Github user rhtyd commented on a diff in the pull request:
https://github.com/apache/cloudstack/pull/1799#discussion_r90397919
--- Diff:
utils/src/main/java/com/cloud/utils/security/CertificateHelper.java ---
@@ -40,123 +46,122 @@
import java.util.ArrayList;
import java.u
Github user rhtyd commented on a diff in the pull request:
https://github.com/apache/cloudstack/pull/1799#discussion_r90397788
--- Diff:
utils/src/main/java/com/cloud/utils/security/CertificateHelper.java ---
@@ -40,123 +46,122 @@
import java.util.ArrayList;
import java.u
Github user rhtyd commented on a diff in the pull request:
https://github.com/apache/cloudstack/pull/1799#discussion_r90397502
--- Diff:
utils/src/main/java/com/cloud/utils/security/CertificateHelper.java ---
@@ -40,123 +46,122 @@
import java.util.ArrayList;
import java.u
Github user blueorangutan commented on the issue:
https://github.com/apache/cloudstack/pull/1800
@rhtyd a Jenkins job has been kicked to build packages. I'll keep you
posted as I make progress.
---
If your project is set up for it, you can reply to this email and have your
reply appe
Github user rhtyd commented on the issue:
https://github.com/apache/cloudstack/pull/828
@sureshanaparti @sateesh-chodapuneedi this looks like a useful patch, can
you change the base branch of your PR to 4.9?
---
If your project is set up for it, you can reply to this email and have y
Github user rhtyd commented on the issue:
https://github.com/apache/cloudstack/pull/1800
@blueorangutan package
---
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 rhtyd commented on the issue:
https://github.com/apache/cloudstack/pull/873
@blueorangutan package
---
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 blueorangutan commented on the issue:
https://github.com/apache/cloudstack/pull/873
@rhtyd a Jenkins job has been kicked to build packages. I'll keep you
posted as I make progress.
---
If your project is set up for it, you can reply to this email and have your
reply appea
GitHub user nitin-maharana opened a pull request:
https://github.com/apache/cloudstack/pull/1804
CLOUDSTACK-9639: Unable to create shared network with vLan isolation
Description:
=
Create shared network fails with Error.
While creating a shared network it fails to
Github user rhtyd commented on the issue:
https://github.com/apache/cloudstack/pull/1794
@karuturi ping
---
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
Github user sateesh-chodapuneedi commented on the issue:
https://github.com/apache/cloudstack/pull/828
LGTM. Verified manually in my VMware setup.
---
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 h
Github user rhtyd commented on a diff in the pull request:
https://github.com/apache/cloudstack/pull/1579#discussion_r90394494
--- Diff: api/src/com/cloud/network/Network.java ---
@@ -215,6 +216,8 @@ public static Provider getProvider(String providerName)
{
public stat
Github user blueorangutan commented on the issue:
https://github.com/apache/cloudstack/pull/828
@rhtyd a Jenkins job has been kicked to build packages. I'll keep you
posted as I make progress.
---
If your project is set up for it, you can reply to this email and have your
reply appea
Github user rhtyd commented on the issue:
https://github.com/apache/cloudstack/pull/828
LGTM.
@blueorangutan package
---
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 a
Github user rhtyd commented on the issue:
https://github.com/apache/cloudstack/pull/1803
LGTM, @nitin-maharana can you rebase and change PR's base branch to 4.9.
This is a small and useful change.
---
If your project is set up for it, you can reply to this email and have your
reply a
Github user blueorangutan commented on the issue:
https://github.com/apache/cloudstack/pull/1802
@rhtyd a Trillian-Jenkins test job (centos7 mgmt + vmware-55u3) has been
kicked to run smoke tests
---
If your project is set up for it, you can reply to this email and have your
reply ap
Github user rhtyd commented on the issue:
https://github.com/apache/cloudstack/pull/1802
LGTM, let's wait for the result.
@blueorangutan test centos7 vmware-55u3
---
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
Github user rhtyd commented on the issue:
https://github.com/apache/cloudstack/pull/1579
I'm seeing no new errors except for `test_06_download_detached_volume` and
`test_01_create_template`. @prashanthvarma can you have a look, they may be
intermittent in nature.
@jburwell ca
Github user asfgit closed the pull request at:
https://github.com/apache/cloudstack/pull/1801
---
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 commented on the issue:
https://github.com/apache/cloudstack/pull/1800
@borisstoyanov can you have a look at failure in
`test_01_snapshot_root_disk`?
@jburwell Jenkins is not reliable, with my latest fix Travis will do
noredist build checks and RAT-license checks
Github user rhtyd commented on the issue:
https://github.com/apache/cloudstack/pull/1801
LGTM. Thanks @murali-reddy I'll merge this.
---
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 featu
Github user blueorangutan commented on the issue:
https://github.com/apache/cloudstack/pull/1802
@murali-reddy a Trillian-Jenkins test job (centos7 mgmt + kvm-centos7) has
been kicked to run smoke tests
---
If your project is set up for it, you can reply to this email and have your
r
Github user murali-reddy commented on the issue:
https://github.com/apache/cloudstack/pull/1802
@blueorangutan test
---
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 wi
Github user blueorangutan commented on the issue:
https://github.com/apache/cloudstack/pull/1802
Packaging result: âcentos6 âcentos7 âdebian. JID-291
---
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 d
Github user jburwell commented on the issue:
https://github.com/apache/cloudstack/pull/1579
@prashanthvarma agreed regarding the most common causes of failures.
@murali-reddy @borisstoyanov and @abhinandanprateek have been working to
address these issues as they are encountered. To
Github user jburwell commented on the issue:
https://github.com/apache/cloudstack/pull/1435
@PaulAngus @rhtyd what are your thoughts on automating the testing of the
Docker container?
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitH
Github user nitin-maharana commented on the issue:
https://github.com/apache/cloudstack/pull/1803
---
If your project is set up for it, you can reply to this email and hav
Github user jburwell commented on the issue:
https://github.com/apache/cloudstack/pull/1435
@pdion891 can you please investigate the Jenkins failure?
---
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 no
Github user jburwell commented on the issue:
https://github.com/apache/cloudstack/pull/1435
@pdion891 if possible, I would like to get this merged for 4.9.1.0. How
does this PR relate to #1789? Also, could you please investigate the Jenkins
failure and create JIRA ticket for this is
Github user jburwell commented on the issue:
https://github.com/apache/cloudstack/pull/1784
@murali-reddy is internal DNS service impacted by having multiple NICs?
---
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 p
Github user blueorangutan commented on the issue:
https://github.com/apache/cloudstack/pull/1802
@murali-reddy a Jenkins job has been kicked to build packages. I'll keep
you posted as I make progress.
---
If your project is set up for it, you can reply to this email and have your
rep
Github user jburwell commented on a diff in the pull request:
https://github.com/apache/cloudstack/pull/1784#discussion_r90380293
--- Diff: test/integration/smoke/test_router_dnsservice.py ---
@@ -0,0 +1,268 @@
+# Licensed to the Apache Software Foundation (ASF) under one
+#
Github user jburwell commented on a diff in the pull request:
https://github.com/apache/cloudstack/pull/1784#discussion_r90380926
--- Diff: test/integration/smoke/test_router_dnsservice.py ---
@@ -0,0 +1,268 @@
+# Licensed to the Apache Software Foundation (ASF) under one
+#
Github user jburwell commented on a diff in the pull request:
https://github.com/apache/cloudstack/pull/1784#discussion_r90380579
--- Diff: test/integration/smoke/test_router_dnsservice.py ---
@@ -0,0 +1,268 @@
+# Licensed to the Apache Software Foundation (ASF) under one
+#
Github user jburwell commented on a diff in the pull request:
https://github.com/apache/cloudstack/pull/1784#discussion_r90380262
--- Diff: test/integration/smoke/test_router_dnsservice.py ---
@@ -0,0 +1,268 @@
+# Licensed to the Apache Software Foundation (ASF) under one
+#
Github user jburwell commented on a diff in the pull request:
https://github.com/apache/cloudstack/pull/1784#discussion_r90380120
--- Diff: test/integration/smoke/test_router_dnsservice.py ---
@@ -0,0 +1,268 @@
+# Licensed to the Apache Software Foundation (ASF) under one
+#
Github user jburwell commented on a diff in the pull request:
https://github.com/apache/cloudstack/pull/1784#discussion_r90380041
--- Diff: test/integration/smoke/test_router_dnsservice.py ---
@@ -0,0 +1,268 @@
+# Licensed to the Apache Software Foundation (ASF) under one
+#
Github user murali-reddy commented on the issue:
https://github.com/apache/cloudstack/pull/1802
@blueorangutan package
---
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
Github user murali-reddy commented on the issue:
https://github.com/apache/cloudstack/pull/1801
test_router_dhcp_opts is passing, and we have necessary LGTM's. can we
merge this?
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as
Github user blueorangutan commented on the issue:
https://github.com/apache/cloudstack/pull/1754
Trillian test result (tid-514)
Environment: xenserver-65sp1 (x2), Advanced Networking with Mgmt server 6
Total time taken: 35803 seconds
Marvin logs:
https://github.com/blueoran
Github user jburwell commented on the issue:
https://github.com/apache/cloudstack/pull/1798
@marcaurele there appear to be Travis failures. Could you please check
into them?
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as wel
Github user jburwell commented on a diff in the pull request:
https://github.com/apache/cloudstack/pull/1798#discussion_r90372709
--- Diff:
api/src/org/apache/cloudstack/api/command/user/affinitygroup/UpdateVMAffinityGroupCmd.java
---
@@ -96,6 +96,10 @@ public Long getId() {
Github user priyankparihar closed the pull request at:
https://github.com/apache/cloudstack/pull/1777
---
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 fe
Github user pdion891 commented on the issue:
https://github.com/apache/cloudstack/pull/1435
Hi @jburwell, do you think we can merge this in master or you are in a code
freeze for 4.10.0.0 ?
Thanks,
---
If your project is set up for it, you can reply to this email and have yo
Github user pdion891 commented on the issue:
https://github.com/apache/cloudstack/pull/1435
@resmo I've updated the PR. I've tested the simulator and marvin.
---
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 proje
Github user serg38 commented on the issue:
https://github.com/apache/cloudstack/pull/828
LGTM. This patch has been working well in production for over a year now.
---
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 pr
Github user sureshanaparti commented on the issue:
https://github.com/apache/cloudstack/pull/828
@rhtyd Rebased against lastest master
---
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 fea
Github user prashanthvarma commented on the issue:
https://github.com/apache/cloudstack/pull/1579
@jburwell Thank you for the update, I was planning to investigate the same
test failure (test_router_dhcp_opts) as unlike other test failures it looked
like an actual reproducible issue f
Github user jburwell commented on the issue:
https://github.com/apache/cloudstack/pull/1711
@syed yes, you will add the changes to `schema-4910to4920` for the `4.9`
port. This will cover anyone upgrading from a version >= 4.9.1.0 to 4.9.2.0.
I do not plan to automatically forward me
Github user jburwell commented on a diff in the pull request:
https://github.com/apache/cloudstack/pull/1799#discussion_r90342302
--- Diff: server/src/org/apache/cloudstack/network/lb/CertServiceImpl.java
---
@@ -339,190 +348,182 @@ public SslCertResponse createCertResponse(SslCert
Github user jburwell commented on a diff in the pull request:
https://github.com/apache/cloudstack/pull/1799#discussion_r90342459
--- Diff: server/src/org/apache/cloudstack/network/lb/CertServiceImpl.java
---
@@ -339,190 +348,182 @@ public SslCertResponse createCertResponse(SslCert
Github user jburwell commented on the issue:
https://github.com/apache/cloudstack/pull/1803
@nitin-maharana could you please add screen shots of the change to the PR?
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If you
Github user jburwell commented on a diff in the pull request:
https://github.com/apache/cloudstack/pull/1799#discussion_r90341780
--- Diff: server/src/org/apache/cloudstack/network/lb/CertServiceImpl.java
---
@@ -339,190 +348,182 @@ public SslCertResponse createCertResponse(SslCert
Github user jburwell commented on a diff in the pull request:
https://github.com/apache/cloudstack/pull/1799#discussion_r90340642
--- Diff: server/src/org/apache/cloudstack/network/lb/CertServiceImpl.java
---
@@ -339,190 +348,182 @@ public SslCertResponse createCertResponse(SslCert
Github user jburwell commented on a diff in the pull request:
https://github.com/apache/cloudstack/pull/1799#discussion_r90341385
--- Diff: server/src/org/apache/cloudstack/network/lb/CertServiceImpl.java
---
@@ -339,190 +348,182 @@ public SslCertResponse createCertResponse(SslCert
Github user jburwell commented on a diff in the pull request:
https://github.com/apache/cloudstack/pull/1799#discussion_r90341157
--- Diff: server/src/org/apache/cloudstack/network/lb/CertServiceImpl.java
---
@@ -339,190 +348,182 @@ public SslCertResponse createCertResponse(SslCert
Github user jburwell commented on a diff in the pull request:
https://github.com/apache/cloudstack/pull/1799#discussion_r90341463
--- Diff: server/src/org/apache/cloudstack/network/lb/CertServiceImpl.java
---
@@ -339,190 +348,182 @@ public SslCertResponse createCertResponse(SslCert
Github user jburwell commented on a diff in the pull request:
https://github.com/apache/cloudstack/pull/1799#discussion_r90341553
--- Diff: server/src/org/apache/cloudstack/network/lb/CertServiceImpl.java
---
@@ -339,190 +348,182 @@ public SslCertResponse createCertResponse(SslCert
Github user jburwell commented on a diff in the pull request:
https://github.com/apache/cloudstack/pull/1799#discussion_r90340513
--- Diff: server/src/org/apache/cloudstack/network/lb/CertServiceImpl.java
---
@@ -339,190 +348,182 @@ public SslCertResponse createCertResponse(SslCert
Github user jburwell commented on a diff in the pull request:
https://github.com/apache/cloudstack/pull/1799#discussion_r90341649
--- Diff: server/src/org/apache/cloudstack/network/lb/CertServiceImpl.java
---
@@ -339,190 +348,182 @@ public SslCertResponse createCertResponse(SslCert
Github user jburwell commented on a diff in the pull request:
https://github.com/apache/cloudstack/pull/1799#discussion_r90339611
--- Diff: server/src/org/apache/cloudstack/network/lb/CertServiceImpl.java
---
@@ -339,190 +348,182 @@ public SslCertResponse createCertResponse(SslCert
Github user jburwell commented on a diff in the pull request:
https://github.com/apache/cloudstack/pull/1799#discussion_r90339269
--- Diff: server/src/org/apache/cloudstack/network/lb/CertServiceImpl.java
---
@@ -339,190 +348,182 @@ public SslCertResponse createCertResponse(SslCert
Github user jburwell commented on a diff in the pull request:
https://github.com/apache/cloudstack/pull/1799#discussion_r90315044
--- Diff: server/src/org/apache/cloudstack/network/lb/CertServiceImpl.java
---
@@ -111,37 +116,37 @@ public CertServiceImpl() {
@DB
@O
Github user jburwell commented on a diff in the pull request:
https://github.com/apache/cloudstack/pull/1799#discussion_r90310482
--- Diff:
utils/src/main/java/com/cloud/utils/security/CertificateHelper.java ---
@@ -40,123 +46,122 @@
import java.util.ArrayList;
import jav
Github user jburwell commented on a diff in the pull request:
https://github.com/apache/cloudstack/pull/1799#discussion_r90310924
--- Diff:
utils/src/main/java/com/cloud/utils/security/CertificateHelper.java ---
@@ -40,123 +46,122 @@
import java.util.ArrayList;
import jav
Github user jburwell commented on a diff in the pull request:
https://github.com/apache/cloudstack/pull/1799#discussion_r90339078
--- Diff: server/src/org/apache/cloudstack/network/lb/CertServiceImpl.java
---
@@ -339,190 +348,182 @@ public SslCertResponse createCertResponse(SslCert
Github user jburwell commented on a diff in the pull request:
https://github.com/apache/cloudstack/pull/1799#discussion_r90315713
--- Diff: server/src/org/apache/cloudstack/network/lb/CertServiceImpl.java
---
@@ -180,16 +185,16 @@ public void deleteSslCert(DeleteSslCertCmd
deleteS
Github user jburwell commented on a diff in the pull request:
https://github.com/apache/cloudstack/pull/1799#discussion_r90338425
--- Diff: server/src/org/apache/cloudstack/network/lb/CertServiceImpl.java
---
@@ -240,68 +245,71 @@ public void deleteSslCert(DeleteSslCertCmd
deleteS
Github user jburwell commented on a diff in the pull request:
https://github.com/apache/cloudstack/pull/1799#discussion_r90314823
--- Diff: server/test/org/apache/cloudstack/network/lb/CertServiceTest.java
---
@@ -125,48 +125,48 @@ public void runUploadSslCertWithCAChain() throws
Github user jburwell commented on a diff in the pull request:
https://github.com/apache/cloudstack/pull/1799#discussion_r90339460
--- Diff: server/src/org/apache/cloudstack/network/lb/CertServiceImpl.java
---
@@ -339,190 +348,182 @@ public SslCertResponse createCertResponse(SslCert
Github user jburwell commented on a diff in the pull request:
https://github.com/apache/cloudstack/pull/1799#discussion_r90313790
--- Diff: server/test/org/apache/cloudstack/network/lb/CertServiceTest.java
---
@@ -245,48 +245,48 @@ public void runUploadSslCertSelfSignedNoPassword()
Github user jburwell commented on a diff in the pull request:
https://github.com/apache/cloudstack/pull/1799#discussion_r90315176
--- Diff: server/src/org/apache/cloudstack/network/lb/CertServiceImpl.java
---
@@ -111,37 +116,37 @@ public CertServiceImpl() {
@DB
@O
Github user jburwell commented on a diff in the pull request:
https://github.com/apache/cloudstack/pull/1799#discussion_r90309869
--- Diff:
utils/src/main/java/com/cloud/utils/security/CertificateHelper.java ---
@@ -40,123 +46,122 @@
import java.util.ArrayList;
import jav
Github user jburwell commented on a diff in the pull request:
https://github.com/apache/cloudstack/pull/1799#discussion_r90315448
--- Diff: server/src/org/apache/cloudstack/network/lb/CertServiceImpl.java
---
@@ -150,26 +155,26 @@ public SslCertResponse uploadSslCert(UploadSslCertC
Github user jburwell commented on a diff in the pull request:
https://github.com/apache/cloudstack/pull/1799#discussion_r90312936
--- Diff:
utils/src/main/java/com/cloud/utils/security/CertificateHelper.java ---
@@ -40,123 +46,122 @@
import java.util.ArrayList;
import jav
Github user jburwell commented on a diff in the pull request:
https://github.com/apache/cloudstack/pull/1799#discussion_r90340246
--- Diff: server/src/org/apache/cloudstack/network/lb/CertServiceImpl.java
---
@@ -339,190 +348,182 @@ public SslCertResponse createCertResponse(SslCert
Github user jburwell commented on a diff in the pull request:
https://github.com/apache/cloudstack/pull/1799#discussion_r90312667
--- Diff:
utils/src/main/java/com/cloud/utils/security/CertificateHelper.java ---
@@ -40,123 +46,122 @@
import java.util.ArrayList;
import jav
Github user jburwell commented on a diff in the pull request:
https://github.com/apache/cloudstack/pull/1799#discussion_r90338915
--- Diff: server/src/org/apache/cloudstack/network/lb/CertServiceImpl.java
---
@@ -339,190 +348,182 @@ public SslCertResponse createCertResponse(SslCert
Github user jburwell commented on a diff in the pull request:
https://github.com/apache/cloudstack/pull/1799#discussion_r90315376
--- Diff: server/src/org/apache/cloudstack/network/lb/CertServiceImpl.java
---
@@ -111,37 +116,37 @@ public CertServiceImpl() {
@DB
@O
Github user jburwell commented on a diff in the pull request:
https://github.com/apache/cloudstack/pull/1799#discussion_r90338636
--- Diff: server/src/org/apache/cloudstack/network/lb/CertServiceImpl.java
---
@@ -240,68 +245,71 @@ public void deleteSslCert(DeleteSslCertCmd
deleteS
Github user jburwell commented on a diff in the pull request:
https://github.com/apache/cloudstack/pull/1799#discussion_r90314765
--- Diff: server/test/org/apache/cloudstack/network/lb/CertServiceTest.java
---
@@ -176,18 +176,18 @@ public void runUploadSslCertSelfSignedWithPassword
Github user jburwell commented on a diff in the pull request:
https://github.com/apache/cloudstack/pull/1799#discussion_r90314781
--- Diff: server/test/org/apache/cloudstack/network/lb/CertServiceTest.java
---
@@ -226,14 +226,14 @@ public void runUploadSslCertSelfSignedNoPassword()
Github user jburwell commented on a diff in the pull request:
https://github.com/apache/cloudstack/pull/1799#discussion_r90310643
--- Diff:
utils/src/main/java/com/cloud/utils/security/CertificateHelper.java ---
@@ -40,123 +46,122 @@
import java.util.ArrayList;
import jav
Github user jburwell commented on a diff in the pull request:
https://github.com/apache/cloudstack/pull/1799#discussion_r90312334
--- Diff:
utils/src/main/java/com/cloud/utils/security/CertificateHelper.java ---
@@ -40,123 +46,122 @@
import java.util.ArrayList;
import jav
Github user jburwell commented on a diff in the pull request:
https://github.com/apache/cloudstack/pull/1799#discussion_r90311210
--- Diff:
utils/src/main/java/com/cloud/utils/security/CertificateHelper.java ---
@@ -40,123 +46,122 @@
import java.util.ArrayList;
import jav
Github user jburwell commented on a diff in the pull request:
https://github.com/apache/cloudstack/pull/1799#discussion_r90310775
--- Diff:
utils/src/main/java/com/cloud/utils/security/CertificateHelper.java ---
@@ -40,123 +46,122 @@
import java.util.ArrayList;
import jav
Github user jburwell commented on a diff in the pull request:
https://github.com/apache/cloudstack/pull/1799#discussion_r90310270
--- Diff:
utils/src/main/java/com/cloud/utils/security/CertificateHelper.java ---
@@ -40,123 +46,122 @@
import java.util.ArrayList;
import jav
Github user sureshanaparti commented on the issue:
https://github.com/apache/cloudstack/pull/1253
Addressed all the changes suggested and rebased. With vSphere 4.0/4.1, when
the ports per dvportgroup is set to less than 256 (current default), then it is
set to the default 256, else it
Github user blueorangutan commented on the issue:
https://github.com/apache/cloudstack/pull/1800
Trillian test result (tid-509)
Environment: xenserver-65sp1 (x2), Advanced Networking with Mgmt server 7
Total time taken: 33518 seconds
Marvin logs:
https://github.com/blueoran
Github user blueorangutan commented on the issue:
https://github.com/apache/cloudstack/pull/1801
Trillian test result (tid-510)
Environment: vmware-55u3 (x2), Advanced Networking with Mgmt server 7
Total time taken: 33387 seconds
Marvin logs:
https://github.com/blueoranguta
Github user pdion891 commented on the issue:
https://github.com/apache/cloudstack/pull/1769
ok, I'll update the PR and merge it I think the PR go all required LGTM.
I'll confirmed with John, in case he is in a code freeze period.
---
If your project is set up for it, you can reply to
Github user resmo commented on the issue:
https://github.com/apache/cloudstack/pull/1769
@pdion891 I would say merge #1435 first
---
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
e
Github user pdion891 commented on the issue:
https://github.com/apache/cloudstack/pull/1769
@resmo could you do a PR on top of #1435 ? or this have to be merged
first? so we can get your chances and some of mine.
---
If your project is set up for it, you can reply to this email and
Github user pdion891 commented on the issue:
https://github.com/apache/cloudstack/pull/1769
So, should we go forward and merge #1435 + #1769 to master?
I might just need to rebase and update version number of the current commit.
With those 2 PR, we would have master back o
Github user syed commented on the issue:
https://github.com/apache/cloudstack/pull/1711
Thanks Guys for testing this out. I had a question @jburwell You wanted me
to port this to 4.9 and then it would be forward merged to 4.10 and master. How
would the DB changes be handled? right now
GitHub user nitin-maharana opened a pull request:
https://github.com/apache/cloudstack/pull/1803
CLOUDSTACK-9636: The host alerts box should be named as hosts in Alerts.
The host Alerts box shows hosts in Alerts. The name host Alerts is
misleading,
it should be changed to hosts
Github user jburwell commented on the issue:
https://github.com/apache/cloudstack/pull/1579
@prashanthvarma the redundant router/VPC failures are known. We have a fix
pending for the `test_snapshot` failures in XenServer (#1800). However, I am
concerned about that`test_router_dhcp_o
Github user jburwell commented on the issue:
https://github.com/apache/cloudstack/pull/1800
@borisstoyanov there is a Jenkins failure, but the results page returns a
404. Could you please re-push this PR to re-trigger the Jenkins/Travis run by
running `git commit --amend --no-edit &&
Github user jburwell commented on the issue:
https://github.com/apache/cloudstack/pull/1579
@prashanthvarma please ignore my last comment. It appears that we have
found an issue with the `test_router_dhcp_opts` test case. #1801 has been
submitted to fix it. Once #1800 and #1801 hav
Github user blueorangutan commented on the issue:
https://github.com/apache/cloudstack/pull/1579
Trillian test result (tid-507)
Environment: vmware-55u3 (x2), Advanced Networking with Mgmt server 7
Total time taken: 40678 seconds
Marvin logs:
https://github.com/blueoranguta
1 - 100 of 168 matches
Mail list logo
