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

OOTB: Remove conditional compilation and model dependency from pluto #3620

Closed
3 tasks
Tracked by #3615
zmrow opened this issue Nov 29, 2023 · 0 comments · Fixed by #3828
Closed
3 tasks
Tracked by #3615

OOTB: Remove conditional compilation and model dependency from pluto #3620

zmrow opened this issue Nov 29, 2023 · 0 comments · Fixed by #3828
Assignees
Labels
area/out-of-tree-builds Related to the efforts of making it easy to create builds outside of the main Bottlerocket repo

Comments

@zmrow
Copy link
Contributor

zmrow commented Nov 29, 2023

Currently the program is conditionally compiled based on the platform (AWS). It also makes calls to the API and pulls the entire set of settings from the model - which will need to change for OOTB.

  • Investigate if some of pluto's behavior could be rolled into the Kubernetes settings extension (perhaps kubernetes setting generator could call out to other binaries)
  • Remove the conditional compilation directives and exclude the package from non-AWS variants
  • Remove the model dependency (probably read from a file rendered by schnauzer)
@zmrow zmrow added this to the no-conditional-compilation milestone Nov 29, 2023
@jmt-lab jmt-lab self-assigned this Feb 15, 2024
@webern webern added the area/out-of-tree-builds Related to the efforts of making it easy to create builds outside of the main Bottlerocket repo label Feb 29, 2024
@jmt-lab jmt-lab mentioned this issue Mar 18, 2024
10 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/out-of-tree-builds Related to the efforts of making it easy to create builds outside of the main Bottlerocket repo
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants