Skip to content
This repository has been archived by the owner on Dec 19, 2023. It is now read-only.

Kanister profile #4

Closed
wants to merge 2 commits into from
Closed

Kanister profile #4

wants to merge 2 commits into from

Conversation

gauthamsunjay
Copy link

What this PR does / why we need it:

  • Provides the option to install a profile CR along with the operator
  • the profile CR is installed via a subchart

Which issue this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close that issue when PR gets merged): fixes #

Special notes for your reviewer:

Are we planning to move this chart to the kanister repo?

@gauthamsunjay gauthamsunjay self-assigned this Jun 7, 2018
@@ -0,0 +1,5 @@
dependencies:
- name: profile
version: 0.1.0
Copy link
Member

Choose a reason for hiding this comment

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

This can't be templated, correct? What happens when we update the profile chart?

Choose a reason for hiding this comment

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

from https://github.com/kubernetes/helm/blob/master/docs/helm/helm_dependency.md
The 'version' field should contain a semantic version or version range.

Choose a reason for hiding this comment

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

something like version: =>0.1.0 should work

Copy link
Author

Choose a reason for hiding this comment

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

@depohmel
Copy link

To push it upstream you would have to bump chart version. Also @ntolia wanted to drop v form chart version

@gauthamsunjay
Copy link
Author

Sure, Ill do that. What should I change the chart version to?

@depohmel
Copy link

closing since we have our chart repo added into https://hub.helm.sh/

@depohmel depohmel closed this Sep 11, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Development

Successfully merging this pull request may close these issues.

3 participants