Skip to content

Release: 2 fixes, 1 refactor, 1 chore → Main#9407

Merged
h0lybyte merged 5 commits intomainfrom
dev
Apr 1, 2026
Merged

Release: 2 fixes, 1 refactor, 1 chore → Main#9407
h0lybyte merged 5 commits intomainfrom
dev

Conversation

@github-actions
Copy link
Copy Markdown
Contributor

@github-actions github-actions Bot commented Mar 31, 2026

Release: Dev → Main

4 atomic commits ready for main

Bug Fixes

Refactoring

Chores


This PR is automatically maintained by CI — KBVE Studio

…uild once, test, promote (#9406)

Replace Playwright-based e2e with vitest + dependsOn container build,
matching the axum-kbve-e2e pattern. The e2e target now depends on
discordsh-bot:container which builds the Docker image first, then
runs smoke tests against the locally-built image. No double builds.

- Add container:ci config to discordsh-bot project.json
- Rewrite e2e from Playwright to vitest with health endpoint smoke tests
- Remove Playwright configs (playwright.config.ts, docker.config.ts, teardown)
- Add vitest.config.ts and helpers/http.ts polling utility

Fixes #9347
Ref #9286
axios was listed in root and @kbve/devops package.json but never
imported anywhere in the codebase. Removes dead dependency.
@github-actions
Copy link
Copy Markdown
Contributor Author

github-actions Bot commented Mar 31, 2026

Dependency Review

✅ No vulnerabilities or license issues or OpenSSF Scorecard issues found.

Snapshot Warnings

⚠️: No snapshots were found for the head SHA 51e19ec.
Ensure that dependencies are being submitted on PR branches and consider enabling retry-on-snapshot-warnings. See the documentation for more information and troubleshooting advice.

Scanned Files

  • packages/npm/devops/package.json
  • pnpm-lock.yaml

#9408)

Switch chuckrpg.com ACME solver from nginx ingress back to
gatewayHTTPRoute with sectionName: http. The original solver
failed because it didn't specify which listener to attach to,
so the HTTPRoute could land on HTTPS instead of HTTP port 80.

Adding sectionName: http ensures the ACME challenge route binds
to the Cilium gateway's HTTP listener (port 80), where Let's
Encrypt sends the verification request.

Also reverts nginx re-addition from kustomization — nginx is
not needed when the Gateway API solver works correctly.

Post-merge: delete stale challenge + order to force retry:
  kubectl delete challenge -n chuckrpg --all
  kubectl delete order -n chuckrpg --all
@github-actions github-actions Bot changed the title Release: 1 refactor → Main Release: 1 fix, 1 refactor, 1 chore → Main Mar 31, 2026
…#9411)

Axios was removed from package.json but still imported in api.ts,
yt.ts, and their test files. Replace all axios.get/post calls with
native fetch and update test mocks from vi.mock('axios') to
vi.stubGlobal('fetch', fetchMock).

Fixes #9410
@github-actions github-actions Bot changed the title Release: 1 fix, 1 refactor, 1 chore → Main Release: 2 fixes, 1 refactor, 1 chore → Main Mar 31, 2026
…re (#9412)

Missing url field caused the kubernetes provider to connect to
localhost:443 instead of the cluster API server. The arc-runners
SecretStore has url: 'https://kubernetes.default.svc' — our copy
was missing it.

Fixes: "dial tcp [::1]:443: connect: connection refused"
Unblocks: KEDA github-runner auth → VM auto-start/stop
@h0lybyte h0lybyte merged commit 5e822d6 into main Apr 1, 2026
4 checks passed
@github-project-automation github-project-automation Bot moved this from Review to Done in KBVE Apr 1, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

1 participant