Skip to content

fix(KFLUXBUGS-1487): Duplicated build PLRs in GitLab e2e test#1399

Merged
psturc merged 2 commits into
konflux-ci:mainfrom
dheerajodha:KFLUXBUGS-1487
Oct 3, 2024
Merged

fix(KFLUXBUGS-1487): Duplicated build PLRs in GitLab e2e test#1399
psturc merged 2 commits into
konflux-ci:mainfrom
dheerajodha:KFLUXBUGS-1487

Conversation

@dheerajodha

@dheerajodha dheerajodha commented Sep 23, 2024

Copy link
Copy Markdown
Member

Description

  • Removes the unused PaC secret in the GitLab test suite.
  • But that is not the root cause, still finding that one...

Issue ticket number and link

KFLUXBUGS-1487

Type of change

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • This change requires a documentation update

How Has This Been Tested?

Please describe the tests that you ran to verify your changes. Provide instructions so we can reproduce. Please also list any relevant details for your test configuration

Checklist:

  • I have performed a self-review of my code
  • I have commented my code, particularly in hard-to-understand areas
  • I have made corresponding changes to the documentation
  • I have added meaningful description with JIRA/GitHub issue key(if applicable), for example HASSuiteDescribe("STONE-123456789 devfile source")
  • I have updated labels (if needed)

@openshift-ci

openshift-ci Bot commented Sep 23, 2024

Copy link
Copy Markdown

Skipping CI for Draft Pull Request.
If you want CI signal for your change, please convert it to an actual PR.
You can still manually trigger a test run with /test all

@dheerajodha

Copy link
Copy Markdown
Member Author

/retest

1 similar comment
@dheerajodha

Copy link
Copy Markdown
Member Author

/retest

@dheerajodha dheerajodha changed the title fix: Duplicated build PLRs in GitLab e2e test fix(KFLUXBUGS-1487): Duplicated build PLRs in GitLab e2e test Oct 1, 2024
* This commit deletes the webhooks, along
  with other things, even after the test
  fails.
* This is needed because maybe the bug is
  occuring due to leftover webhooks within
  the repo.

Signed-off-by: Dheeraj<djodha@redhat.com>
@dheerajodha dheerajodha marked this pull request as ready for review October 1, 2024 11:54
@openshift-ci openshift-ci Bot requested review from dirgim and hongweiliu17 October 1, 2024 11:54
@dheerajodha

Copy link
Copy Markdown
Member Author

/retest

@tisutisu

tisutisu commented Oct 1, 2024

Copy link
Copy Markdown
Contributor

/lgtm

@tisutisu

tisutisu commented Oct 1, 2024

Copy link
Copy Markdown
Contributor

/approve

@openshift-ci openshift-ci Bot added the approved label Oct 1, 2024
@openshift-ci

openshift-ci Bot commented Oct 1, 2024

Copy link
Copy Markdown

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: kasemAlem, tisutisu

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

The pull request process is described here

Details 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

@konflux-ci-qe-bot

Copy link
Copy Markdown

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

PipelineRun Name Status Rerun command Build Log Test Log
konflux-e2e-56r4n-2hvcf 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-56r4n-2hvcf

@psturc

psturc commented Oct 3, 2024

Copy link
Copy Markdown
Member

konflux-ci pipeline is having issues due to failures in provision-rosa task

will try to validate this PR via openshift-ci:
/test redhat-appstudio-e2e

@dheerajodha

Copy link
Copy Markdown
Member Author

Thank you @psturc and the reviewers!
Looks like the openshiftci job is also taking abnormally long to finish. Retrying:
/test redhat-appstudio-e2e

@psturc

psturc commented Oct 3, 2024

Copy link
Copy Markdown
Member

e2e job in openshift-ci has passed

merging

@psturc psturc merged commit ad5c68f into konflux-ci:main Oct 3, 2024
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.

5 participants