Skip to content

Nexus Standalone: Delete#9654

Open
stephanos wants to merge 1 commit intostephanos/nexus-standalone-cancelfrom
stephanos/nexus-standalone-del
Open

Nexus Standalone: Delete#9654
stephanos wants to merge 1 commit intostephanos/nexus-standalone-cancelfrom
stephanos/nexus-standalone-del

Conversation

@stephanos
Copy link
Contributor

@stephanos stephanos commented Mar 25, 2026

How did you test it?

  • built
  • run locally and tested manually
  • covered by existing tests
  • added new unit test(s)
  • added new functional test(s)

Can't verify functional tests yet due to missing impl.

@stephanos stephanos changed the base branch from main to feature/nexus-standalone March 25, 2026 02:18
@stephanos stephanos force-pushed the stephanos/nexus-standalone-del branch 6 times, most recently from ac37168 to 44cdbf9 Compare March 25, 2026 21:06
@stephanos stephanos changed the base branch from feature/nexus-standalone to stephanos/nexus-standalone-cancel March 25, 2026 22:16
@stephanos stephanos force-pushed the stephanos/nexus-standalone-del branch 4 times, most recently from 44193d9 to bb2664f Compare March 26, 2026 00:06
len(req.GetOperationId()), config.MaxIDLengthLimit())
}
if len(req.GetRunId()) > config.MaxIDLengthLimit() {
return serviceerror.NewInvalidArgumentf("run_id exceeds length limit. Length=%d Limit=%d",
Copy link
Contributor Author

@stephanos stephanos Mar 26, 2026

Choose a reason for hiding this comment

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

SAA is parsing the UUID which I thought wasn't necessary here as we don't do that anywhere else either AFAIK.

@stephanos stephanos force-pushed the stephanos/nexus-standalone-del branch 3 times, most recently from 8c294e5 to 6ed485f Compare March 26, 2026 00:13
@stephanos stephanos force-pushed the stephanos/nexus-standalone-del branch from 6ed485f to 05f023d Compare March 26, 2026 00:14
@stephanos stephanos marked this pull request as ready for review March 26, 2026 00:17
@stephanos stephanos requested review from a team as code owners March 26, 2026 00:17
@stephanos stephanos requested review from bergundy and gow March 26, 2026 01:04
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