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
[v1.0.x] *: fix run packagemanifests #3895
[v1.0.x] *: fix run packagemanifests #3895
Conversation
…ifests (operator-framework#3856) * internal/olm/operator: fix operator-registry image for run packagemanifests * internal/olm/client/client.go: return error for failed CSV * use switch for DoCSVWait phase logic * pin to olm 0.15.1 for integration tests
…tory to write files (operator-framework#3867)
223d94b
to
ce9cf4f
Compare
if isRunningOnKind() { | ||
By("loading the bundle image into Kind cluster") | ||
err = tc.LoadImageToKindClusterWithName(bundleImage) | ||
Expect(err).Should(Succeed()) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nit: I know this is a backport and this line is basically unchanged in this PR but for better readability this probably should be:
Expect(err).NotTo(HaveOccurred())
Succeed()
is used more for the function itself:
Ω(DoSomethingSimple()).Should(Succeed())
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
+1 I'll make this comment into an issue. There are probably lots of instances of this that we can fix on master (probably not worth backporting those changes though).
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
/lgtm |
Description of the change:
Backport #3856 and #3867
Motivation for the change:
Supported branches should get bug fixes. This unblocks CI, which is currently failing due to issues with OLM v0.16.1's caching of catalog data.
Checklist
If the pull request includes user-facing changes, extra documentation is required:
changelog/fragments
(seechangelog/fragments/00-template.yaml
)website/content/en/docs