need validate nasConfig

[hougangliu]

Now for a studyjob, we never check if a nasConfig is valid or not.
#339 is nasrl suggestion service, which always expects graphConfig (numlayers, input size,outputsize) and operations field present. we need validate nasconfig filed before store studyjob info into db.

@andreyvelich WDYT?

[hougangliu]

/cc @DeeperMind

[andreyvelich]

@hougangliu
This is a good point!
Do you think, we should check it in the controller step?
For example, if something wrong with nasConfig structure, studyJob status should be Failed and controller should stop the job?

[andreyvelich]

/area nas

[johnugeorge]

See validateStudy in

katib/pkg/controller/studyjob/katib_api_util.go

Line 28 in f11c13e

if validErr := validateStudy(instance, ns); validErr != nil {

[andreyvelich]

@johnugeorge
Thats cool!
Should I add a part there for the NAS job ?

[andreyvelich]

/assign