12435: fix for quick start Cluster Manager issue

[abhioncbr]

while working on my local build, I saw the //LIVEINSTANCES issue and found the reported bug related to it.

I think, this issue got introduced with the PR, Just putting a simple fix for the issue.

cc: @jadami10, @xiangfu0

[jadami10]

hmm, it seems I definitely introduced this, but can you tell which part of the home page is making this happen? It seems like the UI itself is unaffected, but there's just stray, bad requests?

[abhioncbr]

hmm, it seems I definitely introduced this, but can you tell which part of the home page is making this happen? It seems like the UI itself is unaffected, but there's just stray, bad requests?

I wasn't able to track from where the requests were getting originated. Console log was mentioning main.js 79161, but some of the requests for the LIVEINSTANCES was coming without clusterName and hence I thought to check whether we have the value for it or not.

[jadami10]

there's unfortunately no better testing here. so as long as you click around and nothing is actively broken, this is fine with me

[codecov-commenter]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 61.81%. Comparing base (59551e4) to head (2253dd1).
Report is 100 commits behind head on master.

Additional details and impacted files

@@             Coverage Diff              @@
##             master   #12610      +/-   ##
============================================
+ Coverage     61.75%   61.81%   +0.06%     
+ Complexity      207      198       -9     
============================================
  Files          2436     2450      +14     
  Lines        133233   133748     +515     
  Branches      20636    20743     +107     
============================================
+ Hits          82274    82677     +403     
- Misses        44911    44965      +54     
- Partials       6048     6106      +58     

Flag	Coverage Δ
custom-integration1	<0.01% <ø> (-0.01%)	⬇️
integration	<0.01% <ø> (-0.01%)	⬇️
integration1	<0.01% <ø> (-0.01%)	⬇️
integration2	0.00% <ø> (ø)
java-11	61.73% <ø> (+0.02%)	⬆️
java-21	61.69% <ø> (+0.07%)	⬆️
skip-bytebuffers-false	61.77% <ø> (+0.02%)	⬆️
skip-bytebuffers-true	61.65% <ø> (+33.92%)	⬆️
temurin	61.81% <ø> (+0.06%)	⬆️
unittests	61.81% <ø> (+0.06%)	⬆️
unittests1	46.92% <ø> (+0.03%)	⬆️
unittests2	27.71% <ø> (-0.02%)	⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

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

[karthi07]

@abhioncbr This is a minor frontend bug. I can create a PR if required.

The API was called before the clusterName useState was loaded.

[Jackie-Jiang]

@karthi07 I guess you just found the root cause! Yes please file a PR!

[karthi07]

@Jackie-Jiang created PR, let me know if additional details are required, this is my first PR for Pinot :)

#12632