Re: RHEV-M: avoid failure of GET instance details with test fixes

2013-02-27 Thread jvlcek
On 02/27/2013 06:24 PM, jvl...@redhat.com wrote: > A backtrace was sporadically produced for a > GET /api/instance/ > > When one instance is being destroyed during a GET for > another instance, a backtrace is produced. This is because > a query for all instances (vms) was being done. Th

[jira] [Updated] (DTACLOUD-462) RHEVM calling GET /api/instances/instance_id after the instance is stopped, errors out

2013-02-27 Thread Joseph J. VLcek (JIRA)
[ https://issues.apache.org/jira/browse/DTACLOUD-462?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Joseph J. VLcek updated DTACLOUD-462: - Attachment: 0001-RHEV-M-avoid-failure-of-GET-instance-details.patch An updated patch, t

RHEV-M: avoid failure of GET instance details with test fixes

2013-02-27 Thread jvlcek
A backtrace was sporadically produced for a GET /api/instance/ When one instance is being destroyed during a GET for another instance, a backtrace is produced. This is because a query for all instances (vms) was being done. The list of all instances is gathered, one instance is destr

Deltacloud - Tests - adding blackbox tests for addresses and drivers collections

2013-02-27 Thread rlandy
Skeleton tests for the last two collections missing tests. Rakefile update to include those tests in the rake task

[PATCH] Deltacloud - Tests - adding blackbox tests for addresses and drivers collections

2013-02-27 Thread rlandy
From: Ronelle Landy --- tests/Rakefile |2 +- tests/deltacloud/addresses_test.rb | 28 ++ tests/deltacloud/common_tests_collections.rb |6 ++-- tests/deltacloud/drivers_test.rb | 28 ++

Deltacloud Network API discussion

2013-02-27 Thread mar...@redhat.com
Hi after today's Deltacloud Network API discussion and as general information for anyone that wasn't on the call - we agreed that we would each: 1. fill in the spreadsheet @ https://docs.google.com/spreadsheet/ccc?key=0AnIzjKffwLzqdEV5RGFBWmpfeFZqSXJBZHZzQUNQbEE#gid=0 (please feel free to change

[jira] [Commented] (DTACLOUD-480) Digitalocean driver: undefined method `valid_credentials?' error when using using "?force_auth" parameter

2013-02-27 Thread Marios Andreou (JIRA)
[ https://issues.apache.org/jira/browse/DTACLOUD-480?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13588546#comment-13588546 ] Marios Andreou commented on DTACLOUD-480: - thanks - pushed with commit # de51348

[jira] [Resolved] (DTACLOUD-480) Digitalocean driver: undefined method `valid_credentials?' error when using using "?force_auth" parameter

2013-02-27 Thread Marios Andreou (JIRA)
[ https://issues.apache.org/jira/browse/DTACLOUD-480?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Marios Andreou resolved DTACLOUD-480. - Resolution: Fixed > Digitalocean driver: undefined method `valid_credentials?' erro

[jira] [Closed] (DTACLOUD-480) Digitalocean driver: undefined method `valid_credentials?' error when using using "?force_auth" parameter

2013-02-27 Thread Marios Andreou (JIRA)
[ https://issues.apache.org/jira/browse/DTACLOUD-480?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Marios Andreou closed DTACLOUD-480. --- > Digitalocean driver: undefined method `valid_credentials?' error when using > using "?f

[jira] [Closed] (DTACLOUD-477) Openstack provider: Realms id not correctly reported from GET /api/instance/Instance_id

2013-02-27 Thread Marios Andreou (JIRA)
[ https://issues.apache.org/jira/browse/DTACLOUD-477?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Marios Andreou closed DTACLOUD-477. --- > Openstack provider: Realms id not correctly reported from GET > /api/instance/Instance_i

[jira] [Resolved] (DTACLOUD-477) Openstack provider: Realms id not correctly reported from GET /api/instance/Instance_id

2013-02-27 Thread Marios Andreou (JIRA)
[ https://issues.apache.org/jira/browse/DTACLOUD-477?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Marios Andreou resolved DTACLOUD-477. - Resolution: Fixed > Openstack provider: Realms id not correctly reported from GET

[jira] [Commented] (DTACLOUD-477) Openstack provider: Realms id not correctly reported from GET /api/instance/Instance_id

2013-02-27 Thread Marios Andreou (JIRA)
[ https://issues.apache.org/jira/browse/DTACLOUD-477?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13588544#comment-13588544 ] Marios Andreou commented on DTACLOUD-477: - re-recorded fixtures pushed after ACK

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

2013-02-27 Thread Ronelle Landy
ACK to series - GET /api/addresses and GET /cimi/addresses both work now. Was able to gem install sinatra-rabbit-1.1.5 > From: mfoj...@redhat.com > To: dev@deltacloud.apache.org > Sent: Wednesday, February 27, 2013 9:23:34 AM > Subject: [PATCH core 1/3] CIMI: Removed debug statements from > Addre

Re: [PATCH] Fixed processing of error responses

2013-02-27 Thread Michal Fojtik
On 02/27, jprov...@redhat.com wrote: ACK. Pushing. > From: Jan Provaznik > > Error responses from server don't have to contain backend node, only a > message. > For example 401 error response looks like: > > Authentication required > > > This patch makes sure that dc-client doesn't fail i

[PATCH] Fixed processing of error responses

2013-02-27 Thread jprovazn
From: Jan Provaznik Error responses from server don't have to contain backend node, only a message. For example 401 error response looks like: Authentication required This patch makes sure that dc-client doesn't fail if 'backend' node is not present. --- client/lib/deltacloud.rb | 6 --

[PATCH core 3/3] Core: Change prefix for the Sinatra::Rabbit classes

2013-02-27 Thread mfojtik
From: Michal Fojtik Sinatra::Rabbit for CIMI is now CIMI::Rabbit Sinatra::Rabbit for Deltacloud is now Deltacloud::CIMI Signed-off-by: Michal fojtik --- server/lib/deltacloud/drivers/openstack/openstack_driver.rb| 6 +++--- server/lib/deltacloud/helpers/deltacloud_helper.rb |

[PATCH core 2/3] Core: Fixed Rabbit namespacing issues with Addresses (DTACLOUD-500)

2013-02-27 Thread mfojtik
From: Michal Fojtik NOTE: A sinatra-rabbit >= 1.15 is required to make this work and to fix the bug. Signed-off-by: Michal fojtik --- server/deltacloud-core.gemspec| 2 +- server/lib/cimi/collections/base.rb | 4 server/lib/deltacloud/collections/base.rb | 4 3 file

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

2013-02-27 Thread mfojtik
From: Michal Fojtik Signed-off-by: Michal fojtik --- server/lib/cimi/collections/addresses.rb | 2 -- 1 file changed, 2 deletions(-) diff --git a/server/lib/cimi/collections/addresses.rb b/server/lib/cimi/collections/addresses.rb index 92aca3e..c34068d 100644 --- a/server/lib/cimi/collection

Re: Deltacloud - Tests - adding testpoint for creating an image from a stopped instance - if supported

2013-02-27 Thread Ronelle Landy
> From: "Dies Koper" > To: dev@deltacloud.apache.org > Sent: Tuesday, February 26, 2013 7:24:38 PM > Subject: RE: Deltacloud - Tests - adding testpoint for creating an image from > a stopped instance - if supported > > Hi Ronelle, > > Thanks for picking this up. It was on my todo list but have

[jira] [Commented] (DTACLOUD-500) CIMI: GET /cimi/addresses, using application/xml errors out

2013-02-27 Thread Michal Fojtik (JIRA)
[ https://issues.apache.org/jira/browse/DTACLOUD-500?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13588341#comment-13588341 ] Michal Fojtik commented on DTACLOUD-500: Fix is here: http://tracker.deltacloud

[PATCH core 2/2] Core: Fixed Rabbit namespacing issues with Addresses (DTACLOUD-500)

2013-02-27 Thread mfojtik
From: Michal Fojtik NOTE: A sinatra-rabbit >= 1.15 is required to make this work and to fix the bug. Signed-off-by: Michal fojtik --- server/deltacloud-core.gemspec| 2 +- server/lib/cimi/collections/base.rb | 4 server/lib/deltacloud/collections/base.rb | 4 3 file

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

2013-02-27 Thread mfojtik
From: Michal Fojtik Signed-off-by: Michal fojtik --- server/lib/cimi/collections/addresses.rb | 2 -- 1 file changed, 2 deletions(-) diff --git a/server/lib/cimi/collections/addresses.rb b/server/lib/cimi/collections/addresses.rb index 92aca3e..c34068d 100644 --- a/server/lib/cimi/collection

[jira] [Commented] (DTACLOUD-500) CIMI: GET /cimi/addresses, using application/xml errors out

2013-02-27 Thread Michal Fojtik (JIRA)
[ https://issues.apache.org/jira/browse/DTACLOUD-500?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13588266#comment-13588266 ] Michal Fojtik commented on DTACLOUD-500: Indeed, this is most likely a Rabbit bu

Re: [PATCH] CIMI schema: make it possible to build an array of references

2013-02-27 Thread Michal Fojtik
On 02/26, lut...@redhat.com wrote: ACK! Pushing. -- Michal > From: David Lutterkort > > It's now possible to say > > array :things, :ref => CIMI::Model::Target > > to have an array things where each entry is a TargetRef > > Fixes https://issues.apache.org/jira/browse/DTACLOUD-499 > ---