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

fix: default to use a temporary directory for applying helm charts #4384

Merged
merged 2 commits into from
Jun 24, 2019

Conversation

jstrachan
Copy link
Member

fixes #4383

Signed-off-by: James Strachan james.strachan@gmail.com

fixes jenkins-x#4383

Signed-off-by: James Strachan <james.strachan@gmail.com>
Signed-off-by: James Strachan <james.strachan@gmail.com>
@jstrachan
Copy link
Member Author

/test bdd

@codecov
Copy link

codecov bot commented Jun 24, 2019

Codecov Report

Merging #4384 into master will decrease coverage by <.01%.
The diff coverage is 15.78%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #4384      +/-   ##
==========================================
- Coverage   43.36%   43.35%   -0.01%     
==========================================
  Files         810      810              
  Lines      101180   101199      +19     
==========================================
+ Hits        43874    43875       +1     
- Misses      53644    53659      +15     
- Partials     3662     3665       +3
Flag Coverage Δ
#e2e 31.16% <5.26%> (-0.01%) ⬇️
#integration 41.7% <15.78%> (-0.01%) ⬇️
Impacted Files Coverage Δ
pkg/spring/model.go 0% <ø> (ø) ⬆️
pkg/cmd/step/helm/step_helm_apply.go 50.8% <15.78%> (-2.91%) ⬇️
pkg/cmd/uninstall/uninstall.go 62.59% <0%> (-1.53%) ⬇️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update d0979d3...a005b7d. Read the comment docs.

@pmuir
Copy link
Contributor

pmuir commented Jun 24, 2019

/lgtm

@jenkins-x-bot
Copy link
Contributor

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: pmuir

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@jenkins-x-bot jenkins-x-bot merged commit 3e39386 into jenkins-x:master Jun 24, 2019
daveconde pushed a commit to daveconde/jx that referenced this pull request Apr 7, 2020
…enkins-x#4384)

* fix: default to use a temporary directory for applying helm charts

fixes jenkins-x#4383

Signed-off-by: James Strachan <james.strachan@gmail.com>

* fix: lets use https for spring boot model

Signed-off-by: James Strachan <james.strachan@gmail.com>
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.

jx step helm apply should use a temporary directory
3 participants