Bug 40563 - SSH connection check should exclude docker instances
SSH connection check should exclude docker instances
Status: CLOSED FIXED
Product: UCS
Classification: Unclassified
Component: UMC - System diagnostic
UCS 4.1
Other Linux
: P5 normal (vote)
: UCS 4.1-1-errata
Assigned To: Florian Best
Dirk Wiesenthal
:
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2016-02-02 21:50 CET by Stefan Gohmann
Modified: 2016-04-13 14:53 CEST (History)
1 user (show)

See Also:
What kind of report is it?: ---
What type of bug is this?: ---
Who will be affected by this bug?: ---
How will those affected feel about the bug?: ---
User Pain:
Enterprise Customer affected?:
School Customer affected?:
ISV affected?:
Waiting Support:
Flags outvoted (downgraded) after PO Review:
Ticket number:
Bug group (optional):
Max CVSS v3 score:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Stefan Gohmann univentionstaff 2016-02-02 21:50:13 CET
SSH connection check should exclude docker instances because the docker instances don't start a ssh daemon by default:

--------------------------------------------------------------------------------
The ssh connection to at least one other UCS server failed. The following list shows the affected remote servers and the reason for the failed ssh connection:

tecart-1454054491105368.deadlock0.intranet - [Errno -2] Name or service not known
tecart-1454054491105368 - [Errno -2] Name or service not known
--------------------------------------------------------------------------------
Comment 1 Dirk Wiesenthal univentionstaff 2016-03-04 09:33:30 CET
Docker hosts have:

udm computers/memberserver list --filter name=tecart-1454054491105368 | grep objectFlag
  objectFlag: docker

There is also
  ucr get appcenter/apps/tecart/hostdn
Comment 2 Florian Best univentionstaff 2016-03-21 16:06:48 CET
univention-appcenter (5.0.20-31):
r68218 | Bug #40563: exclude docker instances in ssh connections

univention-appcenter.yaml:
r68222 | YAML Bug #38530, Bug #39896, Bug #40563, Bug #39907
Comment 3 Dirk Wiesenthal univentionstaff 2016-04-11 23:18:09 CEST
Please revert the patch, it does not really have any impact. While it is true that the functions find docker hosts, they won't in reality: Only DC Master and DC Backup servers are searched for and those will never be docker hosts.

The only function that would benefit from it is get_all_hosts and this function is not used anymore. It was for collecting data for the frontend, but this is now done by the new code that already filters docker hosts.

Anyway, this bug has nothing to do with the App Center. You need to patch

management/univention-management-console-module-diagnostic/umc/python/diagnostic/plugins/ssh_connection.py
Comment 4 Florian Best univentionstaff 2016-04-12 08:21:50 CEST
Oups, I didn't see the component...

r68544 | Revert "Bug #40563: exclude docker instances in ssh connections"
Comment 5 Florian Best univentionstaff 2016-04-12 08:40:21 CEST
univention-management-console-module-diagnostic.yaml:
r68546 | YAML Bug #40563

univention-management-console-module-diagnostic (2.0.0-5):
r68545 | Bug #40563: exclude docker hosts in SSH connection check
Comment 6 Dirk Wiesenthal univentionstaff 2016-04-12 12:53:29 CEST
Changeset: OK
YAML: OK
Comment 7 Janek Walkenhorst univentionstaff 2016-04-13 14:53:41 CEST
<http://errata.software-univention.de/ucs/4.1/151.html>