-
Notifications
You must be signed in to change notification settings - Fork 1.9k
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(argo-cd): Add missing applicaton & events create ClusterRole perms #1625
fix(argo-cd): Add missing applicaton & events create ClusterRole perms #1625
Conversation
…issions to argo-cd-server Signed-off-by: Nick Fisher <nxf5025@gmail.com>
Hi @nxf5025 great job ! I can confirm the issue with creating events How to reproduce the issue with creating applications ? |
Signed-off-by: Petr Drastil <petr.drastil@gmail.com>
@EladDolev - If you take a look at the first two code blocks in the proposal you should be able to reproduce. (https://github.com/argoproj/argo-cd/blob/master/docs/proposals/003-applications-outside-argocd-namespace.md#proposal). The application creation will fail if create isn't added to the clusterrole |
Well @nxf5025 reason I'm asking is because I'm unable to reproduce 😄 Looking at the proposal gives me no idea how to reproduce unfortunately... |
@EladDolev Weirdly enough when I execute the examples in the code blocks of the proposal I hit the issue:
As soon as I add the |
Signed-off-by: Nick Fisher <nxf5025@gmail.com>
Signed-off-by: Nick Fisher <nxf5025@gmail.com>
@nxf5025 did you set |
I set it to |
Signed-off-by: Nick Fisher <nxf5025@gmail.com>
This pull request has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions. |
This expands upon #1621 and provides the full fix for allowing to create applications in all namespaces.
argocd-server log before the change:
Both of the errors are cleared up with this PR.
Signed-off-by: Nick Fisher nxf5025@gmail.com
Checklist:
Changes are automatically published when merged to
main
. They are not published on branches.