aglinxinyuan commented on PR #8293: URL: https://github.com/apache/texera/pull/8293#issuecomment-5480667487
CI note for reviewers — the one red check on this PR is **not** caused by this diff. **The fix itself is verified on a real runner.** `build / pyamber (ubuntu-latest, 3.12)` in run [33404550705](https://github.com/apache/texera/actions/runs/33404550705) passed the previously-failing step with: ``` OK: 110 Python packages match LICENSE-binary. ``` and `pytest` reported `1298 passed, 1 deselected, 1 xfailed`. The 3.11 and 3.13 legs passed too. The same license step also passed on the `release/v1.2` backport run [33404551297](https://github.com/apache/texera/actions/runs/33404551297), so the backport lands correctly as well (`apply-check` is green — the cherry-pick applies cleanly). **The red check** is `backport (release/v1.2) / amber-integration (ubuntu-latest, 17)`, which failed *after every test had passed* (`Tests: succeeded 553, failed 0`, `Passed: Total 6, Failed 0`) with: ``` .github/scripts/smoke-boot.sh: No such file or directory ##[error]Process completed with exit code 127. ``` The backport check runs `main`'s `build.yml` against the release branch's tree, and `smoke-boot.sh` was added to `main` in #6274 — it does not exist on `release/v1.2`. A one-line change to a Python license manifest cannot create or remove that script. Filed as #8295, with the full scope. It surfaced here only because `pyamber` had to be applied **by hand** for this PR to be tested at all (`amber/LICENSE-binary-python` matches no `.github/labeler.yml` glob — #8294), and `pyamber` also enables `amber-integration`. Across the last 30 Backport Checks runs, this is the only one that ran that leg. Also worth ignoring: a `Required Checks fail 4s` row from run [33402590171](https://github.com/apache/texera/actions/runs/33402590171). That is the pre-label run, whose `precheck` was cancelled when the relabel superseded it. The live gate is run 33404550705. Happy to drop the `pyamber` label if you'd rather this go green — the verification above is already recorded in those runs — but I've left it on so the license check is visibly gating the fix. -- 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: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
