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

Fix repository creation in push pipeline #1802

Open
wants to merge 3 commits into
base: main
Choose a base branch
from

Conversation

chmeliik
Copy link
Contributor

@chmeliik chmeliik commented Jan 9, 2025

  • The script wasn't adding the "task-" or "pipeline-" prefix to repositories
  • Ignore strange "Repository already exists" errors
  • Run build-bundles after create-repositories-if-missing

The script wasn't adding the "task-" or "pipeline-" prefix to
repositories

Signed-off-by: Adam Cmiel <[email protected]>
Inexplicably, for some task repos, quay.io returns 401 in response to
the initial request but then returns "Repository already exists" when we
try to create the repo.

Ignore these errors.

Example:

    Checking konflux-ci/tekton-catalog/task-fips-operator-bundle-check, http code: 401
    Missing task bundle repo: tekton-catalog/task-fips-operator-bundle-check in konflux-ci, creating...
      % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
                                     Dload  Upload   Total   Spent    Left  Speed
    100   362  100   218  100   144    323    213 --:--:-- --:--:-- --:--:--   537
    WARNING: repository creation failed, but the error message was 'Repository already exists'. Assuming that's fine.

Signed-off-by: Adam Cmiel <[email protected]>
@chmeliik chmeliik requested a review from a team as a code owner January 9, 2025 09:51
The create-repositories-if-missing creates the quay repositories for the
build-bundles task. Run them in the correct order.

Signed-off-by: Adam Cmiel <[email protected]>
@chmeliik
Copy link
Contributor Author

chmeliik commented Jan 9, 2025

I noticed the push pipeline was consistently failing, e.g. https://github.com/konflux-ci/build-definitions/runs/35339588858

Attempting to fix

@chmeliik
Copy link
Contributor Author

chmeliik commented Jan 9, 2025

/retest

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.

1 participant