Skip to content

Add GC controls, e2e regexes (v4 branch)#496

Merged
k8s-ci-robot merged 3 commits intokubernetes:masterfrom
thockin:master
Mar 8, 2022
Merged

Add GC controls, e2e regexes (v4 branch)#496
k8s-ci-robot merged 3 commits intokubernetes:masterfrom
thockin:master

Conversation

@thockin
Copy link
Copy Markdown
Member

@thockin thockin commented Feb 24, 2022

Fixes #439.

This does not solve the timeout problem, and it does not do ideal repo-packing, but that is bigger and will have to come in v4 only.

Set gc.autoDetach to "false"

Add --git-gc flag to control GC on each sync
Values:

"auto" - run git gc --auto (default, respects git gc.* configs)
"always" - run git gc
"aggressive" - run git gc --aggressive (may require a longer timeout)
"off" - do not run git gc on each sync (good for --one-time use)

Also makes e2e test treat args as regexes.

In particular, this sets `gc.autoDetach` to "false".
@k8s-ci-robot k8s-ci-robot added do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. size/L Denotes a PR that changes 100-499 lines, ignoring generated files. labels Feb 24, 2022
@k8s-ci-robot k8s-ci-robot added the approved Indicates a PR has been approved by an approver from all required OWNERS files. label Feb 24, 2022
Values:
* "auto" - run `git gc --auto` (default, respects git gc.* configs)
* "always" - run `git gc`
* "aggressive" - run `git gc --aggressive` (may require a longer timeout)
* "off" - do not run `git gc` on each sync (good for --one-time use)
@thockin thockin changed the title WIP: Add GC controls (v4 branch) Add GC controls (v4 branch) Feb 24, 2022
@k8s-ci-robot k8s-ci-robot removed the do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. label Feb 24, 2022
@thockin
Copy link
Copy Markdown
Member Author

thockin commented Feb 24, 2022

OK, I think this is ready for review.

@thockin thockin changed the title Add GC controls (v4 branch) Add GC controls, e2e regexes (v4 branch) Feb 24, 2022
@thockin
Copy link
Copy Markdown
Member Author

thockin commented Mar 4, 2022

ping on this if anyone cares

@stp-ip
Copy link
Copy Markdown
Member

stp-ip commented Mar 6, 2022

On my radar ;)

Copy link
Copy Markdown
Member

@stp-ip stp-ip left a comment

Choose a reason for hiding this comment

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

/lgtm

Readme out of sync on this one as well.

@k8s-ci-robot k8s-ci-robot added the lgtm "Looks good to me", indicates that a PR is ready to be merged. label Mar 8, 2022
@k8s-ci-robot
Copy link
Copy Markdown
Contributor

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: stp-ip, thockin

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Details Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@k8s-ci-robot k8s-ci-robot merged commit e16e919 into kubernetes:master Mar 8, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

approved Indicates a PR has been approved by an approver from all required OWNERS files. cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. lgtm "Looks good to me", indicates that a PR is ready to be merged. size/L Denotes a PR that changes 100-499 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants