Skip to content

[Network]Support azure firewall policy and its child resource/ support fqdns for network/application rules#1033

Merged
zikalino merged 8 commits intoAzure:masterfrom
mmyyrroonn:new-azure-firewall
Oct 31, 2019
Merged

[Network]Support azure firewall policy and its child resource/ support fqdns for network/application rules#1033
zikalino merged 8 commits intoAzure:masterfrom
mmyyrroonn:new-azure-firewall

Conversation

@mmyyrroonn
Copy link
Contributor


Azure/azure-cli#10703
Azure/azure-cli#10725

This checklist is used to make sure that common guidelines for a pull request are followed.

General Guidelines

  • Have you run azdev style <YOUR_EXT> locally? (pip install azdev required)
  • Have you run python scripts/ci/test_index.py -q locally?

For new extensions:

@zikalino
Copy link

looks very good. i am really surprised how you can make such large implementation without codegen :-)
couldn't check every single detail but focused on ux and tests.
so it's good to go.

@zikalino zikalino merged commit 4b78c59 into Azure:master Oct 31, 2019
ManuInNZ pushed a commit to ManuInNZ/azure-cli-extensions that referenced this pull request Apr 11, 2020
…t fqdns for network/application rules (Azure#1033)

* Bump SDK Dependency

* support firewall policy and its child resource CRUD and support fqdns for network/applicatoin rule

* add test

* recording the test

* fix style and add required for one argument
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.

2 participants