I would name node by the HA pair, 1a/1b and 2a/2b. I think that the node name should be in the aggr name. If you have a system with different type of drives I think adding the type would be useful for administration (ssd/scsi/sata). I'm sure the information is there in the display but sometime it's easier to have the this information there for admin. Just my 2 cents.