| *** ijw has quit IRC | 00:00 | |
| *** gouthamr has quit IRC | 00:04 | |
| *** marst has quit IRC | 00:04 | |
| *** dtp has quit IRC | 00:05 | |
| *** thorst_ has quit IRC | 00:10 | |
| openstackgerrit | Sivasathurappan Radhakrishnan proposed openstack/nova: Refactor _post_live_migratiion https://review.openstack.org/413021 | 00:11 |
|---|---|---|
| openstackgerrit | Sivasathurappan Radhakrishnan proposed openstack/nova: [WIP] Add version field to LibvirtLiveMigrateData https://review.openstack.org/404607 | 00:11 |
| *** ducttape_ has quit IRC | 00:11 | |
| *** ducttape_ has joined #openstack-nova | 00:11 | |
| *** Swami_ has joined #openstack-nova | 00:12 | |
| *** jose-phillips has quit IRC | 00:14 | |
| *** Swami__ has joined #openstack-nova | 00:15 | |
| *** diga has joined #openstack-nova | 00:15 | |
| *** Swami has quit IRC | 00:15 | |
| diga | gibi: Hi | 00:15 |
| diga | gibi: there ? | 00:15 |
| *** yingjun has joined #openstack-nova | 00:17 | |
| *** Swami_ has quit IRC | 00:18 | |
| mriedem | we have some placement issues https://bugs.launchpad.net/nova/+bug/1654102 | 00:19 |
| openstack | Launchpad bug 1654102 in OpenStack Compute (nova) "Ironic: TypeError: unsupported operand type(s) for *: 'NoneType' and 'int' - during select_destinations()" [Medium,Confirmed] | 00:19 |
| mriedem | sdague: seems our safe_connect isn't working there | 00:19 |
| mriedem | keystone v3 auth issues when trying to talk to the placement service | 00:20 |
| mriedem | i need to remember to dig into that with jay in the morning | 00:20 |
| *** rfolco has joined #openstack-nova | 00:20 | |
| *** rfolco has quit IRC | 00:20 | |
| diga | mriedem: do you see jaypipes around ? I need to talk to them on nova-notifications | 00:21 |
| *** sdague has quit IRC | 00:21 | |
| mriedem | diga: no he's gone for the night | 00:21 |
| diga | mriedem: ohh | 00:21 |
| mriedem | he's EST time | 00:21 |
| mriedem | plus he lives in florida where everyone goes to dinner at 4pm | 00:22 |
| diga | mriedem: okay | 00:22 |
| diga | mriedem: he will be online tomorrow ? | 00:23 |
| mriedem | i hope so | 00:23 |
| mriedem | i expect so | 00:23 |
| diga | okay | 00:23 |
| diga | okay | 00:23 |
| diga | Thank you! | 00:23 |
| diga | mriedem: do you know how I can contact gibi ? I started working on nova-notifications for placement, it seems gibi has done lot of work on that | 00:24 |
| mriedem | he'll be around tomorrow i assume | 00:24 |
| mriedem | i think he's in the EU so it'd be late for him | 00:25 |
| diga | mriedem: okay | 00:25 |
| *** ducttape_ has quit IRC | 00:25 | |
| diga | mriedem: it is very difficult to catch them, I live in IST :) I should catch them my evening time | 00:26 |
| *** jwcroppe has quit IRC | 00:27 | |
| mriedem | or email i guess, or post questions to the #openstack-dev mailing list | 00:28 |
| mriedem | tagged with [nova] in the subject | 00:28 |
| *** chyka has quit IRC | 00:29 | |
| diga | okay | 00:31 |
| mriedem | sdague_: AHA https://review.openstack.org/#/c/416599/ | 00:31 |
| diga | mriedem: how to run nova-placement-api service separately | 00:32 |
| mriedem | separately? | 00:32 |
| diga | I saw the there is way to run it, but somehow i am not able to find it | 00:32 |
| diga | yes | 00:32 |
| diga | seperately means along with nova-api | 00:33 |
| diga | mriedem: http://docs.openstack.org/developer/nova/placement.html#steps | 00:33 |
| *** amotoki has quit IRC | 00:34 | |
| diga | this is where it is explained we can run nova placement api also | 00:34 |
| *** hfu has joined #openstack-nova | 00:37 | |
| *** hfu has quit IRC | 00:37 | |
| *** dimtruck is now known as zz_dimtruck | 00:39 | |
| mriedem | diga: you'd probably have to look a bit into the linked devstack patches on how they run it as a separate service, | 00:39 |
| mriedem | i've got to head home | 00:39 |
| *** xinliang has quit IRC | 00:40 | |
| *** hongbin has quit IRC | 00:40 | |
| diga | mriedem: okay. | 00:40 |
| mriedem | diga: or if you're just setting up devstack the placement-api service is setup automatically by default now | 00:40 |
| *** jwcroppe has joined #openstack-nova | 00:40 | |
| *** mriedem has quit IRC | 00:41 | |
| diga | mriedem: I setup is devstack just 2 days ago but dont see placement-api service, but will look at it | 00:41 |
| *** jwcroppe has quit IRC | 00:45 | |
| *** hoangcx has joined #openstack-nova | 00:47 | |
| *** gongysh has quit IRC | 00:47 | |
| openstackgerrit | Hieu LE proposed openstack/nova: Remove mox from unit/virt/libvirt/test_imagebackend.py (2) https://review.openstack.org/335411 | 00:51 |
| openstackgerrit | Hieu LE proposed openstack/nova: Remove mox from unit/virt/libvirt/test_imagebackend.py (3) https://review.openstack.org/335765 | 00:51 |
| *** ducttape_ has joined #openstack-nova | 00:51 | |
| openstackgerrit | Hieu LE proposed openstack/nova: Remove mox from unit/virt/libvirt/test_imagebackend.py (4) https://review.openstack.org/335820 | 00:51 |
| *** ducttape_ has quit IRC | 00:52 | |
| openstackgerrit | Hieu LE proposed openstack/nova: Remove mox from unit/virt/libvirt/test_imagebackend.py (5) https://review.openstack.org/335921 | 00:52 |
| *** ducttape_ has joined #openstack-nova | 00:52 | |
| openstackgerrit | Hieu LE proposed openstack/nova: Remove mox from unit/virt/libvirt/test_imagebackend.py (end) https://review.openstack.org/335925 | 00:52 |
| *** thorst_ has joined #openstack-nova | 00:54 | |
| *** xinliang has joined #openstack-nova | 00:56 | |
| *** nic has quit IRC | 00:58 | |
| *** rama_y has quit IRC | 00:59 | |
| *** jamielennox is now known as jamielennox|away | 01:02 | |
| *** jose-phillips has joined #openstack-nova | 01:03 | |
| *** sreeram_v has joined #openstack-nova | 01:04 | |
| *** phuongnh has joined #openstack-nova | 01:05 | |
| *** sreeram_v has quit IRC | 01:08 | |
| *** tuanluong has joined #openstack-nova | 01:12 | |
| *** jose-phillips has quit IRC | 01:12 | |
| *** tuanluong_ has joined #openstack-nova | 01:12 | |
| *** liuyulong has joined #openstack-nova | 01:14 | |
| *** diga has quit IRC | 01:16 | |
| *** ijw has joined #openstack-nova | 01:17 | |
| *** ijw has quit IRC | 01:20 | |
| *** Jinxing has joined #openstack-nova | 01:20 | |
| *** jamielennox|away is now known as jamielennox | 01:21 | |
| *** beagles_biab is now known as beagles | 01:26 | |
| *** browne has quit IRC | 01:28 | |
| *** Swami__ has quit IRC | 01:28 | |
| *** kevinz has joined #openstack-nova | 01:30 | |
| openstackgerrit | Sivasathurappan Radhakrishnan proposed openstack/nova: Initiate network changes from source during live migration https://review.openstack.org/404607 | 01:31 |
| *** mriedem has joined #openstack-nova | 01:31 | |
| openstackgerrit | Huan Xie proposed openstack/nova: xenapi: support the hotplug of a neutron port https://review.openstack.org/365505 | 01:33 |
| *** ducttape_ has quit IRC | 01:37 | |
| *** bkopilov has quit IRC | 01:41 | |
| *** yufei has joined #openstack-nova | 01:43 | |
| *** ducttape_ has joined #openstack-nova | 01:45 | |
| *** jwcroppe has joined #openstack-nova | 01:47 | |
| openstackgerrit | Zhenyu Zheng proposed openstack/nova: Add query parameters white list for server list/detail https://review.openstack.org/408571 | 01:49 |
| *** bkopilov has joined #openstack-nova | 01:52 | |
| *** raunak has quit IRC | 01:53 | |
| *** yuntongjin has joined #openstack-nova | 01:54 | |
| *** baoli has joined #openstack-nova | 01:55 | |
| *** gongysh has joined #openstack-nova | 02:00 | |
| *** dave-mccowan has joined #openstack-nova | 02:02 | |
| *** baoli has quit IRC | 02:06 | |
| *** ijw has joined #openstack-nova | 02:10 | |
| *** ijw has quit IRC | 02:11 | |
| openstackgerrit | Matt Riedemann proposed openstack/nova: Handle Unauthorized exception in report client's safe_connect() https://review.openstack.org/416810 | 02:13 |
| *** jwcroppe has quit IRC | 02:14 | |
| *** Jinxing has quit IRC | 02:18 | |
| *** wlhc has joined #openstack-nova | 02:18 | |
| *** unicell has quit IRC | 02:20 | |
| *** ducttape_ has quit IRC | 02:21 | |
| *** zz_dimtruck is now known as dimtruck | 02:22 | |
| *** ducttape_ has joined #openstack-nova | 02:23 | |
| *** unicell has joined #openstack-nova | 02:23 | |
| mriedem | dansmith: is save() on an object supposed to noop if no fields have changed? | 02:29 |
| openstackgerrit | Ken'ichi Ohmichi proposed openstack/nova: Add type of reboot on api-ref https://review.openstack.org/416813 | 02:29 |
| *** dave-mccowan has quit IRC | 02:33 | |
| *** trananhkma has quit IRC | 02:34 | |
| *** takashin has joined #openstack-nova | 02:35 | |
| *** ljxiash has joined #openstack-nova | 02:41 | |
| *** unicell has quit IRC | 02:42 | |
| *** baoli has joined #openstack-nova | 02:42 | |
| *** wlhc_ has joined #openstack-nova | 02:44 | |
| openstackgerrit | Takashi NATSUME proposed openstack/nova: Enable cold migration with target host(1/2) https://review.openstack.org/408955 | 02:44 |
| mriedem | dansmith: nvm not the bug i was looking for | 02:45 |
| *** wlhc has quit IRC | 02:47 | |
| *** tiendc has joined #openstack-nova | 02:48 | |
| *** coreywright has quit IRC | 02:54 | |
| *** baoli has quit IRC | 02:55 | |
| *** dave-mccowan has joined #openstack-nova | 02:55 | |
| *** coreywright has joined #openstack-nova | 02:57 | |
| *** baoli has joined #openstack-nova | 02:58 | |
| openstackgerrit | Takashi NATSUME proposed openstack/nova: Enable cold migration with target host(2/2) https://review.openstack.org/408964 | 03:02 |
| openstackgerrit | Takashi NATSUME proposed openstack/nova: api-ref: Add parameters in cold migrate action https://review.openstack.org/410042 | 03:02 |
| openstackgerrit | Takashi NATSUME proposed openstack/nova: Add functional tests for cold migration to same host https://review.openstack.org/414926 | 03:04 |
| *** bkopilov has quit IRC | 03:04 | |
| *** rmk has quit IRC | 03:05 | |
| *** harlowja has quit IRC | 03:05 | |
| *** tonyb has quit IRC | 03:05 | |
| *** rha has quit IRC | 03:05 | |
| *** s1rp_ has quit IRC | 03:05 | |
| *** snikitin has quit IRC | 03:05 | |
| *** kaisers1 has quit IRC | 03:05 | |
| *** jgriffith has quit IRC | 03:05 | |
| *** Oku_OS-away has quit IRC | 03:05 | |
| *** sulo has quit IRC | 03:05 | |
| *** yhvh has quit IRC | 03:05 | |
| *** jose-phillips has joined #openstack-nova | 03:05 | |
| *** s1rp has joined #openstack-nova | 03:05 | |
| *** tonyb has joined #openstack-nova | 03:05 | |
| *** snikitin has joined #openstack-nova | 03:05 | |
| *** Oku_OSz has joined #openstack-nova | 03:05 | |
| *** jgriffith has joined #openstack-nova | 03:06 | |
| *** yhvh has joined #openstack-nova | 03:06 | |
| *** kaisers1 has joined #openstack-nova | 03:06 | |
| *** sulo has joined #openstack-nova | 03:06 | |
| *** rha has joined #openstack-nova | 03:06 | |
| *** jose-phillips has quit IRC | 03:06 | |
| *** jose-phillips has joined #openstack-nova | 03:07 | |
| *** rmk has joined #openstack-nova | 03:07 | |
| *** rmk has quit IRC | 03:07 | |
| *** rmk has joined #openstack-nova | 03:07 | |
| *** jose-phillips has quit IRC | 03:09 | |
| openstackgerrit | Matt Riedemann proposed openstack/nova: Fix TypeError in _update_from_compute_node race https://review.openstack.org/416822 | 03:14 |
| mriedem | JayF: jlvillal: ^ | 03:14 |
| *** thorst_ has quit IRC | 03:14 | |
| *** thorst_ has joined #openstack-nova | 03:15 | |
| *** ljxiash has quit IRC | 03:15 | |
| *** hongbin has joined #openstack-nova | 03:15 | |
| *** ljxiash has joined #openstack-nova | 03:15 | |
| *** mriedem has quit IRC | 03:16 | |
| *** raunak has joined #openstack-nova | 03:17 | |
| *** baoli has quit IRC | 03:18 | |
| *** baoli has joined #openstack-nova | 03:19 | |
| *** thorst_ has quit IRC | 03:19 | |
| *** ljxiash has quit IRC | 03:20 | |
| *** gouthamr has joined #openstack-nova | 03:20 | |
| *** yamahata has quit IRC | 03:22 | |
| *** mdnadeem has joined #openstack-nova | 03:23 | |
| *** sudipto_ has joined #openstack-nova | 03:24 | |
| *** sudipto has joined #openstack-nova | 03:24 | |
| *** dimtruck is now known as zz_dimtruck | 03:24 | |
| *** yuntongjin has quit IRC | 03:24 | |
| *** thorst_ has joined #openstack-nova | 03:25 | |
| *** thorst_ has quit IRC | 03:25 | |
| *** wlhc_ has quit IRC | 03:28 | |
| *** wlhc_ has joined #openstack-nova | 03:28 | |
| *** amotoki has joined #openstack-nova | 03:30 | |
| *** ljxiash has joined #openstack-nova | 03:31 | |
| *** marst has joined #openstack-nova | 03:34 | |
| *** baoli has quit IRC | 03:36 | |
| *** raunak has quit IRC | 03:36 | |
| *** links has joined #openstack-nova | 03:40 | |
| *** mdnadeem has quit IRC | 03:41 | |
| *** raunak has joined #openstack-nova | 03:47 | |
| *** ijw has joined #openstack-nova | 03:53 | |
| *** jwcroppe has joined #openstack-nova | 03:54 | |
| *** dave-mccowan has quit IRC | 03:54 | |
| *** ducttape_ has quit IRC | 03:55 | |
| *** yufei has quit IRC | 03:59 | |
| *** thorst_ has joined #openstack-nova | 04:01 | |
| *** thorst_ has quit IRC | 04:01 | |
| *** tyrola has quit IRC | 04:04 | |
| *** ljxiash has quit IRC | 04:04 | |
| *** tyrola has joined #openstack-nova | 04:04 | |
| *** ljxiash has joined #openstack-nova | 04:04 | |
| *** jwcroppe has quit IRC | 04:05 | |
| *** Jeffrey4l has quit IRC | 04:06 | |
| *** Jeffrey4l has joined #openstack-nova | 04:07 | |
| *** ljxiash has quit IRC | 04:09 | |
| *** sreeram_v has joined #openstack-nova | 04:09 | |
| *** sreeram_v has joined #openstack-nova | 04:09 | |
| *** raunak has quit IRC | 04:11 | |
| *** yingjun has quit IRC | 04:12 | |
| *** haplo37 has quit IRC | 04:12 | |
| *** tyrola has quit IRC | 04:14 | |
| *** tyrola has joined #openstack-nova | 04:14 | |
| *** raunak has joined #openstack-nova | 04:18 | |
| *** raunak has quit IRC | 04:20 | |
| *** haplo37 has joined #openstack-nova | 04:21 | |
| *** yingjun has joined #openstack-nova | 04:22 | |
| *** dave-mccowan has joined #openstack-nova | 04:24 | |
| *** psachin has joined #openstack-nova | 04:29 | |
| *** bkopilov has joined #openstack-nova | 04:30 | |
| *** ratailor has joined #openstack-nova | 04:34 | |
| *** mdnadeem has joined #openstack-nova | 04:34 | |
| *** nicolasbock has joined #openstack-nova | 04:36 | |
| *** tjones1 has joined #openstack-nova | 04:43 | |
| *** gouthamr has quit IRC | 04:43 | |
| *** tjones1 has quit IRC | 04:43 | |
| *** ijw has quit IRC | 04:45 | |
| *** ducttape_ has joined #openstack-nova | 04:46 | |
| *** ijw has joined #openstack-nova | 04:49 | |
| *** ljxiash has joined #openstack-nova | 04:53 | |
| *** jose-phillips has joined #openstack-nova | 04:55 | |
| openstackgerrit | Maciej Szankin proposed openstack/nova: Move quota options to a config group. https://review.openstack.org/346869 | 04:57 |
| *** ljxiash has quit IRC | 04:57 | |
| *** tlian has quit IRC | 05:01 | |
| *** tjones1 has joined #openstack-nova | 05:02 | |
| *** sudipto has quit IRC | 05:02 | |
| *** sudipto_ has quit IRC | 05:02 | |
| *** ljxiash has joined #openstack-nova | 05:06 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/nova: Updated from global requirements https://review.openstack.org/415978 | 05:11 |
| *** yuntongjin has joined #openstack-nova | 05:13 | |
| *** ducttape_ has quit IRC | 05:15 | |
| *** claudiub has quit IRC | 05:15 | |
| *** tjones has quit IRC | 05:16 | |
| *** nandal has joined #openstack-nova | 05:21 | |
| *** yuntongjin has quit IRC | 05:21 | |
| *** jose-phillips has quit IRC | 05:22 | |
| *** jamielennox is now known as jamielennox|away | 05:23 | |
| *** tjones1 has quit IRC | 05:23 | |
| *** hongbin has quit IRC | 05:24 | |
| *** sudipto has joined #openstack-nova | 05:27 | |
| *** sudipto_ has joined #openstack-nova | 05:27 | |
| *** tyrola has quit IRC | 05:27 | |
| *** prateek has joined #openstack-nova | 05:28 | |
| *** tyrola has joined #openstack-nova | 05:28 | |
| *** prateek has quit IRC | 05:30 | |
| *** jamielennox|away is now known as jamielennox | 05:30 | |
| *** tiendc has quit IRC | 05:30 | |
| *** prateek has joined #openstack-nova | 05:30 | |
| *** Jeffrey4l_ has joined #openstack-nova | 05:31 | |
| *** Jeffrey4l has quit IRC | 05:34 | |
| *** tiendc has joined #openstack-nova | 05:36 | |
| *** pbandark has joined #openstack-nova | 05:39 | |
| *** salv-orlando has joined #openstack-nova | 05:41 | |
| *** mdnadeem has quit IRC | 05:41 | |
| *** unicell has joined #openstack-nova | 05:41 | |
| *** jwcroppe has joined #openstack-nova | 05:43 | |
| *** sreeram_v has quit IRC | 05:45 | |
| *** mdnadeem has joined #openstack-nova | 05:45 | |
| *** ayogi has joined #openstack-nova | 05:48 | |
| *** nandal has quit IRC | 05:51 | |
| *** nandal has joined #openstack-nova | 05:52 | |
| *** jwcroppe has quit IRC | 05:54 | |
| *** yuikotakadamori has joined #openstack-nova | 05:54 | |
| *** yuikotakadamori has left #openstack-nova | 05:54 | |
| *** sreeram_v has joined #openstack-nova | 05:55 | |
| *** ljxiash_ has joined #openstack-nova | 05:59 | |
| *** ekuris has joined #openstack-nova | 05:59 | |
| *** sreeram_v has quit IRC | 05:59 | |
| *** sreeram_v has joined #openstack-nova | 06:00 | |
| *** thorst_ has joined #openstack-nova | 06:02 | |
| *** ljxiash has quit IRC | 06:02 | |
| *** sreeram_v has quit IRC | 06:03 | |
| *** sreeram_v has joined #openstack-nova | 06:04 | |
| *** sreeram_v has quit IRC | 06:05 | |
| *** sreeram_v has joined #openstack-nova | 06:06 | |
| *** sreeram_v has quit IRC | 06:06 | |
| *** thorst_ has quit IRC | 06:07 | |
| *** yufei has joined #openstack-nova | 06:08 | |
| *** Guest72087 has quit IRC | 06:09 | |
| *** pbandark has quit IRC | 06:13 | |
| *** pbandark has joined #openstack-nova | 06:14 | |
| *** unicell has quit IRC | 06:14 | |
| *** pbandark has quit IRC | 06:14 | |
| *** unicell has joined #openstack-nova | 06:14 | |
| *** gouthamr has joined #openstack-nova | 06:14 | |
| *** ducttape_ has joined #openstack-nova | 06:15 | |
| *** pbandark has joined #openstack-nova | 06:16 | |
| *** ducttape_ has quit IRC | 06:20 | |
| *** ljxiash_ has quit IRC | 06:22 | |
| *** ljxiash has joined #openstack-nova | 06:22 | |
| *** unicell1 has joined #openstack-nova | 06:27 | |
| *** sreeram_v has joined #openstack-nova | 06:27 | |
| *** thorst_ has joined #openstack-nova | 06:28 | |
| *** unicell has quit IRC | 06:29 | |
| *** sreeram_v has quit IRC | 06:32 | |
| *** thorst_ has quit IRC | 06:33 | |
| *** nkrinner_afk is now known as nkrinner | 06:40 | |
| *** sreeram_v has joined #openstack-nova | 06:43 | |
| *** sridharg has joined #openstack-nova | 06:43 | |
| *** diga has joined #openstack-nova | 06:45 | |
| openstackgerrit | Taku Izumi proposed openstack/nova: virt/ironic: Implement rescue and unrescue https://review.openstack.org/416487 | 06:47 |
| *** _gryf_ has joined #openstack-nova | 06:53 | |
| *** _gryf_ is now known as gryf | 06:53 | |
| *** gryf has quit IRC | 06:53 | |
| *** hoangcx_ has joined #openstack-nova | 06:54 | |
| *** pbandark has quit IRC | 06:56 | |
| *** hoangcx has quit IRC | 06:56 | |
| *** pbandark has joined #openstack-nova | 06:57 | |
| *** pbandark has quit IRC | 06:57 | |
| *** pbandark has joined #openstack-nova | 06:57 | |
| *** pbandark has quit IRC | 06:57 | |
| *** pbandark has joined #openstack-nova | 06:58 | |
| *** moshele has joined #openstack-nova | 07:03 | |
| *** satyar has joined #openstack-nova | 07:05 | |
| *** pcaruana has joined #openstack-nova | 07:05 | |
| *** tesseract has joined #openstack-nova | 07:09 | |
| *** salv-orl_ has joined #openstack-nova | 07:13 | |
| *** salv-orl_ has quit IRC | 07:13 | |
| *** salv-orlando has quit IRC | 07:16 | |
| *** lpetrut has joined #openstack-nova | 07:16 | |
| *** rcernin has joined #openstack-nova | 07:17 | |
| *** yuikotak_ has joined #openstack-nova | 07:24 | |
| *** tbachman has quit IRC | 07:27 | |
| *** armax has joined #openstack-nova | 07:30 | |
| *** adisky_ has joined #openstack-nova | 07:30 | |
| openstackgerrit | Huan Xie proposed openstack/nova: xenapi: support the hotplug of a neutron port https://review.openstack.org/365505 | 07:31 |
| *** yuikotak_ has left #openstack-nova | 07:31 | |
| *** jwcroppe has joined #openstack-nova | 07:33 | |
| *** yamahata has joined #openstack-nova | 07:39 | |
| *** markus_z has joined #openstack-nova | 07:42 | |
| openstackgerrit | Takashi NATSUME proposed openstack/nova: api-ref: Fix parameters and response in os-quota-sets.inc https://review.openstack.org/416461 | 07:44 |
| *** jwcroppe has quit IRC | 07:44 | |
| *** ducttape_ has joined #openstack-nova | 07:46 | |
| *** ekuris has quit IRC | 07:47 | |
| *** openstackgerrit has quit IRC | 07:50 | |
| *** ducttape_ has quit IRC | 07:51 | |
| *** hoangcx has joined #openstack-nova | 07:53 | |
| *** gouthamr has quit IRC | 07:53 | |
| *** gouthamr has joined #openstack-nova | 07:54 | |
| *** hoangcx_ has quit IRC | 07:56 | |
| *** lpetrut has quit IRC | 07:58 | |
| *** ekuris has joined #openstack-nova | 08:02 | |
| gibi | diga: ping | 08:05 |
| *** gouthamr has quit IRC | 08:06 | |
| *** armax has quit IRC | 08:13 | |
| *** ijw has quit IRC | 08:15 | |
| gibi | diga: I'm back from vacation, I work in CET (UTC+1) mostly from UTC 8:00 to UTC 17:00 | 08:17 |
| *** sahid has joined #openstack-nova | 08:19 | |
| *** armax has joined #openstack-nova | 08:22 | |
| *** jwcroppe has joined #openstack-nova | 08:28 | |
| *** thorst_ has joined #openstack-nova | 08:29 | |
| *** sahid has quit IRC | 08:30 | |
| gibi | diga: I susbscribed to the bp placement-notifications | 08:31 |
| *** sahid has joined #openstack-nova | 08:31 | |
| *** thorst_ has quit IRC | 08:34 | |
| *** rha has quit IRC | 08:34 | |
| *** ratailor has quit IRC | 08:35 | |
| *** jwcroppe has quit IRC | 08:37 | |
| *** openstackgerrit has joined #openstack-nova | 08:40 | |
| openstackgerrit | Balazs Gibizer proposed openstack/nova: Transform missing delete notifications https://review.openstack.org/410297 | 08:40 |
| *** rha has joined #openstack-nova | 08:43 | |
| *** rha has joined #openstack-nova | 08:43 | |
| diga | gibi: Hi | 08:49 |
| diga | gibi: jaypipes told me to work with you on nova-placement notifications | 08:50 |
| *** ratailor has joined #openstack-nova | 08:55 | |
| *** wlhc_ has quit IRC | 08:56 | |
| *** zzzeek has quit IRC | 09:00 | |
| *** roeyc has joined #openstack-nova | 09:01 | |
| *** zzzeek has joined #openstack-nova | 09:03 | |
| diga | gibi: you there ? | 09:06 |
| *** mnestratov|2 has joined #openstack-nova | 09:14 | |
| *** armax has quit IRC | 09:14 | |
| *** khushbu_ has joined #openstack-nova | 09:14 | |
| *** mnestratov|2 has quit IRC | 09:15 | |
| *** claudiub has joined #openstack-nova | 09:16 | |
| *** ducttape_ has joined #openstack-nova | 09:17 | |
| khushbu_ | Hi! everyone I am looking forward to make some contribution. I have tried keeping track of VM to do load balancing and stuff also learning from openstack administration fundamentals any advice to work on thing to collobrate with someone on existing things ? | 09:19 |
| *** ducttape_ has quit IRC | 09:21 | |
| *** jwcroppe has joined #openstack-nova | 09:22 | |
| *** lucas-afk is now known as lucasagomes | 09:24 | |
| openstackgerrit | Pawel Koniszewski proposed openstack/nova: live-mig: Add claims and proper resource tracking https://review.openstack.org/244489 | 09:28 |
| openstackgerrit | Pawel Koniszewski proposed openstack/nova: libvirt: pass libvirt reference methods to migration https://review.openstack.org/328997 | 09:28 |
| openstackgerrit | Pawel Koniszewski proposed openstack/nova: libvirt: live-migrate updates NUMA and cpus in the XML https://review.openstack.org/286744 | 09:28 |
| *** romainh has joined #openstack-nova | 09:28 | |
| *** yassine has joined #openstack-nova | 09:29 | |
| *** yassine is now known as Guest42786 | 09:30 | |
| *** openstackgerrit has quit IRC | 09:33 | |
| *** jwcroppe has quit IRC | 09:33 | |
| *** yamahata has quit IRC | 09:34 | |
| *** diga has quit IRC | 09:36 | |
| gibi | diga: I'm here | 09:39 |
| *** nicolasbock has quit IRC | 09:45 | |
| *** derekh has joined #openstack-nova | 09:47 | |
| *** dikonoor has joined #openstack-nova | 09:47 | |
| *** openstackgerrit has joined #openstack-nova | 09:47 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/nova: Updated from global requirements https://review.openstack.org/415978 | 09:47 |
| *** lpetrut has joined #openstack-nova | 09:48 | |
| *** little has quit IRC | 09:48 | |
| *** sahid has quit IRC | 09:50 | |
| *** rmart04 has joined #openstack-nova | 09:52 | |
| *** rmart04 has quit IRC | 09:52 | |
| *** nicolasbock has joined #openstack-nova | 09:52 | |
| *** sahid has joined #openstack-nova | 09:53 | |
| *** mvk has quit IRC | 09:53 | |
| *** amotoki has quit IRC | 09:55 | |
| *** hoangcx has quit IRC | 09:57 | |
| *** cdent has joined #openstack-nova | 10:00 | |
| openstackgerrit | Takashi NATSUME proposed openstack/nova: api-ref: Fix path parameters in os-hypervisors.inc https://review.openstack.org/416911 | 10:00 |
| *** rmart04 has joined #openstack-nova | 10:01 | |
| *** khushbu_ has quit IRC | 10:01 | |
| *** sahid has quit IRC | 10:02 | |
| *** dikonoor has quit IRC | 10:02 | |
| *** sheel has joined #openstack-nova | 10:03 | |
| *** yingjun has quit IRC | 10:05 | |
| *** sreeram_v has quit IRC | 10:09 | |
| *** nicolasbock has quit IRC | 10:10 | |
| *** phuongnh has quit IRC | 10:11 | |
| *** phuongnh has joined #openstack-nova | 10:11 | |
| *** nicolasbock has joined #openstack-nova | 10:12 | |
| *** hieulq has quit IRC | 10:13 | |
| *** openstackstatus has quit IRC | 10:13 | |
| *** openstack has joined #openstack-nova | 10:15 | |
| *** phuongnh has quit IRC | 10:16 | |
| *** sudipto_ has quit IRC | 10:16 | |
| *** sudipto has quit IRC | 10:16 | |
| *** kevinz has quit IRC | 10:17 | |
| *** jwcroppe has joined #openstack-nova | 10:17 | |
| openstackgerrit | Zhenyu Zheng proposed openstack/nova: Add query parameters white list for server list/detail https://review.openstack.org/408571 | 10:17 |
| *** satyar has quit IRC | 10:19 | |
| *** mvk has joined #openstack-nova | 10:25 | |
| *** romainh1 has joined #openstack-nova | 10:26 | |
| *** tiendc has quit IRC | 10:26 | |
| *** jwcroppe has quit IRC | 10:27 | |
| *** romainh has quit IRC | 10:27 | |
| openstackgerrit | Matthew Booth proposed openstack/nova: Port libvirt.test_firewall to Python 3 https://review.openstack.org/403423 | 10:27 |
| openstackgerrit | Matthew Booth proposed openstack/nova: Port libvirt.test_vif to Python 3 https://review.openstack.org/403419 | 10:27 |
| *** dikonoor has joined #openstack-nova | 10:28 | |
| mdbooth | sfinucan: Thanks for the +2s yesterday. Just rebased and fixed test-py3 merge conflicts in ^^^. I've also made one depend on the other so it doesn't happen again. | 10:28 |
| sfinucan | mdbooth: Sweet. I'll take a look again shortly | 10:30 |
| mdbooth | sfinucan: Thanks | 10:30 |
| *** zhs_ has quit IRC | 10:30 | |
| *** thorst_ has joined #openstack-nova | 10:30 | |
| *** zhs_ has joined #openstack-nova | 10:30 | |
| *** rmart04 has quit IRC | 10:31 | |
| *** gongysh has quit IRC | 10:31 | |
| *** pbandark is now known as pbandark`mtg | 10:32 | |
| *** rmart04 has joined #openstack-nova | 10:33 | |
| *** thorst_ has quit IRC | 10:35 | |
| openstackgerrit | Jiri Suchomel proposed openstack/nova: Retry few times when creating the initial connection to libvirtd. https://review.openstack.org/416928 | 10:41 |
| *** ducttape_ has joined #openstack-nova | 10:47 | |
| *** rfolco has joined #openstack-nova | 10:49 | |
| *** cdent has quit IRC | 10:49 | |
| *** salv-orlando has joined #openstack-nova | 10:52 | |
| *** ducttape_ has quit IRC | 10:52 | |
| *** salv-orlando has quit IRC | 10:57 | |
| *** salv-orlando has joined #openstack-nova | 10:57 | |
| *** sudipto has joined #openstack-nova | 11:00 | |
| *** sudipto_ has joined #openstack-nova | 11:01 | |
| *** salv-orlando has quit IRC | 11:02 | |
| *** salv-orlando has joined #openstack-nova | 11:04 | |
| *** tbachman has joined #openstack-nova | 11:04 | |
| *** ratailor has quit IRC | 11:05 | |
| *** armax has joined #openstack-nova | 11:05 | |
| *** dikonoor has quit IRC | 11:08 | |
| *** ekuris has quit IRC | 11:09 | |
| *** yufei has quit IRC | 11:10 | |
| *** yingjun has joined #openstack-nova | 11:13 | |
| *** romainh1 has quit IRC | 11:16 | |
| *** dikonoor has joined #openstack-nova | 11:16 | |
| *** amotoki has joined #openstack-nova | 11:16 | |
| *** diga has joined #openstack-nova | 11:16 | |
| *** sudipto_ has quit IRC | 11:16 | |
| *** sudipto has quit IRC | 11:16 | |
| *** ratailor has joined #openstack-nova | 11:18 | |
| *** salv-orlando has quit IRC | 11:19 | |
| *** salv-orlando has joined #openstack-nova | 11:19 | |
| *** ijw has joined #openstack-nova | 11:20 | |
| *** ekuris has joined #openstack-nova | 11:22 | |
| *** yingjun has quit IRC | 11:24 | |
| *** salv-orlando has quit IRC | 11:24 | |
| *** ijw has quit IRC | 11:26 | |
| *** armax has quit IRC | 11:26 | |
| *** cdent has joined #openstack-nova | 11:29 | |
| *** lpetrut has quit IRC | 11:29 | |
| *** lpetrut has joined #openstack-nova | 11:31 | |
| *** diga has quit IRC | 11:37 | |
| *** tbachman has quit IRC | 11:39 | |
| *** pbandark`mtg has quit IRC | 11:40 | |
| *** rmart04 has quit IRC | 11:40 | |
| *** rmart04 has joined #openstack-nova | 11:41 | |
| *** rmart04 has quit IRC | 11:41 | |
| *** rmart04 has joined #openstack-nova | 11:42 | |
| *** psachin has quit IRC | 11:43 | |
| *** psachin has joined #openstack-nova | 11:44 | |
| openstackgerrit | Rafael Folco proposed openstack/nova: DELETE all inventory for a resource provider https://review.openstack.org/416669 | 11:44 |
| *** eglynn has joined #openstack-nova | 11:44 | |
| *** gongysh has joined #openstack-nova | 11:45 | |
| *** pbandark has joined #openstack-nova | 11:46 | |
| cdent | rfolco: things going okay with ^ ? | 11:49 |
| sfinucan | claudiub: I have that vNUMA patch reviewed again. If you get a chance, could you take a shot at updating this page [1] in openstack-manuals to reflect the new functionality? [1] http://docs.openstack.org/admin-guide/compute-cpu-topologies.html | 11:49 |
| sfinucan | ...assuming any of the HyperV stuff in documented in that guide | 11:50 |
| rfolco | cdent, minor issue with reno, another one with latest microversion test, fixed. waiting test results to see if all good now | 11:51 |
| sfinucan | claudiub: I'd guess you'd want to update the feature matrix too. johnthetubaguy might have some input on what you should do there | 11:51 |
| rfolco | cdent, thanks for your great assistance | 11:51 |
| cdent | you're very welcome, thanks for picking up the work. the more people involved the better. | 11:51 |
| claudiub | sfinucan: hi. thanks for reviewing. :) there isn't any hyper-v documentation in the docs about vnuma, since it wasn't merged yet. will start working on it as soon as i can. | 11:52 |
| *** sahid has joined #openstack-nova | 11:53 | |
| sfinucan | claudiub: Sure thing. Might be worth asking sahid to take a look at that too | 11:54 |
| *** derekh has quit IRC | 11:55 | |
| *** tdurakov has quit IRC | 11:55 | |
| *** grumpycatt has quit IRC | 11:55 | |
| *** john51 has quit IRC | 11:55 | |
| *** jgrassle1 has quit IRC | 11:55 | |
| *** tlbr_ has quit IRC | 11:55 | |
| *** rdo has quit IRC | 11:55 | |
| *** mtreinish has quit IRC | 11:55 | |
| *** sileht has quit IRC | 11:55 | |
| *** esmiurium has quit IRC | 11:55 | |
| *** afazekas has quit IRC | 11:55 | |
| *** blaisebool has quit IRC | 11:55 | |
| *** bauzas has quit IRC | 11:55 | |
| *** emerson has quit IRC | 11:55 | |
| *** dansmith has quit IRC | 11:55 | |
| *** kaisers has quit IRC | 11:55 | |
| *** htruta has quit IRC | 11:55 | |
| *** haukebruno has quit IRC | 11:55 | |
| *** devananda has quit IRC | 11:55 | |
| *** smcginnis has quit IRC | 11:55 | |
| *** dosaboy has quit IRC | 11:55 | |
| *** Anticimex has quit IRC | 11:55 | |
| *** jgrassler has joined #openstack-nova | 11:55 | |
| *** htruta has joined #openstack-nova | 11:55 | |
| *** afazekas has joined #openstack-nova | 11:55 | |
| *** haukebruno has joined #openstack-nova | 11:55 | |
| *** dosaboy has joined #openstack-nova | 11:55 | |
| *** rdo has joined #openstack-nova | 11:55 | |
| *** smcginnis has joined #openstack-nova | 11:55 | |
| *** john51 has joined #openstack-nova | 11:56 | |
| *** bauzas has joined #openstack-nova | 11:56 | |
| *** blaisebool has joined #openstack-nova | 11:56 | |
| *** kaisers has joined #openstack-nova | 11:56 | |
| *** devananda has joined #openstack-nova | 11:56 | |
| *** dansmith has joined #openstack-nova | 11:56 | |
| *** derekh has joined #openstack-nova | 11:56 | |
| *** mtreinish has joined #openstack-nova | 11:56 | |
| *** esmiurium has joined #openstack-nova | 11:56 | |
| *** dansmith is now known as Guest41100 | 11:56 | |
| *** armax has joined #openstack-nova | 11:56 | |
| *** sileht has joined #openstack-nova | 11:56 | |
| *** blaisebool is now known as Guest58715 | 11:56 | |
| *** tdurakov has joined #openstack-nova | 11:56 | |
| *** tlbr has joined #openstack-nova | 11:57 | |
| openstackgerrit | Stephen Finucane proposed openstack/nova: console: introduce basic framework for security proxying https://review.openstack.org/345396 | 11:57 |
| openstackgerrit | Stephen Finucane proposed openstack/nova: console: introduce framework for RFB authentication https://review.openstack.org/345397 | 11:57 |
| openstackgerrit | Stephen Finucane proposed openstack/nova: console: introduce the VeNCrypt RFB authentication scheme https://review.openstack.org/345398 | 11:57 |
| openstackgerrit | Stephen Finucane proposed openstack/nova: console: provide an RFB security proxy implementation https://review.openstack.org/345399 | 11:57 |
| *** smatzek has joined #openstack-nova | 11:58 | |
| *** BobBall has quit IRC | 11:58 | |
| *** serverascode has quit IRC | 11:59 | |
| *** diga has joined #openstack-nova | 12:00 | |
| *** serverascode has joined #openstack-nova | 12:01 | |
| frickler | wznoinsk: Intel NFV CI looks unstable, see e.g. https://review.openstack.org/416822 | 12:02 |
| *** pbandark has quit IRC | 12:02 | |
| *** yufei has joined #openstack-nova | 12:03 | |
| *** tiendc has joined #openstack-nova | 12:03 | |
| *** JoseMello has joined #openstack-nova | 12:05 | |
| *** lucasagomes is now known as lucas-hungry | 12:05 | |
| *** kevinz has joined #openstack-nova | 12:05 | |
| *** jwcroppe has joined #openstack-nova | 12:07 | |
| gibi | diga: I'm still here | 12:11 |
| *** mdnadeem has quit IRC | 12:12 | |
| *** links has quit IRC | 12:13 | |
| *** rtheis has joined #openstack-nova | 12:14 | |
| *** gongysh has quit IRC | 12:14 | |
| *** tuanluong_ has quit IRC | 12:14 | |
| *** tuanluong has quit IRC | 12:14 | |
| *** haplo37 has quit IRC | 12:14 | |
| *** jwcroppe has quit IRC | 12:15 | |
| *** voelzmo has joined #openstack-nova | 12:16 | |
| openstackgerrit | Balazs Gibizer proposed openstack/nova: add availability_zone field in service.update https://review.openstack.org/415857 | 12:17 |
| sfinucan | johnthetubaguy: Floating IP quotas only apply when using nova-network, right? | 12:18 |
| *** romainh has joined #openstack-nova | 12:18 | |
| *** ducttape_ has joined #openstack-nova | 12:18 | |
| johnthetubaguy | sfinucan: I believe so | 12:19 |
| johnthetubaguy | sfinucan: there might be a config for that... | 12:19 |
| sfinucan | johnthetubaguy: There is, as part of https://review.openstack.org/#/c/346869/10 | 12:19 |
| sfinucan | which unfortunately for poor, long-suffering macsz got in a merge conflict | 12:20 |
| *** bkopilov has quit IRC | 12:20 | |
| johnthetubaguy | sfinucan: honestly, I recon we could move that one, its can't get removed till will kill cells v1 | 12:21 |
| *** diga_ has joined #openstack-nova | 12:21 | |
| diga_ | gibi: Hi | 12:21 |
| diga_ | sorry got disconnected | 12:21 |
| *** bvanhav_ has joined #openstack-nova | 12:21 | |
| *** bvanhav_ is now known as bvanhav | 12:21 | |
| diga_ | gibi: you still there ? | 12:21 |
| *** ijw has joined #openstack-nova | 12:21 | |
| sfinucan | True. What about the 'driver' one? | 12:21 |
| *** hwoarang has quit IRC | 12:22 | |
| *** ducttape_ has quit IRC | 12:22 | |
| *** hwoarang has joined #openstack-nova | 12:22 | |
| johnthetubaguy | sfinucan: well I don't quite get why we deprecated that I suppose, I guess we will kill the no op driver, but I suspect folks use that | 12:22 |
| gibi | diga_: yes. how can I help you? | 12:22 |
| diga_ | gibi: jaypipes told me to work with you on nova-placement-notifications | 12:23 |
| johnthetubaguy | sfinucan: I thought it was just going to move to stevedore, in which case, the setting remains, its just we do a fallback on the values used | 12:23 |
| *** diga has quit IRC | 12:23 | |
| diga_ | gibi: just want to know, what is remaining, what should take up on first priority | 12:24 |
| sfinucan | johnthetubaguy: Looks like the work of dansmith (https://github.com/openstack/nova/commit/43063888) | 12:24 |
| gibi | diga_: I think nothing was done on the notification side in the placement work | 12:24 |
| johnthetubaguy | yeah, was just digging into that | 12:24 |
| *** haplo37 has joined #openstack-nova | 12:24 | |
| gibi | diga_: one I saw a WIP patch from cdent | 12:24 |
| gibi | diga_: let me dig up a link | 12:25 |
| diga_ | gibi: ok | 12:25 |
| johnthetubaguy | sfinucan: I mean if you set everything to -1, you get the same behaviour as no-op I think | 12:25 |
| openstackgerrit | Stephen Finucane proposed openstack/nova: conf: Deprecate yet another nova-net option https://review.openstack.org/416959 | 12:25 |
| cdent | diga_, gibi : https://review.openstack.org/#/c/308503/ | 12:25 |
| cdent | there's very little there | 12:25 |
| johnthetubaguy | sfinucan: I totally approved the change, just looked odd at a second glance | 12:25 |
| *** links has joined #openstack-nova | 12:25 | |
| diga_ | cdent: yes, this is one you posted in the meeting | 12:26 |
| cdent | diga_: feel free to take that over, or ignore it, whichever seems best | 12:26 |
| gibi | cdent: thanks, that one :) | 12:26 |
| sfinucan | johnthetubaguy: Ah, so we only have two "plugins" and one of them can do the work of the other? Definitely worth killing then | 12:26 |
| *** mdnadeem has joined #openstack-nova | 12:26 | |
| johnthetubaguy | sfinucan: tempting to not move the driver one, given its needed rework | 12:26 |
| diga_ | cdent: sure | 12:26 |
| johnthetubaguy | sfinucan: well, one is much slower at doing nothing, I presume, but I don't know so | 12:26 |
| diga_ | cdent: just want to know how to run placement-api seperately | 12:26 |
| gibi | diga_: also jaypipes drafted the blueprint with some TODO list https://blueprints.launchpad.net/nova/+spec/placement-notifications | 12:26 |
| johnthetubaguy | sfinucan: I suspect its only a test hang over thing | 12:26 |
| diga_ | gibi: yes, he assigned to me | 12:27 |
| sfinucan | johnthetubaguy: I can't see why someone would want to disable quotas entirely, tbh | 12:27 |
| sfinucan | Yes, exactly | 12:27 |
| *** ijw has quit IRC | 12:27 | |
| gibi | diga_: based on that bp, the next thing is a small spec describing what notifications the placement service wants to emit | 12:27 |
| *** mat128 is now known as mat128|afk | 12:28 | |
| diga_ | gibi: okay | 12:28 |
| gibi | diga_: do you need a link for the spec template? | 12:28 |
| cdent | diga_: What do you mean by "separately"? It is automatically enabled in devstack if nova is enabled and runs under mod_wsgi. Do you want to run it without devstack? | 12:28 |
| *** romainh1 has joined #openstack-nova | 12:28 | |
| diga_ | gibi: yeah, will be helpful | 12:28 |
| sfinucan | Hmm, OK I hold off on a reworked version that doesn't move driver | 12:28 |
| *** hwoarang_ has joined #openstack-nova | 12:28 | |
| diga_ | cdent: I just see n-api in devstack | 12:28 |
| diga_ | cdent: is that all ? | 12:28 |
| sfinucan | I also have this to deprecate that floating IPs opt so perhaps we shouldn't move that too (but I don't care so much about this one) https://review.openstack.org/#/c/416959/ | 12:29 |
| *** romainh has quit IRC | 12:29 | |
| gibi | diga_: here is the template you need to fill https://github.com/openstack/nova-specs/blob/master/specs/pike-template.rst | 12:29 |
| diga_ | gibi: okay | 12:29 |
| cdent | diga: the service is called placement-api and it will be enabled unless it is explicitly turned off | 12:29 |
| gibi | diga_: then push the spec up for review in the nova-specs repo | 12:29 |
| diga_ | gibi: will go through it first | 12:30 |
| diga_ | okay | 12:30 |
| diga_ | cdent: okay | 12:30 |
| *** nandal has quit IRC | 12:30 | |
| diga_ | gibi: will submit the spec soon | 12:31 |
| *** hwoarang has quit IRC | 12:31 | |
| *** ljxiash has quit IRC | 12:31 | |
| diga_ | gibi: I think it is necessary to submit the spec spec & then start the implementation | 12:32 |
| diga_ | ? | 12:32 |
| sfinucan | mdbooth: +2'd | 12:32 |
| *** ljxiash has joined #openstack-nova | 12:32 | |
| gibi | diga_: you can start the implementation as early as you would like but you can only expect code review on your implementation after the related spec is approved | 12:33 |
| diga_ | gibi: ohh..kk | 12:33 |
| diga_ | gibi: got it, will start on spec first then | 12:33 |
| gibi | diga_: you don't have to rush as nova folks are pretty busy finishing Ocata these days | 12:33 |
| *** tiendc has quit IRC | 12:34 | |
| gibi | diga_: cool | 12:34 |
| diga_ | gibi: yep | 12:34 |
| *** ljxiash has quit IRC | 12:36 | |
| openstackgerrit | Stephen Finucane proposed openstack/os-vif: vif_plug_ovs: Always set MTU when plugging devices https://review.openstack.org/408779 | 12:37 |
| *** catintheroof has quit IRC | 12:38 | |
| sfinucan | johnthetubaguy: Ah, mriedem already pipped us to the mark https://review.openstack.org/#/c/410996/ | 12:38 |
| gibi | diga_: here are two example spec about adding new notifications https://review.openstack.org/#/c/321336/ and https://review.openstack.org/#/c/340715/ | 12:39 |
| diga_ | gibi: Thanks for sharing this, this is very helpful :) | 12:40 |
| diga_ | gibi: will go through it, add it accordingly | 12:41 |
| diga_ | gibi: brb | 12:41 |
| *** mat128|afk is now known as mat128 | 12:43 | |
| *** catintheroof has joined #openstack-nova | 12:44 | |
| *** takashin has left #openstack-nova | 12:45 | |
| openstackgerrit | Zhenyu Zheng proposed openstack/nova: Add query parameters white list for server list/detail https://review.openstack.org/408571 | 12:46 |
| *** thorst_ has joined #openstack-nova | 12:46 | |
| *** sreeram_v has joined #openstack-nova | 12:47 | |
| *** psachin has quit IRC | 12:48 | |
| *** ratailor has quit IRC | 12:50 | |
| *** sdague has joined #openstack-nova | 12:51 | |
| *** sdague_ has quit IRC | 12:54 | |
| *** ljxiash has joined #openstack-nova | 12:54 | |
| *** armax has quit IRC | 12:54 | |
| *** psachin has joined #openstack-nova | 12:54 | |
| *** links has quit IRC | 12:55 | |
| *** sdague_ has joined #openstack-nova | 12:56 | |
| *** sdague_ has quit IRC | 12:56 | |
| *** catintheroof has quit IRC | 12:57 | |
| *** cdent has quit IRC | 12:58 | |
| *** pradk has joined #openstack-nova | 12:58 | |
| *** catintheroof has joined #openstack-nova | 12:58 | |
| *** ljxiash has quit IRC | 12:59 | |
| *** catintheroof has quit IRC | 13:02 | |
| *** catintheroof has joined #openstack-nova | 13:04 | |
| *** sudipto_ has joined #openstack-nova | 13:04 | |
| *** sudipto has joined #openstack-nova | 13:04 | |
| *** edmondsw has joined #openstack-nova | 13:05 | |
| *** cdent has joined #openstack-nova | 13:06 | |
| openstackgerrit | Chris Dent proposed openstack/nova: [WIP] Structure for simply managing placement-api-ref https://review.openstack.org/409340 | 13:07 |
| *** tbachman has joined #openstack-nova | 13:07 | |
| *** links has joined #openstack-nova | 13:07 | |
| cdent | sdague: if you get a chance to look at that ^ to sanity check that would be awesome. It's not done, but I want to make sure I'm not heading down a bad path. You'll also likely have some insight on how to make the testing it tries to do happen in the right place. | 13:08 |
| *** voelzmo has quit IRC | 13:09 | |
| *** ducttape_ has joined #openstack-nova | 13:09 | |
| *** catintheroof has quit IRC | 13:11 | |
| sdague | cdent: looking | 13:12 |
| cdent | awesome, thank | 13:13 |
| cdent | s | 13:13 |
| johnthetubaguy | sfinucan: ah, good spot | 13:16 |
| sdague | cdent: yeh, that catch looks generally good, though the gate docs target actually doesn't run tox -e docs, so that's not going to fail in the gate when it's failing locally | 13:19 |
| cdent | yeah | 13:19 |
| cdent | (mentioned that in the commit message) | 13:19 |
| openstackgerrit | Alex Szarka proposed openstack/nova: Transform instance-exists notification https://review.openstack.org/403660 | 13:19 |
| cdent | it not failing is fine for now since there aren't any docs, but it would be nice to flip a switch at the bottom of the stack after all existing routes are docced | 13:20 |
| cdent | s/bottom/top/ <- dan's ordering | 13:20 |
| *** prateek has quit IRC | 13:21 | |
| johnthetubaguy | sfinucan: I think we just move them all | 13:22 |
| *** pradk has quit IRC | 13:23 | |
| *** ijw has joined #openstack-nova | 13:23 | |
| *** pradk has joined #openstack-nova | 13:24 | |
| *** Anticimex has joined #openstack-nova | 13:25 | |
| *** tpatzig_ has joined #openstack-nova | 13:25 | |
| *** carthaca_ has joined #openstack-nova | 13:25 | |
| *** dgonzalez_ has joined #openstack-nova | 13:25 | |
| *** david_1 has joined #openstack-nova | 13:25 | |
| *** ayogi has quit IRC | 13:25 | |
| *** carthaca_ has quit IRC | 13:27 | |
| *** tpatzig_ has quit IRC | 13:27 | |
| *** dgonzalez_ has quit IRC | 13:27 | |
| *** david_1 has quit IRC | 13:27 | |
| *** lucas-hungry is now known as lucasagomes | 13:27 | |
| *** ijw has quit IRC | 13:28 | |
| sdague | cdent: well, it causes the local tox rule to fail | 13:30 |
| cdent | sdague: yeah, what I mean is: it would be nice to make the lack of docs to cause a docs failure (somehow), after all the existing routes are documented | 13:31 |
| *** jheroux has joined #openstack-nova | 13:33 | |
| *** pradk has quit IRC | 13:33 | |
| *** liusheng has quit IRC | 13:34 | |
| *** mdrabe has joined #openstack-nova | 13:36 | |
| mdbooth | sfinucan: Thanks. | 13:38 |
| *** sreeram_v has quit IRC | 13:39 | |
| openstackgerrit | Gleb Stepanov proposed openstack/python-novaclient: Add evacuate host method https://review.openstack.org/349179 | 13:40 |
| *** cdent has quit IRC | 13:41 | |
| sfinucan | johnthetubaguy: I agree | 13:41 |
| openstackgerrit | Alex Szarka proposed openstack/nova: Transform instance.rebuild notification https://review.openstack.org/384922 | 13:43 |
| *** abalutoiu has joined #openstack-nova | 13:44 | |
| *** catintheroof has joined #openstack-nova | 13:47 | |
| *** ducttape_ has quit IRC | 13:48 | |
| *** liverpooler has quit IRC | 13:51 | |
| *** baoli has joined #openstack-nova | 13:51 | |
| *** jamesdenton has joined #openstack-nova | 13:52 | |
| *** cdelatte has joined #openstack-nova | 13:53 | |
| *** salv-orlando has joined #openstack-nova | 13:56 | |
| *** jwcroppe has joined #openstack-nova | 13:56 | |
| *** marst has quit IRC | 13:57 | |
| wznoinsk | frickler, we're on it, thanks | 14:00 |
| *** bvanhav has quit IRC | 14:00 | |
| *** rfolco has quit IRC | 14:01 | |
| *** links has quit IRC | 14:02 | |
| *** tblakes has joined #openstack-nova | 14:03 | |
| *** bvanhav has joined #openstack-nova | 14:04 | |
| *** marst has joined #openstack-nova | 14:05 | |
| *** jwcroppe has quit IRC | 14:08 | |
| *** marst has quit IRC | 14:10 | |
| *** bkopilov has joined #openstack-nova | 14:10 | |
| johnthetubaguy | cdent: I got a bit confused with a bit of gabbi, can you check I am interpreting it correctly please: https://review.openstack.org/#/c/407742 | 14:10 |
| johnthetubaguy | ah, oops, he is not here | 14:10 |
| *** efried has joined #openstack-nova | 14:10 | |
| *** marst has joined #openstack-nova | 14:11 | |
| edleafe | johnthetubaguy: I'm kinda fuzzy on that part, too | 14:14 |
| *** pkoniszewski has quit IRC | 14:15 | |
| *** pkoniszewski has joined #openstack-nova | 14:15 | |
| *** ducttape_ has joined #openstack-nova | 14:16 | |
| *** baoli has quit IRC | 14:17 | |
| *** baoli has joined #openstack-nova | 14:17 | |
| *** hoangcx has joined #openstack-nova | 14:17 | |
| *** cjvolzka has joined #openstack-nova | 14:18 | |
| *** xyang1 has joined #openstack-nova | 14:18 | |
| *** tlian has joined #openstack-nova | 14:21 | |
| *** Jeffrey4l_ has quit IRC | 14:21 | |
| *** voelzmo has joined #openstack-nova | 14:22 | |
| *** cdent has joined #openstack-nova | 14:23 | |
| openstackgerrit | Gleb Stepanov proposed openstack/nova: Add _rebuild_instance helper method https://review.openstack.org/376004 | 14:23 |
| *** voelzmo has quit IRC | 14:23 | |
| *** smatzek has quit IRC | 14:24 | |
| *** tbachman has quit IRC | 14:24 | |
| *** ijw has joined #openstack-nova | 14:24 | |
| *** voelzmo has joined #openstack-nova | 14:25 | |
| *** tbachman has joined #openstack-nova | 14:28 | |
| *** cjvolzka_ has joined #openstack-nova | 14:29 | |
| *** ijw has quit IRC | 14:30 | |
| *** raunak has joined #openstack-nova | 14:33 | |
| *** Jeffrey4l_ has joined #openstack-nova | 14:35 | |
| openstackgerrit | Alex Szarka proposed openstack/nova: Transform instance-exists notification https://review.openstack.org/403660 | 14:36 |
| *** tblakes has quit IRC | 14:36 | |
| *** eharney has joined #openstack-nova | 14:37 | |
| *** mlavalle has joined #openstack-nova | 14:37 | |
| macsz | sfinucan: my gerrit weltschmerz-ism precedes me | 14:39 |
| *** liverpooler has joined #openstack-nova | 14:39 | |
| *** efried has quit IRC | 14:41 | |
| *** lamt has joined #openstack-nova | 14:42 | |
| *** efried_ has joined #openstack-nova | 14:43 | |
| *** ducttape_ has quit IRC | 14:43 | |
| *** mriedem has joined #openstack-nova | 14:45 | |
| openstackgerrit | sahid proposed openstack/nova: libvirt: fix uncaugth exception during soft reboot https://review.openstack.org/376350 | 14:46 |
| openstackgerrit | sahid proposed openstack/nova: libvirt: make coherent logs when reboot success https://review.openstack.org/416994 | 14:46 |
| openstackgerrit | sahid proposed openstack/nova: libvirt: rework _destroy method https://review.openstack.org/416995 | 14:46 |
| *** ekuris has quit IRC | 14:49 | |
| mriedem | wznoinsk: intel nfv ci having some issues today? | 14:50 |
| *** thorst_ is now known as thorst_afk | 14:50 | |
| *** salv-orlando has quit IRC | 14:50 | |
| *** hongbin has joined #openstack-nova | 14:51 | |
| *** jwcroppe has joined #openstack-nova | 14:51 | |
| *** efried_ has quit IRC | 14:52 | |
| openstackgerrit | Jeremy Liu proposed openstack/nova: [2/3]Replace six.iteritems() with .items() https://review.openstack.org/413470 | 14:52 |
| *** efried has joined #openstack-nova | 14:52 | |
| *** Jeffrey4l_ has quit IRC | 14:52 | |
| openstackgerrit | Marco Voelz proposed openstack/nova-specs: Add port deletion on termination of server https://review.openstack.org/416997 | 14:52 |
| *** satyar has joined #openstack-nova | 14:53 | |
| voelzmo | @johnthetubaguy remember our conversation about pre-created ports being deleted on VM termination? proposed that to nova-specs for pike: ^^ | 14:53 |
| *** tblakes has joined #openstack-nova | 14:55 | |
| *** dikonoor has quit IRC | 14:56 | |
| *** mdnadeem has quit IRC | 14:57 | |
| *** yingjun has joined #openstack-nova | 14:58 | |
| *** tblakes has quit IRC | 15:00 | |
| *** READ10 has joined #openstack-nova | 15:00 | |
| mriedem | voelzmo: just left some comments | 15:01 |
| mriedem | personally i'd consider that super low priority since if you can create the port outside of nova you should be able to delete it too | 15:02 |
| *** jwcroppe has quit IRC | 15:03 | |
| *** cleong has joined #openstack-nova | 15:03 | |
| mriedem | the delete_on_termination stuff with volumes has gotten us into problems with cinder state management too | 15:03 |
| cdent | sdague: having two different api-ref's in the nova tree makes for some ick with trying to reuse existing api-ref jobs and job groups. Would you suggest making a standalone placement-api-ref job and annotating it appropriately? Or something else? | 15:03 |
| mriedem | where we can't delete the server because nova can't delete the volume | 15:03 |
| mriedem | cdent: i'd separate them | 15:04 |
| mriedem | it's just different subdirectories and tox targets | 15:04 |
| mriedem | the project-config stuff should be fairly simple too | 15:04 |
| cdent | mriedem: I was referring to the fact that there is a lot of automation already in place for {name}-api-ref that looks for directories of certain names, which falls flat in the face of placement-api-ref, thus the need for newer/separate job | 15:05 |
| *** Jeffrey4l_ has joined #openstack-nova | 15:05 | |
| *** psachin has quit IRC | 15:06 | |
| mriedem | automation in nova or in project-config? | 15:07 |
| sfinucan | sahid: Had a gawk at https://review.openstack.org/#/c/409261 - one comment left | 15:07 |
| cdent | mriedem: project-config | 15:07 |
| cdent | nothing that can't be overcome relatively easily | 15:07 |
| sfinucan | johnthetubaguy: I just re +2'd that patch we were discussing earlier, if you wanted to do the same https://review.openstack.org/#/c/346869/ | 15:08 |
| *** raunak has quit IRC | 15:10 | |
| *** sapcc-bot has quit IRC | 15:12 | |
| *** sapcc-bot has joined #openstack-nova | 15:13 | |
| *** prateek has joined #openstack-nova | 15:13 | |
| *** ducttape_ has joined #openstack-nova | 15:14 | |
| *** d34dh0r53 has quit IRC | 15:14 | |
| sahid | sfinucan: ok thanks i will have a look shortly | 15:14 |
| *** d34dh0r53 has joined #openstack-nova | 15:14 | |
| Guest41100 | mriedem: I reordered my set to put these two cleanups in front of the next patch I'm working on so we can get them out of the way: https://review.openstack.org/#/c/410471/6 | 15:15 |
| *** Guest41100 is now known as dansmith | 15:15 | |
| yingjun | The flavor versioned notifications patches are all ready, the first patch got a +2 from mriedem, it would be much appreciated if any other core could take a look at that :) | 15:15 |
| yingjun | link: https://review.openstack.org/#/q/topic:bp/flavor-notifications+status:open | 15:15 |
| mriedem | sfinucan: sahid: would be nice to use a namedtuple in https://review.openstack.org/#/c/409260/ rather than a naked old tuple | 15:15 |
| mriedem | mapping to tuples always grosses me out | 15:15 |
| *** lamt has quit IRC | 15:15 | |
| *** eandersson has quit IRC | 15:15 | |
| *** dansmith is now known as Guest57355 | 15:15 | |
| mriedem | Guest57355: ok :) | 15:15 |
| Guest57355 | eff | 15:16 |
| sfinucan | Yeah, I thought a dict but then thought it might be a little...heavy | 15:16 |
| mriedem | i think https://review.openstack.org/#/c/416585/ is also having issues still | 15:16 |
| sfinucan | namedtuple works | 15:16 |
| *** lamt has joined #openstack-nova | 15:16 | |
| *** Guest57355 is now known as dansmith | 15:16 | |
| *** ljxiash has joined #openstack-nova | 15:16 | |
| *** kevinz has quit IRC | 15:17 | |
| dansmith | mriedem: yeah I just pushed another attempt | 15:17 |
| *** diga_ has quit IRC | 15:18 | |
| sdague | dansmith: you want to do this db create reordering - https://review.openstack.org/#/c/384916/ ? | 15:18 |
| dansmith | sdague: yeah, probably so | 15:20 |
| sdague | ok, want to throw a +1 on that for posterity? | 15:20 |
| sdague | Then I'll +2 it | 15:20 |
| dansmith | sure | 15:21 |
| dansmith | sdague: thanks | 15:21 |
| voelzmo | @mriedem thanks for the comments. I understand that from a nova perspective it is undesirable that you cannot delete the server when cinder cannot delete a volume, or that it ends up in some intermediate state. But that's the tragedy of distributed systems, right? | 15:21 |
| mriedem | dansmith: sdague: why do i feel like we already went through this ordering change once already | 15:21 |
| dansmith | mriedem: I had the same feeling, but I think it must have been another detail.. just creating the database shouldn't be a problem | 15:22 |
| voelzmo | @mriedem So that means, if – for some reasons – you would need to clean up manually afterwards, you wouldn't be worse off than right now, correct? | 15:22 |
| voelzmo | whereas, when everything goes right, you would be better of, as things are cleaned up automatically | 15:22 |
| *** jwcroppe has joined #openstack-nova | 15:22 | |
| *** sridharg has quit IRC | 15:22 | |
| voelzmo | so imho, this would still be an improvement | 15:22 |
| mriedem | voelzmo: well http://docs.openstack.org/developer/nova/project_scope.html?highlight=proxy#api-scope | 15:22 |
| *** smatzek has joined #openstack-nova | 15:22 | |
| mriedem | voelzmo: unless we totally screw it up and you can't delete one resource or the other because of state management bugs | 15:23 |
| *** voelzmo has quit IRC | 15:23 | |
| mriedem | doh | 15:23 |
| sdague | I think there was a proposal to move it around before, which I objected to, as I thought the tools should be resilient | 15:23 |
| sdague | the tools are now resilient | 15:23 |
| *** voelzmo has joined #openstack-nova | 15:23 | |
| sdague | the reorder ends up removing a warning | 15:23 |
| *** zz_dimtruck is now known as dimtruck | 15:23 | |
| voelzmo | @mriedem are you arguing this change would make nova a proxy? | 15:23 |
| mriedem | sdague: that might be, i know we've made some improvements in the nova-manage CLIs since newton | 15:24 |
| mriedem | voelzmo: well, it adds more orchestration logic to nova | 15:24 |
| mriedem | in some already gorpy code in the neutronv2 api in nova | 15:24 |
| *** prateek has quit IRC | 15:24 | |
| mriedem | this is probably less of a problem for ports because they don't have the same state issues tied to servers as volumes do (i think) | 15:24 |
| mriedem | so nova can just delete them w/o issue | 15:25 |
| voelzmo | right | 15:25 |
| mriedem | however, i still see it as super low priority since if you can create the port and provide it on server boot, then i don't see what the issue is with removing it after server delete too | 15:25 |
| *** ijw has joined #openstack-nova | 15:26 | |
| mriedem | for the very basic flows, nova creates a volume or a port for you and cleans those up afterward, we want users providing ports and volumes at server create time for more advanced cases | 15:26 |
| mriedem | and to not get involved in cleaning those up (or not, depending on what the user specified) | 15:26 |
| voelzmo | @mriedem so you're arguing, we should also remove the deletion of the boot volume from nova? | 15:26 |
| voelzmo | and users should do that themselves? | 15:27 |
| mriedem | voelzmo: not directly no...but it's not a bad idea | 15:27 |
| mriedem | honestly the major bug around delete_on_termination with volumes is out of my head right now, but there is a thread about it in the ML somewhere i could find if needed, and a bug and several patches related to problems with it | 15:28 |
| mriedem | i'm sure hemna and scottda remember details there too | 15:28 |
| *** rfolco has joined #openstack-nova | 15:29 | |
| scottda | mriedem: I'd have to think on that one. But I do recall we had issues.... | 15:29 |
| smcginnis | It was the whole discussion that nova is not an orchestration layer, it's just another component of the cloud, so why should it coordinate all actions. IIRC. | 15:29 |
| *** Guest42786 has quit IRC | 15:29 | |
| voelzmo | @mriedem Not sure if deleting a thing and therefore also deleting things associated with it is already orchestration. But that might be nitpicking on the definition of words. | 15:30 |
| *** prateek has joined #openstack-nova | 15:30 | |
| *** yassine has joined #openstack-nova | 15:30 | |
| *** yassine is now known as Guest50714 | 15:31 | |
| *** ijw has quit IRC | 15:31 | |
| voelzmo | The case I'm making is: I have to use pre-created ports to know their mac addresses up front, although I don't care about ports as separate entities once the VM is gone | 15:31 |
| scottda | It might be that it's bad to automatically delete a persistent volume when deleting the VM that boots from it....user might want more control. I'll have to dig back for details... | 15:31 |
| voelzmo | same as with boot volumes, thus the analogy | 15:31 |
| smcginnis | If "deleting a thing" where the thing is from somewhere else, that's orchestration. | 15:31 |
| voelzmo | and I'm not arguing to take control away from the user, I'm adding a new case in which you can give up that control if you like | 15:31 |
| *** prateek has quit IRC | 15:31 | |
| voelzmo | all existing cases of "more control" still exist and remain even the default | 15:32 |
| *** prateek has joined #openstack-nova | 15:32 | |
| *** catintheroof has quit IRC | 15:32 | |
| voelzmo | What's even worse: The thing we're arguing about used to work like I'm proposing until the "bugfix" in Kilo | 15:32 |
| mriedem | scottda: yes that's my assumption on the original use case for delete_on_termination with volumes - nova creates the volume, but you want it leftover after the server is deleted | 15:32 |
| mriedem | voelzmo: well before kilo we had the other bug that nova would delete pre-created neutron ports when the server was deleted | 15:33 |
| mriedem | which was also a nasty bug that pissed everyone off | 15:33 |
| voelzmo | I can also create the volume upfront and provide it to boot_from_volume | 15:33 |
| voelzmo | and still have nova delete it on VM deletion | 15:33 |
| voelzmo | @mriedem that's the bug I'm talking about :) | 15:33 |
| mriedem | voelzmo: i know | 15:33 |
| *** salv-orlando has joined #openstack-nova | 15:34 | |
| *** jaypipes has joined #openstack-nova | 15:35 | |
| *** Jeffrey4l_ has quit IRC | 15:36 | |
| mriedem | voelzmo: to be clear, i'm not totally against the idea, i'm just hesitant to add more stuff like this into nova | 15:36 |
| mriedem | anyway, we're not close to pike specs anyway yet, so it'll simmer for awhile | 15:37 |
| mriedem | either way, welcome :) | 15:37 |
| *** ducttape_ has quit IRC | 15:37 | |
| voelzmo | @mriedem understood, thanks for the comments. I thought it will most likely not make it into ocata, so I just submitted it directly to pike | 15:37 |
| sahid | sfinucan: mriedem ok no problem will update the code based on your comments | 15:37 |
| sahid | thanks for the reviews | 15:37 |
| *** yingjun has quit IRC | 15:38 | |
| *** tblakes_ has joined #openstack-nova | 15:38 | |
| *** yingjun has joined #openstack-nova | 15:41 | |
| macsz | mriedem: there was no meaningful response from the operators on https://review.openstack.org/#/c/397800/ (ML: http://lists.openstack.org/pipermail/openstack-operators/2016-November/012087.html ). do you think we can go forward and remove -2? I will address comments from PS2 today | 15:41 |
| *** tblakes_ is now known as tblakes | 15:41 | |
| mriedem | macsz: can you follow up with johnthetubaguy or mikal to make sure rax isn't using that hook mechanism before we yank it? | 15:42 |
| mriedem | macsz: i'll also fish in the operators channel one last time today | 15:44 |
| mordred | mriedem: morning sir! in http://developer.openstack.org/api-ref/compute/?expanded=show-rate-and-absolute-limits-detail the example response shows a bunch of fields starting with total - but they are not in the documented set of responses | 15:44 |
| macsz | mriedem: sir, yes, sir! | 15:44 |
| mordred | mriedem: any idea which is correct? | 15:44 |
| mriedem | http://developer.openstack.org/api-ref/compute/?expanded=show-rate-and-absolute-limits-detail#id215 ? | 15:45 |
| mriedem | mordred: let me dig | 15:45 |
| wznoinsk | mriedem, yes, trivial, we're working on a permanent fix while workarounded and recheck in-flight (via zuul cli rather than gerrit) | 15:45 |
| mordred | mriedem: thanks! | 15:45 |
| mriedem | wznoinsk: thanks | 15:45 |
| mriedem | sdague: reminder ping on https://review.openstack.org/#/c/415511/ | 15:46 |
| openstackgerrit | Rafael Folco proposed openstack/nova: DELETE all inventory for a resource provider https://review.openstack.org/416669 | 15:46 |
| *** mvk has quit IRC | 15:47 | |
| *** Jeffrey4l_ has joined #openstack-nova | 15:48 | |
| sdague | mordred: I think the total* fields are real, just not documented | 15:48 |
| *** sneti has joined #openstack-nova | 15:48 | |
| mordred | sdague: awesome. should I file a bug? | 15:48 |
| sdague | mordred: sure | 15:49 |
| mordred | \o/ | 15:49 |
| *** ducttape_ has joined #openstack-nova | 15:49 | |
| mriedem | mordred: os-used-limits | 15:49 |
| mriedem | that's the thing that provides those | 15:49 |
| mriedem | https://github.com/openstack/nova/blob/master/nova/api/openstack/compute/used_limits.py#L42 | 15:49 |
| mriedem | if i'm reading that gorp correctly, os-used-limits extends the limits API | 15:50 |
| mordred | gotcha. so it used to be an extensoin - is now likely a part of the limits api due to miroversions yeah? | 15:50 |
| mriedem | and adds the used limits (totals) to the response object from the limits controller | 15:50 |
| sdague | mordred: yeh | 15:51 |
| *** rama_y has joined #openstack-nova | 15:51 | |
| *** yingjun has quit IRC | 15:51 | |
| mriedem | mordred: well it's still an extension, but :) | 15:51 |
| mordred | also - should I expect that the floatingips and security group fields will go away with proxy removal? | 15:51 |
| sdague | mriedem: so, for good form on https://review.openstack.org/#/c/415511 | 15:51 |
| mordred | mriedem: :) | 15:51 |
| sdague | mordred: yes, the code drops those at 2.36 | 15:51 |
| mordred | oh. duh. yah - I see that now | 15:51 |
| mriedem | floatingip and security groups will work with microversion < 2.36 | 15:51 |
| mordred | awesome | 15:51 |
| mriedem | after we drop nova-net, those won't work for nova-net backed systems at any microversion | 15:52 |
| mriedem | because the code will literally be gone | 15:52 |
| sdague | mriedem: on https://review.openstack.org/#/c/415511 ... could you put the body snippets of what the resources end up looking like? Mostly just because REST changes are much easier to historically review in specs when it shows actual sample response | 15:52 |
| mriedem | mordred: anyway, yeah a bug for not having os-used-limits documented in the api-ref would be good | 15:52 |
| mriedem | sdague: sure | 15:53 |
| sdague | mriedem: the spec works for me otherwise | 15:53 |
| mordred | mriedem: https://bugs.launchpad.net/nova/+bug/1654316 | 15:53 |
| openstack | Launchpad bug 1654316 in OpenStack Compute (nova) "total* fields in limits example response undocumented" [Undecided,New] | 15:53 |
| *** salv-orl_ has joined #openstack-nova | 15:53 | |
| *** thorst_afk is now known as thorst_ | 15:53 | |
| mordred | sdague, mriedem: while I'm bothering you - do you happen to know off the top of your head if glance/neutron have equiv limits api calls? | 15:54 |
| openstackgerrit | Matthew Booth proposed openstack/nova: libvirt: Add create_from_image and create_from_func to Backend https://review.openstack.org/333244 | 15:54 |
| mriedem | i know they have quotas... | 15:54 |
| mordred | yah - quotas was the only thing I found | 15:54 |
| mordred | is there a conceptual differences between quotas and limits? | 15:55 |
| *** salv-orlando has quit IRC | 15:56 | |
| mriedem | i don't think so... | 15:56 |
| mriedem | the limits API gets it's values from the quotas engine | 15:56 |
| mriedem | the only difference is that os-used-limits extension which shows how much of a given quota you're using | 15:57 |
| mriedem | which i realize it's totally awesome that we have like 6 different APIs related to quotas, | 15:57 |
| mriedem | probably half of which don't actually do anything, like quota classes | 15:57 |
| mriedem | which is why i want to kill the quota classes api but jay won't let me :P | 15:58 |
| openstackgerrit | Diana Clarke proposed openstack/nova: api-ref: use the examples with paging links https://review.openstack.org/416312 | 15:58 |
| mordred | \o/ | 15:58 |
| mordred | it's also neat that some of these are very much admin functions | 15:58 |
| mordred | like - for neutron: | 15:58 |
| mordred | r=c.get('/v2.0/quotas') | 15:58 |
| mordred | keystoneauth1.exceptions.http.Forbidden: Forbidden (HTTP 403) (Request-ID: req-17d9d1a0-cf68-4e96-bbb0-362bb2f9be2e) | 15:58 |
| mordred | you know - a user could certainly never want to know what their quota is :) | 15:59 |
| mriedem | mordred: for neutron, there is no limits api | 15:59 |
| *** nkrinner is now known as nkrinner_afk | 15:59 | |
| mordred | yah. only quotas | 15:59 |
| mriedem | mordred: when creating a server(s), nova gets the port quota (hard limit) and then lists ports for the project, and then does the math inside nova | 16:00 |
| mordred | the nova limits api totally works for end user - so thank you | 16:00 |
| mordred | mriedem: wow. that's awesome | 16:00 |
| mriedem | mordred: yup | 16:00 |
| *** ljxiash has quit IRC | 16:00 | |
| mordred | mriedem: I'm so glad I came back from vacation | 16:00 |
| mriedem | mordred: hey, at least we only ask for the port id in the response now rather than the full port body response for all of them like we used to :) | 16:00 |
| mriedem | just to do len(ports) | 16:01 |
| *** markus_z has quit IRC | 16:01 | |
| openstackgerrit | Alex Szarka proposed openstack/nova: Transform instance-exists notification https://review.openstack.org/403660 | 16:03 |
| openstackgerrit | Alex Szarka proposed openstack/nova: Transform instance.rebuild.error notification https://review.openstack.org/396621 | 16:04 |
| *** ijw has joined #openstack-nova | 16:05 | |
| *** rcernin has quit IRC | 16:07 | |
| *** voelzmo has quit IRC | 16:08 | |
| *** voelzmo has joined #openstack-nova | 16:08 | |
| *** voelzmo has quit IRC | 16:08 | |
| johnthetubaguy | mriedem: FWIW, we don't use that plugin system that macsz is looking to remove | 16:08 |
| *** tbachman has quit IRC | 16:09 | |
| *** voelzmo has joined #openstack-nova | 16:09 | |
| johnthetubaguy | mriedem: my only slight reservation is folks that use the glance file backend attached to shared storage that is also attached to all their hypervisors, no idea if anyone has ever made that work though | 16:09 |
| jlvillal | mriedem: Just wanted to remind you to talk to jaypipes, if you haven't already on that issue from last night. | 16:10 |
| *** ijw has quit IRC | 16:10 | |
| openstackgerrit | Stephen Finucane proposed openstack/nova: console: introduce basic framework for security proxying https://review.openstack.org/345396 | 16:11 |
| openstackgerrit | Stephen Finucane proposed openstack/nova: console: introduce framework for RFB authentication https://review.openstack.org/345397 | 16:11 |
| openstackgerrit | Stephen Finucane proposed openstack/nova: console: introduce the VeNCrypt RFB authentication scheme https://review.openstack.org/345398 | 16:11 |
| openstackgerrit | Stephen Finucane proposed openstack/nova: console: provide an RFB security proxy implementation https://review.openstack.org/345399 | 16:11 |
| *** moshele has quit IRC | 16:13 | |
| *** rmart04 has quit IRC | 16:13 | |
| *** voelzmo has quit IRC | 16:14 | |
| *** dmsimard has joined #openstack-nova | 16:14 | |
| dmsimard | o/ I noticed that there's an entry point for wsgi_scripts with the placement api: https://github.com/openstack/nova/blob/3d45ab20457abff81be03db7a1c3780b713afde6/setup.cfg#L72-L73 | 16:14 |
| dmsimard | Could there also be one for the nova-api wsgi ? | 16:14 |
| openstackgerrit | Alex Szarka proposed openstack/nova: Transform instance.rebuild notification https://review.openstack.org/384922 | 16:16 |
| dmsimard | Right now it's experimental (really?) and hidden in nova/wsgi/ https://github.com/openstack/nova/blob/master/nova/wsgi/nova-api.py | 16:18 |
| *** tbachman has joined #openstack-nova | 16:20 | |
| *** tesseract has quit IRC | 16:21 | |
| *** roeyc has quit IRC | 16:22 | |
| openstackgerrit | Stephen Finucane proposed openstack/nova: Fix backwards compatibility for InstanceNUMACell https://review.openstack.org/396184 | 16:24 |
| sfinucan | sahid: Sweet. Any chance you could take a look at ^ again after? | 16:24 |
| *** dtp has joined #openstack-nova | 16:26 | |
| dtp | what's the best way to respond to points in gerrit reviews? should i just reply and reference the same line numbers? | 16:27 |
| *** rama_y has quit IRC | 16:28 | |
| mdbooth | dtp: I might be missing some context here. You just reply directly in gerrit? The line number context is already there. | 16:28 |
| *** hoangcx has quit IRC | 16:29 | |
| dtp | no you are good. let me try | 16:29 |
| dtp | thank you | 16:29 |
| openstackgerrit | Alex Szarka proposed openstack/nova: Transform instance.rebuild.error notification https://review.openstack.org/396621 | 16:29 |
| *** baoli has quit IRC | 16:31 | |
| *** tbachman has quit IRC | 16:31 | |
| *** catintheroof has joined #openstack-nova | 16:32 | |
| *** tjones has joined #openstack-nova | 16:32 | |
| dansmith | dtp: if you click on a line of code it will open you up the option to type, and/or hit reply on someone else's comment | 16:34 |
| dtp | cool. thanks | 16:34 |
| jaypipes | mriedem: mornin. what did I miss from jlvillal? | 16:35 |
| mriedem | jaypipes: oh tons of fun | 16:35 |
| mriedem | sec | 16:35 |
| mriedem | jaypipes: two things, https://review.openstack.org/#/c/416810/ | 16:35 |
| mriedem | and https://review.openstack.org/#/c/416822/ | 16:36 |
| mriedem | we were over in the ironic channel debugging some scheduler/resource tracker failures for a couple of hours last night, | 16:36 |
| mriedem | and those were the resulting fixes | 16:36 |
| jaypipes | mriedem: ok, cool. will review right away. | 16:36 |
| *** hwoarang_ is now known as hwoarang | 16:36 | |
| mriedem | the 401 issue was from the keystone v3 only job which has been fixed in devstack already | 16:36 |
| mriedem | but every time the RT ran to talk to the placement API we got a 401 and weren't disabling that check | 16:37 |
| *** weshay is now known as weshay_errand | 16:37 | |
| jaypipes | I see | 16:37 |
| mriedem | the other thing was a latent race in the scheduler host manager stuff where it picks up newly created compute nodes from the db before they have free_disk_gb set | 16:37 |
| jaypipes | ok | 16:37 |
| mriedem | seems we shouldn't even pull those types of compute nodes from the database in the first place, but i'm not very familiar with that host manager code | 16:38 |
| jaypipes | mriedem: odd that the scheduler report client would affect compute_nodes.free_disk_gb, considering the scheduler report client never looks at that field. | 16:39 |
| mriedem | it's not that | 16:39 |
| jaypipes | I'm reading your bug report comment. | 16:39 |
| mriedem | update_resource_stats in the scheduler report client calls compute_node.save(), | 16:39 |
| jaypipes | right | 16:39 |
| mriedem | jaypipes: it's probably easier to just read the commit message | 16:39 |
| mriedem | as the bug report is a bit scatterbrained during debug | 16:39 |
| jaypipes | :) trying to do that now... | 16:39 |
| sahid | sfinucan: hum :/ i commented i think we can't remove that code so i would like you to remove the TODO | 16:43 |
| jaypipes | sahid: did you see my review comment about renaming cpu_overhead to cpus_reserved? | 16:44 |
| sahid | jaypipes: yes, yes i did not get time to do it so far | 16:44 |
| jaypipes | sahid: no worries, just wanted to make sure it didn't get lost in the rebases and revisions :) | 16:45 |
| sahid | no, no don't worry - i wanted to ask you if you want me to change the name only for the field or everywhere ? | 16:45 |
| sahid | well i guess everywhere :) | 16:46 |
| *** prateek_ has joined #openstack-nova | 16:46 | |
| *** gouthamr has joined #openstack-nova | 16:46 | |
| *** unicell1 has quit IRC | 16:47 | |
| mriedem | sdague: you may also enjoy https://review.openstack.org/#/c/416810/ | 16:47 |
| *** jvgrant_ has joined #openstack-nova | 16:48 | |
| sfinucan | sahid: Ugh, yeah. Whoever thought it was a good idea to put JSON in the database :( I'll do that now | 16:48 |
| *** raunak has joined #openstack-nova | 16:49 | |
| *** artom has joined #openstack-nova | 16:49 | |
| *** prateek has quit IRC | 16:49 | |
| sahid | sfinucan: yes it's a particularity of the numa part | 16:49 |
| jaypipes | mriedem: +2 on both patches. nice work with jlvillal and JayF. | 16:50 |
| mriedem | thanks | 16:50 |
| JayF | mriedem did most of the work | 16:50 |
| jaypipes | sahid: yes, everywhere :) | 16:50 |
| JayF | I just followed along in irc | 16:50 |
| JayF | lol | 16:50 |
| jlvillal | jaypipes: Oh I only started the conversation. mriedem and JayF did all the heavy lifting. I had to go do a 1:1 with my manager and then leave for our holiday party :) | 16:51 |
| openstackgerrit | Matt Riedemann proposed openstack/nova-specs: Return uuid from os-aggregates API (spec) https://review.openstack.org/415511 | 16:51 |
| mriedem | sdague: added the example response ^ | 16:51 |
| jlvillal | Big thanks to mriedem and JayF :D | 16:51 |
| *** jvgrant has quit IRC | 16:51 | |
| mriedem | supporting me talk to myself through debugging a problem is half the battle :) | 16:51 |
| mriedem | johnthetubaguy: want to re-apply your +2 to https://review.openstack.org/#/c/415511/ ? i just added an api response example for GET | 16:52 |
| openstackgerrit | Jiri Suchomel proposed openstack/nova: Retry few times when creating the initial connection to libvirtd. https://review.openstack.org/416928 | 16:52 |
| jaypipes | jlvillal: holiday party definitely more important. | 16:53 |
| jlvillal | heh :) | 16:53 |
| jaypipes | :) | 16:53 |
| JayF | mriedem: quack? | 16:54 |
| *** liverpooler has quit IRC | 16:55 | |
| *** browne has joined #openstack-nova | 16:55 | |
| *** amotoki has quit IRC | 16:55 | |
| openstackgerrit | Stephen Finucane proposed openstack/nova: Fix backwards compatibility for InstanceNUMACell https://review.openstack.org/396184 | 16:55 |
| *** diana_clarke has quit IRC | 16:57 | |
| *** pcaruana has quit IRC | 16:58 | |
| *** tlian has quit IRC | 17:00 | |
| *** liverpooler has joined #openstack-nova | 17:00 | |
| *** bvanhav has quit IRC | 17:00 | |
| *** tjones has quit IRC | 17:00 | |
| *** ijw has joined #openstack-nova | 17:01 | |
| *** tbachman has joined #openstack-nova | 17:02 | |
| dansmith | mriedem: houston, we have a problem | 17:02 |
| mriedem | noooooo | 17:02 |
| mriedem | i hope it's just about that regex :) | 17:02 |
| openstackgerrit | Derek Higgins proposed openstack/nova: libvirt: fix nova can't delete the instance with nvram https://review.openstack.org/357190 | 17:02 |
| dansmith | mriedem: so, I need to go dig, but.. there's something about cellsv1 that means we don't have hosts in the database to do the cell mappings when we're doing the rest of it | 17:03 |
| dansmith | I remember something very faintly about it, but I don't know the details.. | 17:03 |
| dansmith | I'm thinking it's something like we won't get hosts reported to the api cell until after we've been running for a bit or something | 17:04 |
| dansmith | which will be a major wrench in the works | 17:04 |
| mriedem | so this is for the nova-manage discover_hosts command? | 17:04 |
| mriedem | by hosts do you mean compute node records? | 17:04 |
| dansmith | well, map_cell_and_hosts, but same difference yeah | 17:05 |
| mriedem | do we get the cell mapping record? | 17:06 |
| dansmith | no, it fails early because no hosts | 17:06 |
| dansmith | https://github.com/openstack/nova/blob/master/nova/cmd/manage.py#L1244-L1247 | 17:06 |
| jaypipes | looks like dansmith needs https://twitter.com/jaypipes/status/801422826402148353 | 17:06 |
| dansmith | won't help unless I'm rechecking to see if anyone else has fixed it | 17:07 |
| mriedem | jaypipes: if that's intel nfv ci it's a known broken i think | 17:07 |
| jaypipes | mriedem: intel-nfv-ci, xen ci, the list goes on :) | 17:07 |
| mriedem | wznoinsk: i've removed intel nfv ci from voting on nova, let me know when i can reenable it | 17:08 |
| jaypipes | dansmith, mriedem: here, this'll cheer you up: https://twitter.com/jaypipes/status/801233017993719808 | 17:08 |
| dansmith | mriedem: I'm almost wondering if we never get those records up in the api node, actually | 17:08 |
| dansmith | johnthetubaguy: around? | 17:08 |
| dansmith | mriedem: I guess I should see how bad it would be to continue to support the old way of listing just for cellsv1 and measure that against the pain of a real fix | 17:09 |
| jaypipes | bauzas: around? you handling the rebase of https://review.openstack.org/392569? If not, happy to do it. just lemme know. | 17:10 |
| mriedem | ok i see http://logs.openstack.org/85/416585/4/experimental/gate-tempest-dsvm-cells-ubuntu-xenial/fb5df4f/logs/devstacklog.txt.gz#_2017-01-05_15_37_14_035 | 17:10 |
| *** diana_clarke has joined #openstack-nova | 17:10 | |
| dansmith | mriedem: yeah | 17:11 |
| bauzas | jaypipes: ooooh :( | 17:11 |
| bauzas | jaypipes: I was literrally clicking on submitting the follow-up changre | 17:11 |
| bauzas | jaypipes: hold on, rebasing then | 17:11 |
| jaypipes | bauzas: no worries. | 17:11 |
| mriedem | dansmith: i guess it doesn't matter if we're using nova.conf or nova-cell.conf in this context because they both point to the same dbs | 17:11 |
| dansmith | mriedem: do they? | 17:12 |
| mriedem | i know that devstack starts the cells stuff in a different order, i'm wondering if we're running this before the child cell has started? | 17:12 |
| mriedem | http://logs.openstack.org/85/416585/4/experimental/gate-tempest-dsvm-cells-ubuntu-xenial/fb5df4f/logs/etc/nova/nova.conf.txt.gz | 17:12 |
| mriedem | [database] connection = mysql+pymysql://root:secretmysql@127.0.0.1/nova?charset=utf8 | 17:12 |
| mriedem | http://logs.openstack.org/85/416585/4/experimental/gate-tempest-dsvm-cells-ubuntu-xenial/fb5df4f/logs/etc/nova/nova-cells.conf.txt.gz | 17:12 |
| mriedem | [database] connection = mysql+pymysql://root:secretmysql@127.0.0.1/nova_cell?charset=utf8 | 17:12 |
| mriedem | oh i guess not | 17:12 |
| *** lucasagomes is now known as lucas-afk | 17:12 | |
| *** derekh has quit IRC | 17:12 | |
| mriedem | so... | 17:12 |
| dansmith | yeah, so that's worth a try | 17:12 |
| *** raunak has quit IRC | 17:13 | |
| dansmith | especially since laski's comment in there says to use nova-cell.conf | 17:13 |
| mriedem | we should be using nova-cells.conf when running nova-manage right? to get the compute nodes from the child cell | 17:13 |
| mriedem | comment in where? | 17:13 |
| dansmith | mriedem: houston, we may have a problem, stand by | 17:13 |
| dansmith | mriedem: https://github.com/openstack/nova/blob/master/nova/cmd/manage.py#L1305 | 17:13 |
| mriedem | dansmith: in this scenario, are you tom hanks or kevin bacon? | 17:13 |
| dansmith | mriedem: definitely tom hanks | 17:14 |
| bauzas | mriedem: please, gravity. | 17:14 |
| mriedem | gravity was terrible | 17:14 |
| dansmith | mriedem: you can be mr. bacon | 17:14 |
| dansmith | I know you have special feelings for him | 17:14 |
| mriedem | unless you love smug ass george clooney | 17:14 |
| mriedem | i do love kevin bacon | 17:14 |
| jaypipes | johnthetubaguy, mriedem: seems https://review.openstack.org/#/c/415896/ might be a devstack[-gate] failure? | 17:14 |
| mriedem | jaypipes: yeah hit a gate failure | 17:15 |
| mriedem | cinder v1 unit tests failed, but those were dropped | 17:15 |
| mriedem | rechecking | 17:15 |
| jaypipes | mriedem: something being worked on, or should I look into it? | 17:15 |
| jaypipes | ah, k | 17:15 |
| mriedem | should be fine | 17:15 |
| bauzas | oh snap | 17:15 |
| openstackgerrit | Jay Jahns proposed openstack/nova: Add connection pool size to vSphere settings https://review.openstack.org/414085 | 17:16 |
| bauzas | jaypipes: cdent: so we merged something using allowed_filters :( | 17:16 |
| *** bvanhav has joined #openstack-nova | 17:16 | |
| *** yamahata has joined #openstack-nova | 17:17 | |
| jaypipes | bauzas: yeah. will need to be modded to use JSONSchema. | 17:17 |
| bauzas | jaypipes: I need to look at the change | 17:17 |
| jaypipes | bauzas: whichever one of those landed first, the other would need modding. | 17:17 |
| jaypipes | bauzas: just a fact of life in this code at this moment. | 17:18 |
| bauzas | sure, I just need to understand the logic of the merged patch | 17:18 |
| dansmith | dtp: yep, just like that :) | 17:18 |
| dtp | cool | 17:18 |
| jaypipes | bauzas: shouldn't be much needed other than adding 'member_of' into the JSONSChema for get. | 17:18 |
| bauzas | jaypipes: no worries, just looking at the merged change | 17:18 |
| dtp | thanks. also, why is no one fighting over gary sinise? | 17:18 |
| mriedem | sdague: i meant to ask you, is there a hook in grenade such that we can do some stuff AFTER ocata code is installed but BEFORE the services are restarted to use that code? | 17:18 |
| mriedem | dtp: i couldn't remember his last name :) | 17:18 |
| mriedem | i knew it was gary though | 17:19 |
| openstackgerrit | Balazs Gibizer proposed openstack/nova: Transform missing delete notifications https://review.openstack.org/410297 | 17:19 |
| mriedem | 'the other guy from forest gump' | 17:19 |
| *** sneti has quit IRC | 17:19 | |
| dtp | oof | 17:19 |
| dtp | lt. dan | 17:19 |
| dansmith | dtp: um, pretty sure he never said that without screaming, right? | 17:19 |
| mriedem | or bill paxton | 17:19 |
| dansmith | LT DAAYUNN! | 17:19 |
| mriedem | https://www.youtube.com/watch?v=dsx2vdn7gpY | 17:20 |
| mriedem | what a whiner | 17:20 |
| *** jose-phillips has joined #openstack-nova | 17:20 | |
| *** tlian has joined #openstack-nova | 17:20 | |
| mriedem | i literally had a nightmare the other nighth that the aliens from aliens were chasing me through a space station, was terrible | 17:20 |
| dtp | true dansmith | 17:21 |
| dansmith | but it turns out they all had "cinder" on their shirts and they were screaming "multiattach" right? | 17:21 |
| mriedem | heh | 17:21 |
| mriedem | multiattach is one thing i haven't lost sleep over this week | 17:22 |
| dansmith | heh, well, congrats then.. aliens seem not so bad | 17:22 |
| *** moshele has joined #openstack-nova | 17:23 | |
| *** prateek_ has quit IRC | 17:24 | |
| openstackgerrit | Sylvain Bauza proposed openstack/nova: Expose a REST API for a specific list of RPs https://review.openstack.org/392569 | 17:25 |
| openstackgerrit | Sylvain Bauza proposed openstack/nova: Put all the magic in a PlacementClient https://review.openstack.org/417111 | 17:25 |
| bauzas | jaypipes: cdent: your eyes will be helpful | 17:25 |
| bauzas | at least to make sure I haven't trampled the member_of lofic | 17:25 |
| bauzas | logic | 17:25 |
| jaypipes | yup, on it. | 17:25 |
| *** roeyc has joined #openstack-nova | 17:25 | |
| *** dmsimard has left #openstack-nova | 17:26 | |
| *** sahid has quit IRC | 17:26 | |
| *** roeyc has quit IRC | 17:26 | |
| *** nic has joined #openstack-nova | 17:26 | |
| *** annegentle has joined #openstack-nova | 17:27 | |
| *** LiberalCarrot has joined #openstack-nova | 17:28 | |
| *** LiberalCarrot has left #openstack-nova | 17:28 | |
| cdent | bauzas: will look after dinner | 17:28 |
| *** sneti has joined #openstack-nova | 17:28 | |
| bauzas | jaypipes: found a miss | 17:29 |
| mriedem | bauzas: yo'ure going to need to rebase https://review.openstack.org/#/c/417111/1 on top of a change of mine probably | 17:29 |
| bauzas | jaypipes: just uploading now | 17:29 |
| jaypipes | bauzas: k | 17:29 |
| openstackgerrit | Sylvain Bauza proposed openstack/nova: Put all the magic in a PlacementClient https://review.openstack.org/417111 | 17:29 |
| openstackgerrit | Sylvain Bauza proposed openstack/nova: Expose a REST API for a specific list of RPs https://review.openstack.org/392569 | 17:29 |
| bauzas | jaypipes: that's it ^ | 17:29 |
| bauzas | mriedem: roger. | 17:29 |
| mriedem | bauzas: not sure how though, you could just approve my fix and rebase once it's merged | 17:30 |
| bauzas | mriedem: hell, good point you found | 17:30 |
| bauzas | mriedem: indeed :) | 17:30 |
| bauzas | my stats are so poor this week | 17:30 |
| bauzas | thanks for helping me to resurrect my stats | 17:30 |
| bauzas | note for self : never ever take vacations | 17:31 |
| bauzas | that's loose time | 17:31 |
| mriedem | bauzas: i can feed you plenty of things that are ready to go :) | 17:32 |
| bauzas | mriedem: <3 | 17:32 |
| bauzas | I'm pretty sure I can find those | 17:32 |
| mriedem | your time is probably better served right now on getting the scheduler bits up | 17:32 |
| bauzas | magic of dashboards ! | 17:32 |
| bauzas | anyway, +W'd | 17:32 |
| mriedem | would be awesome to at least have a WIP of that before next week | 17:32 |
| bauzas | mriedem: yeah, my goal | 17:32 |
| bauzas | we're 4 weeks away, and I'm a bit stressed | 17:33 |
| mriedem | me too | 17:33 |
| bauzas | the scheduler patch shouldn't be a big deal tho | 17:33 |
| mriedem | i'm on vacation the week of o-3 bw | 17:33 |
| jaypipes | cdent: did we decide not to add a microversion to the API when exposing member_of param in the GET /r-p API? | 17:33 |
| mriedem | *btw | 17:33 |
| mriedem | bauzas: so you'll have to handle the beta release tag duties | 17:33 |
| bauzas | mriedem: fine with me, I can handle that | 17:34 |
| mriedem | thanks | 17:34 |
| mriedem | i'm out of country 1/21-1/28 | 17:34 |
| jaypipes | I'm on PTO that week as well. | 17:34 |
| cdent | jaypipes: not that I'm aware of, the code under review has a microversion bump right now | 17:34 |
| *** sneti has quit IRC | 17:34 | |
| bauzas | mriedem: good to know | 17:34 |
| cdent | jaypipes: but I'm happy to change that if we don't care | 17:35 |
| bauzas | mriedem, hope you're not visiting Mexico by that time, or you could be trumped | 17:35 |
| jaypipes | cdent: but technically, the handler *already* accounts for member_of (since it looks at the ResourceProviderList.allowed_filters... | 17:35 |
| mriedem | bauzas: i am :) | 17:35 |
| mriedem | shit that's right | 17:35 |
| bauzas | mriedem: oooooh | 17:35 |
| mriedem | i might not get back in | 17:35 |
| jaypipes | cdent: so the API-facing change has already landed :( | 17:35 |
| cdent | ? | 17:35 |
| bauzas | mriedem: I could give you the French nationality but you would have to rant every day | 17:35 |
| jaypipes | cdent: meaning, the code in nova *right now* supports specifying member_of in the GET params. | 17:36 |
| jaypipes | cdent: and will route it to the ResourceProviderList.get_all_by_filters() call. | 17:36 |
| bauzas | jaypipes: good point, we silently accepted a change in the API | 17:36 |
| bauzas | that's what I thought when I rebased | 17:36 |
| openstackgerrit | Gleb Stepanov proposed openstack/python-novaclient: Add evacuate host method https://review.openstack.org/349179 | 17:37 |
| *** moshele has quit IRC | 17:37 | |
| bauzas | since we're contracting on allowed_filters | 17:37 |
| *** lpetrut has quit IRC | 17:37 | |
| bauzas | but anyway, it's merged | 17:37 |
| cdent | it won't work very well | 17:37 |
| jaypipes | cdent: this was the main issue with having the handler just look at the object allowed_filters key that me switching to JSONSChema representation of the GET request solved. | 17:37 |
| cdent | sure | 17:37 |
| cdent | I was just using what somebody else had already done | 17:37 |
| jaypipes | understood. | 17:37 |
| *** moshele has joined #openstack-nova | 17:37 | |
| cdent | the right thing to do at this point is to just merge the next thing and then clean it up | 17:37 |
| jaypipes | cdent: let's get your microversion patch merged ASAP. | 17:37 |
| bauzas | cdent: but name and uuid were supported because we microversioned | 17:38 |
| cdent | jinx | 17:38 |
| jaypipes | jinx :) | 17:38 |
| jaypipes | fuck. | 17:38 |
| * cdent falls in a black hole | 17:38 | |
| bauzas | cdent: jaypipes: which microversion patch ? | 17:38 |
| jaypipes | cdent: link to that patch pls? | 17:38 |
| jaypipes | bauzas: jinx :) | 17:38 |
| cdent | https://review.openstack.org/#/c/407742/ | 17:38 |
| bauzas | oh heh | 17:38 |
| bauzas | nice potential conflict to mine \o/ | 17:39 |
| jaypipes | bauzas: that is the patch I *thought* had already merged. | 17:39 |
| bauzas | jaypipes: no, I checked when rebasing | 17:39 |
| bauzas | I need to bail out for a few hours for social and parental reasons | 17:39 |
| jaypipes | bauzas: no, I mean, when I said cdent's patch had merged, that was the one I thought had merged, not the object change one. | 17:39 |
| bauzas | but I should be back | 17:39 |
| jaypipes | k | 17:40 |
| cdent | I think this sort of thing is bound to happen with the way we've (or maybe just I've) habitually separated object change with http change | 17:40 |
| bauzas | jaypipes: but the change of allowed_filters impacts the API, right? | 17:40 |
| *** moshele has quit IRC | 17:40 | |
| *** unicell has joined #openstack-nova | 17:40 | |
| bauzas | jaypipes: because we verify that object field at the API level | 17:40 |
| *** unicell has quit IRC | 17:40 | |
| bauzas | so we changed our contract when we modified that list | 17:40 |
| cdent | we're not really in any current danger from that contract change | 17:40 |
| bauzas | another reason like you said to have proper schemas | 17:40 |
| *** unicell has joined #openstack-nova | 17:40 | |
| jaypipes | bauzas: yes, unfortunatgely there'll be one or two commits that will have an API change that is technically different from the published microversion 1.3 change. | 17:41 |
| bauzas | :/ | 17:41 |
| bauzas | either way, /me drops | 17:41 |
| openstackgerrit | Matt Riedemann proposed openstack/nova: Update docstring of _schema_validation_helper https://review.openstack.org/417118 | 17:41 |
| jaypipes | bauzas: for a couple commits, GET /resource_providers will accept a member_of param that doesn't use in: | 17:41 |
| cdent | which is not really a big deal is it? anyoe who is a trunk chaser will chase trunk and never notice. and anyone else will never notice | 17:42 |
| jaypipes | cdent: yes, that is my thought as well. | 17:42 |
| *** jose-phillips has quit IRC | 17:42 | |
| cdent | and above and beyond that: the only real user is the resource tracker at this point and it doesn't send anything we don't control | 17:42 |
| jaypipes | right | 17:42 |
| cdent | so the exposure is conceptually bad and but reality-wise not so uch | 17:42 |
| cdent | s/uch/much/ | 17:42 |
| jaypipes | right | 17:42 |
| cdent | cool | 17:42 |
| *** tblakes has quit IRC | 17:43 | |
| mriedem | btw, we have query parameter json schema validation now | 17:44 |
| mriedem | but would have to be duplicated in the placement API i guess to use it? | 17:44 |
| cdent | mriedem: jay's added something very similar in the patch bauzas is working on | 17:46 |
| *** Kevin_Zheng has quit IRC | 17:47 | |
| *** rtheis has quit IRC | 17:48 | |
| jaypipes | cdent: reviewed. -1 for wording. | 17:48 |
| cdent | roger than jaypipes, will look | 17:48 |
| sdague | mriedem: the upgrade script does the service restart as well at the end, so it could be worked in there one way or another | 17:50 |
| *** LiberalSquash has joined #openstack-nova | 17:51 | |
| *** LiberalSquash has left #openstack-nova | 17:51 | |
| mriedem | sdague: ok i was thinking that grenade was calling devstack to deploy the new code and restart the services in a single shot, contained within devstack, | 17:51 |
| mriedem | which i wouldn't have a hook into possibly | 17:51 |
| mriedem | between install and start of services i mean | 17:52 |
| melwitt | nic: hey, are you around? we've been looking at a rbd bug recently where after shelve/unshelve, the snapshot that was made, can't be deleted from glance. have you run into the same problem? https://bugs.launchpad.net/nova/+bug/1653953 | 17:53 |
| openstack | Launchpad bug 1653953 in OpenStack Compute (nova) "Unable to remove snapshots after an instance is unshelved when using the rbd imagebackend" [Undecided,New] | 17:53 |
| openstackgerrit | Chris Dent proposed openstack/nova: HTTP interface for resource providers by aggregates https://review.openstack.org/407742 | 17:55 |
| *** jose-phillips has joined #openstack-nova | 17:56 | |
| *** liverpooler has quit IRC | 17:56 | |
| cdent | jaypipes: ^ better? | 17:56 |
| cdent | btw, I made your suggested array slice change. I rarely use that, not sure why. I guess I find it semantically ugly. Meaning: I don't like to count. I guess. | 17:57 |
| *** liverpooler has joined #openstack-nova | 17:58 | |
| mriedem | i for one hate slice notation | 17:58 |
| mriedem | because i suck at it | 17:58 |
| *** yamahata has quit IRC | 17:58 | |
| jaypipes | +2 from me cdent. mriedem https://review.openstack.org/#/c/407742/ could use your eyes. | 17:58 |
| mriedem | ok i need food first | 17:59 |
| dansmith | slices are the best thing | 17:59 |
| *** yamahata has joined #openstack-nova | 17:59 | |
| dansmith | since at least, I dunno, sliced bread | 17:59 |
| jaypipes | lol :) | 17:59 |
| mriedem | *rimshot* | 17:59 |
| dansmith | slices are usually the first thing I miss in other languages | 17:59 |
| jaypipes | and since we're going to be rewriting everything in Golang, and slices are a full-fledge type in Golang, we really should use them to their fullest. | 17:59 |
| mriedem | maybe i'll get a slice of pizza | 18:00 |
| dtp | is the [reply done] feature in gerrit used/useful for reviewers? | 18:01 |
| *** tjones has joined #openstack-nova | 18:01 | |
| mriedem | dtp: yes | 18:01 |
| mriedem | as an ack that you saw the comment and handled it | 18:01 |
| cfriesen | any chance of getting a second +2 on https://review.openstack.org/#/c/393487 ? Pretty trivial backport to newton, fixes a real issue. (I just tripped over it in Mitaka) | 18:01 |
| dtp | k | 18:01 |
| *** Swami has joined #openstack-nova | 18:02 | |
| jaypipes | dtp: yeah, I use it. | 18:02 |
| dtp | coo | 18:02 |
| edleafe | jaypipes: cdent: not crazy about the slice change, since we might need to support "nin" too ("not in") | 18:04 |
| *** AnxiousGarlic has joined #openstack-nova | 18:04 | |
| *** AnxiousGarlic has left #openstack-nova | 18:04 | |
| jaypipes | edleafe: meh? :) | 18:05 |
| cdent | edleafe: likely all that hoopa joop is gonna change as a result of the switch to jsonschema validation of query params | 18:05 |
| edleafe | for anti-affinity and such | 18:05 |
| cdent | and nin is yagni talk | 18:05 |
| jaypipes | edleafe: what cdent said :) | 18:05 |
| jaypipes | about the jsonschema thing, not the yagni thing. | 18:05 |
| *** bvanhav_ has joined #openstack-nova | 18:05 | |
| cdent | won't someone please think of the yagni | 18:05 |
| * cdent goes to a PTA meeting | 18:06 | |
| edleafe | cdent: I am, as in "you ARE gonna need it" :) | 18:06 |
| *** bvanhav has quit IRC | 18:06 | |
| cdent | [t 25H] | 18:07 |
| purplerbot | <jaypipes> edleafe: meh? :) [2017-01-05 18:05:05] [n 25H] | 18:07 |
| *** browne has quit IRC | 18:10 | |
| *** cdent has quit IRC | 18:12 | |
| dtp | what is soft delete? | 18:15 |
| dtp | tentative delete but kept on disk for a bit? | 18:17 |
| *** weshay_errand is now known as weshay | 18:17 | |
| melwitt | dtp: yeah. "soft delete" as a feature is a deferred delete where there's a config option for low long before it gets really deleted forever. in the meantime, it can be restored | 18:17 |
| dansmith | dtp: soft delete is an abomination | 18:18 |
| dtp | thanks | 18:18 |
| dtp | oh? | 18:18 |
| melwitt | dtp: we often say "soft delete" to mean the way the "deleted" column is used in a lot of the database schemas too, fyi. in the code though, soft_delete means the feature | 18:20 |
| dansmith | yes, there are two soft deletes | 18:20 |
| dansmith | and they're both abominations | 18:20 |
| dansmith | but in different ways | 18:20 |
| dansmith | i.e. two roads to hell | 18:20 |
| dtp | better bring a cooler | 18:21 |
| dtp | when a soft-deleted instance is deleted for realzies, does it go through task_state=DELETING like normal? | 18:21 |
| dansmith | and, that's why it's an abominaton | 18:21 |
| dansmith | because to answer that, I'd have to look at the code, get the answer wrong once, then try again, etc | 18:22 |
| dtp | i see. lets leave that alone for now | 18:22 |
| * dansmith wonders if dtp is picking up on his hatred of this feature | 18:22 | |
| *** READ10 has quit IRC | 18:22 | |
| dtp | i detect no emotional anomalies | 18:22 |
| dansmith | heh | 18:23 |
| *** rama_y has joined #openstack-nova | 18:24 | |
| mriedem | btw, | 18:24 |
| mriedem | BTW, | 18:24 |
| mriedem | soft delete in the API is configured via a config option that is actually only used on the compute that the instance is running on | 18:25 |
| mriedem | and if those are different, oops! | 18:25 |
| * dansmith hears hell calling | 18:25 | |
| mriedem | s/configured/toggled/ in the api | 18:25 |
| mriedem | so a config-driven API feature | 18:25 |
| mriedem | not that we dont already have crap like that based on which virt driver is on the compute host the instance is running on...but that's more of a known evil | 18:26 |
| melwitt | that's even worse than I thought | 18:26 |
| *** roeyc has joined #openstack-nova | 18:27 | |
| ildikov | Hi Nova people and Happy New Year! | 18:27 |
| ildikov | I see you found nice topics for the first week already :) | 18:27 |
| ildikov | Is there any one around who could lecture me a bit on how to configure Nova to use the Cinder v3 API? | 18:28 |
| mriedem | ildikov: scottda has a WIP for that | 18:28 |
| mriedem | sec | 18:28 |
| mriedem | https://review.openstack.org/#/c/385682/ i think | 18:29 |
| ildikov | I thought I found the right config option for that and now in my one node Devstack install and the nova API succeeds with the v3 api call and the compute tells me it cannot find the endpoint... | 18:29 |
| *** eglynn has quit IRC | 18:29 | |
| mriedem | ildikov: i believe it would be either endpoint_template | 18:29 |
| ildikov | mriedem: right, so I set the catalog_info to v3 | 18:29 |
| ildikov | in the nova.conf | 18:30 |
| mriedem | or service_type or service_name, one of those is cinderv3 i think | 18:30 |
| ildikov | it's volumev3:cinderv3:publicURL | 18:30 |
| ildikov | in my config | 18:30 |
| mriedem | and that fails? | 18:30 |
| ildikov | only in the compute | 18:31 |
| mriedem | is it a new devstack? | 18:31 |
| ildikov | it says: EndpointNotFound: publicURL endpoint for volumev3 service named cinderv3 in RegionOne region not found | 18:31 |
| ildikov | yes, new devstack | 18:31 |
| mriedem | what's in the catalog? did you do openstack catalog list | 18:31 |
| mriedem | or whatever that command is | 18:31 |
| ildikov | this is the log from the API: Cinder api version 3 and url http://127.0.0.1:8776/v3/0f810b0913db4fc0a77135d2e40dfd23 | 18:32 |
| ildikov | it's a one node devstack install | 18:32 |
| ildikov | cinderv3 is listed among the endpoints | 18:32 |
| ildikov | I'm sure it's something dumb obvious misconfiguration, but I cannot figure out what it can be | 18:33 |
| mriedem | but can you list volumes via the nova api? | 18:33 |
| mriedem | like: nova --os-compute-api-version 2.35 volume-list ? | 18:33 |
| mriedem | that will proxy through to cinder | 18:33 |
| ildikov | with volume-attachments it works | 18:36 |
| mriedem | volume-attachments doesn't call cinder | 18:36 |
| mriedem | it uses the block_device_mappings table in the nova db | 18:36 |
| mriedem | i've got a newish devstack, i'll see if i can reproduce | 18:36 |
| *** nicolasbock has quit IRC | 18:37 | |
| mriedem | this may take awhile, the cafeteria was playing don henley and i can't get it out of my head | 18:38 |
| mriedem | in a bad way | 18:39 |
| *** dharinic is now known as dharinic|lunch | 18:39 | |
| ildikov | pfff, ok, I will pick another one then as volume-list wasn't in the list | 18:39 |
| mriedem | yeah we deleted that... | 18:40 |
| mriedem | forgot | 18:40 |
| *** sreeram_v has joined #openstack-nova | 18:40 | |
| *** sneti has joined #openstack-nova | 18:40 | |
| mriedem | i'm going to install an older novaclient in a venv | 18:40 |
| *** raunak has joined #openstack-nova | 18:41 | |
| scottda | ildikov: I haven't tested that in a while. Let me set it up and see.... | 18:41 |
| ildikov | scottda: if that works I will build that in to my patch, I'm currently only trying to make catalog_info work with Cinder v3 | 18:42 |
| ildikov | scottda: using the latest attach code is hardcoded atm | 18:42 |
| ildikov | mriedem: I was trying volume-attach | 18:42 |
| *** Guest50714 has quit IRC | 18:43 | |
| *** baoli has joined #openstack-nova | 18:43 | |
| *** sneti has quit IRC | 18:43 | |
| scottda | ildikov: Well, just adding 'catalog_info = volumev3:cinderv3:publicURL' should just work, without any patches, for existing nova-cinder API | 18:43 |
| scottda | If it doesn't, there's a problem. | 18:44 |
| ildikov | mriedem: that's from the API log: GET http://127.0.0.1:8776/v3/0f810b0913db4fc0a77135d2e40dfd23/volumes/3277e3a3-3c4a-43dc-b6b2-001536c6de89 | 18:44 |
| *** sreeram_v has quit IRC | 18:44 | |
| ildikov | mriedem: and that's the compute log: EndpointNotFound: publicURL endpoint for volumev3 service named cinderv3 in RegionOne region not found | 18:45 |
| *** mvk has joined #openstack-nova | 18:45 | |
| ildikov | in the same attach flow on the same Devstack single node | 18:45 |
| ildikov | scottda: I'm experiencing the above ^^ | 18:45 |
| scottda | hmmmm....I get a real IP from "openstack endpoint list", not 127.0.0.1 | 18:47 |
| scottda | ie. 192.168.122.197 | 18:48 |
| scottda | ildikov: What do you get? | 18:48 |
| mriedem | scottda: ildikov: works for me | 18:48 |
| *** browne has joined #openstack-nova | 18:48 | |
| ildikov | scottda: I think I set it this way, it's 127.0.0.1 everywhere | 18:49 |
| ildikov | scottda: but that shouldn't affect whether nova can read the catalog or not I guess | 18:49 |
| scottda | ok | 18:49 |
| ildikov | mriedem: hmm, then I might tear down the whole thing and try again | 18:50 |
| mriedem | this is what i have | 18:50 |
| mriedem | cinder.catalog_info = volumev3:cinderv3:publicURL | 18:50 |
| mriedem | this is that part of the catalog http://paste.openstack.org/show/594037/ | 18:50 |
| ildikov | mine is the same | 18:50 |
| ildikov | mriedem: have you ever experienced that the API and the compute in a single node devstack install behaves differently regarding finding or not the endpoints? | 18:51 |
| mriedem | nope | 18:51 |
| ildikov | I always knew I can do magic, I only wish it would not mess up things all the time but the other way around... :S :) | 18:52 |
| scottda | BTW mriedem , what are your thoughts on making v3 endpoint the default for Nova to use? | 18:53 |
| mriedem | scottda: umm, | 18:53 |
| ildikov | scottda: mriedem: I would be pretty happy :) | 18:53 |
| mriedem | we don't ci that way at all anywhere in nova's gate | 18:53 |
| scottda | I'm not sure if Cinder folks are ready for the switch. Just bringing it up. | 18:53 |
| mriedem | scottda: i think we start by switching one of the nova jobs over to using cinder v3 | 18:54 |
| mriedem | probably the placement job | 18:54 |
| mriedem | which is our test bed for new things | 18:54 |
| scottda | mriedem: OK, just bringing it up since we're talking about it. | 18:54 |
| mriedem | is there a devstack configure to set the cinder catalog info in nova.conf? | 18:54 |
| mriedem | *config | 18:54 |
| scottda | I don't think so, but not sure. | 18:55 |
| mriedem | CINDER_NOVA_CATALOG_INFO | 18:55 |
| mriedem | oh that's to configure cinder | 18:55 |
| mriedem | if there were a devstack variable we could piggyback on from the cinder side that would be good, | 18:56 |
| mriedem | because we can't yet pass localrc through from the job defintiion through to devstack | 18:56 |
| mriedem | and the devstack cores aren't happy about adding more one-off config variables like that | 18:56 |
| mriedem | i don't see anything | 18:57 |
| mriedem | you could talk to sdague about the plumbing bits...but my guess is he'll say he wants it done with localrc | 18:57 |
| dansmith | mriedem: well, progress. now all that fails is "all the listing tests" which is what that patch touches | 18:58 |
| *** Guest50714 has joined #openstack-nova | 18:58 | |
| mriedem | from tempest? | 18:58 |
| mriedem | well, baby steps :) | 18:59 |
| mriedem | that's probably an easier fix than what we started thinking was broken this morning | 18:59 |
| dansmith | yeah | 18:59 |
| dansmith | that's my point | 18:59 |
| mriedem | unless the cells v1 api isn't getting any instances from the cell mapping | 19:00 |
| mriedem | that might suck | 19:00 |
| *** ayogi has joined #openstack-nova | 19:00 | |
| dansmith | well, it's likely easier to resolve still | 19:00 |
| mriedem | does map_cell_and_hosts also call map_instances or whatever? | 19:00 |
| mriedem | my guess is the instances aren't mapped? | 19:00 |
| dansmith | there are no instances at that point, and all created ones should be mapped | 19:01 |
| mriedem | https://github.com/openstack/nova/blob/master/nova/cmd/manage.py#L1182 | 19:01 |
| mriedem | oh right | 19:01 |
| *** shaohe_feng has quit IRC | 19:02 | |
| *** shaohe_feng has joined #openstack-nova | 19:03 | |
| openstackgerrit | Matt Riedemann proposed openstack/nova: WIP Get Cinder v3 microversion https://review.openstack.org/385682 | 19:04 |
| mriedem | scottda: ildikov: ^ | 19:04 |
| *** eliqiao has quit IRC | 19:04 | |
| *** eliqiao has joined #openstack-nova | 19:04 | |
| mriedem | that's not going to work right no | 19:05 |
| mriedem | *now | 19:05 |
| ildikov | mriedem: sweet, thank you! | 19:05 |
| scottda | ildikov: Just tested using cinder v3 in nova.conf and it works for me. | 19:06 |
| openstackgerrit | Matt Riedemann proposed openstack/nova: WIP Get Cinder v3 microversion https://review.openstack.org/385682 | 19:07 |
| mriedem | ok well ^ should see if things work in a tempest run | 19:07 |
| openstackgerrit | Dan Smith proposed openstack/nova: Multicell support for instance listing https://review.openstack.org/396775 | 19:08 |
| openstackgerrit | Dan Smith proposed openstack/nova: Fix non-parameterized service id in hypervisors sample tests https://review.openstack.org/410471 | 19:08 |
| openstackgerrit | Dan Smith proposed openstack/nova: Make the SingleCellSimple fixture a little more comprehensive https://review.openstack.org/410472 | 19:08 |
| * dansmith plays https://www.youtube.com/watch?v=sil76t2X_DE and waits | 19:09 | |
| *** rcernin has joined #openstack-nova | 19:11 | |
| mriedem | speaking of, | 19:12 |
| mriedem | i always thought they said 'fucked up music' in https://www.youtube.com/watch?v=HV3zWSawJiw | 19:12 |
| mriedem | is that just me? | 19:12 |
| mriedem | i'm pretty sure it's 'funked up' | 19:13 |
| *** eliqiao has quit IRC | 19:13 | |
| * mriedem fast forwards to the sax solo | 19:14 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/nova: Updated from global requirements https://review.openstack.org/415978 | 19:14 |
| ildikov | scottda: thanks, I will reinstall my Devstack then and see | 19:14 |
| *** romainh1 has quit IRC | 19:15 | |
| *** harlowja has joined #openstack-nova | 19:16 | |
| ildikov | mriedem: does not really sound 'funked up' to me, but maybe I'm not native enough to hear it especially after having a hint of what I should hear instead :) | 19:17 |
| mriedem | heh | 19:17 |
| mriedem | well, | 19:17 |
| mriedem | i wouldn't go out tonight, | 19:17 |
| *** dharinic|lunch is now known as dharinic | 19:17 | |
| mriedem | there is a bathroom on the right | 19:17 |
| *** sneti has joined #openstack-nova | 19:18 | |
| *** yufei has quit IRC | 19:18 | |
| mriedem | https://www.youtube.com/watch?v=ExViB7ArQNA | 19:18 |
| *** lpetrut has joined #openstack-nova | 19:19 | |
| ildikov | LOL :) | 19:19 |
| *** sudipto_ has quit IRC | 19:21 | |
| *** sudipto has quit IRC | 19:21 | |
| *** voelzmo has joined #openstack-nova | 19:21 | |
| openstackgerrit | Ildiko Vancsa proposed openstack/nova: Implement new attach/detach Cinder flow https://review.openstack.org/330285 | 19:22 |
| openstackgerrit | Ildiko Vancsa proposed openstack/nova: Remove check_attach https://review.openstack.org/335358 | 19:22 |
| *** Guest50714 has quit IRC | 19:22 | |
| *** cdent has joined #openstack-nova | 19:22 | |
| ildikov | mriedem: BTW, as you don't seem to go out tonight, if you would happen to have time to check the Remove check_attach patch ^^ that would be great :) | 19:25 |
| openstackgerrit | Sujitha proposed openstack/nova: Add post hook script to run tempest with old n-api https://review.openstack.org/401395 | 19:25 |
| mriedem | my dance card is full all of a sudden | 19:26 |
| * dansmith heads out to lunch while his dance card still has room | 19:27 | |
| ildikov | dansmith: enjoy lunch :) | 19:31 |
| *** yamahata has quit IRC | 19:32 | |
| *** sneti has quit IRC | 19:32 | |
| openstackgerrit | Matt Riedemann proposed openstack/nova: Return uuid attribute for aggregates https://review.openstack.org/415031 | 19:33 |
| mriedem | jaypipes: ok i think the change here ^ tightens up the api sample test validation a bit | 19:33 |
| *** emerson has joined #openstack-nova | 19:34 | |
| *** unicell has quit IRC | 19:37 | |
| *** gouthamr has quit IRC | 19:39 | |
| *** gouthamr has joined #openstack-nova | 19:40 | |
| *** romainh has joined #openstack-nova | 19:40 | |
| cdent | macsz: is there code up for review for this bug or are you just indicating it is in progress because there is code in progress on someone's computer? https://bugs.launchpad.net/nova/+bug/1632852 | 19:40 |
| openstack | Launchpad bug 1632852 in OpenStack Compute (nova) "placement api responses should not be cacehable" [Medium,In progress] - Assigned to Prateek Arora (parora) | 19:40 |
| *** salv-orl_ has quit IRC | 19:41 | |
| macsz | cdent: there is an assignee, the policy is that if someone is assigned the status should be 'In progress'. Normally I would question if someone is still working on it, but Prateek was assigned just a month ago, and there were Christmas on the way, so effectively it leaves like two weeks of work | 19:43 |
| cdent | macsz: Secure, was just curious; wondering if maybe it was up for review but the link to gerrit didn't get in there for some reason. If so, I wanted to look :) | 19:44 |
| macsz | cdent: I would probably make it to this bug again next week or two and ask for a status | 19:44 |
| macsz | cdent: cool :) | 19:44 |
| cdent | s/Secure/Sure/ (I have no idea what I was thinking there ... ) | 19:45 |
| macsz | well, secureness cheered me up for a moment :D | 19:46 |
| cdent | :) | 19:48 |
| *** abalutoiu has quit IRC | 19:50 | |
| *** romainh1 has joined #openstack-nova | 19:51 | |
| *** rami_ has joined #openstack-nova | 19:51 | |
| *** sean-k-mooney is now known as sean-k-mooneyAFK | 19:51 | |
| *** rami_ has left #openstack-nova | 19:51 | |
| *** romainh has quit IRC | 19:51 | |
| *** cleong has quit IRC | 19:53 | |
| *** Swami has quit IRC | 19:54 | |
| *** sneti has joined #openstack-nova | 19:55 | |
| cdent | edleafe: did you achieve satisfaction? | 19:56 |
| mriedem | jaypipes: cdent: ok comments in https://review.openstack.org/#/c/407742/ | 19:56 |
| *** burt has joined #openstack-nova | 19:56 | |
| *** mtanino has joined #openstack-nova | 19:57 | |
| cdent | thanks mriedem will chew on that | 19:57 |
| cdent | yeah that all makes sense, will dig | 19:58 |
| mriedem | hmm http://docs.sqlalchemy.org/en/latest/core/sqlelement.html#sqlalchemy.sql.operators.ColumnOperators.in_ | 19:59 |
| mriedem | str is a tuple right? | 19:59 |
| *** markvoelker has quit IRC | 19:59 | |
| mriedem | that would be a pretty gd sneaky bug | 19:59 |
| *** markvoelker has joined #openstack-nova | 19:59 | |
| * cdent starts by adding the missing test | 20:00 | |
| *** rfolco has quit IRC | 20:01 | |
| *** marst_ has joined #openstack-nova | 20:01 | |
| *** cleong has joined #openstack-nova | 20:02 | |
| *** bvanhav_ is now known as bvanhav | 20:02 | |
| *** Sukhdev has joined #openstack-nova | 20:02 | |
| *** marst has quit IRC | 20:05 | |
| cdent | mriedem: I reckon the thing to do is to just special case member_of for 'in:'. The filter code is only doing the in_ sql call on member_of | 20:06 |
| cdent | so we just need to make sure that's a list (even if it is just one) | 20:06 |
| openstackgerrit | Sujitha proposed openstack/nova: Add post hook script to run tempest with old n-api https://review.openstack.org/401395 | 20:07 |
| *** rfolco has joined #openstack-nova | 20:07 | |
| *** rfolco has quit IRC | 20:07 | |
| *** yamahata has joined #openstack-nova | 20:10 | |
| mordred | sdague: we're still heading towards not having project_id in the catalog endpoint for nova, yeah? | 20:10 |
| sdague | mordred: yeh, you can do that now | 20:11 |
| sdague | we've been testing that way for about a year | 20:11 |
| mordred | sdague: cool - so lack of project_id in the url just means "use the project from the currently scoped user" ? | 20:12 |
| mordred | s/user/token/ | 20:12 |
| mriedem | cdent: that's what i was thinking | 20:12 |
| sdague | mordred: yes | 20:12 |
| sdague | mordred: which is actually the way the code always worked | 20:12 |
| cdent | ✔ | 20:12 |
| mriedem | cdent: we could also ask zzzeek but to be safe it's probably safest to just always make it a list | 20:12 |
| openstackgerrit | Sarafraj Singh proposed openstack/nova: WIP: Add service_token for nova-cinder interaction https://review.openstack.org/397399 | 20:13 |
| mriedem | did i mention that safety is very important :) | 20:13 |
| sdague | the project id in the url was nothing more than belt and suspenders since I've been on the project | 20:13 |
| *** voelzmo has quit IRC | 20:13 | |
| sdague | mriedem / dims - something just seems to have started melting down on python3 unit tests for nova - http://status.openstack.org/zuul/ | 20:14 |
| sdague | on gate queue | 20:14 |
| sdague | http://logs.openstack.org/23/403423/9/gate/gate-nova-python35-db/60675e6/console.html#_2017-01-05_19_02_54_521661 | 20:14 |
| mordred | sdague: ah! neat. good to know | 20:16 |
| mordred | sdague: if I am an admin with access rights and I add a project id to the url, that should show me things in that project? | 20:16 |
| sdague | no | 20:17 |
| sdague | it's... weird, and we all had to read that code a bunch to convince ourselves that wasn't working that way | 20:18 |
| *** voelzmo has joined #openstack-nova | 20:18 | |
| sdague | but basically if project_id is in the url, it is checked against token project_id, if they aren't the same, 403 | 20:19 |
| sdague | the token project_id is always used | 20:19 |
| sdague | some resources also let you specify project_id as a param or all_tenants, and then if you are in the policy to have access to those things (i.e. is_admin) you do | 20:19 |
| mordred | sdague: thanks. super helpful. SO - I'm asking all this because I'm reviewing a change to add support for reading limits which means I'm learning about it and stuff ... and the API docs do not mention /limits taking any parameters - but python-novaclient (which I just now looked at) seems to be able to pass tenant_id and restricted as parameters | 20:22 |
| mordred | sdague: should I just assume that's a docs bug I shoudl file? | 20:22 |
| mriedem | sdague: we removed cinder v1 support from nova | 20:23 |
| mriedem | sdague: are you sure that wasn't just a gate hiccup | 20:23 |
| mriedem | ? | 20:23 |
| mriedem | like that change sliding in before some other things | 20:23 |
| mriedem | sdague: https://review.openstack.org/#/c/409904/ | 20:23 |
| sdague | mriedem: 2 changes in a row bounced like that | 20:23 |
| *** ayogi has quit IRC | 20:23 | |
| sdague | mordred: that's probably a docs bug | 20:24 |
| mordred | sdague: cool. thanks! | 20:24 |
| mriedem | sdague: oh it's a new test as part of that change https://review.openstack.org/#/c/409904/2/nova/tests/unit/test_cinder.py@124 | 20:24 |
| mriedem | so probably a busted global mock or something? | 20:25 |
| *** unicell has joined #openstack-nova | 20:25 | |
| sdague | mordred: https://github.com/openstack/nova/blob/1ccdfa4240d67d8f91273ec36ad8d7292aecd0f2/nova/api/openstack/wsgi.py#L644-L652 | 20:26 |
| openstackgerrit | Chris Dent proposed openstack/nova: HTTP interface for resource providers by aggregates https://review.openstack.org/407742 | 20:27 |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/nova: Updated from global requirements https://review.openstack.org/415978 | 20:27 |
| *** ducttape_ has quit IRC | 20:28 | |
| *** ducttape_ has joined #openstack-nova | 20:28 | |
| mordred | sdague: awesome | 20:28 |
| *** bvanhav has quit IRC | 20:30 | |
| cdent | macsz: I've responded to your question on that review about the typo. We should fix that pretty much now | 20:31 |
| cdent | macsz: I guess I never saw that because it didn't have the placement tag, thanks for adding it | 20:31 |
| cdent | I'll go ahead and fix it right now. | 20:32 |
| macsz | cdent: np | 20:32 |
| macsz | cdent: ok | 20:32 |
| mriedem | cdent: looks good, just waiting on test results | 20:33 |
| *** ducttape_ has quit IRC | 20:34 | |
| *** ducttape_ has joined #openstack-nova | 20:34 | |
| *** unicell has quit IRC | 20:37 | |
| openstackgerrit | Chris Dent proposed openstack/nova: [placement] fix typo in call to create auth middleware https://review.openstack.org/417178 | 20:38 |
| *** tblakes has joined #openstack-nova | 20:41 | |
| *** dtp_ has joined #openstack-nova | 20:44 | |
| *** catintheroof has quit IRC | 20:45 | |
| *** dtp has quit IRC | 20:47 | |
| *** roeyc has quit IRC | 20:48 | |
| *** roeyc has joined #openstack-nova | 20:50 | |
| mriedem | nova meeting in 7 minutes | 20:53 |
| mriedem | bauzas: ^ fyi just for you | 20:53 |
| *** takashin has joined #openstack-nova | 20:53 | |
| openstackgerrit | Ed Leafe proposed openstack/nova: Remove the Allocation.create() method. https://review.openstack.org/416751 | 20:53 |
| openstackgerrit | Ed Leafe proposed openstack/nova: Add check for invalid allocation amounts https://review.openstack.org/416752 | 20:53 |
| *** eharney has quit IRC | 20:53 | |
| mriedem | sdague: yeah damn i just had something fail in check http://logs.openstack.org/18/417118/1/check/gate-nova-python27-db-ubuntu-xenial/0c0c8c1/console.html#_2017-01-05_17_52_16_199447 | 20:53 |
| mriedem | that test must be busted | 20:53 |
| *** dtp_ is now known as dtp | 20:54 | |
| dansmith | I think I had that fail earlier today too | 20:54 |
| mriedem | yeah opening a bug | 20:55 |
| mriedem | damn | 20:55 |
| dansmith | we should probably make the nova meeting short so you can get back to looking at that | 20:56 |
| mriedem | gonna make it extra long | 20:57 |
| mriedem | just for you | 20:57 |
| dansmith | >:| | 20:57 |
| mriedem | https://bugs.launchpad.net/nova/+bug/1654393 | 20:57 |
| openstack | Launchpad bug 1654393 in OpenStack Compute (nova) ""keystoneauth1.exceptions.catalog.EmptyCatalog: The service catalog is empty." in test_cinderclient_unsupported_v1" [High,Confirmed] | 20:57 |
| *** satyar has quit IRC | 20:59 | |
| bauzas | mriedem: <3 | 21:01 |
| mriedem | ok e-r query patch is up | 21:01 |
| bauzas | but this time, my google agenda worked :) | 21:01 |
| *** voelzmo has quit IRC | 21:02 | |
| *** mlavalle has quit IRC | 21:07 | |
| *** voelzmo has joined #openstack-nova | 21:07 | |
| *** salv-orlando has joined #openstack-nova | 21:14 | |
| *** cdelatte has quit IRC | 21:19 | |
| *** _ducttape_ has joined #openstack-nova | 21:21 | |
| *** armax has joined #openstack-nova | 21:22 | |
| *** ducttape_ has quit IRC | 21:22 | |
| *** JoseMello has quit IRC | 21:25 | |
| *** voelzmo has quit IRC | 21:29 | |
| *** smatzek has quit IRC | 21:29 | |
| openstackgerrit | Sujitha proposed openstack/nova: Add post hook script to run tempest with old n-api https://review.openstack.org/401395 | 21:29 |
| *** Jeffrey4l_ has quit IRC | 21:35 | |
| *** takashin has left #openstack-nova | 21:36 | |
| bauzas | cdent: bonus point for the HAS-A vs. IS-A comment | 21:36 |
| bauzas | very good point | 21:37 |
| bauzas | I was just lazy | 21:37 |
| cdent | it's not a deal breaker or anything, just seemed...tidier? | 21:37 |
| openstackgerrit | Balazs Gibizer proposed openstack/nova: add availability_zone field in service.update https://review.openstack.org/415857 | 21:37 |
| *** cjvolzka has left #openstack-nova | 21:37 | |
| *** takashin has joined #openstack-nova | 21:38 | |
| bauzas | cdent: well, I think it's better to have a self.placement_client instead of a super().__init__() | 21:39 |
| * cdent nods | 21:39 | |
| *** tovin07 has joined #openstack-nova | 21:39 | |
| *** eharney has joined #openstack-nova | 21:41 | |
| *** tovin07 has left #openstack-nova | 21:42 | |
| cdent | I think my eyes are going to fall out, time to leave the puter. | 21:43 |
| * cdent waves | 21:43 | |
| mriedem | o/ | 21:43 |
| *** cdent has quit IRC | 21:43 | |
| *** dave-mccowan has quit IRC | 21:47 | |
| *** chyka has joined #openstack-nova | 21:47 | |
| *** Jeffrey4l_ has joined #openstack-nova | 21:48 | |
| *** dimtruck is now known as zz_dimtruck | 21:50 | |
| *** liverpooler has quit IRC | 21:51 | |
| dtp | what are the valid cell_types for cells v2? | 21:51 |
| *** salv-orl_ has joined #openstack-nova | 21:53 | |
| *** cleong has quit IRC | 21:54 | |
| mriedem | dtp: that doesn't apply for cells v2 | 21:55 |
| mriedem | well, it's not used i don't think | 21:55 |
| mriedem | e.g. api vs compute | 21:55 |
| *** salv-orlando has quit IRC | 21:56 | |
| openstackgerrit | Matt Riedemann proposed openstack/nova: Fix ksa mocking in test_cinderclient_unsupported_v1 https://review.openstack.org/417206 | 21:58 |
| mriedem | sdague: here's to hoping this fixes the unit test race bug ^ | 21:58 |
| dansmith | mriedem: dtp right, no cell types in v2 | 22:01 |
| *** zz_dimtruck is now known as dimtruck | 22:01 | |
| dansmith | any reference to a cell type is cellsv1 code | 22:02 |
| melwitt | mriedem, dtp: yeah, it's not used in cells v2. it's just that we're trying to determine how to avoid cells v1 running a piece of code in compute cells. and there's no way to tell that from the cell_type option because it defaults to 'compute' | 22:03 |
| melwitt | if we say "cell_type != compute" then it would prevent the code running in cells v2 also | 22:03 |
| *** thorst_ has quit IRC | 22:04 | |
| dtp | i was confused as to what code path would lead to https://github.com/openstack/nova/blob/master/nova/compute/api.py#L1815 being executed in a cell vs the top level | 22:04 |
| melwitt | we want it to run 1) always in cells v2 2) only for the api cell in cells v1 | 22:04 |
| mriedem | so, if CONF.cells.enable and CONF.cells.cell_type == 'api' ? | 22:05 |
| mriedem | cells.enable will only be True for cells v1 | 22:05 |
| melwitt | mriedem: oooh, that sounds like it would work | 22:05 |
| openstackgerrit | Matt Riedemann proposed openstack/nova: Fix ksa mocking in test_cinderclient_unsupported_v1 https://review.openstack.org/417206 | 22:05 |
| mriedem | ok i think *that* ^ fixes that mock issue | 22:05 |
| mriedem | based on https://docs.python.org/3/library/unittest.mock.html#id5 | 22:05 |
| melwitt | dtp: yeah, it's complicated. and one of many reasons we are doing cells v2 and will get rid of cells v1 | 22:05 |
| mriedem | mocking statics from imported libraries is a mess | 22:05 |
| smcginnis | mriedem: I seem to remember puzzling over that as well at some point. | 22:07 |
| *** darrenc is now known as darrenc_afk | 22:07 | |
| smcginnis | mriedem: Looks "more correct" now. | 22:07 |
| mriedem | i just 'feels' right | 22:07 |
| smcginnis | :) | 22:07 |
| mriedem | i code like trump governs | 22:07 |
| mriedem | with the gut | 22:07 |
| mriedem | well, or just not at all | 22:08 |
| smcginnis | It'll be huuge. | 22:08 |
| mriedem | i'll tweet about this bug and shame someone else into fixing it for me | 22:08 |
| mriedem | but take all of the credit when it's fixed | 22:08 |
| smcginnis | hehe | 22:08 |
| mriedem | i mean, his 10 year old son can use a computer, so it can't be that hard | 22:08 |
| jroll | mriedem: that's how we know you're a ptl | 22:08 |
| smcginnis | LOL | 22:09 |
| dtp | melwitt so you're thinking if ! cells.enable, delete the tokens? | 22:09 |
| melwitt | dtp: if not cells.enable or cells.enable and cell_type == 'api'. I think. I'm looking through the code reminding myself how/when replay in a compute cell happens | 22:12 |
| dansmith | melwitt: does it hurt to just run it always? | 22:12 |
| *** dimtruck is now known as zz_dimtruck | 22:13 | |
| dansmith | melwitt: if we ignore something missing we don't expect, we can always nuke I would think | 22:13 |
| dansmith | when in doubt, throw it out | 22:13 |
| melwitt | dansmith: it would send an rpc message out to nowhere, I guess | 22:13 |
| dansmith | if it might crush, give it a flush | 22:13 |
| melwitt | if it's a call, it would time out. but I'm guessing it's a cast | 22:14 |
| dansmith | if it's a cast, let it pass | 22:14 |
| melwitt | lol. I was wondering if you'd have something for that | 22:14 |
| dansmith | if you wonder, let it plunder | 22:14 |
| melwitt | oh boy ... | 22:15 |
| dtp | but we still don't want this running in...whatever compute cells are called now? | 22:15 |
| dansmith | dtp: it won't, that's api-only code | 22:15 |
| dtp | oh, great | 22:15 |
| dansmith | that's theconfusing part of cellsv1, | 22:15 |
| dansmith | the api code is re-played in the compute cell | 22:15 |
| dansmith | well, one of the confusing parts | 22:15 |
| dtp | gotcha | 22:15 |
| dtp | thanks, this helped | 22:16 |
| melwitt | dansmith: I was just getting around to that. why is there a "if cell_type == 'api'" in there if it doesn't replay it? from what I see in nova/compute/cells_api.py it doesn't cast down to cells for delete | 22:16 |
| dansmith | melwitt: I mean cellsv2 doesn't replay | 22:17 |
| melwitt | dansmith: yeah, I'm looking in cells v1 and it looks like delete doesn't replay in a compute cell compute/api | 22:17 |
| melwitt | guh | 22:17 |
| dtp | that's what i was seeing as well | 22:17 |
| dansmith | I'm lost | 22:17 |
| melwitt | so I was wondering, why is there a "if cell_type == 'api'" in compute/api in the first place. maybe residue | 22:18 |
| melwitt | or rather, it's really just saying "if this is cells v1, do this" | 22:18 |
| dansmith | we definitely have to delete in both places in cellsv1 right? | 22:18 |
| dansmith | I think we call delete in the compute cell and then it's replicated backwards in that case right? | 22:19 |
| dansmith | the opposite of most calls | 22:19 |
| melwitt | dansmith: from the code, it looks like it doesn't re-run the compute/api part in compute cells, is what I was observing. the use of the conditional made me think it was | 22:19 |
| dansmith | melwitt: but the comment says it does right? | 22:19 |
| melwitt | dansmith: it does. maybe it used to? unless I'm blind as I look in cells_api.py | 22:20 |
| dansmith | well, we force a local delete in the api, | 22:21 |
| melwitt | looks like the delete_everywhere will do a replay | 22:22 |
| dansmith | then call direct to compute via rpc in there | 22:22 |
| melwitt | but not the normal delete | 22:22 |
| dansmith | ah | 22:22 |
| melwitt | gawd | 22:22 |
| * dansmith gouges out his eyes and hands one each to melwitt and dtp | 22:22 | |
| melwitt | lol! | 22:23 |
| dtp | :) so summary: | 22:23 |
| dtp | no cell types in v2. ignore that if in there. just delete the tokens. | 22:23 |
| dtp | with the conditional melwitt outlined above | 22:23 |
| dtp | why even have the conditional if cells v1 is gone in ocata? | 22:23 |
| dansmith | I would prefer it if we can avoid the conditional, but if we can't then yes | 22:24 |
| dansmith | dtp: unfortunately cellsv1 will be with us for longer than that | 22:24 |
| dansmith | dtp: until cellsv2 is a replacement and we have a migration path | 22:24 |
| openstackgerrit | Ed Leafe proposed openstack/nova: Remove the Allocation.create() method. https://review.openstack.org/416751 | 22:24 |
| melwitt | dtp: we can technically avoid the conditional because the delete_tokens rpc api is a cast i.e. broadcast that doesn't wait for a response. so it could just get sent anyway even if nothing will be there to listen to it | 22:25 |
| *** darrenc_afk is now known as darrenc | 22:25 | |
| dansmith | melwitt: what won't be there to listen for it? | 22:25 |
| dansmith | melwitt: the instance having moved? | 22:25 |
| melwitt | dansmith: the consoleauth service in compute cells | 22:25 |
| dansmith | oh waiut | 22:25 |
| dansmith | right, right | 22:25 |
| dansmith | okay then you should do the conditional | 22:26 |
| dansmith | IMHO of course | 22:26 |
| *** baoli has quit IRC | 22:26 | |
| melwitt | yeah, I hadn't thought of the possibility of just letting it run in compute cells and it not hurting anything | 22:27 |
| *** nic has quit IRC | 22:27 | |
| dansmith | dtp: so, all good? | 22:28 |
| melwitt | dansmith: wait, did you say do the conditional or did you mean don't do it? | 22:29 |
| dtp | i think so. thanks y'all | 22:29 |
| dansmith | melwitt: I mean _do_ the conditional so we're not just firing off a message into a queue that we know is unlistened to | 22:29 |
| melwitt | yeah, okay | 22:29 |
| dansmith | melwitt: I was originally thinking the reason for the conditional was something else | 22:29 |
| melwitt | I was thinking you said the opposite | 22:29 |
| melwitt | dtp: so yes on the conditional | 22:30 |
| dtp | roger that | 22:30 |
| dansmith | melwitt: remember when we told dtp this would be all easysauce? good times. | 22:31 |
| melwitt | dansmith: yeah, I know. I keep wanting to apologize to him | 22:31 |
| melwitt | sorry dtp | 22:31 |
| dtp | hehe | 22:31 |
| dansmith | it's like telling the driver to get into the other lane because it's faster and then getting stuck behind an accident or something | 22:31 |
| dansmith | should'a kept quiet | 22:32 |
| melwitt | haha | 22:32 |
| *** jheroux has quit IRC | 22:32 | |
| *** salv-orl_ has quit IRC | 22:33 | |
| dtp | you can make it up to be by having a look at https://review.openstack.org/#/c/336293/ :D | 22:36 |
| *** salv-orlando has joined #openstack-nova | 22:36 | |
| *** dave-mccowan has joined #openstack-nova | 22:37 | |
| *** jamesdenton has quit IRC | 22:37 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/nova: Updated from global requirements https://review.openstack.org/415978 | 22:40 |
| melwitt | dtp: hah, okay. hm, new config option | 22:40 |
| *** sreeram_v has joined #openstack-nova | 22:41 | |
| dtp | yes :) it is a very, very exciting config option | 22:42 |
| mriedem | yes i've already passively aggressively said this is a blueprint | 22:43 |
| dtp | oh, sorry mriedem. should i create a blueprint? | 22:44 |
| mriedem | well, see, | 22:44 |
| mriedem | the thing i'm really going to say is this would be for pike | 22:44 |
| mriedem | w/o wanting to hurt your feelings | 22:44 |
| mriedem | that may be fine for you guys as i know godaddy would already be running with this patch and you're just upstreaming it | 22:45 |
| *** zz_dimtruck is now known as dimtruck | 22:45 | |
| dtp | yes | 22:45 |
| *** sreeram_v has quit IRC | 22:45 | |
| *** _ducttape_ has quit IRC | 22:46 | |
| dtp | so a pike blueprint? | 22:47 |
| mriedem | yeah | 22:48 |
| dims | mriedem : sdague : dansmith : i got a heat integration test going (devstack+python3.5), since it exercises nova, we can spot what we need to fix in nova too - http://logs.openstack.org/86/412486/17/check/gate-heat-dsvm-functional-convg-mysql-lbaasv2-ubuntu-xenial/960bacc/logs/screen-n-cpu.txt.gz | 22:48 |
| mriedem | https://blueprints.launchpad.net/nova/pike | 22:48 |
| *** dave-mccowan has quit IRC | 22:48 | |
| dtp | thanks | 22:48 |
| mriedem | dims: you know we don't have to officially care about py3 until pike right? :P | 22:48 |
| dims | mriedem : haha, true, giving you a way to figure out, so it can proceed at a leisurely pace :) | 22:49 |
| openstackgerrit | Derek Higgins proposed openstack/nova: libvirt: fix nova can't delete the instance with nvram https://review.openstack.org/357190 | 22:49 |
| openstackgerrit | Derek Higgins proposed openstack/nova: Remove mox in libvirt destory tests https://review.openstack.org/417227 | 22:49 |
| mriedem | i'm going to lean on you for all of this you know | 22:49 |
| mriedem | you and gcb | 22:49 |
| *** edmondsw has quit IRC | 22:51 | |
| dims | mriedem : ack. | 22:51 |
| dims | gcb : look there and start fixing :) | 22:52 |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/nova: Updated from global requirements https://review.openstack.org/415978 | 22:52 |
| *** romainh1 has quit IRC | 22:53 | |
| mriedem | dansmith: melwitt: before i head out, https://review.openstack.org/#/c/417206/ should fix the current CI failure mess | 22:53 |
| mriedem | ooo looky there top check queue failure http://status.openstack.org//elastic-recheck/index.html | 22:53 |
| * mriedem heads out | 22:54 | |
| *** mriedem has quit IRC | 22:54 | |
| openstackgerrit | Samantha Blanco proposed openstack/nova: Add missing CLI commands in support-matrix.ini https://review.openstack.org/417232 | 22:57 |
| *** lpetrut has quit IRC | 22:58 | |
| *** marst_ has quit IRC | 22:59 | |
| *** claudiub|2 has joined #openstack-nova | 23:01 | |
| *** claudiub has quit IRC | 23:01 | |
| *** xyang1 has quit IRC | 23:06 | |
| openstackgerrit | Anusha Unnam proposed openstack/nova: Add post hook script to run tempest with old n-api https://review.openstack.org/401395 | 23:08 |
| *** sneti has quit IRC | 23:13 | |
| *** eandersson has joined #openstack-nova | 23:17 | |
| openstackgerrit | Takashi NATSUME proposed openstack/nova: api-ref: Fix a parameter in os-availability-zone.inc https://review.openstack.org/417233 | 23:19 |
| *** gouthamr has quit IRC | 23:21 | |
| *** Sukhdev has quit IRC | 23:22 | |
| *** gouthamr has joined #openstack-nova | 23:22 | |
| *** dtp has quit IRC | 23:28 | |
| *** dtp has joined #openstack-nova | 23:29 | |
| *** cjvolzka_ has quit IRC | 23:30 | |
| *** tblakes has quit IRC | 23:31 | |
| *** annegentle has quit IRC | 23:32 | |
| *** unicell has joined #openstack-nova | 23:34 | |
| *** unicell has quit IRC | 23:36 | |
| *** unicell has joined #openstack-nova | 23:42 | |
| *** salv-orlando has quit IRC | 23:45 | |
| *** ducttape_ has joined #openstack-nova | 23:48 | |
| *** unicell1 has joined #openstack-nova | 23:48 | |
| *** unicell has quit IRC | 23:49 | |
| *** claudiub|2 has quit IRC | 23:49 | |
| *** mdrabe has quit IRC | 23:50 | |
| *** ducttape_ has quit IRC | 23:53 | |
| *** Sukhdev has joined #openstack-nova | 23:53 | |
| *** baoli has joined #openstack-nova | 23:56 | |
| *** mvk has quit IRC | 23:57 | |
| *** jwcroppe has quit IRC | 23:58 | |
| *** jwcroppe has joined #openstack-nova | 23:59 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!