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

Ignore topologyupdater in nodefeaturediscovery manifests #44

Merged
merged 1 commit into from
Feb 22, 2024

Conversation

larsks
Copy link
Member

@larsks larsks commented Feb 21, 2024

There appears to be some weirdness around the CRD for the
NodeFeatureDiscovery operator: you need to set the topologyUpdater
attribute, but at least on nerc-ocp-test, when you fetch the resource, you
see toplogyupdater (note capitalization difference). This causes argocd
to think the app is out of sync.

This commit instructs argocd to ignore either of these names in the
resource.

There appears to be some weirdness around the CRD for the
NodeFeatureDiscovery operator: you need to set the `topologyUpdater`
attribute, but at least on nerc-ocp-test, when you fetch the resource, you
see `toplogyupdater` (note capitalization difference). This causes argocd
to think the app is out of sync.

This commit instructs argocd to ignore either of these names in the
resource.
@larsks larsks requested review from jtriley and computate February 21, 2024 21:19
@larsks larsks self-assigned this Feb 21, 2024
@larsks larsks merged commit b6a065e into OCP-on-NERC:main Feb 22, 2024
1 check passed
@larsks larsks deleted the fix/nfd-operator-test branch February 22, 2024 03:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants