-
Notifications
You must be signed in to change notification settings - Fork 33
feat(kms): Add KMS under beta #935
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
Merged
Merged
Changes from 1 commit
Commits
Show all changes
71 commits
Select commit
Hold shift + click to select a range
7ee923b
pkg setup for kms
5a7af4f
cmd config setup
72a60d6
beta kms cmd added
84541d6
(missed) add kms to the beta
13bbc8c
added unit tests
1c4301d
added generated documentation updates
ab298b5
added kms to go.mod and go.sum
615d4bf
Merge branch 'main' into kms-beta
JanStern 6a4716d
Merge branch 'main' into kms-beta
JanStern 5e76f28
Merge branch 'main' into kms-beta
JanStern 6459764
Merge branch 'main' into kms-beta
JanStern eeed803
Merge branch 'main' into kms-beta
JanStern 36daef5
Merge branch 'main' into kms-beta
JanStern 3ea181c
fix spelling
ada9a54
yaml/json format added
7bbcd7a
empty responses also handeld in json/yaml format
b63c674
make test variables constant
f92f4c5
Can't remove backend yet, due to missing kms v0.5.1 support
07da0b7
linting improvement
2f9e6d3
generate-docs update
2614575
Upgrade KMS to v1.0.0
de5e097
Merge branch 'main' into kms-beta
JanStern b925f4e
fix spelling
6581f82
added wait for key creation
26490c9
removed uuid validation in parseInput
77c6d96
nil pointer safe
f10d6c1
json/yaml output type safe
c1ba5cc
updated docs
7351b96
Merge branch 'main' into kms-beta
JanStern 411b25d
Merge branch 'main' into kms-beta
JanStern 070d21d
Merge branch 'main' into kms-beta
JanStern 60ddebf
Merge branch 'main' into kms-beta
JanStern d5545f7
Merge branch 'main' into kms-beta
JanStern 056ffdb
remove comments
2bac600
stakit -> stackit
d4a1a1e
clearner loop
03406f1
fix spelling
bba5217
1. Corrected the output format for all requests
e39d775
updated the docs
2f68ead
Merge branch 'main' into kms-beta
JanStern 6dd4e61
Added the 'nil response' test case for the outputResult()
e7c6c71
Update internal/cmd/beta/kms/key/delete/delete.go
JanStern ff4b53a
Update internal/cmd/beta/kms/wrappingkey/create/create.go
JanStern e9d3719
Update internal/cmd/beta/kms/wrappingkey/wrappingkey.go
JanStern c19a855
Update internal/cmd/beta/kms/wrappingkey/create/create.go
JanStern fe09771
Update internal/cmd/beta/kms/wrappingkey/delete/delete.go
JanStern 0852ada
Update internal/cmd/beta/kms/wrappingkey/list/list.go
JanStern 78871c2
removed risky delete output
6639d4e
addressing nil pointer derferences
8d3d9f5
updated docs
9ca8501
Merge branch 'main' into kms-beta
JanStern 67806db
Merge branch 'main' into kms-beta
JanStern 32cb334
Addressed PR #1016
e58bc38
fix name in example form --key-ring to correct --key-ring-id
519f724
Fix feedback for async operations
ae9a66a
fix name in example form --key to correct --key-id
996d1ee
linter, test & docs
407d671
Merge branch 'main' into kms-beta
JanStern da17679
Merge branch 'main' into kms-beta
JanStern 9824515
`--key-ring-id` changed to `--keyring-id`
78506a8
Added `EnumValues` to the flag options
54332f0
Merge branch 'main' into kms-beta
JanStern 8880348
argument ID always in uppercase & typo fix
ae0efe3
Removed default flag value
eae8ec2
added import from file
5869606
better examples
119c00c
Merge branch 'main' into kms-beta
JanStern 53845d3
fix the path
de6aba7
Fix more spelling mistakes
9b79a43
proper capitalization for flag and argument examples & typos & nitpicks
34bf29f
Make testRegion const
File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
fix spelling
- Loading branch information
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Uh oh!
There was an error while loading. Please reload this page.