[GitHub] cloudstack pull request: Cwe 190

Github user remibergsma commented on the pull request: https://github.com/apache/cloudstack/pull/1057#issuecomment-158314116 FYI: @DaanHoogland My tests show 29 test failures. I will retest to be sure, but please hold until this is done. --- If your project is set up for it, you can

[GitHub] cloudstack pull request: [4.5] Metrics view

Github user bhaisaab commented on the pull request: https://github.com/apache/cloudstack/pull/1091#issuecomment-158313998 @remibergsma @karuturi this is confusion, we've been backporting changes for long. These are simply UI changes that don't affect backend/core/mgmt server. I would

[GitHub] cloudstack pull request: [4.5] Metrics view

Github user bhaisaab commented on the pull request: https://github.com/apache/cloudstack/pull/1091#issuecomment-158313579 @karuturi I can revert the metrics view feature, but would keep the other css/js improvements wrt sorting etc. --- If your project is set up for it, you can reply

[GitHub] cloudstack pull request: [4.6] metrics view

Github user bhaisaab closed the pull request at: https://github.com/apache/cloudstack/pull/1090 --- 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: L10N : Updated locale files.

Github user DaanHoogland commented on the pull request: https://github.com/apache/cloudstack/pull/1096#issuecomment-158311555 @barunhalder1234 Can you squash these commits, please? --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub a

Build failed in Jenkins: simulator-4.5-singlerun #278

See -- Started by upstream project "build-4.5-simulator" build number 332 originally caused by: Started by upstream project "build-4.5" build number 538 originally caused by: Started by an

[GitHub] cloudstack pull request: [4.5] Metrics view

Github user karuturi commented on the pull request: https://github.com/apache/cloudstack/pull/1091#issuecomment-158295188 Please wait. This lands a feature/fix/enhancement in 4.5 and 4.7 while not in 4.6. This will just confuse the users. This basically means that votes for on

[DISCUSS] ACS 4.5.3 release

Hi all, I want to ask how happy people are with the last 4.5.2 release and if there are any issues they want to report or want to be fixed in a future minor release. If we’ve enough demand, we can work towards a last 4.5 minor release. Thanks. Rohit Yadav Software Architect [cid:image003.png@0

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

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

[GitHub] cloudstack pull request: [4.5] Metrics view

Github user bhaisaab commented on the pull request: https://github.com/apache/cloudstack/pull/1091#issuecomment-158293387 Merging on 4.5 based on no objections and popular demands. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub a

[GitHub] cloudstack pull request: [4.5] Metrics view

Github user asfgit closed the pull request at: https://github.com/apache/cloudstack/pull/1091 --- 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: Metrics views for CloudStack UI

Github user serverchief commented on the pull request: https://github.com/apache/cloudstack/pull/1038#issuecomment-158291609 The metrics UI is not really a major feature in my opinion. We are not changing the core and any apis. All we are doing is re-presenting already present

[GitHub] cloudstack pull request: Metrics views for CloudStack UI

Github user serverchief commented on the pull request: https://github.com/apache/cloudstack/pull/1038#issuecomment-158289806 +1, please merge On 11/18/15 11:03 PM, Rohit Yadav wrote: > Guys can we get some +1s on this, so let's at least merge on master? cc > @wilderrodr

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

GitHub user barunhalder1234 opened a pull request: https://github.com/apache/cloudstack/pull/1096 L10N : Updated locale files. You can merge this pull request into a Git repository by running: $ git pull https://github.com/barunhalder1234/cloudstack master Alternatively you c

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

The error signifies that the creation of file storage repository failed on xenserver, check that the nfs can be mounted and is writable. The log file mentioned by Jayapal will give you further clues to the problem. Regards, Abhinandan Prateek M: +91 9701199011 abhinandan.prat...@shapeblue.co

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

Hi Prakash, Can you also check the xenserver logs at /var/log/SMLog when this error is coming. Thanks, Jayapal > On 20-Nov-2015, at 9:23 am, B Prakash wrote: > > Please can someone help me with this? I am trying to setup NFS secondary > storage on ACS 4.6 hosted on Ubuntu 14.04 with host XS

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

Please can someone help me with this? I am trying to setup NFS secondary storage on ACS 4.6 hosted on Ubuntu 14.04 with host XS 6.5. NFS mount and its access is working fine on both the servers, but when I try to set up a secondary storage, I get this exception; 2015-11-19 13:24:00,841 WARN

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

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

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

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

Re: [STABILITY]} Large KVM Infrastructure with ACS

How many MS do you have in your environment? On Thu, Nov 19, 2015 at 7:56 PM, Paul Angus wrote: > Hi, > > In the past a couple of clients of our have had issues with indirect > agents (KVM hosts and system VMs) connecting over port 8250, particularly > if connectivity was lost to the management

RE: [STABILITY]} Large KVM Infrastructure with ACS

Hi, In the past a couple of clients of our have had issues with indirect agents (KVM hosts and system VMs) connecting over port 8250, particularly if connectivity was lost to the management server(s). They both had 300+ indirect agents active. In these circumstances we have found that running

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

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

[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-158200709 @ustcweizhou Is that actually necessary? --- 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 projec

[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-158200813 Why is that access control done differently? --- 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

Re: [STABILITY]} Large KVM Infrastructure with ACS

Rafael, Please see response in-line: On 11/18/15 4:16 PM, Rafael Weingärtner wrote: > When you say 250+, you mean 250+ host spread in lots of cluster, right? > If I am not mistaken, ACS limits the number of KVM hosts in a cluster, > something like 50? I do not remember now if that value can be co

[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-158164720 @pdube try this change: ``` diff --git a/server/src/com/cloud/acl/AffinityGroupAccessChecker.java b/server/src/com/cloud/acl/AffinityGroupA

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

GitHub user borisroman opened a pull request: https://github.com/apache/cloudstack/pull/1095 CLOUDSTACK-9076: Changed ownership of directory /var/lib/cloudstack to cloud. Now the management server can create directories inside this folder. It's a packaging issue. Testing ca

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

Github user DaanHoogland commented on the pull request: https://github.com/apache/cloudstack/pull/924#issuecomment-158148817 code reviewed LGTM, @remibergsma kan you start a bubble with this? --- If your project is set up for it, you can reply to this email and have your reply appear

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

See Changes: [wrodrigues] CLOUDSTACK-9015 - Make sure the Backup router can talk to the Master [wrodrigues] CLOUDSTACK-9015 - Add test to cover the rVPC routers stop/start/reboot [wrodrigues] CLOUDSTACK-9067 - Remove old

[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-158144546 Before merge, I want to retry the two failed tests and see if they work for me. --- If your project is set up for it, you can reply to this email and have your

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

Github user DaanHoogland commented on the pull request: https://github.com/apache/cloudstack/pull/1089#issuecomment-158141395 [1088.test_network.results.txt](https://github.com/apache/cloudstack/files/39300/1088.test_network.results.txt) [1088.test_vpc.results.txt](https://github.

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

GitHub user nvazquez opened a pull request: https://github.com/apache/cloudstack/pull/1094 CLOUDSTACK-9074: Support shared networking in NiciraNVP Plugin JIRA TICKET: https://issues.apache.org/jira/browse/CLOUDSTACK-9074 Design Document: https://cwiki.apache.org/c

[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-158136481 @fmaximus preffered charset will be utf8 if it is available in the virtual machine (afaik always) otherwise it is the platform default --- If your project is

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

Github user jlk commented on the pull request: https://github.com/apache/cloudstack/pull/1089#issuecomment-158127621 LGTM, thanks Daan! --- 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] cloudstack pull request: CLOUDSTACK-9067 - As I developer I want t...

Github user asfgit closed the pull request at: https://github.com/apache/cloudstack/pull/1084 --- 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: CLOUDSTACK-9067 - As I developer I want t...

Github user remibergsma commented on the pull request: https://github.com/apache/cloudstack/pull/1084#issuecomment-158117634 I also agree we shouldn't keep unused files in the active tree. We already got some PRs against files that were not in use and that is a waste of time for every

[GitHub] cloudstack pull request: [4.6] metrics view

Github user remibergsma commented on the pull request: https://github.com/apache/cloudstack/pull/1090#issuecomment-158117109 Merged to master, cannot merge to 4.6 as it would conflict when forward merging. It will be in 4.7. --- If your project is set up for it, you can reply to this

[GitHub] cloudstack pull request: [4.6.1] CLOUDSTACK-9015 - Redundant VPC V...

Github user asfgit closed the pull request at: https://github.com/apache/cloudstack/pull/1070 --- 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: [4.6.1] CLOUDSTACK-9015 - Redundant VPC V...

Github user remibergsma commented on the pull request: https://github.com/apache/cloudstack/pull/1070#issuecomment-158116184 LGTM based on these tests: ``` nosetests --with-marvin --marvin-config=${marvinCfg} -s -a tags=advanced,required_hardware=true \ component/test_

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

Github user fmaximus commented on the pull request: https://github.com/apache/cloudstack/pull/1092#issuecomment-158100736 I think we should use StringUtils.getUtf8Bytes() instead, as we are using StringUtils.newStringUtf8(Base64.decodeBase64(cmsUserPassBase64)) on the other side (Nuag

[Marvin] Failed to wget from VM in "/test/integration/component/test_vpc_network_staticnatrule.py"

Hi All, I get an error "Failed to wget from VM..." while running the Marvin test "/test/integration/component/test_vpc_network_staticnatrule.py" with CentOS 5.5 template (KVM hypervisor). I understand that a http server service needs to be initialized/installed in the VM template for this test to

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

See Changes: [Rohit Yadav] CLOUDSTACK-9020: Add new status icons and css rules [Rohit Yadav] CLOUDSTACK-9020: Method to remove last panel from the breadcrumb [Rohit Yadav] CLOUDSTACK-9020: Implement sorting for tables [Ro

[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-158075714 @ustcweizhou I am working on it now, but have a few questions about resource ownership --- If your project is set up for it, you can reply to this email and have your

Re: distribution of Cloudstack API plugins

Great! Glad to hear it's working now. :) On Thursday, November 19, 2015, Schubert, Sven wrote: > Hello Mike, > > I successfully created a plugin by following your instructions from the > video :). I guess the problem was that working on 4.5.2 was a bad idea, as > the poms no longer have the mave

[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-158062783 @wilderrodrigues Fixed your comments, refactoring is for the next round ;) --- If your project is set up for it, you can reply to this email and have your reply

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

Github user remibergsma commented on the pull request: https://github.com/apache/cloudstack/pull/1072#issuecomment-158053349 LGTM, did an upgrade from `4.4.4` to `4.6.1-SNAPSHOT` and that worked now with only the `4.6` template registered. This was done with XenServer. ```

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

Github user nnesic commented on the pull request: https://github.com/apache/cloudstack/pull/924#issuecomment-158052503 Hello, how is the review on this going? --- 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] cloudstack pull request: [4.6.1] CLOUDSTACK-9015 - Redundant VPC V...

Github user wilderrodrigues commented on the pull request: https://github.com/apache/cloudstack/pull/1070#issuecomment-158051578 @remibergsma: some time to re-run the tests? Need a 2nd LGTM. --- If your project is set up for it, you can reply to this email and have your reply appear o

[GitHub] cloudstack pull request: CLOUDSTACK-9042: remove unused edithosts....

Github user wilderrodrigues commented on the pull request: https://github.com/apache/cloudstack/pull/1093#issuecomment-158050587 @resmo: that file is used by HyperV. I have refactored it and removed the files that have to be removed. Have a look at PR #1084 It was difficult t

[GitHub] cloudstack pull request: CLOUDSTACK-9067 - As I developer I want t...

Github user wilderrodrigues commented on the pull request: https://github.com/apache/cloudstack/pull/1084#issuecomment-158049050 @jayapalu, I'm with @borisroman on that one. We are no longer using shared file systems to keep files stored. We have a good version control system and the

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

Github user wilderrodrigues commented on the pull request: https://github.com/apache/cloudstack/pull/1083#issuecomment-158048085 @borisroman @wido I made some comments in the PR. Those are not blocking, but would be very fruitful to get them in place in a second round of the refactor.

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

Github user wilderrodrigues commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1083#discussion_r45335913 --- Diff: core/src/com/cloud/storage/template/S3TemplateDownloader.java --- @@ -342,47 +274,38 @@ public InputStream getS3ObjectInputStream() {

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

Github user wilderrodrigues commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1083#discussion_r45335845 --- Diff: core/src/com/cloud/storage/template/S3TemplateDownloader.java --- @@ -19,303 +19,235 @@ package com.cloud.storage.template;

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

Github user wilderrodrigues commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1083#discussion_r45335326 --- Diff: api/src/org/apache/cloudstack/api/command/admin/storage/AddImageStoreS3CMD.java --- @@ -201,15 +209,15 @@ public int hashCode() {

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

Github user wilderrodrigues commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1083#discussion_r45335015 --- Diff: api/src/com/cloud/agent/api/to/S3TO.java --- @@ -155,6 +161,7 @@ public int hashCode() { result = 31 * result + (secretKey

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

Github user wilderrodrigues commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1083#discussion_r45335103 --- Diff: api/src/org/apache/cloudstack/api/command/admin/storage/AddImageStoreS3CMD.java --- @@ -119,17 +129,16 @@ public void execute() throws

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

Github user wilderrodrigues commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1083#discussion_r45334945 --- Diff: api/src/com/cloud/agent/api/to/S3TO.java --- @@ -51,8 +52,8 @@ public S3TO() { } public S3TO(final Long id, fina

[GitHub] cloudstack pull request: CLOUDSTACK-9042: remove unused edithosts....

GitHub user resmo opened a pull request: https://github.com/apache/cloudstack/pull/1093 CLOUDSTACK-9042: remove unused edithosts.sh As discovered in CLOUDSTACK-9042, this script contains some issues but since it was replaced since 4.6, we remove it on master. Also see #1

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

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

[GitHub] cloudstack pull request: [4.6.1] CLOUDSTACK-9015 - Redundant VPC V...

Github user borisroman commented on the pull request: https://github.com/apache/cloudstack/pull/1070#issuecomment-158036108 @miguelaferreira @remibergsma @wilderrodrigues LGTM :+1: ``` nosetests --with-marvin --marvin-config=/data/shared/marvin/mct-zone1-kvm1-ISOLATED.cf

[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-158033304 thanks @nlivens lgtm but will run regression tests. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as w

[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-158030442 Our test case on ACS 4.5.1: # Before fix ## vanilla router deployed: ~~~ # related script ae693b5ef2d9b1fed1307f6b38213823 /opt/cloud/bin/edit

[GitHub] cloudstack pull request: Metrics views for CloudStack UI

Github user asfgit closed the pull request at: https://github.com/apache/cloudstack/pull/1038 --- 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: Metrics views for CloudStack UI

Github user bhaisaab commented on the pull request: https://github.com/apache/cloudstack/pull/1038#issuecomment-158027141 @remibergsma alright, merging on master now. If there is no objection, I would like to merge this on 4.5 as whole, if that's not acceptable at least the other usef

[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-158026133 @DaanHoogland, this PR contains the fixes for the build-master-slowbuild --- If your project is set up for it, you can reply to this email and have your reply appea

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

GitHub user nlivens opened a pull request: https://github.com/apache/cloudstack/pull/1092 Fix NuageVsp errors for build-master-slowbuild You can merge this pull request into a Git repository by running: $ git pull https://github.com/nlivens/cloudstack master Alternatively you

[GitHub] cloudstack pull request: Metrics views for CloudStack UI

Github user borisroman commented on the pull request: https://github.com/apache/cloudstack/pull/1038#issuecomment-158021643 @remibergsma Agree. --- 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

[GitHub] cloudstack pull request: CLOUDSTACK-9067 - As I developer I want t...

Github user jayapalu commented on the pull request: https://github.com/apache/cloudstack/pull/1084#issuecomment-158020225 @wilderrodrigues @karuturi VR scripts got stabilised over years. If any thing wrong in new implementation there is chance to look into the VR scripts and

[GitHub] cloudstack pull request: Metrics views for CloudStack UI

Github user remibergsma commented on the pull request: https://github.com/apache/cloudstack/pull/1038#issuecomment-158021203 @bhaisaab I'm OK with merging this to master now, but that will also mean it will not end up in 4.6. I think @DaanHoogland already said that no new features sho

[GitHub] cloudstack pull request: CLOUDSTACK-9067 - As I developer I want t...

Github user borisroman commented on the pull request: https://github.com/apache/cloudstack/pull/1084#issuecomment-158020779 @jayapalu That's we're git's versioning system comes into play. We want to keep the current working directory as clean as possible. Meaning, without unused/dead

[GitHub] cloudstack pull request: Metrics views for CloudStack UI

Github user bhaisaab commented on the pull request: https://github.com/apache/cloudstack/pull/1038#issuecomment-158018041 @karuturi Remi asked for a 4.6 based PR. Given some votes and if we don't have an objections should I go ahead and merge on master? --- If your project is set up

[GitHub] cloudstack pull request: Metrics views for CloudStack UI

Github user karuturi commented on the pull request: https://github.com/apache/cloudstack/pull/1038#issuecomment-158017542 Hi @bhaisaab you should create PRs for (4.5 and 4.6) OR (master). Not for all the branches. Based on the discussion and votes above I think the decision w

[GitHub] cloudstack pull request: CLOUDSTACK-9067 - As I developer I want t...

Github user borisroman commented on the pull request: https://github.com/apache/cloudstack/pull/1084#issuecomment-158017282 @wilderrodrigues LGTM :+1: Always in support of removing old/dead code! :) Testing: CentOS 7.1 Setup Managment/Hypervisor ```

[GitHub] cloudstack pull request: CLOUDSTACK-9067 - As I developer I want t...

Github user wilderrodrigues commented on the pull request: https://github.com/apache/cloudstack/pull/1084#issuecomment-158015787 Ping @DaanHoogland @miguelaferreira @karuturi @borisroman @bhaisaab @remibergsma More test results, guys! It looks good to me... but my vote doesn'

[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-158012460 @wilderrodrigues I added some unit tests. you can test it via api: command=updateVmNicIp&response=json&nicId=92ad2e8b-f8d8-4aec-a975-16ee1e4ef0

[GitHub] cloudstack pull request: Metrics views for CloudStack UI

Github user bhaisaab commented on the pull request: https://github.com/apache/cloudstack/pull/1038#issuecomment-158008482 @remibergsma rebased again the branch against latest master, created PRs; 4.6: #1090 4.5: #1091 --- If your project is set up for it, you can reply to th

[GitHub] cloudstack pull request: [4.5] Metrics view

GitHub user bhaisaab opened a pull request: https://github.com/apache/cloudstack/pull/1091 [4.5] Metrics view From #1038, for 4.5 branch You can merge this pull request into a Git repository by running: $ git pull https://github.com/shapeblue/cloudstack metrics-4.5 Alternative

[GitHub] cloudstack pull request: [4.6] metrics view

GitHub user bhaisaab opened a pull request: https://github.com/apache/cloudstack/pull/1090 [4.6] metrics view From #1038, for 4.6 branch You can merge this pull request into a Git repository by running: $ git pull https://github.com/shapeblue/cloudstack 4.6-metrics-master Alte

[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-158006677 @pdube If you are almost done, I will wait for your PR. Otherwise, I can work on porting it. --- If your project is set up for it, you can reply to this email a

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

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

[GitHub] cloudstack pull request: CLOUDSTACK-9067 - As I developer I want t...

Github user wilderrodrigues commented on the pull request: https://github.com/apache/cloudstack/pull/1084#issuecomment-157994184 Thanks, @karuturi! I'm testing against XenServer 6.2 now. Given the changes on the HyperV Resource class, I'm not expecting problem there. The chang

Re: [Proposal] Template for CloudStack API Reference Pages

Hi, Yiping, Thanks for your suggestions. I have updated the template based on your suggestions. Please find attached the updated template for your reference. I agree with your suggestion on proof-reading the descriptions. This is an important part of ensuring effective API references. I think the

RE: distribution of Cloudstack API plugins

Hello Mike, I successfully created a plugin by following your instructions from the video :). I guess the problem was that working on 4.5.2 was a bad idea, as the poms no longer have the maven SNAPSHOT tag in their version. Working on a fresh branch from the 4.6.0 master worked perfectly. Rega

[GitHub] cloudstack pull request: Metrics views for CloudStack UI

Github user abhinandanprateek commented on the pull request: https://github.com/apache/cloudstack/pull/1038#issuecomment-157988733 +1 have tried the feature and used bits of it in revamped quota UI. --- If your project is set up for it, you can reply to this email and have your reply

Re: [Proposal] Template for CloudStack API Reference Pages

Thanks. Format is to domonstrate the usage of the API. I think that the format along with an example would help users understand how they can use the API. For example, the migrateto parameter in the API has the following format: migrateto[volume-index].volume=&migrateto[volume-index].pool=Where,

[GitHub] cloudstack pull request: Metrics views for CloudStack UI

Github user wido commented on the pull request: https://github.com/apache/cloudstack/pull/1038#issuecomment-157987945 I would like to give a LGTM, but my JavaScript skills aren't that good. The small Java changes look good to me. --- If your project is set up for it, you can

[GitHub] cloudstack pull request: Metrics views for CloudStack UI

Github user NuxRo commented on the pull request: https://github.com/apache/cloudstack/pull/1038#issuecomment-157983748 +1 to merge on master --- 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 th