Skip to content
This repository has been archived by the owner on Jul 5, 2021. It is now read-only.

Adding credstash backend #140

Merged
merged 6 commits into from
Jan 13, 2021
Merged

Adding credstash backend #140

merged 6 commits into from
Jan 13, 2021

Conversation

knelasevero
Copy link
Contributor

Adding credstash backend heavily based on ouzi-dev/credstash-operator.

@knelasevero knelasevero marked this pull request as draft November 24, 2020 21:02
@knelasevero knelasevero force-pushed the backend/credstash branch 2 times, most recently from 28e78c9 to e186b35 Compare November 24, 2020 21:05
@codecov
Copy link

codecov bot commented Nov 24, 2020

Codecov Report

Merging #140 (722becb) into master (943f9d9) will decrease coverage by 10.29%.
The diff coverage is 31.64%.

Impacted file tree graph

@@             Coverage Diff             @@
##           master     #140       +/-   ##
===========================================
- Coverage   80.85%   70.55%   -10.30%     
===========================================
  Files           9       10        +1     
  Lines         350      411       +61     
===========================================
+ Hits          283      290        +7     
- Misses         53      104       +51     
- Partials       14       17        +3     
Impacted Files Coverage Δ
pkg/utils/utils.go 26.47% <0.00%> (-26.48%) ⬇️
pkg/credstash/backend.go 39.34% <39.34%> (ø)
pkg/asm/backend.go 84.37% <100.00%> (-5.43%) ⬇️

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 943f9d9...2c1196f. Read the comment docs.

@knelasevero knelasevero changed the title [WIP] Adding credstash backend (That usus AWS KMS and DynamoDB underneath) [WIP] Adding credstash backend (That uses AWS KMS and DynamoDB underneath) Nov 24, 2020
@knelasevero
Copy link
Contributor Author

knelasevero commented Nov 24, 2020

Just leaving the draft up here. Will get back to it when I have time to write the tests and properly manually try things. I can still get feedback if someone has some 😄

@knelasevero knelasevero changed the title [WIP] Adding credstash backend (That uses AWS KMS and DynamoDB underneath) Adding credstash backend Nov 25, 2020
@knelasevero knelasevero marked this pull request as ready for review November 25, 2020 16:04
Copy link
Contributor

@jonatasbaldin jonatasbaldin left a comment

Choose a reason for hiding this comment

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

Hey mate, thanks for this work!

I couldn't test on a real AWS setup, but I left some comments on the code itself ✨

pkg/credstash/backend.go Show resolved Hide resolved
pkg/credstash/backend.go Outdated Show resolved Hide resolved
go.mod Outdated Show resolved Hide resolved
@knelasevero
Copy link
Contributor Author

i am going to merge this since it is taking a long time to get merged (my fault 😅 ). After we are confortable with all the backends that got integrated on master we can prepare to launch another release!

@knelasevero
Copy link
Contributor Author

/approve

@knelasevero
Copy link
Contributor Author

/label merge

@paul-the-alien paul-the-alien bot added the merge label Jan 13, 2021
@paul-the-alien paul-the-alien bot merged commit 89d57f4 into master Jan 13, 2021
@paul-the-alien paul-the-alien bot deleted the backend/credstash branch January 13, 2021 16:00
amouat pushed a commit that referenced this pull request May 26, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants