Hi Hugo,

On Mon, Dec 1, 2014 at 7:27 PM, <h...@apache.org> wrote:

> Repository: cloudstack
> Updated Branches:
>   refs/heads/master ce2b2a1fc -> ae0cb8279
>
>
> Both scripts should be able to use the data-server introduced in 4.4
>

What does this commit fix? Sorry, I've no idea about the data-server
introduced in 4.4.
Verbose git commit messages are always welcome.

Regards.


>
>
> Project: http://git-wip-us.apache.org/repos/asf/cloudstack/repo
> Commit: http://git-wip-us.apache.org/repos/asf/cloudstack/commit/ae0cb827
> Tree: http://git-wip-us.apache.org/repos/asf/cloudstack/tree/ae0cb827
> Diff: http://git-wip-us.apache.org/repos/asf/cloudstack/diff/ae0cb827
>
> Branch: refs/heads/master
> Commit: ae0cb8279225db0b6d0f642a546792ba6f3f1117
> Parents: ce2b2a1
> Author: Hugo Trippaers <htrippa...@schubergphilis.com>
> Authored: Fri Nov 28 14:08:47 2014 +0100
> Committer: Hugo Trippaers <htrippa...@schubergphilis.com>
> Committed: Mon Dec 1 14:56:58 2014 +0100
>
> ----------------------------------------------------------------------
>  setup/bindir/cloud-set-guest-password.in | 69 +++++++++++++--------------
>  setup/bindir/cloud-set-guest-sshkey.in   | 36 ++++++--------
>  2 files changed, 48 insertions(+), 57 deletions(-)
> ----------------------------------------------------------------------
>
>
>
> http://git-wip-us.apache.org/repos/asf/cloudstack/blob/ae0cb827/setup/bindir/cloud-set-guest-password.in
> ----------------------------------------------------------------------
> diff --git a/setup/bindir/cloud-set-guest-password.in b/setup/bindir/
> cloud-set-guest-password.in
> index 4c7c575..2c7304b 100755
> --- a/setup/bindir/cloud-set-guest-password.in
> +++ b/setup/bindir/cloud-set-guest-password.in
> @@ -40,45 +40,44 @@ for DHCP_FILE in $DHCP_FOLDERS; do
>
>                 if [ -n "$PASSWORD_SERVER_IP" ]; then
>                         logger -t "cloud" "Found password server IP
> $PASSWORD_SERVER_IP in $DHCP_FILE"
> -                       logger -t "cloud" "Sending request to password
> server at $PASSWORD_SERVER_IP"
> -
> -                       password=$(wget -q -t 3 -T 20 -O - --header
> "DomU_Request: send_my_password" $PASSWORD_SERVER_IP:$PASSWORD_SERVER_PORT)
> -
> -                       if [ $? -eq 0 ]; then
> -                               password=$(echo $password | tr -d '\r')
> -                               logger -t "cloud" "Got response from
> server at $PASSWORD_SERVER_IP"
> -
> -                               case $password in
> -                                       "")
> -                                               logger -t "cloud"
> "Password server at $PASSWORD_SERVER_IP did not have any password for the
> VM"
> -                                               continue
> -                                               ;;
> -                                       "bad_request")
> -                                               logger -t "cloud" "VM sent
> an invalid request to password server at $PASSWORD_SERVER_IP"
> -
>  error_count=$((error_count+1))
> -                                               continue
> -                                               ;;
> -                                       "saved_password")
> -                                               logger -t "cloud" "VM has
> already saved a password from the password server at $PASSWORD_SERVER_IP"
> -                                               continue
> -                                               ;;
> -                                       *)
> -                                               logger -t "cloud" "VM got
> a valid password from server at $PASSWORD_SERVER_IP"
> -                                               password_received=1
> -                                               break
> -                                               ;;
> -                                       esac
> -                       else
> -                               logger -t "cloud" "Failed to send request
> to password server at $PASSWORD_SERVER_IP"
> -                               error_count=$((error_count+1))
> -                       fi
> -               else
> -                       logger -t "cloud" "Could not find password server
> IP in $DHCP_FILE"
> -                       error_count=$((error_count+1))
> +                        break
>                 fi
>         fi
>  done
>
> +if [ -z "$PASSWORD_SERVER_IP" ] ; then
> +         logger -t "cloud" "Unable to determine the password server,
> falling back to data-server"
> +         PASSWORD_SERVER_IP=data-server
> +fi
> +
> +logger -t "cloud" "Sending request to password server at
> $PASSWORD_SERVER_IP"
> +password=$(wget -q -t 3 -T 20 -O - --header "DomU_Request:
> send_my_password" http://$PASSWORD_SERVER_IP:$PASSWORD_SERVER_PORT)
> +if [ $? -eq 0 ]; then
> +        password=$(echo $password | tr -d '\r')
> +        logger -t "cloud" "Got response from server at
> $PASSWORD_SERVER_IP"
> +
> +        case $password in
> +                "")
> +                        logger -t "cloud" "Password server at
> $PASSWORD_SERVER_IP did not have any password for the VM"
> +                        ;;
> +                "bad_request")
> +                        logger -t "cloud" "VM sent an invalid request to
> password server at $PASSWORD_SERVER_IP"
> +                        error_count=$((error_count+1))
> +                        ;;
> +                "saved_password")
> +                        logger -t "cloud" "VM has already saved a
> password from the password server at $PASSWORD_SERVER_IP"
> +                        ;;
> +                *)
> +                        logger -t "cloud" "VM got a valid password from
> server at $PASSWORD_SERVER_IP"
> +                        password_received=1
> +                        ;;
> +                esac
> +else
> +        logger -t "cloud" "Failed to send request to password server at
> $PASSWORD_SERVER_IP"
> +        error_count=$((error_count+1))
> +fi
> +
> +
>  if [ "$password_received" == "0" ]; then
>         if [ "$error_count" == "$file_count" ]; then
>                 logger -t "cloud" "Failed to get password from any server"
>
>
> http://git-wip-us.apache.org/repos/asf/cloudstack/blob/ae0cb827/setup/bindir/cloud-set-guest-sshkey.in
> ----------------------------------------------------------------------
> diff --git a/setup/bindir/cloud-set-guest-sshkey.in b/setup/bindir/
> cloud-set-guest-sshkey.in
> index 4955dea..0472a7c 100755
> --- a/setup/bindir/cloud-set-guest-sshkey.in
> +++ b/setup/bindir/cloud-set-guest-sshkey.in
> @@ -37,32 +37,24 @@ for DHCP_FILE in $DHCP_FOLDERS; do
>          SSHKEY_SERVER_IP=$(grep dhcp-server-identifier $DHCP_FILE | tail
> -1 | awk '{print $NF}' | tr -d '\;')
>
>          if [ -n "$SSHKEY_SERVER_IP" ]; then
> -            logger -t "cloud" "Sending request to ssh key server at
> $SSHKEY_SERVER_IP"
> -
> -            publickey=$(wget -q -t 3 -T 20 -O - http://
> $SSHKEY_SERVER_IP/latest/public-keys)
> -
> -            if [ $? -eq 0 ]; then
> -                logger -t "cloud" "Got response from server at
> $SSHKEY_SERVER_IP"
> -                keys_received=1
> -                break
> -            fi
> +            logger -t "cloud" "Found ssh key server at $SSHKEY_SERVER_IP"
> +            break
>          else
> -            logger -t "cloud" "Could not find ssh key server IP in
> $DHCP_FILE trying with the name data-server "
> +            logger -t "cloud" "Could not find ssh key server IP in
> $DHCP_FILE"
>          fi
>      fi
>  done
>
> -if [ "$keys_received" == "0" ]; then
> -    SSHKEY_SERVER_IP=$(nslookup data-server | grep Address |tr '\n' ' '|
> awk '{print $4}')
> -    logger -t "cloud" "Sending request to ssh key server at
> $SSHKEY_SERVER_IP"
> -     publickey=$(wget -q -t 3 -T 20 -O -
> http://data-server/latest/public-keys)
> -     if [ $? -eq 0 ]
> -     then
> -        logger -t "cloud" "Got response from server at $SSHKEY_SERVER_IP"
> -        keys_received=1
> -     else
> -        logger -t "cloud" "Could not resolve the name data-server"
> -     fi
> +if [ -z "$SSHKEY_SERVER_IP" ]; then
> +     logger -t "cloud" "Unable to determine the password server, falling
> back to data-server"
> +     SSHKEY_SERVER_IP=data-server
> +fi
> +
> +logger -t "cloud" "Sending request to ssh key server at $SSHKEY_SERVER_IP"
> +publickey=$(wget -q -t 3 -T 20 -O - http://
> $SSHKEY_SERVER_IP/latest/public-keys)
> +if [ $? -eq 0 ]; then
> +    logger -t "cloud" "Got response from server at $SSHKEY_SERVER_IP"
> +    keys_received=1
>  fi
>
>  if [ "$keys_received" == "0" ]; then
> @@ -94,4 +86,4 @@ echo "$publickey" >> $authorized
>
>  which restorecon && restorecon -R -v $sshdir
>
> -exit 0
> \ No newline at end of file
> +exit 0
>
>

Reply via email to