Re: CIMI::Models split to CIMI::Services (rev 2) WIP

2013-03-01 Thread David Lutterkort
On Fri, 2013-03-01 at 14:40 +0100, Michal Fojtik wrote: > - ResourceMetada are now broken and need to be fixed. > (an informative warning included ;-) I added one more patch to your series and rerecorded as http://tracker.deltacloud.org/set/364 > - The filter_by and select_by methods moved to

[jira] [Resolved] (DTACLOUD-484) RHEVm Need to handle GET for instances while some could be being destroyed.

2013-03-01 Thread Joseph J. VLcek (JIRA)
[ https://issues.apache.org/jira/browse/DTACLOUD-484?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Joseph J. VLcek resolved DTACLOUD-484. -- Resolution: Fixed Fix is available in this patch and at https://github.com/jvlcek/rbo

[jira] [Updated] (DTACLOUD-484) RHEVm Need to handle GET for instances while some could be being destroyed.

2013-03-01 Thread Joseph J. VLcek (JIRA)
[ https://issues.apache.org/jira/browse/DTACLOUD-484?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Joseph J. VLcek updated DTACLOUD-484: - Attachment: 0001-RHEV-M-avoid-failure-when-accessing-instances-when-a.patch I've tested

[jira] [Commented] (DTACLOUD-498) mock driver doesn't update bucket size when a blob is created

2013-03-01 Thread Marios Andreou (JIRA)
[ https://issues.apache.org/jira/browse/DTACLOUD-498?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13590916#comment-13590916 ] Marios Andreou commented on DTACLOUD-498: - thanks! i had some merge conflicts a

[jira] [Updated] (DTACLOUD-498) mock driver doesn't update bucket size when a blob is created

2013-03-01 Thread Ronelle Landy (JIRA)
[ https://issues.apache.org/jira/browse/DTACLOUD-498?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Ronelle Landy updated DTACLOUD-498: --- Attachment: 0002-DTACLOUD-498-patch-minor-corrections.patch Attaching a second patch. With

[PATCH 1/2] DTACLOUD-498 - nit in mock driver create/delete blobs - update bucket size

2013-03-01 Thread rlandy
From: marios https://issues.apache.org/jira/browse/DTACLOUD-498 Signed-off-by: marios TrackedAt: http://tracker-mfojtik.rhcloud.com/patch/03b3210c6e63a36f3881904468dc00e34d858aad --- server/lib/deltacloud/drivers/mock/mock_driver.rb | 14 ++ 1 files changed, 14 insertions(+), 0

[PATCH 2/2] DTACLOUD-498 - patch - minor corrections

2013-03-01 Thread rlandy
From: Ronelle Landy --- server/lib/deltacloud/drivers/mock/mock_driver.rb |4 ++-- 1 files changed, 2 insertions(+), 2 deletions(-) diff --git a/server/lib/deltacloud/drivers/mock/mock_driver.rb b/server/lib/deltacloud/drivers/mock/mock_driver.rb index 7d7b17b..07c67e2 100644 --- a/server/

DTACLOUD-498 - patch - minor corrections

2013-03-01 Thread rlandy
Originally, NACK'ed http://tracker.deltacloud.org/set/362 because the changes caused an error (see DTACOUD-498 for details). However, it looks like the patch was just missing two ')'. [PATCH 2/2] adds these to the patch. Testing with both patches, the mock driver shows that the bucket number do

[jira] [Commented] (DTACLOUD-498) mock driver doesn't update bucket size when a blob is created

2013-03-01 Thread Ronelle Landy (JIRA)
[ https://issues.apache.org/jira/browse/DTACLOUD-498?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13590808#comment-13590808 ] Ronelle Landy commented on DTACLOUD-498: Applying this patch and starting the De

[jira] [Updated] (DTACLOUD-498) mock driver doesn't update bucket size when a blob is created

2013-03-01 Thread Marios Andreou (JIRA)
[ https://issues.apache.org/jira/browse/DTACLOUD-498?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Marios Andreou updated DTACLOUD-498: Attachment: 0001-DTACLOUD-498-nit-in-mock-driver-create-delete-blobs-.patch patch also at

[jira] [Commented] (DTACLOUD-478) Add digitalocean to list of supported providers + drivers

2013-03-01 Thread Marios Andreou (JIRA)
[ https://issues.apache.org/jira/browse/DTACLOUD-478?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13590759#comment-13590759 ] Marios Andreou commented on DTACLOUD-478: - patch @ https://issues.apache.org/jir

[jira] [Updated] (DTACLOUD-479) Inconsistent lists of supported providers/drivers in Deltacloud doc

2013-03-01 Thread Marios Andreou (JIRA)
[ https://issues.apache.org/jira/browse/DTACLOUD-479?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Marios Andreou updated DTACLOUD-479: Attachment: 0001-SITE-addresses-DTACLOUD-478-DTACLOUD-479-and-removes.patch patch also at

[jira] [Created] (DTACLOUD-505) CIMI/Deltacloud: Starting deltacloud server with multiple frontend fails - depends on order of arguments

2013-03-01 Thread Ronelle Landy (JIRA)
Ronelle Landy created DTACLOUD-505: -- Summary: CIMI/Deltacloud: Starting deltacloud server with multiple frontend fails - depends on order of arguments Key: DTACLOUD-505 URL: https://issues.apache.org/jira/browse/

Re: [PATCH core] Core: Added support for jruby Puma server

2013-03-01 Thread Francesco Vollero
After a quick chat with Michal on #aeolus @ freenode I decided to ACK it because it work and Michal promised will add the missing line that was breaking. Cheers, -FV On Mar 1, 2013, at 11:17 AM, mfoj...@redhat.com wrote: > From: Michal Fojtik > > The 'puma' web server is much faster and reli

Re: [PATCH core] Core: Added support for jruby Puma server

2013-03-01 Thread Michal Fojtik
On 03/01, Francesco Vollero wrote: Yeah, this is not related to this patch. For some reason we removed the org.sqlite.JDBC initialization. The patch: server/lib/initializers/database_initialize.rb +if RUBY_PLATFORM == 'java' + require 'jdbc/sqlite3' + Jdbc::SQLite3.load_driver +end I'll send

Re: [PATCH core] Core: Added support for jruby Puma server

2013-03-01 Thread Francesco Vollero
I tested the patch within the mock driver and seems to work from curl and from browser While I was running the tests on mock driver I hit a problem and the backtrace is here: Obviously before testing it, I run it from a "clean" gemset and run bundle install on it. -FV ## Backtrace [fvolle

CIMI::Models split to CIMI::Services (rev 2) WIP

2013-03-01 Thread Michal Fojtik
Hi, This is an extended version of David patch. The goal here is to migrate all CIMI 'service' methods to CIMI::Service models and to make CIMI::Model contain just schema. This patch adds/remove/breaks: - ResourceMetada are now broken and need to be fixed. (an informative warning included ;-)

[PATCH core 9/9] CIMI: Use CIMI::Service namespace in Rabbit collections

2013-03-01 Thread mfojtik
From: Michal Fojtik Signed-off-by: Michal fojtik --- server/lib/cimi/collections/address_templates.rb | 6 +++--- server/lib/cimi/collections/addresses.rb | 6 +++--- server/lib/cimi/collections/cloud_entry_point.rb | 2 +- server/lib/cimi/collections/credentials.rb

[PATCH core 6/9] CIMI: Added Service::Base#parse method

2013-03-01 Thread mfojtik
From: Michal Fojtik Make possible to call .parse on Service models from Rabbit collections. Signed-off-by: Michal fojtik --- server/lib/cimi/service/base.rb | 5 + 1 file changed, 5 insertions(+) diff --git a/server/lib/cimi/service/base.rb b/server/lib/cimi/service/base.rb index 9f2c765.

[PATCH core 7/9] FIXME: The ResourceMetadata are broken at this point

2013-03-01 Thread mfojtik
From: Michal Fojtik Signed-off-by: Michal fojtik --- server/lib/cimi/models/schema.rb | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/server/lib/cimi/models/schema.rb b/server/lib/cimi/models/schema.rb index 4ce16b9..6cbf8cc 100644 --- a/server/lib/cimi/models/schema.rb +

[PATCH core 5/9] CIMI: Migrated MachineImage to services

2013-03-01 Thread mfojtik
From: Michal Fojtik Signed-off-by: Michal fojtik --- server/lib/cimi/collections/machine_images.rb | 6 ++-- server/lib/cimi/models/machine_image.rb | 28 --- server/lib/cimi/service.rb| 1 + server/lib/cimi/service/machine_image.rb | 51 +++

[PATCH core 4/9] CIMI: Send list of invalid parameters in message

2013-03-01 Thread mfojtik
From: Michal Fojtik Signed-off-by: Michal fojtik --- server/tests/cimi/collections/machine_templates_test.rb | 1 - 1 file changed, 1 deletion(-) diff --git a/server/tests/cimi/collections/machine_templates_test.rb b/server/tests/cimi/collections/machine_templates_test.rb index 696052a..cfe2

[PATCH core 3/9] CIMI: Apply $filter and $select in Base#list

2013-03-01 Thread mfojtik
From: Michal Fojtik No need to have ugly chained methods in collections. Signed-off-by: Michal fojtik --- server/lib/cimi/collections/machines.rb | 2 +- server/lib/cimi/service/base.rb | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/server/lib/cimi/collections/m

[PATCH core 2/9] CIMI: Fix 'views' folder location

2013-03-01 Thread mfojtik
From: Michal Fojtik In CIMI we don't have views except the error ones. Signed-off-by: Michal fojtik --- server/lib/cimi/collections/base.rb | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/server/lib/cimi/collections/base.rb b/server/lib/cimi/collections/base.rb index 3

[PATCH core 1/9] CIMI: split models into model and service objects

2013-03-01 Thread mfojtik
From: David Lutterkort The current CIMI::Model classes address two concerns: * serialization/deserialization of CIMI objects * interaction with the current driver and the DB This patch splits these two concerns into two separate class hierarchies: CIMI::Model for (de)serialization and CIMI:

Re: RFC: Separate CIMI (de)serialization from server functionality

2013-03-01 Thread Michal Fojtik
On 02/28, lut...@redhat.com wrote: > > This patch shows how we can split the current CIMI::Model classes into two > different class hierarchies, CIMI::Model and CIMI::Service. After the > split, the former will strictly be restricted to (de)serialization of CIMI > resources whereas the latter will

[PATCH core] Switch the rubygems.org source to https://rubygems.org

2013-03-01 Thread mfojtik
From: Michal Fojtik Using HTTPS connection is now prefered way how to access rubygems. Under MRI 2.0 having non-https URL will print a huge ugly warning. Signed-off-by: Michal fojtik --- server/Gemfile | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/server/Gemfile b/server/

[PATCH core] Core: Added support for jruby Puma server

2013-03-01 Thread mfojtik
From: Michal Fojtik The 'puma' web server is much faster and reliable than java webrick. Signed-off-by: Michal fojtik --- server/Gemfile | 1 + server/bin/deltacloudd | 24 +--- 2 files changed, 18 insertions(+), 7 deletions(-) diff --git a/server/Gemfile b/server

[jira] [Commented] (DTACLOUD-456) Opennebula: Error occurs when requesting all instances against opennebula

2013-03-01 Thread Christian Karnath (JIRA)
[ https://issues.apache.org/jira/browse/DTACLOUD-456?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13590372#comment-13590372 ] Christian Karnath commented on DTACLOUD-456: Hi Ronelle, Thank you for doin

Re: [PATCH core 1/3] CIMI: Removed debug statements from AddressCollection

2013-03-01 Thread Michal Fojtik
On 02/27, Ronelle Landy wrote: > ACK to series - GET /api/addresses and GET /cimi/addresses both work now. > Was able to gem install sinatra-rabbit-1.1.5 Thanks! Pushed. > > > From: mfoj...@redhat.com > > To: dev@deltacloud.apache.org > > Sent: Wednesday, February 27, 2013 9:23:34 AM > > Subject

Re: Minor cleanup patches

2013-03-01 Thread Michal Fojtik
On 02/28, lut...@redhat.com wrote: ACK, pushing. > > These patches address a few small annoyances > > David -- Michal Fojtik Deltacloud API, CloudForms

Re: Minor cleanup patches

2013-03-01 Thread mar...@redhat.com
On 01/03/13 02:08, lut...@redhat.com wrote: > These patches address a few small annoyances > > David > ACK