App metadata behavior is mispelled in metadata.yaml for snmp app

Bug #2063224 reported by Enzo Candotti
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
StarlingX
Fix Released
Low
Enzo Candotti

Bug Description

Brief Description
During review of app metadata.yaml it was observed that two apps have 'behaviour' in metadata.yaml instead of 'behavior'. This is probably causing the app to omit 'platform_managed_app' option.

    ./snmp-armada-app/stx-snmp-helm/stx-snmp-helm/files/metadata.yaml

Severity
Minor

Steps to Reproduce
list="snmp-armada-app/stx-snmp-helm/stx-snmp-helm/files/metadata.yaml
ptp-notification-armada-app/stx-ptp-notification-helm/stx-ptp-notification-helm/files/metadata.yaml"

grep "behaviour" $list

Expected Behavior
The spelling should be the expected spelling as presented in config repo files:

    sysinv/sysinv/sysinv/sysinv/common/constants.py
    sysinv/sysinv/sysinv/sysinv/common/app_metadata.py

Actual Behavior
Mispelled.

Reproducibility
100%

System Configuration
Any, n/a

Load info (eg: 2022-03-10_20-00-07)
Master branch, latest

Last Pass
N/A, day one

Timestamp/Logs
N/A

Alarms
N/A

Test Activity
Reviewing platform app metadata

Workaround
N/A

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to snmp-armada-app (master)
Changed in starlingx:
status: New → In Progress
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to snmp-armada-app (master)

Reviewed: https://review.opendev.org/c/starlingx/snmp-armada-app/+/916807
Committed: https://opendev.org/starlingx/snmp-armada-app/commit/fee10f59d177ab215a239bbad335bdebb771f027
Submitter: "Zuul (22348)"
Branch: master

commit fee10f59d177ab215a239bbad335bdebb771f027
Author: Enzo Candotti <email address hidden>
Date: Tue Apr 23 13:07:30 2024 -0300

    Fix app metadata mispelling

    The snmp app have 'behaviour' in metadata.yaml instead of 'behavior'.
    This is causing the app to omit 'platform_managed_app' option.

    This commit fixes this issue.

    Test Plan:
    PASS: Build and deploy updated snmp
    PASS: Patch system to update snmp application, verify that
    app is automatically upgraded.

    Closes-bug: 2063224

    Signed-off-by: Enzo Candotti <email address hidden>
    Change-Id: I1f44243316eb884a2dabfad13c2f1341b9154be8

Changed in starlingx:
status: In Progress → Fix Released
Ghada Khalil (gkhalil)
Changed in starlingx:
importance: Undecided → Low
tags: added: stx.10.0 stx.apps
Changed in starlingx:
assignee: nobody → Enzo Candotti (ecandotti)
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.