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

Various fixes #38

Merged
merged 5 commits into from
Aug 27, 2019
Merged

Various fixes #38

merged 5 commits into from
Aug 27, 2019

Conversation

jfsmig
Copy link

@jfsmig jfsmig commented Aug 26, 2019 •
edited
Loading

  • gridinit_cmd: Set the exit status to 1 when the socket cannot be reached
  • gridinit: Remove the socket at exit
  • gridinit: Remove the pidfile at exit
  • gridinit: Close file descriptor between fork() and execve()
  • gridinit: Lighter variable associations

/cc @jkasarherou

@jfsmig jfsmig added the bug label Aug 26, 2019
@jfsmig jfsmig requested a review from fvennetier August 26, 2019 15:18
@jfsmig jfsmig self-assigned this Aug 26, 2019
@jfsmig jfsmig changed the title cmd: Set the exit status to 1 when the socket cannot be reached Various fixes Aug 26, 2019
@jfsmig jfsmig merged commit d8104ec into open-io:master Aug 27, 2019
@jfsmig jfsmig deleted the M-fixes-20190826 branch September 30, 2020 15:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@fvennetier fvennetier Awaiting requested review from fvennetier

Assignees

@jfsmig jfsmig

Labels
bug
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@jfsmig