Skip to content

release workflow: increase timeout to 30 minutes & remove Go setup action#7259

Merged
dmcgowan merged 2 commits intocontainerd:mainfrom
samuelkarp:release-workflow
Aug 5, 2022
Merged

release workflow: increase timeout to 30 minutes & remove Go setup action#7259
dmcgowan merged 2 commits intocontainerd:mainfrom
samuelkarp:release-workflow

Conversation

@samuelkarp
Copy link
Member

In the 1.6.7 release, we saw significantly longer execution time for producing builds that exceeded the previous timeout of 10 minutes, causing the workflow to fail. After increasing to 20 minutes in the release/1.6 branch, we continued to see one failure (which succeeded on retry).

Increase to 30 minutes to provide additional buffer for the build to complete.


Release builds are performed from within a Dockerfile-defined environment and do not require Go to be installed in the GitHub Actions runner environment.


See https://github.com/samuelkarp/containerd/actions/runs/2805142467 for a successful run.

In the 1.6.7 release, we saw significantly longer execution time for
producing builds that exceeded the previous timeout of 10 minutes,
causing the workflow to fail.  After increasing to 20 minutes in the
release/1.6 branch, we continued to see one failure (which succeeded on
retry).

Increase to 30 minutes to provide additional buffer for the build to
complete.

Signed-off-by: Samuel Karp <samuelkarp@google.com>
Release builds are performed from within a Dockerfile-defined
environment and do not require Go to be installed in the GitHub Actions
runner environment.

Signed-off-by: Samuel Karp <samuelkarp@google.com>
@samuelkarp samuelkarp requested a review from dmcgowan August 5, 2022 17:39
@dmcgowan
Copy link
Member

dmcgowan commented Aug 5, 2022

Did a test run with this change https://github.com/dmcgowan/containerd/actions/runs/2805405593

Everything looks good, increasing the timeout makes sense since we don't want to have our release blocked if there is some gha weirdness slowing the builds down

Copy link
Member

@dcantah dcantah left a comment

Choose a reason for hiding this comment

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

LGTM

@dmcgowan dmcgowan merged commit de68174 into containerd:main Aug 5, 2022
@samuelkarp samuelkarp deleted the release-workflow branch August 6, 2022 01:10
@mxpv mxpv added cherry-picked/1.5.x PR commits are cherry-picked into release/1.5 branch cherry-picked/1.6.x PR commits are cherry-picked into release/1.6 branch and removed cherry-pick/1.5.x labels Aug 9, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

cherry-picked/1.5.x PR commits are cherry-picked into release/1.5 branch cherry-picked/1.6.x PR commits are cherry-picked into release/1.6 branch

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants