Skip to content
This repository has been archived by the owner on Jul 30, 2021. It is now read-only.

checkpointer: bump default checkpointer version. #826

Merged
merged 1 commit into from
Jan 8, 2018

Conversation

diegs
Copy link
Contributor

@diegs diegs commented Jan 8, 2018

Also use new flag (in this checkpointer version) to vastly shorten the
grace period in the checkpoint tests.

@diegs diegs self-assigned this Jan 8, 2018
@diegs diegs requested a review from rphillips January 8, 2018 19:04
@k8s-ci-robot k8s-ci-robot added size/XS Denotes a PR that changes 0-9 lines, ignoring generated files. cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. labels Jan 8, 2018
@@ -448,6 +448,7 @@ spec:
image: %s
command:
- /checkpoint
- --checkpoint-grace-period=5s
Copy link
Contributor

Choose a reason for hiding this comment

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

tabs look off

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Thanks! Fixed

Also use new flag (in this checkpointer version) to vastly shorten the
grace period in the checkpoint tests.
@@ -448,6 +448,7 @@ spec:
image: %s
command:
- /checkpoint
- --checkpoint-grace-period=5s
Copy link
Contributor

Choose a reason for hiding this comment

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

whitespace issue

@rphillips
Copy link
Contributor

/lgtm assuming the tests pass

@diegs
Copy link
Contributor Author

diegs commented Jan 8, 2018

coreosbot run e2e

@diegs
Copy link
Contributor Author

diegs commented Jan 8, 2018

Flake I think, ref #824

@diegs
Copy link
Contributor Author

diegs commented Jan 8, 2018

coreosbot run e2e

@diegs diegs merged commit 1bf2017 into kubernetes-retired:master Jan 8, 2018
@diegs diegs deleted the cp branch January 8, 2018 21:52
dghubble added a commit to poseidon/terraform-render-bootstrap that referenced this pull request Feb 15, 2018
* No notable changes except a grace period flag we don't use
* kubernetes-retired/bootkube#826
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. size/XS Denotes a PR that changes 0-9 lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants