Skip to content

update localstack helm chart to default to localstack-pro#148

Merged
alexrashed merged 5 commits intomainfrom
ENG-350-update-image-consolidate
Mar 12, 2026
Merged

update localstack helm chart to default to localstack-pro#148
alexrashed merged 5 commits intomainfrom
ENG-350-update-image-consolidate

Conversation

@alexrashed
Copy link
Member

Motivation

Currently the helm chart in this repo defaults to use LocalStack Community.
This will require an auth token after the consolidation of our images.
This is why this PR prepares for this change by defaulting to the localstack-pro image, describing the usage of the auth token, and updating CI to also use a token.

Changes

  • Change the default value of image.repository to localstack/localstack-pro to prepare for the upcoming image consolidation.
  • Update the docs to mention that the auth token is required.
  • Update CI such that it also sets a secret CI auth token when running the tests.

@alexrashed alexrashed force-pushed the ENG-350-update-image-consolidate branch from c712780 to 08aea76 Compare March 11, 2026 13:51
@alexrashed alexrashed force-pushed the ENG-350-update-image-consolidate branch from 08aea76 to 1a48560 Compare March 11, 2026 14:11
@alexrashed alexrashed force-pushed the ENG-350-update-image-consolidate branch from 1a48560 to 328d7cb Compare March 11, 2026 14:17
@alexrashed alexrashed marked this pull request as ready for review March 11, 2026 14:22
@alexrashed alexrashed requested a review from a team March 11, 2026 14:22
Copy link
Contributor

@simonrw simonrw left a comment

Choose a reason for hiding this comment

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

Thanks for updating this. Changes look good

@alexrashed alexrashed merged commit f3bb44b into main Mar 12, 2026
2 checks passed
@alexrashed alexrashed deleted the ENG-350-update-image-consolidate branch March 12, 2026 09:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants