[SRU] Deprecated usage of cpu_util

Bug #2088620 reported by Bryan Fraschetti
22
This bug affects 3 people
Affects Status Importance Assigned to Milestone
Ubuntu Cloud Archive
Status tracked in Epoxy
Antelope
New
Undecided
Unassigned
Bobcat
New
Undecided
Unassigned
Caracal
Fix Released
Undecided
Unassigned
Dalmation
Fix Released
Undecided
Unassigned
Epoxy
Fix Released
Undecided
Unassigned
Yoga
New
Undecided
Unassigned
Zed
New
Undecided
Unassigned
watcher (Ubuntu)
Confirmed
Undecided
Bryan Fraschetti
Focal
Confirmed
Undecided
Unassigned
Jammy
Confirmed
Undecided
Unassigned
Noble
Fix Released
Undecided
Unassigned
Oracular
Fix Released
Undecided
Unassigned

Bug Description

Watcher uses cpu_util wich was deprecated.
The `cpu_util` metric was deprecated and replaced with raw CPU time measurements stored in Gnocchi.

There is an upstream fix: https://review.opendev.org/c/openstack/watcher/+/898791

Tags: patch sts
affects: cloud-archive → watcher (Ubuntu)
Changed in watcher (Ubuntu):
assignee: nobody → Bryan Fraschetti (bryanfraschetti)
summary: - Deprecated usage of cpu_util
+ [SRU] Deprecated usage of cpu_util
tags: added: sts
Revision history for this message
Bryan Fraschetti (bryanfraschetti) wrote :

Unsigned debdiff for bobcat that resolves the issue. It contains two patches
The first patch corresponds to the commit here: https://opendev.org/openstack/watcher/commit/c3640ed64e6e11438c62e412c17f8b0e174c4db4, which updates the url from which tox pulls its constraints. This is necessary to ensure the passing of the tests which are required by debuild.
The second directly resolves the issue and is cherry-picked from https://opendev.org/openstack/watcher/commit/40e93407c77c227b5c9159462d75b7015116aa9f

Revision history for this message
Bryan Fraschetti (bryanfraschetti) wrote :

Unsigned debdiff for antelope that resolves the issue. It contains one patch which can be seen upstream: https://opendev.org/openstack/watcher/commit/40e93407c77c227b5c9159462d75b7015116aa9f

Revision history for this message
Bryan Fraschetti (bryanfraschetti) wrote :

Unsigned debdiff for zed that resolves the issue. It contains one patch which can be seen upstream: https://opendev.org/openstack/watcher/commit/40e93407c77c227b5c9159462d75b7015116aa9f

Revision history for this message
Bryan Fraschetti (bryanfraschetti) wrote :

Unsigned debdiff for yoga that resolves the issue. It contains one patch which can be seen upstream: https://opendev.org/openstack/watcher/commit/40e93407c77c227b5c9159462d75b7015116aa9f

Revision history for this message
Ubuntu Foundations Team Bug Bot (crichton) wrote :

The attachment "lp2088620_bobcat.debdiff" seems to be a debdiff. The ubuntu-sponsors team has been subscribed to the bug report so that they can review and hopefully sponsor the debdiff. If the attachment isn't a patch, please remove the "patch" flag from the attachment, remove the "patch" tag, and if you are member of the ~ubuntu-sponsors, unsubscribe the team.

[This is an automated message performed by a Launchpad user owned by ~brian-murray, for any issue please contact him.]

tags: added: patch
no longer affects: watcher (Ubuntu Noble)
Changed in watcher (Ubuntu Noble):
status: New → Fix Released
Changed in watcher (Ubuntu Oracular):
status: New → Fix Released
Revision history for this message
Launchpad Janitor (janitor) wrote :

Status changed to 'Confirmed' because the bug affects multiple users.

Changed in watcher (Ubuntu Focal):
status: New → Confirmed
Changed in watcher (Ubuntu Jammy):
status: New → Confirmed
Changed in watcher (Ubuntu):
status: New → Confirmed
Revision history for this message
James Page (james-page) wrote :

@bryanfrashetti

Please can you either use the commit from upstream including the patch headers or add you own headers according to https://dep-team.pages.debian.net/deps/dep3/

Thanks!

Revision history for this message
James Page (james-page) wrote :

A few more comments:

- update-tox-ini-constraints.patch is not needed for the distro so please drop.
- please reference this bug in the changelog entry using (LP: #2088620).

Revision history for this message
Bryan Fraschetti (bryanfraschetti) wrote :
Revision history for this message
Bryan Fraschetti (bryanfraschetti) wrote :

Antelope patch with DEP-3 headers added to the patch file, and Launchpad bug referenced in the changelog

Revision history for this message
Bryan Fraschetti (bryanfraschetti) wrote :

Zed debdiff with DEP-3 headers added to the patch file, and Launchpad bug referenced in the changelog

Revision history for this message
Bryan Fraschetti (bryanfraschetti) wrote :

Yoga debdiff with DEP-3 headers added to the patch file, and Launchpad bug referenced in the changelog

Revision history for this message
Bryan Fraschetti (bryanfraschetti) wrote :

Hi @james-page

Thanks for taking a look at the patch. I added headers to the patch file, with the upstream commit message as the description, and referenced the bug in the changelog. I also removed the tox constraints patch from the bobcat diff.

Let me know if any further changes are required

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.