Skip to content

update CI to use localstack CI auth token#96

Merged
alexrashed merged 2 commits intomasterfrom
switch-to-pro
Feb 12, 2026
Merged

update CI to use localstack CI auth token#96
alexrashed merged 2 commits intomasterfrom
switch-to-pro

Conversation

@alexrashed
Copy link
Member

Motivation

With the upcoming image consolidation, LocalStack will require an auth token to start up.
This PR updates the CI of this project to use the CI token configured as a secret for this project.

Fixes ENG-327.

Changes

  • Set LOCALSTACK_AUTH_TOKEN when starting LocalStack.
  • Remove unnecessary docker pull and wait timeout

Updated LocalStack step to include authentication token and removed unnecessary docker pull command.
@alexrashed alexrashed requested a review from k-a-il February 12, 2026 13:52
Copy link

@k-a-il k-a-il left a comment

Choose a reason for hiding this comment

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

LGTM 👍

@github-actions
Copy link

Smoke Tests

 6 files  ±0  12 suites  ±0   3m 23s ⏱️ +2s
 4 tests ±0   4 ✅ ±0  0 💤 ±0  0 ❌ ±0 
24 runs  ±0  24 ✅ ±0  0 💤 ±0  0 ❌ ±0 

Results for commit 3551cd1. ± Comparison against base commit e08cb3b.

@alexrashed alexrashed merged commit 691ec32 into master Feb 12, 2026
8 checks passed
@alexrashed alexrashed deleted the switch-to-pro branch February 12, 2026 14:54
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.

2 participants