Github user rhtyd commented on a diff in the pull request:
https://github.com/apache/cloudstack/pull/1663#discussion_r76561209
--- Diff: test/integration/smoke/test_router_dns.py ---
@@ -0,0 +1,286 @@
+# Licensed to the Apache Software Foundation (ASF) under one
+# or more c
Github user rhtyd commented on a diff in the pull request:
https://github.com/apache/cloudstack/pull/1663#discussion_r76561273
--- Diff: test/integration/smoke/test_router_dns.py ---
@@ -0,0 +1,286 @@
+# Licensed to the Apache Software Foundation (ASF) under one
+# or more c
Github user rhtyd commented on a diff in the pull request:
https://github.com/apache/cloudstack/pull/1663#discussion_r76561300
--- Diff: test/integration/smoke/test_router_dns.py ---
@@ -0,0 +1,286 @@
+# Licensed to the Apache Software Foundation (ASF) under one
+# or more c
Github user rhtyd commented on a diff in the pull request:
https://github.com/apache/cloudstack/pull/1663#discussion_r76561383
--- Diff: test/integration/smoke/test_router_dns.py ---
@@ -0,0 +1,286 @@
+# Licensed to the Apache Software Foundation (ASF) under one
+# or more c
Github user rhtyd commented on a diff in the pull request:
https://github.com/apache/cloudstack/pull/1663#discussion_r76561440
--- Diff: test/integration/smoke/test_router_dns.py ---
@@ -0,0 +1,286 @@
+# Licensed to the Apache Software Foundation (ASF) under one
+# or more c
Github user rhtyd commented on the issue:
https://github.com/apache/cloudstack/pull/1663
@jburwell @NuxRo thanks for the reviews, I've fixed the outstanding
issues. Please re-review.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitH
Github user rhtyd commented on the issue:
https://github.com/apache/cloudstack/pull/1636
@jburwell @milamberspace due to version change in poms already merged on
4.8 without a valid db upgrade path (which I had added in a separate PR), we'll
see db failures for all PRs targeting 4.8 b
thanks Simon and Rohit for the valuable inputs! After applying the
following procedure the ssl errors are gone and host state is UP.
# service cloudstack-management stop
mysql> delete from configuration where name = "ssl.keystore" ;
# mv /etc/cloudstack/management/cloudmanagementserver.keystore
Github user rhtyd commented on a diff in the pull request:
https://github.com/apache/cloudstack/pull/1635#discussion_r76569173
--- Diff:
engine/api/src/org/apache/cloudstack/engine/cloud/entity/api/VirtualMachineEntity.java
---
@@ -115,6 +115,12 @@ String reserve(DeploymentPlanner
Github user rhtyd closed the pull request at:
https://github.com/apache/cloudstack/pull/1663
---
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 rhtyd reopened a pull request:
https://github.com/apache/cloudstack/pull/1663
[LTS/blocker] CLOUDSTACK-6432: Prevent DNS reflection attacks
CLOUDSTACK-6432: Prevent DNS reflection attacks
DNS on VR should not be publically accessible as it may be prone to
Github user rhtyd commented on the issue:
https://github.com/apache/cloudstack/pull/1653
@NuxRo this PR can be closed as we've moved to a new one with a marvin test
#1663
Thanks.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitH
Github user NuxRo closed the pull request at:
https://github.com/apache/cloudstack/pull/1653
---
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 NuxRo commented on the issue:
https://github.com/apache/cloudstack/pull/1663
Thanks a lot for implementing this properly :)
---
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 fe
Github user ProjectMoon commented on the issue:
https://github.com/apache/cloudstack/pull/1623
Yeah. Unfortunately I have not had any time to work on the pull requests
lately. When I have time I will reopen it against 4.9.
---
If your project is set up for it, you can reply to this e
Github user ProjectMoon closed the pull request at:
https://github.com/apache/cloudstack/pull/1624
---
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 featu
Github user ProjectMoon commented on the issue:
https://github.com/apache/cloudstack/pull/1624
Jenkins build log has vanished into 404 oblivion. Will see what happened
with Travis (https://travis-ci.org/apache/cloudstack/builds/154436801).
---
If your project is set up for it, you ca
GitHub user ProjectMoon reopened a pull request:
https://github.com/apache/cloudstack/pull/1624
Fixes regarding VOLUME_DELETE events resulting from account deletion
New version of #1491.
**Original Description**
New version of #1373, but updated for the 4.7 branch with a
Github user ProjectMoon commented on the issue:
https://github.com/apache/cloudstack/pull/1624
Have rebased to latest 4.8. Travis build failed with what looks like
`Unable to execute
/home/travis/build/apache/cloudstack/developer/target/db/templates.simulator.sql:
Unknown column 'sta
Github user rhtyd commented on the issue:
https://github.com/apache/cloudstack/pull/1624
Until #1654 is fixed and merged, all 4.8 will fail due to db upgrade
failures. /cc @jburwell
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHu
Github user borisstoyanov commented on the issue:
https://github.com/apache/cloudstack/pull/1660
@jburwell I don't have a VMware setup, maybe when we got Trillian back I
could have a look.
---
If your project is set up for it, you can reply to this email and have your
reply appear o
Github user koushik-das commented on a diff in the pull request:
https://github.com/apache/cloudstack/pull/1618#discussion_r76588109
--- Diff: api/src/org/apache/cloudstack/api/response/SnapshotResponse.java
---
@@ -199,4 +207,20 @@ public boolean isRevertable() {
public v
Github user koushik-das commented on the issue:
https://github.com/apache/cloudstack/pull/1606
@leprechau You may add an unit test as well for
NetUtils.validateGuestCidr(). Check NetUtilsTest.java, there is already an
existing test for 192.168.
---
If your project is set up for it,
Github user koushik-das commented on the issue:
https://github.com/apache/cloudstack/pull/1618
Please add/modify Marvin tests based on these scenarios.
---
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
Github user koushik-das commented on a diff in the pull request:
https://github.com/apache/cloudstack/pull/1618#discussion_r76588305
--- Diff: api/src/org/apache/cloudstack/api/response/SnapshotResponse.java
---
@@ -102,6 +102,14 @@
@Param(description = "indicates whether
Github user koushik-das commented on a diff in the pull request:
https://github.com/apache/cloudstack/pull/1622#discussion_r76588913
--- Diff: api/src/org/apache/cloudstack/api/response/TemplateResponse.java
---
@@ -169,6 +169,10 @@
@Param(description = "additional key/val
Github user koushik-das commented on a diff in the pull request:
https://github.com/apache/cloudstack/pull/1622#discussion_r76589032
--- Diff: api/src/org/apache/cloudstack/api/response/TemplateResponse.java
---
@@ -347,4 +351,11 @@ public String getZoneId() {
return z
Github user koushik-das commented on the issue:
https://github.com/apache/cloudstack/pull/1586
code change LGTM
---
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
Github user borisstoyanov commented on the issue:
https://github.com/apache/cloudstack/pull/1621
LGTM
---
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
Github user serg38 commented on the issue:
https://github.com/apache/cloudstack/pull/1660
@jburwell With this PR test_network_acl is successfull
[root@ussarlabcsmgt41 Aug_23_2016_16_49_32_8HYYKG]# tail results.txt
Test network ACL lists and items in VPC ... === TestName: test_n
Github user bvbharatk commented on the issue:
https://github.com/apache/cloudstack/pull/1511
### ACS CI BVT Run
**Sumarry:**
Build Number 79
Hypervisor xenserver
NetworkType Advanced
Passed=100
Failed=1
Skipped=3
_Link to logs Folder (search b
We have done more investigations. An error happens when the script
/usr/share/cloudstack-common/scripts/network/domr/router_proxy.sh is
executed:
MGMT LOGS:
2016-08-29 16:00:11,342 DEBUG [c.c.a.t.Request]
(AgentManager-Handler-12:null) (logid:) Seq 8-1837187172990452475:
Processing: { Ans: , Mgm
Github user nvazquez commented on the issue:
https://github.com/apache/cloudstack/pull/1660
@jburwell Commit message amended. Is it ok like this? Yes, it should be ran
against VMware.
We also made some individual tests for `SshHelper.sshExecute` by adding a
test API method w
Github user sateesh-chodapuneedi commented on the issue:
https://github.com/apache/cloudstack/pull/1664
@jburwell I will check with @nitt10prashant on this. Will try to add the
tests in PR #1417 to this PR.
---
If your project is set up for it, you can reply to this email and have yo
Github user bvbharatk commented on the issue:
https://github.com/apache/cloudstack/pull/1511
### ACS CI BVT Run
**Sumarry:**
Build Number 81
Hypervisor xenserver
NetworkType Advanced
Passed=100
Failed=1
Skipped=3
_Link to logs Folder (search b
Hi All,
The Nuage Networks team is investing in ACS 4.10.0 and has several feature
PR's outstanding for review :
- https://github.com/apache/cloudstack/pull/1578
- https://github.com/apache/cloudstack/pull/1580
- https://github.com/apache/cloudstack/pull/1579
- https://github.com/a
Github user serg38 commented on the issue:
https://github.com/apache/cloudstack/pull/1660
@rhtyd @jburwell Can this be merged?
---
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
ena
Github user serg38 commented on the issue:
https://github.com/apache/cloudstack/pull/1593
@rhtyd @jburwell Can this be merged?
---
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
ena
Github user serg38 commented on the issue:
https://github.com/apache/cloudstack/pull/1560
@rhtyd @jburwell Can this be merged?
---
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
ena
Github user bvbharatk commented on the issue:
https://github.com/apache/cloudstack/pull/1511
### ACS CI BVT Run
**Sumarry:**
Build Number 82
Hypervisor xenserver
NetworkType Advanced
Passed=100
Failed=1
Skipped=3
_Link to logs Folder (search b
Github user rhtyd commented on the issue:
https://github.com/apache/cloudstack/pull/1560
LGTM (code/logic seems alright, did not run any tests).
---
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 hav
Github user rhtyd commented on a diff in the pull request:
https://github.com/apache/cloudstack/pull/1593#discussion_r76730939
--- Diff: debian/cloudstack-usage.postinst ---
@@ -43,6 +43,10 @@ case "$1" in
rm -rf /etc/cloudstack/usage/key
ln -s /etc
Github user rhtyd commented on a diff in the pull request:
https://github.com/apache/cloudstack/pull/1593#discussion_r76730973
--- Diff: usage/conf/log4j-cloud_usage.xml.in ---
@@ -48,7 +48,7 @@ under the License.
-
+
--- E
Github user rhtyd commented on a diff in the pull request:
https://github.com/apache/cloudstack/pull/1593#discussion_r76731031
--- Diff: usage/src/com/cloud/usage/UsageSanityChecker.java ---
@@ -43,9 +43,9 @@
protected static final int DEFAULT_AGGREGATION_RANGE = 1440;
Github user rhtyd commented on a diff in the pull request:
https://github.com/apache/cloudstack/pull/1660#discussion_r76731318
--- Diff: utils/src/main/java/com/cloud/utils/ssh/SshHelper.java ---
@@ -196,6 +198,10 @@ public static void scpTo(String host, int port, String
user, File
Github user rhtyd commented on the issue:
https://github.com/apache/cloudstack/pull/1660
Minor improvements suggested, LGTM.
---
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
enabl
If you could get 2 LGTMs(any one from your team can also review
and give LGTM) and if you can run BVT suite on
PR(https://github.com/apache/cloudstack/tree/master/test/integration/smoke),
they can be immediately merged.
~ Rajani
http://cloudplatform.accelerite.com/
On August 30, 2016 at 3:47 AM,
Github user jburwell commented on the issue:
https://github.com/apache/cloudstack/pull/1654
@rhtyd I have updated this patch to properly support four position version
numbers, as well as, remove the requirement that all versions have a
``DbUpgrade`` class defined and/or an upgrade pat
48 matches
Mail list logo
