Skip to content

Comments

pmdabcc: removed completely now after retirement last release#2499

Closed
natoscott wants to merge 2 commits intoperformancecopilot:mainfrom
natoscott:drop-bcc-pmda
Closed

pmdabcc: removed completely now after retirement last release#2499
natoscott wants to merge 2 commits intoperformancecopilot:mainfrom
natoscott:drop-bcc-pmda

Conversation

@natoscott
Copy link
Member

No description provided.

Copy link
Contributor

@myllynen myllynen left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, thanks a lot for this!

Copy link
Member

@kmcdonell kmcdonell left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Seems pretty complete to me (with a few exceptions noted below).
Ignorance prevents me from commenting on rpm packaging or equivalence of metrics for atop.
Debian packaging is simple and OK (lucky we don't have per-PMDA packages there!).
Missing?

  • Comment block at head of src/pmdas/bpf/Upgrade probably needs some post cut-n-paste care to speak about bcc -> bpf, rather than notready and OpenMetrics.
  • Also I was surprised to see no changes below qa/admin/package-lists and for qa/admin/other-packages/manifest (not just python bcc or bpfcc but all the other things dragged in for QA of the bcc PMDA)
  • Shouldn't ‎src/pmdas/bcc/modules/fs/__init__.python be deleted? shows as "Whitespace-only changes" ... ditto for src/pmdas/bcc/modules/__init__.python
  • for retired qa tests, I presume the plan is to use qa/really-retire on 'em in a subsequent round of changes, or you could do it now? (so these ones will be like qa/1152 that is already really-retired)

Drop common.bcc, qa/really-retire all of the BCC tests and
updates to manifests and package lists to drop what we can.
@natoscott
Copy link
Member Author

* Comment block at head of src/pmdas/bpf/Upgrade probably needs some post cut-n-paste care to speak about bcc -> bpf, rather than notready and OpenMetrics.

Fixed, thanks.

* Also I was surprised to see no changes below qa/admin/package-lists and for qa/admin/other-packages/manifest (not just python bcc or bpfcc but all the other things dragged in for QA of the bcc PMDA)

Should all be resolved now via second commit on that branch.

* Shouldn't ‎src/pmdas/bcc/modules/fs/__init__.python be deleted? shows as "Whitespace-only changes" ... ditto for src/pmdas/bcc/modules/__init__.python

It is/was git-rm deleted, not sure why its showing up like that there.

* for retired qa tests, I presume the plan is to use qa/really-retire on 'em in a subsequent round of changes, or you could do it now? (so these ones will be like qa/1152 that is already really-retired)

I've done that now, and also dropped common.bcc which I missed before - all in the second commit if you could cast an eye over it once more.

@kmcdonell kmcdonell self-requested a review February 20, 2026 01:32
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Strictly speaking, you don't need to make changes here ... this is a record of fact wrt the package availability for the distros. Mentioning python3-bcc or bpfcc-tools here will have no effect if they have been culled from the manfest.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Could probably add a note here about PMDA being retired and today's date.

Copy link
Member

@kmcdonell kmcdonell left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Couple of small nits in per-file comments as I passed through.
But if this compiles, then ship it!

@natoscott
Copy link
Member Author

Thanks Ken, tweaked and merged.

@natoscott natoscott closed this Feb 20, 2026
@natoscott natoscott deleted the drop-bcc-pmda branch February 20, 2026 03:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants