Skip to content

Fix typo#34

Closed
Manozco wants to merge 1 commit into
opencontainers:masterfrom
Manozco:master
Closed

Fix typo#34
Manozco wants to merge 1 commit into
opencontainers:masterfrom
Manozco:master

Conversation

@Manozco
Copy link
Copy Markdown

@Manozco Manozco commented Jun 25, 2015

No description provided.

@mrunalp
Copy link
Copy Markdown
Contributor

mrunalp commented Jun 26, 2015

Thanks for the PR. Closing as this was fixed in #33

@mrunalp mrunalp closed this Jun 26, 2015
stefanberger pushed a commit to stefanberger/runc that referenced this pull request Sep 8, 2017
hooks: start with pre-start and post-stop hooks.
stefanberger pushed a commit to stefanberger/runc that referenced this pull request Sep 8, 2017
The language from 15dee2e (runtime: Add prestart/poststop hooks,
2015-08-03, opencontainers#34) landed well before we had glossary entries for the
runtime and container namespaces (from 5dad125, config-linux: Specify
host mount namespace for namespace paths, 2015-12-18, opencontainers#275).  Now that
we do have language to cover that concept, it's better to explicitly
say that hooks run in the runtime namespace instead of leaving it to
the reader to extrapolate from the filesystem requirement.

With the new namespace wording, the "host's filesystem" wording is
somewhat redundant.  I've left it in though, because I think it helps
to have a more gradual transition from hook paths to namespaces.

Signed-off-by: W. Trevor King <wking@tremily.us>
haircommander pushed a commit to haircommander/runc that referenced this pull request Apr 15, 2021
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.

2 participants