Skip to content
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

Cluster up should no longer set service_catalog_landing_page to true #16415

Closed
spadgett opened this issue Sep 18, 2017 · 10 comments
Closed

Cluster up should no longer set service_catalog_landing_page to true #16415

spadgett opened this issue Sep 18, 2017 · 10 comments
Assignees

Comments

@spadgett
Copy link
Member

In origin-web-console, we've made the service catalog landing page the default (openshift/origin-web-console#2078). Cluster up should no longer set service_catalog_landing_page to true, and that value will be ignored if set.

https://github.com/openshift/origin/blob/master/pkg/oc/bootstrap/docker/openshift/helper.go#L849

@bparees
Copy link
Contributor

bparees commented Sep 18, 2017

If that's the default now, what happens when we bring up cluster up w/o the service catalog?

@spadgett
Copy link
Member Author

It still uses the new experience, just with no service catalog resources.

@bparees
Copy link
Contributor

bparees commented Sep 18, 2017

if we don't set this, people won't be able to use older versions of the web console w/ the SC. Is there any harm in leaving it set?

@bparees
Copy link
Contributor

bparees commented Sep 18, 2017

(this logic is tied to the oc client binary, not the version of the cluster/webconsole they are deploying)

@spadgett
Copy link
Member Author

No, it will just be ignored.

But I thought it wasn't supported already per #16314 (comment) ?

@spadgett
Copy link
Member Author

I'm OK leaving it as-is.

@bparees
Copy link
Contributor

bparees commented Sep 18, 2017

We dropped 3.6 support. someone on a slightly older 3.7 cluster could still have issues. it's not a big deal, just want to make sure you're aware.

@bparees
Copy link
Contributor

bparees commented Sep 18, 2017

When did the web console start treating this as the default?

@spadgett
Copy link
Member Author

When did the web console start treating this as the default?

About 15 minutes ago :)

@spadgett
Copy link
Member Author

We dropped 3.6 support. someone on a slightly older 3.7 cluster could still have issues. it's not a big deal, just want to make sure you're aware.

OK, then probably no change needed. Closing.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants