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

dellemc_change_power_state not idempotent #115

Closed
tsquillario opened this issue Apr 24, 2020 · 4 comments
Closed

dellemc_change_power_state not idempotent #115

tsquillario opened this issue Apr 24, 2020 · 4 comments
Labels
area/idrac idrac modules/roles type/bug Something isn't working

Comments

@tsquillario
Copy link

tsquillario commented Apr 24, 2020

When you try to power on a server that is already powered on you receive an error. The dellemc_change_power_state module should catch this error and report status as unchanged.

Current workaround is to ignore_errors.

FAILED! => {"ansible_facts": {"discovered_interpreter_python": "/usr/bin/python"}, "changed": false, "msg": {"Data": {"Message": "returned status code doesn't match with the expected success code", "Status": "Failed", "StatusCode": 409}, "Message": "none", "Status": "Failed", "StatusCode": 409, "error": {"error": {"@Message.ExtendedInfo": [{"Message": "Server is already powered ON.", "MessageArgs": [], "MessageArgs@odata.count": 0, "MessageId": "IDRAC.2.1.PSU501", "RelatedProperties": [], "RelatedProperties@odata.count": 0, "Resolution": "No response action is required.", "Severity": "Informational"}], "code": "Base.1.5.GeneralError", "message": "A general error has occurred. See ExtendedInfo for more information"}}, "retval": true}}

@jagadeeshnv
Copy link
Contributor

Thank you @tsquillario we have taken note of this issue. Will fix in the next releases

@anupamaloke anupamaloke added the type/bug Something isn't working label Jul 21, 2020
@rajeevarakkal rajeevarakkal added the area/idrac idrac modules/roles label Jul 28, 2020
@Sajna-Shetty
Copy link
Contributor

@tsquillario We have replaced 'dellemc_change_power_state' module with newer module 'redfish_powerstate'. Verify idempotency behaviour in the newer module.

@jagadeeshnv
Copy link
Contributor

@tsquillario Can you please verify and close this issue?

@jagadeeshnv
Copy link
Contributor

@tsquillario This has been fixed module 'redfish_powerstate' as stated by @Sajna-Shetty. Hence closing this issue. Thank You

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/idrac idrac modules/roles type/bug Something isn't working
Projects
None yet
Development

No branches or pull requests

5 participants