Bug: No Script stdout if 'cross-group'

[wmoore]

If I run a script in a group that is not my eventContext.groupId by passing omero.group to runScript(), no stdout is returned (I assume no stderr is returned either).

Updated the gatewaytest/multi_group testRunScript() to include this - now fails (commit below).

[wmoore]

Failing gatewaytest: ​https://github.com/aleksandra-tarkowska/openmicroscopy/commit/7c1c7676ea6763a4db214a87114845d78efbf3d3

jrs-macbookpro-25107:test will$ python gatewaytest/multi_group.py ScriptTest.testRunScript
WARNING:omero.client:Ignoring error in client.__del__:<class 'Glacier2.SessionNotExistException'>
WARNING:omero.client:Ignoring error in client.__del__:<class 'Glacier2.SessionNotExistException'>
WARNING:omero.client:Ignoring error in client.__del__:<class 'Glacier2.SessionNotExistException'>
WARNING:omero.client:Ignoring error in client.__del__:<class 'Glacier2.SessionNotExistException'>
WARNING:omero.client:Ignoring error in client.__del__:<class 'Glacier2.SessionNotExistException'>
WARNING:omero.client:Ignoring error in client.__del__:<class 'Glacier2.SessionNotExistException'>
F
======================================================================
FAIL: testRunScript (__main__.ScriptTest)
----------------------------------------------------------------------
Traceback (most recent call last):
  File "gatewaytest/multi_group.py", line 176, in testRunScript
    self.assertTrue('stdout' in results, "Failed to return stdout Original File. #8614")
AssertionError: Failed to return stdout Original File. #8614

----------------------------------------------------------------------
Ran 1 test in 4.170s

FAILED (failures=1)

[jburel]

Moved from sprint 2012-05-08 (14)

[Will Moore <will@…>]

(In [7c1c7676ea6763a4db214a87114845d78efbf3d3/ome.git] on branch develop) Script stdout gatewaytest fails. See #8614

[jmoore]

Added call context parameter to process.getResults() and the test now passes. (Pushed to ola/sprint15-web-bugfixes)

I'm closing this. If we want to do a full review of adding "-1" server-side we can, or as Chris suggested making -1 the default. But this isn't particular to scripts, so no need to keep this open.

[jmoore <josh@…>]

(In [9ce721da663f7a45c65e905635a12ff1d3dcf903/ome.git] on branch develop) Fix call-context script test (See #8614)

Call to process.getResults() did not have the
SERVICE_OPTS like the other method invocations and
so the original file object was not found.

[wmoore]

I have just found that this is failing again. Scripts ran with omero.group set to a different group than my default group don't return stdout.

I have tried to re-run the test above, but I'm getting import problems with the gatewaytest setup. I just tried to rebuild server on origin/develop but now it won't start (need DB upgrade)? so I'll try again tomorrow...

[wmoore]

Test test is still passing, so it seems to be a webclient issue.

[wmoore]

Don't understand this:

When I print the script results like this:

                proc = omero.grid.ScriptProcessPrx.checkedCast(conn.c.ic.stringToProxy(cbString))
                cb = omero.scripts.ProcessCallbackI(conn.c, proc)
                # check if we get something back from the handle...
                if cb.block(0): # ms.
                    cb.close()
                    try:
                        results = proc.getResults(0)        # we can only retrieve this ONCE - must save results
                    except Exception, x:
                        logger.error(traceback.format_exc())
                        continue

                    for key, value in results.items():
                        print key, value

I don't see stdout if the group that I'm running the script in isn't the default group.
I DO see stdout if I'm running the script in the default group.

In both cases, the stdout file appears as expected in the binary repository.

[wmoore]

Fixed ​https://github.com/aleksandra-tarkowska/openmicroscopy/commit/8641d203be3b8472f794cd24137935f124195559

In fact, I think this has never worked in web.