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

fix: handle non-zero return code from Azure CLI #635

Merged
merged 1 commit into from
Apr 11, 2024

Conversation

blgm
Copy link
Member

@blgm blgm commented Apr 3, 2024

Resolves #633

When running "om vm-lifecycle create-vm" on Azure, a problem has recently occurred where querying a blob status using "az storage blob show" has started to exit with a non-zero code and error "There is currently a pending copy operation". Because of the error, the command terminates. It's likely that this is because of a change in the backend Azure API. As we are trying to determine whether the copy operation has finished, it's safe to ignore this error and continue polling.

@cf-gitbot
Copy link
Member

We have created an issue in Pivotal Tracker to manage this. Unfortunately, the Pivotal Tracker project is private so you may be unable to view the contents of the story.

The labels on this github issue will be updated when the story is started.

When running "om vm-lifecycle create-vm" on Azure, a problem has
recently occurred where querying a blob status using "az storage blob
show" has started to exit with a non-zero code and error "There is
currently a pending copy operation". Because of the error, the command
terminates. It's likely that this is because of a change in the backend
Azure API. As we are trying to determine whether the copy operation has
finished, it's safe to ignore this error and continue polling.

[#187303104](https://www.pivotaltracker.com/story/show/187303104)
@wayneadams
Copy link
Contributor

Merging after running VMManagers Suite tests locally:

root@e5e63a908f73:/workspace/om/vmlifecycle/vmmanagers# ginkgo
Running Suite: VMManagers Suite
===============================
Random Seed: 1712855812
Will run 224 of 224 specs

••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••
Ran 224 of 224 Specs in 0.960 seconds
SUCCESS! -- 224 Passed | 0 Failed | 0 Pending | 0 Skipped
PASS

Ginkgo ran 1 suite in 6.475538713s
Test Suite Passed

@wayneadams wayneadams merged commit 36320d4 into pivotal-cf:main Apr 11, 2024
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Upgrade-opsman and create-vm fail in Azure
3 participants