[GitHub] cloudstack issue #1626: [blocker] Fix systemvm template build

2016-08-04 Thread karuturi
Github user karuturi commented on the issue: https://github.com/apache/cloudstack/pull/1626 Can you please create a JIRA ticket for this? --- 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 issue #1612: packaging: Marvin and integration-tests packages

2016-08-04 Thread karuturi
Github user karuturi commented on the issue: https://github.com/apache/cloudstack/pull/1612 Can you please create a JIRA ticket for this? --- 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 issue #1612: packaging: Marvin and integration-tests packages

2016-08-04 Thread karuturi
Github user karuturi commented on the issue: https://github.com/apache/cloudstack/pull/1612 @rhtyd will test this tomorrow and commit --- 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 issue #1625: [blocker] cloudstack: fix upgrade paths to 4.10.0

2016-08-04 Thread karuturi
Github user karuturi commented on the issue: https://github.com/apache/cloudstack/pull/1625 yup... started testing.. in progress.. will be done probably in 30 min.. you started automation run.. should we wait for that? --- If your project is set up for it, you can reply to this

[GitHub] cloudstack issue #1625: [blocker] cloudstack: fix upgrade paths to 4.10.0

2016-08-04 Thread karuturi
Github user karuturi commented on the issue: https://github.com/apache/cloudstack/pull/1625 Where did you see the OOM error? I dont see it in travis logs (this is for the previous travis job) https://s3.amazonaws.com/archive.travis-ci.org/jobs/149684289/log.txt --- If your

[GitHub] cloudstack issue #1625: [blocker] cloudstack: fix upgrade paths to 4.10.0

2016-08-04 Thread karuturi
Github user karuturi commented on the issue: https://github.com/apache/cloudstack/pull/1625 @blueorangutan the link doesnt work --- 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

[GitHub] cloudstack issue #1625: [blocker] cloudstack: fix upgrade paths to 4.10.0

2016-08-04 Thread karuturi
Github user karuturi commented on the issue: https://github.com/apache/cloudstack/pull/1625 code looks good. I am seeing the following travis errors `[ERROR] Failed to execute goal on project travis-build-deps: Could not resolve dependencies for project

Re: 4.10.0 release

2016-08-03 Thread Rajani Karuturi
September 2016 * RC Voting: 19 - 25 September 2016 * Release: 26 September 2016 master is open for 4.10.0. It still means that only PRs will be merged and they will be merged only by RMs ( For 4.10.0, its John Burwell and Rajani Karuturi) Every PR should have a JIRA bug ID, 1 code review and 1 test

4.10.0 release

2016-08-03 Thread Rajani Karuturi
open for 4.10.0. It still means that only PRs will be merged and they will be merged only by RMs ( For 4.10.0, its John Burwell and Rajani Karuturi)Every PR should have a JIRA bug ID, 1 code review and 1 test review.It would help in reviewing if the contributor could put information about the feature

Re: [PROPOSAL] Early LTS Initial Release

2016-08-02 Thread Rajani Karuturi
) john.burw...@shapeblue.com ( john.burw...@shapeblue.com ) www.shapeblue.com ( http://www.shapeblue.com )53 Chandos Place, Covent Garden, London VA WC2N 4HSUK@shapeblue On Aug 2, 2016, at 3:57 AM, Rajani Karuturi wrote: We already maintain the release branches and do regularreleases(of the past two

[GitHub] cloudstack issue #1621: Fix a little issue from PR1610 if the db.properties ...

2016-08-02 Thread karuturi
Github user karuturi commented on the issue: https://github.com/apache/cloudstack/pull/1621 We dont need a PR for master. It will be forward merged. If you can create a jira ticket for this (even better would be to reuse the one used in PR1610), it will be useful. --- If your

Re: [PROPOSAL] Early LTS Initial Release

2016-08-02 Thread Rajani Karuturi
We already maintain the release branches and do regular releases(of the past two releases) on them. What are we achieving through this LTS model? How is 4.9.1 different from 4.9.0.0_1.0? ~ Rajanihttp://cloudplatform.accelerite.com/ On August 2, 2016 at 2:33 AM, John Burwell (john.burw...@shapeblue

[GitHub] cloudstack issue #1616: Added missing rules on router config, fixed ordering...

2016-07-26 Thread karuturi
Github user karuturi commented on the issue: https://github.com/apache/cloudstack/pull/1616 @swill missing required LGTMs.. --- 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

Re: Starting jetty on another port

2016-07-18 Thread Rajani Karuturi
Try changing in client/pom.xmlhttps://github.com/apache/cloudstack/blob/master/client/pom.xml#L459 ~ Rajanihttp://cloudplatform.accelerite.com/ On July 19, 2016 at 1:11 AM, Syed Mushtaq (syed1.mush...@gmail.com) wrote:Hi, Is there a way to start Jetty on a port other than :8080? I've tried with mvn

[GitHub] cloudstack issue #1600: Support Backup of Snapshots for Managed Storage

2016-07-13 Thread karuturi
Github user karuturi commented on the issue: https://github.com/apache/cloudstack/pull/1600 BTw, does this work only with xenserver? I dont see any vmware related changes --- 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 issue #1600: Support Backup of Snapshots for Managed Storage

2016-07-13 Thread karuturi
Github user karuturi commented on the issue: https://github.com/apache/cloudstack/pull/1600 `locationType` Standard and Archive refer to either secondary or primary based on the storage type and is confusing(atleast for me). Can we make this simple and use the locationType as primary

Re: JIRA Permission

2016-07-12 Thread Rajani Karuturi
I added you to the contributors list. you should have the required permissions now. ~Rajani On Fri, Jul 8, 2016 at 2:37 PM, Priyank Parihar < priyank.pari...@accelerite.com> wrote: > Hi, > I am unable to assign issues (e.g. - CLOUDSTACK-9104) to me. Please give > me permission. > > -- > Thanks &

[GitHub] cloudstack pull request #1600: Support Backup of Snapshots for Managed Stora...

2016-07-12 Thread karuturi
Github user karuturi commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1600#discussion_r70563575 --- Diff: api/src/org/apache/cloudstack/api/response/SnapshotResponse.java --- @@ -82,6 +82,10 @@ @Param(description = "valid types are h

[GitHub] cloudstack pull request #1600: Support Backup of Snapshots for Managed Stora...

2016-07-12 Thread karuturi
Github user karuturi commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1600#discussion_r70425148 --- Diff: api/src/org/apache/cloudstack/api/command/user/snapshot/CreateSnapshotCmd.java --- @@ -195,24 +196,37 @@ public void create() throws

[GitHub] cloudstack issue #1610: packaging: Add db.X.driver=jdbc:mysql to db.properti...

2016-07-12 Thread karuturi
Github user karuturi commented on the issue: https://github.com/apache/cloudstack/pull/1610 👍LGTM --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if

[GitHub] cloudstack issue #1607: SSVM downloader now handles redirects properly.

2016-07-10 Thread karuturi
Github user karuturi commented on the issue: https://github.com/apache/cloudstack/pull/1607 changes looks good. It would be great if a test case can be added. --- 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-docs-admin pull request #39: corrected a typo

2016-06-29 Thread karuturi
GitHub user karuturi opened a pull request: https://github.com/apache/cloudstack-docs-admin/pull/39 corrected a typo change the default openldap user object configuration value from interorgperson to inetorgperson You can merge this pull request into a Git repository by running

Re: ApiXmlDocReader.java - Used by anyone???

2016-06-16 Thread Rajani Karuturi
I dont know how but, I think marvin depends on it to generate command objects for the apis. Here is a doc from CodeGenerator.py Apache CloudStack- marvin python classes can be generated from the json returned by API discovery or from the xml spec of commands generated by the ApiDocWriter. This cl

Re: 4.9+ release

2016-06-16 Thread Rajani Karuturi
nt Garden, London VA WC2N 4HSUK > @shapeblue > > > On Jun 15, 2016, at 5:31 AM, Rajani Karuturi wrote: > > > > I like this discussion. But, my original question was not about what > should > > the next release number be? > > > > i was ch

Re: Release-Management Question about New API Plug-in

2016-06-15 Thread Rajani Karuturi
"We will only fix bugs in this release branch, no back porting of new features" - https://cwiki.apache.org/confluence/display/CLOUDSTACK/Release+principles+for+Apache+CloudStack+4.6+and+up I would prefer a PR on master for 4.9+ ~Rajani On Thu, Jun 16, 2016 at 7:12 AM, Tutkowski, Mike wrote:

Re: 4.9+ release

2016-06-15 Thread Rajani Karuturi
den, London VA WC2N 4HSUK > > > @shapeblue > > > > > > > > > On Jun 14, 2016, at 2:02 PM, Paul Angus > > wrote: > > >> > > >> +1 Daan. > > >> > > >> My recollection was that major version number changes w

4.9+ release

2016-06-14 Thread Rajani Karuturi
4.10 or 5.0? We are in the 4.* release cycle from a long time. Just wanted to check if anyone is planning on major changes which warrants 5.0 ~Rajani

Re: RM after 4.9

2016-05-30 Thread Rajani Karuturi
.@shapeblue.com > www.shapeblue.com > 53 Chandos Place, Covent Garden, London VA WC2N 4HSUK > @shapeblue > > > On May 27, 2016, at 12:00 PM, Tutkowski, Mike > wrote: > > > > Thanks, Rajani! :) > > > > From: Rajani K

Re: RM after 4.9

2016-05-27 Thread Rajani Karuturi
I volunteer to be the RM for next release Thanks On 03-May-2016 4:13 AM, "Will Stevens" wrote: > Hello All, > As you all know, I am the RM for the 4.9 release. Unfortunately, this is > not a position I will be able to maintain for the long term, so we should > start this discussion sooner rathe

[GitHub] cloudstack pull request: CLOUDSTACK-9238: Fix URL length to 2048 f...

2016-05-27 Thread karuturi
Github user karuturi commented on the pull request: https://github.com/apache/cloudstack/pull/1567#issuecomment-222099870 ahh.. my bad.. thanks for the clarification. so, we need to introduce a new CommandType.URL for api parameters and use java.net.URL in VOs. LGTM in

[GitHub] cloudstack pull request: CLOUDSTACK-9238: Fix URL length to 2048 f...

2016-05-27 Thread karuturi
Github user karuturi commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1567#discussion_r64875174 --- Diff: setup/db/db/schema-471to480.sql --- @@ -27,3 +27,4 @@ ALTER TABLE `cloud`.`image_store` MODIFY COLUMN `url` varchar(2048); ALTER TABLE

[GitHub] cloudstack pull request: CLOUDSTACK-9238: Fix URL length to 2048 f...

2016-05-27 Thread karuturi
Github user karuturi commented on the pull request: https://github.com/apache/cloudstack/pull/1567#issuecomment-222093757 Why dont we change the default length in [GenericDaoBase](https://github.com/apache/cloudstack/blob/master/framework/db/src/com/cloud/utils/db

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

2016-05-26 Thread karuturi
Github user karuturi commented on the pull request: https://github.com/apache/cloudstack/pull/1562#issuecomment-221825423 commit 5c35bd1b3e0184842ed80a9c465549c2a61f5fd9 copied all the data from template_host_ref to template_store_ref --- If your project is set up for it, you can

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

2016-05-26 Thread karuturi
Github user karuturi commented on the pull request: https://github.com/apache/cloudstack/pull/1471#issuecomment-221824946 LGTM 👍(code review only) --- 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] cloudstack pull request: CLOUDSTACK-9238: Change template_host_ref...

2016-05-26 Thread karuturi
Github user karuturi commented on the pull request: https://github.com/apache/cloudstack/pull/1562#issuecomment-221821581 This table is not used anymore. the table should be dropped. --- If your project is set up for it, you can reply to this email and have your reply appear on

[GitHub] cloudstack pull request: dynamic-roles: packaging improvements

2016-05-19 Thread karuturi
Github user karuturi commented on the pull request: https://github.com/apache/cloudstack/pull/1551#issuecomment-220273077 👍 (code review only) --- 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

Re: How to fix Jenkins git workspace issues

2016-05-18 Thread Rajani Karuturi
Can we add git clean -fdx before the the checkout? That would clean all the untracked files from the git workspace. On Wed, May 18, 2016 at 10:32 PM Will Stevens wrote: > I have seen quite a few situations where tests are failing and they are > referencing files that are not even included in th

Re: hidden configuration items revisited

2016-05-09 Thread Rajani Karuturi
'Hidden' and 'Secure' are both encrypted in db only difference being hidden values are not shown. you could just change the category in configuration table. ~Rajani On Mon, May 9, 2016 at 10:56 PM, Erik Weber wrote: > listConfiguration returns unencrypted values for Secure items, but they > nee

Re: I cannot start agent - Cloudstack 4.8 - Connection closed with -1 on reading size

2016-05-02 Thread Rajani Karuturi
also check java version and upgrade nss http://blog.backslasher.net/java-ssl-crash.html ~Rajani On Tue, May 3, 2016 at 7:36 AM, Simon Weller wrote: > Ellie, > > Can you try forcing ACS Management to only bind to IPV4? > Normally this is done in the JAVA_OPTS by setting > -Djava.net.preferIPv4St

Re: ACS PRs Status - 2016/05/02

2016-05-02 Thread Rajani Karuturi
great job Simon David and Will. Given that we dont have github labels, this is very useful for the RM to pickup the prs to merge and for others to review pending. ~Rajani On Tue, May 3, 2016 at 9:47 AM, Simon Weller wrote: > All, > > Will mentioned a little project we've been working on to try

Re: Incorrect system vm templates for master build.

2016-04-28 Thread Rajani Karuturi
ah..makes sense. Thanks Erik. I will revert the config change. ~Rajani On Thu, Apr 28, 2016 at 4:11 PM, Erik Weber wrote: > AFAIK it hasn't been necessary to update the systemvm template yet, that is > probably why it still shows as 4.6.0. > > -- > Erik > > On Thu, Apr 28, 2016 at 12:04 PM, Bha

Re: Incorrect system vm templates for master build.

2016-04-28 Thread Rajani Karuturi
In the job configuration, I see the version as 4.6.0. I am not sure if that is where it reads from. I changed it to 4.9.0 - export version=4.6.0 + export version=4.9.0 I will keep an eye on the job. Let me know if there are any issues with this change. ~Rajani On Thu, Apr 28, 2016 at 3:34 PM,

Re: [ANNOUNCE] Open source distributed virtual machine scheduling platform

2016-04-26 Thread Rajani Karuturi
congratulations guys.. ~Rajani On Wed, Apr 27, 2016 at 2:29 AM, Gabriel Beims Bräscher < gabrasc...@gmail.com> wrote: > Hello CloudStack community members (@dev and @users), > > This email is meant to announce the publication of a project on Github that > provides a distributed virtual machine s

[GitHub] cloudstack pull request: CLOUDSTACK-9100: ISO.CREATE/TEMPLATE.CREA...

2016-04-20 Thread karuturi
Github user karuturi commented on the pull request: https://github.com/apache/cloudstack/pull/1157#issuecomment-212760624 LGTM based on code review :+1: --- 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

Re: Apache Jira rights for the ACS project

2016-04-19 Thread Rajani Karuturi
you should be able to assign issues now. ~Rajani On Wed, Apr 20, 2016 at 2:19 AM, Simon Weller wrote: > > > >From: Erik Weber > >Sent: Tuesday, April 19, 2016 3:44 PM > >To: dev > >Subject: Re: Apache Jira rights for the ACS project > > > >>On Tue, Apr 1

Re: [JIRA] cleanup

2016-04-12 Thread Rajani Karuturi
Thanks Pierre-Luc. ~Rajani On Wed, Apr 13, 2016 at 8:50 AM, Pierre-Luc Dion wrote: > Hi, > > I've "released" in Jira our version that got released over the last few > months, so now the only unreleased versions are: 4.9.0, 4.8.1, 4.7.2. > > I've archived release that I don't think we will relea

Re: Package Repositories

2015-11-30 Thread Rajani Karuturi
+1 on the categories. ~Rajani On Mon, Nov 30, 2015 at 3:38 PM, sebgoa wrote: > Hi folks, we need to resolve this. > > 1-But I have to start with one comment: > Apache open office releases binaries, users don't compile from source. So > it is possible within ASF to officially release binaries. >

[GitHub] cloudstack pull request: CLOUDSTACK-9092: L10n fix in "Add LDAP Ac...

2015-11-29 Thread karuturi
GitHub user karuturi opened a pull request: https://github.com/apache/cloudstack/pull/1139 CLOUDSTACK-9092: L10n fix in "Add LDAP Account page" fixed two strings on the "Add LDAP Account page" After the change: (with a test string in messages_j

Re: Package Repositories

2015-11-26 Thread Rajani Karuturi
+1 to what Remi said. One source for packages on a generic domain with everyone in community trying to make it better. And, please do not mention different domains for systemvm templates in documentation. That is even more confusing. ~ Rajani -Original Message- From: Nux! Repl

[GitHub] cloudstack pull request: CLOUDSTACK-9020: UI enhancements from met...

2015-11-26 Thread karuturi
Github user karuturi commented on the pull request: https://github.com/apache/cloudstack/pull/1106#issuecomment-159901255 I think #1119 can be closed as this PR will be fwd-merged --- 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-9020: UI enhancements from met...

2015-11-26 Thread karuturi
Github user karuturi commented on the pull request: https://github.com/apache/cloudstack/pull/1106#issuecomment-159901047 verified on 4.6.. working fine.. ![screen shot 2015-11-26 at 5 59 33 pm](https://cloud.githubusercontent.com/assets/186833/11423099/8a583474-9467-11e5-9fad

[GitHub] cloudstack pull request: Fix event UUIDS missing on event bus

2015-11-26 Thread karuturi
Github user karuturi commented on the pull request: https://github.com/apache/cloudstack/pull/1127#issuecomment-159889944 Thanks @ProjectMoon :+1: from me based on my previous tests. ping @remibergsma and @DaanHoogland to merge --- If your project is set up for it, you can

[GitHub] cloudstack pull request: Fix event UUIDS missing on event bus

2015-11-26 Thread karuturi
Github user karuturi commented on the pull request: https://github.com/apache/cloudstack/pull/#issuecomment-159884682 No cherry-picking. The PR should be only for 4.6 which will be fwd-merged to master(and hence will be available in 4.7 and up) I wish it was as easy as

[GitHub] cloudstack pull request: Fix event UUIDS missing on event bus

2015-11-26 Thread karuturi
Github user karuturi commented on the pull request: https://github.com/apache/cloudstack/pull/#issuecomment-159883325 I just realised the target branch should be 4.6 meaning the PR should be for 4.6 and fwd-merged to master. Can you take care of that please @ProjectMoon --- If

[GitHub] cloudstack pull request: Change references to people.apache.org to...

2015-11-25 Thread karuturi
Github user karuturi commented on the pull request: https://github.com/apache/cloudstack/pull/1123#issuecomment-159816344 @srikanteswartalluri 1. no direct commits to master please.. 2. as per the review process, wait for two lgtms before doing a merge (and use the git pr

Re: [NOTICE] people.apache.org web space is moving to home.apache.org

2015-11-25 Thread Rajani Karuturi
We have references to people.apache.org in our testdata. ~Rajani On Wed, Nov 25, 2015 at 5:50 PM, Daniel Gruno wrote: > Hi folks, > as the subject says, people.apache.org is being decommissioned soon, and > personal web space is being moved to a new home, aptly named > home.apache.org ( https:/

[GitHub] cloudstack-www pull request: downloads: State packages on download...

2015-11-25 Thread karuturi
Github user karuturi commented on the pull request: https://github.com/apache/cloudstack-www/pull/20#issuecomment-159589449 @abhinandanprateek Please dont get me wrong. This is not a question of good or bad. I am not denying anyone's part in community neither am I saying that

[GitHub] cloudstack-www pull request: downloads: State packages on download...

2015-11-25 Thread karuturi
Github user karuturi commented on the pull request: https://github.com/apache/cloudstack-www/pull/20#issuecomment-159583159 well, it got evolved during the 4.0 release and no one objected. May be, first we should start a discuss/vote first on what can we include

[GitHub] cloudstack-www pull request: downloads: State packages on download...

2015-11-25 Thread karuturi
Github user karuturi commented on the pull request: https://github.com/apache/cloudstack-www/pull/20#issuecomment-159576804 @jburwell apt-get is just a temporary spot until we get to packages.cloudstack.apache.org yes, we should mention that these are not official but just the

[GitHub] cloudstack-www pull request: downloads: State packages on download...

2015-11-25 Thread karuturi
Github user karuturi commented on the pull request: https://github.com/apache/cloudstack-www/pull/20#issuecomment-159576408 @abhinandanprateek I think they were added silently in the first place. We are doing one rpm build as a community for user convenience(which was

[GitHub] cloudstack pull request: Fix event UUIDS missing on event bus

2015-11-25 Thread karuturi
Github user karuturi commented on the pull request: https://github.com/apache/cloudstack/pull/#issuecomment-159568824 @remibergsma @DaanHoogland can you take a look at this please? Its a good fix to have in 4.6.1 I didnt run the test suite. But, manually tested this fix

[GitHub] cloudstack-www pull request: downloads: State packages on download...

2015-11-25 Thread karuturi
Github user karuturi commented on the pull request: https://github.com/apache/cloudstack-www/pull/20#issuecomment-159568400 I agree with @remibergsma -1 for adding more repos which community may not have control. This will definitely confuse users. Since these are build from the

[GitHub] cloudstack pull request: Fix event UUIDS missing on event bus

2015-11-24 Thread karuturi
Github user karuturi commented on the pull request: https://github.com/apache/cloudstack/pull/#issuecomment-159519154 manually tested it with rabbitmq integration Here are the events before and after the fix Before: {"eventDateTime":"2015-11-25

[GitHub] cloudstack pull request: CLOUDSTACK-9081 Migration of vm across cl...

2015-11-24 Thread karuturi
GitHub user karuturi opened a pull request: https://github.com/apache/cloudstack/pull/1118 CLOUDSTACK-9081 Migration of vm across clusters fails in clustered MS setup Storage motion of vm across clusters/xenserver-pools fails in a clustered management server setup. In xen

Re: New Defects reported by Coverity Scan for cloudstack

2015-11-24 Thread Rajani Karuturi
eported by Coverity Scan for cloudstack >On Tue, Nov 24, 2015 at 7:46 AM, Rajani Karuturi > wrote: > >> *98* New defects. >> All the latest merged PR authors, can you please go through the list of >> new defects? >> > >Could we integrate it with travis and/or

[GitHub] cloudstack pull request: Fix event UUIDS missing on event bus

2015-11-24 Thread karuturi
Github user karuturi commented on the pull request: https://github.com/apache/cloudstack/pull/#issuecomment-159318964 Hi @ProjectMoon Thanks for taking this up. If possible, can you put in a sample event before and after the fix? I will test this tomorrow with rabbitmq

Re: New Defects reported by Coverity Scan for cloudstack

2015-11-23 Thread Rajani Karuturi
*98* New defects. All the latest merged PR authors, can you please go through the list of new defects? ~ Rajani -Original Message- From: "scan-ad...@coverity.com" Reply-To: "dev@cloudstack.apache.org" Date: Monday, 23 November 2015 at 11:44 PM To: "dev@cloudstack.apache.org" S

[GitHub] cloudstack pull request: CLOUDSTACK-9080: Resource limits for Prim...

2015-11-23 Thread karuturi
GitHub user karuturi opened a pull request: https://github.com/apache/cloudstack/pull/1107 CLOUDSTACK-9080: Resource limits for Primary arent respected during attach primary store resource limit check is not performed while attaching a volume to a vm. Added them same. Also

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

2015-11-20 Thread karuturi
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 pull request: [4.5] Metrics view

2015-11-20 Thread karuturi
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

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

2015-11-19 Thread karuturi
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

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

2015-11-19 Thread karuturi
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

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

2015-11-18 Thread karuturi
Github user karuturi commented on the pull request: https://github.com/apache/cloudstack/pull/1084#issuecomment-157945578 @wilderrodrigues nope. No HyperV environment. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If

Re: Hello 4.6.0, now what?!

2015-11-17 Thread Rajani Karuturi
Daan, I wont be able to continue. I will be OoO during the release timeframe. ~Rajani On Tue, Nov 17, 2015 at 1:55 PM, Daan Hoogland wrote: > Sebastien, > > On Tue, Nov 17, 2015 at 8:21 AM, Sebastien Goasguen > wrote: > > > > > > On Nov 17, 2015, at 7:33 AM, Wilder Rodrigues < > > wrodrig...@s

[GitHub] cloudstack-docs-install pull request: 4.6.0 release update, url, p...

2015-11-16 Thread karuturi
Github user karuturi commented on the pull request: https://github.com/apache/cloudstack-docs-install/pull/25#issuecomment-157277510 Can you remove ./source/installation.rst.old file? I think its not used anymore. one minor correction :+1: otherwise --- If your project is

[GitHub] cloudstack-docs-install pull request: 4.6.0 release update, url, p...

2015-11-16 Thread karuturi
Github user karuturi commented on a diff in the pull request: https://github.com/apache/cloudstack-docs-install/pull/25#discussion_r45024579 --- Diff: source/installation.rst.old --- @@ -86,7 +86,7 @@ Management Server may be placed on a virtual machine. - Operating

[GitHub] cloudstack-www pull request: Update the list of known CloudStack u...

2015-11-16 Thread karuturi
Github user karuturi commented on the pull request: https://github.com/apache/cloudstack-www/pull/17#issuecomment-157274083 tested. the page load fine. :+1: --- 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-www pull request: 4.6.0 is out!

2015-11-16 Thread karuturi
Github user karuturi commented on the pull request: https://github.com/apache/cloudstack-www/pull/18#issuecomment-157273709 release notes, install instructions links arent live yet. I am assuming they will be once the branch is created. One minor correction: "Conf

Re: [DISCUSS] Shared Networks on Nicira NVP Plugin

2015-11-16 Thread Rajani Karuturi
check this thread http://markmail.org/thread/xzpiaxnganny3ian ~Rajani On Tue, Nov 17, 2015 at 1:16 AM, Nicolás Vázquez wrote: > Thanks Rajani! > > Is there any planned release date for 4.7 version? > > 2015-11-13 7:51 GMT-03:00 Rajani Karuturi : > > > Hi Nicolas, > &

[GitHub] cloudstack pull request: Merge 4.6 release branch to master

2015-11-16 Thread karuturi
Github user karuturi commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1071#discussion_r44907513 --- Diff: build/replace.properties --- @@ -26,4 +26,4 @@ AGENTLOG=logs/agent.log MSMNTDIR=/mnt COMPONENTS-SPEC=components.xml REMOTEHOST

[GitHub] cloudstack pull request: Merge 4.6 release branch to master

2015-11-16 Thread karuturi
GitHub user karuturi opened a pull request: https://github.com/apache/cloudstack/pull/1071 Merge 4.6 release branch to master Initial merge of 4.6 to master ignored pom.xml version number changes and changes to debian/changelog and engine/schema/src/com/cloud/upgrade

[GitHub] cloudstack-docs pull request: Based on my conversation with the us...

2015-11-15 Thread karuturi
Github user karuturi commented on the pull request: https://github.com/apache/cloudstack-docs/pull/12#issuecomment-156944122 :+1: merging --- 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-docs pull request: The help topic created for the DHCP/...

2015-11-15 Thread karuturi
Github user karuturi commented on the pull request: https://github.com/apache/cloudstack-docs/pull/11#issuecomment-156944043 @rajsekharkpally how did you verify that its rendering properly? I think this should be .rst for sphinx to understand and render it properly. This

[GitHub] cloudstack-docs-install pull request: include argument for tomcat7...

2015-11-15 Thread karuturi
Github user karuturi commented on the pull request: https://github.com/apache/cloudstack-docs-install/pull/24#issuecomment-156943469 There is no 4.6 yet. I am merging this to master --- If your project is set up for it, you can reply to this email and have your reply appear on

[GitHub] cloudstack pull request: Make master 4.7.0-SNAPSHOT and implement ...

2015-11-15 Thread karuturi
Github user karuturi commented on the pull request: https://github.com/apache/cloudstack/pull/1068#issuecomment-156939746 remi mentioned on slack that master already has 461to470. so, the merge wont be a problem. In future, during the first merge after any release, we should take

[GitHub] cloudstack-docs-rn pull request: 4.6

2015-11-15 Thread karuturi
Github user karuturi commented on the pull request: https://github.com/apache/cloudstack-docs-rn/pull/25#issuecomment-156922033 Thanks for taking this up @pdion891 Thanks for fixing all the package urls to one consistent source :+1: Reviewed through the preview in github LGTM

Re: DevCloud deployment

2015-11-15 Thread Rajani Karuturi
I find it easier to use devcloud4[1]. It the easiest way for xen6.2 devcloud setup. The documentation is also good. Just install the required tools/IP configs mentioned in README [2]. Then, go to the advanced and follow the readme[3] there for a advanced zone setup. [1] https://github.com/apache/c

[GitHub] cloudstack pull request: Make master 4.7.0-SNAPSHOT and implement ...

2015-11-15 Thread karuturi
Github user karuturi commented on the pull request: https://github.com/apache/cloudstack/pull/1068#issuecomment-156911523 4.6 has 4.6 to 4.6.1 upgrade path and master has 4.6 to 4.7 upgrade path once 4.6 is merged to master, wont there be a problem with upgrade path on

Re: [VOTE] Apache CloudStack 4.6.0 (round 2)

2015-11-13 Thread Rajani Karuturi
+1 (Binding) setup: RHEL 6.5 Management server, xen 6.5 Hypervisor, NFS storage, Advanced zone without sg (Manual testing) Rpm creation, install worked fine Was able to succesfully launch a vm, destroy etc. Upload template worked fine LDAP new as well existing integration worked fine RabbitMQ i

Re: [DISCUSS] Shared Networks on Nicira NVP Plugin

2015-11-13 Thread Rajani Karuturi
Hi Nicolas, Right now master is frozen and RC2 is in progress (welcome to test and got :)). Once RC2 is voted, 4.6 branch would be created and master would become 4.7 At this point, schema-460to470.sql will also be created on master. Your changes should go in this file. 4.6.* will only be a bug f

[GitHub] cloudstack pull request: CLOUDSTACK-9023 filtering privatekey info...

2015-11-10 Thread karuturi
Github user karuturi commented on the pull request: https://github.com/apache/cloudstack/pull/1055#issuecomment-155403829 did a mvn build on the pr and manually verified the api call. the private key isn't logged anymore. before ``` 2015-11-10 17:13:18,462

[GitHub] cloudstack pull request: Fix the new line bug (invalid char on UI)...

2015-11-10 Thread karuturi
Github user karuturi commented on the pull request: https://github.com/apache/cloudstack/pull/1053#issuecomment-155398572 verified in spanish and english that that UI loads fine. did not see any js errors in spanish. LGTM :+1: --- If your project is set up for it, you can

[GitHub] cloudstack pull request: Fix some small UI bugs

2015-11-10 Thread karuturi
Github user karuturi commented on the pull request: https://github.com/apache/cloudstack/pull/1034#issuecomment-155389974 Thanks @ustcweizhou LGTM :+1: --- 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: Fix some small UI bugs

2015-11-09 Thread karuturi
Github user karuturi commented on the pull request: https://github.com/apache/cloudstack/pull/1034#issuecomment-155309530 [UI] fix typo for user data field - verfied verified password dialog change [UI] Network names are not being displayed in Infra » VRs » router » NICS

[ANNOUNCE] New committer: Boris Schrijver

2015-11-08 Thread Rajani Karuturi
The Project Management Committee (PMC) for Apache CloudStack has asked Boris Schrijver to become a committer and we are pleased to announce that he has accepted. Boris is an active code contributor, tester. Helps resolving issues, reviewing PRs and testing them. Being a committer allows many cont

Re: [VOTE] Apache CloudStack 4.6.0

2015-11-07 Thread Rajani Karuturi
+1 (Binding) setup: RHEL 6.5 Management server, xen 6.5 Hypervisor, NFS storage, Advanced zone without sg (Manual testing) Rpm creation, install worked fine Was able to succesfully launch a vm, destroy etc. Upload template worked fine LDAP new as well existing integration worked fine RabbitMQ i

[GitHub] cloudstack-docs-admin pull request: JIRA CLOUDSTACK-9034

2015-11-04 Thread karuturi
Github user karuturi commented on the pull request: https://github.com/apache/cloudstack-docs-admin/pull/32#issuecomment-153963381 @ronatartifact next time, in addition to jira id, please add more information to the subject line. something like "CLOUDSTACK-9034: Fixed bad b

[GitHub] cloudstack-docs-admin pull request: JIRA CLOUDSTACK-9034

2015-11-04 Thread karuturi
Github user karuturi commented on the pull request: https://github.com/apache/cloudstack-docs-admin/pull/32#issuecomment-153963135 Thanks for the PR. looks good. verified in github preview. merging now. --- If your project is set up for it, you can reply to this email and have your

Re: [VOTE] Apache CloudStack 4.6.0

2015-11-04 Thread Rajani Karuturi
I started jenkins builds for cloudstack RPM packages and systemvm templates for this branch here http://jenkins.buildacloud.org/view/parameterized/job/cloudstack-rpm-packages-with-branch-parameter/19/console http://jenkins.buildacloud.org/view/parameterized/job/parameterized-sytemvm/3/console We

RE: test_vpc_vpn.py failing 2 BVTs on all Advnetworking config

2015-11-04 Thread Rajani Karuturi
Its merged. ~Rajani Sent from my Windows Phone From: Raja Pullela Sent: ‎04-‎11-‎15 15:55 To: dev@cloudstack.apache.org Subject: RE: test_vpc_vpn.py failing 2 BVTs on all Advnetworking config Remi,

<    1   2   3   4   5   6   7   8   9   10   >