Verigreen fails to get status for jenkins jobs if the 'jenkins.branchParamName' parameter is not the first parameter

[aleitamar]

When verigreen tries to get the status of jenkins jobs, it assumes that the first parameter of the job will be the 'jenkins.branchParamName' parameter.

verigreen/verigreen-collector-impl/src/main/java/com/verigreen/collector/jobs/CallJenkinsJob.java

Line 211 in 93e4efe

JsonObject parameterJsonObject = (JsonObject) jsonParametersArray.get(0);

If the first parameter is a different one, verigreen fails with 'TRIGGER_FAILED', even after triggering the jenkins job successfully.

This should be fixed or documented (maybe as part of the FAQ 'TRIGGER_FAILED' section?)

[soninob]

sounds like a bug, thank you!
@borzamircea @AndreeaDora - have time to look at that?
I guess we should call it by name

[borzamircea]

@soninob I think this could be fixed by getting hte parameter by key and not by index.

Something like:

JsonObject parameterJsonObject = (JsonObject) jsonParametersArray.get("branchParamName");

[soninob]

Thanks Mircea, care to supply a PR?

[borzamircea]

Will do in a couple of days once things are quieter.

[borzamircea]

Did some quick reading and the jsonArray only has get methods by index and not key: https://docs.oracle.com/javaee/7/api/javax/json/JsonArray.html

[borzamircea]

But apparently searching by key should do the trick.

https://stackoverflow.com/questions/42549545/get-value-by-key-jsonarray/42549752

[borzamircea]

JsonArray jsonParametersArray =  parameterJsonObjectArray.getAsJsonArray("parameters");

for(int i=0;i<jsonParametersArray.length();i++)
{
	JSONObject parameterJsonObject = jsonArray.getJSONObject(i);
	if (parameterJsonObject.keySet().contains("branchParamName"))
	{
		values.setBranchName(parameterJsonObject.get("branchParamName").getAsString());
		break;
	}
}

This should work.

[soninob]

good! Thank you, waiting for your PR :-)