argo-helm icon indicating copy to clipboard operation
argo-helm copied to clipboard

fix(argo-cd): Add cluster role to allow a namespaced applicationset to be handled by a controller

Open borisssmidtCET opened this issue 10 months ago • 6 comments

I recently enabled the name spacing feature of the application controller and ran into the the issue that the application controller couldn't create/read the applications in the required namespace.

Its probably best to add a cluster role for this case?

Checklist:

  • [x] I have bumped the chart version according to versioning
  • [x] I have updated the documentation according to documentation
  • [x] I have updated the chart changelog with all the changes that come with this pull request according to changelog.
  • [x] Any new values are backwards compatible and/or have sensible default.
  • [x] I have signed off all my commits as required by DCO.
  • [ ] My build is green (troubleshooting builds).

borisssmidtCET avatar Aug 17 '23 15:08 borisssmidtCET

Please fix your PR title to match requirement of

example: fix(argo-cd): blah blah blah

Also please sign all of your commits to pass DCO. Thanks.

jmeridth avatar Aug 17 '23 22:08 jmeridth

Hi @borisssmidtCET , thank you for your PR. Please handle below (items written in PR description).

[ ] I have bumped the chart version according to versioning

-> Please bump patch version.

[ ] I have updated the chart changelog with all the changes that come with this pull request according to changelog.

-> Please remove old ones and add new ones in changelog.

yu-croco avatar Aug 18 '23 07:08 yu-croco

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.

github-actions[bot] avatar Oct 21 '23 01:10 github-actions[bot]

Hi, I think this is still relevant! Is there anything to be done to get this going again?

sadoMasupilami avatar Nov 27 '23 12:11 sadoMasupilami

i will be create pr for that.

mugioka avatar Dec 25 '23 07:12 mugioka

https://github.com/argoproj/argo-helm/pull/2402

mugioka avatar Dec 26 '23 02:12 mugioka

Closing in favor of #2402 as it has recent interactions and movement. Thank you for contributing @borisssmidtCET.

jmeridth avatar Feb 23 '24 17:02 jmeridth