feat: ability to edit kube yaml

[cdrage]

feat: ability to edit kube yaml

What does this PR do?

• Adds the ability to edit kube yaml with two buttons, "revert" and
  "apply"
• Renames Kube section to "Edit"

Screenshot / video of UI

Screen.Recording.2024-02-09.at.4.46.08.PM.mov

What issues does this PR fix or reference?

Closes #5662

How to test this PR?

1. Go to Kube tab
2. Edit a YAML (Deployment only right now)
3. Click on apply / revert buttons.

Signed-off-by: Charlie Drage charlie@charliedrage.com

[slemeur]

linking to #519

[odockal]

@cdrage Nice feat. It smells like some E2E tests will be needed. Can you please open new epic similar to ie. this (#5484) one?

[cdrage]

@cdrage Nice feat. It smells like some E2E tests will be needed. Can you please open new epic similar to ie. this (#5484) one?

Created! #6001

[cdrage]

Notes!

This is a very simple implementation of applying the Kubernetes YAML to the cluster / being able to edit.

However there are NO tests for MonacoEditor. There was already no tests for MonacoEditor, as well as the fact that monaco-editor is dynamically important making it difficult to test see [1] and [2].

There have been tests added however for the Kubernetes client.

I have also opened up #6001 in order to create an e2e test for editing YAML which has been assigned to @odockal

[feloy]

Tested editing recplicas, image of pod, rollingUpdate params, works great!

[cdrage]

@feloy You're right! That'd be beneficial to have some form of explanation of what's going on when pressing apply.

I couldn't find a good way to add this as a description on the same page (no space as the editor fills it entirely), but instead I've added a tooltip.

Screen.Recording.2024-02-22.at.11.00.29.AM.mov

[deboer-tim]

Tested. This is super cool, and really handy to make quick edits. 🚀

Some nits:

• I still prefer calling this tab 'Kube' or "YAML'. Yes, it's where you edit, but we could allow editing in other tabs and I'd rather name it based on what it is.
• Apply should be disabled until you type something, and disabled again if you revert.
• Likewise, Revert should be disabled until you type, and again after reverting.
• Our buttons are transparent (to work on any background), which makes it a little awkward b/c the editor text can show up through the button. Could be a follow-up PR, maybe we allow an option to set the bg.

In the future I'd also like an option to hide/remove things like status/managedFields/last-applied-configuration to make it easier to see and edit just the configuration of a resource - probably a good follow-up question for UX call.

[cdrage]

In the future I'd also like an option to hide/remove things like status/managedFields/last-applied-configuration to make it easier to see and edit just the configuration of a resource - probably a good follow-up question for UX call.

Ready for another review!

I've implemented the following changes:

• Buttons are disabled until a change is detected
• It's Kube not Edit

Issue for managed fields is here: #5936

Issue for transparent background for buttons is here: #6117

Ready for another test / review @feloy @deboer-tim

[lstocchi]

LGTM

[feloy]

LGTM, very useful feature, thanks

[cdrage]

Merging as it's been now tested multiple times and reviewed by @deboer-tim @feloy and @lstocchi

Thank you for being patient and doing lots of testing on this!