Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[v23.3.x] gha: use oidc #23452

Merged
merged 8 commits into from
Sep 26, 2024
Merged

[v23.3.x] gha: use oidc #23452

merged 8 commits into from
Sep 26, 2024

Conversation

andrewhsu
Copy link
Member

@andrewhsu andrewhsu commented Sep 24, 2024

Backport of PR #23423
Fixes #23437
jira: PESDLC-1736

This PR updates the workflow files triggered from the v23.3.x branch that use action aws-actions/configure-aws-credentials to run with OIDC tokens.

There were many conflicts when I first attempted to cherry-pick all the commits from PR #23423 that were unrelated to OIDC tokens and/or changed workflow files that wouldn't run on this v23.3.x branch, so I only cherry-picked a subset of the commits that are needed. This reduced the changeset to just these files:

  1. cloud-installpack-bk-trigger.yml
  2. promote.yml
  3. release-rp-storage-tool.yml

Backports Required

  • none - not a bug fix
  • none - this is a backport
  • none - issue does not exist in previous branches
  • none - papercut/not impactful enough to backport
  • v24.2.x
  • v24.1.x
  • v23.3.x

Release Notes

  • none

(cherry picked from commit 0020a8a)
(cherry picked from commit 58cf73d)
(cherry picked from commit eb7121b)
These expressions will not guard against anything with the workflow only
triggering on tags with "v*".

(cherry picked from commit f504034)
@andrewhsu andrewhsu force-pushed the PESDLC-1736-v23.3.x-gh-oidc branch 2 times, most recently from c3720c7 to 42bf760 Compare September 25, 2024 18:22
@andrewhsu andrewhsu marked this pull request as ready for review September 25, 2024 18:49
@andrewhsu andrewhsu requested a review from a team as a code owner September 25, 2024 18:49
@andrewhsu andrewhsu requested review from hobbseltoff, ivotron and WillemKauf and removed request for a team and hobbseltoff September 25, 2024 18:49
@andrewhsu
Copy link
Member Author

@WillemKauf it seems that even if i backport commits from #23036 #23045 to this v23.3.x branch, there is still a rust error. is there something else i should also backport? or should i not backport those PR changes to this branch?

@WillemKauf
Copy link
Contributor

WillemKauf commented Sep 25, 2024

Yeah looks like the transform SDK has issues with the bumped Rust version.

I'm not sure if the transform SDK can have its Rust version easily bumped and backported as well, or if you would prefer to just not backport either of those Rust related commits. For simplicity, I don't think it's a big deal to leave them out.

@andrewhsu andrewhsu force-pushed the PESDLC-1736-v23.3.x-gh-oidc branch from 327e892 to dadc289 Compare September 25, 2024 18:57
@andrewhsu
Copy link
Member Author

For simplicity, I don't think it's a big deal to leave them out.

i dropped those commits. ready for review.

Copy link
Member

@ivotron ivotron left a comment

Choose a reason for hiding this comment

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

lgtm. i don't think the installpack workflow gets exercised since 23.3 is not available in our claoud anymore

@andrewhsu andrewhsu merged commit d749c59 into v23.3.x Sep 26, 2024
11 checks passed
@andrewhsu andrewhsu deleted the PESDLC-1736-v23.3.x-gh-oidc branch September 26, 2024 18:00
@BenPope BenPope added this to the v23.3.22 milestone Oct 21, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants