fix(argo-cd): Add missing applicaton & events create ClusterRole perms#1625
fix(argo-cd): Add missing applicaton & events create ClusterRole perms#1625nxf5025 wants to merge 5 commits intoargoproj:mainfrom
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.