This isn't really a high priority issue, especially considering that common.SLE is really just a base class for the SLE interfaces. But still ...
if not self._hostnames or not self._port:
msg = 'Connection configuration missing hostnames ({}) or port ({})'
msg = msg.format(self._hostnames, self._port)
ait.core.log.error(msg)
raise ValueError(msg)
SLE.__init__ has checks for hostname and port information but these attributes aren't defined default in SLE. They're instead defaulted in the inherited interface classes (RAF, RCF, etc.). Should clean that up unless there's a compelling reason to leave it in its current state.
kmarwah
commented
4 years ago
This isn't really a high priority issue, especially considering that
common.SLEis really just a base class for the SLE interfaces. But still ...SLE.__init__has checks for hostname and port information but these attributes aren't defined default inSLE. They're instead defaulted in the inherited interface classes (RAF,RCF, etc.). Should clean that up unless there's a compelling reason to leave it in its current state.