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

RHTAP-3027 Jenkins/Github advanced test with promotion pipeline #162

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

Conversation

jkopriva
Copy link
Collaborator

No description provided.

@jkopriva jkopriva marked this pull request as ready for review September 24, 2024 08:10
@jkopriva
Copy link
Collaborator Author

jkopriva commented Oct 9, 2024

/retest

2 similar comments
@jkopriva
Copy link
Collaborator Author

/retest

@jkopriva
Copy link
Collaborator Author

/retest

Copy link

sonarcloud bot commented Nov 15, 2024

Quality Gate Failed Quality Gate failed

Failed conditions
2 Security Hotspots
34.5% Duplication on New Code (required ≤ 15%)

See analysis details on SonarQube Cloud

@konflux-ci-qe-bot
Copy link
Collaborator

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

PipelineRun Name Status Rerun command Build Log Test Log
rhtap-min-install-e2e-szs2q 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/rhtap-team/rhtap-cli:rhtap-min-install-e2e-szs2q

Comment on lines +205 to +208
/**
* Trigger a promotion Pull Request in Gitops repository to promote stage image to prod environment
*/
it('trigger pull request promotion to promote from stage to prod environment', async () => {
Copy link
Collaborator

Choose a reason for hiding this comment

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

This is afaik promotion from dev to stage.

/**
* Trigger and wait for Jenkins job to finish(it will also run deplyment pipeline)
*/
it(`Trigger job and wait for ${gptTemplate} jenkins job to finish`, async () => {
Copy link
Collaborator

Choose a reason for hiding this comment

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

There are duplicated "It" node names which makes the second occurence of this test case to be skipped. Pls make sure you are using unique names for It nodes.

/**
* Trigger a promotion Pull Request in Gitops repository to promote stage image to prod environment
*/
it('trigger pull request promotion to promote from stage to prod environment', async () => {
Copy link
Collaborator

Choose a reason for hiding this comment

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

This is first duplicate.

/**
* Merge the gitops Pull Request with the new image value. Expect that argocd will sync the new image in stage
*/
it(`merge gitops pull request to sync new image in stage environment`, async () => {
Copy link
Collaborator

Choose a reason for hiding this comment

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

Duplicate and incorrect name - it should sync the prod env, no?

/**
* Trigger and wait for Jenkins job to finish(it will also run deplyment pipeline)
*/
it(`Trigger job and wait for ${gptTemplate} jenkins job to finish`, async () => {
Copy link
Collaborator

Choose a reason for hiding this comment

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

duplicate

/**
* Obtain the openshift Route for the component and verify that the previous builded image was synced in the cluster and deployed in prod environment
*/
it('container component is successfully synced by gitops in prod environment', async () => {
Copy link
Collaborator

Choose a reason for hiding this comment

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

duplicate

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.

3 participants