[MGNLRES-277] ResourceCleanupTask fail leads to blocking migration Created: 30/May/16  Updated: 29/Mar/22  Resolved: 15/Mar/21

Status: Closed
Project: Magnolia Resources Module
Component/s: updates
Affects Version/s: 2.4.5
Fix Version/s: None

Type: Improvement Priority: Neutral
Reporter: Viet Nguyen Assignee: Unassigned
Resolution: Obsolete Votes: 0
Labels: None
Remaining Estimate: Not Specified
Time Spent: Not Specified
Original Estimate: Not Specified
Environment:

Production Telia


Issue Links:
Relates
Template:
Acceptance criteria:
Empty
Task DoD:
[ ]* Doc/release notes changes? Comment present?
[ ]* Downstream builds green?
[ ]* Solution information and context easily available?
[ ]* Tests
[ ]* FixVersion filled and not yet released
[ ]  Architecture Decision Record (ADR)
Date of First Response:
Team: Nucleus

 Description   

Description from customer - Marcus from Telia:
The reason appears to be that the resource modules ResourceCleanupTask with the below line (ResourceCleanUpTask.java:377 in the trace)
Node tempNode = (BINARY_RESOURCES_TEMPLATE.equals(template)) ? resourceNode.getNode(JcrResourceOrigin.BINARY_NODE_NAME) : resourceNode;
This line, which translates to
Node tempNode = ("resources:binary".equals(template)) ? resourceNode.getNode("binary") : resourceNode;
this assume that all nodes with the resources:binary template contain a "binary" subnode. This assumption is incorrect in our content, not just for the attached node example, which is the one that fails the upgrade task, but for all nodes in this tree with the resources:binary template. (see attached export from failing node in 5.4.6)



 Comments   
Comment by Viet Nguyen [ 30/May/16 ]

Expected result:
In case client's customized data of template 'resources:binary' have no attached 'binary' content node, the migration tasks should continue with a warning message instead of blocking the whole process at 650 tasks as linked issue.

Generated at Mon Feb 12 06:49:07 CET 2024 using Jira 9.4.2#940002-sha1:46d1a51de284217efdcb32434eab47a99af2938b.