Github user asfgit closed the pull request at:
https://github.com/apache/cloudstack/pull/1071
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is
Github user DaanHoogland commented on the pull request:
https://github.com/apache/cloudstack/pull/1071#issuecomment-157299325
Had a look yesterday and lgtm, didn't want to comment untill I saw the
test results. (It seemed so small ;)
---
If your project is set up for it, you can rep
Github user wilderrodrigues commented on the pull request:
https://github.com/apache/cloudstack/pull/1071#issuecomment-157286509
Hi @remibergsma and @karuturi
Went through the code, which was quite straight forward. Based on that,
this PR LGTM :+1:
Cheers,
Wilde
Github user remibergsma commented on the pull request:
https://github.com/apache/cloudstack/pull/1071#issuecomment-157169270
@DaanHoogland @wilderrodrigues @miguelaferreira Can either of you review so
we can merge this and open master for new features?
---
If your project is set up f
Github user remibergsma commented on the pull request:
https://github.com/apache/cloudstack/pull/1071#issuecomment-157168301
LGTM:
```
nosetests --with-marvin --marvin-config=${marvinCfg} -s -a
tags=advanced,required_hardware=true \
component/test_vpc_redundant.py \
Github user karuturi commented on a diff in the pull request:
https://github.com/apache/cloudstack/pull/1071#discussion_r44907513
--- Diff: build/replace.properties ---
@@ -26,4 +26,4 @@ AGENTLOG=logs/agent.log
MSMNTDIR=/mnt
COMPONENTS-SPEC=components.xml
REMOTEHOST=l
Github user remibergsma commented on a diff in the pull request:
https://github.com/apache/cloudstack/pull/1071#discussion_r44907228
--- Diff: build/replace.properties ---
@@ -26,4 +26,4 @@ AGENTLOG=logs/agent.log
MSMNTDIR=/mnt
COMPONENTS-SPEC=components.xml
REMOTEHOS
Github user remibergsma commented on the pull request:
https://github.com/apache/cloudstack/pull/1071#issuecomment-156980076
Thanks @karuturi will give it a test-drive soon!
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well.
GitHub user karuturi opened a pull request:
https://github.com/apache/cloudstack/pull/1071
Merge 4.6 release branch to master
Initial merge of 4.6 to master
ignored pom.xml version number changes and changes to debian/changelog and
engine/schema/src/com/cloud/upgrade/DatabaseUpg
