Merge ~troyanov/maas:backport-lp1929478-to-3.2 into maas:3.2

Proposed by Anton Troyanov
Status: Merged
Approved by: Anton Troyanov
Approved revision: 3132cb22bc5aee66dbc161aac72979aa7e345140
Merge reported by: MAAS Lander
Merged at revision: not available
Proposed branch: ~troyanov/maas:backport-lp1929478-to-3.2
Merge into: maas:3.2
Diff against target: 13 lines (+1/-1)
1 file modified
src/metadataserver/builtin_scripts/commissioning_scripts/bmc_config.py (+1/-1)
Reviewer Review Type Date Requested Status
Anton Troyanov Approve
Review via email: mp+427404@code.launchpad.net

Commit message

Fix utf-8 character error when bmc-config

Related LP and article

https://bugs.launchpad.net/bugs/1892983
https://bugs.launchpad.net/bugs/1929478
https://discourse.maas.io/t/maas-failed-commissioning/4904
(cherry picked from commit 3eec5b5161515a23f58c72638a5ff8aac7486622)

To post a comment you must log in.
Revision history for this message
Anton Troyanov (troyanov) wrote :

self approve backport MP

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
diff --git a/src/metadataserver/builtin_scripts/commissioning_scripts/bmc_config.py b/src/metadataserver/builtin_scripts/commissioning_scripts/bmc_config.py
index a4c791f..4964f08 100755
--- a/src/metadataserver/builtin_scripts/commissioning_scripts/bmc_config.py
+++ b/src/metadataserver/builtin_scripts/commissioning_scripts/bmc_config.py
@@ -242,7 +242,7 @@ class IPMIBase(BMCConfig):
242 )242 )
243 raise243 raise
244 section = None244 section = None
245 for line in proc.stdout.decode().splitlines():245 for line in proc.stdout.decode(errors="surrogateescape").splitlines():
246 line = line.split("#")[0].strip()246 line = line.split("#")[0].strip()
247 if not line:247 if not line:
248 continue248 continue

Subscribers

People subscribed via source and target branches