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

Use prune option in Pygit2 provider when fetching #49984

Merged
merged 2 commits into from
Oct 18, 2018

Conversation

chotea
Copy link
Contributor

@chotea chotea commented Oct 10, 2018

Pygit2 version 0.26.2 added support for pruning when fetching.
In this way Pygit2 provider will no longer need to leverage
git command line utility for pruning the remote.

Signed-off-by: Cristian Hotea [email protected]

Tests written?

No

Commits signed with GPG?

No

Copy link
Contributor

@terminalmage terminalmage left a comment

Choose a reason for hiding this comment

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

I like this, but I'm trying to move away from version checks when I can, the reason being that if the versioning ends up being determined via some sort of git describe logic (for those installing pygit2 from the upstream project's master branch), it may interfere with this kind of version check. Here are some recommended changes that can help us avoid explicit version checking and instead rely on the attributes we know should be available.

Thanks very much!

salt/utils/gitfs.py Show resolved Hide resolved
salt/utils/gitfs.py Outdated Show resolved Hide resolved
salt/utils/gitfs.py Outdated Show resolved Hide resolved
Pygit2 version 0.26.2 added support for pruning when fetching.
In this way Pygit2 provider will no longer need to leverage
git commanand line utility for pruning the remote.

Signed-off-by: Cristian Hotea <[email protected]>
@chotea chotea force-pushed the gitfs_pygit2_prune branch from d8af165 to f480fea Compare October 11, 2018 09:55
@cachedout
Copy link
Contributor

@terminalmage Re-review requested.

@rallytime rallytime merged commit b87bfd3 into saltstack:develop Oct 18, 2018
@Ch3LL Ch3LL added the bugfix-bckport will be be back-ported to an older release branch by creating a PR against that branch label Apr 10, 2019
@Ch3LL Ch3LL added ZZZ[Done]-back-ported-bf RETIRED The pull request has been back-ported to an older branch. and removed bugfix-bckport will be be back-ported to an older release branch by creating a PR against that branch labels Apr 10, 2019
dwoz added a commit that referenced this pull request Apr 10, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ZZZ[Done]-back-ported-bf RETIRED The pull request has been back-ported to an older branch.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants