restore rollbackSnapshot

[Steve-Mcl]

fizes #2026

Description

restore rollbackSnapshot

TODO

• capture this in a frontend api unit test (OR raise an issue to pick this up at a later time)
  • Issue raised: Ensure the "missing rollbackSnapshot function" is caught in an API test to prevent regression #2032

Related Issue(s)

#2026

Acceptance

• Test operation on staging before updating prod

Checklist

• I have read the contribution guidelines
• Suitable unit/system level tests have been added and they pass
  • Issue raised: Ensure the "missing rollbackSnapshot function" is caught in an API test to prevent regression #2032
• 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

[Pezmc]

Comment around renaming, but not going to block!

[Steve-Mcl]

@Pezmc could you please pick up the TODO if I raise an issue?

[Pezmc]

@Steve-Mcl What todo are you planning to open?

[knolleary]

@Pezmc the TODO is the fact this PR has no unit test to cover the new api being added.

[Pezmc]

This needs a basic happy path E2E acceptance test to ensure rollback instance is working and we don't his this regression again, happy to help write it if needed!

[Steve-Mcl]

This needs a basic happy path E2E acceptance test to ensure rollback instance is working and we don't his this regression again, happy to help write it if needed!

Will raise

"Ensure the "missing rollbackSnapshot function" is caught in an API test to prevent regression"

Done: #2032

[Pezmc]

@Steve-Mcl @knolleary And I spoke in a Huddle, happy for this PR to move forward now so we can backport a fix, but we must address #2032 as a follow up.