Skip to content

[AppConfig] Configure max line width for YAML parser to stop wrapping output - #17401

Merged
Feiyue Yu (qwordy) merged 1 commit into
Azure:devfrom
avanigupta:avanigupta/YamlDumpWordWrappingBug
Mar 24, 2021
Merged

Feiyue Yu (qwordy) merged 1 commit into
Azure:devfrom
avanigupta:avanigupta/YamlDumpWordWrappingBug

Conversation

@avanigupta

Copy link
Copy Markdown
Member

Description

Fix #491
PyYAML parser has a default width of 80 characters per line for output dumping. In order to output long string values in the same line without wrapping, we need to set the width option to an arbitrarily high number in yaml.safe_dump() call.

Testing Guide

  1. Create a key in AppConfig with a very long string value like ""a a a a a a a a a a a a a a a a a a a a a a a a a a a a a a a a a a a a a a a a a a a a a a a a a a a a a a a a a a a a a a a a a a a a a a a a a a a a a a a a a a a a a a a a a a a a a a a a a a a a a a a a a a a a a a a a a a a a a a a a a a a a a a a a a a a a a a a a a a a a a a a a a a a a a a a a a a a a a a a a a a a a a a a a a a a a a a a a a a a a a a a a a a a a a a a a a a a a a a a a a a a a a a a a a a a a a a a a a a a a a a a a a a a a a a a a a a a a a a a a a a a a a a a a a a a a a a a a a a a a a a a a a a"

  2. Export key-values to a YAML file:
    az appconfig kv export --format yaml -d file --path export.yaml -n test-appconfig-00

The output in file will contain wrapped value like this:

LongValue: a a a a a a a a a a a a a a a a a a a a a a a a a a a a a a a a a a a a
  a a a a a a a a a a a a a a a a a a a a a a a a a a a a a a a a a a a a a a a a
  a a a a a a a a a a a a a a a a a a a a a a a a a a a a a a a a a a a a a a a a
  a a a a a a a a a a a a a a a a a a a a a a a a a a a a a a a a a a a a a a a a
  a a a a a a a a a a a a a a a a a a a a a a a a a a a a a a a a a a a a a a a a
  a a a a a a a a a a a a a a a a a a a a a a a a a a a a a a a a a a a a a a a a
  a a a a a a a a a a a a a a a a a a a a a a a a a a a a a a

History Notes

[Component Name 1] BREAKING CHANGE: az command a: Make some customer-facing breaking change.
[Component Name 2] az command b: Add some customer-facing feature.


This checklist is used to make sure that common guidelines for a pull request are followed.

@avanigupta

Copy link
Copy Markdown
Member Author

cc Shuai Wang (@shenmuxiaosen)

@qwordy

Copy link
Copy Markdown
Member

/azp run

@azure-pipelines

Copy link
Copy Markdown
Azure Pipelines successfully started running 2 pipeline(s).

@qwordy
Feiyue Yu (qwordy) merged commit 0097fa3 into Azure:dev Mar 24, 2021
@avanigupta
Avani Gupta (avanigupta) deleted the avanigupta/YamlDumpWordWrappingBug branch March 24, 2021 23:27
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.

Long string with spaces is wrapped after 80 characters with az appconfig kv export --format yaml

3 participants