Maybe @cverna or @vivekanand1101 have an idea.

I don't know if pgimport has a --debug option or so that would give us a little more info

I was able to import tickets from a different fedorahosted project. So it looks like the issue is on the FH side. Something with the other project is causing issues which I will need to investigate. Please close this issue.

Thanks,
Mark

pgimport has no --debug option, but it is a good idea just need to be done :smile:

I ll try to import this project on my side and see if I can get more info.

Yeah its odd, https://fedorahosted.org/nunc-stans/ works, but https://fedorahosted.org/389/ does not. Could the project name being all numbers be an issue? <grasping at straws>

This is the answer of fedorahosted for the post request pgimport does

{u'id': 1, u'result': None, u'error': {u'message': u'ServiceException details : unsupported type for timedelta microseconds component: unicode', u'code': -32603, u'name': u'JSONRPCError'}}

Not sure if the name of the project plays a part here, I ll try to find another project with only number as name and try :)
`

Should we move this?

Should we move this?

Maybe you are already there :joy:

(Second attempt :/)

Opened a fedora infrastructure ticket to investigate this from their side:

https://pagure.io/fedora-infrastructure/issue/5573

The problem with the 389 repo was that there were these odd empty/zombie tickets in the trac database. Once they were deleted the import continued, but we ran into another problem. Some of the tickets, for some reason, were missing attachments and this broke the importer. I have a patch for this actually, but I need to clean it up. Once its done I'll attach that patch to this issue for review.

Once its done I'll attach that patch to this issue for review.

Great! Thank you :)

Patch proposal, please review.

Looks good :)

I would just prefer If we could use the If foobar is not None syntax. For example if attachment_resp is not None instead of if attachment_resp. This is recommended by pep8

If you don't want to make another patch, I can put a commit on top of your patch

Thanks

Thanks for the review! New patch attached that passes pep8

Thanks, just merged it to develop