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

RFC - Enterprise Bootstrapping #101

Open
wants to merge 2 commits into
base: main
Choose a base branch
from
Open

RFC - Enterprise Bootstrapping #101

wants to merge 2 commits into from

Conversation

enekofb
Copy link
Contributor

@enekofb enekofb commented Jan 5, 2023

There is no single definition of what an enterprise platform is nor what enterprise bootstrapping for a platform is. However, we have been running Weave Gitops Enterprise in 2022 which has created a body of knowledge on the limitations that enterprise customers have found while bootstrapping its platform with WGE. This RFC
looks at them and proposes a solution to allow solving what we know now allowing a degree of flexibility to
adapt to the unknowns.

4. it shoudl works with 'any' infrastructure layer (ex not only work with terraform
provisioned infra not just capi provisioned infra)

### alternative a: leverage terraform for configuration manager
Copy link
Contributor Author

Choose a reason for hiding this comment

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

to validate with @JamWils whether aligns with the product strategy for weave gitops


### alternative a: leverage terraform for configuration manager

Same approach as take for cluster bootstrap controller but leveraging terraform and terraform controller
Copy link
Contributor Author

Choose a reason for hiding this comment

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

From @MostafaMegahid policy agent needs cert manager, how to provision this?

### alternative a: leverage terraform for configuration manager

Same approach as take for cluster bootstrap controller but leveraging terraform and terraform controller
for bootstrapping via terraform doing the configuration management.
Copy link
Contributor Author

Choose a reason for hiding this comment

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

policy agent is used for validate infrastructure - is there a chicken and egg solution?


- infrastructure should not be linked to cluster api

### alternative b: same as a) but via cluster bootstrapping
Copy link
Contributor Author

Choose a reason for hiding this comment

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

@ahsayde if you want to complete

@enekofb enekofb changed the title bootstrapping-next wip - weave gitops bootstrapping next Jan 17, 2023

TBA

### alternative c: do nothing, leverage to CRE
Copy link
Contributor Author

Choose a reason for hiding this comment

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

@enekofb enekofb changed the title wip - weave gitops bootstrapping next RFC - Enterprise Bootstrapping Jan 20, 2023
@enekofb enekofb marked this pull request as ready for review January 20, 2023 19:12
@enekofb enekofb force-pushed the bootstrapping-next branch from c39c683 to f0bbbfa Compare January 20, 2023 19:15
@enekofb enekofb requested a review from bigkevmcd January 20, 2023 19:16
namespace: flux-system
```

A PoC could be found [here](https://github.com/weaveworks/clusters-config/tree/cluster-wge2205/)
Copy link

@waleedhammam waleedhammam Jan 31, 2023

Choose a reason for hiding this comment

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

readme here: https://github.com/weaveworks/clusters-config/blob/cluster-waleed-terraform/eksctl-clusters/leaf-terraform/README.md

Blockers and manual work

  • needs to apply flux namespace first more info here
  • tf-controller needs service account to work in case of aws or gke currently implemented using aws provider if otherwise needs to change the provider configurations

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants