pmdabcc: removed completely now after retirement last release#2499
pmdabcc: removed completely now after retirement last release#2499natoscott wants to merge 2 commits intoperformancecopilot:mainfrom
Conversation
myllynen
left a comment
There was a problem hiding this comment.
LGTM, thanks a lot for this!
1b3c2a3 to
f9971ee
Compare
kmcdonell
left a comment
There was a problem hiding this comment.
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.
f9971ee to
ff3dac8
Compare
Fixed, thanks.
Should all be resolved now via second commit on that branch.
It is/was git-rm deleted, not sure why its showing up like that there.
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. |
There was a problem hiding this comment.
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.
There was a problem hiding this comment.
Could probably add a note here about PMDA being retired and today's date.
kmcdonell
left a comment
There was a problem hiding this comment.
Couple of small nits in per-file comments as I passed through.
But if this compiles, then ship it!
|
Thanks Ken, tweaked and merged. |
No description provided.