temporal icon indicating copy to clipboard operation
temporal copied to clipboard
verified publisher icon temporalio

Fix legacy DescribeTaskQueue for non-root partitions

Open ShahabT opened this issue 1 year ago • 1 comments

What changed?

DescribeTaskQueue has to support non-root partitions for legacy mode to keep supporting tctl and temporal CLIs.

This PR is fixing a regression caused by https://github.com/temporalio/temporal/pull/6419/files

Why?

The mentioned PR is validating DescribeTaskQueue input similar to other APIs and rejects when task queue name is a non-root partition (starts with /_sys/ prefix). This is not correct, DescribeTaskQueue is different from other API and it has to keep accepting names with this prefix to remain compatible with supported CLIs.

How did you test it?

Added functioonal and unit tests.

Potential risks

Documentation

Is hotfix candidate?

Yes

ShahabT avatar Oct 03 '24 23:10 ShahabT

cc @rodrigozhou

ShahabT avatar Oct 03 '24 23:10 ShahabT