Skip to content

Commit 4648a87

Browse files
committed
docs(reconcile): match the fetchCurrent comment to the disabled-users doc
1 parent 345417c commit 4648a87

1 file changed

Lines changed: 3 additions & 3 deletions

File tree

internal/reconcile/platformuser_reconciler.go

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -121,11 +121,11 @@ func (r *PlatformUserReconciler) Export(ctx context.Context) (any, error) {
121121
}
122122

123123
// fetchCurrent lists the platform principals the reconciler manages. Disabled
124-
// principals are out of scope: ListPlatformUsers resolves users through a store
124+
// users are out of scope: ListPlatformUsers resolves users through a store
125125
// query that filters out disabled accounts, so a disabled admin never appears
126126
// here. It is therefore neither exported nor removed by an empty file, and a file
127-
// that lists a disabled principal cannot converge. Re-enable the user to manage
128-
// their platform access.
127+
// that lists a disabled user cannot converge. Re-enable the user to manage their
128+
// platform access. Service users cannot be disabled today, so they are always listed.
129129
func (r *PlatformUserReconciler) fetchCurrent(ctx context.Context) ([]platformPrincipal, error) {
130130
resp, err := r.client.ListPlatformUsers(ctx, authReq(&frontierv1beta1.ListPlatformUsersRequest{}, r.header))
131131
if err != nil {

0 commit comments

Comments
 (0)