Skip to content

Conversation

@NucleoFusion
Copy link
Contributor

@NucleoFusion NucleoFusion commented Nov 5, 2025

resolves #164 .
Builds upon work on PR #165

Harbor User Group Management – CLI Reference

Available Commands

Command Description
create create user group
delete delete user group
list list user groups
update update user group

Flags

Flag Description
-h help for usergroup
--help help for usergroup

Global Flags

Flag Description
-c, --config string config file (default is $HOME/.config/harbor-cli/config.yaml)
-o, --output-format Output format. One of: json | yaml
-v, --verbose verbose output

List

2025-11-22.11-26-56.mp4

Delete

2025-11-22.11-29-16.mp4

Create

2025-11-22.11-28-35.mp4

Update

2025-11-22.11-29-39.mp4

Testing for LDAP

Samba + OpenLDAP

2025-12-03_18-40-27.mp4

Microsoft AD

TBD

@codecov
Copy link

codecov bot commented Nov 5, 2025

Codecov Report

❌ Patch coverage is 0.94937% with 313 lines in your changes missing coverage. Please review.
✅ Project coverage is 6.31%. Comparing base (60ad0bd) to head (5a191d3).
⚠️ Report is 62 commits behind head on main.

Files with missing lines Patch % Lines
pkg/api/usergroup_handler.go 0.00% 70 Missing ⚠️
cmd/harbor/root/usergroup/list.go 0.00% 42 Missing ⚠️
pkg/views/usergroup/update/view.go 0.00% 38 Missing ⚠️
cmd/harbor/root/usergroup/create.go 0.00% 30 Missing ⚠️
pkg/views/usergroup/delete/delete.go 0.00% 28 Missing ⚠️
pkg/views/usergroup/create/view.go 0.00% 27 Missing ⚠️
cmd/harbor/root/usergroup/delete.go 0.00% 25 Missing ⚠️
cmd/harbor/root/usergroup/update.go 0.00% 20 Missing ⚠️
pkg/views/usergroup/list/view.go 0.00% 20 Missing ⚠️
cmd/harbor/root/usergroup/cmd.go 0.00% 13 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff            @@
##             main    #566      +/-   ##
=========================================
- Coverage   10.99%   6.31%   -4.68%     
=========================================
  Files         173     270      +97     
  Lines        8671   13158    +4487     
=========================================
- Hits          953     831     -122     
- Misses       7612   12221    +4609     
  Partials      106     106              

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

@NucleoFusion NucleoFusion changed the title [WIP] Feature: Adding Usergroup Command Feature: Adding Usergroup Command Nov 22, 2025
@NucleoFusion NucleoFusion marked this pull request as ready for review November 22, 2025 05:50
@NucleoFusion
Copy link
Contributor Author

@bupd This is PR is ready for review now!

@bupd bupd self-requested a review November 23, 2025 19:19
Copy link
Collaborator

@bupd bupd left a comment

Choose a reason for hiding this comment

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

merge conflicts.

mahbub570 and others added 8 commits November 25, 2025 21:44
Signed-off-by: mahbub570 <[email protected]>
Signed-off-by: NucleoFusion <[email protected]>
Signed-off-by: mahbub570 <[email protected]>
Signed-off-by: NucleoFusion <[email protected]>
Signed-off-by: NucleoFusion <[email protected]>
Signed-off-by: NucleoFusion <[email protected]>
@NucleoFusion
Copy link
Contributor Author

merge conflicts.

Fixed!

@bupd
Copy link
Collaborator

bupd commented Nov 25, 2025

@NucleoFusion Please do test with the OpenLDAP/Samba and Microsoft Active Directory. To make sure you don't have any edge cases.

and update the description with which you had tested these.

@bupd
Copy link
Collaborator

bupd commented Dec 9, 2025

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Implement Usergroup Command

3 participants