'tc' utility is missing from StarlingX since recent iproute updates

Bug #1951989 reported by M. Vefa Bicakci
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
StarlingX
Fix Released
High
M. Vefa Bicakci

Bug Description

Since iproute has been updated with the following commits, tc has been missing from StarlingX ISO installations.

https://review.opendev.org/c/starlingx/integ/+/814682

https://review.opendev.org/c/starlingx/integ/+/815907

This bug is reported to re-add the tc utility to the StarlingX ISO.

Changed in starlingx:
assignee: nobody → M. Vefa Bicakci (vbicakci)
status: New → Confirmed
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to integ (master)

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

Changed in starlingx:
status: Confirmed → In Progress
Revision history for this message
Ghada Khalil (gkhalil) wrote :

screening: stx.6.0 / high - tc utility is used for traffic shaping. issue introduced recently and should be fixed.

tags: added: stx.6.0 stx.distro.other stx.integ
Changed in starlingx:
importance: Undecided → High
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to integ (master)

Reviewed: https://review.opendev.org/c/starlingx/integ/+/819021
Committed: https://opendev.org/starlingx/integ/commit/2421d5a3aa316b447a91eb14f55ed26739f8c2c8
Submitter: "Zuul (22348)"
Branch: master

commit 2421d5a3aa316b447a91eb14f55ed26739f8c2c8
Author: M. Vefa Bicakci <email address hidden>
Date: Tue Nov 23 11:02:49 2021 -0500

    Add iproute-tc to the ISO image

    iproute was updated from 4.11 to 5.12 with commit a7760b40a266
    ("iproute: Update from 4.11.0-14 to 5.9.0-4") and commit 36673774ee3c
    ("iproute-5.12, iptables-1.8.4, and libnftnl-1.1.5"). In CentOS 7, the
    iproute-4.11 RPM package included the 'tc' utility, but with CentOS 8,
    tc was moved to a separate RPM package, "iproute-tc".

    The original version of the second aforementioned commit added the
    iproute-tc RPM package in the ISO, but iproute-tc was later removed from
    the ISO during the code review, as it was not known at the time that tc
    had been split into a separate package (iproute-tc) between CentOS 7's
    iproute 4.11 and CentOS 8's 5.9/5.12, and that tc used to be included in
    the StarlingX ISO before the iproute updates.

    This commit explicitly adds the iproute-tc package into the ISO image to
    make tc available in StarlingX installations again.

    To prevent another similar issue, the iproute spec file was scanned for
    %package declarations to see if any other new sub-packages had been
    introduced since CentOS 7, and only the iproute-tc package was seen.

    Verification:
    - ISO build was successful.
    - Resulting bootimage.iso file was confirmed to contain the iproute-tc
      RPM package via "iso-info -f -i bootimage.iso | grep iproute-tc".
    - Note that run-time testing of the tc utility had been carried out in
      the past for eBPF testing by manually installing the iproute-tc RPM
      package to a running StarlingX installation.

    Closes-Bug: #1951989

    Signed-off-by: M. Vefa Bicakci <email address hidden>
    Change-Id: I8422f0aa872101e85502f1f3f0d3dc62ec19b8bb

Changed in starlingx:
status: In Progress → Fix Released
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.