Skip to content

Commit

Permalink
Merge pull request #1545 from thomasmckay/4257-cherry-pick
Browse files Browse the repository at this point in the history
OCM-4257 | fix: doesn't check /labels for HCP
  • Loading branch information
ciaranRoche authored Oct 13, 2023
2 parents 68a1e6e + 58d33c8 commit 132b354
Showing 1 changed file with 8 additions and 4 deletions.
12 changes: 8 additions & 4 deletions cmd/edit/ingress/cmd.go
Original file line number Diff line number Diff line change
Expand Up @@ -207,10 +207,14 @@ func run(cmd *cobra.Command, argv []string) {
interactive.Enable()
}

hasLegacyIngressSupport, err := r.OCMClient.HasLegacyIngressSupport(cluster)
if err != nil {
r.Reporter.Errorf("There was a problem checking version compatibility: %v", err)
os.Exit(1)
hasLegacyIngressSupport := true
if !ocm.IsHyperShiftCluster(cluster) {
var err error
hasLegacyIngressSupport, err = r.OCMClient.HasLegacyIngressSupport(cluster)
if err != nil {
r.Reporter.Errorf("There was a problem checking version compatibility: %v", err)
os.Exit(1)
}
}

if hasLegacyIngressSupport && IsIngressV2SetViaCLI(cmd.Flags()) {
Expand Down

0 comments on commit 132b354

Please sign in to comment.