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

Added statement for non-idempotent resources #284

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Added statement for non-idempotent resources #284

wants to merge 1 commit into from

Conversation

riconem
Copy link

@riconem riconem commented Sep 14, 2020

Description

Added statement for non-idempotent operation.

Issues Resolved

Before these changes the oneview_sas_logical_interconnect, oneview_logical_interconnect and oneview_logical_enclosure resources will fail the update_from_group or set_compliance method for several puppet runs with no changes.

Check List

  • New functionality includes testing.
    • All tests pass ($ rake test).
  • New functionality has been documented in the README if applicable.
    • New functionality has been thoroughly documented in the examples (please include helpful comments).
  • Changes are documented in the CHANGELOG.

@riconem
Copy link
Author

riconem commented Sep 14, 2020

I am not sure, why the tests are failing. I tested the code on a synergy. Maybe it's a synergy specific problem. I am not able to test it on a C7000.

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.

1 participant