search

packit / deployment

Ansible playbooks and scripts for deploying packit-service to OpenShift
MIT License
8 stars 25 forks source link

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

Closed majamassarini closed 2 months ago

majamassarini commented 4 months ago

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

softwarefactory-project-zuul[bot] commented 2 months ago

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

:heavy_check_mark: pre-commit SUCCESS in 1m 40s :x: deployment-tests NODE_FAILURE Node request 200-0007575933 failed in 0s

softwarefactory-project-zuul[bot] commented 2 months ago

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

:heavy_check_mark: pre-commit SUCCESS in 1m 36s :x: deployment-tests NODE_FAILURE Node request 200-0007576015 failed in 0s

softwarefactory-project-zuul[bot] commented 2 months ago

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

:heavy_check_mark: pre-commit SUCCESS in 1m 41s :x: deployment-tests NODE_FAILURE Node request 200-0007576041 failed in 0s

softwarefactory-project-zuul[bot] commented 2 months ago

Build succeeded. https://softwarefactory-project.io/zuul/t/packit-service/buildset/44558e2f3b924aaba16b540f88b6d31c

:heavy_check_mark: pre-commit SUCCESS in 1m 40s

majamassarini commented 2 months ago

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 commented 2 months ago

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

mfocko commented 2 months ago

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 :shrug: flakey > nothing IMO

softwarefactory-project-zuul[bot] commented 2 months ago

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

:heavy_check_mark: pre-commit SUCCESS in 1m 37s