Skip to content

Scheduled instance restarts #6285

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.

Sign up for GitHub

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

Jump to bottom
Merged
hardillb merged 14 commits into from
Nov 12, 2025
Merged

Scheduled instance restarts #6285

hardillb merged 14 commits into from
Nov 12, 2025

Conversation

@cstns
Copy link
Contributor

@cstns cstns commented Nov 11, 2025 •
edited
Loading

Description

Adds UI to for the scheduled instance restarts/upgrades

Related Issue(s)

closes #5756

Checklist

  • I have read the contribution guidelines
  • Suitable unit/system level tests have been added and they pass
  • Documentation has been updated
    • Upgrade instructions
    • Configuration details
    • Concepts
  • Changes flowforge.yml?
    • Issue/PR raised on FlowFuse/helm to update ConfigMap Template
    • Issue/PR raised on FlowFuse/CloudProject to update values for Staging/Production
  • Link to Changelog Entry PR, or note why one is not needed.

Labels

  • Includes a DB migration? -> add the area:migration label

cstns added 6 commits November 10, 2025 10:49
@cstns
feat(ui): add "Maintenance" section to instance settings with routing…
29831b5 
… and base component
@cstns
Merge remote-tracking branch 'origin/scheduled-upgrade-restart' into …
1e0a36b 
…scheduled-instance-restarts
@cstns
Merge remote-tracking branch 'origin/scheduled-upgrade-restart' into …
44005b0 
…scheduled-instance-restarts
@cstns
Merge remote-tracking branch 'origin/scheduled-upgrade-restart' into …
fdb0c49 
…scheduled-instance-restarts
@cstns
Merge remote-tracking branch 'origin/scheduled-upgrade-restart' into …
debaf49 
…scheduled-instance-restarts
@cstns
feat(ui): implement scheduled upgrade settings in maintenance section
97b215e 
- Added API methods (`getUpdateSchedule`, `setUpdateSchedule`, `removeUpdateSchedule`) for managing scheduled upgrades.
- Updated `Maintenance` settings page with UI to enable, configure, and disable scheduled upgrades.
- Enhanced `DateTimePicker` component to support locale fallback and added styling changes.
@cstns cstns requested a review from hardillb November 11, 2025 09:32
@cstns cstns self-assigned this Nov 11, 2025
@cstns cstns changed the base branch from main to scheduled-upgrade-restart November 11, 2025 09:34
@hardillb
Copy link
Contributor

hardillb commented Nov 11, 2025

This should read between "01:00 and 02:00" not "01:00 and 01:00"

Also a note should be added to say all times are in UTC

image

@codecov
Copy link

codecov bot commented Nov 11, 2025 •
edited
Loading

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 76.47%. Comparing base (56deb93) to head (2759327).
⚠️ Report is 15 commits behind head on scheduled-upgrade-restart.

Additional details and impacted files 
@@                    Coverage Diff                     @@
##           scheduled-upgrade-restart    #6285   +/-   ##
==========================================================
  Coverage                      76.47%   76.47%           
==========================================================
  Files                            388      388           
  Lines                          19596    19596           
  Branches                        4708     4708           
==========================================================
  Hits                           14986    14986           
  Misses                          4610     4610
Flag Coverage Δ
backend 76.47% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@hardillb
Copy link
Contributor

hardillb commented Nov 11, 2025

Something still not right here:

Still showing same start and end times and the hour selected is out by an hour to what is displayed

image

@cstns
Copy link
Contributor Author

cstns commented Nov 12, 2025

the end time slipped through the cracks without conversion to utc, please give it a go. I also added the note

knolleary
knolleary reviewed Nov 12, 2025
View reviewed changes
@hardillb
Copy link
Contributor

hardillb commented Nov 12, 2025

Merging this into the backend branch

@hardillb hardillb merged commit f7e6cad into scheduled-upgrade-restart Nov 12, 2025
25 checks passed
@hardillb hardillb deleted the scheduled-instance-restarts branch November 12, 2025 11:31
@cstns cstns mentioned this pull request Nov 12, 2025
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@knolleary knolleary knolleary left review comments

@hardillb hardillb Awaiting requested review from hardillb

Assignees

@cstns cstns

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Design UI for scheduling instance restart

4 participants

@cstns @hardillb @knolleary