starlingx/fault
Code Issues Proposed changes

debian: Fix python3 syntax warning

Browse Source 
From https://docs.python.org/3/whatsnew/3.8.html#changes-in-python-behavior:

The compiler now produces a SyntaxWarning when identity
checks (is and is not) are used with certain types of
literals (e.g. strings, numbers). These can often work
by accident in CPython, but are not guaranteed by the
language spec. The warning advises users to use
equality tests (== and !=) instead.

Test Plan:
PASS Apply patch test the output of fm alarm-list on Debian 11.
PASS Apply patch test the output of fm alarm-list on Centos 7.

Closes-Bug: 1960958

Signed-off-by: Chuck Short <charles.short@windriver.com>
Change-Id: I12c43f336a449876fab8592e34eace674518d0da
This commit is contained in:
Chuck Short 2022-03-24 14:11:42 +00:00 committed by Al Bailey
parent cb34d876a8
commit 30e9761ede
1 changed files with 1 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
python-fmclient/fmclient/fmclient/common/http.py
View File

@ -153,7 +153,7 @@ class HTTPClient(_BaseHTTPClient):
if kwargs.get('insecure', False) is True: if kwargs.get('insecure', False) is True:
self.session.verify = False self.session.verify = False
else: else:
if kwargs.get('cacert', None) is not '': # noqa: F632 if kwargs.get('cacert', None) != '':
self.session.verify = kwargs.get('cacert', True) self.session.verify = kwargs.get('cacert', True)
self.session.cert = (kwargs.get('cert_file'), self.session.cert = (kwargs.get('cert_file'),