Feature: get/add/remove roles

[jkrilov]

Description

New module to add/remove roles for a DB user.

How Has This Been Tested?

Integration tests

Types of changes

• Bug fix (non-breaking change which fixes an issue) - Fixes #
• New feature (non-breaking change which adds functionality)

Checklist:

• I have read/followed the CONTRIBUTING document.
• I have read/followed the PR Quick Start Guide
• I have added tests to cover my changes or they are N/A.
• I have added a changelog fragment describing the changes.
• New module options/parameters include a version_added property.

[codecov-commenter]

Codecov Report

Patch coverage: 100.00% and project coverage change: -4.21% ⚠️

Comparison is base (104572c) 93.81% compared to head (96a62a0) 89.60%.

Additional details and impacted files

@@            Coverage Diff             @@
##             main     #162      +/-   ##
==========================================
- Coverage   93.81%   89.60%   -4.21%     
==========================================
  Files          64       54      -10     
  Lines        2198     1453     -745     
==========================================
- Hits         2062     1302     -760     
- Misses        136      151      +15     

Flag	Coverage Δ
sanity	100.00% <100.00%> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

Files Changed	Coverage Δ
plugins/modules/role_member.py	100.00% <100.00%> (ø)
plugins/modules/role_member_info.py	100.00% <100.00%> (ø)

... and 15 files with indirect coverage changes

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[jkrilov]

@lowlydba I'm seeing intermittent failures of sp_whoisactive.

Edit: Now I'm seeing the error every time on the Windows runners. I'll take a look this afternoon to see if I can determine the cause.

[lowlydba]

Still no update on the Github 403 issues from my support ticket, so we can proceed with those still existing for now.

A few change requests, but otherwise looks good. Let me know if/when its ready for a final review.

[lowlydba]

@Joey40 If you rebase from main, I removed the EOL 2.11 version so there are fewer jobs now. Github support indicated its a rate limit issue mostly (per hour) so if this continues to be a problem giving a 60 minute cooloff should help the jobs to work.

[jkrilov]

Sounds good. Sorry I've been quiet for a couple weeks with the holidays and new year. I'll see if I can get this completed next week.

[Jvekka]

This was actually something I was starting to build, but I noticed it already existed here.
Can I help with something to get going again?

[jkrilov]

This was actually something I was starting to build, but I noticed it already existed here.

Can I help with something to get going again?

I'll look this over today. I had some other work to do and honestly forgot to finish this.

If there's something you'd like that I haven't added feel free to let me know. I'd be happy to have the help.

[lowlydba]

Going to close this for now as it is a bit stale, but feel free to re-open if you want to resume it.