Skip to content

Conversation

@armishra
Copy link
Contributor

@armishra armishra commented Oct 28, 2025

🚀 What

Add deployment_name CLI parameter to the truss chains CLI command.

Part 2/x (This is a hard dependency): https://github.com/basetenlabs/baseten/pull/14016

this is Part 3/x

💻 How

🔬 Testing

Locally tested:

$ pip install -e /path/to/truss
$ truss chains push --help | grep -i "deployment-name"
│ --deployment-name                    TEXT                                  Name of the deployment created by the │
$ truss chains push hello.py --log=d --remote=baseten-staging --promote --deployment-name="archit-deployment-staging"

Deployment succeeded.
Screenshot 2025-10-30 at 4 17 28 PM

Automatic naming mechanism:
Screenshot 2025-10-31 at 10 08 04 AM

@linear
Copy link

linear bot commented Oct 28, 2025

@armishra armishra marked this pull request as ready for review October 28, 2025 23:19
Copy link
Contributor

@tyranitar tyranitar left a comment

Choose a reason for hiding this comment

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

Looks good but I think we need to thread all the way to deploy_chain_atomic. Also we need to deploy changes to the backend mutation first.

@armishra armishra changed the title feat(truss): add deployment_name CLI option to truss chains [COR-215][2/x] feat(truss): add deployment_name CLI option to truss chains [COR-215][3/x] Oct 29, 2025
Copy link
Contributor

@ndeepak-baseten ndeepak-baseten left a comment

Choose a reason for hiding this comment

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

LGTM, tiny comment. Thanks!

@ndeepak-baseten
Copy link
Contributor

@armishra Be sure to run integration tests prior to merge. Github Actions > Run integration tests > your branch.

@armishra armishra force-pushed the architmishra/cor-215-support-deployment-name-for-chains-p2 branch 2 times, most recently from f22db0a to 9b8e036 Compare November 11, 2025 22:55
@armishra armishra force-pushed the architmishra/cor-215-support-deployment-name-for-chains-p2 branch from 9b8e036 to 630dbf3 Compare November 11, 2025 22:58
@armishra armishra merged commit 253882f into main Nov 11, 2025
33 of 34 checks passed
@armishra armishra deleted the architmishra/cor-215-support-deployment-name-for-chains-p2 branch November 11, 2025 23:43
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.

4 participants