Skip to content

chore(model): reduce dependency of model from secrets#1856

Merged
poroh merged 1 commit into
NVIDIA:mainfrom
poroh:model-secrets-dep-reduction
May 26, 2026
Merged

chore(model): reduce dependency of model from secrets#1856
poroh merged 1 commit into
NVIDIA:mainfrom
poroh:model-secrets-dep-reduction

Conversation

@poroh
Copy link
Copy Markdown
Contributor

@poroh poroh commented May 21, 2026

Description

secrets crate brings some heavy-weight dependencies like vault-rs and other. We can start compiling model earlier if this dependency is reduced.

Type of Change

  • Add - New feature or capability
  • Change - Changes in existing functionality
  • Fix - Bug fixes
  • Remove - Removed features or deprecated functionality
  • Internal - Internal changes (refactoring, tests, docs, etc.)

Related Issues (Optional)

Breaking Changes

  • This PR contains breaking changes

Testing

  • Unit tests added/updated
  • Integration tests added/updated
  • Manual testing performed
  • No testing required (docs, internal refactor, etc.)

Additional Notes

@poroh poroh requested a review from a team as a code owner May 21, 2026 05:17
@poroh poroh requested a review from prbinu-nvidia May 21, 2026 05:21
@poroh poroh force-pushed the model-secrets-dep-reduction branch from 11a01d5 to 6725562 Compare May 21, 2026 17:41
Comment thread crates/api-model/src/tenant/identity_config.rs Outdated
Copy link
Copy Markdown
Contributor

@prbinu-nvidia prbinu-nvidia left a comment

Choose a reason for hiding this comment

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

LGTM

secrets crate brings some heavy-weight dependencies like
vault-rs and other. We can start compiling model earlier
if this dependency is reduced.

Signed-off-by: Dmitry Porokh <dporokh@nvidia.com>
@poroh poroh force-pushed the model-secrets-dep-reduction branch from 6725562 to 28bb3c6 Compare May 26, 2026 17:59
@poroh poroh enabled auto-merge (squash) May 26, 2026 17:59
Copy link
Copy Markdown
Contributor

@prbinu-nvidia prbinu-nvidia left a comment

Choose a reason for hiding this comment

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

approved.

@poroh poroh merged commit 0c1d10f into NVIDIA:main May 26, 2026
52 checks passed
Sign up for free to 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.

2 participants