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

fix(scripts): make aws deploy scripts work with new sdk #16586

Merged
merged 1 commit into from
Oct 24, 2024

Conversation

shlokamin
Copy link
Member

@shlokamin shlokamin commented Oct 23, 2024

Overview

After #15267 merged which updated the aws sdk from 2 to 3 our web deploy scripts stopped working for staging and production releases. This PR updates our scripts to use v3 of the aws sdk (hopefully correctly).

I verified that staging promotions work, and production should work too but I'm not gonna do a prod release right now.

closes AUTH-979

Review requests

  • @jerader try doing a staging deploy, it should work!

Risk assessment

Medium

@shlokamin shlokamin changed the title fix(scripts): make aws command work with new sdk fix(scripts): make aws commands work with new sdk Oct 23, 2024
@shlokamin shlokamin marked this pull request as ready for review October 24, 2024 14:04
@shlokamin shlokamin requested a review from a team as a code owner October 24, 2024 14:04
@shlokamin shlokamin requested review from jerader and removed request for a team October 24, 2024 14:04
@shlokamin shlokamin changed the title fix(scripts): make aws commands work with new sdk fix(scripts): make aws deploy scrips work with new sdk Oct 24, 2024
@shlokamin shlokamin changed the title fix(scripts): make aws deploy scrips work with new sdk fix(scripts): make aws deploy scripts work with new sdk Oct 24, 2024
@jerader jerader requested a review from ecormany October 24, 2024 15:15
Copy link
Collaborator

@jerader jerader 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 tested promoting PD to staging.

@shlokamin shlokamin merged commit e32c0f3 into edge Oct 24, 2024
12 checks passed
@shlokamin shlokamin deleted the scripts_fix-web-deploy-scripts branch October 24, 2024 15:38
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