jenkins-infra / helpdesk

Open your Infrastructure related issues here for the Jenkins project
https://github.com/jenkins-infra/helpdesk/issues/new/choose
17 stars 10 forks source link

plugin stats for bouncycastle-api are missing data. #4213

Open jtnord opened 3 months ago

jtnord commented 3 months ago

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.

daniel-beck commented 3 months ago

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.

jtnord commented 3 months ago

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 :)

daniel-beck commented 3 months ago

show what gets reported

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

MarkEWaite commented 3 months ago

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:

daniel-beck commented 3 months ago

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.

dduportal commented 3 months ago

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

mawinter69 commented 2 months ago

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?

timja commented 2 months ago

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

dduportal commented 2 months ago

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

dduportal commented 2 weeks ago

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?

MarkEWaite commented 2 weeks ago

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

daniel-beck commented 1 week ago

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.

dduportal commented 1 week ago

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