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

🔧 Allow binary arguments via helm chart #201

Merged
merged 1 commit into from
Jan 26, 2023

Conversation

toddtee
Copy link
Contributor

@toddtee toddtee commented Jan 25, 2023

This commit allows to pass arguments to the helm chart. This is useful if you want to specify arguments for the helm-dashboard binary whilst deploying via helm.

Fixes Issue

Closes #200 .

Changes proposed

Arguments to the helm-dashboard binary can be supplied by utilising the extraArgs parameter within the supplied helm values.

Check List (Check all the applicable boxes)

  • My code follows the code style of this project.
  • My change requires changes to the documentation.
  • I have updated the documentation accordingly.
  • All new and existing tests passed.
  • The title of my pull request is a short description of the requested changes.

This commit allows to pass arguments to the helm chart.
This is useful if you want to specify arguments for the
helm-dashboard binary whilst deploying via helm.
@toddtee toddtee force-pushed the allow-arguments-via-helm-chart branch from fa560c0 to 2a78731 Compare January 25, 2023 01:50
@@ -111,6 +111,10 @@ autoscaling:

nodeSelector: {}

extraArgs:
Copy link
Collaborator

Choose a reason for hiding this comment

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

can we make two d

@undera
Copy link
Collaborator

undera commented Jan 26, 2023

Ok, let's try this in the quick minor release

@undera undera merged commit 7e32008 into komodorio:main Jan 26, 2023
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.

Allow binary arguments to be supplied via helm installation
2 participants