Skip to content

[Fix-18286] Use real existing workerGroups more accurately instead of defaulting to 'default'#18287

Closed
KaiSong-UK wants to merge 6 commits into
apache:devfrom
KaiSong-UK:fix-18286-workerGroup-default
Closed

[Fix-18286] Use real existing workerGroups more accurately instead of defaulting to 'default'#18287
KaiSong-UK wants to merge 6 commits into
apache:devfrom
KaiSong-UK:fix-18286-workerGroup-default

Conversation

@KaiSong-UK
Copy link
Copy Markdown

close #18286

Purpose

When user configures custom workerGroup (e.g. g_suyc) and there is no default group in workerGroup management list, the system still defaults to default as workerGroup in the API layer, causing ambiguity and potential errors when:

  • Creating/updating schedules (project preferences)
  • Executing workflows
  • Starting workflow/task checks

This is a server-side fix that removes the hardcoded defaultValue = " default\ in the API controllers, allowing the upstream logic to handle the default based on actual available workerGroups.

Changes

  • SchedulerController.java: Removed defaultValue = \default\ from createSchedule, updateSchedule, updateScheduleByWorkflowDefinitionCode methods
  • ExecutorController.java: Removed defaultValue = \default\ from startCheckResult, execTaskInstance, execute methods

Verify

@SbloodyS
Copy link
Copy Markdown
Member

This issue has been claimed by the author, and only the issue labeled good first issue is the first contributor-friendly issue.

@SbloodyS SbloodyS closed this May 24, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Improvement][ApiServer] Use real existing workerGroups more accurately instead of defaulting to 'default', to avoid ambiguity.

2 participants