Is this for production or for lab?
That usually is right but what about redundancy if the management server fails? We segregate resources into management and resource clusters when we usually have cloud environment. When it's a virtualization environment, the management/infra servers would be AD, DB, DHCP, DNS etc with your vCenter and other VMware related products like vShield, SRM etc.
If its just vCenter that you are using and other infra servers then it's better to have them in single cluster so they can failover to other hosts.
So to not confuse you and put it in simple words, your architecture is fine if its production but you should also think about management host failure as its single point of failure if all infra sit on a single host.
Most of the features like HA, DRS, VSAN come with vCenter and ESXi itself. For SRM you would need a seperate windows machine.
If its lab then put all of them in single cluster so they can fail over during host failures.