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

remove all non-essential parameters from the heartbeat executable #3646

Closed
tbeerman opened this issue May 27, 2020 · 0 comments
Closed

remove all non-essential parameters from the heartbeat executable #3646

tbeerman opened this issue May 27, 2020 · 0 comments

Comments

@tbeerman
Copy link
Contributor

Motivation

The executable across the daemons is not uniform. Some only use the daemon name, some use the full path and others the path plus all cli arguments. E.g.,

rucio-undertaker
/usr/bin/rucio-reaper2
/usr/bin/rucio-conveyor-receiver --total-threads 5

The sharding is based on the executable and in the example above if two receivers would be started, one with 5 threads and one with 10 threads, they wouldn't share the load.

Modification

Remove the path and all non-essential cli arguments.

@tbeerman tbeerman added the patch label May 27, 2020
@tbeerman tbeerman self-assigned this May 27, 2020
tbeerman added a commit to tbeerman/rucio that referenced this issue May 27, 2020
bari12 added a commit that referenced this issue Jun 2, 2020
…rams

remove unnecessary parts from the heartbeats #3646
bari12 added a commit that referenced this issue Jun 2, 2020
…rams

remove unnecessary parts from the heartbeats #3646
@bari12 bari12 added this to the 1.22.7 milestone Jun 2, 2020
@bari12 bari12 closed this as completed Jun 2, 2020
tbeerman added a commit to tbeerman/rucio that referenced this issue Jun 10, 2020
bari12 added a commit that referenced this issue Jun 11, 2020
bari12 added a commit that referenced this issue Jun 11, 2020
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