Skip to content

Conversation

penguinjournals
Copy link
Contributor

@penguinjournals penguinjournals commented Apr 21, 2022

feat: deploy cloudconnector on apprunner

side-quest

  • naming consolidation
    • simple > simple-ecs on both examples and test
    • cloud-connector > cloud-connector-ecs

@penguinjournals penguinjournals changed the title Cloudconnector deployment on AppRunner feat: cloudconnector deployment on AppRunner Apr 21, 2022
@wideawakening wideawakening removed their request for review April 25, 2022 07:52
@penguinjournals penguinjournals requested review from wideawakening and removed request for wideawakening April 25, 2022 07:52
README.md Outdated
## Usage

- There are several ways to deploy this in you AWS infrastructure, gathered under **[`/examples`](./examples)**
- [Single Account](#--single-account)
Copy link
Contributor

@wideawakening wideawakening Apr 28, 2022

Choose a reason for hiding this comment

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

⚠️ i would say "Single Account on ECS" here, following previous naming convention

### - Single-Account on AppRunner

Sysdig workload will be deployed using AppRunner in the same account where user's resources will be watched.<br/>
More info in [`./examples/single-account-apprunner`](https://github.com/sysdiglabs/terraform-aws-secure-for-cloud/tree/master/examples/single-account-apprunner)
Copy link
Contributor

Choose a reason for hiding this comment

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

⚠️
i would point all diagrams to master already, or you're gonna need an extra PR for that :D

@wideawakening
Copy link
Contributor

almost ready to go @penguinjournals ! left you some comments for changes ( ⚠️ )

edited the main description to add side-quest changes for clarification and for next iteration we need to sync apprunner with #84 and disable scanning by default. but that for another PR :D

README.md Outdated
If you already own a Kubernetes Cluster on AWS, you can use it to deploy Sysdig Secure for Cloud, instead of default ECS cluster.<br/>
More info in [`./examples/single-account-k8s`](https://github.com/sysdiglabs/terraform-aws-secure-for-cloud/tree/master/examples/single-account-k8s)

![single-account with pre-existing kubernetes cluster](https://raw.githubusercontent.com/sysdiglabs/terraform-aws-secure-for-cloud/feat-apprunner-poc/examples/single-account-k8s/diagram.png)
Copy link
Contributor

Choose a reason for hiding this comment

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

this one to master too :P

@penguinjournals penguinjournals merged commit e02bad8 into master May 3, 2022
@penguinjournals penguinjournals deleted the feat-apprunner-poc branch May 3, 2022 09:46
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.

3 participants