8000 feat: Dedicated cyclotron images by benjackwhite · Pull Request #31191 · PostHog/posthog · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

feat: Dedicated cyclotron images #31191

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.

Sign up for GitHub

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

Jump to bottom
Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

feat: Dedicated cyclotron images #31191

wants to merge 1 commit into from
+9 −0

Conversation

benjackwhite
Copy link
Contributor

Problem

I'm attempting to move cyclotron to our nicer posthog-rust chart and to do that i need separate state file entries https://github.com/PostHog/charts/pull/4243

Changes

  • Does that

Does this work well for both Cloud and self-hosted?

How did you test this code?

@benjackwhite benjackwhite requested a review from a team April 14, 2025 14:47
greptile-apps[bot]
greptile-apps bot reviewed Apr 14, 2025
View reviewed changes
Copy link
Contributor
@greptile-apps greptile-apps bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

PR Summary

This PR adds dedicated deployment configurations for cyclotron-fetch and cyclotron-janitor images in the CI/CD pipeline to support migration to the posthog-rust chart.

  • Added separate release entries for cyclotron-fetch and cyclotron-janitor in the deploy matrix
  • Maintained the original cyclotron release with a note indicating future removal
  • Each new release entry references its corresponding image digest from the build outputs
  • This change enables separate state file entries required for the chart migration referenced in PostHog/charts#4243

1 file(s) reviewed, no comment(s)
Edit PR Review Bot Settings | Greptile

@posthog-bot
Copy link
Contributor

This PR hasn't seen activity in a week! Should it be merged, closed, or further worked on? If you want to keep it open, post a comment or remove the stale label – otherwise this will be closed in another week. If you want to permanentely keep it open, use the waiting label.

@posthog-bot posthog-bot added stale and removed stale labels Apr 22, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@greptile-apps greptile-apps[bot] greptile-apps[bot] left review comments

@eli-r-ph eli-r-ph eli-r-ph approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@benjackwhite @posthog-bot @eli-r-ph
0