-
Notifications
You must be signed in to change notification settings - Fork 393
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
[JENKINS-64383] combined refrepo became our bottleneck, support a fanout location too #644
Open
jimklimov
wants to merge
147
commits into
jenkinsci:master
Choose a base branch
from
jimklimov:refrepo-args
base: master
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
Changes from all commits
Commits
Show all changes
147 commits
Select commit
Hold shift + click to select a range
8f5ec33
Introduce support for reference repository paths ending with /${GIT_U…
jimklimov 022bbc6
Refactor: move findParameterizedReferenceRepository() into LegacyComp…
jimklimov 6210adb
Add isParameterizedReferenceRepository() and support for parameterize…
jimklimov 06d4859
Handle the possibility that the URL for parameterized checkout is nul…
jimklimov b5a6a7b
LegacyCompatibleGitAPIImpl.java: document params for new routines
jimklimov f90bdca
LegacyCompatibleGitAPIImpl.java: refactor urlNormalized generation in…
jimklimov e09eff1
LegacyCompatibleGitAPIImpl.java: introduce support for .../${GIT_URL_…
jimklimov 17010ca
WorkspaceWithRepo.java (test): extend localMirror() with customizable…
jimklimov 72a4a05
GitClientCloneTest.java: add test_clone_reference_parameterized_sha25…
jimklimov c6c010f
LegacyCompatibleGitAPIImpl.java: findParameterizedReferenceRepository…
jimklimov bbedefd
Fix referencePath getName() => getPath()
jimklimov 5306163
Update tests for fRefrepoBase being under target/ dir from CWD of the…
jimklimov bc7e198
GitClientCloneTest: check if refrepo dir exists already
jimklimov 3ff19df
WorkspaceWithRepo.java: localMirror(): normalize() the returned pathn…
jimklimov a509386
GitClientCloneTest.java: extend API choices for assertAlternateFilePo…
jimklimov 603e390
GitClientCloneTest.java: fix test_clone_reference_parameterized_sha25…
jimklimov 66e23a1
LegacyCompatibleGitAPIImpl.java CliGitAPIImpl.java JGitAPIImpl.java: …
jimklimov 5af4400
LegacyCompatibleGitAPIImpl.java: add variants of findParameterizedRef…
jimklimov 509c5ec
[TMP] LegacyCompatibleGitAPIImpl.java: trace findParameterizedReferen…
jimklimov 0b0fcbf
[TMP] WorkspaceWithRepo.java (test): trace localMirror() processing t…
jimklimov 6a9c79d
LegacyCompatibleGitAPIImpl.java: refactor to use referenceExpanded to…
jimklimov 2641971
LegacyCompatibleGitAPIImpl.java: add GIT_URL_FALLBACK and GIT_URL_SHA…
jimklimov c10231c
GitClientCloneTest.java: add cases for parameterized reference repo w…
jimklimov 2016719
LegacyCompatibleGitAPIImpl.java: refactor normalizeGitUrl()
jimklimov 71c0850
[WIP] LegacyCompatibleGitAPIImpl.java: skeleton for GIT_SUBMODULES an…
jimklimov ab04ae7
Introduce getRemoteUrls() and getRemotePushUrls()
jimklimov 179f5c1
LegacyCompatibleGitAPIImpl.java: normalizeGitUrl(): revise for local …
jimklimov 7c0ad4f
[WIP] LegacyCompatibleGitAPIImpl.java: skeleton for GIT_SUBMODULES an…
jimklimov ead03a6
CliGitAPIImpl.java: extend getRemote{,Push}Urls() with filters for AC…
jimklimov 5fde3c9
GitClientCloneTest.java: if we expect particular wsRefrepo spelling, …
jimklimov 6c7374c
GitClientCloneTest.java: for fallback case, expect wsRefrepoBase itself
jimklimov a2cf84f
GitClientCloneTest.java: use the LegacyCompatibleGitAPIImpl.normalize…
jimklimov 823bd0f
GitClientCloneTest.java: for fallback case, expect wsRefrepoBase itse…
jimklimov 0e5e0cd
LegacyCompatibleGitAPIImpl.java: skeleton for getSubmodulesUrls() as …
jimklimov 172e40e
LegacyCompatibleGitAPIImpl.java: update comments for ideas about subm…
jimklimov d6cebe8
LegacyCompatibleGitAPIImpl.java: getSubmodulesUrls(): implement the s…
jimklimov 864e34f
LegacyCompatibleGitAPIImpl.java: move arrDirNames declaration higher …
jimklimov 55fda14
LegacyCompatibleGitAPIImpl.java: update commented thoughts
jimklimov 3e81723
LegacyCompatibleGitAPIImpl.java: prepare to parse submodule workspaces
jimklimov ff65341
LegacyCompatibleGitAPIImpl.java: uncomment needle tracking in a loop …
jimklimov 122746f
LegacyCompatibleGitAPIImpl.java: fix @param usage
jimklimov bf978ee
LegacyCompatibleGitAPIImpl.java: implement handling output of getSubm…
jimklimov aacc0cd
LegacyCompatibleGitAPIImpl.java: getSubmodulesUrls() / findParameteri…
jimklimov 1a8306a
LegacyCompatibleGitAPIImpl.java: findParameterizedReferenceRepository…
jimklimov 769a2d4
LegacyCompatibleGitAPIImpl.java: log-tracing for getSubmodulesUrls()
jimklimov 654362a
LegacyCompatibleGitAPIImpl.java: for now look in subdirs regardless o…
jimklimov 5b6ff6d
LegacyCompatibleGitAPIImpl.java: comment updated
jimklimov 8d8514c
LegacyCompatibleGitAPIImpl.java: rename getObjectPath() to less misle…
jimklimov 925eaba
LegacyCompatibleGitAPIImpl.java: in findParameterizedReferenceReposit…
jimklimov a103bdd
LegacyCompatibleGitAPIImpl.java: in findParameterizedReferenceReposit…
jimklimov aaaef9c
LegacyCompatibleGitAPIImpl.java: in findParameterizedReferenceReposit…
jimklimov a77dab9
LegacyCompatibleGitAPIImpl.java: in findParameterizedReferenceReposit…
jimklimov d664eb3
LegacyCompatibleGitAPIImpl.java: pass not-normalized URL from findPar…
jimklimov e29d947
LegacyCompatibleGitAPIImpl.java: check also not-normalized URL in fin…
jimklimov 1816632
LegacyCompatibleGitAPIImpl.java: only add ".git" to reference if it i…
jimklimov 8b693fd
LegacyCompatibleGitAPIImpl.java: fix tracing into private LOGGER inst…
jimklimov e49b985
LegacyCompatibleGitAPIImpl.java: trace looking for the needle in arrD…
jimklimov ab29c68
LegacyCompatibleGitAPIImpl.java: comment reshuffle
jimklimov 66798fd
LegacyCompatibleGitAPIImpl.java: refactor findParameterizedReferenceR…
jimklimov 776a249
LegacyCompatibleGitAPIImpl.java: trace which sub-git workspaces we do…
jimklimov 094de4d
LegacyCompatibleGitAPIImpl.java: fix getSubmodulesUrls() to explicitl…
jimklimov d30de21
LegacyCompatibleGitAPIImpl.java: fix getSubmodulesUrls() to look in "…
jimklimov 7ed0f0f
GitClient and its implementations: extend API with a newGit(somedir) …
jimklimov 14cf345
LegacyCompatibleGitAPIImpl.java: fix getSubmodulesUrls() to look in r…
jimklimov 2b61767
LegacyCompatibleGitAPIImpl.java: fix getSubmodulesUrls() to return fA…
jimklimov 1c9a8f5
LegacyCompatibleGitAPIImpl.java: fix getSubmodulesUrls() to track rel…
jimklimov a29b00c
LegacyCompatibleGitAPIImpl.java: clearer trace messages in getSubmodu…
jimklimov 25bef67
LegacyCompatibleGitAPIImpl.java: track referenceBaseDirAbs to compare…
jimklimov 39dd9b4
LegacyCompatibleGitAPIImpl.java: fix getSubmodulesUrls() to return fA…
jimklimov c349b25
LegacyCompatibleGitAPIImpl.java: comment the ends of long logical blocks
jimklimov 6a7bb18
LegacyCompatibleGitAPIImpl.java: use absolute pathnames "natively" in…
jimklimov f475606
LegacyCompatibleGitAPIImpl.java: do not need to check bare-mode dirs …
jimklimov 2ddd7df
LegacyCompatibleGitAPIImpl.java: check basename-like dirs for being g…
jimklimov 721d791
LegacyCompatibleGitAPIImpl.java: check the parent refrepo dir for hos…
jimklimov 69f5b59
LegacyCompatibleGitAPIImpl.java: trace what getRemoteUrls() gave us
jimklimov 9c48cb6
LegacyCompatibleGitAPIImpl.java: leave a TODO block for checking subm…
jimklimov f465258
LegacyCompatibleGitAPIImpl.java: trace mis-processing an assumed git …
jimklimov f6e756a
LegacyCompatibleGitAPIImpl.java: comment about abs dirnames in second…
jimklimov ce1827e
LegacyCompatibleGitAPIImpl.java: fix GitClient for abs dirnames in se…
jimklimov c5e863f
LegacyCompatibleGitAPIImpl.java: Log into build console the start and…
jimklimov d0bf23c
LegacyCompatibleGitAPIImpl.java: Refactor a useless needle to be null…
jimklimov 0b761da
LegacyCompatibleGitAPIImpl.java: Refactor getSubmodulesUrls() to unif…
jimklimov 6046f4b
LegacyCompatibleGitAPIImpl.java: Refactor getSubmodulesUrls() to name…
jimklimov 3f7b4e4
LegacyCompatibleGitAPIImpl.java: re-use arrDirnames in getSubmodulesU…
jimklimov 61a7d0b
LegacyCompatibleGitAPIImpl.java: comment the plan about submodule sea…
jimklimov b91e00b
LegacyCompatibleGitAPIImpl.java: update comments around shorter getSu…
jimklimov 65684f3
LegacyCompatibleGitAPIImpl.java: only track absolute dirname values i…
jimklimov 0c99c95
LegacyCompatibleGitAPIImpl.java: comment the plan for submodule handling
jimklimov 327cb48
LegacyCompatibleGitAPIImpl.java: finally, do recursion in getSubmodul…
jimklimov 45d268c
LegacyCompatibleGitAPIImpl.java: check not only for ".git" but also "…
jimklimov 0413d95
LegacyCompatibleGitAPIImpl.java: for GIT_SUBMODULES mode search, if t…
jimklimov fad3727
LegacyCompatibleGitAPIImpl.java: implement a use-case of ".git" file …
jimklimov d37fe49
LegacyCompatibleGitAPIImpl.java: refactor getSubmodulesUrls() to use …
jimklimov 6afacda
LegacyCompatibleGitAPIImpl.java: refactor getSubmodulesUrls() to retu…
jimklimov b278523
LegacyCompatibleGitAPIImpl.java: add logging for getObjectsFile()
jimklimov 4468443
LegacyCompatibleGitAPIImpl.java: in getObjectsFile(), comment why it …
jimklimov 0edcc0c
JGitAPIImpl.java / CliGitAPIImpl.java: inform the build console log r…
jimklimov 0767c2a
LegacyCompatibleGitAPIImpl.java: in getObjectsFile() method comment, …
jimklimov 03b3056
LegacyCompatibleGitAPIImpl.java: little clean-up in getObjectsFile()
jimklimov fd88900
LegacyCompatibleGitAPIImpl.java: referenceBaseDirAbs in getSubmodules…
jimklimov d72999b
LegacyCompatibleGitAPIImpl.java: suggest a selection of results[] *si…
jimklimov f1f478a
LegacyCompatibleGitAPIImpl.java: mark helpers as public static
jimklimov 91ed630
LegacyCompatibleGitAPIImpl.java: check if referenceBaseDir is usable …
jimklimov de98cd1
GitClientCloneTest.java: reword original test for GIT_URL into checki…
jimklimov aa9eaad
LegacyCompatibleGitAPIImpl.java: findParameterizedReferenceRepository…
jimklimov 82a43eb
LegacyCompatibleGitAPIImpl.java: findParameterizedReferenceRepository…
jimklimov 23745a8
LegacyCompatibleGitAPIImpl.java: fix spotbugs complaints: listFiles()…
jimklimov f51e60f
LegacyCompatibleGitAPIImpl.java: fix spotbugs complaints: getSubmodul…
jimklimov a5fdbd3
LegacyCompatibleGitAPIImpl.java: fix spotbugs complaints: have non-ze…
jimklimov 4341cd2
LegacyCompatibleGitAPIImpl.java: fix spotbugs complaints: "new File()…
jimklimov 4cd45c8
LegacyCompatibleGitAPIImpl.java: fix spotbugs complaints: use a file …
jimklimov fd293d7
LegacyCompatibleGitAPIImpl.java: normalizeUrl() failed with full path…
jimklimov c45cedd
LegacyCompatibleGitAPIImpl.java: normalizeUrl(): @SuppressFBWarnings …
jimklimov be57864
CliGitAPIImpl.java: launchCommandIn(): report workDir when throwing e…
jimklimov 8f843d4
LegacyCompatibleGitAPIImpl.java: getSubmodulesUrls(): do not fail che…
jimklimov 4ee6935
JGitAPIImpl.java + CliGitAPIImpl.java: report "reference" pathname th…
jimklimov e726d40
Merge branch 'master' as of 2022-08-01 into refrepo-args
jimklimov 558d076
[JENKINS-69193] GitAPITestCase: test_submodule_update_with_error(): r…
jimklimov 4402820
[JENKINS-69193] GitAPITestUpdateCliGit: rely less on strict wording o…
jimklimov 5d97abe
Merge branch 'JENKINS-69193-bis' into refrepo-args
jimklimov 4222f4e
Merge branch 'master' into refrepo-args
MarkEWaite 63ca351
Merge branch 'master' into refrepo-args
MarkEWaite 2544b99
Merge branch 'master' into refrepo-args
jimklimov 059f7a2
Merge branch 'master' into refrepo-args
MarkEWaite 7c28d4d
Merge branch 'spotless' into refrepo-args
MarkEWaite c882050
Format source changes
MarkEWaite 9863085
Fix merge mistake
MarkEWaite 8334390
Merge remote-tracking branch 'upstream/master' into refrepo-args
jimklimov eeb47b4
GitClient.java: minor fix to javadoc for subGit()
jimklimov 9f162aa
Reconcile code style with changes in the PR #644 (added by @markewait…
jimklimov d3a6d56
Mark @Override methods like in master branch
jimklimov 6840c75
Change PrintWriter use of UTF8 like in master branch
jimklimov d8ddcdb
GitClient.java: minor fix to javadoc for newGit()
jimklimov 896c2c6
LegacyCompatibleGitAPIImpl: import DigestUtils to use by short reference
jimklimov 62355d5
GitClientCloneTest: drop use of FileUtils.readFileToString() like in …
jimklimov 2c10b59
spotless is clueless
jimklimov 59e7474
Merge branch 'master' into refrepo-args
MarkEWaite 4e1f3ab
Remove redundant null check
MarkEWaite f844f23
Resolve spotbugs warnings for toString() and unread vars
MarkEWaite 43c38ba
Resolve spotbugs warnings on toLowerCase
MarkEWaite 65b55c5
Assure stream is closed on exception
MarkEWaite 9da15fa
Merge branch 'master' into refrepo-args
MarkEWaite 1f379a7
Merge branch 'master' into refrepo-args
MarkEWaite cedfd7f
Merge remote-tracking branch 'upstream/master' into refrepo-args
jimklimov b2d7fef
Merge remote-tracking branch 'upstream/master' as of 2024-10-31 into …
jimklimov 9dc373e
Merge remote-tracking branch 'upstream/master' as of 2025-01-16 into …
jimklimov d2fd099
Merge remote-tracking branch 'upstream/master' into refrepo-args
jimklimov File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There don't appear to be any tests for this new method. The method was not called from the modified tests in GitClientCloneTest.
I'd like tests that confirm the method is well-behaved. There need to be tests of the method before it is merged.
The tests could be created in GitClientTest or in a new test class created based on GitClientTest. Because GitClientTest is parameterized and iterates over
git
,jgit
, andjgitapache
, it increases the coverage without dramatically increasing the amount of test code.