Upgrade build environment of sonar jobs on CircleCI

[ohbus]

Description

• Upgraded CircleCI to use JDK 11
• Should fix Upgrade Sonar CircleCI job to Java 11 #558
• Updated to docker based environment from machine

Type of change

• Bug fix (non-breaking change which fixes an issue)
• Feature (non-breaking change which adds functionality)
• Breaking change (fix or feature that would cause existing functionality to change
• Chore (non-breaking change which doesn't affect codebase;
  test, version modification, documentation, etc.)

Checklist

• I have read the contributing guidelines
• I signed-off my commit with a user that has signed the Eclipse Contributor Agreement
• I Added CHANGELOG entry
• I have implemented unit tests to cover my changes
• I have updated the documentation accordingly
• No new bugs, code smells, etc. in SonarCloud report
• I tested my code in Kubernetes
• I tested my code in OpenShift

[rohanKanojia]

Seems like ThorntailV2HealthCheckEnricherTest is failing on JDK11. I've seen this failure myself while working with JDK11. Maybe we should refactor/remove it.

[INFO] Results:
[INFO] 
[ERROR] Errors: 
[ERROR]   ThorntailV2HealthCheckEnricherTest.setUp:67 » ClassCast class jdk.internal.loa...
[ERROR]   ThorntailV2HealthCheckEnricherTest.setUp:67 » IllegalArgument Value of type or...
[ERROR]   ThorntailV2HealthCheckEnricherTest.setUp:67 » IllegalArgument Value of type or...
[ERROR]   ThorntailV2HealthCheckEnricherTest.setUp:67 » IllegalArgument Value of type or...

[manusa]

I'll take a look

[rohanKanojia]

ohk, Thanks a lot!

[ohbus]

Seems like ThorntailV2HealthCheckEnricherTest is failing on JDK11. I've seen this failure myself while working with JDK11. Maybe we should refactor/remove it.

[INFO] Results:
[INFO] 
[ERROR] Errors: 
[ERROR]   ThorntailV2HealthCheckEnricherTest.setUp:67 » ClassCast class jdk.internal.loa...
[ERROR]   ThorntailV2HealthCheckEnricherTest.setUp:67 » IllegalArgument Value of type or...
[ERROR]   ThorntailV2HealthCheckEnricherTest.setUp:67 » IllegalArgument Value of type or...
[ERROR]   ThorntailV2HealthCheckEnricherTest.setUp:67 » IllegalArgument Value of type or...

I believe for the same reason I was not able to build the project on my local system, as I am also using JDK 11.

But the GitHub Actions jobs I suppose are all on JDK 11 right?

You may raise an issue regarding this and mention me, so that I can take a look at it to see if I can lend some helping hand.

Thanks.

[manusa]

I believe for the same reason I was not able to build the project on my local system, as I am also using JDK 11.

But the GitHub Actions jobs I suppose are all on JDK 11 right?

You may raise an issue regarding this and mention me, so that I can take a look at it to see if I can lend some helping hand.

Thanks.

GitHub actions are on 11, but are only compiling the project (no unit tests are run).

These actions are used for e2e tests that are hosted in https://github.com/jkubeio/jkube-integration-tests/

I already created a PR with a fix #563. Once it's merged, you'll have to rebase your PR onto master and hopefully tests will be passing then and we'll be able to merge yours.

[sonarcloud]

Kudos, SonarCloud Quality Gate passed!

0 Bugs
0 Vulnerabilities
0 Security Hotspots
0 Code Smells

No Coverage information
No Duplication information

[codecov]

Codecov Report

Merging #562 (42394a1) into master (659a0c5) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master     #562   +/-   ##
=========================================
  Coverage     42.04%   42.04%           
  Complexity     2840     2840           
=========================================
  Files           402      402           
  Lines         19461    19461           
  Branches       2777     2777           
=========================================
  Hits           8183     8183           
  Misses        10287    10287           
  Partials        991      991           

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 659a0c5...42394a1. Read the comment docs.

[manusa]

LGTM, thank you very much for your contribution!