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

LibGit2/test: Print failed process PID on challenge prompt failure #45798

Closed
wants to merge 1 commit into from

Conversation

Keno
Copy link
Member

@Keno Keno commented Jun 23, 2022

Makes it easier to locate the failed process in an rr trace after
the fact (cf #45797).

Makes it easier to locate the failed process in an rr trace after
the fact (cf #45797).
Copy link
Member

@vtjnash vtjnash left a comment

Choose a reason for hiding this comment

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

It seems like this should be findable by looking for ppid=579 signal=15 in rr ps?

@Keno
Copy link
Member Author

Keno commented Jun 23, 2022

You'd think so, but I don't see any such process, so it may have exited without the parent noticing or something.

@vtjnash
Copy link
Member

vtjnash commented Jun 23, 2022

pid 1978 in julia-9 is the only child that reports to be killed with signal 15

@Keno
Copy link
Member Author

Keno commented Jun 23, 2022

pid 1978 in julia-9 is the only child that reports to be killed with signal 15

Oh, so it does. I missed it.

@Keno
Copy link
Member Author

Keno commented Jun 23, 2022

rr doesn't like that particular pid either: rr-debugger/rr#3285

@DilumAluthge
Copy link
Member

@Keno @vtjnash Once CI is all green, is this good to merge?

@staticfloat
Copy link
Member

I have an alternative solution, which is to print the PID in jl_critical_error() when we are signalled: #45798

@giordano
Copy link
Contributor

Is this PR still relevant after #45954?

I have an alternative solution, which is to print the PID in jl_critical_error() when we are signalled: #45798

I presume you meant to link #45905 instead of this same PR?

@Keno
Copy link
Member Author

Keno commented Jul 10, 2022

Probably worth doing both. @staticfloat's change relies on the child being alive enough to run it's signals handlers, which it may not be sometimes.

@vtjnash vtjnash closed this Aug 31, 2023
@vtjnash vtjnash deleted the kf/libgit2testprintpid branch August 31, 2023 01:20
@vtjnash
Copy link
Member

vtjnash commented Aug 31, 2023

May be worth doing still, but heavily conflicted now for a small change. And hasn't seemed necessary lately (just a test enhancement)

@vtjnash vtjnash added test This change adds or pertains to unit tests libgit2 The libgit2 library or the LibGit2 stdlib module labels Aug 31, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
libgit2 The libgit2 library or the LibGit2 stdlib module test This change adds or pertains to unit tests
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants