hintofbasil

/remove-lifecycle stale

Hi @rifelpet, My work is very interested in getting this functionality added in. Is there anything we can do to help you push this feature over the line?

Thanks for the context. We will bring this up with AWS ourselves. If we decide that we need this feature before AWS can provide it do you mind if we...

Hi Kishorj, Yes, we need both controller to modify the same target group. NLBs only support one listener per port, and each listener only supports a single target group. So...

/remove-lifecycle rotten

Hi @kishorj, thanks for agreeing to take a look. A few people in Skyscanner have agreed to review it in the mean time. Matteo has already put one review in....

This has been rebased and the formatting changes above implemented.

I'm unable to replicate the e2e test failure. I've added the following resources to a cluster running this branch of the controller. ``` apiVersion: v1 kind: Namespace metadata: name: test...

/retest

> I didn't see it while reading the code, but I'm wondering whether there are more IAM actions that we'll need to allow in the IAM policy used by this...