application upload fail without lifecycle hook parameter

Bug #2091973 reported by Wey-Yi W Guy
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
StarlingX
Fix Released
High
Wey-Yi W Guy

Bug Description

Brief Description
-----------------
remove lifecycle hook in perform_app_upload cause upload not function correctly

Severity
--------
Critical

Steps to Reproduce
------------------
perform "system application-upload" with specific tgz file

Expected Behavior
------------------
upload should completed with error

Actual Behavior
----------------
Report "Missing 1 parameter - lifecycle_hook_info

Reproducibility
---------------
100%

System Configuration
--------------------
N/A

Branch/Pull Time/Commit
-----------------------
N/A

Last Pass
---------
N/A

Timestamp/Logs
--------------
N/A

Test Activity
-------------
Sanity

Workaround
----------
N/A

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

Reviewed: https://review.opendev.org/c/starlingx/config/+/937909
Committed: https://opendev.org/starlingx/config/commit/594dcb57d6f23558871dfd388fdb21ea0356ff54
Submitter: "Zuul (22348)"
Branch: master

commit 594dcb57d6f23558871dfd388fdb21ea0356ff54
Author: Wey-Yi W Guy <email address hidden>
Date: Tue Dec 17 20:17:59 2024 +0000

    Revert "Remove unused lifecycle parameters from perform_app_upload"

    This reverts commit 7018cd73277f19a57d1930d78cbf154fd25a5edf.

    Reason for revert: cause some issue on upload application

    Closes-bug: #2091973

    Change-Id: Id001121bb068cc8f7ef5b587f29ca3bf32ebf2e4

Changed in starlingx:
status: In Progress → Fix Released
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to config (master)

Fix proposed to branch: master
Review: https://review.opendev.org/c/starlingx/config/+/937997

Ghada Khalil (gkhalil)
Changed in starlingx:
assignee: nobody → Wey-Yi W Guy (wwguy)
importance: Undecided → High
tags: added: stx.10.0 stx.apps
Revision history for this message
Ghada Khalil (gkhalil) wrote (last edit ):

This sanity issue was introduced by https://review.opendev.org/c/starlingx/config/+/937395 and was addressed by reverting this change: https://review.opendev.org/c/starlingx/config/+/937909

Note: It looks like the re-work of the original fix is also linked to this LP as well (when it really shouldn't). A note has been added to this review: https://review.opendev.org/c/starlingx/config/+/937997

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.