[GitHub] cloudstack pull request: Cloudstack 9339: Virtual Routers do not h...

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

Re: XenServer 7

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

Re: Refactoring CitrixResourceBase

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

Re: XenServer 7

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

[GitHub] cloudstack pull request: Cloudstack 9339: Virtual Routers do not h...

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: XenServer 7

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

Re: XenServer 7

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

[GitHub] cloudstack pull request: Cloudstack 9339: Virtual Routers do not h...

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

[GitHub] cloudstack pull request: Cloudstack 9339: Virtual Routers do not h...

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

Re: XenServer 7

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

[GitHub] cloudstack pull request: CLOUDSTACK-9368: DS template copies don��...

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

Re: XenServer 7

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”

[GitHub] cloudstack pull request: Remodeling of Nuage VSP Plugin + CLOUDSTA...

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

Re: Refactoring CitrixResourceBase

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

Re: XenServer 7

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

Re: Refactoring CitrixResourceBase

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.

Re: Refactoring CitrixResourceBase

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

[GitHub] cloudstack pull request: Bug-ID: CLOUDSTACK-8870: Skip external de...

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

[GitHub] cloudstack pull request: CLOUDSTACK-9180: Optimize concurrent VM d...

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] cloudstack pull request: Hyper-V communication broken by change in...

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

[GitHub] cloudstack pull request: Convert patchviasocket to python (removes...

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

[GitHub] cloudstack pull request: CLOUDSTACK-9380: fix NPE in listDomains A...

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

[GitHub] cloudstack pull request: CLOUDSTACK-8970 Centos 6.{1,2,3,4,5} gues...

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] cloudstack pull request: Refactor system VM default network creati...

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

[GitHub] cloudstack pull request: Remodeling of Nuage VSP Plugin + CLOUDSTA...

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

Re: Refactoring CitrixResourceBase

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

[GitHub] cloudstack pull request: Add Java Default Certificat Authorities i...

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

Re: Refactoring CitrixResourceBase

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

[GitHub] cloudstack pull request: Lower the time we wait for interfaces to ...

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] cloudstack pull request: CLOUDSTACK-9317: Enable/disable static NA...

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

[GitHub] cloudstack pull request: Made the adding new keyboard language sup...

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

Re: Refactoring CitrixResourceBase

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

[GitHub] cloudstack pull request: Dynamically load drivers before creating ...

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

[GitHub] cloudstack pull request: add DHCP lease folders for Ubuntu

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

[GitHub] cloudstack pull request: Add Java Default Certificat Authorities i...

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

[GitHub] cloudstack pull request: [CLOUDSTACK-8973] Fix create template fro...

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

[GitHub] cloudstack pull request: CLOUDSTACK-9203 Implement security group ...

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

[GitHub] cloudstack pull request: CLOUDSTACK-9358: StringIndexOutOfBoundsEx...

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

[GitHub] cloudstack pull request: CLOUDSTACK-9368: Fix for Support configur...

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

[GitHub] cloudstack pull request: Bug-ID: CLOUDSTACK-8870: Skip external de...

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

Re: Refactoring CitrixResourceBase

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

[GitHub] cloudstack pull request: Bug-ID: CLOUDSTACK-8870: Skip external de...

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

[GitHub] cloudstack pull request: Dynamically load drivers before creating ...

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] cloudstack pull request: Add Java Default Certificat Authorities i...

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

[GitHub] cloudstack pull request: Fixes regarding VOLUME_DELETE events resu...

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

Re: Refactoring CitrixResourceBase

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: >

[GitHub] cloudstack pull request: Undetected bug correct and refactor of th...

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

[GitHub] cloudstack pull request: Made the adding new keyboard language sup...

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

[GitHub] cloudstack pull request: CLOUDSTACK-9388: Remove string conversion...

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

[GitHub] cloudstack pull request: CLOUDSTACK-6975: Prevent dnsmasq from sta...

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

[GitHub] cloudstack pull request: CLOUDSTACK-6928: fix issue disk I/O throt...

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

[GitHub] cloudstack pull request: Add Java Default Certificat Authorities i...

Github user milamberspace commented on the pull request: https://github.com/apache/cloudstack/pull/1555#issuecomment-221497756 I add the noprompt option. Travis are green. Ready to merge. cc @swill @footplus --- If your project is set up for it, you can reply to this email

[GitHub] cloudstack pull request: Cloudstack 9339: Virtual Routers do not h...

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

[GitHub] cloudstack pull request: CLOUDSTACK-9238: Change template_host_ref...

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

[GitHub] cloudstack pull request: Cloudstack 9339: Virtual Routers do not h...

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

[GitHub] cloudstack pull request: CLOUDSTACK-9388: Remove string conversion...

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

[GitHub] cloudstack pull request: Cloudstack 9339: Virtual Routers do not h...

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

[GitHub] cloudstack pull request: Cloudstack 9339: Virtual Routers do not h...

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

[GitHub] cloudstack pull request: CLOUDSTACK-9238: Change template_host_ref...

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

[GitHub] cloudstack pull request: CLOUDSTACK-9238: Change template_host_ref...

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] cloudstack pull request: CLOUDSTACK-9238: Change template_host_ref...

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

XenServer 7

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

Re: XenServer 7

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

[GitHub] cloudstack pull request: Add Java Default Certificat Authorities i...

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

[GitHub] cloudstack pull request: [CLOUDSTACK-9296] Start ipsec for client ...

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

Re: Refactoring CitrixResourceBase

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

[GitHub] cloudstack pull request: Dynamically load drivers before creating ...

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

[GitHub] cloudstack pull request: CLOUDSTACK-9368: DS template copies don��...

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

[GitHub] cloudstack pull request: CLOUDSTACK-9368: DS template copies don��...

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

[GitHub] cloudstack pull request: Remodeling of Nuage VSP Plugin + CLOUDSTA...

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] cloudstack pull request: [CLOUDSTACK-9296] Start ipsec for client ...

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

[GitHub] cloudstack pull request: Bug-ID: CLOUDSTACK-8870: Skip external de...

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

[GitHub] cloudstack pull request: Remodeling of Nuage VSP Plugin + CLOUDSTA...

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

[GitHub] cloudstack pull request: CLOUDSTACK-9280: System VM volumes can be...

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] cloudstack pull request: CLOUDSTACK-9368: DS template copies don��...

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

[GitHub] cloudstack pull request: Cloudstack 9339: Virtual Routers do not h...

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