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

Refresh datastore files through EMS #170

Merged

Conversation

agrare
Copy link
Member

@agrare agrare commented Jan 4, 2018

Refreshing datastore files just use the VIM SDK which can be done
through the EMS not the Host.

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

Refreshing datastore files just use the VIM SDK which can be done
through the EMS not the Host.

https://bugzilla.redhat.com/show_bug.cgi?id=1529725
@miq-bot
Copy link
Member

miq-bot commented Jan 4, 2018

Checked commit agrare@50f8330 with ruby 2.3.3, rubocop 0.47.1, haml-lint 0.20.0, and yamllint 1.10.0
2 files checked, 3 offenses detected

app/models/manageiq/providers/vmware/infra_manager.rb

@agrare
Copy link
Member Author

agrare commented Jan 4, 2018

cc @roliveri

@Fryguy Fryguy merged commit 95d1303 into ManageIQ:master Jan 15, 2018
@Fryguy Fryguy added this to the Sprint 77 Ending Jan 15, 2018 milestone Jan 15, 2018
@agrare agrare deleted the bz_1529725_refresh_storages_through_ems branch January 15, 2018 16:29
simaishi pushed a commit that referenced this pull request Jan 15, 2018
@simaishi
Copy link
Contributor

Gaprindashvili backport details:

$ git log -1
commit 0f1be9296d91fd6a7ac1b864a7c677d16d0fe65e
Author: Jason Frey <[email protected]>
Date:   Mon Jan 15 11:28:54 2018 -0500

    Merge pull request #170 from agrare/bz_1529725_refresh_storages_through_ems
    
    Refresh datastore files through EMS
    (cherry picked from commit 95d1303f75d1ba3fc1f9f71f33dd7ad2cef8e1f0)
    
    https://bugzilla.redhat.com/show_bug.cgi?id=1534753

agrare pushed a commit to agrare/manageiq-providers-vmware that referenced this pull request Apr 15, 2019
v2v: Pre-check install_drivers checkbox for windows VMs
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