----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/11657/ -----------------------------------------------------------
Review request for cloudstack. Description ------- This patch replaces the incorrect secondary DNS addresses (4.4.4.4 -> 8.8.4.4). There are several places where 4.4.4.4 is defined as secondary DNS. Should this instead be 8.8.4.4 which is the Google [1] secondary DNS address? I also brought this up on the dev mailing list [2] [1] https://developers.google.com/speed/public-dns/ [2] http://mail-archives.apache.org/mod_mbox/cloudstack-dev/201306.mbox/%3C921C9A2F781547A08CEAC84589A84A60%40digitalsanctum.com%3E This addresses bug https://issues.apache.org/jira/browse/CLOUDSTACK-2859. Diffs ----- tools/appliance/definitions/systemvmtemplate/postinstall.sh 7c06796 tools/appliance/definitions/systemvmtemplate64/postinstall.sh 35a4e4a tools/devcloud-kvm/devcloud-kvm.cfg b3f048a tools/devcloud/devcloud.cfg e6ab71b tools/devcloud/devcloud_internal-mgt.cfg fe3dd1b tools/devcloud/quickcloud.cfg a2613d2 tools/marvin/marvin/configGenerator.py c970ada tools/marvin/marvin/jsonHelper.py 37363bc Diff: https://reviews.apache.org/r/11657/diff/ Testing ------- I ran `mvn clean install -P developer,systemvm` with no issues. Thanks, Shane Witbeck
