Skip to content

Commit 8ca8f3d

Browse files
committed
Permanently switch to the orchestrator for evaluate/compile/execute
1 parent 3baab79 commit 8ca8f3d

File tree

6 files changed

+16
-784
lines changed

6 files changed

+16
-784
lines changed

.github/workflows/ci.yml

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -254,7 +254,6 @@ jobs:
254254
PLAYGROUND_UI_ROOT: server/build/
255255
PLAYGROUND_CORS_ENABLED: true
256256
PLAYGROUND_GITHUB_TOKEN: "${{ secrets.PLAYGROUND_GITHUB_TOKEN }}"
257-
PLAYGROUND_ORCHESTRATOR_ENABLED: 1
258257
PLAYGROUND_EXECUTE_VIA_WEBSOCKET_THRESHOLD: 1.0
259258
run: |-
260259
killall -q ui || true

ci/workflows.yml

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -362,7 +362,6 @@ workflows:
362362
PLAYGROUND_UI_ROOT: server/build/
363363
PLAYGROUND_CORS_ENABLED: true
364364
PLAYGROUND_GITHUB_TOKEN: ${{ secrets.PLAYGROUND_GITHUB_TOKEN }}
365-
PLAYGROUND_ORCHESTRATOR_ENABLED: 1
366365
PLAYGROUND_EXECUTE_VIA_WEBSOCKET_THRESHOLD: 1.0
367366
run: |-
368367
killall -q ui || true

ui/src/main.rs

Lines changed: 0 additions & 14 deletions
Original file line numberDiff line numberDiff line change
@@ -45,7 +45,6 @@ struct Config {
4545
cors_enabled: bool,
4646
gh_token: Option<String>,
4747
metrics_token: Option<String>,
48-
orchestrator_enabled: bool,
4948
feature_flags: FeatureFlags,
5049
port: u16,
5150
root: PathBuf,
@@ -100,8 +99,6 @@ impl Config {
10099

101100
let cors_enabled = env::var_os("PLAYGROUND_CORS_ENABLED").is_some();
102101

103-
let orchestrator_enabled = env::var_os("PLAYGROUND_ORCHESTRATOR_ENABLED").is_some();
104-
105102
let execute_via_websocket_threshold =
106103
env::var("PLAYGROUND_EXECUTE_VIA_WEBSOCKET_THRESHOLD")
107104
.ok()
@@ -116,7 +113,6 @@ impl Config {
116113
cors_enabled,
117114
gh_token,
118115
metrics_token,
119-
orchestrator_enabled,
120116
feature_flags,
121117
port,
122118
root,
@@ -135,10 +131,6 @@ impl Config {
135131
self.cors_enabled
136132
}
137133

138-
fn use_orchestrator(&self) -> bool {
139-
self.orchestrator_enabled
140-
}
141-
142134
fn metrics_token(&self) -> Option<MetricsToken> {
143135
self.metrics_token.as_deref().map(MetricsToken::new)
144136
}
@@ -182,12 +174,6 @@ impl MetricsToken {
182174
enum Error {
183175
#[snafu(display("Sandbox creation failed: {}", source))]
184176
SandboxCreation { source: sandbox::Error },
185-
#[snafu(display("Compilation operation failed: {}", source))]
186-
Compilation { source: sandbox::Error },
187-
#[snafu(display("Execution operation failed: {}", source))]
188-
Execution { source: sandbox::Error },
189-
#[snafu(display("Evaluation operation failed: {}", source))]
190-
Evaluation { source: sandbox::Error },
191177
#[snafu(display("Linting operation failed: {}", source))]
192178
Linting { source: sandbox::Error },
193179
#[snafu(display("Expansion operation failed: {}", source))]

ui/src/metrics.rs

Lines changed: 0 additions & 13 deletions
Original file line numberDiff line numberDiff line change
@@ -425,19 +425,6 @@ where
425425
track_metric_common_async(request, body, |_| {}).await
426426
}
427427

428-
pub(crate) async fn track_metric_force_endpoint_async<Req, B, Resp>(
429-
request: Req,
430-
endpoint: Endpoint,
431-
body: B,
432-
) -> sandbox::Result<Resp>
433-
where
434-
Req: GenerateLabels,
435-
for<'req> B: FnOnce(&'req Req) -> BoxFuture<'req, sandbox::Result<Resp>>,
436-
Resp: SuccessDetails,
437-
{
438-
track_metric_common_async(request, body, |labels| labels.endpoint = endpoint).await
439-
}
440-
441428
async fn track_metric_common_async<Req, B, Resp, F>(
442429
request: Req,
443430
body: B,

0 commit comments

Comments
 (0)