Conversation
|
Note that you can redo or rebase your branch, and force push it back to the same branch on github, and github will update the existing pull request (unless it has already been pulled). No need to create new ones. |
bruceg
left a comment
There was a problem hiding this comment.
I tried this patch locally (with a simpler test case) and it breaks the restart logic for services that exit 100. See my comments on bug #20. I don't think this is fixable, so I would prefer to just drop this as all the other cases are already handled.
bruceg
left a comment
There was a problem hiding this comment.
I will likely not be taking any of these before I do a release (soon).
| until svok test.sv | ||
| do | ||
| sleep 1 | ||
| sleep 0 |
There was a problem hiding this comment.
I'm puzzled. What's the point of sleep 0?
There was a problem hiding this comment.
It's faster than sleep 1. =) Seriously, I didn't think fractional sleep times, or usleep/nanosleep, etc would be portable. So, short of adding a proprietary sleep in the distro, the only other idea I had to waste a bit of time was something like "echo foo >/dev/null".
There was a problem hiding this comment.
On second thought, what about something like sleeper -n -d 100000000 for a 100 ms sleep instead? At this point, adding another switch would be trivial.
|
On the race in supervise when dealing with svc -du, here's how I initially tested. Isn't that what #20 was asking for? Late Edit: I see the problem now. The -du patch breaks the non-du use of exit 100. In other words, the service restarts where it's already documented not to. Skip it for now. Piling even more ugly on that section of code probably isn't a good idea. |
Continued from PR #58, rebased on recent commits and fixed the sleeper -d switch.
====
This pull is a collection of patches. I thought I'd throw it out there to show where I'm at now.
It includes:
The pending whitespace tidy patch.
The sleeper patch for out of order sigcont/sigterm, rebased to current master.
Some svscan tests for the recent sigterm handler addition.
A Solaris portability fix for the new supervise-start test.
The addition of a makefifo program, strictly for use in rts. (Is mkfifo portable?)
Addition of sleeper switches to facilitate better test integration.
A proposed patch for Issue #20. Partially fixed in 7569393, but still failed to restart if the service exited 100. This patch fixes that and adds tests for exit codes 0, 1, and 100.
A change in the way supervise handles throttling, switching to iopause instead of deepsleep, eliminating the one second of unresponsiveness each time a child starts or stops.
Various rts tweaks, reducing run time to around 10 sec.
And that would bring me to the end of meeting my goals on this -- the "missing sigcont" issue and speeding up the tests -- with a couple extras thrown in just because I already had my head wrapped around it. =)