)]}'
{"jenkins/jobs/devstack-gate.yaml":[{"author":{"_account_id":18893,"name":"Miles Gould","email":"mgould@redhat.com","username":"pozorvlak"},"change_message_id":"b5ca216066dc9dd4961c9e5d7ce5ddfb00c1bc95","unresolved":false,"context_lines":[{"line_number":1386,"context_line":"      - console-log"},{"line_number":1387,"context_line":""},{"line_number":1388,"context_line":"- job-template:"},{"line_number":1389,"context_line":"    name: \u0027{pipeline}-tempest-dsvm-ironic-pxe_libvirt{job-suffix}\u0027"},{"line_number":1390,"context_line":"    node: \u0027{node}\u0027"},{"line_number":1391,"context_line":""},{"line_number":1392,"context_line":"    wrappers:"}],"source_content_type":"text/x-yaml","patch_set":3,"id":"dab17558_9184c65d","side":"PARENT","line":1389,"updated":"2016-05-13 14:43:19.000000000","message":"I think this job could run tinyipa too, but I\u0027m not sure.","commit_id":"a2782514dabb14fb61dd35ea95a565169d9c5dbe"},{"author":{"_account_id":6773,"name":"Lucas Alvares Gomes","email":"lucasagomes@gmail.com","username":"lucasagomes"},"change_message_id":"0ae84c47d5a1be9943aaf9c2bb8fa13fb126d131","unresolved":false,"context_lines":[{"line_number":684,"context_line":"              export DEVSTACK_LOCAL_CONFIG+\u003d$\u0027\\n\u0027\"IRONIC_DEPLOY_DRIVER_ISCSI_WITH_IPA\u003dTrue\""},{"line_number":685,"context_line":"              # Only a single ramdisk was supported in kilo/liberty, so don\u0027t set this"},{"line_number":686,"context_line":"              if [ \"$ZUUL_BRANCH\" !\u003d \"stable/kilo\" -a \"$ZUUL_BRANCH\" !\u003d \"stable/liberty\" ] ; then"},{"line_number":687,"context_line":"                  export IRONIC_RAMDISK_TYPE\u003d{ramdisk-type}"},{"line_number":688,"context_line":"              fi"},{"line_number":689,"context_line":"              export DEVSTACK_LOCAL_CONFIG+\u003d$\u0027\\n\u0027\"IRONIC_RAMDISK_TYPE\u003d$IRONIC_RAMDISK_TYPE\""},{"line_number":690,"context_line":"          fi"}],"source_content_type":"text/x-yaml","patch_set":3,"id":"dab17558_b1874a11","line":687,"updated":"2016-05-13 14:54:39.000000000","message":"export DEVSTACK_LOCAL_CONFIG+\u003d$\u0027\\n\u0027\"IRONIC_RAMDISK_TYPE\u003d${ramdisk-type}\"\n\nand get rid of the L689 otherwise you may pass an empty IRONIC_RAMDISK_TYPE\u003d to the config?","commit_id":"281c903a24b79b7017a7d653e58e0862526b6af9"},{"author":{"_account_id":10342,"name":"Jay Faulkner","display_name":"JayF","email":"jay@jvf.cc","username":"JayF","status":"youtube.com/@oss-gr / podcast.gr-oss.io"},"change_message_id":"7c926956260eb146942ad7e3e9fe5055e9300f8a","unresolved":false,"context_lines":[{"line_number":684,"context_line":"              export DEVSTACK_LOCAL_CONFIG+\u003d$\u0027\\n\u0027\"IRONIC_DEPLOY_DRIVER_ISCSI_WITH_IPA\u003dTrue\""},{"line_number":685,"context_line":"              # Only a single ramdisk was supported in kilo/liberty, so don\u0027t set this"},{"line_number":686,"context_line":"              if [ \"$ZUUL_BRANCH\" !\u003d \"stable/kilo\" -a \"$ZUUL_BRANCH\" !\u003d \"stable/liberty\" ] ; then"},{"line_number":687,"context_line":"                  export IRONIC_RAMDISK_TYPE\u003d{ramdisk-type}"},{"line_number":688,"context_line":"              fi"},{"line_number":689,"context_line":"              export DEVSTACK_LOCAL_CONFIG+\u003d$\u0027\\n\u0027\"IRONIC_RAMDISK_TYPE\u003d$IRONIC_RAMDISK_TYPE\""},{"line_number":690,"context_line":"          fi"}],"source_content_type":"text/x-yaml","patch_set":3,"id":"dab17558_d2a230ad","line":687,"in_reply_to":"dab17558_b1874a11","updated":"2016-05-17 20:04:08.000000000","message":"Done","commit_id":"281c903a24b79b7017a7d653e58e0862526b6af9"},{"author":{"_account_id":18893,"name":"Miles Gould","email":"mgould@redhat.com","username":"pozorvlak"},"change_message_id":"b5ca216066dc9dd4961c9e5d7ce5ddfb00c1bc95","unresolved":false,"context_lines":[{"line_number":826,"context_line":"      - console-log"},{"line_number":827,"context_line":""},{"line_number":828,"context_line":"- job-template:"},{"line_number":829,"context_line":"    name: \u0027{pipeline}-tempest-dsvm-ironic-pxe_ipa-tinyipa-src{job-suffix}\u0027"},{"line_number":830,"context_line":"    node: \u0027{node}\u0027"},{"line_number":831,"context_line":""},{"line_number":832,"context_line":"    wrappers:"}],"source_content_type":"text/x-yaml","patch_set":3,"id":"dab17558_e68116fd","line":829,"updated":"2016-05-13 14:43:19.000000000","message":"Should the name of this job change since it\u0027s now using iPXE?","commit_id":"281c903a24b79b7017a7d653e58e0862526b6af9"},{"author":{"_account_id":10342,"name":"Jay Faulkner","display_name":"JayF","email":"jay@jvf.cc","username":"JayF","status":"youtube.com/@oss-gr / podcast.gr-oss.io"},"change_message_id":"7c926956260eb146942ad7e3e9fe5055e9300f8a","unresolved":false,"context_lines":[{"line_number":826,"context_line":"      - console-log"},{"line_number":827,"context_line":""},{"line_number":828,"context_line":"- job-template:"},{"line_number":829,"context_line":"    name: \u0027{pipeline}-tempest-dsvm-ironic-pxe_ipa-tinyipa-src{job-suffix}\u0027"},{"line_number":830,"context_line":"    node: \u0027{node}\u0027"},{"line_number":831,"context_line":""},{"line_number":832,"context_line":"    wrappers:"}],"source_content_type":"text/x-yaml","patch_set":3,"id":"dab17558_128038f6","line":829,"in_reply_to":"dab17558_e68116fd","updated":"2016-05-17 20:04:08.000000000","message":"I\u0027m -1 to changing any names in this as 1) it invalidates any data keyed off name on job performance and 2) pxe is still the name of the driver, even with ipxe enabled.","commit_id":"281c903a24b79b7017a7d653e58e0862526b6af9"},{"author":{"_account_id":6618,"name":"Ruby Loo","email":"opensrloo@gmail.com","username":"rloo"},"change_message_id":"2b81215f7773dd40da522c216f4430e7893747e0","unresolved":false,"context_lines":[{"line_number":675,"context_line":"             export DEVSTACK_PROJECT_FROM_GIT\u003d\"pyghmi,$DEVSTACK_PROJECT_FROM_GIT\""},{"line_number":676,"context_line":"          fi"},{"line_number":677,"context_line":""},{"line_number":678,"context_line":"          # The IPA ramdisk needs at least 1GB of RAM to run"},{"line_number":679,"context_line":"          export DEVSTACK_LOCAL_CONFIG\u003d\"IRONIC_VM_SPECS_RAM\u003d{bm-vm-ram}\""},{"line_number":680,"context_line":"          export DEVSTACK_LOCAL_CONFIG+\u003d$\u0027\\n\u0027\"IRONIC_VM_COUNT\u003d{bm-vm-count}\""},{"line_number":681,"context_line":""}],"source_content_type":"text/x-yaml","patch_set":4,"id":"dab17558_56fc6bb4","line":678,"updated":"2016-05-18 01:51:54.000000000","message":"You might want to update this since it depends on the ramdisk being used. Or delete the comment?","commit_id":"86425653403676e82ad99886f029f02489cbadd7"},{"author":{"_account_id":10342,"name":"Jay Faulkner","display_name":"JayF","email":"jay@jvf.cc","username":"JayF","status":"youtube.com/@oss-gr / podcast.gr-oss.io"},"change_message_id":"26428fdca4bdff2dda558e0f0ca997c85a63d35a","unresolved":false,"context_lines":[{"line_number":675,"context_line":"             export DEVSTACK_PROJECT_FROM_GIT\u003d\"pyghmi,$DEVSTACK_PROJECT_FROM_GIT\""},{"line_number":676,"context_line":"          fi"},{"line_number":677,"context_line":""},{"line_number":678,"context_line":"          # The IPA ramdisk needs at least 1GB of RAM to run"},{"line_number":679,"context_line":"          export DEVSTACK_LOCAL_CONFIG\u003d\"IRONIC_VM_SPECS_RAM\u003d{bm-vm-ram}\""},{"line_number":680,"context_line":"          export DEVSTACK_LOCAL_CONFIG+\u003d$\u0027\\n\u0027\"IRONIC_VM_COUNT\u003d{bm-vm-count}\""},{"line_number":681,"context_line":""}],"source_content_type":"text/x-yaml","patch_set":4,"id":"bab6814e_3ebd7902","line":678,"in_reply_to":"dab17558_56fc6bb4","updated":"2016-05-26 21:35:37.000000000","message":"Going to make this mostly irrelevant by pivoting vm size on ramdisk chosen.","commit_id":"86425653403676e82ad99886f029f02489cbadd7"},{"author":{"_account_id":6618,"name":"Ruby Loo","email":"opensrloo@gmail.com","username":"rloo"},"change_message_id":"2b81215f7773dd40da522c216f4430e7893747e0","unresolved":false,"context_lines":[{"line_number":682,"context_line":"          export DEPLOY_WITH_IPA\u003d{deploy-with-ipa}"},{"line_number":683,"context_line":"          if [ \"$DEPLOY_WITH_IPA\" \u003d\u003d \"1\" ] ; then"},{"line_number":684,"context_line":"              export DEVSTACK_LOCAL_CONFIG+\u003d$\u0027\\n\u0027\"IRONIC_DEPLOY_DRIVER_ISCSI_WITH_IPA\u003dTrue\""},{"line_number":685,"context_line":"              # Only a single ramdisk was supported in kilo/liberty, so don\u0027t set this"},{"line_number":686,"context_line":"              if [ \"$ZUUL_BRANCH\" !\u003d \"stable/kilo\" -a \"$ZUUL_BRANCH\" !\u003d \"stable/liberty\" ] ; then"},{"line_number":687,"context_line":"                  export IRONIC_RAMDISK_TYPE\u003d{ramdisk-type}"},{"line_number":688,"context_line":"                  export DEVSTACK_LOCAL_CONFIG+\u003d$\u0027\\n\u0027\"IRONIC_RAMDISK_TYPE\u003d$IRONIC_RAMDISK_TYPE\""}],"source_content_type":"text/x-yaml","patch_set":4,"id":"dab17558_b69cb7c0","line":685,"updated":"2016-05-18 01:51:54.000000000","message":"nit: if you look at some other comments (L699, 718), they are worded differently (and I think clearer). Eg \"different ramdisks are supported in mitaka\"","commit_id":"86425653403676e82ad99886f029f02489cbadd7"},{"author":{"_account_id":10342,"name":"Jay Faulkner","display_name":"JayF","email":"jay@jvf.cc","username":"JayF","status":"youtube.com/@oss-gr / podcast.gr-oss.io"},"change_message_id":"26428fdca4bdff2dda558e0f0ca997c85a63d35a","unresolved":false,"context_lines":[{"line_number":682,"context_line":"          export DEPLOY_WITH_IPA\u003d{deploy-with-ipa}"},{"line_number":683,"context_line":"          if [ \"$DEPLOY_WITH_IPA\" \u003d\u003d \"1\" ] ; then"},{"line_number":684,"context_line":"              export DEVSTACK_LOCAL_CONFIG+\u003d$\u0027\\n\u0027\"IRONIC_DEPLOY_DRIVER_ISCSI_WITH_IPA\u003dTrue\""},{"line_number":685,"context_line":"              # Only a single ramdisk was supported in kilo/liberty, so don\u0027t set this"},{"line_number":686,"context_line":"              if [ \"$ZUUL_BRANCH\" !\u003d \"stable/kilo\" -a \"$ZUUL_BRANCH\" !\u003d \"stable/liberty\" ] ; then"},{"line_number":687,"context_line":"                  export IRONIC_RAMDISK_TYPE\u003d{ramdisk-type}"},{"line_number":688,"context_line":"                  export DEVSTACK_LOCAL_CONFIG+\u003d$\u0027\\n\u0027\"IRONIC_RAMDISK_TYPE\u003d$IRONIC_RAMDISK_TYPE\""}],"source_content_type":"text/x-yaml","patch_set":4,"id":"bab6814e_5ed5dd40","line":685,"in_reply_to":"dab17558_b69cb7c0","updated":"2016-05-26 21:35:37.000000000","message":"Done","commit_id":"86425653403676e82ad99886f029f02489cbadd7"},{"author":{"_account_id":6618,"name":"Ruby Loo","email":"opensrloo@gmail.com","username":"rloo"},"change_message_id":"2b81215f7773dd40da522c216f4430e7893747e0","unresolved":false,"context_lines":[{"line_number":686,"context_line":"              if [ \"$ZUUL_BRANCH\" !\u003d \"stable/kilo\" -a \"$ZUUL_BRANCH\" !\u003d \"stable/liberty\" ] ; then"},{"line_number":687,"context_line":"                  export IRONIC_RAMDISK_TYPE\u003d{ramdisk-type}"},{"line_number":688,"context_line":"                  export DEVSTACK_LOCAL_CONFIG+\u003d$\u0027\\n\u0027\"IRONIC_RAMDISK_TYPE\u003d$IRONIC_RAMDISK_TYPE\""},{"line_number":689,"context_line":"              fi"},{"line_number":690,"context_line":"          fi"},{"line_number":691,"context_line":""},{"line_number":692,"context_line":"          export IPXE_ENABLED\u003d{ipxe-enabled}"}],"source_content_type":"text/x-yaml","patch_set":4,"id":"dab17558_b6903744","line":689,"updated":"2016-05-18 01:51:54.000000000","message":"What I don\u0027t understand is why you need to wrap this in an if, when the original didn\u0027t have the if. Is it because this patch changes things so that  {ramdisk-type} isn\u0027t always set? (I mean, the original was working with stable/kilo \u0026 stable/liberty, right?)","commit_id":"86425653403676e82ad99886f029f02489cbadd7"},{"author":{"_account_id":10342,"name":"Jay Faulkner","display_name":"JayF","email":"jay@jvf.cc","username":"JayF","status":"youtube.com/@oss-gr / podcast.gr-oss.io"},"change_message_id":"26428fdca4bdff2dda558e0f0ca997c85a63d35a","unresolved":false,"context_lines":[{"line_number":686,"context_line":"              if [ \"$ZUUL_BRANCH\" !\u003d \"stable/kilo\" -a \"$ZUUL_BRANCH\" !\u003d \"stable/liberty\" ] ; then"},{"line_number":687,"context_line":"                  export IRONIC_RAMDISK_TYPE\u003d{ramdisk-type}"},{"line_number":688,"context_line":"                  export DEVSTACK_LOCAL_CONFIG+\u003d$\u0027\\n\u0027\"IRONIC_RAMDISK_TYPE\u003d$IRONIC_RAMDISK_TYPE\""},{"line_number":689,"context_line":"              fi"},{"line_number":690,"context_line":"          fi"},{"line_number":691,"context_line":""},{"line_number":692,"context_line":"          export IPXE_ENABLED\u003d{ipxe-enabled}"}],"source_content_type":"text/x-yaml","patch_set":4,"id":"bab6814e_5b1736a3","line":689,"in_reply_to":"dab17558_b6903744","updated":"2016-05-26 21:35:37.000000000","message":"The only ramdisk we supported, and worked at all, prior to kilo and liberty was CoreOS. Given all the jobs were configured for CoreOS already, that was no big deal. Now that we\u0027re also changing them to TinyIPA, we need to fix it here. \n\nIn fact, I might need to handle VM size in a similar way.","commit_id":"86425653403676e82ad99886f029f02489cbadd7"},{"author":{"_account_id":6618,"name":"Ruby Loo","email":"opensrloo@gmail.com","username":"rloo"},"change_message_id":"2b81215f7773dd40da522c216f4430e7893747e0","unresolved":false,"context_lines":[{"line_number":891,"context_line":"          branch-override: \u0027{branch-override}\u0027"},{"line_number":892,"context_line":"          ramdisk-type: \u0027tinyipa\u0027"},{"line_number":893,"context_line":"          gate-tempest-full: 1"},{"line_number":894,"context_line":"          bm-vm-ram: 384"},{"line_number":895,"context_line":"          bm-vm-count: 7"},{"line_number":896,"context_line":"          multitenant-network: 0"},{"line_number":897,"context_line":""}],"source_content_type":"text/x-yaml","patch_set":4,"id":"dab17558_317e1119","line":894,"updated":"2016-05-18 01:51:54.000000000","message":"If we can use 384 here, who don\u0027t we use this instead of 512 elsewhere?","commit_id":"86425653403676e82ad99886f029f02489cbadd7"},{"author":{"_account_id":10342,"name":"Jay Faulkner","display_name":"JayF","email":"jay@jvf.cc","username":"JayF","status":"youtube.com/@oss-gr / podcast.gr-oss.io"},"change_message_id":"26428fdca4bdff2dda558e0f0ca997c85a63d35a","unresolved":false,"context_lines":[{"line_number":891,"context_line":"          branch-override: \u0027{branch-override}\u0027"},{"line_number":892,"context_line":"          ramdisk-type: \u0027tinyipa\u0027"},{"line_number":893,"context_line":"          gate-tempest-full: 1"},{"line_number":894,"context_line":"          bm-vm-ram: 384"},{"line_number":895,"context_line":"          bm-vm-count: 7"},{"line_number":896,"context_line":"          multitenant-network: 0"},{"line_number":897,"context_line":""}],"source_content_type":"text/x-yaml","patch_set":4,"id":"bab6814e_7eeb8113","line":894,"in_reply_to":"dab17558_317e1119","updated":"2016-05-26 21:35:37.000000000","message":"I\u0027m going to refactor this a bit to set a default vm size for a given ramdisk, so there\u0027s only one knob to turn. I\u0027m going to set it to 384 for now, but really it\u0027s a \"decision\" if we want CI to break if TinyIPA begins requiring \u003e384mb of RAM to run. \n\nI think that\u0027s a sane value (384mb) so I\u0027m going to make that the default :).","commit_id":"86425653403676e82ad99886f029f02489cbadd7"},{"author":{"_account_id":6618,"name":"Ruby Loo","email":"opensrloo@gmail.com","username":"rloo"},"change_message_id":"2b81215f7773dd40da522c216f4430e7893747e0","unresolved":false,"context_lines":[{"line_number":1024,"context_line":""},{"line_number":1025,"context_line":"    wrappers:"},{"line_number":1026,"context_line":"      - build-timeout:"},{"line_number":1027,"context_line":"          timeout: 180"},{"line_number":1028,"context_line":"      - timestamps"},{"line_number":1029,"context_line":""},{"line_number":1030,"context_line":"    builders:"}],"source_content_type":"text/x-yaml","patch_set":4,"id":"dab17558_711ed9bb","line":1027,"updated":"2016-05-18 01:51:54.000000000","message":"it is supposed to be faster with tinyipa so we could reduce this but we\u0027re going to leave it cuz it doesn\u0027t matter, right? cuz it shouldn\u0027t timeout anyway? :)","commit_id":"86425653403676e82ad99886f029f02489cbadd7"},{"author":{"_account_id":10342,"name":"Jay Faulkner","display_name":"JayF","email":"jay@jvf.cc","username":"JayF","status":"youtube.com/@oss-gr / podcast.gr-oss.io"},"change_message_id":"26428fdca4bdff2dda558e0f0ca997c85a63d35a","unresolved":false,"context_lines":[{"line_number":1024,"context_line":""},{"line_number":1025,"context_line":"    wrappers:"},{"line_number":1026,"context_line":"      - build-timeout:"},{"line_number":1027,"context_line":"          timeout: 180"},{"line_number":1028,"context_line":"      - timestamps"},{"line_number":1029,"context_line":""},{"line_number":1030,"context_line":"    builders:"}],"source_content_type":"text/x-yaml","patch_set":4,"id":"bab6814e_fec65198","line":1027,"in_reply_to":"dab17558_711ed9bb","updated":"2016-05-26 21:35:37.000000000","message":"Honestly, I\u0027m changing as little as possible to avoid affecting the stability of the gate. This seems like a value we\u0027d only want to change after collecting data on the new configurations.","commit_id":"86425653403676e82ad99886f029f02489cbadd7"},{"author":{"_account_id":6618,"name":"Ruby Loo","email":"opensrloo@gmail.com","username":"rloo"},"change_message_id":"2b81215f7773dd40da522c216f4430e7893747e0","unresolved":false,"context_lines":[{"line_number":1406,"context_line":"      - devstack-virtual-ironic:"},{"line_number":1407,"context_line":"          postgres: 0"},{"line_number":1408,"context_line":"          build-ramdisk: 1"},{"line_number":1409,"context_line":"          deploy_driver: pxe_libvirt"},{"line_number":1410,"context_line":"          deploy-with-ipa: 0"},{"line_number":1411,"context_line":"          client-from-source: 0"},{"line_number":1412,"context_line":"          ironic-lib-from-source: 0"}],"source_content_type":"text/x-yaml","patch_set":4,"id":"dab17558_b164c121","line":1409,"updated":"2016-05-18 01:51:54.000000000","message":"This is a bonus question. I didn\u0027t think we had such a driver?","commit_id":"86425653403676e82ad99886f029f02489cbadd7"},{"author":{"_account_id":10342,"name":"Jay Faulkner","display_name":"JayF","email":"jay@jvf.cc","username":"JayF","status":"youtube.com/@oss-gr / podcast.gr-oss.io"},"change_message_id":"26428fdca4bdff2dda558e0f0ca997c85a63d35a","unresolved":false,"context_lines":[{"line_number":1406,"context_line":"      - devstack-virtual-ironic:"},{"line_number":1407,"context_line":"          postgres: 0"},{"line_number":1408,"context_line":"          build-ramdisk: 1"},{"line_number":1409,"context_line":"          deploy_driver: pxe_libvirt"},{"line_number":1410,"context_line":"          deploy-with-ipa: 0"},{"line_number":1411,"context_line":"          client-from-source: 0"},{"line_number":1412,"context_line":"          ironic-lib-from-source: 0"}],"source_content_type":"text/x-yaml","patch_set":4,"id":"bab6814e_7e9d41bb","line":1409,"in_reply_to":"dab17558_b164c121","updated":"2016-05-26 21:35:37.000000000","message":"News to me too. I suspect we\u0027ll need a followup at some point to cleanup old jobs that aren\u0027t used anymore, but AIUI none of these run unless they are specified in zuul/layout.yaml","commit_id":"86425653403676e82ad99886f029f02489cbadd7"},{"author":{"_account_id":6618,"name":"Ruby Loo","email":"opensrloo@gmail.com","username":"rloo"},"change_message_id":"2b81215f7773dd40da522c216f4430e7893747e0","unresolved":false,"context_lines":[{"line_number":1452,"context_line":"          inspector-enabled: 1"},{"line_number":1453,"context_line":"          tempest-regex: InspectorBasicTest"},{"line_number":1454,"context_line":"          branch-override: \u0027{branch-override}\u0027"},{"line_number":1455,"context_line":"          ramdisk-type: \u0027coreos\u0027"},{"line_number":1456,"context_line":"          gate-tempest-full: 0"},{"line_number":1457,"context_line":"          bm-vm-ram: 1024"},{"line_number":1458,"context_line":"          bm-vm-count: 1"}],"source_content_type":"text/x-yaml","patch_set":4,"id":"dab17558_b132010e","line":1455,"updated":"2016-05-18 01:51:54.000000000","message":"is this because inspector doesn\u0027t work with tinyipa (yet)?","commit_id":"86425653403676e82ad99886f029f02489cbadd7"},{"author":{"_account_id":10342,"name":"Jay Faulkner","display_name":"JayF","email":"jay@jvf.cc","username":"JayF","status":"youtube.com/@oss-gr / podcast.gr-oss.io"},"change_message_id":"26428fdca4bdff2dda558e0f0ca997c85a63d35a","unresolved":false,"context_lines":[{"line_number":1452,"context_line":"          inspector-enabled: 1"},{"line_number":1453,"context_line":"          tempest-regex: InspectorBasicTest"},{"line_number":1454,"context_line":"          branch-override: \u0027{branch-override}\u0027"},{"line_number":1455,"context_line":"          ramdisk-type: \u0027coreos\u0027"},{"line_number":1456,"context_line":"          gate-tempest-full: 0"},{"line_number":1457,"context_line":"          bm-vm-ram: 1024"},{"line_number":1458,"context_line":"          bm-vm-count: 1"}],"source_content_type":"text/x-yaml","patch_set":4,"id":"bab6814e_fe6ad1b1","line":1455,"in_reply_to":"dab17558_b132010e","updated":"2016-05-26 21:35:37.000000000","message":"I\u0027m unsure; but wanted to leave inspector gate changes as a separate change. If Dmitry or other inspector folks want this changed now as well, I\u0027ll gladly do it now.","commit_id":"86425653403676e82ad99886f029f02489cbadd7"},{"author":{"_account_id":6618,"name":"Ruby Loo","email":"opensrloo@gmail.com","username":"rloo"},"change_message_id":"2b81215f7773dd40da522c216f4430e7893747e0","unresolved":false,"context_lines":[{"line_number":3074,"context_line":"      - \u0027{pipeline}-tempest-dsvm-full-lio{job-suffix}\u0027"},{"line_number":3075,"context_line":"      - \u0027{pipeline}-tempest-dsvm-full-test-accounts{job-suffix}\u0027"},{"line_number":3076,"context_line":"      - \u0027{pipeline}-tempest-dsvm-full{job-suffix}\u0027"},{"line_number":3077,"context_line":"      - \u0027{pipeline}-tempest-dsvm-ironic-agent_ssh-src{job-suffix}\u0027"},{"line_number":3078,"context_line":"      - \u0027{pipeline}-tempest-dsvm-ironic-agent_ssh{job-suffix}\u0027"},{"line_number":3079,"context_line":"      - \u0027{pipeline}-tempest-dsvm-ironic-agent_ipmitool{job-suffix}\u0027"},{"line_number":3080,"context_line":"      - \u0027{pipeline}-tempest-dsvm-ironic-parallel{job-suffix}\u0027"}],"source_content_type":"text/x-yaml","patch_set":4,"id":"dab17558_b17fa068","line":3077,"updated":"2016-05-18 01:51:54.000000000","message":"I don\u0027t really understand what these entries are for. If we should be listing all the ironic jobs described above, we should list them here in the same order to make it easier to see if any are missing.\n\nIt seems like there are some missing, but I don\u0027t know if they are missing cuz we don\u0027t want them here, or missing cuz we missed them.","commit_id":"86425653403676e82ad99886f029f02489cbadd7"},{"author":{"_account_id":10342,"name":"Jay Faulkner","display_name":"JayF","email":"jay@jvf.cc","username":"JayF","status":"youtube.com/@oss-gr / podcast.gr-oss.io"},"change_message_id":"26428fdca4bdff2dda558e0f0ca997c85a63d35a","unresolved":false,"context_lines":[{"line_number":3074,"context_line":"      - \u0027{pipeline}-tempest-dsvm-full-lio{job-suffix}\u0027"},{"line_number":3075,"context_line":"      - \u0027{pipeline}-tempest-dsvm-full-test-accounts{job-suffix}\u0027"},{"line_number":3076,"context_line":"      - \u0027{pipeline}-tempest-dsvm-full{job-suffix}\u0027"},{"line_number":3077,"context_line":"      - \u0027{pipeline}-tempest-dsvm-ironic-agent_ssh-src{job-suffix}\u0027"},{"line_number":3078,"context_line":"      - \u0027{pipeline}-tempest-dsvm-ironic-agent_ssh{job-suffix}\u0027"},{"line_number":3079,"context_line":"      - \u0027{pipeline}-tempest-dsvm-ironic-agent_ipmitool{job-suffix}\u0027"},{"line_number":3080,"context_line":"      - \u0027{pipeline}-tempest-dsvm-ironic-parallel{job-suffix}\u0027"}],"source_content_type":"text/x-yaml","patch_set":4,"id":"bab6814e_1e451542","line":3077,"in_reply_to":"dab17558_b17fa068","updated":"2016-05-26 21:35:37.000000000","message":"This is well outside of the scope of my PR, but I believe these are just groupings of jobs. I\u0027m not sure it actually impacts configuration significantly, but my understanding of these pieces is limited.\n\nNone of these will actually run unless in zuul/layout.yaml.","commit_id":"86425653403676e82ad99886f029f02489cbadd7"},{"author":{"_account_id":6618,"name":"Ruby Loo","email":"opensrloo@gmail.com","username":"rloo"},"change_message_id":"2b81215f7773dd40da522c216f4430e7893747e0","unresolved":false,"context_lines":[{"line_number":3075,"context_line":"      - \u0027{pipeline}-tempest-dsvm-full-test-accounts{job-suffix}\u0027"},{"line_number":3076,"context_line":"      - \u0027{pipeline}-tempest-dsvm-full{job-suffix}\u0027"},{"line_number":3077,"context_line":"      - \u0027{pipeline}-tempest-dsvm-ironic-agent_ssh-src{job-suffix}\u0027"},{"line_number":3078,"context_line":"      - \u0027{pipeline}-tempest-dsvm-ironic-agent_ssh{job-suffix}\u0027"},{"line_number":3079,"context_line":"      - \u0027{pipeline}-tempest-dsvm-ironic-agent_ipmitool{job-suffix}\u0027"},{"line_number":3080,"context_line":"      - \u0027{pipeline}-tempest-dsvm-ironic-parallel{job-suffix}\u0027"},{"line_number":3081,"context_line":"      - \u0027{pipeline}-tempest-dsvm-ironic-pxe_ipa-src{job-suffix}\u0027"}],"source_content_type":"text/x-yaml","patch_set":4,"id":"dab17558_7149b860","line":3078,"updated":"2016-05-18 01:51:54.000000000","message":"and agent_ssh-tinyipa-src?\n\nand agent_ssh-coreos-src?","commit_id":"86425653403676e82ad99886f029f02489cbadd7"},{"author":{"_account_id":6618,"name":"Ruby Loo","email":"opensrloo@gmail.com","username":"rloo"},"change_message_id":"2b81215f7773dd40da522c216f4430e7893747e0","unresolved":false,"context_lines":[{"line_number":3079,"context_line":"      - \u0027{pipeline}-tempest-dsvm-ironic-agent_ipmitool{job-suffix}\u0027"},{"line_number":3080,"context_line":"      - \u0027{pipeline}-tempest-dsvm-ironic-parallel{job-suffix}\u0027"},{"line_number":3081,"context_line":"      - \u0027{pipeline}-tempest-dsvm-ironic-pxe_ipa-src{job-suffix}\u0027"},{"line_number":3082,"context_line":"      - \u0027{pipeline}-tempest-dsvm-ironic-pxe_ipa-tinyipa{job-suffix}\u0027"},{"line_number":3083,"context_line":"      - \u0027{pipeline}-tempest-dsvm-ironic-pxe_ipa{job-suffix}\u0027"},{"line_number":3084,"context_line":"      - \u0027{pipeline}-tempest-dsvm-ironic-pxe_ipa-postgres{job-suffix}\u0027"},{"line_number":3085,"context_line":"      - \u0027{pipeline}-tempest-dsvm-ironic-pxe_ipmitool{job-suffix}\u0027"}],"source_content_type":"text/x-yaml","patch_set":4,"id":"dab17558_711b7874","line":3082,"updated":"2016-05-18 01:51:54.000000000","message":"do we need to add another entry with ...tinyipa-src{}?\nand ..pxe_ipa-coreos-src{}?","commit_id":"86425653403676e82ad99886f029f02489cbadd7"},{"author":{"_account_id":6618,"name":"Ruby Loo","email":"opensrloo@gmail.com","username":"rloo"},"change_message_id":"2b81215f7773dd40da522c216f4430e7893747e0","unresolved":false,"context_lines":[{"line_number":3080,"context_line":"      - \u0027{pipeline}-tempest-dsvm-ironic-parallel{job-suffix}\u0027"},{"line_number":3081,"context_line":"      - \u0027{pipeline}-tempest-dsvm-ironic-pxe_ipa-src{job-suffix}\u0027"},{"line_number":3082,"context_line":"      - \u0027{pipeline}-tempest-dsvm-ironic-pxe_ipa-tinyipa{job-suffix}\u0027"},{"line_number":3083,"context_line":"      - \u0027{pipeline}-tempest-dsvm-ironic-pxe_ipa{job-suffix}\u0027"},{"line_number":3084,"context_line":"      - \u0027{pipeline}-tempest-dsvm-ironic-pxe_ipa-postgres{job-suffix}\u0027"},{"line_number":3085,"context_line":"      - \u0027{pipeline}-tempest-dsvm-ironic-pxe_ipmitool{job-suffix}\u0027"},{"line_number":3086,"context_line":"      - \u0027{pipeline}-tempest-dsvm-ironic-pxe_ipmitool-pyghmi-src{job-suffix}\u0027"}],"source_content_type":"text/x-yaml","patch_set":4,"id":"dab17558_b14a005a","line":3083,"updated":"2016-05-18 01:51:54.000000000","message":"..pxe_ipa-full isn\u0027t here either","commit_id":"86425653403676e82ad99886f029f02489cbadd7"},{"author":{"_account_id":6618,"name":"Ruby Loo","email":"opensrloo@gmail.com","username":"rloo"},"change_message_id":"2b81215f7773dd40da522c216f4430e7893747e0","unresolved":false,"context_lines":[{"line_number":3085,"context_line":"      - \u0027{pipeline}-tempest-dsvm-ironic-pxe_ipmitool{job-suffix}\u0027"},{"line_number":3086,"context_line":"      - \u0027{pipeline}-tempest-dsvm-ironic-pxe_ipmitool-pyghmi-src{job-suffix}\u0027"},{"line_number":3087,"context_line":"      - \u0027{pipeline}-tempest-dsvm-ironic-pxe_ssh{job-suffix}\u0027"},{"line_number":3088,"context_line":"      - \u0027{pipeline}-tempest-dsvm-ironic-inspector{job-suffix}\u0027"},{"line_number":3089,"context_line":"      - \u0027{pipeline}-tempest-dsvm-layer4{job-suffix}\u0027"},{"line_number":3090,"context_line":"      - \u0027{pipeline}-tempest-dsvm-lxc-f23{job-suffix}\u0027"},{"line_number":3091,"context_line":"      - \u0027{pipeline}-tempest-dsvm-neutron-dvr-ovs-native{job-suffix}\u0027"}],"source_content_type":"text/x-yaml","patch_set":4,"id":"dab17558_f16be8b7","line":3088,"updated":"2016-05-18 01:51:54.000000000","message":"also missing: \n  {pipeline}-tempest-dsvm-python-ironicclient-src{job-suffix}\n  {pipeline}-tempest-dsvm-ironic-lib-src{job-suffix}\n  {pipeline}-tempest-dsvm-ironic-multitenant-network{job-suffix}\n  {pipeline}-tempest-dsvm-ironic-pxe_libvirt{job-suffix}","commit_id":"86425653403676e82ad99886f029f02489cbadd7"},{"author":{"_account_id":10239,"name":"Dmitry Tantsur","email":"dtantsur@protonmail.com","username":"dtantsur"},"change_message_id":"3315ebdba0e42332b23538fdd8b4bb5df6f8c25b","unresolved":false,"context_lines":[{"line_number":1407,"context_line":"          inspector-enabled: 1"},{"line_number":1408,"context_line":"          tempest-regex: InspectorBasicTest"},{"line_number":1409,"context_line":"          branch-override: \u0027{branch-override}\u0027"},{"line_number":1410,"context_line":"          ramdisk-type: \u0027coreos\u0027"},{"line_number":1411,"context_line":"          gate-tempest-full: 0"},{"line_number":1412,"context_line":"          bm-vm-count: 1"},{"line_number":1413,"context_line":"          multitenant-network: 0"}],"source_content_type":"text/x-yaml","patch_set":6,"id":"9abb7d3a_08d664f0","line":1410,"range":{"start_line":1410,"start_character":25,"end_line":1410,"end_character":31},"updated":"2016-06-01 15:40:31.000000000","message":"this can be changed too","commit_id":"af18c75c7ffeb32eee2e38f8eff1807dcc6ba0bc"},{"author":{"_account_id":10343,"name":"Jim Rollenhagen","email":"jim@jimrollenhagen.com","username":"jimrollenhagen"},"change_message_id":"a64b0c5c5e1ec9212592c0c3c76378fa6089854b","unresolved":false,"context_lines":[{"line_number":689,"context_line":"                          export DEVSTACK_LOCAL_CONFIG+\u003d$\u0027\\n\u0027\"IRONIC_VM_SPECS_RAM\u003d384\""},{"line_number":690,"context_line":"                          ;;"},{"line_number":691,"context_line":"                      \u0027coreos\u0027)"},{"line_number":692,"context_line":"                          export DEVSTACK_LOCAL_CONFIG+\u003d$\u0027\\n\u0027\"IRONIC_VM_SPECS_RAM\u003d1024\""},{"line_number":693,"context_line":"                          ;;"},{"line_number":694,"context_line":"                  # if using a ramdisk without a known good value, use the devstack"},{"line_number":695,"context_line":"                  # default by not exporting any value for IRONIC_VM_SPECS_RAM"}],"source_content_type":"text/x-yaml","patch_set":8,"id":"9abb7d3a_cb8f2473","line":692,"updated":"2016-06-01 21:19:01.000000000","message":"We still want to export this in stable/liberty, right?","commit_id":"21f07be50d8fba6325d87508888ae7a9ab541710"}],"jenkins/jobs/projects.yaml":[{"author":{"_account_id":18893,"name":"Miles Gould","email":"mgould@redhat.com","username":"pozorvlak"},"change_message_id":"b5ca216066dc9dd4961c9e5d7ce5ddfb00c1bc95","unresolved":false,"context_lines":[{"line_number":2493,"context_line":"          node: ubuntu-trusty"},{"line_number":2494,"context_line":"          job-suffix: \u0027-nv\u0027"},{"line_number":2495,"context_line":"          branch-override: default"},{"line_number":2496,"context_line":"      - \u0027{pipeline}-tempest-dsvm-ironic-agent_ssh-coreos-src{job-suffix}\u0027:"},{"line_number":2497,"context_line":"          pipeline: gate"},{"line_number":2498,"context_line":"          node: ubuntu-trusty"},{"line_number":2499,"context_line":"          job-suffix: \u0027\u0027"}],"source_content_type":"text/x-yaml","patch_set":3,"id":"dab17558_71c84275","line":2496,"updated":"2016-05-13 14:43:19.000000000","message":"This is under the \"ironic\" project - does this (and line 2506 below) mean that Ironic is still gating on coreos?","commit_id":"281c903a24b79b7017a7d653e58e0862526b6af9"},{"author":{"_account_id":10342,"name":"Jay Faulkner","display_name":"JayF","email":"jay@jvf.cc","username":"JayF","status":"youtube.com/@oss-gr / podcast.gr-oss.io"},"change_message_id":"7c926956260eb146942ad7e3e9fe5055e9300f8a","unresolved":false,"context_lines":[{"line_number":2493,"context_line":"          node: ubuntu-trusty"},{"line_number":2494,"context_line":"          job-suffix: \u0027-nv\u0027"},{"line_number":2495,"context_line":"          branch-override: default"},{"line_number":2496,"context_line":"      - \u0027{pipeline}-tempest-dsvm-ironic-agent_ssh-coreos-src{job-suffix}\u0027:"},{"line_number":2497,"context_line":"          pipeline: gate"},{"line_number":2498,"context_line":"          node: ubuntu-trusty"},{"line_number":2499,"context_line":"          job-suffix: \u0027\u0027"}],"source_content_type":"text/x-yaml","patch_set":3,"id":"dab17558_52cc8069","line":2496,"in_reply_to":"dab17558_71c84275","updated":"2016-05-17 20:04:08.000000000","message":"The location of the job in this yaml file is mainly for sorting; as you can see in zuul/layout.yaml, these jobs gate for ironic-python-agent, not Ironic proper.","commit_id":"281c903a24b79b7017a7d653e58e0862526b6af9"}]}