Bug 54728 - UVMM: stop auto polling on logout
UVMM: stop auto polling on logout
Status: CLOSED WONTFIX
Product: UCS
Classification: Unclassified
Component: UMC - Virtual machines (UVMM)
UCS 4.4
Other Linux
: P5 normal (vote)
: ---
Assigned To: UMC maintainers
UMC maintainers
:
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2022-05-06 16:11 CEST by Florian Best
Modified: 2023-06-28 10:51 CEST (History)
0 users

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 Florian Best univentionstaff 2022-05-06 16:11:38 CEST
The polling in UVMM does not stop if one is logged out.

UVMM should watch for login and logout events and stop/start its polling based on this.
This causes a memory leak in UMC-Server (Bug #52361):

(In reply to Florian Best from comment #4)
> (In reply to Florian Best from comment #3)
> > The logs are full of messages like:
> > 06.05.22 10:27:12.116  MAIN        ( WARN    ) : Module None (command=u'uvmm/query', id=u'165182563204742-873172') does not exists anymore
> > 06.05.22 10:27:12.130  MAIN        ( WARN    ) : Module None (command=u'uvmm/node/query', id=u'165182563204952-873173') does not exists anymore
> > 06.05.22 10:27:12.139  MAIN        ( WARN    ) : Module None (command=u'uvmm/cloud/query', id=u'165182563205075-873174') does not exists anymore
> My theory about this:
> The UVMM module doesn't stop the polling if one is logged out (e.g. due to
> SAML session end time (=5 minutes), UMC-Webserver restarts, etc.)
> This causes that the request is made but the reloaded ACL's don't know this
> module anymore and therefore this message is raised.
> The code then does:
>  493 »   »   »   CORE.warn('Module %r (command=%r, id=%r) does not exists
> anymore' % (module_name, command, msg.id))
>  494 »   »   »   moduleManager.load()
>  495 »   »   »   self._reload_acls_and_permitted_commands()
> 
> which was causing the memory leak as explained in Bug #52361 comment 0.
Comment 1 Philipp Hahn univentionstaff 2023-06-28 10:51:01 CEST
UVMM and virtualization with UCS is deprecated and will no longer be developed in UCS 4.4; they have already been removed from UCS 5.0.