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

Update fis.py #1887

Merged
merged 1 commit into from
Apr 2, 2021
Merged

Update fis.py #1887

merged 1 commit into from
Apr 2, 2021

Conversation

vasinov
Copy link
Contributor

@vasinov vasinov commented Apr 1, 2021

Even though the CF docs say [1] that FIS resource keys are in camelCase they are actually in PascalCase (just like any other standard AWS resource). It's easily testable by submitting a CF template. I'm contacting AWS to notify them about this discrepancy.

  1. https://docs.aws.amazon.com/AWSCloudFormation/latest/UserGuide/aws-properties-fis-experimenttemplate-experimenttemplateaction.html

Even though the CF docs say [1] that FIS resource keys are in camelCase they are actually in PascalCase (just like any other standard AWS resource). It's easily testable by submitting a CF template. I'm contacting AWS to notify them about this discrepancy.

1. https://docs.aws.amazon.com/AWSCloudFormation/latest/UserGuide/aws-properties-fis-experimenttemplate-experimenttemplateaction.html
@markpeek markpeek merged commit 57c577b into cloudtools:master Apr 2, 2021
@markpeek
Copy link
Member

markpeek commented Apr 2, 2021

Thanks! Yes, the previous spec file must have had the camelCase but is now fixed.

@vasinov vasinov deleted the patch-1 branch April 2, 2021 14:35
@vasinov vasinov mentioned this pull request Apr 16, 2021
markpeek pushed a commit that referenced this pull request Apr 16, 2021
Even though the CF docs say [1] that FIS resource keys are in camelCase they are actually in PascalCase (just like any other standard AWS resource). It's easily testable by submitting a CF template. I'm contacting AWS to notify them about this discrepancy.

1. https://docs.aws.amazon.com/AWSCloudFormation/latest/UserGuide/aws-properties-fis-experimenttemplate-experimenttemplateaction.html
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.

None yet

2 participants