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

[Test Proxy] Fix parameter reference in stop_record_or_playback #25725

Merged
merged 1 commit into from
Aug 16, 2022

Conversation

mccoyp
Copy link
Member

@mccoyp mccoyp commented Aug 16, 2022

Description

The test_output parameter in the test proxy infra's stop_record_or_playback method was recently changed to (more accurately) be test_variables. As part of the test tooling refactor, this name change got mismatched such that this code raises an error over an undefined test_output variable.

All SDK Contribution checklist:

  • The pull request does not introduce [breaking changes]
  • CHANGELOG is updated for new features, bug fixes or other significant changes.
  • I have read the contribution guidelines.

General Guidelines and Best Practices

  • Title of the pull request is clear and informative.
  • There are a small number of commits, each of which have an informative message. This means that previously merged commits do not appear in the history of the PR. For more information on cleaning up the commits in your PR, see this page.

Testing Guidelines

  • Pull request includes test coverage for the included changes.

@mccoyp mccoyp added the EngSys This issue is impacting the engineering system. label Aug 16, 2022
@mccoyp mccoyp requested a review from scbedd August 16, 2022 23:13
@mccoyp
Copy link
Member Author

mccoyp commented Aug 16, 2022

/check-enforcer override

@mccoyp mccoyp merged commit 7c835fb into main Aug 16, 2022
@mccoyp mccoyp deleted the proxy-recordstopfix branch August 16, 2022 23:18
wonder6845 pushed a commit to wonder6845/azure-sdk-for-python that referenced this pull request Aug 23, 2022
mccoyp added a commit to mccoyp/azure-sdk-for-python that referenced this pull request Sep 22, 2022
azure-sdk pushed a commit to azure-sdk/azure-sdk-for-python that referenced this pull request Sep 22, 2023
Release elasticsan microsoft.elastic san 2023 01 01 ga (Azure#25687)

* Adds base for updating Microsoft.ElasticSan from version preview/2022-12-01-preview to version 2023-01-01

* Updates readme

* Updates API version in new specs and examples

* Added snapshot API & SourceId change (Azure#25073)

* Added snapshot API & SourceId change

* removed state from virtualNetworkRule

* add CMK changes

* add object type on properties

* add managedby property

* Removed PATCH Snapshot API, Added forceDeleteFlags for volume, added provisioning state from volume, added publicNetworkAccess

* Added proper examples

* resolved Prettier check

* removed export, added enum for force-delete

* Updated operationID & resolved comments

---------

Co-authored-by: Harshjeet Aulakh <[email protected]>

* Adding update to x-ms-mutability (Azure#25725)

* Adding update to x-ms-mutability

* updated examples as per need of teraform

* added managedBy in Update properties

* prettier check

---------

Co-authored-by: Harshjeet Aulakh <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
EngSys This issue is impacting the engineering system.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants