Created attachment 596485 [details] screen shot of power management configuration Description of problem: The Power Management options presented in RHEV-M include ilo and ilo3 but not iLO2 (which works like the original ilo) or iLO4 (which works like iLO3). Please add iLO2 and more importantly, iLO4 as options. Version-Release number of selected component (if applicable): RHEV 3.0 How reproducible: Always Steps to Reproduce: 1. Add a host and configure power management. 2. 3. Actual results: Notice that ilo and ilo3 are the only ilo* options. (See attachment) Expected results: Would like to see iLO, iLO2, iLO3 and iLO4 as options. Bonus points for fixing the capitalization if possible. iLO2 should have the same configuration options as iLO. iLO4 should have the same configuration options as iLO3. Perhaps these can be provided as just two options: iLO/iLO2 and iLO3/iLO4. Additional info: There should be no new iLO versions for a while. :-)
i'm for just fixing the text. though need to verify if this means something during upgrade, or just display name. need to consider rest api backward compatibility for names if relevant. perry - any thoughts/considerations from your side?
note: may be easier to just add in the config ilo2 and ilo4 to avoid upgrade and backward compatibility
(In reply to comment #2) > note: may be easier to just add in the config ilo2 and ilo4 to avoid upgrade > and backward compatibility Works for me. It might also be better in case we uncover an incompatibility or optimization that only applies to one flavor down the road.
(In reply to comment #1) > i'm for just fixing the text. > though need to verify if this means something during upgrade, or just > display name. > need to consider rest api backward compatibility for names if relevant. > > perry - any thoughts/considerations from your side? Since iLO2 and iLO have the same config and usage of fence_ilo and iLO3/iLO4 have the same config/usage of fence_ipmilan, I think you are correct in suggesting that this could simply be a cosmetic change to the drop down box. However if changing the display name generates issues with backward compat, I also see no issue in introducing two new items in the drop down for iLO2 and iLO4
I suggest to 1) Add iLo2 and iLo4 as new entries 2) We have the infrastructure to map iLo4 to iLo3 and iLo2 to iLo, this is exactly how we implemented iLo3 via ipmilan This way, the whole change will be in the configuration settings (0000config.sql) and will be backward compatible. please approve
Sounds good to me. Thanks.
http://gerrit.ovirt.org/#/c/8492/
(In reply to comment #6) > Sounds good to me. Thanks. Linda, will you be able to test it? I'm not sure we have either in our labs (we have iLO3).
I'm traveling this week but I should be able to test it when I'm back. I just need to know where/how to get the bits. Can I just test the patched file or do I need to update everything? -- ljk
QA_ACK+ - HP can test it.
fixed in commit : 154ac74
see comment 10. cannot be verified in our lab.
This bug is currently attached to errata RHEA-2013:14491. If this change is not to be documented in the text for this errata please either remove it from the errata, set the requires_doc_text flag to minus (-), or leave a "Doc Text" value of "--no tech note required" if you do not have permission to alter the flag. Otherwise to aid in the development of relevant and accurate release documentation, please fill out the "Doc Text" field above with these four (4) pieces of information: * Cause: What actions or circumstances cause this bug to present. * Consequence: What happens when the bug presents. * Fix: What was done to fix the bug. * Result: What now happens when the actions or circumstances above occur. (NB: this is not the same as 'the bug doesn't present anymore') Once filled out, please set the "Doc Type" field to the appropriate value for the type of change made and submit your edits to the bug. For further details on the Cause, Consequence, Fix, Result format please refer to: https://bugzilla.redhat.com/page.cgi?id=fields.html#cf_release_notes Thanks in advance.
Sorry, just got back from being away for several weeks and saw the request to test this change. I can test the change but in the meantime, changing the support to tech preview makes no sense. Even without this change, iLO2 and iLO4 were supported, they just didn't have their own pulldown menu options. Instead, one had to select iLO to get iLO2 or select iLO3 to get iLO4 support. This change only adds the explicit menu options.
(In reply to Linda Knippers from comment #14) > Sorry, just got back from being away for several weeks and saw the request > to test this change. I can test the change but in the meantime, changing > the support to tech preview makes no sense. Even without this change, iLO2 > and iLO4 were supported, they just didn't have their own pulldown menu > options. Instead, one had to select iLO to get iLO2 or select iLO3 to get > iLO4 support. This change only adds the explicit menu options. This is going into the online release notes and not to the packaged release notes. If you can test it in the next 5 days we'll probably be able to prevent even transient mention of this tech-preview state.
We have verified iLO3 and iLO4. iLO worked before as should have been unchanged. I'm looking for someone to try iLO2.
Since the problem described in this bug report should be resolved in a recent advisory, it has been closed with a resolution of ERRATA. For information on the advisory, and where to find the updated files, follow the link below. If the solution does not work for you, open a new bug report. http://rhn.redhat.com/errata/RHSA-2013-0888.html