Skip to content

Add work-around and fixes for blocked SIGCHLD environments#2

Open
cmcantalupo wants to merge 25 commits into
bgeltz:issue-3930from
cmcantalupo:issue-3930
Open

Add work-around and fixes for blocked SIGCHLD environments#2
cmcantalupo wants to merge 25 commits into
bgeltz:issue-3930from
cmcantalupo:issue-3930

Conversation

@cmcantalupo
Copy link
Copy Markdown

@cmcantalupo cmcantalupo commented Oct 9, 2025 •
edited
Loading

  • Modify cache clean up code so that the cache is always present, not just on demand
  • Adds work around to hook code that unblocks sigchld
  • Add fix to libgeopmd that unblocks sigchld prior to calling popen()

bgeltz and others added 25 commits October 1, 2025 10:03
@bgeltz
Refactor the PBS hook
ba12c62 
- Split the server configuration from the compute.

Signed-off-by: Brad Geltz <brad.geltz@intel.com>
@bgeltz
Remove unused variables
a293459 
Signed-off-by: Brad Geltz <brad.geltz@intel.com>
@bgeltz
Split the install scripts
f6e2a4e 
Signed-off-by: Brad Geltz <brad.geltz@intel.com>
@bgeltz
Fixups
3594ae3 
- Remmove unneeded code.
- Align names of resources across install script and hook implementation.

Signed-off-by: Brad Geltz <brad.geltz@intel.com>
@bgeltz
Docu updates
b81e1d9 
Signed-off-by: Brad Geltz <brad.geltz@intel.com>
@bgeltz
Update the test
2b0928c 
Signed-off-by: Brad Geltz <brad.geltz@intel.com>
@bgeltz
Set env for LevelZero
ad158e2 
Signed-off-by: Brad Geltz <brad.geltz@intel.com>
@bgeltz
Support MODIFYJOB events
4e61821 
Signed-off-by: Brad Geltz <brad.geltz@intel.com>
@bgeltz
Minimize or remove server and event calls
a7e25fb 
- Read from the resources file and replace calls
  to pbs.server() with this file.
- Call pbs.event() once and pass all other
  usages by reference.

Signed-off-by: Brad Geltz <brad.geltz@intel.com>
@bgeltz
DEBUGGING
164bc93 
Signed-off-by: Brad Geltz <brad.geltz@intel.com>
@bgeltz
DEBUGGING 2
5552f00 
Signed-off-by: Brad Geltz <brad.geltz@intel.com>
@bgeltz
Add the hook name to the front of all log messages
de01872 
Signed-off-by: Brad Geltz <brad.geltz@intel.com>
@bgeltz
Instrument read_controls()
e7c4773 
Signed-off-by: Brad Geltz <brad.geltz@intel.com>
@bgeltz
DEBUGGING: Add instrumentation to read_signal() paths
47a0185 
Signed-off-by: Brad Geltz <brad.geltz@intel.com>
@bgeltz
DEBUGGING: Print environment
35e2a5c 
Signed-off-by: Brad Geltz <brad.geltz@intel.com>
@cmcantalupo @bgeltz
Use subprocess to interact with PlatformIO
d2c46da 
- Avoid having pbs process become a client of the GPU

Signed-off-by: Christopher M. Cantalupo <christopher.m.cantalupo@intel.com>
@cmcantalupo
Fix options to subprocess.run() for backward compatibility
f235ae8 
Signed-off-by: Christopher M. Cantalupo <christopher.m.cantalupo@intel.com>
@bgeltz
Purge last use of pio
051f77a 
Signed-off-by: Brad Geltz <brad.geltz@intel.com>
@bgeltz
Add hostname to debug prints
cf7d591 
Signed-off-by: Brad Geltz <brad.geltz@intel.com>
@bgeltz
DEBUGGING: Instrument PlatformTopo:create_cache()
4007d17 
Signed-off-by: Brad Geltz <brad.geltz@intel.com>
@bgeltz
Fix format strings
21fae31 
Signed-off-by: Brad Geltz <brad.geltz@intel.com>
@cmcantalupo @bgeltz
Use system python to create topo cache as a subprocess
d68fb4d 
Signed-off-by: Christopher M. Cantalupo <christopher.m.cantalupo@intel.com>
@cmcantalupo
Move command string
4fcb6e7 
Signed-off-by: Christopher M. Cantalupo <christopher.m.cantalupo@intel.com>
@cmcantalupo
Unblock SIGCHLD in libgeopmd and in PBS hook
27ba9c5 
Signed-off-by: Christopher M. Cantalupo <christopher.m.cantalupo@intel.com>
@cmcantalupo
Remove topo cache file prior to creating the service object
f5b0aef 
- Expectation is the topo cache will be created at object
  creation time.

Signed-off-by: Christopher M. Cantalupo <christopher.m.cantalupo@intel.com>
@cmcantalupo cmcantalupo changed the title Issue 3930 Add work-around and fixes for blocked SIGCHLD environments Oct 9, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@cmcantalupo @bgeltz