-
Notifications
You must be signed in to change notification settings - Fork 1.7k
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
[v1.0.x] fix: modify olm-status-descriptor-test in scorecard #4088
[v1.0.x] fix: modify olm-status-descriptor-test in scorecard #4088
Conversation
Modify `olm-status-descriptor-test` to not check the presence of `status` field in CR, instead validate only if status-descriptors are present in owned CRDs.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
/lgtm
This doesn't quite fix the issue described in #3999 /hold |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
/lgtm
the nit isn't worth fixing since the PR looks good.
# release notes and/or the migration guide | ||
entries: | ||
- description: > | ||
Modify `olm-status-descriptors-test` to only validate if the status-descriptors are present in CRD. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
nit: changelog should be past tense: Modified
Once this is merged, #4093 can also be cherry picked |
/hold cancel |
closing this and opening #4141 instead with same changes. |
This is an automated cherry-pick of #4009
/assign varshaprasad96