Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[v6r20] catch mroe exception in the ReqClient for displaying FTS Jobs #3829

Merged
merged 1 commit into from
Sep 21, 2018

Conversation

chaen
Copy link
Contributor

@chaen chaen commented Sep 13, 2018

Replaces #3802
@marianne013 can you please test this hack we talked about ? It will just avoid your script to crash, but it won't display the requests. Sorry I did not try it.

BEGINRELEASENOTES

*RMS
FIX: catch more exception in the ReqClient when trying to display the associated FTS jobs

ENDRELEASENOTES

@coveralls
Copy link

Coverage Status

Coverage increased (+0.003%) to 22.265% when pulling fbf9575 on chaen:rel-v6r20_FIX_reqClientcatchMore into 456ff2f on DIRACGrid:rel-v6r20.

@marianne013
Copy link
Contributor

lx02:diracdev_v6r20p13 > dirac-rms-request test2009.orig
Request name='test2009.orig' ID=17 Status='Waiting' ('Assigned' in DB)
Created 2018-09-20 10:12:21, Updated 2018-09-20 10:13:12, NotBefore 2018-09-20 10:12:21
Owner: '/C=UK/O=eScience/OU=Imperial/L=Physics/CN=daniela bauer', Group: gridpp_user
[0] Operation Type='ReplicateAndRegister' ID=23 Order=0 Status='Waiting'
TargetSE: UKI-LT2-QMUL-disk - Created 2018-09-20 10:12:21, Updated 2018-09-20 10:12:21
[01] ID=23 LFN='/gridpp/user/d/daniela.bauer/simonisstillannoying4' Status='Waiting' Checksum='1d15bf1b'
exception in FTS3JSONDecoder No module named sqlalchemy for type FTS3TransferOperation

(Not sure about the last exception, that existed before as well)

@atsareg
Copy link
Contributor

atsareg commented Sep 21, 2018

Is the test conclusive for this PR ?

@marianne013
Copy link
Contributor

From my point of view, yes

@atsareg atsareg merged commit cff8ddd into DIRACGrid:rel-v6r20 Sep 21, 2018
@chaen chaen deleted the rel-v6r20_FIX_reqClientcatchMore branch August 14, 2019 14:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants