Skip to content

[Network] DNS Record Set Create Updates#1583

Merged
tjprescott merged 2 commits into
Azure:masterfrom
tjprescott:DnsFixes
Dec 16, 2016
Merged

[Network] DNS Record Set Create Updates#1583
tjprescott merged 2 commits into
Azure:masterfrom
tjprescott:DnsFixes

Conversation

@tjprescott

Copy link
Copy Markdown
Member

Fixes #1129 (and pieces of other DNS issues).

  • Updates help text
  • Adds the following parameters to dnz record-set create: --metadata, --if-match, --if-none-match
  • Removes record type SOA from the --type enum because that type cannot be created, only updated.
  • Adds -t alias for --type and -z alias for --zone-name

@tjprescott tjprescott added the Network az network vnet/lb/nic/dns/etc... label Dec 15, 2016

@derekbekoe derekbekoe left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM (one small question)

register_cli_argument('network dns record txt add', 'value', nargs='+')
register_cli_argument('network dns record txt remove', 'value', nargs='+')

register_cli_argument('network dns record-set create', 'record_set_type', options_list=('--type', '-t'), help='The type of DNS records in the record set.', **enum_choice_list(modified_record_type))

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

If this is similar to line 512, create an arg_type and refer to it in both places?

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Help might be different otherwise?

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

You can still reference an arg_type and then override a piece of it, so this is a great suggestion. 👍

@tjprescott tjprescott merged commit 76e4ac5 into Azure:master Dec 16, 2016
@tjprescott tjprescott deleted the DnsFixes branch December 16, 2016 17:17
@haroldrandom haroldrandom added cla-not-required Network az network vnet/lb/nic/dns/etc... labels Oct 25, 2019
00Kai0 pushed a commit to 00Kai0/azure-cli that referenced this pull request Apr 7, 2021
Triggered by Azure CLI Extensions Release Pipeline - ADO_BUILD_ID=367568

Last commit: Azure/azure-cli-extensions@8877ae4
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

cla-not-required Network az network vnet/lb/nic/dns/etc...

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants