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: prevent parallel builds from causing conflicts #1900

Merged
merged 2 commits into from
Mar 3, 2021

Conversation

nixpanic
Copy link
Member

@nixpanic nixpanic commented Mar 2, 2021

When the container image needs to be rebuild, two parallel jobs will try
to attempt that. With recent versions of Podman, this now fails.

When the image needs to be rebuild, do so in the stage where it would
otherwise get pulled. This makes sure the image gets build only once.


Show available bot commands

These commands are normally not required, but in case of issues, leave any of
the following bot commands in an otherwise empty comment in this PR:

  • /retest ci/centos/<job-name>: retest the <job-name> after unrelated
    failure (please report the failure too!)
  • /retest all: run this in case the CentOS CI failed to start/report any test
    progress or results

nixpanic added 2 commits March 2, 2021 14:52
When the container image needs to be rebuild, two parallel jobs will try
to attempt that. With recent versions of Podman, this now fails.

When the image needs to be rebuild, do so in the stage where it would
otherwise get pulled. This makes sure the image gets build only once.

Signed-off-by: Niels de Vos <ndevos@redhat.com>
Fixes: ceph#1670
Signed-off-by: Niels de Vos <ndevos@redhat.com>
@nixpanic nixpanic added the component/testing Additional test cases or CI work label Mar 2, 2021
@nixpanic nixpanic requested review from Madhu-1 and Yuggupta27 March 2, 2021 13:57
@Madhu-1 Madhu-1 requested a review from humblec March 3, 2021 03:46
@mergify mergify bot merged commit 415abea into ceph:ci/centos Mar 3, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
component/testing Additional test cases or CI work
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants