petoju / terraform-provider-mysql

Terraform MySQL provider – unofficial fork
https://registry.terraform.io/providers/petoju/mysql
Mozilla Public License 2.0
63 stars 40 forks source link

Support `SET DEFAULT ROLE` Statement #119

Closed alanwu4321 closed 2 months ago

alanwu4321 commented 3 months ago

Do we have support for SET DEFAULT ROLE on user?

SET DEFAULT ROLE
    {NONE | ALL | role [, role ] ...}
    TO user [, user ] ...

If not I can help contribute :) Would like some pointers

Maybe something like this? @petoju

...

resource "mysql_default_role" "developer" {
  user     = mysql_user.jdoe.user
  host     = mysql_user.jdoe.host
  role    = mysql_role.developer.name
}
petoju commented 3 months ago

Why would it need a database?

But otherwise, it looks good. As long as you write at least some acceptance tests (create resource, change resource, delete resource), I'd accept that.

petoju commented 2 months ago

The new resource was released as a part of provider version 3.0.55 - thanks to @rneascu5