Skip to content

PIE-3516 part 2: use AWS SSM instead and simplify the release workflow#39

Merged
zhming0 merged 1 commit intomainfrom
ming/pie-3516-part-2
Mar 25, 2025
Merged

PIE-3516 part 2: use AWS SSM instead and simplify the release workflow#39
zhming0 merged 1 commit intomainfrom
ming/pie-3516-part-2

Conversation

@zhming0
Copy link
Contributor

@zhming0 zhming0 commented Mar 25, 2025

  • After clarifying with other engineers, I replace Buildkite Secret with AWS SSM.
  • I also tweak the release flow so it's much simplified.

If this is successful, we can probably apply the same practice to other Test Engine open source repos.

@zhming0 zhming0 requested review from a team and nprizal March 25, 2025 22:43
1. Open a new PR bumping the version number in `constants.py`, make sure the PR title contains `[release]`.
2. Get the PR approved and merged, this will trigger the release pipeline.
3. (Optional) In the event of step 3 failure, run `.buildkite/steps/release-pypi` locally with your own credentials.
4. Create a [new github release](https://github.com/buildkite/test-collector-python/releases) for prosperity, you can create a tag as you create the release.
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Ideally, we should get automate tag and release creation. but beyond the scope of this PR I guess.

variables:
TWINE_PASSWORD: test-collector-python-pypi-api-token
- aws-assume-role-with-web-identity#v1.2.0:
role-arn: "arn:aws:iam::445615400570:role/pipeline-buildkite-test-collector-python-build"
Copy link
Contributor

Choose a reason for hiding this comment

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

👍 this aligns with bktec release pipeline

@zhming0 zhming0 merged commit c0a5c8a into main Mar 25, 2025
11 checks passed
@zhming0 zhming0 deleted the ming/pie-3516-part-2 branch March 25, 2025 23:28
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