Skip to content

Get-AzVMBootDiagnostics api change to use RetrieveBootDiagnosticsData() #13233

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

Merged
merged 30 commits into from
Nov 9, 2020

Conversation

Sandido
Copy link
Contributor

@Sandido Sandido commented Oct 15, 2020

Description

Checklist

  • I have read the Submitting Changes section of CONTRIBUTING.md
  • The title of the PR is clear and informative
  • The appropriate ChangeLog.md file(s) has been updated:
    • For any service, the ChangeLog.md file can be found at src/{{SERVICE}}/{{SERVICE}}/ChangeLog.md
    • A snippet outlining the change(s) made in the PR should be written under the ## Upcoming Release header -- no new version header should be added
  • The PR does not introduce breaking changes
  • If applicable, the changes made in the PR have proper test coverage
  • For public API changes to cmdlets:
    • a cmdlet design review was approved for the changes in this repository (Microsoft internal only)
    • the markdown help files have been regenerated using the commands listed here

@Sandido Sandido requested review from msJinLei and isra-fel October 28, 2020 14:50
@isra-fel isra-fel self-assigned this Oct 29, 2020
@Sandido
Copy link
Contributor Author

Sandido commented Oct 30, 2020

Not getting input from PS reviewers in understanding the CI check failures, so marking this as Ready for Review to try and get feedback.

to see if any effect on the CI checks.
@isra-fel
Copy link
Member

isra-fel commented Nov 2, 2020

/azp run

@Sandido Sandido changed the title Get-AzVMBootDiagnostics api change to use RetrieveBootDiagnosticsData() ! Do Not Merge ! Get-AzVMBootDiagnostics api change to use RetrieveBootDiagnosticsData() Nov 3, 2020
@Sandido
Copy link
Contributor Author

Sandido commented Nov 3, 2020

Adding Do Not Merge because we are concerned with using a preview version for Azure.Storage.Blobs, despite it being required by the CI checks and PS team.

@Sandido Sandido changed the title ! Do Not Merge ! Get-AzVMBootDiagnostics api change to use RetrieveBootDiagnosticsData() Get-AzVMBootDiagnostics api change to use RetrieveBootDiagnosticsData() Nov 3, 2020
@Sandido
Copy link
Contributor Author

Sandido commented Nov 4, 2020

/azp run azure-powershell - powershell-core

@azure-pipelines
Copy link
Contributor

Azure Pipelines successfully started running 1 pipeline(s).

@Sandido
Copy link
Contributor Author

Sandido commented Nov 4, 2020

/azp run azure-powershell - windows-powershell (Test)

@azure-pipelines
Copy link
Contributor

No pipelines are associated with this pull request.

@Sandido
Copy link
Contributor Author

Sandido commented Nov 4, 2020

/azp run azure-powershell - windows-powershell

@azure-pipelines
Copy link
Contributor

Azure Pipelines successfully started running 1 pipeline(s).

@Sandido
Copy link
Contributor Author

Sandido commented Nov 5, 2020

/azp run azure-powershell - powershell-core

@azure-pipelines
Copy link
Contributor

Azure Pipelines successfully started running 1 pipeline(s).

@azure-pipelines
Copy link
Contributor

No pipelines are associated with this pull request.

@isra-fel
Copy link
Member

isra-fel commented Nov 9, 2020

/azp run azure-powershell - powershell-core

@azure-pipelines
Copy link
Contributor

Azure Pipelines successfully started running 1 pipeline(s).

@isra-fel isra-fel merged commit 7ca83a6 into master Nov 9, 2020
@isra-fel isra-fel deleted the SandidoBootDiagDataAPI branch November 9, 2020 03:07
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.

6 participants