PR#22: Update .fmf files with CentOS Stream related data - rpms/cups

rpms / cups

#22 Update .fmf files with CentOS Stream related data

Closed a year ago by zdohnal. Opened a year ago by zdohnal.

rpms/ zdohnal/cups fmf_updates into rawhide

Asterisks don't work - use pure fedora/centos-stream

Zdenek Dohnal • a year ago

4e69c81

Update .fmf files with CentOS Stream related data

Zdenek Dohnal • a year ago

2a15400

ci.fmf

file added

		`@@ -0,0 +1,1 @@`
		`+ resultsdb-testcase: separate`

gating.yaml

file modified

+4 -2

		`@@ -4,7 +4,7 @@`
		`decision_context: bodhi_update_push_testing`
		`subject_type: koji_build`
		`rules:`
		`- - !PassingTestCaseRule {test_case_name: fedora-ci.koji-build.tier0.functional}`
		`+ - !PassingTestCaseRule {test_case_name: fedora-ci.koji-build./plans/public.functional}`

		`#Rawhide`
		`--- !Policy`
		`@@ -13,7 +13,7 @@`
		`decision_context: bodhi_update_push_stable`
		`subject_type: koji_build`
		`rules:`
		`- - !PassingTestCaseRule {test_case_name: fedora-ci.koji-build.tier0.functional}`
		`+ - !PassingTestCaseRule {test_case_name: fedora-ci.koji-build./plans/public.functional}`

		`#gating rhel`
		`--- !Policy`
		`@@ -24,3 +24,5 @@`
		`- !PassingTestCaseRule {test_case_name: baseos-ci.brew-build.tier1.functional}`
		`- !PassingTestCaseRule {test_case_name: baseos-ci.brew-build.tedude.validation}`
		`- !PassingTestCaseRule {test_case_name: baseos-ci.brew-build.acceptance-tier.functional}`
		`+ - !PassingTestCaseRule {test_case_name: osci.brew-build./plans/tier1-internal.functional}`
		`+ - !PassingTestCaseRule {test_case_name: osci.brew-build./plans/public.functional}`

plans/public.fmf ~~plans/all.fmf~~

file renamed

+1 -1

		`@@ -1,4 +1,4 @@`
		`- summary: Test plan with cups tests`
		`+ summary: Test plan that runs all tests from cups tests repo.`
		`discover:`
		`how: fmf`
		`url: https://src.fedoraproject.org/tests/cups.git`

plans/tier1-internal.fmf

file added

+12

		`@@ -0,0 +1,12 @@`
		`+ summary: CI plan, picks internal Tier1 tests, runs in beakerlib.`
		`+ discover:`
		`+ - name: rhel`
		`+ how: fmf`
		`+ filter: 'tier: 1'`
		`+ url: git://pkgs.devel.redhat.com/tests/cups`
		`+ execute:`
		`+ how: tmt`
		`+ adjust:`
		`+ enabled: false`
		`+ when: distro == centos-stream or distro == fedora`
		`+ because: They don't have access to internal repos.`

zdohnal commented a year ago

@omejzlik would you mind reviewing the PR whether it is correct?

omejzlik commented a year ago

Looks ok.

omejzlik commented a year ago

Looks like the * wildcard is not working as expected.
Use when: distro == fedora, centos-stream
to disable all centos and fedora versions.

zdohnal commented a year ago

@omejzlik thx! I'll apply it.

1 new commit added

Asterisks don't work - use pure fedora/centos-stream

a year ago

omejzlik commented a year ago

The error above seems to be because testing farm did not find any enabled plan in the internal plan which is intentionally not enabled.

@mvadkert
The idea here is that the plans will eventually make their way to centos and rhel.
But fedora and centos can not see into internal infrastructure so the internal plan is disabled on fedora and centos and will only be used by RH internal systems.
Similar setup has been used for example in centos gitlab repos and it works there.

Can the ci here be set to ignore plans which are not enabled here?

mvadkert commented a year ago

it is a Fedora CI bug ...

mvadkert commented a year ago

https://pagure.io/fedora-ci/general pls

zdohnal commented a year ago

Ok - pvalena explained the problem to me, so now I know more :) . I'll file the issue.

zdohnal commented a year ago

Reported as https://pagure.io/fedora-ci/general/issue/398

msrb commented a year ago

[citest]

msrb commented a year ago

Pagure won't remove the old result, but restart only triggered /plans/public now.

omejzlik commented a year ago

[citest]

omejzlik commented a year ago

The current problem seems to be broken testing farm.

mvadkert commented a year ago

[citest]

mvadkert commented a year ago

[citest]

mvadkert commented a year ago

@omejzlik it luks butter

zdohnal commented a year ago

[citest]

zdohnal commented a year ago

It seems the tier1-internal is not triggered by [citest], which might be a sign it is ignored. I'll check how it will look in a new PR, I'm closing this one for this purpose.

Pull-Request has been closed by zdohnal

a year ago