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

Feature 520. Get rid of the complex pipelines and issue a single call for listing DIDs #525

Open
wants to merge 4 commits into
base: master
Choose a base branch
from

Conversation

MytsV
Copy link
Contributor

@MytsV MytsV commented Mar 15, 2025

[GSoC 2025]

Fix #520

Requires merging and deployment of changes from rucio/rucio#7496 or any other PR addressing the issue.

Summary

The application has two views that require listing DIDs. The first one, DIDs List, only displays their scope and name, while loading the metadata on demand. On the other hand, during the rule creation a "deep" size of a dataset/container should be fetched, as well as its status (open).

Hence the decision to split the existing feature into two: one loads a simple list of names from the Rucio Server, the other fetches extended information with size and status.

Changes

  • Made the original list DIDs features not include DIDType
  • Implement an endpoint for fetching extended DID data
  • Create the clean architecture feature for fetching extended DIDs
  • Align the existing tests with the new expected outputs
  • Cover the new gateway and feature with unit tests
  • Refactor the reviewed tests to avoid duplication in stream processing
  • Connect the Create Rule view to the newly added feature

Benefits

  • Reduces API calls needed by WebUI 2.0, significantly decreasing latency

Followup

  • Some issue was introduced to the storybooks by some earlier updates. The AGGrid doesn't become ready, making manual tests impossible. I will address it in another PR.
  • Many more tests contain stream processing duplication, which should be addressed in the future.

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.

GSoC 2025 Qualification Task: Integrate New Single-Call DID Listing in Rucio WebUI
1 participant