Skip to content

{App Service}: Updating the adhoc network api-version to the latest#23586

Draft
panchagnula wants to merge 2 commits intoAzure:devfrom
panchagnula:sisirap/networkDefaults
Draft

{App Service}: Updating the adhoc network api-version to the latest#23586
panchagnula wants to merge 2 commits intoAzure:devfrom
panchagnula:sisirap/networkDefaults

Conversation

@panchagnula
Copy link
Copy Markdown
Contributor

@panchagnula panchagnula commented Aug 18, 2022

Resolves #23389

Related command

Description

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.

@ghost ghost requested a review from yonzhan August 18, 2022 23:52
@ghost ghost added the Auto-Assign Auto assign by bot label Aug 18, 2022
@ghost ghost requested a review from wangzelin007 August 18, 2022 23:52
@ghost ghost assigned necusjz Aug 18, 2022
@ghost ghost added this to the Aug 2022 (2022-09-06) milestone Aug 18, 2022
@ghost ghost added the Network az network vnet/lb/nic/dns/etc... label Aug 18, 2022
@ghost ghost assigned zhoxing-ms Aug 18, 2022
@ghost ghost added the App Services az appservice label Aug 18, 2022
@panchagnula panchagnula marked this pull request as draft August 18, 2022 23:53
@yonzhan
Copy link
Copy Markdown
Collaborator

yonzhan commented Aug 18, 2022

App Service

@panchagnula panchagnula force-pushed the sisirap/networkDefaults branch from 6d5e1e2 to b718715 Compare August 19, 2022 00:18
@panchagnula panchagnula requested a review from madsd August 19, 2022 00:47
@panchagnula panchagnula marked this pull request as ready for review August 19, 2022 04:03
'container_network': '2018-08-01',
'appservice_network': '2020-04-01',
'appservice_ensure_subnet': '2019-02-01'
'appservice_network': '2021-08-01'
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.

I think it's better for us to totally remove adhoc api versions and use

ResourceType.MGMT_NETWORK: '2021-08-01',

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Makes sense - will make this change , after rebasing, since looks like I need to re-record some tests anyway :)

@madsd
Copy link
Copy Markdown
Contributor

madsd commented Aug 19, 2022

The change to use this model was not done by me - but @fengzhou-msft
image

6173ee4

@panchagnula
Copy link
Copy Markdown
Contributor Author

The change to use this model was not done by me - but @fengzhou-msft image

6173ee4

Btw while I did see the core-cli make this change - seems like they moved the versions from the code to adhoc versions under_shared.py , which I assumed you were originally the creator of. Apologize if this assumption was incorrect & randomizing you. 6173ee4#diff-8d2a65b8bdb03df2a25f1e9e5f2fa12fc680887b4274ed1e5b76bdf9ef26b6dc

@panchagnula
Copy link
Copy Markdown
Contributor Author

Marking Draft, since we need to wait for the new SDK/ api-version update on networking extension.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

act-observability-squad act-quality-productivity-squad App Services az appservice Auto-Assign Auto assign by bot Network az network vnet/lb/nic/dns/etc...

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Remove ad-hoc api versions

7 participants