)]}'
{"monasca_api/v2/reference/__init__.py":[{"author":{"_account_id":11809,"name":"Craig Bryant","email":"craig.bryant@hpe.com","username":"craig-bryant"},"change_message_id":"a9d338d52bdd567bd95f75e73d3a3d436431652d","unresolved":false,"context_lines":[{"line_number":41,"context_line":"                 cfg.ListOpt(\u0027agent_authorized_roles\u0027, default\u003d[\u0027agent\u0027],"},{"line_number":42,"context_line":"                             help\u003d\u0027Roles that are only allowed to POST to \u0027"},{"line_number":43,"context_line":"                                  \u0027the API\u0027),"},{"line_number":44,"context_line":"                 cfg.ListOpt(\u0027read_only_authorized_roles\u0027, default\u003d[\u0027monasca-user\u0027],"},{"line_number":45,"context_line":"                             help\u003d\u0027Roles that are only allowed to GET from \u0027"},{"line_number":46,"context_line":"                                  \u0027the API\u0027),"},{"line_number":47,"context_line":"                 cfg.ListOpt(\u0027delegate_authorized_roles\u0027, default\u003d[\u0027admin\u0027],"}],"source_content_type":"text/x-python","patch_set":3,"id":"dada55a8_8c8a90bd","line":44,"range":{"start_line":44,"start_character":69,"end_line":44,"end_character":82},"updated":"2016-07-26 22:15:02.000000000","message":"Does this cause issues since devstack is configured with users whose role is monasca-user?","commit_id":"be89be735d95d40762776c4428af324aab5514dc"},{"author":{"_account_id":8126,"name":"Bradley Klein","email":"bradley.klein@charter.com","username":"bklei"},"change_message_id":"b29c8c27009606ff18a6a047ca8602d09b6b4acb","unresolved":false,"context_lines":[{"line_number":41,"context_line":"                 cfg.ListOpt(\u0027agent_authorized_roles\u0027, default\u003d[\u0027agent\u0027],"},{"line_number":42,"context_line":"                             help\u003d\u0027Roles that are only allowed to POST to \u0027"},{"line_number":43,"context_line":"                                  \u0027the API\u0027),"},{"line_number":44,"context_line":"                 cfg.ListOpt(\u0027read_only_authorized_roles\u0027, default\u003d[\u0027monasca-user\u0027],"},{"line_number":45,"context_line":"                             help\u003d\u0027Roles that are only allowed to GET from \u0027"},{"line_number":46,"context_line":"                                  \u0027the API\u0027),"},{"line_number":47,"context_line":"                 cfg.ListOpt(\u0027delegate_authorized_roles\u0027, default\u003d[\u0027admin\u0027],"}],"source_content_type":"text/x-python","patch_set":3,"id":"dada55a8_cf691a8e","line":44,"range":{"start_line":44,"start_character":69,"end_line":44,"end_character":82},"in_reply_to":"dada55a8_8c8a90bd","updated":"2016-07-26 22:24:42.000000000","message":"Perhaps -- I can change to monasca-read-only-user, but since it\u0027s only a default, I\u0027m assuming the checked-in config files are overriding anything set here...","commit_id":"be89be735d95d40762776c4428af324aab5514dc"},{"author":{"_account_id":14517,"name":"Ryan Brandt","email":"ryan.brandt@hpe.com","username":"ryan-brandt"},"change_message_id":"235445881ab07ab9a3dbb2a2403b5267c9388ef4","unresolved":false,"context_lines":[{"line_number":41,"context_line":"                 cfg.ListOpt(\u0027agent_authorized_roles\u0027, default\u003d[\u0027agent\u0027],"},{"line_number":42,"context_line":"                             help\u003d\u0027Roles that are only allowed to POST to \u0027"},{"line_number":43,"context_line":"                                  \u0027the API\u0027),"},{"line_number":44,"context_line":"                 cfg.ListOpt(\u0027read_only_authorized_roles\u0027, default\u003d[\u0027monasca-read-only--user\u0027],"},{"line_number":45,"context_line":"                             help\u003d\u0027Roles that are only allowed to GET from \u0027"},{"line_number":46,"context_line":"                                  \u0027the API\u0027),"},{"line_number":47,"context_line":"                 cfg.ListOpt(\u0027delegate_authorized_roles\u0027, default\u003d[\u0027admin\u0027],"}],"source_content_type":"text/x-python","patch_set":4,"id":"bacf61ea_1c920558","line":44,"range":{"start_line":44,"start_character":87,"end_line":44,"end_character":88},"updated":"2016-07-27 15:14:16.000000000","message":"extra \u0027-\u0027 here","commit_id":"a4dc23c8c6e0eea969732712f914236efc3d8a59"}],"monasca_tempest_tests/tests/api/test_read_only_role.py":[{"author":{"_account_id":11809,"name":"Craig Bryant","email":"craig.bryant@hpe.com","username":"craig-bryant"},"change_message_id":"a9d338d52bdd567bd95f75e73d3a3d436431652d","unresolved":false,"context_lines":[{"line_number":36,"context_line":"    @classmethod"},{"line_number":37,"context_line":"    def resource_cleanup(cls):"},{"line_number":38,"context_line":"        super(TestReadOnlyRole, cls).resource_cleanup()"},{"line_number":39,"context_line":""},{"line_number":40,"context_line":"    @test.attr(type\u003d\"gate\")"},{"line_number":41,"context_line":"    @test.attr(type\u003d[\u0027negative\u0027])"},{"line_number":42,"context_line":"    def test_delete_alarms_fails(self):"}],"source_content_type":"text/x-python","patch_set":3,"id":"dada55a8_acd5d4e5","line":39,"updated":"2016-07-26 22:15:02.000000000","message":"Why no positive test?","commit_id":"be89be735d95d40762776c4428af324aab5514dc"},{"author":{"_account_id":8126,"name":"Bradley Klein","email":"bradley.klein@charter.com","username":"bklei"},"change_message_id":"b29c8c27009606ff18a6a047ca8602d09b6b4acb","unresolved":false,"context_lines":[{"line_number":36,"context_line":"    @classmethod"},{"line_number":37,"context_line":"    def resource_cleanup(cls):"},{"line_number":38,"context_line":"        super(TestReadOnlyRole, cls).resource_cleanup()"},{"line_number":39,"context_line":""},{"line_number":40,"context_line":"    @test.attr(type\u003d\"gate\")"},{"line_number":41,"context_line":"    @test.attr(type\u003d[\u0027negative\u0027])"},{"line_number":42,"context_line":"    def test_delete_alarms_fails(self):"}],"source_content_type":"text/x-python","patch_set":3,"id":"dada55a8_2fcc36ba","line":39,"in_reply_to":"dada55a8_acd5d4e5","updated":"2016-07-26 22:24:42.000000000","message":"I\u0027m having difficulty with the oslo cred_provider to change roles/users after the class is instantiated.  To perform a positive test I\u0027d need to use the full admin role to create something to GET, then switch to read-only and verify GET works.  If you have an example of how to make this work I\u0027d love to see it...","commit_id":"be89be735d95d40762776c4428af324aab5514dc"},{"author":{"_account_id":14517,"name":"Ryan Brandt","email":"ryan.brandt@hpe.com","username":"ryan-brandt"},"change_message_id":"235445881ab07ab9a3dbb2a2403b5267c9388ef4","unresolved":false,"context_lines":[{"line_number":39,"context_line":""},{"line_number":40,"context_line":"    @test.attr(type\u003d\"gate\")"},{"line_number":41,"context_line":"    @test.attr(type\u003d[\u0027negative\u0027])"},{"line_number":42,"context_line":"    def test_delete_alarms_fails(self):"},{"line_number":43,"context_line":"        self.assertRaises(exceptions.Unauthorized,"},{"line_number":44,"context_line":"                          self.monasca_client.delete_alarm, \"foo\")"},{"line_number":45,"context_line":""}],"source_content_type":"text/x-python","patch_set":4,"id":"bacf61ea_17481e2d","line":42,"range":{"start_line":42,"start_character":4,"end_line":42,"end_character":39},"updated":"2016-07-27 15:14:16.000000000","message":"Seems like it would be best if all endpoints were tested, perhaps grouped into negative and positive?","commit_id":"a4dc23c8c6e0eea969732712f914236efc3d8a59"},{"author":{"_account_id":2419,"name":"Roland Hochmuth","email":"roland.hochmuth@hpe.com","username":"roland-hochmuth-s"},"change_message_id":"e22cd13316fc1931338f6b857533436d72c21ec1","unresolved":false,"context_lines":[{"line_number":1,"context_line":"# (C) Copyright 2016 Hewlett Packard Enterprise Development Company LP"},{"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":7,"id":"bacf61ea_4e9b9c18","line":1,"updated":"2016-08-04 16:22:35.000000000","message":"As this is a completely new file, I don\u0027t think it shoudl be copyright HPE.","commit_id":"c818c815d32f9e4d3368dbe3716bb784f00a780a"}]}