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

Update ContainerState.Termination to ContainerState.Terminated #8923

Merged
merged 2 commits into from May 29, 2015

Conversation

caesarxuchao
Copy link
Member

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
Copy link
Member Author

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

@thockin
Copy link
Member

thockin commented May 28, 2015

Negative on CI

@caesarxuchao
Copy link
Member 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 notifications@github.com 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
Copy link
Member 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.

None yet

6 participants