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

AWS Profile #416

Merged
merged 11 commits into from Jan 14, 2019
Merged

AWS Profile #416

merged 11 commits into from Jan 14, 2019

Conversation

marky-mark
Copy link
Contributor

closed #389

Added aws_profile

* Decrypt
* Command Line
* Master Key
* Encrypt
* .sops.yaml

Tested locally, the tests pass except for 1...i think i'm missing an imported key

Mark Kelly added 7 commits January 11, 2019 14:40
* Decrypt
* Command Line
* Master Key
* Encrypt
* .sops.yaml marshall
* Command Line argument fix
* Command Line argument fix for group add
* Decrypt
* Command Line
* Master Key
* Encrypt
* .sops.yaml
@autrilla autrilla self-requested a review January 11, 2019 17:22
Copy link
Contributor

@autrilla autrilla left a comment

Choose a reason for hiding this comment

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

lgtm, all changes are basically small nits. Once they're handled, we can merge this!

README.rst Outdated Show resolved Hide resolved
README.rst Outdated Show resolved Hide resolved
README.rst Outdated Show resolved Hide resolved
README.rst Outdated Show resolved Hide resolved
README.rst Outdated Show resolved Hide resolved
cmd/sops/main.go Outdated Show resolved Hide resolved
kms/keysource.go Outdated Show resolved Hide resolved
kms/keysource.go Outdated Show resolved Hide resolved
kms/keysource.go Outdated Show resolved Hide resolved
kms/keysource.go Outdated Show resolved Hide resolved
@marky-mark
Copy link
Contributor Author

Great thanks, working through the comments now

@marky-mark
Copy link
Contributor Author

ok all updated :D

@codecov-io
Copy link

codecov-io commented Jan 14, 2019

Codecov Report

Merging #416 into master will decrease coverage by 0.24%.
The diff coverage is 62.5%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #416      +/-   ##
==========================================
- Coverage   43.38%   43.14%   -0.24%     
==========================================
  Files          16       16              
  Lines        2206     2209       +3     
==========================================
- Hits          957      953       -4     
- Misses       1160     1166       +6     
- Partials       89       90       +1
Impacted Files Coverage Δ
stores/stores.go 0% <0%> (ø) ⬆️
config/config.go 77.21% <100%> (ø) ⬆️
kms/keysource.go 50.99% <80%> (+0.32%) ⬆️
stores/flatten.go 87.28% <0%> (-4.24%) ⬇️

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 cfc6fa1...eb7b22a. Read the comment docs.

Copy link
Contributor

@autrilla autrilla left a comment

Choose a reason for hiding this comment

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

👍 lgtm! Thanks for the patch!

@autrilla autrilla merged commit 3383f2d into getsops:master Jan 14, 2019
@marky-mark
Copy link
Contributor Author

Great thanks. Is there any idea when we can get a release so that we can get it automatically via homebrew?

@autrilla
Copy link
Contributor

@marky-mark I got an email with a comment saying there was a small bug, has that been fixed?

@marky-mark
Copy link
Contributor Author

Not a bug, i forgot to add the profile to the encrypted file. All good

@autrilla
Copy link
Contributor

Okay, great. Once #420 and #421 land, I'll cut a new release.

@autrilla autrilla mentioned this pull request Jan 24, 2019
@marky-mark marky-mark mentioned this pull request Jan 26, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Support AWS profile setting in creation_rules
3 participants