Skip to content

Adding the Microsoft.Security NetworkData API#7730

Closed
NicolasMunoz1 wants to merge 58 commits intoAzure:masterfrom
NicolasMunoz1:master
Closed

Adding the Microsoft.Security NetworkData API#7730
NicolasMunoz1 wants to merge 58 commits intoAzure:masterfrom
NicolasMunoz1:master

Conversation

@NicolasMunoz1
Copy link

Created new PR because the previous owner of the repo is on Maternity leave
Previous open PR

NetworkData is a new feature in Azure Security Center.
Security Center's Network Map displays a visualization of the various network elements such as VNets, Subnets, VM's etc. connectivity and traffic information. The network traffic REST API presents the underlying information upon which the map is constructed, including the allowed connections and actual traffic data for each VM.

@AutorestCI
Copy link

AutorestCI commented Nov 7, 2019

Automation for azure-sdk-for-python

A PR has been created for you based on this PR content.

Once this PR will be merged, content will be added to your service PR:
Azure/azure-sdk-for-python#8450

@AutorestCI
Copy link

AutorestCI commented Nov 7, 2019

Automation for azure-sdk-for-go

A PR has been created for you based on this PR content.

Once this PR will be merged, content will be added to your service PR:
Azure/azure-sdk-for-go#6275

@NicolasMunoz1 NicolasMunoz1 reopened this Nov 7, 2019
@azuresdkci
Copy link
Contributor

Can one of the admins verify this patch?

@chlahav chlahav added the DoNotMerge <valid label in PR review process> use to hold merge after approval label Nov 7, 2019
@akning-ms
Copy link
Contributor

Hi @NicolasMunoz1 Can we know what is this PR status? as it is pending a long time.

@NicolasMunoz1
Copy link
Author

NicolasMunoz1 commented Mar 8, 2020

Hi @NicolasMunoz1 Can we know what is this PR status? as it is pending a long time.

Hi @akning-ms, NetworkData is new feature, currently the feature is in private preview. I continue the previous PR because the person that started the PR in maternity leave.

We still need to make some changes on the files.

@akning-ms
Copy link
Contributor

If so, I suggest you put lable 'NotReadyForReview until it is ready.

@akning-ms akning-ms added the NotReadyForReview <valid label in PR review process>It is in draft for swagger or not swagger PR label Mar 27, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

DoNotMerge <valid label in PR review process> use to hold merge after approval NotReadyForReview <valid label in PR review process>It is in draft for swagger or not swagger PR

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants