-
Notifications
You must be signed in to change notification settings - Fork 68
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 kubernetes-aws-java
and container-aws-java
templates
#880
base: master
Are you sure you want to change the base?
Conversation
I'm running into pulumi/pulumi-awsx#820 when deploying a project created from the
The deployment runs, but fails creating the service with:
|
Also running into an issue deploying
|
.portMappings(List.of( | ||
TaskDefinitionPortMappingArgs.builder() | ||
.containerPort(containerPort) | ||
.targetGroup(loadbalancer.defaultTargetGroup()) |
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 guess this is the output that causes the issue. iirc wrapping this in pulumi.output(…)
(or the java equivalent) is a way to work around this. I can give it a try later
.portMappings(List.of( | ||
TaskDefinitionPortMappingArgs.builder() | ||
.containerPort(containerPort) | ||
.targetGroup(loadbalancer.defaultTargetGroup()) |
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.
This is working for me:
.targetGroup(loadbalancer.defaultTargetGroup()) | |
.targetGroup(Output.of(loadbalancer.defaultTargetGroup()).apply(Function.identity())) |
Note, this needs this import: import java.util.function.Function;
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.
As Florian mentioned in Slack, the workaround isn't actually working. We need to fix Java to serialize output values: pulumi/pulumi-java#1662. Working on this now.
Fixes #876
This change adds
kubernetes-aws-java
andcontainer-aws-java
templates.