rhtyd commented on a change in pull request #3075: KVM: Prevent regenerating 
keystore on provisionCertificate API
URL: https://github.com/apache/cloudstack/pull/3075#discussion_r290569265
 
 

 ##########
 File path: scripts/util/keystore-cert-import
 ##########
 @@ -38,9 +38,6 @@ if [ -z "${KS_PASS// }" ]; then
     exit 1
 fi
 
-# Use a new keystore file
-NEW_KS_FILE="$KS_FILE.new"
 
 Review comment:
   The script was create to be idempotent for jks creation and cert import. 
I'll have to test concurrent cases and see if this script would pass/fail. I'll 
test and review today and keep you posted @nvazquez thanks.

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services

Reply via email to