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

feat: Enclave connect #2117

Merged
merged 11 commits into from Feb 6, 2024
Merged

feat: Enclave connect #2117

merged 11 commits into from Feb 6, 2024

Conversation

laurentluce
Copy link
Contributor

@laurentluce laurentluce commented Feb 5, 2024

Description:

We want the user to be able to connect from their local machine to an enclave with a start from the UI. A connect button is added to the enclave page with the CLI commands to paste. A CLI enclave connect command is added.

Is this change user facing?

YES

References (if applicable):

(https://www.notion.so/kurtosistech/Connect-local-CLI-to-running-enclave-workflow-for-cloud-based-enclaves-via-frontend-636d41ca937c4e49b54161aa498704b1?pvs=4)

Copy link

cloudflare-pages bot commented Feb 5, 2024

Deploying with  Cloudflare Pages  Cloudflare Pages

Latest commit: 690d92a
Status: ✅  Deploy successful!
Preview URL: https://4d23065e.kurtosis-docs.pages.dev
Branch Preview URL: https://laurent-enclave-connect.kurtosis-docs.pages.dev

View logs

Copy link
Contributor

@adschwartz adschwartz left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good, just a few comments to check out

@laurentluce laurentluce added this pull request to the merge queue Feb 6, 2024
Merged via the queue into main with commit d33de7b Feb 6, 2024
38 checks passed
@laurentluce laurentluce deleted the laurent/enclave-connect branch February 6, 2024 21:30
github-merge-queue bot pushed a commit that referenced this pull request Feb 7, 2024
#2127)

## Description:
The enclave connect button was the wrong one in the EM UI. Updated to
one corresponding to the action.

## Is this change user facing?
YES

## References (if applicable):
#2117
github-merge-queue bot pushed a commit that referenced this pull request Feb 7, 2024
🤖 I have created a release *beep* *boop*
---


##
[0.86.15](0.86.14...0.86.15)
(2024-02-07)


### Features

* Enclave connect
([#2117](#2117))
([d33de7b](d33de7b))


### Bug Fixes

* Fix the enclave connect button and modal help string in the EM UI
([#2127](#2127))
([a11fb4c](a11fb4c))

---
This PR was generated with [Release
Please](https://github.com/googleapis/release-please). See
[documentation](https://github.com/googleapis/release-please#release-please).

---------

Co-authored-by: kurtosisbot <kurtosisbot@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants