Skip to content

docs: Add information about version validation of models#2379

Merged
triceo merged 1 commit into
TimefoldAI:mainfrom
diogodanielsoaresferreira:enforce_version_requirements_for_registered_models
Jun 24, 2026
Merged

docs: Add information about version validation of models#2379
triceo merged 1 commit into
TimefoldAI:mainfrom
diogodanielsoaresferreira:enforce_version_requirements_for_registered_models

Conversation

@diogodanielsoaresferreira

Copy link
Copy Markdown
Contributor

- The request query string includes parameters such as `type`, `registrationKey`, `handleSubscription`, and (for shared type) multiple `restrictedTo` entries for tenants
- If the platform responds with 409 (conflict) and `timefold.model.overwrite=true`, the plugin will PATCH `/api/platform/v1/models/{registrationKey}` to update the existing model
- The plugin reads the model descriptor to obtain name/id by extracting `timefold-model-descriptor.json` from the archive (or reading `target/timefold/timefold-model-descriptor.json` when present)
- The platform validates the model version (`timefold.application.version`) on registration. It must be a release version `vN` (e.g. `v1`), a git commit hash (1–40 lowercase hex characters), or the literal `SNAPSHOT`; any other value is rejected with HTTP 400.

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Does the maven plugin also provide explanation of the reason why the registration failed with HTTP 400?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

We return the error message in the 4xx error, and the maven plugin should show it in case of failed model register.

@triceo triceo merged commit 208d5b5 into TimefoldAI:main Jun 24, 2026
4 of 5 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.

4 participants