Skip to content

Add Configured condition to User controller - #686

Merged
felix-kaestner merged 1 commit into
ironcore-dev:mainfrom
lparz1val:feat/configured-condition-user
Oct 26, 2026
Merged

felix-kaestner merged 1 commit into
ironcore-dev:mainfrom
lparz1val:feat/configured-condition-user

Conversation

@lparz1val

Copy link
Copy Markdown
Contributor

Adds a Configured condition to the User controller, following the pattern introduced for Banner (#426). Removes the old type-rename hack that set a Configured-flavored condition directly on Ready. The clientutil.NewClient Secret fetch path correctly does not set Configured — K8s API errors say nothing about device config state.

Changes:

  • user_types.go — add Configured print column (priority=1)
  • user_controller.go — InitializeConditions adds ConfiguredCondition; deferred RecomputeReady; removed type-rename hack; EnsureUser result sets Configured directly
  • user_controller_test.go — 2 → 3 conditions in happy-path assertion
  • Regenerated CRD and Helm chart

Testing

make vet lint test test-gnmi all pass.

Verified live against cisco1 (nx.cisco) on kind dev cluster:

NAME             USERNAME   DEVICE   READY   CONFIGURED   PAUSED   AGE
user-live-test   notest     cisco1   True    True         False    10s
status:
  conditions:
  - lastTransitionTime: "2026-10-26T08:36:53Z"
    message: All conditions are ready
    reason: Ready
    status: "True"
    type: Ready
  - lastTransitionTime: "2026-10-26T08:36:53Z"
    message: Configured successfully
    reason: Configured
    status: "True"
    type: Configured
  - lastTransitionTime: "2026-10-26T08:36:52Z"
    message: ""
    reason: NotPaused
    status: "False"
    type: Paused

Signed-off-by: Yurii Levchenko <yurii.levchenko@sap.com>
@felix-kaestner
felix-kaestner enabled auto-merge (rebase) October 26, 2026 12:15
@felix-kaestner
felix-kaestner merged commit fe65dc3 into ironcore-dev:main Oct 26, 2026
18 of 19 checks passed
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.

2 participants