)]}'
{"swift/common/storage_policy.py":[{"author":{"_account_id":7233,"name":"Matthew Oliver","email":"matt@oliver.net.au","username":"mattoliverau"},"change_message_id":"16627ce6f3ff1ab3b2a5e62c14bde0cf3db6c247","unresolved":false,"context_lines":[{"line_number":473,"context_line":"                    \u0027In a future release, this will prevent services from \u0027"},{"line_number":474,"context_line":"                    \u0027starting unless the policy is marked as deprecated.\u0027)"},{"line_number":475,"context_line":"                # raise ValueError(\u0027Storage policy %s MUST be deprecated\u0027 %"},{"line_number":476,"context_line":"                #                  self.name)"},{"line_number":477,"context_line":""},{"line_number":478,"context_line":"        # Initialize PyECLib EC backend"},{"line_number":479,"context_line":"        try:"}],"source_content_type":"text/x-python","patch_set":2,"id":"3a461143_15618105","line":476,"updated":"2017-01-30 02:41:10.000000000","message":"We need to remember to do this in the future. Having a todo here seems to be adding some future technical debt. So we need to make sure we have something to remind us of this.\n\nMaybe just mentioning it in the bug once it gets down-greaded after this partial fix is in place is good enough. Or some reminder on Swift priority or ideas page. I dunno.","commit_id":"45f8b7f347feef44df6d03561ba711c8493216a0"},{"author":{"_account_id":15343,"name":"Tim Burke","email":"tburke@nvidia.com","username":"tburke"},"change_message_id":"a4724cc5565cea860b7d9e6cc7465eebb9f540e5","unresolved":false,"context_lines":[{"line_number":473,"context_line":"                    \u0027In a future release, this will prevent services from \u0027"},{"line_number":474,"context_line":"                    \u0027starting unless the policy is marked as deprecated.\u0027)"},{"line_number":475,"context_line":"                # raise ValueError(\u0027Storage policy %s MUST be deprecated\u0027 %"},{"line_number":476,"context_line":"                #                  self.name)"},{"line_number":477,"context_line":""},{"line_number":478,"context_line":"        # Initialize PyECLib EC backend"},{"line_number":479,"context_line":"        try:"}],"source_content_type":"text/x-python","patch_set":2,"id":"3a461143_212d4695","line":476,"in_reply_to":"3a461143_15618105","updated":"2017-01-30 18:43:04.000000000","message":"I\u0027ll add a note to the bug after this lands. Something tells me John will keep it in mind anyway, though :-)","commit_id":"45f8b7f347feef44df6d03561ba711c8493216a0"}],"test/unit/common/test_storage_policy.py":[{"author":{"_account_id":12279,"name":"Janie Richling","email":"jrichli@us.ibm.com","username":"jrichli"},"change_message_id":"318ce84941f58ad2c3438d902a9a9928bbc4a2be","unresolved":false,"context_lines":[{"line_number":645,"context_line":"        policy_type \u003d erasure_coding"},{"line_number":646,"context_line":"        ec_type \u003d isa_l_rs_vand"},{"line_number":647,"context_line":"        ec_num_data_fragments \u003d 10"},{"line_number":648,"context_line":"        ec_num_parity_fragments \u003d 5"},{"line_number":649,"context_line":"        \"\"\")"},{"line_number":650,"context_line":""},{"line_number":651,"context_line":"        with capture_logging(\u0027swift.common.storage_policy\u0027) as records:"}],"source_content_type":"text/x-python","patch_set":2,"id":"3a461143_39d2085d","line":648,"updated":"2017-01-28 06:14:18.000000000","message":"you could add another policy here that is deprecated and make policy 0 the default.  Then you can assert that the \"In a future release\" statement is not used for a deprecated policy.","commit_id":"45f8b7f347feef44df6d03561ba711c8493216a0"},{"author":{"_account_id":7233,"name":"Matthew Oliver","email":"matt@oliver.net.au","username":"mattoliverau"},"change_message_id":"16627ce6f3ff1ab3b2a5e62c14bde0cf3db6c247","unresolved":false,"context_lines":[{"line_number":645,"context_line":"        policy_type \u003d erasure_coding"},{"line_number":646,"context_line":"        ec_type \u003d isa_l_rs_vand"},{"line_number":647,"context_line":"        ec_num_data_fragments \u003d 10"},{"line_number":648,"context_line":"        ec_num_parity_fragments \u003d 5"},{"line_number":649,"context_line":"        \"\"\")"},{"line_number":650,"context_line":""},{"line_number":651,"context_line":"        with capture_logging(\u0027swift.common.storage_policy\u0027) as records:"}],"source_content_type":"text/x-python","patch_set":2,"id":"3a461143_954c919f","line":648,"in_reply_to":"3a461143_39d2085d","updated":"2017-01-30 02:41:10.000000000","message":"+1 here, seeing as you\u0027ve added the deprecation check, we should really test it.","commit_id":"45f8b7f347feef44df6d03561ba711c8493216a0"},{"author":{"_account_id":15343,"name":"Tim Burke","email":"tburke@nvidia.com","username":"tburke"},"change_message_id":"a4724cc5565cea860b7d9e6cc7465eebb9f540e5","unresolved":false,"context_lines":[{"line_number":645,"context_line":"        policy_type \u003d erasure_coding"},{"line_number":646,"context_line":"        ec_type \u003d isa_l_rs_vand"},{"line_number":647,"context_line":"        ec_num_data_fragments \u003d 10"},{"line_number":648,"context_line":"        ec_num_parity_fragments \u003d 5"},{"line_number":649,"context_line":"        \"\"\")"},{"line_number":650,"context_line":""},{"line_number":651,"context_line":"        with capture_logging(\u0027swift.common.storage_policy\u0027) as records:"}],"source_content_type":"text/x-python","patch_set":2,"id":"3a461143_2106261d","line":648,"in_reply_to":"3a461143_954c919f","updated":"2017-01-30 18:43:04.000000000","message":"Done","commit_id":"45f8b7f347feef44df6d03561ba711c8493216a0"},{"author":{"_account_id":12279,"name":"Janie Richling","email":"jrichli@us.ibm.com","username":"jrichli"},"change_message_id":"fe1f254b007e50f3fab71b61757a09c2bdbeb194","unresolved":false,"context_lines":[{"line_number":661,"context_line":"        self.assertIn(\u0027In a future release, this will prevent services from \u0027"},{"line_number":662,"context_line":"                      \u0027starting\u0027, records[1].message)"},{"line_number":663,"context_line":""},{"line_number":664,"context_line":"        slightly_less_bad_conf \u003d self._conf(\"\"\""},{"line_number":665,"context_line":"        [storage-policy:0]"},{"line_number":666,"context_line":"        name \u003d bad-policy"},{"line_number":667,"context_line":"        policy_type \u003d erasure_coding"}],"source_content_type":"text/x-python","patch_set":4,"id":"3a461143_7260f701","line":664,"updated":"2017-01-31 07:24:56.000000000","message":"great name :-)","commit_id":"c7bffd6cee400b703c76e47f8b8602ca775d69ec"}]}