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/ (+1/-1)
Reviewer Review Type Date Requested Status
Anton Troyanov Approve
Review via email:

Commit message

Fix utf-8 character error when bmc-config

Related LP and article
(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
1diff --git a/src/metadataserver/builtin_scripts/commissioning_scripts/ b/src/metadataserver/builtin_scripts/commissioning_scripts/
2index a4c791f..4964f08 100755
3--- a/src/metadataserver/builtin_scripts/commissioning_scripts/
4+++ b/src/metadataserver/builtin_scripts/commissioning_scripts/
5@@ -242,7 +242,7 @@ class IPMIBase(BMCConfig):
6 )
7 raise
8 section = None
9- for line in proc.stdout.decode().splitlines():
10+ for line in proc.stdout.decode(errors="surrogateescape").splitlines():
11 line = line.split("#")[0].strip()
12 if not line:
13 continue


People subscribed via source and target branches