Stuck on an issue?

Lightrun Answers was designed to reduce the constant googling that comes with debugging 3rd party libraries. It collects links to all the places you might be looking at while hunting down a tough bug.

And, if you’re still stuck at the end, we’re happy to hop on a call to see how we can help out.

Improve messaging when `arc` is missing on jenkins nodes

See original GitHub issue

Original title: “FATAL: net.sf.json.JSONNull cannot be cast to java.lang.String”

context: I had a phab diff already under review. I updated it via the phab web UI by uploading a new .patch file with the latest changes.

CI bombed. Looking at the build logs I see this:

Checking out Revision 67cf3661e48c02f8a8a7df24a6714486fd760a26 (refs/remotes/origin/master)
 > git config core.sparsecheckout # timeout=10
 > git checkout -f 67cf3661e48c02f8a8a7df24a6714486fd760a26
 > git rev-list 67cf3661e48c02f8a8a7df24a6714486fd760a26 # timeout=10
[phabricator:harbormaster] Sending Harbormaster BUILD_URL via PHID: PHID-HMBT-utnue5zrz33sokyy3pob
[phabricator:conduit] Fetching differential from Conduit API
FATAL: net.sf.json.JSONNull cannot be cast to java.lang.String
java.lang.ClassCastException: net.sf.json.JSONNull cannot be cast to java.lang.String
	at com.uber.jenkins.phabricator.conduit.Differential.getBaseCommit(Differential.java:116)
	at com.uber.jenkins.phabricator.PhabricatorBuildWrapper.setUp(PhabricatorBuildWrapper.java:124)
	at hudson.model.Build$BuildExecution.doRun(Build.java:156)
	at hudson.model.AbstractBuild$AbstractBuildExecution.run(AbstractBuild.java:534)
	at hudson.model.Run.execute(Run.java:1720)
	at hudson.model.FreeStyleBuild.run(FreeStyleBuild.java:43)
	at hudson.model.ResourceController.execute(ResourceController.java:98)
	at hudson.model.Executor.run(Executor.java:404)

it seems logical that a String could be null … but the mechanism for achieving that translation here appears to be broken.

Issue Analytics

State:
Created 7 years ago
Comments:10 (7 by maintainers)

Top GitHub Comments

1reaction

Stefan-Schmidtcommented, Sep 27, 2017

For me it failed on differentials created from raw diffs over the phab web UI. It started to work fine once I tested with real arc diffs.

Maybe the differential coming from the web UI are missing some patch metdata and thus the error gets triggered?

1reaction

ascandellacommented, Jan 25, 2017

Oh, that’s not good, we definetely need to provide better documentation and error messages around that.

Top Results From Across the Web

Managing Nodes - Jenkins

Configure your node to use the "Launch agents by connecting it to the master" launch method. Click Save ; Note the command required...

'Jenkins' missing node label 'master' after v. 2.307+ upgrade

Otherwise, you can go to the Dashboard -> Manage Jenkins -> Manage Nodes and Clouds, then press downwards-arrow when you hover over a...

Setting up a CI/CD pipeline w/ Jenkins, Nexus, Kubernetes

We'll use Kublr to manage our Kubernetes cluster, Jenkins, Nexus, ... If a physical node fails and volume data is lost, a new...

Bountysource

Improve messaging when `arc` is missing on jenkins nodes.

Untitled

... ethernet: fs_enet: Add missing MODULE_LICENSE (Michael Ellerman) - misdn: dsp: select CONFIG_BITREVERSE (Arnd Bergmann) to fix build error on ARC (Randy ...