Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Conversation

caesarxuchao
Copy link
Contributor

Update ContainerState.Termination to ContainerState.Terminated in pkg/api/types.go and pkg/api/v1/types.go
#7018 #6979

@krousey @bgrant0607

@dchen1107
Copy link
Member

Thanks for fixing this for me. :-)

@bgrant0607 bgrant0607 self-assigned this May 28, 2015
@bgrant0607
Copy link
Member

LGTM

@bgrant0607 bgrant0607 added the lgtm "Looks good to me", indicates that a PR is ready to be merged. label May 28, 2015
@thockin
Copy link
Member

thockin commented May 28, 2015

needs rebase

@caesarxuchao caesarxuchao force-pushed the TerminationToTerminated branch from b38213c to 6ac41ac Compare May 28, 2015 23:14
@caesarxuchao
Copy link
Contributor Author

Rebased. Let's see if it still passess all tests.

@thockin
Copy link
Member

thockin commented May 28, 2015

Negative on CI

@caesarxuchao caesarxuchao force-pushed the TerminationToTerminated branch from 6ac41ac to 813b7a8 Compare May 28, 2015 23:30
@caesarxuchao
Copy link
Contributor Author

It's because the Goimports on my machine randomly removes actual dependencies. I can't fix it. Have you ever met similar problems?

@thockin
Copy link
Member

thockin commented May 28, 2015

Fix your GOPATH

On Thu, May 28, 2015 at 4:33 PM, Chao Xu [email protected] wrote:

It's because the Goimports on my machine randomly removes actual
dependencies. I can't fix it. Have you ever met similar problems?


Reply to this email directly or view it on GitHub
#8923 (comment)
.

@caesarxuchao caesarxuchao force-pushed the TerminationToTerminated branch from 813b7a8 to 671ff07 Compare May 28, 2015 23:57
@caesarxuchao caesarxuchao force-pushed the TerminationToTerminated branch from 671ff07 to 9f5e0b6 Compare May 29, 2015 00:20
@caesarxuchao
Copy link
Contributor Author

Shippable is green

rjnagal added a commit that referenced this pull request May 29, 2015
Update ContainerState.Termination to ContainerState.Terminated
@rjnagal rjnagal merged commit 233ce46 into kubernetes:master May 29, 2015
@caesarxuchao caesarxuchao deleted the TerminationToTerminated branch May 12, 2020 23:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
lgtm "Looks good to me", indicates that a PR is ready to be merged.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants