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

StorageLVMSubscriptionNS.yaml has workload annotation in wrong section #2086

Open
dav1x opened this issue Oct 24, 2024 · 0 comments
Open

StorageLVMSubscriptionNS.yaml has workload annotation in wrong section #2086

dav1x opened this issue Oct 24, 2024 · 0 comments

Comments

@dav1x
Copy link

dav1x commented Oct 24, 2024

The storage LVM subscription has the workload annotation in the wrong section as a label.

https://github.com/openshift-kni/cnf-features-deploy/blob/release-4.16/ztp/source-crs/StorageLVMSubscriptionNS.yaml

skipping pod CPUs requests modifications because the openshift-storage namespace is not annotated with workload.openshift.io/allowed

If you compare to the SRIOV NS:
https://github.com/openshift-kni/cnf-features-deploy/blob/release-4.16/ztp/source-crs/SriovSubscriptionNS.yaml#L6

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

1 participant