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

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

Open
chu11 opened this issue Apr 4, 2024 · 1 comment
Open

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

chu11 opened this issue Apr 4, 2024 · 1 comment

Comments

@chu11
Copy link
Member

chu11 commented Apr 4, 2024

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
Copy link
Member Author

chu11 commented Apr 30, 2024

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

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

No branches or pull requests

1 participant