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

DX-805 get and show pod connection details for livebook smart cell #228

Merged
merged 4 commits into from
Oct 2, 2024

Conversation

Fadhil
Copy link
Contributor

@Fadhil Fadhil commented Sep 30, 2024

Summary

In this PR, we update the kubernetes_pod query to additionally get the node_name and cookie for the pod and display those values along with a short instruction on how the user can connect to a selected node.

Ticket: DX-805

The changes in action:

Screenshot 2024-10-01 at 3 35 16 PM

Testing out the provided details in a livebook:
Screenshot 2024-09-30 at 6 58 57 PM

What's Changed

Show pod connection details and instructions on how to use them in the application instances connect flow

Updated schema and kubernetes_pods.graphql to get node_name and cookie

@Fadhil Fadhil force-pushed the feature/show-smart-cell-details branch from 2ded9f0 to 00d38a6 Compare September 30, 2024 11:33
@onimsha onimsha requested a review from jk-gan October 1, 2024 03:04
@Fadhil Fadhil requested a review from jk-gan October 1, 2024 07:16
@sauron-droid
Copy link

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: Fadhil, jk-gan

The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@onimsha onimsha merged commit c92cbba into main Oct 2, 2024
6 checks passed
@sauron-droid sauron-droid deleted the feature/show-smart-cell-details branch October 2, 2024 07:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Development

Successfully merging this pull request may close these issues.

4 participants