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

Add VmClarity cloud-formation starting point #1

Merged
2 commits merged into from
Nov 17, 2022
Merged

Conversation

ghost
Copy link

@ghost ghost commented Nov 4, 2022

No description provided.

@ghost ghost force-pushed the vmclarity_cloud_formation branch 2 times, most recently from 73a26e7 to 5ee3e7b Compare November 9, 2022 10:38
Adds a cloud-formation manifest which creates a VPC, Network and RBAC
for the VMClarity backend. It spins up a generic ubuntu VM which
notifies cloud formation when its ready.
The vmclarity systemd unit will configure and start the VMClarity
backend container. It will watch the status and restart it, if it
stops for whatever reason.

The container it starts right now is a generic ubuntu container until
we've got a VMClarity backend container published.
@ghost ghost force-pushed the vmclarity_cloud_formation branch from 5ee3e7b to 3ea303e Compare November 16, 2022 13:59
Comment on lines +161 to +164
- IpProtocol: tcp
FromPort: 80
ToPort: 80
CidrIp: 0.0.0.0/0
Copy link
Member

Choose a reason for hiding this comment

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

remove for now

Copy link
Author

Choose a reason for hiding this comment

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

Will address in the follow up task

Comment on lines +215 to +229
- "ec2:DetachVolume"
- "ec2:AttachVolume"
- "ec2:ExportImage"
- "ec2:TerminateInstances"
- "ec2:DeleteTags"
- "ec2:CreateTags"
- "ec2:RunInstances"
- "ec2:StopInstances"
- "ec2:DescribeVolumeAttribute"
- "ec2:CreateVolume"
- "ec2:DescribeStoreImageTasks"
- "ec2:DeleteVolume"
- "ec2:StartInstances"
- "ec2:CreateSnapshots"
- "ec2:CreateSnapshot"
Copy link
Member

Choose a reason for hiding this comment

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

separate (with a comment for now even) permissions needed for the entire account and only for our VPC

Copy link
Author

Choose a reason for hiding this comment

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

Will address in the follow up task

@ghost ghost merged commit b60e8a8 into main Nov 17, 2022
@ghost ghost deleted the vmclarity_cloud_formation branch November 17, 2022 11:54
This pull request was closed.
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.

None yet

3 participants