search

project-everest / everest-ci

CI scripts for project everest
3 stars 8 forks source link

Darrenge vale kremlin #67

Closed darrenge closed 7 years ago

darrenge commented 7 years ago

Double checked with Barry on whether he is ok with this and he is.

msprotz commented 7 years ago

It seems like this doesn't use the latest version of the code I had written in the original github issue. Have you tried it out? This was just an (untested) sketch...

Thanks,

Jonathan

darrenge commented 7 years ago

My mistake ... I tested it out before you updated it. My mistake was that I thought the new build of Kremlin after Everest build would push out what we needed. I will update with your updated sketch and retest it.

darrenge commented 7 years ago

Updated with your new suggestions. The code was correct ... just needed to add a couple packages to the Linux agent docker container (informed Irina which ones were needed). Barry is aware of changes to be needed for NoVerify which will be in Kremlin

msprotz commented 7 years ago

Excellent. Thanks!

darrenge commented 7 years ago

One of these days, I will do it right the first time. I'll keep at it ...