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

Jterry75/containerd runhcs shim fixes #286

Merged
merged 4 commits into from
Aug 17, 2018

Conversation

jterry75
Copy link
Contributor

Fixes a bug in runhcs state where we return an error rather than status unknown if the gcs cannot be reached but in the starting phase.
Removes an additional newline in an error making it easier to match.
Adds runhcs create-scratch that creates a ext4 formatted vhdx at the target path.
Adds runhcs tart2vhd that converts a given tar to vhd at the target path.

@jterry75 jterry75 requested a review from lowenna August 15, 2018 20:14
@jterry75
Copy link
Contributor Author

@crosbymichael - FYI only df6c81b is needed for non lcow work

@crosbymichael
Copy link

@jterry75 thanks!

@jterry75
Copy link
Contributor Author

@jhowardmsft - I thought we already fixed this issue with CI?

@lowenna
Copy link
Contributor

lowenna commented Aug 15, 2018

I thought we already fixed this issue with CI?

Whaaaa? That shouldn't be happening.

Occasionally the product will fail the state query and return "operation
is not valid in the current sate". Rather than returning an error in
this case we want to return state "unknown" with a valid json output for
clients to be able to handle.

Signed-off-by: Justin Terry (VM) <juterry@microsoft.com>
Signed-off-by: Justin Terry (VM) <juterry@microsoft.com>
Signed-off-by: Justin Terry (VM) <juterry@microsoft.com>
Signed-off-by: Justin Terry (VM) <juterry@microsoft.com>
@jterry75 jterry75 force-pushed the jterry75/containerd-runhcs-shim_fixes branch from a9678b0 to 56a3adc Compare August 15, 2018 21:46
@jterry75
Copy link
Contributor Author

@jhowardmsft - PTAL

Copy link
Contributor

@lowenna lowenna left a comment

Choose a reason for hiding this comment

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

LGTM

@lowenna lowenna merged commit c328a0e into master Aug 17, 2018
@lowenna lowenna deleted the jterry75/containerd-runhcs-shim_fixes branch August 17, 2018 17:30
@crosbymichael
Copy link

Nice, thanks!

dcantah pushed a commit to dcantah/hcsshim that referenced this pull request Mar 17, 2021
Change HotRemove NIC to warning instead of error
princepereira pushed a commit to princepereira/hcsshim that referenced this pull request Aug 29, 2024
…unhcs-shim_fixes

Jterry75/containerd runhcs shim fixes
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.

3 participants