YARN-11971: Add validation to block adding queue capacity in partition view if the queue does not have the accessible label configured - #8641
Conversation
…mode when adding queue capacity in label partition
…as capacity in partition view
|
🎊 +1 overall
This message was automatically generated. |
|
🎊 +1 overall
This message was automatically generated. |
|
Thanks @Hean-Chhinling for taking care of this issue! If i understand correctly with the mutable conf api the users can creates capacity configs what will be invalid. May i ask you to attach some screenshot to the jira ticket how the manual test happened? |
You are right @K0K0V0K! |
Description of PR
Queue capacity could be set in partition view even though the queue does not have access to that label. Leaving the only queue's label capacity in /scheduler-conf endpoint. E.g.
Resulting into a bug when the label is deleted and created a new queue. Detail in the jira ticket description.
Therefore, a new validation is needed to make sure the queue has access to the label before allowing to set capacity in the partition view. Vice versa, the accessible label could not be removed if the queue has capacity in that partition. Both have to be changed at the same time.
How was this patch tested?
Tested locally plus unit tests and work as expected.
For code changes:
LICENSE,LICENSE-binary,NOTICE-binaryfiles?AI Tooling
If an AI tool was used:
where is the name of the AI tool used.
https://www.apache.org/legal/generative-tooling.html