)]}'
{"id":"openstack%2Fdevstack~529944","triplet_id":"openstack%2Fdevstack~master~Id0095763eb91592c2fd1a913526883987df704bd","project":"openstack/devstack","branch":"master","topic":"irrelevant","hashtags":[],"change_id":"Id0095763eb91592c2fd1a913526883987df704bd","subject":"devstack job: Add irrelevant-files","status":"MERGED","created":"2017-12-23 19:22:16.000000000","updated":"2018-01-12 09:58:48.000000000","submitted":"2018-01-12 09:58:48.000000000","submitter":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"total_comment_count":0,"unresolved_comment_count":0,"has_review_started":true,"submission_id":"529944-1515751128143-24c9093e","meta_rev_id":"84cae294dbad5912a79561793f8c1860b3d68f45","_number":529944,"virtual_id_number":529944,"owner":{"_account_id":6547,"name":"Andreas Jaeger","email":"jaegerandi@gmail.com","username":"jaegerandi"},"actions":{},"labels":{"Verified":{"approved":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"all":[{"value":0,"_account_id":11904,"name":"Sean McGinnis","email":"sean.mcginnis@gmail.com","username":"SeanM"},{"value":0,"date":"2017-12-23 21:39:35.000000000","_account_id":10385,"name":"Citrix XenServer CI","username":"citrix_xenserver_ci","tags":["SERVICE_USER"]},{"value":0,"_account_id":1921,"name":"Andrea Frittoli","email":"andrea.frittoli@gmail.com","username":"andrea-frittoli"},{"value":0,"date":"2017-12-23 19:50:39.000000000","_account_id":14595,"name":"z Systems KVM","email":"zkvm-ci@linux.vnet.ibm.com","username":"ibm-zkvm-ci","tags":["SERVICE_USER"]},{"value":0,"date":"2018-01-11 08:06:15.000000000","_account_id":6547,"name":"Andreas Jaeger","email":"jaegerandi@gmail.com","username":"jaegerandi"},{"value":0,"_account_id":1653,"name":"garyk","email":"gkotton@vmware.com","username":"garyk"},{"value":0,"date":"2017-12-23 20:00:06.000000000","permitted_voting_range":{"min":0,"max":1},"_account_id":10118,"name":"IBM PowerKVM CI","email":"kvmpower@linux.vnet.ibm.com","username":"powerkvm","tags":["SERVICE_USER"]},{"value":0,"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},{"value":2,"date":"2018-01-12 09:58:47.000000000","permitted_voting_range":{"min":2,"max":2},"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},{"value":0,"_account_id":7118,"name":"Ian Wienand","email":"iwienand@redhat.com","username":"iwienand"}],"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":1653,"name":"garyk","email":"gkotton@vmware.com","username":"garyk"},"recommended":{"_account_id":11904,"name":"Sean McGinnis","email":"sean.mcginnis@gmail.com","username":"SeanM"},"all":[{"value":1,"date":"2018-01-11 19:40:59.000000000","permitted_voting_range":{"min":1,"max":1},"_account_id":11904,"name":"Sean McGinnis","email":"sean.mcginnis@gmail.com","username":"SeanM"},{"value":0,"_account_id":10385,"name":"Citrix XenServer CI","username":"citrix_xenserver_ci","tags":["SERVICE_USER"]},{"value":0,"_account_id":1921,"name":"Andrea Frittoli","email":"andrea.frittoli@gmail.com","username":"andrea-frittoli"},{"value":0,"_account_id":14595,"name":"z Systems KVM","email":"zkvm-ci@linux.vnet.ibm.com","username":"ibm-zkvm-ci","tags":["SERVICE_USER"]},{"value":0,"_account_id":6547,"name":"Andreas Jaeger","email":"jaegerandi@gmail.com","username":"jaegerandi"},{"value":2,"date":"2018-01-01 15:08:11.000000000","_account_id":1653,"name":"garyk","email":"gkotton@vmware.com","username":"garyk"},{"value":0,"_account_id":10118,"name":"IBM PowerKVM CI","email":"kvmpower@linux.vnet.ibm.com","username":"powerkvm","tags":["SERVICE_USER"]},{"value":1,"date":"2018-01-11 07:57:59.000000000","permitted_voting_range":{"min":1,"max":2},"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},{"value":0,"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},{"value":2,"date":"2018-01-12 05:42:55.000000000","permitted_voting_range":{"min":2,"max":2},"_account_id":7118,"name":"Ian Wienand","email":"iwienand@redhat.com","username":"iwienand"}],"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":"","value":1,"default_value":0,"optional":true},"Workflow":{"approved":{"_account_id":7118,"name":"Ian Wienand","email":"iwienand@redhat.com","username":"iwienand"},"all":[{"value":0,"_account_id":11904,"name":"Sean McGinnis","email":"sean.mcginnis@gmail.com","username":"SeanM"},{"value":0,"_account_id":10385,"name":"Citrix XenServer CI","username":"citrix_xenserver_ci","tags":["SERVICE_USER"]},{"value":0,"_account_id":1921,"name":"Andrea Frittoli","email":"andrea.frittoli@gmail.com","username":"andrea-frittoli"},{"value":0,"_account_id":14595,"name":"z Systems KVM","email":"zkvm-ci@linux.vnet.ibm.com","username":"ibm-zkvm-ci","tags":["SERVICE_USER"]},{"value":0,"_account_id":6547,"name":"Andreas Jaeger","email":"jaegerandi@gmail.com","username":"jaegerandi"},{"value":0,"_account_id":1653,"name":"garyk","email":"gkotton@vmware.com","username":"garyk"},{"value":0,"_account_id":10118,"name":"IBM PowerKVM CI","email":"kvmpower@linux.vnet.ibm.com","username":"powerkvm","tags":["SERVICE_USER"]},{"value":0,"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},{"value":0,"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},{"value":1,"date":"2018-01-12 05:42:55.000000000","permitted_voting_range":{"min":1,"max":1},"_account_id":7118,"name":"Ian Wienand","email":"iwienand@redhat.com","username":"iwienand"}],"values":{"-1":"Work in progress"," 0":"Ready for reviews","+1":"Approved"},"description":"","default_value":0,"optional":true},"Review-Priority":{"all":[{"value":0,"_account_id":11904,"name":"Sean McGinnis","email":"sean.mcginnis@gmail.com","username":"SeanM"},{"value":0,"_account_id":10385,"name":"Citrix XenServer CI","username":"citrix_xenserver_ci","tags":["SERVICE_USER"]},{"value":0,"_account_id":1921,"name":"Andrea Frittoli","email":"andrea.frittoli@gmail.com","username":"andrea-frittoli"},{"value":0,"_account_id":14595,"name":"z Systems KVM","email":"zkvm-ci@linux.vnet.ibm.com","username":"ibm-zkvm-ci","tags":["SERVICE_USER"]},{"value":0,"_account_id":6547,"name":"Andreas Jaeger","email":"jaegerandi@gmail.com","username":"jaegerandi"},{"value":0,"_account_id":1653,"name":"garyk","email":"gkotton@vmware.com","username":"garyk"},{"value":0,"_account_id":10118,"name":"IBM PowerKVM CI","email":"kvmpower@linux.vnet.ibm.com","username":"powerkvm","tags":["SERVICE_USER"]},{"value":0,"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},{"value":0,"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},{"value":0,"_account_id":7118,"name":"Ian Wienand","email":"iwienand@redhat.com","username":"iwienand"}],"values":{"-1":"Branch Freeze"," 0":"No Priority","+1":"Important Change","+2":"Gate Blocker Fix / Urgent Change"},"description":"","default_value":0,"optional":true}},"removable_reviewers":[],"reviewers":{"REVIEWER":[{"_account_id":1653,"name":"garyk","email":"gkotton@vmware.com","username":"garyk"},{"_account_id":1921,"name":"Andrea Frittoli","email":"andrea.frittoli@gmail.com","username":"andrea-frittoli"},{"_account_id":6547,"name":"Andreas Jaeger","email":"jaegerandi@gmail.com","username":"jaegerandi"},{"_account_id":7118,"name":"Ian Wienand","email":"iwienand@redhat.com","username":"iwienand"},{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},{"_account_id":10118,"name":"IBM PowerKVM CI","email":"kvmpower@linux.vnet.ibm.com","username":"powerkvm","tags":["SERVICE_USER"]},{"_account_id":10385,"name":"Citrix XenServer CI","username":"citrix_xenserver_ci","tags":["SERVICE_USER"]},{"_account_id":11904,"name":"Sean McGinnis","email":"sean.mcginnis@gmail.com","username":"SeanM"},{"_account_id":14595,"name":"z Systems KVM","email":"zkvm-ci@linux.vnet.ibm.com","username":"ibm-zkvm-ci","tags":["SERVICE_USER"]},{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]}]},"pending_reviewers":{},"reviewer_updates":[{"updated":"2017-12-23 19:22:55.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":"2017-12-23 19:50:39.000000000","updated_by":{"_account_id":14595,"name":"z Systems KVM","email":"zkvm-ci@linux.vnet.ibm.com","username":"ibm-zkvm-ci","tags":["SERVICE_USER"]},"reviewer":{"_account_id":14595,"name":"z Systems KVM","email":"zkvm-ci@linux.vnet.ibm.com","username":"ibm-zkvm-ci","tags":["SERVICE_USER"]},"state":"REVIEWER"},{"updated":"2017-12-23 20:00:06.000000000","updated_by":{"_account_id":10118,"name":"IBM PowerKVM CI","email":"kvmpower@linux.vnet.ibm.com","username":"powerkvm","tags":["SERVICE_USER"]},"reviewer":{"_account_id":10118,"name":"IBM PowerKVM CI","email":"kvmpower@linux.vnet.ibm.com","username":"powerkvm","tags":["SERVICE_USER"]},"state":"REVIEWER"},{"updated":"2017-12-23 21:39:35.000000000","updated_by":{"_account_id":10385,"name":"Citrix XenServer CI","username":"citrix_xenserver_ci","tags":["SERVICE_USER"]},"reviewer":{"_account_id":10385,"name":"Citrix XenServer CI","username":"citrix_xenserver_ci","tags":["SERVICE_USER"]},"state":"REVIEWER"},{"updated":"2018-01-01 15:08:11.000000000","updated_by":{"_account_id":1653,"name":"garyk","email":"gkotton@vmware.com","username":"garyk"},"reviewer":{"_account_id":1653,"name":"garyk","email":"gkotton@vmware.com","username":"garyk"},"state":"REVIEWER"},{"updated":"2018-01-11 07:57:59.000000000","updated_by":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"reviewer":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"state":"REVIEWER"},{"updated":"2018-01-11 19:40:59.000000000","updated_by":{"_account_id":11904,"name":"Sean McGinnis","email":"sean.mcginnis@gmail.com","username":"SeanM"},"reviewer":{"_account_id":11904,"name":"Sean McGinnis","email":"sean.mcginnis@gmail.com","username":"SeanM"},"state":"REVIEWER"},{"updated":"2018-01-12 05:42:55.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":"2018-01-12 09:58:47.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":"7749d035c6bbdb6c9f51c644e42fe06b29fb2ba9","author":{"_account_id":6547,"name":"Andreas Jaeger","email":"jaegerandi@gmail.com","username":"jaegerandi"},"date":"2017-12-23 19:22:16.000000000","message":"Uploaded patch set 1.","accounts_in_message":[],"_revision_number":1},{"id":"768d8c1fee5d83ff0c5d3e772b3fad8e4b25ac3b","author":{"_account_id":14595,"name":"z Systems KVM","email":"zkvm-ci@linux.vnet.ibm.com","username":"ibm-zkvm-ci","tags":["SERVICE_USER"]},"date":"2017-12-23 19:23:28.000000000","message":"Patch Set 1:\n\nTesting failed ubuntu-xenial-s390x. For rechecking only on the ubuntu-xenial-s390x CI, add a review comment with recheck-zkvm. Contact info: zkvm-ci@linux.vnet.ibm.com. For more information, see https://wiki.openstack.org/wiki/ThirdPartySystems/IBM_zKVM_CI\n\n- check-tempest-dsvm-neutron-full-ubuntu-xenial-s390x http://sng01.objectstorage.softlayer.net/v1/AUTH_1940ea10-6e82-4501-b2f9-eb236510e575/ibmzkvmci/production/529944/1/check-tempest-dsvm-neutron-full-ubuntu-xenial-s390x/5f11ed9 : FAILURE in 34s","accounts_in_message":[],"_revision_number":1},{"id":"250c4e884d1510e1e5ff48a03c6ccbaaa18573bd","author":{"_account_id":10118,"name":"IBM PowerKVM CI","email":"kvmpower@linux.vnet.ibm.com","username":"powerkvm","tags":["SERVICE_USER"]},"date":"2017-12-23 19:28:01.000000000","message":"Patch Set 1:\n\nBuild failed. Test completed on IBM PowerKVM platform. For rechecking only on the IBM PowerKVM CI, add a review comment with pkvm: recheck. For contact and more information, see https://wiki.openstack.org/wiki/PowerKVM\n\n- tempest-dsvm-full-xenial https://dal05.objectstorage.softlayer.net/v1/AUTH_3d8e6ecb-f597-448c-8ec2-164e9f710dd6/pkvmci/devstack/44/529944/1/check/tempest-dsvm-full-xenial/675b655/ : FAILURE in 5m 29s","accounts_in_message":[],"_revision_number":1},{"id":"a47d460fa11cb5cf1618f7a73d47b64da5d2ea60","author":{"_account_id":6547,"name":"Andreas Jaeger","email":"jaegerandi@gmail.com","username":"jaegerandi"},"date":"2017-12-23 19:45:37.000000000","message":"Uploaded patch set 2: Commit message was updated.","accounts_in_message":[],"_revision_number":2},{"id":"1bcb724f2ec00d42a6751c73c3fd6f5160d7440e","author":{"_account_id":14595,"name":"z Systems KVM","email":"zkvm-ci@linux.vnet.ibm.com","username":"ibm-zkvm-ci","tags":["SERVICE_USER"]},"date":"2017-12-23 19:46:44.000000000","message":"Patch Set 2:\n\nTesting failed ubuntu-xenial-s390x. For rechecking only on the ubuntu-xenial-s390x CI, add a review comment with recheck-zkvm. Contact info: zkvm-ci@linux.vnet.ibm.com. For more information, see https://wiki.openstack.org/wiki/ThirdPartySystems/IBM_zKVM_CI\n\n- check-tempest-dsvm-neutron-full-ubuntu-xenial-s390x http://sng01.objectstorage.softlayer.net/v1/AUTH_1940ea10-6e82-4501-b2f9-eb236510e575/ibmzkvmci/production/529944/2/check-tempest-dsvm-neutron-full-ubuntu-xenial-s390x/1d5d224 : FAILURE in 34s","accounts_in_message":[],"_revision_number":2},{"id":"3e1cb59566a3c3b40963fa9add77c693c27e1371","author":{"_account_id":6547,"name":"Andreas Jaeger","email":"jaegerandi@gmail.com","username":"jaegerandi"},"date":"2017-12-23 19:49:38.000000000","message":"Uploaded patch set 3.","accounts_in_message":[],"_revision_number":3},{"id":"3bce210844cef779b6fe1430d2418129299d3293","author":{"_account_id":14595,"name":"z Systems KVM","email":"zkvm-ci@linux.vnet.ibm.com","username":"ibm-zkvm-ci","tags":["SERVICE_USER"]},"date":"2017-12-23 19:50:39.000000000","message":"Patch Set 3:\n\nTesting failed ubuntu-xenial-s390x. For rechecking only on the ubuntu-xenial-s390x CI, add a review comment with recheck-zkvm. Contact info: zkvm-ci@linux.vnet.ibm.com. For more information, see https://wiki.openstack.org/wiki/ThirdPartySystems/IBM_zKVM_CI\n\n- check-tempest-dsvm-neutron-full-ubuntu-xenial-s390x http://sng01.objectstorage.softlayer.net/v1/AUTH_1940ea10-6e82-4501-b2f9-eb236510e575/ibmzkvmci/production/529944/3/check-tempest-dsvm-neutron-full-ubuntu-xenial-s390x/9e306d2 : FAILURE in 33s","accounts_in_message":[],"_revision_number":3},{"id":"9d5aaebf2d25eae236a2d7d887a75c5524edde28","author":{"_account_id":10118,"name":"IBM PowerKVM CI","email":"kvmpower@linux.vnet.ibm.com","username":"powerkvm","tags":["SERVICE_USER"]},"date":"2017-12-23 20:00:06.000000000","message":"Patch Set 3:\n\nBuild failed. Test completed on IBM PowerKVM platform. For rechecking only on the IBM PowerKVM CI, add a review comment with pkvm: recheck. For contact and more information, see https://wiki.openstack.org/wiki/PowerKVM\n\n- tempest-dsvm-full-xenial https://dal05.objectstorage.softlayer.net/v1/AUTH_3d8e6ecb-f597-448c-8ec2-164e9f710dd6/pkvmci/devstack/44/529944/3/check/tempest-dsvm-full-xenial/2acfdce/ : FAILURE in 10m 10s","accounts_in_message":[],"_revision_number":3},{"id":"133f703716d9d94fd1d2f3a81df74615425f2da7","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2017-12-23 21:16:04.000000000","message":"Patch Set 3: Verified+1\n\nBuild succeeded (check pipeline).\n\n- tempest-full http://logs.openstack.org/44/529944/3/check/tempest-full/54db4ec/ : SUCCESS in 1h 24m 23s\n- legacy-grenade-dsvm-neutron http://logs.openstack.org/44/529944/3/check/legacy-grenade-dsvm-neutron/ad6d24f/ : SUCCESS in 48m 49s\n- legacy-tempest-dsvm-py35 http://logs.openstack.org/44/529944/3/check/legacy-tempest-dsvm-py35/136507a/ : SUCCESS in 1h 20m 31s\n- build-openstack-sphinx-docs http://logs.openstack.org/44/529944/3/check/build-openstack-sphinx-docs/1d9e35a/html/ : SUCCESS in 3m 45s\n- openstack-tox-bashate http://logs.openstack.org/44/529944/3/check/openstack-tox-bashate/e471f99/ : SUCCESS in 3m 20s\n- legacy-devstack-unit-tests http://logs.openstack.org/44/529944/3/check/legacy-devstack-unit-tests/8c9ec9e/ : SUCCESS in 3m 46s\n- ironic-tempest-dsvm-ipa-wholedisk-bios-agent_ipmitool-tinyipa http://logs.openstack.org/44/529944/3/check/ironic-tempest-dsvm-ipa-wholedisk-bios-agent_ipmitool-tinyipa/4e8473b/ : SUCCESS in 40m 20s (non-voting)\n- legacy-devstack-dsvm-updown http://logs.openstack.org/44/529944/3/check/legacy-devstack-dsvm-updown/c847a31/ : SUCCESS in 25m 18s\n- legacy-devstack-dsvm-py35-updown http://logs.openstack.org/44/529944/3/check/legacy-devstack-dsvm-py35-updown/203e8eb/ : SUCCESS in 24m 34s (non-voting)\n- legacy-swift-dsvm-functional http://logs.openstack.org/44/529944/3/check/legacy-swift-dsvm-functional/8ae723a/ : SUCCESS in 23m 54s (non-voting)\n- legacy-grenade-dsvm-neutron-multinode http://logs.openstack.org/44/529944/3/check/legacy-grenade-dsvm-neutron-multinode/2a4c441/ : SUCCESS in 55m 11s\n- legacy-tempest-dsvm-neutron-full-centos-7 http://logs.openstack.org/44/529944/3/check/legacy-tempest-dsvm-neutron-full-centos-7/c79ad55/ : FAILURE in 1h 24m 43s (non-voting)\n- legacy-tempest-dsvm-neutron-linuxbridge http://logs.openstack.org/44/529944/3/check/legacy-tempest-dsvm-neutron-linuxbridge/3f606b4/ : SUCCESS in 1h 18m 11s\n- legacy-tempest-dsvm-neutron-full-opensuse-423 http://logs.openstack.org/44/529944/3/check/legacy-tempest-dsvm-neutron-full-opensuse-423/13e5f7d/ : FAILURE in 1h 11m 42s (non-voting)\n- legacy-tempest-dsvm-neutron-multinode-full http://logs.openstack.org/44/529944/3/check/legacy-tempest-dsvm-neutron-multinode-full/408cfba/ : FAILURE in 1h 16m 02s (non-voting)\n- devstack http://logs.openstack.org/44/529944/3/check/devstack/3c5e427/ : SUCCESS in 22m 34s\n- devstack-multinode http://logs.openstack.org/44/529944/3/check/devstack-multinode/d69458c/ : FAILURE in 20m 07s (non-voting)","accounts_in_message":[],"_revision_number":3},{"id":"82a3f35956e55b7427fe841673330dfbcdc21f2d","author":{"_account_id":10385,"name":"Citrix XenServer CI","username":"citrix_xenserver_ci","tags":["SERVICE_USER"]},"date":"2017-12-23 21:39:35.000000000","message":"Patch Set 3:\n\nBuild succeeded.\n\n- dsvm-tempest-neutron-network http://dd6b71949550285df7dc-dda4e480e005aaa13ec303551d2d8155.r49.cf1.rackcdn.com/44/529944/3/check/dsvm-tempest-neutron-network/9f2dca7 : SUCCESS in 1h 49m 32s","accounts_in_message":[],"_revision_number":3},{"id":"2259831e589aa303c3302ca40b1e50097cce9676","author":{"_account_id":1653,"name":"garyk","email":"gkotton@vmware.com","username":"garyk"},"date":"2018-01-01 15:08:11.000000000","message":"Patch Set 3: Code-Review+2","accounts_in_message":[],"_revision_number":3},{"id":"8ac32a2632041271384c47ad39a11a7f560e6dd6","author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"date":"2018-01-11 01:10:36.000000000","message":"Patch Set 3: Code-Review-1\n\nThanks, this will avoid redefining the same set of files everywhere. \n\nI abandon the same kind of approach in https://review.openstack.org/#/c/525038/1, as per below issues and discussion with clark. If we resolve/clarify those then i like these changes. \n\n1. If other inheritted jobs define \u0027files\u0027 then, it leads to have \u0027files\u0027 and \u0027irrelevant-files\u0027 both in same job. is that ok?\n\n2. if other inherited jobs want to override irrelevant-files with some extra files then they have to override the complete set of files including define in parent job not just appending few files as irrelevant-files is not dict in zuul ?","accounts_in_message":[],"_revision_number":3},{"id":"1b98257aa6de74517a5bf0c211abe42c1cfd6c06","author":{"_account_id":6547,"name":"Andreas Jaeger","email":"jaegerandi@gmail.com","username":"jaegerandi"},"date":"2018-01-11 07:27:30.000000000","message":"Patch Set 3:\n\nGhanshyam,\n\nThis is exactly the same you do in https://review.openstack.org/#/c/531195/ .\n\nhttps://review.openstack.org/#/c/525038/1/.zuul.yaml was wrong since it would *only* run the job if roles, playbooks gets changed.\n\nRegarding your two items:\nAd 2): Any other repo can inherit from this one or add to the job directly in the project section its own irrelevant-files list and that overrides this one. This list is the default - and if you look at the referenced followup (see needed-by), it will make project-config much simpler.\n\nAd 1) Yes, mixing files and irrelevant-files is a challenge. You can have both, it\u0027s not an error per se but this is not tested and might open some interesting effects.\n\nSee the followup - do you have a better idea to simplify this?","accounts_in_message":[],"_revision_number":3},{"id":"955c35205be073c38a5bc3895df37f44c685f04e","author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"date":"2018-01-11 07:57:59.000000000","message":"Patch Set 3: Code-Review+1\n\n- https://review.openstack.org/#/c/531195/ this is integrated job not very base job and hardly anyone override from this.\n\n\u003eAd 2): .... it will make project-config much simpler.\n\ngmann - +1 on this and i agree the benefit of this with overriding the irrelevant-files if needed. \n\n\u003eAd 1) Yes, mixing files and irrelevant-files is a challenge. You \u003ecan have both, it\u0027s not an error per se but this is not tested \u003eand might open some interesting effects.\n\u003eSee the followup - do you have a better idea to simplify this?\n\ngmann- yea this is tricky and we do not know the behavior. 1 idea is to change in zuul logic - Give Preference \u0027files\u0027 over irrelevant-files if both are present. \n\nm +1 on this as it is not error and we can handle when we have any side effect.\n\n\u003eSee the followup\nyou have any followup patch fir testing the 1st case?","accounts_in_message":[],"_revision_number":3},{"id":"7ffe66e49c9614623d0b28009fb9add5f8857476","author":{"_account_id":6547,"name":"Andreas Jaeger","email":"jaegerandi@gmail.com","username":"jaegerandi"},"date":"2018-01-11 08:06:15.000000000","message":"Patch Set 3:\n\nThere\u0027s only the one followup mentioned here.","accounts_in_message":[],"_revision_number":3},{"id":"00696744482acec3a262132595a6ff4684de54d1","author":{"_account_id":11904,"name":"Sean McGinnis","email":"sean.mcginnis@gmail.com","username":"SeanM"},"date":"2018-01-11 19:40:59.000000000","message":"Patch Set 3: Code-Review+1","accounts_in_message":[],"_revision_number":3},{"id":"a8fd4e5903b6179a2c8ddeb52cbf1c61c49076a6","author":{"_account_id":7118,"name":"Ian Wienand","email":"iwienand@redhat.com","username":"iwienand"},"date":"2018-01-12 05:42:55.000000000","message":"Patch Set 3: Workflow+1 Code-Review+2\n\nnot sure we have translations, but this is good as we often get little typo fixes","accounts_in_message":[],"_revision_number":3},{"id":"baedcd39c20760d95b37e642dc7827e9b1b83820","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2018-01-12 06:15:15.000000000","message":"Patch Set 3: -Verified\n\nStarting gate jobs.","accounts_in_message":[],"_revision_number":3},{"id":"dec9243697e43acbf68779369771a0a93868e97f","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2018-01-12 09:58:47.000000000","message":"Patch Set 3: Verified+2\n\nBuild succeeded (gate pipeline).\n\n- tempest-full http://logs.openstack.org/44/529944/3/gate/tempest-full/1c61cdb/ : SUCCESS in 1h 23m 11s\n- legacy-grenade-dsvm-neutron http://logs.openstack.org/44/529944/3/gate/legacy-grenade-dsvm-neutron/eefaa76/ : SUCCESS in 45m 15s\n- legacy-tempest-dsvm-py35 http://logs.openstack.org/44/529944/3/gate/legacy-tempest-dsvm-py35/bcb7b58/ : SUCCESS in 1h 16m 04s\n- build-openstack-sphinx-docs http://logs.openstack.org/44/529944/3/gate/build-openstack-sphinx-docs/8ce6bd4/html/ : SUCCESS in 4m 10s\n- openstack-tox-bashate http://logs.openstack.org/44/529944/3/gate/openstack-tox-bashate/bfbee0d/ : SUCCESS in 3m 45s\n- legacy-grenade-dsvm-neutron-multinode http://logs.openstack.org/44/529944/3/gate/legacy-grenade-dsvm-neutron-multinode/959a2a5/ : SUCCESS in 59m 05s\n- legacy-tempest-dsvm-neutron-linuxbridge http://logs.openstack.org/44/529944/3/gate/legacy-tempest-dsvm-neutron-linuxbridge/771f74a/ : SUCCESS in 1h 52m 01s\n- devstack http://logs.openstack.org/44/529944/3/gate/devstack/d41dcae/ : SUCCESS in 26m 34s","accounts_in_message":[],"_revision_number":3},{"id":"0a4ae1c951f6b37c6340de6366c4e00c8ded9cb0","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2018-01-12 09:58:48.000000000","message":"Change has been successfully merged by Zuul","accounts_in_message":[],"_revision_number":3}],"current_revision_number":3,"current_revision":"b4d772b9fdb00fbdec1304ce6fbeafaa77915071","revisions":{"a05c239aabaabd3a444c2f8d98dc14b5058e30b6":{"kind":"REWORK","_number":1,"created":"2017-12-23 19:22:16.000000000","uploader":{"_account_id":6547,"name":"Andreas Jaeger","email":"jaegerandi@gmail.com","username":"jaegerandi"},"ref":"refs/changes/44/529944/1","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/devstack","ref":"refs/changes/44/529944/1","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/devstack refs/changes/44/529944/1 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/devstack refs/changes/44/529944/1 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/devstack refs/changes/44/529944/1 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/devstack refs/changes/44/529944/1"}}},"commit":{"parents":[{"commit":"c1700973886bce1ec212c788e28a6fa372329b2d","subject":"Merge \"Updated from generate-devstack-plugins-list\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/c1700973886bce1ec212c788e28a6fa372329b2d"}]}],"author":{"name":"Andreas Jaeger","email":"aj@suse.com","date":"2017-12-23 19:17:33.000000000","tz":60},"committer":{"name":"Andreas Jaeger","email":"aj@suse.com","date":"2017-12-23 19:21:54.000000000","tz":60},"subject":"devstack job: Add irrelevant-files","message":"devstack job: Add irrelevant-files\n\nDo not run devstack and its siblings if only rst files, releasenotes, or\nfiles in doc directory change.\n\nThis is the minimal set of irrelevant files shared by most projects\nalready.\n\nChange-Id: Id0095763eb91592c2fd1a913526883987df704bd\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/a05c239aabaabd3a444c2f8d98dc14b5058e30b6"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/a05c239aabaabd3a444c2f8d98dc14b5058e30b6"}]},"branch":"refs/heads/master"},"a73db2c39ebe832f99cec74f5db65b6b95015258":{"kind":"NO_CODE_CHANGE","_number":2,"created":"2017-12-23 19:45:37.000000000","uploader":{"_account_id":6547,"name":"Andreas Jaeger","email":"jaegerandi@gmail.com","username":"jaegerandi"},"ref":"refs/changes/44/529944/2","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/devstack","ref":"refs/changes/44/529944/2","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/devstack refs/changes/44/529944/2 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/devstack refs/changes/44/529944/2 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/devstack refs/changes/44/529944/2 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/devstack refs/changes/44/529944/2"}}},"commit":{"parents":[{"commit":"c1700973886bce1ec212c788e28a6fa372329b2d","subject":"Merge \"Updated from generate-devstack-plugins-list\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/c1700973886bce1ec212c788e28a6fa372329b2d"}]}],"author":{"name":"Andreas Jaeger","email":"aj@suse.com","date":"2017-12-23 19:17:33.000000000","tz":60},"committer":{"name":"Andreas Jaeger","email":"aj@suse.com","date":"2017-12-23 19:43:56.000000000","tz":60},"subject":"devstack job: Add irrelevant-files","message":"devstack job: Add irrelevant-files\n\nDo not run devstack and its siblings if only rst files, releasenotes, or\nfiles in doc directory change.\n\nThis is the minimal set of irrelevant files shared by most projects\nalready.\n\nNeeded-By: Ie8504ba3d5d46f6338a228ed2d248ba6363e37ae\nChange-Id: Id0095763eb91592c2fd1a913526883987df704bd\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/a73db2c39ebe832f99cec74f5db65b6b95015258"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/a73db2c39ebe832f99cec74f5db65b6b95015258"}]},"branch":"refs/heads/master"},"b4d772b9fdb00fbdec1304ce6fbeafaa77915071":{"kind":"REWORK","_number":3,"created":"2017-12-23 19:49:38.000000000","uploader":{"_account_id":6547,"name":"Andreas Jaeger","email":"jaegerandi@gmail.com","username":"jaegerandi"},"ref":"refs/changes/44/529944/3","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/devstack","ref":"refs/changes/44/529944/3","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/devstack refs/changes/44/529944/3 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/devstack refs/changes/44/529944/3 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/devstack refs/changes/44/529944/3 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/devstack refs/changes/44/529944/3"}}},"commit":{"parents":[{"commit":"c1700973886bce1ec212c788e28a6fa372329b2d","subject":"Merge \"Updated from generate-devstack-plugins-list\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/c1700973886bce1ec212c788e28a6fa372329b2d"}]}],"author":{"name":"Andreas Jaeger","email":"aj@suse.com","date":"2017-12-23 19:17:33.000000000","tz":60},"committer":{"name":"Andreas Jaeger","email":"aj@suse.com","date":"2017-12-23 19:49:15.000000000","tz":60},"subject":"devstack job: Add irrelevant-files","message":"devstack job: Add irrelevant-files\n\nDo not run devstack and its siblings if only rst files, releasenotes, or\nfiles in doc directory change.\n\nThis is the minimal set of irrelevant files shared by most projects\nalready.\n\nNeeded-By: Ie8504ba3d5d46f6338a228ed2d248ba6363e37ae\nChange-Id: Id0095763eb91592c2fd1a913526883987df704bd\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/b4d772b9fdb00fbdec1304ce6fbeafaa77915071"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/b4d772b9fdb00fbdec1304ce6fbeafaa77915071"}]},"branch":"refs/heads/master"}},"requirements":[],"submit_records":[],"submit_requirements":[]}
