Delete bulk-send and inspec-scanner IAM users

[sastels]

Description

As a Notify developer, I need the repo to have as little unused code as possible.

WHY are we building?

We no longer use the bulk-send and inspec-scanner IAM users (the credentials were disabled months ago without complaints) so we can delete the users.

WHAT are we building?

Delete the bulk-send and inspec-scanner IAM users

VALUE created by our solution

cleaner code code runs better

Acceptance Criteria

Given some context, when (X) action occurs, then (Y) outcome is achieved.

• [x] bulk-send and inspec-scanner users deleted

QA Steps

• [x] these users don't exist anymore
• [x] Send a bulk send notifications with CSV

[sastels]

https://github.com/cds-snc/notification-terraform/pull/851

[ben851]

Ben to review PR

[ben851]

Need to release the force destroy PR to prod before merging to staging.

[ben851]

Merge to staging initially failed due to click-ops account credentials. Steve manually removed and merge succeeded. Steve to check prod for credentials.

[sastels]

inspec-scanner creds manually deleted in prod

[jimleroyer]

Requires a Terraform release. Plan to do it today. Credentials were deleted in Prod last week.

[sastels]

released in prod

[jimleroyer]

I will QA today!

[jimleroyer]

QA'ed:

• Bulk sent and working.
• Users up for deletion checked they really does not exist.