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

Test pct using shallow clone #1613

Closed
wants to merge 9 commits into from
Closed

Conversation

olamy
Copy link
Member

@olamy olamy commented Nov 30, 2022

@olamy olamy changed the title Use incremental build from https://github.com/jenkinsci/plugin-compat-tester/pull/382 Use incremental build from https://github.com/jenkinsci/plugin-compat-tester/pull/382 to test shallow clone Nov 30, 2022
@MarkEWaite MarkEWaite added the full-test Test all LTS lines in this PR and do not halt upon first error. label Nov 30, 2022
@jglick jglick marked this pull request as draft December 1, 2022 17:59
@jglick jglick changed the title Use incremental build from https://github.com/jenkinsci/plugin-compat-tester/pull/382 to test shallow clone Test shallow clone Dec 1, 2022
@olamy olamy changed the title Test shallow clone Test pct using shallow clone Dec 2, 2022
MarkEWaite added a commit to MarkEWaite/git-client-plugin that referenced this pull request Dec 5, 2022
Do not attempt to use the local repository as a reference repository
when the local repository is a shallow clone.  A shallow clone
repository has no content to offer as a reference.

Fixes 64 test failures when the test repository is a shallow clone.

jenkinsci/bom#1613 describes the experiment to
run the plugin compatibility tests with shallow clones.

jenkinsci/git-plugin#1365 implemented a similar
change in the git plugin so that shallow cloned repositories will not
fail the tests in that plugin.

Will make a few tests in a shallow repository a little slower because
they will require a full clone of the git client plugin repository
rather than reusing the existing clone in the test workspace.  Doubtful
that the performance difference will be detectable within the general
variability of CI test jobs.
MarkEWaite added a commit to jenkinsci/git-client-plugin that referenced this pull request Dec 5, 2022
Do not attempt to use the local repository as a reference repository
when the local repository is a shallow clone.  A shallow clone
repository has no content to offer as a reference.

Fixes 64 test failures when the test repository is a shallow clone.

jenkinsci/bom#1613 describes the experiment to
run the plugin compatibility tests with shallow clones.

jenkinsci/git-plugin#1365 implemented a similar
change in the git plugin so that shallow cloned repositories will not
fail the tests in that plugin.

Will make a few tests in a shallow repository a little slower because
they will require a full clone of the git client plugin repository
rather than reusing the existing clone in the test workspace.  Doubtful
that the performance difference will be detectable within the general
variability of CI test jobs.
@MarkEWaite
Copy link
Contributor

Git plugin 4.14.2 and git client plugin 3.13.1 pass all their tests when run from a shallow clone. The special case code for git plugin and git client plugin can be removed.

MarkEWaite added a commit to jenkinsci/git-client-plugin that referenced this pull request Dec 13, 2022
Do not attempt to use the local repository as a reference repository
when the local repository is a shallow clone.  A shallow clone
repository has no content to offer as a reference.

Fixes 64 test failures when the test repository is a shallow clone.

jenkinsci/bom#1613 describes the experiment to
run the plugin compatibility tests with shallow clones.

jenkinsci/git-plugin#1365 implemented a similar
change in the git plugin so that shallow cloned repositories will not
fail the tests in that plugin.

Will make a few tests in a shallow repository a little slower because
they will require a full clone of the git client plugin repository
rather than reusing the existing clone in the test workspace.  Doubtful
that the performance difference will be detectable within the general
variability of CI test jobs.

(cherry picked from commit 5c98434)
MarkEWaite added a commit to MarkEWaite/bom that referenced this pull request Dec 13, 2022
https://issues.jenkins.io/browse/JENKINS-70174 notes that the git client
plugin automated tests failed when using a shallow clone (depth == 1)
of a single branch.  The issue was fixed for Jenkins 2.346.3 and newer
with the release of git client plugin 3.13.1.  This release is a backport
of that fix.

No plugin functionality was changed in this release.  It changes the
automated tests when run from a shallow clone (depth == 1) of a single
branch.

jenkinsci#1613 is the key consumer in the
plugin bill of materials.

jenkinsci/plugin-compat-tester#382 is the key
consumer in the plugin compatibility tester.
jglick pushed a commit that referenced this pull request Dec 14, 2022
https://issues.jenkins.io/browse/JENKINS-70174 notes that the git client
plugin automated tests failed when using a shallow clone (depth == 1)
of a single branch.  The issue was fixed for Jenkins 2.346.3 and newer
with the release of git client plugin 3.13.1.  This release is a backport
of that fix.

No plugin functionality was changed in this release.  It changes the
automated tests when run from a shallow clone (depth == 1) of a single
branch.

#1613 is the key consumer in the
plugin bill of materials.

jenkinsci/plugin-compat-tester#382 is the key
consumer in the plugin compatibility tester.
@olamy olamy marked this pull request as ready for review December 20, 2022 08:41
@jglick jglick marked this pull request as draft December 21, 2022 16:58
@olamy
Copy link
Member Author

olamy commented Jan 19, 2023

closing it as jenkinsci/plugin-compat-tester#382 has been closed.

@olamy olamy closed this Jan 19, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
full-test Test all LTS lines in this PR and do not halt upon first error.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants