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

chore: revert EM UI build changes #1813

Merged
merged 3 commits into from
Nov 16, 2023

Conversation

Dartoxian
Copy link
Contributor

Description:

This PR reverts the changes to the enclave build process

@Dartoxian Dartoxian requested a review from adschwartz November 16, 2023 22:19
@adschwartz adschwartz changed the title chore: revert enclave build step chore: revert EM UI build changs Nov 16, 2023
@adschwartz adschwartz changed the title chore: revert EM UI build changs chore: revert EM UI build changes Nov 16, 2023
@adschwartz adschwartz merged commit 76029c8 into main Nov 16, 2023
17 of 19 checks passed
@adschwartz adschwartz deleted the bgazzard/chore-remove-enclave-build-step branch November 16, 2023 22:21
Dartoxian added a commit that referenced this pull request Nov 17, 2023
@Dartoxian Dartoxian mentioned this pull request Nov 17, 2023
github-merge-queue bot pushed a commit that referenced this pull request Jan 10, 2024
## Description:
This PR brings the emui into the ci build/release process. It is a
reversion of last nights
#1813.

Yesterday the build/release change for emui was abandoned because after
several attempts the publish step was not working. In
4ed9328
I have demonstrated that this pr works by:
* Removing the body of the publish jobs and replacing them with echo
statements
* Triggering publish jobs on `bgazzard-*` tags
* Pushing a tag `bgazzard-0.0.1` (which I have now deleted) to trigger
the `publish` workflow
* The publish workflow can be seen here
https://app.circleci.com/pipelines/github/kurtosis-tech/kurtosis/9407/workflows/ca6cef3b-4f71-481a-ab08-8787002ed9d0
and demonstrates that the `build_enclave_manager_webapp` job is now
successfully run as a dependency of `publish_engine_server_image`.

The existing build/test path which was approved previously is unchanged
by this pr and can be seen working in this workflow
https://app.circleci.com/pipelines/github/kurtosis-tech/kurtosis/9408/workflows/bb48e469-2ff7-433e-8da9-ad32620b0bbc.

## Is this change user facing?
NO
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.

2 participants