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

Cannot update existing VMs due to Priority being set #968

Closed
TheRSP opened this issue Sep 2, 2022 · 4 comments
Closed

Cannot update existing VMs due to Priority being set #968

TheRSP opened this issue Sep 2, 2022 · 4 comments
Assignees
Labels
bug Something isn't working

Comments

@TheRSP
Copy link
Contributor

TheRSP commented Sep 2, 2022

Trying to update a VM which was created with an older version of farmer now fails as of #894

{
    "status": "Failed",
    "error": {
        "code": "BadRequest",
        "message": "The specified priority value 'Regular' cannot be applied to the Virtual Machine '...' since no priority was specified when the Virtual Machine was created. For more information, see http://aka.ms/AzureSpot/errormessages."
    }
}
@isaacabraham
Copy link
Member

cc @bigjonroberts

@isaacabraham isaacabraham added the bug Something isn't working label Sep 10, 2022
@bigjonroberts
Copy link
Contributor

I think it should be fairly straightforward to change it to only output when set. I should have a chance to peek at this tomorrow morning.

@ninjarobot
Copy link
Collaborator

I think this is fixed by #969 that @TheRSP contributed. It was just released yesterday, but does what you describe @isaacabraham. Unless the explicitly set a priority then no priority field is emitted.

@ninjarobot
Copy link
Collaborator

Fixed by #969

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

4 participants