Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[WOR-1348] Fix GCP Enhanced Bucket Monitoring on Clone #2654

Merged
merged 4 commits into from
Dec 6, 2023

Conversation

tlangs
Copy link
Contributor

@tlangs tlangs commented Dec 6, 2023

Ticket: https://broadworkbench.atlassian.net/browse/WOR-1348

Rawls needs to look at the source workspace bucket to determine a cloned workspace's enhanced bucket monitoring, not the destination bucket, which can happen if the destination workspace has a bucket name.


PR checklist

  • Include the JIRA issue number in the PR description and title
  • Make sure Swagger is updated if API changes
    • ...and Orchestration's Swagger too!
  • If you changed anything in model/, then you should publish a new official rawls-model and update rawls-model in Orchestration's dependencies.
  • Get two thumbsworth of PR review
  • Verify all tests go green, including CI tests
  • Squash commits and merge to develop (branches are automatically deleted after merging)
  • Inform other teams of any substantial changes via Slack and/or email

@tlangs tlangs marked this pull request as draft December 6, 2023 14:59
@tlangs tlangs marked this pull request as ready for review December 6, 2023 16:02
@cahrens cahrens changed the title ID-1348 Fix GCP Enhanced Bucket Monitoring on Clone [WOR-1348] Fix GCP Enhanced Bucket Monitoring on Clone Dec 6, 2023
testData.testProject1Name.value,
baseWorkspaceName,
Map.empty,
authorizationDomain = Option(Set(testData.dbGapAuthorizedUsersGroup))
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think the auth domain case worked before, and what failed was when the original workspace had "enahncedBucketLogging" set to true but no auth domains.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Intersting, I got a test failure when I removed my fix, and a pass when I included it.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hmmm, the bug we are seeing from terra-ui is specific to not checking "enhancedBucketLogging" but that's probably because we force "enhancedBucketLogging" to be true for workspaces that have an auth domain. So if we didn't send the option in the clone request I guess we would see the bug then also.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

updated the test case to be even clearer

@tlangs
Copy link
Contributor Author

tlangs commented Dec 6, 2023

jenkins retest

@cahrens cahrens requested review from a team, marctalbott and aherbst-broad and removed request for a team December 6, 2023 18:29
@tlangs
Copy link
Contributor Author

tlangs commented Dec 6, 2023

jenkins retest

@tlangs tlangs merged commit c0e8484 into develop Dec 6, 2023
12 checks passed
@tlangs tlangs deleted the tl_WOR-1348_fix_gcp_enhanced_monitoring_clone branch December 6, 2023 19:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants