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

Add power_state,health_state and vendor properties in parse #40

Merged
merged 3 commits into from
Apr 11, 2017
Merged

Add power_state,health_state and vendor properties in parse #40

merged 3 commits into from
Apr 11, 2017

Conversation

walteraa
Copy link
Member

@walteraa walteraa commented Apr 7, 2017

Now the refresh is parsing the follow new properties:

  • health_state
  • power_state
  • vendor

Add POWER_STATE_MAP and HEALTH_STATE_MAP to parse power_state and health_state status.
Add :vendor as "Lenovo" in Hash's parse.
@miq-bot
Copy link
Member

miq-bot commented Apr 7, 2017

Checked commits https://github.com/lenovo/manageiq-providers-lenovo/compare/51d46c826851c8e2bba6e09ffed223cfb8c1fbb5~...b752b03127302091f663644127274317ed77e1a5 with ruby 2.2.6, rubocop 0.47.1, and haml-lint 0.20.0
1 file checked, 0 offenses detected
Everything looks good. 🏆

@juliancheal juliancheal merged commit 3d3d928 into ManageIQ:master Apr 11, 2017
@juliancheal
Copy link
Member

@miq-bot add_label fine/yes

simaishi pushed a commit that referenced this pull request Apr 11, 2017
Add power_state,health_state and vendor properties in parse
(cherry picked from commit 3d3d928)
@simaishi
Copy link
Contributor

Fine backport details:

$ git log -1
commit ce53607b011f316f05858674ee70731525060d04
Author: Julian Cheal <julian.cheal@gmail.com>
Date:   Tue Apr 11 12:43:10 2017 +0100

    Merge pull request #40 from lenovo/add_properties_in_parse
    
    Add power_state,health_state and vendor properties in parse
    (cherry picked from commit 3d3d9287c63d60163e596e5c3e74ce5bc723a889)

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.

4 participants