Skip to content

Conversation

@talyaacovi
Copy link
Contributor

@talyaacovi talyaacovi commented Oct 23, 2025

🚀 What

Pass environment_name to create_model_from_truss mutation instead of erroring. This PR needs to be merged first, which will replicate the error unless a feature flag is enabled.

Also will return an error if both environment and promote flags are used, as currently that behavior is buggy and we want to move away from eager promotion.

image

💻 How

Remove error messaging if a new model is being pushed with a non-production environment name.

🔬 Testing

Locally, confirmed same error message if feature flag is disabled and that push succeeds if the flag is on. Will test in dev as well.

@talyaacovi talyaacovi force-pushed the tal/pass-environment-name-to-create-model-from-truss branch from c3f5eaa to 25ff984 Compare October 23, 2025 16:37
@talyaacovi talyaacovi force-pushed the tal/pass-environment-name-to-create-model-from-truss branch from 1f24c34 to 6b8f893 Compare October 27, 2025 22:31
@talyaacovi talyaacovi force-pushed the tal/pass-environment-name-to-create-model-from-truss branch from 6b8f893 to 50e42ae Compare October 27, 2025 22:55
@talyaacovi talyaacovi force-pushed the tal/pass-environment-name-to-create-model-from-truss branch from 50e42ae to 536d534 Compare October 29, 2025 21:14
@talyaacovi talyaacovi merged commit 747eb14 into main Oct 29, 2025
62 of 63 checks passed
@talyaacovi talyaacovi deleted the tal/pass-environment-name-to-create-model-from-truss branch October 29, 2025 22:48
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