Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add uaenorth to non-zonal regions #1926

Merged
merged 1 commit into from
Feb 2, 2022
Merged

Conversation

25region
Copy link
Contributor

@25region 25region commented Feb 1, 2022

Which issue this PR addresses:

Fixes an issue with uaenorth RP deployment. The gateway deployment is failing due to zoning issues.

What this PR does / why we need it:

Adds uaenorth to non-zonal exceptions for the gateway deployment

Test plan for issue:

Existing tests.

Is there any documentation that needs to be updated for this PR?

N/A

@github-actions
Copy link

github-actions bot commented Feb 1, 2022

Please rebase pull request.

@bennerv
Copy link
Collaborator

bennerv commented Feb 1, 2022

/azp run ci

@azure-pipelines
Copy link

Pull request contains merge conflicts.

@github-actions github-actions bot removed the needs-rebase branch needs a rebase label Feb 1, 2022
Copy link
Contributor

@ross-bryan ross-bryan left a comment

Choose a reason for hiding this comment

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

lgtm

Copy link
Collaborator

@bennerv bennerv left a comment

Choose a reason for hiding this comment

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

LGTM - deployed successfully

@bennerv bennerv merged commit 9155a92 into Azure:master Feb 2, 2022
@25region 25region deleted the uaenorth-bug branch February 2, 2022 14:42
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.

3 participants