-
Notifications
You must be signed in to change notification settings - Fork 65
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: rename gitea secret for better description #224
Conversation
partially fixes cnoe-io#216 This only covers the gitea secret renaming, as apparently the argocd one is generated by argo on installation, and is hardcoded to `argocd-initial-admin-secret`. See here: https://github.com/argoproj/argo-cd/blob/0f11dfb5961361807962aafc68b11426b8a47490/cmd/argocd/commands/admin/initial_password.go#L16 There's some world where we could replicate the secret into one with a name we like better, but that sounds like a recipe for confusion. Signed-off-by: Bradon Kanyid (rattboi) <[email protected]>
d701d50
to
2c0884f
Compare
Approved ! |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM. If you could fix up my mistake I'd appreciate it :)
Signed-off-by: Bradon Kanyid (rattboi) <[email protected]>
637176f
to
a046fe2
Compare
@rattboi @nabuskey Im getting below error while pull the latest code and deploy. Im not sure if this is related to this PR.
|
are you using a rebuilt |
Yea just need to update the secret name in the example. |
I think you both are right. I didnt rebuild the binary, let me try that out! Thanks! |
@nabuskey Could we have a patch release for this breaking change? |
+1 on a micro release if the latest binary doesnt go with the ref implementation. Also another good reason to separate the examples repo, so we can cut versions both on the binary and the examples. |
Yeah we really need to decouple the two and pin to a version |
partially fixes #216
This only covers the gitea secret renaming, as apparently the argocd one is generated by argo on installation, and is hardcoded to
argocd-initial-admin-secret
.See here: https://github.com/argoproj/argo-cd/blob/0f11dfb5961361807962aafc68b11426b8a47490/cmd/argocd/commands/admin/initial_password.go#L16
There's some world where we could replicate the secret into one with a name we like better, but that sounds like a recipe for confusion.