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

chore(stoneintg-1092): refactor group snapshots tests #1504

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

Conversation

jencull
Copy link
Contributor

@jencull jencull commented Feb 5, 2025

Description

Please include a summary of the changes and the related issue. Please also include relevant motivation and context. List any dependencies that are required for this change.

Issue ticket number and link

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)

Copy link

openshift-ci bot commented Feb 5, 2025

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by:
Once this PR has been reviewed and has the lgtm label, please assign psturc for approval. For more information see the Code Review Process.

The full list of commands accepted by this bot can be found 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

@jsztuka
Copy link
Contributor

jsztuka commented Feb 5, 2025

Changes look fine in general, let see if the CI passes.

@jsztuka
Copy link
Contributor

jsztuka commented Feb 5, 2025

I think once you change the commit message from chore(stoneintg-1092) to chore(stoneintg-1092): conventional commit check will pass.

@jencull
Copy link
Contributor Author

jencull commented Feb 5, 2025

I think once you change the commit message from chore(stoneintg-1092) to chore(stoneintg-1092): conventional commit check will pass.

Yep, have changed it but its not showing in the title. Will come back to it.

This is a WIP, I am just testing this before i start merging additional code from the status-reporting file. Will then test those changes and, finally, update the readme. So its marked as a WIP for now as I work my way through the stages.

@jencull jencull changed the title chore(stoneintg-1092) refactor group snapshots tests chore(stoneintg-1092): refactor group snapshots tests Feb 6, 2025
@jencull
Copy link
Contributor Author

jencull commented Feb 6, 2025

/retest

@jencull
Copy link
Contributor Author

jencull commented Feb 12, 2025

/retest

@jencull
Copy link
Contributor Author

jencull commented Feb 25, 2025

/retest

@jencull jencull force-pushed the stoneintg-1092 branch 2 times, most recently from 3dfe1ce to 33dab36 Compare March 4, 2025 16:39
@jencull
Copy link
Contributor Author

jencull commented Mar 11, 2025

/retest

1 similar comment
@jencull
Copy link
Contributor Author

jencull commented Mar 13, 2025

/retest

@konflux-ci-qe-bot
Copy link

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

PipelineRun Name Status Rerun command Build Log Test Log
konflux-e2e-5krrf 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-5krrf

Test results analysis

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

➡️ [failed] [It] [integration-service-suite Integration Service E2E tests] with happy path for general flow of Integration service when An snapshot of push event is created checks if a Release is created successfully [integration-service]

Click to view logs

Expected success, but got an error:
    : 
    context deadline exceeded
    {}

➡️ [failed] [It] [build-service-suite Build service E2E tests] Creating component with container image source should not trigger a PipelineRun [build-service]

Click to view logs

Expected success, but got an error:
    : 
    context deadline exceeded
    {}

➡️ [failed] [It] [build-service-suite Build service E2E tests] test PaC component build github when a new component without specified branch is created and with visibility private a related PipelineRun should be deleted after deleting the component [build-service, github-webhook, pac-build, pipeline, image-controller, pac-custom-default-branch]

Click to view logs

Expected success, but got an error:
    : 
    context deadline exceeded
    {}

➡️ [failed] [It] [build-service-suite Build service E2E tests] A secret with dummy quay.io credentials is created in the testing namespace should not be possible to push to quay.io repo (PipelineRun should fail) [build-service]

Click to view logs

Expected success, but got an error:
    : 
    context deadline exceeded
    {}

➡️ [failed] [It] [integration-service-suite Creation of group snapshots for monorepo and multiple repos] with status reporting of Integration tests in CheckRuns when creating and testing multiple components when the Snapshot testing for python-component-bletzp is completed successfully should merge the init PaC PR successfully [integration-service, group-snapshot-creation]

Click to view logs

Expected success, but got an error:
    : 
    context deadline exceeded
    {}

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.

4 participants