)]}'
{"openstackclient/compute/v2/aggregate.py":[{"author":{"_account_id":6482,"name":"Steve Martinelli","email":"s.martinelli@gmail.com","username":"stevemar"},"change_message_id":"a273aa9e6f9181704df6042a622559362275872d","unresolved":false,"context_lines":[{"line_number":1,"context_line":"#   Copyright 2013 Nebula Inc."},{"line_number":2,"context_line":"#"},{"line_number":3,"context_line":"#   Licensed under the Apache License, Version 2.0 (the \"License\"); you may"},{"line_number":4,"context_line":"#   not use this file except in compliance with the License. You may obtain"}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAAP3%2F%2FvaM%3D","line":1,"updated":"2013-07-12 04:59:59.000000000","message":"Openstack LLC","commit_id":"2362f0db634629d71510267cee0dc334af42bb96"},{"author":{"_account_id":6482,"name":"Steve Martinelli","email":"s.martinelli@gmail.com","username":"stevemar"},"change_message_id":"a273aa9e6f9181704df6042a622559362275872d","unresolved":false,"context_lines":[{"line_number":146,"context_line":"        columns \u003d ("},{"line_number":147,"context_line":"            \"ID\","},{"line_number":148,"context_line":"            \"Name\","},{"line_number":149,"context_line":"            \"Availability Zone\","},{"line_number":150,"context_line":"        )"},{"line_number":151,"context_line":"        data \u003d compute_client.aggregates.list()"},{"line_number":152,"context_line":"        return (columns,"}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAAP3%2F%2FvXY%3D","line":149,"updated":"2013-07-12 04:59:59.000000000","message":"--long option for property, not sure what the status quo is?","commit_id":"2362f0db634629d71510267cee0dc334af42bb96"},{"author":{"_account_id":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},"change_message_id":"03d2343e2d10d9e7006bbd1189fee2b53db5c2cd","unresolved":false,"context_lines":[{"line_number":146,"context_line":"        columns \u003d ("},{"line_number":147,"context_line":"            \"ID\","},{"line_number":148,"context_line":"            \"Name\","},{"line_number":149,"context_line":"            \"Availability Zone\","},{"line_number":150,"context_line":"        )"},{"line_number":151,"context_line":"        data \u003d compute_client.aggregates.list()"},{"line_number":152,"context_line":"        return (columns,"}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAAP3%2F%2FfGI%3D","line":149,"in_reply_to":"AAAAP3%2F%2FvXY%3D","updated":"2013-07-17 15:10:02.000000000","message":"The only additional field to show is metadata, I don\u0027t think we have a consistent policy on whether to include that in --long lists.","commit_id":"2362f0db634629d71510267cee0dc334af42bb96"},{"author":{"_account_id":6482,"name":"Steve Martinelli","email":"s.martinelli@gmail.com","username":"stevemar"},"change_message_id":"a273aa9e6f9181704df6042a622559362275872d","unresolved":false,"context_lines":[{"line_number":275,"context_line":"        self.log.debug(\u0027take_action(%s)\u0027 % parsed_args)"},{"line_number":276,"context_line":""},{"line_number":277,"context_line":"        compute_client \u003d self.app.client_manager.compute"},{"line_number":278,"context_line":"        data \u003d utils.find_resource("},{"line_number":279,"context_line":"            compute_client.aggregates,"},{"line_number":280,"context_line":"            parsed_args.aggregate,"},{"line_number":281,"context_line":"        )"}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAAP3%2F%2FvW4%3D","line":278,"updated":"2013-07-12 04:59:59.000000000","message":"purely formatting, this affects anywhere you call data._info.\n\nThere is an availability zone field, which is fine,\nbut then there are two issues:\n\n1) the availability zone is repeated in the metadata field, (i guess AZ is just special metadata).\nand \n2) the metadata is printed out with dictionary encodings and looks ugly -\u003e {u\u0027blah\u0027: u\u0027blah\u0027} \n\nI think I addressed this in volume.py and type.py, which probably should have been slotted into utils already - my bad :)\n\nExample of 1 and 2:\n$ openstack aggregate show agg2\n| Field             | Value                                                  |\n| availability_zone | myzone2                                                |\n| metadata          | {u\u0027key1\u0027: u\u0027value1\u0027, u\u0027availability_zone\u0027: u\u0027myzone2\u0027} |","commit_id":"2362f0db634629d71510267cee0dc334af42bb96"},{"author":{"_account_id":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},"change_message_id":"03d2343e2d10d9e7006bbd1189fee2b53db5c2cd","unresolved":false,"context_lines":[{"line_number":275,"context_line":"        self.log.debug(\u0027take_action(%s)\u0027 % parsed_args)"},{"line_number":276,"context_line":""},{"line_number":277,"context_line":"        compute_client \u003d self.app.client_manager.compute"},{"line_number":278,"context_line":"        data \u003d utils.find_resource("},{"line_number":279,"context_line":"            compute_client.aggregates,"},{"line_number":280,"context_line":"            parsed_args.aggregate,"},{"line_number":281,"context_line":"        )"}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAAP3%2F%2Fh8I%3D","line":278,"in_reply_to":"AAAAP3%2F%2FvW4%3D","updated":"2013-07-17 15:10:02.000000000","message":"There are a couple of problems with the availability_zone handling in the aggregates API. https://bugs.launchpad.net/nova/+bug/1200369 describes one that stomps on the metadata (fix committed yesterday).  The issue of it appearing both as a property and in the metadata is due to a change in Nova in https://github.com/openstack/nova/commit/9f4534ab584faeee1e24d4c1bb38a2b194f24626 and is seen as a backward-compatibility issue.  We\u0027ll have to clean it ourselves.","commit_id":"2362f0db634629d71510267cee0dc334af42bb96"},{"author":{"_account_id":6482,"name":"Steve Martinelli","email":"s.martinelli@gmail.com","username":"stevemar"},"change_message_id":"99c2b905d72c582d1a4862b2509b2c3c47870b88","unresolved":false,"context_lines":[{"line_number":1,"context_line":"#   Copyright 2013 Nebula Inc."},{"line_number":2,"context_line":"#"},{"line_number":3,"context_line":"#   Licensed under the Apache License, Version 2.0 (the \"License\"); you may"},{"line_number":4,"context_line":"#   not use this file except in compliance with the License. You may obtain"}],"source_content_type":"text/x-python","patch_set":2,"id":"AAAAQn%2F%2F%2F3M%3D","line":1,"updated":"2013-07-17 19:04:41.000000000","message":"Nebula in the header","commit_id":"50c8998eae7d25ed399f234328352bae4bd50f4f"}]}