A relação deveria ManyToMany mesmo. OneToMany seria se as roles fossem de apenas um usuário. Por exemplo: Tabela pessoa e telefone, teria um atributo telefone em pessoa anotado com OneToMany, uma pessoa tem vários telefones, mas o telefone só pertence a uma pessoa.
Coloca o bloco do try antes do gerar tabelas e vê se resolveu, não postou o método persistindo inteiro aqui né? tem o método save ou merge nele para persistir?
Vi que é possível manter o Dao com as operações no banco e o service para a lógica de negócio. Para usar minha classe service eu apenas anoto com @service e injeto ela no controller e chamos o métodos que criei? Tem alguma convenção no nome dos métodos? Quantas classes de service precisa ter para manter a coesão?