Skip to content

Conversation

@imabedalghafer
Copy link
Contributor

Updating the fetch_repair_vm function to use the small letters in the query instead of capital letters


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

Related command

General Guidelines

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

For new extensions:

About Extension Publish

There is a pipeline to automatically build, upload and publish extension wheels.
Once your pull request is merged into main branch, a new pull request will be created to update src/index.json automatically.
The precondition is to put your code inside this repository and upgrade the version in the pull request but do not modify src/index.json.

@ghost ghost added the customer-reported Issues that are reported by GitHub users external to the Azure organization. label Aug 28, 2022
@ghost
Copy link

ghost commented Aug 28, 2022

Thank you for your contribution imabedalghafer! We will review the pull request and get back to you soon.

@yonzhan
Copy link
Collaborator

yonzhan commented Aug 28, 2022

vm repair

@imabedalghafer
Copy link
Contributor Author

Hello @zhoxing-ms ,

Would you please have a look at this PR, as currently the vm-repair extension is not able to perform the restore and run command due to the error that it cannot find the resources tagged with vm-repair. The issue is reproducible with the current version.

Regards,
Ibrahim Abedalghafer

@zhoxing-ms zhoxing-ms merged commit c3d43f9 into Azure:main Aug 29, 2022
@azclibot
Copy link
Collaborator

[Release] Update index.json for extension [ vm-repair ] : https://dev.azure.com/azure-sdk/internal/_build/results?buildId=1813287&view=results

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

Labels

customer-reported Issues that are reported by GitHub users external to the Azure organization.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants