Scale back on how many warnings we issue
* Remove "Request returned failure status" warning, it does not carry
any helpful information for an operator (or anyone).
* Demote retry warnings to debug level. Retrying is essentially a normal
thing in the real world, and currently we're spamming logs with warnings.
http://logs.openstack.org/93/255793/3/gate/gate-tempest-dsvm-ironic-agent_ssh/25175ed/logs/screen-n-cpu.txt.gz?level=TRACE
Change-Id: I6c40581b9dc5ab1fe3af4b882ede17a13edacc66
Closes-Bug: #1525002
(cherry picked from commit 8224859c02
)
This commit is contained in:
parent
6d782fd962
commit
ae2c0995ad
|
@ -186,7 +186,7 @@ def with_retries(func):
|
|||
LOG.error(msg)
|
||||
raise
|
||||
else:
|
||||
LOG.warn(msg)
|
||||
LOG.debug(msg)
|
||||
time.sleep(self.conflict_retry_interval)
|
||||
|
||||
return wrapper
|
||||
|
@ -345,7 +345,6 @@ class HTTPClient(VersionNegotiationMixin):
|
|||
self.log_http_response(resp)
|
||||
|
||||
if 400 <= resp.status < 600:
|
||||
LOG.warn("Request returned failure status.")
|
||||
error_json = _extract_error_json(body_str)
|
||||
raise exc.from_response(
|
||||
resp, error_json.get('faultstring'),
|
||||
|
|
Loading…
Reference in New Issue