Skip to content
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

Evaluate the deprecation of the beat deployment #177

Closed
guidoiaquinti opened this issue Nov 3, 2021 · 2 comments · Fixed by #184
Closed

Evaluate the deprecation of the beat deployment #177

guidoiaquinti opened this issue Nov 3, 2021 · 2 comments · Fixed by #184
Assignees
Labels
deployments enhancement New feature or request P2 nice to have

Comments

@guidoiaquinti
Copy link
Contributor

Is your feature request related to a problem?

We currently run two specific deployments for beat and workers. We could probably simplify this setup by deprecating the beat deployment and reconfigure workers to run ./bin/docker-worker-celery --with-scheduler instead of ./bin/docker-worker-celery.

I'm not aware of any downside doing that (as it's also how we run PostHog Cloud) but we should probably double check everything works as expected before moving forward.

Describe the solution you'd like

Less resources to maintain.

Describe alternatives you've considered

Do nothing.

Additional context

See conversation in our internal Slack

@guidoiaquinti guidoiaquinti added the enhancement New feature or request label Nov 3, 2021
@mariusandra
Copy link
Contributor

Beat was separated in the chart since that's how it was in the original template I adapted. We then integrated redbeat to not have an extra beat dyno on heroku, costing money. It has worked without a hitch so far, and I guess the separate beat pod can be killed

@guidoiaquinti
Copy link
Contributor Author

Thank you for the additional context @mariusandra ! 🙇

@fuziontech fuziontech added P2 nice to have deployments labels Nov 10, 2021
@guidoiaquinti guidoiaquinti linked a pull request Nov 11, 2021 that will close this issue
6 tasks
@guidoiaquinti guidoiaquinti self-assigned this Nov 22, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
deployments enhancement New feature or request P2 nice to have
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants