Keep backwards compatibility in kill api. by calavera · Pull Request #15098 · moby/moby · GitHub
Skip to content

Keep backwards compatibility in kill api. - #15098

Merged
icecrime merged 1 commit into
moby:masterfrom
calavera:backwards_compat_kill_error
Jul 29, 2015
Merged

icecrime merged 1 commit into
moby:masterfrom
calavera:backwards_compat_kill_error

Conversation

@calavera

@calavera calavera commented Jul 28, 2015 •

Copy link
Copy Markdown
Contributor

Return an error when the container is stopped only in api versions
equal or greater than 1.20 (docker 1.8).

Signed-off-by: David Calavera david.calavera@gmail.com

@thaJeztah

Copy link
Copy Markdown
Member

Comment thread integration-cli/docker_cli_kill_test.go Outdated

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

instead of OldApi, can we specify either 119 or Pre120

@cpuguy83

Copy link
Copy Markdown
Member

@thaJeztah issue number now refers to this PR

LGTM other than being specific about API versions in the test name.

@thaJeztah

Copy link
Copy Markdown
Member

@cpuguy83 oh darn! the previous number was a non existing issue, but was 10000 off of the PR that introduced this. Probably had the wrong issue on my clipboard, I'll search the right one.

@thaJeztah

Copy link
Copy Markdown
Member

third time's a charm, think I got the right one now. Thanks Brian

Return an error when the container is stopped only in api versions
equal or greater than 1.20 (docker 1.8).

Signed-off-by: David Calavera <david.calavera@gmail.com>
@calavera
calavera force-pushed the backwards_compat_kill_error branch from b58f9c7 to 621e3d8 Compare July 28, 2015 19:25
@calavera

Copy link
Copy Markdown
Contributor Author

Test name fixed. Thank you guys!

@thaJeztah

Copy link
Copy Markdown
Member

LGTM (jankies are running)

@icecrime

Copy link
Copy Markdown
Contributor

LGTM

@crosbymichael

Copy link
Copy Markdown
Contributor

LGTM

@icecrime

Copy link
Copy Markdown
Contributor

Green janky: merging.

icecrime pushed a commit that referenced this pull request Jul 29, 2015
Keep backwards compatibility in kill api.
@icecrime
icecrime merged commit 4a71323 into moby:master Jul 29, 2015
@calavera
calavera deleted the backwards_compat_kill_error branch July 29, 2015 23:43
@calavera

Copy link
Copy Markdown
Contributor Author

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.

Error when attempting to kill a non-running container breaks backwards compatibility

6 participants