AppFwk: Auto update flag doesn't work for non platform managed apps
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
StarlingX |
Fix Released
|
Medium
|
Dan Voiculeasa |
Bug Description
Brief Description
-----------------
Take ptp for example which lacks platform_
behavior:
platform_
When the new ptp version contains the auto update metadata it will not trigger the auto update after sysinv restart
upgrades:
auto_update: true
Auto apply apps will not be called because it is gated by this lines:
We need to allow apps to be auto updated to newer versions, regardless if they are managed on not. This will enhance the patchback experience, allowing no manul intervention to update apps.
Severity
--------
Critical: Manual input needed on large systems.
Steps to Reproduce
------------------
Write down the steps to reproduce the issue
Expected Behavior
------------------
auto update apps.
Actual Behavior
----------------
Need manual intervention for updating apps.
Reproducibility
---------------
100%
System Configuration
-------
Any
Branch/Pull Time/Commit
-------
3rd March 2023
Last Pass
---------
N/A
Timestamp/Logs
--------------
N/A
Test Activity
-------------
Developer Testing
Workaround
----------
manual update/ write a script for patches to detect and update.
Changed in starlingx: | |
assignee: | nobody → Dan Voiculeasa (dvoicule) |
Changed in starlingx: | |
status: | New → In Progress |
tags: | added: stx.9.0 stx.apps |
Changed in starlingx: | |
importance: | Undecided → Medium |
Reviewed: https:/ /review. opendev. org/c/starlingx /config/ +/876327 /opendev. org/starlingx/ config/ commit/ d8abc9e7ffac78e cbb7a862cb612fd 2ec6671ad5
Committed: https:/
Submitter: "Zuul (22348)"
Branch: master
commit d8abc9e7ffac78e cbb7a862cb612fd 2ec6671ad5
Author: Dan Voiculeasa <email address hidden>
Date: Fri Mar 3 14:45:55 2023 +0200
AppFwk: Enable auto update for non platform managed apps
We have a gap in design where we want apps to be auto updated if
specific metadata is present in the app on the N+1 side. Metadata
represented by [1].
The gap in design is that the N version must also contain specific
metadata to register the app as 'platform managed app'. Metadata
represented by [2].
Tests on AIO-SX:
PASS: apply ptp-notification app in a patch, then restart sysinv
observed the newer version of ptp-notification is auto-applied
[1]: https:/ /opendev. org/starlingx/ app-security- profiles- operator/ src/commit/ 31b3f77eeefdc0b e4c9c1f53c4e728 88313a8d0e/ stx-security- profiles- operator- helm/stx- security- profiles- operator- helm/files/ metadata. yaml#L6- L7 /opendev. org/starlingx/ platform- armada- app/src/ commit/ 4c765c35500da92 d920d3fbf3c79b4 cf02d4712b/ stx-platform- helm/stx- platform- helm/files/ metadata. yaml#L5- L6 012e7dcb6ac7fa4 c513f382bc2
[2]: https:/
Closes-Bug: 2009187
Signed-off-by: Dan Voiculeasa <email address hidden>
Change-Id: Ia6f5646e78b8ed