)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":10239,"name":"Dmitry Tantsur","email":"dtantsur@protonmail.com","username":"dtantsur"},"change_message_id":"488164b469e6ce2d1259987f331ff81e84912171","unresolved":false,"context_lines":[{"line_number":8,"context_line":""},{"line_number":9,"context_line":" * Add q-agt to enabled services"},{"line_number":10,"context_line":" * set deploy driver to agent_ssh"},{"line_number":11,"context_line":" * disable automated cleaning"},{"line_number":12,"context_line":" * unset VIRT_DRIVER as it is a trigger to check that nova is"},{"line_number":13,"context_line":"   enabled."},{"line_number":14,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"3a461143_ddc71096","line":11,"range":{"start_line":11,"start_character":3,"end_line":11,"end_character":29},"updated":"2017-01-30 14:22:34.000000000","message":"could you please leave a comment why these two changes are done?","commit_id":"ec97e38b68937889c97b259b2bd7dc1aa35639c2"},{"author":{"_account_id":14525,"name":"Vasyl Saienko","email":"vsaienko@mirantis.com","username":"vsaienko"},"change_message_id":"0ba0b93641dcf48f60a2382f0c1aed3a22d343b9","unresolved":false,"context_lines":[{"line_number":8,"context_line":""},{"line_number":9,"context_line":" * Add q-agt to enabled services"},{"line_number":10,"context_line":" * set deploy driver to agent_ssh"},{"line_number":11,"context_line":" * disable automated cleaning"},{"line_number":12,"context_line":" * unset VIRT_DRIVER as it is a trigger to check that nova is"},{"line_number":13,"context_line":"   enabled."},{"line_number":14,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"3a461143_9dde680f","line":11,"range":{"start_line":11,"start_character":3,"end_line":11,"end_character":29},"in_reply_to":"3a461143_ddc71096","updated":"2017-01-30 14:31:33.000000000","message":"Done","commit_id":"ec97e38b68937889c97b259b2bd7dc1aa35639c2"},{"author":{"_account_id":10239,"name":"Dmitry Tantsur","email":"dtantsur@protonmail.com","username":"dtantsur"},"change_message_id":"bb76d362166d10788fcac4867fa9edae46d88962","unresolved":false,"context_lines":[{"line_number":9,"context_line":" * Add q-agt to enabled services"},{"line_number":10,"context_line":" * set deploy driver to agent_ipmitool: to configure devstack with"},{"line_number":11,"context_line":"   tempurl support. pxe_ipmitool will work also."},{"line_number":12,"context_line":" * disable automated cleaning: as tests for manual cleaning are"},{"line_number":13,"context_line":"   added in initial commit."},{"line_number":14,"context_line":" * unset VIRT_DRIVER as it is a trigger to check that nova is"},{"line_number":15,"context_line":"   enabled."},{"line_number":16,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"3a461143_1ab973cc","line":13,"range":{"start_line":12,"start_character":0,"end_line":13,"end_character":27},"updated":"2017-01-31 13:34:51.000000000","message":"manual cleaning is not strictly the same as automated cleaning code-wise. I wonder if we actually want it..","commit_id":"ae3c77ecbade50b80b4b63d44519857ff6c96f20"},{"author":{"_account_id":14525,"name":"Vasyl Saienko","email":"vsaienko@mirantis.com","username":"vsaienko"},"change_message_id":"a1fb8fe0e41104e89dbce0eb4dde3cf7293c7c68","unresolved":false,"context_lines":[{"line_number":9,"context_line":" * Add q-agt to enabled services"},{"line_number":10,"context_line":" * set deploy driver to agent_ipmitool: to configure devstack with"},{"line_number":11,"context_line":"   tempurl support. pxe_ipmitool will work also."},{"line_number":12,"context_line":" * disable automated cleaning: as tests for manual cleaning are"},{"line_number":13,"context_line":"   added in initial commit."},{"line_number":14,"context_line":" * unset VIRT_DRIVER as it is a trigger to check that nova is"},{"line_number":15,"context_line":"   enabled."},{"line_number":16,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"3a461143_1a66d329","line":13,"range":{"start_line":12,"start_character":0,"end_line":13,"end_character":27},"in_reply_to":"3a461143_1ab973cc","updated":"2017-01-31 13:42:56.000000000","message":"We still have automated cleaning enabled in other jobs, I would prefer to disable it here to save a time.","commit_id":"ae3c77ecbade50b80b4b63d44519857ff6c96f20"}],"jenkins/jobs/ironic.yaml":[{"author":{"_account_id":10239,"name":"Dmitry Tantsur","email":"dtantsur@protonmail.com","username":"dtantsur"},"change_message_id":"488164b469e6ce2d1259987f331ff81e84912171","unresolved":false,"context_lines":[{"line_number":1217,"context_line":"      - ironic-tempest-plugin"},{"line_number":1218,"context_line":"      - ironic-tempest-regex:"},{"line_number":1219,"context_line":"          tempest-regex: ironic_standalone"},{"line_number":1220,"context_line":"      - ironic-extra-vars:"},{"line_number":1221,"context_line":"          script: |"},{"line_number":1222,"context_line":"            export PROJECTS\u003d\"openstack/ironic $PROJECTS\""},{"line_number":1223,"context_line":"            export PROJECTS\u003d\"openstack/ironic-lib $PROJECTS\""},{"line_number":1224,"context_line":"            export PROJECTS\u003d\"openstack/ironic-python-agent $PROJECTS\""}],"source_content_type":"text/x-yaml","patch_set":1,"id":"3a461143_7de29c30","line":1221,"range":{"start_line":1220,"start_character":0,"end_line":1221,"end_character":19},"updated":"2017-01-30 14:22:34.000000000","message":"ironic-extra-vars is used by ironic-virtual builder which you don\u0027t include","commit_id":"ec97e38b68937889c97b259b2bd7dc1aa35639c2"},{"author":{"_account_id":14525,"name":"Vasyl Saienko","email":"vsaienko@mirantis.com","username":"vsaienko"},"change_message_id":"0ba0b93641dcf48f60a2382f0c1aed3a22d343b9","unresolved":false,"context_lines":[{"line_number":1217,"context_line":"      - ironic-tempest-plugin"},{"line_number":1218,"context_line":"      - ironic-tempest-regex:"},{"line_number":1219,"context_line":"          tempest-regex: ironic_standalone"},{"line_number":1220,"context_line":"      - ironic-extra-vars:"},{"line_number":1221,"context_line":"          script: |"},{"line_number":1222,"context_line":"            export PROJECTS\u003d\"openstack/ironic $PROJECTS\""},{"line_number":1223,"context_line":"            export PROJECTS\u003d\"openstack/ironic-lib $PROJECTS\""},{"line_number":1224,"context_line":"            export PROJECTS\u003d\"openstack/ironic-python-agent $PROJECTS\""}],"source_content_type":"text/x-yaml","patch_set":1,"id":"3a461143_bd0a4488","line":1221,"range":{"start_line":1220,"start_character":0,"end_line":1221,"end_character":19},"in_reply_to":"3a461143_7de29c30","updated":"2017-01-30 14:31:33.000000000","message":"added source ironic-extra-vars here.","commit_id":"ec97e38b68937889c97b259b2bd7dc1aa35639c2"},{"author":{"_account_id":10239,"name":"Dmitry Tantsur","email":"dtantsur@protonmail.com","username":"dtantsur"},"change_message_id":"bb76d362166d10788fcac4867fa9edae46d88962","unresolved":false,"context_lines":[{"line_number":1217,"context_line":"      - ironic-tempest-plugin"},{"line_number":1218,"context_line":"      - ironic-tempest-regex:"},{"line_number":1219,"context_line":"          tempest-regex: ironic_standalone"},{"line_number":1220,"context_line":"      - ironic-extra-vars:"},{"line_number":1221,"context_line":"          script: |"},{"line_number":1222,"context_line":"            export PROJECTS\u003d\"openstack/ironic $PROJECTS\""},{"line_number":1223,"context_line":"            export PROJECTS\u003d\"openstack/ironic-lib $PROJECTS\""},{"line_number":1224,"context_line":"            export PROJECTS\u003d\"openstack/ironic-python-agent $PROJECTS\""}],"source_content_type":"text/x-yaml","patch_set":2,"id":"3a461143_9a7f2314","line":1221,"range":{"start_line":1220,"start_character":0,"end_line":1221,"end_character":19},"updated":"2017-01-31 13:34:51.000000000","message":"this change still look incorrect. now you have ironic-extra-vars that source ironic-extra-vars.","commit_id":"ae3c77ecbade50b80b4b63d44519857ff6c96f20"},{"author":{"_account_id":14525,"name":"Vasyl Saienko","email":"vsaienko@mirantis.com","username":"vsaienko"},"change_message_id":"a1fb8fe0e41104e89dbce0eb4dde3cf7293c7c68","unresolved":false,"context_lines":[{"line_number":1217,"context_line":"      - ironic-tempest-plugin"},{"line_number":1218,"context_line":"      - ironic-tempest-regex:"},{"line_number":1219,"context_line":"          tempest-regex: ironic_standalone"},{"line_number":1220,"context_line":"      - ironic-extra-vars:"},{"line_number":1221,"context_line":"          script: |"},{"line_number":1222,"context_line":"            export PROJECTS\u003d\"openstack/ironic $PROJECTS\""},{"line_number":1223,"context_line":"            export PROJECTS\u003d\"openstack/ironic-lib $PROJECTS\""},{"line_number":1224,"context_line":"            export PROJECTS\u003d\"openstack/ironic-python-agent $PROJECTS\""}],"source_content_type":"text/x-yaml","patch_set":2,"id":"3a461143_9a7ac3bb","line":1221,"range":{"start_line":1220,"start_character":0,"end_line":1221,"end_character":19},"in_reply_to":"3a461143_9a7f2314","updated":"2017-01-31 13:42:56.000000000","message":"got it","commit_id":"ae3c77ecbade50b80b4b63d44519857ff6c96f20"}]}
