Skip to content

Fix broken workflows since the introduction of private ecr repository#4220

Merged
landonxjames merged 3 commits intomainfrom
ysaito/fix-broken-workflows
Jul 16, 2025
Merged

Fix broken workflows since the introduction of private ecr repository#4220
landonxjames merged 3 commits intomainfrom
ysaito/fix-broken-workflows

Conversation

@ysaito1001
Copy link
Contributor

Motivation and Context

Minor cleanup following #4153

Description

The following workflows have been broken due to minor issues—such as missing secrets, not using the latest upload scripts, and so on:

  • pull-request-updating-lockfiles.yml
    • update-lockfiles.yml and manual-update-lockfiles.yml are callers of the given workflow
  • ci-main.yml
  • dry-run-release-scheduled.yml

Testing

The changes to ci-main.yml and dry-run-release-scheduled.yml haven't been verified, as these workflows only run in live environments (if they are still broken after this PR, will look into them further).

For the changes to pull-request-updating-lockfiles.yml, manually triggered manual-update-lockfiles.yml and confirmed that #4219 has been created.


By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

@ysaito1001 ysaito1001 requested review from a team as code owners July 16, 2025 19:37
@github-actions
Copy link

A new generated diff is ready to view.

  • No codegen difference in the AWS SDK
  • No codegen difference in the Client Test
  • No codegen difference in the Server Test
  • No codegen difference in the Server Test Python
  • No codegen difference in the Server Test Typescript

A new doc preview is ready to view.

@landonxjames landonxjames merged commit fca4d08 into main Jul 16, 2025
48 checks passed
@landonxjames landonxjames deleted the ysaito/fix-broken-workflows branch July 16, 2025 20:40
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