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 test / PR-traffic-light PR check is failing #5296

Open
shefali-malhotra opened this issue Nov 13, 2024 · 7 comments
Open

Operator test / PR-traffic-light PR check is failing #5296

shefali-malhotra opened this issue Nov 13, 2024 · 7 comments

Comments

@shefali-malhotra
Copy link
Contributor

Seeing the operator PR traffiv light check for the following PR #5291
Any leads in the fixing that.

@shefali-malhotra
Copy link
Contributor Author

@mporrato Could you please help look into this .

@mporrato
Copy link
Collaborator

@shefali-malhotra your csv is malformed:

$ operator-sdk bundle validate 1.7.0
FATA[0000] unable to find a csv in bundle directory 1.7.0/manifests 
$ find 1.7.0
1.7.0
1.7.0/manifests
1.7.0/manifests/containerstoragemodules.storage.dell.com.crd.yaml
1.7.0/manifests/dell-csm-operator.v1.7.0.clusterserviceversion.yaml
1.7.0/metadata
1.7.0/metadata/annotations.yaml
1.7.0/tests
1.7.0/tests/scorecard
1.7.0/tests/scorecard/config.yaml
$ yamllint -c <(printf "rules:\n  line-length: disable\n") 1.7.0/
1.7.0/manifests/dell-csm-operator.v1.7.0.clusterserviceversion.yaml
  4447:5    error    syntax error: expected <block end>, but found '?' (syntax)

@shefali-malhotra
Copy link
Contributor Author

Thanks @mporrato for pointing out the CSV issue. After fixing all the formatting issue now the operator upgrade and olm install tests are failing in PR.
From the logs it looks operator controller pod is going in ImagePullBackoff error but exactly which image it is unable to pull could you please provide those details if possible. Similar error is being observed with olm install.

@mporrato
Copy link
Collaborator

@shefali-malhotra seems to be the kube-rbac-proxy image: https://github.com/k8s-operatorhub/community-operators/actions/runs/11837159427/job/32983683857?pr=5291#step:6:7146
The CI requires all images to be pullable without authentication.

@shefali-malhotra
Copy link
Contributor Author

@mporrato thanks for pointing the issue. Is there any alternative to use the redHat packaged images as our customers are expecting to use the same.
Also, for community and certified operator the pipeline tests were passed with the same image and PRs are merged.
We have also checked that if same image is available as part of other redHat registries that do not need authentication, but this image is only available on registry.redhat.io which requires authentication.

@shefali-malhotra
Copy link
Contributor Author

I tried updating the image to use the one present on docker hub and all PR checks passed. But our current requirement is to consume RedHat packaged image . Please suggest the workaround for the same.

@mporrato
Copy link
Collaborator

@shefali-malhotra your requirement is not compatible with this repository: the operators in this repo are supposed to be installable on any kubernetes cluster, not just openshift clusters.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

No branches or pull requests

2 participants