chaos / powerman

cluster power control
GNU General Public License v2.0
43 stars 19 forks source link

should we add "setresult" to older powerman device files #171

Open chu11 opened 7 months ago

chu11 commented 7 months ago

With setresult soon to be merged (#168) there's a question if it should be used in older device files, notably redfishpower and ipmipower device files.

pro: it'll offer folks behavior that is more in line with what they are expecting

con: it breaks old behavior, especially device files that have been around for like 20 years

chu11 commented 6 months ago

or alternately, we create new device files like ipmipower_errorhandling.dev or whatever