)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":7973,"name":"Douglas Mendizábal","email":"dmendiza@redhat.com","username":"dougmendizabal"},"change_message_id":"7a061d97de405bc57a08fdc747ea641eeced29c7","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"db1c230a_14765b11","updated":"2022-02-11 16:55:47.000000000","message":"This change looks good, but it needs a release not as it is a change to the default values (yes, even though the existing default is broken and would never affect anything 😅)","commit_id":"927aa80005b157323a9bcce4702ae39d1fb8d236"},{"author":{"_account_id":16465,"name":"Kristi Nikolla","email":"knikolla@bu.edu","username":"knikolla"},"change_message_id":"2d588f2d0f5fe4d3134a3f2aa5e2c36de6c57eb6","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"9a1e543e_c16429c0","updated":"2022-03-18 12:57:05.000000000","message":"We discussed this during last weeks reviewathon. I\u0027m of the opinion that this is changing default behavior, and therefore the correct approach would be to remove the nonexistent event from the list of opt out events, rather than correct the spelling and change the default behavior.","commit_id":"927aa80005b157323a9bcce4702ae39d1fb8d236"},{"author":{"_account_id":14250,"name":"Grzegorz Grasza","email":"xek@redhat.com","username":"xek"},"change_message_id":"30b60bd4ccdbffb112d6e641a067292379934c28","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"06ee9f44_28842b0c","updated":"2022-02-25 16:54:29.000000000","message":"imo this needs a release note","commit_id":"927aa80005b157323a9bcce4702ae39d1fb8d236"},{"author":{"_account_id":14250,"name":"Grzegorz Grasza","email":"xek@redhat.com","username":"xek"},"change_message_id":"a08067fbfec210f6e1eda183cfa9f000e15d9bcb","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"1d26e25e_5e52b0d1","updated":"2022-02-25 16:57:12.000000000","message":"it also needs to be fixed in the docs here: doc/source/event_notifications.rst","commit_id":"927aa80005b157323a9bcce4702ae39d1fb8d236"},{"author":{"_account_id":21420,"name":"Gage Hugo","email":"gagehugo@gmail.com","username":"ghugo"},"change_message_id":"0616f09d9cdf0ec231aec2702b0b1210afd14f05","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"2935e23f_490fa2a1","updated":"2021-12-13 15:56:31.000000000","message":"recheck","commit_id":"927aa80005b157323a9bcce4702ae39d1fb8d236"},{"author":{"_account_id":13478,"name":"Boris Bobrov","email":"b.bobrov@sap.com","username":"bbobrov"},"change_message_id":"746779e449ba7687356bb57307a76ab08648313b","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"47273194_df380aff","updated":"2021-12-14 16:26:51.000000000","message":"recheck","commit_id":"927aa80005b157323a9bcce4702ae39d1fb8d236"},{"author":{"_account_id":13478,"name":"Boris Bobrov","email":"b.bobrov@sap.com","username":"bbobrov"},"change_message_id":"6073682d960a26dea65ec17adf0269c29a5ec08e","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"62756a30_8134cb0c","updated":"2021-12-14 09:35:55.000000000","message":"recheck","commit_id":"927aa80005b157323a9bcce4702ae39d1fb8d236"},{"author":{"_account_id":14250,"name":"Grzegorz Grasza","email":"xek@redhat.com","username":"xek"},"change_message_id":"4793ab94e1c5db39fb1dbfaffc62728cdc2d0cf9","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":2,"id":"4d6b3ad7_8640f106","in_reply_to":"35be55f7_a7218d31","updated":"2022-10-12 08:12:10.000000000","message":"I concur with Kristi, we should keep the default behavior.","commit_id":"927aa80005b157323a9bcce4702ae39d1fb8d236"},{"author":{"_account_id":8313,"name":"Lajos Katona","display_name":"lajoskatona","email":"katonalala@gmail.com","username":"elajkat","status":"Ericsson Software Technology"},"change_message_id":"c790cdfc0b2dc08304b1af407269145592d02334","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":2,"id":"b3bf1c60_bc8c39c1","in_reply_to":"4d6b3ad7_8640f106","updated":"2022-10-13 07:35:52.000000000","message":"ack, I will check that way than","commit_id":"927aa80005b157323a9bcce4702ae39d1fb8d236"},{"author":{"_account_id":8313,"name":"Lajos Katona","display_name":"lajoskatona","email":"katonalala@gmail.com","username":"elajkat","status":"Ericsson Software Technology"},"change_message_id":"b9db3b0a682f8169480df19a7458a5557cd6816a","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"35be55f7_a7218d31","in_reply_to":"9a1e543e_c16429c0","updated":"2022-10-05 06:47:54.000000000","message":"Hi, we also run into this low-hanging issue :-) My understanding is that the users/deployers anyway change this default at last when they realize that they have more events than expected.\nMy idea is that this is a default behavior so correcting it help users. Of course I can live with your option also as worst case.","commit_id":"927aa80005b157323a9bcce4702ae39d1fb8d236"},{"author":{"_account_id":8313,"name":"Lajos Katona","display_name":"lajoskatona","email":"katonalala@gmail.com","username":"elajkat","status":"Ericsson Software Technology"},"change_message_id":"4d8c7dff507c81bf1d30d09f518bf01d97c0b567","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"75e39008_1d432e93","updated":"2022-10-05 11:35:17.000000000","message":"recheck\nbased on quick search (see [1]) the failure of test_identity_list_credentials from keystone-protection-functional is not related to this patch\n\n[1]: https://opensearch.logs.openstack.org/_dashboards/app/discover/?security_tenant\u003dglobal#/?_g\u003d(filters:!(),refreshInterval:(pause:!t,value:0),time:(from:\u00272022-09-14T11:03:15.172Z\u0027,to:now))\u0026_a\u003d(columns:!(build_name,build_status),filters:!(),index:\u002794869730-aea8-11ec-9e6a-83741af3fdcd\u0027,interval:auto,query:(language:kuery,query:\u0027message:%22Could%20not%20find%20credential%22\u0027),sort:!())","commit_id":"8632e0fc212cfeb05e8a2a1535bcc06eec65f8df"},{"author":{"_account_id":8313,"name":"Lajos Katona","display_name":"lajoskatona","email":"katonalala@gmail.com","username":"elajkat","status":"Ericsson Software Technology"},"change_message_id":"80129dbeca6c800111d1b1908c0dced8c148686e","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"dc0ae702_703a8d11","updated":"2022-10-17 09:03:30.000000000","message":"recheck\nIssue with keystone_tempest_plugin.tests.rbac.v3.test_credential.SystemMemberTests\t.test_identity_list_credentials \u0026 keystone_tempest_plugin.tests.rbac.v3.test_credential.ProjectMemberTests.test_identity_list_credentials seems unrelated to the patch by the logs","commit_id":"0b46eab168c87f71ec23b3bf6bda027c8c35524e"}]}
