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

Do not throw error when disk path is not known yet #944

Merged
merged 1 commit into from
Jan 16, 2020

Conversation

koikonom
Copy link
Contributor

If the path to an attached disk is interpolated, then sometimes we may
end up in a situation where the diff for the VM having the disks runs
before the disk itself has been processed. In this case since the path
has no value we throw an error.

This changes the behaviour so we check if the path has been calculated
before we consider it final or not.

If the path to an attached disk is interpolated, then sometimes we may
end up in a situation where the diff for the VM having the disks runs
before the disk itself has been processed. In this case since the path
has no value we throw an error.

This changes the behaviour so we check if the path has been calculated
before we consider it final or not.
@koikonom koikonom requested a review from a team January 15, 2020 11:27
@ghost ghost added the size/m Relative Sizing: Medium label Jan 15, 2020
@aareet aareet added the bug Type: Bug label Jan 15, 2020
Copy link
Contributor

@bill-rich bill-rich left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@koikonom koikonom merged commit 1aa34d7 into master Jan 16, 2020
@koikonom koikonom deleted the interpolated_disk_paths branch January 16, 2020 18:26
@ghost ghost locked and limited conversation to collaborators Apr 18, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
bug Type: Bug size/m Relative Sizing: Medium
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants