)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":18893,"name":"Miles Gould","email":"mgould@redhat.com","username":"pozorvlak"},"change_message_id":"c98f7e8bcd558f0f749752ac979ba78e7bfca223","unresolved":false,"context_lines":[{"line_number":4,"context_line":"Commit:     Lucas Alvares Gomes \u003clucasagomes@gmail.com\u003e"},{"line_number":5,"context_line":"CommitDate: 2016-11-02 17:09:47 +0000"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Switch ironic-ipa jobs to Xenial"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Change-Id: I7812d23c11ed92859cb59d32ddaac770ea610449"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"ba5da102_763e1729","line":7,"updated":"2016-11-02 17:16:27.000000000","message":"I think this could be a little clearer: \"to Ubuntu 16.04 (Xenial)\", perhaps.","commit_id":"7f2738c4391c018a150e23e870263eaa53f3524a"}],"jenkins/jobs/ironic.yaml":[{"author":{"_account_id":10239,"name":"Dmitry Tantsur","email":"dtantsur@protonmail.com","username":"dtantsur"},"change_message_id":"9bfae1c4525213e67bd5ec840c6bd7b398ffd62c","unresolved":false,"context_lines":[{"line_number":558,"context_line":"      - console-log"},{"line_number":559,"context_line":""},{"line_number":560,"context_line":"- job-template:"},{"line_number":561,"context_line":"    name: \u0027{pipeline}-tempest-dsvm-ironic-ipa-{image-type}-{driver}-{ramdisk-type}-{node}{src}{job-suffix}\u0027"},{"line_number":562,"context_line":"    node: \u0027{node}\u0027"},{"line_number":563,"context_line":""},{"line_number":564,"context_line":"    wrappers:"}],"source_content_type":"text/x-yaml","patch_set":4,"id":"ba5da102_f380204c","line":561,"range":{"start_line":561,"start_character":82,"end_line":561,"end_character":89},"updated":"2016-11-04 14:56:49.000000000","message":"{node} is a generic thing, so it should go after {src} (which is ironic-specific)","commit_id":"04ad9e3f6e7098cad169e67654f7f03e13ab9d4d"},{"author":{"_account_id":6773,"name":"Lucas Alvares Gomes","email":"lucasagomes@gmail.com","username":"lucasagomes"},"change_message_id":"ad84e2fdb64e484ca70e31aef3ebe584f73a4064","unresolved":false,"context_lines":[{"line_number":558,"context_line":"      - console-log"},{"line_number":559,"context_line":""},{"line_number":560,"context_line":"- job-template:"},{"line_number":561,"context_line":"    name: \u0027{pipeline}-tempest-dsvm-ironic-ipa-{image-type}-{driver}-{ramdisk-type}-{node}{src}{job-suffix}\u0027"},{"line_number":562,"context_line":"    node: \u0027{node}\u0027"},{"line_number":563,"context_line":""},{"line_number":564,"context_line":"    wrappers:"}],"source_content_type":"text/x-yaml","patch_set":4,"id":"ba5da102_defde309","line":561,"range":{"start_line":561,"start_character":82,"end_line":561,"end_character":89},"in_reply_to":"ba5da102_f380204c","updated":"2016-11-04 15:07:20.000000000","message":"Done","commit_id":"04ad9e3f6e7098cad169e67654f7f03e13ab9d4d"}],"jenkins/jobs/projects.yaml":[{"author":{"_account_id":10239,"name":"Dmitry Tantsur","email":"dtantsur@protonmail.com","username":"dtantsur"},"change_message_id":"0b90a82196b15712e3cedae324491c1497e59c3f","unresolved":false,"context_lines":[{"line_number":5558,"context_line":"          suffix: \u0027\u0027"},{"line_number":5559,"context_line":"      - api-ref-jobs:"},{"line_number":5560,"context_line":"          service: baremetal"},{"line_number":5561,"context_line":"      - \u0027{pipeline}-tempest-dsvm-ironic-ipa-{image-type}-{driver}-{ramdisk-type}{src}{job-suffix}\u0027:"},{"line_number":5562,"context_line":"          pipeline: gate"},{"line_number":5563,"context_line":"          node: ubuntu-xenial"},{"line_number":5564,"context_line":"          branch-override: default"},{"line_number":5565,"context_line":"      - \u0027{pipeline}-tempest-dsvm-ironic-ipa-{image-type}-{driver}-{ramdisk-type}{src}{job-suffix}\u0027:"},{"line_number":5566,"context_line":"          pipeline: gate"},{"line_number":5567,"context_line":"          node: ubuntu-xenial"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"ba5da102_7ce97fb6","line":5564,"range":{"start_line":5561,"start_character":0,"end_line":5564,"end_character":34},"updated":"2016-11-03 09:39:08.000000000","message":"let\u0027s limit our effort to only this job for now. I think we can parametrize it by node:\n\n - \u0027{pipeline}-tempest-dsvm-ironic-ipa-{image-type}-{driver}-{ramdisk-type}{src}{job-suffix}\u0027:\n     pipeline: gate\n     node:\n         - ubuntu-trusty\n         - ubuntu-xenial\n     branch-override: default\n\n(I\u0027m not entirely sure if it\u0027s ok to leave \"node\" here or it should be moved above - something to experiment with).\n\nthen in the \"exclude\" above we can probably remove xenial jobs for ssh and trusty jobs for ipmitool, like\n\n exclude:\n     - driver: \u0027pxe_ipmitool\u0027\n       node: \u0027ubuntu-trusty\u0027\n     - driver: \u0027pxe_ssh\u0027\n       node: \u0027ubuntu-xenial\u0027","commit_id":"7f2738c4391c018a150e23e870263eaa53f3524a"},{"author":{"_account_id":10239,"name":"Dmitry Tantsur","email":"dtantsur@protonmail.com","username":"dtantsur"},"change_message_id":"a9fa9774b7f6403d9cce24c7625bd06a8c488998","unresolved":false,"context_lines":[{"line_number":5558,"context_line":"          suffix: \u0027\u0027"},{"line_number":5559,"context_line":"      - api-ref-jobs:"},{"line_number":5560,"context_line":"          service: baremetal"},{"line_number":5561,"context_line":"      - \u0027{pipeline}-tempest-dsvm-ironic-ipa-{image-type}-{driver}-{ramdisk-type}{src}{job-suffix}\u0027:"},{"line_number":5562,"context_line":"          pipeline: gate"},{"line_number":5563,"context_line":"          node: ubuntu-xenial"},{"line_number":5564,"context_line":"          branch-override: default"},{"line_number":5565,"context_line":"      - \u0027{pipeline}-tempest-dsvm-ironic-ipa-{image-type}-{driver}-{ramdisk-type}{src}{job-suffix}\u0027:"},{"line_number":5566,"context_line":"          pipeline: gate"},{"line_number":5567,"context_line":"          node: ubuntu-xenial"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"ba5da102_f78da8d3","line":5564,"range":{"start_line":5561,"start_character":0,"end_line":5564,"end_character":34},"in_reply_to":"ba5da102_177cdcea","updated":"2016-11-03 10:12:24.000000000","message":"\u003e I probably should add -{node}- to the templates name\n\nIt won\u0027t hurt for sure, even though I think it\u0027s not mandatory.","commit_id":"7f2738c4391c018a150e23e870263eaa53f3524a"},{"author":{"_account_id":6773,"name":"Lucas Alvares Gomes","email":"lucasagomes@gmail.com","username":"lucasagomes"},"change_message_id":"a4de80c47072661aefc18ad628f87f33169e993d","unresolved":false,"context_lines":[{"line_number":5558,"context_line":"          suffix: \u0027\u0027"},{"line_number":5559,"context_line":"      - api-ref-jobs:"},{"line_number":5560,"context_line":"          service: baremetal"},{"line_number":5561,"context_line":"      - \u0027{pipeline}-tempest-dsvm-ironic-ipa-{image-type}-{driver}-{ramdisk-type}{src}{job-suffix}\u0027:"},{"line_number":5562,"context_line":"          pipeline: gate"},{"line_number":5563,"context_line":"          node: ubuntu-xenial"},{"line_number":5564,"context_line":"          branch-override: default"},{"line_number":5565,"context_line":"      - \u0027{pipeline}-tempest-dsvm-ironic-ipa-{image-type}-{driver}-{ramdisk-type}{src}{job-suffix}\u0027:"},{"line_number":5566,"context_line":"          pipeline: gate"},{"line_number":5567,"context_line":"          node: ubuntu-xenial"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"ba5da102_177cdcea","line":5564,"range":{"start_line":5561,"start_character":0,"end_line":5564,"end_character":34},"in_reply_to":"ba5da102_7ce97fb6","updated":"2016-11-03 10:09:06.000000000","message":"Thanks for the snippets, I\u0027m going to try it out.\n\n(I probably should add -{node}- to the templates name, no ?)","commit_id":"7f2738c4391c018a150e23e870263eaa53f3524a"},{"author":{"_account_id":6773,"name":"Lucas Alvares Gomes","email":"lucasagomes@gmail.com","username":"lucasagomes"},"change_message_id":"0ff3d3aee9da8532e54a75a0bc7738ff67f9bacf","unresolved":false,"context_lines":[{"line_number":5572,"context_line":"          branch-override: default"},{"line_number":5573,"context_line":"      - \u0027{pipeline}-tempest-dsvm-ironic-ipa-{image-type}-{driver}-{ramdisk-type}{src}-{node}{job-suffix}\u0027:"},{"line_number":5574,"context_line":"          pipeline: gate"},{"line_number":5575,"context_line":"          node: ubuntu-trusty"},{"line_number":5576,"context_line":"          branch-override: default"},{"line_number":5577,"context_line":"          driver: \u0027pxe_ssh\u0027"},{"line_number":5578,"context_line":"          job-suffix: \u0027\u0027"}],"source_content_type":"text/x-yaml","patch_set":6,"id":"9a629dbe_6c4effc2","line":5575,"updated":"2016-11-07 11:32:53.000000000","message":"I wonder if I should update this job to xenial + ipmitool. It runs as an experimental job for now","commit_id":"29729d7144cfa9fa804d9eef49941b4e5767aaeb"},{"author":{"_account_id":6773,"name":"Lucas Alvares Gomes","email":"lucasagomes@gmail.com","username":"lucasagomes"},"change_message_id":"14a499342a7ca7776f94c6a88bf5cc816a5f1db8","unresolved":false,"context_lines":[{"line_number":5572,"context_line":"          branch-override: default"},{"line_number":5573,"context_line":"      - \u0027{pipeline}-tempest-dsvm-ironic-ipa-{image-type}-{driver}-{ramdisk-type}{src}-{node}{job-suffix}\u0027:"},{"line_number":5574,"context_line":"          pipeline: gate"},{"line_number":5575,"context_line":"          node: ubuntu-trusty"},{"line_number":5576,"context_line":"          branch-override: default"},{"line_number":5577,"context_line":"          driver: \u0027pxe_ssh\u0027"},{"line_number":5578,"context_line":"          job-suffix: \u0027\u0027"}],"source_content_type":"text/x-yaml","patch_set":6,"id":"9a629dbe_d8d14efd","line":5575,"in_reply_to":"9a629dbe_023d192b","updated":"2016-11-08 12:11:19.000000000","message":"Ok, talked to sam on IRC and he still see value on this job. So I\u0027m just going to update it to use xenial + ipmitool.\n\nLogs:\n\n\u003clucasagomes\u003e sambetts, around ? Do we still need this tinyipa256 job here: https://github.com/openstack-infra/project-config/blob/master/jenkins/jobs/projects.yaml#L5574-L5581 ?\n\u003clucasagomes\u003e sambetts, it runs on experimental so I\u0027m wondering whether I should also switch it to use ipmitool + xenial or simple get rid of it\n\u003clucasagomes\u003e (if not used anymore)\n\u003csambetts\u003e lucasagomes: that job was around as an experimental to be able to check tinyipa against 256mb of RAM with the goal that we would eventually be able to reduce all the tinyipa jobs to 256mb of RAM\n\u003csambetts\u003e per VM\n\u003clucasagomes\u003e sambetts, right, so the job is still around\n\u003clucasagomes\u003e in the experimetal gate as you said... Should we keep it there (but updated to use ipmitool and xenial) or you don\u0027t see any value on it anymore and we can delete it ?\n\u003csambetts\u003e I still see some value in it at the moment, it would be nice to has around as we improve our CI if we can use it to get the jobs running more VMs\n\u003clucasagomes\u003e sambetts, right ok, I will just update it then \n\u003csambetts\u003e awesome thanks :)","commit_id":"29729d7144cfa9fa804d9eef49941b4e5767aaeb"},{"author":{"_account_id":10239,"name":"Dmitry Tantsur","email":"dtantsur@protonmail.com","username":"dtantsur"},"change_message_id":"1ead4b5e9a12367af7163ffed564c41e1d803b53","unresolved":false,"context_lines":[{"line_number":5572,"context_line":"          branch-override: default"},{"line_number":5573,"context_line":"      - \u0027{pipeline}-tempest-dsvm-ironic-ipa-{image-type}-{driver}-{ramdisk-type}{src}-{node}{job-suffix}\u0027:"},{"line_number":5574,"context_line":"          pipeline: gate"},{"line_number":5575,"context_line":"          node: ubuntu-trusty"},{"line_number":5576,"context_line":"          branch-override: default"},{"line_number":5577,"context_line":"          driver: \u0027pxe_ssh\u0027"},{"line_number":5578,"context_line":"          job-suffix: \u0027\u0027"}],"source_content_type":"text/x-yaml","patch_set":6,"id":"9a629dbe_e24bc529","line":5575,"in_reply_to":"9a629dbe_241ead63","updated":"2016-11-08 11:28:21.000000000","message":"I\u0027m not sure we even need this job.. We should talk to Sam about it, I suspect he introduced it.","commit_id":"29729d7144cfa9fa804d9eef49941b4e5767aaeb"},{"author":{"_account_id":6773,"name":"Lucas Alvares Gomes","email":"lucasagomes@gmail.com","username":"lucasagomes"},"change_message_id":"a067014e21d9622898d8387b2464e2d188e597a6","unresolved":false,"context_lines":[{"line_number":5572,"context_line":"          branch-override: default"},{"line_number":5573,"context_line":"      - \u0027{pipeline}-tempest-dsvm-ironic-ipa-{image-type}-{driver}-{ramdisk-type}{src}-{node}{job-suffix}\u0027:"},{"line_number":5574,"context_line":"          pipeline: gate"},{"line_number":5575,"context_line":"          node: ubuntu-trusty"},{"line_number":5576,"context_line":"          branch-override: default"},{"line_number":5577,"context_line":"          driver: \u0027pxe_ssh\u0027"},{"line_number":5578,"context_line":"          job-suffix: \u0027\u0027"}],"source_content_type":"text/x-yaml","patch_set":6,"id":"9a629dbe_241ead63","line":5575,"in_reply_to":"9a629dbe_63e44d0d","updated":"2016-11-08 10:57:00.000000000","message":"Cool, I will put a patch up on top of this one for it.","commit_id":"29729d7144cfa9fa804d9eef49941b4e5767aaeb"},{"author":{"_account_id":6547,"name":"Andreas Jaeger","email":"jaegerandi@gmail.com","username":"jaegerandi"},"change_message_id":"a6fbd4d989e1cf41a7fafaafe53e1c867b563e72","unresolved":false,"context_lines":[{"line_number":5572,"context_line":"          branch-override: default"},{"line_number":5573,"context_line":"      - \u0027{pipeline}-tempest-dsvm-ironic-ipa-{image-type}-{driver}-{ramdisk-type}{src}-{node}{job-suffix}\u0027:"},{"line_number":5574,"context_line":"          pipeline: gate"},{"line_number":5575,"context_line":"          node: ubuntu-trusty"},{"line_number":5576,"context_line":"          branch-override: default"},{"line_number":5577,"context_line":"          driver: \u0027pxe_ssh\u0027"},{"line_number":5578,"context_line":"          job-suffix: \u0027\u0027"}],"source_content_type":"text/x-yaml","patch_set":6,"id":"9a629dbe_63e44d0d","line":5575,"in_reply_to":"9a629dbe_6c4effc2","updated":"2016-11-07 12:04:21.000000000","message":"Please do ;) Can be separate change","commit_id":"29729d7144cfa9fa804d9eef49941b4e5767aaeb"},{"author":{"_account_id":6773,"name":"Lucas Alvares Gomes","email":"lucasagomes@gmail.com","username":"lucasagomes"},"change_message_id":"494fc9b86615812a7218a314aff36f395809d645","unresolved":false,"context_lines":[{"line_number":5572,"context_line":"          branch-override: default"},{"line_number":5573,"context_line":"      - \u0027{pipeline}-tempest-dsvm-ironic-ipa-{image-type}-{driver}-{ramdisk-type}{src}-{node}{job-suffix}\u0027:"},{"line_number":5574,"context_line":"          pipeline: gate"},{"line_number":5575,"context_line":"          node: ubuntu-trusty"},{"line_number":5576,"context_line":"          branch-override: default"},{"line_number":5577,"context_line":"          driver: \u0027pxe_ssh\u0027"},{"line_number":5578,"context_line":"          job-suffix: \u0027\u0027"}],"source_content_type":"text/x-yaml","patch_set":6,"id":"9a629dbe_023d192b","line":5575,"in_reply_to":"9a629dbe_e24bc529","updated":"2016-11-08 11:39:39.000000000","message":"I will check with him. But I would prefer to leave this as-is in this patch and remove/update later.","commit_id":"29729d7144cfa9fa804d9eef49941b4e5767aaeb"},{"author":{"_account_id":10239,"name":"Dmitry Tantsur","email":"dtantsur@protonmail.com","username":"dtantsur"},"change_message_id":"2e5c5d4a6b81877804eed372c987e4c5e5458524","unresolved":false,"context_lines":[{"line_number":5516,"context_line":"      - image-type: \u0027partition\u0027"},{"line_number":5517,"context_line":"        driver: \u0027agent_ssh\u0027"},{"line_number":5518,"context_line":"        job-suffix: \u0027\u0027"},{"line_number":5519,"context_line":"      #- image-type: \u0027partition\u0027"},{"line_number":5520,"context_line":"      #  driver: \u0027agent_ipmitool\u0027"},{"line_number":5521,"context_line":"      # pxe_* drivers with whole disk images are experimental"},{"line_number":5522,"context_line":"      - image-type: \u0027wholedisk\u0027"},{"line_number":5523,"context_line":"        driver: \u0027pxe_ssh\u0027"}],"source_content_type":"text/x-yaml","patch_set":7,"id":"9a629dbe_37701ddc","line":5520,"range":{"start_line":5519,"start_character":0,"end_line":5520,"end_character":33},"updated":"2016-11-10 16:39:36.000000000","message":"we should undo this eventually, I guess","commit_id":"b59f8e9479b156ad762b616f04f50f484fda885c"},{"author":{"_account_id":10239,"name":"Dmitry Tantsur","email":"dtantsur@protonmail.com","username":"dtantsur"},"change_message_id":"2e5c5d4a6b81877804eed372c987e4c5e5458524","unresolved":false,"context_lines":[{"line_number":5570,"context_line":"            - ubuntu-trusty"},{"line_number":5571,"context_line":"            - ubuntu-xenial"},{"line_number":5572,"context_line":"          branch-override: default"},{"line_number":5573,"context_line":"      - \u0027{pipeline}-tempest-dsvm-ironic-ipa-{image-type}-{driver}-{ramdisk-type}{src}-{node}{job-suffix}\u0027:"},{"line_number":5574,"context_line":"          pipeline: gate"},{"line_number":5575,"context_line":"          node: ubuntu-trusty"},{"line_number":5576,"context_line":"          branch-override: default"},{"line_number":5577,"context_line":"          driver: \u0027pxe_ssh\u0027"},{"line_number":5578,"context_line":"          job-suffix: \u0027\u0027"},{"line_number":5579,"context_line":"          ramdisk-type: \u0027tinyipa256\u0027"},{"line_number":5580,"context_line":"          src: \u0027-src\u0027"},{"line_number":5581,"context_line":"      - \u0027{pipeline}-tempest-dsvm-ironic-ipa-{image-type}-{driver}-{ramdisk-type}{src}-multinode{job-suffix}\u0027:"},{"line_number":5582,"context_line":"          pipeline: gate"},{"line_number":5583,"context_line":"          node: ubuntu-trusty-2-node"}],"source_content_type":"text/x-yaml","patch_set":7,"id":"9a629dbe_d200cf03","line":5580,"range":{"start_line":5573,"start_character":0,"end_line":5580,"end_character":21},"updated":"2016-11-10 16:39:36.000000000","message":"this job won\u0027t run on master any more, we have to switch it to xenial for that","commit_id":"b59f8e9479b156ad762b616f04f50f484fda885c"},{"author":{"_account_id":6773,"name":"Lucas Alvares Gomes","email":"lucasagomes@gmail.com","username":"lucasagomes"},"change_message_id":"5ed0fdd3a0e085a29367a82298e3b0281edc5180","unresolved":false,"context_lines":[{"line_number":5570,"context_line":"            - ubuntu-trusty"},{"line_number":5571,"context_line":"            - ubuntu-xenial"},{"line_number":5572,"context_line":"          branch-override: default"},{"line_number":5573,"context_line":"      - \u0027{pipeline}-tempest-dsvm-ironic-ipa-{image-type}-{driver}-{ramdisk-type}{src}-{node}{job-suffix}\u0027:"},{"line_number":5574,"context_line":"          pipeline: gate"},{"line_number":5575,"context_line":"          node: ubuntu-trusty"},{"line_number":5576,"context_line":"          branch-override: default"},{"line_number":5577,"context_line":"          driver: \u0027pxe_ssh\u0027"},{"line_number":5578,"context_line":"          job-suffix: \u0027\u0027"},{"line_number":5579,"context_line":"          ramdisk-type: \u0027tinyipa256\u0027"},{"line_number":5580,"context_line":"          src: \u0027-src\u0027"},{"line_number":5581,"context_line":"      - \u0027{pipeline}-tempest-dsvm-ironic-ipa-{image-type}-{driver}-{ramdisk-type}{src}-multinode{job-suffix}\u0027:"},{"line_number":5582,"context_line":"          pipeline: gate"},{"line_number":5583,"context_line":"          node: ubuntu-trusty-2-node"}],"source_content_type":"text/x-yaml","patch_set":7,"id":"9a629dbe_12a32701","line":5580,"range":{"start_line":5573,"start_character":0,"end_line":5580,"end_character":21},"in_reply_to":"9a629dbe_d200cf03","updated":"2016-11-10 16:41:11.000000000","message":"Right, I added a patch for it on top of this one: https://review.openstack.org/#/c/394927/1","commit_id":"b59f8e9479b156ad762b616f04f50f484fda885c"}],"zuul/layout.yaml":[{"author":{"_account_id":10239,"name":"Dmitry Tantsur","email":"dtantsur@protonmail.com","username":"dtantsur"},"change_message_id":"9bfae1c4525213e67bd5ec840c6bd7b398ffd62c","unresolved":false,"context_lines":[{"line_number":1959,"context_line":"  - name: ^gate-tempest-dsvm-ironic-parallel-nv$"},{"line_number":1960,"context_line":"    branch: ^(?!stable)"},{"line_number":1961,"context_line":""},{"line_number":1962,"context_line":"  - name: ^gate-tempest-dsvm-ironic-ipa-partition-pxe_ipmitool-tinyipa-ubuntu-xenial-nv$"},{"line_number":1963,"context_line":"    branch: ^(?!stable/liberty).*$"},{"line_number":1964,"context_line":""},{"line_number":1965,"context_line":"  - name: ^gate-tempest-dsvm-ironic-pxe_ipmitool-pyghmi-src-nv$"},{"line_number":1966,"context_line":"    branch: ^(?!stable/liberty).*$"}],"source_content_type":"text/x-yaml","patch_set":4,"id":"ba5da102_53f70cf5","line":1963,"range":{"start_line":1962,"start_character":0,"end_line":1963,"end_character":34},"updated":"2016-11-04 14:56:49.000000000","message":"somewhere in this file there is a guard in place to ensure nothing with -ubuntu-xenial postfix runs on liberty/mitaka. This is redundant.","commit_id":"04ad9e3f6e7098cad169e67654f7f03e13ab9d4d"},{"author":{"_account_id":6773,"name":"Lucas Alvares Gomes","email":"lucasagomes@gmail.com","username":"lucasagomes"},"change_message_id":"ad84e2fdb64e484ca70e31aef3ebe584f73a4064","unresolved":false,"context_lines":[{"line_number":1959,"context_line":"  - name: ^gate-tempest-dsvm-ironic-parallel-nv$"},{"line_number":1960,"context_line":"    branch: ^(?!stable)"},{"line_number":1961,"context_line":""},{"line_number":1962,"context_line":"  - name: ^gate-tempest-dsvm-ironic-ipa-partition-pxe_ipmitool-tinyipa-ubuntu-xenial-nv$"},{"line_number":1963,"context_line":"    branch: ^(?!stable/liberty).*$"},{"line_number":1964,"context_line":""},{"line_number":1965,"context_line":"  - name: ^gate-tempest-dsvm-ironic-pxe_ipmitool-pyghmi-src-nv$"},{"line_number":1966,"context_line":"    branch: ^(?!stable/liberty).*$"}],"source_content_type":"text/x-yaml","patch_set":4,"id":"ba5da102_3e0dff37","line":1963,"range":{"start_line":1962,"start_character":0,"end_line":1963,"end_character":34},"in_reply_to":"ba5da102_53f70cf5","updated":"2016-11-04 15:07:20.000000000","message":"Done","commit_id":"04ad9e3f6e7098cad169e67654f7f03e13ab9d4d"},{"author":{"_account_id":10239,"name":"Dmitry Tantsur","email":"dtantsur@protonmail.com","username":"dtantsur"},"change_message_id":"9bfae1c4525213e67bd5ec840c6bd7b398ffd62c","unresolved":false,"context_lines":[{"line_number":1968,"context_line":"  - name: ^gate-tempest-dsvm-ironic-pxe_ipmitool-virtualbmc-src-nv$"},{"line_number":1969,"context_line":"    branch: ^(?!stable/liberty).*$"},{"line_number":1970,"context_line":""},{"line_number":1971,"context_line":"  - name: ^gate-tempest-dsvm-ironic-ipa-wholedisk-agent_ipmitool-tinyipa-ubuntu-xenial-nv$"},{"line_number":1972,"context_line":"    branch: ^(?!stable/liberty).*$"},{"line_number":1973,"context_line":""},{"line_number":1974,"context_line":"  - name: ^gate-ironic-python-agent-tox-func-ubuntu-(trusty|xenial)$"},{"line_number":1975,"context_line":"    voting: false"}],"source_content_type":"text/x-yaml","patch_set":4,"id":"ba5da102_13d4b445","line":1972,"range":{"start_line":1971,"start_character":0,"end_line":1972,"end_character":34},"updated":"2016-11-04 14:56:49.000000000","message":"ditto","commit_id":"04ad9e3f6e7098cad169e67654f7f03e13ab9d4d"},{"author":{"_account_id":6773,"name":"Lucas Alvares Gomes","email":"lucasagomes@gmail.com","username":"lucasagomes"},"change_message_id":"ad84e2fdb64e484ca70e31aef3ebe584f73a4064","unresolved":false,"context_lines":[{"line_number":1968,"context_line":"  - name: ^gate-tempest-dsvm-ironic-pxe_ipmitool-virtualbmc-src-nv$"},{"line_number":1969,"context_line":"    branch: ^(?!stable/liberty).*$"},{"line_number":1970,"context_line":""},{"line_number":1971,"context_line":"  - name: ^gate-tempest-dsvm-ironic-ipa-wholedisk-agent_ipmitool-tinyipa-ubuntu-xenial-nv$"},{"line_number":1972,"context_line":"    branch: ^(?!stable/liberty).*$"},{"line_number":1973,"context_line":""},{"line_number":1974,"context_line":"  - name: ^gate-ironic-python-agent-tox-func-ubuntu-(trusty|xenial)$"},{"line_number":1975,"context_line":"    voting: false"}],"source_content_type":"text/x-yaml","patch_set":4,"id":"ba5da102_5e12b359","line":1972,"range":{"start_line":1971,"start_character":0,"end_line":1972,"end_character":34},"in_reply_to":"ba5da102_13d4b445","updated":"2016-11-04 15:07:20.000000000","message":"Done","commit_id":"04ad9e3f6e7098cad169e67654f7f03e13ab9d4d"},{"author":{"_account_id":10239,"name":"Dmitry Tantsur","email":"dtantsur@protonmail.com","username":"dtantsur"},"change_message_id":"9bfae1c4525213e67bd5ec840c6bd7b398ffd62c","unresolved":false,"context_lines":[{"line_number":10293,"context_line":"      - ironic-coverage-db-ubuntu-trusty"},{"line_number":10294,"context_line":"      - ironic-coverage-db-ubuntu-xenial"},{"line_number":10295,"context_line":"    gate:"},{"line_number":10296,"context_line":"      - gate-tempest-dsvm-ironic-ipa-wholedisk-agent_ssh-tinyipa-ubuntu-trusty"},{"line_number":10297,"context_line":"      - gate-tempest-dsvm-ironic-ipa-partition-pxe_ssh-tinyipa-ubuntu-trusty"},{"line_number":10298,"context_line":"      - gate-grenade-dsvm-ironic"},{"line_number":10299,"context_line":"      - ironic-coverage-db-ubuntu-trusty"},{"line_number":10300,"context_line":"      - ironic-coverage-db-ubuntu-xenial"}],"source_content_type":"text/x-yaml","patch_set":4,"id":"ba5da102_130914de","line":10297,"range":{"start_line":10296,"start_character":0,"end_line":10297,"end_character":76},"updated":"2016-11-04 14:56:49.000000000","message":"this change leaves us without dsvm gate jobs on master :( I\u0027m afraid we need to make the ipmitool jobs voting in the same patch.","commit_id":"04ad9e3f6e7098cad169e67654f7f03e13ab9d4d"},{"author":{"_account_id":6773,"name":"Lucas Alvares Gomes","email":"lucasagomes@gmail.com","username":"lucasagomes"},"change_message_id":"ad84e2fdb64e484ca70e31aef3ebe584f73a4064","unresolved":false,"context_lines":[{"line_number":10293,"context_line":"      - ironic-coverage-db-ubuntu-trusty"},{"line_number":10294,"context_line":"      - ironic-coverage-db-ubuntu-xenial"},{"line_number":10295,"context_line":"    gate:"},{"line_number":10296,"context_line":"      - gate-tempest-dsvm-ironic-ipa-wholedisk-agent_ssh-tinyipa-ubuntu-trusty"},{"line_number":10297,"context_line":"      - gate-tempest-dsvm-ironic-ipa-partition-pxe_ssh-tinyipa-ubuntu-trusty"},{"line_number":10298,"context_line":"      - gate-grenade-dsvm-ironic"},{"line_number":10299,"context_line":"      - ironic-coverage-db-ubuntu-trusty"},{"line_number":10300,"context_line":"      - ironic-coverage-db-ubuntu-xenial"}],"source_content_type":"text/x-yaml","patch_set":4,"id":"ba5da102_5e39d3d7","line":10297,"range":{"start_line":10296,"start_character":0,"end_line":10297,"end_character":76},"in_reply_to":"ba5da102_130914de","updated":"2016-11-04 15:07:20.000000000","message":"oh indeed, I\u0027m on it!","commit_id":"04ad9e3f6e7098cad169e67654f7f03e13ab9d4d"},{"author":{"_account_id":10239,"name":"Dmitry Tantsur","email":"dtantsur@protonmail.com","username":"dtantsur"},"change_message_id":"9bfae1c4525213e67bd5ec840c6bd7b398ffd62c","unresolved":false,"context_lines":[{"line_number":10407,"context_line":"      - gate-ironic-inspector-dsvm-ipa-src-nv"},{"line_number":10408,"context_line":"      - gate-tempest-dsvm-ironic-inspector-src-nv"},{"line_number":10409,"context_line":"    gate:"},{"line_number":10410,"context_line":"      - gate-tempest-dsvm-ironic-ipa-wholedisk-agent_ssh-coreos-ubuntu-trusty-src"},{"line_number":10411,"context_line":"      - gate-tempest-dsvm-ironic-ipa-wholedisk-agent_ssh-tinyipa-ubuntu-trusty-src"},{"line_number":10412,"context_line":"      - gate-tempest-dsvm-ironic-ipa-partition-pxe_ssh-coreos-ubuntu-trusty-src"},{"line_number":10413,"context_line":"      - gate-tempest-dsvm-ironic-ipa-partition-pxe_ssh-tinyipa-ubuntu-trusty-src"},{"line_number":10414,"context_line":"    experimental:"},{"line_number":10415,"context_line":"      - gate-tempest-dsvm-ironic-ipa-partition-pxe_ssh-tinyipa256-ubuntu-trusty-src"},{"line_number":10416,"context_line":"    post:"}],"source_content_type":"text/x-yaml","patch_set":4,"id":"ba5da102_53138c0d","line":10413,"range":{"start_line":10410,"start_character":0,"end_line":10413,"end_character":80},"updated":"2016-11-04 14:56:49.000000000","message":"ditto","commit_id":"04ad9e3f6e7098cad169e67654f7f03e13ab9d4d"},{"author":{"_account_id":6547,"name":"Andreas Jaeger","email":"jaegerandi@gmail.com","username":"jaegerandi"},"change_message_id":"9951de9afde0ee9645a8df821447c7c4bc1cfc24","unresolved":false,"context_lines":[{"line_number":3504,"context_line":"      - gate-devstack-bashate"},{"line_number":3505,"context_line":"      - gate-devstack-unit-tests"},{"line_number":3506,"context_line":"      - gate-grenade-dsvm-forward"},{"line_number":3507,"context_line":"      - gate-tempest-dsvm-ironic-ipa-wholedisk-agent_ssh-tinyipa-ubuntu-trusty-nv"},{"line_number":3508,"context_line":"      - gate-devstack-dsvm-updown"},{"line_number":3509,"context_line":"      - gate-tempest-dsvm-neutron-identity-v3-only-full-ubuntu-xenial-nv"},{"line_number":3510,"context_line":"      - gate-swift-dsvm-functional-ubuntu-trusty-nv"}],"source_content_type":"text/x-yaml","patch_set":5,"id":"ba5da102_fea61252","line":3507,"updated":"2016-11-05 23:32:24.000000000","message":"This means that the job will *not* run on newton or master due to our policy to not run ubuntu-trusty there. Don\u0027t you want to add a xenial job as well?","commit_id":"40db1d0725ae723a839a2ce12d0f54d5d17cc11d"},{"author":{"_account_id":6773,"name":"Lucas Alvares Gomes","email":"lucasagomes@gmail.com","username":"lucasagomes"},"change_message_id":"85e17e33affd2ec141e3fa9b7971f84a57ea770f","unresolved":false,"context_lines":[{"line_number":3504,"context_line":"      - gate-devstack-bashate"},{"line_number":3505,"context_line":"      - gate-devstack-unit-tests"},{"line_number":3506,"context_line":"      - gate-grenade-dsvm-forward"},{"line_number":3507,"context_line":"      - gate-tempest-dsvm-ironic-ipa-wholedisk-agent_ssh-tinyipa-ubuntu-trusty-nv"},{"line_number":3508,"context_line":"      - gate-devstack-dsvm-updown"},{"line_number":3509,"context_line":"      - gate-tempest-dsvm-neutron-identity-v3-only-full-ubuntu-xenial-nv"},{"line_number":3510,"context_line":"      - gate-swift-dsvm-functional-ubuntu-trusty-nv"}],"source_content_type":"text/x-yaml","patch_set":5,"id":"9a629dbe_80523cdc","line":3507,"in_reply_to":"ba5da102_fea61252","updated":"2016-11-07 09:58:47.000000000","message":"Ops, yes we should. I will ad a new patch set. Thanks for pointing it out.","commit_id":"40db1d0725ae723a839a2ce12d0f54d5d17cc11d"},{"author":{"_account_id":6547,"name":"Andreas Jaeger","email":"jaegerandi@gmail.com","username":"jaegerandi"},"change_message_id":"a6fbd4d989e1cf41a7fafaafe53e1c867b563e72","unresolved":false,"context_lines":[{"line_number":3504,"context_line":"      - gate-devstack-bashate"},{"line_number":3505,"context_line":"      - gate-devstack-unit-tests"},{"line_number":3506,"context_line":"      - gate-grenade-dsvm-forward"},{"line_number":3507,"context_line":"      - gate-tempest-dsvm-ironic-ipa-wholedisk-agent_ssh-tinyipa-ubuntu-trusty-nv"},{"line_number":3508,"context_line":"      - gate-tempest-dsvm-ironic-ipa-wholedisk-agent_ipmitool-tinyipa-ubuntu-xenial-nv"},{"line_number":3509,"context_line":"      - gate-devstack-dsvm-updown"},{"line_number":3510,"context_line":"      - gate-tempest-dsvm-neutron-identity-v3-only-full-ubuntu-xenial-nv"}],"source_content_type":"text/x-yaml","patch_set":6,"id":"9a629dbe_e393fd4d","line":3507,"range":{"start_line":3507,"start_character":8,"end_line":3507,"end_character":81},"updated":"2016-11-07 12:04:21.000000000","message":"This one is now run on both liberty and mitaka, your previous regex had the job not run on liberty. Is this intented?","commit_id":"29729d7144cfa9fa804d9eef49941b4e5767aaeb"},{"author":{"_account_id":10239,"name":"Dmitry Tantsur","email":"dtantsur@protonmail.com","username":"dtantsur"},"change_message_id":"1ead4b5e9a12367af7163ffed564c41e1d803b53","unresolved":false,"context_lines":[{"line_number":3504,"context_line":"      - gate-devstack-bashate"},{"line_number":3505,"context_line":"      - gate-devstack-unit-tests"},{"line_number":3506,"context_line":"      - gate-grenade-dsvm-forward"},{"line_number":3507,"context_line":"      - gate-tempest-dsvm-ironic-ipa-wholedisk-agent_ssh-tinyipa-ubuntu-trusty-nv"},{"line_number":3508,"context_line":"      - gate-tempest-dsvm-ironic-ipa-wholedisk-agent_ipmitool-tinyipa-ubuntu-xenial-nv"},{"line_number":3509,"context_line":"      - gate-devstack-dsvm-updown"},{"line_number":3510,"context_line":"      - gate-tempest-dsvm-neutron-identity-v3-only-full-ubuntu-xenial-nv"}],"source_content_type":"text/x-yaml","patch_set":6,"id":"9a629dbe_c281614c","line":3507,"range":{"start_line":3507,"start_character":8,"end_line":3507,"end_character":81},"in_reply_to":"9a629dbe_04cfe956","updated":"2016-11-08 11:28:21.000000000","message":"I don\u0027t remember why we didn\u0027t run the job on liberty, but I think it\u0027s fine to run it.","commit_id":"29729d7144cfa9fa804d9eef49941b4e5767aaeb"},{"author":{"_account_id":6773,"name":"Lucas Alvares Gomes","email":"lucasagomes@gmail.com","username":"lucasagomes"},"change_message_id":"a067014e21d9622898d8387b2464e2d188e597a6","unresolved":false,"context_lines":[{"line_number":3504,"context_line":"      - gate-devstack-bashate"},{"line_number":3505,"context_line":"      - gate-devstack-unit-tests"},{"line_number":3506,"context_line":"      - gate-grenade-dsvm-forward"},{"line_number":3507,"context_line":"      - gate-tempest-dsvm-ironic-ipa-wholedisk-agent_ssh-tinyipa-ubuntu-trusty-nv"},{"line_number":3508,"context_line":"      - gate-tempest-dsvm-ironic-ipa-wholedisk-agent_ipmitool-tinyipa-ubuntu-xenial-nv"},{"line_number":3509,"context_line":"      - gate-devstack-dsvm-updown"},{"line_number":3510,"context_line":"      - gate-tempest-dsvm-neutron-identity-v3-only-full-ubuntu-xenial-nv"}],"source_content_type":"text/x-yaml","patch_set":6,"id":"9a629dbe_04cfe956","line":3507,"range":{"start_line":3507,"start_character":8,"end_line":3507,"end_character":81},"in_reply_to":"9a629dbe_e393fd4d","updated":"2016-11-08 10:57:00.000000000","message":"Hmm... I\u0027m a bit confused, cause the regex removed was to prevent the ipmitool jobs to run on liberty right ? I bet agent_ssh was already running on it, so it should not be a problem.\n\nThat said, I removed the regex following the advise at [0].\n\n[0] https://review.openstack.org/#/c/392890/4/zuul/layout.yaml","commit_id":"29729d7144cfa9fa804d9eef49941b4e5767aaeb"},{"author":{"_account_id":10239,"name":"Dmitry Tantsur","email":"dtantsur@protonmail.com","username":"dtantsur"},"change_message_id":"1ead4b5e9a12367af7163ffed564c41e1d803b53","unresolved":false,"context_lines":[{"line_number":10276,"context_line":"      - gate-tempest-dsvm-ironic-pxe_ipa-postgres-ubuntu-trusty-nv"},{"line_number":10277,"context_line":"      - gate-tempest-dsvm-ironic-pxe_ipa-postgres-ubuntu-xenial-nv"},{"line_number":10278,"context_line":"      - gate-tempest-dsvm-ironic-ipa-wholedisk-agent_ssh-tinyipa-ubuntu-trusty"},{"line_number":10279,"context_line":"      - gate-tempest-dsvm-ironic-ipa-wholedisk-pxe_ssh-tinyipa-ubuntu-trusty-nv"},{"line_number":10280,"context_line":"      - gate-tempest-dsvm-ironic-ipa-partition-agent_ssh-tinyipa-ubuntu-trusty-nv"},{"line_number":10281,"context_line":"      - gate-tempest-dsvm-ironic-ipa-partition-pxe_ssh-tinyipa-ubuntu-trusty"},{"line_number":10282,"context_line":"      - gate-ironic-inspector-dsvm-ipa-nv"},{"line_number":10283,"context_line":"      - gate-tempest-dsvm-ironic-inspector-nv"}],"source_content_type":"text/x-yaml","patch_set":6,"id":"9a629dbe_426ab181","line":10280,"range":{"start_line":10279,"start_character":0,"end_line":10280,"end_character":81},"updated":"2016-11-08 11:28:21.000000000","message":"I don\u0027t see xenial+ipmitool variants of these jobs, we need to have them now that we disable ssh jobs.","commit_id":"29729d7144cfa9fa804d9eef49941b4e5767aaeb"},{"author":{"_account_id":6773,"name":"Lucas Alvares Gomes","email":"lucasagomes@gmail.com","username":"lucasagomes"},"change_message_id":"494fc9b86615812a7218a314aff36f395809d645","unresolved":false,"context_lines":[{"line_number":10276,"context_line":"      - gate-tempest-dsvm-ironic-pxe_ipa-postgres-ubuntu-trusty-nv"},{"line_number":10277,"context_line":"      - gate-tempest-dsvm-ironic-pxe_ipa-postgres-ubuntu-xenial-nv"},{"line_number":10278,"context_line":"      - gate-tempest-dsvm-ironic-ipa-wholedisk-agent_ssh-tinyipa-ubuntu-trusty"},{"line_number":10279,"context_line":"      - gate-tempest-dsvm-ironic-ipa-wholedisk-pxe_ssh-tinyipa-ubuntu-trusty-nv"},{"line_number":10280,"context_line":"      - gate-tempest-dsvm-ironic-ipa-partition-agent_ssh-tinyipa-ubuntu-trusty-nv"},{"line_number":10281,"context_line":"      - gate-tempest-dsvm-ironic-ipa-partition-pxe_ssh-tinyipa-ubuntu-trusty"},{"line_number":10282,"context_line":"      - gate-ironic-inspector-dsvm-ipa-nv"},{"line_number":10283,"context_line":"      - gate-tempest-dsvm-ironic-inspector-nv"}],"source_content_type":"text/x-yaml","patch_set":6,"id":"9a629dbe_02cfd956","line":10280,"range":{"start_line":10279,"start_character":0,"end_line":10280,"end_character":81},"in_reply_to":"9a629dbe_426ab181","updated":"2016-11-08 11:39:39.000000000","message":"Done","commit_id":"29729d7144cfa9fa804d9eef49941b4e5767aaeb"},{"author":{"_account_id":6547,"name":"Andreas Jaeger","email":"jaegerandi@gmail.com","username":"jaegerandi"},"change_message_id":"a6fbd4d989e1cf41a7fafaafe53e1c867b563e72","unresolved":false,"context_lines":[{"line_number":10389,"context_line":"    check:"},{"line_number":10390,"context_line":"      - gate-ironic-python-agent-tox-func-ubuntu-trusty"},{"line_number":10391,"context_line":"      - gate-ironic-python-agent-tox-func-ubuntu-xenial"},{"line_number":10392,"context_line":"      - gate-tempest-dsvm-ironic-ipa-wholedisk-agent_ssh-coreos-src-ubuntu-trusty"},{"line_number":10393,"context_line":"      - gate-tempest-dsvm-ironic-ipa-partition-agent_ssh-coreos-src-ubuntu-trusty-nv"},{"line_number":10394,"context_line":"      - gate-tempest-dsvm-ironic-ipa-wholedisk-agent_ssh-tinyipa-src-ubuntu-trusty"},{"line_number":10395,"context_line":"      - gate-tempest-dsvm-ironic-ipa-partition-agent_ssh-tinyipa-src-ubuntu-trusty-nv"}],"source_content_type":"text/x-yaml","patch_set":6,"id":"9a629dbe_231175ad","line":10392,"range":{"start_line":10392,"start_character":8,"end_line":10392,"end_character":81},"updated":"2016-11-07 12:04:21.000000000","message":"I don\u0027t see Xenial version of this - is this correct?","commit_id":"29729d7144cfa9fa804d9eef49941b4e5767aaeb"},{"author":{"_account_id":6773,"name":"Lucas Alvares Gomes","email":"lucasagomes@gmail.com","username":"lucasagomes"},"change_message_id":"a067014e21d9622898d8387b2464e2d188e597a6","unresolved":false,"context_lines":[{"line_number":10389,"context_line":"    check:"},{"line_number":10390,"context_line":"      - gate-ironic-python-agent-tox-func-ubuntu-trusty"},{"line_number":10391,"context_line":"      - gate-ironic-python-agent-tox-func-ubuntu-xenial"},{"line_number":10392,"context_line":"      - gate-tempest-dsvm-ironic-ipa-wholedisk-agent_ssh-coreos-src-ubuntu-trusty"},{"line_number":10393,"context_line":"      - gate-tempest-dsvm-ironic-ipa-partition-agent_ssh-coreos-src-ubuntu-trusty-nv"},{"line_number":10394,"context_line":"      - gate-tempest-dsvm-ironic-ipa-wholedisk-agent_ssh-tinyipa-src-ubuntu-trusty"},{"line_number":10395,"context_line":"      - gate-tempest-dsvm-ironic-ipa-partition-agent_ssh-tinyipa-src-ubuntu-trusty-nv"}],"source_content_type":"text/x-yaml","patch_set":6,"id":"9a629dbe_a4dd5d44","line":10392,"range":{"start_line":10392,"start_character":8,"end_line":10392,"end_character":81},"in_reply_to":"9a629dbe_231175ad","updated":"2016-11-08 10:57:00.000000000","message":"It\u0027s been replaced yes, it\u0027s on line 10400 (gate-tempest-dsvm-ironic-ipa-wholedisk-agent_ipmitool-coreos-src-ubuntu-xenial)","commit_id":"29729d7144cfa9fa804d9eef49941b4e5767aaeb"},{"author":{"_account_id":10239,"name":"Dmitry Tantsur","email":"dtantsur@protonmail.com","username":"dtantsur"},"change_message_id":"1ead4b5e9a12367af7163ffed564c41e1d803b53","unresolved":false,"context_lines":[{"line_number":10389,"context_line":"    check:"},{"line_number":10390,"context_line":"      - gate-ironic-python-agent-tox-func-ubuntu-trusty"},{"line_number":10391,"context_line":"      - gate-ironic-python-agent-tox-func-ubuntu-xenial"},{"line_number":10392,"context_line":"      - gate-tempest-dsvm-ironic-ipa-wholedisk-agent_ssh-coreos-src-ubuntu-trusty"},{"line_number":10393,"context_line":"      - gate-tempest-dsvm-ironic-ipa-partition-agent_ssh-coreos-src-ubuntu-trusty-nv"},{"line_number":10394,"context_line":"      - gate-tempest-dsvm-ironic-ipa-wholedisk-agent_ssh-tinyipa-src-ubuntu-trusty"},{"line_number":10395,"context_line":"      - gate-tempest-dsvm-ironic-ipa-partition-agent_ssh-tinyipa-src-ubuntu-trusty-nv"}],"source_content_type":"text/x-yaml","patch_set":6,"id":"9a629dbe_e2dd8571","line":10392,"range":{"start_line":10392,"start_character":8,"end_line":10392,"end_character":81},"in_reply_to":"9a629dbe_a4dd5d44","updated":"2016-11-08 11:28:21.000000000","message":"Please place trusty and xenial variants one below the other for easier reviewing","commit_id":"29729d7144cfa9fa804d9eef49941b4e5767aaeb"},{"author":{"_account_id":6773,"name":"Lucas Alvares Gomes","email":"lucasagomes@gmail.com","username":"lucasagomes"},"change_message_id":"494fc9b86615812a7218a314aff36f395809d645","unresolved":false,"context_lines":[{"line_number":10389,"context_line":"    check:"},{"line_number":10390,"context_line":"      - gate-ironic-python-agent-tox-func-ubuntu-trusty"},{"line_number":10391,"context_line":"      - gate-ironic-python-agent-tox-func-ubuntu-xenial"},{"line_number":10392,"context_line":"      - gate-tempest-dsvm-ironic-ipa-wholedisk-agent_ssh-coreos-src-ubuntu-trusty"},{"line_number":10393,"context_line":"      - gate-tempest-dsvm-ironic-ipa-partition-agent_ssh-coreos-src-ubuntu-trusty-nv"},{"line_number":10394,"context_line":"      - gate-tempest-dsvm-ironic-ipa-wholedisk-agent_ssh-tinyipa-src-ubuntu-trusty"},{"line_number":10395,"context_line":"      - gate-tempest-dsvm-ironic-ipa-partition-agent_ssh-tinyipa-src-ubuntu-trusty-nv"}],"source_content_type":"text/x-yaml","patch_set":6,"id":"9a629dbe_22dcdd11","line":10392,"range":{"start_line":10392,"start_character":8,"end_line":10392,"end_character":81},"in_reply_to":"9a629dbe_e2dd8571","updated":"2016-11-08 11:39:39.000000000","message":"Done","commit_id":"29729d7144cfa9fa804d9eef49941b4e5767aaeb"},{"author":{"_account_id":6547,"name":"Andreas Jaeger","email":"jaegerandi@gmail.com","username":"jaegerandi"},"change_message_id":"a6fbd4d989e1cf41a7fafaafe53e1c867b563e72","unresolved":false,"context_lines":[{"line_number":16770,"context_line":"      - gate-grenade-dsvm-multinode"},{"line_number":16771,"context_line":"      - gate-grenade-dsvm-neutron-multinode"},{"line_number":16772,"context_line":"      - gate-grenade-dsvm-neutron-ubuntu-xenial"},{"line_number":16773,"context_line":"      - gate-tempest-dsvm-ironic-ipa-wholedisk-agent_ssh-tinyipa-ubuntu-trusty-nv"},{"line_number":16774,"context_line":"      - gate-tempest-dsvm-ironic-ipa-wholedisk-agent_ipmitool-tinyipa-ubuntu-xenial-nv"},{"line_number":16775,"context_line":"      - gate-tempest-dsvm-neutron-full-ssh"},{"line_number":16776,"context_line":"      - gate-tempest-dsvm-neutron-identity-v3-only-full-ubuntu-xenial-nv"},{"line_number":16777,"context_line":"      - gate-tempest-dsvm-full-devstack-plugin-ceph-ubuntu-trusty"}],"source_content_type":"text/x-yaml","patch_set":6,"id":"9a629dbe_a374a5fd","line":16774,"range":{"start_line":16773,"start_character":5,"end_line":16774,"end_character":84},"updated":"2016-11-07 12:04:21.000000000","message":"This kind of change is not clear to me from your commit message. Could you enhance your commit message, please?","commit_id":"29729d7144cfa9fa804d9eef49941b4e5767aaeb"},{"author":{"_account_id":6773,"name":"Lucas Alvares Gomes","email":"lucasagomes@gmail.com","username":"lucasagomes"},"change_message_id":"a067014e21d9622898d8387b2464e2d188e597a6","unresolved":false,"context_lines":[{"line_number":16770,"context_line":"      - gate-grenade-dsvm-multinode"},{"line_number":16771,"context_line":"      - gate-grenade-dsvm-neutron-multinode"},{"line_number":16772,"context_line":"      - gate-grenade-dsvm-neutron-ubuntu-xenial"},{"line_number":16773,"context_line":"      - gate-tempest-dsvm-ironic-ipa-wholedisk-agent_ssh-tinyipa-ubuntu-trusty-nv"},{"line_number":16774,"context_line":"      - gate-tempest-dsvm-ironic-ipa-wholedisk-agent_ipmitool-tinyipa-ubuntu-xenial-nv"},{"line_number":16775,"context_line":"      - gate-tempest-dsvm-neutron-full-ssh"},{"line_number":16776,"context_line":"      - gate-tempest-dsvm-neutron-identity-v3-only-full-ubuntu-xenial-nv"},{"line_number":16777,"context_line":"      - gate-tempest-dsvm-full-devstack-plugin-ceph-ubuntu-trusty"}],"source_content_type":"text/x-yaml","patch_set":6,"id":"9a629dbe_444721bb","line":16774,"range":{"start_line":16773,"start_character":5,"end_line":16774,"end_character":84},"in_reply_to":"9a629dbe_a374a5fd","updated":"2016-11-08 10:57:00.000000000","message":"Sure thing, will do","commit_id":"29729d7144cfa9fa804d9eef49941b4e5767aaeb"}]}