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

Inventory host serial number #139

Merged
merged 1 commit into from
Nov 17, 2017
Merged

Inventory host serial number #139

merged 1 commit into from
Nov 17, 2017

Conversation

jameswnl
Copy link
Contributor

@jameswnl jameswnl commented Nov 16, 2017

@jameswnl
Copy link
Contributor Author

@miq-bot add_labels enhancement

@agrare agrare self-assigned this Nov 17, 2017
@agrare
Copy link
Member

agrare commented Nov 17, 2017

👍 confirmed this works. I released a new version of vmware_web_service (v0.2.3) can you require this in the gemspec?

@miq-bot
Copy link
Member

miq-bot commented Nov 17, 2017

Checked commit https://github.com/jameswnl/manageiq-providers-vmware/commit/d093212743c1908b74c37fbe8e756a113e49ee2d with ruby 2.3.3, rubocop 0.47.1, haml-lint 0.20.0, and yamllint 1.10.0
2 files checked, 0 offenses detected
Everything looks fine. ⭐

@@ -14,7 +14,7 @@ Gem::Specification.new do |s|
s.files = Dir["{app,config,lib}/**/*"]

s.add_dependency("fog-vcloud-director", ["~> 0.1.8"])
s.add_dependency "vmware_web_service", "~>0.2.0"
s.add_dependency "vmware_web_service", "~>0.2.3"
Copy link
Contributor Author

Choose a reason for hiding this comment

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

@agrare here you go....

Copy link
Member

@agrare agrare left a comment

Choose a reason for hiding this comment

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

Thanks @jameswnl !

@agrare agrare merged commit ead619e into ManageIQ:master Nov 17, 2017
@agrare agrare added this to the Sprint 74 Ending Nov 27, 2017 milestone Nov 17, 2017
@agrare
Copy link
Member

agrare commented Nov 17, 2017

@simaishi this is needed for ManageIQ/manageiq-providers-lenovo#93

@jameswnl jameswnl deleted the host-sn branch November 17, 2017 20:24
simaishi pushed a commit that referenced this pull request Nov 20, 2017
Inventory host serial number
(cherry picked from commit ead619e)
@simaishi
Copy link
Contributor

Gaprindashvili backport details:

$ git log -1
commit 8f16ff345ce5550abe798d32c09ac5daae54b681
Author: Adam Grare <agrare@redhat.com>
Date:   Fri Nov 17 14:49:37 2017 -0500

    Merge pull request #139 from jameswnl/host-sn
    
    Inventory host serial number
    (cherry picked from commit ead619e9335a193fcd43e3ea05f166773be7e123)

agrare pushed a commit to agrare/manageiq-providers-vmware that referenced this pull request Apr 15, 2019
v2v: Fix list drivers on missing ISO domain
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.

5 participants