diff options
author | Stephen Finucane <stephenfin@redhat.com> | 2023-03-22 12:05:06 +0000 |
---|---|---|
committer | Stephen Finucane <stephenfin@redhat.com> | 2023-04-17 10:17:44 +0100 |
commit | 7d62b3664e4acad9161d849a111bdb8b4d707f61 (patch) | |
tree | 4ea11f3b3e7cc419c009d2457fac5d85be9f4b52 /.zuul.yaml | |
parent | 04acea8cbbf7a989e07dad66fd40ac1d3ba4e0d2 (diff) | |
download | oslo-db-7d62b3664e4acad9161d849a111bdb8b4d707f61.tar.gz |
Match exceptions with multiple lines
We were creating regexes without the DOTALL flag, which meant '.' wasn't
matching newlines. This meant exceptions that contained multiple lines
would not be caught. For example, in my environment where Kerberos is
used, I see the following otherwise harmless message:
(psycopg2.OperationalError) connection to server at "localhost" (::1),
port 5432 failed: could not initiate GSSAPI security context: No
credentials were supplied, or the credentials were unavailable or
inaccessible: Configuration file does not specify default realm
connection to server at "localhost" (::1), port 5432 failed: FATAL:
database "non_existent_database" does not exist
The presence of that newline causes our matchers to fail and the
exception is not wrapped. Correct this.
In the meanwhile, we reformat the function that does the wrapping to
make it a little flatter. This was difficult to modify (for debugging
purposes) due to the level of indentation.
Change-Id: I5396a5a3272e6984954d819cfc71507283c775db
Signed-off-by: Stephen Finucane <stephenfin@redhat.com>
Diffstat (limited to '.zuul.yaml')
0 files changed, 0 insertions, 0 deletions