Skip to content

feat: Add configurable workflow status subscription for HTTP webhook publisher#720

Open
akhilpathivada wants to merge 1 commit intoconductor-oss:mainfrom
akhilpathivada:feature/configurable-workflow-status-publishing
Open

feat: Add configurable workflow status subscription for HTTP webhook publisher#720
akhilpathivada wants to merge 1 commit intoconductor-oss:mainfrom
akhilpathivada:feature/configurable-workflow-status-publishing

Conversation

@akhilpathivada
Copy link

@akhilpathivada akhilpathivada commented Jan 20, 2026

Resolves #710

Summary

Enables configuration-based control over which workflow status changes trigger HTTP webhook events, matching the existing task status publishing behavior.

Problem

The HTTP webhook publisher (conductor.workflow-status-listener.type=workflow_publisher) currently hardcodes publishing only COMPLETED and TERMINATED workflow events, with no way to configure which statuses to receive. This is inconsistent with the task publisher which allows full configurability via subscribed-task-statuses.

Solution

Added subscribedWorkflowStatuses configuration property to enable selective workflow event publishing:

# New configuration property
conductor.status-notifier.notification.subscribed-workflow-statuses=RUNNING,COMPLETED,TERMINATED,PAUSED

# Valid values: RUNNING, COMPLETED, FAILED, TIMED_OUT, TERMINATED, PAUSED, RESUMED, RESTARTED, RETRIED, RERAN, FINALIZED

Changes

  1. StatusNotifierNotificationProperties: Added subscribedWorkflowStatuses field with getters/setters
  2. StatusChangePublisherConfiguration: Inject workflow status config and pass to publisher
  3. StatusChangePublisher: Implement conditional checks in all 9 workflow lifecycle methods
  4. StatusChangePublisherTest: Added comprehensive unit tests (17 test cases)
  5. config.properties: Added configuration example with documentation

Testing

✅ All 9 workflow status methods tested with various configurations
✅ Null and empty list handling
✅ Build successful with no linter errors

@akhilpathivada akhilpathivada force-pushed the feature/configurable-workflow-status-publishing branch from eaa2a31 to df6b305 Compare January 20, 2026 13:19
@akhilpathivada akhilpathivada force-pushed the feature/configurable-workflow-status-publishing branch from df6b305 to 4c32dc2 Compare January 20, 2026 18:41
@akhilpathivada akhilpathivada marked this pull request as ready for review January 20, 2026 18:44
@akhilpathivada
Copy link
Author

@v1r3n @manan164 could you please help reviewing this?

@v1r3n
Copy link
Collaborator

v1r3n commented Jan 21, 2026

Yes we are on it and should be part of next release if all looks good.

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.

[FEATURE] Add Configurable Workflow Status Subscription for HTTP Webhook Publisher

3 participants