-
-
Notifications
You must be signed in to change notification settings - Fork 9.3k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
analytics: optionally duplicate to InfluxDB #13356
analytics: optionally duplicate to InfluxDB #13356
Conversation
Review period will end on 2022-06-01 at 16:05:13 UTC. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nice work so far! Great to see this get some traction.
Review period ended. |
This pull request has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. |
8fcfcdd
to
14d2e08
Compare
b1d84a3
to
79db9ee
Compare
67f27ae
to
0b549e6
Compare
The brew domain didn't work for me, we'll have to see if the paid version allows custom domains but otherwise I think we're stuck on the current one. |
0b549e6
to
d3d41c8
Compare
@SMillerDev This would be a hard blocker on full adoption from me, I'm afraid 😭. I can help when this PR is a bit further along, though. A suggestion here would be to gate this functionality behind |
Why would using an external domain for analytics be a blocker for you? Isn't the same situation we have now?
That's fine by me |
This is not just an external domain but one specific to a particular cloud provider and region.
We're self-hosting afterwards which means I want to set a higher standard for any future migrations. |
The current implementation isn't actually self-hosted. It's using InfluxDB Cloud (DBaaS). |
Cool. Let's get in touch with them and see what they can do for custom domains. What's the pricing like there? |
The pricing is here: https://www.influxdata.com/influxdb-cloud-pricing/ |
@SMillerDev Thanks! May want to see if the "annual plan" is better and will need to watch our cardinality. |
d3d41c8
to
4e0b432
Compare
This pull request has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. |
4e0b432
to
d8d3a01
Compare
246f5ae
to
96add4c
Compare
@MikeMcQuaid actually forgot to update here. But we also can't do a vanity domain if we setup through Google Cloud. That would only be available if we self-host which I'd like to avoid. Most of the other TODO items are now done. |
96add4c
to
00ec468
Compare
50e6a27
to
fde8d2c
Compare
Should analytics report being Linuxbrew on Linux? I thought it was all Homebrew now? |
Yes, all Homebrew and a single analytics "bucket" with differing OS would be ideal. |
c0789f0
to
654f147
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good to me once suggested changes are applied and tests are 🟢. Sorry for all the back-and-forth.
654f147
to
bd562d4
Compare
bd562d4
to
5225afc
Compare
5225afc
to
59ebdab
Compare
@MikeMcQuaid any suggestions what I can do about the timeouts? |
@SMillerDev For now: ignore them. Are you ready to merge as-is? If so: I can do it manually. |
Yeah, I think this is ready to merge. |
brew style
with your changes locally?brew typecheck
with your changes locally?brew tests
with your changes locally?Starting the work that was open for GSoC but is not part of GSoC
Closes https://github.com/Homebrew/gsoc/issues/46
TODO:
install
andinstall_on_request
metrics