Hudson 3.0.0 no longer supports /api/python interface

[c.jones@…]

The HudsonTracPlugin requires that hudson makes available the API for python on the URL /api/python at the moment, however now that Eclipse have taken on hudson, and version 3.0.0 has been released, the python API is no longer supported. Once hudson is upgraded to version 3.0.0 you get this error in the TRAC timeline:

Trac Error

Hudson Builds event provider (HudsonTracPlugin) failed:
IOError: Error getting build info from 
'http://myserver:8080/job/jobname/api/python?
tree=builds[building,timestamp,duration,result,description,url,
fullDisplayName,changeSet[items[revision,id,user,author[fullName]]],
culprits[fullName],actions[causes[userName]]]': 
HTTPError: HTTP Error 404: Not Found. 
This most likely means you configured a wrong job_url, username, or password.

See discussion here, that explains the python interface is no longer supported: ​http://www.eclipse.org/forums/index.php/t/457665/

Apparently though, the api is almost identical to the JSON interface anyway, so perhaps HudsonTracPlugin can be fairly easily modified to point to /api/json instead?

[TCF]

Without this fix, this plug-in is completely broken (and in fact it breaks the timeline feature in Trac as noted in the bug and thus is actually disruptive). So, made a fix for this myself, replacing the python API with the one for JSON, and also replacing the code used to handle the HTTP calls with the requests library (from python-requests.org). It works fine for my Hudson installation but it probably needs more testing against other configurations of Hudson and Jenkins. Also, since this is my first time writing python, I could use a little coaching in how to make sure I bundle it correctly so it knows the requests library is now a dependency.

Can someone help me with those two items? Then I could give this code to whoever runs this project and have them update it as the next release.

[Ryan J Ollos]

Replying to tcfenstermaker:

[...] Also, since this is my first time writing python, I could use a little coaching in how to make sure I bundle it correctly so it knows the requests library is now a dependency.

You'll just need to add an entry in install_requires. Here​ is an example, and the relevant ​documentation for setuptools.

Can someone help me with those two items? Then I could give this code to whoever runs this project and have them update it as the next release.

It will probably be easiest if you just attach the code to this ticket, or give us a link to a fork of the code in GitHub, BitBucket, etc...

[TCF]

Thanks for the help rjollos; I'll try to get to this this week. I'll probably just attach the code to the ticket, as it's only one module. I just need to clean it up a bit.

[Cinc]

I created a patch to get the plugin to work with Trac 0.12.3 and Hudson 3.2.x:

Trac/Plugins/HudsonTrac/HudsonTrac-0.5-py2.7.egg/HudsonTrac/HudsonTracPlugin.py

@@ -15 +15 @@
 import urllib2
 import base64
 from datetime import datetime
+import json
 
 from genshi.builder import tag
 from trac.core import *
@@ -29 +30 @@
     from trac.timeline.api import ITimelineEventProvider
 except ImportError:
     from trac.Timeline import ITimelineEventProvider
-try:
-    from ast import literal_eval
-except ImportError:
-    def literal_eval(str):
-        return eval(str, {"__builtins__":None}, {"True":True, "False":False})
 
+
 class HudsonTracPlugin(Component):
     """
     Display Hudson results in the timeline and an entry in the main navigation
@@ -105 +102 @@
         api_url = unicode_quote(self.job_url, '/%:@')
         if api_url and api_url[-1] != '/':
             api_url += '/'
-        api_url += 'api/python'
+        api_url += 'api/json'
 
         # set up http authentication
         if self.username and self.api_token:
@@ -172 +169 @@
                 cset = resp.info().getparam('charset') or 'ISO-8859-1'
 
                 ct   = resp.info().gettype()
-                if ct != 'text/x-python':
+                if ct != 'application/javascript':
                     local_exc = True
                     raise IOError(
                         "Error getting build info from '%s': returned document "
-                        "has unexpected type '%s' (expected 'text/x-python'). "
+                        "has unexpected type '%s' (expected 'application/javascript'). "
                         "The returned text is:\n%s" %
                         (self.info_url, ct, unicode(resp.read(), cset)))
 
-                info = literal_eval(resp.read())
+                info = json.loads(resp.read())
 
                 return info, cset
             except Exception:

Note that the revisions are our internal SVN revisions and the path must be adapted properly because the plugin was imported into our private tree.

Sorry for this inconvenience.

[Cinc]

Patch to get ththe plugin to work with Hudson 3.2.x. Note that the path in this file must be corrected properly before applying. Sorry for this inconvenience.

[Dirk Stöcker]

As I don't use Hudson, but Jenkins (current version works): Is this still relevant?

[Dirk Stöcker]

In 18579:

fix #10905 - support json interface and make it default