Skip to content

ci: add helm chart publisher#4

Merged
ndbaker1 merged 1 commit intomainfrom
release-ci
May 14, 2025
Merged

ci: add helm chart publisher#4
ndbaker1 merged 1 commit intomainfrom
release-ci

Conversation

@ndbaker1
Copy link
Contributor

@ndbaker1 ndbaker1 commented Apr 23, 2025

Issue #, if available:

Description of changes:

This PR adds a workflow to publish helm charts once a new commit is pushed to main

Testing Done:

Pushed initial chart version to github pages - https://github.com/aws/eks-node-monitoring-agent/tree/gh-pages

helm repo add eks-node-monitoring-agent https://aws.github.io/eks-node-monitoring-agent
helm search repo eks-node --versions
NAME                                                    CHART VERSION	APP VERSION	DESCRIPTION
eks-node-monitoring-agent/eks-node-monitoring-a...      1.2.0        	1.2.0      	A Helm chart for eks-node-montoring-agent

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

@ndbaker1 ndbaker1 force-pushed the release-ci branch 2 times, most recently from 9886987 to ec58184 Compare April 23, 2025 06:06
@ndbaker1 ndbaker1 changed the title feat(ci): add helm chart publisher ci: add helm chart publisher Apr 23, 2025
@ndbaker1 ndbaker1 requested review from mselim00 and wwvela April 23, 2025 20:49
Copy link

@wwvela wwvela left a comment

Choose a reason for hiding this comment

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

Nice!

@ndbaker1 ndbaker1 marked this pull request as draft April 24, 2025 22:27
@ndbaker1
Copy link
Contributor Author

was putting this on hold to explore some other hosting options, but we can go ahead and set this up as one medium

@ndbaker1 ndbaker1 marked this pull request as ready for review May 14, 2025 23:04
@ndbaker1 ndbaker1 merged commit 9e7f827 into main May 14, 2025
1 check passed
@ndbaker1 ndbaker1 deleted the release-ci branch May 14, 2025 23:11
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.

2 participants