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.

Sign up for GitHub

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

Jump to bottom

Fixed PV recycler ActiveDeadlineSeconds precision #10152

Merged
merged 1 commit into from
Jun 22, 2015
Merged

Fixed PV recycler ActiveDeadlineSeconds precision #10152

merged 1 commit into from
Jun 22, 2015

Conversation

markturansky
Copy link
Contributor

@k8s-bot
Copy link

k8s-bot commented Jun 20, 2015

GCE e2e build/test passed for commit 39c5152.

@derekwaynecarr
Copy link
Member

Much better :-)

Lgtm

Sent from my iPhone

On Jun 20, 2015, at 12:36 AM, Mark Turansky notifications@github.com wrote:

@derekwaynecarr

You can view, comment on, or merge this pull request online at:

#10152

Commit Summary

fixed pv recycler timeout precision
File Changes

M pkg/volume/host_path/host_path.go (3)
M pkg/volume/nfs/nfs.go (3)
Patch Links:

https://github.com/GoogleCloudPlatform/kubernetes/pull/10152.patch
https://github.com/GoogleCloudPlatform/kubernetes/pull/10152.diff
—
Reply to this email directly or view it on GitHub.

@derekwaynecarr derekwaynecarr self-assigned this Jun 20, 2015
@derekwaynecarr derekwaynecarr added the lgtm "Looks good to me", indicates that a PR is ready to be merged. label Jun 20, 2015
j3ffml added a commit that referenced this pull request Jun 22, 2015
@j3ffml
Merge pull request #10152 from markturansky/pv_recycler_precision_fix
cbf0a4f 
Fixed PV recycler ActiveDeadlineSeconds precision
@j3ffml j3ffml merged commit cbf0a4f into kubernetes:master Jun 22, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees

@derekwaynecarr derekwaynecarr

Labels
lgtm "Looks good to me", indicates that a PR is ready to be merged.
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
5 participants
@markturansky @k8s-bot @derekwaynecarr @j3ffml @googlebot