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

updated value for device.state (#155) #156

Merged
merged 1 commit into from
Feb 3, 2018
Merged

Conversation

sandss
Copy link
Contributor

@sandss sandss commented Oct 30, 2017

changed value 'enabled' to 'Enabled' to fix the error:

[ErrorCode]: ERR-xml-parse-error[ErrorDescription]: XML PARSING ERROR: Element 'lsbootVMedia', attribute 'state': [facet 'enumeration'] The value 'enabled' is not an element of the set {'Disabled', 'Enabled'}. Element 'lsbootVMedia', attribute 'state': 'enabled' is not a valid value of the local atomic type.

@sandss
Copy link
Contributor Author

sandss commented Oct 30, 2017

Tested to see if changes corrected issue. No errors were encountered while running boot_order_precision_set()

@jlothian
Copy link

jlothian commented Dec 4, 2017

This bit me today, and updating this value fixed it as well.

@vvb
Copy link
Member

vvb commented Jan 30, 2018

@sandss @jlothian What versions of IMC are you using? This is because of some screw up in the earlier versions of IMC. So, from what I have seen Enabled would work only for some early versions on IMC.. but enabled is expected to work for all. We have been using this code for sometime now and not seen this issue. So, please let me know the versions you are using when you get time.

@sandss
Copy link
Contributor Author

sandss commented Jan 30, 2018

2.0(8g) on one and I've also ran it against one of the newest versions 3.1(1d) (confirmed using "Enabled" not "enabled"). I have not tested using "enabled" on the newest version to see if it fails.

@britthouser
Copy link
Contributor

I just hit this on version CIMC version 2.0(6d)

@vvb
Copy link
Member

vvb commented Feb 3, 2018

so, sorry for the delayed response.

I had to go and check this with the imc team. So looks like there were several inconsistencies in the property values in the earlier releases.

In relatively newer versions both enabled and Enabled should work.
In older versions

  • for some MO Properties enabled works, Enabled does not.
  • for some other MO Properties Enabled works, enabled does not.

In this specific case, Enabled works for both old and new versions.

I am approving this change.

Thank you for submitting this PR.

@vvb vvb merged commit 5768bab into CiscoUcs:master Feb 3, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants