Skip to content

feat: expose invocation ID as NEL_INVOCATION_ID env var#894

Merged
wprazuch merged 2 commits intomainfrom
agronskiy/feat/invocation-id-as-env-var
Apr 3, 2026
Merged

feat: expose invocation ID as NEL_INVOCATION_ID env var#894
wprazuch merged 2 commits intomainfrom
agronskiy/feat/invocation-id-as-env-var

Conversation

@agronskiy
Copy link
Copy Markdown
Collaborator

Summary

  • Exports NEL_INVOCATION_ID environment variable at the top of both docker (local) and slurm execution scripts
  • Allows downstream tooling to reference the invocation ID during evaluation runs

Test plan

  • Verify docker run.sh contains export NEL_INVOCATION_ID=... at the top
  • Verify slurm run.sub contains export NEL_INVOCATION_ID=... after TASK_DIR
  • Run a local dry-run and inspect generated script

🤖 Generated with Claude Code

@copy-pr-bot
Copy link
Copy Markdown

copy-pr-bot Bot commented Apr 3, 2026

This pull request requires additional validation before any workflows can run on NVIDIA's runners.

Pull request vetters can view their responsibilities here.

Contributors can view more details about this message here.

@agronskiy agronskiy marked this pull request as ready for review April 3, 2026 17:58
@agronskiy agronskiy requested review from a team as code owners April 3, 2026 17:58
agronskiy and others added 2 commits April 3, 2026 19:58
…scripts

Makes the invocation ID available at the top of both docker (local) and
slurm execution scripts so downstream tooling can reference it.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Signed-off-by: Alex Gronskiy <agronskiy@nvidia.com>
Consistent with TASK_DIR which is also a plain shell variable.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Signed-off-by: Alex Gronskiy <agronskiy@nvidia.com>
@agronskiy agronskiy force-pushed the agronskiy/feat/invocation-id-as-env-var branch from d99b4fa to 463de6e Compare April 3, 2026 17:58
Copy link
Copy Markdown
Contributor

@wprazuch wprazuch left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@wprazuch wprazuch enabled auto-merge (squash) April 3, 2026 18:40
@wprazuch
Copy link
Copy Markdown
Contributor

wprazuch commented Apr 3, 2026

/ok to test 463de6e

@wprazuch wprazuch merged commit 8c56e10 into main Apr 3, 2026
48 checks passed
@wprazuch wprazuch deleted the agronskiy/feat/invocation-id-as-env-var branch April 3, 2026 18:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants