Inventory. Error in Cornell's environment (might be a follow-up on MODINV-829)

Description

Overview: After https://folio-org.atlassian.net/browse/MODINV-829#icft=MODINV-829 was fixed then a new problem might have been introduced and is seen in Cornell's test environment and in Bugfest Orchid. It's happening when there are 100 items or more

See videos from Cornell's environment:

Steps to Reproduce:

  1. Log into FOLIO Bugfest Orchid environment as User folio (password: folio)

  2. Go to Inventory, in the Holdings segment select search on Holdings HRID

  3. Search for Holdings HRID: ho28708

  4. The instance title Accountancy displays and the holdings with 103 items are listed in the top of the instance record in the third pane

  5. Display the item list

  6. Click on any item

Expected Results:
The item displays in detailed view.

Actual Results:
Get an error message saying: "ERROR: in module @folio/inventory, operation GET on resource 'instanceRecords' failed, saying: Not Found"

Additional Information: The mod-inventory fix (https://folio-org.atlassian.net/browse/MODINV-829#icft=MODINV-829) was deployed to bugfest Orchid on 6/29:
URL: https://bugfest-orchid.int.aws.folio.org/inventory/view/9da417a0-c26d-4d87-be32-907fdcefe8dc?qindex=hrid&query=ho28708&segment=holdings&sort=title
Interested parties:

Description of what was done in this Jira
submitted the patch for code review that should mitigate this problem. It will not actually solve the underlying issue, but should work with

  • any holding record that has no bound-with items, regardless of how many normal items it has. I tested with 500+.

  • any holding record that has a mix of directly-linked and bound-with items, as long as there are not more than 300 unique holdings records directly linked to those items.

  • Details in the PR.

Orchid Bugfest Instances with Many Items

CSP justification

Describe issue impact on business

Unable to complete requests for any holdings record that contain over a 100 items. 

What institutions are affected? (field “Effected Institutions” in Jira to be populated)

Any institutions that have a holdings record that contain over a 100 items 

What is the workaround if exists?

No reasonable workaround 

What areas will be impacted by fix (i.e. what areas need to be retested)

  • GET holdings for an instance record 

  • GET items for a holdings record

Brief explanation of technical implementation and the level of effort (in workdays) and technical risk (low/medium/high)

  • Description is in comments.

  • LOE: 3 days 

  • Risk: Low 

Brief explanation of testing required and level of effort (in workdays). Provide test plan agreed with by QA Manager and PO.

 Test plan:

  • Generate a holdings record with more than 100 but less than 1000 items 

  • Verify that user can view all items tied to the holdings record 

  • Verify that user can view the holdings record and items record 

  • Verify with / without bound-withs 

Tests

  • New test:

    • Need to add tests

  • Existing tests (for regression testing):

    • No existing tests exists for a large set of records. 

What is the roll back plan in case the fix does not work?

  • Revert change 

CSP Request Details

Approved by RMS on 7/3/2023. MODINV-838 and UIIN-2478 are required for the implementation of MODINV-832. Both UIIN-2478 and MODINV-832 have been through the RMS voting approval process.

CSP Rejection Details

None

Potential Workaround

None

Attachments

13
  • 31 Jul 2023, 09:34 AM
  • 31 Jul 2023, 09:34 AM
  • 31 Jul 2023, 09:34 AM
  • 31 Jul 2023, 09:34 AM
  • 31 Jul 2023, 09:32 AM
  • 06 Jul 2023, 12:23 PM
  • 05 Jul 2023, 12:13 PM
  • 04 Jul 2023, 08:28 AM
  • 04 Jul 2023, 08:28 AM
  • 30 Jun 2023, 01:13 PM
  • 30 Jun 2023, 01:13 PM
  • 30 Jun 2023, 12:34 PM

Checklist

hide

TestRail: Results

Activity

Show:

Charlotte Whitt July 31, 2023 at 9:36 AM

Manual test in FOLIO Snapshot version @folio/inventory 9.4.9, using Chrome.

Following instance records all behave as expected:
Instance in190451 (103 items)
Instance in794997 (24,000+ items)
Instance in2386991 (5,953 items)

Big thank you to and Yellow Star Yellow Star Yellow Star

I'll close the ticket as done.

CC:

Ann-Marie Breaux July 25, 2023 at 3:29 PM

I'll clone this so that we have Orchid CSP 4 and Poppy tickets, and then we'll need to release the Orchid CSP

Charlotte Whitt July 18, 2023 at 3:09 PM

- I can talk with a little later today, and get you an update.

Khalilah Gambrell July 17, 2023 at 3:16 PM
Edited

You are right . I reviewed another build and forgot it was not snapshot. I moved this to Code Review. Apologies for confusion.

 

cc:  

Ann-Marie Breaux July 17, 2023 at 3:12 PM

Hi Looks like it, plus there's a PR that hasn't been merged yet. Could you create a UIIN Orchid CSP release story (or if there already is one, add this Jira to it) - but don't release until all PRs are merged, and it's tested on Snapshot. Thank you!

Done

Details

Assignee

Reporter

Tester Assignee

Priority

Sprint

Development Team

Sif

Fix versions

Release

Orchid (R1 2023) Service Patch #4

CSP Approved

Yes

RCA Group

Implementation coding issue

Affected Institution

Cornell

TestRail: Cases

Open TestRail: Cases

TestRail: Runs

Open TestRail: Runs

Created June 30, 2023 at 12:31 PM
Updated September 12, 2023 at 8:24 PM
Resolved July 25, 2023 at 3:29 PM
TestRail: Cases
TestRail: Runs

Flag notifications