PR#88: Clarify how test namespace with a dot must match topic - fedora-ci/messages

fedora-ci / messages

#88 Clarify how test namespace with a dot must match topic

Merged 4 years ago by mvadkert. Opened 4 years ago by lholecek.

fedora-ci/ lholecek/messages clarify-namespace into master

Clarify how test namespace with a dot must match topic

Lukas Holecek • 4 years ago

84f7069

schemas/test-common.yaml

file modified

		`@@ -58,6 +58,12 @@`
		`for testcase names belonging to another namespace. The impact of this depends`
		`on how test results are used, typically for gating or automatic promotion of`
		`content.`
		`+`
		`+ If it contains a dot, only the text before the first dot must match`
		`+ the namespace in the topic hierarchy.`
		`+`
		`+ For example, if namespace is "a-team.some.test.prefix", the`
		`+ namespace in topic must be "a-team".`
		`examples:`
		`- osci`
		`- rtt`

lholecek commented 4 years ago

Signed-off-by: Lukas Holecek hluk@email.cz

lholecek commented 4 years ago

@mvadkert, @ralph This came up on UMB consortium meeting, so I feel the need to clarify this here.

BTW, for simplicity, all three components for the resulting test case name <namespace>.<type>.<category>, could be in just a single field in the message (instead of making single field with just the two last ones as discussed on the meeting).

ralph commented 4 years ago

Works for me. @mvadkert?

mvadkert commented 4 years ago

BTW, for simplicity, all three components for the resulting test case name <namespace>.<type>.<category>, could be in just a single field in the message (instead of making single field with just the two last ones as discussed on the meeting).

Yeah, my thoughts were to leave the concatenation of test namespace and (a new field) test.name. Will propose a patch for that. We can discuss there if it would make more sense just to let the concatenation be and have just a test.name, which could even make test.namespace redundant.

Otherwise this clarifies it for me :)