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

Use "hcsschema" in internal/hcs #1901

Merged
merged 1 commit into from
Oct 18, 2023
Merged

Conversation

helsaawy
Copy link
Contributor

Use hcsschema.ProcessModifyRequest (and associated structs) in internal/hcs/process when making modify requests and getting process properties.

Update hcschema structs to match documentation (swagger seems to default to int32 for generated fields regardless of the type specified in the documentation).

Use `hcsschema.ProcessModifyRequest` (and associated structs) in
`internal/hcs/process` when making modify requests and getting process
properties.

Update `hcschema` structs to match documentation (swagger seems to
default to `int32` for generated fields regardless of the type specified
in the documentation).

Signed-off-by: Hamza El-Saawy <[email protected]>
@helsaawy helsaawy requested a review from a team as a code owner September 14, 2023 18:26
@helsaawy helsaawy merged commit cff9473 into microsoft:main Oct 18, 2023
@helsaawy helsaawy deleted the hcs-modify-request branch October 18, 2023 16:15
darracott pushed a commit to KenGordon/hcsshim that referenced this pull request Oct 30, 2023
Use `hcsschema.ProcessModifyRequest` (and associated structs) in
`internal/hcs/process` when making modify requests and getting process
properties.

Update `hcschema` structs to match documentation (swagger seems to
default to `int32` for generated fields regardless of the type specified
in the documentation).

Signed-off-by: Hamza El-Saawy <[email protected]>
princepereira pushed a commit to princepereira/hcsshim that referenced this pull request Aug 29, 2024
Use `hcsschema.ProcessModifyRequest` (and associated structs) in
`internal/hcs/process` when making modify requests and getting process
properties.

Update `hcschema` structs to match documentation (swagger seems to
default to `int32` for generated fields regardless of the type specified
in the documentation).

Signed-off-by: Hamza El-Saawy <[email protected]>
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