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

chore: bump ui in kommander/opsportal #389

Merged
merged 1 commit into from
Jan 31, 2020

Conversation

juliangieseke
Copy link
Contributor

@juliangieseke juliangieseke commented Jan 31, 2020

this fixes a small issue with kommander ui not being available due to ingress bug

@juliangieseke juliangieseke requested review from a team January 31, 2020 19:10
@alejandroEsc
Copy link
Contributor

Could I get some details on the Ingress bug?

@juliangieseke
Copy link
Contributor Author

sure! we've been putting the wrong url (with prefix) in our chart, fix here: https://github.com/mesosphere/kommander/pull/1613

alejandroEsc
alejandroEsc previously approved these changes Jan 31, 2020
Copy link
Contributor

@alejandroEsc alejandroEsc left a comment

Choose a reason for hiding this comment

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

awesome, thanks for the quick reply.

@juliangieseke juliangieseke force-pushed the jg/bump-kommander-20200131-2 branch 2 times, most recently from e9e28cd to 75c95e7 Compare January 31, 2020 19:37
@juliangieseke juliangieseke force-pushed the jg/bump-kommander-20200131-2 branch from 75c95e7 to 8952b0f Compare January 31, 2020 19:38
@juliangieseke
Copy link
Contributor Author

rebased 👍

@alejandroEsc alejandroEsc self-requested a review January 31, 2020 19:42
@juliangieseke juliangieseke merged commit bf3eeb8 into master Jan 31, 2020
@juliangieseke juliangieseke deleted the jg/bump-kommander-20200131-2 branch January 31, 2020 20:03
mesosphere-teamcity pushed a commit that referenced this pull request Jan 31, 2020
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