)]}'
{"lib/ironic":[{"author":{"_account_id":11929,"name":"Michael Turek","email":"mjturek@linux.vnet.ibm.com","username":"mjturek"},"change_message_id":"0188e5b4de260b06d7ac06d2aece9ec768170689","unresolved":false,"context_lines":[{"line_number":578,"context_line":"        local ironic_node_ram\u003d$IRONIC_VM_SPECS_RAM"},{"line_number":579,"context_line":"        local ironic_node_disk\u003d$IRONIC_VM_SPECS_DISK"},{"line_number":580,"context_line":"        local ironic_ephemeral_disk\u003d$IRONIC_VM_EPHEMERAL_DISK"},{"line_number":581,"context_line":"        local ironic_node_arch\u003dx86_64"},{"line_number":582,"context_line":"        local ironic_hwinfo_file\u003d$IRONIC_VM_MACS_CSV_FILE"},{"line_number":583,"context_line":"        local node_options\u003d\"\\"},{"line_number":584,"context_line":"            -i deploy_kernel\u003d$IRONIC_DEPLOY_KERNEL_ID \\"}],"source_content_type":"application/x-shellscript","patch_set":1,"id":"ba15a1d1_d7d6614e","line":581,"updated":"2015-09-23 14:03:36.000000000","message":"Hey Mauro, I might be missing something but why is x86_64 hardcoded here? I thought the point of the patch is to allow the user to specify the architecture of  the nodes. Shouldn\u0027t this be $IRONIC_HW_ARCH ?","commit_id":"28c898b2c3ee6085c1d3861753d179288b6881e3"},{"author":{"_account_id":5174,"name":"Mauro Sergio Martins Rodrigues","email":"maurosr@linux.vnet.ibm.com","username":"maurosr"},"change_message_id":"c055f0ec42d4c78c9651848bc0945f98c361ec9e","unresolved":false,"context_lines":[{"line_number":578,"context_line":"        local ironic_node_ram\u003d$IRONIC_VM_SPECS_RAM"},{"line_number":579,"context_line":"        local ironic_node_disk\u003d$IRONIC_VM_SPECS_DISK"},{"line_number":580,"context_line":"        local ironic_ephemeral_disk\u003d$IRONIC_VM_EPHEMERAL_DISK"},{"line_number":581,"context_line":"        local ironic_node_arch\u003dx86_64"},{"line_number":582,"context_line":"        local ironic_hwinfo_file\u003d$IRONIC_VM_MACS_CSV_FILE"},{"line_number":583,"context_line":"        local node_options\u003d\"\\"},{"line_number":584,"context_line":"            -i deploy_kernel\u003d$IRONIC_DEPLOY_KERNEL_ID \\"}],"source_content_type":"application/x-shellscript","patch_set":1,"id":"ba15a1d1_ca45536f","line":581,"in_reply_to":"ba15a1d1_d7d6614e","updated":"2015-09-23 16:40:33.000000000","message":"This patch is only to enable us to specify arch for the target BM node, I\u0027m still working on a solution for the the guests. So this is hardcoded cause it meant to keep it as x86 for guests (as the if conditional points: !is_ironic_hardware).","commit_id":"28c898b2c3ee6085c1d3861753d179288b6881e3"}],"stackrc":[{"author":{"_account_id":7118,"name":"Ian Wienand","email":"iwienand@redhat.com","username":"iwienand"},"change_message_id":"2409dfcff0e41e8b428e98399edb01e50d11a4fd","unresolved":false,"context_lines":[{"line_number":611,"context_line":"                DEFAULT_IMAGE_NAME\u003d${DEFAULT_IMAGE_NAME:-cirros-${CIRROS_VERSION}-${CIRROS_ARCH}-uec}"},{"line_number":612,"context_line":"            fi"},{"line_number":613,"context_line":"            IMAGE_URLS+\u003d\"http://download.cirros-cloud.net/${CIRROS_VERSION}/cirros-${CIRROS_VERSION}-${CIRROS_ARCH}-uec.tar.gz\""},{"line_number":614,"context_line":"            IMAGE_URLS+\u003d\",http://download.cirros-cloud.net/${CIRROS_VERSION}/cirros-${CIRROS_VERSION}-${CIRROS_ARCH}-disk.img\";;"},{"line_number":615,"context_line":"        *) # Default to Cirros with kernel, ramdisk and disk image"},{"line_number":616,"context_line":"            DEFAULT_IMAGE_NAME\u003d${DEFAULT_IMAGE_NAME:-cirros-${CIRROS_VERSION}-${CIRROS_ARCH}-uec}"},{"line_number":617,"context_line":"            IMAGE_URLS+\u003d\"http://download.cirros-cloud.net/${CIRROS_VERSION}/cirros-${CIRROS_VERSION}-${CIRROS_ARCH}-uec.tar.gz\";;"}],"source_content_type":"application/x-shellscript","patch_set":3,"id":"9a8ffd7b_65108ea9","line":614,"updated":"2015-12-01 03:10:06.000000000","message":"is this totally safe?  i notice arm for example doesn\u0027t have a qcow image [1] \n\n[1] http://download.cirros-cloud.net/0.3.4/","commit_id":"98ae96ce4c7daddc0e58e38e3e71af1de3b535f3"},{"author":{"_account_id":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},"change_message_id":"79c1ff64a5d4c4e07b3252f675e8d6a99129f44f","unresolved":false,"context_lines":[{"line_number":611,"context_line":"                DEFAULT_IMAGE_NAME\u003d${DEFAULT_IMAGE_NAME:-cirros-${CIRROS_VERSION}-${CIRROS_ARCH}-uec}"},{"line_number":612,"context_line":"            fi"},{"line_number":613,"context_line":"            IMAGE_URLS+\u003d\"http://download.cirros-cloud.net/${CIRROS_VERSION}/cirros-${CIRROS_VERSION}-${CIRROS_ARCH}-uec.tar.gz\""},{"line_number":614,"context_line":"            IMAGE_URLS+\u003d\",http://download.cirros-cloud.net/${CIRROS_VERSION}/cirros-${CIRROS_VERSION}-${CIRROS_ARCH}-disk.img\";;"},{"line_number":615,"context_line":"        *) # Default to Cirros with kernel, ramdisk and disk image"},{"line_number":616,"context_line":"            DEFAULT_IMAGE_NAME\u003d${DEFAULT_IMAGE_NAME:-cirros-${CIRROS_VERSION}-${CIRROS_ARCH}-uec}"},{"line_number":617,"context_line":"            IMAGE_URLS+\u003d\"http://download.cirros-cloud.net/${CIRROS_VERSION}/cirros-${CIRROS_VERSION}-${CIRROS_ARCH}-uec.tar.gz\";;"}],"source_content_type":"application/x-shellscript","patch_set":3,"id":"7a740942_cd696c27","line":614,"in_reply_to":"9a8ffd7b_65108ea9","updated":"2015-12-09 20:27:10.000000000","message":"The only thing new about that issue is introducing it to the Ironic code path...it already exists in the default below.","commit_id":"98ae96ce4c7daddc0e58e38e3e71af1de3b535f3"}]}