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

POST /daemon/restart #412

Merged
merged 14 commits into from
Sep 15, 2023
Merged

Conversation

cgalibern
Copy link
Contributor

No description provided.

The unit may be restarted from the opensvc-agent control group
we need a separate flow here to avoid left-over process (om) in control group during stop

    systemd[1]: Stopping OpenSVC agent...
    systemd[1]: opensvc-agent.service: Succeeded.
    systemd[1]: Stopped OpenSVC agent.
    systemd[1]: opensvc-agent.service: Found left-over process 2899690 (om) in control group while starting unit. Ignoring.
    systemd[1]: This usually indicates unclean termination of a previous run, or service implementation deficiencies.
    systemd[1]: opensvc-agent.service: Found left-over process 2899697 (systemctl) in control group while starting unit. Ignoring.
    systemd[1]: This usually indicates unclean termination of a previous run, or service implementation deficiencies.
    systemd[1]: Starting OpenSVC agent...
    systemd[1]: Started OpenSVC agent.
This commit fix missing implementation for om daemon restart --node xxx
we can use instead: om daemon stop && om daemon start --foreground
cvaroqui and others added 4 commits September 15, 2023 16:24
Because endOrchestration is the only func reseting the
orchestration_id.

(cherry picked from commit 7620c72)
#6c21dbc2927d2bcadedd53fb7f498cd793ea3622 regression:
   Use POST /cluster/action/* and POST /node/action/drain from cmd
@cgalibern cgalibern merged commit 363d310 into opensvc:main Sep 15, 2023
1 check passed
@cgalibern cgalibern deleted the feature-api-daemon-restart branch September 30, 2023 12:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants