Skip to content

Upgrade on CAN#1311

Open
THardy98 wants to merge 12 commits intomainfrom
upgrade-on-can
Open

Upgrade on CAN#1311
THardy98 wants to merge 12 commits intomainfrom
upgrade-on-can

Conversation

@THardy98
Copy link
Contributor

@THardy98 THardy98 commented Jan 29, 2026

What was changed

Propagate CAN versioning and CAN suggested reasons to Python.

Why?

Users can specify auto-upgrade behavior on CAN.

  1. Part of Expose upgrade on continue as new in SDKs features#738

  2. How was this tested:
    Integration tests

  3. Any docs updates needed?
    Maybe?

@THardy98 THardy98 changed the title uUpgrade on CAN (needs test) Upgrade on CAN (needs test) Jan 29, 2026
self._current_history_length = act.history_length
self._current_history_size = act.history_size_bytes
self._continue_as_new_suggested = act.continue_as_new_suggested
self._suggested_continue_as_new_reasons = act.suggest_continue_as_new_reasons
Copy link
Member

Choose a reason for hiding this comment

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

Can you confirm from a Core side that once continue as new reasons are set on a workflow, they come over for every activation after that? (also worth confirming this for the suggested bool)

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Added a test (very similar to the test_workflow_history_info we were discussing) to check that CAN suggestion & reasons persist between WFTs

@THardy98 THardy98 changed the title Upgrade on CAN (needs test) Upgrade on CAN (waiting on core) Feb 2, 2026
@THardy98 THardy98 marked this pull request as ready for review February 3, 2026 16:44
@THardy98 THardy98 requested a review from a team as a code owner February 3, 2026 16:44
@THardy98 THardy98 changed the title Upgrade on CAN (waiting on core) Upgrade on CAN Feb 3, 2026
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.

3 participants