Skip to content

feat: adding jobservice dashboard feature to cli#776

Open
NishchayRajput wants to merge 2 commits intogoharbor:mainfrom
NishchayRajput:feat/jobservice-dashboard
Open

feat: adding jobservice dashboard feature to cli#776
NishchayRajput wants to merge 2 commits intogoharbor:mainfrom
NishchayRajput:feat/jobservice-dashboard

Conversation

@NishchayRajput
Copy link
Copy Markdown

@NishchayRajput NishchayRajput commented Mar 28, 2026

Description

Adds a new harbor jobservice command group to bring key Harbor Job Service Dashboard operations into CLI workflows for system administrators.

This change is needed to support terminal-first operations for queues, workers, schedules, and job logs without relying on the UI, and to make these workflows automation-friendly.

Proposal: https://www.notion.so/Jobservice-Dashboard-Support-to-Harbor-CLI-335da66bff138008a4f5c425be2f7384?source=copy_link

Type of Change

Please select the relevant type.

  • Bug fix
  • New feature
  • Refactor
  • Documentation update
  • Chore / maintenance

Changes

  • Added harbor jobservice root command with subcommands for pools, workers, queues, schedules, and jobs.
  • Added worker commands for listing and release operations:
    • harbor jobservice workers list
    • harbor jobservice workers free --job-id <id>
    • harbor jobservice workers free-all
  • Added queue commands for listing and lifecycle control:
    • harbor jobservice queues list
    • harbor jobservice queues stop|pause|resume
    • Supports --type (single, repeated, comma-separated, all) and interactive multi-select mode with state-aware filtering.
  • Added schedule commands:
    • harbor jobservice schedules list --page --page-size
    • harbor jobservice schedules status
    • harbor jobservice schedules pause-all|resume-all
  • Added job log command:
    • harbor jobservice jobs log --job-id <id>
  • Added/updated API handlers and views to support worker pools/workers, queue actions, schedules, and job log output.

Signed-off-by: Nishchay Rajput <nishchayr@iitbhilai.ac.in>
@NishchayRajput NishchayRajput mentioned this pull request Mar 28, 2026
5 tasks
Signed-off-by: Nishchay Rajput <nishchayr@iitbhilai.ac.in>
@NishchayRajput NishchayRajput marked this pull request as ready for review March 30, 2026 17:50
@qcserestipy qcserestipy self-requested a review April 1, 2026 17:43
@codecov
Copy link
Copy Markdown

codecov bot commented Apr 1, 2026

Codecov Report

❌ Patch coverage is 0.45593% with 655 lines in your changes missing coverage. Please review.
✅ Project coverage is 7.44%. Comparing base (60ad0bd) to head (2d77b0c).
⚠️ Report is 123 commits behind head on main.

Files with missing lines Patch % Lines
cmd/harbor/root/jobservice/queues/list.go 0.00% 192 Missing ⚠️
cmd/harbor/root/jobservice/schedules/list.go 0.00% 110 Missing ⚠️
pkg/api/jobservice_handler.go 0.00% 76 Missing ⚠️
cmd/harbor/root/jobservice/workers/list.go 0.00% 49 Missing ⚠️
cmd/harbor/root/jobservice/workers/free.go 0.00% 46 Missing ⚠️
pkg/views/jobservice/workers/view.go 0.00% 33 Missing ⚠️
cmd/harbor/root/jobservice/jobs/log.go 0.00% 32 Missing ⚠️
cmd/harbor/root/jobservice/pools/list.go 0.00% 31 Missing ⚠️
cmd/harbor/root/jobservice/workers/workers.go 0.00% 21 Missing ⚠️
pkg/views/jobservice/pools/view.go 0.00% 20 Missing ⚠️
... and 3 more
Additional details and impacted files
@@            Coverage Diff            @@
##             main    #776      +/-   ##
=========================================
- Coverage   10.99%   7.44%   -3.55%     
=========================================
  Files         173     283     +110     
  Lines        8671   13827    +5156     
=========================================
+ Hits          953    1030      +77     
- Misses       7612   12684    +5072     
- Partials      106     113       +7     

☔ 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.

@qcserestipy
Copy link
Copy Markdown
Collaborator

@NishchayRajput Thank you very much for this contribution! Since this PR contains nearly 3000 changed lines would it be possible to split it into several smaller PRs? It would be easier to review if it is split into smaller parts.

Copy link
Copy Markdown
Collaborator

@qcserestipy qcserestipy left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please split to smaller PRs subsequently introducing the sub commands.

@qcserestipy qcserestipy added the Changes Requesed feedback that must be addressed before merging. label Apr 1, 2026
@qcserestipy
Copy link
Copy Markdown
Collaborator

Please also make sure that linting passes in the split PRs

@NishchayRajput
Copy link
Copy Markdown
Author

NishchayRajput commented Apr 1, 2026

However it might be less than 3000 as it's because of doc file generated. But sure will raise PR for sub-commands! @qcserestipy

@NishchayRajput
Copy link
Copy Markdown
Author

#793 #795 #796 are split PRs @qcserestipy

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Changes Requesed feedback that must be addressed before merging.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[feature]: Implementing Jobservice Dashboard in CLI

2 participants