[GitHub] cloudstack pull request: CLOUDSTACK-9057 remove old system vm upgr...

Github user asfgit closed the pull request at: https://github.com/apache/cloudstack/pull/1072 --- 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-9051: update nic IP address of...

Github user ustcweizhou commented on the pull request: https://github.com/apache/cloudstack/pull/1086#issuecomment-158360198 @jayapalu 1. The /etc/dhcphosts.txt will not be changed inside VR during this process. root@r-7542-VM:~# cat /etc/dhcphosts.txt

[GitHub] cloudstack pull request: javascript:TrailingComma sonacube issue c...

Github user koushik-das commented on the pull request: https://github.com/apache/cloudstack/pull/1069#issuecomment-158375503 LGTM based on the changes --- 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: 4.6 release

On 11/20/2015 02:04 PM, Paul Angus wrote: > http://docs.cloudstack.apache.org/projects/cloudstack-release-notes/en/4.6.0/upgrade/upgrade-4.5.html#hypervisor-vmware > states: > 'Warning > For VMware hypervisor CloudStack management server packages must be build > using “noredist”. Refer to

[GitHub] cloudstack pull request: Cwe 190

Github user DaanHoogland commented on the pull request: https://github.com/apache/cloudstack/pull/1057#issuecomment-158400754 to many unexplained failures for such a small refactor. --- 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: Cwe 190

Github user DaanHoogland closed the pull request at: https://github.com/apache/cloudstack/pull/1057 --- 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

[GitHub] cloudstack pull request: CLOUDSTACK-8485: listAPIs are taking too ...

Github user DaanHoogland commented on the pull request: https://github.com/apache/cloudstack/pull/1021#issuecomment-158401181 also running the regressions suites on it now --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well.

[GitHub] cloudstack pull request: javascript:TrailingComma sonacube issue c...

Github user wido commented on the pull request: https://github.com/apache/cloudstack/pull/1069#issuecomment-158403924 The fix seems good to me. I would like to LGTM, but I can't test JavaScript that easily. Isn't there a lint-check for JS? --- If your project is set up for

[GitHub] cloudstack pull request: CLOUDSTACK-9074: Support shared networkin...

Github user remibergsma commented on the pull request: https://github.com/apache/cloudstack/pull/1094#issuecomment-158334828 LGTM, based on a set of tests that I run on this branch (which I rebased myself first). Screenshot is from an experimental Jenkins job (that runs the same

[GitHub] cloudstack pull request: Usage event fixes for deleted accounts

Github user remibergsma commented on the pull request: https://github.com/apache/cloudstack/pull/924#issuecomment-158348133 I tried rebasing against current master but I run into build issues. Tried again but same thing. @nnesic Could you try rebasing the PR against master

Request for CloudStack Dev Slack channel access

Hi, Can you please grant me access and point me to the CloudStack Dev Slack channel. With regards, Prashanth

[GitHub] cloudstack pull request: Fix NuageVsp errors for build-master-slow...

Github user DaanHoogland commented on the pull request: https://github.com/apache/cloudstack/pull/1092#issuecomment-158376719 @nlivens @fmaximus I have to ask you to reconsider. if getPreffferedCharset() does not return UTF8 it is because it is not supported on the platform. how do

[GitHub] cloudstack pull request: [WIP] CLOUDSTACK-6276: project support in...

Github user ustcweizhou commented on the pull request: https://github.com/apache/cloudstack/pull/508#issuecomment-158342455 @pdube you may notice this code in UserVmManagerImpl.java _accountMgr.checkAccess(caller, null, true, owner, ag); normally it will check (1)

Re: [DISCUSS] ACS 4.5.3 release

Hi Rohit We at swiss txt would appreciate a 4.5.3 to be released asap, it already contains this fix https://github.com/apache/cloudstack/pull/922 which lets us finally activate DRS in our vmware setup. Huge gain on performance and reliability. Another one related to VR which has _not_ yet been

[GitHub] cloudstack pull request: CLOUDSTACK-9051: update nic IP address of...

Github user koushik-das commented on the pull request: https://github.com/apache/cloudstack/pull/1086#issuecomment-158346095 @ustcweizhou Some questions. 1. Is updateVmNicIp API admin only? 2. How the caller knows if the IP address is available and what is the CIDR associated

[GitHub] cloudstack pull request: [4.5] CLOUDSTACK-9042: VR: DHCP: fix miss...

Github user resmo commented on the pull request: https://github.com/apache/cloudstack/pull/1045#issuecomment-158350734 @remibergsma @bhaisaab, no, not yet, created a patch for master before release of 4.6 #1042 but I was told it is not necessary, script is not used, was replaced by a

[GitHub] cloudstack pull request: CLOUDSTACK-9063 CitrixResourceBase refact...

Github user DaanHoogland commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1066#discussion_r45456092 --- Diff: plugins/hypervisors/xenserver/src/com/cloud/hypervisor/xenserver/resource/CitrixResourceData.java --- @@ -0,0 +1,289 @@ +package

Re: 4.6 release

Paul, this is not helpful. please supply changes for what you would like to see instead so we can discuss. On Fri, Nov 20, 2015 at 10:10 AM, Paul Angus wrote: > Guys, > > > > I’m out and about this morning, but I’ve noticed that the release notes > state that baseurl

[GitHub] cloudstack pull request: CLOUDSTACK-9062: Improve S3 implementatio...

Github user borisroman commented on the pull request: https://github.com/apache/cloudstack/pull/1083#issuecomment-158344376 @remibergsma Of course I tested manually! Nice Jenkins Job btw ;) I see three LGTM's, two of which are based on code review, one on running integration

[GitHub] cloudstack pull request: CLOUDSTACK-9051: update nic IP address of...

Github user DaanHoogland commented on the pull request: https://github.com/apache/cloudstack/pull/1086#issuecomment-158353414 @jayapalu this discussion is mirrored on dev@ so it is a discussion on the mailing list. In that sense the process reqs are met. Also @wilderrodrigues already

[GitHub] cloudstack pull request: CLOUDSTACK-8485: listAPIs are taking too ...

Github user DaanHoogland commented on the pull request: https://github.com/apache/cloudstack/pull/1021#issuecomment-158374179 code reviewed. LGTM awaiting tests. it is running on one of the INFRA-10703 hosts but I know @abayer did some work on that issue... --- If your project is

[GitHub] cloudstack pull request: Fix NuageVsp errors for build-master-slow...

Github user nlivens commented on the pull request: https://github.com/apache/cloudstack/pull/1092#issuecomment-158381857 @DaanHoogland, how about the following piece of code then? In this case we are using the getPrefferedCharset() ```java public static String

[GitHub] cloudstack pull request: Fix NuageVsp errors for build-master-slow...

Github user DaanHoogland commented on the pull request: https://github.com/apache/cloudstack/pull/1092#issuecomment-158385988 @nlivens looks good, but I have some questions still 1. So why wouldn't you encorporate this in your getBytesUtf8() and newStringUtf8() methods? 2. How

Re: 4.6 release

On Fri, Nov 20, 2015 at 2:04 PM, Paul Angus wrote: > Which version (noredist or oss) have the packages on cloudstack.apt-get.eu > been built with? > ​these used to be build noredist. Has anything changed?​ I think we must replace them if it has. -- Daan

[GitHub] cloudstack pull request: CLOUDSTACK-9057 remove old system vm upgr...

Github user wilderrodrigues commented on the pull request: https://github.com/apache/cloudstack/pull/1072#issuecomment-158335845 The change makes sense and the upgrade was tested by @remibergsma. LGTM :+1: Cheers, Wilder --- If your project is set up for it,

[GitHub] cloudstack pull request: CLOUDSTACK-9053 security upgrade as per C...

Github user remibergsma commented on the pull request: https://github.com/apache/cloudstack/pull/1089#issuecomment-158338131 @DaanHoogland the tests succeed for me: ``` nosetests --with-marvin --marvin-config=${marvinCfg} -s -a tags=advanced,required_hardware=true

[GitHub] cloudstack pull request: CLOUDSTACK-9051: update nic IP address of...

Github user jayapalu commented on the pull request: https://github.com/apache/cloudstack/pull/1086#issuecomment-158351915 @ustcweizhou @wilderrodrigues @koushik-das It is an enhancement/feature to the product. Also a new API is introduced here. So I would recommend to go

[GitHub] cloudstack pull request: CLOUDSTACK-9051: update nic IP address of...

Github user ustcweizhou commented on the pull request: https://github.com/apache/cloudstack/pull/1086#issuecomment-158358371 @koushik-das answers: 1. not only admin , but domain admin and normal user 2. for isolated network, the user can get the information (ip, cidr)

[GitHub] cloudstack pull request: CLOUDSTACK-8485: listAPIs are taking too ...

Github user koushik-das commented on the pull request: https://github.com/apache/cloudstack/pull/1021#issuecomment-158336847 @DaanHoogland did you get a chance to test it? --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well.

[GitHub] cloudstack pull request: CLOUDSTACK-9053 security upgrade as per C...

Github user asfgit closed the pull request at: https://github.com/apache/cloudstack/pull/1089 --- 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-9063 CitrixResourceBase refact...

Github user koushik-das commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1066#discussion_r45450911 --- Diff: plugins/hypervisors/xenserver/src/com/cloud/hypervisor/xenserver/resource/CitrixResourceData.java --- @@ -0,0 +1,289 @@ +package

[GitHub] cloudstack pull request: Fix NuageVsp errors for build-master-slow...

Github user nlivens commented on the pull request: https://github.com/apache/cloudstack/pull/1092#issuecomment-158375131 @DaanHoogland, @fmaximus was right about the encoding. I rewrote that part so we're always using the same encoding :) --- If your project is set up for it, you

RE: 4.6 release

http://docs.cloudstack.apache.org/projects/cloudstack-release-notes/en/4.6.0/upgrade/upgrade-4.5.html#hypervisor-vmware states: 'Warning For VMware hypervisor CloudStack management server packages must be build using “noredist”. Refer to Building from Source.' I can't find the same warning in the

[GitHub] cloudstack pull request: CLOUDSTACK-8485: listAPIs are taking too ...

Github user DaanHoogland commented on the pull request: https://github.com/apache/cloudstack/pull/1021#issuecomment-158347545 @koushik-das sorry, no. I have not much test infra and run a lot of tests. Could you force push your changes, please. I'll review your last changes. --- If

[GitHub] cloudstack pull request: CLOUDSTACK-8485: listAPIs are taking too ...

Github user koushik-das commented on the pull request: https://github.com/apache/cloudstack/pull/1021#issuecomment-158366425 @DaanHoogland I have force pushed again. Hopefully a good jenkins slave will pick this up :) --- If your project is set up for it, you can reply to this email

[GitHub] cloudstack pull request: CLOUDSTACK-8485: listAPIs are taking too ...

Github user DaanHoogland commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1021#discussion_r45461347 --- Diff: server/src/com/cloud/api/response/ApiResponseSerializer.java --- @@ -127,53 +151,74 @@ public static String

CloudStack - New Feature: Root Disk/Volume Resize for XenServer

Hi, I would like to know if you are planing to add "ROOT DISK/VOLUME RESIZE" feature for XenServer 6.2/6.5 in CloudStack? It is already present for KVM. Best regrads, -- Srdačan pozdrav,Milan Boharević Lead System Engineer E. milan.bohare...@netcast.rs M. +381 62 888 13 50

[GitHub] cloudstack pull request: Fix NuageVsp errors for build-master-slow...

Github user DaanHoogland commented on the pull request: https://github.com/apache/cloudstack/pull/1092#issuecomment-158397080 @nlivens .1 : my bad .2 remains but is not specific to this patch and even this plugin. LGTM I asked @remibergsma on slack to have a look as well. --- If

[GitHub] cloudstack pull request: CLOUDSTACK-9062: Improve S3 implementatio...

Github user remibergsma commented on the pull request: https://github.com/apache/cloudstack/pull/1083#issuecomment-158341279 LGTM, based on a set of tests that I run on this branch (which I rebased myself first). Screenshot is from an experimental Jenkins job (that runs the same

Re: [STABILITY]} Large KVM Infrastructure with ACS

First I thought the problem could be a consequence of over commitment, but per your answers I do not see problems with over provisioning of resources. I would check the MS and KVM agent code, but I believe the problem is number of MS, imagine a singles MS to manage, monitor and orchestrate 250+

Re: Request for CloudStack Dev Slack channel access

yab On Fri, Nov 20, 2015 at 11:32 AM, Prashanth Manthena < prashanth.manth...@nuagenetworks.net> wrote: > Hi, > > Can you please grant me access and point me to the CloudStack Dev Slack > channel. > > With regards, > Prashanth > -- Daan

Re: Request for CloudStack Dev Slack channel access

Thank you, Daan :) On Fri, Nov 20, 2015 at 11:47 AM, Daan Hoogland wrote: > yab > > On Fri, Nov 20, 2015 at 11:32 AM, Prashanth Manthena < > prashanth.manth...@nuagenetworks.net> wrote: > > > Hi, > > > > Can you please grant me access and point me to the CloudStack Dev

[GitHub] cloudstack pull request: [4.5] CLOUDSTACK-9042: VR: DHCP: fix miss...

Github user bhaisaab commented on the pull request: https://github.com/apache/cloudstack/pull/1045#issuecomment-158349810 LGTM, @remibergsma did we have the fix on 4.6/master already? --- If your project is set up for it, you can reply to this email and have your reply appear on

[GitHub] cloudstack pull request: Fix NuageVsp errors for build-master-slow...

Github user nlivens commented on the pull request: https://github.com/apache/cloudstack/pull/1092#issuecomment-158394211 @DaanHoogland 1. getBytesUtf8() and newStringUtf8() would no longer be used. 2. With this new code, we won't any longer be only dependent on UTF-8, but

[GitHub] cloudstack pull request: [WIP] CLOUDSTACK-6276: project support in...

Github user ustcweizhou commented on the pull request: https://github.com/apache/cloudstack/pull/508#issuecomment-158450166 @pdube you are right. system/root admin can access every entity, but root admin cannot create a vm using other account' affinity groups.

[GitHub] cloudstack pull request: [WIP] CLOUDSTACK-6276: project support in...

Github user pdube commented on the pull request: https://github.com/apache/cloudstack/pull/508#issuecomment-158434572 @ustcweizhou Thanks for the response. I understand the code. I was wondering why there was a need for a special access checker for AffinityGroups. After more

[GitHub] cloudstack pull request: Cwe 190

Github user rafaelweingartner commented on the pull request: https://github.com/apache/cloudstack/pull/1057#issuecomment-158434570 Hi @DaanHoogland, Do not give up now, I really liked your PR. Those errors can be easily explained. You changed the behavior of

Re: [DISCUSS] ACS 4.5.3 release

Hi Rohit, I've reported one Bug[1], that affects the network.throttling.rate. I assume, this is just some bad calculation. Maybe this could be fixed for 4.5.3? :) [1] https://issues.apache.org/jira/browse/CLOUDSTACK-8936 Am Freitag, den 20.11.2015, 06:27 + schrieb Rohit Yadav: > Hi all,

[GitHub] cloudstack pull request: Cwe 190

Github user DaanHoogland commented on the pull request: https://github.com/apache/cloudstack/pull/1057#issuecomment-158486604 @rafaelweingartner thanks for picking this up. it is not a matter of giving up but of priorities and I didn't want to keep it open knowing there was work to

[GitHub] cloudstack pull request: Cwe 190

GitHub user DaanHoogland reopened a pull request: https://github.com/apache/cloudstack/pull/1057 Cwe 190 coverity warnings of this type adressed. Some where dismissed and maybe with reason but it seemed possible to remove them and hence obligatory ;p You can merge this pull

[GitHub] cloudstack pull request: Cwe 190

Github user rafaelweingartner commented on the pull request: https://github.com/apache/cloudstack/pull/1057#issuecomment-158499809 I understand why you closed the PR, but after a quick review, I thought that I could help you, so let’s see if now everything is ok --- If your

[GitHub] cloudstack pull request: CLOUDSTACK-9055: fix NPE in updating Redu...

Github user asfgit closed the pull request at: https://github.com/apache/cloudstack/pull/1073 --- 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: Shuffling the password to avoid having a ...

Github user remibergsma commented on the pull request: https://github.com/apache/cloudstack/pull/1058#issuecomment-158530260 I will run some tests on this branch and report back. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as

[GitHub] cloudstack pull request: update for comment in cancelMaintin on st...

Github user asfgit closed the pull request at: https://github.com/apache/cloudstack/pull/1065 --- 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:

Github user remibergsma commented on the pull request: https://github.com/apache/cloudstack/commit/3381154fafb7fa4f0a61d538f7c2550e48247787#commitcomment-14536914 In scripts/vm/systemvm/injectkeys.sh: In scripts/vm/systemvm/injectkeys.sh on line 89: This breaks CentOS7, see

[GitHub] cloudstack pull request: CLOUDSTACK-9062: Improve S3 implementatio...

Github user asfgit closed the pull request at: https://github.com/apache/cloudstack/pull/1083 --- 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: [4.5] Metrics view

Github user karuturi commented on the pull request: https://github.com/apache/cloudstack/pull/1091#issuecomment-158326799 I am ok with merging the fix to 4.5+. I am not against that. But, missing one release is not acceptable. Given that its already merged on 4.7, merging in 4.5 now

Build failed in Jenkins: build-4.5 #539

See Changes: [Rohit Yadav] ui: Remove Metrics UI feature -- [...truncated 2923 lines...] [INFO] --- maven-compiler-plugin:2.5.1:compile (default-compile) @ cloud-plugin-network-internallb ---

[GitHub] cloudstack pull request: Cwe 190

Github user DaanHoogland commented on the pull request: https://github.com/apache/cloudstack/pull/1057#issuecomment-158327637 @remibergsma running them myself as well, now --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well.

[GitHub] cloudstack pull request: Fix NuageVsp errors for build-master-slow...

Github user DaanHoogland commented on the pull request: https://github.com/apache/cloudstack/pull/1092#issuecomment-158328407 [1092.test_network.results.txt](https://github.com/apache/cloudstack/files/39874/1092.test_network.results.txt)

[GitHub] cloudstack pull request: L10N : Updated locale files.

Github user DaanHoogland commented on the pull request: https://github.com/apache/cloudstack/pull/1096#issuecomment-158328710 @barunhalder1234 no need to close the PR, you can force push to your branch and it will update the PR --- If your project is set up for it, you can reply to

[GitHub] cloudstack pull request: CLOUDSTACK-9076: Changed ownership of dir...

Github user wido commented on the pull request: https://github.com/apache/cloudstack/pull/1095#issuecomment-158328758 Seems to be a spacing issue? Keep in mind that the debian/rules files uses TABs and not spaces. With space it will fail. --- If your project is set up for it, you

[GitHub] cloudstack pull request: CLOUDSTACK-9062: Improve S3 implementatio...

Github user wido commented on the pull request: https://github.com/apache/cloudstack/pull/1083#issuecomment-158328981 LGTM. Reviewed the code and looks good. --- 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: [4.5] Metrics view

Github user karuturi commented on the pull request: https://github.com/apache/cloudstack/pull/1091#issuecomment-158329652 For future reference: this PR is reverted in commit https://github.com/apache/cloudstack/commit/ea5aecbe07f360bdf5f461c5ea2bce73c3a0e7df --- If your project is

[GitHub] cloudstack-www pull request: add swiss txt to cloudstack users

GitHub user resmo opened a pull request: https://github.com/apache/cloudstack-www/pull/19 add swiss txt to cloudstack users You can merge this pull request into a Git repository by running: $ git pull https://github.com/resmo/cloudstack-www patch-1 Alternatively you can

4.6 release

Guys, I'm out and about this morning, but I've noticed that the release notes state that baseurl for 4.6 is http://cloudstack.apt-get.eu/rhel/4.5/ And that somewhat buried in the vmware install information is the fact that the apt-get repo only includes the oss build. 1. I fundamentally

Build failed in Jenkins: build-master-slowbuild #2674

See -- [...truncated 27717 lines...] [INFO] --- maven-compiler-plugin:3.2:compile (default-compile) @ cloud-quickcloud --- [INFO] No sources to compile [INFO] [INFO] >>>

Re: [STABILITY]} Large KVM Infrastructure with ACS

2 MS is typical. There is also an issue with starting stopped MS. For example if you start 1MS and within next 30 seconds start the second MS (first MS is loading all hosts), there is a chance they will come up - but really not functional. Specifically sync jobs would fail and status updates

[GitHub] cloudstack pull request: L10N : Updated locale files.

Github user barunhalder1234 closed the pull request at: https://github.com/apache/cloudstack/pull/1096 --- 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

RE: [STABILITY]} Large KVM Infrastructure with ACS

Hi Rafael, These are clients of our support or consulting services, therefore I can only ever talk in very broad terms regarding their architectures. One client had a single management server, the other had a pair which were load balanced behind a physical appliance. Regards, Paul Angus VP

[GitHub] cloudstack pull request: L10N : Updated locale files with latest t...

GitHub user barunhalder1234 opened a pull request: https://github.com/apache/cloudstack/pull/1097 L10N : Updated locale files with latest translation. You can merge this pull request into a Git repository by running: $ git pull https://github.com/barunhalder1234/cloudstack

Re: [STABILITY]} Large KVM Infrastructure with ACS

On Thu, Nov 19, 2015 at 9:31 PM, ilya wrote: > Maybe I need to ping LeaseWeb and ExtremePC folks.. > ​We have many smaller installations, nothing near those limits.​ (we being LeaseWeb) -- Daan

Re: SR_BACKEND_FAILURE_61File SR creation error [opterr=fail to mount FileSR. Errno is 32]

Thanks Jayapal, Abhinandan. log SMlog doesnt giveout much, I am able to mount and edit file on mount from both server and agent. I get the same error as follows (timestamp is different because I tried it again), is there anything else I am missing?: Nov 20 16:22:01 agent SM: [5432] ['mount',

Re: Read/Write splitting on database

Is there anyone to answer my question? Thanks. - Original Message - From: Alireza Eskandari To: Dev Sent: Wednesday, November 18, 2015 12:30 PM Subject: Read/Write splitting on database Hi all, According to link below, does