Merge ~ltrager/maas:lp1730524 into maas:master
Proposed by
Lee Trager
Status: | Merged |
---|---|
Approved by: | Lee Trager |
Approved revision: | 3923d84a1a312adaa0d0bdc5ff5d07fdaedf5ac2 |
Merge reported by: | MAAS Lander |
Merged at revision: | not available |
Proposed branch: | ~ltrager/maas:lp1730524 |
Merge into: | maas:master |
Diff against target: |
24 lines (+5/-1) 1 file modified
src/metadataserver/user_data/templates/commissioning.template (+5/-1) |
Related bugs: |
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
MAAS Lander | Approve | ||
Blake Rouse (community) | Approve | ||
Review via email: mp+334894@code.launchpad.net |
Commit message
LP: #1730524 - Ping the metadata service before starting commissioning.
Before starting commissioning ping the metadata service, if unable exit. This
prevents the BMC password from being changed with the ephemeral environment
having no way to signal.
To post a comment you must log in.
Looks good.
This is a good change, but I would like to see this even more robust.
1. Check we can talk to MAAS. (as you have added)
2. Save current password on BMC.
3. Update password on BMC.
4. Update MAAS with new BMC.
5. If #4 fails then reset BMC password back to saved from #2.
I think that #5 would really make this a robust change.
I am not going to block you on this branch, as I know your trying to make this a quick fix. But it would be a good change to see.