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.

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

[FEAT] - Split deployment of Greenhouse controller manager #808

Open
1 task
IvoGoman opened this issue Dec 17, 2024 · 0 comments · May be fixed by #855
Open
1 task

[FEAT] - Split deployment of Greenhouse controller manager #808

IvoGoman opened this issue Dec 17, 2024 · 0 comments · May be fixed by #855
Assignees
@gciezkowski-acc
Labels
feature ready Backlog Item is ready for Sprint Backlog

Comments

@IvoGoman
Copy link
Contributor

IvoGoman commented Dec 17, 2024 •
edited
Loading

Priority

(Medium) I'm annoyed but I'll live

User Story

As an Operator of the Greenhouse platform I want to deploy the operator and the webhooks separately, so that a failing operator deployment does not block Admission and Helm rollback.

Description

The Greenhouse Helm chart has one deployment to rollout the controller manager, which runs both the Operator and the Webhooks.
The goal of the issue is that the Operator and the Webhooks are deployed separately. This ensures that among others that Greenhouse CR's can still be applied when the Operator is failing.
There are already environment variables in place to run the operator in webhook only or controller only mode.

Acceptance Criteria

  • Helm Chart has separate deployments for webhook and operator

Reference Issues

No response
@github-project-automation github-project-automation bot moved this to Sprint Backlog in Greenhouse Core Roadmap Dec 17, 2024
@IvoGoman IvoGoman added the ready Backlog Item is ready for Sprint Backlog label Dec 17, 2024
@IvoGoman IvoGoman moved this from Sprint Backlog to Todo in Greenhouse Core Roadmap Jan 8, 2025
@gciezkowski-acc gciezkowski-acc self-assigned this Jan 15, 2025
gciezkowski-acc added a commit that referenced this issue Jan 16, 2025
@gciezkowski-acc
feat(deployments) split deployment to webhooks and controllers (#808)
5f00e94
gciezkowski-acc added a commit that referenced this issue Jan 17, 2025
@gciezkowski-acc
feat(deployments) split deployment to webhooks and controllers (#808)
5301889
gciezkowski-acc added a commit that referenced this issue Jan 17, 2025
@gciezkowski-acc
feat(controllers) Side effects after splitting deployments (#808)
ed799f5
gciezkowski-acc added a commit that referenced this issue Jan 20, 2025
@gciezkowski-acc
feat(deployments) split deployment to webhooks and controllers (#808)
26aa43f
gciezkowski-acc added a commit that referenced this issue Jan 20, 2025
@gciezkowski-acc
feat(controllers) Side effects after splitting deployments (#808)
e119c0b
gciezkowski-acc added a commit that referenced this issue Jan 20, 2025
@gciezkowski-acc
feat(deployments) Linter fixes (#808)
2ed8027
gciezkowski-acc added a commit that referenced this issue Jan 20, 2025
@gciezkowski-acc
feat(deployments) split deployment to webhooks and controllers (#808)
d50e2b9
gciezkowski-acc added a commit that referenced this issue Jan 20, 2025
@gciezkowski-acc
feat(controllers) Side effects after splitting deployments (#808)
ac025ab
gciezkowski-acc added a commit that referenced this issue Jan 20, 2025
@gciezkowski-acc
feat(deployments) Linter fixes (#808)
b424ea5
@gciezkowski-acc gciezkowski-acc linked a pull request Jan 20, 2025 that will close this issue
Open
24 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@gciezkowski-acc gciezkowski-acc

Labels
feature ready Backlog Item is ready for Sprint Backlog
Projects
Greenhouse Core Roadmap
Status: Todo
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

feat(greenhouse): split deployment of greenhouse cloudoperators/greenhouse
2 participants
@IvoGoman @gciezkowski-acc