search

awslabs / landing-zone-accelerator-on-aws

Deploy a multi-account cloud foundation to support highly-regulated workloads and complex compliance requirements.
https://aws.amazon.com/solutions/implementations/landing-zone-accelerator-on-aws/
Apache License 2.0
564 stars 449 forks source link

fix: `get-accelerator-metadata` getLastSuccessfulPipelineExecution #623

Closed StephenBedard closed 3 weeks ago

StephenBedard commented 3 weeks ago

Issue #, if available:

Description of changes:

The getLastSuccessfulPipelineExecution function was getting stuck in an infinite loop due to odd behavior from the ListPipelineExecutions API returning a nextToken even when there were no more results to get. I've changed this function to make use of the ListPipelineExecutions paginator.

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

erwaxler commented 3 weeks ago

Hi @StephenBedard , thank you for your contribution to the Landing Zone Accelerator!

I'm pleased to say this work has been integrated into the code base and will be available in the next major release. I'll close this PR in the meantime.

Thank you for your support of the LZA!