DT-1202: Add "inherit steward" support to snapshot creation

[pshapiro4broad]

Jira ticket: https://broadworkbench.atlassian.net/browse/DT-1202

Addresses

Add support to snapshot creation so the dataset custodian is optionally granted steward permissions on the snapshot.

Summary of changes

In the snapshot creation flight, if the inheritSteward flag is set:

• the parent is passed to Sam when creating the snapshot
• the dataset custodian user is added as a reader on the BQ project
• the dataset custodian user is added as a job executor to the BQ project

Testing Strategy

• unit tests
• connected tests - skip for now, add later once feature is exposed at the API layer
• manual local testing to create snapshot and observe its behavior

[fboulnois]

Overall looks good, some minor suggestions:

[fboulnois]

Should we pull this into a common method somewhere?

[pshapiro4broad]

This code is temporary until https://broadworkbench.atlassian.net/browse/DT-1197 is done. That will replace this code, which reads from the flight input map, to read from a field in sourceDataset. I'm expecting it would look something like

    if (sourceDataset.inheritSteward()) {
      var datasetPolicyMap =
          iamService.retrievePolicyEmails(userReq, IamResourceType.DATASET, sourceDataset.getId());
      emails.add(datasetPolicyMap.get(IamRole.CUSTODIAN));
    }

So I think a lot of it will go away. If there's still a need to refactor common code it can be done then.

[sonarqubecloud]

Quality Gate passed

Issues
11 New issues
0 Accepted issues

Measures
0 Security Hotspots
93.9% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud