Merge ~andreserl/maas:2.2_lp1717287 into maas:2.2

Proposed by Andres Rodriguez
Status: Merged
Approved by: Andres Rodriguez
Approved revision: bfb5a54a520a1763d28889635a01039710c930b5
Merge reported by: MAAS Lander
Merged at revision: not available
Proposed branch: ~andreserl/maas:2.2_lp1717287
Merge into: maas:2.2
Diff against target: 26 lines (+3/-1)
2 files modified
docs/changelog.rst (+2/-0)
src/metadataserver/user_data/templates/snippets/maas_enlist.sh (+1/-1)
Reviewer Review Type Date Requested Status
Andres Rodriguez (community) Approve
Review via email: mp+331060@code.launchpad.net

Commit message

Backport 21fb76a967c2d0196dc34b73d0ccab4e4dbd523b from master.

LP: #1717287 - maas-enlist not supporting IPv6 addresses in server_url

To post a comment you must log in.
Revision history for this message
Andres Rodriguez (andreserl) :
review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1diff --git a/docs/changelog.rst b/docs/changelog.rst
2index bfefebd..86ba2b3 100644
3--- a/docs/changelog.rst
4+++ b/docs/changelog.rst
5@@ -30,6 +30,8 @@ LP: #1704444 [2.2] MAAS API returns 500 internal server error instead of rais
6
7 LP: #1676992 Force Postgresql restart on maas-region-controller installation.
8
9+LP: #1717287 maas-enlist not supporting IPv6 addresses in server_url.
10+
11
12 2.2.2
13 =====
14diff --git a/src/metadataserver/user_data/templates/snippets/maas_enlist.sh b/src/metadataserver/user_data/templates/snippets/maas_enlist.sh
15index 27bff90..a579801 100644
16--- a/src/metadataserver/user_data/templates/snippets/maas_enlist.sh
17+++ b/src/metadataserver/user_data/templates/snippets/maas_enlist.sh
18@@ -231,7 +231,7 @@ fi
19 if echo "$serverurl" | egrep -q '(^[a-z]+://|^)[a-z0-9\.\-]+($|/$)'; then
20 api_url="MAAS/api/2.0/machines/"
21 else
22- api_url=`echo $serverurl | sed 's#^\(\|[a-z]\+://\)[a-zA-Z0-9\.\-]\+\(\|\:[0-9]\+\)/##'`
23+ api_url=`echo $serverurl | sed 's#^\(\|[a-z]\+://\)\([a-zA-Z0-9\.]\+\|\(\[[0-9a-fA-F:]\+\]\)\)\(\|\:[0-9]\+\)/##'`
24 fi
25
26 #TODO: Auto-detect hostname?

Subscribers

People subscribed via source and target branches