DeltaCloud
  1. DeltaCloud
  2. DTACLOUD-216

exception when using 'run command' in GUI on an instance with no public ip

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Minor Minor
    • Resolution: Fixed
    • Component/s: Server
    • Labels:
      None

      Description

      Looking at the code, this applies to all drivers that support the run_on_instance operation.
      The error is: undefined method `address' for nil:NilClass

      I'd like to suggest we add a selection box to the GUI to display all public and private (DC could be running on a VM in the same network after all) IP addresses of the VM so the user can chose. That should also prevent this error.

        Activity

        Hide
        Dies Koper added a comment -

        patch that implements the suggestion

        Show
        Dies Koper added a comment - patch that implements the suggestion
        Hide
        Michal Fojtik added a comment -

        Pushed:

        commit aa7c79be75f5636138209a266bc3e69a732715b5
        Author: Dies Koper <diesk@fast.au.fujitsu.com>
        Date: Mon May 21 10:02:59 2012 +1000

        Allow user to chose target ip for run_on_instance

        Show
        Michal Fojtik added a comment - Pushed: commit aa7c79be75f5636138209a266bc3e69a732715b5 Author: Dies Koper <diesk@fast.au.fujitsu.com> Date: Mon May 21 10:02:59 2012 +1000 Allow user to chose target ip for run_on_instance
        Hide
        Dies Koper added a comment -

        The patch I submitted worked perfectly fine at the time. Now that you've merged it in, I get an error when I click on the run command button:

        http://localhost:3001/api/run_instances/inst1;id=inst1
        Sinatra doesn’t know this ditty.

        Fails both on mock driver and fgcp driver.

        Maybe something in the modular patches changed the url and broke it?

        Show
        Dies Koper added a comment - The patch I submitted worked perfectly fine at the time. Now that you've merged it in, I get an error when I click on the run command button: http://localhost:3001/api/run_instances/inst1;id=inst1 Sinatra doesn’t know this ditty. Fails both on mock driver and fgcp driver. Maybe something in the modular patches changed the url and broke it?
        Hide
        Marios Andreou added a comment -

        This was fixed in commit # 5fddc78a85d89203c77088d7b90c4abe6736ab35 "Core: Fixed typo in instance HAML view"

        Show
        Marios Andreou added a comment - This was fixed in commit # 5fddc78a85d89203c77088d7b90c4abe6736ab35 "Core: Fixed typo in instance HAML view"

          People

          • Assignee:
            Michal Fojtik
            Reporter:
            Dies Koper
          • Votes:
            0 Vote for this issue
            Watchers:
            3 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development