Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up
Appearance settings

[API] Add stop upgrade endpoint #71

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

Open
mirkoCrobu wants to merge 2 commits into main
base: main
Choose a base branch
Loading
from issue_382_api_stop_running_upgrade

Conversation

@mirkoCrobu
Copy link
Contributor

@mirkoCrobu mirkoCrobu commented Nov 12, 2025
edited
Loading

Motivation

closes https://github.com/bcmi-labs/orchestrator/issues/382

Change description

Adding API endpoint PUT /v1/system/update/stop.
It stops service and platform updates.

The update cancellation is implemented via a SIGTERM signal (context cancellation) to allow for a graceful shutdown.

apt-get, In some phases, traps this signal to complete its current critical operation before exiting, delaying the interruption.

Additional Notes

Test

  • Installing orchestrator with new features
  • Start listening to events
    curl -X GET localhost:8800/v1/system/update/events
  • Start updates
    curl -X PUT localhost:8800/v1/system/update/apply
  • After 0,5 -1-3 seconds, stop updates
    curl -X PUT localhost:8800/v1/system/update/stop

Reviewer checklist

  • PR addresses a single concern.
  • PR title and description are properly filled.
  • Changes will be merged in main.
  • Changes are covered by tests.
  • Logging is meaningful in case of troubleshooting.

@mirkoCrobu mirkoCrobu self-assigned this Nov 12, 2025
@mirkoCrobu mirkoCrobu requested a review from a team November 12, 2025 14:47
Copy link
Contributor

@mirkoCrobu can you fix the conflicts?

mirkoCrobu reacted with rocket emoji

@mirkoCrobu mirkoCrobu force-pushed the issue_382_api_stop_running_upgrade branch from 552db79 to dfc7c3d Compare December 16, 2025 16:45
@mirkoCrobu mirkoCrobu requested review from a team and giulio93 December 16, 2025 16:45
@mirkoCrobu mirkoCrobu force-pushed the issue_382_api_stop_running_upgrade branch from 9054051 to d488509 Compare December 17, 2025 15:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@giulio93 giulio93 Awaiting requested review from giulio93

At least 1 approving review is required to merge this pull request.

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

AltStyle によって変換されたページ (->オリジナル) /