-
Notifications
You must be signed in to change notification settings - Fork 47
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
Dependency Updates - October 23, 2023 #4133
Dependency Updates - October 23, 2023 #4133
Conversation
…-4' into Dependency-updates-2023-oct-week-4
Nice! Is the new ECR image on the Court lower account, too? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks like the reindex was manually approved. What was the underlying reason it wasn't going to auto-approve?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Is there an update for the AWS provider used in our Terraform files?
I... believe so? I published the ECR image with the script we have and that should provide the image everywhere, right?
Yeah, I talked to Jim about this, and the way the wait-for script works, you have to manually approve the step if the destination table has more records than the source table. There was a discrepancy in the number of judges, which I believe is something you saw when you were updating dependencies last week.
There wasn't when I created the first PR, but I see there is one now. I could include that here or leave it. I'm open. |
It depends on which credentials are set while the script is run. It needs to be run once with Flexion creds (for exp envs) and once with USTC lower env creds (for test/staging). Once the image is built, you should just need to re-tag and re-push it using the other account. Happy to help |
Thanks! We got this resolved, and I updated the documentation to be clearer about this. |
updated minor versions
updated ECR image due to awscli version update
Successful migration