Skip to content

feat: Add Lepton secret handling - #348

Open
ansjindal wants to merge 1 commit into
NVIDIA-NeMo:mainfrom
ansjindal:feat/lepton-secret-handling
Open

feat: Add Lepton secret handling #348
ansjindal wants to merge 1 commit into
NVIDIA-NeMo:mainfrom
ansjindal:feat/lepton-secret-handling

Conversation

@ansjindal

@ansjindalansjindal commented Oct 6, 2025

Copy link
Copy Markdown
Contributor
  • Add secure environment variable handling with Lepton workspace secrets
  • Support mixed environment variables (secrets + regular values)
  • Create detailed Lepton examples with distributed training workflows
  • Update documentation with usage guides

Key changes:

  • LeptonExecutor now supports {'value_from': {'secret_name_ref': 'SECRET_NAME'}} syntax

Example

 env_vars={
# Use Lepton secret reference instead of hardcoded token
"HF_TOKEN": {"value_from": {"secret_name_ref": "HUGGING_FACE_HUB_TOKEN_read"}},
# Basic distributed training configuration (regular env vars)
"NCCL_DEBUG": "INFO",
"TORCH_DISTRIBUTED_DEBUG": "INFO", "CUDA_LAUNCH_BLOCKING": "1",
},

…ng support
- Add secure environment variable handling with Lepton workspace secrets
- Support mixed environment variables (secrets + regular values)
- Implement EnvValue import and processing in LeptonExecutor
- Create detailed Lepton examples with distributed training workflows
- Update documentation with security best practices and usage guides
Key changes:
* LeptonExecutor now supports {'value_from': {'secret_name_ref': 'SECRET_NAME'}} syntax
Signed-off-by: ansjindal <ansjindal@nvidia.com>
@ansjindal
ansjindalforce-pushed the feat/lepton-secret-handling branch from 9ffd6d2 to 4677c96CompareOctober 6, 2025 08:40
@ansjindalansjindal changed the title feat: Add Lepton secret handling and comprehensive distributed traini…feat: Add Lepton secret handling Oct 7, 2025
@ansjindal

Copy link
Copy Markdown
ContributorAuthor

@roclark could you please review it, a customer was asking about it?

@roclark
roclark self-requested a review November 12, 2025 15:29
Sign up for freeto 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

@ansjindal