PR#121: Added missing fields for brew-build.test.complete.yaml - fedora-ci/messages

fedora-ci / messages

#121 Added missing fields for brew-build.test.complete.yaml

Merged 2 years ago by lholecek. Opened 2 years ago by draval.

fedora-ci/ draval/messages master into master

brew_build added missing fields

DivyangRaval • 2 years ago

f26c607

mappings/results/brew-build.test.complete.yaml

file modified

+10

		`@@ -7,3 +7,13 @@`
		`data:`
		`item:`
		`- "${body.artifact.nvr}"`
		`+ system_provider:`
		`+ - "${body.system[0].provider}"`
		`+ system_architecture:`
		`+ - "${body.system[0].architecture}"`
		`+ system_variant:`
		`+ - "${body.system[0].variant}"`
		`+ scenario:`
		`+ - "#if(${body.test.scenario})${body.test.scenario}#{else}not available#end"`
		`+ scratch:`
		`+ - "${body.artifact.scratch}"`

draval commented 2 years ago

Fields Added:
- scenario
- system_architecture
- system_variant
- system_provider
- scratch

draval commented 2 years ago

@lholecek @pholica
Kindly review & merge.

lholecek commented 2 years ago

+1 Thanks! Can you squash the commits into a single one?

pholica commented 2 years ago

Seems to look good. Thank you.

Edited 2 years ago by pholica

rebased onto f26c607

2 years ago

draval commented 2 years ago

@lholecek
Oh my bad, I thought while merging the PR user will get an option to squash the commit.
Anyway, no worries, I have squashed them from my side.
So should be ready to merge.