Remove redundant condition in jvmStats test

[lukas-vlcek]

Description

Removing second use of the frequentyl() function. This function is alraedy used in the beginning of the testing block immediately after the declaration of jvmStats variable. The second use of the frequently() could cause all the other variables that were carefully prepared for the constructor to not be used at all.

Related Issues

n/a

Check List

• Functionality includes testing.
  - [ ] API changes companion pull request created, if applicable.
  - [ ] Public documentation issue/PR created, if applicable.

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

[lukas-vlcek]

I do not think this needs to go into CHANGELOG (unless the process was changed). Can someone add skip-changelog flog to this PR please?

[github-actions]

❌ Gradle check result for eddf6ee: FAILURE

Please examine the workflow log, locate, and copy-paste the failure(s) below, then iterate to green. Is the failure a flaky test unrelated to your change?

[github-actions]

❌ Gradle check result for f5bf84b: FAILURE

Please examine the workflow log, locate, and copy-paste the failure(s) below, then iterate to green. Is the failure a flaky test unrelated to your change?

[andrross]

@lukas-vlcek Looks like a spotless formatting error on the build

[lukas-vlcek]

@andrross Ooops, sorry, fixed.

[github-actions]

❌ Gradle check result for 54dc5e4:

Please examine the workflow log, locate, and copy-paste the failure(s) below, then iterate to green. Is the failure a flaky test unrelated to your change?

[github-actions]

❌ Gradle check result for 54dc5e4:

Please examine the workflow log, locate, and copy-paste the failure(s) below, then iterate to green. Is the failure a flaky test unrelated to your change?

[lukas-vlcek]

I am not sure about the build failure:

cause-exception : com.thoughtworks.xstream.mapper.CannotResolveClassException
cause-message : com.michelin.cio.hudson.plugins.rolestrategy.RoleBasedAuthorizationStrategy
class : hudson.model.Hudson required-type : hudson.model.Hudson
converter-type : hudson.util.RobustReflectionConverter
path : /hudson/authorizationStrategy line number : 10 version : not available

Is this a CI system failure?

[dblock]

Is this a CI system failure?

Probably. Kicked it.

[github-actions]

❕ Gradle check result for 54dc5e4: UNSTABLE

• TEST FAILURES:

      1 org.opensearch.repositories.azure.AzureBlobContainerRetriesTests.testWriteBlobWithRetries

Please review all flaky tests that succeeded after retry and create an issue if one does not already exist to track the flaky failure.

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 71.61%. Comparing base (b15cb0c) to head (54dc5e4).
Report is 385 commits behind head on main.

Additional details and impacted files

@@             Coverage Diff              @@
##               main   #14075      +/-   ##
============================================
+ Coverage     71.42%   71.61%   +0.19%     
- Complexity    59978    61554    +1576     
============================================
  Files          4985     5081      +96     
  Lines        282275   289071    +6796     
  Branches      40946    41836     +890     
============================================
+ Hits         201603   207008    +5405     
- Misses        63999    65004    +1005     
- Partials      16673    17059     +386     

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