search

PaloAltoNetworks / aws-transit-vpc

automated AWS transit vpc
41 stars 34 forks source link

CF JSON to YML? #63

Closed gliptak closed 5 years ago

gliptak commented 5 years ago

Would there be an interest in a PR converting https://github.com/PaloAltoNetworks/aws-transit-vpc/tree/master/cfts/*.json to *.yml?

https://github.com/PaloAltoNetworks/aws-transit-vpc/issues/52

gliptak commented 5 years ago

@jpeezus please comment. Somebody would also need to update the PDFs (as sources to generate from weren't made available)

jpeezus commented 5 years ago

@gliptak At this point in time I would say no. We have a team that converts or creates templates in the language of our focus. Right now although YML is very popular we don't have plans to convert this to YML or update any documentation based on any outside conversions. That could change in the future. We appreciate your feedback and interaction and look forward to continuing the interaction.

gliptak commented 5 years ago

@jpeezus thank you for the details

panwce commented 4 years ago

You can flip between json and yaml using cfn-flip I have "flipped" this template in the past without issues https://github.com/awslabs/aws-cfn-template-flip https://cfnflip.com/

gliptak commented 4 years ago

@panwce thank you for the pointer. The thought here is that YAML is friendlier to source control diffs allowing to understand update impact quicker.