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

flux-uri(1) pid resolver does not work on macos #6513

Open
garlick opened this issue Dec 16, 2024 · 4 comments
Open

flux-uri(1) pid resolver does not work on macos #6513

garlick opened this issue Dec 16, 2024 · 4 comments
Labels

Comments

@garlick
Copy link
Member

garlick commented Dec 16, 2024

Problem: the flux-uri(1) pid resolver relies on /proc but that does not exist on macos

$ flux uri
flux-uri: ERROR: No such file or directory: '/proc/65570/task'
@grondo
Copy link
Contributor

grondo commented Dec 16, 2024

It appears perhaps ps -Eww may be able to get the environment for a target PID on macos. However, when the target is a broker, the pid resolver has to attempt to find a child, since FLUX_URI is not set in the broker itself.

Since it would be a bit of work to get this working on macos, and the pid resolver is not necessary functionality (and also we don't have any immediate need for Flux on macos afaik) I'd be inclined to leave this as is or maybe just detect macos and emit a "not supported" error.

Unfortunately this means we're going to have to disable a bunch of tests on macos, maybe by adding a new test prereq NOT_MACOS or similar? Hopefully that would be less work than getting the pid resolver (and anything else that uses /proc) working.

@garlick
Copy link
Member Author

garlick commented Dec 16, 2024

Fine with me. I mainly just wanted to get some issues open for macos so interested parties could dig in should they choose. Not a priority.

@grondo
Copy link
Contributor

grondo commented Dec 16, 2024

Sounds good. Maybe we want to create a macos label for these?

@garlick
Copy link
Member Author

garlick commented Dec 16, 2024

Good idea, I'll do that.

@garlick garlick added the macos label Dec 16, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants