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

operator [R] cockroachdb (2.0.9 2.1.1 2.1.11 3.0.7) #428

Closed
wants to merge 1 commit into from

Conversation

mporrato
Copy link
Collaborator

No description provided.

Signed-off-by: Maurizio Porrato <mporrato@redhat.com>
@github-actions
Copy link
Contributor

Dockerfile or bundle.Dockerfile is added/changed. Note that for security reasons none of these files are going to be used when building bundle. Docker file will be generated and all label information is taken from annotations.yaml.

@github-actions github-actions bot changed the title remove cockroachdb/5.0.3 operator [R] cockroachdb (2.0.9 2.1.1 2.1.11 3.0.7) Oct 13, 2023
@github-actions
Copy link
Contributor

Dear @mporrato,
Some errors and/or warnings were found while doing the check of your operator (cockroachdb/3.0.7) against the entire suite of validators for Operator Framework with Operator-SDK version v1.30.0 and the command $ operator-sdk bundle validate <bundle-path> --select-optional suite=operatorframework.

Errors (:bug:) must be fixed while warnings (:warning:) are informative, and fixing them might improve the quality of your solution.

Type Message
🐛 Field metadata.annotations[api-approved.kubernetes.io], Value : metadata.annotations[api-approved.kubernetes.io]: Required value: protected groups must have approval annotation "api-approved.kubernetes.io", see kubernetes/enhancements#1111
🐛 Field spec.preserveUnknownFields, Value 0xc000ba66f8: spec.preserveUnknownFields: Invalid value: true: cannot set to true, set x-kubernetes-preserve-unknown-fields to true in spec.versions[*].schema instead
⚠️ Value : found olm.properties annotation, please define these properties in metadata/properties.yaml instead
⚠️ Value cockroachdb.v3.0.7: this bundle is using APIs which were deprecated and removed in v1.22. More info: https://kubernetes.io/docs/reference/using-api/deprecation-guide/#v1-22. Migrate the API(s) for CRD: (["cockroachdbs.charts.helm.k8s.io"])
⚠️ Value cockroachdb.v3.0.7: this bundle is using APIs which were deprecated and removed in v1.25. More info: https://kubernetes.io/docs/reference/using-api/deprecation-guide/#v1-25. Migrate the API(s) for poddisruptionbudgets: (["ClusterServiceVersion.Spec.InstallStrategy.StrategySpec.Permissions[0].Rules[6]"])
⚠️ Value cockroachdb.v3.0.7: unable to find the resource requests for the container: (cockroachdb). It is recommended to ensure the resource request for CPU and Memory. Be aware that for some clusters configurations it is required to specify requests or limits for those values. Otherwise, the system or quota may reject Pod creation. More info: https://master.sdk.operatorframework.io/docs/best-practices/managing-resources/
⚠️ Value cockroachdb.v3.0.7: CSV contains permissions to create CRD. An Operator shouldn't deploy or manage other operators (such patterns are known as meta or super operators or include CRDs in its Operands). It's the Operator Lifecycle Manager's job to manage the deployment and lifecycle of operators. Please, review the design of your solution and if you should not be using Dependency Resolution from OLM instead. More info: https://sdk.operatorframework.io/docs/best-practices/common-recommendation/

@mporrato mporrato closed this Oct 13, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants