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

Add debug logging around local channel process launch #3493

Merged
merged 2 commits into from
Jun 24, 2024

Conversation

benclifford
Copy link
Collaborator

This is driven by a case where it is unclear when the Parsl scaling code is launching processes to monitor batch queue status.

Changed Behaviour

This will add 4 debug lines per block started, block cancelled, and per poll (limited by provider.status_polling_interval() and config.strategy_period)

Type of change

  • New feature

This is driven by a case where it is unclear when the Parsl scaling
code is launching processes to monitor batch queue status.

This will add 4 debug lines per block started, block cancelled, and
per poll (limited by provider.status_polling_interval() and
config.strategy_period)
@benclifford
Copy link
Collaborator Author

cc @cms21 - this is extra logging I mentioned in Globus Compute's #help slack channel for debugging excessive qstat calls

parsl/channels/local/local.py Outdated Show resolved Hide resolved
parsl/channels/local/local.py Outdated Show resolved Hide resolved
@benclifford benclifford merged commit 7a440f3 into master Jun 24, 2024
7 checks passed
@benclifford benclifford deleted the benc-channel-debug branch June 24, 2024 12:35
@cms21
Copy link
Contributor

cms21 commented Jun 28, 2024

Thanks @benclifford, when will this be available in Globus Compute? As soon as it's released in the public Parsl version?

@rjmello
Copy link
Contributor

rjmello commented Jun 28, 2024

We expect to include this in the July 10th Globus Compute release.

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.

None yet

4 participants