helpdesk icon indicating copy to clipboard operation
helpdesk copied to clipboard

Published 20 hours ago • verified publisher icon jenkins-infra

plugin stats for bouncycastle-api are missing data.

Open jtnord opened this issue 1 year ago • 13 comments

Service(s)

stats.jenkins.io

Summary

The statistics for the bouncycastle-api plugin are missing details of the last 4 CD releases.
There is additionally a release present (221.v8c725fc275c7) that appears to be bogus.

Reproduction steps

  1. go to https://stats.jenkins.io/pluginversions/bouncycastle-api.html
  2. check the version of the plugin in the top row
  3. check the plugins releases and tags (notice the last four releases and the lack of a 221.v8c725fc275c7 version
  4. download https://updates.jenkins.io/current/plugin-versions.json and check the known versions of bouncycastle-api

Note that there are missing releases from the stats data and also release that is bogus.

jtnord avatar Aug 07 '24 08:08 jtnord

Versions appear in the HTML source, my guess is https://github.com/jenkins-infra/infra-statistics/commit/6d7432c852006f3b67a77f7b2aec98d0b94c9043 wasn't comprehensive enough and is missing legal punctuation.

daniel-beck avatar Aug 07 '24 09:08 daniel-beck

from @daniel-beck in a PM

completely bogus release

We do not filter plugins by known public versions, and show what gets reported.

explains the 221.v8c725fc275c7 that has come from someone then :)

jtnord avatar Aug 07 '24 09:08 jtnord

show what gets reported

Noting that we could, like we do for core, we just don't bother at the moment.

daniel-beck avatar Aug 07 '24 09:08 daniel-beck

Refer to a similar issue that was reported earlier. At that time, it was decided that the results from the fix were worse than the issue:

  • #3815

MarkEWaite avatar Aug 13 '24 14:08 MarkEWaite

So allowlisting public releases would be the way to go, no filtering needed later. Do we know enough about where usage-log-decrypter is run to ensure any such proposal would end up merged and used? Right now the repo state does not look good…

Alternatively, we could check for known public releases in infra-statistics and filter there.

daniel-beck avatar Aug 13 '24 16:08 daniel-beck

Any objection if this issue moved to the repository (as there are no actionable for the Jenkins infra team)?

dduportal avatar Aug 20 '24 12:08 dduportal

I assume this is related but for all plugins there is no update for the installation count. The last data is from June e.g. for script-security image Is there some script not running that updates the stats or is it failing?

mawinter69 avatar Sep 05 '24 07:09 mawinter69

Yes, after changes to require VPN Kohsuke's machine can't connect, @dduportal was in contact to resolve last I heard

timja avatar Sep 05 '24 08:09 timja

Absolutely: we are tracking the missing stats in https://github.com/jenkins-infra/helpdesk/issues/4285

dduportal avatar Sep 10 '24 17:09 dduportal

https://github.com/jenkins-infra/helpdesk/issues/4285 has been closed: stats until 29 Oct. have been integrated.

@jtnord @daniel-beck I still can't see the expected releases in the HTML but I'm not sure how it is generated. Are there additional work to do as per the discussion here?

dduportal avatar Nov 14 '24 17:11 dduportal

I think that the issue with "-" characters in the version string is still the reason that those releases are not displayed.

MarkEWaite avatar Nov 14 '24 18:11 MarkEWaite

I think that

As confirmed in https://github.com/jenkins-infra/helpdesk/issues/4213#issuecomment-2273025837.

https://github.com/jenkins-infra/helpdesk/issues/4213#issuecomment-2286672473 still needs an answer.

daniel-beck avatar Nov 19 '24 09:11 daniel-beck

We need help on this one as the infra team does not have enough bandwidth to work on the statistics at this level

dduportal avatar Nov 20 '24 07:11 dduportal