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

Backend for stack restart API #1615

Merged
merged 1 commit into from
Jan 31, 2023
Merged

Backend for stack restart API #1615

merged 1 commit into from
Jan 31, 2023

Conversation

hardillb
Copy link
Contributor

@hardillb hardillb commented Jan 25, 2023

Description

Add API endpoint /projects/<project id>/actions/restartStack

This suspends and then restarts the project.

Useful for upgrading bare pod to Deployment (when FlowFuse/driver-k8s#71 is merged)

Related Issue(s)

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 flowforge/helm to update ConfigMap Template
    • [-] Issue/PR raised on flowforge/CloudProject to update values for Staging/Production

Labels

  • [-] Backport needed? -> add the backport label
  • [-] Includes a DB migration? -> add the area:migration label

@hardillb hardillb marked this pull request as ready for review January 26, 2023 11:11
@joepavitt
Copy link
Contributor

Just checking no UI changes required for this in order to expose the action? API entry-point only?

@hardillb
Copy link
Contributor Author

@joepavitt UI to follow later (we may hit this via a script as part of the K8s upgrade)

Copy link
Contributor

@joepavitt joepavitt left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Form the UI side, I'm happy with the Icon-use and Verbose descriptions. I worked with Ben yesterday on assuring our logging is appropriate server-side too, but will await @knolleary approval before merging

@knolleary knolleary merged commit f2301ea into main Jan 31, 2023
@knolleary knolleary deleted the project-restart-option branch January 31, 2023 09:56
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.

3 participants