Skip to content

Nexus Standalone: Feature flag#9567

Merged
stephanos merged 4 commits intofeature/nexus-standalonefrom
stephanos/nexus-standalone-feature-flag
Mar 19, 2026
Merged

Nexus Standalone: Feature flag#9567
stephanos merged 4 commits intofeature/nexus-standalonefrom
stephanos/nexus-standalone-feature-flag

Conversation

@stephanos
Copy link
Copy Markdown
Contributor

@stephanos stephanos commented Mar 18, 2026

What changed?

Add Nexus Standalone feature flag.

How did you test it?

Tests will be added to respective API impl.

@stephanos stephanos changed the title Stephanos/nexus standalone feature flag Nexus Standalone: Feature flag Mar 18, 2026
@stephanos stephanos force-pushed the stephanos/nexus-standalone-feature-flag branch from fbebde9 to aad2aa4 Compare March 18, 2026 17:44
@stephanos stephanos force-pushed the stephanos/nexus-standalone-feature-flag branch from aad2aa4 to 1404d9d Compare March 18, 2026 17:54
if !h.config.Enabled(req.GetNamespace()) {
return nil, ErrStandaloneNexusOperationDisabled
}
return nil, serviceerror.NewUnimplemented("DeleteNexusOperationExecution not implemented")
Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

^ fix after the feature branch was rebased on main

@stephanos stephanos marked this pull request as ready for review March 18, 2026 18:05
@stephanos stephanos requested review from a team as code owners March 18, 2026 18:05
@stephanos stephanos requested review from bergundy and gow March 18, 2026 18:05
func (h *frontendHandler) StartNexusOperationExecution(context.Context, *workflowservice.StartNexusOperationExecutionRequest) (*workflowservice.StartNexusOperationExecutionResponse, error) {
// IsStandaloneNexusOperationEnabled checks if standalone Nexus operations are enabled for the given namespace.
func (h *frontendHandler) IsStandaloneNexusOperationEnabled(namespaceName string) bool {
return h.config.Enabled(namespaceName)
Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Should we also check ChasmEnabled here?

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Yeah, we can't enable CHASM by default until OSS 1.32 unfortunately to support rollback to the previous minor. 1.31 is the first OSS version where CHASM would become stable.

Copy link
Copy Markdown
Member

@bergundy bergundy left a comment

Choose a reason for hiding this comment

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

Overall LGTM

func (h *frontendHandler) StartNexusOperationExecution(context.Context, *workflowservice.StartNexusOperationExecutionRequest) (*workflowservice.StartNexusOperationExecutionResponse, error) {
// IsStandaloneNexusOperationEnabled checks if standalone Nexus operations are enabled for the given namespace.
func (h *frontendHandler) IsStandaloneNexusOperationEnabled(namespaceName string) bool {
return h.config.Enabled(namespaceName)
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Yeah, we can't enable CHASM by default until OSS 1.32 unfortunately to support rollback to the previous minor. 1.31 is the first OSS version where CHASM would become stable.

@stephanos stephanos merged commit aef0fef into feature/nexus-standalone Mar 19, 2026
14 checks passed
@stephanos stephanos deleted the stephanos/nexus-standalone-feature-flag branch March 19, 2026 15:53
stephanos added a commit that referenced this pull request Apr 6, 2026
## What changed?

Add Nexus Standalone feature flag.

## How did you test it?

Tests will be added to respective API impl.
stephanos added a commit that referenced this pull request Apr 7, 2026
## What changed?

Add Nexus Standalone feature flag.

## How did you test it?

Tests will be added to respective API impl.
stephanos added a commit that referenced this pull request Apr 7, 2026
## What changed?

Add Nexus Standalone feature flag.

## How did you test it?

Tests will be added to respective API impl.
stephanos added a commit that referenced this pull request Apr 8, 2026
## What changed?

Add Nexus Standalone feature flag.

## How did you test it?

Tests will be added to respective API impl.
stephanos added a commit that referenced this pull request Apr 8, 2026
## What changed?

Add Nexus Standalone feature flag.

## How did you test it?

Tests will be added to respective API impl.
stephanos added a commit that referenced this pull request Apr 9, 2026
## What changed?

Add Nexus Standalone feature flag.

## How did you test it?

Tests will be added to respective API impl.
stephanos added a commit that referenced this pull request Apr 9, 2026
## What changed?

Add Nexus Standalone feature flag.

## How did you test it?

Tests will be added to respective API impl.
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