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 #4684 (closed)

Opened 13 years ago

Closed 12 years ago

Bug: tests change passwords leading to other failed tests

Reported by: jamoore Owned by: cxallan
Priority: minor Milestone: OMERO-4.4
Component: General Version: n.a.
Keywords: n.a. Cc:
Resources: n.a. Referenced By: n.a.
References: n.a. Remaining Time: n.a.
Sprint: n.a.

Description

See <http://hudson.openmicroscopy.org.uk/job/OMERO-trunk-components/./component=py,label=x86-windows/388/changes>
...
scons: `TreeList\AllProjects.class' is up to date.
python TreeList/Main.py '--omero.host=10.2.1.194' '--omero.port=04064' 'hudson' 'ome'
Traceback (most recent call last):
 File "TreeList/Main.py", line 16, in <module>
   factory = client.createSession(user, pasw)
 File "C:\hudson\workspace\OMERO-trunk-components\component\py\label\x86-windows\src\dist\lib\python\omero\clients.py", line 448, in createSession
   prx = self.getRouter(self.__ic).createSession(username, password, ctx)
 File "C:\Ice-3.3.1-VC90\python\Glacier2_Router_ice.py", line 107, in createSession
   return _M_Glacier2.Router._op_createSession.invoke(self, ((userId, password), _ctx))
Glacier2.PermissionDeniedException: exception ::Glacier2::PermissionDeniedException
{
   reason = Password check failed
}
scons: *** [Main.py] Error 1
scons: building terminated because of errors.

I checked the password table on *.194, and sure enough, the password was not the hash of "ome". One of the other tests had changed the password.

We can add a changePassword before the execution of the examples in order to rollback any changes, but this is only a partial fix; this may break other tests (though that is fairly unlikely), and also does not fix the underlying race condition. In general, each section may need to be started with a different user to prevent this from happening.

Change History (7)

comment:1 Changed 13 years ago by cxallan

  • Owner set to cxallan

comment:2 Changed 13 years ago by jmoore

  • Milestone changed from OMERO-Beta4.3 to OMERO-Beta4.3.1

Moving to 4.3.1 for handling

comment:3 Changed 13 years ago by jburel

  • Milestone changed from OMERO-Beta4.3.1 to OMERO-Beta4.3.2
  • Sprint changed from 2011-07-07 (1) to 2011-07-21 (1)

Moved from sprint 2011-07-07 (1)

comment:4 Changed 13 years ago by cxallan

  • Sprint changed from 2011-07-21 (1) to 2011-08-04 (2)

Moved from sprint 2011-07-21 (1)

comment:5 Changed 13 years ago by cxallan

  • Sprint 2011-08-04 (2) deleted

comment:6 Changed 13 years ago by jburel

  • Milestone changed from OMERO-Beta4.3.2 to OME-5.0

comment:7 Changed 12 years ago by cxallan

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

It's not clear whether this is even still valid and it needs real investigation in order to be adequately addressed. Closing as invalid, we can re-address this going forward.

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.67046 sec.)

We're Hiring!