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

Submodule to onboard AWS account #8

Merged
merged 33 commits into from
Oct 2, 2024
Merged

Submodule to onboard AWS account #8

merged 33 commits into from
Oct 2, 2024

Conversation

shireesh-illumio
Copy link
Collaborator

@shireesh-illumio shireesh-illumio commented Sep 30, 2024

Add aws_account nested module which defines an AWS IAM role and policies to allow Illumio CloudSecure's AWS account to read/write an AWS account's resources and onboard that AWS account with CloudSecure.

Simplify the example for each module.
Update GitHub action to regenerate docs for examples.

modules/aws_account_permissions/main.tf Outdated Show resolved Hide resolved
modules/aws_account_permissions/main.tf Outdated Show resolved Hide resolved
examples/aws_account_permissions/versions.tf Outdated Show resolved Hide resolved
modules/aws_account_permissions/variables.tf Outdated Show resolved Hide resolved
modules/aws_account_permissions/variables.tf Outdated Show resolved Hide resolved
modules/aws_account_permissions/variables.tf Outdated Show resolved Hide resolved
modules/aws_account_permissions/outputs.tf Outdated Show resolved Hide resolved
modules/aws_account_permissions/outputs.tf Outdated Show resolved Hide resolved
modules/aws_account_permissions/variables.tf Outdated Show resolved Hide resolved
modules/aws_account_permissions/main.tf Outdated Show resolved Hide resolved
Copy link
Collaborator

@rlenglet rlenglet left a comment

Choose a reason for hiding this comment

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

11 comments are still unresolved.

@rlenglet rlenglet changed the title Submodule to onboard aws account Submodule to onboard AWS account Oct 2, 2024
@rlenglet
Copy link
Collaborator

rlenglet commented Oct 2, 2024

@evanj80-illumio can you please take a look at this? I fixed a few things in the k8s_cluster example, as well as the GitHub action, and I'd like to get your review.

@rlenglet
Copy link
Collaborator

rlenglet commented Oct 2, 2024

@itsesterline can you please take a look at this? Esp. the docs & examples.

modules/k8s_cluster/README.md Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
@evanj80-illumio
Copy link
Collaborator

No comments on any of the TF changes just some on the GHA and readme

rlenglet
rlenglet previously approved these changes Oct 2, 2024
evanj80-illumio
evanj80-illumio previously approved these changes Oct 2, 2024
@rlenglet rlenglet dismissed stale reviews from evanj80-illumio and themself via 7fce273 October 2, 2024 21:33
@rlenglet rlenglet self-requested a review October 2, 2024 21:35
@rlenglet
Copy link
Collaborator

rlenglet commented Oct 2, 2024

@busi-reddy-karnati FYI

Copy link
Collaborator

@itsesterline itsesterline left a comment

Choose a reason for hiding this comment

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

Approved

@rlenglet rlenglet merged commit 48db4dc into main Oct 2, 2024
4 checks passed
@rlenglet rlenglet deleted the aws_tf_module branch October 2, 2024 23:55
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.

4 participants