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

Reconfigure VM: Add / Remove Network Adapters #16700

Merged
merged 2 commits into from
Feb 13, 2018

Conversation

kruge002
Copy link
Contributor

@kruge002 kruge002 commented Dec 20, 2017

This PR adds the option to reconfigure network adapters at the Reconfigure VM webpage.

This PR is part of a set of PRs:
ManageIQ/vmware_web_service#25 (merged by @agrare)
ManageIQ/manageiq-providers-vmware#163
#16700
ManageIQ/manageiq-ui-classic#3121

More info: ManageIQ/manageiq-ui-classic#3119

@miq-bot
Copy link
Member

miq-bot commented Jan 10, 2018

This pull request is not mergeable. Please rebase and repush.

@gmcculloug
Copy link
Member

@kruge002 Please rebase.

@kruge002
Copy link
Contributor Author

@gmcculloug Thanks for your reply. I will fix this in the coming days.

@kruge002
Copy link
Contributor Author

@gmcculloug Due to a shift in priorities this will take a bit longer.

@miq-bot
Copy link
Member

miq-bot commented Feb 12, 2018

Checked commits kruge002/manageiq@65dba55~...3d5b4d9 with ruby 2.3.3, rubocop 0.52.0, haml-lint 0.20.0, and yamllint 1.10.0
2 files checked, 0 offenses detected
Everything looks fine. 🍪

@gmcculloug
Copy link
Member

Looks good. Thanks @kruge002

@gmcculloug gmcculloug merged commit 6a0dc62 into ManageIQ:master Feb 13, 2018
@gmcculloug gmcculloug added this to the Sprint 79 Ending Feb 12, 2018 milestone Feb 13, 2018
@simaishi
Copy link
Contributor

simaishi commented Jun 1, 2018

@gmcculloug Can this be gaprindashvili/yes? ManageIQ/manageiq-providers-vmware#272 needs reconfigure network feature in G-branch..

@AllenBW
Copy link
Member

AllenBW commented Jun 1, 2018

I for one, would love this to be backported, bin/update fails like a :godmode: on G presently

@gmcculloug
Copy link
Member

@simaishi Label added.

simaishi pushed a commit that referenced this pull request Jun 1, 2018
Reconfigure VM: Add / Remove Network Adapters
(cherry picked from commit 6a0dc62)
@simaishi
Copy link
Contributor

simaishi commented Jun 1, 2018

Gaprindashvili backport details:

$ git log -1
commit f59e11c188edfaaed18113b8ad4e1d0530405984
Author: Greg McCullough <gmccullo@redhat.com>
Date:   Tue Feb 13 09:03:33 2018 -0500

    Merge pull request #16700 from kruge002/ReconfigureNetwork
    
    Reconfigure VM: Add / Remove Network Adapters
    (cherry picked from commit 6a0dc623a49f2d01cabe0a931c0d456b812deefd)

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.

6 participants