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

Bring che operator in line with the latest changes coming to Che server to support single-host mode. #73

Merged
merged 1 commit into from
Sep 10, 2019

Conversation

metlos
Copy link
Contributor

@metlos metlos commented Aug 23, 2019

Che server will require different ingress annotations and a new config variable to support single-host mode. This brings the operator inline with what is going to be required once eclipse-che/che#14134 is merged.

@metlos
Copy link
Contributor Author

metlos commented Sep 5, 2019

@davidfestal could we please move this forward?

Copy link
Contributor

@nickboldt nickboldt left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

+0, I have no idea if this change is correct, nor what it does. Deferring to experts who know the codebase.

Copy link
Contributor

@davidfestal davidfestal left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I assume it's OK.

For my understanding, are this changes OK for both the single-host and multi-host cases ?

@davidfestal
Copy link
Contributor

@metlos are we still waiting for something of can we merge this now ?

@metlos
Copy link
Contributor Author

metlos commented Sep 10, 2019

@davidfestal we can merge this now

@davidfestal davidfestal merged commit 3a28252 into eclipse-che:master Sep 10, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants