Github user rhtyd commented on the issue:
https://github.com/apache/cloudstack/pull/1762
@abhinandanprateek can you help reviewing this one, thanks.
---
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
Github user rhtyd commented on the issue:
https://github.com/apache/cloudstack/pull/1769
LGTM. @pdion891 would you like to review?
---
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
Github user rhtyd commented on the issue:
https://github.com/apache/cloudstack/pull/1757
LGTM, merging this based on code review and test results.
---
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 the issue:
https://github.com/apache/cloudstack/pull/1745
LGTM, merging this based on last test run and code reviews.
---
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
Github user asfgit closed the pull request at:
https://github.com/apache/cloudstack/pull/1757
---
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/1711
Good progress :smiley:
---
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 wishe
Github user rhtyd commented on the issue:
https://github.com/apache/cloudstack/pull/1554
@marcaurele if I squash while merging, the git sha of the commit will
change and the PR won't be marked as 'merged'. Which is why we use the `git-pr`
tool that uses a merge strategy to merge the P
Github user asfgit closed the pull request at:
https://github.com/apache/cloudstack/pull/1745
---
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/1765
LGTM, last test run looks okay. Failed tests around vpc/rvr are known
intermittent issues.
@abhinandanprateek can you squash your commits, thanks.
---
If your project is set up for it, you ca
Will, this issue did happen for me in the remote access vpn of vpc with
redundant VRs.
Andrei, if you confirm it works, I will create a PR.
2016-11-22 20:58 GMT+01:00 Andrei Mikhailovsky :
> Dag from the users mailing list has pointed to this:
> https://issues.apache.org/jira/browse/CLOUDSTACK-9
Github user rhtyd commented on the issue:
https://github.com/apache/cloudstack/pull/1766
@murali-reddy can you check why `test_router_dhcp_opts` failed?
@blueorangutan package
---
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/1766
@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/1681
LGTM, failures related to vpc/rvr are known intermittent failures. We'll
kick another test round on #1753, #1754 etc. and work towards fixing them for
vmware and xenserver separately. Merging thi
Github user asfgit closed the pull request at:
https://github.com/apache/cloudstack/pull/1681
---
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/1680
LGTM. Merging this based on review and tests.
Failures in vpc/rvr are known intermittent issues and will be addressed
separately for xen/vmware.
---
If your project is set up for it, you can
Github user rhtyd commented on the issue:
https://github.com/apache/cloudstack/pull/1659
@murali-reddy can you check why we've test failures for KVM? Also, if the
integration tests don't take a lot of time (few minutes to half hour) can you
move it to smoke tests?
---
If your projec
Github user asfgit closed the pull request at:
https://github.com/apache/cloudstack/pull/1680
---
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/1597
@vcharmcaster can you fix the space/tab issue.
---
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 rhtyd commented on the issue:
https://github.com/apache/cloudstack/pull/1735
@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/1735
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 the fe
Github user rhtyd commented on the issue:
https://github.com/apache/cloudstack/pull/1710
Do we have LGTMs on 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 feature
enabled and wis
Github user rhtyd commented on the issue:
https://github.com/apache/cloudstack/pull/1622
Thanks @ProjectMoon
@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 f
Github user rhtyd commented on the issue:
https://github.com/apache/cloudstack/pull/1622
@ProjectMoon can you add a JIRA id for this, also in the commit's summary
---
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 blueorangutan commented on the issue:
https://github.com/apache/cloudstack/pull/1622
@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/1623
@ProjectMoon thanks
@murali-reddy can you help review this?
@blueorangutan package
---
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/1623
@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/1566
LGTM. @ProjectMoon thanks, can you add a JIRA id for this, and also in the
commit's summary
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub
Github user rhtyd commented on a diff in the pull request:
https://github.com/apache/cloudstack/pull/1580#discussion_r89277508
--- Diff:
plugins/network-elements/nuage-vsp/src/com/cloud/api/response/NuageVlanIpRangeResponse.java
---
@@ -0,0 +1,38 @@
+//
+// Licensed to the
Github user rhtyd commented on a diff in the pull request:
https://github.com/apache/cloudstack/pull/1580#discussion_r89277328
--- Diff:
plugins/network-elements/nuage-vsp/src/com/cloud/api/commands/DisableNuageUnderlayVlanIpRangeCmd.java
---
@@ -0,0 +1,110 @@
+//
+// Lice
Github user rhtyd commented on a diff in the pull request:
https://github.com/apache/cloudstack/pull/1580#discussion_r89277386
--- Diff:
plugins/network-elements/nuage-vsp/src/com/cloud/api/commands/EnableNuageUnderlayVlanIpRangeCmd.java
---
@@ -0,0 +1,110 @@
+//
+// Licen
Github user rhtyd commented on a diff in the pull request:
https://github.com/apache/cloudstack/pull/1580#discussion_r89277160
--- Diff: client/tomcatconf/commands.properties.in ---
@@ -0,0 +1,804 @@
+# Licensed to the Apache Software Foundation (ASF) under one
--- End diff
Github user rhtyd commented on a diff in the pull request:
https://github.com/apache/cloudstack/pull/1580#discussion_r89277419
--- Diff:
plugins/network-elements/nuage-vsp/src/com/cloud/api/commands/ListNuageUnderlayVlanIpRangesCmd.java
---
@@ -0,0 +1,80 @@
+//
+// License
Github user rhtyd commented on the issue:
https://github.com/apache/cloudstack/pull/1604
@romain-dartigues I'll have a look at the failure, meanwhile can you see my
comments against the changes
https://github.com/apache/cloudstack/pull/1604/files
---
If your project is set up for it
Github user rhtyd commented on the issue:
https://github.com/apache/cloudstack/pull/1618
@ProjectMoon thanks, can you add a JIRA id for this and also use that in
your commit's PR.
@blueorangutan package
---
If your project is set up for it, you can reply to this email and have yo
Github user blueorangutan commented on the issue:
https://github.com/apache/cloudstack/pull/1618
@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/1542
LGTM, merging based on code reviews and tests, ignoring known intermittent
errors in vpc/rvr.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHu
Github user rhtyd commented on the issue:
https://github.com/apache/cloudstack/pull/756
Thanks, merging this 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 project does not have this feature
enabled and wish
Github user asfgit closed the pull request at:
https://github.com/apache/cloudstack/pull/1542
---
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 asfgit closed the pull request at:
https://github.com/apache/cloudstack/pull/756
---
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/1559
Thanks @ProjectMoon can you squash your commits.
@murali-reddy @abhinandanprateek can you help review this?
---
If your project is set up for it, you can reply to this email and have your
repl
Github user blueorangutan commented on the issue:
https://github.com/apache/cloudstack/pull/1766
Packaging result: âcentos6 âcentos7 âdebian. JID-229
---
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 abhinandanprateek commented on the issue:
https://github.com/apache/cloudstack/pull/1765
@rhtyd squashed
---
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 w
Github user rhtyd commented on the issue:
https://github.com/apache/cloudstack/pull/1765
@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/1765
@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 blueorangutan commented on the issue:
https://github.com/apache/cloudstack/pull/1623
Packaging result: âcentos6 âcentos7 âdebian. JID-231
---
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 ProjectMoon commented on the issue:
https://github.com/apache/cloudstack/pull/1559
@rhtyd, in my opinion the two commits are functionally separate enough that
they should remain separate (and this might have been asked for in one of the
various previous incarnations of thi
Github user blueorangutan commented on the issue:
https://github.com/apache/cloudstack/pull/1622
Packaging result: âcentos6 âcentos7 âdebian. JID-230
---
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
Mauricio,
As far as I'm aware, we don't have any tooling to support this but you're right
such a tool would consume response from listApis call.
You may want to write such a tool for your use-case and maybe publish it on
github/pypi/etc.
Regards.
From: Mauri
The idea to have a WSDL or a WADL is interesting. It helps in integrations
between system-system (without human coding/support), and allows more
sophisticated software to consume APIs.
Instead of creating something to generate the API externally, why not add a
plugin into CloudStack that generate t
Github user blueorangutan commented on the issue:
https://github.com/apache/cloudstack/pull/1618
Packaging result: âcentos6 âcentos7 âdebian. JID-232
---
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 blueorangutan commented on the issue:
https://github.com/apache/cloudstack/pull/1765
Packaging result: âcentos6 âcentos7 âdebian. JID-233
---
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 rhtyd commented on the issue:
https://github.com/apache/cloudstack/pull/1752
@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/1753
@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/1753
@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/1754
@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/1752
@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 blueorangutan commented on the issue:
https://github.com/apache/cloudstack/pull/1754
@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 wido commented on the issue:
https://github.com/apache/cloudstack/pull/1545
@jburwell @rhtyd I fixed most of the comments.
The NPE should be fixed and the StringUtils comments as well. I didn't
refactor the other stuff @jburwell mentioned as my code is in-line with
GitHub user nitin-maharana opened a pull request:
https://github.com/apache/cloudstack/pull/1771
CLOUDSTACK-9611: Dedicating a Guest VLAN range to Project does not work.
Description:
=
Trying to dedicate a guest VLAN range to an account fails. Either API
documen
Github user rhtyd commented on the issue:
https://github.com/apache/cloudstack/pull/1545
@wido can you confirm if you've fixed the NPE issue I've shared in the
comment above?
java.lang.NullPointerException
at java.io.File.(File.java:277)
at
com.cloud.hypervisor.kvm.resourc
Github user wido commented on the issue:
https://github.com/apache/cloudstack/pull/1545
Yes @rhtyd, I added a check for NULL there.
---
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 featur
Github user rhtyd commented on the issue:
https://github.com/apache/cloudstack/pull/1545
@wido alright, I'll initiate a final test round -- if that passes we can
consider merging. /cc @jburwell
---
If your project is set up for it, you can reply to this email and have your
reply app
Github user rhtyd commented on the issue:
https://github.com/apache/cloudstack/pull/1545
@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 ozhanrk commented on the issue:
https://github.com/apache/cloudstack/pull/1710
Hi @kiwiflyer @nathanejohnson i saw that you are merged this fix and using
it on your ENA CloudStack branch, when you have time could you check the code
for possible LGTM's please.
---
If your
Github user blueorangutan commented on the issue:
https://github.com/apache/cloudstack/pull/1545
@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 murali-reddy commented on the issue:
https://github.com/apache/cloudstack/pull/1766
@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/1766
@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 blueorangutan commented on the issue:
https://github.com/apache/cloudstack/pull/1753
Packaging result: âcentos6 âcentos7 âdebian. JID-234
---
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 blueorangutan commented on the issue:
https://github.com/apache/cloudstack/pull/1752
Packaging result: âcentos6 âcentos7 âdebian. JID-235
---
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 blueorangutan commented on the issue:
https://github.com/apache/cloudstack/pull/1754
Packaging result: âcentos6 âcentos7 âdebian. JID-236
---
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 blueorangutan commented on the issue:
https://github.com/apache/cloudstack/pull/1545
Packaging result: âcentos6 âcentos7 âdebian. JID-237
---
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 blueorangutan commented on the issue:
https://github.com/apache/cloudstack/pull/1545
@rhtyd 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
reply ap
Github user rhtyd commented on the issue:
https://github.com/apache/cloudstack/pull/1545
@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 wishes so
Github user rhtyd commented on the issue:
https://github.com/apache/cloudstack/pull/1754
@blueorangutan test matrix
---
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/1754
@rhtyd a Trillian-Jenkins matrix job (centos6 mgmt + xs65sp1, centos7 mgmt
+ vmware55u3, centos7 mgmt + kvmcentos7) has been kicked to run smoke tests
---
If your project is set up for it
Github user rhtyd commented on the issue:
https://github.com/apache/cloudstack/pull/1545
@wido please fix the build failures:
[INFO] Starting audit...
/jenkins/workspace/acs-pr-centos7-pkg-builder/dist/rpmbuild/BUILD/cloudstack-4.10.0.0-SNAPSHOT/plugins/hypervisors/kvm/src/com/
Github user prashanthvarma commented on a diff in the pull request:
https://github.com/apache/cloudstack/pull/1580#discussion_r89308445
--- Diff:
plugins/network-elements/nuage-vsp/src/com/cloud/api/response/NuageVlanIpRangeResponse.java
---
@@ -0,0 +1,38 @@
+//
+// Licens
Github user prashanthvarma commented on the issue:
https://github.com/apache/cloudstack/pull/1580
@rhtyd I have resolved and responded to your comments, and updated the PR
accordingly. Moreover, I have just kicked on our internal regression tests
(including the added Marvin tests in t
Github user romain-dartigues commented on a diff in the pull request:
https://github.com/apache/cloudstack/pull/1604#discussion_r89310394
--- Diff: systemvm/patches/debian/config/opt/cloud/bin/cs/CsHelper.py ---
@@ -179,12 +179,12 @@ def get_hostname():
return line.stri
Github user romain-dartigues commented on the issue:
https://github.com/apache/cloudstack/pull/1604
@rhtyd < separate function done.
---
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 rhtyd commented on the issue:
https://github.com/apache/cloudstack/pull/1753
@blueorangutan test matrix
---
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/1752
@rhtyd a Trillian-Jenkins matrix job (centos6 mgmt + xs65sp1, centos7 mgmt
+ vmware55u3, centos7 mgmt + kvmcentos7) has been kicked to run smoke tests
---
If your project is set up for it
Github user rhtyd commented on the issue:
https://github.com/apache/cloudstack/pull/1752
@blueorangutan test matrix
---
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/1753
@rhtyd a Trillian-Jenkins matrix job (centos6 mgmt + xs65sp1, centos7 mgmt
+ vmware55u3, centos7 mgmt + kvmcentos7) has been kicked to run smoke tests
---
If your project is set up for it
Github user rhtyd commented on the issue:
https://github.com/apache/cloudstack/pull/1604
okay @romain-dartigues I'll kick some tests
@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 proje
Github user blueorangutan commented on the issue:
https://github.com/apache/cloudstack/pull/1604
@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 pdion891 commented on the issue:
https://github.com/apache/cloudstack/pull/1684
btw, I don't think this PR should be backport since it would change
behavior for someone that would upgrade from 4.x.x to 4.9.0 or 4.9.x(that would
include this PR)
---
If your project is set
Github user blueorangutan commented on the issue:
https://github.com/apache/cloudstack/pull/1604
Packaging result: âcentos6 âcentos7 âdebian. JID-238
---
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 wido commented on the issue:
https://github.com/apache/cloudstack/pull/1545
@rhtyd Yes, fixed. Forgot to commit that change locally. Pushed a new
version.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If yo
Github user nathanejohnson commented on the issue:
https://github.com/apache/cloudstack/pull/1710
LGTM! We're running it in our labs.
---
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 mike-tutkowski commented on the issue:
https://github.com/apache/cloudstack/pull/1710
This code 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 wish
Github user rhtyd commented on the issue:
https://github.com/apache/cloudstack/pull/1545
@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/1545
@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 ozhanrk commented on the issue:
https://github.com/apache/cloudstack/pull/1710
Hi @rhtyd we have got the required LGTM's 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 project does not have this
Github user jburwell commented on the issue:
https://github.com/apache/cloudstack/pull/1762
@rhtyd I am -1 on this 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 your project does not have this feature
enabled and
Github user blueorangutan commented on the issue:
https://github.com/apache/cloudstack/pull/1545
Packaging result: âcentos6 âcentos7 âdebian. JID-239
---
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/1735
@nvazquez could you please check into the Travis failures?
---
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 n
Github user serg38 commented on the issue:
https://github.com/apache/cloudstack/pull/1762
@jburwell We've been running this fix as a part of proprietary CS for
several weeks now. We are observing elimination of deadlocks and no DB
corruption. Retry seems to be the only realistic way o
Github user jburwell commented on the issue:
https://github.com/apache/cloudstack/pull/1762
@serg38 corruption could happen at any point -- it's a ticking time bomb.
From a ACID perspective, this patch fails from a consistency perspective. All
data being updated must be re-queried a
Github user serg38 commented on the issue:
https://github.com/apache/cloudstack/pull/1762
What about if the author can figure out a way to identify all part of
transaction being cancelled and retry all parts? Or retry the whole
transaction? It would be nice to open a path for the au
1 - 100 of 171 matches
Mail list logo
