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

Chassis power types should be dynamically listed #1012

Closed
Treydone opened this issue Apr 21, 2020 · 2 comments · Fixed by #1040
Closed

Chassis power types should be dynamically listed #1012

Treydone opened this issue Apr 21, 2020 · 2 comments · Fixed by #1040

Comments

@Treydone
Copy link

All the chassis power types are listed in ui/src/app/machines/views/AddChassis/chassisPowerTypes.js. If we add a new chassis power types, we have to change the backend and the ui. A better way is to use the settings not on scope SETTING_SCOPE.NODE registered in each driver and available in /MAAS/api/2.0/machines/?op=power_parameters

@squidsoup
Copy link
Contributor

Thanks for submitting an issue @Treydone, that's certainly something we should investigate!

@Caleb-Ellis
Copy link
Contributor

Each power type returned from general.power_types now has a boolean field chassis which we can use to get a list of chassis power types.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants