)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"change_message_id":"05b1e4fecc97c63db617477147a9dfe11c426ac6","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"d61e6b1b_5c041d8a","updated":"2021-11-07 11:05:55.000000000","message":"Good catch /o\\","commit_id":"47c4d56e446a49726e7145c6cbc3bb7620a431f7"},{"author":{"_account_id":13177,"name":"Emma Foley","email":"efoley@redhat.com","username":"emma-l-foley"},"change_message_id":"6f3b9d583c6b0716893995e175a48ae42e475734","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"3212f4dd_106179da","updated":"2021-11-05 16:16:34.000000000","message":"Hey, what\u0027s the status on this, what does it need to get merged? \nIt\u0027s a follow-up to an existing patch, which is backported as far as train.","commit_id":"47c4d56e446a49726e7145c6cbc3bb7620a431f7"},{"author":{"_account_id":13177,"name":"Emma Foley","email":"efoley@redhat.com","username":"emma-l-foley"},"change_message_id":"5e7db66ffada8cc2e1e8cbd09f41eb9cd3f132e3","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"c30c5be9_554a31b7","updated":"2021-11-08 10:36:41.000000000","message":"Thanks Stephen!","commit_id":"47c4d56e446a49726e7145c6cbc3bb7620a431f7"}],"releasenotes/notes/redis-connect-retries-c9adfc81eb06a4ab.yaml":[{"author":{"_account_id":28522,"name":"Hervé Beraud","email":"herveberaud.pro@gmail.com","username":"hberaud"},"change_message_id":"68d83f1e392442b84205c6a16004fc18b3fbda7d","unresolved":true,"context_lines":[{"line_number":1,"context_line":"---"},{"line_number":2,"context_line":"features:"},{"line_number":3,"context_line":"  - |"},{"line_number":4,"context_line":"    Redis driver retries actions for up to 15 times when met with error"},{"line_number":5,"context_line":"    connecting to Redis."}],"source_content_type":"text/x-yaml","patch_set":3,"id":"d5731369_53599fa9","line":4,"range":{"start_line":4,"start_character":40,"end_line":4,"end_character":52},"updated":"2021-06-16 17:05:39.000000000","message":"Could be less strict concerning the limit of the retry:\n\n\"until the retry limit is reached (default to 15 times)\"","commit_id":"47c4d56e446a49726e7145c6cbc3bb7620a431f7"}],"tooz/drivers/redis.py":[{"author":{"_account_id":28522,"name":"Hervé Beraud","email":"herveberaud.pro@gmail.com","username":"hberaud"},"change_message_id":"2a2d404612b065f8a0f669b15c225f5371d8b450","unresolved":true,"context_lines":[{"line_number":54,"context_line":"                    # retry ntries times and then raise a connection error"},{"line_number":55,"context_line":"                    ntries -\u003d 1"},{"line_number":56,"context_line":"                    if not ntries:"},{"line_number":57,"context_line":"                        utils.raise_with_cause("},{"line_number":58,"context_line":"                            coordination.ToozConnectionError,"},{"line_number":59,"context_line":"                            encodeutils.exception_to_unicode(e),"},{"line_number":60,"context_line":"                            cause\u003de)"}],"source_content_type":"text/x-python","patch_set":2,"id":"6e21116f_cc42238f","line":57,"range":{"start_line":57,"start_character":24,"end_line":57,"end_character":47},"updated":"2021-06-16 13:48:19.000000000","message":"LGTM, although, we will raise an exception, I wonder if we shouldn\u0027t explicitly indicate that we reached our n retries. It Could help during debug.\n\nExample:\n```\nLOG.debug(\"Redis connection error, retry limit has been reached, aborting - %s\", e)\n```","commit_id":"e667547acb8f29d3f172aab12050b8013b4af2a9"},{"author":{"_account_id":9542,"name":"Pavlo Shchelokovskyy","email":"pshchelokovskyy@mirantis.com","username":"pshchelo"},"change_message_id":"311eded2ae3b08970b34def3a7bf451382da3e0b","unresolved":false,"context_lines":[{"line_number":54,"context_line":"                    # retry ntries times and then raise a connection error"},{"line_number":55,"context_line":"                    ntries -\u003d 1"},{"line_number":56,"context_line":"                    if not ntries:"},{"line_number":57,"context_line":"                        utils.raise_with_cause("},{"line_number":58,"context_line":"                            coordination.ToozConnectionError,"},{"line_number":59,"context_line":"                            encodeutils.exception_to_unicode(e),"},{"line_number":60,"context_line":"                            cause\u003de)"}],"source_content_type":"text/x-python","patch_set":2,"id":"c307332f_987b655d","line":57,"range":{"start_line":57,"start_character":24,"end_line":57,"end_character":47},"in_reply_to":"6e21116f_cc42238f","updated":"2021-06-16 16:37:25.000000000","message":"Ack","commit_id":"e667547acb8f29d3f172aab12050b8013b4af2a9"},{"author":{"_account_id":9542,"name":"Pavlo Shchelokovskyy","email":"pshchelokovskyy@mirantis.com","username":"pshchelo"},"change_message_id":"dc7565bbe53b00f7dadf8c88ace0fdc69cc893d5","unresolved":false,"context_lines":[{"line_number":54,"context_line":"                    # retry ntries times and then raise a connection error"},{"line_number":55,"context_line":"                    ntries -\u003d 1"},{"line_number":56,"context_line":"                    if not ntries:"},{"line_number":57,"context_line":"                        utils.raise_with_cause("},{"line_number":58,"context_line":"                            coordination.ToozConnectionError,"},{"line_number":59,"context_line":"                            encodeutils.exception_to_unicode(e),"},{"line_number":60,"context_line":"                            cause\u003de)"}],"source_content_type":"text/x-python","patch_set":2,"id":"7f606f03_02b67954","line":57,"range":{"start_line":57,"start_character":24,"end_line":57,"end_character":47},"in_reply_to":"c307332f_987b655d","updated":"2021-06-16 16:45:34.000000000","message":"done","commit_id":"e667547acb8f29d3f172aab12050b8013b4af2a9"}]}
