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

Include JID in build log #1190

Open
1 task done
dag-erling opened this issue Nov 25, 2024 · 0 comments
Open
1 task done

Include JID in build log #1190

dag-erling opened this issue Nov 25, 2024 · 0 comments
Assignees
Milestone

Comments

@dag-erling
Copy link
Member

Prerequisites

  • Have you checked for an existing issue describing your idea?

What is your proposal?

Include the numeric jail ID for each jail in the build log.

What is the existing behavior, if any?

Each build log starts with a summary of the environment, e.g.:

=>> Building x11/mate-terminal
build started at Sun Nov 24 21:01:13 UTC 2024
port directory: /usr/ports/x11/mate-terminal
package name: mate-terminal-1.28.1
building for: FreeBSD pkg.des.dev 14.1-RELEASE-p6 FreeBSD 14.1-RELEASE-p6 amd64
maintained by: [email protected]
Makefile datestamp: -rw-r--r--  1 root wheel 730 Mar  7  2024 /usr/ports/x11/mate-terminal/Makefile
Ports top last git commit: 2401d480290f
Ports top unclean checkout: no
Port dir last git commit: 9a57a782af70
Port dir unclean checkout: no
Poudriere version: poudriere-git-3.4.2
Host OSVERSION: 1500027
Jail OSVERSION: 1401000
Job Id: 10

What is the motivation / use case for the change?

This would we useful for troubleshooting, e.g. when a build fails due to a segfault, as the jid will be included in the kernel message:

Nov 24 21:01:55 pkg kernel: pid 28694 (python3.11), jid 269, uid 65534: exited on signal 11 (core dumped)

Did you consider any alternatives?

No.

Is this really a ports feature request?

No.

Example

Jail Id: 269

Additional context

Note that the jail does not know its own ID (ps -ojid= $$ always prints 0) and the host does not find out until after the jail is created, so it would have to be passed in somehow.

@bdrewery bdrewery self-assigned this Nov 26, 2024
@bdrewery bdrewery added the Easy label Nov 26, 2024
@bdrewery bdrewery added this to the 3.5.0 milestone Dec 9, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants