Skip to content

Send a topic's descriptionFile as metadata.description - #146

Merged
99-not-out merged 1 commit into
mainfrom
fix/topic-description-file
Oct 7, 2026
Merged

99-not-out merged 1 commit into
mainfrom
fix/topic-description-file

Conversation

@99-not-out

@99-not-out 99-not-out commented Oct 6, 2026 •

Copy link
Copy Markdown
Contributor

TL;DR: Topics using descriptionFile apply again on the current Console.

What & why

conduktor apply reads a Topic's conduktor.io/descriptionFile label into a conduktor.io/description label. The Console now keeps a topic's description in metadata.description and refuses that label (Couldn't decode key. at 'metadata.labels.conduktor.io/description'), so any Topic using descriptionFile fails to apply. Found while making the VS Code extension's includes match ctl's.

Changes

  • The file's contents go to metadata.description, and the descriptionFile label is removed as before.
  • Labels left empty once the file label is read are dropped, rather than sent as {}.
  • Tests updated to the new target, plus a case showing other labels are kept.

Notes

  • Broken since Console 1.29.0 (Nov 2024): console-plus #2412 moved the description to metadata.description and removed description from the accepted conduktor.io/ labels. ctl's descriptionFile support (add support for description from file in topic resource #52) predates that and was never updated.
  • Checked against a local Console (main image): a dry run of a Topic with descriptionFile reports Created with this branch, and fails with the released CLI.
  • make lint reports two existing unparam findings in cmd/template.go on main; untouched here.

The Console keeps a topic's description in metadata.description and
refuses the conduktor.io/description label the file used to expand
to, so applying any Topic with a conduktor.io/descriptionFile label
failed. Labels left empty once the file label is read are dropped.

Co-Authored-By: Claude Opus 5.5 <[email protected]>
Claude-Session: https://claude.ai/code/session_01B464sFQc8q6VkehfkByBaX
@99-not-out
99-not-out marked this pull request as ready for review October 6, 2026 12:49
@99-not-out
99-not-out merged commit 59f71fc into main Oct 7, 2026
15 of 19 checks passed
@99-not-out
99-not-out deleted the fix/topic-description-file branch October 7, 2026 08:25
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