cloudstack-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Hugo Trippaers <HTrippa...@schubergphilis.com>
Subject RE: Review Request 13491: Fix specifying db passwd in cloud-install-sys-tmplt
Date Mon, 12 Aug 2013 09:16:44 GMT
:D  This is going to be hard to beat unless we start counting the changes in bits when indivdual
characters change.


Cheers,

Hugo

From: Dave Cahill [mailto:dcahill@midokura.com]
Sent: maandag 12 augustus 2013 11:05
To: Hugo Trippaers
Cc: cloudstack
Subject: Re: Review Request 13491: Fix specifying db passwd in cloud-install-sys-tmplt

Thanks! I'm going for the smallest patch of the year prize (1 character). :)
On Mon, Aug 12, 2013 at 6:03 PM, Hugo Trippaers <htrippaers@schubergphilis.com<mailto:htrippaers@schubergphilis.com>>
wrote:
This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/13491/



On August 12th, 2013, 8:59 a.m. UTC, Hugo Trippaers wrote:

Ship It!

master 062eab8df44ba76dad28800599d87ef131bb0bec

4.2 9d053d04ecd08ca3ae7cbf6d6530cf96c3007687


- Hugo


On August 12th, 2013, 7:14 a.m. UTC, Dave Cahill wrote:
Review request for cloudstack.
By Dave Cahill.

Updated Aug. 12, 2013, 7:14 a.m.
Repository: cloudstack-git
Description

Previously database password was specified as a no-argument option, however the rest of the
script expected an argument.


Testing

Tested installing System VM template using the script and specifying db username / pass before
change (didn't work) and after (worked).


Diffs

  *   scripts/storage/secondary/cloud-install-sys-tmplt (7237ffa840878e3ce31e71964430fba38f93efb2)

View Diff<https://reviews.apache.org/r/13491/diff/>



Mime
  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message