Remove STATUS column from SP provider table output#21
Merged
ygalblum merged 1 commit intoApr 23, 2026
Conversation
## What Remove the STATUS column from `dcm sp provider` table output after the upstream API dropped the `status` field from the Provider schema. ## Why The Service Provider Manager API (commit ec59988) removed the `status` property from the Provider object. The CLI table was displaying a field that no longer exists in API responses. ## Changes - Removed STATUS from SP provider table headers and row function (6 columns → 5) - Removed `status` field from test sample responses and related assertions - Added negative assertion confirming STATUS column is absent - Updated spec, test plan, and checkpoint docs to reflect 5-column layout ## Testing Updated existing Ginkgo unit tests for SP provider list and get commands to remove status-related assertions and verify the STATUS column is no longer present. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> Signed-off-by: Ygal Blum <ygal.blum@gmail.com>
ygalblum
requested review from
ebichman-1,
gabriel-farache,
gciavarrini,
jenniferubah and
machacekondra
as code owners
gciavarrini
approved these changes
Apr 22, 2026
jenniferubah
approved these changes
Apr 23, 2026
machacekondra
approved these changes
Apr 23, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
4 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
What
Remove the STATUS column from
dcm sp providertable output after the upstream API dropped thestatusfield from the Provider schema.Why
The Service Provider Manager API (commit ec59988) removed the
statusproperty from the Provider object. The CLI table was displaying a field that no longer exists in API responses.Changes
statusfield from test sample responses and related assertionsTesting
Updated existing Ginkgo unit tests for SP provider list and get commands to remove status-related assertions and verify the STATUS column is no longer present.