Given that you're doing this in OnPlayerRequestClass, don't forget to remove them from that group again when they change skin - otherwise they will just stay in that group indefinately.
Which is why I suggested storing the classid in a variable and set the group after selection (OnPlayerSpawn). Not sure how that would work out, though, as I've never used y_groups before.