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

ID-1023 Use new listResourcesV2 endpoint. #2714

Closed
wants to merge 6 commits into from

Conversation

Ghost-in-a-Jar
Copy link
Contributor

@Ghost-in-a-Jar Ghost-in-a-Jar commented Jan 26, 2024

Ticket: https://broadworkbench.atlassian.net/browse/ID-1023

Updates the sam client and uses the new listResourcesV2 endpoint that is more performant. There is still work to be done to lazily load public resources in the ui which will further reduce load on sam.

(the branch name is referring to a duplicate ticket we have since closed)


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

@Ghost-in-a-Jar Ghost-in-a-Jar changed the title WOR-1464 Use new listResourcesV2 endpoint. ID-1023 Use new listResourcesV2 endpoint. Jan 26, 2024
@Ghost-in-a-Jar Ghost-in-a-Jar force-pushed the WOR-1464-use-new-list-resources-v2 branch from e220823 to f410fea Compare January 26, 2024 19:07
@tlangs
Copy link
Contributor

tlangs commented Jan 29, 2024

jenkins retest

@Ghost-in-a-Jar
Copy link
Contributor Author

jenkins retest again

i saw service unavailable, gateway timeout, and

group AuthDomainOne_9ca17289-ad4b-4622-bab9-541776b726d6 cannot be deleted because it is a member of at least 1 other group

@aherbst-broad
Copy link
Contributor

@Ghost-in-a-Jar I'm closing this as it looks fairly out of date. Feel free to reopen if this is still in progress.

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