)]}'
{"jenkins/jobs/ironic.yaml":[{"author":{"_account_id":10239,"name":"Dmitry Tantsur","email":"dtantsur@protonmail.com","username":"dtantsur"},"change_message_id":"caf29d1eae3be52c04221c5ea5f010c76238d1f3","unresolved":false,"context_lines":[{"line_number":75,"context_line":"      - console-log"},{"line_number":76,"context_line":""},{"line_number":77,"context_line":"- job-template:"},{"line_number":78,"context_line":"    name: \u0027{pipeline}-ironic-inspector-dsvm-{ramdisk}{job-suffix}\u0027"},{"line_number":79,"context_line":"    node: \u0027{node}\u0027"},{"line_number":80,"context_line":""},{"line_number":81,"context_line":"    wrappers:"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"5a5ae5dd_75a58fb3","line":78,"range":{"start_line":78,"start_character":44,"end_line":78,"end_character":53},"updated":"2016-02-09 14:08:39.000000000","message":"I suggest we rename this variable ramdisk-\u003eflavor (or type), and then use it to distinguish between jobs instead of copy-pasting","commit_id":"2a41979e2a9c8ef6f2ff0e18c354f76c782940ac"},{"author":{"_account_id":7882,"name":"Yuiko Takada Mori","email":"yuiko-mori@nec.com","username":"YuikoTakada"},"change_message_id":"c867573d1881331529cc7adfacc54d3d2e5d46fc","unresolved":false,"context_lines":[{"line_number":174,"context_line":"          # IPA requires at least 1 GiB of RAM"},{"line_number":175,"context_line":"          export DEVSTACK_LOCAL_CONFIG+\u003d$\u0027\\n\u0027\"IRONIC_VM_SPECS_RAM\u003d1024\"$\u0027\\n\u0027\"IRONIC_VM_COUNT\u003d1\""},{"line_number":176,"context_line":"          # enable discovery hook"},{"line_number":177,"context_line":"          export DEVSTACK_LOCAL_CONFIG+\u003d$\u0027\\n\u0027\"IRONIC_INSPECTOR_NODE_NOT_FOUND_HOOK\u003ddiscovery\""},{"line_number":178,"context_line":""},{"line_number":179,"context_line":"          function post_test_hook {{"},{"line_number":180,"context_line":"                source $BASE/new/devstack/openrc admin admin"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"bae84128_02cddc65","line":177,"range":{"start_line":177,"start_character":83,"end_line":177,"end_character":92},"updated":"2016-02-23 03:10:07.000000000","message":"not discovery but enroll?","commit_id":"2a41979e2a9c8ef6f2ff0e18c354f76c782940ac"},{"author":{"_account_id":13636,"name":"aarefiev","display_name":"Anton","email":"arefyevanton@gmail.com","username":"aarefiev"},"change_message_id":"22e01de3f89060b9c409d5e6ac1ab4572785400e","unresolved":false,"context_lines":[{"line_number":174,"context_line":"          # IPA requires at least 1 GiB of RAM"},{"line_number":175,"context_line":"          export DEVSTACK_LOCAL_CONFIG+\u003d$\u0027\\n\u0027\"IRONIC_VM_SPECS_RAM\u003d1024\"$\u0027\\n\u0027\"IRONIC_VM_COUNT\u003d1\""},{"line_number":176,"context_line":"          # enable discovery hook"},{"line_number":177,"context_line":"          export DEVSTACK_LOCAL_CONFIG+\u003d$\u0027\\n\u0027\"IRONIC_INSPECTOR_NODE_NOT_FOUND_HOOK\u003ddiscovery\""},{"line_number":178,"context_line":""},{"line_number":179,"context_line":"          function post_test_hook {{"},{"line_number":180,"context_line":"                source $BASE/new/devstack/openrc admin admin"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"bae84128_ece6e6ae","line":177,"range":{"start_line":177,"start_character":83,"end_line":177,"end_character":92},"in_reply_to":"bae84128_02cddc65","updated":"2016-02-23 08:46:00.000000000","message":"yes, it was renamed, I\u0027ll update this patch as far as inspector work will be done","commit_id":"2a41979e2a9c8ef6f2ff0e18c354f76c782940ac"},{"author":{"_account_id":7118,"name":"Ian Wienand","email":"iwienand@redhat.com","username":"iwienand"},"change_message_id":"8e41db519f51574371683831ee1dc7522c31e56f","unresolved":false,"context_lines":[{"line_number":164,"context_line":"              export OVERRIDE_ZUUL_BRANCH\u003d$BRANCH_OVERRIDE"},{"line_number":165,"context_line":"          fi"},{"line_number":166,"context_line":""},{"line_number":167,"context_line":"          export PROJECTS\u003d\"openstack/ironic openstack/ironic-inspector $PROJECTS\""},{"line_number":168,"context_line":"          export DEVSTACK_LOCAL_CONFIG\u003d\"enable_plugin ironic git://git.openstack.org/openstack/ironic\""},{"line_number":169,"context_line":"          export DEVSTACK_LOCAL_CONFIG+\u003d$\u0027\\n\u0027\"enable_plugin ironic-inspector https://git.openstack.org/openstack/ironic-inspector\""},{"line_number":170,"context_line":""}],"source_content_type":"text/x-yaml","patch_set":2,"id":"7af24918_181f1ab8","line":167,"updated":"2016-03-03 00:51:47.000000000","message":"we usually split these but doesn\u0027t matter","commit_id":"53054e9d8c5b8301cd2d514036db32b9524c5e0a"},{"author":{"_account_id":7118,"name":"Ian Wienand","email":"iwienand@redhat.com","username":"iwienand"},"change_message_id":"8e41db519f51574371683831ee1dc7522c31e56f","unresolved":false,"context_lines":[{"line_number":179,"context_line":"          function post_test_hook {{"},{"line_number":180,"context_line":"                source $BASE/new/devstack/openrc admin admin"},{"line_number":181,"context_line":"                cd /opt/stack/new/ironic-inspector"},{"line_number":182,"context_line":"                sudo -E -u stack ./devstack/discovery_exercise.sh"},{"line_number":183,"context_line":"          }}"},{"line_number":184,"context_line":"          export -f post_test_hook"},{"line_number":185,"context_line":""}],"source_content_type":"text/x-yaml","patch_set":2,"id":"7af24918_5897e2eb","line":182,"updated":"2016-03-03 00:51:47.000000000","message":"this really seems odd ... i know you\u0027re just probably copying this from the others, but do you really need this?","commit_id":"53054e9d8c5b8301cd2d514036db32b9524c5e0a"},{"author":{"_account_id":13636,"name":"aarefiev","display_name":"Anton","email":"arefyevanton@gmail.com","username":"aarefiev"},"change_message_id":"0409c36b1b8f6fe78113a474d5ce3d59650eabb4","unresolved":false,"context_lines":[{"line_number":179,"context_line":"          function post_test_hook {{"},{"line_number":180,"context_line":"                source $BASE/new/devstack/openrc admin admin"},{"line_number":181,"context_line":"                cd /opt/stack/new/ironic-inspector"},{"line_number":182,"context_line":"                sudo -E -u stack ./devstack/discovery_exercise.sh"},{"line_number":183,"context_line":"          }}"},{"line_number":184,"context_line":"          export -f post_test_hook"},{"line_number":185,"context_line":""}],"source_content_type":"text/x-yaml","patch_set":2,"id":"7af24918_7cc1e053","line":182,"in_reply_to":"7af24918_5897e2eb","updated":"2016-03-03 08:45:56.000000000","message":"could you specify what do you mean by \u0027this\u0027? It will run test script discovery_exercise.sh","commit_id":"53054e9d8c5b8301cd2d514036db32b9524c5e0a"},{"author":{"_account_id":10239,"name":"Dmitry Tantsur","email":"dtantsur@protonmail.com","username":"dtantsur"},"change_message_id":"4e522b65496402cdd252f6d8178494ae4e56e359","unresolved":false,"context_lines":[{"line_number":179,"context_line":"          function post_test_hook {{"},{"line_number":180,"context_line":"                source $BASE/new/devstack/openrc admin admin"},{"line_number":181,"context_line":"                cd /opt/stack/new/ironic-inspector"},{"line_number":182,"context_line":"                sudo -E -u stack ./devstack/discovery_exercise.sh"},{"line_number":183,"context_line":"          }}"},{"line_number":184,"context_line":"          export -f post_test_hook"},{"line_number":185,"context_line":""}],"source_content_type":"text/x-yaml","patch_set":2,"id":"7af24918_d82d9947","line":182,"in_reply_to":"7af24918_7cc1e053","updated":"2016-03-03 10:38:24.000000000","message":"I also didn\u0027t get the question.. Ian, could you elaborate?","commit_id":"53054e9d8c5b8301cd2d514036db32b9524c5e0a"},{"author":{"_account_id":10239,"name":"Dmitry Tantsur","email":"dtantsur@protonmail.com","username":"dtantsur"},"change_message_id":"4555b799080ced42c3723a281889e863d82991d9","unresolved":false,"context_lines":[{"line_number":179,"context_line":"          function post_test_hook {{"},{"line_number":180,"context_line":"                source $BASE/new/devstack/openrc admin admin"},{"line_number":181,"context_line":"                cd /opt/stack/new/ironic-inspector"},{"line_number":182,"context_line":"                sudo -E -u stack ./devstack/discovery_exercise.sh"},{"line_number":183,"context_line":"          }}"},{"line_number":184,"context_line":"          export -f post_test_hook"},{"line_number":185,"context_line":""}],"source_content_type":"text/x-yaml","patch_set":2,"id":"7af24918_ed24ee0d","line":182,"in_reply_to":"7af24918_9bb80777","updated":"2016-03-03 19:56:19.000000000","message":"I remember me copy-pasting it from somewhere, but I\u0027m not sure :) I think Anton can try removing it and see what happens. It\u0027s a non-voting job after all :)","commit_id":"53054e9d8c5b8301cd2d514036db32b9524c5e0a"},{"author":{"_account_id":13636,"name":"aarefiev","display_name":"Anton","email":"arefyevanton@gmail.com","username":"aarefiev"},"change_message_id":"b8922183238ccce61b49c39b987c761abfde629c","unresolved":false,"context_lines":[{"line_number":179,"context_line":"          function post_test_hook {{"},{"line_number":180,"context_line":"                source $BASE/new/devstack/openrc admin admin"},{"line_number":181,"context_line":"                cd /opt/stack/new/ironic-inspector"},{"line_number":182,"context_line":"                sudo -E -u stack ./devstack/discovery_exercise.sh"},{"line_number":183,"context_line":"          }}"},{"line_number":184,"context_line":"          export -f post_test_hook"},{"line_number":185,"context_line":""}],"source_content_type":"text/x-yaml","patch_set":2,"id":"7af24918_3250edd3","line":182,"in_reply_to":"7af24918_d098a96c","updated":"2016-03-03 21:51:36.000000000","message":"We are using virsh for manage bare metal vms, so looks like we need sudo here","commit_id":"53054e9d8c5b8301cd2d514036db32b9524c5e0a"},{"author":{"_account_id":7118,"name":"Ian Wienand","email":"iwienand@redhat.com","username":"iwienand"},"change_message_id":"8c10798bbdce884d0d792ac9bc2c7f00b89ed6ee","unresolved":false,"context_lines":[{"line_number":179,"context_line":"          function post_test_hook {{"},{"line_number":180,"context_line":"                source $BASE/new/devstack/openrc admin admin"},{"line_number":181,"context_line":"                cd /opt/stack/new/ironic-inspector"},{"line_number":182,"context_line":"                sudo -E -u stack ./devstack/discovery_exercise.sh"},{"line_number":183,"context_line":"          }}"},{"line_number":184,"context_line":"          export -f post_test_hook"},{"line_number":185,"context_line":""}],"source_content_type":"text/x-yaml","patch_set":2,"id":"7af24918_9bb80777","line":182,"in_reply_to":"7af24918_d82d9947","updated":"2016-03-03 19:43:43.000000000","message":"sorry, i mean do you really need the sudo part of this?","commit_id":"53054e9d8c5b8301cd2d514036db32b9524c5e0a"},{"author":{"_account_id":7118,"name":"Ian Wienand","email":"iwienand@redhat.com","username":"iwienand"},"change_message_id":"8fbe95b8abc4add963649dfea9ad4b9ff85ebd50","unresolved":false,"context_lines":[{"line_number":179,"context_line":"          function post_test_hook {{"},{"line_number":180,"context_line":"                source $BASE/new/devstack/openrc admin admin"},{"line_number":181,"context_line":"                cd /opt/stack/new/ironic-inspector"},{"line_number":182,"context_line":"                sudo -E -u stack ./devstack/discovery_exercise.sh"},{"line_number":183,"context_line":"          }}"},{"line_number":184,"context_line":"          export -f post_test_hook"},{"line_number":185,"context_line":""}],"source_content_type":"text/x-yaml","patch_set":2,"id":"7af24918_d098a96c","line":182,"in_reply_to":"7af24918_ed24ee0d","updated":"2016-03-03 20:09:26.000000000","message":"yeah, so i mean unless you\u0027re touching parts of devstack setup as \"stack\" i would not think this is required ...","commit_id":"53054e9d8c5b8301cd2d514036db32b9524c5e0a"}]}