PR#98: Add productmd-compose.build.* schemas - fedora-ci/messages

fedora-ci / messages

#98 Add productmd-compose.build.* schemas

Merged 4 years ago by ralph. Opened 4 years ago by ralph.

productmd-compose-build into master

Drop build reference from productmd-compose

Ralph Bean • 4 years ago

988de3b

Add productmd-compose.build.* schemas

Ralph Bean • 4 years ago

9be8bdd

schemas/productmd-compose.build.complete.yaml

file added

+27

		`@@ -0,0 +1,27 @@`
		`+ $id: productmd-compose.build.complete`
		`+ $schema: http://json-schema.org/draft-07/schema#`
		`+`
		`+ description:`
		`+ Building of the compose has been completed.`
		`+ This is a mandatory message.`
		`+`
		`+ properties:`
		`+ contact:`
		`+ $ref: contact.json`
		`+ run:`
		`+ $ref: run.json`
		`+ artifact:`
		`+ $ref: productmd-compose.json`
		`+ generated_at:`
		`+ $ref: common.json#properties/generated_at`
		`+ version:`
		`+ $ref: common.json#properties/version`
		`+`
		`+ required:`
		`+ - contact`
		`+ - run`
		`+ - artifact`
		`+ - generated_at`
		`+ - version`
		`+`
		`+ type: object`

schemas/productmd-compose.build.error.yaml

file added

+30

		`@@ -0,0 +1,30 @@`
		`+ $id: productmd-compose.build.error`
		`+ $schema: http://json-schema.org/draft-07/schema#`
		`+`
		`+ description:`
		`+ Building of the compose has failed.`
		`+ This is a mandatory message.`
		`+`
		`+ properties:`
		`+ contact:`
		`+ $ref: contact.json`
		`+ run:`
		`+ $ref: run.json`
		`+ artifact:`
		`+ $ref: productmd-compose.json`
		`+ error:`
		`+ $ref: error.json`
		`+ generated_at:`
		`+ $ref: common.json#properties/generated_at`
		`+ version:`
		`+ $ref: common.json#properties/version`
		`+`
		`+ required:`
		`+ - contact`
		`+ - run`
		`+ - artifact`
		`+ - error`
		`+ - generated_at`
		`+ - version`
		`+`
		`+ type: object`

no initial comment

lsedlar commented 4 years ago

Generally looks good to me.

A couple questions:
* in the build property, the only required key is repository. What does that mean for a compose?
* there are issues with the artifact payload as well: #97