Cloud Cost UI goes blank with no cloud-integration.json as a secret

[jjarrett21]

What does this PR change?

• Checks for cloud cost status

Does this PR relate to any other PRs?

• no

How will this PR impact users?

• should show an error message and a docs link if they don't have cloud cost enabled

Does this PR address any GitHub or Zendesk issues?

• Closes Cloud Cost UI goes blank with no cloud-integration.json as a secret #2284

How was this PR tested?

• self

Does this PR require changes to documentation?

• no but the included docs link is wrong and @mattray will need to provide the correct one.

Have you labeled this PR and its corresponding Issue as "next release" if it should be part of the next OpenCost release? If not, why not?

[vercel]

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name	Status	Preview	Comments	Updated (UTC)
opencost	✅ Ready (Inspect)	Visit Preview	💬 Add feedback	Nov 28, 2023 11:34am

[jjarrett21]

Updated the styles to be a bit more consistent with what we have previously

[dwbrown2]

Would it be easy to remove the upper error box? If so, like that change?

[jjarrett21]

@dwbrown2 changed it so that it should only show one or the other.

[dwbrown2]

@jjarrett21 I'm just seeing the error message now, should we instead expect to see the "no cloud integration" message?

[jjarrett21]

how bout now?

[dwbrown2]

I'm not seeing anything in preview: https://opencost-qbpimiolx-kubecost.vercel.app/

Have a screenshot?

[jjarrett21]

@dwbrown2 😅

[dwbrown2]

Haven't reviewed code, but this looks good to me!

This isn't the right docs page though, we want to use an OpenCost page: https://docs.kubecost.com/using-kubecost/navigating-the-kubecost-ui/cloud-costs-explorer#installation-and-configuration

Nit: I might also open the docs link in a new window.

[jjarrett21]

done and done

[dwbrown2]

I'm still seeing kubecost docs referenced in code, but once that's done I think the UX is good! Would be great to get a maintainer review when ready.

[jjarrett21]

Oh @dwbrown2 I was moving fast and I thought the link you posted was the open cost one lol. I'll ask @mattray for the proper docs link

[dwbrown2]

I know @mattray is likely out for the weekend, so sharing this in case helpful: https://www.opencost.io/docs/configuration/#cloud-costs

We may want to change/improve but this at least feels like a fine starting point.

[mattray]

Tested locally