-
Notifications
You must be signed in to change notification settings - Fork 384
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
Update data_aws_policies.go #311
Conversation
@stikkireddy in next release we should add "byovpc" attributes to data |
Travis tests have failedHey @stikkireddy, 1st Buildcurl -sSL "https://github.com/gotestyourself/gotestsum/releases/download/v0.4.2/gotestsum_0.4.2_linux_amd64.tar.gz" | sudo tar -xz -C /usr/local/bin gotestsum
curl -sSfL https://github.com/golangci/golangci-lint/master/install.sh | sh -s -- -b $(go env GOPATH)/bin v1.25.0
time make lint
time make test
time make build
2nd Buildcurl -sSL "https://github.com/gotestyourself/gotestsum/releases/download/v0.4.2/gotestsum_0.4.2_linux_amd64.tar.gz" | sudo tar -xz -C /usr/local/bin gotestsum
curl -sSfL https://github.com/golangci/golangci-lint/master/install.sh | sh -s -- -b $(go env GOPATH)/bin v1.25.0
time make lint
time make test
time make build
TravisBuddy Request Identifier: 1d6e7560-f346-11ea-addc-35ae975c2fab |
Travis tests have failedHey @stikkireddy, 2nd Buildcurl -sSL "https://github.com/gotestyourself/gotestsum/releases/download/v0.4.2/gotestsum_0.4.2_linux_amd64.tar.gz" | sudo tar -xz -C /usr/local/bin gotestsum
curl -sSfL https://github.com/golangci/golangci-lint/master/install.sh | sh -s -- -b $(go env GOPATH)/bin v1.25.0
time make test
TravisBuddy Request Identifier: a701e140-f346-11ea-addc-35ae975c2fab |
Hey @stikkireddy, TravisBuddy Request Identifier: 3dcd0530-f349-11ea-addc-35ae975c2fab |
Codecov Report
@@ Coverage Diff @@
## master #311 +/- ##
==========================================
+ Coverage 64.13% 64.18% +0.05%
==========================================
Files 55 55
Lines 6730 6741 +11
==========================================
+ Hits 4316 4327 +11
Misses 2044 2044
Partials 370 370
|
The new default for E2 is to support NPIP for when not bringing your own VPC. For this, there is a requirement to support creation of
This additions was created using the information provided in the docs and existing policy in the code and performing
set() - set() =
I also noticed that two items were removed that are required for BYOVPC:
I did not remove them to keep the superset between both BYOVPC and Databricks Managed VPC
I also added the field to allow the policy to be a superset of the BYOVPC: