)]}'
{"id":"opendev%2Fsubunit2sql~552152","triplet_id":"opendev%2Fsubunit2sql~master~I31468c825cf259b8df62134e578f31d96af6ac75","project":"opendev/subunit2sql","branch":"master","topic":"attachments","hashtags":[],"change_id":"I31468c825cf259b8df62134e578f31d96af6ac75","subject":"Add attachments flag to get_test_runs_by_status","status":"MERGED","created":"2018-03-12 21:54:32.000000000","updated":"2019-07-18 05:52:22.000000000","submitted":"2019-07-18 05:52:22.000000000","submitter":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"total_comment_count":18,"unresolved_comment_count":0,"has_review_started":true,"submission_id":"552152-1563429142638-cc061412","meta_rev_id":"903ebaba7fb64da25d5817c4b323329530e09aa6","_number":552152,"virtual_id_number":552152,"owner":{"_account_id":20378,"name":"Trevor McCasland","email":"TM2086@att.com","username":"twm2016"},"actions":{},"labels":{"Verified":{"approved":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"all":[{"value":0,"_account_id":1921,"name":"Andrea Frittoli","email":"andrea.frittoli@gmail.com","username":"andrea-frittoli"},{"value":0,"_account_id":1,"name":"James E. Blair","email":"jim@acmegating.com","username":"corvus"},{"value":0,"_account_id":4162,"name":"Paul Belanger","email":"pabelanger@redhat.com","username":"pabelanger"},{"value":0,"_account_id":2,"name":"Monty Taylor","email":"mordred@inaugust.com","username":"mordred"},{"value":0,"_account_id":13252,"name":"Dr. Jens Harbott","display_name":"Jens Harbott (frickler)","email":"frickler@offenerstapel.de","username":"jrosenboom"},{"value":0,"_account_id":9061,"name":"David Moreau Simard","email":"moi@dmsimard.com","username":"dmsimard"},{"value":2,"date":"2019-07-18 05:52:22.000000000","post_submit":true,"permitted_voting_range":{"min":2,"max":2},"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},{"value":0,"_account_id":5196,"name":"Matthew Treinish","email":"mtreinish@kortar.org","username":"treinish"},{"value":0,"_account_id":7118,"name":"Ian Wienand","email":"iwienand@redhat.com","username":"iwienand"},{"value":0,"_account_id":5263,"name":"Jeremy Stanley","display_name":"fungi","email":"fungi@yuggoth.org","username":"fungi","status":"missing, presumed fed"},{"value":0,"_account_id":4146,"name":"Clark Boylan","email":"cboylan@sapwetik.org","username":"cboylan"},{"value":0,"_account_id":1106,"name":"Ricardo Carrillo Cruz","email":"ricardo.carrillo.cruz@gmail.com","username":"rcc"},{"value":0,"_account_id":6133,"name":"yolanda.robla","email":"yroblamo@redhat.com","username":"yolanda.robla"},{"value":0,"_account_id":6167,"name":"Ken\u0027ichi Ohmichi","email":"ken1ohmichi@gmail.com","username":"oomichi"},{"value":0,"_account_id":6488,"name":"Clint Byrum","email":"clint@fewbar.com","username":"clint-fewbar"},{"value":0,"_account_id":5689,"name":"Masayuki Igawa","email":"masayuki@igawa.io","username":"igawa"},{"value":0,"_account_id":20378,"name":"Trevor McCasland","email":"TM2086@att.com","username":"twm2016"},{"value":0,"_account_id":5690,"name":"Masayuki Igawa","inactive":true},{"value":0,"_account_id":3099,"name":"David Shrewsbury","email":"dshrewsb@redhat.com","username":"dshrews"},{"value":0,"_account_id":9725,"name":"Dong Ma","email":"winterma.dong@gmail.com","username":"larainema"},{"value":0,"_account_id":7069,"name":"Joshua Hesketh","email":"josh@nitrotech.org","username":"jhesketh"},{"value":0,"_account_id":17887,"name":"Doug Schveninger","email":"ds6901@att.com","username":"Doug.Schveninger"}],"values":{"-2":"Fails","-1":"Doesn\u0027t seem to work"," 0":"No score","+1":"Works for me","+2":"Verified"},"description":"","default_value":0,"optional":true},"Code-Review":{"approved":{"_account_id":5196,"name":"Matthew Treinish","email":"mtreinish@kortar.org","username":"treinish"},"all":[{"value":0,"_account_id":1921,"name":"Andrea Frittoli","email":"andrea.frittoli@gmail.com","username":"andrea-frittoli"},{"value":0,"_account_id":1,"name":"James E. Blair","email":"jim@acmegating.com","username":"corvus"},{"value":0,"_account_id":4162,"name":"Paul Belanger","email":"pabelanger@redhat.com","username":"pabelanger"},{"value":0,"_account_id":2,"name":"Monty Taylor","email":"mordred@inaugust.com","username":"mordred"},{"value":0,"_account_id":13252,"name":"Dr. Jens Harbott","display_name":"Jens Harbott (frickler)","email":"frickler@offenerstapel.de","username":"jrosenboom"},{"value":0,"_account_id":9061,"name":"David Moreau Simard","email":"moi@dmsimard.com","username":"dmsimard"},{"value":0,"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},{"value":2,"date":"2019-01-09 16:27:23.000000000","permitted_voting_range":{"min":2,"max":2},"_account_id":5196,"name":"Matthew Treinish","email":"mtreinish@kortar.org","username":"treinish"},{"value":0,"_account_id":7118,"name":"Ian Wienand","email":"iwienand@redhat.com","username":"iwienand"},{"value":2,"date":"2019-07-17 22:31:47.000000000","permitted_voting_range":{"min":2,"max":2},"_account_id":5263,"name":"Jeremy Stanley","display_name":"fungi","email":"fungi@yuggoth.org","username":"fungi","status":"missing, presumed fed"},{"value":0,"_account_id":4146,"name":"Clark Boylan","email":"cboylan@sapwetik.org","username":"cboylan"},{"value":0,"_account_id":1106,"name":"Ricardo Carrillo Cruz","email":"ricardo.carrillo.cruz@gmail.com","username":"rcc"},{"value":0,"_account_id":6133,"name":"yolanda.robla","email":"yroblamo@redhat.com","username":"yolanda.robla"},{"value":0,"_account_id":6167,"name":"Ken\u0027ichi Ohmichi","email":"ken1ohmichi@gmail.com","username":"oomichi"},{"value":0,"_account_id":6488,"name":"Clint Byrum","email":"clint@fewbar.com","username":"clint-fewbar"},{"value":0,"_account_id":5689,"name":"Masayuki Igawa","email":"masayuki@igawa.io","username":"igawa"},{"value":0,"_account_id":20378,"name":"Trevor McCasland","email":"TM2086@att.com","username":"twm2016"},{"value":0,"_account_id":5690,"name":"Masayuki Igawa","inactive":true},{"value":0,"_account_id":3099,"name":"David Shrewsbury","email":"dshrewsb@redhat.com","username":"dshrews"},{"value":0,"_account_id":9725,"name":"Dong Ma","email":"winterma.dong@gmail.com","username":"larainema"},{"value":0,"_account_id":7069,"name":"Joshua Hesketh","email":"josh@nitrotech.org","username":"jhesketh"},{"value":0,"_account_id":17887,"name":"Doug Schveninger","email":"ds6901@att.com","username":"Doug.Schveninger"}],"values":{"-2":"Do not merge","-1":"This patch needs further work before it can be merged"," 0":"No score","+1":"Looks good to me, but someone else must approve","+2":"Looks good to me (core reviewer)"},"description":"","default_value":0,"optional":true},"Workflow":{"approved":{"_account_id":5196,"name":"Matthew Treinish","email":"mtreinish@kortar.org","username":"treinish"},"all":[{"value":0,"date":"2019-01-08 15:04:42.000000000","permitted_voting_range":{"min":0,"max":1},"_account_id":1921,"name":"Andrea Frittoli","email":"andrea.frittoli@gmail.com","username":"andrea-frittoli"},{"value":0,"date":"2019-01-28 15:25:02.000000000","permitted_voting_range":{"min":0,"max":1},"_account_id":1,"name":"James E. Blair","email":"jim@acmegating.com","username":"corvus"},{"value":0,"date":"2019-01-28 15:25:02.000000000","_account_id":4162,"name":"Paul Belanger","email":"pabelanger@redhat.com","username":"pabelanger"},{"value":0,"date":"2019-01-28 15:25:02.000000000","_account_id":2,"name":"Monty Taylor","email":"mordred@inaugust.com","username":"mordred"},{"value":0,"date":"2019-01-28 15:25:02.000000000","permitted_voting_range":{"min":0,"max":1},"_account_id":13252,"name":"Dr. Jens Harbott","display_name":"Jens Harbott (frickler)","email":"frickler@offenerstapel.de","username":"jrosenboom"},{"value":0,"date":"2019-01-28 15:25:02.000000000","_account_id":9061,"name":"David Moreau Simard","email":"moi@dmsimard.com","username":"dmsimard"},{"value":0,"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},{"value":1,"date":"2019-07-17 22:27:46.000000000","permitted_voting_range":{"min":1,"max":1},"_account_id":5196,"name":"Matthew Treinish","email":"mtreinish@kortar.org","username":"treinish"},{"value":0,"date":"2019-01-28 15:25:02.000000000","permitted_voting_range":{"min":0,"max":1},"_account_id":7118,"name":"Ian Wienand","email":"iwienand@redhat.com","username":"iwienand"},{"value":0,"_account_id":5263,"name":"Jeremy Stanley","display_name":"fungi","email":"fungi@yuggoth.org","username":"fungi","status":"missing, presumed fed"},{"value":0,"date":"2019-01-28 15:25:02.000000000","permitted_voting_range":{"min":0,"max":1},"_account_id":4146,"name":"Clark Boylan","email":"cboylan@sapwetik.org","username":"cboylan"},{"value":0,"date":"2019-01-28 15:25:02.000000000","_account_id":1106,"name":"Ricardo Carrillo Cruz","email":"ricardo.carrillo.cruz@gmail.com","username":"rcc"},{"value":0,"date":"2019-01-28 15:25:02.000000000","_account_id":6133,"name":"yolanda.robla","email":"yroblamo@redhat.com","username":"yolanda.robla"},{"value":0,"date":"2019-01-28 15:25:02.000000000","permitted_voting_range":{"min":0,"max":1},"_account_id":6167,"name":"Ken\u0027ichi Ohmichi","email":"ken1ohmichi@gmail.com","username":"oomichi"},{"value":0,"date":"2019-01-28 15:25:02.000000000","permitted_voting_range":{"min":0,"max":1},"_account_id":6488,"name":"Clint Byrum","email":"clint@fewbar.com","username":"clint-fewbar"},{"value":0,"date":"2019-01-28 15:25:02.000000000","permitted_voting_range":{"min":0,"max":1},"_account_id":5689,"name":"Masayuki Igawa","email":"masayuki@igawa.io","username":"igawa"},{"value":0,"_account_id":20378,"name":"Trevor McCasland","email":"TM2086@att.com","username":"twm2016"},{"value":0,"date":"2019-01-04 18:41:57.000000000","_account_id":5690,"name":"Masayuki Igawa","inactive":true},{"value":0,"date":"2019-01-28 15:25:02.000000000","_account_id":3099,"name":"David Shrewsbury","email":"dshrewsb@redhat.com","username":"dshrews"},{"value":0,"date":"2019-01-08 15:04:39.000000000","permitted_voting_range":{"min":0,"max":1},"_account_id":9725,"name":"Dong Ma","email":"winterma.dong@gmail.com","username":"larainema"},{"value":0,"date":"2019-01-10 15:40:09.000000000","_account_id":7069,"name":"Joshua Hesketh","email":"josh@nitrotech.org","username":"jhesketh"},{"value":0,"_account_id":17887,"name":"Doug Schveninger","email":"ds6901@att.com","username":"Doug.Schveninger"}],"values":{"-1":"Work in progress"," 0":"Ready for reviews","+1":"Approved"},"description":"","default_value":0,"optional":true}},"removable_reviewers":[],"reviewers":{"REVIEWER":[{"_account_id":1,"name":"James E. Blair","email":"jim@acmegating.com","username":"corvus"},{"_account_id":2,"name":"Monty Taylor","email":"mordred@inaugust.com","username":"mordred"},{"_account_id":1106,"name":"Ricardo Carrillo Cruz","email":"ricardo.carrillo.cruz@gmail.com","username":"rcc"},{"_account_id":1921,"name":"Andrea Frittoli","email":"andrea.frittoli@gmail.com","username":"andrea-frittoli"},{"_account_id":3099,"name":"David Shrewsbury","email":"dshrewsb@redhat.com","username":"dshrews"},{"_account_id":4146,"name":"Clark Boylan","email":"cboylan@sapwetik.org","username":"cboylan"},{"_account_id":4162,"name":"Paul Belanger","email":"pabelanger@redhat.com","username":"pabelanger"},{"_account_id":5196,"name":"Matthew Treinish","email":"mtreinish@kortar.org","username":"treinish"},{"_account_id":5263,"name":"Jeremy Stanley","display_name":"fungi","email":"fungi@yuggoth.org","username":"fungi","status":"missing, presumed fed"},{"_account_id":5689,"name":"Masayuki Igawa","email":"masayuki@igawa.io","username":"igawa"},{"_account_id":5690,"name":"Masayuki Igawa","inactive":true},{"_account_id":6133,"name":"yolanda.robla","email":"yroblamo@redhat.com","username":"yolanda.robla"},{"_account_id":6167,"name":"Ken\u0027ichi Ohmichi","email":"ken1ohmichi@gmail.com","username":"oomichi"},{"_account_id":6488,"name":"Clint Byrum","email":"clint@fewbar.com","username":"clint-fewbar"},{"_account_id":7069,"name":"Joshua Hesketh","email":"josh@nitrotech.org","username":"jhesketh"},{"_account_id":7118,"name":"Ian Wienand","email":"iwienand@redhat.com","username":"iwienand"},{"_account_id":9061,"name":"David Moreau Simard","email":"moi@dmsimard.com","username":"dmsimard"},{"_account_id":9725,"name":"Dong Ma","email":"winterma.dong@gmail.com","username":"larainema"},{"_account_id":13252,"name":"Dr. Jens Harbott","display_name":"Jens Harbott (frickler)","email":"frickler@offenerstapel.de","username":"jrosenboom"},{"_account_id":17887,"name":"Doug Schveninger","email":"ds6901@att.com","username":"Doug.Schveninger"},{"_account_id":20378,"name":"Trevor McCasland","email":"TM2086@att.com","username":"twm2016"},{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]}]},"pending_reviewers":{},"reviewer_updates":[{"updated":"2018-07-29 21:54:32.000000000","updated_by":{"_account_id":17887,"name":"Doug Schveninger","email":"ds6901@att.com","username":"Doug.Schveninger"},"reviewer":{"_account_id":17887,"name":"Doug Schveninger","email":"ds6901@att.com","username":"Doug.Schveninger"},"state":"REVIEWER"},{"updated":"2019-01-04 18:41:57.000000000","updated_by":{"_account_id":5690,"name":"Masayuki Igawa","inactive":true},"reviewer":{"_account_id":5690,"name":"Masayuki Igawa","inactive":true},"state":"REVIEWER"},{"updated":"2019-01-08 15:04:39.000000000","updated_by":{"_account_id":9725,"name":"Dong Ma","email":"winterma.dong@gmail.com","username":"larainema"},"reviewer":{"_account_id":9725,"name":"Dong Ma","email":"winterma.dong@gmail.com","username":"larainema"},"state":"REVIEWER"},{"updated":"2019-01-08 15:04:42.000000000","updated_by":{"_account_id":1921,"name":"Andrea Frittoli","email":"andrea.frittoli@gmail.com","username":"andrea-frittoli"},"reviewer":{"_account_id":1921,"name":"Andrea Frittoli","email":"andrea.frittoli@gmail.com","username":"andrea-frittoli"},"state":"REVIEWER"},{"updated":"2019-01-10 15:40:09.000000000","updated_by":{"_account_id":7069,"name":"Joshua Hesketh","email":"josh@nitrotech.org","username":"jhesketh"},"reviewer":{"_account_id":7069,"name":"Joshua Hesketh","email":"josh@nitrotech.org","username":"jhesketh"},"state":"REVIEWER"},{"updated":"2019-01-28 15:25:02.000000000","updated_by":{"_account_id":1,"name":"James E. Blair","email":"jim@acmegating.com","username":"corvus"},"reviewer":{"_account_id":1,"name":"James E. Blair","email":"jim@acmegating.com","username":"corvus"},"state":"REVIEWER"},{"updated":"2019-01-28 15:25:02.000000000","updated_by":{"_account_id":2,"name":"Monty Taylor","email":"mordred@inaugust.com","username":"mordred"},"reviewer":{"_account_id":2,"name":"Monty Taylor","email":"mordred@inaugust.com","username":"mordred"},"state":"REVIEWER"},{"updated":"2019-01-28 15:25:02.000000000","updated_by":{"_account_id":1106,"name":"Ricardo Carrillo Cruz","email":"ricardo.carrillo.cruz@gmail.com","username":"rcc"},"reviewer":{"_account_id":1106,"name":"Ricardo Carrillo Cruz","email":"ricardo.carrillo.cruz@gmail.com","username":"rcc"},"state":"REVIEWER"},{"updated":"2019-01-28 15:25:02.000000000","updated_by":{"_account_id":3099,"name":"David Shrewsbury","email":"dshrewsb@redhat.com","username":"dshrews"},"reviewer":{"_account_id":3099,"name":"David Shrewsbury","email":"dshrewsb@redhat.com","username":"dshrews"},"state":"REVIEWER"},{"updated":"2019-01-28 15:25:02.000000000","updated_by":{"_account_id":4146,"name":"Clark Boylan","email":"cboylan@sapwetik.org","username":"cboylan"},"reviewer":{"_account_id":4146,"name":"Clark Boylan","email":"cboylan@sapwetik.org","username":"cboylan"},"state":"REVIEWER"},{"updated":"2019-01-28 15:25:02.000000000","updated_by":{"_account_id":4162,"name":"Paul Belanger","email":"pabelanger@redhat.com","username":"pabelanger"},"reviewer":{"_account_id":4162,"name":"Paul Belanger","email":"pabelanger@redhat.com","username":"pabelanger"},"state":"REVIEWER"},{"updated":"2019-01-28 15:25:02.000000000","updated_by":{"_account_id":5689,"name":"Masayuki Igawa","email":"masayuki@igawa.io","username":"igawa"},"reviewer":{"_account_id":5689,"name":"Masayuki Igawa","email":"masayuki@igawa.io","username":"igawa"},"state":"REVIEWER"},{"updated":"2019-01-28 15:25:02.000000000","updated_by":{"_account_id":6133,"name":"yolanda.robla","email":"yroblamo@redhat.com","username":"yolanda.robla"},"reviewer":{"_account_id":6133,"name":"yolanda.robla","email":"yroblamo@redhat.com","username":"yolanda.robla"},"state":"REVIEWER"},{"updated":"2019-01-28 15:25:02.000000000","updated_by":{"_account_id":6167,"name":"Ken\u0027ichi Ohmichi","email":"ken1ohmichi@gmail.com","username":"oomichi"},"reviewer":{"_account_id":6167,"name":"Ken\u0027ichi Ohmichi","email":"ken1ohmichi@gmail.com","username":"oomichi"},"state":"REVIEWER"},{"updated":"2019-01-28 15:25:02.000000000","updated_by":{"_account_id":6488,"name":"Clint Byrum","email":"clint@fewbar.com","username":"clint-fewbar"},"reviewer":{"_account_id":6488,"name":"Clint Byrum","email":"clint@fewbar.com","username":"clint-fewbar"},"state":"REVIEWER"},{"updated":"2019-01-28 15:25:02.000000000","updated_by":{"_account_id":7118,"name":"Ian Wienand","email":"iwienand@redhat.com","username":"iwienand"},"reviewer":{"_account_id":7118,"name":"Ian Wienand","email":"iwienand@redhat.com","username":"iwienand"},"state":"REVIEWER"},{"updated":"2019-01-28 15:25:02.000000000","updated_by":{"_account_id":9061,"name":"David Moreau Simard","email":"moi@dmsimard.com","username":"dmsimard"},"reviewer":{"_account_id":9061,"name":"David Moreau Simard","email":"moi@dmsimard.com","username":"dmsimard"},"state":"REVIEWER"},{"updated":"2019-01-28 15:25:02.000000000","updated_by":{"_account_id":13252,"name":"Dr. Jens Harbott","display_name":"Jens Harbott (frickler)","email":"frickler@offenerstapel.de","username":"jrosenboom"},"reviewer":{"_account_id":13252,"name":"Dr. Jens Harbott","display_name":"Jens Harbott (frickler)","email":"frickler@offenerstapel.de","username":"jrosenboom"},"state":"REVIEWER"},{"updated":"2019-07-17 22:27:46.000000000","updated_by":{"_account_id":5196,"name":"Matthew Treinish","email":"mtreinish@kortar.org","username":"treinish"},"reviewer":{"_account_id":5196,"name":"Matthew Treinish","email":"mtreinish@kortar.org","username":"treinish"},"state":"REVIEWER"},{"updated":"2019-07-17 22:31:47.000000000","updated_by":{"_account_id":5263,"name":"Jeremy Stanley","display_name":"fungi","email":"fungi@yuggoth.org","username":"fungi","status":"missing, presumed fed"},"reviewer":{"_account_id":5263,"name":"Jeremy Stanley","display_name":"fungi","email":"fungi@yuggoth.org","username":"fungi","status":"missing, presumed fed"},"state":"REVIEWER"},{"updated":"2019-07-18 05:52:22.000000000","updated_by":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"reviewer":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"state":"REVIEWER"}],"messages":[{"id":"bb010759446083f6f51e59a239b99221a15e382d","author":{"_account_id":20378,"name":"Trevor McCasland","email":"TM2086@att.com","username":"twm2016"},"date":"2018-03-12 21:54:32.000000000","message":"Uploaded patch set 1.","accounts_in_message":[],"_revision_number":1},{"id":"f4141313d217aec4dbcddff6144acdb9987b609a","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2018-03-12 22:10:01.000000000","message":"Patch Set 1: Verified-1\n\nBuild failed (check pipeline).  For information on how to proceed, see\nhttp://docs.openstack.org/infra/manual/developers.html#automated-testing\n\n\n- tox-pep8 http://logs.openstack.org/52/552152/1/check/tox-pep8/6515876/ : FAILURE in 3m 52s\n- tox-py27 http://logs.openstack.org/52/552152/1/check/tox-py27/f223dc0/ : SUCCESS in 6m 26s\n- tox-py35 http://logs.openstack.org/52/552152/1/check/tox-py35/b3e29b9/ : FAILURE in 7m 31s\n- build-openstack-sphinx-docs http://logs.openstack.org/52/552152/1/check/build-openstack-sphinx-docs/599c472/html/ : SUCCESS in 4m 16s","accounts_in_message":[],"_revision_number":1},{"id":"1981e2559f0ae3f3719d1d3b554828bc5bcb15d6","author":{"_account_id":20378,"name":"Trevor McCasland","email":"TM2086@att.com","username":"twm2016"},"date":"2018-03-12 22:12:33.000000000","message":"Patch Set 1: Workflow-1","accounts_in_message":[],"_revision_number":1},{"id":"0e9d921c9317099fa228e42d7c841bd3261c199b","author":{"_account_id":5196,"name":"Matthew Treinish","email":"mtreinish@kortar.org","username":"treinish"},"date":"2018-03-12 22:16:25.000000000","message":"Patch Set 1: Code-Review-1\n\n(3 comments)\n\nThanks, this looks good for the most part. Just some inline comments about some concerns about the \u003e1 case.","accounts_in_message":[],"_revision_number":1},{"id":"8b10f7c9c891a8fcdf3dd61492c350dbcdfea5c9","author":{"_account_id":5196,"name":"Matthew Treinish","email":"mtreinish@kortar.org","username":"treinish"},"date":"2018-03-12 22:16:43.000000000","message":"Patch Set 1:\n\noh, oops I see you workflow -1\u0027d this while I was reviewing it :)","accounts_in_message":[],"_revision_number":1},{"id":"6925d30bba93337556ccc96611e3202a05611bb9","author":{"_account_id":20378,"name":"Trevor McCasland","email":"TM2086@att.com","username":"twm2016"},"date":"2018-03-12 22:22:18.000000000","message":"Uploaded patch set 2.","accounts_in_message":[],"_revision_number":2},{"id":"5f2b09d51367588e63b0ec3a33f020e57c6c32da","author":{"_account_id":20378,"name":"Trevor McCasland","email":"TM2086@att.com","username":"twm2016"},"date":"2018-03-12 22:26:45.000000000","message":"Patch Set 1:\n\n\u003e (3 comments)\n \u003e \n \u003e Thanks, this looks good for the most part. Just some inline\n \u003e comments about some concerns about the \u003e1 case.\n\nI was noticing some weird behavior while testing with \u003e1 case, i\u0027ll give it another shot. I\u0027ll make sure I get the other comments too.","accounts_in_message":[],"_revision_number":1},{"id":"92263aa78fa08a841021bef45e9ad3914e5236e9","author":{"_account_id":20378,"name":"Trevor McCasland","email":"TM2086@att.com","username":"twm2016"},"date":"2018-03-12 22:27:07.000000000","message":"Patch Set 2: Workflow-1","accounts_in_message":[],"_revision_number":2},{"id":"165832e7d071a3ce7a0497b58f0b424659322bca","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2018-03-12 22:34:11.000000000","message":"Patch Set 2: Verified+1\n\nBuild succeeded (check pipeline).\n\n- tox-pep8 http://logs.openstack.org/52/552152/2/check/tox-pep8/0b66ce8/ : SUCCESS in 4m 12s\n- tox-py27 http://logs.openstack.org/52/552152/2/check/tox-py27/745cd45/ : SUCCESS in 9m 17s\n- tox-py35 http://logs.openstack.org/52/552152/2/check/tox-py35/a7dc6f1/ : SUCCESS in 6m 23s\n- build-openstack-sphinx-docs http://logs.openstack.org/52/552152/2/check/build-openstack-sphinx-docs/3c01c1e/html/ : SUCCESS in 4m 06s","accounts_in_message":[],"_revision_number":2},{"id":"3fded46c4b6c7098487e61c84f4909d2820ce7f0","author":{"_account_id":20378,"name":"Trevor McCasland","email":"TM2086@att.com","username":"twm2016"},"date":"2018-03-13 22:18:05.000000000","message":"Uploaded patch set 3.","accounts_in_message":[],"_revision_number":3},{"id":"ee711d891e63d38f5aa34155ef6e81da48715a7c","author":{"_account_id":20378,"name":"Trevor McCasland","email":"TM2086@att.com","username":"twm2016"},"date":"2018-03-13 22:25:58.000000000","message":"Patch Set 3:\n\n(3 comments)","accounts_in_message":[],"_revision_number":3},{"id":"eed0011383ce7ef2601fdc446bb63268042254f9","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2018-03-13 22:45:51.000000000","message":"Patch Set 3: Verified-1\n\nBuild failed (check pipeline).  For information on how to proceed, see\nhttp://docs.openstack.org/infra/manual/developers.html#automated-testing\n\n\n- tox-pep8 http://logs.openstack.org/52/552152/3/check/tox-pep8/4d5899e/ : FAILURE in 4m 15s\n- tox-py27 http://logs.openstack.org/52/552152/3/check/tox-py27/8a8386a/ : FAILURE in 6m 41s\n- tox-py35 http://logs.openstack.org/52/552152/3/check/tox-py35/e421909/ : FAILURE in 9m 25s\n- build-openstack-sphinx-docs http://logs.openstack.org/52/552152/3/check/build-openstack-sphinx-docs/4a02f4a/html/ : SUCCESS in 5m 44s","accounts_in_message":[],"_revision_number":3},{"id":"3b15c6be3b3499185eda6ed5f421f74eb8af660d","author":{"_account_id":20378,"name":"Trevor McCasland","email":"TM2086@att.com","username":"twm2016"},"date":"2018-03-15 18:15:14.000000000","message":"Uploaded patch set 4.","accounts_in_message":[],"_revision_number":4},{"id":"71edc21c98e24eeab0354e7e19b33dc0c515f0f4","author":{"_account_id":20378,"name":"Trevor McCasland","email":"TM2086@att.com","username":"twm2016"},"date":"2018-03-15 18:21:07.000000000","message":"Patch Set 4:\n\n(1 comment)","accounts_in_message":[],"_revision_number":4},{"id":"8596fa6a511bdd88fa00b9196f57e6ff5d19b20b","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2018-03-15 19:38:36.000000000","message":"Patch Set 4: Verified+1\n\nBuild succeeded (check pipeline).\n\n- tox-pep8 http://logs.openstack.org/52/552152/4/check/tox-pep8/6c312de/ : SUCCESS in 3m 36s\n- tox-py27 http://logs.openstack.org/52/552152/4/check/tox-py27/2b86496/ : SUCCESS in 7m 30s\n- tox-py35 http://logs.openstack.org/52/552152/4/check/tox-py35/4958002/ : SUCCESS in 6m 32s\n- build-openstack-sphinx-docs http://logs.openstack.org/52/552152/4/check/build-openstack-sphinx-docs/2026f72/html/ : SUCCESS in 3m 49s","accounts_in_message":[],"_revision_number":4},{"id":"569a350bff593e0a394b7eb9b76cdff6f0cfc80b","author":{"_account_id":20378,"name":"Trevor McCasland","email":"TM2086@att.com","username":"twm2016"},"date":"2018-04-06 17:06:14.000000000","message":"Patch Set 4:\n\nI hope the reviewer\u0027s concerns were addressed with the patchsets 2-4. Is there anything else I should consider to add?","accounts_in_message":[],"_revision_number":4},{"id":"e8dd6240a837397c8a7ed1ce23181c9b59bba48b","author":{"_account_id":20378,"name":"Trevor McCasland","email":"TM2086@att.com","username":"twm2016"},"date":"2018-04-06 17:06:56.000000000","message":"Patch Set 4:\n\n(1 comment)","accounts_in_message":[],"_revision_number":4},{"id":"0d56c853e30f160a4f737bf32f722e5da5c94e4b","author":{"_account_id":5196,"name":"Matthew Treinish","email":"mtreinish@kortar.org","username":"treinish"},"date":"2018-04-06 17:29:22.000000000","message":"Patch Set 4: Code-Review-1\n\n(1 comment)\n\nOops just realized that I never actually submitted my comment here and just left it as a draft.\n\nThat line is actually problematic on a couple of fronts. Besides the potential backwards incompatibility you highlighted there isn\u0027t actually a guarantee the content object has a string representation. While most of the attachments we\u0027re dealing with are text and that would work, there isn\u0027t any guarantee that an attachment is a text type. Is there are reason we need to do that?","accounts_in_message":[],"_revision_number":4},{"id":"82e260c51024fd6882bd03257fb62562dfffc048","author":{"_account_id":20378,"name":"Trevor McCasland","email":"TM2086@att.com","username":"twm2016"},"date":"2018-04-06 18:32:50.000000000","message":"Patch Set 4:\n\n\u003e (1 comment)\n \u003e \n \u003e there isn\u0027t any guarantee that an attachment is a text type. \n\nok, that makes sense, maybe we can handle each type individually if it\u0027s not too much work?\n\n \u003e Is there are reason we need to do that?\n\nYeah the reason is to improve the fail data output, because I have to handle the raw Testools.Content object information in the frontend. I made an openstack-health patch, sorry i\u0027ll link it in the commit message next time around.\n\nhttps://review.openstack.org/#/c/552153/\n* To improve the fail data output, the testtools Content\n    object is stored using it\u0027s as_text method.","accounts_in_message":[],"_revision_number":4},{"id":"e5e8d82987dc820aad61aeeda61b80223a5c5aa4","author":{"_account_id":20378,"name":"Trevor McCasland","email":"TM2086@att.com","username":"twm2016"},"date":"2018-04-13 13:56:03.000000000","message":"Patch Set 4: Workflow-1\n\ni\u0027ll see if I can deal with testtools.content object in the frontend i guess.","accounts_in_message":[],"_revision_number":4},{"id":"23fe6ca44df8d1cdad3308bd48ca6270e24f92a9","author":{"_account_id":20378,"name":"Trevor McCasland","email":"TM2086@att.com","username":"twm2016"},"date":"2018-04-13 18:29:18.000000000","message":"Patch Set 4:\n\n(1 comment)\n\nim going to upload a change and then ill work on removing the as_text() method. The issue i\u0027m addressing is inline 1874 db/api.py. Looks like I didn\u0027t test this with the logstash.openstack.org database and just used our local one, sorry about that :(","accounts_in_message":[],"_revision_number":4},{"id":"8025e75fca7aa49373e0fc18458828b7c3b9136c","author":{"_account_id":20378,"name":"Trevor McCasland","email":"TM2086@att.com","username":"twm2016"},"date":"2018-04-13 18:32:34.000000000","message":"Patch Set 4:\n\n(1 comment)","accounts_in_message":[],"_revision_number":4},{"id":"470cdc07814669c565bd86e198fc4177f153ec85","author":{"_account_id":20378,"name":"Trevor McCasland","email":"TM2086@att.com","username":"twm2016"},"date":"2018-04-13 20:02:04.000000000","message":"Uploaded patch set 5.","accounts_in_message":[],"_revision_number":5},{"id":"130b1ed94120e948a9354185e8e0d2a11b2db08b","author":{"_account_id":20378,"name":"Trevor McCasland","email":"TM2086@att.com","username":"twm2016"},"date":"2018-04-13 20:04:26.000000000","message":"Uploaded patch set 6: Commit message was updated.","accounts_in_message":[],"_revision_number":6},{"id":"d2b0ac2a7005ce308eac1c6d8d4fe27434eb6245","author":{"_account_id":20378,"name":"Trevor McCasland","email":"TM2086@att.com","username":"twm2016"},"date":"2018-04-13 20:06:06.000000000","message":"Patch Set 6: Workflow-1\n\nstill need to work on removing the as_text() method as discussed on PS4","accounts_in_message":[],"_revision_number":6},{"id":"51cebc6a6877a209451b9f82ad464102a2def391","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2018-04-13 20:16:10.000000000","message":"Patch Set 6: Verified-1\n\nBuild failed (check pipeline).  For information on how to proceed, see\nhttp://docs.openstack.org/infra/manual/developers.html#automated-testing\n\n\n- tox-pep8 http://logs.openstack.org/52/552152/6/check/tox-pep8/7f02e56/ : FAILURE in 3m 54s\n- tox-py27 http://logs.openstack.org/52/552152/6/check/tox-py27/cd78387/ : SUCCESS in 5m 53s\n- tox-py35 http://logs.openstack.org/52/552152/6/check/tox-py35/cbc8d75/ : SUCCESS in 6m 23s\n- build-openstack-sphinx-docs http://logs.openstack.org/52/552152/6/check/build-openstack-sphinx-docs/13617cf/html/ : SUCCESS in 4m 05s","accounts_in_message":[],"_revision_number":6},{"id":"62b3a2514402307005a27e3a9160d5b81ed09a2d","author":{"_account_id":20378,"name":"Trevor McCasland","email":"TM2086@att.com","username":"twm2016"},"date":"2018-04-24 17:15:37.000000000","message":"Uploaded patch set 7.","accounts_in_message":[],"_revision_number":7},{"id":"20a5997a5fb77a7b5ff3848ad46bb30fb3a0dcff","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2018-04-24 18:44:39.000000000","message":"Patch Set 7: Verified+1\n\nBuild succeeded (check pipeline).\n\n- tox-pep8 http://logs.openstack.org/52/552152/7/check/tox-pep8/eb80cfb/ : SUCCESS in 4m 25s\n- tox-py27 http://logs.openstack.org/52/552152/7/check/tox-py27/981fe18/ : SUCCESS in 9m 37s\n- tox-py35 http://logs.openstack.org/52/552152/7/check/tox-py35/710fbec/ : SUCCESS in 12m 07s\n- build-openstack-releasenotes http://logs.openstack.org/52/552152/7/check/build-openstack-releasenotes/6a46f09/html/ : SUCCESS in 4m 07s\n- build-openstack-sphinx-docs http://logs.openstack.org/52/552152/7/check/build-openstack-sphinx-docs/82612e1/html/ : SUCCESS in 4m 13s","accounts_in_message":[],"_revision_number":7},{"id":"dcd64e8b70b3fa3ab6a34992ff32e72a99144c42","author":{"_account_id":20378,"name":"Trevor McCasland","email":"TM2086@att.com","username":"twm2016"},"date":"2018-04-24 21:30:16.000000000","message":"Patch Set 4:\n\n(1 comment)","accounts_in_message":[],"_revision_number":4},{"id":"b528928dd208a98396a53637d31983ba58207066","author":{"_account_id":20378,"name":"Trevor McCasland","email":"TM2086@att.com","username":"twm2016"},"date":"2018-04-24 21:31:45.000000000","message":"Patch Set 7:\n\n(1 comment)","accounts_in_message":[],"_revision_number":7},{"id":"85d17ed9b05daf74bbb5497fa94a2bc7f8cf216a","author":{"_account_id":20378,"name":"Trevor McCasland","email":"TM2086@att.com","username":"twm2016"},"date":"2018-06-22 13:43:17.000000000","message":"Patch Set 4:\n\n(1 comment)","accounts_in_message":[],"_revision_number":4},{"id":"0d13303d7cc4db5efc707d61835ab81ef201323b","author":{"_account_id":17887,"name":"Doug Schveninger","email":"ds6901@att.com","username":"Doug.Schveninger"},"date":"2018-07-29 21:54:32.000000000","message":"Patch Set 7: Code-Review+1","accounts_in_message":[],"_revision_number":7},{"id":"1caff42353fa6c4eb291020dd9148521315401f7","author":{"_account_id":5196,"name":"Matthew Treinish","email":"mtreinish@kortar.org","username":"treinish"},"date":"2018-12-24 20:41:44.000000000","message":"Patch Set 7:\n\n(1 comment)\n\nSorry this took so long for me to get back to. This iteration looks better to me. I have one inline question about the potential performance impacts when we set the flag to true, since we end up making the same query (which is mostly the same) twice. We should be able to conditionally add on the other join and attachments values if the flag is true and handle the post processing with a single query.\n\nThat being said if you didn\u0027t test the performance we can leave that as a future optimization (considering how long this sat and things really aren\u0027t ever that fast)  if you add a todo comment above that block mentioning we can optimize this I\u0027ll be fine and we can merge this.","accounts_in_message":[],"_revision_number":7},{"id":"b35757050a664771057afc7016a601ef9f60c270","author":{"_account_id":5196,"name":"Matthew Treinish","email":"mtreinish@kortar.org","username":"treinish"},"date":"2018-12-24 20:41:50.000000000","message":"Patch Set 7:\n\nrecheck","accounts_in_message":[],"_revision_number":7},{"id":"63cbaaf6d90b4beef0c0c3ae1c0f3c8dcf768bc5","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2018-12-24 20:56:07.000000000","message":"Patch Set 7:\n\nBuild succeeded (check pipeline).\n\n- tox-pep8 http://logs.openstack.org/52/552152/7/check/tox-pep8/e3af8c0/ : SUCCESS in 3m 45s\n- tox-py27 http://logs.openstack.org/52/552152/7/check/tox-py27/148279f/ : SUCCESS in 5m 53s\n- tox-py35 http://logs.openstack.org/52/552152/7/check/tox-py35/4d09659/ : SUCCESS in 13m 22s\n- build-openstack-releasenotes http://logs.openstack.org/52/552152/7/check/build-openstack-releasenotes/57683af/html/ : SUCCESS in 3m 12s\n- build-openstack-sphinx-docs http://logs.openstack.org/52/552152/7/check/build-openstack-sphinx-docs/ec0a01b/html/ : SUCCESS in 3m 49s","accounts_in_message":[],"_revision_number":7},{"id":"00ac7c9be3fee017e1308e72eb89975b26e41a54","author":{"_account_id":20378,"name":"Trevor McCasland","email":"TM2086@att.com","username":"twm2016"},"date":"2018-12-26 18:26:08.000000000","message":"Patch Set 7: Workflow-1\n\n(2 comments)\n\n\u003e (1 comment)\n \u003e \n \u003e Sorry this took so long for me to get back to. This iteration looks\n \u003e better to me. I have one inline question about the potential\n \u003e performance impacts when we set the flag to true, since we end up\n \u003e making the same query (which is mostly the same) twice. We should\n \u003e be able to conditionally add on the other join and attachments\n \u003e values if the flag is true and handle the post processing with a\n \u003e single query.\n \u003e \n \u003e That being said if you didn\u0027t test the performance we can leave\n \u003e that as a future optimization (considering how long this sat and\n \u003e things really aren\u0027t ever that fast)  if you add a todo comment\n \u003e above that block mentioning we can optimize this I\u0027ll be fine and\n \u003e we can merge this.\n\nGood point, I haven\u0027t noticed this query taking any longer than 0.1 sec with 37 test_runs. I\u0027ll update the commit message with some performance analysis with a larger number of test_runs.\n\nI also believe we should add a test where at least one of the jobs has more than one failing test but with unique attachments so that we can test the separation of attachments between jobs and as well as the separation of attachments between tests per job","accounts_in_message":[],"_revision_number":7},{"id":"0e39a40e12ffda98c3801e39b4ba485d3c244458","author":{"_account_id":20378,"name":"Trevor McCasland","email":"TM2086@att.com","username":"twm2016"},"date":"2018-12-26 22:12:23.000000000","message":"Uploaded patch set 8.","accounts_in_message":[],"_revision_number":8},{"id":"a184e6dc8a68eb62cc2935493e6b59fcfcdf96da","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2018-12-26 22:28:23.000000000","message":"Patch Set 8: Verified-1\n\nBuild failed (check pipeline).  For information on how to proceed, see\nhttp://docs.openstack.org/infra/manual/developers.html#automated-testing\n\n\n- tox-pep8 http://logs.openstack.org/52/552152/8/check/tox-pep8/9a380f7/ : SUCCESS in 3m 42s\n- tox-py27 http://logs.openstack.org/52/552152/8/check/tox-py27/fb91d9e/ : SUCCESS in 14m 54s\n- tox-py35 http://logs.openstack.org/52/552152/8/check/tox-py35/2c971bb/ : FAILURE in 7m 55s\n- build-openstack-releasenotes http://logs.openstack.org/52/552152/8/check/build-openstack-releasenotes/77dede0/html/ : SUCCESS in 3m 32s\n- build-openstack-sphinx-docs http://logs.openstack.org/52/552152/8/check/build-openstack-sphinx-docs/7074092/html/ : SUCCESS in 4m 26s","accounts_in_message":[],"_revision_number":8},{"id":"35b86715ec76266059dad17afe3e18d849ec306c","author":{"_account_id":20378,"name":"Trevor McCasland","email":"TM2086@att.com","username":"twm2016"},"date":"2018-12-26 22:33:27.000000000","message":"Uploaded patch set 9.","accounts_in_message":[],"_revision_number":9},{"id":"e28dad1835d759407a2506d968e4b8ee076383b4","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2018-12-26 22:47:44.000000000","message":"Patch Set 9: Verified+1\n\nBuild succeeded (check pipeline).\n\n- tox-pep8 http://logs.openstack.org/52/552152/9/check/tox-pep8/e638e0f/ : SUCCESS in 3m 48s\n- tox-py27 http://logs.openstack.org/52/552152/9/check/tox-py27/6aa7170/ : SUCCESS in 5m 45s\n- tox-py35 http://logs.openstack.org/52/552152/9/check/tox-py35/a36a584/ : SUCCESS in 7m 02s\n- build-openstack-releasenotes http://logs.openstack.org/52/552152/9/check/build-openstack-releasenotes/e902256/html/ : SUCCESS in 3m 17s\n- build-openstack-sphinx-docs http://logs.openstack.org/52/552152/9/check/build-openstack-sphinx-docs/21147e4/html/ : SUCCESS in 4m 04s","accounts_in_message":[],"_revision_number":9},{"id":"0f146b7ff520b5a5438b358bb8f1a505a4ae465a","author":{"_account_id":20378,"name":"Trevor McCasland","email":"TM2086@att.com","username":"twm2016"},"date":"2018-12-27 15:12:51.000000000","message":"Patch Set 7:\n\n(1 comment)","accounts_in_message":[],"_revision_number":7},{"id":"1623c2fecf32ad740171f72eb310f033c8ed724c","author":{"_account_id":5196,"name":"Matthew Treinish","email":"mtreinish@kortar.org","username":"treinish"},"date":"2019-01-09 16:27:23.000000000","message":"Patch Set 9: Code-Review+2\n\nLGTM, thanks for respinning","accounts_in_message":[],"_revision_number":9},{"id":"ec733a91c14c999b80559a19148be4552346a1df","author":{"_account_id":5196,"name":"Matthew Treinish","email":"mtreinish@kortar.org","username":"treinish"},"date":"2019-07-17 22:27:46.000000000","message":"Patch Set 9: Workflow+1\n\nI don\u0027t think anyone else will review this it\u0027s been sitting long enough.","accounts_in_message":[],"_revision_number":9},{"id":"f2e295b0bb9727849eeca06bf010c52e4620ce1c","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2019-07-17 22:27:57.000000000","message":"Patch Set 9: -Verified\n\nStarting gate jobs.","accounts_in_message":[],"_revision_number":9},{"id":"d7329fb06775a69f57137dcb089c19aace092403","author":{"_account_id":5263,"name":"Jeremy Stanley","display_name":"fungi","email":"fungi@yuggoth.org","username":"fungi","status":"missing, presumed fed"},"date":"2019-07-17 22:31:47.000000000","message":"Patch Set 9: Code-Review+2","accounts_in_message":[],"_revision_number":9},{"id":"d751a89757270265d6a0c9aae2fa6d4c4f0c1797","author":{"_account_id":5263,"name":"Jeremy Stanley","display_name":"fungi","email":"fungi@yuggoth.org","username":"fungi","status":"missing, presumed fed"},"date":"2019-07-17 22:33:13.000000000","message":"Patch Set 9:\n\nYep, sorry, had missed it before but looks like a fine improvement.","accounts_in_message":[],"_revision_number":9},{"id":"53c212b7974a4658236388272dc5aec052f49349","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2019-07-17 22:45:18.000000000","message":"Patch Set 9: Verified-2\n\nBuild failed (gate pipeline).  For information on how to proceed, see\nhttp://docs.openstack.org/infra/manual/developers.html#automated-testing\n\n\n- tox-pep8 http://logs.openstack.org/52/552152/9/gate/tox-pep8/6d03d16/ : RETRY_LIMIT in 1m 49s\n- tox-py27 http://logs.openstack.org/52/552152/9/gate/tox-py27/56f4f40/ : RETRY_LIMIT in 2m 08s\n- tox-py35 http://logs.openstack.org/52/552152/9/gate/tox-py35/22a7cad/ : RETRY_LIMIT in 2m 12s\n- build-openstack-releasenotes http://logs.openstack.org/52/552152/9/gate/build-openstack-releasenotes/ae6a997/html/ : SUCCESS in 3m 05s\n- openstack-tox-docs http://logs.openstack.org/52/552152/9/gate/openstack-tox-docs/16bd96e/html/ : SUCCESS in 2m 49s","accounts_in_message":[],"_revision_number":9},{"id":"77b9cacacbcf47d96557648e7756fae6e0c1283b","author":{"_account_id":5196,"name":"Matthew Treinish","email":"mtreinish@kortar.org","username":"treinish"},"date":"2019-07-18 05:14:24.000000000","message":"Patch Set 9:\n\nrecheck","accounts_in_message":[],"_revision_number":9},{"id":"2c1f44f04be2255d4a738c86856cde5847f1d983","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2019-07-18 05:26:37.000000000","message":"Patch Set 9: Verified+1\n\nBuild succeeded (check pipeline).\n\n- tox-pep8 http://logs.openstack.org/52/552152/9/check/tox-pep8/2710ece/ : SUCCESS in 3m 44s\n- tox-py27 http://logs.openstack.org/52/552152/9/check/tox-py27/ae6ed7c/ : SUCCESS in 5m 56s\n- tox-py35 http://logs.openstack.org/52/552152/9/check/tox-py35/5e82c15/ : SUCCESS in 6m 16s\n- build-openstack-releasenotes http://logs.openstack.org/52/552152/9/check/build-openstack-releasenotes/34ce427/html/ : SUCCESS in 3m 21s\n- openstack-tox-docs http://logs.openstack.org/52/552152/9/check/openstack-tox-docs/8f5da1a/html/ : SUCCESS in 3m 35s","accounts_in_message":[],"_revision_number":9},{"id":"db356c8d2048819a3979e9504e79260b6be44cfd","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2019-07-18 05:26:49.000000000","message":"Patch Set 9: -Verified\n\nStarting gate jobs.","accounts_in_message":[],"_revision_number":9},{"id":"f19df29c421f37347d923ec66515172fe510dd18","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2019-07-18 05:52:22.000000000","message":"Change has been successfully merged by Zuul","accounts_in_message":[],"_revision_number":9},{"id":"0d0d7f96f519492e331fde32d4889b9df07e8e7d","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2019-07-18 05:52:22.000000000","message":"Patch Set 9: Verified+2\n\nBuild succeeded (gate pipeline).\n\n- tox-pep8 http://logs.openstack.org/52/552152/9/gate/tox-pep8/6f50603/ : SUCCESS in 4m 40s\n- tox-py27 http://logs.openstack.org/52/552152/9/gate/tox-py27/5ea3d7a/ : SUCCESS in 7m 10s\n- tox-py35 http://logs.openstack.org/52/552152/9/gate/tox-py35/3a69d6f/ : SUCCESS in 10m 06s\n- build-openstack-releasenotes http://logs.openstack.org/52/552152/9/gate/build-openstack-releasenotes/6a8ff3f/html/ : SUCCESS in 3m 28s\n- openstack-tox-docs http://logs.openstack.org/52/552152/9/gate/openstack-tox-docs/7032a71/html/ : SUCCESS in 4m 46s","accounts_in_message":[],"_revision_number":9}],"current_revision_number":9,"current_revision":"464a9e0adc1f6ebed14b95e83a81b1e343f5465d","revisions":{"022c4c2d7ea6feae3e0da7263c5aea1c9cde2a8c":{"kind":"REWORK","_number":1,"created":"2018-03-12 21:54:32.000000000","uploader":{"_account_id":20378,"name":"Trevor McCasland","email":"TM2086@att.com","username":"twm2016"},"ref":"refs/changes/52/552152/1","fetch":{"anonymous http":{"url":"https://review.opendev.org/opendev/subunit2sql","ref":"refs/changes/52/552152/1","commands":{"Checkout":"git fetch https://review.opendev.org/opendev/subunit2sql refs/changes/52/552152/1 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/opendev/subunit2sql refs/changes/52/552152/1 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/opendev/subunit2sql refs/changes/52/552152/1 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/opendev/subunit2sql refs/changes/52/552152/1"}}},"commit":{"parents":[{"commit":"02a25056c4ce508e59fd7e5fefbf60bea975175b","subject":"Honor dpi cli flag for ci_time graph","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/opendev/subunit2sql/commit/02a25056c4ce508e59fd7e5fefbf60bea975175b"}]}],"author":{"name":"mccasland, trevor (tm2086)","email":"TM2086@att.com","date":"2018-03-12 21:47:30.000000000","tz":-300},"committer":{"name":"mccasland, trevor (tm2086)","email":"TM2086@att.com","date":"2018-03-12 21:53:24.000000000","tz":-300},"subject":"Add attachments flag to get_test_runs_by_status","message":"Add attachments flag to get_test_runs_by_status\n\nBy adding an \u0027include_attachments\u0027 flag to the get_test_runs_by_status\nmethod we can join the Attachments table with the same structure as\nthe RunMetadata table allowing us to optionally include attachments or\nmetadata with test_runs fitlered by a given status.\n\nThis change is needed by an existing change in openstack-health\nto render the attachments data on the home page.\n\nChange-Id: I31468c825cf259b8df62134e578f31d96af6ac75\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/opendev/subunit2sql/commit/022c4c2d7ea6feae3e0da7263c5aea1c9cde2a8c"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/opendev/subunit2sql/commit/022c4c2d7ea6feae3e0da7263c5aea1c9cde2a8c"}]},"branch":"refs/heads/master"},"3f62d07b720867a4525476985902cf770e52d0b5":{"kind":"REWORK","_number":2,"created":"2018-03-12 22:22:18.000000000","uploader":{"_account_id":20378,"name":"Trevor McCasland","email":"TM2086@att.com","username":"twm2016"},"ref":"refs/changes/52/552152/2","fetch":{"anonymous http":{"url":"https://review.opendev.org/opendev/subunit2sql","ref":"refs/changes/52/552152/2","commands":{"Checkout":"git fetch https://review.opendev.org/opendev/subunit2sql refs/changes/52/552152/2 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/opendev/subunit2sql refs/changes/52/552152/2 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/opendev/subunit2sql refs/changes/52/552152/2 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/opendev/subunit2sql refs/changes/52/552152/2"}}},"commit":{"parents":[{"commit":"02a25056c4ce508e59fd7e5fefbf60bea975175b","subject":"Honor dpi cli flag for ci_time graph","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/opendev/subunit2sql/commit/02a25056c4ce508e59fd7e5fefbf60bea975175b"}]}],"author":{"name":"mccasland, trevor (tm2086)","email":"TM2086@att.com","date":"2018-03-12 21:47:30.000000000","tz":-300},"committer":{"name":"mccasland, trevor (tm2086)","email":"TM2086@att.com","date":"2018-03-12 22:22:00.000000000","tz":-300},"subject":"Add attachments flag to get_test_runs_by_status","message":"Add attachments flag to get_test_runs_by_status\n\nBy adding an \u0027include_attachments\u0027 flag to the get_test_runs_by_status\nmethod we can join the Attachments table with the same structure as\nthe RunMetadata table allowing us to optionally include attachments or\nmetadata with test_runs fitlered by a given status.\n\nThis change is needed by an existing change in openstack-health\nto render the attachments data on the home page.\n\nChange-Id: I31468c825cf259b8df62134e578f31d96af6ac75\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/opendev/subunit2sql/commit/3f62d07b720867a4525476985902cf770e52d0b5"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/opendev/subunit2sql/commit/3f62d07b720867a4525476985902cf770e52d0b5"}]},"branch":"refs/heads/master"},"ae23e24c11e8fb233ab048c555f32e05c82e8b2c":{"kind":"REWORK","_number":3,"created":"2018-03-13 22:18:05.000000000","uploader":{"_account_id":20378,"name":"Trevor McCasland","email":"TM2086@att.com","username":"twm2016"},"ref":"refs/changes/52/552152/3","fetch":{"anonymous http":{"url":"https://review.opendev.org/opendev/subunit2sql","ref":"refs/changes/52/552152/3","commands":{"Checkout":"git fetch https://review.opendev.org/opendev/subunit2sql refs/changes/52/552152/3 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/opendev/subunit2sql refs/changes/52/552152/3 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/opendev/subunit2sql refs/changes/52/552152/3 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/opendev/subunit2sql refs/changes/52/552152/3"}}},"commit":{"parents":[{"commit":"02a25056c4ce508e59fd7e5fefbf60bea975175b","subject":"Honor dpi cli flag for ci_time graph","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/opendev/subunit2sql/commit/02a25056c4ce508e59fd7e5fefbf60bea975175b"}]}],"author":{"name":"mccasland, trevor (tm2086)","email":"TM2086@att.com","date":"2018-03-12 21:47:30.000000000","tz":-300},"committer":{"name":"mccasland, trevor (tm2086)","email":"TM2086@att.com","date":"2018-03-13 22:16:50.000000000","tz":-300},"subject":"Add attachments flag to get_test_runs_by_status","message":"Add attachments flag to get_test_runs_by_status\n\nBy adding an \u0027include_attachments\u0027 flag to the get_test_runs_by_status\nmethod we can join the Attachments table with the same structure as\nthe RunMetadata table allowing us to optionally include attachments or\nmetadata with test_runs filtered by a given status.\n\nThis change is needed by an existing change in openstack-health\nto render the attachments data on the home page.\n\nChange-Id: I31468c825cf259b8df62134e578f31d96af6ac75\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/opendev/subunit2sql/commit/ae23e24c11e8fb233ab048c555f32e05c82e8b2c"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/opendev/subunit2sql/commit/ae23e24c11e8fb233ab048c555f32e05c82e8b2c"}]},"branch":"refs/heads/master"},"932b7c1fb39e06291065e6e4f0c5d0bc2b896313":{"kind":"REWORK","_number":4,"created":"2018-03-15 18:15:14.000000000","uploader":{"_account_id":20378,"name":"Trevor McCasland","email":"TM2086@att.com","username":"twm2016"},"ref":"refs/changes/52/552152/4","fetch":{"anonymous http":{"url":"https://review.opendev.org/opendev/subunit2sql","ref":"refs/changes/52/552152/4","commands":{"Checkout":"git fetch https://review.opendev.org/opendev/subunit2sql refs/changes/52/552152/4 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/opendev/subunit2sql refs/changes/52/552152/4 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/opendev/subunit2sql refs/changes/52/552152/4 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/opendev/subunit2sql refs/changes/52/552152/4"}}},"commit":{"parents":[{"commit":"02a25056c4ce508e59fd7e5fefbf60bea975175b","subject":"Honor dpi cli flag for ci_time graph","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/opendev/subunit2sql/commit/02a25056c4ce508e59fd7e5fefbf60bea975175b"}]}],"author":{"name":"mccasland, trevor (tm2086)","email":"TM2086@att.com","date":"2018-03-12 21:47:30.000000000","tz":-300},"committer":{"name":"mccasland, trevor (tm2086)","email":"TM2086@att.com","date":"2018-03-15 18:14:55.000000000","tz":-300},"subject":"Add attachments flag to get_test_runs_by_status","message":"Add attachments flag to get_test_runs_by_status\n\nBy adding an \u0027include_attachments\u0027 flag to the get_test_runs_by_status\nmethod we can join the Attachments table with the same structure as\nthe RunMetadata table allowing us to optionally include attachments or\nmetadata with test_runs filtered by a given status.\n\nThis change is needed by an existing change in openstack-health\nto render the attachments data on the home page.\n\nChange-Id: I31468c825cf259b8df62134e578f31d96af6ac75\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/opendev/subunit2sql/commit/932b7c1fb39e06291065e6e4f0c5d0bc2b896313"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/opendev/subunit2sql/commit/932b7c1fb39e06291065e6e4f0c5d0bc2b896313"}]},"branch":"refs/heads/master"},"cd749f97662a20a1bc242fe0a242d53a6c9b9e11":{"kind":"REWORK","_number":5,"created":"2018-04-13 20:02:04.000000000","uploader":{"_account_id":20378,"name":"Trevor McCasland","email":"TM2086@att.com","username":"twm2016"},"ref":"refs/changes/52/552152/5","fetch":{"anonymous http":{"url":"https://review.opendev.org/opendev/subunit2sql","ref":"refs/changes/52/552152/5","commands":{"Checkout":"git fetch https://review.opendev.org/opendev/subunit2sql refs/changes/52/552152/5 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/opendev/subunit2sql refs/changes/52/552152/5 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/opendev/subunit2sql refs/changes/52/552152/5 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/opendev/subunit2sql refs/changes/52/552152/5"}}},"commit":{"parents":[{"commit":"02a25056c4ce508e59fd7e5fefbf60bea975175b","subject":"Honor dpi cli flag for ci_time graph","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/opendev/subunit2sql/commit/02a25056c4ce508e59fd7e5fefbf60bea975175b"}]}],"author":{"name":"mccasland, trevor (tm2086)","email":"TM2086@att.com","date":"2018-03-12 21:47:30.000000000","tz":-300},"committer":{"name":"mccasland, trevor (tm2086)","email":"tm2086@att.com","date":"2018-04-13 20:01:22.000000000","tz":-300},"subject":"Add attachments flag to get_test_runs_by_status","message":"Add attachments flag to get_test_runs_by_status\n\nBy adding an \u0027include_attachments\u0027 flag to the get_test_runs_by_status\nmethod we can join the Attachments table with the same structure as\nthe RunMetadata table allowing us to optionally include attachments or\nmetadata with test_runs filtered by a given status.\n\nNotes on attachment storage:\n  currently the attachments are stored using the testtools Content\n  python object[1]. I will investigate if I can use this on the\n  openstack-health api side and propose a change if so. It seems like\n  its returned as a string so I didn\u0027t think it was possible without\n  something like an eval call (that\u0027s bad practice i think?[2]) or\n  parsing the string for content.\n\nNotes on testing:\n  logstash.o.o does not store attachments[2]\n\n[1] https://github.com/openstack-infra/subunit2sql/blob/fed81d0901a5783d34aaf8302392782bd390a604/subunit2sql/read_subunit.py#L76\n[2] https://stackoverflow.com/questions/1832940/why-is-using-eval-a-bad-practice?\n[3] http://paste.openstack.org/show/719192/\nNeeded-By: I3ce2fc50ada9462de3df29b5a8b76b12a548fd12\nChange-Id: I31468c825cf259b8df62134e578f31d96af6ac75\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/opendev/subunit2sql/commit/cd749f97662a20a1bc242fe0a242d53a6c9b9e11"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/opendev/subunit2sql/commit/cd749f97662a20a1bc242fe0a242d53a6c9b9e11"}]},"branch":"refs/heads/master"},"add5c803a120a1b43d47e12591f517ec376a993d":{"kind":"NO_CODE_CHANGE","_number":6,"created":"2018-04-13 20:04:26.000000000","uploader":{"_account_id":20378,"name":"Trevor McCasland","email":"TM2086@att.com","username":"twm2016"},"ref":"refs/changes/52/552152/6","fetch":{"anonymous http":{"url":"https://review.opendev.org/opendev/subunit2sql","ref":"refs/changes/52/552152/6","commands":{"Checkout":"git fetch https://review.opendev.org/opendev/subunit2sql refs/changes/52/552152/6 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/opendev/subunit2sql refs/changes/52/552152/6 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/opendev/subunit2sql refs/changes/52/552152/6 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/opendev/subunit2sql refs/changes/52/552152/6"}}},"commit":{"parents":[{"commit":"02a25056c4ce508e59fd7e5fefbf60bea975175b","subject":"Honor dpi cli flag for ci_time graph","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/opendev/subunit2sql/commit/02a25056c4ce508e59fd7e5fefbf60bea975175b"}]}],"author":{"name":"mccasland, trevor (tm2086)","email":"TM2086@att.com","date":"2018-03-12 21:47:30.000000000","tz":-300},"committer":{"name":"mccasland, trevor (tm2086)","email":"tm2086@att.com","date":"2018-04-13 20:03:59.000000000","tz":-300},"subject":"Add attachments flag to get_test_runs_by_status","message":"Add attachments flag to get_test_runs_by_status\n\nBy adding an \u0027include_attachments\u0027 flag to the get_test_runs_by_status\nmethod we can join the Attachments table with the same structure as\nthe RunMetadata table allowing us to optionally include attachments or\nmetadata with test_runs filtered by a given status.\n\nNotes on attachment storage:\n  currently the attachments are stored using the testtools Content\n  python object[1]. I will investigate if I can use this on the\n  openstack-health api side and propose a change if so. It seems like\n  its returned as a string so I didn\u0027t think it was possible without\n  something like an eval call (that\u0027s bad practice i think?[2]) or\n  parsing the string for content.\n\nNotes on testing:\n  logstash.o.o does not store attachments[3]\n\n[1] https://github.com/openstack-infra/subunit2sql/blob/fed81d0901a5783d34aaf8302392782bd390a604/subunit2sql/read_subunit.py#L76\n[2] https://stackoverflow.com/questions/1832940/why-is-using-eval-a-bad-practice?\n[3] http://paste.openstack.org/show/719192/\nNeeded-By: I3ce2fc50ada9462de3df29b5a8b76b12a548fd12\nChange-Id: I31468c825cf259b8df62134e578f31d96af6ac75\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/opendev/subunit2sql/commit/add5c803a120a1b43d47e12591f517ec376a993d"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/opendev/subunit2sql/commit/add5c803a120a1b43d47e12591f517ec376a993d"}]},"branch":"refs/heads/master"},"948b641d6b8d5ed09111ce4c87119eeb3489c074":{"kind":"REWORK","_number":7,"created":"2018-04-24 17:15:37.000000000","uploader":{"_account_id":20378,"name":"Trevor McCasland","email":"TM2086@att.com","username":"twm2016"},"ref":"refs/changes/52/552152/7","fetch":{"anonymous http":{"url":"https://review.opendev.org/opendev/subunit2sql","ref":"refs/changes/52/552152/7","commands":{"Checkout":"git fetch https://review.opendev.org/opendev/subunit2sql refs/changes/52/552152/7 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/opendev/subunit2sql refs/changes/52/552152/7 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/opendev/subunit2sql refs/changes/52/552152/7 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/opendev/subunit2sql refs/changes/52/552152/7"}}},"commit":{"parents":[{"commit":"08222d7a2b09250ed0048f3d7e767c55191fcc20","subject":"Amend releasenotes for new subunit2sql functions and options","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/opendev/subunit2sql/commit/08222d7a2b09250ed0048f3d7e767c55191fcc20"}]}],"author":{"name":"mccasland, trevor (tm2086)","email":"TM2086@att.com","date":"2018-03-12 21:47:30.000000000","tz":-300},"committer":{"name":"mccasland, trevor (tm2086)","email":"tm2086@att.com","date":"2018-04-24 17:08:45.000000000","tz":-300},"subject":"Add attachments flag to get_test_runs_by_status","message":"Add attachments flag to get_test_runs_by_status\n\nBy adding an \u0027include_attachments\u0027 flag to the get_test_runs_by_status\nmethod we can join the Attachments table with the same structure as\nthe RunMetadata table allowing us to optionally include attachments or\nmetadata with test_runs filtered by a given status.\n\nNotes on testing:\n  * As far as I could tell, logstash.o.o doesn\u0027t store attachments[1]\n  so to test this patch you will have to setup/use another subunit2sql\n  db with attachments stored in it\n  * you can find the openstack-health patch to render the attachments\n  in the Needed-By change of the footer of this commit message.\n\n[1] http://paste.openstack.org/show/719192/\nNeeded-By: I3ce2fc50ada9462de3df29b5a8b76b12a548fd12\nChange-Id: I31468c825cf259b8df62134e578f31d96af6ac75\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/opendev/subunit2sql/commit/948b641d6b8d5ed09111ce4c87119eeb3489c074"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/opendev/subunit2sql/commit/948b641d6b8d5ed09111ce4c87119eeb3489c074"}]},"branch":"refs/heads/master"},"57c2bfd810d65b1f51375fc991f5dbcee6a1ff9a":{"kind":"REWORK","_number":8,"created":"2018-12-26 22:12:23.000000000","uploader":{"_account_id":20378,"name":"Trevor McCasland","email":"TM2086@att.com","username":"twm2016"},"ref":"refs/changes/52/552152/8","fetch":{"anonymous http":{"url":"https://review.opendev.org/opendev/subunit2sql","ref":"refs/changes/52/552152/8","commands":{"Checkout":"git fetch https://review.opendev.org/opendev/subunit2sql refs/changes/52/552152/8 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/opendev/subunit2sql refs/changes/52/552152/8 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/opendev/subunit2sql refs/changes/52/552152/8 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/opendev/subunit2sql refs/changes/52/552152/8"}}},"commit":{"parents":[{"commit":"4202e4c195dc090eee8c4b562ed129cc61131d91","subject":"Merge \"Update min tox version to 2.0\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/opendev/subunit2sql/commit/4202e4c195dc090eee8c4b562ed129cc61131d91"}]}],"author":{"name":"mccasland, trevor (tm2086)","email":"TM2086@att.com","date":"2018-03-12 21:47:30.000000000","tz":-300},"committer":{"name":"mccasland, trevor (tm2086)","email":"tm2086@att.com","date":"2018-12-26 21:54:28.000000000","tz":-360},"subject":"Add attachments flag to get_test_runs_by_status","message":"Add attachments flag to get_test_runs_by_status\n\nBy adding an \u0027include_attachments\u0027 flag to the get_test_runs_by_status\nmethod we can join the Attachments table with the same structure as\nthe RunMetadata table allowing us to optionally include attachments or\nmetadata with test_runs filtered by a given status.\n\nNotes on testing:\n  * As far as I could tell, logstash.o.o doesn\u0027t store attachments[1]\n  so to test this patch you will have to setup/use another subunit2sql\n  db with attachments stored in it\n  * you can find the openstack-health patch to render the attachments\n  in the Needed-By change of the footer of this commit message.\n\nNotes on performance:\n  * on average loading attachments is about 13% slower when calling this\n    api with python[2]. The query times in the paste were produced by calling\n    the api method get_test_runs_by_status_for_run_ids 100x in a for loop with\n    49 test_runs being returned from the call, which is the number of failures\n    in the last 10 runs.\n\n[1] http://paste.openstack.org/show/719192/\n[2] http://paste.openstack.org/show/738662/\nNeeded-By: I3ce2fc50ada9462de3df29b5a8b76b12a548fd12\nChange-Id: I31468c825cf259b8df62134e578f31d96af6ac75\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/opendev/subunit2sql/commit/57c2bfd810d65b1f51375fc991f5dbcee6a1ff9a"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/opendev/subunit2sql/commit/57c2bfd810d65b1f51375fc991f5dbcee6a1ff9a"}]},"branch":"refs/heads/master"},"464a9e0adc1f6ebed14b95e83a81b1e343f5465d":{"kind":"REWORK","_number":9,"created":"2018-12-26 22:33:27.000000000","uploader":{"_account_id":20378,"name":"Trevor McCasland","email":"TM2086@att.com","username":"twm2016"},"ref":"refs/changes/52/552152/9","fetch":{"anonymous http":{"url":"https://review.opendev.org/opendev/subunit2sql","ref":"refs/changes/52/552152/9","commands":{"Checkout":"git fetch https://review.opendev.org/opendev/subunit2sql refs/changes/52/552152/9 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/opendev/subunit2sql refs/changes/52/552152/9 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/opendev/subunit2sql refs/changes/52/552152/9 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/opendev/subunit2sql refs/changes/52/552152/9"}}},"commit":{"parents":[{"commit":"4202e4c195dc090eee8c4b562ed129cc61131d91","subject":"Merge \"Update min tox version to 2.0\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/opendev/subunit2sql/commit/4202e4c195dc090eee8c4b562ed129cc61131d91"}]}],"author":{"name":"mccasland, trevor (tm2086)","email":"TM2086@att.com","date":"2018-03-12 21:47:30.000000000","tz":-300},"committer":{"name":"mccasland, trevor (tm2086)","email":"tm2086@att.com","date":"2018-12-26 22:15:27.000000000","tz":-360},"subject":"Add attachments flag to get_test_runs_by_status","message":"Add attachments flag to get_test_runs_by_status\n\nBy adding an \u0027include_attachments\u0027 flag to the get_test_runs_by_status\nmethod we can join the Attachments table with the same structure as\nthe RunMetadata table allowing us to optionally include attachments or\nmetadata with test_runs filtered by a given status.\n\nNotes on testing:\n  * As far as I could tell, logstash.o.o doesn\u0027t store attachments[1]\n  so to test this patch you will have to setup/use another subunit2sql\n  db with attachments stored in it\n  * you can find the openstack-health patch to render the attachments\n  in the Needed-By change of the footer of this commit message.\n\nNotes on performance:\n  * on average loading attachments is about 13% slower when calling this\n    api with python[2]. The query times in the paste were produced by calling\n    the api method get_test_runs_by_status_for_run_ids 100x in a for loop with\n    49 test_runs being returned from the call, which is the number of failures\n    in the last 10 runs.\n\n[1] http://paste.openstack.org/show/719192/\n[2] http://paste.openstack.org/show/738662/\nNeeded-By: I3ce2fc50ada9462de3df29b5a8b76b12a548fd12\nChange-Id: I31468c825cf259b8df62134e578f31d96af6ac75\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/opendev/subunit2sql/commit/464a9e0adc1f6ebed14b95e83a81b1e343f5465d"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/opendev/subunit2sql/commit/464a9e0adc1f6ebed14b95e83a81b1e343f5465d"}]},"branch":"refs/heads/master"}},"requirements":[],"submit_records":[],"submit_requirements":[]}