[DOC] cloudstack-docs-admin git merge strategy

[Rajani Karuturi]

While fixing a doc issue, I added *.po and *.mo files to the merge ignore
list(see .gitattributes file) and merged from 4.3 to 4.4 to master.

If we could follow this, it would be really helpful.
Basically, fix in the lowest possible version and merge upwards. The merges
would be very trivial as they would most likely be your change only.

Please let me know if you see any issues with this.

$ git branch --contains fe5b65de1b0eaa7de7dd5ee0f0dcb0e23dc8a618
  4.3
  4.4
* master

peace :)


~Rajani

Re: [TRAVIS] - failing for pull request #18

[Ian Duffy]

Hi Wilder,

The travis stuff is experimental.

I will take a look at this for you and see if its anything to be concerned
about.

Thanks,

Ian

On 10 September 2014 07:00, Wilder Rodrigues 
wrote:

> Hi guys,
>
> Last night I sent a pull request, after rebasing and fixing some conflicts
> in VirtualNetworkApplianceManagerImpl class. Unfortunately, the 3 builds
> are not okay on Travis. For the first job, I got the following:
>
>
> Still running (30 of 30): ./tools/travis/before_script.sh
>
> Timeout (30 minutes) reached. Terminating "./tools/travis/before_script.sh"
>
> The command "travis_wait 30 ./tools/travis/before_script.sh" failed and
> exited with 1 during .
>
> The second job is a bit stranger than that because all the tests that I
> actually executed on our environment, and passed, just failed on that job:
>
> test_privategw_acl
> test_01_reset_vm_on_reboot
> test_routers
>
> Concerning the 3rd job, same story: tests that have passed are now failing:
>
> test_service_offerings
> test_vpc_vpn
>
> Stil on the 3rd job I got problems with test_volumes and test_templates.
> The test_vm_life_cycle is still failing for everybody.
>
> My test environment is the following:
>
>
> · XenServer (host) running in our VMWare zone (which is managed by
> CloudStack).
>
> o   MySQL
>
> o   Secondary Storage
>
> o   Local Primary Storage
>
> · Management Server
>
> o   Debian 7 VM (inside Virtual Box)
>
> o   Pre/Post routing to allow my Debian to get connected to the host via
> its public IPs
>
> o   SSH reverse tunnel to allow communication between the host and the
> management server (for the Agent)
>
> · Tests
>
> o   I create a separate project with a bunch of ACS Marving tests
>
> §  Changed storage type in my tests from "shared" to "local"
>
> §  Changed DataCenter configuration
>
> · IP ranges
>
> §  Changed VPCs CIDRs and Network Gatways
>
> · To cope with my IP ranges
>
> · All the tests below are passing:
>
> o   test_01_create_account
>
> o   test_01_add_vm_to_subdomain
>
> o   test_DeleteDomain
>
> o   test_forceDeleteDomain
>
> o   test_updateAdminDetails
>
> o   test_updateDomainAdminDetails
>
> o   test_updateUserDetails
>
> o   test_LoginApiDomain
>
> o   test_LoginApiUuidResponse
>
> o   test_privategw_acl
>
> o   test_01_reset_vm_on_reboot
>
> o   test_03_restart_network_cleanup
>
> o   test_05_router_basic
>
> o   test_06_router_advanced
>
> o   test_07_stop_router
>
> o   test_08_start_router
>
> o   test_09_reboot_router
>
> o   test_01_create_service_offering
>
> o   test_02_edit_service_offering
>
> o   test_03_delete_service_offering
>
> o   test_01_start_stop_router_after_addition_of_one_guest_network
>
> o   test_02_reboot_router_after_addition_of_one_guest_network
>
> o   test_04_chg_srv_off_router_after_addition_of_one_guest_network
>
> o   test_05_destroy_router_after_addition_of_one_guest_network
>
> o   test_01_stop_start_router_after_creating_vpc
>
> o   test_02_reboot_router_after_creating_vpc
>
> o   test_04_change_service_offerring_vpc
>
> o   test_05_destroy_router_after_creating_vpc
>
> o   test_vpc_remote_access_vpn
>
> o   test_vpc_site2site_vpn
>
> Unfortunately, Travis does not give much help about what is going on, it
> only says "exception". Is there anyone out there that could give some help?
>
> The pull request is: VPC and Virtual Network Managers refactor. Part of
> the VPC Redundant Router work #18
>
> Thanks in advance.
>
> Cheers,
> Wilder
>

RE: [TRAVIS] - failing for pull request #18

[Wilder Rodrigues]

Hi Ian,

Super thanks!

I’m executing the tests again to see if there is any hiccup with the 
environment, although I did that twice yesterday.

In case you find some strangeness in the pull request, please, do not hesitate 
to contact me.

Cheers,
Wilder

From: Ian Duffy [mailto:i...@ianduffy.ie]
Sent: woensdag 10 september 2014 9:05
To: CloudStack Dev
Cc: int-toolkit; int-cloud
Subject: Re: [TRAVIS] - failing for pull request #18

Hi Wilder,

The travis stuff is experimental.

I will take a look at this for you and see if its anything to be concerned 
about.

Thanks,

Ian

On 10 September 2014 07:00, Wilder Rodrigues 
mailto:wrodrig...@schubergphilis.com>> wrote:
Hi guys,

Last night I sent a pull request, after rebasing and fixing some conflicts in 
VirtualNetworkApplianceManagerImpl class. Unfortunately, the 3 builds are not 
okay on Travis. For the first job, I got the following:


Still running (30 of 30): ./tools/travis/before_script.sh

Timeout (30 minutes) reached. Terminating "./tools/travis/before_script.sh"

The command "travis_wait 30 ./tools/travis/before_script.sh" failed and exited 
with 1 during .

The second job is a bit stranger than that because all the tests that I 
actually executed on our environment, and passed, just failed on that job:

test_privategw_acl
test_01_reset_vm_on_reboot
test_routers

Concerning the 3rd job, same story: tests that have passed are now failing:

test_service_offerings
test_vpc_vpn

Stil on the 3rd job I got problems with test_volumes and test_templates. The 
test_vm_life_cycle is still failing for everybody.

My test environment is the following:


· XenServer (host) running in our VMWare zone (which is managed by 
CloudStack).

o   MySQL

o   Secondary Storage

o   Local Primary Storage

· Management Server

o   Debian 7 VM (inside Virtual Box)

o   Pre/Post routing to allow my Debian to get connected to the host via its 
public IPs

o   SSH reverse tunnel to allow communication between the host and the 
management server (for the Agent)

· Tests

o   I create a separate project with a bunch of ACS Marving tests

§  Changed storage type in my tests from "shared" to "local"

§  Changed DataCenter configuration

· IP ranges

§  Changed VPCs CIDRs and Network Gatways

· To cope with my IP ranges

· All the tests below are passing:

o   test_01_create_account

o   test_01_add_vm_to_subdomain

o   test_DeleteDomain

o   test_forceDeleteDomain

o   test_updateAdminDetails

o   test_updateDomainAdminDetails

o   test_updateUserDetails

o   test_LoginApiDomain

o   test_LoginApiUuidResponse

o   test_privategw_acl

o   test_01_reset_vm_on_reboot

o   test_03_restart_network_cleanup

o   test_05_router_basic

o   test_06_router_advanced

o   test_07_stop_router

o   test_08_start_router

o   test_09_reboot_router

o   test_01_create_service_offering

o   test_02_edit_service_offering

o   test_03_delete_service_offering

o   test_01_start_stop_router_after_addition_of_one_guest_network

o   test_02_reboot_router_after_addition_of_one_guest_network

o   test_04_chg_srv_off_router_after_addition_of_one_guest_network

o   test_05_destroy_router_after_addition_of_one_guest_network

o   test_01_stop_start_router_after_creating_vpc

o   test_02_reboot_router_after_creating_vpc

o   test_04_change_service_offerring_vpc

o   test_05_destroy_router_after_creating_vpc

o   test_vpc_remote_access_vpn

o   test_vpc_site2site_vpn

Unfortunately, Travis does not give much help about what is going on, it only 
says "exception". Is there anyone out there that could give some help?

The pull request is: VPC and Virtual Network Managers refactor. Part of the VPC 
Redundant Router work #18

Thanks in advance.

Cheers,
Wilder

[DRAFT] Report Q3 Apache Cloudstack

[Hugo Trippaers]

Hey All,

This is the draft board report for Apache CloudStack for this quarter.

Please let me know if you have any feedback.

Cheers,

Hugo


DESCRIPTION
Apache CloudStack is open source software designed to deploy and manage 
large networks of virtual machines, as a highly available, highly scalable 
Infrastructure as a Service (IaaS) cloud computing platform.

ISSUES

None

CURRENT ACTIVITY
* The community is working on maintenance releases for both the 4.3.0 and 
  the 4.4.0 releases.
* The agreed release schedule is not met at the moment as we regularly identify
  quality issues in pending releases. This is an ongoing process, but slowly we 
are
  improving our QA processes and introducting more testing capabilities.
* Apache CloudStack CloudMonkey 5.2.0 was released on Aug 28 2014. Apache
  CloudStack CloudMonkey is a CLI tool for interacting with Apache
  CloudStack and is distributed separate from Apache CloudStack  
* Apache CloudStack 4.4.0 was released July 26 2014. Some issues were
  identified with this release after the release. The comunity decided
  not to market this release to a bid audience and work on a maintenance
  release immediately.
* The community is working with The Linux Foundation to prepare
  the second European CloudStack Collaboration Conference
  (CCCEU). CCCEU will planned following ApacheCon in 
  Budapest from November 19 till 21. It will have a new format
  which will hopefully engage more users and developers.

RELEASES
Apache CloudStack 4.4.0 was released on July 26, 2014.
Apache CloudStack CloudMonkey 5.2.0 was released on Aug 28, 2014

COMMUNITY
Including the following additions, CloudStack has 96 (+5) committers and 29 
(+1) 
PMC members.

New Committers:
Amogh Vasekar (amoghvk) - May 29
Pierre-Luc Dion (pdion891) - June 5
Will Stevens (swill) - June 16
Santhosh Edukulla (santhoshedukulla) - June 24
Rajani Karuturi (rajani) - July 17

New PMC Members:
Mark R. Hinkle (mrhinkle) - July 3

The Apache CloudStack project remains a high volume community:

//FIXME Old subscribers figures, volumes updated.
dev@ 761 729(+32) subs / msgs = Jun: 1852, Jul: 1555, Aug: 1613
users@ 1042(-59) subs / msgs = Jun: 663, Jul: 710, Aug: 640
issues@ 192(+15) subs / msgs = Jun: 1386, Jul: 1381, Aug: 1635
commits@ 202(+8) subs / msgs = Jun: 700, Jul: 790, Aug: 808
marketing@ 178(+15) subs / msgs = Jun: 251, Jul: 189, Aug:99 
users-cn@ 508(+40) subs / msgs = Jun: 54, Jul: 133, Aug: 73

Jenkins build is still unstable: simulator-singlerun #330

[jenkins]

See 

Re: [DOC] cloudstack-docs-admin git merge strategy

[Sebastien Goasguen]

On Sep 10, 2014, at 3:04 AM, Rajani Karuturi  wrote:

> While fixing a doc issue, I added *.po and *.mo files to the merge ignore
> list(see .gitattributes file) and merged from 4.3 to 4.4 to master.
> 

FYI: not sure what you are merging, but the .po and .mo are for the 
localization.
Since the docs are different for each release the po files have different 
strings, so we can't really mix them.
the .mo are actually binary files that technically should not be in the repo 
but there seems to be a bug in readthedocs and that's the only way to actually 
build the japanese and chinese pages.

> If we could follow this, it would be really helpful.
> Basically, fix in the lowest possible version and merge upwards. The merges
> would be very trivial as they would most likely be your change only.
> 
> Please let me know if you see any issues with this.
> 
> $ git branch --contains fe5b65de1b0eaa7de7dd5ee0f0dcb0e23dc8a618
>  4.3
>  4.4
> * master
> 
> peace :)
> 
> 
> ~Rajani

RE: [DRAFT] Report Q3 Apache Cloudstack

[Stephen Turner]

Two trivial typos in this sentence:

The comunity decided
 not to market this release to a bid audience and work on a maintenance
 release immediately.

community (two m's) and big (not bid).

-- 
Stephen Turner


-Original Message-
From: Trippie [mailto:trip...@gmail.com] On Behalf Of Hugo Trippaers
Sent: 10 September 2014 08:38
To: 
Subject: [DRAFT] Report Q3 Apache Cloudstack

Hey All,

This is the draft board report for Apache CloudStack for this quarter.

Please let me know if you have any feedback.

Cheers,

Hugo


DESCRIPTION
Apache CloudStack is open source software designed to deploy and manage large 
networks of virtual machines, as a highly available, highly scalable 
Infrastructure as a Service (IaaS) cloud computing platform.

ISSUES

None

CURRENT ACTIVITY
* The community is working on maintenance releases for both the 4.3.0 and
  the 4.4.0 releases.
* The agreed release schedule is not met at the moment as we regularly identify
  quality issues in pending releases. This is an ongoing process, but slowly we 
are
  improving our QA processes and introducting more testing capabilities.
* Apache CloudStack CloudMonkey 5.2.0 was released on Aug 28 2014. Apache
  CloudStack CloudMonkey is a CLI tool for interacting with Apache
  CloudStack and is distributed separate from Apache CloudStack
* Apache CloudStack 4.4.0 was released July 26 2014. Some issues were
  identified with this release after the release. The comunity decided
  not to market this release to a bid audience and work on a maintenance
  release immediately.
* The community is working with The Linux Foundation to prepare
  the second European CloudStack Collaboration Conference
  (CCCEU). CCCEU will planned following ApacheCon in
  Budapest from November 19 till 21. It will have a new format
  which will hopefully engage more users and developers.

RELEASES
Apache CloudStack 4.4.0 was released on July 26, 2014.
Apache CloudStack CloudMonkey 5.2.0 was released on Aug 28, 2014

COMMUNITY
Including the following additions, CloudStack has 96 (+5) committers and 29 
(+1) PMC members.

New Committers:
Amogh Vasekar (amoghvk) - May 29
Pierre-Luc Dion (pdion891) - June 5
Will Stevens (swill) - June 16
Santhosh Edukulla (santhoshedukulla) - June 24 Rajani Karuturi (rajani) - July 
17

New PMC Members:
Mark R. Hinkle (mrhinkle) - July 3

The Apache CloudStack project remains a high volume community:

//FIXME Old subscribers figures, volumes updated.
dev@ 761 729(+32) subs / msgs = Jun: 1852, Jul: 1555, Aug: 1613 users@ 
1042(-59) subs / msgs = Jun: 663, Jul: 710, Aug: 640 issues@ 192(+15) subs / 
msgs = Jun: 1386, Jul: 1381, Aug: 1635 commits@ 202(+8) subs / msgs = Jun: 700, 
Jul: 790, Aug: 808 marketing@ 178(+15) subs / msgs = Jun: 251, Jul: 189, Aug:99 
users-cn@ 508(+40) subs / msgs = Jun: 54, Jul: 133, Aug: 73

Re: Review Request 25476: CLOUDSTACK-7520: [UI] keep advanced search parameters visible after search has been run.

[Rohit Yadav]

> On Sept. 9, 2014, 2:02 p.m., Rohit Yadav wrote:
> > ui/scripts/ui/widgets/listView.js, line 1857
> > 
> >
> > Should n't this be declared as var, also this is not used or referenced 
> > in the function definition.
> 
> Mihaela Stoica wrote:
> This is where the saved advanced search parameters are cleared when a 
> basic search is run.
> The variable is declared in the scope of makeListView() function (ln 
> 1934) and is referenced in three places:
> - populated after the advanced search is run (ln 1960)
> - read when the advanced search box is invocked (ln 1949)
> - cleared when a basic search is run (ln 1857)

Thanks, LGTM.


- Rohit


---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/25476/#review52718
---


On Sept. 9, 2014, 1:47 p.m., Mihaela Stoica wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/25476/
> ---
> 
> (Updated Sept. 9, 2014, 1:47 p.m.)
> 
> 
> Review request for cloudstack, Brian Federle and Jessica Wang.
> 
> 
> Repository: cloudstack-git
> 
> 
> Description
> ---
> 
> - Preserve the advanced search parameters, so that when the advanced search 
> box is shown again,
> it is populated with the values selected/entered previously, unless they have 
> navigated away from the search results page,
> or applied any additional filters/search parameters.
> 
> 
> Diffs
> -
> 
>   ui/scripts/ui/widgets/listView.js 22182c1 
> 
> Diff: https://reviews.apache.org/r/25476/diff/
> 
> 
> Testing
> ---
> 
> 
> Thanks,
> 
> Mihaela Stoica
> 
>

Re: [DOC] cloudstack-docs-admin git merge strategy

[Rajani Karuturi]

On Wed, Sep 10, 2014 at 1:14 PM, Sebastien Goasguen 
wrote:

>
> On Sep 10, 2014, at 3:04 AM, Rajani Karuturi  wrote:
>
> > While fixing a doc issue, I added *.po and *.mo files to the merge ignore
> > list(see .gitattributes file) and merged from 4.3 to 4.4 to master.
> >
>
> FYI: not sure what you are merging, but the .po and .mo are for the
> localization.
> Since the docs are different for each release the po files have different
> strings, so we can't really mix them.
> the .mo are actually binary files that technically should not be in the
> repo but there seems to be a bug in readthedocs and that's the only way to
> actually build the japanese and chinese pages.
>
>
.po and .mo files arent merged. They are left untouched in any branch even
after a merge.

~Rajani

Re: Marvin test cleanup

[Daan Hoogland]

Alex,

I like the idea. What worries me is the possibility to run test cases on a
life system. I think it is useful for some operators to be able to do that.
Those people must not lose any objects that were created outside of the
test suite. Therefore it can be very tricky, not very different from your
parallel use-case, though.
The other side of the medal is of course the exhausted resources Prasanna
described.

+1


On Wed, Sep 10, 2014 at 8:15 AM, Gaurav Aradhye 
wrote:

> One thing that is important to consider here is "order in which we cleanup
> resources". Many times, cleanup operation fails if the resource can't be
> deleted unless few other resources are deleted (child/dependent).
>
> Alex, if you think if this too can be handled efficiently with the
> prototype you are likely to build, then it will be great! If not, please
> consider this too.
>
> Also, please let me know if I can help. I had started fixing the cleanup
> problems in test cases (identifying the missing items, fixing the cleanup
> orders), but due to other priorities I could not finish it. I will take it
> up soon anyway to complete it. But this change in framework itself to
> accommodate cleanup operation will be great!
>
> Regards,
> Gaurav
>
> On Wed, Sep 10, 2014 at 9:13 AM, Prasanna Santhanam 
> wrote:
>
> > On Wed, Sep 10, 2014 at 5:42 AM, Alex Brett 
> wrote:
> > > Hello all,
> > >
> > > At the moment we have a lot of Marvin tests that follow a pattern that
> > looks roughly like this:
> > >
> > > 1. Setup some resources (e.g. accounts, service offerings, VMs etc)
> > > 2. Add the resources to a list in the testcase (often called
> > self.cleanup)
> > > 3. Do the test(s)
> > > 4. Call cleanup_resources with the list of resources from 2
> > >
> > > (obviously in some cases resources get created/allocated during the
> > actual test rather than in setup, but it's a similar principle)
> > >
> > > In theory this is fine, however there are a number of cases where
> > resources are being created and then not added to the cleanup list, which
> > results in things being 'left behind', potentially using up resources
> which
> > may then affect future tests. For example I'm currently attempting to run
> > various tests in parallel (to speed up execution), and I'm hitting some
> > issues I believe to be caused by this.
> > >
> > > The thought that occurs to me here is do we actually need the testcase
> > to have to manually add resources to a list to cleanup, with the inherent
> > risk of resources getting missed etc - could we not make this something
> the
> > framework does for us (at least by default, with the option to override
> the
> > behaviour if needed).
> > >
> > > I've got some ideas as to how this could be done (one example that's a
> > bit of a layer violation but might be acceptable would be to wrap/extend
> > the apiClient to have a method that can be called on it from the various
> > object create methods to add the resulting object for cleanup), but
> before
> > I go ahead and start trying to prototype something I wanted to see if
> > anybody had any reasons why this sort of automatic cleanup behaviour
> might
> > be a bad idea or has investigated anything similar in the past?
> > >
> >
> > +1 - the cleanup of cloud resources is nasty especially when VMs and
> > billable entities are left behind. Many tests that follow fail as a
> > result of exhausted capacity. The nose framework sort of imposes the
> > setUp - test - tearDown model of test case authoring and perhaps led
> > us down that path of accumulating entities in cleanup lists. It would
> > be elegant if you can identify the corresponding Object.destroy method
> > and add the necessary finalizer automatically during create. Sort of
> > like the way unittest2 and py.test do it. The ACS API is quite
> > expressive and broken down easily into this model where you can figure
> > out the entity/resource you operate upon. Each entity has its API for
> > creation and a corresponding one for listing and destroy/deletion. If
> > that's too involved the overloaded apiClient works fine too.
> >
> > > Cheers,
> > > Alex
> > >
> >
>



-- 
Daan

Jenkins build is back to normal : build-master #1578

[jenkins]

See 

Re: [DOC] cloudstack-docs-admin git merge strategy

[sebgoa]

On Sep 10, 2014, at 10:47 AM, Rajani Karuturi  wrote:

> On Wed, Sep 10, 2014 at 1:14 PM, Sebastien Goasguen 
> wrote:
> 
>> 
>> On Sep 10, 2014, at 3:04 AM, Rajani Karuturi  wrote:
>> 
>>> While fixing a doc issue, I added *.po and *.mo files to the merge ignore
>>> list(see .gitattributes file) and merged from 4.3 to 4.4 to master.
>>> 
>> 
>> FYI: not sure what you are merging, but the .po and .mo are for the
>> localization.
>> Since the docs are different for each release the po files have different
>> strings, so we can't really mix them.
>> the .mo are actually binary files that technically should not be in the
>> repo but there seems to be a bug in readthedocs and that's the only way to
>> actually build the japanese and chinese pages.
>> 
>> 
> .po and .mo files arent merged. They are left untouched in any branch even
> after a merge.

roger that, thx

> 
> ~Rajani

Re: [DOC] cloudstack-docs-admin git merge strategy

[Daan Hoogland]

see you managed to do this strategy in the docs, great. plusone. any
resolutions on the how to keep the po files up to date?


On Wed, Sep 10, 2014 at 10:47 AM, Rajani Karuturi  wrote:

> On Wed, Sep 10, 2014 at 1:14 PM, Sebastien Goasguen 
> wrote:
>
> >
> > On Sep 10, 2014, at 3:04 AM, Rajani Karuturi  wrote:
> >
> > > While fixing a doc issue, I added *.po and *.mo files to the merge
> ignore
> > > list(see .gitattributes file) and merged from 4.3 to 4.4 to master.
> > >
> >
> > FYI: not sure what you are merging, but the .po and .mo are for the
> > localization.
> > Since the docs are different for each release the po files have different
> > strings, so we can't really mix them.
> > the .mo are actually binary files that technically should not be in the
> > repo but there seems to be a bug in readthedocs and that's the only way
> to
> > actually build the japanese and chinese pages.
> >
> >
> .po and .mo files arent merged. They are left untouched in any branch even
> after a merge.
>
> ~Rajani
>



-- 
Daan

Re: Review Request 25474: CLOUDSTACK-7519: Using bound/unbound methods instead of directly calling API methods from test case

[SrikanteswaraRao Talluri]

---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/25474/#review52839
---

Ship it!


Thanks for this followup patch.

pushed to master 81d5dd9bcda185ff83820f8dd6233cf80f8efaed

- SrikanteswaraRao Talluri


On Sept. 9, 2014, 12:17 p.m., Gaurav Aradhye wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/25474/
> ---
> 
> (Updated Sept. 9, 2014, 12:17 p.m.)
> 
> 
> Review request for cloudstack, sanjeev n, Santhosh Edukulla, and 
> SrikanteswaraRao Talluri.
> 
> 
> Bugs: CLOUDSTACK-7519
> https://issues.apache.org/jira/browse/CLOUDSTACK-7519
> 
> 
> Repository: cloudstack-git
> 
> 
> Description
> ---
> 
> As suggested in RR: https://reviews.apache.org/r/25433/
> 
> test_templates.py file has many instances where direct APIs are called from 
> the test case. Instead we have the methods in base library that can be used.
> 
> 
> Diffs
> -
> 
>   test/integration/smoke/test_templates.py b7fe49d 
>   tools/marvin/marvin/lib/base.py 2a763ce 
> 
> Diff: https://reviews.apache.org/r/25474/diff/
> 
> 
> Testing
> ---
> 
> Yes.
> 
> Tested on KVM setup.
> 
> Log:
> Test create public & private template ... === TestName: 
> test_01_create_template | Status : SUCCESS ===
> ok
> Test Edit template ... === TestName: test_02_edit_template | Status : SUCCESS 
> ===
> ok
> Test delete template ... === TestName: test_03_delete_template | Status : 
> SUCCESS ===
> ok
> Test for extract template ... === TestName: test_04_extract_template | Status 
> : SUCCESS ===
> ok
> Update & Test for template permissions ... === TestName: 
> test_05_template_permissions | Status : SUCCESS ===
> ok
> Test for copy template from one zone to another ... === TestName: 
> test_06_copy_template | Status : SUCCESS ===
> ok
> Test only public templates are visible to normal user ... === TestName: 
> test_07_list_public_templates | Status : SUCCESS ===
> ok
> Test System templates are not visible to normal user ... === TestName: 
> test_08_list_system_templates | Status : SUCCESS ===
> ok
> 
> --
> Ran 8 tests in 1007.401s
> 
> OK
> 
> 
> Thanks,
> 
> Gaurav Aradhye
> 
>

Re: [DOC] cloudstack-docs-admin git merge strategy

[sebgoa]

On Sep 10, 2014, at 11:09 AM, Daan Hoogland  wrote:

> see you managed to do this strategy in the docs, great. plusone. any
> resolutions on the how to keep the po files up to date?

this is very much a manual process, the po files need to be generated by hand 
for each release based on the current english docs.
po files are just a list of strings in all the markdown files
the mo files are also generated by hand, by pulling the translated strings from 
transifex and building them with sphinx…

see the translation section in the readme at:
https://github.com/apache/cloudstack-docs-admin

or maybe I not getting your question.

> 
> 
> On Wed, Sep 10, 2014 at 10:47 AM, Rajani Karuturi  wrote:
> 
>> On Wed, Sep 10, 2014 at 1:14 PM, Sebastien Goasguen 
>> wrote:
>> 
>>> 
>>> On Sep 10, 2014, at 3:04 AM, Rajani Karuturi  wrote:
>>> 
 While fixing a doc issue, I added *.po and *.mo files to the merge
>> ignore
 list(see .gitattributes file) and merged from 4.3 to 4.4 to master.
 
>>> 
>>> FYI: not sure what you are merging, but the .po and .mo are for the
>>> localization.
>>> Since the docs are different for each release the po files have different
>>> strings, so we can't really mix them.
>>> the .mo are actually binary files that technically should not be in the
>>> repo but there seems to be a bug in readthedocs and that's the only way
>> to
>>> actually build the japanese and chinese pages.
>>> 
>>> 
>> .po and .mo files arent merged. They are left untouched in any branch even
>> after a merge.
>> 
>> ~Rajani
>> 
> 
> 
> 
> -- 
> Daan

Jenkins build is back to normal : build-master-noredist #3494

[jenkins]

See 

Re: [DOC] cloudstack-docs-admin git merge strategy

[Daan Hoogland]

Your getting the question, I just had the tense wrong. I meant in the
future (implied by the 'resolution')


On Wed, Sep 10, 2014 at 11:12 AM, sebgoa  wrote:

>
> On Sep 10, 2014, at 11:09 AM, Daan Hoogland 
> wrote:
>
> > see you managed to do this strategy in the docs, great. plusone. any
> > resolutions on the how to keep the po files up to date?
>
> this is very much a manual process, the po files need to be generated by
> hand for each release based on the current english docs.
> po files are just a list of strings in all the markdown files
> the mo files are also generated by hand, by pulling the translated strings
> from transifex and building them with sphinx…
>
> see the translation section in the readme at:
> https://github.com/apache/cloudstack-docs-admin
>
> or maybe I not getting your question.
>
> >
> >
> > On Wed, Sep 10, 2014 at 10:47 AM, Rajani Karuturi 
> wrote:
> >
> >> On Wed, Sep 10, 2014 at 1:14 PM, Sebastien Goasguen 
> >> wrote:
> >>
> >>>
> >>> On Sep 10, 2014, at 3:04 AM, Rajani Karuturi 
> wrote:
> >>>
>  While fixing a doc issue, I added *.po and *.mo files to the merge
> >> ignore
>  list(see .gitattributes file) and merged from 4.3 to 4.4 to master.
> 
> >>>
> >>> FYI: not sure what you are merging, but the .po and .mo are for the
> >>> localization.
> >>> Since the docs are different for each release the po files have
> different
> >>> strings, so we can't really mix them.
> >>> the .mo are actually binary files that technically should not be in the
> >>> repo but there seems to be a bug in readthedocs and that's the only way
> >> to
> >>> actually build the japanese and chinese pages.
> >>>
> >>>
> >> .po and .mo files arent merged. They are left untouched in any branch
> even
> >> after a merge.
> >>
> >> ~Rajani
> >>
> >
> >
> >
> > --
> > Daan
>
>


-- 
Daan

Re: [TRAVIS] - failing for pull request #18

[Ian Duffy]

HI Wilder,

I cloned your branch, compiled it and ran it in simulator mode.

mvn -pl :cloud-client-ui jetty:run -D simulator

I browsed about the UI for a bit, I seen an exception been thrown when I
went to the accounts page, the page would not successfully load.

I went ahead and deployed the advanced.cfg file with marvin:

python -m marvin.deployDataCenter setup/dev/advanced.cfg

I executed the first set of tests travis does and seen some failures:

=== TestName: test_DeployVmAntiAffinityGroup | Status : EXCEPTION ===

=== TestName: test_deployvm_firstfit | Status : EXCEPTION ===

=== TestName: test_deployvm_userconcentrated | Status : EXCEPTION ===

=== TestName: test_deployvm_userdispersing | Status : EXCEPTION ===

=== TestName: test_delete_account | Status : EXCEPTION ===

If you run them on your own machine you'll see the JSON response from
cloudstack that causes the exceptions within: /tmp//MarvinLogs and you can
begin to debug.

>From what I can tell the failures are genuine and the changes do break
things.

On 10 September 2014 08:15, Wilder Rodrigues 
wrote:

>  Hi Ian,
>
>
>
> Super thanks!
>
>
>
> I’m executing the tests again to see if there is any hiccup with the
> environment, although I did that twice yesterday.
>
>
>
> In case you find some strangeness in the pull request, please, do not
> hesitate to contact me.
>
>
>
> Cheers,
>
> Wilder
>
>
>
> *From:* Ian Duffy [mailto:i...@ianduffy.ie]
> *Sent:* woensdag 10 september 2014 9:05
> *To:* CloudStack Dev
> *Cc:* int-toolkit; int-cloud
> *Subject:* Re: [TRAVIS] - failing for pull request #18
>
>
>
> Hi Wilder,
>
>
>
> The travis stuff is experimental.
>
>
>
> I will take a look at this for you and see if its anything to be concerned
> about.
>
>
>
> Thanks,
>
>
>
> Ian
>
>
>
> On 10 September 2014 07:00, Wilder Rodrigues <
> wrodrig...@schubergphilis.com> wrote:
>
> Hi guys,
>
> Last night I sent a pull request, after rebasing and fixing some conflicts
> in VirtualNetworkApplianceManagerImpl class. Unfortunately, the 3 builds
> are not okay on Travis. For the first job, I got the following:
>
>
> Still running (30 of 30): ./tools/travis/before_script.sh
>
> Timeout (30 minutes) reached. Terminating "./tools/travis/before_script.sh"
>
> The command "travis_wait 30 ./tools/travis/before_script.sh" failed and
> exited with 1 during .
>
> The second job is a bit stranger than that because all the tests that I
> actually executed on our environment, and passed, just failed on that job:
>
> test_privategw_acl
> test_01_reset_vm_on_reboot
> test_routers
>
> Concerning the 3rd job, same story: tests that have passed are now failing:
>
> test_service_offerings
> test_vpc_vpn
>
> Stil on the 3rd job I got problems with test_volumes and test_templates.
> The test_vm_life_cycle is still failing for everybody.
>
> My test environment is the following:
>
>
> · XenServer (host) running in our VMWare zone (which is managed by
> CloudStack).
>
> o   MySQL
>
> o   Secondary Storage
>
> o   Local Primary Storage
>
> · Management Server
>
> o   Debian 7 VM (inside Virtual Box)
>
> o   Pre/Post routing to allow my Debian to get connected to the host via
> its public IPs
>
> o   SSH reverse tunnel to allow communication between the host and the
> management server (for the Agent)
>
> · Tests
>
> o   I create a separate project with a bunch of ACS Marving tests
>
> §  Changed storage type in my tests from "shared" to "local"
>
> §  Changed DataCenter configuration
>
> · IP ranges
>
> §  Changed VPCs CIDRs and Network Gatways
>
> · To cope with my IP ranges
>
> · All the tests below are passing:
>
> o   test_01_create_account
>
> o   test_01_add_vm_to_subdomain
>
> o   test_DeleteDomain
>
> o   test_forceDeleteDomain
>
> o   test_updateAdminDetails
>
> o   test_updateDomainAdminDetails
>
> o   test_updateUserDetails
>
> o   test_LoginApiDomain
>
> o   test_LoginApiUuidResponse
>
> o   test_privategw_acl
>
> o   test_01_reset_vm_on_reboot
>
> o   test_03_restart_network_cleanup
>
> o   test_05_router_basic
>
> o   test_06_router_advanced
>
> o   test_07_stop_router
>
> o   test_08_start_router
>
> o   test_09_reboot_router
>
> o   test_01_create_service_offering
>
> o   test_02_edit_service_offering
>
> o   test_03_delete_service_offering
>
> o   test_01_start_stop_router_after_addition_of_one_guest_network
>
> o   test_02_reboot_router_after_addition_of_one_guest_network
>
> o   test_04_chg_srv_off_router_after_addition_of_one_guest_network
>
> o   test_05_destroy_router_after_addition_of_one_guest_network
>
> o   test_01_stop_start_router_after_creating_vpc
>
> o   test_02_reboot_router_after_creating_vpc
>
> o   test_04_change_service_offerring_vpc
>
> o   test_05_destroy_router_after_creating_vpc
>
> o   test_vpc_remote_access_vpn
>
> o   test_vpc_site2site_vpn
>
> Unfortunately, Travis does not give much help about what is going on, it
> only says "exception". Is there anyone out there that could give some help?

RE: [TRAVIS] - failing for pull request #18

[Wilder Rodrigues]

Thanks Ian,

I will follow the same steps, because it’s really weird.

I’m running with mvn –pl :cloud-ui jetty:run, but no simulator.

Accounts page looks fine, I even executed the accounts related tests. Advance 
zone was also deployed successfully.

The data form my accounts page:

admin  Admin  ROOT  enabled
baremetal-system-account User ROOT  enabled
test-a-TestVpcRemoteAccessVpn-4SYLK3   User ROOT  enabled
test-a-TestVpcRemoteAccessVpn-5WDDUJ User ROOT  enabled

You clone the “test-cmd-rebase” branch,  right?

I was busy running all the tests I have again, the result was successful and I 
also kept tracking the “running.txt” Marvin log file during execution.

The only difference IPs I have in the advanced.cfg, because I do not use the 
ranges that are there.



Test Create Account and user for that account ... === TestName: 
test_01_create_account | Status : SUCCESS ===
ok
Test Sub domain allowed to launch VM  when a Domain level zone is created ... 
=== TestName: test_01_add_vm_to_subdomain | Status : SUCCESS ===
ok
Test delete domain without force option ... === TestName: test_DeleteDomain | 
Status : SUCCESS ===
ok
Test delete domain with force option ... === TestName: test_forceDeleteDomain | 
Status : SUCCESS ===
ok
Test update admin details ... === TestName: test_updateAdminDetails | Status : 
SUCCESS ===
ok
Test update domain admin details ... === TestName: 
test_updateDomainAdminDetails | Status : SUCCESS ===
ok
Test user update API ... === TestName: test_updateUserDetails | Status : 
SUCCESS ===
ok
Test login API with domain ... === TestName: test_LoginApiDomain | Status : 
SUCCESS ===
ok
Test if Login API does not return UUID's ... === TestName: 
test_LoginApiUuidResponse | Status : SUCCESS ===
ok

--
Ran 9 tests in 832.310s

OK

test_privategw_acl (ekholabs.acs.tests.test_privategw_acl.TestPrivateGwACL) ... 
=== TestName: test_privategw_acl | Status : SUCCESS ===
ok

--
Ran 1 test in 100.582s

OK

Test reset virtual machine on reboot ... === TestName: 
test_01_reset_vm_on_reboot | Status : SUCCESS ===
ok

--
Ran 1 test in 183.507s

OK

Test router internal basic zone ... SKIP: Marvin configuration has no host 
credentials to check router services
Test router internal advanced zone ... SKIP: Marvin configuration has no host 
credentials to check router services
Test restart network ... === TestName: test_03_restart_network_cleanup | Status 
: SUCCESS ===
ok
Test restart network without cleanup ... SKIP: Marvin configuration has no host 
credentials to check router services
Test router basic setup ... === TestName: test_05_router_basic | Status : 
SUCCESS ===
ok
Test router advanced setup ... === TestName: test_06_router_advanced | Status : 
SUCCESS ===
ok
Test stop router ... === TestName: test_07_stop_router | Status : SUCCESS ===
ok
Test start router ... === TestName: test_08_start_router | Status : SUCCESS ===
ok
Test reboot router ... === TestName: test_09_reboot_router | Status : SUCCESS 
===
ok

--
Ran 9 tests in 354.245s

OK (SKIP=3)

Test to create service offering ... === TestName: 
test_01_create_service_offering | Status : SUCCESS ===
ok
Test to update existing service offering ... === TestName: 
test_02_edit_service_offering | Status : SUCCESS ===
ok
Test to delete service offering ... === TestName: 
test_03_delete_service_offering | Status : SUCCESS ===
ok

--
Ran 3 tests in 158.207s

OK

Test start/stop of router after addition of one guest network ... === TestName: 
test_01_start_stop_router_after_addition_of_one_guest_network | Status : 
SUCCESS ===
ok
Test reboot of router after addition of one guest network ... === TestName: 
test_02_reboot_router_after_addition_of_one_guest_network | Status : SUCCESS ===
ok
Test migrate of router after addition of one guest network ... SKIP: No host 
available for migration. Test requires atleast 2 hosts
Test to change service offering of router after addition of one guest network 
... === TestName: 
test_04_chg_srv_off_router_after_addition_of_one_guest_network | Status : 
SUCCESS ===
ok
Test destroy of router after addition of one guest network ... === TestName: 
test_05_destroy_router_after_addition_of_one_guest_network | Status : SUCCESS 
===
ok
Test to stop and start router after creation of VPC ... === TestName: 
test_01_stop_start_router_after_creating_vpc | Status : SUCCESS ===
ok
Test to reboot the router after creating a VPC ... === TestName: 
test_02_reboot_router_after_creating_vpc | Status : SUCCESS ===
ok
Test migration of router to another host after creating VPC ... SKIP: No host 
available for migration. Test requires atleast 2 hosts
Tests to change service offering of the

Re: [TRAVIS] - failing for pull request #18

[Ian Duffy]

> You clone the “test-cmd-rebase” branch,  right?

Yes positive:

*$ *git status

On branch test-cmd-rebase

Your branch is up-to-date with 'origin/test-cmd-rebase'.


Try your work against the simulator.

On 10 September 2014 10:44, Wilder Rodrigues 
wrote:

>  Thanks Ian,
>
>
>
> I will follow the same steps, because it’s really weird.
>
>
>
> I’m running with mvn –pl :cloud-ui jetty:run, but no simulator.
>
>
>
> Accounts page looks fine, I even executed the accounts related tests.
> Advance zone was also deployed successfully.
>
>
>
> The data form my accounts page:
>
>
>
> admin  Admin  ROOT  enabled
>
> baremetal-system-account User ROOT  enabled
>
> test-a-TestVpcRemoteAccessVpn-4SYLK3   User ROOT  enabled
>
> test-a-TestVpcRemoteAccessVpn-5WDDUJ User ROOT  enabled
>
>
>
> You clone the “test-cmd-rebase” branch,  right?
>
>
>
> I was busy running all the tests I have again, the result was successful
> and I also kept tracking the “running.txt” Marvin log file during execution.
>
>
>
> The only difference IPs I have in the advanced.cfg, because I do not use
> the ranges that are there.
>
>
>
>
>
>
>
> Test Create Account and user for that account ... === TestName:
> test_01_create_account | Status : SUCCESS ===
>
> ok
>
> Test Sub domain allowed to launch VM  when a Domain level zone is created
> ... === TestName: test_01_add_vm_to_subdomain | Status : SUCCESS ===
>
> ok
>
> Test delete domain without force option ... === TestName:
> test_DeleteDomain | Status : SUCCESS ===
>
> ok
>
> Test delete domain with force option ... === TestName:
> test_forceDeleteDomain | Status : SUCCESS ===
>
> ok
>
> Test update admin details ... === TestName: test_updateAdminDetails |
> Status : SUCCESS ===
>
> ok
>
> Test update domain admin details ... === TestName:
> test_updateDomainAdminDetails | Status : SUCCESS ===
>
> ok
>
> Test user update API ... === TestName: test_updateUserDetails | Status :
> SUCCESS ===
>
> ok
>
> Test login API with domain ... === TestName: test_LoginApiDomain | Status
> : SUCCESS ===
>
> ok
>
> Test if Login API does not return UUID's ... === TestName:
> test_LoginApiUuidResponse | Status : SUCCESS ===
>
> ok
>
>
>
> --
>
> Ran 9 tests in 832.310s
>
>
>
> OK
>
>
>
> test_privategw_acl
> (ekholabs.acs.tests.test_privategw_acl.TestPrivateGwACL) ... === TestName:
> test_privategw_acl | Status : SUCCESS ===
>
> ok
>
>
>
> --
>
> Ran 1 test in 100.582s
>
>
>
> OK
>
>
>
> Test reset virtual machine on reboot ... === TestName:
> test_01_reset_vm_on_reboot | Status : SUCCESS ===
>
> ok
>
>
>
> --
>
> Ran 1 test in 183.507s
>
>
>
> OK
>
>
>
> Test router internal basic zone ... SKIP: Marvin configuration has no host
> credentials to check router services
>
> Test router internal advanced zone ... SKIP: Marvin configuration has no
> host credentials to check router services
>
> Test restart network ... === TestName: test_03_restart_network_cleanup |
> Status : SUCCESS ===
>
> ok
>
> Test restart network without cleanup ... SKIP: Marvin configuration has no
> host credentials to check router services
>
> Test router basic setup ... === TestName: test_05_router_basic | Status :
> SUCCESS ===
>
> ok
>
> Test router advanced setup ... === TestName: test_06_router_advanced |
> Status : SUCCESS ===
>
> ok
>
> Test stop router ... === TestName: test_07_stop_router | Status : SUCCESS
> ===
>
> ok
>
> Test start router ... === TestName: test_08_start_router | Status :
> SUCCESS ===
>
> ok
>
> Test reboot router ... === TestName: test_09_reboot_router | Status :
> SUCCESS ===
>
> ok
>
>
>
> --
>
> Ran 9 tests in 354.245s
>
>
>
> OK (SKIP=3)
>
>
>
> Test to create service offering ... === TestName:
> test_01_create_service_offering | Status : SUCCESS ===
>
> ok
>
> Test to update existing service offering ... === TestName:
> test_02_edit_service_offering | Status : SUCCESS ===
>
> ok
>
> Test to delete service offering ... === TestName:
> test_03_delete_service_offering | Status : SUCCESS ===
>
> ok
>
>
>
> --
>
> Ran 3 tests in 158.207s
>
>
>
> OK
>
>
>
> Test start/stop of router after addition of one guest network ... ===
> TestName: test_01_start_stop_router_after_addition_of_one_guest_network |
> Status : SUCCESS ===
>
> ok
>
> Test reboot of router after addition of one guest network ... ===
> TestName: test_02_reboot_router_after_addition_of_one_guest_network |
> Status : SUCCESS ===
>
> ok
>
> Test migrate of router after addition of one guest network ... SKIP: No
> host available for migration. Test requires atleast 2 hosts
>
> Test to change service offering of router after addition of one guest
> network ... === TestName:
> test_04_chg_srv_off_rou

Re: Review Request 25476: CLOUDSTACK-7520: [UI] keep advanced search parameters visible after search has been run.

[Rajani Karuturi]

---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/25476/#review52841
---

Ship it!


Ship It!

- Rajani Karuturi


On Sept. 9, 2014, 1:47 p.m., Mihaela Stoica wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/25476/
> ---
> 
> (Updated Sept. 9, 2014, 1:47 p.m.)
> 
> 
> Review request for cloudstack, Brian Federle and Jessica Wang.
> 
> 
> Repository: cloudstack-git
> 
> 
> Description
> ---
> 
> - Preserve the advanced search parameters, so that when the advanced search 
> box is shown again,
> it is populated with the values selected/entered previously, unless they have 
> navigated away from the search results page,
> or applied any additional filters/search parameters.
> 
> 
> Diffs
> -
> 
>   ui/scripts/ui/widgets/listView.js 22182c1 
> 
> Diff: https://reviews.apache.org/r/25476/diff/
> 
> 
> Testing
> ---
> 
> 
> Thanks,
> 
> Mihaela Stoica
> 
>

Jenkins build is still unstable: simulator-singlerun #331

[jenkins]

See 

Re: Review Request 25476: CLOUDSTACK-7520: [UI] keep advanced search parameters visible after search has been run.

[Rajani Karuturi]

> On Sept. 10, 2014, 10:07 a.m., Rajani Karuturi wrote:
> > Ship It!

commit 75cd79a2382486385b5ae901c5b5c45cbd1505fc


- Rajani


---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/25476/#review52841
---


On Sept. 9, 2014, 1:47 p.m., Mihaela Stoica wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/25476/
> ---
> 
> (Updated Sept. 9, 2014, 1:47 p.m.)
> 
> 
> Review request for cloudstack, Brian Federle and Jessica Wang.
> 
> 
> Repository: cloudstack-git
> 
> 
> Description
> ---
> 
> - Preserve the advanced search parameters, so that when the advanced search 
> box is shown again,
> it is populated with the values selected/entered previously, unless they have 
> navigated away from the search results page,
> or applied any additional filters/search parameters.
> 
> 
> Diffs
> -
> 
>   ui/scripts/ui/widgets/listView.js 22182c1 
> 
> Diff: https://reviews.apache.org/r/25476/diff/
> 
> 
> Testing
> ---
> 
> 
> Thanks,
> 
> Mihaela Stoica
> 
>

Re: Review Request 25435: [CLOUDSTACK-7159] Add "usageid" parameter to the "listUsageRecords" API call.

[Rohit Yadav]

---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/25435/#review52845
---


Merged on master:
commit 70142c4acb84a2d2b1fe8806c159493e4a556532
Author: Ilia Shakitko 
AuthorDate: Tue Sep 9 15:42:07 2014 +0200
Commit: Rohit Yadav 
CommitDate: Wed Sep 10 12:19:34 2014 +0200

- Rohit Yadav


On Sept. 9, 2014, 1:57 p.m., Ilia Shakitko wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/25435/
> ---
> 
> (Updated Sept. 9, 2014, 1:57 p.m.)
> 
> 
> Review request for cloudstack, Alena Prokharchyk, Kishan Kavala, and Sheng 
> Yang.
> 
> 
> Bugs: CLOUDSTACK-7510
> https://issues.apache.org/jira/browse/CLOUDSTACK-7510
> 
> 
> Repository: cloudstack-git
> 
> 
> Description
> ---
> 
> Working with Usage server and usage records very often I need to get only 
> records for that particular usage ID. For example when filtering out 
> network_bytes_received/sent with big amount of data it's not very fast to 
> process hundreds of objects looking for the only one you need.
> It would be useful to have an ability to filter out usage records only for 
> specific resource ID.
> 
> This parch brings that to the API.
> 
> 
> Diffs
> -
> 
>   api/src/org/apache/cloudstack/api/ApiConstants.java 6baa95c 
>   
> api/src/org/apache/cloudstack/api/command/admin/usage/GetUsageRecordsCmd.java 
> 21a7e4a 
>   server/src/com/cloud/usage/UsageServiceImpl.java d1f62aa 
> 
> Diff: https://reviews.apache.org/r/25435/diff/
> 
> 
> Testing
> ---
> 
> Tested cases:
> - usageid is specified w/o "type": an exception is thrown (correct)
> - provided usageid is not exists: an empty response is returned (since no 
> records were found, correct)
> - no usageid specified: work as is
> - an existing usageid specified (with type, for example type=4 or type=5): 
> only records for that usage type is returned
> 
> 
> Thanks,
> 
> Ilia Shakitko
> 
>

Re: Review Request 25430: live migration is failing for vm deployed using dynaic compute offerings with NPE

[Rohit Yadav]

---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/25430/#review52846
---


Just want to check with other reviewers, if this contribution is good to merge 
on master? Bharat, if you may confirm that it is well tested on Xen/KVM?

- Rohit Yadav


On Sept. 8, 2014, 1:55 p.m., bharat kumar wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/25430/
> ---
> 
> (Updated Sept. 8, 2014, 1:55 p.m.)
> 
> 
> Review request for cloudstack, Alena Prokharchyk, Devdeep Singh, edison su, 
> Koushik Das, Mike Tutkowski, and Nitin Mehta.
> 
> 
> Bugs: CLOUDSTACK-6099
> https://issues.apache.org/jira/browse/CLOUDSTACK-6099
> 
> 
> Repository: cloudstack-git
> 
> 
> Description
> ---
> 
> live migration is failing for vm deployed using dynaic compute offerings with 
> NPE
> https://issues.apache.org/jira/browse/CLOUDSTACK-6099
> 
> 
> Diffs
> -
> 
>   engine/orchestration/src/com/cloud/vm/VirtualMachineManagerImpl.java 
> 8edef77 
> 
> Diff: https://reviews.apache.org/r/25430/diff/
> 
> 
> Testing
> ---
> 
> Tested live migration on master using xenserver.
> 
> 
> Thanks,
> 
> bharat kumar
> 
>

Re: Review Request 25266: Simulator build support need to extends for RPM build

[Rohit Yadav]

---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/25266/#review52847
---


This is great, can you also add rules for creating deb packages as well?

- Rohit Yadav


On Sept. 2, 2014, 7:09 p.m., Rayees Namathponnan wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/25266/
> ---
> 
> (Updated Sept. 2, 2014, 7:09 p.m.)
> 
> 
> Review request for cloudstack, Frank Zhang and Hugo Trippaers.
> 
> 
> Bugs: CLOUDSTACK-7469
> https://issues.apache.org/jira/browse/CLOUDSTACK-7469
> 
> 
> Repository: cloudstack-git
> 
> 
> Description
> ---
> 
> Currently there is no option to build rpm with simulator,  as part this patch 
> modified package.sh file to accept simulator
> 
> also updated cloud.spec file to build both oss and nooss simulator buids
> 
> 
> To build noredist and simulator, you need to use below package command
> ./package.sh --pack noredist --s simulator
> 
> default package with simulato, need to call  ./package.sh
> 
> 
> Diffs
> -
> 
>   packaging/centos63/cloud.spec 7306d1f 
>   packaging/centos63/package.sh 6de66e6 
> 
> Diff: https://reviews.apache.org/r/25266/diff/
> 
> 
> Testing
> ---
> 
> Yes
> 
> 
> Thanks,
> 
> Rayees Namathponnan
> 
>

Re: Review Request 25429: CLOUDSTACK-7502: UI: Host detail page - Display KVM agent version and Qemu version

[Rohit Yadav]

---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/25429/#review52848
---


Thanks for your contribution, can you share with us if this was tested and with 
UI contributions can we see a screenshot with/without KVM hosts.

- Rohit Yadav


On Sept. 8, 2014, 9:13 a.m., Mihaela Stoica wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/25429/
> ---
> 
> (Updated Sept. 8, 2014, 9:13 a.m.)
> 
> 
> Review request for cloudstack, Brian Federle and Jessica Wang.
> 
> 
> Repository: cloudstack-git
> 
> 
> Description
> ---
> 
> - Added Agent version and Qemu version fields to the Host detail page, with 
> the values from the host's details field
> - Only displayed for KVM hosts, as the details field is not defined for other 
> hypervisors
> 
> 
> Diffs
> -
> 
>   client/WEB-INF/classes/resources/messages.properties 4655973 
>   ui/dictionary.jsp 0102144 
>   ui/scripts/system.js cf3b9dc 
> 
> Diff: https://reviews.apache.org/r/25429/diff/
> 
> 
> Testing
> ---
> 
> 
> Thanks,
> 
> Mihaela Stoica
> 
>

Re: Review Request 25426: CLOUDSTACK-7370: Fixed password visible in plain text if password passed is in the end in url

[Anshul Gangwar]

---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/25426/
---

(Updated Sept. 10, 2014, 10:34 a.m.)


Review request for cloudstack, Abhinandan Prateek and Devdeep Singh.


Changes
---

updated the patch with required changes


Bugs: CLOUDSTACK-7370
https://issues.apache.org/jira/browse/CLOUDSTACK-7370


Repository: cloudstack-git


Description
---

updated the regular expression which is used to clean password


Diffs (updated)
-

  utils/src/com/cloud/utils/StringUtils.java 17b8fc0 

Diff: https://reviews.apache.org/r/25426/diff/


Testing
---

tested by passing affected strings through regular expressions.


Thanks,

Anshul Gangwar

RE: [TRAVIS] - failing for pull request #18

[Wilder Rodrigues]

Okay…

I’m cloning it in a different place, just to make sure I follow the same steps 
as you.

Once I get it tested… will share my thoughts again. ☺

Thanks again for the support.

Cheers,
Wilder

From: Ian Duffy [mailto:i...@ianduffy.ie]
Sent: woensdag 10 september 2014 12:06
To: Wilder Rodrigues
Cc: CloudStack Dev; int-toolkit; int-cloud
Subject: Re: [TRAVIS] - failing for pull request #18

> You clone the “test-cmd-rebase” branch,  right?

Yes positive:
$ git status
On branch test-cmd-rebase
Your branch is up-to-date with 'origin/test-cmd-rebase'.

Try your work against the simulator.

On 10 September 2014 10:44, Wilder Rodrigues 
mailto:wrodrig...@schubergphilis.com>> wrote:
Thanks Ian,

I will follow the same steps, because it’s really weird.

I’m running with mvn –pl :cloud-ui jetty:run, but no simulator.

Accounts page looks fine, I even executed the accounts related tests. Advance 
zone was also deployed successfully.

The data form my accounts page:

admin  Admin  ROOT  enabled
baremetal-system-account User ROOT  enabled
test-a-TestVpcRemoteAccessVpn-4SYLK3   User ROOT  enabled
test-a-TestVpcRemoteAccessVpn-5WDDUJ User ROOT  enabled

You clone the “test-cmd-rebase” branch,  right?

I was busy running all the tests I have again, the result was successful and I 
also kept tracking the “running.txt” Marvin log file during execution.

The only difference IPs I have in the advanced.cfg, because I do not use the 
ranges that are there.



Test Create Account and user for that account ... === TestName: 
test_01_create_account | Status : SUCCESS ===
ok
Test Sub domain allowed to launch VM  when a Domain level zone is created ... 
=== TestName: test_01_add_vm_to_subdomain | Status : SUCCESS ===
ok
Test delete domain without force option ... === TestName: test_DeleteDomain | 
Status : SUCCESS ===
ok
Test delete domain with force option ... === TestName: test_forceDeleteDomain | 
Status : SUCCESS ===
ok
Test update admin details ... === TestName: test_updateAdminDetails | Status : 
SUCCESS ===
ok
Test update domain admin details ... === TestName: 
test_updateDomainAdminDetails | Status : SUCCESS ===
ok
Test user update API ... === TestName: test_updateUserDetails | Status : 
SUCCESS ===
ok
Test login API with domain ... === TestName: test_LoginApiDomain | Status : 
SUCCESS ===
ok
Test if Login API does not return UUID's ... === TestName: 
test_LoginApiUuidResponse | Status : SUCCESS ===
ok

--
Ran 9 tests in 832.310s

OK

test_privategw_acl (ekholabs.acs.tests.test_privategw_acl.TestPrivateGwACL) ... 
=== TestName: test_privategw_acl | Status : SUCCESS ===
ok

--
Ran 1 test in 100.582s

OK

Test reset virtual machine on reboot ... === TestName: 
test_01_reset_vm_on_reboot | Status : SUCCESS ===
ok

--
Ran 1 test in 183.507s

OK

Test router internal basic zone ... SKIP: Marvin configuration has no host 
credentials to check router services
Test router internal advanced zone ... SKIP: Marvin configuration has no host 
credentials to check router services
Test restart network ... === TestName: test_03_restart_network_cleanup | Status 
: SUCCESS ===
ok
Test restart network without cleanup ... SKIP: Marvin configuration has no host 
credentials to check router services
Test router basic setup ... === TestName: test_05_router_basic | Status : 
SUCCESS ===
ok
Test router advanced setup ... === TestName: test_06_router_advanced | Status : 
SUCCESS ===
ok
Test stop router ... === TestName: test_07_stop_router | Status : SUCCESS ===
ok
Test start router ... === TestName: test_08_start_router | Status : SUCCESS ===
ok
Test reboot router ... === TestName: test_09_reboot_router | Status : SUCCESS 
===
ok

--
Ran 9 tests in 354.245s

OK (SKIP=3)

Test to create service offering ... === TestName: 
test_01_create_service_offering | Status : SUCCESS ===
ok
Test to update existing service offering ... === TestName: 
test_02_edit_service_offering | Status : SUCCESS ===
ok
Test to delete service offering ... === TestName: 
test_03_delete_service_offering | Status : SUCCESS ===
ok

--
Ran 3 tests in 158.207s

OK

Test start/stop of router after addition of one guest network ... === TestName: 
test_01_start_stop_router_after_addition_of_one_guest_network | Status : 
SUCCESS ===
ok
Test reboot of router after addition of one guest network ... === TestName: 
test_02_reboot_router_after_addition_of_one_guest_network | Status : SUCCESS ===
ok
Test migrate of router after addition of one guest network ... SKIP: No host 
available for migration. Test requires atleast 2 hosts
Test to change service offering of router after addition of one guest network 
... === TestName: 
test_04_chg_s

Re: [VOTE] Release Apache CloudStack 4.3.1, round #3

[Rohit Yadav]

Hi Sebastien,

On 09-Sep-2014, at 9:13 pm, sebgoa  wrote:
> Ok, so I would like to get confirmation from other folks on whether this is 
> the right way to do it or not.
> And if it's the current version or not.
>
> I am cc Abhi and animesh who worked on 4.3.0

After my fixes, I see that builds are going fine. At present, all the builds 
are succeeding:
http://jenkins.buildacloud.org/view/4.3/

Using 885c02dbd8b40d654f4a84fca8474ff88bc7ca5e on 4.3, I created new deb/rpm 
packages which built fine and are available here now:
http://packages.bhaisaab.org/cloudstack/main (4.3.1-voting-03 suffixes)

I checked what I did is alright, since in 4.3 branch we build xapi from 
deps/XenServer. I request that you cancel this voting because of the systemvm 
and main artifacts build fixes. (Thanks to Lucian for reporting).

Please start a new voting round with the latest HEAD/4.3 candidate. I hope this 
would be the last re-vote request from my side and we should release 4.3.1 with 
the next round.

Regards,
Rohit Yadav
Software Architect, ShapeBlue
M. +41 779015219 | rohit.ya...@shapeblue.com
Blog: bhaisaab.org | Twitter: @_bhaisaab

Find out more about ShapeBlue and our range of CloudStack related services

IaaS Cloud Design & Build
CSForge – rapid IaaS deployment framework
CloudStack Consulting
CloudStack Infrastructure 
Support
CloudStack Bootcamp Training Courses

This email and any attachments to it may be confidential and are intended 
solely for the use of the individual to whom it is addressed. Any views or 
opinions expressed are solely those of the author and do not necessarily 
represent those of Shape Blue Ltd or related companies. If you are not the 
intended recipient of this email, you must neither take any action based upon 
its contents, nor copy or show it to anyone. Please contact the sender if you 
believe you have received this email in error. Shape Blue Ltd is a company 
incorporated in England & Wales. ShapeBlue Services India LLP is a company 
incorporated in India and is operated under license from Shape Blue Ltd. Shape 
Blue Brasil Consultoria Ltda is a company incorporated in Brasil and is 
operated under license from Shape Blue Ltd. ShapeBlue SA Pty Ltd is a company 
registered by The Republic of South Africa and is traded under license from 
Shape Blue Ltd. ShapeBlue is a registered trademark.

Re: Review Request 25429: CLOUDSTACK-7502: UI: Host detail page - Display KVM agent version and Qemu version

[Mihaela Stoica]

---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/25429/
---

(Updated Sept. 10, 2014, 11:50 a.m.)


Review request for cloudstack, Brian Federle and Jessica Wang.


Repository: cloudstack-git


Description
---

- Added Agent version and Qemu version fields to the Host detail page, with the 
values from the host's details field
- Only displayed for KVM hosts, as the details field is not defined for other 
hypervisors


Diffs
-

  client/WEB-INF/classes/resources/messages.properties 4655973 
  ui/dictionary.jsp 0102144 
  ui/scripts/system.js cf3b9dc 

Diff: https://reviews.apache.org/r/25429/diff/


Testing (updated)
---

Tested with a KVM host: the two fields are displayed.
Tested with a XenServer host: fields not shown.

Screenshots attached to the ticket.


Thanks,

Mihaela Stoica

Re: Review Request 25429: CLOUDSTACK-7502: UI: Host detail page - Display KVM agent version and Qemu version

[Mihaela Stoica]

> On Sept. 10, 2014, 11:32 a.m., Rohit Yadav wrote:
> > Thanks for your contribution, can you share with us if this was tested and 
> > with UI contributions can we see a screenshot with/without KVM hosts.

Thank you for your comments. I've just updated the "testing" section. Also 
attached screenshots to the ticket CLOUDSTACK-7502.


- Mihaela


---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/25429/#review52848
---


On Sept. 10, 2014, 11:50 a.m., Mihaela Stoica wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/25429/
> ---
> 
> (Updated Sept. 10, 2014, 11:50 a.m.)
> 
> 
> Review request for cloudstack, Brian Federle and Jessica Wang.
> 
> 
> Repository: cloudstack-git
> 
> 
> Description
> ---
> 
> - Added Agent version and Qemu version fields to the Host detail page, with 
> the values from the host's details field
> - Only displayed for KVM hosts, as the details field is not defined for other 
> hypervisors
> 
> 
> Diffs
> -
> 
>   client/WEB-INF/classes/resources/messages.properties 4655973 
>   ui/dictionary.jsp 0102144 
>   ui/scripts/system.js cf3b9dc 
> 
> Diff: https://reviews.apache.org/r/25429/diff/
> 
> 
> Testing
> ---
> 
> Tested with a KVM host: the two fields are displayed.
> Tested with a XenServer host: fields not shown.
> 
> Screenshots attached to the ticket.
> 
> 
> Thanks,
> 
> Mihaela Stoica
> 
>

Re: Review Request 25429: CLOUDSTACK-7502: UI: Host detail page - Display KVM agent version and Qemu version

[Mihaela Stoica]

---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/25429/
---

(Updated Sept. 10, 2014, 10:52 a.m.)


Review request for cloudstack, Brian Federle and Jessica Wang.


Bugs: CLOUDSTACK-7502
https://issues.apache.org/jira/browse/CLOUDSTACK-7502


Repository: cloudstack-git


Description
---

- Added Agent version and Qemu version fields to the Host detail page, with the 
values from the host's details field
- Only displayed for KVM hosts, as the details field is not defined for other 
hypervisors


Diffs
-

  client/WEB-INF/classes/resources/messages.properties 4655973 
  ui/dictionary.jsp 0102144 
  ui/scripts/system.js cf3b9dc 

Diff: https://reviews.apache.org/r/25429/diff/


Testing
---

Tested with a KVM host: the two fields are displayed.
Tested with a XenServer host: fields not shown.

Screenshots attached to the ticket.


Thanks,

Mihaela Stoica

Re: Review Request 25429: CLOUDSTACK-7502: UI: Host detail page - Display KVM agent version and Qemu version

[Rohit Yadav]

---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/25429/#review52850
---



ui/scripts/system.js


This probably does not apply for ACS? Is this for CCP?

Can you fix/test with ACS master?


- Rohit Yadav


On Sept. 10, 2014, 10:52 a.m., Mihaela Stoica wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/25429/
> ---
> 
> (Updated Sept. 10, 2014, 10:52 a.m.)
> 
> 
> Review request for cloudstack, Brian Federle and Jessica Wang.
> 
> 
> Bugs: CLOUDSTACK-7502
> https://issues.apache.org/jira/browse/CLOUDSTACK-7502
> 
> 
> Repository: cloudstack-git
> 
> 
> Description
> ---
> 
> - Added Agent version and Qemu version fields to the Host detail page, with 
> the values from the host's details field
> - Only displayed for KVM hosts, as the details field is not defined for other 
> hypervisors
> 
> 
> Diffs
> -
> 
>   client/WEB-INF/classes/resources/messages.properties 4655973 
>   ui/dictionary.jsp 0102144 
>   ui/scripts/system.js cf3b9dc 
> 
> Diff: https://reviews.apache.org/r/25429/diff/
> 
> 
> Testing
> ---
> 
> Tested with a KVM host: the two fields are displayed.
> Tested with a XenServer host: fields not shown.
> 
> Screenshots attached to the ticket.
> 
> 
> Thanks,
> 
> Mihaela Stoica
> 
>

Review Request 25499: CLOUDSTACK-7351: Passing hypervisor type while deploying VM using ISO

[Gaurav Aradhye]

---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/25499/
---

Review request for cloudstack, sanjeev n, Santhosh Edukulla, and 
SrikanteswaraRao Talluri.


Bugs: CLOUDSTACK-7153
https://issues.apache.org/jira/browse/CLOUDSTACK-7153


Repository: cloudstack-git


Description
---

Hypervisor type was not passed while deploying VM using ISO. It must be passed 
for successful deployment.


Diffs
-

  test/integration/component/test_stopped_vm.py 65b9767 

Diff: https://reviews.apache.org/r/25499/diff/


Testing
---

Yes.

Log:
Test Deploy HA enabled Virtual Machine with startvm=false ... SKIP: skip
Test Deploy HA enabled Virtual Machine from ISO ... === TestName: 
test_02_deploy_ha_vm_from_iso | Status : SUCCESS ===
ok
Test Deploy HA enabled Virtual Machine from ISO with startvm=false ... SKIP: 
skip

--
Ran 3 tests in 248.029s

OK (SKIP=2)


Thanks,

Gaurav Aradhye

Jenkins build is still unstable: simulator-singlerun #332

[jenkins]

See 

Re: Review Request 25430: live migration is failing for vm deployed using dynaic compute offerings with NPE

[Devdeep Singh]

---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/25430/#review52852
---

Ship it!


Ship It!

- Devdeep Singh


On Sept. 8, 2014, 1:55 p.m., bharat kumar wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/25430/
> ---
> 
> (Updated Sept. 8, 2014, 1:55 p.m.)
> 
> 
> Review request for cloudstack, Alena Prokharchyk, Devdeep Singh, edison su, 
> Koushik Das, Mike Tutkowski, and Nitin Mehta.
> 
> 
> Bugs: CLOUDSTACK-6099
> https://issues.apache.org/jira/browse/CLOUDSTACK-6099
> 
> 
> Repository: cloudstack-git
> 
> 
> Description
> ---
> 
> live migration is failing for vm deployed using dynaic compute offerings with 
> NPE
> https://issues.apache.org/jira/browse/CLOUDSTACK-6099
> 
> 
> Diffs
> -
> 
>   engine/orchestration/src/com/cloud/vm/VirtualMachineManagerImpl.java 
> 8edef77 
> 
> Diff: https://reviews.apache.org/r/25430/diff/
> 
> 
> Testing
> ---
> 
> Tested live migration on master using xenserver.
> 
> 
> Thanks,
> 
> bharat kumar
> 
>

Re: Review Request 25430: live migration is failing for vm deployed using dynaic compute offerings with NPE

[Devdeep Singh]

> On Sept. 10, 2014, 10:29 a.m., Rohit Yadav wrote:
> > Just want to check with other reviewers, if this contribution is good to 
> > merge on master? Bharat, if you may confirm that it is well tested on 
> > Xen/KVM?

The fix looks correct and it can be merged. The issue was that with custom 
offerings the profile wasn't getting initialized correctly and it lacked the 
cpu and memory details. This would cause a null pointer exception in the 
allocators. I can apply the patch if no one else is working on it.


- Devdeep


---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/25430/#review52846
---


On Sept. 8, 2014, 1:55 p.m., bharat kumar wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/25430/
> ---
> 
> (Updated Sept. 8, 2014, 1:55 p.m.)
> 
> 
> Review request for cloudstack, Alena Prokharchyk, Devdeep Singh, edison su, 
> Koushik Das, Mike Tutkowski, and Nitin Mehta.
> 
> 
> Bugs: CLOUDSTACK-6099
> https://issues.apache.org/jira/browse/CLOUDSTACK-6099
> 
> 
> Repository: cloudstack-git
> 
> 
> Description
> ---
> 
> live migration is failing for vm deployed using dynaic compute offerings with 
> NPE
> https://issues.apache.org/jira/browse/CLOUDSTACK-6099
> 
> 
> Diffs
> -
> 
>   engine/orchestration/src/com/cloud/vm/VirtualMachineManagerImpl.java 
> 8edef77 
> 
> Diff: https://reviews.apache.org/r/25430/diff/
> 
> 
> Testing
> ---
> 
> Tested live migration on master using xenserver.
> 
> 
> Thanks,
> 
> bharat kumar
> 
>

Re: [VOTE] Release Apache CloudStack 4.3.1, round #3

[sebgoa]

On Sep 10, 2014, at 12:50 PM, Rohit Yadav  wrote:

> Hi Sebastien,
> 
> On 09-Sep-2014, at 9:13 pm, sebgoa  wrote:
>> Ok, so I would like to get confirmation from other folks on whether this is 
>> the right way to do it or not.
>> And if it's the current version or not.
>> 
>> I am cc Abhi and animesh who worked on 4.3.0
> 
> After my fixes, I see that builds are going fine. At present, all the builds 
> are succeeding:
> http://jenkins.buildacloud.org/view/4.3/
> 
> Using 885c02dbd8b40d654f4a84fca8474ff88bc7ca5e on 4.3, I created new deb/rpm 
> packages which built fine and are available here now:
> http://packages.bhaisaab.org/cloudstack/main (4.3.1-voting-03 suffixes)
> 
> I checked what I did is alright, since in 4.3 branch we build xapi from 
> deps/XenServer.

Can you point me to what change since 4.3.0 that created the issue...

> I request that you cancel this voting because of the systemvm and main 
> artifacts build fixes. (Thanks to Lucian for reporting).
> 

I see that you put a fix that make the system debian 7.5 instead of 7.0 ….Does 
that constitutes a system upgrade that users will need to take care of ?

> Please start a new voting round with the latest HEAD/4.3 candidate. I hope 
> this would be the last re-vote request from my side and we should release 
> 4.3.1 with the next round.
> 
> Regards,
> Rohit Yadav
> Software Architect, ShapeBlue
> M. +41 779015219 | rohit.ya...@shapeblue.com
> Blog: bhaisaab.org | Twitter: @_bhaisaab
> 
> Find out more about ShapeBlue and our range of CloudStack related services
> 
> IaaS Cloud Design & Build
> CSForge – rapid IaaS deployment framework
> CloudStack Consulting
> CloudStack Infrastructure 
> Support
> CloudStack Bootcamp Training 
> Courses
> 
> This email and any attachments to it may be confidential and are intended 
> solely for the use of the individual to whom it is addressed. Any views or 
> opinions expressed are solely those of the author and do not necessarily 
> represent those of Shape Blue Ltd or related companies. If you are not the 
> intended recipient of this email, you must neither take any action based upon 
> its contents, nor copy or show it to anyone. Please contact the sender if you 
> believe you have received this email in error. Shape Blue Ltd is a company 
> incorporated in England & Wales. ShapeBlue Services India LLP is a company 
> incorporated in India and is operated under license from Shape Blue Ltd. 
> Shape Blue Brasil Consultoria Ltda is a company incorporated in Brasil and is 
> operated under license from Shape Blue Ltd. ShapeBlue SA Pty Ltd is a company 
> registered by The Republic of South Africa and is traded under license from 
> Shape Blue Ltd. ShapeBlue is a registered trademark.

Re: Review Request 25430: live migration is failing for vm deployed using dynaic compute offerings with NPE

[Rohit Yadav]

---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/25430/#review52853
---

Ship it!


Thanks. I merged them on 4.3 and master, sent out request for 4.4;
57c69a8ba02a9d327629fd8665262efc94813d3a 4.3
2b7b837b28f847eedf3cd7bd034dc23ba43d8a63 4.4
56c7fc800ad12e2e3ccbf7fc79aaacd709f036b2 hotfix/4.4/CLOUDSTACK-6099

- Rohit Yadav


On Sept. 8, 2014, 1:55 p.m., bharat kumar wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/25430/
> ---
> 
> (Updated Sept. 8, 2014, 1:55 p.m.)
> 
> 
> Review request for cloudstack, Alena Prokharchyk, Devdeep Singh, edison su, 
> Koushik Das, Mike Tutkowski, and Nitin Mehta.
> 
> 
> Bugs: CLOUDSTACK-6099
> https://issues.apache.org/jira/browse/CLOUDSTACK-6099
> 
> 
> Repository: cloudstack-git
> 
> 
> Description
> ---
> 
> live migration is failing for vm deployed using dynaic compute offerings with 
> NPE
> https://issues.apache.org/jira/browse/CLOUDSTACK-6099
> 
> 
> Diffs
> -
> 
>   engine/orchestration/src/com/cloud/vm/VirtualMachineManagerImpl.java 
> 8edef77 
> 
> Diff: https://reviews.apache.org/r/25430/diff/
> 
> 
> Testing
> ---
> 
> Tested live migration on master using xenserver.
> 
> 
> Thanks,
> 
> bharat kumar
> 
>

Re: [VOTE] Release Apache CloudStack 4.3.1, round #3

[Rohit Yadav]

Hi Sebastien,

On 10-Sep-2014, at 1:27 pm, sebgoa  wrote:

> Can you point me to what change since 4.3.0 that created the issue…

Yes, this commit (short id: 41c0eaa) tried to remove the SNAPSHOT string from 
poms and removed it from deps/XenServer.

https://github.com/apache/cloudstack/commit/41c0eaa14d2f8cd5b58bca71f2254c2390cc6913#diff-03d0955964f7b3c8fddf550c9ffa1aeaL28

>> I request that you cancel this voting because of the systemvm and main 
>> artifacts build fixes. (Thanks to Lucian for reporting).
>>
>
> I see that you put a fix that make the system debian 7.5 instead of 7.0 
> ….Does that constitutes a system upgrade that users will need to take care of 
> ?

The fix (a1d230b88b8ae4e618201928079c0b17c994a3b8) I made was to detect vmware 
ova/vmdk file using a wildcard number matcher [0-9] and not search a disk by 
suffix 1. This fixed the systemvm builds on jenkins.buildacloud.org

By using 7.5.0 release we get the latest stable debian release. There is not 
much difference, as when building systemvms we do a apt-get upgrade as well to 
get latest debian stuff, by using the latest ISO we reduce some bandwidth 
usage. I also see that few fixes went into systemvm/ and agent since 4.3.0 
release, so users should ideally upgrade their systemvms to 4.3.1’s systemvms. 
But I think old systemvms (4.3.0) should work fine with 4.3.1 release as well.

Regards,
Rohit Yadav
Software Architect, ShapeBlue
M. +41 779015219 | rohit.ya...@shapeblue.com
Blog: bhaisaab.org | Twitter: @_bhaisaab

Find out more about ShapeBlue and our range of CloudStack related services

IaaS Cloud Design & Build
CSForge – rapid IaaS deployment framework
CloudStack Consulting
CloudStack Infrastructure 
Support
CloudStack Bootcamp Training Courses

This email and any attachments to it may be confidential and are intended 
solely for the use of the individual to whom it is addressed. Any views or 
opinions expressed are solely those of the author and do not necessarily 
represent those of Shape Blue Ltd or related companies. If you are not the 
intended recipient of this email, you must neither take any action based upon 
its contents, nor copy or show it to anyone. Please contact the sender if you 
believe you have received this email in error. Shape Blue Ltd is a company 
incorporated in England & Wales. ShapeBlue Services India LLP is a company 
incorporated in India and is operated under license from Shape Blue Ltd. Shape 
Blue Brasil Consultoria Ltda is a company incorporated in Brasil and is 
operated under license from Shape Blue Ltd. ShapeBlue SA Pty Ltd is a company 
registered by The Republic of South Africa and is traded under license from 
Shape Blue Ltd. ShapeBlue is a registered trademark.

Re: [VOTE] Release Apache CloudStack 4.3.1, round #3

[Nux!]

So, round 4?

Lucian

--
Sent from the Delta quadrant using Borg technology!

Nux!
www.nux.ro


- Original Message -
> From: "Rohit Yadav" 
> To: dev@cloudstack.apache.org
> Cc: "Abhinandan Prateek" , 
> "animesh.chaturv...@citrix.com Chaturvedi"
> 
> Sent: Wednesday, 10 September, 2014 12:40:18 PM
> Subject: Re: [VOTE] Release Apache CloudStack 4.3.1, round #3
> 
> Hi Sebastien,
> 
> On 10-Sep-2014, at 1:27 pm, sebgoa  wrote:
> 
> > Can you point me to what change since 4.3.0 that created the issue…
> 
> Yes, this commit (short id: 41c0eaa) tried to remove the SNAPSHOT string from
> poms and removed it from deps/XenServer.
> 
> https://github.com/apache/cloudstack/commit/41c0eaa14d2f8cd5b58bca71f2254c2390cc6913#diff-03d0955964f7b3c8fddf550c9ffa1aeaL28
> 
> >> I request that you cancel this voting because of the systemvm and main
> >> artifacts build fixes. (Thanks to Lucian for reporting).
> >>
> >
> > I see that you put a fix that make the system debian 7.5 instead of 7.0
> > ….Does that constitutes a system upgrade that users will need to take care
> > of ?
> 
> The fix (a1d230b88b8ae4e618201928079c0b17c994a3b8) I made was to detect
> vmware ova/vmdk file using a wildcard number matcher [0-9] and not search a
> disk by suffix 1. This fixed the systemvm builds on jenkins.buildacloud.org
> 
> By using 7.5.0 release we get the latest stable debian release. There is not
> much difference, as when building systemvms we do a apt-get upgrade as well
> to get latest debian stuff, by using the latest ISO we reduce some bandwidth
> usage. I also see that few fixes went into systemvm/ and agent since 4.3.0
> release, so users should ideally upgrade their systemvms to 4.3.1’s
> systemvms. But I think old systemvms (4.3.0) should work fine with 4.3.1
> release as well.
> 
> Regards,
> Rohit Yadav
> Software Architect, ShapeBlue
> M. +41 779015219 | rohit.ya...@shapeblue.com
> Blog: bhaisaab.org | Twitter: @_bhaisaab
> 
> Find out more about ShapeBlue and our range of CloudStack related services
> 
> IaaS Cloud Design & Build
> CSForge – rapid IaaS deployment framework
> CloudStack Consulting
> CloudStack Infrastructure
> Support
> CloudStack Bootcamp Training
> Courses
> 
> This email and any attachments to it may be confidential and are intended
> solely for the use of the individual to whom it is addressed. Any views or
> opinions expressed are solely those of the author and do not necessarily
> represent those of Shape Blue Ltd or related companies. If you are not the
> intended recipient of this email, you must neither take any action based
> upon its contents, nor copy or show it to anyone. Please contact the sender
> if you believe you have received this email in error. Shape Blue Ltd is a
> company incorporated in England & Wales. ShapeBlue Services India LLP is a
> company incorporated in India and is operated under license from Shape Blue
> Ltd. Shape Blue Brasil Consultoria Ltda is a company incorporated in Brasil
> and is operated under license from Shape Blue Ltd. ShapeBlue SA Pty Ltd is a
> company registered by The Republic of South Africa and is traded under
> license from Shape Blue Ltd. ShapeBlue is a registered trademark.
> 

Re: [VOTE] Release Apache CloudStack 4.3.1, round #3

[sebgoa]

On Sep 10, 2014, at 1:40 PM, Rohit Yadav  wrote:

> Hi Sebastien,
> 
> On 10-Sep-2014, at 1:27 pm, sebgoa  wrote:
> 
>> Can you point me to what change since 4.3.0 that created the issue…
> 
> Yes, this commit (short id: 41c0eaa) tried to remove the SNAPSHOT string from 
> poms and removed it from deps/XenServer.
> 
> https://github.com/apache/cloudstack/commit/41c0eaa14d2f8cd5b58bca71f2254c2390cc6913#diff-03d0955964f7b3c8fddf550c9ffa1aeaL28
> 

then we need to fix the tools/build/build_asf.sh script

>>> I request that you cancel this voting because of the systemvm and main 
>>> artifacts build fixes. (Thanks to Lucian for reporting).
>>> 
>> 
>> I see that you put a fix that make the system debian 7.5 instead of 7.0 
>> ….Does that constitutes a system upgrade that users will need to take care 
>> of ?
> 
> The fix (a1d230b88b8ae4e618201928079c0b17c994a3b8) I made was to detect 
> vmware ova/vmdk file using a wildcard number matcher [0-9] and not search a 
> disk by suffix 1. This fixed the systemvm builds on jenkins.buildacloud.org
> 
> By using 7.5.0 release we get the latest stable debian release. There is not 
> much difference, as when building systemvms we do a apt-get upgrade as well 
> to get latest debian stuff, by using the latest ISO we reduce some bandwidth 
> usage. I also see that few fixes went into systemvm/ and agent since 4.3.0 
> release, so users should ideally upgrade their systemvms to 4.3.1’s 
> systemvms. But I think old systemvms (4.3.0) should work fine with 4.3.1 
> release as well.

Ok, everybody happy with this ?

> 
> Regards,
> Rohit Yadav
> Software Architect, ShapeBlue
> M. +41 779015219 | rohit.ya...@shapeblue.com
> Blog: bhaisaab.org | Twitter: @_bhaisaab
> 
> Find out more about ShapeBlue and our range of CloudStack related services
> 
> IaaS Cloud Design & Build
> CSForge – rapid IaaS deployment framework
> CloudStack Consulting
> CloudStack Infrastructure 
> Support
> CloudStack Bootcamp Training 
> Courses
> 
> This email and any attachments to it may be confidential and are intended 
> solely for the use of the individual to whom it is addressed. Any views or 
> opinions expressed are solely those of the author and do not necessarily 
> represent those of Shape Blue Ltd or related companies. If you are not the 
> intended recipient of this email, you must neither take any action based upon 
> its contents, nor copy or show it to anyone. Please contact the sender if you 
> believe you have received this email in error. Shape Blue Ltd is a company 
> incorporated in England & Wales. ShapeBlue Services India LLP is a company 
> incorporated in India and is operated under license from Shape Blue Ltd. 
> Shape Blue Brasil Consultoria Ltda is a company incorporated in Brasil and is 
> operated under license from Shape Blue Ltd. ShapeBlue SA Pty Ltd is a company 
> registered by The Republic of South Africa and is traded under license from 
> Shape Blue Ltd. ShapeBlue is a registered trademark.

Review Request 25500: CLOUDSTACK-7134: CLOUDSTACK-7134: Fixed test cases for basic zone, services dict was getting overwritten which passed zone type as advanced even in basic zone setup

[Gaurav Aradhye]

---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/25500/
---

Review request for cloudstack, sanjeev n, Santhosh Edukulla, and 
SrikanteswaraRao Talluri.


Bugs: CLOUDSTACK-7134
https://issues.apache.org/jira/browse/CLOUDSTACK-7134


Repository: cloudstack-git


Description
---

The line self.services = Services.services() in setUp and in test case caused 
the "networktype" in self.services to be overwritten as "advanced". Hence in 
basic zone, these test cases failed while deploying VM (acquiring public IP, 
which is not possible in basic zone).

Removed the lines which causes this defect.


Diffs
-

  test/integration/component/test_reset_ssh_keypair.py 204690e 

Diff: https://reviews.apache.org/r/25500/diff/


Testing
---

Tested on KVM basic zone setup. The VM deployment was successful, but test 
cases failed while SSHing, which is already tracked under separate defect.

Log:
Test Reset SSH keys for VM  already having SSH key ... === TestName: 
test_01_reset_ssh_keys | Status : FAILED ===
FAIL

==
FAIL: Test Reset SSH keys for VM  already having SSH key
--
Traceback (most recent call last):
  File "/root/test/test_reset_ssh_keypair.py", line 360, in 
test_01_reset_ssh_keys
(virtual_machine.name, e))
AssertionError: Failed to SSH into VM with new keypair: 
TestVM-91f66646-aba7-4dc0-90b3-3c1474d5346f, SSH connection has Failed. Waited 
600s. Error is SSH Connection
Failed
 >> begin captured stdout << -
=== TestName: test_01_reset_ssh_keys | Status : FAILED ===


Thanks,

Gaurav Aradhye

Re: Review Request 25471: CLOUDSTACK-7516: Fixed resource permission issue in test_snapshots.py, account was using template registered with other account

[SrikanteswaraRao Talluri]

---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/25471/#review52855
---


I am looking at the reason why this behaviour has changed.
These tests were passing before 5th Sept.

- SrikanteswaraRao Talluri


On Sept. 9, 2014, 7:22 a.m., Gaurav Aradhye wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/25471/
> ---
> 
> (Updated Sept. 9, 2014, 7:22 a.m.)
> 
> 
> Review request for cloudstack, sanjeev n, Santhosh Edukulla, and 
> SrikanteswaraRao Talluri.
> 
> 
> Bugs: CLOUDSTACK-7516
> https://issues.apache.org/jira/browse/CLOUDSTACK-7516
> 
> 
> Repository: cloudstack-git
> 
> 
> Description
> ---
> 
> Template was registered with admin api client, hence there was conflict in 
> permissions available to the user account.
> The test case failed while deploying template with the custom template.
> 
> Error:
> Execute cmd: deployvirtualmachine failed, due to: errorCode: 531, 
> errorText:Acct[51d00171-895e-4893-90c8-6630b98f852a-test-TestCreateVMSnapshotTemplate-BJ9XFN]
>  does not have permission to operate with resource 
> Acct[e7b7973c-3512-11e4-9ac6-1a6f7bb0d0a8-admin]
> 
> Changes:
> Used api client of the user account for all the operations so that there is 
> no conflict in permissions.
> 
> 
> Diffs
> -
> 
>   test/integration/component/test_snapshots.py 8a5c7ff 
> 
> Diff: https://reviews.apache.org/r/25471/diff/
> 
> 
> Testing
> ---
> 
> Yes.
> 
> Ran two test cases separately.
> 
> Log:
> 
> [1]
> Create Template from snapshot ... === TestName: 
> test_07_template_from_snapshot | Status : SUCCESS ===
> ok
> 
> --
> Ran 1 test in 637.534s
> 
> OK
> 
> 
> [2]
> Test create VM, Snapshot and Template ... === TestName: 
> test_01_createVM_snapshotTemplate | Status : SUCCESS ===
> ok
> 
> --
> Ran 1 test in 614.318s
> 
> OK
> 
> 
> Thanks,
> 
> Gaurav Aradhye
> 
>

Build failed in Jenkins: build-master-noredist #3499

[jenkins]

See 

Changes:

[Rohit Yadav] CLOUDSTACK-6099 live migration is failing for vm deployed using 
dynaic compute offerings with NPE

--
[...truncated 5058 lines...]
[INFO] --- maven-site-plugin:3.3:attach-descriptor (attach-descriptor) @ 
cloud-plugin-user-authenticator-md5 ---
[INFO] 
[INFO] --- maven-install-plugin:2.3.1:install (default-install) @ 
cloud-plugin-user-authenticator-md5 ---
[INFO] Installing 

 to 
/jenkins/.m2/repository/org/apache/cloudstack/cloud-plugin-user-authenticator-md5/4.5.0-SNAPSHOT/cloud-plugin-user-authenticator-md5-4.5.0-SNAPSHOT.jar
[INFO] Installing 

 to 
/jenkins/.m2/repository/org/apache/cloudstack/cloud-plugin-user-authenticator-md5/4.5.0-SNAPSHOT/cloud-plugin-user-authenticator-md5-4.5.0-SNAPSHOT.pom
[INFO] 
[INFO] 
[INFO] Building Apache CloudStack Plugin - User Authenticator Plain Text 
4.5.0-SNAPSHOT
[INFO] 
[INFO] 
[INFO] --- maven-clean-plugin:2.5:clean (default-clean) @ 
cloud-plugin-user-authenticator-plaintext ---
[INFO] Deleting 

 (includes = [**/*], excludes = [])
[INFO] Deleting 

 (includes = [target, dist], excludes = [])
[INFO] 
[INFO] --- maven-checkstyle-plugin:2.11:check (cloudstack-checkstyle) @ 
cloud-plugin-user-authenticator-plaintext ---
[INFO] Starting audit...
Audit done.

[INFO] 
[INFO] --- maven-remote-resources-plugin:1.3:process (default) @ 
cloud-plugin-user-authenticator-plaintext ---
[INFO] 
[INFO] --- maven-resources-plugin:2.5:resources (default-resources) @ 
cloud-plugin-user-authenticator-plaintext ---
[debug] execute contextualize
[INFO] Using 'UTF-8' encoding to copy filtered resources.
[INFO] Copying 2 resources
[INFO] Copying 3 resources
[INFO] 
[INFO] --- maven-compiler-plugin:2.5.1:compile (default-compile) @ 
cloud-plugin-user-authenticator-plaintext ---
[INFO] Compiling 1 source file to 

[INFO] 
[INFO] --- maven-resources-plugin:2.5:testResources (default-testResources) @ 
cloud-plugin-user-authenticator-plaintext ---
[debug] execute contextualize
[INFO] Using 'UTF-8' encoding to copy filtered resources.
[INFO] skip non existing resourceDirectory 

[INFO] Copying 3 resources
[INFO] 
[INFO] --- maven-compiler-plugin:2.5.1:testCompile (default-testCompile) @ 
cloud-plugin-user-authenticator-plaintext ---
[INFO] No sources to compile
[INFO] 
[INFO] --- maven-surefire-plugin:2.12:test (default-test) @ 
cloud-plugin-user-authenticator-plaintext ---
[INFO] Surefire report directory: 


---
 T E S T S
---

Results :

Tests run: 0, Failures: 0, Errors: 0, Skipped: 0

[INFO] 
[INFO] --- maven-jar-plugin:2.4:jar (default-jar) @ 
cloud-plugin-user-authenticator-plaintext ---
[INFO] Building jar: 

[INFO] 
[INFO] --- maven-site-plugin:3.3:attach-descriptor (attach-descriptor) @ 
cloud-plugin-user-authenticator-plaintext ---
[INFO] 
[INFO] --- maven-install-plugin:2.3.1:install (default-install) @ 
cloud-plugin-user-authenticator-plaintext ---
[INFO] Installing 

 to 
/jenkins/.m2/repository/org/apache/cloudstack/cloud-plugin-user-authenticator-plaintext/4.5.0-SNAPSHOT/cloud-plugin-user-authenticator-plaintext-4.5.0-SNAPSHOT.jar
[INFO] Installing 

 to 
/jenkins/.m2/repository/org/apache/cloudstack/cloud-plugin-user-authenticator-plaintext/4.5.0-SNAPSHOT/cloud-plugin-user-authenticator-plaintext-4.5.0-SNAPSHOT.pom
[INFO]  

Re: [ACS44][DISCUSS]release 4.4.1

[Sally Khudairi]

Hi folks --how are we looking for the release?

If there are any additions/changes that might need to be reflected in the 
announcement press release, do let me know.

Thanks for this,
Sally
 



 From: Daan Hoogland 
To: dev ; Francois Gaudreault 
 
Cc: Sally Khudairi  
Sent: Monday, 1 September 2014, 10:42
Subject: Re: [ACS44][DISCUSS]release 4.4.1
 


I will start some basic testing myself and have some unrelated $dayjob so an 
ATA of end of workweek is fine by me.

Daan



On Mon, Sep 1, 2014 at 4:39 PM, Francois Gaudreault  
wrote:

Daan,
>
>We are working on fixing SSL offload for projects. Any chance we can slide 
>that in 4.4.1 or should we target 4.5? Assuming we have the code working later 
>this week...
>
>FG
>
>
>On 2014-09-01, 10:30 AM, Wido den Hollander wrote:
>
>
>>
>>On 09/01/2014 04:20 PM, Daan Hoogland wrote:
>>
>>SO May 1st and 1st manday in september are both due to may 4th...
>>>
>>>Nice read Stephen.
>>>
>>>Now back to 4.4.x
>>>
>>>I would like to create a new release if people feel we have something to
>>>release. How is the general population of this list thinking about that? Is
>>>there still focus on the 4.4 branch? Are all major issues from the 4.4.0
>>>release solved?
>>>
>>>
>>Would I say all issues? I don't know. I do know however that all issues that 
>>I encountered have been fixed in 4.4.1
>>
>>Wido
>>
>>
>>thanks for any (encouraging) responds,
>>>Daan
>>>
>>>
>>>On Mon, Sep 1, 2014 at 10:33 AM, Stephen Turner 
>>>wrote:
>>>
>>>
>>>Interestingly, the reason Labour Day is on May 1 in the rest of the world
is the same reason it's not in May in the US and Canada, to do with
communists and anarchists commemorating a riot in Chicago. See
http://en.wikipedia.org/wiki/Labor_Day
http://en.wikipedia.org/wiki/Haymarket_affair

-- 
Stephen Turner


-Original Message-
From: Daan Hoogland [mailto:daan.hoogl...@gmail.com]
Sent: 01 September 2014 09:05
To: dev; Mike Tutkowski
Cc: Francois Gaudreault
Subject: Re: [ACS44][DISCUSS]release 4.4.1

Hey Mike, You guys are not supposed to add random red holidays to your
calendars. You're the commy bashers. May 1st not enough for ya'll?


On Mon, Sep 1, 2014 at 12:00 AM, Mike Tutkowski <
mike.tutkow...@solidfire.com> wrote:


Just an FYI that it's Labor Day Weekend in the U.S., so you might not
>get many, if any, replies until Tuesday.
>
>
>On Sat, Aug 30, 2014 at 2:06 PM, Francois Gaudreault <
>fgaudrea...@cloudops.com> wrote:
>
>
>On 2014-08-30, 4:01 PM, Daan Hoogland wrote:
>>
>>
>>H,
>>>
>>>How is the felling on list about 4.4.1? Do we have a stable enough
>>>
branch
>
>4.4 at this moment? Especially the answer on this of KVM saffy
>>>people is appreciated.
>>>
>>>  I am running on 4.4.1 since a week now. I did hit couple small
>>>bugs,
>>>
but
>
>no real blocker. We also try to fix the SSL offload code to work
>>with projects. We would really really like to get this feature in
>>
4.4.1.


>>FG
>>
>>-- 
>>Francois Gaudreault
>>Gestionnaire de Produit | Product Manager - Cloud Platform &
>>Services
>>t:514-629-6775
>>
>>CloudOps Votre partenaire infonuagique | Cloud Solutions Experts
>>420 rue Guy | Montreal | Quebec | H3J 1S6
>>w: cloudops.com | tw: @CloudOps_
>>
>>
>>
>
>-- 
>*Mike Tutkowski*
>*Senior CloudStack Developer, SolidFire Inc.*
>e: mike.tutkow...@solidfire.com
>o: 303.746.7302
>Advancing the way the world uses the cloud
>*™*
>
>


-- 
Daan


>>>
>>>
>>>
>>
>>
>
>-- 
>Francois Gaudreault
>Gestionnaire de Produit | Product Manager - Cloud Platform & Services
>t:514-629-6775
>
>CloudOps Votre partenaire infonuagique | Cloud Solutions Experts
>420 rue Guy | Montreal | Quebec | H3J 1S6
>w: cloudops.com | tw: @CloudOps_
>
>


-- 
Daan 

Re: Review Request 25429: CLOUDSTACK-7502: UI: Host detail page - Display KVM agent version and Qemu version

[Mihaela Stoica]

---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/25429/
---

(Updated Sept. 10, 2014, 1:19 p.m.)


Review request for cloudstack, Brian Federle and Jessica Wang.


Changes
---

Updated diff: The new fields should only be visible if the host has these 
details.


Bugs: CLOUDSTACK-7502
https://issues.apache.org/jira/browse/CLOUDSTACK-7502


Repository: cloudstack-git


Description
---

- Added Agent version and Qemu version fields to the Host detail page, with the 
values from the host's details field
- Only displayed for KVM hosts, as the details field is not defined for other 
hypervisors


Diffs (updated)
-

  client/WEB-INF/classes/resources/messages.properties 4655973 
  ui/dictionary.jsp 0102144 
  ui/scripts/system.js ed53fd2 

Diff: https://reviews.apache.org/r/25429/diff/


Testing
---

Tested with a KVM host: the two fields are displayed.
Tested with a XenServer host: fields not shown.

Screenshots attached to the ticket.


Thanks,

Mihaela Stoica

Re: [VOTE] Release Apache CloudStack 4.3.1, round #3

[sebgoa]

On Sep 10, 2014, at 1:40 PM, Rohit Yadav  wrote:

> Hi Sebastien,
> 
> On 10-Sep-2014, at 1:27 pm, sebgoa  wrote:
> 
>> Can you point me to what change since 4.3.0 that created the issue…
> 
> Yes, this commit (short id: 41c0eaa) tried to remove the SNAPSHOT string from 
> poms and removed it from deps/XenServer.
> 
> https://github.com/apache/cloudstack/commit/41c0eaa14d2f8cd5b58bca71f2254c2390cc6913#diff-03d0955964f7b3c8fddf550c9ffa1aeaL28
> 

I am still confused by this one. The release build script removed the -SNAPSHOT 
from deps/XenServer/pom.xml  see:

https://github.com/apache/cloudstack/commit/41c0eaa14d2f8cd5b58bca71f2254c2390cc6913#diff-03d0955964f7b3c8fddf550c9ffa1aeaL28

But your fix was in the root pom.xml to remove the -SNAPSHOT as well

https://git-wip-us.apache.org/repos/asf?p=cloudstack.git;a=blobdiff;f=pom.xml;h=83b62e2d4c1eb9d69d4802259d1203c71ca58eab;hp=19e4d35f32c1a516c71bd777dfdda51ab9c65ba2;hb=b6ee5e6455176e75c67b74241b1a9f6160cc62a1;hpb=33911723f851fe9a816c400bb4739539b8087cd3

But animesh added it back in Jan 22:

https://github.com/apache/cloudstack/commit/361aaca794fd885d0b72573a89ad8799bdef301a

So which one is correct for the Xapi artifact ?


>>> I request that you cancel this voting because of the systemvm and main 
>>> artifacts build fixes. (Thanks to Lucian for reporting).
>>> 
>> 
>> I see that you put a fix that make the system debian 7.5 instead of 7.0 
>> ….Does that constitutes a system upgrade that users will need to take care 
>> of ?
> 
> The fix (a1d230b88b8ae4e618201928079c0b17c994a3b8) I made was to detect 
> vmware ova/vmdk file using a wildcard number matcher [0-9] and not search a 
> disk by suffix 1. This fixed the systemvm builds on jenkins.buildacloud.org
> 
> By using 7.5.0 release we get the latest stable debian release. There is not 
> much difference, as when building systemvms we do a apt-get upgrade as well 
> to get latest debian stuff, by using the latest ISO we reduce some bandwidth 
> usage. I also see that few fixes went into systemvm/ and agent since 4.3.0 
> release, so users should ideally upgrade their systemvms to 4.3.1’s 
> systemvms. But I think old systemvms (4.3.0) should work fine with 4.3.1 
> release as well.
> 
> Regards,
> Rohit Yadav
> Software Architect, ShapeBlue
> M. +41 779015219 | rohit.ya...@shapeblue.com
> Blog: bhaisaab.org | Twitter: @_bhaisaab
> 
> Find out more about ShapeBlue and our range of CloudStack related services
> 
> IaaS Cloud Design & Build
> CSForge – rapid IaaS deployment framework
> CloudStack Consulting
> CloudStack Infrastructure 
> Support
> CloudStack Bootcamp Training 
> Courses
> 
> This email and any attachments to it may be confidential and are intended 
> solely for the use of the individual to whom it is addressed. Any views or 
> opinions expressed are solely those of the author and do not necessarily 
> represent those of Shape Blue Ltd or related companies. If you are not the 
> intended recipient of this email, you must neither take any action based upon 
> its contents, nor copy or show it to anyone. Please contact the sender if you 
> believe you have received this email in error. Shape Blue Ltd is a company 
> incorporated in England & Wales. ShapeBlue Services India LLP is a company 
> incorporated in India and is operated under license from Shape Blue Ltd. 
> Shape Blue Brasil Consultoria Ltda is a company incorporated in Brasil and is 
> operated under license from Shape Blue Ltd. ShapeBlue SA Pty Ltd is a company 
> registered by The Republic of South Africa and is traded under license from 
> Shape Blue Ltd. ShapeBlue is a registered trademark.

Re: Review Request 25429: CLOUDSTACK-7502: UI: Host detail page - Display KVM agent version and Qemu version

[Mihaela Stoica]

> On Sept. 10, 2014, 11:55 a.m., Rohit Yadav wrote:
> > ui/scripts/system.js, lines 16106-16107
> > 
> >
> > This probably does not apply for ACS? Is this for CCP?
> > 
> > Can you fix/test with ACS master?

Please see the updated diff. The new fields should only be visible if the host 
has these details.
Thanks.


- Mihaela


---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/25429/#review52850
---


On Sept. 10, 2014, 1:19 p.m., Mihaela Stoica wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/25429/
> ---
> 
> (Updated Sept. 10, 2014, 1:19 p.m.)
> 
> 
> Review request for cloudstack, Brian Federle and Jessica Wang.
> 
> 
> Bugs: CLOUDSTACK-7502
> https://issues.apache.org/jira/browse/CLOUDSTACK-7502
> 
> 
> Repository: cloudstack-git
> 
> 
> Description
> ---
> 
> - Added Agent version and Qemu version fields to the Host detail page, with 
> the values from the host's details field
> - Only displayed for KVM hosts, as the details field is not defined for other 
> hypervisors
> 
> 
> Diffs
> -
> 
>   client/WEB-INF/classes/resources/messages.properties 4655973 
>   ui/dictionary.jsp 0102144 
>   ui/scripts/system.js ed53fd2 
> 
> Diff: https://reviews.apache.org/r/25429/diff/
> 
> 
> Testing
> ---
> 
> Tested with a KVM host: the two fields are displayed.
> Tested with a XenServer host: fields not shown.
> 
> Screenshots attached to the ticket.
> 
> 
> Thanks,
> 
> Mihaela Stoica
> 
>

[GITHUB] turning on comment notifications

[sebgoa]

With pull requests enabled , I thought it would be good to also receive 
comments made in-line in a patch through the github UI.

So don't be surprise to see those emails come in as well.

Of course if this becomes too much email load we can redirect to issues@ or 
commit@ or create a new ML.
And of course we can always turn it off if there is a consensus that this was a 
terrible idea.

-sebastien

Re: Review Request 25429: CLOUDSTACK-7502: UI: Host detail page - Display KVM agent version and Qemu version

[Rohit Yadav]

---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/25429/#review52858
---



ui/scripts/system.js


I need to test this, I think we don't have CCP.* keys in listHosts outputs, 
perhaps it's from Citrix's commercial distribution? Are you using Apache 
CloudStack or not?

In case this has nothing to do with Apache CloudStack, we cannot accept 
this for ACS.


- Rohit Yadav


On Sept. 10, 2014, 12:19 p.m., Mihaela Stoica wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/25429/
> ---
> 
> (Updated Sept. 10, 2014, 12:19 p.m.)
> 
> 
> Review request for cloudstack, Brian Federle and Jessica Wang.
> 
> 
> Bugs: CLOUDSTACK-7502
> https://issues.apache.org/jira/browse/CLOUDSTACK-7502
> 
> 
> Repository: cloudstack-git
> 
> 
> Description
> ---
> 
> - Added Agent version and Qemu version fields to the Host detail page, with 
> the values from the host's details field
> - Only displayed for KVM hosts, as the details field is not defined for other 
> hypervisors
> 
> 
> Diffs
> -
> 
>   client/WEB-INF/classes/resources/messages.properties 4655973 
>   ui/dictionary.jsp 0102144 
>   ui/scripts/system.js ed53fd2 
> 
> Diff: https://reviews.apache.org/r/25429/diff/
> 
> 
> Testing
> ---
> 
> Tested with a KVM host: the two fields are displayed.
> Tested with a XenServer host: fields not shown.
> 
> Screenshots attached to the ticket.
> 
> 
> Thanks,
> 
> Mihaela Stoica
> 
>

Jenkins build is still unstable: simulator-singlerun #333

[jenkins]

See 

Re: [VOTE] Release Apache CloudStack 4.3.1, round #3

[Rohit Yadav]

On 10-Sep-2014, at 2:20 pm, sebgoa  wrote:

> I am still confused by this one. The release build script removed the 
> -SNAPSHOT from deps/XenServer/pom.xml  see:
>
> https://github.com/apache/cloudstack/commit/41c0eaa14d2f8cd5b58bca71f2254c2390cc6913#diff-03d0955964f7b3c8fddf550c9ffa1aeaL28
>
> But your fix was in the root pom.xml to remove the -SNAPSHOT as well
>
> https://git-wip-us.apache.org/repos/asf?p=cloudstack.git;a=blobdiff;f=pom.xml;h=83b62e2d4c1eb9d69d4802259d1203c71ca58eab;hp=19e4d35f32c1a516c71bd777dfdda51ab9c65ba2;hb=b6ee5e6455176e75c67b74241b1a9f6160cc62a1;hpb=33911723f851fe9a816c400bb4739539b8087cd3
>
> But animesh added it back in Jan 22:
>
> https://github.com/apache/cloudstack/commit/361aaca794fd885d0b72573a89ad8799bdef301a
>
> So which one is correct for the Xapi artifact ?

Alright, the way maven and dependencies work is that when you build an 
artifact/jar they get a version and to use this jar we have to specify the same 
version in dependency block. In this case the xapi jar (deps/XenServerJava) has 
some version and we need to use the same version in xen hypervisor plugin 
(plugins/hypervisor/xen/pom.xml) which pick the version string from root 
pom.xml. So, as long as these versions are same at both places, it does not 
matter whether you want to append some prefix since we’re consuming the 
artifact we build from deps/XenServer.

To make this less confusing, I’ve reverted my fix and made the pom in 
deps/XenServerJava to use -SNAPSHOT suffix like it was before.

Regards,
Rohit Yadav
Software Architect, ShapeBlue
M. +41 779015219 | rohit.ya...@shapeblue.com
Blog: bhaisaab.org | Twitter: @_bhaisaab

Find out more about ShapeBlue and our range of CloudStack related services

IaaS Cloud Design & Build
CSForge – rapid IaaS deployment framework
CloudStack Consulting
CloudStack Infrastructure 
Support
CloudStack Bootcamp Training Courses

This email and any attachments to it may be confidential and are intended 
solely for the use of the individual to whom it is addressed. Any views or 
opinions expressed are solely those of the author and do not necessarily 
represent those of Shape Blue Ltd or related companies. If you are not the 
intended recipient of this email, you must neither take any action based upon 
its contents, nor copy or show it to anyone. Please contact the sender if you 
believe you have received this email in error. Shape Blue Ltd is a company 
incorporated in England & Wales. ShapeBlue Services India LLP is a company 
incorporated in India and is operated under license from Shape Blue Ltd. Shape 
Blue Brasil Consultoria Ltda is a company incorporated in Brasil and is 
operated under license from Shape Blue Ltd. ShapeBlue SA Pty Ltd is a company 
registered by The Republic of South Africa and is traded under license from 
Shape Blue Ltd. ShapeBlue is a registered trademark.

Re: Review Request 25429: CLOUDSTACK-7502: UI: Host detail page - Display KVM agent version and Qemu version

[Mihaela Stoica]

> On Sept. 10, 2014, 1:26 p.m., Rohit Yadav wrote:
> > ui/scripts/system.js, lines 15939-15947
> > 
> >
> > I need to test this, I think we don't have CCP.* keys in listHosts 
> > outputs, perhaps it's from Citrix's commercial distribution? Are you using 
> > Apache CloudStack or not?
> > 
> > In case this has nothing to do with Apache CloudStack, we cannot accept 
> > this for ACS.

This change has been developed by Citrix, but I thought that these new values 
will be added to the acs master. I will check and update the review request 
accordingly.


- Mihaela


---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/25429/#review52858
---


On Sept. 10, 2014, 1:19 p.m., Mihaela Stoica wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/25429/
> ---
> 
> (Updated Sept. 10, 2014, 1:19 p.m.)
> 
> 
> Review request for cloudstack, Brian Federle and Jessica Wang.
> 
> 
> Bugs: CLOUDSTACK-7502
> https://issues.apache.org/jira/browse/CLOUDSTACK-7502
> 
> 
> Repository: cloudstack-git
> 
> 
> Description
> ---
> 
> - Added Agent version and Qemu version fields to the Host detail page, with 
> the values from the host's details field
> - Only displayed for KVM hosts, as the details field is not defined for other 
> hypervisors
> 
> 
> Diffs
> -
> 
>   client/WEB-INF/classes/resources/messages.properties 4655973 
>   ui/dictionary.jsp 0102144 
>   ui/scripts/system.js ed53fd2 
> 
> Diff: https://reviews.apache.org/r/25429/diff/
> 
> 
> Testing
> ---
> 
> Tested with a KVM host: the two fields are displayed.
> Tested with a XenServer host: fields not shown.
> 
> Screenshots attached to the ticket.
> 
> 
> Thanks,
> 
> Mihaela Stoica
> 
>

Re: Review Request 25500: CLOUDSTACK-7134: CLOUDSTACK-7134: Fixed test cases for basic zone, services dict was getting overwritten which passed zone type as advanced even in basic zone setup

[SrikanteswaraRao Talluri]

---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/25500/#review52861
---

Ship it!


Good catch. This has been failing for a long time.

- SrikanteswaraRao Talluri


On Sept. 10, 2014, 11:52 a.m., Gaurav Aradhye wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/25500/
> ---
> 
> (Updated Sept. 10, 2014, 11:52 a.m.)
> 
> 
> Review request for cloudstack, sanjeev n, Santhosh Edukulla, and 
> SrikanteswaraRao Talluri.
> 
> 
> Bugs: CLOUDSTACK-7134
> https://issues.apache.org/jira/browse/CLOUDSTACK-7134
> 
> 
> Repository: cloudstack-git
> 
> 
> Description
> ---
> 
> The line self.services = Services.services() in setUp and in test case caused 
> the "networktype" in self.services to be overwritten as "advanced". Hence in 
> basic zone, these test cases failed while deploying VM (acquiring public IP, 
> which is not possible in basic zone).
> 
> Removed the lines which causes this defect.
> 
> 
> Diffs
> -
> 
>   test/integration/component/test_reset_ssh_keypair.py 204690e 
> 
> Diff: https://reviews.apache.org/r/25500/diff/
> 
> 
> Testing
> ---
> 
> Tested on KVM basic zone setup. The VM deployment was successful, but test 
> cases failed while SSHing, which is already tracked under separate defect.
> 
> Log:
> Test Reset SSH keys for VM  already having SSH key ... === TestName: 
> test_01_reset_ssh_keys | Status : FAILED ===
> FAIL
> 
> ==
> FAIL: Test Reset SSH keys for VM  already having SSH key
> --
> Traceback (most recent call last):
>   File "/root/test/test_reset_ssh_keypair.py", line 360, in 
> test_01_reset_ssh_keys
> (virtual_machine.name, e))
> AssertionError: Failed to SSH into VM with new keypair: 
> TestVM-91f66646-aba7-4dc0-90b3-3c1474d5346f, SSH connection has Failed. 
> Waited 600s. Error is SSH Connection
> Failed
>  >> begin captured stdout << -
> === TestName: test_01_reset_ssh_keys | Status : FAILED ===
> 
> 
> Thanks,
> 
> Gaurav Aradhye
> 
>

Re: [VOTE] Release Apache CloudStack 4.3.1, round #3

[sebgoa]

On Sep 10, 2014, at 2:36 PM, Rohit Yadav  wrote:

> 
> On 10-Sep-2014, at 2:20 pm, sebgoa  wrote:
> 
>> I am still confused by this one. The release build script removed the 
>> -SNAPSHOT from deps/XenServer/pom.xml  see:
>> 
>> https://github.com/apache/cloudstack/commit/41c0eaa14d2f8cd5b58bca71f2254c2390cc6913#diff-03d0955964f7b3c8fddf550c9ffa1aeaL28
>> 
>> But your fix was in the root pom.xml to remove the -SNAPSHOT as well
>> 
>> https://git-wip-us.apache.org/repos/asf?p=cloudstack.git;a=blobdiff;f=pom.xml;h=83b62e2d4c1eb9d69d4802259d1203c71ca58eab;hp=19e4d35f32c1a516c71bd777dfdda51ab9c65ba2;hb=b6ee5e6455176e75c67b74241b1a9f6160cc62a1;hpb=33911723f851fe9a816c400bb4739539b8087cd3
>> 
>> But animesh added it back in Jan 22:
>> 
>> https://github.com/apache/cloudstack/commit/361aaca794fd885d0b72573a89ad8799bdef301a
>> 
>> So which one is correct for the Xapi artifact ?
> 
> Alright, the way maven and dependencies work is that when you build an 
> artifact/jar they get a version and to use this jar we have to specify the 
> same version in dependency block. In this case the xapi jar 
> (deps/XenServerJava) has some version and we need to use the same version in 
> xen hypervisor plugin (plugins/hypervisor/xen/pom.xml) which pick the version 
> string from root pom.xml. So, as long as these versions are same at both 
> places, it does not matter whether you want to append some prefix since we’re 
> consuming the artifact we build from deps/XenServer.
> 

Ok thanks for the explanation, I just would like confirmation that we should 
not have the -SNAPSHOT

> To make this less confusing, I’ve reverted my fix and made the pom in 
> deps/XenServerJava to use -SNAPSHOT suffix like it was before.
> 

I think you went too fast, there is no need to revert it if that's correct. The 
build script will remove the -SNAPSHOT from the deps/XenServerJava/pom.xml, so 
it seems the way to avoid this in the future is really to remove the 
-SNAPSHOT...



> Regards,
> Rohit Yadav
> Software Architect, ShapeBlue
> M. +41 779015219 | rohit.ya...@shapeblue.com
> Blog: bhaisaab.org | Twitter: @_bhaisaab
> 
> Find out more about ShapeBlue and our range of CloudStack related services
> 
> IaaS Cloud Design & Build
> CSForge – rapid IaaS deployment framework
> CloudStack Consulting
> CloudStack Infrastructure 
> Support
> CloudStack Bootcamp Training 
> Courses
> 
> This email and any attachments to it may be confidential and are intended 
> solely for the use of the individual to whom it is addressed. Any views or 
> opinions expressed are solely those of the author and do not necessarily 
> represent those of Shape Blue Ltd or related companies. If you are not the 
> intended recipient of this email, you must neither take any action based upon 
> its contents, nor copy or show it to anyone. Please contact the sender if you 
> believe you have received this email in error. Shape Blue Ltd is a company 
> incorporated in England & Wales. ShapeBlue Services India LLP is a company 
> incorporated in India and is operated under license from Shape Blue Ltd. 
> Shape Blue Brasil Consultoria Ltda is a company incorporated in Brasil and is 
> operated under license from Shape Blue Ltd. ShapeBlue SA Pty Ltd is a company 
> registered by The Republic of South Africa and is traded under license from 
> Shape Blue Ltd. ShapeBlue is a registered trademark.

Re: Review Request 25500: CLOUDSTACK-7134: CLOUDSTACK-7134: Fixed test cases for basic zone, services dict was getting overwritten which passed zone type as advanced even in basic zone setup

[SrikanteswaraRao Talluri]

---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/25500/#review52862
---


Please resubmit the patch with the correct author.

It shows like this 

commit dbb7392cce2f5da014d064bc82f4113ee5c7d2aa
Author: Girish Shilamkar 
Date:   Wed Sep 10 16:29:22 2014 +0530

CLOUDSTACK-7134: Fixed test cases for basic zone, services dict was getting 
overwritten which passed zone type as advanced even in basic zone setup

- SrikanteswaraRao Talluri


On Sept. 10, 2014, 11:52 a.m., Gaurav Aradhye wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/25500/
> ---
> 
> (Updated Sept. 10, 2014, 11:52 a.m.)
> 
> 
> Review request for cloudstack, sanjeev n, Santhosh Edukulla, and 
> SrikanteswaraRao Talluri.
> 
> 
> Bugs: CLOUDSTACK-7134
> https://issues.apache.org/jira/browse/CLOUDSTACK-7134
> 
> 
> Repository: cloudstack-git
> 
> 
> Description
> ---
> 
> The line self.services = Services.services() in setUp and in test case caused 
> the "networktype" in self.services to be overwritten as "advanced". Hence in 
> basic zone, these test cases failed while deploying VM (acquiring public IP, 
> which is not possible in basic zone).
> 
> Removed the lines which causes this defect.
> 
> 
> Diffs
> -
> 
>   test/integration/component/test_reset_ssh_keypair.py 204690e 
> 
> Diff: https://reviews.apache.org/r/25500/diff/
> 
> 
> Testing
> ---
> 
> Tested on KVM basic zone setup. The VM deployment was successful, but test 
> cases failed while SSHing, which is already tracked under separate defect.
> 
> Log:
> Test Reset SSH keys for VM  already having SSH key ... === TestName: 
> test_01_reset_ssh_keys | Status : FAILED ===
> FAIL
> 
> ==
> FAIL: Test Reset SSH keys for VM  already having SSH key
> --
> Traceback (most recent call last):
>   File "/root/test/test_reset_ssh_keypair.py", line 360, in 
> test_01_reset_ssh_keys
> (virtual_machine.name, e))
> AssertionError: Failed to SSH into VM with new keypair: 
> TestVM-91f66646-aba7-4dc0-90b3-3c1474d5346f, SSH connection has Failed. 
> Waited 600s. Error is SSH Connection
> Failed
>  >> begin captured stdout << -
> === TestName: test_01_reset_ssh_keys | Status : FAILED ===
> 
> 
> Thanks,
> 
> Gaurav Aradhye
> 
>

Re: Review Request 25499: CLOUDSTACK-7351: Passing hypervisor type while deploying VM using ISO

[SrikanteswaraRao Talluri]

---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/25499/#review52866
---

Ship it!


pushed to master 3411fbaccafbce8f99af4714a26f38a2c17d8a9e

- SrikanteswaraRao Talluri


On Sept. 10, 2014, 11:13 a.m., Gaurav Aradhye wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/25499/
> ---
> 
> (Updated Sept. 10, 2014, 11:13 a.m.)
> 
> 
> Review request for cloudstack, sanjeev n, Santhosh Edukulla, and 
> SrikanteswaraRao Talluri.
> 
> 
> Bugs: CLOUDSTACK-7153
> https://issues.apache.org/jira/browse/CLOUDSTACK-7153
> 
> 
> Repository: cloudstack-git
> 
> 
> Description
> ---
> 
> Hypervisor type was not passed while deploying VM using ISO. It must be 
> passed for successful deployment.
> 
> 
> Diffs
> -
> 
>   test/integration/component/test_stopped_vm.py 65b9767 
> 
> Diff: https://reviews.apache.org/r/25499/diff/
> 
> 
> Testing
> ---
> 
> Yes.
> 
> Log:
> Test Deploy HA enabled Virtual Machine with startvm=false ... SKIP: skip
> Test Deploy HA enabled Virtual Machine from ISO ... === TestName: 
> test_02_deploy_ha_vm_from_iso | Status : SUCCESS ===
> ok
> Test Deploy HA enabled Virtual Machine from ISO with startvm=false ... SKIP: 
> skip
> 
> --
> Ran 3 tests in 248.029s
> 
> OK (SKIP=2)
> 
> 
> Thanks,
> 
> Gaurav Aradhye
> 
>

Re: [VOTE] Release Apache CloudStack 4.3.1, round #3

[sebgoa]

On Sep 10, 2014, at 2:41 PM, sebgoa  wrote:

> 
> On Sep 10, 2014, at 2:36 PM, Rohit Yadav  wrote:
> 
>> 
>> On 10-Sep-2014, at 2:20 pm, sebgoa  wrote:
>> 
>>> I am still confused by this one. The release build script removed the 
>>> -SNAPSHOT from deps/XenServer/pom.xml  see:
>>> 
>>> https://github.com/apache/cloudstack/commit/41c0eaa14d2f8cd5b58bca71f2254c2390cc6913#diff-03d0955964f7b3c8fddf550c9ffa1aeaL28
>>> 
>>> But your fix was in the root pom.xml to remove the -SNAPSHOT as well
>>> 
>>> https://git-wip-us.apache.org/repos/asf?p=cloudstack.git;a=blobdiff;f=pom.xml;h=83b62e2d4c1eb9d69d4802259d1203c71ca58eab;hp=19e4d35f32c1a516c71bd777dfdda51ab9c65ba2;hb=b6ee5e6455176e75c67b74241b1a9f6160cc62a1;hpb=33911723f851fe9a816c400bb4739539b8087cd3
>>> 
>>> But animesh added it back in Jan 22:
>>> 
>>> https://github.com/apache/cloudstack/commit/361aaca794fd885d0b72573a89ad8799bdef301a
>>> 
>>> So which one is correct for the Xapi artifact ?
>> 
>> Alright, the way maven and dependencies work is that when you build an 
>> artifact/jar they get a version and to use this jar we have to specify the 
>> same version in dependency block. In this case the xapi jar 
>> (deps/XenServerJava) has some version and we need to use the same version in 
>> xen hypervisor plugin (plugins/hypervisor/xen/pom.xml) which pick the 
>> version string from root pom.xml. So, as long as these versions are same at 
>> both places, it does not matter whether you want to append some prefix since 
>> we’re consuming the artifact we build from deps/XenServer.
>> 
> 
> Ok thanks for the explanation, I just would like confirmation that we should 
> not have the -SNAPSHOT
> 
>> To make this less confusing, I’ve reverted my fix and made the pom in 
>> deps/XenServerJava to use -SNAPSHOT suffix like it was before.
>> 
> 
> I think you went too fast, there is no need to revert it if that's correct. 
> The build script will remove the -SNAPSHOT from the 
> deps/XenServerJava/pom.xml, so it seems the way to avoid this in the future 
> is really to remove the -SNAPSHOT…
> 

Found the proper fix in 4.4 from Daan:

https://git-wip-us.apache.org/repos/asf?p=cloudstack.git;a=blobdiff;f=tools/build/build_asf.sh;h=ea9b272d1d32a107ce06362efb163b6295969be1;hp=72f2111c303abe36f299d8f6a882a34439e33103;hb=58a4c7a89158ece348d890774c866e02f52e5d80;hpb=7fe353d4995a2b55c7baed5e27056dd2e2f7509f

Even though if we don't use -SNAPSHOT for this artifact in the first place, we 
should never face the issue…:(

> 
> 
>> Regards,
>> Rohit Yadav
>> Software Architect, ShapeBlue
>> M. +41 779015219 | rohit.ya...@shapeblue.com
>> Blog: bhaisaab.org | Twitter: @_bhaisaab
>> 
>> Find out more about ShapeBlue and our range of CloudStack related services
>> 
>> IaaS Cloud Design & Build
>> CSForge – rapid IaaS deployment framework
>> CloudStack Consulting
>> CloudStack Infrastructure 
>> Support
>> CloudStack Bootcamp Training 
>> Courses
>> 
>> This email and any attachments to it may be confidential and are intended 
>> solely for the use of the individual to whom it is addressed. Any views or 
>> opinions expressed are solely those of the author and do not necessarily 
>> represent those of Shape Blue Ltd or related companies. If you are not the 
>> intended recipient of this email, you must neither take any action based 
>> upon its contents, nor copy or show it to anyone. Please contact the sender 
>> if you believe you have received this email in error. Shape Blue Ltd is a 
>> company incorporated in England & Wales. ShapeBlue Services India LLP is a 
>> company incorporated in India and is operated under license from Shape Blue 
>> Ltd. Shape Blue Brasil Consultoria Ltda is a company incorporated in Brasil 
>> and is operated under license from Shape Blue Ltd. ShapeBlue SA Pty Ltd is a 
>> company registered by The Republic of South Africa and is traded under 
>> license from Shape Blue Ltd. ShapeBlue is a registered trademark.
> 

[GitHub] cloudstack pull request: VPC and Virtual Network Managers refactor...

[runseb]

Github user runseb commented on a diff in the pull request:

https://github.com/apache/cloudstack/pull/18#discussion_r17358853
  
--- Diff: 
plugins/network-elements/ovs/src/com/cloud/network/element/OvsElement.java ---
@@ -74,9 +74,11 @@
 import com.cloud.vm.DomainRouterVO;
 import com.cloud.vm.NicProfile;
 import com.cloud.vm.ReservationContext;
+import com.cloud.vm.VirtualMachine;
 import com.cloud.vm.VirtualMachineProfile;
 import com.cloud.vm.dao.DomainRouterDao;
-import com.cloud.vm.VirtualMachine;
+import com.cloud.vm.dao.UserVmDao;
+import com.google.gson.Gson;
--- End diff --

Bogus comment, do not reply: Why did you move this import down ?
Le'ts see if I can reply from the dev@ list 


---
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 enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

Jenkins build is back to normal : build-master-noredist #3500

[jenkins]

See 

Re: [GitHub] cloudstack pull request: VPC and Virtual Network Managers refactor...

[sebgoa]

[discard, this is a test]

Because I like alphabetical order ???
But maybe I missed an empty line..who cares..

On Sep 10, 2014, at 3:02 PM, runseb  wrote:

> Github user runseb commented on a diff in the pull request:
> 
>https://github.com/apache/cloudstack/pull/18#discussion_r17358853
> 
>--- Diff: 
> plugins/network-elements/ovs/src/com/cloud/network/element/OvsElement.java ---
>@@ -74,9 +74,11 @@
> import com.cloud.vm.DomainRouterVO;
> import com.cloud.vm.NicProfile;
> import com.cloud.vm.ReservationContext;
>+import com.cloud.vm.VirtualMachine;
> import com.cloud.vm.VirtualMachineProfile;
> import com.cloud.vm.dao.DomainRouterDao;
>-import com.cloud.vm.VirtualMachine;
>+import com.cloud.vm.dao.UserVmDao;
>+import com.google.gson.Gson;
>--- End diff --
> 
>Bogus comment, do not reply: Why did you move this import down ?
>Le'ts see if I can reply from the dev@ list 
> 
> 
> ---
> 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 enabled but not working, please
> contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
> with INFRA.
> ---

Re: Review Request 23856: Fix VPC distributed routing integration tests

[SrikanteswaraRao Talluri]

---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/23856/#review52867
---


Except for the comment I provided, Patch looks good to me.

I am not able to apply this patch cleanly. Can you please recreate the patch 
against latest master?

Applying: Fix vpc distributed routing tests: - Use local setup in the tests - 
Enable Ovs network provider during setup - Remove space in supported services 
config
/Users/talluri/asf/cloudstack/.git/rebase-apply/patch:71: trailing whitespace.

/Users/talluri/asf/cloudstack/.git/rebase-apply/patch:191: trailing whitespace.

/Users/talluri/asf/cloudstack/.git/rebase-apply/patch:196: trailing whitespace.

/Users/talluri/asf/cloudstack/.git/rebase-apply/patch:202: trailing whitespace.

error: test/integration/component/test_vpc_distributed_routing_offering.py: 
does not match index
error: tools/marvin/marvin/lib/common.py: does not match index
Patch failed at 0001 Fix vpc distributed routing tests: - Use local setup in 
the tests - Enable Ovs network provider during setup - Remove space in 
supported services config
The copy of the patch that failed is found in:
   /Users/talluri/asf/cloudstack/.git/rebase-apply/patch
When you have resolved this problem, run "git am --resolved".
If you prefer to skip this patch, run "git am --skip" instead.
To restore the original branch and stop patching, run "git am --abort".


tools/marvin/marvin/lib/common.py


What if it is an empty list? better to check for empty list.

you can make use of 
validateList() from tools/marvin/marvin/lib/utils.py.


- SrikanteswaraRao Talluri


On July 24, 2014, 1:46 p.m., Miguel Ferreira wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/23856/
> ---
> 
> (Updated July 24, 2014, 1:46 p.m.)
> 
> 
> Review request for cloudstack, daan Hoogland, Murali Reddy, Hugo Trippaers, 
> and SrikanteswaraRao Talluri.
> 
> 
> Repository: cloudstack-git
> 
> 
> Description
> ---
> 
> 1) The first test in the class is failing on asserting that distributed 
> routing is enabled:
> That was because it is using the global services configuration instead of the 
> local one (where the distributed routing is enabled).
> I've changed that and also added two new lines in between every python method 
> to make the class more readable.
> 
> See stack trace:
> Traceback (most recent call last):
>   File 
> "/usr/local/Cellar/python/2.7.8/Frameworks/Python.framework/Versions/2.7/lib/python2.7/unittest/case.py",
>  line 329, in run
> testMethod()
>   File 
> "/Users/mferreira/development/workspace/cloudstack-sbp-vpc-tests/src/vpc-tests/all/test_vpc_distributed_routing_offering.py",
>  line 292, in 
> test_01_create_vpc_offering_with_distributedrouter_service_capability
> self.validate_vpc_offering(vpc_off)
>   File 
> "/Users/mferreira/development/workspace/cloudstack-sbp-vpc-tests/src/vpc-tests/all/test_vpc_distributed_routing_offering.py",
>  line 245, in validate_vpc_offering
> "VPC offering is not set up for Distributed routing"
>   File 
> "/usr/local/Cellar/python/2.7.8/Frameworks/Python.framework/Versions/2.7/lib/python2.7/unittest/case.py",
>  line 513, in assertEqual
> assertion_func(first, second, msg=msg)
>   File 
> "/usr/local/Cellar/python/2.7.8/Frameworks/Python.framework/Versions/2.7/lib/python2.7/unittest/case.py",
>  line 506, in _baseAssertEqual
> raise self.failureException(msg)
> AssertionError: VPC offering is not set up for Distributed routing
>  >> begin captured stdout << -
> === TestName: 
> test_01_create_vpc_offering_with_distributedrouter_service_capability | 
> Status : FAILED ===
> 
> 
> 2) The second test was failing because the connectivity service was not being 
> enabled in te offering due to a typo.
> 
> See stack trace:
> Traceback (most recent call last):
>   File 
> "/usr/local/Cellar/python/2.7.8/Frameworks/Python.framework/Versions/2.7/lib/python2.7/unittest/case.py",
>  line 329, in run
> testMethod()
>   File 
> "/Users/mferreira/development/workspace/cloudstack-sbp-vpc-tests/src/vpc-tests/all/test_vpc_distributed_routing_offering.py",
>  line 369, in test_03_deploy_vms_in_vpc_with_distributedrouter
> conservemode=False
>   File 
> "/Users/mferreira/development/git/cloudstack-sbp/tools/marvin/marvin/lib/base.py",
>  line 2028, in create
> return NetworkOffering(apiclient.createNetworkOffering(cmd).__dict__)
>   File 
> "/Users/mferreira/development/git/cloudstack-sbp/tools/marvin/marvin/cloudstackAPI/cloudstackAPIClient.py",
>  line 1778, in createNetworkOffering
> response = self.connection.marvinRequest(comma

Re: [DRAFT] Report Q3 Apache Cloudstack

[Chip Childers]

+1

On Wed, Sep 10, 2014 at 3:38 AM, Hugo Trippaers  wrote:
> Hey All,
>
> This is the draft board report for Apache CloudStack for this quarter.
>
> Please let me know if you have any feedback.
>
> Cheers,
>
> Hugo
>
>
> DESCRIPTION
> Apache CloudStack is open source software designed to deploy and manage
> large networks of virtual machines, as a highly available, highly scalable
> Infrastructure as a Service (IaaS) cloud computing platform.
>
> ISSUES
>
> None
>
> CURRENT ACTIVITY
> * The community is working on maintenance releases for both the 4.3.0 and
>   the 4.4.0 releases.
> * The agreed release schedule is not met at the moment as we regularly 
> identify
>   quality issues in pending releases. This is an ongoing process, but slowly 
> we are
>   improving our QA processes and introducting more testing capabilities.
> * Apache CloudStack CloudMonkey 5.2.0 was released on Aug 28 2014. Apache
>   CloudStack CloudMonkey is a CLI tool for interacting with Apache
>   CloudStack and is distributed separate from Apache CloudStack
> * Apache CloudStack 4.4.0 was released July 26 2014. Some issues were
>   identified with this release after the release. The comunity decided
>   not to market this release to a bid audience and work on a maintenance
>   release immediately.
> * The community is working with The Linux Foundation to prepare
>   the second European CloudStack Collaboration Conference
>   (CCCEU). CCCEU will planned following ApacheCon in
>   Budapest from November 19 till 21. It will have a new format
>   which will hopefully engage more users and developers.
>
> RELEASES
> Apache CloudStack 4.4.0 was released on July 26, 2014.
> Apache CloudStack CloudMonkey 5.2.0 was released on Aug 28, 2014
>
> COMMUNITY
> Including the following additions, CloudStack has 96 (+5) committers and 29 
> (+1)
> PMC members.
>
> New Committers:
> Amogh Vasekar (amoghvk) - May 29
> Pierre-Luc Dion (pdion891) - June 5
> Will Stevens (swill) - June 16
> Santhosh Edukulla (santhoshedukulla) - June 24
> Rajani Karuturi (rajani) - July 17
>
> New PMC Members:
> Mark R. Hinkle (mrhinkle) - July 3
>
> The Apache CloudStack project remains a high volume community:
>
> //FIXME Old subscribers figures, volumes updated.
> dev@ 761 729(+32) subs / msgs = Jun: 1852, Jul: 1555, Aug: 1613
> users@ 1042(-59) subs / msgs = Jun: 663, Jul: 710, Aug: 640
> issues@ 192(+15) subs / msgs = Jun: 1386, Jul: 1381, Aug: 1635
> commits@ 202(+8) subs / msgs = Jun: 700, Jul: 790, Aug: 808
> marketing@ 178(+15) subs / msgs = Jun: 251, Jul: 189, Aug:99
> users-cn@ 508(+40) subs / msgs = Jun: 54, Jul: 133, Aug: 73
>

Re: Review Request 25293: Test script to verify vm deployment with two networks in SG enabled advanced zone

[SrikanteswaraRao Talluri]

---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/25293/#review52868
---



test/integration/component/test_advancedsg_networks.py


it is better to have a condition for windows and then final else for "other 
OS"



tools/marvin/marvin/config/test_data.py


it would be nice to point to where/how we can get a sample advancedsg.cfg 
file


- SrikanteswaraRao Talluri


On Sept. 3, 2014, 9:58 a.m., sanjeev n wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/25293/
> ---
> 
> (Updated Sept. 3, 2014, 9:58 a.m.)
> 
> 
> Review request for cloudstack, suresh sadhu, Santhosh Edukulla, and 
> SrikanteswaraRao Talluri.
> 
> 
> Repository: cloudstack-git
> 
> 
> Description
> ---
> 
> Test script to verify vm deployment with two networks in SG enabled advanced 
> zone.
> Involves following steps:
> 1.Deploy SG enabled advanced zone with only 4 ip addresses in guest ip range 
> using simulator
> 2.Deploy one guest vm in the default shared guest network so that all the IP 
> addresses will be consumed
> 3.Add another guest network in the zone created at step1
> 4.Deploy vm without passing the network id. It should pick the network with 
> sufficient resources
> 
> 
> Diffs
> -
> 
>   test/integration/component/test_advancedsg_networks.py 85ab697 
>   tools/marvin/marvin/config/test_data.py b658b01 
> 
> Diff: https://reviews.apache.org/r/25293/diff/
> 
> 
> Testing
> ---
> 
> @Desc: VM Cannot deploy to second network in advanced SG network ... === 
> TestName: test_34_DeployVM_in_SecondSGNetwork | Status : SUCCESS ===
> ok
> 
> --
> Ran 1 test in 343.198s
> 
> OK
> 
> 
> Thanks,
> 
> sanjeev n
> 
>

[GitHub] cloudstack commit comment: 41c0eaa14d2f8cd5b58bca71f2254c2390cc691...

[Humbedooh]

Github user Humbedooh commented on the pull request:


https://github.com/apache/cloudstack/commit/41c0eaa14d2f8cd5b58bca71f2254c2390cc6913#commitcomment-7731347
  
In deps/XenServerJava/pom.xml:
In deps/XenServerJava/pom.xml on line 28:
one more test :)


---
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 enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

Re: [VOTE] Release Apache CloudStack 4.3.1, round #3

[Rohit Yadav]

Hi Sebastien,

On 10-Sep-2014, at 2:54 pm, sebgoa  wrote:
> Found the proper fix in 4.4 from Daan:
>
> https://git-wip-us.apache.org/repos/asf?p=cloudstack.git;a=blobdiff;f=tools/build/build_asf.sh;h=ea9b272d1d32a107ce06362efb163b6295969be1;hp=72f2111c303abe36f299d8f6a882a34439e33103;hb=58a4c7a89158ece348d890774c866e02f52e5d80;hpb=7fe353d4995a2b55c7baed5e27056dd2e2f7509f
>
> Even though if we don't use -SNAPSHOT for this artifact in the first place, 
> we should never face the issue…:(

Sorry about the quick fix, I did not know that the build_asf script removes 
SNAPSHOT from all poms.

Looks like Daan is removing the -SNAPSHOT from root pom.xml, which is something 
I did too. You may pick this for 4.3 branch, for change the version string to 
5.6.100-1 etc. I’ll leave that for you to decide/fix.

Regards,
Rohit Yadav
Software Architect, ShapeBlue
M. +41 779015219 | rohit.ya...@shapeblue.com
Blog: bhaisaab.org | Twitter: @_bhaisaab



Find out more about ShapeBlue and our range of CloudStack related services

IaaS Cloud Design & Build
CSForge – rapid IaaS deployment framework
CloudStack Consulting
CloudStack Infrastructure 
Support
CloudStack Bootcamp Training Courses

This email and any attachments to it may be confidential and are intended 
solely for the use of the individual to whom it is addressed. Any views or 
opinions expressed are solely those of the author and do not necessarily 
represent those of Shape Blue Ltd or related companies. If you are not the 
intended recipient of this email, you must neither take any action based upon 
its contents, nor copy or show it to anyone. Please contact the sender if you 
believe you have received this email in error. Shape Blue Ltd is a company 
incorporated in England & Wales. ShapeBlue Services India LLP is a company 
incorporated in India and is operated under license from Shape Blue Ltd. Shape 
Blue Brasil Consultoria Ltda is a company incorporated in Brasil and is 
operated under license from Shape Blue Ltd. ShapeBlue SA Pty Ltd is a company 
registered by The Republic of South Africa and is traded under license from 
Shape Blue Ltd. ShapeBlue is a registered trademark.

[GitHub] cloudstack commit comment: 41c0eaa14d2f8cd5b58bca71f2254c2390cc691...

[Humbedooh]

Github user Humbedooh commented on commit 
41c0eaa14d2f8cd5b58bca71f2254c2390cc6913:


https://github.com/apache/cloudstack/commit/41c0eaa14d2f8cd5b58bca71f2254c2390cc6913#commitcomment-7731409
  
In deps/XenServerJava/pom.xml:
In deps/XenServerJava/pom.xml on line 28:
one more test, hopefully it'll work properly this time :)


---
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 enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] cloudstack commit comment: 3411fbaccafbce8f99af4714a26f38a2c17d8a9...

[runseb]

Github user runseb commented on commit 3411fbaccafbce8f99af4714a26f38a2c17d8a9e:


https://github.com/apache/cloudstack/commit/3411fbaccafbce8f99af4714a26f38a2c17d8a9e#commitcomment-7731424
  
In test/integration/component/test_stopped_vm.py:
In test/integration/component/test_stopped_vm.py on line 773:
[discard] is it working 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 have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

Jenkins build is still unstable: simulator-singlerun #334

[jenkins]

See 

[GitHub] cloudstack pull request: VPC and Virtual Network Managers refactor...

[runseb]

Github user runseb commented on a diff in the pull request:

https://github.com/apache/cloudstack/pull/18#discussion_r17360590
  
--- Diff: 
plugins/network-elements/ovs/src/com/cloud/network/element/OvsElement.java ---
@@ -16,8 +16,6 @@
 // under the License.
 package com.cloud.network.element;
 
--- End diff --

[discard] what if we comment within a PR, can we reply ?


---
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 enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

Re: [GitHub] cloudstack pull request: VPC and Virtual Network Managers refactor...

[sebgoa]

[discard] so if I reply does it gets posted back ?

On Sep 10, 2014, at 3:37 PM, runseb  wrote:

> Github user runseb commented on a diff in the pull request:
> 
>https://github.com/apache/cloudstack/pull/18#discussion_r17360590
> 
>--- Diff: 
> plugins/network-elements/ovs/src/com/cloud/network/element/OvsElement.java ---
>@@ -16,8 +16,6 @@
> // under the License.
> package com.cloud.network.element;
> 
>--- End diff --
> 
>[discard] what if we comment within a PR, can we reply ?
> 
> 
> ---
> 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 enabled but not working, please
> contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
> with INFRA.
> ---

RE: Marvin test cleanup

[Alex Brett]

> I like the idea. What worries me is the possibility to run test cases on a 
> life
> system. I think it is useful for some operators to be able to do that.
> Those people must not lose any objects that were created outside of the
> test suite. Therefore it can be very tricky, not very different from your
> parallel use-case, though.

I'd only be looking to remove objects created by the specific test case, so 
that should be fine. If it ended up deleting other objects it would as you say 
break the parallel use case so I definitely won't be looking to do that.

I'll make a start on this over the next few days and see what I can get 
working...

Alex

Re: [ACS44][DISCUSS]release 4.4.1

[Daan Hoogland]

Sally, there are two blocking issues logged for it, one of which I am
working on. The other one I am waiting for feedback on.


On Wed, Sep 10, 2014 at 2:13 PM, Sally Khudairi 
wrote:

> Hi folks --how are we looking for the release?
>
> If there are any additions/changes that might need to be reflected in the
> announcement press release, do let me know.
>
> Thanks for this,
> Sally
>
>
>   --
>  *From:* Daan Hoogland 
> *To:* dev ; Francois Gaudreault <
> fgaudrea...@cloudops.com>
> *Cc:* Sally Khudairi 
> *Sent:* Monday, 1 September 2014, 10:42
> *Subject:* Re: [ACS44][DISCUSS]release 4.4.1
>
> I will start some basic testing myself and have some unrelated $dayjob so
> an ATA of end of workweek is fine by me.
>
> Daan
>
>
> On Mon, Sep 1, 2014 at 4:39 PM, Francois Gaudreault <
> fgaudrea...@cloudops.com> wrote:
>
> Daan,
>
> We are working on fixing SSL offload for projects. Any chance we can slide
> that in 4.4.1 or should we target 4.5? Assuming we have the code working
> later this week...
>
> FG
>
>
> On 2014-09-01, 10:30 AM, Wido den Hollander wrote:
>
>
>
> On 09/01/2014 04:20 PM, Daan Hoogland wrote:
>
> SO May 1st and 1st manday in september are both due to may 4th...
>
> Nice read Stephen.
>
> Now back to 4.4.x
>
> I would like to create a new release if people feel we have something to
> release. How is the general population of this list thinking about that? Is
> there still focus on the 4.4 branch? Are all major issues from the 4.4.0
> release solved?
>
>
> Would I say all issues? I don't know. I do know however that all issues
> that I encountered have been fixed in 4.4.1
>
> Wido
>
>  thanks for any (encouraging) responds,
> Daan
>
>
> On Mon, Sep 1, 2014 at 10:33 AM, Stephen Turner  >
> wrote:
>
>  Interestingly, the reason Labour Day is on May 1 in the rest of the world
> is the same reason it's not in May in the US and Canada, to do with
> communists and anarchists commemorating a riot in Chicago. See
> http://en.wikipedia.org/wiki/Labor_Day
> http://en.wikipedia.org/wiki/Haymarket_affair
>
> --
> Stephen Turner
>
>
> -Original Message-
> From: Daan Hoogland [mailto:daan.hoogl...@gmail.com]
> Sent: 01 September 2014 09:05
> To: dev; Mike Tutkowski
> Cc: Francois Gaudreault
> Subject: Re: [ACS44][DISCUSS]release 4.4.1
>
> Hey Mike, You guys are not supposed to add random red holidays to your
> calendars. You're the commy bashers. May 1st not enough for ya'll?
>
>
> On Mon, Sep 1, 2014 at 12:00 AM, Mike Tutkowski <
> mike.tutkow...@solidfire.com> wrote:
>
>  Just an FYI that it's Labor Day Weekend in the U.S., so you might not
> get many, if any, replies until Tuesday.
>
>
> On Sat, Aug 30, 2014 at 2:06 PM, Francois Gaudreault <
> fgaudrea...@cloudops.com> wrote:
>
>  On 2014-08-30, 4:01 PM, Daan Hoogland wrote:
>
>  H,
>
> How is the felling on list about 4.4.1? Do we have a stable enough
>
> branch
>
> 4.4 at this moment? Especially the answer on this of KVM saffy
> people is appreciated.
>
>   I am running on 4.4.1 since a week now. I did hit couple small
> bugs,
>
> but
>
> no real blocker. We also try to fix the SSL offload code to work
> with projects. We would really really like to get this feature in
>
> 4.4.1.
>
>
> FG
>
> --
> Francois Gaudreault
> Gestionnaire de Produit | Product Manager - Cloud Platform &
> Services
> t:514-629-6775
>
> CloudOps Votre partenaire infonuagique | Cloud Solutions Experts
> 420 rue Guy | Montreal | Quebec | H3J 1S6
> w: cloudops.com | tw: @CloudOps_
>
>
>
>
> --
> *Mike Tutkowski*
> *Senior CloudStack Developer, SolidFire Inc.*
> e: mike.tutkow...@solidfire.com
> o: 303.746.7302
> Advancing the way the world uses the cloud
> *™*
>
>
>
>
> --
> Daan
>
>
>
>
>
>
>
>
> --
> Francois Gaudreault
> Gestionnaire de Produit | Product Manager - Cloud Platform & Services
> t:514-629-6775
>
> CloudOps Votre partenaire infonuagique | Cloud Solutions Experts
> 420 rue Guy | Montreal | Quebec | H3J 1S6
> w: cloudops.com | tw: @CloudOps_
>
>
>
>
> --
> Daan
>
>
>


-- 
Daan

Re: [ACS44][DISCUSS]release 4.4.1

[Sally Khudairi]

Thanks, Daan.

I know that with the recent CloudStack discussions in the news, I know 
announcing v4.4 would be welcome :-)

Kind thanks,
Sally


[From the mobile; please excuse spelling/spacing errors and brevity]

- Reply message -
From: "Daan Hoogland" 
To: "Sally Khudairi" 
Cc: "dev" , "Francois Gaudreault" 

Subject: [ACS44][DISCUSS]release 4.4.1
Date: Wed, Sep 10, 2014 10:14

Sally, there are two blocking issues logged for it, one of which I am working 
on. The other one I am waiting for feedback on.



On Wed, Sep 10, 2014 at 2:13 PM, Sally Khudairi  wrote:
Hi folks --how are we looking for the release?


If there are any additions/changes that might need to be reflected in the 
announcement press release, do let me know.


Thanks for this,
Sally

 

From: Daan Hoogland 
To: dev ; Francois Gaudreault 
 
Cc: Sally Khudairi  
Sent: Monday, 1 September 2014, 10:42
Subject: Re: [ACS44][DISCUSS]release 4.4.1



I will start some basic testing myself and have some unrelated $dayjob so an 
ATA of end of workweek is fine by me.



Daan


On Mon, Sep 1, 2014 at 4:39 PM, Francois Gaudreault  
wrote:


Daan,



We are working on fixing SSL offload for projects. Any chance we can slide that 
in 4.4.1 or should we target 4.5? Assuming we have the code working later this 
week...



FG



On 2014-09-01, 10:30 AM, Wido den Hollander wrote:






On 09/01/2014 04:20 PM, Daan Hoogland wrote:


SO May 1st and 1st manday in september are both due to may 4th...



Nice read Stephen.



Now back to 4.4.x



I would like to create a new release if people feel we have something to

release. How is the general population of this list thinking about that? Is

there still focus on the 4.4 branch? Are all major issues from the 4.4.0

release solved?






Would I say all issues? I don't know. I do know however that all issues that I 
encountered have been fixed in 4.4.1



Wido




thanks for any (encouraging) responds,

Daan





On Mon, Sep 1, 2014 at 10:33 AM, Stephen Turner 

wrote:




Interestingly, the reason Labour Day is on May 1 in the rest of the world

is the same reason it's not in May in the US and Canada, to do with

communists and anarchists commemorating a riot in Chicago. See

http://en.wikipedia.org/wiki/Labor_Day

http://en.wikipedia.org/wiki/Haymarket_affair



-- 

Stephen Turner





-Original Message-

From: Daan Hoogland [mailto:daan.hoogl...@gmail.com]

Sent: 01 September 2014 09:05

To: dev; Mike Tutkowski

Cc: Francois Gaudreault

Subject: Re: [ACS44][DISCUSS]release 4.4.1



Hey Mike, You guys are not supposed to add random red holidays to your

calendars. You're the commy bashers. May 1st not enough for ya'll?





On Mon, Sep 1, 2014 at 12:00 AM, Mike Tutkowski <

mike.tutkow...@solidfire.com> wrote:




Just an FYI that it's Labor Day Weekend in the U.S., so you might not

get many, if any, replies until Tuesday.





On Sat, Aug 30, 2014 at 2:06 PM, Francois Gaudreault <

fgaudrea...@cloudops.com> wrote:




On 2014-08-30, 4:01 PM, Daan Hoogland wrote:




H,



How is the felling on list about 4.4.1? Do we have a stable enough


branch


4.4 at this moment? Especially the answer on this of KVM saffy

people is appreciated.



  I am running on 4.4.1 since a week now. I did hit couple small

bugs,


but


no real blocker. We also try to fix the SSL offload code to work

with projects. We would really really like to get this feature in


4.4.1.




FG



-- 

Francois Gaudreault

Gestionnaire de Produit | Product Manager - Cloud Platform &

Services

t:514-629-6775



CloudOps Votre partenaire infonuagique | Cloud Solutions Experts

420 rue Guy | Montreal | Quebec | H3J 1S6

w: cloudops.com | tw: @CloudOps_










-- 

*Mike Tutkowski*

*Senior CloudStack Developer, SolidFire Inc.*

e: mike.tutkow...@solidfire.com

o: 303.746.7302

Advancing the way the world uses the cloud

*™*










-- 

Daan




















-- 

Francois Gaudreault

Gestionnaire de Produit | Product Manager - Cloud Platform & Services

t:514-629-6775



CloudOps Votre partenaire infonuagique | Cloud Solutions Experts

420 rue Guy | Montreal | Quebec | H3J 1S6

w: cloudops.com | tw: @CloudOps_







-- 
Daan










-- 
Daan

Re: Marvin test cleanup

[Daan Hoogland]

One more bear on the road: account for object name clashing; if you create
an test_vm in your test and it already existed... you'll have to somehow
gather objects by uuid and only use those.

/me being paranoid and budding into other peoples' project plans,
Daan


On Wed, Sep 10, 2014 at 4:05 PM, Alex Brett  wrote:

> > I like the idea. What worries me is the possibility to run test cases on
> a life
> > system. I think it is useful for some operators to be able to do that.
> > Those people must not lose any objects that were created outside of the
> > test suite. Therefore it can be very tricky, not very different from your
> > parallel use-case, though.
>
> I'd only be looking to remove objects created by the specific test case,
> so that should be fine. If it ended up deleting other objects it would as
> you say break the parallel use case so I definitely won't be looking to do
> that.
>
> I'll make a start on this over the next few days and see what I can get
> working...
>
> Alex
>



-- 
Daan

Re: [ACS44][DISCUSS]release 4.4.1

[Daan Hoogland]

Right, I would like that too. @devs Can we put some extra focus on 4.4.1
the coming days?


On Wed, Sep 10, 2014 at 4:28 PM, Sally Khudairi 
wrote:

> Thanks, Daan.
>
> I know that with the recent CloudStack discussions in the news, I know
> announcing v4.4 would be welcome :-)
>
> Kind thanks,
> Sally
>
>
> [From the mobile; please excuse spelling/spacing errors and brevity]
>
> - Reply message -
> From: "Daan Hoogland" 
> To: "Sally Khudairi" 
> Cc: "dev" , "Francois Gaudreault" <
> fgaudrea...@cloudops.com>
> Subject: [ACS44][DISCUSS]release 4.4.1
> Date: Wed, Sep 10, 2014 10:14
>
> Sally, there are two blocking issues logged for it, one of which I am
> working on. The other one I am waiting for feedback on.
>
>
> On Wed, Sep 10, 2014 at 2:13 PM, Sally Khudairi 
> wrote:
>
>> Hi folks --how are we looking for the release?
>>
>> If there are any additions/changes that might need to be reflected in the
>> announcement press release, do let me know.
>>
>> Thanks for this,
>> Sally
>>
>>
>>   --
>>  *From:* Daan Hoogland 
>> *To:* dev ; Francois Gaudreault <
>> fgaudrea...@cloudops.com>
>> *Cc:* Sally Khudairi 
>> *Sent:* Monday, 1 September 2014, 10:42
>> *Subject:* Re: [ACS44][DISCUSS]release 4.4.1
>>
>> I will start some basic testing myself and have some unrelated $dayjob so
>> an ATA of end of workweek is fine by me.
>>
>> Daan
>>
>>
>> On Mon, Sep 1, 2014 at 4:39 PM, Francois Gaudreault <
>> fgaudrea...@cloudops.com> wrote:
>>
>> Daan,
>>
>> We are working on fixing SSL offload for projects. Any chance we can
>> slide that in 4.4.1 or should we target 4.5? Assuming we have the code
>> working later this week...
>>
>> FG
>>
>>
>> On 2014-09-01, 10:30 AM, Wido den Hollander wrote:
>>
>>
>>
>> On 09/01/2014 04:20 PM, Daan Hoogland wrote:
>>
>> SO May 1st and 1st manday in september are both due to may 4th...
>>
>> Nice read Stephen.
>>
>> Now back to 4.4.x
>>
>> I would like to create a new release if people feel we have something to
>> release. How is the general population of this list thinking about that?
>> Is
>> there still focus on the 4.4 branch? Are all major issues from the 4.4.0
>> release solved?
>>
>>
>> Would I say all issues? I don't know. I do know however that all issues
>> that I encountered have been fixed in 4.4.1
>>
>> Wido
>>
>>  thanks for any (encouraging) responds,
>> Daan
>>
>>
>> On Mon, Sep 1, 2014 at 10:33 AM, Stephen Turner <
>> stephen.tur...@citrix.com>
>> wrote:
>>
>>  Interestingly, the reason Labour Day is on May 1 in the rest of the world
>> is the same reason it's not in May in the US and Canada, to do with
>> communists and anarchists commemorating a riot in Chicago. See
>>  http://en.wikipedia.org/wiki/
>> Labor_Day
>>  
>> http://en.wikipedia.org/wiki/Haymarket_affair
>>
>> --
>> Stephen Turner
>>
>>
>> -Original Message-
>> From: Daan Hoogland [mailto:daan.hoogl...@gmail.com]
>> Sent: 01 September 2014 09:05
>> To: dev; Mike Tutkowski
>> Cc: Francois Gaudreault
>> Subject: Re: [ACS44][DISCUSS]release 4.4.1
>>
>> Hey Mike, You guys are not supposed to add random red holidays to your
>> calendars. You're the commy bashers. May 1st not enough for ya'll?
>>
>>
>> On Mon, Sep 1, 2014 at 12:00 AM, Mike Tutkowski <
>> mike.tutkow...@solidfire.com> wrote:
>>
>>  Just an FYI that it's Labor Day Weekend in the U.S., so you might not
>> get many, if any, replies until Tuesday.
>>
>>
>> On Sat, Aug 30, 2014 at 2:06 PM, Francois Gaudreault <
>> fgaudrea...@cloudops.com> wrote:
>>
>>  On 2014-08-30, 4:01 PM, Daan Hoogland wrote:
>>
>>  H,
>>
>> How is the felling on list about 4.4.1? Do we have a stable enough
>>
>> branch
>>
>> 4.4 at this moment? Especially the answer on this of KVM saffy
>> people is appreciated.
>>
>>   I am running on 4.4.1 since a week now. I did hit couple small
>> bugs,
>>
>> but
>>
>> no real blocker. We also try to fix the SSL offload code to work
>> with projects. We would really really like to get this feature in
>>
>> 4.4.1.
>>
>>
>> FG
>>
>> --
>> Francois Gaudreault
>> Gestionnaire de Produit | Product Manager - Cloud Platform &
>> Services
>> t:514-629-6775
>>
>> CloudOps Votre partenaire infonuagique | Cloud Solutions Experts
>> 420 rue Guy | Montreal | Quebec | H3J 1S6
>> w: cloudops.com | tw: @CloudOps_
>>
>>
>>
>>
>> --
>> *Mike Tutkowski*
>> *Senior CloudStack Developer, SolidFire Inc.*
>> e: mike.tutkow...@solidfire.com
>> o: 303.746.7302
>> Advancing the way the world uses the cloud
>> < 
>> http://solidfire.com/solution/overview/?video=play>*™*
>>
>>
>>
>>
>> --
>> Daan
>>
>>
>>
>>
>>
>>
>>
>>
>> --
>> Francois Gaudreault
>> Gestionnaire de Produit | Product Manager - Cloud Platform & Services
>> t:514-629-6775
>>
>> CloudOps Votre partenaire infonuagique | Cloud Solutions Experts
>> 420 rue Guy | Montreal | Quebec | H3J 1S6
>> w: cloudops.com | tw: @CloudOps

Jenkins build is still unstable: simulator-singlerun #335

[jenkins]

See 

Re: Review Request 25500: CLOUDSTACK-7134: CLOUDSTACK-7134: Fixed test cases for basic zone, services dict was getting overwritten which passed zone type as advanced even in basic zone setup

[Gaurav Aradhye]

---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/25500/
---

(Updated Sept. 10, 2014, 8:25 p.m.)


Review request for cloudstack, sanjeev n, Santhosh Edukulla, and 
SrikanteswaraRao Talluri.


Changes
---

Updated patch with correct author details.


Bugs: CLOUDSTACK-7134
https://issues.apache.org/jira/browse/CLOUDSTACK-7134


Repository: cloudstack-git


Description
---

The line self.services = Services.services() in setUp and in test case caused 
the "networktype" in self.services to be overwritten as "advanced". Hence in 
basic zone, these test cases failed while deploying VM (acquiring public IP, 
which is not possible in basic zone).

Removed the lines which causes this defect.


Diffs (updated)
-

  test/integration/component/test_reset_ssh_keypair.py 204690e 

Diff: https://reviews.apache.org/r/25500/diff/


Testing
---

Tested on KVM basic zone setup. The VM deployment was successful, but test 
cases failed while SSHing, which is already tracked under separate defect.

Log:
Test Reset SSH keys for VM  already having SSH key ... === TestName: 
test_01_reset_ssh_keys | Status : FAILED ===
FAIL

==
FAIL: Test Reset SSH keys for VM  already having SSH key
--
Traceback (most recent call last):
  File "/root/test/test_reset_ssh_keypair.py", line 360, in 
test_01_reset_ssh_keys
(virtual_machine.name, e))
AssertionError: Failed to SSH into VM with new keypair: 
TestVM-91f66646-aba7-4dc0-90b3-3c1474d5346f, SSH connection has Failed. Waited 
600s. Error is SSH Connection
Failed
 >> begin captured stdout << -
=== TestName: test_01_reset_ssh_keys | Status : FAILED ===


Thanks,

Gaurav Aradhye

[CANCEL] [VOTE] Release Apache CloudStack 4.3.1, round #3

[sebgoa]

On Sep 10, 2014, at 3:28 PM, Rohit Yadav  wrote:

> Hi Sebastien,
> 
> On 10-Sep-2014, at 2:54 pm, sebgoa  wrote:
>> Found the proper fix in 4.4 from Daan:
>> 
>> https://git-wip-us.apache.org/repos/asf?p=cloudstack.git;a=blobdiff;f=tools/build/build_asf.sh;h=ea9b272d1d32a107ce06362efb163b6295969be1;hp=72f2111c303abe36f299d8f6a882a34439e33103;hb=58a4c7a89158ece348d890774c866e02f52e5d80;hpb=7fe353d4995a2b55c7baed5e27056dd2e2f7509f
>> 
>> Even though if we don't use -SNAPSHOT for this artifact in the first place, 
>> we should never face the issue…:(
> 
> Sorry about the quick fix, I did not know that the build_asf script removes 
> SNAPSHOT from all poms.
> 
> Looks like Daan is removing the -SNAPSHOT from root pom.xml, which is 
> something I did too. You may pick this for 4.3 branch, for change the version 
> string to 5.6.100-1 etc. I’ll leave that for you to decide/fix.
> 
> Regards,
> Rohit Yadav
> Software Architect, ShapeBlue
> M. +41 779015219 | rohit.ya...@shapeblue.com
> Blog: bhaisaab.org | Twitter: @_bhaisaab
> 
> 
> 
> Find out more about ShapeBlue and our range of CloudStack related services
> 
> IaaS Cloud Design & Build
> CSForge – rapid IaaS deployment framework
> CloudStack Consulting
> CloudStack Infrastructure 
> Support
> CloudStack Bootcamp Training 
> Courses
> 
> This email and any attachments to it may be confidential and are intended 
> solely for the use of the individual to whom it is addressed. Any views or 
> opinions expressed are solely those of the author and do not necessarily 
> represent those of Shape Blue Ltd or related companies. If you are not the 
> intended recipient of this email, you must neither take any action based upon 
> its contents, nor copy or show it to anyone. Please contact the sender if you 
> believe you have received this email in error. Shape Blue Ltd is a company 
> incorporated in England & Wales. ShapeBlue Services India LLP is a company 
> incorporated in India and is operated under license from Shape Blue Ltd. 
> Shape Blue Brasil Consultoria Ltda is a company incorporated in Brasil and is 
> operated under license from Shape Blue Ltd. ShapeBlue SA Pty Ltd is a company 
> registered by The Republic of South Africa and is traded under license from 
> Shape Blue Ltd. ShapeBlue is a registered trademark.

[VOTE] Release Apache CloudStack 4.3.1 round #4

[sebgoa]

Hi All,

I've created a 4.3.1 release, with the following artifacts up for a vote:

Git Branch and Commit SH:
https://git-wip-us.apache.org/repos/asf?p=cloudstack.git;a=shortlog;h=refs/heads/4.3
Commit: 1687bcdd9b8de1e372b308dffda5bf819b5c4296

List of changes:
https://git-wip-us.apache.org/repos/asf?p=cloudstack.git;a=blob;f=CHANGES.md;hb=4.3

Source release (checksums and signatures are available at the same location):
https://dist.apache.org/repos/dist/dev/cloudstack/4.3.1/

PGP release keys (signed using 8F89EB84):
https://dist.apache.org/repos/dist/release/cloudstack/KEYS

Vote will be open for 72 hours.

Please use 
https://cwiki.apache.org/confluence/display/CLOUDSTACK/Release+test+procedure
to test at least for the keys, rat etc.

For sanity in tallying the vote, can PMC members please be sure to indicate 
"(binding)" with their vote?

[ ] +1  approve
[ ] +0  no opinion
[ ] -1  disapprove (and reason why)

Re: S3/Swift Problem around Virtual Size

[Punith S]

yes mike,

i'm fixing the issue w.r.t option 2.

testing the patch is consuming much time, since i have to register the
templates via S3, and it has to download via S3 to staging store.

can i know when is the code freez for 4.5 ?

thanks!

On Tue, Sep 9, 2014 at 9:30 PM, Mike Tutkowski  wrote:

> Yeah, either solution will fix this issue for managed storage in general
> (ex. SolidFire, CloudByte).
>
> On Tue, Sep 9, 2014 at 9:58 AM, Francois Gaudreault <
> fgaudrea...@cloudops.com> wrote:
>
>> That's great Punith :)
>>
>> Thanks for handling this one. I am not too worried about the option, as
>> long as it fixes SF integration for 4.5 :)
>>
>> FG
>>
>> On Tue, Sep 9, 2014 at 11:55 AM, Mike Tutkowski <
>> mike.tutkow...@solidfire.com> wrote:
>>
>>> Thanks for the info, Punith!
>>>
>>> Does that mean you are fixing the issue via Option2?
>>>
>>> On Tue, Sep 9, 2014 at 7:52 AM, Punith S  wrote:
>>>
 yes mike,

 w.r.t to option 1:
 it will be like creating a VM w.r.t ISO, where admin will have to
 specify the os disk(ROOT) disk size.

 for option 2:
 i have figured out the issue, post downloading the template to S3,
 cloudstack will again download the template from S3 to staging nfs store.
 here we need to access the file and process it with VHD processor in
 order to calculate the virtualsize but we are skipping this process,
 hence the virtual size is not being calculated while using the S3 or
 swift.

 the templates already present in the staging nfs storage cannot be
 applied to this process.

 for option 3:
 it's convenient to calculate the template virtual size while it is
 being copied from s3 to staged nfs store instead of staged nfs to primary,
 since admin might be using more than one primary stores.

 i'm fixing the issue, will post the patch ASAP for 4.5.snapshot.

 thanks!

 On Tue, Sep 9, 2014 at 11:13 AM, Mike Tutkowski <
 mike.tutkow...@solidfire.com> wrote:

> By the way, for anyone new to this issue, this is what we're referring
> to here:
>
> https://issues.apache.org/jira/browse/CLOUDSTACK-7406
>
> On Mon, Sep 8, 2014 at 11:41 PM, Mike Tutkowski <
> mike.tutkow...@solidfire.com> wrote:
>
>> Great :) Then a question might be, "Is it too late in the game to
>> interrogate the template to discover its virtual size if we're just about
>> to copy the template to primary storage?"
>>
>> If it's not, this might be the place to run the logic to figure out
>> the virtual size.
>>
>> Really, there are three big possibilities:
>>
>> 1) Just ask the end user to provide the virtual size (not commenting
>> here on what happens for already-uploaded templates)
>>
>> or
>>
>> 2) Figure out the virtual size when the template is copied from
>> object storage to secondary storage and update the DB with this info (not
>> sure what happens if the template has already been copied to
>> (secondary-storage) NFS because it was used before)
>>
>> or
>>
>> 3) Figure out the virtual size when the template is about to be
>> copied from secondary storage to primary storage
>>
>> On Mon, Sep 8, 2014 at 11:35 PM, Sanjeev Neelarapu <
>> sanjeev.neelar...@citrix.com> wrote:
>>
>>> Mike,
>>>
>>> You are right. Template gets copied to (secondary-storage) NFS
>>> before being copied to primary storage
>>>
>>> -Sanjeev
>>>
>>> -Original Message-
>>> From: Mike Tutkowski [mailto:mike.tutkow...@solidfire.com]
>>> Sent: Tuesday, September 09, 2014 10:55 AM
>>> To: dev@cloudstack.apache.org
>>> Cc: Punith S; Francois Gaudreault
>>> Subject: Re: S3/Swift Problem around Virtual Size
>>>
>>> Hi Will,
>>>
>>> Thanks for the input!
>>>
>>> I like the idea of storing the virtual size as metadata in S3 or
>>> Swift although this could require that the end user provide this value 
>>> when
>>> uploading the template.
>>>
>>> However, if we have the ability to determine the virtual size of the
>>> template after it gets downloaded to (secondary-storage) NFS and we're 
>>> able
>>> to update the database with this info, then it would seem we would never
>>> need to ask the user for this value.
>>>
>>> Either way, the tricky part might be if the template in object
>>> storage has already been downloaded to (secondary-storage) NFS (say it 
>>> was
>>> used before). In this case, we won't need to download it to
>>> (secondary-storage) NFS again (at least not in the same zone), so we 
>>> won't
>>> have an easy opportunity to figure out the virtual size upon download 
>>> from
>>> object storage.
>>>
>>> I wonder if it's too late in this process if we figured out the
>>> virtual size before the copied template (now on (second

Re: S3/Swift Problem around Virtual Size

[Mike Tutkowski]

I have not heard recently on when code freeze for 4.5 is, per se.

Regardless, I'd say this is an important-enough issue that we should wait
for your patch.

We're still in the process of getting 4.3.1 and 4.4.1 out the door, so I
think it'll be a bit before 4.5 goes out.

Thanks for your time and effort on this, Punith!

On Wed, Sep 10, 2014 at 10:32 AM, Punith S  wrote:

> yes mike,
>
> i'm fixing the issue w.r.t option 2.
>
> testing the patch is consuming much time, since i have to register the
> templates via S3, and it has to download via S3 to staging store.
>
> can i know when is the code freez for 4.5 ?
>
> thanks!
>
> On Tue, Sep 9, 2014 at 9:30 PM, Mike Tutkowski <
> mike.tutkow...@solidfire.com> wrote:
>
>> Yeah, either solution will fix this issue for managed storage in general
>> (ex. SolidFire, CloudByte).
>>
>> On Tue, Sep 9, 2014 at 9:58 AM, Francois Gaudreault <
>> fgaudrea...@cloudops.com> wrote:
>>
>>> That's great Punith :)
>>>
>>> Thanks for handling this one. I am not too worried about the option, as
>>> long as it fixes SF integration for 4.5 :)
>>>
>>> FG
>>>
>>> On Tue, Sep 9, 2014 at 11:55 AM, Mike Tutkowski <
>>> mike.tutkow...@solidfire.com> wrote:
>>>
 Thanks for the info, Punith!

 Does that mean you are fixing the issue via Option2?

 On Tue, Sep 9, 2014 at 7:52 AM, Punith S 
 wrote:

> yes mike,
>
> w.r.t to option 1:
> it will be like creating a VM w.r.t ISO, where admin will have to
> specify the os disk(ROOT) disk size.
>
> for option 2:
> i have figured out the issue, post downloading the template to S3,
> cloudstack will again download the template from S3 to staging nfs store.
> here we need to access the file and process it with VHD processor in
> order to calculate the virtualsize but we are skipping this process,
> hence the virtual size is not being calculated while using the S3 or
> swift.
>
> the templates already present in the staging nfs storage cannot be
> applied to this process.
>
> for option 3:
> it's convenient to calculate the template virtual size while it is
> being copied from s3 to staged nfs store instead of staged nfs to primary,
> since admin might be using more than one primary stores.
>
> i'm fixing the issue, will post the patch ASAP for 4.5.snapshot.
>
> thanks!
>
> On Tue, Sep 9, 2014 at 11:13 AM, Mike Tutkowski <
> mike.tutkow...@solidfire.com> wrote:
>
>> By the way, for anyone new to this issue, this is what we're
>> referring to here:
>>
>> https://issues.apache.org/jira/browse/CLOUDSTACK-7406
>>
>> On Mon, Sep 8, 2014 at 11:41 PM, Mike Tutkowski <
>> mike.tutkow...@solidfire.com> wrote:
>>
>>> Great :) Then a question might be, "Is it too late in the game to
>>> interrogate the template to discover its virtual size if we're just 
>>> about
>>> to copy the template to primary storage?"
>>>
>>> If it's not, this might be the place to run the logic to figure out
>>> the virtual size.
>>>
>>> Really, there are three big possibilities:
>>>
>>> 1) Just ask the end user to provide the virtual size (not commenting
>>> here on what happens for already-uploaded templates)
>>>
>>> or
>>>
>>> 2) Figure out the virtual size when the template is copied from
>>> object storage to secondary storage and update the DB with this info 
>>> (not
>>> sure what happens if the template has already been copied to
>>> (secondary-storage) NFS because it was used before)
>>>
>>> or
>>>
>>> 3) Figure out the virtual size when the template is about to be
>>> copied from secondary storage to primary storage
>>>
>>> On Mon, Sep 8, 2014 at 11:35 PM, Sanjeev Neelarapu <
>>> sanjeev.neelar...@citrix.com> wrote:
>>>
 Mike,

 You are right. Template gets copied to (secondary-storage) NFS
 before being copied to primary storage

 -Sanjeev

 -Original Message-
 From: Mike Tutkowski [mailto:mike.tutkow...@solidfire.com]
 Sent: Tuesday, September 09, 2014 10:55 AM
 To: dev@cloudstack.apache.org
 Cc: Punith S; Francois Gaudreault
 Subject: Re: S3/Swift Problem around Virtual Size

 Hi Will,

 Thanks for the input!

 I like the idea of storing the virtual size as metadata in S3 or
 Swift although this could require that the end user provide this value 
 when
 uploading the template.

 However, if we have the ability to determine the virtual size of
 the template after it gets downloaded to (secondary-storage) NFS and 
 we're
 able to update the database with this info, then it would seem we would
 never need to ask the user for this value.

 Either

Re: S3/Swift Problem around Virtual Size

[Mike Tutkowski]

Also, of course feel free to put me down as a reviewer when you are ready
and I can review the code shortly after.

Thanks!

On Wed, Sep 10, 2014 at 10:47 AM, Mike Tutkowski <
mike.tutkow...@solidfire.com> wrote:

> I have not heard recently on when code freeze for 4.5 is, per se.
>
> Regardless, I'd say this is an important-enough issue that we should wait
> for your patch.
>
> We're still in the process of getting 4.3.1 and 4.4.1 out the door, so I
> think it'll be a bit before 4.5 goes out.
>
> Thanks for your time and effort on this, Punith!
>
> On Wed, Sep 10, 2014 at 10:32 AM, Punith S  wrote:
>
>> yes mike,
>>
>> i'm fixing the issue w.r.t option 2.
>>
>> testing the patch is consuming much time, since i have to register the
>> templates via S3, and it has to download via S3 to staging store.
>>
>> can i know when is the code freez for 4.5 ?
>>
>> thanks!
>>
>> On Tue, Sep 9, 2014 at 9:30 PM, Mike Tutkowski <
>> mike.tutkow...@solidfire.com> wrote:
>>
>>> Yeah, either solution will fix this issue for managed storage in general
>>> (ex. SolidFire, CloudByte).
>>>
>>> On Tue, Sep 9, 2014 at 9:58 AM, Francois Gaudreault <
>>> fgaudrea...@cloudops.com> wrote:
>>>
 That's great Punith :)

 Thanks for handling this one. I am not too worried about the option, as
 long as it fixes SF integration for 4.5 :)

 FG

 On Tue, Sep 9, 2014 at 11:55 AM, Mike Tutkowski <
 mike.tutkow...@solidfire.com> wrote:

> Thanks for the info, Punith!
>
> Does that mean you are fixing the issue via Option2?
>
> On Tue, Sep 9, 2014 at 7:52 AM, Punith S 
> wrote:
>
>> yes mike,
>>
>> w.r.t to option 1:
>> it will be like creating a VM w.r.t ISO, where admin will have to
>> specify the os disk(ROOT) disk size.
>>
>> for option 2:
>> i have figured out the issue, post downloading the template to S3,
>> cloudstack will again download the template from S3 to staging nfs store.
>> here we need to access the file and process it with VHD processor in
>> order to calculate the virtualsize but we are skipping this process,
>> hence the virtual size is not being calculated while using the S3 or
>> swift.
>>
>> the templates already present in the staging nfs storage cannot be
>> applied to this process.
>>
>> for option 3:
>> it's convenient to calculate the template virtual size while it is
>> being copied from s3 to staged nfs store instead of staged nfs to 
>> primary,
>> since admin might be using more than one primary stores.
>>
>> i'm fixing the issue, will post the patch ASAP for 4.5.snapshot.
>>
>> thanks!
>>
>> On Tue, Sep 9, 2014 at 11:13 AM, Mike Tutkowski <
>> mike.tutkow...@solidfire.com> wrote:
>>
>>> By the way, for anyone new to this issue, this is what we're
>>> referring to here:
>>>
>>> https://issues.apache.org/jira/browse/CLOUDSTACK-7406
>>>
>>> On Mon, Sep 8, 2014 at 11:41 PM, Mike Tutkowski <
>>> mike.tutkow...@solidfire.com> wrote:
>>>
 Great :) Then a question might be, "Is it too late in the game to
 interrogate the template to discover its virtual size if we're just 
 about
 to copy the template to primary storage?"

 If it's not, this might be the place to run the logic to figure out
 the virtual size.

 Really, there are three big possibilities:

 1) Just ask the end user to provide the virtual size (not
 commenting here on what happens for already-uploaded templates)

 or

 2) Figure out the virtual size when the template is copied from
 object storage to secondary storage and update the DB with this info 
 (not
 sure what happens if the template has already been copied to
 (secondary-storage) NFS because it was used before)

 or

 3) Figure out the virtual size when the template is about to be
 copied from secondary storage to primary storage

 On Mon, Sep 8, 2014 at 11:35 PM, Sanjeev Neelarapu <
 sanjeev.neelar...@citrix.com> wrote:

> Mike,
>
> You are right. Template gets copied to (secondary-storage) NFS
> before being copied to primary storage
>
> -Sanjeev
>
> -Original Message-
> From: Mike Tutkowski [mailto:mike.tutkow...@solidfire.com]
> Sent: Tuesday, September 09, 2014 10:55 AM
> To: dev@cloudstack.apache.org
> Cc: Punith S; Francois Gaudreault
> Subject: Re: S3/Swift Problem around Virtual Size
>
> Hi Will,
>
> Thanks for the input!
>
> I like the idea of storing the virtual size as metadata in S3 or
> Swift although this could require that the end user provide this 
> value when
>>

Re: [VOTE] Release Apache CloudStack 4.3.1 round #4

[Wido den Hollander]

+1 for me based on the fact that the Debian packages build fine and the
changelog says 4.3.1

I currently don't have a 4.3 dev environment available, so I wasn't able
to test the functionality.

Wido

On 09/10/2014 05:31 PM, sebgoa wrote:
> Hi All,
> 
> I've created a 4.3.1 release, with the following artifacts up for a vote:
> 
> Git Branch and Commit SH:
> https://git-wip-us.apache.org/repos/asf?p=cloudstack.git;a=shortlog;h=refs/heads/4.3
> Commit: 1687bcdd9b8de1e372b308dffda5bf819b5c4296
> 
> List of changes:
> https://git-wip-us.apache.org/repos/asf?p=cloudstack.git;a=blob;f=CHANGES.md;hb=4.3
> 
> Source release (checksums and signatures are available at the same location):
> https://dist.apache.org/repos/dist/dev/cloudstack/4.3.1/
> 
> PGP release keys (signed using 8F89EB84):
> https://dist.apache.org/repos/dist/release/cloudstack/KEYS
> 
> Vote will be open for 72 hours.
> 
> Please use 
> https://cwiki.apache.org/confluence/display/CLOUDSTACK/Release+test+procedure
> to test at least for the keys, rat etc.
> 
> For sanity in tallying the vote, can PMC members please be sure to indicate 
> "(binding)" with their vote?
> 
> [ ] +1  approve
> [ ] +0  no opinion
> [ ] -1  disapprove (and reason why)
> 

Re: Review Request 24090: Externalized the hard-coded strings from JavaScript files to resource bundles.

[Brian Federle]

---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/24090/#review52917
---


Vetri, I have a conflict in vpc.js...please rebase changes to the latest master 
and regenerate patch. Thanks.

error: patch failed: ui/scripts/vpc.js:698
error: ui/scripts/vpc.js: patch does not apply
Patch failed at 0001 Fixes: Externalized hardcoded strings and splitted up 
dictionary file

- Brian Federle


On Sept. 8, 2014, 11:43 a.m., Vetrivel Chinnasamy wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/24090/
> ---
> 
> (Updated Sept. 8, 2014, 11:43 a.m.)
> 
> 
> Review request for cloudstack, Brian Federle and Jessica Wang.
> 
> 
> Repository: cloudstack-git
> 
> 
> Description
> ---
> 
> Externalized the hard-coded strings from JavaScript files to resource 
> bundles. Updated the dictionary.jsp file accordingly. Also got the 
> externalized strings translated in JA & SC.
> 
> 
> Diffs
> -
> 
>   client/WEB-INF/classes/resources/messages_ja_JP.properties ed6a1b1 
>   client/WEB-INF/classes/resources/messages_zh_CN.properties 2c497bc 
>   ui/dictionary.jsp 0102144 
>   ui/dictionary2.jsp PRE-CREATION 
>   ui/index.jsp 48afa6a 
>   ui/modules/vnmcAsa1000v/vnmcAsa1000v.js 621c52a 
>   ui/modules/vnmcNetworkProvider/vnmcNetworkProvider.js c9295a3 
>   ui/scripts/accounts.js cc4624a 
>   ui/scripts/autoscaler.js c8963fd 
>   ui/scripts/cloudStack.js 38cf501 
>   ui/scripts/configuration.js a70c672 
>   ui/scripts/domains.js 488382e 
>   ui/scripts/events.js 2731cb6 
>   ui/scripts/instances.js 4d536e3 
>   ui/scripts/lbStickyPolicy.js 16995f6 
>   ui/scripts/network.js 8a758c8 
>   ui/scripts/projects.js 53b7964 
>   ui/scripts/regions.js 368c1bf 
>   ui/scripts/sharedFunctions.js bb2d596 
>   ui/scripts/storage.js f4ab6e1 
>   ui/scripts/system.js cf3b9dc 
>   ui/scripts/templates.js 6dcd6da 
>   ui/scripts/ui-custom/autoscaler.js 0aa6c77 
>   ui/scripts/ui-custom/healthCheck.js 4e10f1c 
>   ui/scripts/ui-custom/physicalResources.js 110945e 
>   ui/scripts/ui-custom/regions.js 986e009 
>   ui/scripts/ui-custom/zoneWizard.js f3a1aae 
>   ui/scripts/ui/dialog.js 6c77924 
>   ui/scripts/ui/widgets/listView.js 22182c1 
>   ui/scripts/ui/widgets/multiEdit.js 47e5f43 
>   ui/scripts/vpc.js d3602fc 
>   ui/scripts/zoneWizard.js 4498534 
> 
> Diff: https://reviews.apache.org/r/24090/diff/
> 
> 
> Testing
> ---
> 
> Tested by replacing the modified javascript , dictionary.jsp and properties 
> files.
> 
> 
> Thanks,
> 
> Vetrivel Chinnasamy
> 
>

Re: [VOTE] Release Apache CloudStack 4.3.1 round #4

[Rohit Yadav]

+1 installed on KVM, tested with basic zone, vm lifecycles and other things as 
recommended in the release test procedure but with KVM only.

New deb/rpm repo built out of this voting candidate (4.3.1-voting-04) can be 
found and used from here:
http://packages.bhaisaab.org/cloudstack/main/

On 10-Sep-2014, at 7:17 pm, Wido den Hollander  wrote:
> +1 for me based on the fact that the Debian packages build fine and the
> changelog says 4.3.1
>
> I currently don't have a 4.3 dev environment available, so I wasn't able
> to test the functionality.
>
> Wido
>
> On 09/10/2014 05:31 PM, sebgoa wrote:
>> Hi All,
>>
>> I've created a 4.3.1 release, with the following artifacts up for a vote:
>>
>> Git Branch and Commit SH:
>> https://git-wip-us.apache.org/repos/asf?p=cloudstack.git;a=shortlog;h=refs/heads/4.3
>> Commit: 1687bcdd9b8de1e372b308dffda5bf819b5c4296
>>
>> List of changes:
>> https://git-wip-us.apache.org/repos/asf?p=cloudstack.git;a=blob;f=CHANGES.md;hb=4.3
>>
>> Source release (checksums and signatures are available at the same location):
>> https://dist.apache.org/repos/dist/dev/cloudstack/4.3.1/
>>
>> PGP release keys (signed using 8F89EB84):
>> https://dist.apache.org/repos/dist/release/cloudstack/KEYS
>>
>> Vote will be open for 72 hours.
>>
>> Please use 
>> https://cwiki.apache.org/confluence/display/CLOUDSTACK/Release+test+procedure
>> to test at least for the keys, rat etc.
>>
>> For sanity in tallying the vote, can PMC members please be sure to indicate 
>> "(binding)" with their vote?
>>
>> [ ] +1  approve
>> [ ] +0  no opinion
>> [ ] -1  disapprove (and reason why)
>>

Regards,
Rohit Yadav
Software Architect, ShapeBlue
M. +41 779015219 | rohit.ya...@shapeblue.com
Blog: bhaisaab.org | Twitter: @_bhaisaab



Find out more about ShapeBlue and our range of CloudStack related services

IaaS Cloud Design & Build
CSForge – rapid IaaS deployment framework
CloudStack Consulting
CloudStack Infrastructure 
Support
CloudStack Bootcamp Training Courses

This email and any attachments to it may be confidential and are intended 
solely for the use of the individual to whom it is addressed. Any views or 
opinions expressed are solely those of the author and do not necessarily 
represent those of Shape Blue Ltd or related companies. If you are not the 
intended recipient of this email, you must neither take any action based upon 
its contents, nor copy or show it to anyone. Please contact the sender if you 
believe you have received this email in error. Shape Blue Ltd is a company 
incorporated in England & Wales. ShapeBlue Services India LLP is a company 
incorporated in India and is operated under license from Shape Blue Ltd. Shape 
Blue Brasil Consultoria Ltda is a company incorporated in Brasil and is 
operated under license from Shape Blue Ltd. ShapeBlue SA Pty Ltd is a company 
registered by The Republic of South Africa and is traded under license from 
Shape Blue Ltd. ShapeBlue is a registered trademark.

Review Request 25518: CLOUDSTACK-7035-[Automation] - Automate ACL test cases relating to listNetworks() for isolated and shared networks.

[sangeetha hariharan]

---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/25518/
---

Review request for cloudstack, edison su, Min Chen, Prachi Damle, and Santhosh 
Edukulla.


Bugs: CLOUDSTACK-7035
https://issues.apache.org/jira/browse/CLOUDSTACK-7035


Repository: cloudstack-git


Description
---

Test Suite contains ACL test cases relating to listNetworks() for isolated and 
shared networks when executed by Admin, domain admin and resgular users.


Diffs
-

  test/integration/component/test_acl_listnetwork.py PRE-CREATION 

Diff: https://reviews.apache.org/r/25518/diff/


Testing
---

Tested this suite against a simulator set up from master:

Test listing of Networks by passing no parameters as domain admin ... === 
TestName: test_listNetwork_as_domainadmin | Status : FAILED ===
FAIL
Domain admin should not be allowed to list networks of accounts in other 
domains ... === TestName: test_listNetwork_as_domainadmin_cross_domainid | 
Status : SUCCESS ===
ok
Regular User should not be allowed to list networks of other accounts in other 
domains ... === TestName: 
test_listNetwork_as_domainadmin_cross_domainid_accountid | Status : SUCCESS ===
ok
Test listing of Networks by passing domainId parameter as domain admin ... === 
TestName: test_listNetwork_as_domainadmin_domainid | Status : SUCCESS ===
ok
Test listing of Networks by passing domainId and isrecursive=false parameter as 
domain admin ... === TestName: 
test_listNetwork_as_domainadmin_domainid__rec_false | Status : SUCCESS ===
ok
Test listing of Networks by passing domainId ,accountname parameter as domain 
admin ... === TestName: test_listNetwork_as_domainadmin_domainid_accountid | 
Status : SUCCESS ===
ok
Test listing of Networks by passing domainId ,accountname , listall=false 
parameter as domain admin ... === TestName: 
test_listNetwork_as_domainadmin_domainid_accountid_listall_false | Status : 
SUCCESS ===
ok
Test listing of Networks by passing domainId ,accountname , listall=false and 
isrecursive=false parameter as domain admin ... === TestName: 
test_listNetwork_as_domainadmin_domainid_accountid_listall_false_rec_false | 
Status : SUCCESS ===
ok
Test listing of Networks by passing domainId ,accountname , listall=false and 
isrecursive=true parameter as domain admin ... === TestName: 
test_listNetwork_as_domainadmin_domainid_accountid_listall_false_rec_true | 
Status : SUCCESS ===
ok
Test listing of Networks by passing domainId ,accountname , listall=true 
parameter as domain admin ... === TestName: 
test_listNetwork_as_domainadmin_domainid_accountid_listall_true | Status : 
SUCCESS ===
ok
Test listing of Networks by passing domainId ,accountname , listall=true and 
isrecursive=false parameter as domain admin ... === TestName: 
test_listNetwork_as_domainadmin_domainid_accountid_listall_true_rec_false | 
Status : SUCCESS ===
ok
Test listing of Networks by passing domainId ,accountname , listall=true and 
isrecursive=true parameter as domain admin ... === TestName: 
test_listNetwork_as_domainadmin_domainid_accountid_listall_true_rec_true | 
Status : SUCCESS ===
ok
Test listing of Networks by passing domainId ,accountname and isrecursive=false 
parameter as domain admin ... === TestName: 
test_listNetwork_as_domainadmin_domainid_accountid_rec_false | Status : SUCCESS 
===
ok
Test listing of Networks by passing domainId ,accountname and isrecursive=true 
parameter as domain admin ... === TestName: 
test_listNetwork_as_domainadmin_domainid_accountid_rec_true | Status : SUCCESS 
===
ok
Test listing of Networks by passing domainId ,listall=false parameter as domain 
admin ... === TestName: test_listNetwork_as_domainadmin_domainid_listall_false 
| Status : SUCCESS ===
ok
Test listing of Networks by passing domainId ,listall=false and 
isrecursive=false parameter as domain admin ... === TestName: 
test_listNetwork_as_domainadmin_domainid_listall_false_rec_false | Status : 
SUCCESS ===
ok
Test listing of Networks by passing domainId ,listall=false and 
isrecursive=true parameter as domain admin ... === TestName: 
test_listNetwork_as_domainadmin_domainid_listall_false_rec_true | Status : 
SUCCESS ===
ok
Test listing of Networks by passing domainId and listall=true parameter as 
domain admin ... === TestName: 
test_listNetwork_as_domainadmin_domainid_listall_true | Status : SUCCESS ===
ok
Test listing of Networks by passing domainId ,listall=true and 
isrecurisive=false parameter as domain admin ... === TestName: 
test_listNetwork_as_domainadmin_domainid_listall_true_rec_false | Status : 
SUCCESS ===
ok
Test listing of Networks by passing domainId ,listall=truei and 
isrecursive=false parameter as domain admin ... === TestName: 
test_listNetwork_as_domainadmin_domainid_listall_true_rec_true | Status : 
SUCCESS ===
ok
Test listing of Networks by passing domainId and isrecursive=true parameter as 
domain admin 

Re: Review Request 25518: CLOUDSTACK-7035-[Automation] - Automate ACL test cases relating to listNetworks() for isolated and shared networks.

[sangeetha hariharan]

---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/25518/
---

(Updated Sept. 10, 2014, 9:57 p.m.)


Review request for cloudstack, edison su, Min Chen, Prachi Damle, and Santhosh 
Edukulla.


Bugs: CLOUDSTACK-7035
https://issues.apache.org/jira/browse/CLOUDSTACK-7035


Repository: cloudstack-git


Description
---

Test Suite contains ACL test cases relating to listNetworks() for isolated and 
shared networks when executed by Admin, domain admin and resgular users.


Diffs
-

  test/integration/component/test_acl_listnetwork.py PRE-CREATION 

Diff: https://reviews.apache.org/r/25518/diff/


Testing
---

Tested this suite against a simulator set up from master:

Test listing of Networks by passing no parameters as domain admin ... === 
TestName: test_listNetwork_as_domainadmin | Status : FAILED ===
FAIL
Domain admin should not be allowed to list networks of accounts in other 
domains ... === TestName: test_listNetwork_as_domainadmin_cross_domainid | 
Status : SUCCESS ===
ok
Regular User should not be allowed to list networks of other accounts in other 
domains ... === TestName: 
test_listNetwork_as_domainadmin_cross_domainid_accountid | Status : SUCCESS ===
ok
Test listing of Networks by passing domainId parameter as domain admin ... === 
TestName: test_listNetwork_as_domainadmin_domainid | Status : SUCCESS ===
ok
Test listing of Networks by passing domainId and isrecursive=false parameter as 
domain admin ... === TestName: 
test_listNetwork_as_domainadmin_domainid__rec_false | Status : SUCCESS ===
ok
Test listing of Networks by passing domainId ,accountname parameter as domain 
admin ... === TestName: test_listNetwork_as_domainadmin_domainid_accountid | 
Status : SUCCESS ===
ok
Test listing of Networks by passing domainId ,accountname , listall=false 
parameter as domain admin ... === TestName: 
test_listNetwork_as_domainadmin_domainid_accountid_listall_false | Status : 
SUCCESS ===
ok
Test listing of Networks by passing domainId ,accountname , listall=false and 
isrecursive=false parameter as domain admin ... === TestName: 
test_listNetwork_as_domainadmin_domainid_accountid_listall_false_rec_false | 
Status : SUCCESS ===
ok
Test listing of Networks by passing domainId ,accountname , listall=false and 
isrecursive=true parameter as domain admin ... === TestName: 
test_listNetwork_as_domainadmin_domainid_accountid_listall_false_rec_true | 
Status : SUCCESS ===
ok
Test listing of Networks by passing domainId ,accountname , listall=true 
parameter as domain admin ... === TestName: 
test_listNetwork_as_domainadmin_domainid_accountid_listall_true | Status : 
SUCCESS ===
ok
Test listing of Networks by passing domainId ,accountname , listall=true and 
isrecursive=false parameter as domain admin ... === TestName: 
test_listNetwork_as_domainadmin_domainid_accountid_listall_true_rec_false | 
Status : SUCCESS ===
ok
Test listing of Networks by passing domainId ,accountname , listall=true and 
isrecursive=true parameter as domain admin ... === TestName: 
test_listNetwork_as_domainadmin_domainid_accountid_listall_true_rec_true | 
Status : SUCCESS ===
ok
Test listing of Networks by passing domainId ,accountname and isrecursive=false 
parameter as domain admin ... === TestName: 
test_listNetwork_as_domainadmin_domainid_accountid_rec_false | Status : SUCCESS 
===
ok
Test listing of Networks by passing domainId ,accountname and isrecursive=true 
parameter as domain admin ... === TestName: 
test_listNetwork_as_domainadmin_domainid_accountid_rec_true | Status : SUCCESS 
===
ok
Test listing of Networks by passing domainId ,listall=false parameter as domain 
admin ... === TestName: test_listNetwork_as_domainadmin_domainid_listall_false 
| Status : SUCCESS ===
ok
Test listing of Networks by passing domainId ,listall=false and 
isrecursive=false parameter as domain admin ... === TestName: 
test_listNetwork_as_domainadmin_domainid_listall_false_rec_false | Status : 
SUCCESS ===
ok
Test listing of Networks by passing domainId ,listall=false and 
isrecursive=true parameter as domain admin ... === TestName: 
test_listNetwork_as_domainadmin_domainid_listall_false_rec_true | Status : 
SUCCESS ===
ok
Test listing of Networks by passing domainId and listall=true parameter as 
domain admin ... === TestName: 
test_listNetwork_as_domainadmin_domainid_listall_true | Status : SUCCESS ===
ok
Test listing of Networks by passing domainId ,listall=true and 
isrecurisive=false parameter as domain admin ... === TestName: 
test_listNetwork_as_domainadmin_domainid_listall_true_rec_false | Status : 
SUCCESS ===
ok
Test listing of Networks by passing domainId ,listall=truei and 
isrecursive=false parameter as domain admin ... === TestName: 
test_listNetwork_as_domainadmin_domainid_listall_true_rec_true | Status : 
SUCCESS ===
ok
Test listing of Networks by passing domainId and isrecu

Re: Review Request 25518: CLOUDSTACK-7035-[Automation] - Automate ACL test cases relating to listNetworks() for isolated and shared networks.

[sangeetha hariharan]

---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/25518/
---

(Updated Sept. 10, 2014, 10 p.m.)


Review request for cloudstack, edison su, Min Chen, Prachi Damle, and Santhosh 
Edukulla.


Changes
---

Corrected the error message check for 
CloudstackAclException.NO_PERMISSION_TO_OPERATE_ACCOUNT instead of 
CloudstackAclException.NO_PERMISSION_TO_ACCESS_ACCOUNT


Bugs: CLOUDSTACK-7035
https://issues.apache.org/jira/browse/CLOUDSTACK-7035


Repository: cloudstack-git


Description
---

Test Suite contains ACL test cases relating to listNetworks() for isolated and 
shared networks when executed by Admin, domain admin and resgular users.


Diffs (updated)
-

  test/integration/component/test_acl_listnetwork.py PRE-CREATION 

Diff: https://reviews.apache.org/r/25518/diff/


Testing
---

Tested this suite against a simulator set up from master:

Test listing of Networks by passing no parameters as domain admin ... === 
TestName: test_listNetwork_as_domainadmin | Status : FAILED ===
FAIL
Domain admin should not be allowed to list networks of accounts in other 
domains ... === TestName: test_listNetwork_as_domainadmin_cross_domainid | 
Status : SUCCESS ===
ok
Regular User should not be allowed to list networks of other accounts in other 
domains ... === TestName: 
test_listNetwork_as_domainadmin_cross_domainid_accountid | Status : SUCCESS ===
ok
Test listing of Networks by passing domainId parameter as domain admin ... === 
TestName: test_listNetwork_as_domainadmin_domainid | Status : SUCCESS ===
ok
Test listing of Networks by passing domainId and isrecursive=false parameter as 
domain admin ... === TestName: 
test_listNetwork_as_domainadmin_domainid__rec_false | Status : SUCCESS ===
ok
Test listing of Networks by passing domainId ,accountname parameter as domain 
admin ... === TestName: test_listNetwork_as_domainadmin_domainid_accountid | 
Status : SUCCESS ===
ok
Test listing of Networks by passing domainId ,accountname , listall=false 
parameter as domain admin ... === TestName: 
test_listNetwork_as_domainadmin_domainid_accountid_listall_false | Status : 
SUCCESS ===
ok
Test listing of Networks by passing domainId ,accountname , listall=false and 
isrecursive=false parameter as domain admin ... === TestName: 
test_listNetwork_as_domainadmin_domainid_accountid_listall_false_rec_false | 
Status : SUCCESS ===
ok
Test listing of Networks by passing domainId ,accountname , listall=false and 
isrecursive=true parameter as domain admin ... === TestName: 
test_listNetwork_as_domainadmin_domainid_accountid_listall_false_rec_true | 
Status : SUCCESS ===
ok
Test listing of Networks by passing domainId ,accountname , listall=true 
parameter as domain admin ... === TestName: 
test_listNetwork_as_domainadmin_domainid_accountid_listall_true | Status : 
SUCCESS ===
ok
Test listing of Networks by passing domainId ,accountname , listall=true and 
isrecursive=false parameter as domain admin ... === TestName: 
test_listNetwork_as_domainadmin_domainid_accountid_listall_true_rec_false | 
Status : SUCCESS ===
ok
Test listing of Networks by passing domainId ,accountname , listall=true and 
isrecursive=true parameter as domain admin ... === TestName: 
test_listNetwork_as_domainadmin_domainid_accountid_listall_true_rec_true | 
Status : SUCCESS ===
ok
Test listing of Networks by passing domainId ,accountname and isrecursive=false 
parameter as domain admin ... === TestName: 
test_listNetwork_as_domainadmin_domainid_accountid_rec_false | Status : SUCCESS 
===
ok
Test listing of Networks by passing domainId ,accountname and isrecursive=true 
parameter as domain admin ... === TestName: 
test_listNetwork_as_domainadmin_domainid_accountid_rec_true | Status : SUCCESS 
===
ok
Test listing of Networks by passing domainId ,listall=false parameter as domain 
admin ... === TestName: test_listNetwork_as_domainadmin_domainid_listall_false 
| Status : SUCCESS ===
ok
Test listing of Networks by passing domainId ,listall=false and 
isrecursive=false parameter as domain admin ... === TestName: 
test_listNetwork_as_domainadmin_domainid_listall_false_rec_false | Status : 
SUCCESS ===
ok
Test listing of Networks by passing domainId ,listall=false and 
isrecursive=true parameter as domain admin ... === TestName: 
test_listNetwork_as_domainadmin_domainid_listall_false_rec_true | Status : 
SUCCESS ===
ok
Test listing of Networks by passing domainId and listall=true parameter as 
domain admin ... === TestName: 
test_listNetwork_as_domainadmin_domainid_listall_true | Status : SUCCESS ===
ok
Test listing of Networks by passing domainId ,listall=true and 
isrecurisive=false parameter as domain admin ... === TestName: 
test_listNetwork_as_domainadmin_domainid_listall_true_rec_false | Status : 
SUCCESS ===
ok
Test listing of Networks by passing domainId ,listall=truei and 
isrecursive=false p

Jenkins build is still unstable: simulator-singlerun #336

[jenkins]

See 

Build failed in Jenkins: build-master-noredist #3505

[jenkins]

See 

Changes:

[jessicawang] CLOUDSTACK-7462: UI > Network > VPC > Router > Network ACL Lists 
> click an entry from list > Details tab > ACL List Rules tab > click Edit icon 
on any existing rule > fix the JavaScript error "args.jsonObj is undefined".

--
[...truncated 1815 lines...]
log4j:WARN See http://logging.apache.org/log4j/1.2/faq.html#noconfig for more 
info.
Tests run: 1, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 0.522 sec
Running com.cloud.keystore.KeystoreTest
org.apache.cloudstack.api.response.UserVmResponse/null/{"id":"3","securitygroup":[],"nic":[],"tags":[],"affinitygroup":[]}
org.apache.cloudstack.api.response.AlertResponse/null/{"id":"100","description":"Hello"}
Tests run: 1, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 0.188 sec
Running com.cloud.alert.AlertControlsUnitTest
Tests run: 1, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 0.032 sec
Running com.cloud.capacity.CapacityManagerTest
Tests run: 1, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 0.178 sec
Running com.cloud.servlet.StaticResourceServletTest
Tests run: 12, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 0.178 sec
Running com.cloud.servlet.ConsoleProxyServletTest
Tests run: 1, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 0.003 sec
Running com.cloud.resourcelimit.ResourceLimitManagerImplTest
Tests run: 1, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 0.004 sec
Running com.cloud.network.firewall.FirewallManagerTest
Tests run: 1, Failures: 0, Errors: 0, Skipped: 1, Time elapsed: 0.002 sec
Running com.cloud.network.ExternalLoadBalancerDeviceManagerImplTest
Tests run: 2, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 0.515 sec
Running com.cloud.network.element.VirtualRouterElementTest
Tests run: 2, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 0.913 sec
Running com.cloud.network.UpdatePhysicalNetworkTest
Tests run: 1, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 0.178 sec
Running com.cloud.network.CreatePrivateNetworkTest
Tests run: 1, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 0.018 sec
Running com.cloud.network.NetworkModelTest
Tests run: 1, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 0.024 sec
Running com.cloud.network.router.VirtualNetworkApplianceManagerImplTest
Tests run: 2, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 0.06 sec
Running com.cloud.network.security.SecurityGroupManagerImplTest
Tests run: 4, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 4.081 sec
Running com.cloud.network.security.SecurityGroupQueueTest
Total jobs dequeued = 10, num queued=1010 queue current size=1000
Num Vms= 50 Queue size = 50
Num Vms= 2 Queue size = 2 time=191 ms
Num Vms= 5000 Queue size = 5000 time=2308 ms
Num Vms= 1 Queue size = 1 time=0 ms
Num Vms= 100 Queue size = 100 time=234 ms
Total jobs dequeued = 10, num queued=1010 queue current size=1000
Total jobs dequeued = 1, num queued=1001 queue current size=1000
Total jobs dequeued = 10, num queued=1000 queue current size=990
Total jobs dequeued = 10, num queued=10 queue current size=0
Tests run: 4, Failures: 0, Errors: 1, Skipped: 0, Time elapsed: 3.194 sec <<< 
FAILURE!
Running com.cloud.network.vpc.VpcManagerImplTest
Tests run: 1, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 0.041 sec
Running com.cloud.network.DedicateGuestVlanRangesTest
Tests run: 2, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 0.029 sec
Running com.cloud.network.lb.AssignLoadBalancerTest
Tests run: 4, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 0.109 sec
Running com.cloud.network.dao.NetworkDaoTest
Tests run: 2, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 0.003 sec
Running com.cloud.vm.DeploymentPlanningManagerImplTest
Tests run: 3, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 0.843 sec
Running com.cloud.vm.snapshot.VMSnapshotManagerTest
Tests run: 7, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 0.251 sec
Running com.cloud.vm.UserVmManagerTest
Tests run: 9, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 0.373 sec
Running com.cloud.configuration.ValidateIpRangeTest
Tests run: 3, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 0.021 sec
Running com.cloud.configuration.ConfigurationManagerTest
Tests run: 9, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 0.086 sec
Running com.cloud.server.ConfigurationServerImplTest
Tests run: 4, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 0.008 sec
Running com.cloud.template.TemplateManagerImplTest
Tests run: 1, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 0.01 sec
Running com.cloud.vpc.Site2SiteVpnTest
Tests run: 1, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 3.144 sec
Running com.cloud.vpc.NetworkACLServiceTest
Tests run: 5, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 0.19 sec
Running com.cloud.vpc.NetworkACLManagerTest
Tests run: 6, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 0.197 sec
Running com.cloud.storage.VolumeApiServi

Jenkins build is still unstable: simulator-singlerun #337

[jenkins]

See 

Jenkins build is still unstable: simulator-singlerun #338

[jenkins]

See 

Jenkins build is back to normal : build-master-noredist #3506

[jenkins]

See 

Jenkins build is still unstable: simulator-singlerun #339

[jenkins]

See 

Jenkins build is still unstable: simulator-singlerun #340

[jenkins]

See 

Review Request 25530: CLOUDSTACK-7534: ResetVM for VM with attached datadisk fails when enable.ha.storage.migration is false

[Harikrishna Patnala]

---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/25530/
---

Review request for cloudstack and Kishan Kavala.


Bugs: CLOUDSTACK-7534
https://issues.apache.org/jira/browse/CLOUDSTACK-7534


Repository: cloudstack-git


Description
---

   CLOUDSTACK-7534: ResetVM for VM with attached datadisk fails when 
enable.ha.storage.migration is false

Separate global config to enable/disable Storage Migration during normal 
deployment
Introduced a configuration parameter named enable.storage.migration


Diffs
-

  api/src/com/cloud/vm/VirtualMachineProfile.java 29f3164 
  
engine/orchestration/src/org/apache/cloudstack/engine/orchestration/VolumeOrchestrator.java
 afc6f63 
  server/src/com/cloud/ha/HighAvailabilityManagerImpl.java 310f361 

Diff: https://reviews.apache.org/r/25530/diff/


Testing
---


Thanks,

Harikrishna Patnala

Challenges faced by cloudstack committers, developers, users

[Madan Ganesh V]

Hello CloudStack Developers & Users, 

Trust everyone is doing well.

Myself, Madan, founder & CEO of ActOnMagic.

We’re currently evaluating the possibility of developer edition of 
www.actoncloud.com for CloudStack dev/users. 

A simple SaaS based BI tool to help you analyse & root cause your cloudstack 
dump faster and enabling you to be more productive!

Appreciate if you could share your pain points as a committer, developer or 
user?

Thanks and Regards,
Madan

Founder & CEO
www.actoncloud.com