[MIR] libtracefs

Bug #2008799 reported by Christian Ehrhardt 
8
This bug affects 1 person
Affects Status Importance Assigned to Milestone
libtracefs (Ubuntu)
Won't Fix
Undecided
Unassigned
ndctl (Ubuntu)
Fix Released
Undecided
Lukas Märdian

Bug Description

[Availability]

The package libtracefs is already in Ubuntu universe.
The package libtracefs build for the architectures it is designed to work on.
It currently builds and works for architectures: amd64, arm64, armhf, ppc64el, riscv64, s390x.
Link to package [[https://launchpad.net/ubuntu/+source/libtracefs|libtracefs]]

[Rationale]

libtracefs is a new dependency of ndctl which was introduced by upstream in version 76:

https://github.com/pmem/ndctl/commit/68c72f36e

Since ndctl is in main and libtracefs seems to enable useful features, let's promote it to main and fix the component mismathc.

It would be great and useful to community/processes to have the package libtracefs in Ubuntu main, but there is no hard deadline, hopefully before Lunar release.

[Security]

Nothing was found in the CVE database:

https://cve.mitre.org/cgi-bin/cvekey.cgi?keyword=libtracefs

Also nothing was found in the OSS security mailing list archive.

No CVE in the Ubuntu security tracker:

https://ubuntu.com/security/cves?package=libtracefs

Nor in the Debian security tracker:

https://security-tracker.debian.org/tracker/source-package/libtracefs

- no executables in `/sbin` and `/usr/sbin`
- Package does not install services, timers or recurring jobs
- Packages does not open privileged ports (ports < 1024)
- Packages does not contain extensions to security-sensitive software (filters, scanners, plugins, UI skins, ...)

[Quality assurance - function/usage]

- The package works well right after install

[Quality assurance - maintenance]

- The package is maintained well in Debian/Ubuntu/Upstream and does not have too many, long-term & critical, open bugs
- Ubuntu https://bugs.launchpad.net/ubuntu/+source/libtracefs/+bug
- Debian https://bugs.debian.org/cgi-bin/pkgreport.cgi?src=libtracefs
- The package does not deal with exotic hardware we cannot support

[Quality assurance - testing]

- The package runs a test suite on build time, if it fails it makes the build fail, link to build log:

https://launchpadlibrarian.net/646632616/buildlog_ubuntu-lunar-amd64.libtracefs_1.6.4-1_BUILDING.txt.gz

The package does not implement any DEP-8 test right now. I am adding some here:

https://code.launchpad.net/~lucaskanashiro/ubuntu/+source/libtracefs/+git/libtracefs/+merge/438414

Once it is approved it will be uploaded to Lunar and forwarded to Debian.

Since there is no autopkgtest run against the version in the archive, there is no failure.

[Quality assurance - packaging]

- debian/watch is present and works

- debian/control defines a correct Maintainer field

- This package does not yield massive lintian Warnings, Errors

W: libtracefs source: newer-standards-version 4.6.2 (current is 4.5.0)
P: libtracefs source: package-uses-experimental-debhelper-compat-version 13
P: libtracefs source: rules-requires-root-missing

- Lintian overrides are not present

- This package does not rely on obsolete or about to be demoted packages.

- The package will not be installed by default

- Packaging and build is easy, link to d/rules:

https://git.launchpad.net/ubuntu/+source/libtracefs/tree/debian/rules

[UI standards]

- Application is not end-user facing (does not need translation)

[Dependencies]

- There is one dependency that is not yet in main, MIR for libtraceevent at:

https://bugs.launchpad.net/ubuntu/+source/libtraceevent/+bug/2009715

[Standards compliance]

- This package correctly follows FHS and Debian Policy

[Maintenance/Owner]

- Owning Team will be Server

- Team is not yet, but will subscribe to the package before promotion

- This does not use static builds

- This does not use vendored code

- This package is not rust based

- The package has been built in the archive more recently than the last test rebuild

[Background information]

The Package description explains the package well
Upstream Name is libtracefs
Link to upstream project: https://git.kernel.org/pub/scm/libs/libtrace/libtracefs.git

[Original description]

Found in component mismatches

 ndctl | 75-1 | lunar | source, amd64, arm64, armhf, i386, ppc64el, riscv64, s390x
 ndctl | 76-1 | lunar-proposed | source, amd64, arm64, armhf, ppc64el, riscv64, s390x

The new version depends on libtraceevent + libtracefs

tags: added: server-todo
Changed in libtracefs (Ubuntu):
status: Incomplete → New
assignee: nobody → Lucas Kanashiro (lucaskanashiro)
description: updated
Changed in libtracefs (Ubuntu):
status: New → In Progress
description: updated
description: updated
Revision history for this message
Lucas Kanashiro (lucaskanashiro) wrote :

The Server team is subscribed to the package.

Changed in libtracefs (Ubuntu):
assignee: Lucas Kanashiro (lucaskanashiro) → nobody
status: In Progress → New
assignee: nobody → Lukas Märdian (slyon)
Revision history for this message
Didier Roche-Tolomelli (didrocks) wrote :

I am picking that MIR to review. It seems both were assigned to slyon by error.

Changed in libtracefs (Ubuntu):
assignee: Lukas Märdian (slyon) → Didier Roche-Tolomelli (didrocks)
Revision history for this message
Lukas Märdian (slyon) wrote (last edit ):

I wonder if the rational is strong enough to support promoting libtracefs & libtraceevent to "main"? Especially now that upstream has made those dependencies optional. Should we rather disable it for the time being?

What's the exact usecase we want to support for "cxl monitor" (outside of cxl-monitor.servce, which is currently disabled in Debian anyway)?

Revision history for this message
Lukas Märdian (slyon) wrote :

13:25 <kanashiro[m]> slyon: TBH I had not seen this upstream patch when I was working on it... I believe you are right, let's disable it and avoid the promotion. Would you upload your debdiff?

=> WONTFIX

Changed in libtracefs (Ubuntu):
status: New → Won't Fix
assignee: Didier Roche-Tolomelli (didrocks) → nobody
Changed in ndctl (Ubuntu):
status: New → In Progress
Lukas Märdian (slyon)
Changed in ndctl (Ubuntu):
assignee: nobody → Lukas Märdian (slyon)
Revision history for this message
Launchpad Janitor (janitor) wrote :

This bug was fixed in the package ndctl - 76-1ubuntu1

---------------
ndctl (76-1ubuntu1) lunar; urgency=medium

  * Avoid optional dependency MIRs: libtracefs & libtraceevent.
    Upstream cherry-pick https://github.com/pmem/ndctl/commit/82884ee
  * d/rules: disabled (new) libtracefs meson option
  * d/control: drop libtracesfs & libtraceevent (LP: #2009715, LP: #2008799)

 -- Lukas Märdian <email address hidden> Tue, 21 Mar 2023 14:08:28 +0100

Changed in ndctl (Ubuntu):
status: In Progress → Fix Released
Revision history for this message
Lukas Märdian (slyon) wrote :

unsubscribing ~ubuntu-mir in favour of newer MIR for libtracefs (LP: #2051925)

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.