-
Notifications
You must be signed in to change notification settings - Fork 536
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] [CI] prometheus-exporter-operator (0.2.0 0.2.4 0.7.0) #3249
base: main
Are you sure you want to change the base?
operator [R] [CI] prometheus-exporter-operator (0.2.0 0.2.4 0.7.0) #3249
Conversation
|
Dear @slopezz, Errors (:bug:) must be fixed while warnings (:warning:) are informative, and fixing them might improve the quality of your solution.
|
d64ea4c
to
4ac4872
Compare
|
Dear @slopezz, Errors (:bug:) must be fixed while warnings (:warning:) are informative, and fixing them might improve the quality of your solution.
|
The latest version of Outdated v0.2.4
Current v0.7.0
OperaturHub CISurprisingly, despite all possible breaking changes between the 2 versions, the CI is only failing 1 test:
However it is a bit weird, since this If for any reason there is no easy way of having old |
Dear @slopezz, Errors (:bug:) must be fixed while warnings (:warning:) are informative, and fixing them might improve the quality of your solution.
|
4ac4872
to
d019fdf
Compare
|
Dear @slopezz, Errors (:bug:) must be fixed while warnings (:warning:) are informative, and fixing them might improve the quality of your solution.
|
d019fdf
to
bf16fe5
Compare
|
Dear @slopezz, Errors (:bug:) must be fixed while warnings (:warning:) are informative, and fixing them might improve the quality of your solution.
|
I deleted old versions just in case that's the root problem, so only keeping new However the same CI test fails on the same place:
So I force-pushed again older versions since the error is always the same. I know 100% that operator works, since which it was successfully merged in openshift-operatorhub redhat-openshift-ecosystem/community-operators-prod#3218 using the exact same yamls, and operator upgrade happened seamlessly since OperatorHub version was updated. So, which is the best way to proceed with CI failing test? |
|
Dear @slopezz, Errors (:bug:) must be fixed while warnings (:warning:) are informative, and fixing them might improve the quality of your solution.
|
bf16fe5
to
094310b
Compare
|
Dear @slopezz, Affected versions : More info in 'Kubernetes max version in CSV' section here. |
094310b
to
deeb16d
Compare
Signed-off-by: Sergio López <slopezma@redhat.com>
|
1 similar comment
|
Dear @slopezz, Errors (:bug:) must be fixed while warnings (:warning:) are informative, and fixing them might improve the quality of your solution.
|
1 similar comment
Dear @slopezz, Errors (:bug:) must be fixed while warnings (:warning:) are informative, and fixing them might improve the quality of your solution.
|
Thanks submitting your Operator. Please check below list before you create your Pull Request.
New Submissions
Updates to existing Operators
ci.yaml
file according to the update instructions?replaces
property if you chosereplaces-mode
via theupdateGraph
property inci.yaml
?package.yaml
orannotations.yaml
?Your submission should not
Operator Description must contain (in order)
Operator Metadata should contain
Remember that you can preview your CSV here.
--
1 If you feel your Operator does not fit any of the pre-defined categories, file an issue against this repo and explain your need
2 For more information see here