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

Generate replacement requirements documentation for components #209

Draft
wants to merge 3 commits into
base: main
Choose a base branch
from

Conversation

leifmadsen
Copy link
Contributor

Create a documentation generator that describes the replacements that are required for use of the component within a kustomization.yaml components list.

@leifmadsen leifmadsen added documentation Improvements or additions to documentation do-not-merge Changes are not ready to be merged labels May 2, 2024
@leifmadsen leifmadsen self-assigned this May 2, 2024
Attempt to generate a sample ConfigMap in the component_replacements.md
markdown documentation to make it easier to know how to consume the
component and setup resources.
@leifmadsen
Copy link
Contributor Author

As this is more of a POC at this point, I'm going to probably stop picking at this in favour of obtaining feedback to the approach.

Reason for this PR, is that I felt the common components of the lib/ directory could use some documentation to identify the expected parameters that need to be passed to the component to make it operational in an environment overlay. The replacements do not have default values, and errors will be present when running kustomize build if the components are included in the environment overlay. The generated documents provide the expected parameters to be passed via local-config ConfigMap.

A few things are missing:

  • expectation that only a single ConfigMap name is used
    • create parameter hash per ConfigMap in case components use multiple configmaps
    • generated ConfigMap example would need to be done per-ConfigMap value
    • ConfigMap name needs to be updated

Other potential nice-to-haves:

  • also show example of how to consume the component in an overlay
  • use asciidoc so documentation could be used in product?
  • include built docs in mkdocs

@leifmadsen leifmadsen added question Further information is requested ↡ priority-low Low priority item that doesn't need immediate review labels May 3, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
do-not-merge Changes are not ready to be merged documentation Improvements or additions to documentation ↡ priority-low Low priority item that doesn't need immediate review question Further information is requested
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant