Skip to content

feat(task): add task description label and WithTaskDescription option - #985

Merged
kyasbal merged 4 commits into
GoogleCloudPlatform:mainfrom
kyasbal:push-ppqwlutuuvxx
Sep 11, 2026
Merged

feat(task): add task description label and WithTaskDescription option#985
kyasbal merged 4 commits into
GoogleCloudPlatform:mainfrom
kyasbal:push-ppqwlutuuvxx

Conversation

@kyasbal

@kyasbal kyasbal commented Sep 11, 2026

Copy link
Copy Markdown
Member

Summary

Adds LabelKeyTaskDescription to pkg/core/task along with WithTaskDescription label option to attach human-readable descriptions to inspection tasks.

Details

  • Add LabelKeyTaskDescription in pkg/core/task/task.go.
  • Add WithTaskDescription and its implementation in pkg/core/task/label.go.
  • Add unit test TestWithTaskDescription in pkg/core/task/label_test.go.
  • Attach description to DefaultK8sResourceMergeConfigTask.
  • Propagate form/feature task descriptions to LabelKeyTaskDescription in pkg/task/inspection/inspectioncore/contract.

@kyasbal
kyasbal requested a review from RyuSA as a code owner September 11, 2026 08:16
@kyasbal kyasbal added the type:feature New feature or request label Sep 11, 2026
@kyasbal
kyasbal requested a review from jyane as a code owner September 11, 2026 08:16
@kyasbal kyasbal added area:backend-core Core backend frameworks, DAG task runner, and server API area:backend-parser Log parsers and log timeline mapper tasks labels Sep 11, 2026

@gemini-code-assist gemini-code-assist Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Code Review

This pull request introduces a new task label, LabelKeyTaskDescription, to support human-readable descriptions for tasks, and integrates it across various task and form implementations. The review feedback suggests simplifying the implementation of WithTaskDescription by reusing the existing WithLabelValue helper instead of defining a custom struct.

Comment thread pkg/core/task/label.go Outdated
@kyasbal
kyasbal added this pull request to the merge queue Sep 11, 2026
Merged via the queue into GoogleCloudPlatform:main with commit 52cb728 Sep 11, 2026
17 checks passed
@kyasbal
kyasbal deleted the push-ppqwlutuuvxx branch September 11, 2026 08:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area:backend-core Core backend frameworks, DAG task runner, and server API area:backend-parser Log parsers and log timeline mapper tasks type:feature New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant