Use clusterUUID for discovery nodes remote path

[soosinha]

Description

DiscoveryNodes is being uploaded as part of remote cluster state. The remote path for this object is being constructed using stateUUID. Cluster UUID should be used for the path instead.

Related Issues

#15142

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.

[github-actions]

❌ Gradle check result for e4a5855: 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 225bb90: SUCCESS

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 71.81%. Comparing base (c725431) to head (225bb90).
Report is 5 commits behind head on main.

Additional details and impacted files

@@             Coverage Diff              @@
##               main   #15143      +/-   ##
============================================
- Coverage     71.85%   71.81%   -0.04%     
+ Complexity    62832    62786      -46     
============================================
  Files          5169     5169              
  Lines        294688   294688              
  Branches      42616    42616              
============================================
- Hits         211735   211623     -112     
- Misses        65561    65667     +106     
- Partials      17392    17398       +6     

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

[Bukhtawar]

I am assuming manifest will reflect the real path and so we are good with BWC? Does GC will automatically use the new path based on manifest as well?

[soosinha]

I am assuming manifest will reflect the real path and so we are good with BWC? Does GC will automatically use the new path based on manifest as well?

Yes. The manifest contains the actual path. The entities are downloaded by reading the manifest file. So bwc should not be impacted. GC also uses the path from manifest.