Ignition config should be marshaled as compact JSON #2

maikzumstrull · 2017-06-30T09:27:12Z

It's currently using indented JSON:
https://github.com/terraform-providers/terraform-provider-ignition/blob/master/ignition/resource_ignition_config.go#L122

This doesn't really help with readability, as Terraform logs the JSON blob with additional escaping, ruining the effect, plus raw Ignition configs aren't meant to be human-readable anyway. It does blow up the file size, though, and most cloud providers have a pretty tight size limit on metadata.

adidenko · 2017-09-18T12:19:34Z

Regression:
https://github.com/terraform-providers/terraform-provider-ignition/blob/v1.0.0/ignition/resource_ignition_config.go#L132

Add build release script

mcuadros added the enhancement label Jun 30, 2017

mcuadros mentioned this issue Jun 30, 2017

ignition_config: render to non-indented json #3

Merged

mcuadros closed this as completed in #3 Jun 30, 2017

adidenko mentioned this issue Oct 4, 2017

Regression with indented JSON #21

Closed

kairen pushed a commit to getamis/terraform-provider-ignition that referenced this issue Oct 16, 2020

Merge pull request hashicorp#2 from yussufsh/build

5aff61a

Add build release script

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Ignition config should be marshaled as compact JSON #2

Ignition config should be marshaled as compact JSON #2

maikzumstrull commented Jun 30, 2017

adidenko commented Sep 18, 2017

Ignition config should be marshaled as compact JSON #2

Ignition config should be marshaled as compact JSON #2

Comments

maikzumstrull commented Jun 30, 2017

adidenko commented Sep 18, 2017