Warning: Can't synchronize with repository "(default)" (/home/git/ome.git does not appear to be a Git repository.). Look in the Trac log for more information.
Notice: In order to edit this ticket you need to be either: a Product Owner, The owner or the reporter of the ticket, or, in case of a Task not yet assigned, a team_member"

Task #10900 (closed)

Opened 11 years ago

Closed 11 years ago

Last modified 11 years ago

Bug: web failed on delete dataset with partial MIF

Reported by: jamoore Owned by: wmoore
Priority: critical Milestone: 5.0.0-beta1
Component: Web Version: n.a.
Keywords: fs Cc: fs@…
Resources: n.a. Referenced By: n.a.
References: n.a. Remaining Time: n.a.
Sprint: FS demo 4.2

Description

Occurred during demo 4.1. Server returned GraphConstraintERR but client showed green tick. In OMEROweb.log:

5-14 15:14:49,290 ERROR [      omeroweb.webclient.webclient_utils] (proc.13976) _formatReport:41 Format report: {'warning': '', 'error': ''}
2013-05-14 15:14:49,292 WARNI [                           omero.gateway] (proc.13976) debug:3439 ValueError on <class 'omeroweb.webclient.webclient_gateway.OmeroWebSafeCallWrapper'> to <2b8d8d06-4f63-4bdf-9471-cdfa6ff769d5omero.api.IContainer> getImages(('Dataset', [u'36'], None, <ServiceOptsDict: {'omero.session.uuid': 'dc3ed196-9af7-4d69-9f30-5de86a5a8c17', 'omero.group': '-1', 'omero.client.uuid': '2b8d8d06-4f63-4bdf-9471-cdfa6ff769d5'}>), {})
Traceback (most recent call last):
  File "/home/omero/slave/workspace/OMERO-merge-develop/src/dist/lib/python/omero/gateway/__init__.py", line 3457, in __call__
    return self.f(*args, **kwargs)
  File "/home/omero/slave/workspace/OMERO-merge-develop/src/dist/lib/python/omero_api_IContainer_ice.py", line 106, in getImages
    return _M_omero.api.IContainer._op_getImages.invoke(self, ((rootType, rootIds, options), _ctx))
ValueError: invalid value for element 0 of sequence<long>
2013-05-14 15:14:49,297 ERROR [                omeroweb.webclient.views] (proc.13976) activities:2217 Traceback (most recent call last):
  File "/home/omero/slave/workspace/OMERO-merge-develop/src/dist/lib/python/omeroweb/webclient/views.py", line 2194, in activities
    attempted_imgIds = [i.id.val for i in cs.getImages(dtype, obj_ids, None, conn.SERVICE_OPTS)]
  File "/home/omero/slave/workspace/OMERO-merge-develop/src/dist/lib/python/omero/gateway/__init__.py", line 3460, in __call__
    return self.handle_exception(e, *args, **kwargs)
  File "/home/omero/slave/workspace/OMERO-merge-develop/src/dist/lib/python/omeroweb/webclient/webclient_gateway.py", line 1941, in handle_exception
    e, *args, **kwargs)
  File "/home/omero/slave/workspace/OMERO-merge-develop/src/dist/lib/python/omero/gateway/__init__.py", line 3457, in __call__
    return self.f(*args, **kwargs)
  File "/home/omero/slave/workspace/OMERO-merge-develop/src/dist/lib/python/omero_api_IContainer_ice.py", line 106, in getImages
    return _M_omero.api.IContainer._op_getImages.invoke(self, ((rootType, rootIds, options), _ctx))
ValueError: invalid value for element 0 of sequence<long>

2013-05-14 15:14:49,297 ERROR [                omeroweb.webclient.views] (proc.13976) activities:2218 Status job 'dc3ed196-9af7-4d69-9f30-5de86a5a8c17/IHandlef669c44a-2e24-4e97-86e1-0f083c0fca24 -t:tcp -h 134.36.65.227 -p 52892'error:

Attachments (2)

Screen shot 2013-05-20 at 16.38.38.png (28.3 KB) - added by wmoore 11 years ago.
Delete Dataset with partial MIF
Delete activities gretzky.png (14.5 KB) - added by pwalczysko 11 years ago.

Download all attachments as: .zip

Change History (12)

comment:1 Changed 11 years ago by pwalczysko

Supplementary information: Deletion of a dataset was attempted. The dataset consisted of a MIF, which was not complete (2 out of 3 images only were present in the Dataset). Although the UI showed the success, the Delete action did not go through. After refreshing of the UI, the dataset was still there, it was not deleted and not removed from display. (It was not removed from display after Refresh button was used - prior to Refresh the dataset was missing).

Changed 11 years ago by wmoore

Delete Dataset with partial MIF

comment:2 Changed 11 years ago by wmoore

  • Resolution set to invalid
  • Status changed from new to closed

This seems to be working for me now. See screenshot: https://trac.openmicroscopy.org.uk/ome/attachment/ticket/10900/Screen%20shot%202013-05-20%20at%2016.38.38.png

Maybe a PR was missing from the demo build?

comment:3 Changed 11 years ago by jamoore

  • Resolution invalid deleted
  • Status changed from closed to reopened

This wasn't from a special demo build but rather from the daily build, so it's unlikely a PR was missing. Re-opening and assigning to Petr in case he's reproduced. Note: the issue in the stack trace isn't just the delete but an odd Ice sequence exception. Will, you'll likely need to synthetically cause an exception at that point to see if the green ticket hangs around.

comment:4 Changed 11 years ago by pwalczysko

The discrepancy with @wmoore might be that I am testing on today's GRETZKY (Will probably tests locally).

The bug is still there, exactly as reported.
Once more today's workflow:

  • import as user-4, file /Volumes/ome/data_repo/test_images_good/leica-lif/free edge 6.lif
  • a datset will be created with 16 images of the MIF in it
  • move the last image of the 16 into another dataset
  • select the dataset with (now) 15 images of the MIF in it
  • try to Delete it
  • the Activities will show a following message (see screenshot)
  • after refresh (in OMERO.web) the dataset is still there, undeleted
Last edited 11 years ago by pwalczysko (previous) (diff)

Changed 11 years ago by pwalczysko

comment:5 Changed 11 years ago by wmoore

Very strange: seem that if I try the following against a local server (using latest snoopy merge build) it works OK, but if I try connecting to gretzky and do the same it fails with the error above.

containerService.getImages("Dataset", ["53"], None)

Any ideas why Josh?

comment:6 Changed 11 years ago by wmoore

  • Resolution set to fixed
  • Status changed from reopened to closed

comment:7 Changed 11 years ago by jamoore

Almost certainly due to differences in the model and/or API, i.e. the Ice protocol is just broken between the client & server.

comment:8 Changed 11 years ago by Will Moore <will@…>

(In [be1f1d1d9cbc92cc4fa06eaddaea990fe76b0a84/ome.git] on branch develop) Don't show green tick if delete Error. See #10900

comment:9 Changed 11 years ago by Josh Moore <josh@…>

(In [adea48acfc0f7d7e7c8908f3a59662974dfaa335/ome.git] on branch develop) Merge pull request #1193 from will-moore/activities_web_bugs_May21st

Don't show green tick if delete Error. See #10900

comment:10 Changed 11 years ago by Will Moore <will@…>

(In [9820373d25996d70baa348bc343a3e276051bb10/ome.git] on branch develop) Fix 'invalid value' error on failed MIF delete. See #10900

Note: See TracTickets for help on using tickets. You may also have a look at Agilo extensions to the ticket.

1.3.13-PRO © 2008-2011 Agilo Software all rights reserved (this page was served in: 0.67504 sec.)

We're Hiring!