)]}'
{"doc/source/command-objects/aggregate.rst":[{"author":{"_account_id":17776,"name":"Reedip","email":"reedip.banerjee@gmail.com","username":"Reedip"},"change_message_id":"cac657de9efef56c9b01043877f6b8320fbe22af","unresolved":false,"context_lines":[{"line_number":133,"context_line":"    (repeat option to set multiple properties)"},{"line_number":134,"context_line":""},{"line_number":135,"context_line":".. option:: --no-property"},{"line_number":136,"context_line":""},{"line_number":137,"context_line":"    Remove all properties from :ref:`\\\u003caggregate\\\u003e \u003caggregate_set-aggregate\u003e`"},{"line_number":138,"context_line":""},{"line_number":139,"context_line":".. _aggregate_set-aggregate:"}],"source_content_type":"text/x-rst","patch_set":11,"id":"7a8ec9b2_8c7b61bd","line":136,"updated":"2016-09-19 09:39:29.000000000","message":"If this will be used in conjuction with --property to Overwrite the current values in property with new values, then please add that information as well :)","commit_id":"88f32353c5df7b06d387adbb70e56f9b915b2e0c"},{"author":{"_account_id":8276,"name":"kiwik","email":"chenrui.momo@gmail.com","username":"kiwik"},"change_message_id":"8100173cdb9a5a8c87ae2362e054eaf7dbc3f233","unresolved":false,"context_lines":[{"line_number":133,"context_line":"    (repeat option to set multiple properties)"},{"line_number":134,"context_line":""},{"line_number":135,"context_line":".. option:: --no-property"},{"line_number":136,"context_line":""},{"line_number":137,"context_line":"    Remove all properties from :ref:`\\\u003caggregate\\\u003e \u003caggregate_set-aggregate\u003e`"},{"line_number":138,"context_line":""},{"line_number":139,"context_line":".. _aggregate_set-aggregate:"}],"source_content_type":"text/x-rst","patch_set":11,"id":"5a8bc5a2_7e974890","line":136,"in_reply_to":"7a8ec9b2_8c7b61bd","updated":"2016-09-23 07:25:23.000000000","message":"Done","commit_id":"88f32353c5df7b06d387adbb70e56f9b915b2e0c"},{"author":{"_account_id":8410,"name":"Richard Theis","email":"rtheis@us.ibm.com","username":"rtheis"},"change_message_id":"fea888e71a0c86f10e390d67b5a1841716c546f0","unresolved":false,"context_lines":[{"line_number":136,"context_line":""},{"line_number":137,"context_line":"    Remove all properties from :ref:`\\\u003caggregate\\\u003e \u003caggregate_set-aggregate\u003e`"},{"line_number":138,"context_line":"    (specify both --property and --no-property to"},{"line_number":139,"context_line":"    overwrite the current property)"},{"line_number":140,"context_line":""},{"line_number":141,"context_line":".. _aggregate_set-aggregate:"},{"line_number":142,"context_line":".. describe:: \u003caggregate\u003e"}],"source_content_type":"text/x-rst","patch_set":12,"id":"5a8bc5a2_6269508b","line":139,"range":{"start_line":139,"start_character":26,"end_line":139,"end_character":34},"updated":"2016-09-23 15:21:38.000000000","message":"properties","commit_id":"cb40f77f04fbf24f36e342b468bfb18d9ae1c390"},{"author":{"_account_id":8276,"name":"kiwik","email":"chenrui.momo@gmail.com","username":"kiwik"},"change_message_id":"17e81afd8b2c0b3f280938e89a07bb2b4c0e3bb0","unresolved":false,"context_lines":[{"line_number":136,"context_line":""},{"line_number":137,"context_line":"    Remove all properties from :ref:`\\\u003caggregate\\\u003e \u003caggregate_set-aggregate\u003e`"},{"line_number":138,"context_line":"    (specify both --property and --no-property to"},{"line_number":139,"context_line":"    overwrite the current property)"},{"line_number":140,"context_line":""},{"line_number":141,"context_line":".. _aggregate_set-aggregate:"},{"line_number":142,"context_line":".. describe:: \u003caggregate\u003e"}],"source_content_type":"text/x-rst","patch_set":12,"id":"5a8bc5a2_e41e3562","line":139,"range":{"start_line":139,"start_character":26,"end_line":139,"end_character":34},"in_reply_to":"5a8bc5a2_6269508b","updated":"2016-09-24 08:17:41.000000000","message":"Done","commit_id":"cb40f77f04fbf24f36e342b468bfb18d9ae1c390"}],"doc/source/command-options.rst":[{"author":{"_account_id":8410,"name":"Richard Theis","email":"rtheis@us.ibm.com","username":"rtheis"},"change_message_id":"c20d66c1d768202420474c9bcc3e1ca81c258eeb","unresolved":false,"context_lines":[{"line_number":188,"context_line":"                   \u0027(repeat option to remove multiple properties)\u0027),"},{"line_number":189,"context_line":"        )"},{"line_number":190,"context_line":"        example_property_group.add_argument("},{"line_number":191,"context_line":"            \u0027--all-example-properties\u0027,"},{"line_number":192,"context_line":"            dest\u003d\u0027all_example_properties\u0027,"},{"line_number":193,"context_line":"            action\u003d\u0027store_true\u0027,"},{"line_number":194,"context_line":"            help\u003d_(\u0027Remove all example properties for this \u003cresource\u003e\u0027),"}],"source_content_type":"text/x-rst","patch_set":5,"id":"bab6814e_d1b90517","line":191,"updated":"2016-05-25 17:35:52.000000000","message":"I thought we were avoiding the use of plurals.","commit_id":"a5de7ca561c2fa375f1a4e6154fcd3c18c14a7aa"},{"author":{"_account_id":8410,"name":"Richard Theis","email":"rtheis@us.ibm.com","username":"rtheis"},"change_message_id":"84882c3b902ebaa5aafe691809816e6deb02bbce","unresolved":false,"context_lines":[{"line_number":188,"context_line":"                   \u0027(repeat option to remove multiple properties)\u0027),"},{"line_number":189,"context_line":"        )"},{"line_number":190,"context_line":"        example_property_group.add_argument("},{"line_number":191,"context_line":"            \u0027--all-example-properties\u0027,"},{"line_number":192,"context_line":"            dest\u003d\u0027all_example_properties\u0027,"},{"line_number":193,"context_line":"            action\u003d\u0027store_true\u0027,"},{"line_number":194,"context_line":"            help\u003d_(\u0027Remove all example properties for this \u003cresource\u003e\u0027),"},{"line_number":195,"context_line":"        )"}],"source_content_type":"text/x-rst","patch_set":6,"id":"7aa08908_74d3d538","line":192,"range":{"start_line":191,"start_character":0,"end_line":192,"end_character":42},"updated":"2016-06-17 13:18:43.000000000","message":"Dean, are we okay with plurals here?  I like the --all prefix but it does read odd unless plurals are used.  An alternative is --each prefix to avoid plurals.","commit_id":"951c8f679c2f29e5d0621b250059aabade82dc3c"}],"openstackclient/compute/v2/aggregate.py":[{"author":{"_account_id":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},"change_message_id":"6da09144f7e3e3dd091262b370777a3578a1515e","unresolved":false,"context_lines":[{"line_number":258,"context_line":""},{"line_number":259,"context_line":"        set_property \u003d {}"},{"line_number":260,"context_line":"        if parsed_args.property:"},{"line_number":261,"context_line":"            set_property.update(parsed_args.property)"},{"line_number":262,"context_line":"        if parsed_args.no_property:"},{"line_number":263,"context_line":"            # NOTE(RuiChen): \"availability_zone\" is removed from response of"},{"line_number":264,"context_line":"            #                aggregate show and create commands, don\u0027t see it"}],"source_content_type":"text/x-python","patch_set":3,"id":"bab6814e_b7062a1e","line":261,"updated":"2016-05-24 22:29:47.000000000","message":"This seems backwards to me?\n\nAssume properties {foo\u003dyes, bar\u003dno} in the aggregate, the command line:\n\n  aggregate set --no-property --property foo\u003dno \u003caggregate\u003e\n\n_should_ leave the aggregate properties {foo: no}.  It looks like it will actually be empty...","commit_id":"1880f556b1684afbbceaa6e725446d0f823c9b80"},{"author":{"_account_id":8276,"name":"kiwik","email":"chenrui.momo@gmail.com","username":"kiwik"},"change_message_id":"43b7b85baf2a34b5164fa79116a42a4545f9184e","unresolved":false,"context_lines":[{"line_number":258,"context_line":""},{"line_number":259,"context_line":"        set_property \u003d {}"},{"line_number":260,"context_line":"        if parsed_args.property:"},{"line_number":261,"context_line":"            set_property.update(parsed_args.property)"},{"line_number":262,"context_line":"        if parsed_args.no_property:"},{"line_number":263,"context_line":"            # NOTE(RuiChen): \"availability_zone\" is removed from response of"},{"line_number":264,"context_line":"            #                aggregate show and create commands, don\u0027t see it"}],"source_content_type":"text/x-python","patch_set":3,"id":"bab6814e_48f3dc0a","line":261,"in_reply_to":"bab6814e_b7062a1e","updated":"2016-05-25 01:55:20.000000000","message":"Sorry, my fault, forget to update the code, the overwrite logic should be applied at here.","commit_id":"1880f556b1684afbbceaa6e725446d0f823c9b80"},{"author":{"_account_id":8410,"name":"Richard Theis","email":"rtheis@us.ibm.com","username":"rtheis"},"change_message_id":"fea888e71a0c86f10e390d67b5a1841716c546f0","unresolved":false,"context_lines":[{"line_number":279,"context_line":""},{"line_number":280,"context_line":"        set_property \u003d {}"},{"line_number":281,"context_line":"        if parsed_args.no_property:"},{"line_number":282,"context_line":"            # NOTE(RuiChen): \"availability_zone\" is removed from response of"},{"line_number":283,"context_line":"            #                aggregate show and create commands, don\u0027t see it"},{"line_number":284,"context_line":"            #                anywhere, so pop it, avoid the unexpected server"},{"line_number":285,"context_line":"            #                exception(can\u0027t unset the availability zone from"}],"source_content_type":"text/x-python","patch_set":12,"id":"5a8bc5a2_c2830403","line":282,"updated":"2016-09-23 15:21:38.000000000","message":"Should this be noted in the help text?  That is, document that reserved properties are not removed.","commit_id":"cb40f77f04fbf24f36e342b468bfb18d9ae1c390"},{"author":{"_account_id":8276,"name":"kiwik","email":"chenrui.momo@gmail.com","username":"kiwik"},"change_message_id":"17e81afd8b2c0b3f280938e89a07bb2b4c0e3bb0","unresolved":false,"context_lines":[{"line_number":279,"context_line":""},{"line_number":280,"context_line":"        set_property \u003d {}"},{"line_number":281,"context_line":"        if parsed_args.no_property:"},{"line_number":282,"context_line":"            # NOTE(RuiChen): \"availability_zone\" is removed from response of"},{"line_number":283,"context_line":"            #                aggregate show and create commands, don\u0027t see it"},{"line_number":284,"context_line":"            #                anywhere, so pop it, avoid the unexpected server"},{"line_number":285,"context_line":"            #                exception(can\u0027t unset the availability zone from"}],"source_content_type":"text/x-python","patch_set":12,"id":"5a8bc5a2_447d29e6","line":282,"in_reply_to":"5a8bc5a2_c2830403","updated":"2016-09-24 08:17:41.000000000","message":"I prefer don\u0027t export the nova reserved key to end users, and the following code is internal handing. In aggregate list and show commands, the availability_zone is separated column and it\u0027s not shown in properties column, so I think end users would not unset the \"availability_zone\" key.","commit_id":"cb40f77f04fbf24f36e342b468bfb18d9ae1c390"}],"openstackclient/tests/compute/v2/test_aggregate.py":[{"author":{"_account_id":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},"change_message_id":"6da09144f7e3e3dd091262b370777a3578a1515e","unresolved":false,"context_lines":[{"line_number":315,"context_line":"        self.aggregate_mock.set_metadata.assert_called_once_with("},{"line_number":316,"context_line":"            self.fake_ag, parsed_args.property)"},{"line_number":317,"context_line":"        self.assertIsNone(result)"},{"line_number":318,"context_line":""},{"line_number":319,"context_line":""},{"line_number":320,"context_line":"class TestAggregateShow(TestAggregate):"},{"line_number":321,"context_line":""}],"source_content_type":"text/x-python","patch_set":3,"id":"bab6814e_e228f287","side":"PARENT","line":318,"updated":"2016-05-24 22:29:47.000000000","message":"There needs to be a test that combines --no-property and --property a\u003db.\n\nI have a hunch it\u0027ll fail with the current implementation","commit_id":"5dfedd6445347417e8d68707ec24780f574d6b23"},{"author":{"_account_id":8276,"name":"kiwik","email":"chenrui.momo@gmail.com","username":"kiwik"},"change_message_id":"43b7b85baf2a34b5164fa79116a42a4545f9184e","unresolved":false,"context_lines":[{"line_number":315,"context_line":"        self.aggregate_mock.set_metadata.assert_called_once_with("},{"line_number":316,"context_line":"            self.fake_ag, parsed_args.property)"},{"line_number":317,"context_line":"        self.assertIsNone(result)"},{"line_number":318,"context_line":""},{"line_number":319,"context_line":""},{"line_number":320,"context_line":"class TestAggregateShow(TestAggregate):"},{"line_number":321,"context_line":""}],"source_content_type":"text/x-python","patch_set":3,"id":"bab6814e_48cabca2","side":"PARENT","line":318,"in_reply_to":"bab6814e_e228f287","updated":"2016-05-25 01:55:20.000000000","message":"add it in next patch, thanks.","commit_id":"5dfedd6445347417e8d68707ec24780f574d6b23"}]}