Skip to content

Bump PID used for checkpoint to be > 1000#542

Merged
tjwatson merged 1 commit into
WASdev:vNextfrom
tjwatson:elevateCheckpointPID
Sep 8, 2023
Merged

Bump PID used for checkpoint to be > 1000#542
tjwatson merged 1 commit into
WASdev:vNextfrom
tjwatson:elevateCheckpointPID

Conversation

@tjwatson

@tjwatson tjwatson commented Sep 8, 2023

Copy link
Copy Markdown
Member

No description provided.

Some environments cause higher PIDs to be used in
the container than 100 which leads to potential
conflicts in the PID needed to restore.

This change increases the checkpoint PID to be > 1000.
More investigation is needed to do this efficiently.
If a more efficient way is found then we may want to
increase this to a higher PID value.  To avoid long
delays during a checkpoint the value of 1000 is being
used for now.
@tjwatson tjwatson requested a review from leochr September 8, 2023 18:52

@leochr leochr left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

LGTM. Thanks

@tjwatson tjwatson merged commit fa81a18 into WASdev:vNext Sep 8, 2023
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.

2 participants