Murano does not cache name-to-id mappings of application packages
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
Mirantis OpenStack |
Won't Fix
|
Medium
|
Stan Lagun | ||
7.0.x |
Won't Fix
|
Medium
|
Stan Lagun |
Bug Description
When murano deploys application packages and needs their resources (scripts, files etc), it needs to load them from disk. The contents of that package is usually already there, extracted to the temporary directory during the MuranoPL class load process.
However murano does not maintain a mapping between the package fqn's and the names of the temporary directories (the latter are usually the IDs of the packages in repository), so it has to query Murano-Api to get the ID by fqn each time the resource is requested.
This degrades the performance, but more importantly, it dramatically increases the chance of deployment failure due to access token expiration, as the murano-api needs a valid token to authenticate the request.
Murano needs to maintain a map of loaded packages in its package loader, so subsequent get-id-by-fqn calls do not go to murano-api.
A similar approach (with significantly different implementation though) is already merged in upstream. It should be added to 6.1 as well, especially because it is more vulnerable to access-token expiration issues due to a lack of trust support.
tags: | added: murano |
Changed in mos: | |
status: | New → Confirmed |
Changed in mos: | |
assignee: | nobody → Serg Melikyan (smelikyan) |
assignee: | Serg Melikyan (smelikyan) → Stan Lagun (slagun) |
Changed in mos: | |
milestone: | 6.1-updates → 8.0 |
status: | Confirmed → New |
Changed in mos: | |
importance: | Critical → High |
status: | New → Confirmed |
Changed in mos: | |
milestone: | 8.0 → none |
assignee: | Stan Lagun (slagun) → nobody |
importance: | High → Critical |
importance: | Critical → Undecided |
status: | Confirmed → New |
Changed in mos: | |
milestone: | none → 8.0 |
assignee: | nobody → MOS Murano (mos-murano) |
importance: | Undecided → High |
status: | New → Confirmed |
Changed in mos: | |
status: | Confirmed → Opinion |
Changed in mos: | |
assignee: | nobody → MOS Murano (mos-murano) |
milestone: | none → 8.0 |
importance: | Undecided → High |
status: | New → Confirmed |
This bug is not seen on 8.0