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

Rename GetOCISpec #2252

Closed
devimc opened this issue Nov 22, 2019 · 0 comments · Fixed by #2253
Closed

Rename GetOCISpec #2252

devimc opened this issue Nov 22, 2019 · 0 comments · Fixed by #2253
Assignees
Labels
enhancement Improvement to an existing feature needs-review Needs to be assessed by the team.

Comments

@devimc
Copy link

devimc commented Nov 22, 2019

When I was debugging an issue in kata-runtime, it was difficult to realize that GetOCISpec method returns a patched version of the original spec, to avoid confusions we should rename it to something less generic

@devimc devimc added enhancement Improvement to an existing feature needs-review Needs to be assessed by the team. labels Nov 22, 2019
@devimc devimc self-assigned this Nov 22, 2019
devimc pushed a commit to devimc/kata-runtime that referenced this issue 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 kata-containers#2252

Signed-off-by: Julio Montes <julio.montes@intel.com>
devimc pushed a commit to devimc/kata-runtime that referenced this issue 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 kata-containers#2252

Signed-off-by: Julio Montes <julio.montes@intel.com>
devimc pushed a commit to devimc/kata-runtime that referenced this issue 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 kata-containers#2252

Signed-off-by: Julio Montes <julio.montes@intel.com>
devimc pushed a commit to devimc/kata-runtime that referenced this issue 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 kata-containers#2252

Signed-off-by: Julio Montes <julio.montes@intel.com>
devimc pushed a commit to devimc/kata-runtime that referenced this issue Nov 25, 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 and ContainerConfig.Spec to ContainerConfig.CustomSpec

fixes kata-containers#2252

Signed-off-by: Julio Montes <julio.montes@intel.com>
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
enhancement Improvement to an existing feature needs-review Needs to be assessed by the team.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant