Skip to content

testcases: fetch test definitions from a GitHub source tarball - #96

Open
Matt H (mattface) wants to merge 1 commit into
masterfrom
test-tarballs
Open

Matt H (mattface) wants to merge 1 commit into
masterfrom
test-tarballs

Conversation

@mattface

Copy link
Copy Markdown

Every definition cloned the same repository, 18 times for a pre-merge job, each leaving a .git in the overlay. Render "from: url" against the GitHub archive url instead, so they share one cached fetch.

The url has to name a commit: a branch moves and a tag can be recreated, and the cache would keep serving whichever one it saw first. Resolve the revision with git ls-remote before rendering, and record in the job what it resolved to. Repositories not on github.com, and revisions that cannot be resolved, keep using git.

Every definition cloned the same repository, 18 times for a pre-merge
job, each leaving a .git in the overlay. Render "from: url" against the
GitHub archive url instead, so they share one cached fetch.

The url has to name a commit: a branch moves and a tag can be recreated,
and the cache would keep serving whichever one it saw first. Resolve the
revision with git ls-remote before rendering, and record in the job what
it resolved to. Repositories not on github.com, and revisions that
cannot be resolved, keep using git.

Signed-off-by: Matt Hart <matthart@qti.qualcomm.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant