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

Add guide for changing configuration repository #317

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

carolili
Copy link

This PR adds a guide to the documentation for changing the repository containing the configuration that Flux applies.

Contents:

  • Setting up a new repository and credentials.
  • Defining the new repository setup as Flux resources.
  • Pointing the old Kustomization(s) to the new repository.

The examples aim to be generic and contain fake examples of usernames/credentials. Any feedback is welcome.

Please review the documentation especially with regards to the Kustomization and pointing the Kustomization to a new repo to ensure that no important detail is missed.

Copy link
Member

@dholbach dholbach left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks a lot for your PR. I just had a quick pass over it and checked grammar, spelling, etc. I'll need to defer to somebody who knows Flux better for the actual content.

@dholbach
Copy link
Member

dholbach commented Jun 22, 2021

It looks like @csjiang and @stealthybox are having a discussion about something similar in #187. Stefan said that the two PRs should potentially be merged and maybe not repeat the information from the source-controller docs. Is this something folks would be interested in taking on?

@carolili
Copy link
Author

Thank you, I will add the suggested changes so you can proceed. If the docs in #187 are already covering this use case, feel free to mention changing configuration repository as a use case and reject this PR.

Signed-off-by: Carolina Lindqvist <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants