Github user nlivens commented on the pull request:
https://github.com/apache/cloudstack/pull/1471#issuecomment-221791953
@remibergsma, I understand your concern regarding the wait_until method.
Perhaps we could use the wait_until with a 2s interval and a max wait of 4s?
That's 1 retry
GitHub user rhtyd reopened a pull request:
https://github.com/apache/cloudstack/pull/1511
4.9/master bountycastle changes
This PR adds bountycastle related version and code fixes from @DaanHoogland
on top of https://github.com/apache/cloudstack/pull/1510
I could not fix jav
Github user rhtyd closed the pull request at:
https://github.com/apache/cloudstack/pull/1511
---
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 sanju1010 commented on the pull request:
https://github.com/apache/cloudstack/pull/1550#issuecomment-221788191
I ran bunch of tests(related to network functionality) from the regression
test suite(test/integration/component) on this PR branch and I don't see any
test failu
GitHub user dsclose reopened a pull request:
https://github.com/apache/cloudstack/pull/1519
Cloudstack 9339: Virtual Routers do not handle Multiple Public Interfaces
This PR addresses CLOUDSTACK-9339 and may need a code review from someone
familiar with the System VM scripts. In par
Github user dsclose closed the pull request at:
https://github.com/apache/cloudstack/pull/1519
---
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 i
Github user bvbharatk commented on the pull request:
https://github.com/apache/cloudstack/pull/866#issuecomment-221785775
@swill
Rebased with master and resolved the conflicts.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub
Github user anshul1886 commented on the pull request:
https://github.com/apache/cloudstack/pull/669#issuecomment-221780994
@swill Resolved the merge conflicts. They were because of import statements.
---
If your project is set up for it, you can reply to this email and have your
reply
Github user pavanb018 commented on the pull request:
https://github.com/apache/cloudstack/pull/1561#issuecomment-221780495
Very small change , conversion of firewall port to string was not necessary
and that seems to be removed. Looks Good to me !!
---
If your project is set up for i
Github user asfgit closed the pull request at:
https://github.com/apache/cloudstack/pull/956
---
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/1494
---
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/1556
---
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/1533
---
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/1360
---
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/1550
---
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/1518
---
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/1503
---
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/1424
---
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/1297
---
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/1552
---
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/1555
---
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/1553
---
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/846
---
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/1514
---
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/1410
---
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 swill commented on the pull request:
https://github.com/apache/cloudstack/pull/1251#issuecomment-221764160
Need code review on this one. Thx...
---
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
Github user swill commented on the pull request:
https://github.com/apache/cloudstack/pull/669#issuecomment-221763988
This one needs some work. We have merge conflicts and we need another code
review.
---
If your project is set up for it, you can reply to this email and have your
rep
Github user swill commented on the pull request:
https://github.com/apache/cloudstack/pull/1499#issuecomment-221763535
Bump...
@alexandrelimassantana this one has merge conflicts. Can you rebase and fix
the conflicts? Thanks...
---
If your project is set up for it, you can r
Github user swill commented on the pull request:
https://github.com/apache/cloudstack/pull/1491#issuecomment-221763412
This one needs review as well...
---
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 swill commented on the pull request:
https://github.com/apache/cloudstack/pull/1471#issuecomment-221762946
Can I get some code review on 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
Github user swill commented on the pull request:
https://github.com/apache/cloudstack/pull/1450#issuecomment-221762730
CI is clean and everything is green. I need some code review on this one.
Thanks...
---
If your project is set up for it, you can reply to this email and have your
Github user swill commented on the pull request:
https://github.com/apache/cloudstack/pull/1555#issuecomment-221759242
This is ready to merge now. The issue we are seeing is a known issue not
related to this PR.
---
If your project is set up for it, you can reply to this email and h
Github user swill commented on the pull request:
https://github.com/apache/cloudstack/pull/1555#issuecomment-221759081
### CI RESULTS
```
Tests Run: 82
Skipped: 0
Failed: 0
Errors: 3
Duration: 8h 37m 16s
```
**Summary of the p
Github user swill commented on the pull request:
https://github.com/apache/cloudstack/pull/1553#issuecomment-221758855
This is a known issue, so this one should be ready now. Thx...
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub
Github user swill commented on the pull request:
https://github.com/apache/cloudstack/pull/1553#issuecomment-221758690
### CI RESULTS
```
Tests Run: 83
Skipped: 0
Failed: 0
Errors: 2
Duration: 8h 39m 49s
```
**Summary of the p
cool. :)
*Will STEVENS*
Lead Developer
*CloudOps* *| *Cloud Solutions Experts
420 rue Guy *|* Montreal *|* Quebec *|* H3J 1S6
w cloudops.com *|* tw @CloudOps_
On Wed, May 25, 2016 at 9:45 PM, Syed Mushtaq
wrote:
> Nope. I think they should all be XenServer (they were Citrix earlier) I
> plan
Github user swill commented on the pull request:
https://github.com/apache/cloudstack/pull/846#issuecomment-221758314
This is a cleaner run because this issue is a known issue. I think this
one is ready now...
---
If your project is set up for it, you can reply to this email and hav
Github user swill commented on the pull request:
https://github.com/apache/cloudstack/pull/846#issuecomment-221757982
### CI RESULTS
```
Tests Run: 82
Skipped: 0
Failed: 0
Errors: 3
Duration: 8h 33m 42s
```
**Summary of the pr
Nope. I think they should all be XenServer (they were Citrix earlier) I
plan to clean that up too.
-Syed
On Wed, May 25, 2016 at 9:43 PM, Will Stevens
wrote:
> Is there a reason some are prefixed with 'Xs' and others are prefixed with
> 'XenServer'?
> On May 25, 2016 9:41 PM, "Will Stevens" wr
Is there a reason some are prefixed with 'Xs' and others are prefixed with
'XenServer'?
On May 25, 2016 9:41 PM, "Will Stevens" wrote:
> Ya. I didn't know either and made the same mistake. :)
> On May 25, 2016 9:40 PM, "Syed Mushtaq" wrote:
>
>> Aah I did not know that. Thanks Will .. here is th
Ya. I didn't know either and made the same mistake. :)
On May 25, 2016 9:40 PM, "Syed Mushtaq" wrote:
> Aah I did not know that. Thanks Will .. here is the link
> http://i.imgur.com/5B55XMB.png
>
> On Wed, May 25, 2016 at 6:37 PM, Will Stevens
> wrote:
>
> > Attachments don't work. You will have
Aah I did not know that. Thanks Will .. here is the link
http://i.imgur.com/5B55XMB.png
On Wed, May 25, 2016 at 6:37 PM, Will Stevens
wrote:
> Attachments don't work. You will have to host it publicly and then post a
> link to it.
> On May 25, 2016 4:28 PM, "Syed Ahmed" wrote:
>
> > Forgot to a
Correct, all modern XenServer distros until 7 were centos 5.x. I doubt any
xapi changes would break cloudstack, but the plugins should be retested,
and btw it's also systemd time
On May 25, 2016 2:39 PM, "Remi Bergsma" wrote:
As far as I know previous versions were based on Centos 5 even ;-)
Reg
Attachments don't work. You will have to host it publicly and then post a
link to it.
On May 25, 2016 4:28 PM, "Syed Ahmed" wrote:
> Forgot to attach screenshot
>
> On Wed, May 25, 2016 at 4:09 PM, Syed Ahmed wrote:
>
>> Thanks Rafael,
>>
>> Here is how I am doing. You can see the tree in the sc
As far as I know previous versions were based on Centos 5 even ;-)
Regards, Remi
> On 25 May 2016, at 16:55, Rafael Weingärtner
> wrote:
>
> Oh, I did know that, thanks for sharing Will.
>
> I was reading their Release notes; it seems that their “major changes”
> section does not reflect the
Forgot to attach screenshot
On Wed, May 25, 2016 at 4:09 PM, Syed Ahmed wrote:
> Thanks Rafael,
>
> Here is how I am doing. You can see the tree in the screenshot attached. I
> have started with storage and extracted the storage commands out of
> CitrixResourceBase (I've renamed it to XenServerR
Thanks Rafael,
Here is how I am doing. You can see the tree in the screenshot attached. I
have started with storage and extracted the storage commands out of
CitrixResourceBase (I've renamed it to XenServerResourceBase). This is the
file for refrence. Let me know what you think.
https://github.co
Github user KrisSterckx commented on the pull request:
https://github.com/apache/cloudstack/pull/1494#issuecomment-221666486
Thanks @swill and all involved !
---
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
Github user nvazquez commented on a diff in the pull request:
https://github.com/apache/cloudstack/pull/1560#discussion_r64617338
--- Diff:
plugins/hypervisors/vmware/src/com/cloud/hypervisor/vmware/resource/VmwareResource.java
---
@@ -5510,6 +5501,27 @@ public Answer execute(Dest
Github user swill commented on the pull request:
https://github.com/apache/cloudstack/pull/1519#issuecomment-221637516
@dsclose can you kick it off again. It is just a timeout which caused it
to fail, so kicking it off again could fix it. `No output has been received in
the last 10m
Github user swill commented on the pull request:
https://github.com/apache/cloudstack/pull/1494#issuecomment-221631597
This error is unrelated to this PR. This is ready to merge...
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub
Github user rafaelweingartner commented on a diff in the pull request:
https://github.com/apache/cloudstack/pull/1560#discussion_r64607461
--- Diff:
plugins/hypervisors/vmware/src/com/cloud/hypervisor/vmware/resource/VmwareResource.java
---
@@ -5510,6 +5501,27 @@ public Answer exe
Github user swill commented on the pull request:
https://github.com/apache/cloudstack/pull/1553#issuecomment-221634279
### CI RESULTS
```
Tests Run: 83
Skipped: 0
Failed: 1
Errors: 2
Duration: 8h 32m 15s
```
**Summary of the p
Github user swill commented on the pull request:
https://github.com/apache/cloudstack/pull/1559#issuecomment-221633251
### CI RESULTS
```
Tests Run: 85
Skipped: 0
Failed: 1
Errors: 1
Duration: 10h 41m 20s
```
**Summary of the
Github user swill commented on the pull request:
https://github.com/apache/cloudstack/pull/846#issuecomment-221633655
### CI RESULTS
```
Tests Run: 85
Skipped: 0
Failed: 2
Errors: 1
Duration: 10h 41m 55s
```
**Summary of the p
Github user swill commented on the pull request:
https://github.com/apache/cloudstack/pull/1494#issuecomment-221631430
### CI RESULTS
```
Tests Run: 84
Skipped: 0
Failed: 0
Errors: 1
Duration: 5h 26m 59s
```
**Summary of the p
Github user swill commented on the pull request:
https://github.com/apache/cloudstack/pull/1423#issuecomment-221631305
@syed can you review this error? It seems to be related to your test.
Thanks...
---
If your project is set up for it, you can reply to this email and have your
rep
Github user swill commented on the pull request:
https://github.com/apache/cloudstack/pull/1423#issuecomment-221631046
### CI RESULTS
```
Tests Run: 85
Skipped: 0
Failed: 0
Errors: 1
Duration: 4h 14m 49s
```
**Summary of the p
Github user rafaelweingartner commented on the pull request:
https://github.com/apache/cloudstack/pull/1560#issuecomment-221629942
Great job, thanks @nvazquez
---
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 nvazquez commented on the pull request:
https://github.com/apache/cloudstack/pull/1560#issuecomment-221623024
Hi @rafaelweingartner thanks for your comments!
I extracted to a method and added test cases for this new method. Also
changed Exception to CloudRuntimeExceptio
Github user dsclose commented on the pull request:
https://github.com/apache/cloudstack/pull/1519#issuecomment-221611699
Same issue. I don't think this is related to the PR. I'll leave this be for
now.
---
If your project is set up for it, you can reply to this email and have your
re
Hi Syed,
That is a great job.
I would only suggest breaking the commons a little bit more between
“monitoring” and “common”. On the monitoring side, we could have host
monitoring, VMs monitoring, VMs' status checks (running, stopped, and
others) and maybe other tasks that aim to monitor/check a res
Hey Guys,
To give you an update, I've identified and categorized the functions in
CitrixResourceBase into 4 categories
1) common: These deal with the host as a whole (example getHostInfo,
callPlugin, connection pool etc)
2) compute: Dealing with operations on VMs (start,stop, reboot, update etc)
I had installed Xenserver 7 when it was still in beta. If I remember
correctly, it has a different python version than 6.5 (2.4 in 6.5 and 2.6
in 7). Some scripts had been modified to run on python 2.4 may break with
XS7. Will have to verify this.
On Wed, May 25, 2016 at 10:55 AM, Rafael Weingär
Oh, I did know that, thanks for sharing Will.
I was reading their Release notes; it seems that their “major changes”
section does not reflect the experience you are getting.
On Wed, May 25, 2016 at 11:42 AM, Will Stevens
wrote:
> CentOS changes a LOT, especially around networking, so I would no
CentOS changes a LOT, especially around networking, so I would not be
surprised if it does not work. It still takes me like 20 minutes to figure
out what the hell I am doing when I login to CentOS7. No ifconfig, no
iptables, etc, etc, etc... They pretty much did a wholesale change of
everything y
Github user dsclose commented on the pull request:
https://github.com/apache/cloudstack/pull/1519#issuecomment-221600590
> No output has been received in the last 10m0s, this potentially indicates
a stalled build or something wrong with the build itself.
>
>The build has been t
GitHub user dsclose reopened a pull request:
https://github.com/apache/cloudstack/pull/1519
Cloudstack 9339: Virtual Routers do not handle Multiple Public Interfaces
This PR addresses CLOUDSTACK-9339 and may need a code review from someone
familiar with the System VM scripts. In par
Github user dsclose closed the pull request at:
https://github.com/apache/cloudstack/pull/1519
---
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 i
Considering those scripts you are right; but, my main concerns would be
regarding some XAPI change. The scripts that we create and inject into the
Dom0 should not be that intrusive.
I do not know about the differences between CentOS 6 and 7, but the only
way I see a script stopping working is that
Rafael
We don’t purely use XAPI with XenServer.
XenServer 7 has a CentOS7 dom0 rather than CentOS6. We do a whole load of
stuff behind the scenes with python and shell scripts a chunk of which won't
work anymore.
Kind regards,
Paul Angus
paul.an...@shapeblue.com
www.shapeblue.com
53 Chand
Github user footplus commented on the pull request:
https://github.com/apache/cloudstack/pull/1555#issuecomment-221572152
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 featu
Will there be any change on the XAPI? If not, it already works (just the
new functions, if any, that will not be used).
On Wed, May 25, 2016 at 9:59 AM, Paul Angus
wrote:
> Is anyone here working on XenServer 7 support for CloudStack?
>
>
> Kind regards,
>
> Paul Angus
>
>
> paul.an...@shapeblue
Is anyone here working on XenServer 7 support for CloudStack?
Kind regards,
Paul Angus
paul.an...@shapeblue.com
www.shapeblue.com
53 Chandos Place, Covent Garden, London WC2N 4HSUK
@shapeblue
Github user rhtyd commented on the pull request:
https://github.com/apache/cloudstack/pull/1562#issuecomment-221538713
LGTM, simple fix.
Tested on local database.
tag:easypr
---
If your project is set up for it, you can reply to this email and have your
reply appear on Gi
Github user wido commented on the pull request:
https://github.com/apache/cloudstack/pull/1562#issuecomment-221538332
@rhtyd: Could you take a look? As you can see, it's a very small fix.
---
If your project is set up for it, you can reply to this email and have your
reply appear on G
Github user wido commented on the pull request:
https://github.com/apache/cloudstack/pull/1562#issuecomment-221531859
@swill: Still time to get this one in?
This fix is running in production on our clouds. It is just a ALTER TABLE
to increase the size of a database field.
--
GitHub user wido opened a pull request:
https://github.com/apache/cloudstack/pull/1562
CLOUDSTACK-9238: Change template_host_ref.url to varchar(2048)
In Pull Request #1341 / commit a171bb this was fixed for many other
URL fields, but this was not corrected and is still a varchar(
Github user dsclose closed the pull request at:
https://github.com/apache/cloudstack/pull/1519
---
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 i
GitHub user dsclose reopened a pull request:
https://github.com/apache/cloudstack/pull/1519
Cloudstack 9339: Virtual Routers do not handle Multiple Public Interfaces
This PR addresses CLOUDSTACK-9339 and may need a code review from someone
familiar with the System VM scripts. In par
GitHub user sanju1010 opened a pull request:
https://github.com/apache/cloudstack/pull/1561
CLOUDSTACK-9388: Remove string conversion in assertion statement
Remove string convertion in Assertion statement, since the start port
parameter in listFirewallAPI response is of type integer
Github user dsclose commented on a diff in the pull request:
https://github.com/apache/cloudstack/pull/1519#discussion_r64537215
--- Diff: systemvm/patches/debian/config/opt/cloud/bin/cs/CsAddress.py ---
@@ -288,14 +290,16 @@ def post_configure(self, address):
""" The s
Github user dsclose commented on the pull request:
https://github.com/apache/cloudstack/pull/1519#issuecomment-221501553
@swill taking a look 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 h
Github user milamberspace commented on the pull request:
https://github.com/apache/cloudstack/pull/1555#issuecomment-221497756
I add the noprompt option. Travis&Jenkins are green. Ready to merge.
cc @swill @footplus
---
If your project is set up for it, you can reply to this
84 matches
Mail list logo
