search

rawkintrevo / bluemix-extra-services

0 stars 1 forks source link

[BME-5] Added a check make sure setS3Auth is run prior to running Zep start #6

Closed dustinvanstee closed 7 years ago

dustinvanstee commented 7 years ago

Added a status flag to make sure setS3Auth is run before z.start. If not, an error message will print. Also added a simple check for making sure rootkey.csv exists.

added a new status flag s3authSet = False once setS3Auth is run, then this flag is set True start method checks this flag prior to start. if not set, prints error message and aborts ..

rawkintrevo commented 7 years ago

Looking good @dustinvanstee . A couple of more comments to address after testing then should be good to merge.

Thanks!

dustinvanstee commented 7 years ago

Addressed comments above, and performed some tests using the example deployment scripts ... Looks good to me. Code re-committed

rawkintrevo commented 7 years ago

lgtm if no further comment