Hard-coded "r1" in build scripts and instructions

Bug #1883760 reported by Bruce Jones
8
This bug affects 1 person
Affects Status Importance Assigned to Milestone
StarlingX
Fix Released
Low
M Camp

Bug Description

The build and build documentation include an "r1" in the pathnames of the directories. It's been two years since r1, and I don't think we need it. If we want to include a release distinguishing directory, it should be parameterized across releases. Or just dropped.

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

Marking as Low priority - assuming there is no negative impact. Would be nice to fix, but should not hold up the release.

tags: added: stx.build stx.docs
Changed in starlingx:
importance: Undecided → Low
status: New → Triaged
Changed in starlingx:
assignee: nobody → Susendra Selvaraj (sselvar2)
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to tools (master)

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

Changed in starlingx:
status: Triaged → In Progress
Revision history for this message
Susendra Selvaraj (sselvar2) wrote :

Just dropped Hardcoded 'r1', layered build is working fine.
Patch uploaded for build scripts: https://review.opendev.org/#/c/745651/

Same changes shall be propagated to .rst files for doc changes.

Revision history for this message
Susendra Selvaraj (sselvar2) wrote :

There is patch from Saul for the same: https://review.opendev.org/#/c/687400/
This removes both the r1 and the pike portion of the download path name.
This takes some coordination with the build team possibly due to changes required in Jenkins.
This will be discussed in Build Team Meeting on Tuesday.

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Change abandoned on tools (master)

Change abandoned by Susendra Selvaraj (<email address hidden>) on branch: master
Review: https://review.opendev.org/745651
Reason: abandoned in favor of Saul's similar review.

Revision history for this message
Susendra Selvaraj (sselvar2) wrote :

        - Scott expects this to require 1-2 days effort
        - TBD on when this work can be done.

Revision history for this message
M Camp (mcamp859) wrote :

Changing LP status to Fix Released since the associated review has merged:
https://review.opendev.org/762761

Changed in starlingx:
status: In Progress → Fix Released
assignee: Susendra Selvaraj (sselvar2) → M Camp (mcamp859)
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.