cloudfoundry / cf-acceptance-tests

CF Acceptance tests
Apache License 2.0
69 stars 171 forks source link

Add test for comma delimited ASG destinations #1135

Closed geofffranks closed 1 month ago

geofffranks commented 1 month ago

Are you submitting this PR against the develop branch?

What is this change about?

Adds a test to validate that ASGs with comma delimited destinations function, when configured to do so.

Please provide contextual information.

CAPI must be configured to enable this and it is off by default, so the test is also off by default.

What version of cf-deployment have you run this cf-acceptance-test change against?

Latest.

Please check all that apply for this PR:

Did you update the README as appropriate for this change?

If you are introducing a new acceptance test, what is your rationale for including it CATs rather than your own acceptance test suite?

The functionality requires a combination of CAPI, Diego, cf-networking, and silk to function. This is an integration test across across all the components.

How many more (or fewer) seconds of runtime will this change introduce to CATs?

60-200? The time taken to push an app and validate ASG changes to it.

What is the level of urgency for publishing this change?

Tag your pair, your PM, and/or team!

It's helpful to tag a few other folks on your team or your team alias in case we need to follow up later.