Load-import '--inactive' is not rejected when importing a non-upgradable load

Bug #2043510 reported by Lindley Werner Soares Vieira
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
StarlingX
Fix Released
Low
Lindley Werner Soares Vieira

Bug Description

Brief Description
------------------
Load-import '--inactive' is not rejected when importing a non-upgradable load

We should import only upgradable loads using --inactive parameter

Severity
---------
<Minor: System/Feature is usable with minor issue>

Steps to Reproduce
-------------------
Import unsupported previous release
system --os-region-name SystemController load-import --inactive <.iso> <.sig>

Expected Behavior
-----------------
Load-import should be rejected.

Actual Behavior
---------------
We can import the load

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

System Configuration
--------------------
DC
lab: DC21

Load info (eg: 2022-03-10_20-00-07)
---------
main branch w/ an attempt to import incompatible previous release

Last Pass
----------
new test scenario

Timestamp/Logs
---------------
N/A - issue is reproducible

Alarms
------
n/a

Test Activity
--------------
Feature Testing

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

Changed in starlingx:
assignee: nobody → Lindley Werner Soares Vieira (lindley-vieira)
Changed in starlingx:
status: New → In Progress
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/+/901042

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to config (master)

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

commit e781961d3325c7af2e490ae87f0d9f9dd9ba3d6a
Author: Bin Qian <email address hidden>
Date: Wed Nov 15 14:53:46 2023 +0000

    Fix load-import required patch check

    The required_patch xml tag was mistakenly changed to required_patches in
    previous commit [1]. This change is to make sure the correct tag is used.

    [1] https://review.opendev.org/c/starlingx/config/+/888329

    Test plan:
      PASS: load-import with and without --inactive, both with and without
            required patches applied on from-release.

    Closes-bug: 2043510

    Change-Id: I69504396ffda4565ebcce9b759c88f98d2631564
    Signed-off-by: Bin Qian <email address hidden>

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

Change abandoned by "Lindley Werner Soares Vieira <email address hidden>" on branch: master
Review: https://review.opendev.org/c/starlingx/config/+/900830

Ghada Khalil (gkhalil)
Changed in starlingx:
importance: Undecided → Low
Ghada Khalil (gkhalil)
description: updated
tags: added: stx.9.0 stx.config stx.distcloud
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.