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

feat: Retry if renaming failed with 'name a repository operation is already in progress' error #1497

Merged
merged 1 commit into from
Jan 28, 2025

Conversation

jhutar
Copy link
Collaborator

@jhutar jhutar commented Jan 28, 2025

Description

Getting this:

I0127 20:47:45.961453   24251 logging.go:66] FAIL(80): Repo forking failed: Error renaming XXXXXXXXXXXXXXX/nodejs-devfile-sample-1 to nodejs-devfile-sample-cidaily-61000-0002: PATCH https://api.github.com/repos/XXXXXXXXXXXXXXX/nodejs-devfile-sample-1: 422 Validation Failed [{Resource:Repository Field:name Code:custom Message:name a repository operation is already in progress}]

Looks like we need to retry when renaming the repo here.

Issue ticket number and link

None

Type of change

  • New feature (non-breaking change which adds functionality)

How Has This Been Tested?

It was not tested, will run CI job here

@openshift-ci openshift-ci bot requested review from ascerra and tisutisu January 28, 2025 06:43
@jhutar jhutar force-pushed the fix41-retry-repo-rename branch from c0c5799 to 213bd30 Compare January 28, 2025 06:51
@jhutar
Copy link
Collaborator Author

jhutar commented Jan 28, 2025

/test load-test-ci-10u

@jhutar
Copy link
Collaborator Author

jhutar commented Jan 28, 2025

/retest

@konflux-ci-qe-bot
Copy link

@jhutar: The following test has Failed, say /retest to rerun failed tests.

PipelineRun Name Status Rerun command Build Log Test Log
konflux-e2e-vjmhx Failed /retest View Pipeline Log View Test Logs

Inspecting Test Artifacts

To inspect your test artifacts, follow these steps:

  1. Install ORAS (see the ORAS installation guide).
  2. Download artifacts with the following commands:
mkdir -p oras-artifacts
cd oras-artifacts
oras pull quay.io/konflux-test-storage/konflux-team/e2e-tests:konflux-e2e-vjmhx

Test results analysis

🚨 Error occurred while running the E2E tests, list of failed Spec(s):

➡️ [failed] [It] [integration-service-suite Creation of group snapshots for monorepo and multiple repos] with status reporting of Integration tests in CheckRuns when both the init PaC PRs are merged wait for the last components build to finish [integration-service, group-snapshot-creation]

Click to view logs

Expected success, but got an error:
    <*errors.errorString | 0xc0011c7f80>: 
    Pipelinerun 'go-component-qfkwvs-on-pull-request-ctmbb' didn't succeed
    Logs from failed container 'go-component-qfkwvs-on-pull-request-ctmbb-build-container/step-push': 
    INFO: Using mounted CA bundle: /mnt/trusted-ca/ca-bundle.crt
    '/mnt/trusted-ca/ca-bundle.crt' -> '/etc/pki/ca-trust/source/anchors/ca-bundle.crt'
    Pushing to quay.io/redhat-appstudio-qe/group-pmnm-tenant/go-component-qfkwvs:
    Getting image source signatures
    Copying blob sha256:314640f419c581ddcac8f3618af39342a4571d5dc7a4e1f5b64d60f37e630b49
    Copying blob sha256:a653a5ff4eaef6037f4f354414ab251ac7218d2d42fc8326743fc63ecf2cabd5
    Copying blob sha256:aa1d190862fd4844b8981a1eb7eec58b5285f04d59297976d5b8826e14d89796
    Copying blob sha256:d3f6a420cbadfb30033dc481690b39191ce6d2d841ccd54434c352f474ea54c3
    Copying blob sha256:db77b3de17313a3fab2620178c01a4ef8eb60cbe722a6e9390f3cbb1132a7d22
    Error: pushing image "quay.io/redhat-appstudio-qe/group-pmnm-tenant/go-component-qfkwvs:on-pr-65854a1a78f5d0b5fea856fad09943aeadeab7f3" to "docker://quay.io/redhat-appstudio-qe/group-pmnm-tenant/go-component-qfkwvs:go-component-qfkwvs-on-pull-request-ctmbb-build-container": writing blob: initiating layer upload to /v2/redhat-appstudio-qe/group-pmnm-tenant/go-component-qfkwvs/blobs/uploads/ in quay.io: received unexpected HTTP status: 502 Bad Gateway
    Failed to push sbom image to quay.io/redhat-appstudio-qe/group-pmnm-tenant/go-component-qfkwvs:go-component-qfkwvs-on-pull-request-ctmbb-build-container after 5 tries
    
    {
        s: "Pipelinerun 'go-component-qfkwvs-on-pull-request-ctmbb' didn't succeed\nLogs from failed container 'go-component-qfkwvs-on-pull-request-ctmbb-build-container/step-push': \nINFO: Using mounted CA bundle: /mnt/trusted-ca/ca-bundle.crt\n'/mnt/trusted-ca/ca-bundle.crt' -> '/etc/pki/ca-trust/source/anchors/ca-bundle.crt'\nPushing to quay.io/redhat-appstudio-qe/group-pmnm-tenant/go-component-qfkwvs:\nGetting image source signatures\nCopying blob sha256:314640f419c581ddcac8f3618af39342a4571d5dc7a4e1f5b64d60f37e630b49\nCopying blob sha256:a653a5ff4eaef6037f4f354414ab251ac7218d2d42fc8326743fc63ecf2cabd5\nCopying blob sha256:aa1d190862fd4844b8981a1eb7eec58b5285f04d59297976d5b8826e14d89796\nCopying blob sha256:d3f6a420cbadfb30033dc481690b39191ce6d2d841ccd54434c352f474ea54c3\nCopying blob sha256:db77b3de17313a3fab2620178c01a4ef8eb60cbe722a6e9390f3cbb1132a7d22\nError: pushing image \"quay.io/redhat-appstudio-qe/group-pmnm-tenant/go-component-qfkwvs:on-pr-65854a1a78f5d0b5fea856fad09943aeadeab7f3\" to \"docker://quay.io/redhat-appstudio-qe/group-pmnm-tenant/go-component-qfkwvs:go-component-qfkwvs-on-pull-request-ctmbb-build-container\": writing blob: initiating layer upload to /v2/redhat-appstudio-qe/group-pmnm-tenant/go-component-qfkwvs/blobs/uploads/ in quay.io: received unexpected HTTP status: 502 Bad Gateway\nFailed to push sbom image to quay.io/redhat-appstudio-qe/group-pmnm-tenant/go-component-qfkwvs:go-component-qfkwvs-on-pull-request-ctmbb-build-container after 5 tries\n",
    }

Copy link
Member

@psturc psturc left a comment

Choose a reason for hiding this comment

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

/lgtm

unrelated error, merging

Copy link

openshift-ci bot commented Jan 28, 2025

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: psturc

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@psturc psturc merged commit 0e1f1f6 into konflux-ci:main Jan 28, 2025
12 of 14 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants