RE: [PATCH] FGCP: added support for storage_volume creation from snapshot

2013-02-22 Thread Koper, Dies
> ACK; though there are a couple of things that need improving: (1) the > fact that there are no unit tests and (2) spawning a new thread for the > snapshot creation: I discussed (1) with mfojtik on irc. The problem is that I'm polling the server until the volume's state changes from CREATING to

[jira] [Commented] (DTACLOUD-497) image on /api/instance_states page does not display on Chrome and IE (Windows)

2013-02-22 Thread David Lutterkort (JIRA)
[ https://issues.apache.org/jira/browse/DTACLOUD-497?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13584932#comment-13584932 ] David Lutterkort commented on DTACLOUD-497: --- This might be caused by ImageMagi

Re: [PATCH] GUI: made capitalization in titles and buttons consistent

2013-02-22 Thread David Lutterkort
On Sat, 2013-02-23 at 12:22 +1100, di...@fast.au.fujitsu.com wrote: > From: Dies Koper > > --- > server/views/addresses/show.html.haml | 2 +- > server/views/buckets/show.html.haml| 2 +- > server/views/firewalls/new.html.haml | 4 ++-- > server/views/images/show.h

Re: [PATCH] FGCP: add mapping for GET system and system_templates

2013-02-22 Thread David Lutterkort
Hi Dies, looks good, though I don't understand the networks method at all: On Fri, 2013-02-22 at 23:45 +1100, di...@fast.au.fujitsu.com wrote: > diff --git a/server/lib/deltacloud/drivers/fgcp/fgcp_driver_cimi_methods.rb > b/server/lib/deltacloud/drivers/fgcp/fgcp_driver_cimi_methods.rb > new f

[PATCH] GUI: made capitalization in titles and buttons consistent

2013-02-22 Thread diesk
From: Dies Koper --- server/views/addresses/show.html.haml | 2 +- server/views/buckets/show.html.haml| 2 +- server/views/firewalls/new.html.haml | 4 ++-- server/views/images/show.html.haml | 2 +- server/views/instances/run_command.html.haml | 2 +-

[jira] [Created] (DTACLOUD-497) image on /api/instance_states page does not display on Chrome and IE (Windows)

2013-02-22 Thread Dies Koper (JIRA)
Dies Koper created DTACLOUD-497: --- Summary: image on /api/instance_states page does not display on Chrome and IE (Windows) Key: DTACLOUD-497 URL: https://issues.apache.org/jira/browse/DTACLOUD-497 Projec

[jira] [Created] (DTACLOUD-496) /api/addresses returns 500 when cimi frontend is also enabled

2013-02-22 Thread Dies Koper (JIRA)
Dies Koper created DTACLOUD-496: --- Summary: /api/addresses returns 500 when cimi frontend is also enabled Key: DTACLOUD-496 URL: https://issues.apache.org/jira/browse/DTACLOUD-496 Project: DeltaCloud

Re: [PATCH] FGCP: added support for storage_volume creation from snapshot

2013-02-22 Thread David Lutterkort
On Fri, 2013-02-22 at 12:11 +1100, di...@fast.au.fujitsu.com wrote: > From: Dies Koper > > --- > server/lib/deltacloud/drivers/fgcp/fgcp_client.rb | 44 > +++ > server/lib/deltacloud/drivers/fgcp/fgcp_driver.rb | 38 +++- > server/views/storage_volumes/show.h

[PATCH 9/9] CIMI tests (rmd4_test): simplify the code a little

2013-02-22 Thread lutter
From: David Lutterkort --- tests/cimi/rmd4_test.rb | 68 ++--- 1 file changed, 36 insertions(+), 32 deletions(-) diff --git a/tests/cimi/rmd4_test.rb b/tests/cimi/rmd4_test.rb index 1d721fc..d08a24d 100644 --- a/tests/cimi/rmd4_test.rb +++ b/tests/cim

[PATCH 8/9] CIMI (MachineCreate): pass initialState to driver

2013-02-22 Thread lutter
From: David Lutterkort --- server/lib/cimi/models/machine_create.rb | 2 ++ server/lib/cimi/models/machine_template.rb | 1 + 2 files changed, 3 insertions(+) diff --git a/server/lib/cimi/models/machine_create.rb b/server/lib/cimi/models/machine_create.rb index c0b107c..0f04c01 100644 --- a/

[PATCH 6/9] CIMI models (VolumeConfig): add missing type attribute

2013-02-22 Thread lutter
From: David Lutterkort --- server/lib/cimi/models/volume_configuration.rb | 1 + 1 file changed, 1 insertion(+) diff --git a/server/lib/cimi/models/volume_configuration.rb b/server/lib/cimi/models/volume_configuration.rb index 7f67051..bf94a3a 100644 --- a/server/lib/cimi/models/volume_configu

[PATCH 7/9] CIMI tests: href on MachineConfig is now a proper attribute

2013-02-22 Thread lutter
From: David Lutterkort Fixes a test failure --- tests/cimi/part3_test.rb | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/tests/cimi/part3_test.rb b/tests/cimi/part3_test.rb index 50e7aae..10243ec 100644 --- a/tests/cimi/part3_test.rb +++ b/tests/cimi/part3_test.rb @@ -67,

[PATCH 3/9] CIMI: Use class.superclass instead of opts[:class]

2013-02-22 Thread lutter
From: Michal Fojtik Under latest MRI 1.9.3 the 'opts[:class]' is not visible to 'find' method and it is reported as 'method missing' With 'class.superclass' we get access to the original class we used to create Ref. Signed-off-by: Michal fojtik TrackedAt: http://tracker.deltacloud.org/patch/a

[PATCH 5/9] CIMI Model: do not trip over Fixnums during validation

2013-02-22 Thread lutter
From: David Lutterkort There's no Fixnum#empty?; convert everything to a String before checking for emptiness. --- server/lib/cimi/models/schema.rb | 2 +- server/tests/cimi/model/resource_spec.rb | 40 2 files changed, 41 insertions(+), 1 deletion(-) c

[PATCH 4/9] CIMI models: move validate! to CIMI::Model::Resource

2013-02-22 Thread lutter
From: David Lutterkort It also needs to be available for collections, which do not derive from Base --- server/lib/cimi/models/base.rb | 7 --- server/lib/cimi/models/resource.rb | 7 +++ 2 files changed, 7 insertions(+), 7 deletions(-) diff --git a/server/lib/cimi/models/base.rb b/

Fix test failures

2013-02-22 Thread lutter
The *Create patches seem to have caused quite a bit of test failures. With these patches, both the CIMI unit tests and the blackbox tests pass again. This series also incorporates Michal's earlier patches (patch set 337) David

[PATCH 2/9] CIMI: Added ref_id helper to Base class

2013-02-22 Thread lutter
From: Michal Fojtik The ref_id(url) will extract the last segment of the URL and return it as an 'id'. FIXME: We should use ctx to return the ID of resource properly. Signed-off-by: Michal fojtik TrackedAt: http://tracker.deltacloud.org/patch/90eb17744273b3e64b01808f7692fe57c4cff097 --- serv

[PATCH 1/9] CIMI: Added VolumeCreate model

2013-02-22 Thread lutter
From: Michal Fojtik Signed-off-by: Michal fojtik TrackedAt: http://tracker.deltacloud.org/patch/15544d24bd245a6d5ecee21e0efb3aaa2f3856f2 --- server/lib/cimi/collections/volumes.rb | 3 +- server/lib/cimi/models.rb | 3 +- server/lib/cimi/models/volume.rb

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

2013-02-22 Thread David Lutterkort (JIRA)
[ https://issues.apache.org/jira/browse/DTACLOUD-477?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] David Lutterkort reopened DTACLOUD-477: --- The unit tests now fail, since they require rerecording. > Openstack p

[jira] [Resolved] (DTACLOUD-488) Openstack driver - re-map regions to providers rather than realms

2013-02-22 Thread Marios Andreou (JIRA)
[ https://issues.apache.org/jira/browse/DTACLOUD-488?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Marios Andreou resolved DTACLOUD-488. - Resolution: Fixed > Openstack driver - re-map regions to providers rather than real

[jira] [Commented] (DTACLOUD-488) Openstack driver - re-map regions to providers rather than realms

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

[jira] [Closed] (DTACLOUD-488) Openstack driver - re-map regions to providers rather than realms

2013-02-22 Thread Marios Andreou (JIRA)
[ https://issues.apache.org/jira/browse/DTACLOUD-488?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Marios Andreou closed DTACLOUD-488. --- > Openstack driver - re-map regions to providers rather than realms > -

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

2013-02-22 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-22 Thread Marios Andreou (JIRA)
[ https://issues.apache.org/jira/browse/DTACLOUD-477?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13584448#comment-13584448 ] Marios Andreou commented on DTACLOUD-477: - thanks for the very thorough testing

[jira] [Closed] (DTACLOUD-448) 500 Error when creating a machine template with no properties

2013-02-22 Thread Marios Andreou (JIRA)
[ https://issues.apache.org/jira/browse/DTACLOUD-448?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Marios Andreou closed DTACLOUD-448. --- > 500 Error when creating a machine template with no properties >

[jira] [Resolved] (DTACLOUD-448) 500 Error when creating a machine template with no properties

2013-02-22 Thread Marios Andreou (JIRA)
[ https://issues.apache.org/jira/browse/DTACLOUD-448?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Marios Andreou resolved DTACLOUD-448. - Resolution: Fixed pushed a5953e975f2fc65aefb43f7834ad0b1a5a4455e1 > 5

[jira] [Created] (DTACLOUD-495) ruby client does not autogenerate public_addresses method if corresponding tag is empty

2013-02-22 Thread Giulio Fidente (JIRA)
Giulio Fidente created DTACLOUD-495: --- Summary: ruby client does not autogenerate public_addresses method if corresponding tag is empty Key: DTACLOUD-495 URL: https://issues.apache.org/jira/browse/DTACLOUD-495

Re: [PATCH core 1/3] CIMI: Added VolumeCreate model

2013-02-22 Thread Michal Fojtik
On 02/22, mfoj...@redhat.com wrote: > +# WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. See the > +# License for the specific language governing permissions and limitations > +# under the License. > + > +require 'pry' Please ignore this ^^ Will be removed before push ;-) -- M

[VOTE] Deltacloud Community Call (new day/time)

2013-02-22 Thread Michal Fojtik
Hi, By the Murphy Law, tuesday we used for $SUBJECT became full of various meetings and calls that conflicts with the current time. I started a new poll for the new time. If you want to attend these calls in real-time, please mark days/time slots that fits your TZ and availibity... http://doodle

[PATCH] FGCP: add mapping for GET system and system_templates

2013-02-22 Thread diesk
From: Dies Koper --- server/lib/deltacloud/drivers/fgcp/fgcp_driver.rb | 1 + .../drivers/fgcp/fgcp_driver_cimi_methods.rb | 137 + 2 files changed, 138 insertions(+) create mode 100644 server/lib/deltacloud/drivers/fgcp/fgcp_driver_cimi_methods.rb diff --git a/se

[PATCH core 1/3] CIMI: Added VolumeCreate model

2013-02-22 Thread mfojtik
From: Michal Fojtik Signed-off-by: Michal fojtik --- server/lib/cimi/collections/volumes.rb | 3 +- server/lib/cimi/models.rb | 3 +- server/lib/cimi/models/volume.rb | 40 - server/lib/cimi/models/volume_configuration.rb | 3 +- se

[PATCH core 2/3] CIMI: Added ref_id helper to Base class

2013-02-22 Thread mfojtik
From: Michal Fojtik The ref_id(url) will extract the last segment of the URL and return it as an 'id'. FIXME: We should use ctx to return the ID of resource properly. Signed-off-by: Michal fojtik --- server/lib/cimi/models/base.rb | 4 1 file changed, 4 insertions(+) diff --git a/server

[PATCH core 3/3] CIMI: Use class.superclass instead of opts[:class]

2013-02-22 Thread mfojtik
From: Michal Fojtik Under latest MRI 1.9.3 the 'opts[:class]' is not visible to 'find' method and it is reported as 'method missing' With 'class.superclass' we get access to the original class we used to create Ref. Signed-off-by: Michal fojtik --- server/lib/cimi/models/schema.rb | 8 ++-

[jira] [Commented] (DTACLOUD-448) 500 Error when creating a machine template with no properties

2013-02-22 Thread Martha Chumo (JIRA)
[ https://issues.apache.org/jira/browse/DTACLOUD-448?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13584209#comment-13584209 ] Martha Chumo commented on DTACLOUD-448: --- Patch fixes the problem. $gem update del

[jira] [Updated] (DTACLOUD-448) 500 Error when creating a machine template with no properties

2013-02-22 Thread Marios Andreou (JIRA)
[ https://issues.apache.org/jira/browse/DTACLOUD-448?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Marios Andreou updated DTACLOUD-448: Attachment: (was: 0001-CIMI-bug-create-CIMI-db-resource-with-no-properties-.patch)

Re: [PATCH core 08/11] Core: Fixed JSON and XML serialization of realms

2013-02-22 Thread Michal Fojtik
On 02/21, David Lutterkort wrote: > On Mon, 2013-02-11 at 13:35 +0100, mfoj...@redhat.com wrote: > > From: Michal Fojtik > > > > > > Signed-off-by: Michal fojtik > > ACK; one nit: Thanks! Fixed all two :-) > > > diff --git a/server/lib/deltacloud/models/realm.rb > > b/server/lib/deltacloud

[jira] [Commented] (DTACLOUD-448) 500 Error when creating a machine template with no properties

2013-02-22 Thread Martha Chumo (JIRA)
[ https://issues.apache.org/jira/browse/DTACLOUD-448?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13584156#comment-13584156 ] Martha Chumo commented on DTACLOUD-448: --- Then I probably did something wrong. Let