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

Not exec a container from stopped state #880

Merged
merged 1 commit into from
Aug 27, 2016

Conversation

rajasec
Copy link
Contributor

@rajasec rajasec commented Jun 4, 2016

Do not allow to exec an container from stopped state, which is similar to start.

Signed-off-by: rajasec rajasec79@gmail.com

Signed-off-by: rajasec <rajasec79@gmail.com>
@crosbymichael
Copy link
Member

Can you rebase please?

@rajasec
Copy link
Contributor Author

rajasec commented Aug 24, 2016

Rebased.

@crosbymichael
Copy link
Member

crosbymichael commented Aug 25, 2016

LGTM

Approved with PullApprove

1 similar comment
@hqhq
Copy link
Contributor

hqhq commented Aug 27, 2016

LGTM

Approved with PullApprove

@hqhq hqhq merged commit face64a into opencontainers:master Aug 27, 2016
@rajasec rajasec deleted the exec-status branch September 11, 2016 02:26
stefanberger pushed a commit to stefanberger/runc that referenced this pull request Sep 8, 2017
…pabilities-again

rebase: config: Make capabilities, noNewPrivileges, and rlimits Linux-only (again)
stefanberger pushed a commit to stefanberger/runc that referenced this pull request Sep 8, 2017
Through f4d221c (Merge pull request opencontainers#880 from
dqminh/wking-linux-only-capabilities-again, 2017-07-05).  The rc6
release picked up an earlier version of these notes, and those entries
are mostly unchanged except for:

* The credentialSpec entry, which was opencontainers#814 for credentialspec and now
  also includes opencontainers#859 for credentialSpec.

* The root(.path) Hyper-V entry, which was opencontainers#820 for root.path and now
  also includes opencontainers#838 for root.  I also moved this into the "breaking
  changes" section, because rc5 Hyper-V configs required root to be
  set, and rc6 Hyper-V configs require it to not be set.  Although
  whether rc5 allowed Hyper-V configs at all is not clear to me.

* Fixed indenting for the typo-fixes entry, as well as a number of
  more recent typo-fix PRs.

Signed-off-by: W. Trevor King <wking@tremily.us>
stefanberger pushed a commit to stefanberger/runc that referenced this pull request Sep 8, 2017
Fixing a typo I'd made in 5292e9c (config: Make rlimits
POSIX-specific, 2017-05-23, opencontainers#880).

Signed-off-by: W. Trevor King <wking@tremily.us>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants