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

nmstate instance uses deprecated API #290

Open
dlbewley opened this issue Mar 19, 2024 · 1 comment
Open

nmstate instance uses deprecated API #290

dlbewley opened this issue Mar 19, 2024 · 1 comment

Comments

@dlbewley
Copy link
Contributor

The nmstate resource here

Has the following issues:

  • It uses v1beta1 API which has been deprecated for v1. Observered warning when install to OCP 4.14.
  • The spec has a node selector defined, but I'm not sure why. An empty dict should also work.
$ kustomize build 'https://github.com/redhat-cop/gitops-catalog/nmstate/aggregate/overlays/default?ref=main' | kfilt -k nmstate
---
apiVersion: nmstate.io/v1beta1
kind: NMState
metadata:
  annotations:
    argocd.argoproj.io/sync-options: SkipDryRunOnMissingResource=true
  name: nmstate
spec:
  nodeSelector:
    beta.kubernetes.io/arch: amd64
@strangiato
Copy link
Contributor

Thanks for raising the issue!

I'm seeing references to v1 all the way back to the 4.10 docs so it is probably safe to just update to v1 entirely without having to keep a backwards compatible overlay.

Do you know if there were any important API changes that we need to incorporate between v1beta1 and v1?

Would you be interested in trying out the v1 API on your cluster and updating the objects if everything looks good?

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

No branches or pull requests

2 participants