Homebrew / brew

🍺 The missing package manager for macOS (or Linux)
https://brew.sh
BSD 2-Clause "Simplified" License
41.3k stars 9.71k forks source link

formula brew analytics api does not contain `aws/tap/aws-sam-cli` formula anymore #15528

Closed moelasmar closed 1 year ago

moelasmar commented 1 year ago

brew doctor output

N/A

Verification

brew config output

N/A

What were you trying to do (and why)?

We are using the formula analytics API https://formulae.brew.sh/api/analytics/install/30d.json to get the usage for our formula aws/tap/aws-sam-cli. It was working fine till 5/30, but then this API response does not contain our formula aws/tap/aws-sam-cli anymore. The same issue exist in https://formulae.brew.sh/api/analytics/build-error/30d.jsonx for the installation errors.

What happened (include all command output)?

when we execute curl https://formulae.brew.sh/api/analytics/install/30d.json | jq -er '.items[] | select(.formula=="aws/tap/aws-sam-cli")' nothing retuned.

What did you expect to happen?

we expect some data to be returned, something like the following:

{
  "number": 297,
  "formula": "aws/tap/aws-sam-cli",
  "count": "11,262",
  "percent": "0.06"
}

Step-by-step reproduction instructions (by running brew commands)

run `curl https://formulae.brew.sh/api/analytics/install/30d.json | jq -er '.items[] | select(.formula=="aws/tap/aws-sam-cli")'`
Bo98 commented 1 year ago

Tap prefixes are currently not working (it's incorrectly merges with homebrew-core's aws-sam-cli), not sure if there's another tracking issue somewhere but this is indeed a regression.

MikeMcQuaid commented 1 year ago

not sure if there's another tracking issue somewhere but this is indeed a regression.

There is not but, yes, this is a known regression we need to handle with our InfluxDB data generation CC @SMillerDev.

SMillerDev commented 1 year ago

Looking at InfluxDB I'm not sure how to join 2 fields like this. Looking again later, but it might be easier to just always send the full formula name with the tap and strip them out in JSON generation.

MikeMcQuaid commented 1 year ago

@SMillerDev I'll play around and see if I have any luck.

moelasmar commented 1 year ago

We still have this issue for the build-error data