)]}'
{"devstack/lib/magnum":[{"author":{"_account_id":9591,"name":"Ton Ngo","email":"ton@us.ibm.com","username":"tango"},"change_message_id":"d82592a49ba75297877aa30f8729265b6d7635a3","unresolved":false,"context_lines":[{"line_number":288,"context_line":"        fi"},{"line_number":289,"context_line":"    done"},{"line_number":290,"context_line":"    if [ -z ${image_name} ]; then"},{"line_number":291,"context_line":"        echo \"Do not know what to do with $image_filename\"; false"},{"line_number":292,"context_line":"    fi"},{"line_number":293,"context_line":""},{"line_number":294,"context_line":"    openstack --os-url $GLANCE_SERVICE_PROTOCOL://$GLANCE_HOSTPORT --os-image-api-version 1 image set $image_name $magnum_image_property"}],"source_content_type":"application/x-shellscript","patch_set":31,"id":"fad55976_54ca9150","line":291,"range":{"start_line":291,"start_character":14,"end_line":291,"end_character":57},"updated":"2016-07-15 18:02:51.000000000","message":"Suggestion for a more helpful error message:\n\"Unknown image format, expecting name with extension:  tgz, img, qcow2, iso, vhd, vhdx, tar.gz, img.gz, img.bz2, vhd.gz, vhdx.gz\"","commit_id":"7e7bb92a9b509d2b2b17b492170ddf181005eb55"},{"author":{"_account_id":20498,"name":"Spyros Trigazis","email":"spyridon.trigazis@cern.ch","username":"strigazi"},"change_message_id":"9293d3303471e5121c1ba6750f3b25f26dc107a2","unresolved":false,"context_lines":[{"line_number":288,"context_line":"        fi"},{"line_number":289,"context_line":"    done"},{"line_number":290,"context_line":"    if [ -z ${image_name} ]; then"},{"line_number":291,"context_line":"        echo \"Do not know what to do with $image_filename\"; false"},{"line_number":292,"context_line":"    fi"},{"line_number":293,"context_line":""},{"line_number":294,"context_line":"    openstack --os-url $GLANCE_SERVICE_PROTOCOL://$GLANCE_HOSTPORT --os-image-api-version 1 image set $image_name $magnum_image_property"}],"source_content_type":"application/x-shellscript","patch_set":31,"id":"fad55976_58cb861e","line":291,"range":{"start_line":291,"start_character":14,"end_line":291,"end_character":57},"in_reply_to":"fad55976_54ca9150","updated":"2016-07-16 13:59:46.000000000","message":"Makes sense. It looks much better like this.\n\nAlthough, to justify my choice:\nhttps://github.com/openstack-dev/devstack/blob/master/functions#L327\n\n:)","commit_id":"7e7bb92a9b509d2b2b17b492170ddf181005eb55"},{"author":{"_account_id":9591,"name":"Ton Ngo","email":"ton@us.ibm.com","username":"tango"},"change_message_id":"c7fed1ab765d014cdeea9c18c80414a8d8b7fd1d","unresolved":false,"context_lines":[{"line_number":288,"context_line":"        fi"},{"line_number":289,"context_line":"    done"},{"line_number":290,"context_line":"    if [ -z ${image_name} ]; then"},{"line_number":291,"context_line":"        echo \"Do not know what to do with $image_filename\"; false"},{"line_number":292,"context_line":"    fi"},{"line_number":293,"context_line":""},{"line_number":294,"context_line":"    openstack --os-url $GLANCE_SERVICE_PROTOCOL://$GLANCE_HOSTPORT --os-image-api-version 1 image set $image_name $magnum_image_property"}],"source_content_type":"application/x-shellscript","patch_set":31,"id":"dada55a8_4a6c0b35","line":291,"range":{"start_line":291,"start_character":14,"end_line":291,"end_character":57},"in_reply_to":"fad55976_58cb861e","updated":"2016-07-20 17:04:19.000000000","message":"Yeah I saw that before too. Maybe we can fix that message also while we are at it.","commit_id":"7e7bb92a9b509d2b2b17b492170ddf181005eb55"},{"author":{"_account_id":20498,"name":"Spyros Trigazis","email":"spyridon.trigazis@cern.ch","username":"strigazi"},"change_message_id":"578579296e7e91a95af03ecc7230099115ff0564","unresolved":false,"context_lines":[{"line_number":67,"context_line":"    MAGNUM_BIN_DIR\u003d$(get_python_exec_prefix)"},{"line_number":68,"context_line":"fi"},{"line_number":69,"context_line":""},{"line_number":70,"context_line":"MAGNUM_CONFIGURE_IPTABLES\u003d${MAGNUM_CONFIGURE_IPTABLES:-True}"},{"line_number":71,"context_line":""},{"line_number":72,"context_line":"# Functions"},{"line_number":73,"context_line":"# ---------"}],"source_content_type":"application/x-shellscript","patch_set":58,"id":"bacf61ea_472d4845","line":70,"range":{"start_line":70,"start_character":0,"end_line":70,"end_character":60},"updated":"2016-08-04 14:12:10.000000000","message":"This was maybe crucial! :-)","commit_id":"9acd7c8e7c82c50bc6a1d51c0f76e64ba7fdf928"},{"author":{"_account_id":11650,"name":"Tom Cammann","email":"tom.cammann@hpe.com","username":"cammann"},"change_message_id":"eecb70d7d5aa5f607b349cf652256cb78fffba68","unresolved":false,"context_lines":[{"line_number":278,"context_line":"    # get the image name"},{"line_number":279,"context_line":"    local image_filename\u003d$(basename \"$MAGNUM_GUEST_IMAGE_URL\")"},{"line_number":280,"context_line":"    local image_name\u003d\"\""},{"line_number":281,"context_line":"    for extension in \"tgz\" \"img\" \"qcow2\" \"iso\" \"vhd\" \"vhdx\" \"tar.gz\" \"img.gz\" \"img.bz2\" \"vhd.gz\" \"vhdx.gz\""},{"line_number":282,"context_line":"    do"},{"line_number":283,"context_line":"        if [ $(expr match \"${image_filename}\" \".*\\.${extension}$\") -ne 0 ]; then"},{"line_number":284,"context_line":"            image_name\u003d$(basename \"$image_filename\" \".${extension}\")"}],"source_content_type":"application/x-shellscript","patch_set":59,"id":"bacf61ea_f790fc44","line":281,"updated":"2016-08-08 01:38:03.000000000","message":"move this list to a const somewhere to make this line more readable","commit_id":"d80fbffbf846fac61d98b2a7d1b563f41a336e01"},{"author":{"_account_id":11650,"name":"Tom Cammann","email":"tom.cammann@hpe.com","username":"cammann"},"change_message_id":"eecb70d7d5aa5f607b349cf652256cb78fffba68","unresolved":false,"context_lines":[{"line_number":286,"context_line":"        fi"},{"line_number":287,"context_line":"    done"},{"line_number":288,"context_line":"    if [ -z ${image_name} ]; then"},{"line_number":289,"context_line":"        echo \"Unknown image extension in $image_filename, supported extensions: tgz, img, qcow2, iso, vhd, vhdx, tar.gz, img.gz, img.bz2, vhd.gz, vhdx.gz\"; false"},{"line_number":290,"context_line":"    fi"},{"line_number":291,"context_line":""},{"line_number":292,"context_line":"    openstack --os-url $GLANCE_SERVICE_PROTOCOL://$GLANCE_HOSTPORT --os-image-api-version 1 image set $image_name $magnum_image_property"}],"source_content_type":"application/x-shellscript","patch_set":59,"id":"bacf61ea_777ecc19","line":289,"range":{"start_line":289,"start_character":156,"end_line":289,"end_character":161},"updated":"2016-08-08 01:38:03.000000000","message":"should this be return false?","commit_id":"d80fbffbf846fac61d98b2a7d1b563f41a336e01"},{"author":{"_account_id":11650,"name":"Tom Cammann","email":"tom.cammann@hpe.com","username":"cammann"},"change_message_id":"eecb70d7d5aa5f607b349cf652256cb78fffba68","unresolved":false,"context_lines":[{"line_number":289,"context_line":"        echo \"Unknown image extension in $image_filename, supported extensions: tgz, img, qcow2, iso, vhd, vhdx, tar.gz, img.gz, img.bz2, vhd.gz, vhdx.gz\"; false"},{"line_number":290,"context_line":"    fi"},{"line_number":291,"context_line":""},{"line_number":292,"context_line":"    openstack --os-url $GLANCE_SERVICE_PROTOCOL://$GLANCE_HOSTPORT --os-image-api-version 1 image set $image_name $magnum_image_property"},{"line_number":293,"context_line":"}"},{"line_number":294,"context_line":""},{"line_number":295,"context_line":"# install_magnumclient() - Collect source and prepare"}],"source_content_type":"application/x-shellscript","patch_set":59,"id":"bacf61ea_b774c435","line":292,"updated":"2016-08-08 01:38:03.000000000","message":"break long lines","commit_id":"d80fbffbf846fac61d98b2a7d1b563f41a336e01"}],"magnum/tests/contrib/gate_hook.sh":[{"author":{"_account_id":11536,"name":"hongbin","email":"hongbin034@gmail.com","username":"hongbin"},"change_message_id":"926ecf3dfbc4879ec491c107c802b74adc8d3021","unresolved":false,"context_lines":[{"line_number":26,"context_line":"    echo \"MAGNUM_GUEST_IMAGE_URL\u003dhttps://fedorapeople.org/groups/magnum/ubuntu-14.04.3-mesos-0.25.0.qcow2\" \u003e\u003e $BASE/new/devstack/localrc"},{"line_number":27,"context_line":"elif [ \"$coe\" \u003d \"k8s-coreos\" ]; then"},{"line_number":28,"context_line":"    echo \"MAGNUM_GUEST_IMAGE_URL\u003dhttp://beta.release.core-os.net/amd64-usr/current/coreos_production_openstack_image.img.bz2\" \u003e\u003e $BASE/new/devstack/localrc"},{"line_number":29,"context_line":"elif [ \"${coe}${special}\" \u003d \"k8s-ironic\" ]; then"},{"line_number":30,"context_line":"  # https://fedorapeople.org/groups/magnum/fedora-23-kubernetes-ironic.qcow2"},{"line_number":31,"context_line":"  # https://fedorapeople.org/groups/magnum/fedora-23-kubernetes-ironic.initrd"},{"line_number":32,"context_line":"  # https://fedorapeople.org/groups/magnum/fedora-23-kubernetes-ironic.vmlinuz"}],"source_content_type":"text/x-sh","patch_set":4,"id":"1aa78d24_efe0d4ce","line":29,"range":{"start_line":29,"start_character":8,"end_line":29,"end_character":24},"updated":"2016-07-05 02:54:45.000000000","message":"I wonder why not make \"${coe}\" \u003d \"k8s-ironic\"","commit_id":"da8063810bb77a683bd0cd0dc3b74105b5c22a50"},{"author":{"_account_id":12385,"name":"Motohiro/Yuanying Otsuka","email":"yuanying@fraction.jp","username":"yuanying"},"change_message_id":"c950f110d7bedddd3e20ea6fa934ce6b2f26a547","unresolved":false,"context_lines":[{"line_number":26,"context_line":"    echo \"MAGNUM_GUEST_IMAGE_URL\u003dhttps://fedorapeople.org/groups/magnum/ubuntu-14.04.3-mesos-0.25.0.qcow2\" \u003e\u003e $BASE/new/devstack/localrc"},{"line_number":27,"context_line":"elif [ \"$coe\" \u003d \"k8s-coreos\" ]; then"},{"line_number":28,"context_line":"    echo \"MAGNUM_GUEST_IMAGE_URL\u003dhttp://beta.release.core-os.net/amd64-usr/current/coreos_production_openstack_image.img.bz2\" \u003e\u003e $BASE/new/devstack/localrc"},{"line_number":29,"context_line":"elif [ \"${coe}${special}\" \u003d \"k8s-ironic\" ]; then"},{"line_number":30,"context_line":"  # https://fedorapeople.org/groups/magnum/fedora-23-kubernetes-ironic.qcow2"},{"line_number":31,"context_line":"  # https://fedorapeople.org/groups/magnum/fedora-23-kubernetes-ironic.initrd"},{"line_number":32,"context_line":"  # https://fedorapeople.org/groups/magnum/fedora-23-kubernetes-ironic.vmlinuz"}],"source_content_type":"text/x-sh","patch_set":4,"id":"1aa78d24_222af32c","line":29,"range":{"start_line":29,"start_character":8,"end_line":29,"end_character":24},"in_reply_to":"1aa78d24_efe0d4ce","updated":"2016-07-05 03:38:15.000000000","message":"I think coe for this functional test is k8s and special parameter is ironic.\nPlease see also below patch.\n\n* https://review.openstack.org/#/c/337386/1/jenkins/jobs/projects.yaml","commit_id":"da8063810bb77a683bd0cd0dc3b74105b5c22a50"},{"author":{"_account_id":11536,"name":"hongbin","email":"hongbin034@gmail.com","username":"hongbin"},"change_message_id":"926ecf3dfbc4879ec491c107c802b74adc8d3021","unresolved":false,"context_lines":[{"line_number":27,"context_line":"elif [ \"$coe\" \u003d \"k8s-coreos\" ]; then"},{"line_number":28,"context_line":"    echo \"MAGNUM_GUEST_IMAGE_URL\u003dhttp://beta.release.core-os.net/amd64-usr/current/coreos_production_openstack_image.img.bz2\" \u003e\u003e $BASE/new/devstack/localrc"},{"line_number":29,"context_line":"elif [ \"${coe}${special}\" \u003d \"k8s-ironic\" ]; then"},{"line_number":30,"context_line":"  # https://fedorapeople.org/groups/magnum/fedora-23-kubernetes-ironic.qcow2"},{"line_number":31,"context_line":"  # https://fedorapeople.org/groups/magnum/fedora-23-kubernetes-ironic.initrd"},{"line_number":32,"context_line":"  # https://fedorapeople.org/groups/magnum/fedora-23-kubernetes-ironic.vmlinuz"},{"line_number":33,"context_line":"  # FIXME(yuanying): Folowing tarball must include above 3 images."}],"source_content_type":"text/x-sh","patch_set":4,"id":"1aa78d24_0f056079","line":30,"range":{"start_line":30,"start_character":0,"end_line":30,"end_character":2},"updated":"2016-07-05 02:54:45.000000000","message":"four spaces.","commit_id":"da8063810bb77a683bd0cd0dc3b74105b5c22a50"},{"author":{"_account_id":12385,"name":"Motohiro/Yuanying Otsuka","email":"yuanying@fraction.jp","username":"yuanying"},"change_message_id":"c950f110d7bedddd3e20ea6fa934ce6b2f26a547","unresolved":false,"context_lines":[{"line_number":27,"context_line":"elif [ \"$coe\" \u003d \"k8s-coreos\" ]; then"},{"line_number":28,"context_line":"    echo \"MAGNUM_GUEST_IMAGE_URL\u003dhttp://beta.release.core-os.net/amd64-usr/current/coreos_production_openstack_image.img.bz2\" \u003e\u003e $BASE/new/devstack/localrc"},{"line_number":29,"context_line":"elif [ \"${coe}${special}\" \u003d \"k8s-ironic\" ]; then"},{"line_number":30,"context_line":"  # https://fedorapeople.org/groups/magnum/fedora-23-kubernetes-ironic.qcow2"},{"line_number":31,"context_line":"  # https://fedorapeople.org/groups/magnum/fedora-23-kubernetes-ironic.initrd"},{"line_number":32,"context_line":"  # https://fedorapeople.org/groups/magnum/fedora-23-kubernetes-ironic.vmlinuz"},{"line_number":33,"context_line":"  # FIXME(yuanying): Folowing tarball must include above 3 images."}],"source_content_type":"text/x-sh","patch_set":4,"id":"1aa78d24_82212750","line":30,"range":{"start_line":30,"start_character":0,"end_line":30,"end_character":2},"in_reply_to":"1aa78d24_0f056079","updated":"2016-07-05 03:38:15.000000000","message":"Done","commit_id":"da8063810bb77a683bd0cd0dc3b74105b5c22a50"},{"author":{"_account_id":20498,"name":"Spyros Trigazis","email":"spyridon.trigazis@cern.ch","username":"strigazi"},"change_message_id":"86bb38b6b363e05aade146568fbf608a6da5929f","unresolved":false,"context_lines":[{"line_number":32,"context_line":""},{"line_number":33,"context_line":"    # NOTE(yuanying): Current implementation requires only 1 subnet under network"},{"line_number":34,"context_line":"    export DEVSTACK_LOCAL_CONFIG+\u003d$\u0027\\n\u0027\"IP_VERSION\u003d4\""},{"line_number":35,"context_line":"    export DEVSTACK_LOCAL_CONFIG+\u003d$\u0027\\n\u0027\"disable_service cinder c-sch c-api c-vol\""},{"line_number":36,"context_line":"    export DEVSTACK_LOCAL_CONFIG+\u003d$\u0027\\n\u0027\"SWIFT_ENABLE_TEMPURLS\u003dTrue\""},{"line_number":37,"context_line":"    export DEVSTACK_LOCAL_CONFIG+\u003d$\u0027\\n\u0027\"SWIFT_TEMPURL_KEY\u003dtempurlkey\""},{"line_number":38,"context_line":"    export DEVSTACK_LOCAL_CONFIG+\u003d$\u0027\\n\u0027\"SWIFT_HASH\u003dpassword\""}],"source_content_type":"text/x-sh","patch_set":15,"id":"1aa78d24_e5609e37","line":35,"range":{"start_line":35,"start_character":69,"end_line":35,"end_character":74},"updated":"2016-07-12 08:57:46.000000000","message":"Let\u0027s keep cinder for now.\nhttp://logs.openstack.org/33/336433/15/check/gate-functional-dsvm-magnum-k8s-ironic-nv/83da95e/logs/devstacklog.txt.gz","commit_id":"7353f0674e4edbcaef59c2c98f5df38cc632517b"},{"author":{"_account_id":9591,"name":"Ton Ngo","email":"ton@us.ibm.com","username":"tango"},"change_message_id":"d82592a49ba75297877aa30f8729265b6d7635a3","unresolved":false,"context_lines":[{"line_number":27,"context_line":"elif [ \"$coe\" \u003d \"k8s-coreos\" ]; then"},{"line_number":28,"context_line":"    echo \"MAGNUM_GUEST_IMAGE_URL\u003dhttp://beta.release.core-os.net/amd64-usr/current/coreos_production_openstack_image.img.bz2\" \u003e\u003e $BASE/new/devstack/localrc"},{"line_number":29,"context_line":"elif [ \"${coe}${special}\" \u003d \"k8s-ironic\" ]; then"},{"line_number":30,"context_line":"    export DEVSTACK_LOCAL_CONFIG+\u003d$\u0027\\n\u0027\"MAGNUM_GUEST_IMAGE_URL\u003d\u0027https://test-fedora-23-ironic-image.web.cern.ch/test-fedora-23-ironic-image/fedora-23-kubernetes-ironic.tar.gz\u0027\""},{"line_number":31,"context_line":"    export DEVSTACK_LOCAL_CONFIG+\u003d$\u0027\\n\u0027\"MAGNUM_IMAGE_NAME\u003d\u0027fedora-23-kubernetes-ironic\u0027\""},{"line_number":32,"context_line":""},{"line_number":33,"context_line":"    export DEVSTACK_GATE_VIRT_DRIVER\u003d\"fake_ironic\""}],"source_content_type":"text/x-sh","patch_set":31,"id":"fad55976_65955d3f","line":30,"range":{"start_line":30,"start_character":64,"end_line":30,"end_character":174},"updated":"2016-07-15 18:02:51.000000000","message":"Is this the image tar file that needs to be placed on the fedora site?","commit_id":"7e7bb92a9b509d2b2b17b492170ddf181005eb55"},{"author":{"_account_id":20498,"name":"Spyros Trigazis","email":"spyridon.trigazis@cern.ch","username":"strigazi"},"change_message_id":"9293d3303471e5121c1ba6750f3b25f26dc107a2","unresolved":false,"context_lines":[{"line_number":27,"context_line":"elif [ \"$coe\" \u003d \"k8s-coreos\" ]; then"},{"line_number":28,"context_line":"    echo \"MAGNUM_GUEST_IMAGE_URL\u003dhttp://beta.release.core-os.net/amd64-usr/current/coreos_production_openstack_image.img.bz2\" \u003e\u003e $BASE/new/devstack/localrc"},{"line_number":29,"context_line":"elif [ \"${coe}${special}\" \u003d \"k8s-ironic\" ]; then"},{"line_number":30,"context_line":"    export DEVSTACK_LOCAL_CONFIG+\u003d$\u0027\\n\u0027\"MAGNUM_GUEST_IMAGE_URL\u003d\u0027https://test-fedora-23-ironic-image.web.cern.ch/test-fedora-23-ironic-image/fedora-23-kubernetes-ironic.tar.gz\u0027\""},{"line_number":31,"context_line":"    export DEVSTACK_LOCAL_CONFIG+\u003d$\u0027\\n\u0027\"MAGNUM_IMAGE_NAME\u003d\u0027fedora-23-kubernetes-ironic\u0027\""},{"line_number":32,"context_line":""},{"line_number":33,"context_line":"    export DEVSTACK_GATE_VIRT_DRIVER\u003d\"fake_ironic\""}],"source_content_type":"text/x-sh","patch_set":31,"id":"fad55976_f85fda3e","line":30,"range":{"start_line":30,"start_character":64,"end_line":30,"end_character":174},"in_reply_to":"fad55976_65955d3f","updated":"2016-07-16 13:59:46.000000000","message":"Yes, it\u0027s the same with the current tarball with corrected filenames.\nI didn\u0027t have access to upload it there. hongbin and sdake who are admins and can give access. Hongbin was o vacations and sdake didn\u0027t reply.","commit_id":"7e7bb92a9b509d2b2b17b492170ddf181005eb55"},{"author":{"_account_id":20498,"name":"Spyros Trigazis","email":"spyridon.trigazis@cern.ch","username":"strigazi"},"change_message_id":"be7e6b26100df932b031ee2b97d1868fba62f695","unresolved":false,"context_lines":[{"line_number":27,"context_line":"elif [ \"$coe\" \u003d \"k8s-coreos\" ]; then"},{"line_number":28,"context_line":"    echo \"MAGNUM_GUEST_IMAGE_URL\u003dhttp://beta.release.core-os.net/amd64-usr/current/coreos_production_openstack_image.img.bz2\" \u003e\u003e $BASE/new/devstack/localrc"},{"line_number":29,"context_line":"elif [ \"${coe}${special}\" \u003d \"k8s-ironic\" ]; then"},{"line_number":30,"context_line":"    export DEVSTACK_LOCAL_CONFIG+\u003d$\u0027\\n\u0027\"MAGNUM_GUEST_IMAGE_URL\u003d\u0027https://test-fedora-23-ironic-image.web.cern.ch/test-fedora-23-ironic-image/fedora-23-kubernetes-ironic.tar.gz\u0027\""},{"line_number":31,"context_line":"    export DEVSTACK_LOCAL_CONFIG+\u003d$\u0027\\n\u0027\"MAGNUM_IMAGE_NAME\u003d\u0027fedora-23-kubernetes-ironic\u0027\""},{"line_number":32,"context_line":""},{"line_number":33,"context_line":"    export DEVSTACK_GATE_VIRT_DRIVER\u003d\"fake_ironic\""}],"source_content_type":"text/x-sh","patch_set":31,"id":"dada55a8_6cc43340","line":30,"range":{"start_line":30,"start_character":64,"end_line":30,"end_character":174},"in_reply_to":"fad55976_f85fda3e","updated":"2016-07-17 12:14:06.000000000","message":"sha256sum fedora-23-kubernetes-ironic.tar.gz \n748cc787068db7e7557286cb1fa8dd52d761e4b12ffe3f13e8e40af5312bcb61  fedora-23-kubernetes-ironic.tar.gz","commit_id":"7e7bb92a9b509d2b2b17b492170ddf181005eb55"},{"author":{"_account_id":20498,"name":"Spyros Trigazis","email":"spyridon.trigazis@cern.ch","username":"strigazi"},"change_message_id":"84c7936389b1ee2fc6671bce787124012b528125","unresolved":false,"context_lines":[{"line_number":30,"context_line":"    export DEVSTACK_LOCAL_CONFIG+\u003d$\u0027\\n\u0027\"MAGNUM_GUEST_IMAGE_URL\u003d\u0027https://test-fedora-23-ironic-image.web.cern.ch/test-fedora-23-ironic-image/fedora-23-kubernetes-ironic.tar.gz\u0027\""},{"line_number":31,"context_line":"    export DEVSTACK_LOCAL_CONFIG+\u003d$\u0027\\n\u0027\"MAGNUM_IMAGE_NAME\u003d\u0027fedora-23-kubernetes-ironic\u0027\""},{"line_number":32,"context_line":""},{"line_number":33,"context_line":"    export DEVSTACK_GATE_VIRT_DRIVER\u003d\"fake_ironic\""},{"line_number":34,"context_line":"    # NOTE(yuanying): Current implementation requires only 1 subnet under network"},{"line_number":35,"context_line":"    export DEVSTACK_LOCAL_CONFIG+\u003d$\u0027\\n\u0027\"IP_VERSION\u003d4\""},{"line_number":36,"context_line":"    # NOTE(strigazi) keep cinder"}],"source_content_type":"text/x-sh","patch_set":31,"id":"dada55a8_6981c58f","line":33,"range":{"start_line":33,"start_character":38,"end_line":33,"end_character":49},"updated":"2016-07-17 11:30:34.000000000","message":"let\u0027s pick ironic OR fake.\nI\u0027ll try ironic which worked for me.","commit_id":"7e7bb92a9b509d2b2b17b492170ddf181005eb55"},{"author":{"_account_id":9591,"name":"Ton Ngo","email":"ton@us.ibm.com","username":"tango"},"change_message_id":"99128e636597c90b8182287b95e154139da15215","unresolved":false,"context_lines":[{"line_number":27,"context_line":"elif [ \"$coe\" \u003d \"k8s-coreos\" ]; then"},{"line_number":28,"context_line":"    echo \"MAGNUM_GUEST_IMAGE_URL\u003dhttp://beta.release.core-os.net/amd64-usr/current/coreos_production_openstack_image.img.bz2\" \u003e\u003e $BASE/new/devstack/localrc"},{"line_number":29,"context_line":"elif [ \"${coe}${special}\" \u003d \"k8s-ironic\" ]; then"},{"line_number":30,"context_line":"    export DEVSTACK_LOCAL_CONFIG+\u003d$\u0027\\n\u0027\"MAGNUM_GUEST_IMAGE_URL\u003d\u0027https://test-fedora-23-ironic-image.web.cern.ch/test-fedora-23-ironic-image/fedora-23-kubernetes-ironic.tar.gz\u0027\""},{"line_number":31,"context_line":"    export DEVSTACK_LOCAL_CONFIG+\u003d$\u0027\\n\u0027\"MAGNUM_IMAGE_NAME\u003d\u0027fedora-23-kubernetes-ironic\u0027\""},{"line_number":32,"context_line":""},{"line_number":33,"context_line":"    export DEVSTACK_GATE_VIRT_DRIVER\u003d\"ironic\""}],"source_content_type":"text/x-sh","patch_set":35,"id":"dada55a8_1ef898c2","line":30,"range":{"start_line":30,"start_character":64,"end_line":30,"end_character":174},"updated":"2016-07-20 17:15:37.000000000","message":"I upload this file to the Fedora site, so we can download from there now:\nhttps://fedorapeople.org/groups/magnum/fedora-23-kubernetes-ironic.tar.gz","commit_id":"a0de51492310d4d2808bc6ecb78fef5695d1a54f"},{"author":{"_account_id":12385,"name":"Motohiro/Yuanying Otsuka","email":"yuanying@fraction.jp","username":"yuanying"},"change_message_id":"78fb730e28e9eb7003b487a5a9c0d093ba239870","unresolved":false,"context_lines":[{"line_number":27,"context_line":"elif [ \"$coe\" \u003d \"k8s-coreos\" ]; then"},{"line_number":28,"context_line":"    echo \"MAGNUM_GUEST_IMAGE_URL\u003dhttp://beta.release.core-os.net/amd64-usr/current/coreos_production_openstack_image.img.bz2\" \u003e\u003e $BASE/new/devstack/localrc"},{"line_number":29,"context_line":"elif [ \"${coe}${special}\" \u003d \"k8s-ironic\" ]; then"},{"line_number":30,"context_line":"    export DEVSTACK_LOCAL_CONFIG+\u003d$\u0027\\n\u0027\"MAGNUM_GUEST_IMAGE_URL\u003d\u0027https://test-fedora-23-ironic-image.web.cern.ch/test-fedora-23-ironic-image/fedora-23-kubernetes-ironic.tar.gz\u0027\""},{"line_number":31,"context_line":"    export DEVSTACK_LOCAL_CONFIG+\u003d$\u0027\\n\u0027\"MAGNUM_IMAGE_NAME\u003d\u0027fedora-23-kubernetes-ironic\u0027\""},{"line_number":32,"context_line":""},{"line_number":33,"context_line":"    export DEVSTACK_GATE_VIRT_DRIVER\u003d\"ironic\""}],"source_content_type":"text/x-sh","patch_set":35,"id":"dada55a8_2750d6d3","line":30,"range":{"start_line":30,"start_character":64,"end_line":30,"end_character":174},"in_reply_to":"dada55a8_1ef898c2","updated":"2016-07-21 07:13:37.000000000","message":"Thanks Tom, I fixed it!","commit_id":"a0de51492310d4d2808bc6ecb78fef5695d1a54f"},{"author":{"_account_id":12385,"name":"Motohiro/Yuanying Otsuka","email":"yuanying@fraction.jp","username":"yuanying"},"change_message_id":"eafca48c6e451cb430c460fe17624b244a987c99","unresolved":false,"context_lines":[{"line_number":27,"context_line":"elif [ \"$coe\" \u003d \"k8s-coreos\" ]; then"},{"line_number":28,"context_line":"    echo \"MAGNUM_GUEST_IMAGE_URL\u003dhttp://beta.release.core-os.net/amd64-usr/current/coreos_production_openstack_image.img.bz2\" \u003e\u003e $BASE/new/devstack/localrc"},{"line_number":29,"context_line":"elif [ \"${coe}${special}\" \u003d \"k8s-ironic\" ]; then"},{"line_number":30,"context_line":"    export DEVSTACK_LOCAL_CONFIG+\u003d$\u0027\\n\u0027\"MAGNUM_GUEST_IMAGE_URL\u003d\u0027https://test-fedora-23-ironic-image.web.cern.ch/test-fedora-23-ironic-image/fedora-23-kubernetes-ironic.tar.gz\u0027\""},{"line_number":31,"context_line":"    export DEVSTACK_LOCAL_CONFIG+\u003d$\u0027\\n\u0027\"MAGNUM_IMAGE_NAME\u003d\u0027fedora-23-kubernetes-ironic\u0027\""},{"line_number":32,"context_line":""},{"line_number":33,"context_line":"    export DEVSTACK_GATE_VIRT_DRIVER\u003d\"ironic\""}],"source_content_type":"text/x-sh","patch_set":35,"id":"dada55a8_e7f7ded0","line":30,"range":{"start_line":30,"start_character":64,"end_line":30,"end_character":174},"in_reply_to":"dada55a8_2750d6d3","updated":"2016-07-21 07:14:52.000000000","message":"@Ton: Sorry, My shit IM mistook your name..","commit_id":"a0de51492310d4d2808bc6ecb78fef5695d1a54f"}],"magnum/tests/contrib/post_test_hook.sh":[{"author":{"_account_id":20498,"name":"Spyros Trigazis","email":"spyridon.trigazis@cern.ch","username":"strigazi"},"change_message_id":"2d74269f3f6f89a4fca2d134519d8b237d23b09f","unresolved":false,"context_lines":[{"line_number":36,"context_line":"        local image_name\u003d\"coreos\""},{"line_number":37,"context_line":"        local container_format\u003d\"bare\""},{"line_number":38,"context_line":"    elif [ \"${coe}${special}\" \u003d\u003d \u0027k8s-ironic\u0027 ]; then"},{"line_number":39,"context_line":"        local flavor_id\u003d\u0027baremetal\u0027"},{"line_number":40,"context_line":"        local master_flavor_id\u003d\u0027baremetal\u0027"},{"line_number":41,"context_line":""},{"line_number":42,"context_line":"        # NOTE(yuanying): Workaround fix for ironic issue"},{"line_number":43,"context_line":"        # cf. https://bugs.launchpad.net/ironic/+bug/1596421"}],"source_content_type":"text/x-sh","patch_set":32,"id":"dada55a8_ef40150a","line":40,"range":{"start_line":39,"start_character":8,"end_line":40,"end_character":42},"updated":"2016-07-17 13:59:40.000000000","message":"Since we have the default flavor for master to m1.small, let\u0027s NOT set master flavor to baremetal. It\u0027s a waste of resources anyway.","commit_id":"d9f60c9d621aa98a11ec0f6db9e03d4444151779"},{"author":{"_account_id":20498,"name":"Spyros Trigazis","email":"spyridon.trigazis@cern.ch","username":"strigazi"},"change_message_id":"2d74269f3f6f89a4fca2d134519d8b237d23b09f","unresolved":false,"context_lines":[{"line_number":85,"context_line":"image_id \u003d $IMAGE_ID"},{"line_number":86,"context_line":"nic_id \u003d $NIC_ID"},{"line_number":87,"context_line":"keypair_id \u003d default"},{"line_number":88,"context_line":"flavor_id \u003d $flavor_id"},{"line_number":89,"context_line":"master_flavor_id \u003d $master_flavor_id"},{"line_number":90,"context_line":"copy_logs \u003d true"},{"line_number":91,"context_line":"csr_location \u003d $MAGNUM_DIR/default.csr"}],"source_content_type":"text/x-sh","patch_set":32,"id":"dada55a8_af5d7dee","line":88,"range":{"start_line":88,"start_character":13,"end_line":88,"end_character":22},"updated":"2016-07-17 13:59:40.000000000","message":"Here we should set the id of the baremetal flavor created by ironic\u0027s devstack script. [1]\n\n[1] https://github.com/openstack/ironic/blob/master/devstack/lib/ironic#L1021","commit_id":"d9f60c9d621aa98a11ec0f6db9e03d4444151779"},{"author":{"_account_id":20498,"name":"Spyros Trigazis","email":"spyridon.trigazis@cern.ch","username":"strigazi"},"change_message_id":"2d74269f3f6f89a4fca2d134519d8b237d23b09f","unresolved":false,"context_lines":[{"line_number":86,"context_line":"nic_id \u003d $NIC_ID"},{"line_number":87,"context_line":"keypair_id \u003d default"},{"line_number":88,"context_line":"flavor_id \u003d $flavor_id"},{"line_number":89,"context_line":"master_flavor_id \u003d $master_flavor_id"},{"line_number":90,"context_line":"copy_logs \u003d true"},{"line_number":91,"context_line":"csr_location \u003d $MAGNUM_DIR/default.csr"},{"line_number":92,"context_line":"dns_nameserver \u003d 8.8.8.8"}],"source_content_type":"text/x-sh","patch_set":32,"id":"dada55a8_8f6239ac","line":89,"range":{"start_line":89,"start_character":20,"end_line":89,"end_character":36},"updated":"2016-07-17 13:59:40.000000000","message":"We can leave this as it is.","commit_id":"d9f60c9d621aa98a11ec0f6db9e03d4444151779"},{"author":{"_account_id":20498,"name":"Spyros Trigazis","email":"spyridon.trigazis@cern.ch","username":"strigazi"},"change_message_id":"2d74269f3f6f89a4fca2d134519d8b237d23b09f","unresolved":false,"context_lines":[{"line_number":160,"context_line":""},{"line_number":161,"context_line":"    # Create magnum specific flavor for use in functional tests."},{"line_number":162,"context_line":"    echo_summary \"Create a flavor\""},{"line_number":163,"context_line":"    nova flavor-create $MAGNUM_MASTER_FLAVOR_ID 100 1024 10 1"},{"line_number":164,"context_line":"    nova flavor-create $MAGNUM_FLAVOR_ID 200 512 10 1"},{"line_number":165,"context_line":"}"},{"line_number":166,"context_line":""},{"line_number":167,"context_line":"if ! function_exists echo_summary; then"}],"source_content_type":"text/x-sh","patch_set":32,"id":"dada55a8_ef0e55e4","line":164,"range":{"start_line":163,"start_character":4,"end_line":164,"end_character":53},"updated":"2016-07-17 13:59:40.000000000","message":"This is wrong, we create two flavors named baremetal and ironic\u0027s devstack script has already create one.","commit_id":"d9f60c9d621aa98a11ec0f6db9e03d4444151779"}]}