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

Crmv1 #129

Open
wants to merge 13 commits into
base: main
Choose a base branch
from
Next Next commit
main: Fix exit code on grant/revoke command error
Client command grant/revoke was returning success exit
code on some failures (site not configured, arbitrator,
no tickets given). Test case is running
`booth grant -s $IP; echo $?`, where IP is not configured in the config
file.

Patch fixes this behavior so error code is returned.

Signed-off-by: Jan Friesse <jfriesse@redhat.com>
  • Loading branch information
jfriesse authored and dmuhamedagic committed Oct 20, 2023
commit ed4b362ed3d8da8abd3b4e84d254b0d709a189fb
2 changes: 1 addition & 1 deletion src/main.c
Original file line number Diff line number Diff line change
@@ -762,7 +762,7 @@ static int do_command(cmd_request_t cmd)
else if (cmd == CMD_REVOKE)
op_str = "revoke";

rv = 0;
rv = -1;
site = NULL;

/* Always use TCP for client - at least for now. */