saagie / gradle-saagie-dataops-plugin

Saagie Gradle Plugin for the new version (2.0) of Saagie Dataops Orchestrator
Apache License 2.0
2 stars 0 forks source link

GetJobInstance does not work any more #80

Closed sgokaram-saagie closed 4 years ago

sgokaram-saagie commented 4 years ago

saagies-mbp:Gradle shivakumar$ gradle -b build.gradle.projectsGetJobInstanceStatus projectsGetJobInstanceStatus

FAILURE: Build failed with an exception.

BUILD FAILED in 515ms

plugins { id "io.saagie.gradle-saagie-dataops-plugin" version "1.1.23" }

saagie { server { url = saagieurl login = saagieuserid password = saagiepassword environment = saagieplatformid jwt = true } project { id = saagieprojectid } jobinstance { id = '8ea798c9-e600-4417-a574-c55842c65801' } }

DecampsRenan commented 4 years ago

@sgokaram-saagie I just found the issue: i named the config key according to the graphql schema, jobInstance; or you are using jobinstance with a lower cased i

Should I update the code to reflect the wiki, or should we update the wiki to reflect the graphql API ?

sgokaram-saagie commented 4 years ago

@DecampsRenan - So i checked wiki for what has been out naming convention. So far i dont see any use of mixed case. So lets make it uniform and not use mixed cases just all lower case through out the project. This way we are consistent.

So change the code for now to make it jobinstance lower case i.

Let me know if there are other places of mixed case , i could not find them.

DecampsRenan commented 4 years ago

This issue is fixed with the #50 , so I close it