Skip to content

Remove LCOW (step 9): layer: remove layerstore.setOS(), layerstore.getOS()#43362

Merged
thaJeztah merged 1 commit intomoby:masterfrom
thaJeztah:remove_lcow_setos_getos
Mar 16, 2022
Merged

Remove LCOW (step 9): layer: remove layerstore.setOS(), layerstore.getOS()#43362
thaJeztah merged 1 commit intomoby:masterfrom
thaJeztah:remove_lcow_setos_getos

Conversation

@thaJeztah
Copy link
Copy Markdown
Member

Relates to #39533 / #42170

Found this one on my computer, and looks like I didn't have a PR for this 😂

This removes the setOS() / getOS() functions from the layer store, which were added in fc21bf2 (#33241) and 0380fbf (#34642) in support of LCOW.

- A picture of a cute animal (not mandatory but encouraged)

This removes the `setOS()` / `getOS()` functions from the layer store, which were
added in fc21bf2 and 0380fbf
in support of LCOW.

Signed-off-by: Sebastiaan van Stijn <github@gone.nl>
@thaJeztah thaJeztah added status/2-code-review area/lcow Issues and PR's related to the experimental LCOW feature kind/refactor PR's that refactor, or clean-up code labels Mar 11, 2022
@thaJeztah thaJeztah added this to the 21.xx milestone Mar 11, 2022
@thaJeztah
Copy link
Copy Markdown
Member Author

@tonistiigi PTAL

@thaJeztah
Copy link
Copy Markdown
Member Author

Thanks for review; let's get this one in 👍

@thaJeztah thaJeztah merged commit 45e1f59 into moby:master Mar 16, 2022
@thaJeztah thaJeztah deleted the remove_lcow_setos_getos branch March 16, 2022 15:01
@thaJeztah thaJeztah mentioned this pull request Jul 12, 2023
15 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area/lcow Issues and PR's related to the experimental LCOW feature kind/refactor PR's that refactor, or clean-up code status/2-code-review

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants