)]}'
{"lib/keystone":[{"author":{"_account_id":6482,"name":"Steve Martinelli","email":"s.martinelli@gmail.com","username":"stevemar"},"change_message_id":"b4f7b13b7d3d00f3405ae124c5ce884bb358ac2b","unresolved":false,"context_lines":[{"line_number":332,"context_line":""},{"line_number":333,"context_line":"    # Domain"},{"line_number":334,"context_line":"    local default_domain"},{"line_number":335,"context_line":"    default_domain\u003d$(\u0027get_or_create_domain \"default\"\u0027)"},{"line_number":336,"context_line":""},{"line_number":337,"context_line":"    # The keystone bootstrapping process (performed via keystone-manage bootstrap)"},{"line_number":338,"context_line":"    # creates an admin user, admin role and admin project. As a sanity check"}],"source_content_type":"application/x-shellscript","patch_set":1,"id":"9a68dd71_4605d543","line":335,"range":{"start_line":335,"start_character":22,"end_line":335,"end_character":42},"updated":"2016-01-22 20:28:08.000000000","message":"this should be created already, you could just use \"openstack domain show \"default\" -f value -c id","commit_id":"8a3b29539dabffa84896eaa6db5d7ffbc3bb42f9"},{"author":{"_account_id":15742,"name":"Paulo Ewerton Gomes Fragoso","email":"pauloewerton@lsd.ufcg.edu.br","username":"pauloewerton"},"change_message_id":"cc1de784ca10ae0f2d9e01b49e195d936fe27ce9","unresolved":false,"context_lines":[{"line_number":332,"context_line":""},{"line_number":333,"context_line":"    # Domain"},{"line_number":334,"context_line":"    local default_domain"},{"line_number":335,"context_line":"    default_domain\u003d$(\u0027get_or_create_domain \"default\"\u0027)"},{"line_number":336,"context_line":""},{"line_number":337,"context_line":"    # The keystone bootstrapping process (performed via keystone-manage bootstrap)"},{"line_number":338,"context_line":"    # creates an admin user, admin role and admin project. As a sanity check"}],"source_content_type":"application/x-shellscript","patch_set":1,"id":"9a68dd71_5056ab06","line":335,"range":{"start_line":335,"start_character":22,"end_line":335,"end_character":42},"in_reply_to":"9a68dd71_4605d543","updated":"2016-01-25 13:15:06.000000000","message":"Done","commit_id":"8a3b29539dabffa84896eaa6db5d7ffbc3bb42f9"},{"author":{"_account_id":6482,"name":"Steve Martinelli","email":"s.martinelli@gmail.com","username":"stevemar"},"change_message_id":"b255e214208915ac20e8f20dd8c3524aef16d7eb","unresolved":false,"context_lines":[{"line_number":333,"context_line":"    # The keystone bootstrapping process (performed via keystone-manage bootstrap)"},{"line_number":334,"context_line":"    # creates a default domain, an admin user, admin role and admin project."},{"line_number":335,"context_line":"    # As a sanity check we exercise the CLI to retrieve the IDs for these values."},{"line_number":336,"context_line":"    local default_domain"},{"line_number":337,"context_line":"    default_domain\u003d$(openstack domain show \"default\" -f value -c id)"},{"line_number":338,"context_line":"    local admin_tenant"},{"line_number":339,"context_line":"    admin_tenant\u003d$(openstack project show \"admin\" -f value -c id)"},{"line_number":340,"context_line":"    local admin_user"}],"source_content_type":"application/x-shellscript","patch_set":2,"id":"9a68dd71_963de4c5","line":337,"range":{"start_line":336,"start_character":4,"end_line":337,"end_character":68},"updated":"2016-01-25 21:38:02.000000000","message":"\"default_domain\" appears to be unused, not sure what the point of it is","commit_id":"503e56e55a5e2fb48b940856f80a377af6b4d351"},{"author":{"_account_id":15742,"name":"Paulo Ewerton Gomes Fragoso","email":"pauloewerton@lsd.ufcg.edu.br","username":"pauloewerton"},"change_message_id":"91bfafc5494bb42737670c1fafdef5b90353c9b0","unresolved":false,"context_lines":[{"line_number":333,"context_line":"    # The keystone bootstrapping process (performed via keystone-manage bootstrap)"},{"line_number":334,"context_line":"    # creates a default domain, an admin user, admin role and admin project."},{"line_number":335,"context_line":"    # As a sanity check we exercise the CLI to retrieve the IDs for these values."},{"line_number":336,"context_line":"    local default_domain"},{"line_number":337,"context_line":"    default_domain\u003d$(openstack domain show \"default\" -f value -c id)"},{"line_number":338,"context_line":"    local admin_tenant"},{"line_number":339,"context_line":"    admin_tenant\u003d$(openstack project show \"admin\" -f value -c id)"},{"line_number":340,"context_line":"    local admin_user"}],"source_content_type":"application/x-shellscript","patch_set":2,"id":"9a68dd71_8e128957","line":337,"range":{"start_line":336,"start_character":4,"end_line":337,"end_character":68},"in_reply_to":"9a68dd71_963de4c5","updated":"2016-01-26 14:19:23.000000000","message":"this was just for recovering the value from openstackclient but it doesn\u0027t seem to be useful so I\u0027ll remove in the next patch set.","commit_id":"503e56e55a5e2fb48b940856f80a377af6b4d351"}],"lib/nova_plugins/hypervisor-ironic":[{"author":{"_account_id":6482,"name":"Steve Martinelli","email":"s.martinelli@gmail.com","username":"stevemar"},"change_message_id":"b255e214208915ac20e8f20dd8c3524aef16d7eb","unresolved":false,"context_lines":[{"line_number":47,"context_line":"    # ironic section"},{"line_number":48,"context_line":"    iniset $NOVA_CONF ironic admin_username admin"},{"line_number":49,"context_line":"    iniset $NOVA_CONF ironic admin_password $ADMIN_PASSWORD"},{"line_number":50,"context_line":"    iniset $NOVA_CONF ironic admin_url $KEYSTONE_AUTH_URI/v3"},{"line_number":51,"context_line":"    iniset $NOVA_CONF ironic admin_tenant_name demo"},{"line_number":52,"context_line":"    iniset $NOVA_CONF ironic api_endpoint $IRONIC_SERVICE_PROTOCOL://$IRONIC_HOSTPORT/v1"},{"line_number":53,"context_line":"}"}],"source_content_type":"application/x-shellscript","patch_set":2,"id":"9a68dd71_363d188f","line":50,"range":{"start_line":50,"start_character":4,"end_line":50,"end_character":60},"updated":"2016-01-25 21:38:02.000000000","message":"is this only required because of the depends-on patch?","commit_id":"503e56e55a5e2fb48b940856f80a377af6b4d351"},{"author":{"_account_id":15742,"name":"Paulo Ewerton Gomes Fragoso","email":"pauloewerton@lsd.ufcg.edu.br","username":"pauloewerton"},"change_message_id":"91bfafc5494bb42737670c1fafdef5b90353c9b0","unresolved":false,"context_lines":[{"line_number":47,"context_line":"    # ironic section"},{"line_number":48,"context_line":"    iniset $NOVA_CONF ironic admin_username admin"},{"line_number":49,"context_line":"    iniset $NOVA_CONF ironic admin_password $ADMIN_PASSWORD"},{"line_number":50,"context_line":"    iniset $NOVA_CONF ironic admin_url $KEYSTONE_AUTH_URI/v3"},{"line_number":51,"context_line":"    iniset $NOVA_CONF ironic admin_tenant_name demo"},{"line_number":52,"context_line":"    iniset $NOVA_CONF ironic api_endpoint $IRONIC_SERVICE_PROTOCOL://$IRONIC_HOSTPORT/v1"},{"line_number":53,"context_line":"}"}],"source_content_type":"application/x-shellscript","patch_set":2,"id":"9a68dd71_6ebab50c","line":50,"range":{"start_line":50,"start_character":4,"end_line":50,"end_character":60},"in_reply_to":"9a68dd71_363d188f","updated":"2016-01-26 14:19:23.000000000","message":"actually we would need the depends-on patch because of this line. gate seems ok with it, so I guess we can remove the dependency. ;)","commit_id":"503e56e55a5e2fb48b940856f80a377af6b4d351"}],"lib/tempest":[{"author":{"_account_id":6482,"name":"Steve Martinelli","email":"s.martinelli@gmail.com","username":"stevemar"},"change_message_id":"b255e214208915ac20e8f20dd8c3524aef16d7eb","unresolved":false,"context_lines":[{"line_number":188,"context_line":"    ALT_TENANT_NAME\u003d${ALT_TENANT_NAME:-alt_demo}"},{"line_number":189,"context_line":"    ADMIN_TENANT_ID\u003d$(openstack project list | awk \"/ admin / { print \\$2 }\")"},{"line_number":190,"context_line":""},{"line_number":191,"context_line":"    # Setting Keystone v3 environment variables"},{"line_number":192,"context_line":"    export OS_IDENTITY_API_VERSION\u003d3"},{"line_number":193,"context_line":"    export OS_PROJECT_DOMAIN_ID\u003ddefault"},{"line_number":194,"context_line":"    export OS_USER_DOMAIN_ID\u003ddefault"},{"line_number":195,"context_line":""},{"line_number":196,"context_line":"    if is_service_enabled nova; then"},{"line_number":197,"context_line":"        # If ``DEFAULT_INSTANCE_TYPE`` is not declared, use the new behavior"}],"source_content_type":"application/x-shellscript","patch_set":2,"id":"9a68dd71_f67d406d","line":194,"range":{"start_line":191,"start_character":4,"end_line":194,"end_character":36},"updated":"2016-01-25 21:38:02.000000000","message":"are these needed? it\u0027s already done here: https://github.com/openstack-dev/devstack/blob/master/stack.sh#L1018-L1026","commit_id":"503e56e55a5e2fb48b940856f80a377af6b4d351"},{"author":{"_account_id":15742,"name":"Paulo Ewerton Gomes Fragoso","email":"pauloewerton@lsd.ufcg.edu.br","username":"pauloewerton"},"change_message_id":"91bfafc5494bb42737670c1fafdef5b90353c9b0","unresolved":false,"context_lines":[{"line_number":188,"context_line":"    ALT_TENANT_NAME\u003d${ALT_TENANT_NAME:-alt_demo}"},{"line_number":189,"context_line":"    ADMIN_TENANT_ID\u003d$(openstack project list | awk \"/ admin / { print \\$2 }\")"},{"line_number":190,"context_line":""},{"line_number":191,"context_line":"    # Setting Keystone v3 environment variables"},{"line_number":192,"context_line":"    export OS_IDENTITY_API_VERSION\u003d3"},{"line_number":193,"context_line":"    export OS_PROJECT_DOMAIN_ID\u003ddefault"},{"line_number":194,"context_line":"    export OS_USER_DOMAIN_ID\u003ddefault"},{"line_number":195,"context_line":""},{"line_number":196,"context_line":"    if is_service_enabled nova; then"},{"line_number":197,"context_line":"        # If ``DEFAULT_INSTANCE_TYPE`` is not declared, use the new behavior"}],"source_content_type":"application/x-shellscript","patch_set":2,"id":"9a68dd71_2e091dde","line":194,"range":{"start_line":191,"start_character":4,"end_line":194,"end_character":36},"in_reply_to":"9a68dd71_f67d406d","updated":"2016-01-26 14:19:23.000000000","message":"I guess you\u0027re right, removing it in next patch set.","commit_id":"503e56e55a5e2fb48b940856f80a377af6b4d351"}]}