Skip to content

(feature) - Allow chart version to diff to be specified #1

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.

Sign up for GitHub

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

Jump to bottom
Closed
lachie83 opened this issue Mar 21, 2017 · 6 comments
Closed

(feature) - Allow chart version to diff to be specified #1

lachie83 opened this issue Mar 21, 2017 · 6 comments

Comments

@lachie83
Copy link
Contributor

No description provided.

@databus23
Copy link
Owner

I thought about this as well. Ideally it should be possible to compare arbitrary versions of a release.

I'm just not sure about the CLI UI for that without making it confusing. When comparing two deployed versions the command does not need the otherwise mandatory local directory.

@lachie83
Copy link
Contributor Author

Currently doing something like this as a workaround

helm fetch --untar deis/workflow --version $(WORKFLOW_VERSION)
helm diff deis-workflow ./workflow \
-f ../../env/$(CLUSTER_ID)/workflow-values.yaml

@lachie83
Copy link
Contributor Author

It also doesn't appear to trigger the hooks and display hook changes. Any ideas how hard that would be?

@lachie83 lachie83 mentioned this issue Apr 11, 2017
Closed
@lachie83
Copy link
Contributor Author

@databus23 I've submitted a PR with the ability to specify the chart version. LMK what you think

@databus23 databus23 mentioned this issue Apr 11, 2018
Closed
jeff-knurek pushed a commit to jeff-knurek/helm-diff that referenced this issue Apr 12, 2018
FLAG: allow option for chart version
e85580c 
The first issue databus23#1 and one of the first PRs are still open on this topic.
I think databus23#6 is an entirely unrelated proposal, and if `helm` allows for different version on install, I want to see the diff of what gets installed with that.
@jeff-knurek jeff-knurek mentioned this issue Apr 12, 2018
Closed
@databus23
Copy link
Owner

I believe this has been fixed by a536dfc and is shipped since version 2.9.0+1 of this plugin.

1 similar comment
@databus23
Copy link
Owner

I believe this has been fixed by a536dfc and is shipped since version 2.9.0+1 of this plugin.

@zhilyaev zhilyaev mentioned this issue Nov 11, 2021
Closed
@AndriiOmelianenko AndriiOmelianenko mentioned this issue Jul 4, 2024
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

feat(core): Add ability to specify chart version in diff lachie83/helm-diff
2 participants
@databus23 @lachie83