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

Helm SDK doc section and examples #1543

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

Conversation

gjenkins8
Copy link
Contributor

@gjenkins8 gjenkins8 commented Jan 4, 2024

Create a docs section for the SDK. With a (brief) description of the SDK, and some examples

(the main intent of the PR is to add SDK examples)

Signed-off-by: George Jenkins <[email protected]>
Signed-off-by: George Jenkins <[email protected]>
@@ -0,0 +1,144 @@
module github.com/helm/helm-www/sdkexamples
Copy link
Contributor Author

@gjenkins8 gjenkins8 Jan 19, 2024

Choose a reason for hiding this comment

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

I put the source in this 'sdkexamples' directory. As this directory / the source is meant to be a standalone, buildable go project. Suggestions on how to better structure this welcome

Signed-off-by: George Jenkins <[email protected]>
@gjenkins8 gjenkins8 marked this pull request as ready for review January 19, 2024 05:04
Signed-off-by: George Jenkins <[email protected]>
@rea1shane
Copy link

Will this be merged?

@rea1shane
Copy link

Please add helm template example.

@karenhchu karenhchu added the docs label Oct 8, 2024
@scottrigby scottrigby assigned scottrigby and unassigned scottrigby Oct 24, 2024
@scottrigby scottrigby self-requested a review October 24, 2024 17:26
Copy link
Member

@scottrigby scottrigby left a comment

Choose a reason for hiding this comment

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

@gjenkins8 this all looks great! 👏 Just a few requested changes.

Would you also:

  • ensure the code is up to date (given that it took me 10 months to review)?
  • rebase from main (to make sure we don't plow over any changes that may have been made since then for things like navigation weights, etc)?

@@ -0,0 +1 @@
{{ transform.Highlight (os.ReadFile (.Get "file")) "go" }}
Copy link
Member

Choose a reason for hiding this comment

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

small Git nit, needs newline at EOF

aliases: ["/docs/gosdk"]
weight: 1
---
Helm's Go SDK enables custom software to leverage Helm charts and Helm's functionality for managing Kubernetes software deployment (In fact, the Helm CLI is effectively just one such tool!)
Copy link
Member

Choose a reason for hiding this comment

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

I think we should integrate with this existing Go SDK section under Advanced Helm Techniques https://helm.sh/docs/topics/advanced/#go-sdk

Helm's Go SDK enables custom software to leverage Helm charts and Helm's functionality for managing Kubernetes software deployment (In fact, the Helm CLI is effectively just one such tool!)

Currently, the SDK has been functionally separated from the Helm CLI, and the SDK can (and is) used by standalone tooling. The Helm project has committed to API stability for the SDK. As a warning, the SDK has some rough edges remaining from the initial work to separate the CLI and the SDK. Which the Helm project aims to improve and over time.

Copy link
Member

Choose a reason for hiding this comment

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

may be good to link to the examples page from here. Especially for mobile users—or any user who landed here from a web search—who may not see the examples page in the left navigation.

Suggested change
See [Go SDK Examples]({{< ref "../examples" >}}) for fully working code examples.

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

Successfully merging this pull request may close these issues.

4 participants