This repository was archived by the owner on Jan 30, 2020. It is now read-only.
fleetctl: status and restart commands with global units #1624
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
dongsupark
force-pushed
the
dongsu/fleetctl-restart-global-units
branch
from
ba57a71 to
64f8b0e
Compare
dongsupark
force-pushed
the
master
branch
2 times, most recently
from
54409ab to
a1a21b8
Compare
dongsupark
force-pushed
the
master
branch
3 times, most recently
from
150d30c to
4002bf5
Compare
dongsupark
force-pushed
the
master
branch
7 times, most recently
from
3b60e93 to
875d938
Compare
dongsupark
force-pushed
the
master
branch
2 times, most recently
from
bdc94e8 to
fa5aa3a
Compare
|
@dongsupark have you been using this since you created this PR? How's it holding up? We've been doing something similar outside of fleet using SSH connections and calling Is anyone planning to review and merge this in the near future? |
Author
|
@simonvanderveldt |
dongsupark
force-pushed
the
dongsu/fleetctl-restart-global-units
branch
from
64f8b0e to
fe560b0
Compare
dongsupark
force-pushed
the
master
branch
3 times, most recently
from
20a3e96 to
3aaa1ab
Compare
dongsupark
force-pushed
the
dongsu/fleetctl-restart-global-units
branch
4 times, most recently
from
d591ce2 to
de2fcc7
Compare
dongsupark
force-pushed
the
dongsu/fleetctl-restart-global-units
branch
2 times, most recently
from
0e864e0 to
f77faa3
Compare
dongsupark
force-pushed
the
master
branch
2 times, most recently
from
eb6872f to
365565e
Compare
added 6 commits
December 14, 2016 11:49
fleetctl status should handle machine states of global units, instead of failing with errors. To avoid creating individual ssh connection for each machine for global unit, global units need to be first inserted into a list of global units. They are filtered again to contain only unique pairs of (unit name, machine ID), before fleetctl finally initiates connection to each of the machines.
Fleetctl restart allows users to easily restart units on every machine, without having to run manually like "fleetctl ssh systemctl restart". Both global units and non-global units are supported. Originally written by Kyle Boorky <[email protected]> See also coreos#1238
Now that the "fleetctl restart" command is available, a new unit test is also necessary correspondently.
For start command, we don't need to explicitly check for systemd active states, as fleetctl start is already doing that.
A new functional test TestUnitRestart is also needed to verify that the new command "fleetctl restart" works well.
Now that fleetctl status & restart is able to run with global units, TestScheduleGlobalUnits() needs to run "fleetctl status" as well as "fleetctl restart" also with a global unit, to verify that the new functionality is working too.
dongsupark
force-pushed
the
dongsu/fleetctl-restart-global-units
branch
from
f77faa3 to
060abbf
Compare
Author
|
Fixed minor things, and force-pushed. |
dongsupark
force-pushed
the
master
branch
6 times, most recently
from
39a99ba to
44591b0
Compare
dongsupark
pushed a commit
that referenced
this pull request
Dec 16, 2016
…-units fleetctl: status and restart commands with global units
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
2 participants
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This PR is yet another revision of a long-running issue,
fleetctl restartandstatuswith global units."fleetctl status"to run with global units. For the purpose, implement a helpercmdGlobalMachineState()that filters a list of machines only with unique entries."fleetctl restart". This part is originally taken from fleetctl: Adding restart command #1238 written by "Kyle Boorky [email protected]", committed by @hhoover .fleetctl restart.TestScheduleGlobalUnits().This PR is not about changing API or metadata format in etcd registry. It's not introducing any new mechanism, because I don't think I can afford changing such design at the moment. It's rather a compromise between practical requirements and the current design of fleet. Though we still need to think about how to decrease number of ssh connections when restarting global units, as suggested by @kayrus. See also #1559.
Fixes #975