Skip to content

Commit 3fada07

Browse files
committed
add comments noting why we want git SHAs
1 parent 369ce85 commit 3fada07

File tree

2 files changed

+7
-0
lines changed

2 files changed

+7
-0
lines changed

dev-tools/omdb/build.rs

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -13,6 +13,9 @@ fn main() {
1313
// Define the `VERGEN_GIT_SHA` and `VERGEN_GIT_DIRTY` environment variables
1414
// (accessible via `env!()`) that note the current git commit and whether
1515
// the working tree is dirty at the time of this build.
16+
//
17+
// We use this to check our own git SHA against the git SHA of the Nexus
18+
// that generated blueprint planner debug logs.
1619
let gitcl = GitclBuilder::default()
1720
.sha(/* short= */ false)
1821
.dirty(/* include_untracked= */ false)

nexus/db-model/build.rs

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -13,6 +13,10 @@ fn main() {
1313
// Define the `VERGEN_GIT_SHA` and `VERGEN_GIT_DIRTY` environment variables
1414
// (accessible via `env!()`) that note the current git commit and whether
1515
// the working tree is dirty at the time of this build.
16+
//
17+
// We embed our git SHA in the JSON blobs for blueprint planner debug logs,
18+
// so `omdb` can report if it's out of sync with the Nexus that created the
19+
// debug log.
1620
let gitcl = GitclBuilder::default()
1721
.sha(/* short= */ false)
1822
.dirty(/* include_untracked= */ false)

0 commit comments

Comments
 (0)