ApplyMojo.java: Either re-interrupt this method or rethrow the "InterruptedException" that can be caught here.

[manusa]

Description

The following statement is reported as a Bug by SonarCloud:

https://github.com/eclipse/jkube/blob/af40c8f27af57f9651f8ebb1634df33eaf909f6a/kubernetes-maven-plugin/plugin/src/main/java/org/eclipse/jkube/maven/plugin/mojo/build/ApplyMojo.java#L194-L196

• should be changed to:

        } catch(InterruptedException ex) {
            Thread.currentThread().interrupt();
        } catch (Exception e) {
            throw new MojoExecutionException(e.getMessage(), e);
        }

How to manually test my changes

Kubernetes

If you don't have a real Kubernetes cluster available (most probably), you can use Minikube or Kind to test with a local cluster.

• Minikube get started guide
• Kind (Requires Docker) quick start

OpenShift

If you don't have a real OpenShift cluster available (most probably), you can use Red Hat's developer Sandbox for Red Hat OpenShift. The only requirement is to have a Red Hat account.

Once you have your Sandbox environment, you'll need to download the oc tool from the cluster console.
(Press the ? icon and from the context menu select Command line tools, you'll be redirected to https://$subdomain.openshiftapps.com/command-lines-tools where you'll be able to download the CLI for your platform)

[prathamgandhi]

I'd like to do this

[prathamgandhi]

@manusa could you take a look why the fix didn't work. I submitted the PR without testing with kubernetes, because when I ran it, my laptop started freezing and all the RAM got used up. I thought since I had just added a catch statement, there should be no error, but apparently I was wrong :(