-
Notifications
You must be signed in to change notification settings - Fork 587
add Insights capability to Insights CRDs #2682
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
base: master
Are you sure you want to change the base?
Conversation
This commit adds the capability.openshift.io/name=Insights annotation to all Insights related CRDs. Signed-off-by: Ondrej Pokorny <opokorny@redhat.com>
|
Pipeline controller notification For optional jobs, comment This repository is configured in: LGTM mode |
|
Hello @opokornyy! Some important instructions when contributing to openshift/api: |
📝 WalkthroughWalkthroughThe pull request adds Insights capability declarations across multiple CRD definitions and API versions. It introduces the 🚥 Pre-merge checks | ✅ 3✅ Passed checks (3 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing touches
🧪 Generate unit tests (beta)
Warning There were issues while running some tools. Please review the errors and either fix the tool's configuration or disable the tool if it's a critical failure. 🔧 golangci-lint (2.5.0)Error: build linters: unable to load custom analyzer "kubeapilinter": tools/_output/bin/kube-api-linter.so, plugin: not implemented Tip 🧪 Unit Test Generation v2 is now available!We have significantly improved our unit test generation capabilities. To enable: Add this to your reviews:
finishing_touches:
unit_tests:
enabled: trueTry it out by using the Have feedback? Share your thoughts on our Discord thread! Comment |
PR Compliance Guide 🔍Below is a summary of compliance checks for this PR:
Compliance status legend🟢 - Fully Compliant🟡 - Partial Compliant 🔴 - Not Compliant ⚪ - Requires Further Human Verification 🏷️ - Compliance label |
|||||||||||||||||||||||
PR Code Suggestions ✨No code suggestions found for the PR. |
|
/lgtm |
|
Scheduling tests matching the |
|
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: JoelSpeed The full list of commands accepted by this bot can be found here. The pull request process is described here DetailsNeeds approval from an approver in each of these files:
Approvers can indicate their approval by writing |
|
/override ci/prow/e2e-aws-serial-techpreview |
|
@JoelSpeed: Overrode contexts on behalf of JoelSpeed: ci/prow/e2e-aws-serial, ci/prow/e2e-aws-serial-techpreview DetailsIn response to this:
Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository. |
|
/test e2e-aws-ovn-hypershift-conformance |
|
PR-Agent: could not fine a component named |
|
@opokornyy: all tests passed! Full PR test history. Your PR dashboard. DetailsInstructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository. I understand the commands that are listed here. |
User description
This PR adds the capability.openshift.io/name=Insights annotation to all Insights related CRDs.
PR Type
Enhancement
Description
Add
capability.openshift.io/name=Insightsannotation to all Insights CRDsUpdate type definitions across multiple API versions (v1, v1alpha1, v1alpha2)
Regenerate CRD manifests with capability metadata
Update featuregated CRD manifests and payload manifests
Diagram Walkthrough
File Walkthrough
6 files
Add Insights capability annotation to v1 typeAdd Insights capability annotation to v1alpha1 typeAdd Insights capability annotation to v1alpha2 typeAdd Insights capability annotation to insights v1 typeAdd Insights capability annotation to insights v1alpha1 typeAdd Insights capability annotation to insights v1alpha2 type15 files
Add capability annotation to CustomNoUpgrade manifestAdd capability annotation to DevPreviewNoUpgrade manifestAdd capability annotation to TechPreviewNoUpgrade manifestUpdate Capability field from empty to InsightsAdd capability annotation to featuregated manifestAdd capability annotation to v1alpha1 CustomNoUpgrade manifestAdd capability annotation to v1alpha1 DevPreviewNoUpgrade manifestAdd capability annotation to v1alpha1 TechPreviewNoUpgrade manifestUpdate Capability field from empty to InsightsAdd capability annotation to v1alpha1 featuregated manifestAdd capability annotation to v1alpha2 CustomNoUpgrade manifestAdd capability annotation to v1alpha2 DevPreviewNoUpgrade manifestAdd capability annotation to v1alpha2 TechPreviewNoUpgrade manifestUpdate Capability field from empty to InsightsAdd capability annotation to v1alpha2 featuregated manifest21 files