Skip to content
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

Reference a non-existent CFN parameter in minInstancesInServiceParameters #549

Closed
wants to merge 2 commits into from

Conversation

akash1810
Copy link
Member

@akash1810 akash1810 commented Oct 11, 2024

What does this change?

This is to allow testing of guardian/riff-raff#1383 whilst the riff-raff.yaml generator from GuCDK hasn't been updated.

This riff-raff.yaml is a stripped down version of the generated one, only uploading the application artifact to S3, and deploying the application CloudFormation stack.

This is a test of behaviour when the riff-raff.yaml file references a CFN parameter that does not exist in the CFN template.

How to test?

Deploy this branch; it should fail.

image

Note

This is existing behaviour.

This is to allow testing of guardian/riff-raff#1383
whilst the `riff-raff.yaml` generator from GuCDK hasn't been updated.
@akash1810 akash1810 changed the title Aa/cfn param 404 @akash1810 Reference a non-existent CFN parameter in minInstancesInServiceParameters Oct 11, 2024
@akash1810 akash1810 changed the title @akash1810 Reference a non-existent CFN parameter in minInstancesInServiceParameters Reference a non-existent CFN parameter in minInstancesInServiceParameters Oct 11, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant