feat(ws): Implement Start restart and stop workspace actions

[ElayAharoni]

Fixes: #146

Implemented Start, restart and stop workspace actions present in the workspace table actions kebab dropdown.
with the requested buttons and warning, from the issue description.
snapshots:

1. start with pending updates:
   

2. restart without pending updates:
   

3. stop without pending updates:
   

4. stop with pending updates:
   

5. restart with pending updates:
   

[paulovmr]

Hi @ElayAharoni , thanks for the PR. IMO, we should have one component (and its .tsx file) for each of those modals, instead of managing UI features as React states. This way each component would be isolated and could call the appropriate BFF service to execute its action. In the Workspaces.tsx, we would have only the const [isXModalOpen, setIsXModalOpen] = React.useState(false); boolean state to manage if each popup is opened or not.

This idea is similar to what Model Registry does with ModelVersionsTableRow using the ArchiveModelVersionModal modal.

[ElayAharoni]

i have made changes according to @thesuperzapper requests.
also this is the change after rebasing the latest changes.
https://www.loom.com/share/50623d12b176400e9d9ef1c0189b2afa

[paulovmr]

Hi @ElayAharoni , thanks for adding all the redirect information on this UI. I still think we should have one component (in its own .tsx file) for each of those modals as we discussed before, with a reusable component in a separate file for the redirects information. The WorkspaceActionAlert currently has a lot of responsibilities, and the separation would help the understanding and maintainability of this code.

[ElayAharoni]

hi @paulovmr i did the requested changes

[paulovmr]

Thanks @ElayAharoni , the components modularity looks good to me now! I've left a few comments inline, and I have a few suggestions regarding some naming:

• Consider renaming RestartActionAlert to WorkspaceRestartActionModal (and similarly to the others)
• Consider renaming ActionWithUpdatesBody to WorkspaceRedirectInformationView
• Consider changing the activeActionData state to be only a enum with the type of the action. The already existing selectedWorkspace state could be used to hold the workspace and passed to the modal directly. Also please consider renaming it to activeActionType.

[google-oss-prow]

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: ederign, paulovmr

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Needs approval from an approver in each of these files:

• workspaces/frontend/OWNERS [ederign]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

[ederign]

/lgtm