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

Regression on Managed Namespace Metadata Option #15815

Closed
3 tasks done
jonwraymond opened this issue Oct 4, 2023 · 0 comments · Fixed by #15670
Closed
3 tasks done

Regression on Managed Namespace Metadata Option #15815

jonwraymond opened this issue Oct 4, 2023 · 0 comments · Fixed by #15670
Labels
bug Something isn't working

Comments

@jonwraymond
Copy link

Checklist:

  • I've searched in the docs and FAQ for my answer: https://bit.ly/argocd-faq.
  • I've included steps to reproduce the bug.
  • I've pasted the output of argocd version.

Describe the bug

It looks like this change may have caused a regression that causes a duplicate namespace issue on a single application if a namespace object is has been defined in a manifest along with the managedNamespaceMetadata option enabled of an application. This seems to be related to the argocd.argoproj.io/sync-options": "ServerSideApply=true annotation being applied to namespaces. This causes a resource conflict on the defined namespace object.

The docs here say this should be possible with a namespace object.

To Reproduce

  1. Create a namespace kubernetes manifest without server side apply in an application.
  2. Enable managedNamespaceMetadata: on an application.

Expected behavior

Merge metadata from managedNamespaceMetadata when a namespace object is defined in manifests. This used to work before version v2.8.4

Workaround*

Remove managedNamespaceMetadata block from application and only use k8s namespace manifest.

Version

v2.8.4

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants