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

[FINE] Fix Cockpit console connectivity for RHEV, AWS, GCE #15303

Merged
merged 5 commits into from Jun 13, 2017

Conversation

bmclaughlin
Copy link
Contributor

@bmclaughlin bmclaughlin commented Jun 5, 2017

Builds on logic that fixed Cockpit connectivity in RHEV in #15000 to fix connectivity in AWS and GCE. This PR back-ports #15000 to Fine (did not cherry-pick cleanly) and fixes AWS/GCE connectivity as well.

@miq-bot add_labels bug, core

https://bugzilla.redhat.com/show_bug.cgi?id=1458924
https://bugzilla.redhat.com/show_bug.cgi?id=1458925

@bmclaughlin
Copy link
Contributor Author

@martinpovolny, @simaishi could you review?

@simaishi simaishi self-assigned this Jun 5, 2017
@simaishi
Copy link
Contributor

simaishi commented Jun 5, 2017

@bmclaughlin which PR fixes this for AWS/GCE in 'master' branch?

@bmclaughlin
Copy link
Contributor Author

bmclaughlin commented Jun 5, 2017

@simaishi that PR is still pending as #12506 diverged the code and this Fine solution isn't currently applicable to master. However this is a blocker for Fine and I deemed it wise to get that fix out in a PR.

@simaishi
Copy link
Contributor

simaishi commented Jun 5, 2017

@bmclaughlin agreed. Cloned both BZs for 'Fine' branch. Please use them instead:

https://bugzilla.redhat.com/show_bug.cgi?id=1458924 (clone of 1450109)
https://bugzilla.redhat.com/show_bug.cgi?id=1458925 (clone of 1443296)

@bmclaughlin bmclaughlin force-pushed the use-public-ip-for-aws-cockpit branch from d325f15 to dc40b66 Compare June 6, 2017 19:44
@bmclaughlin
Copy link
Contributor Author

@simaishi, updated BZ references in PR and new commits.

Addressed miq-bot style issues.

@miq-bot add_label blocker

@miq-bot miq-bot added the blocker label Jun 6, 2017
@miq-bot
Copy link
Member

miq-bot commented Jun 6, 2017

Checked commits bmclaughlin/manageiq@3c4ce1e~...dc40b66 with ruby 2.2.6, rubocop 0.47.1, and haml-lint 0.20.0
2 files checked, 0 offenses detected
Everything looks fine. 🍪

@simaishi
Copy link
Contributor

@martinpovolny please approve if this is good to go. Thanks!

@simaishi simaishi merged commit e5c0bab into ManageIQ:fine Jun 13, 2017
@simaishi simaishi added this to the Sprint 63 Ending Jun 19, 2017 milestone Jun 13, 2017
@bmclaughlin bmclaughlin deleted the use-public-ip-for-aws-cockpit branch June 13, 2017 19:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants