Skip to content

[Network] az network dns zone: support - character#13377

Merged
mmyyrroonn merged 1 commit intoAzure:devfrom
stegag:patch-1
May 18, 2020
Merged

[Network] az network dns zone: support - character#13377
mmyyrroonn merged 1 commit intoAzure:devfrom
stegag:patch-1

Conversation

@stegag
Copy link
Member

@stegag stegag commented May 7, 2020

Description
See internal ICM 187136418
We are failing to import zones when domain name in $origin contains hypens.
This is because we don't parse it correctly in the regex (only alphanumeric char are evaluated with "\w" and that doesn't include hypen, like it is for outher resurces like CNAME.

Testing Guide

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.

See internal ICM 187136418
We are failing to import zones when domain name in $origin contains hypens.
This is because we don't parse it correctly in the regex here
@yonzhan
Copy link
Collaborator

yonzhan commented May 7, 2020

add to S169

@mmyyrroonn mmyyrroonn changed the title Update parse_zone_file.py [Network] az network dns zone: support - character May 8, 2020
@mmyyrroonn
Copy link
Contributor

/azp run Azure.azure-cli

@azure-pipelines
Copy link

Azure Pipelines failed to run 1 pipeline(s).

@haroldrandom
Copy link
Contributor

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 2 pipeline(s).

@mmyyrroonn mmyyrroonn changed the title [Network] az network dns zone: support - character [Network] az network dns zone: support "-" character May 12, 2020
@mmyyrroonn
Copy link
Contributor

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 2 pipeline(s).

@mmyyrroonn
Copy link
Contributor

/azp run Azure.azure-cli (Check the Format of Pull Request Title)

@azure-pipelines
Copy link

No pipelines are associated with this pull request.

@mmyyrroonn mmyyrroonn closed this May 12, 2020
@mmyyrroonn mmyyrroonn reopened this May 12, 2020
@mmyyrroonn
Copy link
Contributor

/azp run Azure.azure-cli

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@yonzhan yonzhan modified the milestones: S169 - For Build, S170 May 16, 2020
@yonzhan yonzhan requested a review from jsntcy May 16, 2020 04:54
@mmyyrroonn
Copy link
Contributor

/azp run Azure.azure-cli

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@mmyyrroonn mmyyrroonn changed the title [Network] az network dns zone: support "-" character [Network] az network dns zone: support - character May 18, 2020
@mmyyrroonn
Copy link
Contributor

/azp run Azure.azure-cli

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@mmyyrroonn mmyyrroonn merged commit bc39534 into Azure:dev May 18, 2020
derekbekoe added a commit to derekbekoe/azure-cli that referenced this pull request May 18, 2020
* 'dev' of https://github.com/Azure/azure-cli: (85 commits)
  {Monitor} az monitor autoscale profile show: bug fix (Azure#13515)
  [Monitor] az monitor diagnostic-settings create: support --export-to-specific-resource argument (Azure#13544)
  [aro]Add tests for generate_random_id function (Azure#13522)
  [ARO]Change CLIError to correct flag for --worker-vm-disk-size-gb (Azure#13439)
  [Packaging] Add Ubuntu Focal Package (Azure#13491)
  Update parse_zone_file.py (Azure#13377)
  {Release} Upgrade to Azure CLI 2.6.0 (Azure#13542)
  fix (Azure#13511)
  {Resource} Add example to tell how to pass array to --parameters (Azure#13510)
  [NetAppFiles] Anf 5207 Bugfix - add missing snapshot restore function (Azure#13481)
  {Network} az network nic create: Refine help message (Azure#13513)
  [Storage] az storage blob sync: Fix the incorrect error message when azcopy cannot find the installation location (Azure#9576)  (Azure#13104)
  Support --connect-string for az storage blob sync (Azure#13135)
  Modify the deprecate information for deployment operation (Azure#13390)
  [AppService] Onboard local context for app service (Azure#12984)
  {Monitor} az monitor metrics alert: refine severity explanation (Azure#13512)
  [IoT Hub] Support for 2020-03-01 API and Network Isolation commands (Azure#13467)
  [Compute] az vm list-skus: Update --zone behavior, return all type skus now (Azure#13470)
  Fix that cli doe not fail when user only specifies Windows password (Azure#13418)
  {ACR}: expose --expiration from token commands (Azure#13451)
  ...
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.

4 participants