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.

Sign up for GitHub

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

Jump to bottom

Revamp --follow-exec #962

Merged
merged 21 commits into from
Mar 20, 2022
Merged

Revamp --follow-exec #962

merged 21 commits into from
Mar 20, 2022

Conversation

xd009642
Copy link
Owner

@xd009642
Add failing MRE and update event log
bacffd5 
Event log now stores manifest information just to make my
tarpaulin-viewer output more compact
@xd009642
Change VFORK handling
6fb8cda 
I found out posix_spawn causes a call to vfork, I imagine this is the
most common time it occurs so should factor that in and put a
handle_exec in there
@xd009642 xd009642 mentioned this pull request Mar 2, 2022
Open
xd009642 added 10 commits March 3, 2022 18:17
@xd009642
Fix follow-exec alias
1d306cf
@xd009642
If not config.follow_exec do old behaviour
7101a16
@xd009642
Add test for issue #966
55b9788
@xd009642
Prototype compound actions
db3d99b 
Prototypes compound actions and also handles things being signaled in
some manner.
@xd009642
Change vfork parent continue behaviour.
f93c1b9 
Changes it so we only continue vfork parent in the situation where
--forward is passed
@xd009642
Make signaling less accepting, final trace merge
8467136 
Some binaries that are exec'ed and killed may not show as finishing
before the root test process, and I'm not sure if they ever will show
up? So add a fail-safe merge. Also made signalling less open so only do
the TryContinue on SIGTERM
@xd009642
Remove compound events and add last_wait_attempt
bf3a49b 
So now we don't have compound events at all, and there is a
last_wait_attempt for when the test is finished but tarpaulin is waiting
for other things to finish i.e. executables it's getting exec coverage
on.
@xd009642
Actually make --forward default
ad9537d 
Also, fix debug traces for spawned processes
@xd009642
Fix instrumented state machine
e316610 
Timeouts are impossible, and even if `last_wait_attempt` is called we
don't want to just launch the entire test binary again _just in case_
@xd009642
Remove commented out code
9543cb2
This was referenced Mar 8, 2022
Closed
Open
@xd009642 xd009642 marked this pull request as ready for review March 20, 2022 16:10
@xd009642 xd009642 changed the title WIP work on integration tests which issue a kill to spawned application Revamp --follow-exec Mar 20, 2022
@xd009642 xd009642 merged commit 83f1297 into develop Mar 20, 2022
@xd009642 xd009642 deleted the issue/process-kill-953 branch March 20, 2022 16:19
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.
1 participant
@xd009642