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

ci: use release environment in release workflow integ tests #112

Merged
merged 1 commit into from
Dec 13, 2023

Conversation

jericht
Copy link
Contributor

@jericht jericht commented Dec 12, 2023

What was the problem/requirement? (What/Why)

The new release workflow failed because the integration test stage to configure AWS credentials failed (see https://github.com/casillas2/deadline-cloud-worker-agent/actions/runs/7188624961/job/19578694519#step:3:10). Root cause of this seems to be that we're referencing a release secret environment variable in that stage but we're missing the statement in the workflow file to use the release environment.

What was the solution? (How)

Use the release environment in the integration tests of the release workflow

What is the impact of this change?

release workflow works

How was this change tested?

N/A / TODO

Was this change documented?

No

Is this a breaking change?

No

Signed-off-by: Jericho Tolentino <68654047+jericht@users.noreply.github.com>
@jericht jericht requested a review from a team as a code owner December 12, 2023 23:55
@jusiskin jusiskin changed the title fix: use release environment in release workflow integ tests ci: use release environment in release workflow integ tests Dec 13, 2023
@moorec-aws moorec-aws merged commit 3ae3a16 into mainline Dec 13, 2023
9 checks passed
@moorec-aws moorec-aws deleted the jericht/fix_release branch December 13, 2023 00:48
gmchale79 pushed a commit that referenced this pull request Mar 11, 2024
Signed-off-by: Jericho Tolentino <68654047+jericht@users.noreply.github.com>
Signed-off-by: Graeme McHale <gmchale@amazon.com>
jusiskin pushed a commit to jusiskin/deadline-cloud-worker-agent that referenced this pull request Sep 4, 2024
Signed-off-by: client-software-ci <129794699+client-software-ci@users.noreply.github.com>
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.

3 participants