)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":13252,"name":"Dr. Jens Harbott","display_name":"Jens Harbott (frickler)","email":"frickler@offenerstapel.de","username":"jrosenboom"},"change_message_id":"d79d59aa1552e0efd23060232601a74e933c8067","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"29beda7a_b1e6cadc","updated":"2022-01-13 13:10:45.000000000","message":"Patch seems valid according to the referenced patches, although some stable backports aren\u0027t merged yet. Further question inline.","commit_id":"f6fe297dd50758fbe7e46b93b6eb4b79bbd65517"},{"author":{"_account_id":15197,"name":"Pierre Riteau","email":"pierre@stackhpc.com","username":"priteau","status":"StackHPC"},"change_message_id":"fb69db6af5ec83d75de7954e06da015759065444","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"73fb4370_5e73ed76","updated":"2022-01-13 13:56:49.000000000","message":"Added release note\n\nI also proposed another patch for the other option: https://review.opendev.org/c/openstack/kolla-ansible/+/824568/","commit_id":"073c6959dd041cbbb343dc1d7d85ccbe41c763c5"},{"author":{"_account_id":30491,"name":"Radosław Piliszek","display_name":"Radek","email":"radek@piliszek.it","username":"yoctozepto","status":"self-employed techologist, collaborating mostly with 7bulls.com"},"change_message_id":"e30cd78b9566a1ab27acdd989e620c8cf9e05fb1","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"9c228212_e0a34f32","updated":"2022-01-18 13:54:35.000000000","message":"in general, we should stop owning the defaults","commit_id":"073c6959dd041cbbb343dc1d7d85ccbe41c763c5"},{"author":{"_account_id":15197,"name":"Pierre Riteau","email":"pierre@stackhpc.com","username":"priteau","status":"StackHPC"},"change_message_id":"03b10c57ed5d5d918002371c37f8cc59e5f2e5c8","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"fe64a0ff_c3249a6e","updated":"2022-01-18 15:25:01.000000000","message":"Failure was due to a mirror issue\n\nConnection failed [IP: 188.212.109.26 443]\\","commit_id":"20a3b14001f68d5f64aa70b124d7a6dcd6a4e777"},{"author":{"_account_id":15197,"name":"Pierre Riteau","email":"pierre@stackhpc.com","username":"priteau","status":"StackHPC"},"change_message_id":"65297bdab29b50d3ccf70594465f7e7633ed6096","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"4a44480d_bad6d8eb","updated":"2022-01-18 15:22:57.000000000","message":"Gate failed so I fixed the nit.","commit_id":"20a3b14001f68d5f64aa70b124d7a6dcd6a4e777"}],"ansible/roles/barbican/templates/barbican.conf.j2":[{"author":{"_account_id":13252,"name":"Dr. Jens Harbott","display_name":"Jens Harbott (frickler)","email":"frickler@offenerstapel.de","username":"jrosenboom"},"change_message_id":"d79d59aa1552e0efd23060232601a74e933c8067","unresolved":true,"context_lines":[{"line_number":10,"context_line":"host_href \u003d {{ barbican_public_endpoint }}"},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"backlog \u003d 4096"},{"line_number":13,"context_line":"max_allowed_request_size_in_bytes \u003d 1000000"},{"line_number":14,"context_line":""},{"line_number":15,"context_line":"db_auto_create \u003d False"},{"line_number":16,"context_line":"sql_connection \u003d mysql+pymysql://{{ barbican_database_user }}:{{ barbican_database_password }}@{{ barbican_database_address }}/{{ barbican_database_name }}"}],"source_content_type":"text/x-jinja2","patch_set":1,"id":"898e5660_c2b01d01","line":13,"updated":"2022-01-13 13:10:45.000000000","message":"What about this one, do we still need to override it?\nIt was bumped in the same barbican patch from 15k to 25k, so 1M seems a bit excessive to me.","commit_id":"f6fe297dd50758fbe7e46b93b6eb4b79bbd65517"},{"author":{"_account_id":15197,"name":"Pierre Riteau","email":"pierre@stackhpc.com","username":"priteau","status":"StackHPC"},"change_message_id":"3700b1c0e905e1fdc53622f8b4509fe3fe401e28","unresolved":true,"context_lines":[{"line_number":10,"context_line":"host_href \u003d {{ barbican_public_endpoint }}"},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"backlog \u003d 4096"},{"line_number":13,"context_line":"max_allowed_request_size_in_bytes \u003d 1000000"},{"line_number":14,"context_line":""},{"line_number":15,"context_line":"db_auto_create \u003d False"},{"line_number":16,"context_line":"sql_connection \u003d mysql+pymysql://{{ barbican_database_user }}:{{ barbican_database_password }}@{{ barbican_database_address }}/{{ barbican_database_name }}"}],"source_content_type":"text/x-jinja2","patch_set":1,"id":"2f2ab465_0818c18d","line":13,"in_reply_to":"898e5660_c2b01d01","updated":"2022-01-13 13:19:03.000000000","message":"Indeed, I wanted to discuss this with anyone potentially impacted. It has been set to 1,000,000 since the original barbican role commit in 2016, without justification.","commit_id":"f6fe297dd50758fbe7e46b93b6eb4b79bbd65517"}],"releasenotes/notes/barbican-max-allowed-secret-size-1941307ab5d2a9fd.yaml":[{"author":{"_account_id":30491,"name":"Radosław Piliszek","display_name":"Radek","email":"radek@piliszek.it","username":"yoctozepto","status":"self-employed techologist, collaborating mostly with 7bulls.com"},"change_message_id":"44fdc6c7df77062e963cb57a22b924fcdbfed40e","unresolved":true,"context_lines":[{"line_number":3,"context_line":"  - |"},{"line_number":4,"context_line":"    Removes custom value of ``max_allowed_secret_in_bytes`` in"},{"line_number":5,"context_line":"    ``barbican.conf``. The default maximum size in Barbican was doubled to"},{"line_number":6,"context_line":"    avoid issues with some certificates."}],"source_content_type":"text/x-yaml","patch_set":2,"id":"1969d6d7_842efcc9","line":6,"updated":"2022-01-18 13:53:54.000000000","message":"nit: missing reference to LP but I don\u0027t mind that much as all the info is here","commit_id":"073c6959dd041cbbb343dc1d7d85ccbe41c763c5"},{"author":{"_account_id":15197,"name":"Pierre Riteau","email":"pierre@stackhpc.com","username":"priteau","status":"StackHPC"},"change_message_id":"65297bdab29b50d3ccf70594465f7e7633ed6096","unresolved":false,"context_lines":[{"line_number":3,"context_line":"  - |"},{"line_number":4,"context_line":"    Removes custom value of ``max_allowed_secret_in_bytes`` in"},{"line_number":5,"context_line":"    ``barbican.conf``. The default maximum size in Barbican was doubled to"},{"line_number":6,"context_line":"    avoid issues with some certificates."}],"source_content_type":"text/x-yaml","patch_set":2,"id":"98581ace_d2b90115","line":6,"in_reply_to":"1969d6d7_842efcc9","updated":"2022-01-18 15:22:57.000000000","message":"Done","commit_id":"073c6959dd041cbbb343dc1d7d85ccbe41c763c5"}]}
