Skip to content

Init server/scheduler package and use it as proxy for queue & pubsub#6418

Merged
lafriks merged 1 commit into
woodpecker-ci:mainfrom
6543-forks:init/server-scheduler/package
Apr 10, 2026
Merged

Init server/scheduler package and use it as proxy for queue & pubsub#6418
lafriks merged 1 commit into
woodpecker-ci:mainfrom
6543-forks:init/server-scheduler/package

Conversation

@6543

@6543 6543 commented Apr 9, 2026

Copy link
Copy Markdown
Member

move towards #6317

this does not change any code flow, it just introduces a new package that will later on combine queue and pubsub coordination.

@6543 6543 added the refactor delete or replace old code label Apr 9, 2026
@woodpecker-bot

woodpecker-bot commented Apr 9, 2026

Copy link
Copy Markdown
Contributor

Surge PR preview deployment was removed

@codecov

codecov Bot commented Apr 10, 2026

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 6.55738% with 57 lines in your changes missing coverage. Please review.
✅ Project coverage is 35.39%. Comparing base (d1c1da1) to head (45d4130).
⚠️ Report is 4 commits behind head on main.

Files with missing lines Patch % Lines
server/scheduler/proxy.go 0.00% 26 Missing ⚠️
server/rpc/rpc.go 25.00% 6 Missing ⚠️
server/api/agent.go 28.57% 5 Missing ⚠️
server/scheduler/scheduler.go 0.00% 5 Missing ⚠️
server/api/queue.go 0.00% 4 Missing ⚠️
cmd/server/setup.go 0.00% 3 Missing ⚠️
server/rpc/server.go 0.00% 2 Missing ⚠️
cmd/server/grpc_server.go 0.00% 1 Missing ⚠️
cmd/server/metrics_server.go 0.00% 1 Missing ⚠️
server/api/stream.go 0.00% 1 Missing ⚠️
... and 3 more
Additional details and impacted files
@@            Coverage Diff             @@
##             main    #6418      +/-   ##
==========================================
- Coverage   35.45%   35.39%   -0.06%     
==========================================
  Files         427      429       +2     
  Lines       28659    28689      +30     
==========================================
- Hits        10160    10155       -5     
- Misses      17569    17603      +34     
- Partials      930      931       +1     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@6543 6543 requested a review from lafriks April 10, 2026 07:44
@lafriks lafriks merged commit acc68e5 into woodpecker-ci:main Apr 10, 2026
7 of 9 checks passed
@woodpecker-bot woodpecker-bot mentioned this pull request Apr 10, 2026
1 task
@6543 6543 deleted the init/server-scheduler/package branch April 10, 2026 08:11
@woodpecker-bot woodpecker-bot mentioned this pull request Apr 15, 2026
1 task
@6543 6543 changed the title Init server/scheduler package and use it as proxy for queue&pubsub Init server/scheduler package and use it as proxy for queue & pubsub Apr 15, 2026
@6543 6543 changed the title Init server/scheduler package and use it as proxy for queue & pubsub Init server/scheduler package and use it as proxy for queue & pubsub Apr 15, 2026
@woodpecker-bot woodpecker-bot mentioned this pull request Apr 27, 2026
1 task
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

refactor delete or replace old code

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants