Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

FIX: Append gitconfig instead of override in GitCredentialStorageFileSecretApplier #17554

Merged
merged 1 commit into from
Aug 4, 2020

Conversation

skabashnyuk
Copy link
Contributor

What does this PR do?

FIX: Append gitconfig instead of override in GitCredentialStorageFileSecretApplier

What issues does this PR fix or reference?

GitCredentialStorageFileSecretApplier replaces the content of git-config instead of adding to it.

Release Notes

N/A

Docs PR

N/A

…eSecretApplier

Signed-off-by: Sergii Kabashniuk <skabashniuk@redhat.com>
@skabashnyuk skabashnyuk requested review from sparkoo and mshaposhnik and removed request for nickboldt, l0rd, sleshchenko and amisevsk August 4, 2020 07:27
@che-bot che-bot added kind/bug Outline of a bug - must adhere to the bug report template. status/code-review This issue has a pull request posted for it and is awaiting code review completion by the community. labels Aug 4, 2020
@che-bot
Copy link
Contributor

che-bot commented Aug 4, 2020

✅ E2E Happy path tests succeed 🎉

See Details

Tested with Eclipse Che Multiuser User on K8S (minikube v1.1.1)

@skabashnyuk skabashnyuk merged commit 537f7c9 into eclipse-che:master Aug 4, 2020
@skabashnyuk skabashnyuk deleted the gitfix branch August 4, 2020 08:28
@skabashnyuk skabashnyuk added this to the 7.17 milestone Aug 4, 2020
@che-bot che-bot removed the status/code-review This issue has a pull request posted for it and is awaiting code review completion by the community. label Aug 4, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind/bug Outline of a bug - must adhere to the bug report template.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants