SqlServerDsc icon indicating copy to clipboard operation
SqlServerDsc copied to clipboard

SqlDatabase: Enable snapshot isolation

Open bozho opened this issue 7 years ago • 4 comments

Hi,

Would there be interest for improving/expanding xSQLServerDatabase, xSQLServerDatabasePermission and xSQLServerDatabaseRole resources?

We currently have a need for the following functionality, which we've covered with custom resources:

  • enable snapshot isolation on a DB
  • ~~create roles on a DB~~ Related moved to issue #846 and #847
  • ~~assign permissions to roles~~ Related moved to issue #848

Enabling/disabling snapshot isolation (and perhaps other DB properties) could be added to the xSQLServerDatabase resource.

~~xSQLServerDatabaseRole is a bit confusing, since it doesn't create/delete roles. Instead, it will create a DB user for a login and assign it an existing role. I think it would be better to have a resource for managing DB roles and a separate resource for user mapping and (maybe) role membership.~~

~~xSQLServerDatabasePermission will also add a DB user if it doesn't exist and assign permissions to it. Again, I think we would benefit here from having a separate resource for DB user management and use xSQLServerDatabasePermission to manage permissions on both users and roles.~~

I'm more than happy to start working on this, but I'd like to hear others' thoughts and ideas.

bozho avatar Sep 25 '17 10:09 bozho

Could you please split this issue into at least one per resource? It will be easier to comment then. Thanks. :smile:

johlju avatar Sep 25 '17 12:09 johlju

Done: issues #846, #847 and #848

bozho avatar Sep 25 '17 14:09 bozho

Are we tracking the snapshot isolation on a database in this issue? Could you please change the title and issue description on this issue to reflect that?

I'm all for adding this functionality. Could you please propose a schema.mof with the changes needed for handle the isolation?

johlju avatar Sep 26 '17 12:09 johlju

I updated the title and issue description is this issue so we know what we are tracking with this issue. Please correct me if I'm wrong.

johlju avatar May 30 '19 10:05 johlju