)]}'
{"novaclient/tests/functional/test_readonly_nova.py":[{"author":{"_account_id":4690,"name":"melanie witt","display_name":"melwitt","email":"melwittt@gmail.com","username":"melwitt"},"change_message_id":"aedbef9d4b1b3931b14d5d61075aa4375d94ec5d","unresolved":false,"context_lines":[{"line_number":131,"context_line":"        self.nova(\u0027usage-list\u0027)"},{"line_number":132,"context_line":""},{"line_number":133,"context_line":"    def test_admin_volume_list(self):"},{"line_number":134,"context_line":"        self.nova(\u0027volume-list\u0027)"},{"line_number":135,"context_line":""},{"line_number":136,"context_line":"    def test_admin_volume_snapshot_list(self):"},{"line_number":137,"context_line":"        out \u003d self.nova(\u0027volume-snapshot-list\u0027, merge_stderr\u003dTrue)"}],"source_content_type":"text/x-python","patch_set":8,"id":"5afe65bd_87fd823c","line":134,"updated":"2015-06-03 02:13:52.000000000","message":"I think this got missed for checking the deprecation message","commit_id":"23f13437dd64496fcbc138bbaa9b0ac615a3cf23"},{"author":{"_account_id":1849,"name":"Joe Gordon","email":"joe.gordon0@gmail.com","username":"jogo"},"change_message_id":"762ff18c6e7e7ca4c54efb3b6280233cef8439b7","unresolved":false,"context_lines":[{"line_number":131,"context_line":"        self.nova(\u0027usage-list\u0027)"},{"line_number":132,"context_line":""},{"line_number":133,"context_line":"    def test_admin_volume_list(self):"},{"line_number":134,"context_line":"        self.nova(\u0027volume-list\u0027)"},{"line_number":135,"context_line":""},{"line_number":136,"context_line":"    def test_admin_volume_snapshot_list(self):"},{"line_number":137,"context_line":"        out \u003d self.nova(\u0027volume-snapshot-list\u0027, merge_stderr\u003dTrue)"}],"source_content_type":"text/x-python","patch_set":8,"id":"5afe65bd_3b1b811d","line":134,"in_reply_to":"5afe65bd_87fd823c","updated":"2015-06-03 08:30:20.000000000","message":"I wonder the same thing","commit_id":"23f13437dd64496fcbc138bbaa9b0ac615a3cf23"}],"novaclient/v2/shell.py":[{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"b6e1e7c655c1ba2bce5715bf6b2a205bf9ac4eab","unresolved":false,"context_lines":[{"line_number":67,"context_line":"def emit_volume_deprecation_warning(command_name):"},{"line_number":68,"context_line":"    warnings.warn(_LW(\u0027Command %s is deprecated and will be removed after \u0027"},{"line_number":69,"context_line":"                      \u0027Nova 2016.1 is released. Use python-cinderlient or \u0027"},{"line_number":70,"context_line":"                      \u0027openstackclient instead.\u0027),"},{"line_number":71,"context_line":"                  DeprecationWarning)"},{"line_number":72,"context_line":""},{"line_number":73,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"7a016987_76617ca8","line":70,"updated":"2015-05-22 20:53:04.000000000","message":"oops, probably need the command name in here....","commit_id":"17558cadc0a905b0b382fd15e501faa0f0ac967f"},{"author":{"_account_id":9420,"name":"Matthew Gilliard","email":"matthew.gilliard@hp.com","username":"gilliard"},"change_message_id":"51d3d87d94e3e1b8b2efabdcb3320dabf97c1a45","unresolved":false,"context_lines":[{"line_number":2085,"context_line":"    metavar\u003d\u0027\u003cvolume\u003e\u0027, nargs\u003d\u0027+\u0027,"},{"line_number":2086,"context_line":"    help\u003d_(\u0027Name or ID of the volume(s) to delete.\u0027))"},{"line_number":2087,"context_line":"def do_volume_delete(cs, args):"},{"line_number":2088,"context_line":"    \"\"\"DEPRECATION: Remove volume(s).\"\"\""},{"line_number":2089,"context_line":"    emit_volume_deprecation_warning(\u0027volume-delete\u0027)"},{"line_number":2090,"context_line":"    for volume in args.volume:"},{"line_number":2091,"context_line":"        try:"}],"source_content_type":"text/x-python","patch_set":3,"id":"7a016987_b1801e8c","line":2088,"updated":"2015-05-22 21:59:18.000000000","message":"s/DEPRECATION/DEPRECATED/ to be like all the others\n\n(these docstrings show up in \"nova help\")","commit_id":"8eb4d6c0a401e2e9bfe80bb3d1bf8a39d2a1cc58"},{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"43ab765525ee80d0b38813d9308cbac2145d2aa8","unresolved":false,"context_lines":[{"line_number":2085,"context_line":"    metavar\u003d\u0027\u003cvolume\u003e\u0027, nargs\u003d\u0027+\u0027,"},{"line_number":2086,"context_line":"    help\u003d_(\u0027Name or ID of the volume(s) to delete.\u0027))"},{"line_number":2087,"context_line":"def do_volume_delete(cs, args):"},{"line_number":2088,"context_line":"    \"\"\"DEPRECATION: Remove volume(s).\"\"\""},{"line_number":2089,"context_line":"    emit_volume_deprecation_warning(\u0027volume-delete\u0027)"},{"line_number":2090,"context_line":"    for volume in args.volume:"},{"line_number":2091,"context_line":"        try:"}],"source_content_type":"text/x-python","patch_set":3,"id":"7a016987_7ca4bdb8","line":2088,"in_reply_to":"7a016987_b1801e8c","updated":"2015-05-22 22:13:14.000000000","message":"You\u0027re right, I was probably distracted when doing this.","commit_id":"8eb4d6c0a401e2e9bfe80bb3d1bf8a39d2a1cc58"},{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"1837f715edaf1e00eb03401a97ed4c48b1356c03","unresolved":false,"context_lines":[{"line_number":64,"context_line":"# NOTE(mriedem): Remove this along with the deprecated commands in the first"},{"line_number":65,"context_line":"# python-novaclient release AFTER the nova server 2016.1 \u0027M\u0027 release."},{"line_number":66,"context_line":"def emit_volume_deprecation_warning(command_name):"},{"line_number":67,"context_line":"    print(\u0027Command %s is deprecated and will be removed after Nova 2016.1 is \u0027"},{"line_number":68,"context_line":"          \u0027released. Use python-cinderclient or openstackclient instead.\u0027 %"},{"line_number":69,"context_line":"          command_name, file\u003dsys.stderr)"},{"line_number":70,"context_line":""}],"source_content_type":"text/x-python","patch_set":6,"id":"5afe65bd_cf9cba6b","line":67,"updated":"2015-06-02 20:19:06.000000000","message":"I\u0027ll add a WARNING prefix to this also, I originally though of this being a DeprecationWarning from the warnings module but that change.  The warnings module will prefix with the type of warning.  Otherwise you get something like this:\n\n[root@novaclient python-novaclient]# nova volume-list\n\nCommand volume-list is deprecated and will be removed after Nova 2016.1 is released. Use python-cinderclient or openstackclient instead.\n\n+----+--------+--------------+------+-------------+-------------+\n| ID | Status | Display Name | Size | Volume Type | Attached to |\n+----+--------+--------------+------+-------------+-------------+\n+----+--------+--------------+------+-------------+-------------+\n\n[root@novaclient python-novaclient]#","commit_id":"8d3dd8cd681b797925e55c8d18ad270ce7330ba5"}],"novaclient/v2/volume_snapshots.py":[{"author":{"_account_id":679,"name":"Kevin L. Mitchell","email":"klmitch@mit.edu","username":"klmitch"},"change_message_id":"5eb7ea0d11404db8961be0a2f0f02d9e7de45916","unresolved":false,"context_lines":[{"line_number":21,"context_line":""},{"line_number":22,"context_line":"from novaclient import base"},{"line_number":23,"context_line":""},{"line_number":24,"context_line":"warnings.warn(\u0027The novaclient.v2.volume_snapshots module is deprecated and \u0027"},{"line_number":25,"context_line":"              \u0027will be removed after Nova 2016.1 is released. Use \u0027"},{"line_number":26,"context_line":"              \u0027python-cinderclient or python-openstacksdk instead.\u0027,"},{"line_number":27,"context_line":"              DeprecationWarning)"}],"source_content_type":"text/x-python","patch_set":6,"id":"7a016987_ada64551","line":24,"updated":"2015-05-26 16:00:23.000000000","message":"So this warning will be emitted when novaclient.v2.client is imported, regardless of whether the user is intending to use the volume API.  Is there any chance we could move this into the actual code in some fashion?","commit_id":"8d3dd8cd681b797925e55c8d18ad270ce7330ba5"},{"author":{"_account_id":679,"name":"Kevin L. Mitchell","email":"klmitch@mit.edu","username":"klmitch"},"change_message_id":"d11d60d64d1c71d6eb990e6bc7350afc1f3f9081","unresolved":false,"context_lines":[{"line_number":21,"context_line":""},{"line_number":22,"context_line":"from novaclient import base"},{"line_number":23,"context_line":""},{"line_number":24,"context_line":"warnings.warn(\u0027The novaclient.v2.volume_snapshots module is deprecated and \u0027"},{"line_number":25,"context_line":"              \u0027will be removed after Nova 2016.1 is released. Use \u0027"},{"line_number":26,"context_line":"              \u0027python-cinderclient or python-openstacksdk instead.\u0027,"},{"line_number":27,"context_line":"              DeprecationWarning)"}],"source_content_type":"text/x-python","patch_set":6,"id":"7a016987_ac938af9","line":24,"in_reply_to":"7a016987_0db63cdc","updated":"2015-05-28 15:47:30.000000000","message":"No, we\u0027d have the same issue, at least with SnapshotManager.__init__(): warning emitted when the client is instantiated; see novaclient/v2/client.py:167.  I think we have to bite the bullet and put the warning in the actual action methods :/","commit_id":"8d3dd8cd681b797925e55c8d18ad270ce7330ba5"},{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"0fbdbd37191dce70119e41b6c1e2f5566be6d78c","unresolved":false,"context_lines":[{"line_number":21,"context_line":""},{"line_number":22,"context_line":"from novaclient import base"},{"line_number":23,"context_line":""},{"line_number":24,"context_line":"warnings.warn(\u0027The novaclient.v2.volume_snapshots module is deprecated and \u0027"},{"line_number":25,"context_line":"              \u0027will be removed after Nova 2016.1 is released. Use \u0027"},{"line_number":26,"context_line":"              \u0027python-cinderclient or python-openstacksdk instead.\u0027,"},{"line_number":27,"context_line":"              DeprecationWarning)"}],"source_content_type":"text/x-python","patch_set":6,"id":"5afe65bd_8a6350b1","line":24,"in_reply_to":"7a016987_ac938af9","updated":"2015-06-02 20:40:30.000000000","message":"Oh right, good point. OK.  I might look at adding a dependency on oslo\u0027s debtcollector package here then if it makes it cleaner, but it might not be worth it.  I\u0027ll take a look.  Just getting back to this this week.","commit_id":"8d3dd8cd681b797925e55c8d18ad270ce7330ba5"},{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"326664675516bdede4f6748a0cf3833857418a0e","unresolved":false,"context_lines":[{"line_number":21,"context_line":""},{"line_number":22,"context_line":"from novaclient import base"},{"line_number":23,"context_line":""},{"line_number":24,"context_line":"warnings.warn(\u0027The novaclient.v2.volume_snapshots module is deprecated and \u0027"},{"line_number":25,"context_line":"              \u0027will be removed after Nova 2016.1 is released. Use \u0027"},{"line_number":26,"context_line":"              \u0027python-cinderclient or python-openstacksdk instead.\u0027,"},{"line_number":27,"context_line":"              DeprecationWarning)"}],"source_content_type":"text/x-python","patch_set":6,"id":"7a016987_0db63cdc","line":24,"in_reply_to":"7a016987_ada64551","updated":"2015-05-27 21:41:27.000000000","message":"Oh right, gross.  I was following what was done in the v1_1/__init__.py but that\u0027s not the same issue here.\n\nI could move this into the classes themselves, maybe in an __init__ method.","commit_id":"8d3dd8cd681b797925e55c8d18ad270ce7330ba5"}]}
