[DNS] Issues with 'az network dns record-set list'

[jtuliani]

Issues with 'az network dns record-set list'

• 1. As with 'create' and 'delete', should support -z as short form for --zone-name (PR [DNS] Issues with 'az network dns record-set create' #1129)
• 2a. No way to format results as table whilst still showing record values (RDATA). This will require a change to our output infrastructure and is being tracked by item Table output with multiple tables #1948
• 2b. Also, the record type is not shown. This is a serious regression relative to the old CLI and makes the list output useless. (PR [Network] DNS Fixes #1823).
• 3. Doesn't show metadata in table format either (PR [Network] DNS Fixes #1823)
• 4. JSON output should not show 'null' output for each record type that doesn't apply to the record being listed. Instead, these lines should be suppressed. (PR [Network] DNS Fixes Round 2 #1592)
• 5. Doesn't support --type for listing record sets of a specific type only. This is a regression vs old CLI. (PR [Network] DNS Fixes #1823)
• 6. Remove --top parameter. Where results include a NextLink, this should be followed and each page of results returned. (PR [Network] DNS Fixes #1823)

[JasonRShaver]

• short names
• correct table output (does service return this?) (Add current directory to search path to allow load of modules without… #2, Initial code to sign user in and wire up the credentials in a sample storage command #3)
• Filtering by type (assuming service supports)
• remove top

[jtuliani]

Sample output attached
Sample record set list.txt

[jtuliani]

Also attaching example of current output...you can see all the 'null' items listed, which are a problem as described in item 4 above.
Existing record-set list output.txt

[jtuliani]

As discussed...JSON output over the wire does not include 'null' items, so they should definitely not be in the CLI JSON format. Re formatted sample output, @JasonRShaver to investigate

[tjprescott]

The null values exist because msrest deserializes the response into the strongly typed object. Since the values aren't included in the response, they end up as null values in the resulting object. We can handle this with a transform of the output.