Hi Abhinandan,
I have force reconnect the hosts but no use and i didn't find any error in
SMlog. please suggest me the error is still
generating.
Regards,
Rajasekhar.
On Wed, Sep 16, 2015 at 8:58 AM, Abhinandan Prateek <
abhinandan.prat...@shapeblue.com> wrote:
> Can you check in the SMlog on t
Github user koushik-das commented on a diff in the pull request:
https://github.com/apache/cloudstack/pull/792#discussion_r39827592
--- Diff:
plugins/hypervisors/xenserver/src/com/cloud/hypervisor/xenserver/resource/CitrixResourceBase.java
---
@@ -3407,6 +3390,18 @@ public void ha
Github user sureshanaparti commented on the pull request:
https://github.com/apache/cloudstack/pull/828#issuecomment-141356708
@bhaisaab @sateesh-chodapuneedi Update code as per the existing
conventions. Please check. Working on the unit test. Thanks.
---
If your project is set up f
correction - not "BVT" - "Regression Tests" in Advanced zone.
-Original Message-
From: Raja Pullela [mailto:raja.pull...@citrix.com]
Sent: Friday, September 18, 2015 11:12 AM
To: CloudStack Dev (dev@cloudstack.apache.org)
Subject: [Blocker] PF, static nat, LB, egress rules not working
GitHub user kishankavala opened a pull request:
https://github.com/apache/cloudstack/pull/847
Bug-ID: CLOUDSTACK-8880: calculate free memory on host before deploying Vm.
free memory = total memory - (all vm memory)
With memory over-provisioning set to 1, when mgmt server starts VMs
Hi,
Created a blocker - https://issues.apache.org/jira/browse/CLOUDSTACK-8881
This is failing a bunch of BVTs in Adv for XS/KVM and VMWare.
Can someone please look at this ?
best,
Raja
Github user jayapalu commented on the pull request:
https://github.com/apache/cloudstack/pull/842#issuecomment-141348042
@wilderrodrigues
I have tested the changes on my local against the VR. It is specific to
shared network and the fix is also specific to shared network VR. I thi
Github user bhaisaab commented on a diff in the pull request:
https://github.com/apache/cloudstack/pull/828#discussion_r39824131
--- Diff:
vmware-base/src/com/cloud/hypervisor/vmware/util/VmwareHelper.java ---
@@ -598,6 +601,19 @@ public static void
setBasicVmConfig(VirtualMachine
Github user bhaisaab commented on the pull request:
https://github.com/apache/cloudstack/pull/839#issuecomment-141343706
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
ena
Github user bhaisaab commented on the pull request:
https://github.com/apache/cloudstack/pull/840#issuecomment-141343287
LGTM, pl fix one of the indentation issues and perhaps use better and
verbose log messages.
---
If your project is set up for it, you can reply to this email and h
Github user asfgit closed the pull request at:
https://github.com/apache/cloudstack/pull/817
---
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 bhaisaab commented on a diff in the pull request:
https://github.com/apache/cloudstack/pull/840#discussion_r39823849
--- Diff:
framework/spring/lifecycle/src/main/java/org/apache/cloudstack/spring/lifecycle/CloudStackExtendedLifeCycle.java
---
@@ -96,12 +96,13 @@ publi
Github user bhaisaab commented on the pull request:
https://github.com/apache/cloudstack/pull/817#issuecomment-141342804
LGTM, merging this this is already merged on master and builds are passing
on 4.5 branch now
---
If your project is set up for it, you can reply to this email and
GitHub user kishankavala opened a pull request:
https://github.com/apache/cloudstack/pull/846
Bug-ID: CLOUDSTACK-8870: Skip external device usage collection if no
external devices exist
external network device usage monitor thread that runs every 5mins by
default (based on global c
On 09/17/2015 02:33 PM, Dmytro Shevchenko wrote:
> Nice work. I compiled and install new version into local maven
> repository, but now I can't compile Cloudstack with this library. I
> changed dependency version in pom file to new, but got next exception
> while compiling 'cloud-plugin-hyperviso
Github user remibergsma commented on the pull request:
https://github.com/apache/cloudstack/pull/845#issuecomment-141175510
@wilderrodrigues The steps to test this are simply creating a POD, then
deleting it. It fails in current master and succeeds in this PR. Same with a
zone. If you
Github user remibergsma commented on the pull request:
https://github.com/apache/cloudstack/pull/845#issuecomment-141170729
For the record, I tested against fa5f388fe91bc2fd477b1f6752c2a31ac69d5051.
---
If your project is set up for it, you can reply to this email and have your
reply
Github user rafaelweingartner commented on the pull request:
https://github.com/apache/cloudstack/pull/778#issuecomment-141142775
@wilderrodrigues,
Of course no hard feelings.
I understand the pain that you feel when you stabilize everything and
people come and breaking t
Github user borisroman commented on a diff in the pull request:
https://github.com/apache/cloudstack/pull/765#discussion_r39765711
--- Diff: systemvm/patches/debian/config/opt/cloud/bin/cs/CsAddress.py ---
@@ -414,7 +426,7 @@ def fw_router(self):
self.fw.append(['', '',
Github user wilderrodrigues commented on the pull request:
https://github.com/apache/cloudstack/pull/778#issuecomment-141135958
Hi there @rafaelweingartner,
No hard feelings, dude.
Long story short:
* we are going through a critical phase with ACS - release
Github user remibergsma commented on the pull request:
https://github.com/apache/cloudstack/pull/845#issuecomment-141124718
@borisroman Just successfully deleted a POD and a ZONE.
---
If your project is set up for it, you can reply to this email and have you
Github user remibergsma commented on the pull request:
https://github.com/apache/cloudstack/pull/845#issuecomment-141086717
Hi @borisroman I deployed a DC and then added an extra POD. Then without
creating even a cluster, tried to delete it. It failed saying:
```
The pod c
Nice work. I compiled and install new version into local maven
repository, but now I can't compile Cloudstack with this library. I
changed dependency version in pom file to new, but got next exception
while compiling 'cloud-plugin-hypervisor-kvm':
Konsole output
[ERROR] Failed to execute goal
Github user ustcweizhou commented on the pull request:
https://github.com/apache/cloudstack/pull/844#issuecomment-141065052
@DaanHoogland @miguelaferreira @wilderrodrigues
I have rebase with the lastest master, and force push to github.
let's see if it will be fixed.
---
If
Github user miguelaferreira commented on the pull request:
https://github.com/apache/cloudstack/pull/844#issuecomment-141064763
Thanks for rebasing @ustcweizhou
The pull-analysis is now running.
https://builds.apache.org/job/cloudstack-pull-analysis/583/
Lets keep an e
Github user DaanHoogland commented on the pull request:
https://github.com/apache/cloudstack/pull/845#issuecomment-141061213
@borisroman so please add a description of your manual test scheme in the
description area of this PR
( and help us think of a better dao implementation ;)
Github user borisroman commented on the pull request:
https://github.com/apache/cloudstack/pull/845#issuecomment-141059956
@DaanHoogland That was also one of my questions on Slack. There are no Unit
tests regarding Dao classes in /cloud-engine-schema/test/. So how do I test
these?
Github user miguelaferreira commented on the pull request:
https://github.com/apache/cloudstack/pull/844#issuecomment-141059487
@DaanHoogland @wilderrodrigues @ustcweizhou I agree with Daan, the problem
is most likely elsewhere. But I also agree with Wilder, first we need to figure
o
Github user DaanHoogland commented on the pull request:
https://github.com/apache/cloudstack/pull/844#issuecomment-141058143
@wilderrodrigues that is not fair, it is not @ustcweizhou 's bug that needs
fixing!
---
If your project is set up for it, you can reply to this email and have
Github user DaanHoogland commented on the pull request:
https://github.com/apache/cloudstack/pull/845#issuecomment-141057501
@borisroman there are no (unit) tests. can you add or describe or at least
how to prove this fix?
---
If your project is set up for it, you can reply to this e
Github user DaanHoogland commented on the pull request:
https://github.com/apache/cloudstack/pull/775#issuecomment-141056052
in general: factor out a small chunk of the method into a private method or
helper class. (commit 1)
then unit test that. (commit 2)
then apply your fix
Github user wilderrodrigues commented on the pull request:
https://github.com/apache/cloudstack/pull/844#issuecomment-141052485
@ustcweizhou ,
Which tests did you execute?
I would say: no merge until it's fixed, tested by @ustcweizhou and he let
us know how we should
Github user DaanHoogland commented on the pull request:
https://github.com/apache/cloudstack/pull/844#issuecomment-141051316
@ustcweizhou @miguelaferreira @karuturi @remibergsma so no merges till this
is fixed?
---
If your project is set up for it, you can reply to this email and ha
Github user DaanHoogland commented on the pull request:
https://github.com/apache/cloudstack/pull/829#issuecomment-141050957
If @remibergsma is right then the design is really broken. two events come
in during migration:
# PowerOn (or Off or what ever is actual)
# PowerReportMi
Github user borisroman commented on the pull request:
https://github.com/apache/cloudstack/pull/845#issuecomment-141022408
@remibergsma @wilderrodrigues @karuturi Please review!
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as w
Github user nitin-maharana commented on the pull request:
https://github.com/apache/cloudstack/pull/775#issuecomment-141015775
@DaanHoogland, @rafaelweingartner Yes I understand, I also think there
should be test cases for a change because we don't know the entire logic. As
there is n
Github user wido commented on the pull request:
https://github.com/apache/cloudstack/pull/835#issuecomment-141004854
I agree with @DaanHoogland That should be unit tested.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. I
Github user wido commented on the pull request:
https://github.com/apache/cloudstack/pull/841#issuecomment-141004554
@karuturi I think that 8 is a good thing, that brings in much more
combinations. Maybe even 10?
And I agree with @bhaisaab, no more global settings :)
---
If
Github user wilderrodrigues commented on the pull request:
https://github.com/apache/cloudstack/pull/832#issuecomment-141002870
Thanks, @borisroman ! :)
I will have a look at your PR again later today.
Cheers,
Wilder
---
If your project is set up for it, you can
Github user wilderrodrigues commented on the pull request:
https://github.com/apache/cloudstack/pull/842#issuecomment-141001813
Thanks, @jayapalu
I will try to take some time today to test your PR.
Cheers,
Wilder
---
If your project is set up for it, you can rep
Issue is here: https://issues.apache.org/jira/browse/CLOUDSTACK-8878
Cheers,
Wilder
On 17 Sep 2015, at 10:14, Wilder Rodrigues
mailto:wrodrig...@schubergphilis.com>> wrote:
Hi Rajani,
The last commit from 8 September works:
https://github.com/apache/cloudstack/commit/5e9e9b84fb1a4ca029c32e5b
Hi Rajani,
The last commit from 8 September works:
https://github.com/apache/cloudstack/commit/5e9e9b84fb1a4ca029c32e5b1c305124bfa4d4af
The merge commit just before the 5e9e9b does NOT work:
https://github.com/apache/cloudstack/commit/237e0e4611dd3d0ae56a8c0557dba93e7fbd3440
And the commit be
Github user nitin-maharana closed the pull request at:
https://github.com/apache/cloudstack/pull/790
---
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 fea
Github user nitin-maharana commented on the pull request:
https://github.com/apache/cloudstack/pull/790#issuecomment-140999446
@remibergsma Yes this one should be closed. PR #775 does the same thing.
Thanks.
---
If your project is set up for it, you can reply to this email and have y
Github user asfgit closed the pull request at:
https://github.com/apache/cloudstack/pull/820
---
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
49 matches
Mail list logo
