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

[AutoPR eventhub/resource-manager] EventHub: Revert the NetworkRuleSet #4539

Merged

Conversation

AutorestCI
Copy link
Contributor

Revert "Added VNet API (#5351)"

This reverts commit f2ab4a8.
@AutorestCI
Copy link
Contributor Author

AutorestCI commented Mar 11, 2019

(message created by the CI based on PR content)

Installation instruction

Package azure-mgmt-eventhub

You can install the package azure-mgmt-eventhub of this PR using the following command:
pip install "git+https://github.com/Azure/azure-sdk-for-python@restapi_auto_5363#egg=azure-mgmt-eventhub&subdirectory=azure-mgmt-eventhub"

You can build a wheel to distribute for test using the following command:
pip wheel --no-deps "git+https://github.com/Azure/azure-sdk-for-python@restapi_auto_5363#egg=azure-mgmt-eventhub&subdirectory=azure-mgmt-eventhub"

If you have a local clone of this repository, you can also do:

  • git checkout restapi_auto_5363
  • pip install -e ./azure-mgmt-eventhub

Or build a wheel file to distribute for testing:

  • git checkout restapi_auto_5363
  • pip wheel --no-deps ./azure-mgmt-eventhub

Direct download

Your files can be directly downloaded here:

@codecov-io
Copy link

Codecov Report

Merging #4539 into restapi_auto_eventhub/resource-manager will increase coverage by <.01%.
The diff coverage is n/a.

Impacted file tree graph

@@                            Coverage Diff                             @@
##           restapi_auto_eventhub/resource-manager    #4539      +/-   ##
==========================================================================
+ Coverage                                   49.49%   49.49%   +<.01%     
==========================================================================
  Files                                        6307     6303       -4     
  Lines                                      176700   176618      -82     
==========================================================================
- Hits                                        87450    87420      -30     
+ Misses                                      89250    89198      -52
Impacted Files Coverage Δ
..._04_01/models/event_hub_management_client_enums.py 100% <ø> (ø) ⬆️
...ub/v2017_04_01/operations/namespaces_operations.py 83.8% <ø> (+8.31%) ⬆️
...azure/mgmt/eventhub/v2017_04_01/models/__init__.py 58.82% <ø> (+1.19%) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 4a2da56...287ad2a. Read the comment docs.

@AutorestCI AutorestCI merged commit c52c462 into restapi_auto_eventhub/resource-manager Mar 11, 2019
@AutorestCI AutorestCI deleted the restapi_auto_5363 branch March 11, 2019 23:48
@AutorestCI
Copy link
Contributor Author

This PR has been merged into #4529

lmazuel pushed a commit that referenced this pull request Apr 9, 2019
* Generated from fff07dcafefa324aa6378ba4f71dbc1e8f034f66 (#4521)

Added VNet API

* Generated from 85d0bffabce2f51b454c1662f8fd20b9c14167bc (#4533)

removed unused parameters

* Generated from 0a0a6f440615601504a0f696faaeccdeaf8248f3 (#4539)

Revert "Added VNet API (#5351)"

This reverts commit f2ab4a8.

* [AutoPR eventhub/resource-manager] EventHub: Added NetwrokruleSet API (#4540)

* Generated from 7de09862a31597b258f5724b04d7eecd34130546

Added NetwrokruleSet API

* Generated from d70fd504a3d7361dc46faa7946be60efa995f8b0

updated uri to networkRuleSets as per ARM guideline

* Generated from bb4e947567c5d75b4111bd76ce260e83da7caddb

updated with review comments

* Packaging update of azure-mgmt-eventhub

* updated versioning and change log
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants