Skip to content

Test deployment locally with tmt (from packit branch) - #581

Merged
softwarefactory-project-zuul[bot] merged 12 commits into
mainfrom
tf-openshift-tests
Sep 16, 2024
Merged

softwarefactory-project-zuul[bot] merged 12 commits into
mainfrom
tf-openshift-tests

Conversation

@majamassarini

@majamassarini majamassarini commented Jul 1, 2024 •

Copy link
Copy Markdown
Member

To be able to access secrets and test ansible scripts using github secrets in the testing farm github action I need to create a PR from the packit repo itself (and not from a fork).

This is the reason why the PR is duplicated at the moment.

Should fix #562 #563 #564 #565

@majamassarini

Copy link
Copy Markdown
Member Author

[test]

@softwarefactory-project-zuul

Copy link
Copy Markdown
Contributor

@softwarefactory-project-zuul

Copy link
Copy Markdown
Contributor

@softwarefactory-project-zuul

Copy link
Copy Markdown
Contributor

@softwarefactory-project-zuul

Copy link
Copy Markdown
Contributor

@softwarefactory-project-zuul

Copy link
Copy Markdown
Contributor

@softwarefactory-project-zuul

Copy link
Copy Markdown
Contributor

@softwarefactory-project-zuul

Copy link
Copy Markdown
Contributor

@majamassarini
majamassarini force-pushed the tf-openshift-tests branch 5 times, most recently from b232a4c to 385b341 Compare July 2, 2024 12:45
@softwarefactory-project-zuul

Copy link
Copy Markdown
Contributor

@majamassarini
majamassarini force-pushed the tf-openshift-tests branch 6 times, most recently from b0b87a1 to b6d1528 Compare July 3, 2024 07:32
@softwarefactory-project-zuul

Copy link
Copy Markdown
Contributor

@softwarefactory-project-zuul

Copy link
Copy Markdown
Contributor

@majamassarini

Copy link
Copy Markdown
Member Author

check

@softwarefactory-project-zuul

Copy link
Copy Markdown
Contributor

Build failed.
https://softwarefactory-project.io/zuul/t/packit-service/buildset/508c2a5e73694556bdb8ca10d0489a9d

✔️ pre-commit SUCCESS in 1m 41s
❌ deployment-tests NODE_FAILURE Node request 200-0007574685 failed in 0s

@softwarefactory-project-zuul

Copy link
Copy Markdown
Contributor

Build failed.
https://softwarefactory-project.io/zuul/t/packit-service/buildset/1a101a09e09e4b9fb80bd61c6d1c8919

✔️ pre-commit SUCCESS in 1m 44s
❌ deployment-tests NODE_FAILURE Node request 200-0007574755 failed in 0s

@softwarefactory-project-zuul

Copy link
Copy Markdown
Contributor

Build failed.
https://softwarefactory-project.io/zuul/t/packit-service/buildset/78eeaea734574953a69c04c4297f2d48

✔️ pre-commit SUCCESS in 1m 38s
❌ deployment-tests NODE_FAILURE Node request 200-0007574829 failed in 0s

@softwarefactory-project-zuul

Copy link
Copy Markdown
Contributor

Build failed.
https://softwarefactory-project.io/zuul/t/packit-service/buildset/c1a1dd07b56145e68ccb8472aab8450a

✔️ pre-commit SUCCESS in 1m 42s
❌ deployment-tests NODE_FAILURE Node request 200-0007574967 failed in 0s

@softwarefactory-project-zuul

Copy link
Copy Markdown
Contributor

Build failed.
https://softwarefactory-project.io/zuul/t/packit-service/buildset/53865cbbd5bd45268bb399e78e107697

✔️ pre-commit SUCCESS in 1m 43s
❌ deployment-tests NODE_FAILURE Node request 200-0007575226 failed in 0s

majamassarini and others added 2 commits September 13, 2024 09:45
Co-authored-by: Laura Barcziová <49026743+lbarcziova@users.noreply.github.com>
Co-authored-by: Matej Focko <mfocko@users.noreply.github.com>
Tokman is no longer used.
The github-app-private-key is now used directly by the service and the workers.
@softwarefactory-project-zuul

Copy link
Copy Markdown
Contributor

Build failed.
https://softwarefactory-project.io/zuul/t/packit-service/buildset/ccdab2c531e04bba803738e6304bc196

✔️ pre-commit SUCCESS in 1m 40s
❌ deployment-tests NODE_FAILURE Node request 200-0007575933 failed in 0s

@softwarefactory-project-zuul

Copy link
Copy Markdown
Contributor

Build failed.
https://softwarefactory-project.io/zuul/t/packit-service/buildset/d3bdaaf9bb3241d684927ad086497cb4

✔️ pre-commit SUCCESS in 1m 36s
❌ deployment-tests NODE_FAILURE Node request 200-0007576015 failed in 0s

Co-authored-by: Matej Focko <mfocko@users.noreply.github.com>
@softwarefactory-project-zuul

Copy link
Copy Markdown
Contributor

Build failed.
https://softwarefactory-project.io/zuul/t/packit-service/buildset/4b168a7c3379431881a017e2957e5b28

✔️ pre-commit SUCCESS in 1m 41s
❌ deployment-tests NODE_FAILURE Node request 200-0007576041 failed in 0s

@lbarcziova lbarcziova left a comment •

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Schedule test on Testing Farm / tests (pull_request) Successful in 36m 🥳 🥳 🥳

great job Maja!

as for the Zuul deployment-tests, can we remove them right now or in a follow-up PR? We should then also configure the new check to be required to pass for merging (in the repo settings).

Now we are using testing farm for it.
@softwarefactory-project-zuul

Copy link
Copy Markdown
Contributor

@majamassarini

Copy link
Copy Markdown
Member Author

as for the Zuul deployment-tests, can we remove them right now or in a follow-up PR? We should then also configure the new check to be required to pass for merging (in the repo settings).

I have disabled deployment test from zuul.
Regarding the required check I thought @lachmanfrantisek prefers not to use the github testing farm integration, but for me it is ok if we want to enable it.

@lbarcziova

Copy link
Copy Markdown
Member

@majamassarini I meant just changing the branch protection rules to require tests check to be passing

@mfocko mfocko left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

🚀

Comment thread .github/workflows/tf-tests.yml
@mfocko

mfocko commented Sep 16, 2024

Copy link
Copy Markdown
Member

as for the Zuul deployment-tests, can we remove them right now or in a follow-up PR? We should then also configure the new check to be required to pass for merging (in the repo settings).

I have disabled deployment test from zuul. Regarding the required check I thought @lachmanfrantisek prefers not to use the github testing farm integration, but for me it is ok if we want to enable it.

Yes, it is preferred. Though having no (red) CI right now is IMO bit worse, and you have the GH Action ready 🤷 flakey > nothing IMO

@majamassarini majamassarini added the mergeit Merge via Zuul label Sep 16, 2024
@softwarefactory-project-zuul

Copy link
Copy Markdown
Contributor

Build succeeded (gate pipeline).
https://softwarefactory-project.io/zuul/t/packit-service/buildset/0e345ffd9df146e5a9cc2b3b99680798

✔️ pre-commit SUCCESS in 1m 37s

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

mergeit Merge via Zuul

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants