Skip to content

Commit

Permalink
Merge pull request openshift#1290 from stevekuznetsov/skuznets/consol…
Browse files Browse the repository at this point in the history
…e-routes

ci-operator: read custom routes for console
  • Loading branch information
openshift-merge-robot authored Oct 12, 2020
2 parents 5784e03 + 16c0113 commit daa2263
Showing 1 changed file with 19 additions and 3 deletions.
22 changes: 19 additions & 3 deletions cmd/ci-operator/main.go
Original file line number Diff line number Diff line change
Expand Up @@ -22,6 +22,7 @@ import (
"strings"
"time"

v1 "github.com/openshift/api/route/v1"
"github.com/sirupsen/logrus"

authapi "k8s.io/api/authorization/v1"
Expand Down Expand Up @@ -705,10 +706,25 @@ func (o *options) resolveInputs(steps []api.Step) error {
if routeGetter, err := routeclientset.NewForConfig(o.clusterConfig); err != nil {
log.Printf("could not get route client for cluster config")
} else {
if consoleRoute, err := routeGetter.Routes("openshift-console").Get(context.TODO(), "console", meta.GetOptions{}); err != nil {
log.Printf("could not get route console in namespace openshift-console")
if consoleRoutes, err := routeGetter.Routes("openshift-console").List(context.TODO(), meta.ListOptions{}); err != nil {
log.Printf("could not get routes in namespace openshift-console")
} else {
o.consoleHost = consoleRoute.Spec.Host
hostForRoute := func(name string, routes []v1.Route) string {
for _, route := range routes {
if route.Name == name {
return route.Spec.Host
}
}
return ""
}
// the canonical route for the console may be in one of two routes,
// and we want to prefer the custom one if it is present
for _, routeName := range []string{"console-custom", "console"} {
if host := hostForRoute(routeName, consoleRoutes.Items); host != "" {
o.consoleHost = host
break
}
}
}
}

Expand Down

0 comments on commit daa2263

Please sign in to comment.