Skip to content

[Do Not Merge] Change default of external event publishing - #6006

Draft
rbev wants to merge 1 commit into
masterfrom
change-integration-events-default
Draft

rbev wants to merge 1 commit into
masterfrom
change-integration-events-default

Conversation

@rbev

@rbev rbev commented Oct 15, 2026

Copy link
Copy Markdown
Contributor

Important

Targeted for 7.0, should not be merged to master until it is time to mark that branch for the major update

This pull request updates the configuration for external integrations publishing to use a new positive EnableExternalIntegrationsPublishing setting instead of the previous negative DisableExternalIntegrationsPublishing flag. It also ensures backward compatibility with the legacy setting, adds comprehensive unit tests for the new logic, and updates documentation and acceptance tests accordingly.

Configuration and Feature Flag Changes:

  • Replaces the DisableExternalIntegrationsPublishing flag with EnableExternalIntegrationsPublishing throughout the codebase, updating all conditionals and configuration checks to use the new positive flag. [1] [2] [3] [4]
  • Updates documentation to reflect the new ServiceControl/EnableExternalIntegrationsPublishing setting, while still supporting the legacy ServiceControl/DisableExternalIntegrationsPublishing as an inverted alternative.

Testing and Backward Compatibility:

  • Adds a new unit test class ExternalIntegrationsPublishingSettingsTests to verify the correct behavior of the new and legacy settings, including error handling when both are set.
  • Updates all relevant acceptance tests to explicitly enable external integrations publishing using the new setting for consistent test behavior.

@rbev rbev added this to the NextMajor milestone Oct 15, 2026
@rbev rbev self-assigned this Oct 15, 2026
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.

1 participant