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
[CONTINT-3529] Add basic ecs fargate windows app #571
base: main
Are you sure you want to change the base?
Conversation
ecf3e00
to
a4807ec
Compare
a4807ec
to
078b9d2
Compare
46ededa
to
7289cfe
Compare
ada285c
to
4bbe63c
Compare
|
||
opts = append(opts, pulumi.Parent(EcsFargateComponent)) | ||
|
||
nlb, err := lb.NewNetworkLoadBalancer(e.Ctx, namer.ResourceName("lb"), &lb.NetworkLoadBalancerArgs{ |
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.
Why do you need an LB?
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.
I was thinking of adding a query app too in another PR.
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.
I removed the load balancer so we can add it in another PR if needed
a8af4e9
to
d84b06b
Compare
What does this PR do?
This PR adds a basic ecs fargate windows application.
Which scenarios this will impact?
aws/ecs
Example: https://dddev.datadoghq.com/notebook/7776845/ali-windows-ecs?cell-31ekfdth-live=false&cell-31ekfdth-range=50606&cell-31ekfdth-start=1711630015394&cell-v7z96u34-live=false&cell-v7z96u34-range=1823936&cell-v7z96u34-start=1711628242064
Motivation
We would like to easily setup an ecs fargate windows setup.
Additional Notes