Fix the bug when kubeconfig file content is collapsed to one line after workspace restart

[vinokurig]

What does this PR do?

1. Change the parse format from JSON to YAML as the default format of the oc and kubectl tools is YAML.
2. Do not substring new line when fetching the kubeconfig file content.

Screenshot/screencast of this PR

What issues does this PR fix or reference?

fixes eclipse-che/che#23238

Is it tested? How?

1. Deploy Che with the dashboard pull request image: quay.io/eclipse/che-dashboard:pr-1279
2. Apply Persistent user home.
3. Start a workspace, open the /home/user/.kube/config file, see that the content is YAML.
4. Open a new terminal and oc login to a kubernetes cluster.
5. Restart the workspace, execute oc whoami from the terminal.

See: the oc command works, the kubeconfig file is valid.

Release Notes

Docs PR

[che-bot]

Click here to review and test in web IDE:

[olexii4]

@vinokurig Could you add the proper test for your changes(for the mergeKubeConfig method)?

[github-actions]

Docker image build succeeded: quay.io/eclipse/che-dashboard:pr-1279

kubectl patch command

kubectl patch -n eclipse-che "checluster/eclipse-che" --type=json -p="[{"op": "replace", "path": "/spec/components/dashboard/deployment", "value": {containers: [{image: "quay.io/eclipse/che-dashboard:pr-1279", name: che-dashboard}]}}]"

[ibuziuk]

@vinokurig would it be possible to add test(s) for that ?

[codecov]

Codecov Report

Attention: Patch coverage is 30.00000% with 14 lines in your changes missing coverage. Please review.

Project coverage is 91.80%. Comparing base (6be3790) to head (fed10a3).
Report is 4 commits behind head on main.

Files with missing lines	Patch %	Lines
...d/src/devworkspaceClient/services/kubeConfigApi.ts	31.57%	13 Missing ⚠️
...nd/src/devworkspaceClient/services/helpers/exec.ts	0.00%	1 Missing ⚠️

Additional details and impacted files

@@            Coverage Diff             @@
##             main    #1279      +/-   ##
==========================================
+ Coverage   91.69%   91.80%   +0.10%     
==========================================
  Files         499      499              
  Lines       45651    45653       +2     
  Branches     3178     3187       +9     
==========================================
+ Hits        41860    41912      +52     
+ Misses       3760     3711      -49     
+ Partials       31       30       -1     

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

[vinokurig]

@ibuziuk @olexii4 Added test.

[olexii4]

@vinokurig Could you fix the build-and-test checks, please?

[github-actions]

Docker image build succeeded: quay.io/eclipse/che-dashboard:pr-1279

kubectl patch command

kubectl patch -n eclipse-che "checluster/eclipse-che" --type=json -p="[{"op": "replace", "path": "/spec/components/dashboard/deployment", "value": {containers: [{image: "quay.io/eclipse/che-dashboard:pr-1279", name: che-dashboard}]}}]"

[olexii4]

LGTM

[artaleks9]

Verified on Eclipse Che with quay.io/eclipse/che-dashboard:pr-1279 - the functionality works properly

[openshift-ci]

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: akurinnoy, artaleks9, ibuziuk, olexii4, vinokurig

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

Needs approval from an approver in each of these files:

• OWNERS

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

[devspacesbuild]

Build 3.19 :: dashboard_3.x/607: Console, Changes, Git Data

[devspacesbuild]

Build 3.19 :: sync-to-downstream_3.x/8346: Console, Changes, Git Data

[devspacesbuild]

Build 3.19 :: get-sources-rhpkg-container-build_3.x/8442: Console, Changes, Git Data

[devspacesbuild]

Build 3.19 :: get-sources-rhpkg-container-build_3.x/8442:

dashboard : 3.x :: Failed in : BREW:BUILD/STATUS:UNKNOWN
FAILURE:; copied to quay