update nova to report quantum floating IPs

Bug #1023169 reported by dan wendlandt on 2012-07-10
18
This bug affects 2 people
Affects Status Importance Assigned to Milestone
OpenStack Compute (nova)
High
Akihiro Motoki
Folsom
High
Chuck Short
nova (Ubuntu)
Undecided
Unassigned
Quantal
Undecided
Unassigned

Bug Description

In F-3, we've moving Floating IPs to Quantum.

We'll need to update the Nova/Quantum integration code to include these floating IPs in the nova models.

dan wendlandt (danwent) on 2012-07-10
Changed in quantum:
milestone: none → folsom-3
importance: Undecided → High
tags: added: low-hanging-fruit
dan wendlandt (danwent) on 2012-07-30
Changed in quantum:
status: New → Confirmed
flaviamissi (flaviamissi) wrote :

Hi,

I've look at the nova/quantum integration code and I have some doubts:

 - By "include in the nova models" you mean that we should save the floating ips in nova's db?
 - Also, I haven't seen any related code in python-quantumclient, is this work included on this issue or there's a separate one?

I'm sorry if I'm asking newbie questions, it's my first time contributing o/

Thanks!

dan wendlandt (danwent) wrote :

No, when nova queries for network information, this data is returned in something called a "network model": https://github.com/openstack/nova/blob/master/nova/network/model.py

Both the quantum-server and quantumclient code for this is still in review (https://review.openstack.org/#/c/10994/) and may change in the next few days.

flaviamissi (flaviamissi) wrote :

Hi, Dan!

Thanks for the update, I haven't seen this implementation from quantumclient, only the one in quantum-server.

So correct me if I'm wrong: this issue is about adding a FloatingIp class, equivalent to the FixedIp[1], in network model?

Thanks again!

[1] https://github.com/openstack/nova/blob/master/nova/network/model.py#L80

Changed in quantum:
assignee: nobody → flaviamissi (flaviamissi)
Changed in quantum:
status: Confirmed → In Progress
dan wendlandt (danwent) wrote :

floating IP stuff should merge tonight/tomorrow, so we can start working on this after that.

python-quantumclient changes will also be available.

Changed in quantum:
milestone: folsom-3 → folsom-rc1
Thierry Carrez (ttx) on 2012-08-22
Changed in nova:
importance: Undecided → High
status: New → Confirmed
dan wendlandt (danwent) on 2012-09-04
Changed in quantum:
assignee: flaviamissi (flaviamissi) → nobody
dan wendlandt (danwent) wrote :

I will be speaking to Thierry and Vish about the general Nova/Quantum proxy issues today. Will update this bug with the results.

dan wendlandt (danwent) on 2012-09-05
Changed in quantum:
milestone: folsom-rc1 → none
Changed in quantum:
status: In Progress → Confirmed

Fix proposed to branch: master
Review: https://review.openstack.org/13007

Changed in nova:
assignee: nobody → Akihiro Motoki (amotoki)
status: Confirmed → In Progress

Fix proposed to branch: master
Review: https://review.openstack.org/13176

Akihiro Motoki (amotoki) on 2012-09-18
Changed in quantum:
assignee: nobody → Akihiro Motoki (amotoki)

Reviewed: https://review.openstack.org/13007
Committed: http://github.com/openstack/nova/commit/7948b7a572e33e3fd541e27f2617d832eb43b504
Submitter: Jenkins
Branch: master

commit 7948b7a572e33e3fd541e27f2617d832eb43b504
Author: Akihiro MOTOKI <email address hidden>
Date: Thu Sep 13 21:22:33 2012 +0900

    Proxy floating IP calls to quantum

    Fixes bug 1023169. Also fixes bug 1031119.

    In Folsom, floating IP support is moved to Quantum. By this commit floating IP
    calls to nova are proxied to Quantum and nova command can be used to manage
    floating IPs.

    pip-requires is also updated and now requires python-quantumclient >=2.1 which
    supports floating IP feature in Quantum.

    Change-Id: I2c32948a8fe291601216dc62d546da64a8fb8428

Changed in nova:
status: In Progress → Fix Committed
dan wendlandt (danwent) on 2012-10-30
no longer affects: quantum

Reviewed: https://review.openstack.org/16193
Committed: http://github.com/openstack/nova/commit/3f7788ca4310a7fa34b7687ff6e41f8de4d2cadc
Submitter: Jenkins
Branch: stable/folsom

commit 3f7788ca4310a7fa34b7687ff6e41f8de4d2cadc
Author: Akihiro MOTOKI <email address hidden>
Date: Thu Sep 13 21:22:33 2012 +0900

    Proxy floating IP calls to quantum

    Fixes bug 1023169. Also fixes bug 1031119.

    In Folsom, floating IP support is moved to Quantum. By this commit floating IP
    calls to nova are proxied to Quantum and nova command can be used to manage
    floating IPs.

    pip-requires is also updated and now requires python-quantumclient >=2.1 which
    supports floating IP feature in Quantum.

    Change-Id: I2c32948a8fe291601216dc62d546da64a8fb8428
    (cherry picked from commit 7948b7a572e33e3fd541e27f2617d832eb43b504)

tags: added: in-stable-folsom
Thierry Carrez (ttx) on 2012-11-21
Changed in nova:
milestone: none → grizzly-1
status: Fix Committed → Fix Released
Changed in nova (Ubuntu):
status: New → Fix Released
Changed in nova (Ubuntu Quantal):
status: New → Confirmed

Hello dan, or anyone else affected,

Accepted nova into quantal-proposed. The package will build now and be available at http://launchpad.net/ubuntu/+source/nova/2012.2.1+stable-20121212-a99a802e-0ubuntu1 in a few hours, and then in the -proposed repository.

Please help us by testing this new package. See https://wiki.ubuntu.com/Testing/EnableProposed for documentation how to enable and use -proposed. Your feedback will aid us getting this update out to other Ubuntu users.

If this package fixes the bug for you, please add a comment to this bug, mentioning the version of the package you tested, and change the tag from verification-needed to verification-done. If it does not fix the bug for you, please add a comment stating that, and change the tag to verification-failed. In either case, details of your testing will help us make a better decision.

Further information regarding the verification process can be found at https://wiki.ubuntu.com/QATeam/PerformingSRUVerification . Thank you in advance!

Changed in nova (Ubuntu Quantal):
status: Confirmed → Fix Committed
tags: added: verification-needed
Mark McLoughlin (markmc) on 2013-01-22
tags: removed: in-stable-folsom
Launchpad Janitor (janitor) wrote :
Download full text (8.3 KiB)

This bug was fixed in the package nova - 2012.2.1+stable-20121212-a99a802e-0ubuntu1

---------------
nova (2012.2.1+stable-20121212-a99a802e-0ubuntu1) quantal-proposed; urgency=low

  * Ubuntu updates:
    - debian/control: Ensure novaclient is upgraded with nova,
      require python-keystoneclient >= 1:2.9.0. (LP: #1073289)
    - d/p/avoid_setuptools_git_dependency.patch: Refresh.
  * Dropped patches, applied upstream:
    - debian/patches/CVE-2012-5625.patch: [a99a802]
  * Resynchronize with stable/folsom (b55014ca) (LP: #1085255):
    - [a99a802] create_lvm_image allocates dirty blocks (LP: #1070539)
    - [670b388] RPC exchange name defaults to 'openstack' (LP: #1083944)
    - [3ede373] disassociate_floating_ip with multi_host=True fails
      (LP: #1074437)
    - [22d7c3b] libvirt imagecache should handle shared image storage
      (LP: #1075018)
    - [e787786] Detached and deleted RBD volumes remain associated with insance
      (LP: #1083818)
    - [9265eb0] live_migration missing migrate_data parameter in Hyper-V driver
      (LP: #1066513)
    - [3d99848] use_single_default_gateway does not function correctly
      (LP: #1075859)
    - [65a2d0a] resize does not migrate DHCP host information (LP: #1065440)
    - [102c76b] Nova backup image fails (LP: #1065053)
    - [48a3521] Fix config-file overrides for nova-dhcpbridge
    - [69663ee] Cloudpipe in Folsom: no such option: cnt_vpn_clients
      (LP: #1069573)
    - [6e47cc8] DisassociateAddress can cause Internal Server Error
      (LP: #1080406)
    - [22c3d7b] API calls to dis-associate an auto-assigned floating IP should
      return proper warning (LP: #1061499)
    - [bd11d15] libvirt: if exception raised during volume_detach, volume state
      is inconsistent (LP: #1057756)
    - [dcb59c3] admin can't describe all images in ec2 api (LP: #1070138)
    - [78de622] Incorrect Exception raised during Create server when metadata
      over 255 characters (LP: #1004007)
    - [c313de4] Fixed IP isn't released before updating DHCP host file
      (LP: #1078718)
    - [f4ab42d] Enabling Return Reservation ID with XML create server request
      returns no body (LP: #1061124)
    - [3db2a38] 'BackupCreate' should accept rotation parameter greater than or
      equal to zero (LP: #1071168)
    - [f7e5dde] libvirt reboot sometimes fails to reattach volumes
      (LP: #1073720)
    - [ff776d4] libvirt: detaching volume may fail while terminating other
      instances on the same host concurrently (LP: #1060836)
    - [85a8bc2] Used instance uuid rather than id in remove-fixed-ip
    - [42a85c0] Fix error on invalid delete_on_termination value
    - [6a17579] xenapi migrations fail w/ swap (LP: #1064083)
    - [97649b8] attach-time field for volumes is not updated for detach volume
      (LP: #1056122)
    - [8f6a718] libvirt: rebuild is not using kernel and ramdisk associated with
      the new image (LP: #1060925)
    - [fbe835f] live-migration and volume host assignement (LP: #1066887)
    - [c2a9150] typo prevents volume_tmp_dir flag from working (LP: #1071536)
    - [93efa21] Instances deleted during spawn leak network allocations
      (LP: #1068716)
    - [ebabd02] After restartin...

Read more...

Changed in nova (Ubuntu Quantal):
status: Fix Committed → Fix Released
Thierry Carrez (ttx) on 2013-04-04
Changed in nova:
milestone: grizzly-1 → 2013.1
To post a comment you must log in.
This report contains Public information  Edit
Everyone can see this information.

Other bug subscribers