[build] pin artifacts in a committed manifest - #419
Merged
Conversation
michaelkirk
force-pushed
the
mkirk/artifact-manifest
branch
6 times, most recently
from
August 14, 2026 22:35
9145269 to
933d0ac
Compare
…sion Previously we were relying manually running a 'symlink' command to promote the latest transit graph. Now we track the exact version of the transit graph like we do with everything else. This simplifies some other scripts which no longer need to detect what's latest. Also removes OTP_CONFIG_JSON_YAML since we're no longer using it.
michaelkirk
force-pushed
the
mkirk/artifact-manifest
branch
from
August 14, 2026 22:47
58483e5 to
653db6e
Compare
github-merge-queue
Bot
removed this pull request from the merge queue due to failed status checks
Aug 17, 2026
michaelkirk
enabled auto-merge
August 18, 2026 21:20
michaelkirk
force-pushed
the
mkirk/artifact-manifest
branch
from
August 18, 2026 21:22
31e0479 to
e1d71ac
Compare
This file contains hidden or 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
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
I'm primarily attempting to solve two problems.
currently
this change
There is no longer a HEADWAY_TRANSIT_AREA's env variable. The build process infers which transit zones to build based on the presence of a gtfs_feeds file alone.
When the build completes, those (and the other build artifacts) will be in the artifacts.json, which is then used to drive bin/publish-artifacts and bin/k8s-generate.
It's not less code... so calling it "simpler" is a stretch, but it's not really any more code, it dedupes some state, and decouples more parts of the build->deploy process.
I think it's a win overall, but the proof will be in how it feels after a couple of build/deploy cycles.