fedora-ci / messages

Clone
Source Code
GIT

#131 Add message ID to brew-build results
Merged 2 years ago by lholecek. Opened 2 years ago by lholecek.
fedora-ci/ lholecek/messages add-msg-id  into  master

Add message ID to brew-build results
Lukas Holecek • 2 years ago  
b5f531e
file modified
+2 
@@ -19,3 +19,5 @@ 
 

          - "#if(${body.test.scenario})${body.test.scenario}#{else}#end"
 

      scratch:
 

          - "${body.artifact.scratch}"
 

+     msg_id:
 

+         - "${headers.message-id}"

file modified
+2 
@@ -19,3 +19,5 @@ 
 

          - "#if(${body.test.scenario})${body.test.scenario}#{else}#end"
 

      scratch:
 

          - "${body.artifact.scratch}"
 

+     msg_id:
 

+         - "${headers.message-id}"

file modified
+2 
@@ -19,3 +19,5 @@ 
 

          - "#if(${body.test.scenario})${body.test.scenario}#{else}#end"
 

      scratch:
 

          - "${body.artifact.scratch}"
 

+     msg_id:
 

+         - "${headers.message-id}"

file modified
+2 
@@ -19,3 +19,5 @@ 
 

          - "#if(${body.test.scenario})${body.test.scenario}#{else}#end"
 

      scratch:
 

          - "${body.artifact.scratch}"
 

+     msg_id:
 

+         - "${headers.message-id}"

JIRA: RHELWF-3910

@lholecek,
I guess header name is message-id.

does that change to message_id?

@lholecek,
I guess header name is message-id.

does that change to message_id?

Should I use headers.message-id instead? I thought that was causing issues because of the dash in name. Or maybe it's headers.messageId?

@lholecek
Yes you can use headers. message-id.
That - issue has been fixed already.

rebased onto b5f531e
2 years ago

Yes you can use headers. message-id.
That - issue has been fixed already.

Done. I changed headers.message_id to headers.message-id in the mappings.

Looks good to me

@astepano Can you also review since you filed the request?

LGTM from me

Pull-Request has been merged by lholecek
2 years ago

Merged and tagged with version 1.1.11.
Metadata
None
No Tags
Changes Summary 4
+2 -0
file changed
mappings/results/brew-build.test.complete.yaml
+2 -0
file changed
mappings/results/brew-build.test.error.yaml
+2 -0
file changed
mappings/results/brew-build.test.queued.yaml
+2 -0
file changed
mappings/results/brew-build.test.running.yaml
Powered by Pagure 5.13.3
Documentation • File an Issue • About • SSH Hostkey/Fingerprint
© Red Hat, Inc. and others.