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

feat: support looking for existing secrets in vault #4360

Merged
merged 1 commit into from
Jun 21, 2019

Conversation

pmuir
Copy link
Contributor

@pmuir pmuir commented Jun 20, 2019

Submitter checklist

  • Change is code complete and matches issue description.
  • Change is covered by existing or new tests.

Description

If a secret already exists in vault then it is used and not reprompted

Signed-off-by: Pete Muir pmuir@bleepbleep.org.uk

Special notes for the reviewer(s)

Which issue this PR fixes

#4328

@pmuir pmuir force-pushed the vault_existing1 branch 3 times, most recently from 92b875c to e97868b Compare June 20, 2019 18:42
@pmuir
Copy link
Contributor Author

pmuir commented Jun 21, 2019

/retest

@pmuir pmuir force-pushed the vault_existing1 branch from e97868b to 2b47063 Compare June 21, 2019 08:50
If `"default":"<generated>”` is used on a token or a password then it will default to a generated secret

Signed-off-by: Pete Muir <pmuir@bleepbleep.org.uk>
@pmuir pmuir force-pushed the vault_existing1 branch from 2b47063 to d1c6970 Compare June 21, 2019 10:45
@codecov
Copy link

codecov bot commented Jun 21, 2019

Codecov Report

❗ No coverage uploaded for pull request base (master@921163c). Click here to learn what that means.
The diff coverage is 73.52%.

Impacted file tree graph

@@            Coverage Diff            @@
##             master    #4360   +/-   ##
=========================================
  Coverage          ?   43.36%           
=========================================
  Files             ?      810           
  Lines             ?   101113           
  Branches          ?        0           
=========================================
  Hits              ?    43846           
  Misses            ?    53606           
  Partials          ?     3661
Flag Coverage Δ
#e2e 31.17% <0%> (?)
#integration 41.7% <73.52%> (?)
Impacted Files Coverage Δ
pkg/vault/fake/client.go 19.64% <22.22%> (ø)
pkg/surveyutils/jsonschema.go 59.29% <92%> (ø)

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 921163c...d1c6970. Read the comment docs.

@codecov
Copy link

codecov bot commented Jun 21, 2019

Codecov Report

Merging #4360 into master will increase coverage by 1.4%.
The diff coverage is 75.55%.

Impacted file tree graph

@@            Coverage Diff            @@
##           master    #4360     +/-   ##
=========================================
+ Coverage   41.94%   43.34%   +1.4%     
=========================================
  Files         810      810             
  Lines      101158   101165      +7     
=========================================
+ Hits        42427    43855   +1428     
+ Misses      55369    53649   -1720     
- Partials     3362     3661    +299
Flag Coverage Δ
#e2e 31.16% <0%> (+24.52%) ⬆️
#integration 41.69% <75.55%> (ø) ⬆️
Impacted Files Coverage Δ
pkg/vault/fake/client.go 21.05% <22.22%> (ø) ⬆️
pkg/surveyutils/jsonschema.go 59.77% <88.88%> (+1.32%) ⬆️
pkg/cmd/opts/install.go 8.42% <0%> (+0.33%) ⬆️
pkg/helm/helm_helpers.go 55.13% <0%> (+0.62%) ⬆️
pkg/cmd/opts/helm.go 21.96% <0%> (+0.99%) ⬆️
pkg/cmd/opts/commands.go 35.8% <0%> (+1.23%) ⬆️
pkg/kube/pod.go 23.12% <0%> (+1.36%) ⬆️
pkg/cmd/opts/import.go 37.14% <0%> (+1.42%) ⬆️
pkg/util/strings.go 72.03% <0%> (+1.69%) ⬆️
pkg/cmd/opts/domain.go 20.68% <0%> (+1.72%) ⬆️
... and 48 more

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 921163c...d1c6970. Read the comment docs.

@jstrachan
Copy link
Member

/lgtm

@jenkins-x-bot
Copy link
Contributor

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: jstrachan

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 7b9c9db into jenkins-x:master Jun 21, 2019
daveconde pushed a commit to daveconde/jx that referenced this pull request Apr 7, 2020
…ins-x#4360)

If `"default":"<generated>”` is used on a token or a password then it will default to a generated secret

Signed-off-by: Pete Muir <pmuir@bleepbleep.org.uk>
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.

3 participants