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

Feature: Move all decision logic into the Manager service #1576

Open
2 of 3 tasks
Despire opened this issue Nov 14, 2024 · 0 comments
Open
2 of 3 tasks

Feature: Move all decision logic into the Manager service #1576

Despire opened this issue Nov 14, 2024 · 0 comments
Assignees
Labels
feature New feature groomed Task that everybody agrees to pass the gatekeeper

Comments

@Despire
Copy link
Contributor

Despire commented Nov 14, 2024

Motivation

The manager service was introduced as the "brain" of claudie where the difference between the current state and desired state is create which results in a number of tasks to be worked on by the builder to achieve the desired state.

Description

Currently, not all the decision logic is present in Manager. Ideally, we should move them to the manager service when handling the Diff between the current and desired state. The other services Ansibler, Terraformer, Kuber, Kube-eleven should have only very minimal or ideally no decision logic in them.

Exit criteria

  • Explore if the generation of CIDR can be moved into the manager service and not be delayed until terraformer stage.
  • Ansibler Setup/Teardown of Loadbalancers should be split depending on whether the API endpoint change is needed. For example If the endpoint change is needed it will first Teardown the Lbs and then use the state from the teardown in the Setup The case of changing the API endpoint should be handled in its own task and then the rest of the Loadbalancers should be build, the logic of determining this should be moved to the manager instead of having this in Ansibler and Builder
  • Determine if there is anything else in all of the mentioned services that should be moved to the manager service.
@Despire Despire added the feature New feature label Nov 14, 2024
@bernardhalas bernardhalas added the groomed Task that everybody agrees to pass the gatekeeper label Dec 13, 2024
@Despire Despire self-assigned this Dec 16, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature New feature groomed Task that everybody agrees to pass the gatekeeper
Projects
None yet
Development

No branches or pull requests

2 participants