Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix: Display managed clusters when itsData is null #164

Conversation

krishkumar84
Copy link
Contributor

Display managed clusters when itsData is null

Description

This PR implements a fallback mechanism to display local Kubernetes clusters in the Managed Clusters page when itsData is null. Previously, the page would show an empty table even when valid local clusters were present.

Related Issue

Fixes #163

Changes Made

  • Added fallback logic to handle null itsData in ITS.tsx
  • Added ContextInfo interface for type safety
  • Implemented context to managed cluster format conversion
  • Added proper filtering for -kubeflex contexts

Checklist

  • I have reviewed the project's contribution guidelines.
  • I have tested the changes locally and ensured they work as expected.
  • My code follows the project's coding standards.
  • The code properly handles edge cases (null/undefined values).

Screenshots or Logs (if applicable)

Before:
image

After:
image

Additional Notes

This change improves the developer experience by showing local clusters during development, while maintaining compatibility with the existing itsData structure.

- Add fallback to show local clusters
- Convert contexts to managed cluster format
- Add ContextInfo interface
@krishkumar84
Copy link
Contributor Author

@clubanderson can you review this

@MAVRICK-1 MAVRICK-1 self-requested a review February 15, 2025 16:22
@krishkumar84
Copy link
Contributor Author

hey @MAVRICK-1 can you help me in joining the slack can you send me invitation it would be very helpful?

@clubanderson
Copy link
Contributor

@krishkumar84 Thank you for the PR. Unfortunately the ITS managedcluster list does not include the "kubeflex" clusters. ITS is the KubeStellar inventory and transport service. Any object in an ITS of type managedcluster is what belongs in this dialog.

@clubanderson
Copy link
Contributor

@krishkumar84 I opened a ticket with CNCF for their slack inviter being down.

@clubanderson
Copy link
Contributor

@krishkumar84 the link for CNCF invite is working now

@clubanderson
Copy link
Contributor

@krishkumar84 are there other issues you would like to work on?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

[BUG] Managed Clusters not displaying when itsData is null
2 participants