)]}'
{"README.md":[{"author":{"_account_id":7118,"name":"Ian Wienand","email":"iwienand@redhat.com","username":"iwienand"},"change_message_id":"deb0171a8a520134a4aba392a1df140a506e4f4d","unresolved":false,"context_lines":[{"line_number":61,"context_line":"    # list instances"},{"line_number":62,"context_line":"    nova list"},{"line_number":63,"context_line":""},{"line_number":64,"context_line":"    # Optionally, use IPv6 for the OpenStack CLI"},{"line_number":65,"context_line":"    . openrc -6"},{"line_number":66,"context_line":""},{"line_number":67,"context_line":"If the EC2 API is your cup-o-tea, you can create credentials and use euca2ools:"}],"source_content_type":"text/x-gfm","patch_set":8,"id":"fa32b979_c68e2027","line":64,"updated":"2015-06-22 02:08:35.000000000","message":"maybe configuration.rst , where we\u0027re describing how to setup ipv6?","commit_id":"01cfadeec0395cc8f6923147860cc21f4ab21e5b"},{"author":{"_account_id":16782,"name":"Jeremy Alvis","email":"jeremy.alvis@hp.com","username":"jalvis"},"change_message_id":"f2943e89f16231a4fffa1d7c32956bc68ae518d3","unresolved":false,"context_lines":[{"line_number":61,"context_line":"    # list instances"},{"line_number":62,"context_line":"    nova list"},{"line_number":63,"context_line":""},{"line_number":64,"context_line":"    # Optionally, use IPv6 for the OpenStack CLI"},{"line_number":65,"context_line":"    . openrc -6"},{"line_number":66,"context_line":""},{"line_number":67,"context_line":"If the EC2 API is your cup-o-tea, you can create credentials and use euca2ools:"}],"source_content_type":"text/x-gfm","patch_set":8,"id":"fa32b979_322fe73f","line":64,"in_reply_to":"fa32b979_c68e2027","updated":"2015-06-22 17:55:03.000000000","message":"Done","commit_id":"01cfadeec0395cc8f6923147860cc21f4ab21e5b"}],"openrc":[{"author":{"_account_id":1131,"name":"Brian Haley","email":"haleyb.dev@gmail.com","username":"brian-haley"},"change_message_id":"15cfa714648b24f5b30b33813b5b98e1254befca","unresolved":false,"context_lines":[{"line_number":2,"context_line":"#"},{"line_number":3,"context_line":"# source openrc [options] [username] [tenantname]"},{"line_number":4,"context_line":"#"},{"line_number":5,"context_line":"#       -6      IPv6 support. Sets HOST_IP and SERVICE_IP to IPv6 addresses."},{"line_number":6,"context_line":"#"},{"line_number":7,"context_line":"# Configure a set of credentials for $TENANT/$USERNAME:"},{"line_number":8,"context_line":"#   Set OS_TENANT_NAME to override the default tenant \u0027demo\u0027"}],"source_content_type":"application/octet-stream","patch_set":4,"id":"fa32b979_613b979d","line":5,"updated":"2015-06-19 19:05:39.000000000","message":"Can you make this more generic, something like:\n\nSet service endpoints to be IPv6 addresses","commit_id":"4192379a0af1c187fb9019ee46e468bb9f4d0e85"},{"author":{"_account_id":16782,"name":"Jeremy Alvis","email":"jeremy.alvis@hp.com","username":"jalvis"},"change_message_id":"6d77f2ca3f083d1ef8c3cc5a6111c9398ed706e6","unresolved":false,"context_lines":[{"line_number":2,"context_line":"#"},{"line_number":3,"context_line":"# source openrc [options] [username] [tenantname]"},{"line_number":4,"context_line":"#"},{"line_number":5,"context_line":"#       -6      IPv6 support. Sets HOST_IP and SERVICE_IP to IPv6 addresses."},{"line_number":6,"context_line":"#"},{"line_number":7,"context_line":"# Configure a set of credentials for $TENANT/$USERNAME:"},{"line_number":8,"context_line":"#   Set OS_TENANT_NAME to override the default tenant \u0027demo\u0027"}],"source_content_type":"application/octet-stream","patch_set":4,"id":"fa32b979_be4d3dc6","line":5,"in_reply_to":"fa32b979_613b979d","updated":"2015-06-19 20:23:03.000000000","message":"Done","commit_id":"4192379a0af1c187fb9019ee46e468bb9f4d0e85"},{"author":{"_account_id":1131,"name":"Brian Haley","email":"haleyb.dev@gmail.com","username":"brian-haley"},"change_message_id":"15cfa714648b24f5b30b33813b5b98e1254befca","unresolved":false,"context_lines":[{"line_number":80,"context_line":"# Some exercises call glance directly.  On a single-node installation, Glance"},{"line_number":81,"context_line":"# should be listening on HOST_IP.  If its running elsewhere, it can be set here"},{"line_number":82,"context_line":""},{"line_number":83,"context_line":"if [ $ipv6 \u003d false ]; then"},{"line_number":84,"context_line":"    HOST_IP\u003d${HOST_IP:-127.0.0.1}"},{"line_number":85,"context_line":"    SERVICE_HOST\u003d${SERVICE_HOST:-$HOST_IP}"},{"line_number":86,"context_line":"    GLANCE_HOST\u003d${GLANCE_HOST:-$HOST_IP}"}],"source_content_type":"application/octet-stream","patch_set":4,"id":"fa32b979_81549bca","line":83,"updated":"2015-06-19 19:05:39.000000000","message":"nit: this can be \u0027if $ipv6; then\u0027 just need to switch below statements around.","commit_id":"4192379a0af1c187fb9019ee46e468bb9f4d0e85"},{"author":{"_account_id":16782,"name":"Jeremy Alvis","email":"jeremy.alvis@hp.com","username":"jalvis"},"change_message_id":"6d77f2ca3f083d1ef8c3cc5a6111c9398ed706e6","unresolved":false,"context_lines":[{"line_number":80,"context_line":"# Some exercises call glance directly.  On a single-node installation, Glance"},{"line_number":81,"context_line":"# should be listening on HOST_IP.  If its running elsewhere, it can be set here"},{"line_number":82,"context_line":""},{"line_number":83,"context_line":"if [ $ipv6 \u003d false ]; then"},{"line_number":84,"context_line":"    HOST_IP\u003d${HOST_IP:-127.0.0.1}"},{"line_number":85,"context_line":"    SERVICE_HOST\u003d${SERVICE_HOST:-$HOST_IP}"},{"line_number":86,"context_line":"    GLANCE_HOST\u003d${GLANCE_HOST:-$HOST_IP}"}],"source_content_type":"application/octet-stream","patch_set":4,"id":"fa32b979_7ec6d517","line":83,"in_reply_to":"fa32b979_81549bca","updated":"2015-06-19 20:23:03.000000000","message":"Done","commit_id":"4192379a0af1c187fb9019ee46e468bb9f4d0e85"},{"author":{"_account_id":7118,"name":"Ian Wienand","email":"iwienand@redhat.com","username":"iwienand"},"change_message_id":"deb0171a8a520134a4aba392a1df140a506e4f4d","unresolved":false,"context_lines":[{"line_number":12,"context_line":"# NOTE: support for the old NOVA_* novaclient environment variables has"},{"line_number":13,"context_line":"# been removed."},{"line_number":14,"context_line":""},{"line_number":15,"context_line":"ipv6\u003dfalse"},{"line_number":16,"context_line":""},{"line_number":17,"context_line":"while getopts \":6\" OPT; do"},{"line_number":18,"context_line":"    case $OPT in"}],"source_content_type":"application/octet-stream","patch_set":8,"id":"fa32b979_6679542f","line":15,"updated":"2015-06-22 02:08:35.000000000","message":"minor quibble - globals should be caps.  alost while there best to namespace it, maybe OPT_IPV6 is good","commit_id":"01cfadeec0395cc8f6923147860cc21f4ab21e5b"},{"author":{"_account_id":16782,"name":"Jeremy Alvis","email":"jeremy.alvis@hp.com","username":"jalvis"},"change_message_id":"f2943e89f16231a4fffa1d7c32956bc68ae518d3","unresolved":false,"context_lines":[{"line_number":12,"context_line":"# NOTE: support for the old NOVA_* novaclient environment variables has"},{"line_number":13,"context_line":"# been removed."},{"line_number":14,"context_line":""},{"line_number":15,"context_line":"ipv6\u003dfalse"},{"line_number":16,"context_line":""},{"line_number":17,"context_line":"while getopts \":6\" OPT; do"},{"line_number":18,"context_line":"    case $OPT in"}],"source_content_type":"application/octet-stream","patch_set":8,"id":"fa32b979_1de519a7","line":15,"in_reply_to":"fa32b979_6679542f","updated":"2015-06-22 17:55:03.000000000","message":"Done","commit_id":"01cfadeec0395cc8f6923147860cc21f4ab21e5b"},{"author":{"_account_id":7118,"name":"Ian Wienand","email":"iwienand@redhat.com","username":"iwienand"},"change_message_id":"da6b55e2d6c6eb5b84d1e9aee20cd4f5da44888d","unresolved":false,"context_lines":[{"line_number":87,"context_line":"if [ $OPT_IPV6 ]; then"},{"line_number":88,"context_line":"    HOST_IPV6\u003d${HOST_IPV6:-::1}"},{"line_number":89,"context_line":"    SERVICE_HOST\u003d${SERVICE_HOST:-[$HOST_IPV6]}"},{"line_number":90,"context_line":"    GLANCE_HOST\u003d${GLANCE_HOST:-$HOST_IPV6}"},{"line_number":91,"context_line":"else"},{"line_number":92,"context_line":"    HOST_IP\u003d${HOST_IP:-127.0.0.1}"},{"line_number":93,"context_line":"    SERVICE_HOST\u003d${SERVICE_HOST:-$HOST_IP}"}],"source_content_type":"application/octet-stream","patch_set":15,"id":"fa32b979_6e5dd414","line":90,"updated":"2015-06-26 02:11:27.000000000","message":"this should have [] too?","commit_id":"a0b3a6f6baa0b211d5e67b5aca23cc6a605ceef0"},{"author":{"_account_id":16782,"name":"Jeremy Alvis","email":"jeremy.alvis@hp.com","username":"jalvis"},"change_message_id":"be359c1eed5ecd6911e9a3df1c2a5e9b5e3ea5d7","unresolved":false,"context_lines":[{"line_number":87,"context_line":"if [ $OPT_IPV6 ]; then"},{"line_number":88,"context_line":"    HOST_IPV6\u003d${HOST_IPV6:-::1}"},{"line_number":89,"context_line":"    SERVICE_HOST\u003d${SERVICE_HOST:-[$HOST_IPV6]}"},{"line_number":90,"context_line":"    GLANCE_HOST\u003d${GLANCE_HOST:-$HOST_IPV6}"},{"line_number":91,"context_line":"else"},{"line_number":92,"context_line":"    HOST_IP\u003d${HOST_IP:-127.0.0.1}"},{"line_number":93,"context_line":"    SERVICE_HOST\u003d${SERVICE_HOST:-$HOST_IP}"}],"source_content_type":"application/octet-stream","patch_set":15,"id":"fa32b979_3b38543d","line":90,"in_reply_to":"fa32b979_6e5dd414","updated":"2015-06-26 15:06:08.000000000","message":"You\u0027re right.... I somehow missed that","commit_id":"a0b3a6f6baa0b211d5e67b5aca23cc6a605ceef0"},{"author":{"_account_id":4656,"name":"Sean M. Collins","email":"sean@coreitpro.com","username":"scollins"},"change_message_id":"5db3f9fe553738addf646d514ffc690ddaa4dda1","unresolved":false,"context_lines":[{"line_number":9,"context_line":""},{"line_number":10,"context_line":"# NOTE: support for the old NOVA_* novaclient environment variables has"},{"line_number":11,"context_line":"# been removed."},{"line_number":12,"context_line":""},{"line_number":13,"context_line":"OPT_IPV6\u003dfalse"},{"line_number":14,"context_line":""},{"line_number":15,"context_line":"if [[ -n \"$1\" ]]; then"}],"source_content_type":"application/octet-stream","patch_set":16,"id":"ba3cc151_b3a9c77d","line":12,"updated":"2015-06-30 13:51:36.000000000","message":"I\u0027d like to see a comment here about what this is used for","commit_id":"b8247ce957670f86146bbac4fe40a637ec00f5ac"},{"author":{"_account_id":1131,"name":"Brian Haley","email":"haleyb.dev@gmail.com","username":"brian-haley"},"change_message_id":"36e814aa6a7ac3945e653c6a0f55d38f0b49b85c","unresolved":false,"context_lines":[{"line_number":9,"context_line":""},{"line_number":10,"context_line":"# NOTE: support for the old NOVA_* novaclient environment variables has"},{"line_number":11,"context_line":"# been removed."},{"line_number":12,"context_line":""},{"line_number":13,"context_line":"OPT_IPV6\u003dfalse"},{"line_number":14,"context_line":""},{"line_number":15,"context_line":"if [[ -n \"$1\" ]]; then"}],"source_content_type":"application/octet-stream","patch_set":16,"id":"ba3cc151_24248fad","line":12,"in_reply_to":"ba3cc151_b3a9c77d","updated":"2015-06-30 14:33:50.000000000","message":"# Used to communicate with the APIs over IPv6, including for authentication\n\nIs that good Sean?  Basically, the other changes are enough to update the service catalog, but the initial auth request was still over IPv4.  And there are some other things like Glance that are special...","commit_id":"b8247ce957670f86146bbac4fe40a637ec00f5ac"},{"author":{"_account_id":7118,"name":"Ian Wienand","email":"iwienand@redhat.com","username":"iwienand"},"change_message_id":"2bb5d529f84301bf8237ea95d308d41f38dec134","unresolved":false,"context_lines":[{"line_number":36,"context_line":""},{"line_number":37,"context_line":"# Check for SERVICE_IP_VERSION"},{"line_number":38,"context_line":"if [[ $SERVICE_IP_VERSION \u003d\u003d 6 ]]; then"},{"line_number":39,"context_line":"    OPT_IPV6\u003dtrue"},{"line_number":40,"context_line":"fi"},{"line_number":41,"context_line":""},{"line_number":42,"context_line":"# Get some necessary configuration"}],"source_content_type":"application/octet-stream","patch_set":17,"id":"ba3cc151_46ab8275","line":39,"updated":"2015-07-01 23:11:57.000000000","message":"i\u0027m sorry to keep looping around on this; i think we\u0027ve just taken a couple of different paths here.\n\nit seems we\u0027ve settled on the idea that just setting SERVICE_IP_VERSION is sufficient and if that is set, then reasonably there\u0027s nothing to do but set up the ipv6 hosts here.  so i don\u0027t think we need this option, in the end.\n\nalso, if you want anyone else to review this without having to dig through the prior 17 revisions, i\u0027d strongly recommend you clearly describe in the change/comment why things are done this way -- lest anyone else suggest you go back to a option flag.","commit_id":"54057027e9e8352570f0e0d7ac38cca2d33671f5"},{"author":{"_account_id":16782,"name":"Jeremy Alvis","email":"jeremy.alvis@hp.com","username":"jalvis"},"change_message_id":"d0a2eeb355f81dc1ba856fdb0b50619fbb5bbfc6","unresolved":false,"context_lines":[{"line_number":36,"context_line":""},{"line_number":37,"context_line":"# Check for SERVICE_IP_VERSION"},{"line_number":38,"context_line":"if [[ $SERVICE_IP_VERSION \u003d\u003d 6 ]]; then"},{"line_number":39,"context_line":"    OPT_IPV6\u003dtrue"},{"line_number":40,"context_line":"fi"},{"line_number":41,"context_line":""},{"line_number":42,"context_line":"# Get some necessary configuration"}],"source_content_type":"application/octet-stream","patch_set":17,"id":"ba3cc151_ef4daac4","line":39,"in_reply_to":"ba3cc151_46ab8275","updated":"2015-07-02 15:02:07.000000000","message":"No worries! I see how unnecessary that variable is now. I have updated the code accordingly and added a section in the commit message explaining the progression of changes.","commit_id":"54057027e9e8352570f0e0d7ac38cca2d33671f5"},{"author":{"_account_id":1131,"name":"Brian Haley","email":"haleyb.dev@gmail.com","username":"brian-haley"},"change_message_id":"d3591d0a3628f46608c9e55a36163f0057eb429e","unresolved":false,"context_lines":[{"line_number":57,"context_line":"export OS_REGION_NAME\u003d${REGION_NAME:-RegionOne}"},{"line_number":58,"context_line":""},{"line_number":59,"context_line":"# Set api HOST_IP endpoint.  SERVICE_HOST may also be used to specify the endpoint,"},{"line_number":60,"context_line":"# which is convenient for some localrc configurations."},{"line_number":61,"context_line":""},{"line_number":62,"context_line":"# Some exercises call glance directly.  On a single-node installation, Glance"},{"line_number":63,"context_line":"# should be listening on HOST_IP.  If its running elsewhere, it can be set here"}],"source_content_type":"application/octet-stream","patch_set":18,"id":"ba3cc151_83a4f507","line":60,"updated":"2015-07-02 15:48:33.000000000","message":"nit: the following three comments are related, so we should combine them.  How about:\n\n\"Set the host API endpoint.  This will default to HOST_IP if SERVICE_IP_VERSION is 4, else HOST_IPV6 if it\u0027s 6.\nSERVICE_HOST may also be used to specify the endpoint,\nwhich is convenient for some localrc configurations.\nAdditionally, some exercises call Glance directly. On a single-node installation,\nGlance should be listening on a local IP address, depending on the setting of\nSERVICE_IP_VERSION.  If its running elsewhere, it can be set here.\"","commit_id":"33140ca5fa9c6a9e5c647c10f07a8123a81614c6"},{"author":{"_account_id":16782,"name":"Jeremy Alvis","email":"jeremy.alvis@hp.com","username":"jalvis"},"change_message_id":"d46eb2a0ab587756dbe3a6e78b9a0f0a1b84ae82","unresolved":false,"context_lines":[{"line_number":57,"context_line":"export OS_REGION_NAME\u003d${REGION_NAME:-RegionOne}"},{"line_number":58,"context_line":""},{"line_number":59,"context_line":"# Set api HOST_IP endpoint.  SERVICE_HOST may also be used to specify the endpoint,"},{"line_number":60,"context_line":"# which is convenient for some localrc configurations."},{"line_number":61,"context_line":""},{"line_number":62,"context_line":"# Some exercises call glance directly.  On a single-node installation, Glance"},{"line_number":63,"context_line":"# should be listening on HOST_IP.  If its running elsewhere, it can be set here"}],"source_content_type":"application/octet-stream","patch_set":18,"id":"ba3cc151_e3f4e15b","line":60,"in_reply_to":"ba3cc151_83a4f507","updated":"2015-07-02 15:56:33.000000000","message":"Done","commit_id":"33140ca5fa9c6a9e5c647c10f07a8123a81614c6"}]}