SW Test #16567

SW Test #16559: gCube Release 4.14.0 - FT Testing

SW Test #16561: org.gcube.portlets-user.4-14-0

org.gcube.portlets-user.workspace-portlet.6-22-0

Added by Francesco Mangiacrapa 9 months ago. Updated 9 months ago.

Status:Tested on PreprodStart date:Apr 17, 2019
Priority:UrgentDue date:
Assignee:Pasquale Pagano% Done:

100%

Category:portlets-user
Sprint:gCube Release 4.14.0 - FT Testing
Milestones:
Duration:


Related issues

Related to gCube - Task #12910: Rename and delete operations on shared folder should be m... Closed Nov 20, 2018

History

#1 Updated by Luca Frosini 9 months ago

  • Assignee changed from Luca Frosini to Pasquale Pagano

#2 Updated by Massimiliano Assante 9 months ago

  • Status changed from New to Available

The portlet has been updated an the test can begin https://pre.d4science.org/group/pre/workspace

#3 Updated by Pasquale Pagano 9 months ago

  • Status changed from Available to Under Testing

#4 Updated by Pasquale Pagano 9 months ago

  • % Done changed from 0 to 30

#5 Updated by Pasquale Pagano 9 months ago

  • % Done changed from 30 to 60
  • Assignee changed from Pasquale Pagano to Francesco Mangiacrapa
  • Status changed from Under Testing to Test Issue

The following tests failed:

#2  "Create a new shared folder 
(Go on Workspace Tree View, right click on a folder and choose “New Shared Folder”)"    

ERROR: Sorry, an error has occurred on the server when sharing item. The Error occurs when selecting Write Any.  It works properly selecting Write Own. 

--- 
#7.2    Select a shared folder, right click and click “Rename”  

"ERROR: Error on renaming item. Try again"

---
#8.2    Select a shared folder, right click and click “Delete Item” 

"ERROR:  Error on deleting. Either the item is shared, unshare it and try to delete again or you have not the permission to delete the item"

---
#9.2    "After performed #9.1, right click on shared folder and click “Share”. Now remove one or more users from “Share with users”"

ERROR: the operation goes to the end without any error but the user is not removed. 

---
#9.3    "After performed #9.2, right click on shared folder  and click “Share”. Now, change Permissions for user/s belonging to the share (e.g. from “Write Own” to “Write Any”)"

ERROR: Sorry, an error has occurred on the server when sharing item.

---
#13 "Go into a shared folder where your role is “Write Any”,  and upload file/s"

Cannot be tested because putting Write Any returns always an error

---
#16 "Right click on a shared folder and click “Get Folder Link” (remember the shared folder path)"

ERROR: Error on reading Folder Link for id: fff1095a-fa38-4006-b3b6-c911d6c85ecb

---
#19.2   After performed #8.2, go into the Trash. Is the shared folder in the trash?

Not possible to check since the shared folder cannot be deleted

#6 Updated by Francesco Mangiacrapa 9 months ago

I've added @g.panichi@isti.cnr.it as 'watcher'. Giancarlo can check and fix the points #9.2 e #9.3 e #13 about the sharing facility...

I will check and possibly correct the points:

#2;
#7.2 if renaming of a shared folder is not available on SHUB-side, we can limit this action on WS-side;
#8.2 as commented at #7.2
#16;
#19.2 does not exist because of #8.2.

@lucio.lelii@isti.cnr.it can you confirm the comments reported to #7.2 and #8.2?

#7 Updated by Lucio Lelii 9 months ago

Yes, a shared folder cannot be renamed

#8 Updated by Lucio Lelii 9 months ago

the problem of #2, #9.3, #13 is that reinstalling the jackrabbit database we lost the jcrAll privilege, and there isn't a wiki where is expleined how to add it, so in preprod everithing with the privilege "Write Any" will fail

#9 Updated by Francesco Mangiacrapa 9 months ago

About point #16 it was missing the Application Profile 'Workspace-Explorer-App' in the new root vo of preprod (/pred4s) needed to the Workspace portlet to resolve a public folder.
After creating it, the public "Folder Link" is returned correctly. However, as already done for other operations I need to add a delay :-( on WS-side after control returns to the Workspace because the icon does not change (remaining to a private folder). Instead, it changes forcing a refresh manually (the folder icon changes to the public folder), so a delay should solve this issue...

About the operations reported at #7.2 #8.2, they will not be limited (removing it) but these cases will be managed specifically (by code) and human readable message will be show to explain to users that the operations are not allowed. @massimiliano.assante@isti.cnr.it will provide the message...

#11 Updated by Roberto Cirillo 9 months ago

You are right. it is still missing on data-pre. I'm going to add it

#12 Updated by Roberto Cirillo 9 months ago

The missing webapps have been provisioned. Please could you retry now?

#13 Updated by Massimiliano Assante 9 months ago

About the operations reported at #7.2 #8.2, they will not be limited (removing it) but these cases will be managed specifically (by code) and human readable message will be show to explain to users that the operations are not allowed. @massimiliano.assante@isti.cnr.it will provide the message...

Ops! This operation is not allowed, we're working hard to make this possible soon.

#14 Updated by Francesco Mangiacrapa 9 months ago

  • Related to Task #12910: Rename and delete operations on shared folder should be managed by dedicated message on WS-side added

#16 Updated by Francesco Mangiacrapa 9 months ago

  • % Done changed from 60 to 80

Recap about my tasks:

#7.2 fixed by #12910
#8.2 fixed by #12910
#16 fixed by creating the "Workspace-Explorer-App" GE in the scope /pred4 and by adding a delay time to refresh the list of items.

The Workspace is updated on dev (at https://next.d4science.org/group/next/workspace)

#17 Updated by Giancarlo Panichi 9 months ago

  • Assignee changed from Francesco Mangiacrapa to Pasquale Pagano
  • Status changed from Test Issue to Available

Bugs #9.2, #9.3 and #13 have been fixed.
A new version of the portlet is available in Preprod.
Please @pasquale.pagano@isti.cnr.it, could you repeat the tests?
Thanks.

#18 Updated by Giancarlo Panichi 9 months ago

There is a problem with the refresh of the portlet after the execution of Remove Folder Link
command, present in the context menu of the Workspace Tree.
It's probably a synchronization problem.

#19 Updated by Pasquale Pagano 9 months ago

  • Status changed from Available to Under Testing

#20 Updated by Pasquale Pagano 9 months ago

  • % Done changed from 80 to 90
  • Assignee changed from Pasquale Pagano to Francesco Mangiacrapa
  • Status changed from Under Testing to Tested on Preprod

I performed all the tests and I uploaded the result of the tests in the folder.

All of them have been passed but

a) #14: Go into a shared folder where your role is “Write Any”, and upload file/s.

This test should show an alert and an error is shown instead. It is not blocking but the user may be disoriented since it is not evident that the error is due to the missing privileges on that folder.

b) #31.1, #31.2 and #31.3 tests were not performed since I am not a VRE manager on PRE. These tests have to be performed either by @massimiliano.assante@isti.cnr.it or @francesco.mangiacrapa@isti.cnr.it.

#21 Updated by Luca Frosini 9 months ago

  • Assignee changed from Francesco Mangiacrapa to Massimiliano Assante

@massimiliano.assante@isti.cnr.it can you perform the tests requested by @pasquale.pagano@isti.cnr.it?

#22 Updated by Massimiliano Assante 9 months ago

  • Assignee changed from Massimiliano Assante to Giancarlo Panichi

@g.panichi@isti.cnr.it volunteered to do the remaining tests

#23 Updated by Lucio Lelii 9 months ago

the problem (a) highlighted by Lino is related to the ticket #16630

#24 Updated by Giancarlo Panichi 9 months ago

  • Assignee changed from Giancarlo Panichi to Lucio Lelii

Tests Results:

#14 It is partially solved. In case of Move Files the error message is OK, in case of Upload Files the error message is not completely readble(enlarge the upload window or wrap the message).
#31.1 Using the portlet for assigning user roles, the administrator is set correctly by HL but is not recognized by StorageHub.
#31.2 Permission change operation fails in HL
#31.3 New administrators are not considered by StorageHub

The test plan is updated with these results:
https://data.d4science.org/shub/E_NjRrTmFvZ0VJVC9vYjQvMEQ0azR2a0xrNThlTkl1KzRsVWpxNTQ4UmVvK1g3dXJ4a3lBRWxEZFNPVUxrYWJxcQ==

#25 Updated by Pasquale Pagano 9 months ago

  • Priority changed from Normal to Urgent

#26 Updated by Lucio Lelii 9 months ago

#31.1 how can you say "the administrator is set correctly by HL but is not recognized by StorageHub"? the storageHub service reads the acl from Jackrabbit, so did you check the acls directly there ?
#31.2 HL is the same from 4.11, how is it possible that it worked before ?
#31.3 related to #31.1

#27 Updated by Giancarlo Panichi 9 months ago

Hi @lucio.lelii@isti.cnr.it , it could be a pre-production HL configuration problem. Are we sure that the HL configuration file is correct?
Could it also be a problem related to the installation of JackRabbit on Preprod?
When I try to do the Get Info on preVRE folder, I get this logs back:

http://paste.d4science.org/index.php?id=20190507124726_70597

#28 Updated by Lucio Lelii 9 months ago

  • Assignee changed from Lucio Lelii to Giancarlo Panichi

@g.panichi@isti.cnr.it found that probably the problem in the tests were caused by a change he made in the sharing widget. I'm reassigning the ticket to him.

#29 Updated by Giancarlo Panichi 9 months ago

  • Assignee changed from Giancarlo Panichi to Lucio Lelii

@lucio.lelii@isti.cnr.it , the problem solved is inherent to Get Info dialog, the workspace porltet called a wrong method to show sharing permissions.
Now a new build on etics has been created, so @lucio.lelii@isti.cnr.it and @francesco.mangiacrapa@isti.cnr.it verify that all the issues indicated in the Test Plan have been resolved.

#30 Updated by Lucio Lelii 9 months ago

  • Assignee changed from Lucio Lelii to Giancarlo Panichi

I don't understand why you insist to assign the ticket to me ? I'm not in charge neither of the workspace portlet nor the test of this portlet

#31 Updated by Giancarlo Panichi 9 months ago

  • Assignee changed from Giancarlo Panichi to Pasquale Pagano

By the collaboration of @massimiliano.assante@isti.cnr.it we have updated the portlet both on the Preprod and on the Shadow portal.
Failed tests have been repeated. The result is that:

  1. On the Shadow portal the portlet works correctly and also it provides all the data in the Get Info call.
  2. On the Preprod portal, Get Info does not retrieve all information(i.e. the creation and modification date of VRE Folders) due to HL errors.

So most likely there is a problem in preProd related to the use of HL of the newly installed JackRabbit repository.
Therefore, the hypothesis is that these problems will not occur in Production.

@pasquale.pagano@isti.cnr.it what do you think is convenient to do?
We release in production on Friday and so consider the FIRMS and CWP meetings (May 13-18, 2019) too.

#32 Updated by Pasquale Pagano 9 months ago

  • % Done changed from 90 to 100

Shadow is more representative of Pre gateway (unfortunately)

So, I think this testing activity is complete. However, Pre must become a reliable environment and it is not currently since the new instance of JackRabbit is not equivalent to the production one (e.g. Write All is not supported).

Also available in: Atom PDF