-
Notifications
You must be signed in to change notification settings - Fork 12
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
Can't pull the right certified-operator-index image in a brand new cluster #154
Comments
Hi @sxd I'm not sure I understand the question/issue fully, so let me ask some questions...
|
@sxd Answers to the above would be helpful, but I think I kind of understand. It seems that the As a workaround after the operate is installed and CR reconciled the below should work (replace v4.14 with the version of your cluster):
|
Hi @acornett21 Well you got the issue sorry for the delay in the answer. Now, possible solutions, I've been checking the code and how the ImageStream works, and since we cannot pull all the tags, even if the documentation doesn't say anything about (if you can point me to more explicit documentation, please go ahead), what if we add another options that will be a list of tags something like this:
And starting from this we can create the ImageStreamTags required to pull the proper images, what do you about this idea? Regards! |
@sxd Sorry for the long delay on this, finally had some time to get back to it. I have a PR up that should address this now, and for future versions of OCP with zero impact on the API/user. Zero in the sense there isn't any more work for them to do. I'd like to have this released this week, but we'll see how PR reviews go, since some people are on PTO. |
@acornett21 That's amazing!!! I'll be the first user for sure! I'm guessing this is the PR #160 I'll test it as soon as is released, don't know if there's a way to test before the release, for me it's almost no effort to test, please let me know! Cheers! |
Hey @sxd If you want to install a custom catalog in your cluster to test, another set of eyes is always welcome. Catalog Spec
Then you could either use the UI to create a subscription, or create one manually. Let me know if there you spot any issues. |
@acornett21 perfect! scheduled for tomorrow morning work! I'll let you know as soon as I have results! CheerS1 |
Hey @sxd did you have a chance to test this? Any feedback you have would be great! Thanks! |
Bug Description
When running the operator to certify a new version, we found out that it was pulling the wrong image of the
certified-operator-index
because it was 4.15, which "doesn't exist" yet, leading to a problem like thisEven if the operator only supports v4.14 as a top version, the operator certification downloads the the v4.15 tag and the container can't pull the v4.14, to clarify, I don't know if this is the right place to ask for help, but the certification process doesn't work because of this.
The fix is clear, if you set the operator to have the maximum support on 4.15, it works really nice, that's why I open this issue, if it's not the right please, can you point me the right one?
The text was updated successfully, but these errors were encountered: