Skip to content
This repository was archived by the owner on Mar 23, 2026. It is now read-only.

Rename default container name to localstack-main#9469

Merged
simonrw merged 2 commits intorelease/v3.0from
major/rename-default-container-name
Nov 2, 2023
Merged

Rename default container name to localstack-main#9469
simonrw merged 2 commits intorelease/v3.0from
major/rename-default-container-name

Conversation

@simonrw
Copy link
Copy Markdown
Contributor

@simonrw simonrw commented Oct 25, 2023

Motivation

Hyphens are not allowed in URLs. When addressing the LocalStack container over a docker network, the container name resolves to the IP address of the container. This is a problem in particular for boto3 which refuses to connect.

Changes

Rename localstack_main -> localstack-main

@simonrw simonrw added the semver: major Breaking changes which can be included in a major release only label Oct 25, 2023
@simonrw simonrw self-assigned this Oct 25, 2023
@github-actions
Copy link
Copy Markdown

github-actions Bot commented Oct 25, 2023

LocalStack Community integration with Pro

       2 files  ±0         2 suites  ±0   1h 18m 51s ⏱️ + 8m 39s
2 324 tests ±0  1 747 ✔️ ±0  577 💤 ±0  0 ±0 
2 325 runs  ±0  1 747 ✔️ ±0  578 💤 ±0  0 ±0 

Results for commit 8bc385a. ± Comparison against base commit 6ff5c4c.

♻️ This comment has been updated with latest results.

@coveralls
Copy link
Copy Markdown

coveralls commented Oct 25, 2023

Coverage Status

coverage: 83.723% (-0.03%) from 83.756% when pulling 569f152 on major/rename-default-container-name into 8c937c6 on release/v3.0.

@simonrw simonrw marked this pull request as ready for review October 31, 2023 10:41
@simonrw simonrw requested a review from alexrashed October 31, 2023 10:41
Copy link
Copy Markdown
Member

@alexrashed alexrashed left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Awesome! It's great to see this change getting over the line! There aren't too many changes necessary, so I think this should be fine, especially if we can prepare our users appropriately and update all our own repos and docs in advance! 🚀 🧹

Copy link
Copy Markdown
Member

@joe4dev joe4dev left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM on the LocalStack side 👍

There are some more references in other places to sync:

@alexrashed alexrashed force-pushed the release/v3.0 branch 3 times, most recently from a4a87dc to b484b3a Compare November 1, 2023 14:40
@simonrw simonrw force-pushed the major/rename-default-container-name branch from 569f152 to ebaccf6 Compare November 1, 2023 20:54
@alexrashed alexrashed force-pushed the release/v3.0 branch 6 times, most recently from 19c8422 to 6ff5c4c Compare November 2, 2023 09:24
@simonrw simonrw force-pushed the major/rename-default-container-name branch from ebaccf6 to 8bc385a Compare November 2, 2023 09:28
@simonrw simonrw merged commit c5ffecd into release/v3.0 Nov 2, 2023
@simonrw simonrw deleted the major/rename-default-container-name branch November 2, 2023 11:22
simonrw added a commit to localstack-samples/localstack-demo that referenced this pull request Nov 3, 2023
The previous default container name `localstack_main` is not a valid URL, so
we rename the default to `localstack-main`.

See localstack/localstack#9469.
simonrw added a commit to localstack/localstack-java-utils that referenced this pull request Nov 3, 2023
The previous default container name `localstack_main` is not a valid URL, so
we rename the default to `localstack-main`.

See localstack/localstack#9469.
simonrw added a commit to localstack-samples/localstack-pro-samples that referenced this pull request Nov 3, 2023
The previous default container name `localstack_main` is not a valid URL, so
we rename the default to `localstack-main`.

See localstack/localstack#9469.
simonrw added a commit to localstack-samples/localstack-terraform-samples that referenced this pull request Nov 3, 2023
The previous default container name `localstack_main` is not a valid URL, so
we rename the default to `localstack-main`.

See localstack/localstack#9469.
simonrw added a commit to localstack/docs that referenced this pull request Nov 6, 2023
The previous default container name `localstack_main` is not a valid URL, so
we rename the default to `localstack-main`.

See localstack/localstack#9469.
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

semver: major Breaking changes which can be included in a major release only

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants