-
Notifications
You must be signed in to change notification settings - Fork 18
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
🧹 Updating AWS Policies to Align with Recent Developments #385
Conversation
Signed-off-by: Hossein Rouhani <h_rouhani@hotmail.com>
Signed-off-by: Hossein Rouhani <h_rouhani@hotmail.com>
Signed-off-by: Hossein Rouhani <h_rouhani@hotmail.com>
Signed-off-by: Hossein Rouhani <h_rouhani@hotmail.com>
Signed-off-by: Hossein Rouhani <h_rouhani@hotmail.com>
…t level Signed-off-by: Hossein Rouhani <h_rouhani@hotmail.com>
…3 Buckets Signed-off-by: Hossein Rouhani <h_rouhani@hotmail.com>
Signed-off-by: Hossein Rouhani <h_rouhani@hotmail.com>
Signed-off-by: Hossein Rouhani <h_rouhani@hotmail.com>
e51f99b
to
2a54d6a
Compare
…concurrent execution limits Signed-off-by: Hossein Rouhani <h_rouhani@hotmail.com>
Signed-off-by: Hossein Rouhani <h_rouhani@hotmail.com>
This comment has been minimized.
This comment has been minimized.
Signed-off-by: Hossein Rouhani <h_rouhani@hotmail.com>
This comment has been minimized.
This comment has been minimized.
…ances are configured for deletion on instance termination Signed-off-by: Hossein Rouhani <h_rouhani@hotmail.com>
This comment has been minimized.
This comment has been minimized.
Signed-off-by: Hossein Rouhani <h_rouhani@hotmail.com>
This comment has been minimized.
This comment has been minimized.
Signed-off-by: Hossein Rouhani <h_rouhani@hotmail.com>
This comment has been minimized.
This comment has been minimized.
Signed-off-by: Hossein Rouhani <h_rouhani@hotmail.com>
This comment has been minimized.
This comment has been minimized.
…ion protection enabled Signed-off-by: Hossein Rouhani <h_rouhani@hotmail.com>
This comment has been minimized.
This comment has been minimized.
…h Encryption-at-Rest Signed-off-by: Hossein Rouhani <h_rouhani@hotmail.com>
This comment has been minimized.
This comment has been minimized.
Signed-off-by: Hossein Rouhani <h_rouhani@hotmail.com>
This comment has been minimized.
This comment has been minimized.
…ide encryption KMS & SageMaker test Signed-off-by: Hossein Rouhani <h_rouhani@hotmail.com>
This comment has been minimized.
This comment has been minimized.
Signed-off-by: Hossein Rouhani <h_rouhani@hotmail.com>
This comment has been minimized.
This comment has been minimized.
Signed-off-by: Hossein Rouhani <h_rouhani@hotmail.com>
This comment has been minimized.
This comment has been minimized.
Signed-off-by: Hossein Rouhani <h_rouhani@hotmail.com>
This comment has been minimized.
This comment has been minimized.
Signed-off-by: Hossein Rouhani <h_rouhani@hotmail.com>
This comment has been minimized.
This comment has been minimized.
Signed-off-by: Hossein Rouhani <h_rouhani@hotmail.com>
This comment has been minimized.
This comment has been minimized.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Generally it's a good idea to update the checks to keep in line with the new developments, but there's a problem with changing the asset_filters:
of existing checks to other assets.
Many checks have been moved from the single asset, to the cluster asset, which 1) isn't the direction we're moving with checks in general and 2) more importantly, existing customers might have already worked with the single asset checks.
Would be great if we can keep the asset_filter scopes and just update the checks as needed.
…ion-level concurrent execution limits Signed-off-by: Hossein Rouhani <h_rouhani@hotmail.com>
Signed-off-by: Hossein Rouhani <h_rouhani@hotmail.com>
Signed-off-by: Hossein Rouhani <h_rouhani@hotmail.com>
…Management Service Signed-off-by: Hossein Rouhani <h_rouhani@hotmail.com>
Signed-off-by: Hossein Rouhani <h_rouhani@hotmail.com>
…onfigured for deletion on instance Signed-off-by: Hossein Rouhani <h_rouhani@hotmail.com>
Signed-off-by: Hossein Rouhani <h_rouhani@hotmail.com>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for the changes to the checks @HRouhani . Those all look good.
We need to double check and most likely remove all filters: asset.platform == "aws"
at group level as those will never allow any of the single asset level variants:
to run.
Signed-off-by: Hossein Rouhani <h_rouhani@hotmail.com>
@mm-weber It should be fine now, I did it exactly like what we did in Azure, using asset.runtime. You can test all variant using following script:
Change the cnspec path to your binary as well as the bundle path. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Good stuff @HRouhani !
Thank you.
No description provided.