cgts-client and python-inventoryclient rebuild every time

Bug #1830284 reported by Don Penney
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
StarlingX
Fix Released
Low
Don Penney

Bug Description

Brief Description
-----------------
cgts-client and python-inventoryclient rebuild every time you run build-pkgs due to incomplete SDK removal specfile changes. Additionally, this causes inventory and python-django-horizon to also rebuild, due to dependencies.

Severity
--------
Minor

Steps to Reproduce
------------------
build-pkgs --no-build-info

Expected Behavior
------------------
If "build-pkgs" is run a second time (with --no-build-info), there should be no pkgs rebuilt since there are no changes.

Actual Behavior
----------------
These 4 packages rebuild every time.

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

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

Branch/Pull Time/Commit
-----------------------
master branch, May 23, 2019

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

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

Test Activity
-------------
Build

Don Penney (dpenney)
Changed in starlingx:
assignee: nobody → Don Penney (dpenney)
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to metal (master)

Fix proposed to branch: master
Review: https://review.opendev.org/661119

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/661120

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

Reviewed: https://review.opendev.org/661119
Committed: https://git.openstack.org/cgit/starlingx/metal/commit/?id=061b0f2c3cc34f083ac2e25dc9faba5fbb278a9f
Submitter: Zuul
Branch: master

commit 061b0f2c3cc34f083ac2e25dc9faba5fbb278a9f
Author: Don Penney <email address hidden>
Date: Thu May 23 17:03:11 2019 -0400

    Remove sdk package from python-inventoryclient.spec

    A previous update removed the "%files sdk" section from
    the python-inventoryclient.spec file, but missed removal
    of the "%package sdk" section. As a result, the build
    system looks for the python-inventoryclient-sdk package
    when checking if a rebuild is required. But because the
    "%files" was removed, the package is not built. This leads
    the build system to decide it needs to rebuild.

    Change-Id: I753212a17eca59699c2122c55eac313d1f9ed94f
    Partial-Bug: 1830284
    Signed-off-by: Don Penney <email address hidden>

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

Reviewed: https://review.opendev.org/661120
Committed: https://git.openstack.org/cgit/starlingx/config/commit/?id=a156a13e9dfc5ad4f1ebe6f9d7198d3f4a153e37
Submitter: Zuul
Branch: master

commit a156a13e9dfc5ad4f1ebe6f9d7198d3f4a153e37
Author: Don Penney <email address hidden>
Date: Thu May 23 17:06:43 2019 -0400

    Remove sdk package from cgts-client.spec

    A previous update removed the "%files sdk" section from
    the cgts-client.spec file, but missed removal
    of the "%package sdk" section. As a result, the build
    system looks for the cgts-client-sdk package
    when checking if a rebuild is required. But because the
    "%files" was removed, the package is not built. This leads
    the build system to decide it needs to rebuild.

    Change-Id: I2c293e73ad25b72c8218ba9d090ebf1c47fab44e
    Closes-Bug: 1830284
    Signed-off-by: Don Penney <email address hidden>

Revision history for this message
Ghada Khalil (gkhalil) wrote :

Adding the stx.2.0 release tag since the fix for this bug merged for that release

Changed in starlingx:
importance: Undecided → Low
tags: added: stx.2.0 stx.config
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.