Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

chore: propage permanent mark to task instance generated from deployment #442

Conversation

wvmcastro
Copy link

@wvmcastro wvmcastro commented Dec 4, 2024

It is not strictly needed since the deployment task instance is added to the plan after generation. But as we pretend to do early deployment at generation, the garbage collection would throw away tasks contexts that got merged into this instances.

@wvmcastro wvmcastro requested review from doudou and jhonasiv December 4, 2024 19:48
@wvmcastro wvmcastro self-assigned this Dec 4, 2024
@wvmcastro wvmcastro force-pushed the propagate-permanent-mark-to-deployment-task branch from d66ce2d to f54e3e2 Compare December 4, 2024 21:06
@wvmcastro wvmcastro changed the title chore: propage permanent mark to task instance generated from the dep… chore: propage permanent mark to task instance generated from deployment Dec 4, 2024
@wvmcastro wvmcastro force-pushed the propagate-permanent-mark-to-deployment-task branch from f54e3e2 to 0dbb1f5 Compare December 5, 2024 12:24
…loyment

It is not strictly needed since the deployment task instance is added to the
plan after generation. But as we pretend to do early deployment at generation
the garbage collection would throw away tasks contexts that got merged into
this instances
@wvmcastro wvmcastro force-pushed the propagate-permanent-mark-to-deployment-task branch from 0dbb1f5 to 3b0d88c Compare December 5, 2024 12:30
@wvmcastro
Copy link
Author

Not really the right way to go, replace for #443

@wvmcastro wvmcastro closed this Dec 5, 2024
@wvmcastro wvmcastro deleted the propagate-permanent-mark-to-deployment-task branch December 5, 2024 17:19
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.

1 participant