Skip to content

Commit ef6081e

Browse files
Fix bug where installing HCO on kubernetes+OLM (#2705)
In this case, we miss the CSV, as HCO currenlty only read it on openshift. That eventually causes nil pointer error, and the HCO pod is in crash loop. Signed-off-by: Nahshon Unna-Tsameret <[email protected]> Co-authored-by: Nahshon Unna-Tsameret <[email protected]>
1 parent d23ea5b commit ef6081e

File tree

2 files changed

+2
-2
lines changed

2 files changed

+2
-2
lines changed

pkg/util/own_resources.go

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -68,7 +68,7 @@ func findOwnResources(ctx context.Context, cl client.Reader, logger logr.Logger)
6868
logger.Error(err, "Can't get deployment")
6969
return
7070
}
71-
if GetClusterInfo().IsOpenshift() {
71+
if GetClusterInfo().IsManagedByOLM() {
7272
var err error
7373
or.csv, err = getCSVFromDeployment(or.deployment, cl, operatorNs, logger)
7474
if err != nil {

tests/vendor/github.com/kubevirt/hyperconverged-cluster-operator/pkg/util/own_resources.go

Lines changed: 1 addition & 1 deletion
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

0 commit comments

Comments
 (0)