Merge ~ltrager/maas:lp1730524_2.3 into maas:2.3

Proposed by Lee Trager
Status: Merged
Approved by: Lee Trager
Approved revision: 8478527bb88fc2a0a776588999aece2f04e7f347
Merge reported by: MAAS Lander
Merged at revision: not available
Proposed branch: ~ltrager/maas:lp1730524_2.3
Merge into: maas:2.3
Diff against target: 24 lines (+5/-1)
1 file modified
src/metadataserver/user_data/templates/commissioning.template (+5/-1)
Reviewer Review Type Date Requested Status
Alberto Donato (community) Approve
MAAS Lander Approve
Review via email: mp+335193@code.launchpad.net

Commit message

Backport of af2a66a - 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.
Revision history for this message
MAAS Lander (maas-lander) wrote :

UNIT TESTS
-b lp1730524_2.3 lp:~ltrager/maas into -b 2.3 lp:~maas-committers/maas

STATUS: SUCCESS
COMMIT: 8478527bb88fc2a0a776588999aece2f04e7f347

review: Approve
Revision history for this message
Alberto Donato (ack) wrote :

+1

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1diff --git a/src/metadataserver/user_data/templates/commissioning.template b/src/metadataserver/user_data/templates/commissioning.template
2index f0d49ab..bea840f 100644
3--- a/src/metadataserver/user_data/templates/commissioning.template
4+++ b/src/metadataserver/user_data/templates/commissioning.template
5@@ -27,6 +27,10 @@ add_ipmi_config() {
6 main() {
7 prep_maas_api_helper
8
9+ # LP:1730524 - Make sure we can signal the metadata service before updating
10+ # the BMC username and password.
11+ signal WORKING "Starting [maas-bmc-autodetect]" || exit 1
12+
13 # Install IPMI deps
14 aptget install freeipmi-tools openipmi ipmitool sshpass
15
16@@ -61,7 +65,7 @@ main() {
17 if [ ! -z "$power_settings" ]; then
18 signal \
19 "--power-type=${power_type}" "--power-parameters=${power_settings}" \
20- WORKING "Finished [maas-ipmi-autodetect]"
21+ WORKING "Finished [maas-bmc-autodetect]"
22 fi
23
24 maas-run-remote-scripts "--config=${CRED_CFG}" "${TEMP_D}"

Subscribers

People subscribed via source and target branches