[MESOS-2305] Refactor validators in Master.

• Type: Bug
• Status: Resolved
• Priority: Major
• Resolution: Fixed
• Fix Version/s: 0.22.0
• Epic Link: Persistence
• Sprint: Twitter Mesos Q1 Sprint 1, Twitter Mesos Q1 Sprint 2
• Story Points: 3
• Assignee: Jie Yu
• Reporter: Jie Yu
• Created: 29/Jan/15 22:27
• Updated: 02/Jul/15 18:43
• Resolved: 03/Feb/15 18:59

Description

There are several motivation for this. We are in the process of adding dynamic reservations and persistent volumes support in master. To do that, master needs to validate relevant operations from the framework (See Offer::Operation in mesos.proto). The existing validator style in master is hard to extend, compose and re-use.

Another motivation for this is for unit testing (MESOS-1064). Right now, we write integration tests for those validators which is unfortunate.

Issue Links

• blocks: MESOS-1064 Add tests for validation visitors in master