chore(deps): upgrade backend to Fastify 5 on v2.25.0-fixes - #9244
Conversation
|
Important Review skippedAuto reviews are disabled on base/target branches other than the default branch. 🗂️ Base branches to auto review (4)
Please check the settings in the CodeRabbit UI or the ⚙️ Run configurationConfiguration used: Central YAML (base), Organization UI (inherited) Review profile: CHILL Plan: Pro Plus Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
|
[APPROVALNOTIFIER] This PR is NOT APPROVED This pull-request has been approved by: The full list of commands accepted by this bot can be found here. DetailsNeeds approval from an approver in each of these files:Approvers can indicate their approval by writing |
manaswinidas
left a comment
There was a problem hiding this comment.
create this PR to stable-2.x
|
Created PR to stable-2.x |
https://issues.redhat.com/browse/RHOAIENG-81841
Description
Ports the Fastify 5 backend migration from
main(#9202) onto thev2.25.0-fixesrelease branch. Bumpsfastifyand all version-coupled@fastify/*plugins in the same change set.Dependency changes (
backend/package.json):fastifyfastify-plugin@fastify/autoload@fastify/http-proxy@fastify/reply-from@fastify/static@fastify/websocket@fastify/sensible@fastify/acceptsundicitypescriptserver.ts:logger→loggerInstanceroutes/wss/k8s/index.ts: WebSocket handler usesconnectiondirectly; typed ping/pong/error callbacksroutes/api/storage/index.ts+utils/resourceUtils.ts:log.erroruses pino object formtsconfig.json:skipLibCheck: true(Fastify 5 typings on TS 5.4)backend/package-lock.json: regenerated for Fastify 5 dependency treeHow Has This Been Tested?
Automated
npm run test(backend) — lint, type-check, jest 17/17 passnpm run build(backend) — passnpm ci --dry-run(backend) — passResolved lock versions:
fastify@5.11.3@fastify/websocket@11.3.0@fastify/http-proxy@11.6.0@fastify/autoload@6.5.0@fastify/reply-from@12.6.4undici@7.29.0typescript@5.4.5Manual
Logged in as
htpasswd-cluster-admin-userviamake login+npm run dev./api/health,/api/status,/api/config/api/dsc/status,/api/dsci/status/api/k8s/...proxy/wss/k8s/...watch in browserTest Impact
Request review criteria:
Self checklist (all need to be checked):
If you have UI changes:
After the PR is posted & before it merges:
v2.25.0-fixes