You are viewing a plain text version of this content. The canonical link for it is here.
Posted to notifications@texera.apache.org by "xuang7 (via GitHub)" <gi...@apache.org> on 2026/07/05 02:39:57 UTC
[PR] fix: add missing /pve route to single-node nginx [texera]
xuang7 opened a new pull request, #6130:
URL: https://github.com/apache/texera/pull/6130
### What changes were proposed in this PR?
The single-node NGINX gateway (`bin/single-node/nginx.conf`) has no `location` block for `/pve/`, so Python Virtual Environment (PVE) REST calls fall through to the catch-all `location /` → `dashboard-service`, which does not serve them and returns **404**. In the UI this surfaces as a *"Could not load Python virtual environments"* popup on every Python UDF operator selection.
The PVE endpoints are served by the amber webserver (`PveResource`, `@Path("/pve")`, Jersey mounted under `/api/*`) on `workflow-runtime-coordinator-service:8085`. Both the Angular dev proxy (`frontend/proxy.config.json`) and the Kubernetes gateway (`bin/k8s/templates/base/gateway/gateway-routes.yaml`) already route `/pve` → `/api/pve` to that service; the single-node NGINX config was the only gateway missing it.
This adds the matching `/pve/` block, rewriting `/pve/*` → `/api/pve/*` before proxying to the coordinator — mirroring the dev proxy and k8s gateway.
### Any related issues, documentation, discussions?
Related to #5963 (PVE single-node configuration).
### How was this PR tested?
_Pending — manual verification on the single-node Docker stack before this is marked ready for review._
### Was this PR authored or co-authored using generative AI tooling?
Generated-by: Claude Code (Claude Opus 4.8)
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: notifications-unsubscribe@texera.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] fix: serve PVE requests under /api/pve and drop per-gateway rewrites [texera]
Posted by "Yicong-Huang (via GitHub)" <gi...@apache.org>.
Yicong-Huang commented on code in PR #6130:
URL: https://github.com/apache/texera/pull/6130#discussion_r3548022524
##########
frontend/src/app/workspace/service/virtual-environment/virtual-environment.service.spec.ts:
##########
@@ -45,7 +46,7 @@ describe("WorkflowPveService", () => {
expect(resp.veid).toBe(42);
});
- const req = httpTestingController.expectOne("/pve/db");
+ const req = httpTestingController.expectOne(`${AppSettings.getApiEndpoint()}/pve/db`);
Review Comment:
`/pve/db` is just too bad of name choice: by reading the api, I don't know what operation it is conduction. we should refactor it separately
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: notifications-unsubscribe@texera.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] fix: serve PVE requests under /api/pve and drop per-gateway rewrites [texera]
Posted by "Yicong-Huang (via GitHub)" <gi...@apache.org>.
Yicong-Huang commented on code in PR #6130:
URL: https://github.com/apache/texera/pull/6130#discussion_r3548017511
##########
bin/single-node/nginx.conf:
##########
@@ -95,6 +95,12 @@ http {
proxy_set_header Host $host;
}
+ location /api/pve/ {
+ proxy_pass http://workflow-runtime-coordinator-service:8085;
+ proxy_set_header Host $host;
+ proxy_set_header X-Real-IP $remote_addr;
+ }
Review Comment:
Doesn’t this need to be placed above the /api/ entry? I think the routing uses first-match semantics, right?
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: notifications-unsubscribe@texera.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] fix: serve PVE requests under /api/pve and drop per-gateway rewrites [texera]
Posted by "xuang7 (via GitHub)" <gi...@apache.org>.
xuang7 commented on code in PR #6130:
URL: https://github.com/apache/texera/pull/6130#discussion_r3562547002
##########
bin/single-node/nginx.conf:
##########
@@ -95,6 +95,12 @@ http {
proxy_set_header Host $host;
}
+ location /api/pve/ {
+ proxy_pass http://workflow-runtime-coordinator-service:8085;
+ proxy_set_header Host $host;
+ proxy_set_header X-Real-IP $remote_addr;
+ }
Review Comment:
Updated, thanks.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: notifications-unsubscribe@texera.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] fix(pve): serve PVE requests under /api/pve and drop per-gateway rewrites [texera]
Posted by "xuang7 (via GitHub)" <gi...@apache.org>.
xuang7 merged PR #6130:
URL: https://github.com/apache/texera/pull/6130
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: notifications-unsubscribe@texera.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] fix: add missing /pve route to single-node nginx [texera]
Posted by "github-actions[bot] (via GitHub)" <gi...@apache.org>.
github-actions[bot] commented on PR #6130:
URL: https://github.com/apache/texera/pull/6130#issuecomment-4884597416
<!-- texera-reviewer-suggestion -->
### Automated Reviewer Suggestions
Based on the `git blame` history of the changed files, we recommend the following reviewers:
- **Contributors with relevant context:** `@bobbai00`
You can notify them by mentioning `@bobbai00` in a comment.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: notifications-unsubscribe@texera.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] fix: serve PVE requests under /api/pve and drop per-gateway rewrites [texera]
Posted by "xuang7 (via GitHub)" <gi...@apache.org>.
xuang7 commented on code in PR #6130:
URL: https://github.com/apache/texera/pull/6130#discussion_r3549359900
##########
frontend/src/app/workspace/service/virtual-environment/virtual-environment.service.spec.ts:
##########
@@ -45,7 +46,7 @@ describe("WorkflowPveService", () => {
expect(resp.veid).toBe(42);
});
- const req = httpTestingController.expectOne("/pve/db");
+ const req = httpTestingController.expectOne(`${AppSettings.getApiEndpoint()}/pve/db`);
Review Comment:
Currently, /pve/db handles CRUD operations for the user’s saved environment definitions, while /pve/pves lists the environments materialized on a computing unit. I agree that the naming is not very clear and should be refactored separately.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: notifications-unsubscribe@texera.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] fix: add missing /pve route to single-node nginx [texera]
Posted by "Yicong-Huang (via GitHub)" <gi...@apache.org>.
Yicong-Huang commented on code in PR #6130:
URL: https://github.com/apache/texera/pull/6130#discussion_r3532635799
##########
bin/single-node/nginx.conf:
##########
@@ -95,6 +95,12 @@ http {
proxy_set_header Host $host;
}
+ location /pve/ {
Review Comment:
we should change this to `/api/pve/` in the src code instead of doing a mapping here.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: notifications-unsubscribe@texera.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] fix: serve PVE requests under /api/pve and drop per-gateway rewrites [texera]
Posted by "xuang7 (via GitHub)" <gi...@apache.org>.
xuang7 commented on code in PR #6130:
URL: https://github.com/apache/texera/pull/6130#discussion_r3539903517
##########
bin/single-node/nginx.conf:
##########
@@ -95,6 +95,12 @@ http {
proxy_set_header Host $host;
}
+ location /pve/ {
Review Comment:
Updated, thanks!
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: notifications-unsubscribe@texera.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] fix: serve PVE requests under /api/pve and drop per-gateway rewrites [texera]
Posted by "Yicong-Huang (via GitHub)" <gi...@apache.org>.
Yicong-Huang commented on code in PR #6130:
URL: https://github.com/apache/texera/pull/6130#discussion_r3548028904
##########
frontend/src/app/workspace/service/virtual-environment/virtual-environment.service.ts:
##########
@@ -73,27 +74,27 @@ export class WorkflowPveService {
getSystemPackages(cuid: number): Observable<PackageResponse> {
const params = this.buildBaseParams().set("cuid", cuid.toString());
- return this.http.get<PackageResponse>("/pve/system", { params });
+ return this.http.get<PackageResponse>(`${AppSettings.getApiEndpoint()}/pve/system`, { params });
}
fetchPVEs(cuid: number): Observable<PvePackageResponse[]> {
const params = this.buildBaseParams().set("cuid", cuid.toString());
- return this.http.get<PvePackageResponse[]>("/pve/pves", { params });
+ return this.http.get<PvePackageResponse[]>(`${AppSettings.getApiEndpoint()}/pve/pves`, { params });
}
getUserPackages(cuid: number, pveName: string): Observable<string[]> {
return this.fetchPVEs(cuid).pipe(map(pves => pves.find(pve => pve.pveName === pveName)?.userPackages ?? []));
}
deleteEnvironments(cuid: number) {
- return this.http.delete(`/pve/pves/${cuid}`);
+ return this.http.delete(`${AppSettings.getApiEndpoint()}/pve/pves/${cuid}`);
Review Comment:
same for this, `/pve/pves` what dose it do? the naming is not self-contained
##########
frontend/src/app/workspace/service/virtual-environment/virtual-environment.service.spec.ts:
##########
@@ -45,7 +46,7 @@ describe("WorkflowPveService", () => {
expect(resp.veid).toBe(42);
});
- const req = httpTestingController.expectOne("/pve/db");
+ const req = httpTestingController.expectOne(`${AppSettings.getApiEndpoint()}/pve/db`);
Review Comment:
`/pve/db` is just too bad of name choice: by reading the api, I don't know what operation it is conducting. we should refactor it separately
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: notifications-unsubscribe@texera.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] fix: serve PVE requests under /api/pve and drop per-gateway rewrites [texera]
Posted by "xuang7 (via GitHub)" <gi...@apache.org>.
xuang7 commented on code in PR #6130:
URL: https://github.com/apache/texera/pull/6130#discussion_r3549362912
##########
frontend/src/app/workspace/service/virtual-environment/virtual-environment.service.ts:
##########
@@ -73,27 +74,27 @@ export class WorkflowPveService {
getSystemPackages(cuid: number): Observable<PackageResponse> {
const params = this.buildBaseParams().set("cuid", cuid.toString());
- return this.http.get<PackageResponse>("/pve/system", { params });
+ return this.http.get<PackageResponse>(`${AppSettings.getApiEndpoint()}/pve/system`, { params });
}
fetchPVEs(cuid: number): Observable<PvePackageResponse[]> {
const params = this.buildBaseParams().set("cuid", cuid.toString());
- return this.http.get<PvePackageResponse[]>("/pve/pves", { params });
+ return this.http.get<PvePackageResponse[]>(`${AppSettings.getApiEndpoint()}/pve/pves`, { params });
}
getUserPackages(cuid: number, pveName: string): Observable<string[]> {
return this.fetchPVEs(cuid).pipe(map(pves => pves.find(pve => pve.pveName === pveName)?.userPackages ?? []));
}
deleteEnvironments(cuid: number) {
- return this.http.delete(`/pve/pves/${cuid}`);
+ return this.http.delete(`${AppSettings.getApiEndpoint()}/pve/pves/${cuid}`);
Review Comment:
I agree that the naming is not very clear and should be refactored separately.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: notifications-unsubscribe@texera.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] fix: serve PVE requests under /api/pve and drop per-gateway rewrites [texera]
Posted by "Yicong-Huang (via GitHub)" <gi...@apache.org>.
Yicong-Huang commented on code in PR #6130:
URL: https://github.com/apache/texera/pull/6130#discussion_r3561338469
##########
bin/single-node/nginx.conf:
##########
@@ -95,6 +95,12 @@ http {
proxy_set_header Host $host;
}
+ location /api/pve/ {
+ proxy_pass http://workflow-runtime-coordinator-service:8085;
+ proxy_set_header Host $host;
+ proxy_set_header X-Real-IP $remote_addr;
+ }
Review Comment:
OK, is there a way we can document this " longest-prefix matching rather than first-match matching" next to the conf?
even though it is using `longest-prefix matching`, can we still move it up to be closer to other `/api/xxx` endpoints?
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: notifications-unsubscribe@texera.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] fix: add missing /pve route to single-node nginx [texera]
Posted by "kunwp1 (via GitHub)" <gi...@apache.org>.
kunwp1 commented on code in PR #6130:
URL: https://github.com/apache/texera/pull/6130#discussion_r3532832591
##########
bin/single-node/nginx.conf:
##########
@@ -95,6 +95,12 @@ http {
proxy_set_header Host $host;
}
+ location /pve/ {
Review Comment:
Agreed. No other comments for me.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: notifications-unsubscribe@texera.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] fix: serve PVE requests under /api/pve and drop per-gateway rewrites [texera]
Posted by "xuang7 (via GitHub)" <gi...@apache.org>.
xuang7 commented on code in PR #6130:
URL: https://github.com/apache/texera/pull/6130#discussion_r3549331672
##########
bin/single-node/nginx.conf:
##########
@@ -95,6 +95,12 @@ http {
proxy_set_header Host $host;
}
+ location /api/pve/ {
+ proxy_pass http://workflow-runtime-coordinator-service:8085;
+ proxy_set_header Host $host;
+ proxy_set_header X-Real-IP $remote_addr;
+ }
Review Comment:
Nginx prefix locations use longest-prefix matching rather than first-match matching, so the file order does not matter here. Only regex locations are order-sensitive, and this file does not contain any regex locations.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: notifications-unsubscribe@texera.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] fix: serve PVE requests under /api/pve and drop per-gateway rewrites [texera]
Posted by "Yicong-Huang (via GitHub)" <gi...@apache.org>.
Yicong-Huang commented on code in PR #6130:
URL: https://github.com/apache/texera/pull/6130#discussion_r3548028904
##########
frontend/src/app/workspace/service/virtual-environment/virtual-environment.service.ts:
##########
@@ -73,27 +74,27 @@ export class WorkflowPveService {
getSystemPackages(cuid: number): Observable<PackageResponse> {
const params = this.buildBaseParams().set("cuid", cuid.toString());
- return this.http.get<PackageResponse>("/pve/system", { params });
+ return this.http.get<PackageResponse>(`${AppSettings.getApiEndpoint()}/pve/system`, { params });
}
fetchPVEs(cuid: number): Observable<PvePackageResponse[]> {
const params = this.buildBaseParams().set("cuid", cuid.toString());
- return this.http.get<PvePackageResponse[]>("/pve/pves", { params });
+ return this.http.get<PvePackageResponse[]>(`${AppSettings.getApiEndpoint()}/pve/pves`, { params });
}
getUserPackages(cuid: number, pveName: string): Observable<string[]> {
return this.fetchPVEs(cuid).pipe(map(pves => pves.find(pve => pve.pveName === pveName)?.userPackages ?? []));
}
deleteEnvironments(cuid: number) {
- return this.http.delete(`/pve/pves/${cuid}`);
+ return this.http.delete(`${AppSettings.getApiEndpoint()}/pve/pves/${cuid}`);
Review Comment:
save for this, `/pve/pves` what dose it do? the naming is not self-contained
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: notifications-unsubscribe@texera.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] fix: serve PVE requests under /api/pve and drop per-gateway rewrites [texera]
Posted by "Yicong-Huang (via GitHub)" <gi...@apache.org>.
Yicong-Huang commented on code in PR #6130:
URL: https://github.com/apache/texera/pull/6130#discussion_r3548022524
##########
frontend/src/app/workspace/service/virtual-environment/virtual-environment.service.spec.ts:
##########
@@ -45,7 +46,7 @@ describe("WorkflowPveService", () => {
expect(resp.veid).toBe(42);
});
- const req = httpTestingController.expectOne("/pve/db");
+ const req = httpTestingController.expectOne(`${AppSettings.getApiEndpoint()}/pve/db`);
Review Comment:
`/pve/db` is just too bad of name choice. we should refactor it seperately
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: notifications-unsubscribe@texera.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] fix: serve PVE requests under /api/pve and drop per-gateway rewrites [texera]
Posted by "codecov-commenter (via GitHub)" <gi...@apache.org>.
codecov-commenter commented on PR #6130:
URL: https://github.com/apache/texera/pull/6130#issuecomment-4909420838
## [Codecov](https://app.codecov.io/gh/apache/texera/pull/6130?dropdown=coverage&src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) Report
:x: Patch coverage is `62.50000%` with `3 lines` in your changes missing coverage. Please review.
:white_check_mark: Project coverage is 60.27%. Comparing base ([`b1c4fd2`](https://app.codecov.io/gh/apache/texera/commit/b1c4fd29bd2208abca1d5cc99729c75f850bd127?dropdown=coverage&el=desc&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache)) to head ([`b864745`](https://app.codecov.io/gh/apache/texera/commit/b86474568b20a46bd23b80a7feba5eacab0c5b56?dropdown=coverage&el=desc&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache)).
:warning: Report is 7 commits behind head on main.
| [Files with missing lines](https://app.codecov.io/gh/apache/texera/pull/6130?dropdown=coverage&src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) | Patch % | Lines |
|---|---|---|
| [...virtual-environment/virtual-environment.service.ts](https://app.codecov.io/gh/apache/texera/pull/6130?src=pr&el=tree&filepath=frontend%2Fsrc%2Fapp%2Fworkspace%2Fservice%2Fvirtual-environment%2Fvirtual-environment.service.ts&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache#diff-ZnJvbnRlbmQvc3JjL2FwcC93b3Jrc3BhY2Uvc2VydmljZS92aXJ0dWFsLWVudmlyb25tZW50L3ZpcnR1YWwtZW52aXJvbm1lbnQuc2VydmljZS50cw==) | 62.50% | [3 Missing :warning: ](https://app.codecov.io/gh/apache/texera/pull/6130?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) |
<details><summary>Additional details and impacted files</summary>
```diff
@@ Coverage Diff @@
## main #6130 +/- ##
=========================================
Coverage 60.26% 60.27%
Complexity 3339 3339
=========================================
Files 1133 1133
Lines 43979 43985 +6
Branches 4781 4782 +1
=========================================
+ Hits 26503 26510 +7
+ Misses 16021 16015 -6
- Partials 1455 1460 +5
```
| [Flag](https://app.codecov.io/gh/apache/texera/pull/6130/flags?src=pr&el=flags&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) | Coverage Δ | | *Carryforward flag |
|---|---|---|---|
| [access-control-service](https://app.codecov.io/gh/apache/texera/pull/6130/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) | `70.00% <ø> (ø)` | | Carriedforward from [b1c4fd2](https://app.codecov.io/gh/apache/texera/commit/b1c4fd29bd2208abca1d5cc99729c75f850bd127?utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) |
| [agent-service](https://app.codecov.io/gh/apache/texera/pull/6130/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) | `44.59% <ø> (ø)` | | Carriedforward from [b1c4fd2](https://app.codecov.io/gh/apache/texera/commit/b1c4fd29bd2208abca1d5cc99729c75f850bd127?utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) |
| [amber](https://app.codecov.io/gh/apache/texera/pull/6130/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) | `65.51% <ø> (ø)` | | Carriedforward from [b1c4fd2](https://app.codecov.io/gh/apache/texera/commit/b1c4fd29bd2208abca1d5cc99729c75f850bd127?utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) |
| [computing-unit-managing-service](https://app.codecov.io/gh/apache/texera/pull/6130/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) | `0.00% <ø> (ø)` | | Carriedforward from [b1c4fd2](https://app.codecov.io/gh/apache/texera/commit/b1c4fd29bd2208abca1d5cc99729c75f850bd127?utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) |
| [config-service](https://app.codecov.io/gh/apache/texera/pull/6130/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) | `52.30% <ø> (ø)` | | Carriedforward from [b1c4fd2](https://app.codecov.io/gh/apache/texera/commit/b1c4fd29bd2208abca1d5cc99729c75f850bd127?utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) |
| [file-service](https://app.codecov.io/gh/apache/texera/pull/6130/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) | `63.97% <ø> (ø)` | | Carriedforward from [b1c4fd2](https://app.codecov.io/gh/apache/texera/commit/b1c4fd29bd2208abca1d5cc99729c75f850bd127?utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) |
| [frontend](https://app.codecov.io/gh/apache/texera/pull/6130/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) | `52.00% <62.50%> (+0.02%)` | :arrow_up: | |
| [notebook-migration-service](https://app.codecov.io/gh/apache/texera/pull/6130/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) | `78.57% <ø> (ø)` | | Carriedforward from [b1c4fd2](https://app.codecov.io/gh/apache/texera/commit/b1c4fd29bd2208abca1d5cc99729c75f850bd127?utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) |
| [pyamber](https://app.codecov.io/gh/apache/texera/pull/6130/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) | `91.18% <ø> (ø)` | | Carriedforward from [b1c4fd2](https://app.codecov.io/gh/apache/texera/commit/b1c4fd29bd2208abca1d5cc99729c75f850bd127?utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) |
| [workflow-compiling-service](https://app.codecov.io/gh/apache/texera/pull/6130/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) | `55.14% <ø> (ø)` | | Carriedforward from [b1c4fd2](https://app.codecov.io/gh/apache/texera/commit/b1c4fd29bd2208abca1d5cc99729c75f850bd127?utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) |
*This pull request uses carry forward flags. [Click here](https://docs.codecov.io/docs/carryforward-flags?utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) to find out more.
</details>
[:umbrella: View full report in Codecov by Harness](https://app.codecov.io/gh/apache/texera/pull/6130?dropdown=coverage&src=pr&el=continue&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache).
:loudspeaker: Have feedback on the report? [Share it here](https://about.codecov.io/codecov-pr-comment-feedback/?utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache).
<details><summary> :rocket: New features to boost your workflow: </summary>
- :package: [JS Bundle Analysis](https://docs.codecov.com/docs/javascript-bundle-analysis): Save yourself from yourself by tracking and limiting bundle sizes in JS merges.
</details>
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: notifications-unsubscribe@texera.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org