Skip to content
This repository has been archived by the owner on May 12, 2021. It is now read-only.

virtcontainers: rename GetOCISpec to GetPatchedOCISpec #2253

Merged

Conversation

devimc
Copy link

@devimc devimc commented Nov 22, 2019

GetOCISpec returns a patched version of the original OCI spec, it was modified
to support:

  • capabilities
  • Ephemeral storage
  • k8s empty dir

In order to avoid consusions and make api clear, rename GetOCISpec
to GetPatchedOCISpec.

fixes #2252

Signed-off-by: Julio Montes julio.montes@intel.com

@devimc devimc force-pushed the topic/virtcontainers/GetPatchedOCI branch from cfa642d to 3e5f70c Compare November 22, 2019 19:33
@devimc
Copy link
Author

devimc commented Nov 22, 2019

/test-ubuntu

@devimc devimc force-pushed the topic/virtcontainers/GetPatchedOCI branch from 3e5f70c to 5055463 Compare November 22, 2019 21:14
@devimc
Copy link
Author

devimc commented Nov 22, 2019

/test-ubuntu

Copy link
Contributor

@jodh-intel jodh-intel left a comment

Choose a reason for hiding this comment

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

Thanks @devimc.

lgtm

GetOCISpec returns a patched version of the original OCI spec, it was modified
to support:
* capabilities
* Ephemeral storage
* k8s empty dir

In order to avoid consusions and make api clear, rename GetOCISpec
to GetPatchedOCISpec and ContainerConfig.Spec to ContainerConfig.CustomSpec

fixes kata-containers#2252

Signed-off-by: Julio Montes <julio.montes@intel.com>
@devimc devimc force-pushed the topic/virtcontainers/GetPatchedOCI branch from 5055463 to 613fd0f Compare November 25, 2019 17:23
@devimc
Copy link
Author

devimc commented Nov 25, 2019

/test-ubuntu

@jodh-intel
Copy link
Contributor

Thanks @devimc

@codecov
Copy link

codecov bot commented Nov 25, 2019

Codecov Report

❗ No coverage uploaded for pull request base (master@dd15db3). Click here to learn what that means.
The diff coverage is 88.88%.

@@            Coverage Diff            @@
##             master    #2253   +/-   ##
=========================================
  Coverage          ?   48.76%           
=========================================
  Files             ?      111           
  Lines             ?    16086           
  Branches          ?        0           
=========================================
  Hits              ?     7845           
  Misses            ?     7266           
  Partials          ?      975

@amshinde amshinde merged commit 545d61c into kata-containers:master Nov 25, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Rename GetOCISpec
3 participants