Your one-stop-cake-shop for everything Freshly Baked has to offer

ci: fetch dependencies before evaluate #128

merged opened by a.starrysky.fyi targeting main from private/minion/push-mlmuvqqtptlt

Due to what I suspect must be either an npins or a lix bug, we get this error when evaluating our configuration on new machines:

โ€ฆ while fetching the input 'git+https://gerrit.wikimedia.org/r/mediawiki/extensions/AdvancedSearch?rev=398c9fa782843d8b3aeaa5ebb1c1b3db35c3382f'

error: Cannot find Git revision '398c9fa782843d8b3aeaa5ebb1c1b3db35c3382f' in ref 'refs/heads/master' of repository 'https://gerrit.wikimedia.org/r/mediawiki/extensions/AdvancedSearch'! Please make sure that the rev exists on the ref you've specified or add allRefs = true; to fetchGit.

Now, this is what I'd expect - because we're not meant to be fetching master of this extension - we're meant to be fetching a branch! By using npins to prefetch all dependencies, we get around this.

Due to a nilla bug, fetching dependencies is required for evaluation anyway - so this isn't too much more of an issue than it would already have been...

Labels

None yet.

requested-reviewers

None yet.

approved

None yet.

tested-working

None yet.

rejected

None yet.

assignee

None yet.

Participants 1
AT URI
at://did:plc:uuyqs6y3pwtbteet4swt5i5y/sh.tangled.repo.pull/3mbbxryg4sg22
+6
Interdiff #0 โ†’ #1
+2
.tangled/workflows/packetmix-build.yml
··· 40 40 echo "builders = ssh-ng://remoteBuilds@midnight?ssh-key=/tmp/key-ssh-remote-build x86_64-linux" >> /etc/nix/nix.conf 41 41 - name: Fetch dependencies 42 42 command: | 43 + cd packetmix 43 44 npins update --partial 45 + cd .. 44 46 - name: Evaluate all systems 45 47 command: | 46 48 set +e
+2
.tangled/workflows/packetmix-release.yml
··· 47 47 echo "builders = ssh-ng://remoteBuilds@midnight?ssh-key=/tmp/key-ssh-remote-build x86_64-linux" >> /etc/nix/nix.conf 48 48 - name: Fetch dependencies 49 49 command: | 50 + cd packetmix 50 51 npins update --partial 52 + cd .. 51 53 - name: Evaluate all systems 52 54 command: | 53 55 set +e
+2
.tangled/workflows/packetmix-treefmt.yaml
··· 40 40 echo "max-jobs = 0" >> /etc/nix/nix.conf 41 41 - name: Fetch dependencies 42 42 command: | 43 + cd packetmix 43 44 npins update --partial 45 + cd .. 44 46 - name: Ensure files are formatted with treefmt 45 47 command: | 46 48 set -e

History

15 rounds 0 comments
sign up or login to add to the discussion
1 commit
expand
fix(npins): fetch dependencies from correct ref
1/5 failed, 4/5 success
expand
expand 0 comments
pull request successfully merged
1 commit
expand
fix(npins): fetch dependencies from correct ref
1/5 failed, 4/5 success
expand
expand 0 comments
1 commit
expand
fix(npins): fetch dependencies from correct ref
1/5 failed, 4/5 success
expand
expand 0 comments
1 commit
expand
fix(npins): fetch dependencies from correct ref
1/5 failed, 4/5 success
expand
expand 0 comments
1 commit
expand
fix(npins): fetch dependencies from correct ref
1/5 failed, 4/5 success
expand
expand 0 comments
1 commit
expand
fix(npins): fetch dependencies from correct ref
1/5 failed, 4/5 success
expand
expand 0 comments
1 commit
expand
fix(npins): fetch dependencies from correct ref
1/5 failed, 4/5 success
expand
expand 0 comments
1 commit
expand
fix(npins): fetch dependencies from correct ref
1/5 failed, 4/5 success
expand
expand 0 comments
1 commit
expand
fix(npins): fetch dependencies from correct ref
1/5 failed, 4/5 success
expand
expand 0 comments
1 commit
expand
fix(npins): fetch dependencies from correct ref
expand 0 comments
1 commit
expand
fix(npins): fetch dependencies from correct ref
expand 0 comments
1 commit
expand
fix(npins): fetch dependencies from correct ref
expand 0 comments
1 commit
expand
fix(npins): fetch dependencies from correct ref
expand 0 comments
1 commit
expand
ci: fetch dependencies before evaluate
expand 0 comments
1 commit
expand
ci: fetch dependencies before evaluate
expand 0 comments