| *** itlinux has joined #openstack-nova | 00:03 | |
| openstackgerrit | Peter Penchev proposed openstack/nova master: Add the Nova libvirt StorPool attachment driver. https://review.openstack.org/140733 | 00:08 |
|---|---|---|
| *** thorst has joined #openstack-nova | 00:16 | |
| *** yangyapeng has quit IRC | 00:16 | |
| *** yangyapeng has joined #openstack-nova | 00:16 | |
| *** sree has joined #openstack-nova | 00:17 | |
| openstackgerrit | Takashi NATSUME proposed openstack/nova master: [placement] Add x-openstack-request-id in API ref https://review.openstack.org/523007 | 00:19 |
| *** yangyapeng has quit IRC | 00:21 | |
| *** thorst has quit IRC | 00:21 | |
| *** Oku_OS-away has joined #openstack-nova | 00:22 | |
| *** sree has quit IRC | 00:22 | |
| *** yassine has joined #openstack-nova | 00:29 | |
| *** yassine is now known as Guest68950 | 00:30 | |
| *** david-lyle has joined #openstack-nova | 00:30 | |
| *** david-lyle has quit IRC | 00:31 | |
| *** david-lyle has joined #openstack-nova | 00:31 | |
| *** cheneydc has joined #openstack-nova | 00:33 | |
| *** salv-orlando has joined #openstack-nova | 00:44 | |
| *** jmlowe has joined #openstack-nova | 00:47 | |
| *** salv-orlando has quit IRC | 00:49 | |
| *** thorst has joined #openstack-nova | 00:52 | |
| *** huanxie has quit IRC | 00:54 | |
| *** thorst has quit IRC | 00:57 | |
| *** yangyapeng has joined #openstack-nova | 00:58 | |
| *** slaweq has joined #openstack-nova | 00:59 | |
| *** jichen has joined #openstack-nova | 00:59 | |
| *** psachin has joined #openstack-nova | 01:00 | |
| *** yangyapeng has quit IRC | 01:03 | |
| *** slaweq has quit IRC | 01:03 | |
| *** phuongnh has joined #openstack-nova | 01:06 | |
| *** yangyapeng has joined #openstack-nova | 01:08 | |
| *** trungnv has joined #openstack-nova | 01:17 | |
| *** edmondsw has joined #openstack-nova | 01:19 | |
| openstackgerrit | Yikun Jiang (Kero) proposed openstack/nova master: Add cross cell sort support for get_migrations https://review.openstack.org/517273 | 01:21 |
| *** edmondsw has quit IRC | 01:24 | |
| *** tetsuro has joined #openstack-nova | 01:25 | |
| *** annp has quit IRC | 01:27 | |
| *** thorst has joined #openstack-nova | 01:27 | |
| *** yangyapeng has quit IRC | 01:27 | |
| *** yangyapeng has joined #openstack-nova | 01:28 | |
| *** yamahata has quit IRC | 01:28 | |
| *** thorst has quit IRC | 01:28 | |
| *** thorst has joined #openstack-nova | 01:29 | |
| *** thorst has quit IRC | 01:30 | |
| *** claudiub has quit IRC | 01:35 | |
| *** jmlowe has quit IRC | 01:44 | |
| *** salv-orlando has joined #openstack-nova | 01:45 | |
| *** yamamoto has joined #openstack-nova | 01:45 | |
| *** salv-orlando has quit IRC | 01:49 | |
| *** ankit_ has joined #openstack-nova | 01:49 | |
| openstackgerrit | Takashi NATSUME proposed openstack/python-novaclient master: Microversion 2.56 - Enable cold migration with target host https://review.openstack.org/406707 | 01:49 |
| *** Yingxin has joined #openstack-nova | 01:52 | |
| openstackgerrit | Zhenyu Zheng proposed openstack/nova master: Modify _poll_shelved_instances periodic task call _shelve_offload_instance() https://review.openstack.org/524047 | 01:53 |
| *** gmann_afk is now known as gmann | 01:54 | |
| *** thorst has joined #openstack-nova | 02:01 | |
| *** thorst has quit IRC | 02:06 | |
| *** lei-zh has joined #openstack-nova | 02:06 | |
| *** sree has joined #openstack-nova | 02:08 | |
| openstackgerrit | Yikun Jiang (Kero) proposed openstack/nova master: Update Instance action's updated_at when action event updated. https://review.openstack.org/507473 | 02:08 |
| *** sree_ has joined #openstack-nova | 02:10 | |
| *** thorst has joined #openstack-nova | 02:10 | |
| *** sree_ is now known as Guest25159 | 02:10 | |
| *** Guest25159 has quit IRC | 02:12 | |
| *** sree has quit IRC | 02:14 | |
| openstackgerrit | Zhenyu Zheng proposed openstack/nova master: Add instance action record for shelve_offload instances https://review.openstack.org/523653 | 02:14 |
| *** thorst has quit IRC | 02:14 | |
| *** phuongnh has quit IRC | 02:15 | |
| *** phuongnh has joined #openstack-nova | 02:16 | |
| *** gbarros has quit IRC | 02:23 | |
| *** Apoorva has quit IRC | 02:23 | |
| *** TuanLA has joined #openstack-nova | 02:27 | |
| openstackgerrit | Zhenyu Zheng proposed openstack/nova master: Add instance action record for backup instances https://review.openstack.org/523676 | 02:34 |
| openstackgerrit | Yikun Jiang (Kero) proposed openstack/nova master: Add pagination and changes-since for instance-actions https://review.openstack.org/326326 | 02:38 |
| *** salv-orlando has joined #openstack-nova | 02:45 | |
| *** salv-orlando has quit IRC | 02:50 | |
| *** sridharg has joined #openstack-nova | 02:51 | |
| *** edmondsw has joined #openstack-nova | 03:07 | |
| openstackgerrit | Yikun Jiang (Kero) proposed openstack/nova master: Genericify the instance_list stuff https://review.openstack.org/524253 | 03:09 |
| openstackgerrit | Yikun Jiang (Kero) proposed openstack/nova master: Split instance_list into instance and multi_cell https://review.openstack.org/524302 | 03:09 |
| openstackgerrit | Yikun Jiang (Kero) proposed openstack/nova master: Add migration db and object pagination support. https://review.openstack.org/514904 | 03:09 |
| openstackgerrit | Yikun Jiang (Kero) proposed openstack/nova master: Implement query param schema for migration index https://review.openstack.org/518644 | 03:09 |
| openstackgerrit | Yikun Jiang (Kero) proposed openstack/nova master: Add cross cell sort support for get_migrations https://review.openstack.org/517273 | 03:09 |
| openstackgerrit | Yikun Jiang (Kero) proposed openstack/nova master: Add pagination and Changes-since filter support for os-migrations. https://review.openstack.org/330406 | 03:09 |
| *** edmondsw has quit IRC | 03:12 | |
| *** cheneydc has quit IRC | 03:14 | |
| *** thorst has joined #openstack-nova | 03:17 | |
| *** sree has joined #openstack-nova | 03:20 | |
| *** thorst has quit IRC | 03:22 | |
| openstackgerrit | guanzuoyu proposed openstack/nova master: Use actual availability zone rather than original value https://review.openstack.org/525034 | 03:23 |
| openstackgerrit | Zhenyu Zheng proposed openstack/nova master: Mention API behavior change when over quota limit https://review.openstack.org/515631 | 03:29 |
| *** itlinux has quit IRC | 03:32 | |
| *** edmondsw has joined #openstack-nova | 03:32 | |
| *** edmondsw has quit IRC | 03:36 | |
| *** slaweq has joined #openstack-nova | 03:46 | |
| *** salv-orlando has joined #openstack-nova | 03:46 | |
| *** tetsuro has quit IRC | 03:49 | |
| *** slaweq has quit IRC | 03:50 | |
| *** salv-orlando has quit IRC | 03:51 | |
| *** yamamoto has quit IRC | 03:53 | |
| *** yamamoto has joined #openstack-nova | 03:55 | |
| *** thorst has joined #openstack-nova | 03:57 | |
| *** thorst has quit IRC | 04:02 | |
| *** annp has joined #openstack-nova | 04:10 | |
| *** links has joined #openstack-nova | 04:13 | |
| *** jmlowe has joined #openstack-nova | 04:17 | |
| *** yamahata has joined #openstack-nova | 04:21 | |
| *** sree has quit IRC | 04:23 | |
| *** udesale has joined #openstack-nova | 04:25 | |
| *** udesale has quit IRC | 04:25 | |
| *** sree has joined #openstack-nova | 04:25 | |
| *** udesale has joined #openstack-nova | 04:25 | |
| *** adisky_ has joined #openstack-nova | 04:31 | |
| *** thorst has joined #openstack-nova | 04:37 | |
| *** itlinux has joined #openstack-nova | 04:40 | |
| *** thorst has quit IRC | 04:41 | |
| *** lei-zh has quit IRC | 04:43 | |
| *** salv-orlando has joined #openstack-nova | 04:47 | |
| *** yamamoto_ has joined #openstack-nova | 04:47 | |
| *** yamamoto has quit IRC | 04:51 | |
| *** salv-orlando has quit IRC | 04:52 | |
| openstackgerrit | jichenjc proposed openstack/nova master: remove glance usage inside compute https://review.openstack.org/511392 | 04:53 |
| openstackgerrit | jichenjc proposed openstack/nova master: Remove direct usage of glance.generate_image_url https://review.openstack.org/511397 | 04:53 |
| *** edmondsw has joined #openstack-nova | 04:58 | |
| *** lei-zh has joined #openstack-nova | 05:01 | |
| *** jmlowe has quit IRC | 05:02 | |
| openstackgerrit | jichenjc proposed openstack/nova master: remove reserve_quota_delta https://review.openstack.org/521469 | 05:02 |
| openstackgerrit | jichenjc proposed openstack/nova master: WIP: Remove quota reserve/commit/rollback https://review.openstack.org/521470 | 05:02 |
| *** edmondsw has quit IRC | 05:03 | |
| *** janki has joined #openstack-nova | 05:05 | |
| openstackgerrit | jichenjc proposed openstack/nova master: Remove 'nova-manage host' and 'nova-manage agent' https://review.openstack.org/521837 | 05:07 |
| openstackgerrit | jichenjc proposed openstack/nova master: Remove 'nova-manage shell' command https://review.openstack.org/521835 | 05:07 |
| openstackgerrit | jichenjc proposed openstack/nova master: Remove 'nova-manage account' and 'nova-manage project' https://review.openstack.org/521833 | 05:07 |
| openstackgerrit | jichenjc proposed openstack/nova master: Remove 'nova-manage logs' command https://review.openstack.org/522133 | 05:07 |
| *** thorst has joined #openstack-nova | 05:10 | |
| *** Alex_Staf has joined #openstack-nova | 05:11 | |
| *** gcb has joined #openstack-nova | 05:11 | |
| *** thorst has quit IRC | 05:15 | |
| *** jaosorior has joined #openstack-nova | 05:16 | |
| *** Alex_Staf has quit IRC | 05:20 | |
| *** fragatina has quit IRC | 05:21 | |
| *** fragatina has joined #openstack-nova | 05:21 | |
| *** yikun has quit IRC | 05:26 | |
| *** itlinux has quit IRC | 05:28 | |
| *** anthonyper has quit IRC | 05:29 | |
| openstackgerrit | Chen Hanxiao proposed openstack/nova master: libvirt: validate scsi controller model https://review.openstack.org/525055 | 05:29 |
| *** fragatina has quit IRC | 05:30 | |
| *** zhurong has joined #openstack-nova | 05:31 | |
| *** andymccr has quit IRC | 05:32 | |
| *** andymccr has joined #openstack-nova | 05:33 | |
| *** anthonyper has joined #openstack-nova | 05:33 | |
| *** thorst has joined #openstack-nova | 05:37 | |
| *** anthonyper has quit IRC | 05:38 | |
| *** thorst has quit IRC | 05:42 | |
| *** anthonyper has joined #openstack-nova | 05:42 | |
| *** thorst has joined #openstack-nova | 05:45 | |
| *** ratailor has joined #openstack-nova | 05:46 | |
| *** salv-orlando has joined #openstack-nova | 05:48 | |
| *** thorst has quit IRC | 05:50 | |
| *** moshele has joined #openstack-nova | 05:51 | |
| *** hshiina has joined #openstack-nova | 05:51 | |
| *** salv-orlando has quit IRC | 05:53 | |
| *** clarkb has quit IRC | 06:02 | |
| *** gongysh has joined #openstack-nova | 06:15 | |
| *** thorst has joined #openstack-nova | 06:24 | |
| *** Tom-Tom has joined #openstack-nova | 06:26 | |
| *** thorst has quit IRC | 06:29 | |
| *** yamahata has quit IRC | 06:31 | |
| *** xinliang has quit IRC | 06:32 | |
| *** clarkb has joined #openstack-nova | 06:33 | |
| *** suresh12 has joined #openstack-nova | 06:36 | |
| *** salv-orlando has joined #openstack-nova | 06:40 | |
| *** slaweq has joined #openstack-nova | 06:41 | |
| *** xinliang has joined #openstack-nova | 06:44 | |
| openstackgerrit | Hironori Shiina proposed openstack/nova master: virt/ironic: Implement rescue and unrescue https://review.openstack.org/416487 | 06:44 |
| *** slaweq has quit IRC | 06:45 | |
| *** lajoskatona has joined #openstack-nova | 06:46 | |
| *** edmondsw has joined #openstack-nova | 06:46 | |
| *** edmondsw has quit IRC | 06:51 | |
| *** thorst has joined #openstack-nova | 06:57 | |
| *** zhurong has quit IRC | 06:57 | |
| *** trungnv has quit IRC | 06:59 | |
| *** pooja_jadhav has joined #openstack-nova | 07:00 | |
| *** thorst has quit IRC | 07:01 | |
| *** edand_ has joined #openstack-nova | 07:04 | |
| *** suresh12 has quit IRC | 07:08 | |
| *** ralonsoh has joined #openstack-nova | 07:13 | |
| *** dayou has quit IRC | 07:14 | |
| *** zhurong has joined #openstack-nova | 07:20 | |
| *** fragatina has joined #openstack-nova | 07:27 | |
| *** yikun has joined #openstack-nova | 07:28 | |
| *** sahid has joined #openstack-nova | 07:30 | |
| *** thorst has joined #openstack-nova | 07:36 | |
| *** thorst has quit IRC | 07:41 | |
| *** karthiks has joined #openstack-nova | 07:44 | |
| *** damien_r has quit IRC | 07:46 | |
| *** rcernin has quit IRC | 07:49 | |
| *** dayou has joined #openstack-nova | 08:02 | |
| openstackgerrit | Rodolfo Alonso Hernandez proposed openstack/os-vif master: Migrate from 'ip' commands to 'pyroute2' https://review.openstack.org/484386 | 08:03 |
| *** damien_r has joined #openstack-nova | 08:10 | |
| *** slaweq has joined #openstack-nova | 08:11 | |
| *** tesseract has joined #openstack-nova | 08:13 | |
| *** damien_r has quit IRC | 08:13 | |
| *** damien_r has joined #openstack-nova | 08:13 | |
| *** AlexeyAbashkin has joined #openstack-nova | 08:16 | |
| *** Tom-Tom has quit IRC | 08:17 | |
| *** Tom-Tom has joined #openstack-nova | 08:17 | |
| openstackgerrit | Peter Penchev proposed openstack/nova master: Add the Nova libvirt StorPool attachment driver. https://review.openstack.org/140733 | 08:18 |
| *** Tom-Tom has quit IRC | 08:19 | |
| *** Tom-Tom has joined #openstack-nova | 08:19 | |
| *** zhurong has quit IRC | 08:20 | |
| *** andreas_s has joined #openstack-nova | 08:21 | |
| *** pcaruana has joined #openstack-nova | 08:23 | |
| *** sshwarts has joined #openstack-nova | 08:30 | |
| *** salv-orlando has quit IRC | 08:32 | |
| *** salv-orlando has joined #openstack-nova | 08:32 | |
| *** sree_ has joined #openstack-nova | 08:32 | |
| *** sree_ is now known as Guest25372 | 08:33 | |
| openstackgerrit | Balazs Gibizer proposed openstack/nova master: Test alloc_cands with indirectly sharing RPs https://review.openstack.org/519601 | 08:34 |
| *** sree has quit IRC | 08:35 | |
| *** edmondsw has joined #openstack-nova | 08:35 | |
| *** rcernin has joined #openstack-nova | 08:35 | |
| *** salv-orlando has quit IRC | 08:36 | |
| *** claudiub has joined #openstack-nova | 08:39 | |
| *** edmondsw has quit IRC | 08:40 | |
| *** ragiman has joined #openstack-nova | 08:40 | |
| *** yamamoto_ has quit IRC | 08:47 | |
| *** jpena|off is now known as jpena | 08:48 | |
| *** salv-orlando has joined #openstack-nova | 08:50 | |
| *** claudiub has quit IRC | 08:57 | |
| *** hshiina has quit IRC | 08:58 | |
| *** yamamoto has joined #openstack-nova | 08:58 | |
| *** zhurong has joined #openstack-nova | 09:02 | |
| *** claudiub has joined #openstack-nova | 09:04 | |
| *** mvk has joined #openstack-nova | 09:05 | |
| bauzas | morning folks | 09:05 |
| *** gongysh has quit IRC | 09:07 | |
| *** trungnv has joined #openstack-nova | 09:09 | |
| *** Guest13268 is now known as zigo | 09:09 | |
| *** aloga has quit IRC | 09:21 | |
| *** aloga has joined #openstack-nova | 09:22 | |
| *** Guest25372 has quit IRC | 09:24 | |
| *** sree has joined #openstack-nova | 09:24 | |
| *** gongysh has joined #openstack-nova | 09:25 | |
| *** sree has quit IRC | 09:29 | |
| *** jaianshu has joined #openstack-nova | 09:30 | |
| *** lucas-afk is now known as lucasagomes | 09:31 | |
| giblet | bauzas: good morning | 09:31 |
| *** rodolof has joined #openstack-nova | 09:32 | |
| *** alexchadin has joined #openstack-nova | 09:32 | |
| *** tetsuro has joined #openstack-nova | 09:35 | |
| *** jangutter has joined #openstack-nova | 09:38 | |
| openstackgerrit | Theodoros Tsioutsias proposed openstack/python-novaclient master: CommandError is raised for invalid server fields https://review.openstack.org/525110 | 09:38 |
| *** derekh has joined #openstack-nova | 09:38 | |
| *** sree has joined #openstack-nova | 09:40 | |
| *** lei-zh has quit IRC | 09:41 | |
| *** claudiub has quit IRC | 09:41 | |
| *** moshele has quit IRC | 09:42 | |
| *** slaweq has quit IRC | 09:49 | |
| *** slaweq has joined #openstack-nova | 09:50 | |
| *** slaweq has quit IRC | 09:50 | |
| openstackgerrit | Silvan Kaiser proposed openstack/nova master: Added mount fstype based validation of Quobyte mounts https://review.openstack.org/522245 | 09:52 |
| *** gszasz has joined #openstack-nova | 09:56 | |
| *** slaweq has joined #openstack-nova | 09:56 | |
| *** alexchadin has quit IRC | 10:05 | |
| *** sree has quit IRC | 10:13 | |
| *** alexchadin has joined #openstack-nova | 10:13 | |
| *** sree has joined #openstack-nova | 10:13 | |
| *** ragiman_ has joined #openstack-nova | 10:15 | |
| *** jaianshu has quit IRC | 10:16 | |
| *** jaianshu has joined #openstack-nova | 10:16 | |
| *** ragiman has quit IRC | 10:16 | |
| *** sree has quit IRC | 10:18 | |
| *** sree has joined #openstack-nova | 10:19 | |
| *** sree has quit IRC | 10:23 | |
| *** jaianshu has quit IRC | 10:23 | |
| *** giblet is now known as gibi | 10:25 | |
| *** zhurong has quit IRC | 10:26 | |
| *** Yingxin has quit IRC | 10:27 | |
| *** jaianshu has joined #openstack-nova | 10:28 | |
| *** jichen has quit IRC | 10:28 | |
| *** Yingxin has joined #openstack-nova | 10:30 | |
| *** yamamoto has quit IRC | 10:31 | |
| *** annp has quit IRC | 10:33 | |
| *** phuongnh has quit IRC | 10:37 | |
| *** mvk has quit IRC | 10:40 | |
| *** gszasz has quit IRC | 10:42 | |
| *** edand_ has quit IRC | 10:42 | |
| *** andreas_s has quit IRC | 10:42 | |
| *** andreas_s has joined #openstack-nova | 10:43 | |
| *** slaweq_ has joined #openstack-nova | 10:43 | |
| *** rodolof has quit IRC | 10:45 | |
| *** sambetts|afk is now known as sambetts | 10:47 | |
| *** slaweq_ has quit IRC | 10:48 | |
| *** sridharg has quit IRC | 10:49 | |
| *** gmann is now known as gmann_afk | 10:52 | |
| *** ragiman_ has quit IRC | 10:54 | |
| *** andreas_s has quit IRC | 10:56 | |
| *** ralonsoh has quit IRC | 10:58 | |
| *** Tom-Tom has quit IRC | 10:59 | |
| *** Tom-Tom has joined #openstack-nova | 11:00 | |
| *** jaianshu_ has joined #openstack-nova | 11:02 | |
| *** ralonsoh has joined #openstack-nova | 11:03 | |
| *** Tom-Tom has quit IRC | 11:04 | |
| *** andreas_s has joined #openstack-nova | 11:06 | |
| *** jaianshu has quit IRC | 11:06 | |
| *** ragiman_ has joined #openstack-nova | 11:06 | |
| *** edand_ has joined #openstack-nova | 11:07 | |
| *** mvk has joined #openstack-nova | 11:09 | |
| mdbooth | Anybody got a minute to look at this bugfix: https://review.openstack.org/#/c/524681/ | 11:10 |
| mdbooth | Honesty in advertising: it's only a couple of lines, but probably requires extra care due to its place bouncing around in the middle of the live migration workflow. | 11:11 |
| mdbooth | Still, seems like an obvious fix, and its making users grumpy | 11:11 |
| *** gongysh has quit IRC | 11:14 | |
| *** dayou has quit IRC | 11:15 | |
| *** yamamoto has joined #openstack-nova | 11:16 | |
| *** alexchadin has quit IRC | 11:19 | |
| *** andreas_s has quit IRC | 11:19 | |
| *** alexchadin has joined #openstack-nova | 11:20 | |
| *** andreas_s has joined #openstack-nova | 11:20 | |
| *** andreas_s has quit IRC | 11:24 | |
| *** brault has joined #openstack-nova | 11:25 | |
| gibi | mdbooth: looks good to me | 11:26 |
| *** Tom-Tom has joined #openstack-nova | 11:26 | |
| *** alexchadin has quit IRC | 11:30 | |
| *** alexchadin has joined #openstack-nova | 11:30 | |
| *** dtantsur|afk is now known as dtantsur | 11:30 | |
| *** Tom-Tom has quit IRC | 11:30 | |
| kaisers | finucannot: Hi! (hope i've the right id :) ) Regarding the exception in https://review.openstack.org/#/c/522245 , i'm happy to do this differently. I just need a way to differentiate one of the exceptions in validate volume so if you have suggestions, pls fire away | 11:31 |
| *** finucannot is now known as stephenfin | 11:31 | |
| stephenfin | kaisers: This is for the purposes of the test? | 11:32 |
| kaisers | no, for connect_volume() | 11:32 |
| kaisers | That has a special treatment for the stale mount point situation. It's where i want to detect and handle that special case. | 11:33 |
| *** gszasz has joined #openstack-nova | 11:33 | |
| stephenfin | Ah, I see your comments now | 11:33 |
| kaisers | stephenfin: (i thought this would be faster via irc than a comments discussion... ) | 11:33 |
| stephenfin | kaisers: Hmm, OK then if it's serving a purpose a separate exception makes sense | 11:35 |
| stephenfin | *purpose other than making testing slightly easier | 11:35 |
| stephenfin | and it should be in nova.exception. There are a lot of virt-specific exceptions in there alright | 11:35 |
| stephenfin | At least I can't spot many exceptions kept outside of there | 11:36 |
| kaisers | stephenfin: ok, i see that point | 11:36 |
| kaisers | i'll move the exception and rethink it's naming and send it with the next update | 11:36 |
| openstackgerrit | Ghanshyam Mann proposed openstack/nova master: Fix 'all_tenants' & 'all_projects' type in api-ref https://review.openstack.org/522918 | 11:37 |
| *** moshele has joined #openstack-nova | 11:40 | |
| *** sean-k-mooney2 has joined #openstack-nova | 11:43 | |
| sean-k-mooney2 | o/ | 11:44 |
| *** andreas_s has joined #openstack-nova | 11:44 | |
| mdbooth | gibi: Thanks! | 11:45 |
| *** lpetrut has joined #openstack-nova | 11:46 | |
| *** andreas_s has quit IRC | 11:46 | |
| *** andreas_s has joined #openstack-nova | 11:46 | |
| *** cdent has joined #openstack-nova | 11:51 | |
| *** ratailor has quit IRC | 11:52 | |
| openstackgerrit | Theodoros Tsioutsias proposed openstack/python-novaclient master: CommandError is raised for invalid server fields https://review.openstack.org/525110 | 11:54 |
| *** smatzek has joined #openstack-nova | 11:56 | |
| *** jpena is now known as jpena|lunch | 11:57 | |
| *** gszasz has quit IRC | 11:57 | |
| openstackgerrit | Chris Dent proposed openstack/nova master: [placement] Object changes to support last-modified headers https://review.openstack.org/521639 | 11:58 |
| openstackgerrit | Chris Dent proposed openstack/nova master: [placement] Add cache headers to placement api requests https://review.openstack.org/521640 | 11:58 |
| *** gszasz has joined #openstack-nova | 11:58 | |
| *** TuanLA has quit IRC | 12:01 | |
| *** yangyapeng has quit IRC | 12:02 | |
| *** Tom-Tom has joined #openstack-nova | 12:06 | |
| *** yangyape_ has joined #openstack-nova | 12:06 | |
| *** lennyb__ has quit IRC | 12:07 | |
| *** gcb has quit IRC | 12:10 | |
| *** lucasagomes is now known as lucas-hungry | 12:10 | |
| *** yangyape_ has quit IRC | 12:11 | |
| *** edmondsw has joined #openstack-nova | 12:11 | |
| *** edmondsw has quit IRC | 12:16 | |
| *** lpetrut has quit IRC | 12:17 | |
| *** BryanS68 has joined #openstack-nova | 12:24 | |
| *** lpetrut has joined #openstack-nova | 12:25 | |
| *** kuzko has quit IRC | 12:29 | |
| *** kuzko has joined #openstack-nova | 12:31 | |
| *** tetsuro has quit IRC | 12:31 | |
| *** rcernin has quit IRC | 12:31 | |
| *** kuzko has quit IRC | 12:35 | |
| *** yangyapeng has joined #openstack-nova | 12:36 | |
| *** derekh has quit IRC | 12:36 | |
| *** kuzko has joined #openstack-nova | 12:43 | |
| *** alexchadin has quit IRC | 12:44 | |
| efried | Hey kids! | 12:45 |
| *** mhenkel has quit IRC | 12:47 | |
| *** sridharg has joined #openstack-nova | 12:47 | |
| cdent | hey uncle eric | 12:48 |
| *** links has quit IRC | 12:54 | |
| *** mhenkel has joined #openstack-nova | 12:55 | |
| sean-k-mooney2 | :) hey efried you seem to be in a good mood. good weekend? | 13:02 |
| efried | Yeah, very eventful. My buddy got his BJJ black belt, which is a pretty big deal. | 13:02 |
| *** dtantsur is now known as dtantsur|lunch | 13:03 | |
| sean-k-mooney2 | oh cool. does that mean he can now teach. | 13:03 |
| sean-k-mooney2 | or does he need second dan for that | 13:04 |
| *** gszasz has quit IRC | 13:04 | |
| efried | He's been teaching for a while. | 13:04 |
| *** jpena|lunch is now known as jpena | 13:04 | |
| efried | I guess it doesn't really mean anything except to people in the BJJ community, or near enough to it to recognize the significance. | 13:05 |
| efried | Basically, it means he's good. Really good. | 13:05 |
| sean-k-mooney2 | hehe yes blackbelts in any martial art tend to be :) one of my co workers recently started BJJ | 13:06 |
| *** alexchadin has joined #openstack-nova | 13:07 | |
| *** salv-orlando has quit IRC | 13:07 | |
| *** salv-orlando has joined #openstack-nova | 13:08 | |
| *** lucas-hungry is now known as lucasagomes | 13:08 | |
| *** jaypipes has joined #openstack-nova | 13:08 | |
| efried | BJJ is one of the harder ones, generally speaking. Most times it takes around a decade to get there (though my buddy did it in a little over 7 years, which is fairly quick - he's got talent). | 13:09 |
| *** ttsiouts_ has joined #openstack-nova | 13:11 | |
| sean-k-mooney2 | jaypipes: o/ its faily early still in your neck of the woods right? | 13:11 |
| jaypipes | sean-k-mooney2: quarter after eight | 13:11 |
| *** salv-orlando has quit IRC | 13:12 | |
| sean-k-mooney2 | jaypipes: ya i taught you were -5 from me. | 13:12 |
| *** ttsiouts_ has quit IRC | 13:14 | |
| sean-k-mooney2 | so general question. in pike there was no way to request a spcecific resouce class to be allocated as part of a vm boot request correct e.g. in placement | 13:14 |
| efried | sean-k-mooney2 I believe that is correct, except for the Ironic case. | 13:18 |
| efried | ...which isn't really a resource class. | 13:19 |
| sean-k-mooney2 | right | 13:19 |
| jaypipes | sean-k-mooney2: no, you could/can specify an override in the flavor's extra specs "resources" key. | 13:20 |
| sean-k-mooney2 | ok thats what i taught. am for neted resouce provided sriov is going to be the first example resouce correct | 13:20 |
| sean-k-mooney2 | jaypipes: oh really? | 13:20 |
| jaypipes | sean-k-mooney2: no. Ironic has used resources in the flavor extra specs already | 13:20 |
| jaypipes | sean-k-mooney2: and vCPU, memory and disk can also be overridden from the flavor.vcpus/memory_mb/disk_gb values | 13:21 |
| *** slaweq_ has joined #openstack-nova | 13:21 | |
| efried | sean-k-mooney2 I think the first real world example may be VGPUs in the xen driver. | 13:22 |
| sean-k-mooney2 | jaypipes: and if i was to do that it wont break any of the internal accounting? e.g. if flavor says 4 vcpus and i override the resouce request to 2 or 8 bad thing wont happen? or this is a gun and i should be careful where i point it | 13:23 |
| *** edmondsw has joined #openstack-nova | 13:23 | |
| jaypipes | sean-k-mooney2: accounting will be done with the override. | 13:23 |
| sean-k-mooney2 | ok cool so if i were to do this always claim >= what is used by normal flavor properties. | 13:25 |
| *** slaweq_ has quit IRC | 13:26 | |
| efried | sean-k-mooney2 For the Standard Three (VCPU, MEMORY_MB, and DISK_GB) in pike, it's in fact the exact same code path by the time it gets to actually claiming anything. The override/conversion gets done really early. | 13:26 |
| sean-k-mooney2 | efried: yes but i would assume the xml generation would use the VCPU not the override in resouces | 13:27 |
| efried | sean-k-mooney2 Maybe I'm not understanding what you mean by "override". Where are you overriding? | 13:27 |
| jaypipes | efried: the extra specs resources key/value pair are overrides from what is in, say, flavor.vcpus | 13:27 |
| efried | Right, sean-k-mooney2 in ^ this case, you're safe. | 13:28 |
| efried | For a sec I was scared you were, like, doing something in the virt driver that was different from what came down in the flavor. | 13:28 |
| sean-k-mooney2 | efried: well i was just trying to make sure you couldnt have flavor.vpus=4 and resouces.vcpus=2 and end up spawning a vm with 4 cpus but only claim 2 | 13:30 |
| jaypipes | sean-k-mooney2: no, you can't do that | 13:31 |
| efried | sean-k-mooney2 oh, it's an interesting point. I imagine it's up to the virt driver what they key off of to decide how much resource to allocate. Let me check whether we actually overwrite the flavor.vcpus value for the copy we send down to virt... | 13:31 |
| *** links has joined #openstack-nova | 13:31 | |
| jaypipes | sean-k-mooney2: but you can have flavor.vcpus = 2 and resources:VCPU=2 and the instance will get 2. | 13:31 |
| jaypipes | doh, I meant resources:VCPU=4 and get 4. | 13:31 |
| sean-k-mooney2 | jaypipes: ok well since the flavor is admin only i guess that is ok. if this was available via image metadata that could be bad | 13:33 |
| jaypipes | sean-k-mooney2: ack | 13:33 |
| *** janki has quit IRC | 13:33 | |
| *** yamahata has joined #openstack-nova | 13:34 | |
| *** oanson has quit IRC | 13:35 | |
| sean-k-mooney2 | ok well that good to know it will likely be helpful at some point. | 13:37 |
| *** oanson has joined #openstack-nova | 13:37 | |
| sean-k-mooney2 | have ye heard of vhost-vfio | 13:38 |
| *** dtantsur|lunch is now known as dtantsur | 13:38 | |
| *** rabel has joined #openstack-nova | 13:40 | |
| sean-k-mooney2 | it is a form of vhost offload like vhost-user which leaverages the kenel vfio-pci driver to provide sriov like performance that can also support livemigration/ovs hardware offload. | 13:41 |
| jaypipes | sean-k-mooney2: no, haven't heard of it until just now. sounds like yet another avenue for hard-to-debug issues in live migration :) | 13:43 |
| *** edmondsw has quit IRC | 13:43 | |
| *** jaianshu_ has quit IRC | 13:43 | |
| *** hongbin has joined #openstack-nova | 13:43 | |
| sean-k-mooney2 | jaypipes: hehe proably an im likely going to be one of the first to enjoy that experience | 13:43 |
| jaypipes | sean-k-mooney2: you always are :) | 13:44 |
| jaypipes | strace and gdb ftw | 13:44 |
| sean-k-mooney2 | this is the semi internal poc that im being pull upstream work to work on. it leaverages mdevs instead of vfs to represent the virtual interfaces which is fun... | 13:45 |
| jaypipes | sean-k-mooney2: what advantage does Nvidia's mdev framework have over the SR-IOV VF interface? | 13:46 |
| sean-k-mooney2 | jaypipes: live migrtation and the ability to create them at runtime are 2 of the advantages | 13:47 |
| jaypipes | sean-k-mooney2: why can't VFs be created at runtime? | 13:47 |
| *** jamesdenton has joined #openstack-nova | 13:48 | |
| sean-k-mooney2 | jaypipes: the linux kernel does not have an interface to do so without removeing and reallocating all vfs on the pf | 13:48 |
| sean-k-mooney2 | our nics and technically dpdk can | 13:48 |
| jaypipes | sean-k-mooney2: interesting. why didn't nvidia enhance the existing SR-IOV drivers/VF interface instead of inventing a whole new interface? | 13:49 |
| openstackgerrit | Jay Pipes proposed openstack/nova master: placement: allow filter providers in tree https://review.openstack.org/377215 | 13:49 |
| openstackgerrit | Jay Pipes proposed openstack/nova master: placement: adds REST API for nested providers https://review.openstack.org/384807 | 13:49 |
| openstackgerrit | Jay Pipes proposed openstack/nova master: placement: update client to set parent provider https://review.openstack.org/385693 | 13:49 |
| sean-k-mooney2 | jaypipes: well sriov is a standard by the pcisig and when you create a vf you need to create a new pci endpoint on the host pcie bus for that vf which is no trivial in hardware at runtime | 13:50 |
| jaypipes | alex_xu: great catch on https://review.openstack.org/#/c/384807. I've updated and added a new func test to catch CannotDeleteParentResourceProvider | 13:50 |
| *** itlinux has joined #openstack-nova | 13:50 | |
| jaypipes | sean-k-mooney2: and what's different about mdev in that respect? | 13:51 |
| *** Tom-Tom_ has joined #openstack-nova | 13:51 | |
| *** abalutoiu has joined #openstack-nova | 13:52 | |
| sean-k-mooney2 | they are attached to a new bus /sys/bus/mdev and not the pci bus. you create them by echoing a uuid to a file under a partent device then acress them via the uuid after tehy are created | 13:52 |
| jaypipes | sean-k-mooney2: ack. but why couldn't the PCI spec be changed to allow a similar functionality? I'm just curious | 13:53 |
| *** ttsiouts_ has joined #openstack-nova | 13:53 | |
| *** Tom-Tom has quit IRC | 13:54 | |
| sean-k-mooney2 | jaypipes: im guessing backwards compatiblity but im not sure. one of the other advanatages of vfio-mdevs ist that it does not need hardware support. it can all be done in the driver | 13:54 |
| jaypipes | sean-k-mooney2: ack | 13:54 |
| *** ttsiouts has quit IRC | 13:55 | |
| * efried brags about PowerVM again, where we've had the ability to do live-migrateable SR-IOV VF with dynamic allocation since ocata | 13:55 | |
| *** edleafe- is now known as edleafe | 13:55 | |
| sean-k-mooney2 | so if you had a nic without sriov with say 4 tx/rx pairs and you impmented mdev support you could create 4 mdevs one for each queue pair and use it like 4 nics. sriov would require hardware support to do the same | 13:55 |
| jaypipes | sean-k-mooney2: or just use PowerVM, which solves all problems for everyone. | 13:57 |
| efried | Correction: Since Newton. | 13:57 |
| jaypipes | heh | 13:57 |
| * jaypipes turns off snark-o-meter | 13:57 | |
| sean-k-mooney2 | jaypipes: :) certily solve my problem of exposing intel ad vaule in hardware | 13:58 |
| *** hongbin has quit IRC | 13:58 | |
| *** ttsiouts_ is now known as ttsiouts | 13:58 | |
| *** thorst has joined #openstack-nova | 13:59 | |
| *** hongbin has joined #openstack-nova | 13:59 | |
| openstackgerrit | Lajos Katona proposed openstack/nova master: Extend ServerMovingTests with custom resources https://review.openstack.org/497399 | 13:59 |
| *** ttsiouts is now known as ttsiouts_ | 13:59 | |
| *** yamahata has quit IRC | 13:59 | |
| sean-k-mooney2 | so any way i may have to backport whatever i come upwith to pike hence my earlir question though im trying to avoid that give howmuch is changeing in this space | 13:59 |
| edleafe | Scheduler subteam meeting starting now in #openstack-meeting-alt | 14:00 |
| sean-k-mooney2 | oh im late for a meeting too... | 14:00 |
| efried | sean-k-mooney2 It would be a tough backport, because yeah, queens is going to have huge deltas in this area. | 14:01 |
| *** vladikr has joined #openstack-nova | 14:01 | |
| alex_xu | jaypipes: there is another comment at https://review.openstack.org/#/c/377215/66 | 14:02 |
| *** slaweq_ has joined #openstack-nova | 14:02 | |
| *** liverpooler has joined #openstack-nova | 14:02 | |
| sean-k-mooney2 | efried: yep and doing it in a way that will be upgradable to queens will be very hard which is why im trying to push that out of scope but first i need to understand if its technically possible or not | 14:03 |
| *** takashin_ has joined #openstack-nova | 14:04 | |
| efried | sean-k-mooney2 It might be doable for custom resource classes. What did you have in mind? | 14:04 |
| *** takashin has quit IRC | 14:06 | |
| *** slaweq_ has quit IRC | 14:06 | |
| sean-k-mooney2 | custom resouce class create by neutron ovs agent extention, + os-vif plugin to create mdevs during plug + new vnic type to request a vhost-vfio port like we do with sriov | 14:06 |
| sean-k-mooney2 | + the smallest patch to nova possibel to cover xml generation changes in libvirt driver and doing the resouce claim in placement. | 14:07 |
| *** lyan has joined #openstack-nova | 14:08 | |
| *** yamamoto_ has joined #openstack-nova | 14:08 | |
| *** salv-orlando has joined #openstack-nova | 14:08 | |
| *** cleong has joined #openstack-nova | 14:09 | |
| *** hongbin has quit IRC | 14:11 | |
| *** yamamoto has quit IRC | 14:11 | |
| *** mlavalle has joined #openstack-nova | 14:11 | |
| *** pchavva has joined #openstack-nova | 14:11 | |
| *** karthiks has quit IRC | 14:12 | |
| *** salv-orlando has quit IRC | 14:13 | |
| gibi | sean-k-mooney2: be aware that in pike custom resource classes only works if it is reported by the nova-compute in the RP that represents the compute node. All the other option needs either shared RP or nested RP | 14:13 |
| jaypipes | alex_xu: responded. | 14:13 |
| *** felipemonteiro has joined #openstack-nova | 14:15 | |
| alex_xu | jaypipes: yea, the uuid and name aren't in opposition, but sorry, I didn't get what your mean about the 'member_of' | 14:16 |
| *** felipemonteiro_ has joined #openstack-nova | 14:16 | |
| *** yamamoto_ has quit IRC | 14:20 | |
| openstackgerrit | Silvan Kaiser proposed openstack/nova master: Added mount fstype based validation of Quobyte mounts https://review.openstack.org/522245 | 14:20 |
| *** felipemonteiro has quit IRC | 14:20 | |
| *** alexchadin has quit IRC | 14:22 | |
| openstackgerrit | Silvan Kaiser proposed openstack/nova master: Added mount fstype based validation of Quobyte mounts https://review.openstack.org/522245 | 14:23 |
| *** gbarros has joined #openstack-nova | 14:24 | |
| jaypipes | alex_xu: that was responding to bauzas' comment about using member_of instead of tree. member_of is for aggregates, not for provider trees. | 14:27 |
| *** gcb has joined #openstack-nova | 14:27 | |
| alex_xu | jaypipes: ah, got it | 14:28 |
| *** yamamoto has joined #openstack-nova | 14:29 | |
| *** tssurya has quit IRC | 14:29 | |
| bauzas | jaypipes: oh my bad, you're right | 14:31 |
| *** smatzek has quit IRC | 14:31 | |
| *** takashin_ has left #openstack-nova | 14:31 | |
| *** takashin has joined #openstack-nova | 14:31 | |
| *** smatzek has joined #openstack-nova | 14:31 | |
| bauzas | jaypipes: tbc, I'm not opiniated by which key we should use, just the point that we need to make sure it works with a call having all of them :) | 14:31 |
| openstackgerrit | Matthew Booth proposed openstack/nova master: Add uuid column to BlockDeviceMapping https://review.openstack.org/242602 | 14:32 |
| openstackgerrit | Matthew Booth proposed openstack/nova master: Make BlockDeviceMapping object support uuid https://review.openstack.org/242603 | 14:32 |
| jaypipes | bauzas: ack | 14:32 |
| bauzas | "tree" is maybe a bit difficult to understand, but I'm not an English folk :) | 14:32 |
| *** yamamoto has quit IRC | 14:35 | |
| *** smatzek has quit IRC | 14:36 | |
| *** rabel has quit IRC | 14:40 | |
| *** rabel has joined #openstack-nova | 14:40 | |
| *** cdent has quit IRC | 14:42 | |
| *** gszasz has joined #openstack-nova | 14:42 | |
| sean-k-mooney2 | gibi: ah yes that is a good point. i would prefer to use nested resouce providers as logically the mdev resouce are a nested resouce of a parent device. in my case a pcie device but technically mdevs could be childern of device on buses other the pci bus. | 14:45 |
| *** amodi has joined #openstack-nova | 14:46 | |
| gibi | sean-k-mooney2: I agree that the best would be to have nest RPs for it | 14:46 |
| *** tssurya has joined #openstack-nova | 14:46 | |
| *** thorst has quit IRC | 14:46 | |
| efried | jaypipes bauzas alex_xu Oh, I wasn't following the conversation in here; I left a (verbose, as usual) comment. | 14:46 |
| jaypipes | efried: on what? | 14:47 |
| *** BryanS68 has quit IRC | 14:47 | |
| *** BryanS68 has joined #openstack-nova | 14:47 | |
| jaypipes | efried: in any case, I'm currently adding func tests that add member_of and uuid filters to a tree filter. | 14:48 |
| *** thorst has joined #openstack-nova | 14:48 | |
| *** yamamoto has joined #openstack-nova | 14:52 | |
| *** peter-hamilton has joined #openstack-nova | 14:52 | |
| *** gouthamr has joined #openstack-nova | 14:52 | |
| *** psachin has quit IRC | 14:53 | |
| *** salv-orlando has joined #openstack-nova | 14:57 | |
| *** esberglu has joined #openstack-nova | 14:58 | |
| *** tidwellr has joined #openstack-nova | 14:59 | |
| *** smatzek has joined #openstack-nova | 14:59 | |
| *** edand_ has quit IRC | 15:00 | |
| alex_xu | efried: I don't the tree and resources will work as your expected. The 'GET /resource_providers' just a simple filtering, it is different with the filtering of 'GET /allocation_candidates' | 15:01 |
| efried | jaypipes alex_xu Sorry, I was talking about https://review.openstack.org/#/c/377215/ | 15:02 |
| efried | I'm coming at it from the perspective of how I think the REST API should work, not looking at the code. | 15:02 |
| efried | The func tests jaypipes is suggesting should bear out what I said in my comment; and if not, it's a bug that we can fix, right? | 15:03 |
| *** thorst has quit IRC | 15:03 | |
| openstackgerrit | Takashi NATSUME proposed openstack/nova master: [placement] Add sending global request ID in delete https://review.openstack.org/525231 | 15:04 |
| *** yamamoto has quit IRC | 15:04 | |
| *** burt has quit IRC | 15:06 | |
| *** Tom-Tom_ has quit IRC | 15:06 | |
| jianghuaw_ | bauzas, the vGPU relative calls have been there since the minimal version of XS supported by nova. So there is no need to check XenServer's version. | 15:07 |
| *** takashin has left #openstack-nova | 15:07 | |
| openstackgerrit | Jay Pipes proposed openstack/nova master: placement: allow filter providers in tree https://review.openstack.org/377215 | 15:07 |
| stephenfin | dansmith: Question on this patch for you https://review.openstack.org/#/c/514309/ | 15:07 |
| openstackgerrit | Jay Pipes proposed openstack/nova master: placement: adds REST API for nested providers https://review.openstack.org/384807 | 15:07 |
| openstackgerrit | Jay Pipes proposed openstack/nova master: placement: update client to set parent provider https://review.openstack.org/385693 | 15:07 |
| jaypipes | bauzas, alex_xu, efried: k, func tests added for tree filter + uuid/member_of | 15:07 |
| dansmith | stephenfin: where? | 15:08 |
| bauzas | jianghuaw_: gtk | 15:08 |
| stephenfin | dansmith: The 'tempest-dsvm-neutron-src-oslo.versionedobjects' job | 15:08 |
| stephenfin | dansmith: I migrated that because it seemed we were the only caller | 15:08 |
| bauzas | jianghuaw_: kaboom then | 15:09 |
| stephenfin | But I see now that there are many 'tempest-dsvm-neutron-src-{library}' jobs and '{library}' is usually wildcarded, so that should probably live in oslo.versionedobjects | 15:09 |
| jianghuaw_ | bauzas, thanks:-) | 15:09 |
| dansmith | stephenfin: sorry, I blame the early hour for my confusion but.. what are you asking? if other projects have o.vo jobs? | 15:10 |
| * bauzas going back to write the libvirt create MDEV | 15:10 | |
| *** links has quit IRC | 15:10 | |
| stephenfin | dansmith: Yes. Just a sanity check | 15:11 |
| dansmith | stephenfin: tbh, that's not a thing I know and I would expect it to be more of a core oslo thing, policy for running library jobs against projects that use them and such | 15:12 |
| dansmith | stephenfin: I think nova, neutron, ironic at least use o.vo | 15:12 |
| jianghuaw_ | bauzas, I guess this one would be much easier as you ever +2/+w on it:-) https://review.openstack.org/#/c/516217 | 15:12 |
| bauzas | jianghuaw_: next one :) | 15:13 |
| jianghuaw_ | cool:-) | 15:13 |
| efried | jaypipes It would be nice to have a func test for resources + tree -- I think that was a main concern of alex_xu's. But we can do it in a followup too. | 15:13 |
| stephenfin | dansmith: That's fair. I picked on you as you were the one that enabled the test for nova (commit ac96f95 in openstack-infra/project-config). No good deed :) | 15:13 |
| *** edmondsw has joined #openstack-nova | 15:13 | |
| stephenfin | I'll go bug AJaeger | 15:13 |
| dansmith | stephenfin: probably at the behest (and handholding) of some infra deity :) | 15:13 |
| *** ijw has quit IRC | 15:14 | |
| *** awaugama has joined #openstack-nova | 15:14 | |
| openstackgerrit | Takashi NATSUME proposed openstack/nova master: Fix 500 error while passing 4-byte unicode data https://review.openstack.org/407514 | 15:17 |
| *** edmondsw has quit IRC | 15:18 | |
| *** AlexeyAbashkin has quit IRC | 15:20 | |
| efried | jaypipes I was about to rebase "make _Provider private" -- did you have any feedback on uuid_set() (https://review.openstack.org/#/c/520243/) so I can hit it at the same time? | 15:20 |
| *** thorst has joined #openstack-nova | 15:24 | |
| *** yamamoto has joined #openstack-nova | 15:25 | |
| *** yamamoto has quit IRC | 15:25 | |
| *** itlinux has quit IRC | 15:26 | |
| *** esberglu_ has joined #openstack-nova | 15:27 | |
| *** gbarros has quit IRC | 15:27 | |
| *** thorst has quit IRC | 15:29 | |
| *** esberglu has quit IRC | 15:29 | |
| *** armax has joined #openstack-nova | 15:30 | |
| *** moshele has quit IRC | 15:33 | |
| *** gbarros has joined #openstack-nova | 15:35 | |
| openstackgerrit | Eric Fried proposed openstack/nova master: Make _Provider really private https://review.openstack.org/523932 | 15:37 |
| openstackgerrit | Eric Fried proposed openstack/nova master: ProviderTree.uuid_set() https://review.openstack.org/520243 | 15:37 |
| efried | jaypipes ^ | 15:37 |
| mdbooth | dansmith: You mentioned last week that you didn't think I could add a unique constraint to the uuid column in the first instance. You also say the same explicitly in a comment, so I assume there's a good reason. Do you recall what it is? | 15:40 |
| dansmith | mdbooth: you can if it's nullable I think, but I think I thought you were asserting that you were going to create the column and make it non-nullable/unique right away and that's what I was trying to say was not okay | 15:41 |
| mdbooth | dansmith: Yeah, I was going for unique/nullable. | 15:41 |
| mdbooth | Any reason I shouldn't add the unique constraint to your patch? | 15:42 |
| * mdbooth likes constraints | 15:42 | |
| dansmith | if you can add it to the schema without code changes or moving data, then it's cool to add it yeah | 15:42 |
| mdbooth | Will do, thanks | 15:43 |
| tssurya | mriedem: ping | 15:45 |
| *** hongbin has joined #openstack-nova | 15:48 | |
| *** andreas_s has quit IRC | 15:50 | |
| *** andreas_s has joined #openstack-nova | 15:51 | |
| openstackgerrit | Balazs Gibizer proposed openstack/nova master: Add late server group policy check to rebuild https://review.openstack.org/525242 | 15:51 |
| mdbooth | dansmith: Second question: update by uuid. I'm assuming that when you were adding this first time round you anticipated needing this. I don't think I need it. I could add it for completeness, or omit it until there's a caller. Personal preference would be the latter. | 15:51 |
| *** tidwellr has quit IRC | 15:52 | |
| *** slaweq is now known as slaweq_ | 15:53 | |
| *** tidwellr has joined #openstack-nova | 15:53 | |
| dansmith | mdbooth: I probably had it for tests or something, but sure, no need to have it if there is no user | 15:53 |
| *** gcb has quit IRC | 15:53 | |
| *** esberglu_ has quit IRC | 15:56 | |
| *** esberglu has joined #openstack-nova | 15:56 | |
| *** ttsiouts_ has quit IRC | 15:57 | |
| *** tssurya has quit IRC | 15:57 | |
| *** david-lyle has quit IRC | 15:58 | |
| *** david-lyle has joined #openstack-nova | 15:59 | |
| openstackgerrit | Jackie Truong proposed openstack/python-novaclient master: Microversion 2.57 - Add trusted_image_certificates https://review.openstack.org/500396 | 15:59 |
| *** edand_ has joined #openstack-nova | 15:59 | |
| *** slaweq_ has quit IRC | 16:00 | |
| *** yamamoto has joined #openstack-nova | 16:00 | |
| *** andreas_s has quit IRC | 16:00 | |
| *** andreas_s has joined #openstack-nova | 16:01 | |
| *** sdague has joined #openstack-nova | 16:01 | |
| *** slaweq_ has joined #openstack-nova | 16:03 | |
| mdbooth | dansmith: Thanks | 16:07 |
| *** sshwarts has quit IRC | 16:07 | |
| *** slaweq_ has quit IRC | 16:08 | |
| *** mriedem has joined #openstack-nova | 16:09 | |
| *** coreywright has quit IRC | 16:10 | |
| ildikov | mriedem: hi, I updated the patch with the fixes to your comments | 16:13 |
| ildikov | mriedem: I didn't have the stomach as of yet at least to do the split... | 16:13 |
| *** amodi has quit IRC | 16:13 | |
| mriedem | ildikov: ok | 16:13 |
| *** _pewp_ has quit IRC | 16:14 | |
| *** ttsiouts has joined #openstack-nova | 16:14 | |
| *** _pewp_ has joined #openstack-nova | 16:14 | |
| ildikov | mriedem: if you have time plz continue to review and I will fix it in either one or two patches | 16:14 |
| mriedem | i hope to get to it today, but have to work on some other stuff this morning | 16:15 |
| *** tssurya has joined #openstack-nova | 16:16 | |
| ildikov | appreciated if you can get to it today | 16:16 |
| *** lajoskatona has left #openstack-nova | 16:19 | |
| *** ttsiouts_ has joined #openstack-nova | 16:20 | |
| *** cdent has joined #openstack-nova | 16:21 | |
| *** yamamoto has quit IRC | 16:22 | |
| *** coreywright has joined #openstack-nova | 16:23 | |
| *** slaweq_ has joined #openstack-nova | 16:25 | |
| *** edmondsw has joined #openstack-nova | 16:26 | |
| *** thorst has joined #openstack-nova | 16:28 | |
| *** itlinux has joined #openstack-nova | 16:31 | |
| *** ttsiouts_ has quit IRC | 16:32 | |
| *** artom_ has quit IRC | 16:32 | |
| *** artom has joined #openstack-nova | 16:32 | |
| tssurya | mriedem: I am stuck on this - https://bugs.launchpad.net/nova/+bug/1729806/comments/1 , because both those columns are of type "Text()" and SQL doesn't allow text type columns to have a unique constraint; any pointers on how you usually do this in nova ? | 16:33 |
| openstack | Launchpad bug 1729806 in OpenStack Compute (nova) "nova-manage update_cell does not check if the same combination of transport_url and database_connection already exists" [Low,In progress] - Assigned to Surya Seetharaman (tssurya) | 16:33 |
| *** armax has quit IRC | 16:33 | |
| *** thorst has quit IRC | 16:33 | |
| mriedem | ildikov: i looked at the diff, comments in https://review.openstack.org/#/c/330285/ | 16:33 |
| *** armax has joined #openstack-nova | 16:34 | |
| openstackgerrit | Eric Fried proposed openstack/nova master: Make _Provider really private https://review.openstack.org/523932 | 16:34 |
| dansmith | tssurya: you could check before create, which isn't as good as a constraint, but it'd be an easy thing to do | 16:36 |
| openstackgerrit | Silvan Kaiser proposed openstack/nova master: Added mount fstype based validation of Quobyte mounts https://review.openstack.org/522245 | 16:36 |
| ildikov | mriedem: tnx! | 16:36 |
| *** ragiman_ has quit IRC | 16:36 | |
| *** medberry_ has joined #openstack-nova | 16:36 | |
| mriedem | ildikov: and another one just now | 16:37 |
| mriedem | ildikov: the shelved offloaded case probably needs some thought | 16:37 |
| mriedem | could also see what johnthetubaguy thinks | 16:37 |
| ildikov | mriedem: ok, I'll wait a bit longer before touching it :) | 16:37 |
| mriedem | for all intents and purposes, a reserved volume is going to be the same as an in-use volume to nova | 16:37 |
| ildikov | mriedem: sigh, I will look into the comments shortly | 16:37 |
| johnthetubaguy | I was a touch worried about the change in shelved offloaded | 16:38 |
| mriedem | there is an api behavior change with attaching a volume to a shelved offloaded instance with the new flow, | 16:38 |
| mriedem | where the volume is left 'reserved' rather than 'in-use' | 16:38 |
| johnthetubaguy | yeah, that seemed nasty | 16:38 |
| mriedem | i could see client side app tooling failing / timing out on that | 16:38 |
| mriedem | like tempest even | 16:38 |
| mriedem | lots of client code waits for an attachment to go to in-use before it considers it done | 16:38 |
| johnthetubaguy | yes, good point | 16:39 |
| tssurya | dansmith: Yes, that is what I am doing in my review, I was just trying to add the constraint part too | 16:39 |
| dansmith | tssurya: sounds like we might not be able to do the constraint part.. since urls can't be large, I'm not sure converting it to something other than text is a good idea | 16:39 |
| *** medberry_ has quit IRC | 16:39 | |
| johnthetubaguy | mriedem: I did raise that in a previous version, but forgot about it shortly after, +1 on it being bad | 16:40 |
| dansmith | tssurya: we actually have some long urls ourselves, since we encode all the HA hosts in the transport url, and I think we do the same for our galera endpoints | 16:40 |
| *** medberry_ has joined #openstack-nova | 16:40 | |
| *** slaweq_ has quit IRC | 16:40 | |
| efried | cdent dansmith Did you see my "themed CI results" last week? I made one monochrome for cdent (http://184.172.12.213/manual/htmlify_logs/logs/n-cpu.txt.gz?theme=monochrome) and one solarized for both of you (http://184.172.12.213/manual/htmlify_logs/logs/n-cpu.txt.gz?theme=solarized) | 16:40 |
| tssurya | dansmith: oh okay, yea then I will not pursue it, this came up https://review.openstack.org/#/c/523817/ in parallel to my patchset, so I thought I would investigate this | 16:40 |
| dansmith | tssurya: cool | 16:41 |
| tssurya | dansmith: thanks then, I will remove the constraint part then | 16:41 |
| mriedem | edleafe: i'm going to make a concerted push on the alternate hosts series today, do you plan on rebasing https://review.openstack.org/#/q/topic:bp/return-alternate-hosts+status:open to resolve the merge conflicts at the top? | 16:41 |
| *** AJaeger_ has joined #openstack-nova | 16:41 | |
| *** chyka has joined #openstack-nova | 16:41 | |
| *** trozet has quit IRC | 16:41 | |
| dansmith | efried: I did not, but cool.. that still ends up with ascii in the logs if I curl download it though right? | 16:41 |
| efried | dansmith I've actually got solutions that go both ways. | 16:42 |
| AJaeger_ | sdague: could you review https://review.openstack.org/#/c/524353/ ,please? This updates jobs for os-api-ref and gets rid of tools/tox_install.sh | 16:42 |
| jaypipes | efried: you are currently rebasing the series that ends in 520246, yes? | 16:42 |
| dansmith | efried: .... | 16:42 |
| edleafe | mriedem: I was waiting until there was renewed interest in the series. I can start rebasing now. | 16:42 |
| dansmith | efried: well, from the purely browser-based view, solarized looks good to my eyes :) | 16:42 |
| mriedem | edleafe: yup, it's my #1 priority today | 16:42 |
| efried | dansmith I think what I've got on that server right now preserves the codes, because that's the way I prefer it, and that's my server :) | 16:43 |
| mriedem | edleafe: i plan on having that series +2ed by thursday | 16:43 |
| *** gyee has joined #openstack-nova | 16:43 | |
| edleafe | mriedem: on it | 16:43 |
| efried | dansmith Okay, duly noted. The way I've got it set up, it's fairly easy to add themes, and (in case you didn't notice) there are new links at the top allowing you to flip between themes. | 16:43 |
| *** jpena is now known as jpena|brb | 16:43 | |
| dansmith | efried: yeah, curl gives me ansified files | 16:43 |
| cdent | efried: I did see it, solarized was also easy on my eyes. If there’s a way to curl and get non-ansified files that would be a cherry on top | 16:44 |
| cdent | (sort of as another theme, I guess) | 16:44 |
| ildikov | mriedem: johnthetubaguy: got busy, but will check back on it and we can chat about it | 16:44 |
| dansmith | the theming happens in the browser I imagine | 16:44 |
| dansmith | so doesn't affect curl | 16:44 |
| *** mvk has quit IRC | 16:44 | |
| * cdent misses CGI | 16:44 | |
| efried | dansmith cdent I have solutions that *always* or *never* strip the ANSI colors; what I'd like to get to eventually is a way for you to curl download with a querystring that allows you to choose. | 16:44 |
| dansmith | I'm not a hard -2 on the ansi logs, but I'd prefer it if I can get it | 16:45 |
| *** maciejjozefczyk has quit IRC | 16:45 | |
| dansmith | efried: that'd be cool | 16:45 |
| *** danpawlik_ has quit IRC | 16:45 | |
| dansmith | efried: just put me a curl link on the top somewhere so I can copy that, kinda like the download links in gerrit | 16:45 |
| *** itlinux has quit IRC | 16:46 | |
| efried | dansmith The theming is actually an HTML filter that sits on the apache server and kicks in when the user agent is a browser (or something). Then there's a separate filter that you get when it's a download, that doesn't do the HTML conversion at all. | 16:46 |
| efried | dansmith ++ good call | 16:46 |
| dansmith | efried: ah, okay I dunno how that works, but cool | 16:46 |
| efried | dansmith I usually right-click Save-As the file from the directory view, but a copyable link would be neat. | 16:46 |
| *** danpawlik_ has joined #openstack-nova | 16:46 | |
| efried | dansmith In case you're interested, the theme-ification patch: https://review.openstack.org/#/c/524744/ | 16:46 |
| dansmith | efried: yeah, copyable would be nice, because I wget them to my dev box.. right-click-save doesn't do what I want because it'd be on my local machine instead | 16:47 |
| *** maciejjozefczyk has joined #openstack-nova | 16:47 | |
| efried | dansmith I gotcha. | 16:47 |
| dansmith | but yeah, aside from my complaints, looks awesome | 16:48 |
| dansmith | sometimes being able to just switch the theme based on what you're looking for can be nice, and/or what you've been staring at for hours | 16:48 |
| *** itlinux has joined #openstack-nova | 16:50 | |
| *** yamamoto has joined #openstack-nova | 16:51 | |
| *** medberry_ has quit IRC | 16:51 | |
| *** edand_ has quit IRC | 16:53 | |
| *** sridharg has quit IRC | 16:55 | |
| mriedem | jaypipes: easy refactor https://review.openstack.org/#/c/523492/ | 16:56 |
| * artom feels out of the loop | 16:56 | |
| mriedem | in the alt hosts series | 16:56 |
| artom | What happened to nova-specs? | 16:56 |
| mriedem | artom: ? | 16:56 |
| artom | https://review.openstack.org/#/c/490824/ merged October 3rd | 16:56 |
| *** esberglu has quit IRC | 16:56 | |
| artom | Latest commit in https://github.com/openstack/nova-specs/commits/master/specs is september 21 | 16:57 |
| mriedem | efried: ^ didn't you ask infra about this a week or two ago? | 16:57 |
| *** lpetrut_ has joined #openstack-nova | 16:57 | |
| *** emccormick has joined #openstack-nova | 16:57 | |
| mriedem | http://git.openstack.org/cgit/openstack/nova-specs/log/ | 16:57 |
| mriedem | the github mirror repo is definitely stale | 16:57 |
| artom | Yeah | 16:57 |
| *** esberglu has joined #openstack-nova | 16:58 | |
| * artom will use git.openstack.org | 16:58 | |
| artom | And maybe bring it up in #infra? | 16:58 |
| efried | mriedem After spec freeze I noticed that they weren't getting published to specs.o.o -- sounds like that's a different issue. | 16:58 |
| *** amodi has joined #openstack-nova | 16:58 | |
| efried | ...which was resolved, at least at the time. | 16:58 |
| clarkb | mriedem: artom efried the github mirror being stale is a known issue, the repo is corrupt according to github and won't accept pushes | 16:58 |
| clarkb | if we take a gerrit downtime we can surgery the repo but we haven't had opportunity for that yet | 16:59 |
| artom | clarkb, aha, thanks! | 16:59 |
| efried | artom FWIW, that spec is published: https://specs.openstack.org/openstack/nova-specs/specs/queens/approved/libvirt-qemu-native-luks.html | 16:59 |
| mriedem | oh yes | 16:59 |
| clarkb | its been a low priority because we do host working mirrors elsewhere and github is just barely a thing we support | 16:59 |
| cfriesen | has anyone ever considered an option to "tox -e pep8" that would run it only on the files in the top N commits? | 16:59 |
| artom | efried, doh, of course! | 16:59 |
| efried | cfriesen -- -HEAD | 16:59 |
| mriedem | cfriesen: tox -e fast8 | 16:59 |
| artom | clarkb, yep, fair enough, thanks for, err, being aware? | 16:59 |
| * efried was not aware of fast8 -- what's that about? | 17:00 | |
| mriedem | fast8 is different | 17:00 |
| mriedem | fast8 is pep8 on just the files changed | 17:00 |
| artom | mriedem, I thought we're only at Fast and Furious 6 or 7 | 17:00 |
| clarkb | with the holidays coming up, things tend to slow down and we can probably more easily sneak in an extended gerrit downtime | 17:00 |
| efried | Ah, so same as -- -HEAD | 17:00 |
| mriedem | yes | 17:00 |
| clarkb | I'll add it to tomorrows meeting agenda | 17:00 |
| *** lpetrut has quit IRC | 17:00 | |
| mriedem | because typing 'tox -e pep8 -- -HEAD' was too tedious | 17:00 |
| openstackgerrit | Ed Leafe proposed openstack/nova master: Add Selection objects https://review.openstack.org/499239 | 17:00 |
| openstackgerrit | Ed Leafe proposed openstack/nova master: Refactor the code to check for sufficient hosts https://review.openstack.org/520242 | 17:00 |
| openstackgerrit | Ed Leafe proposed openstack/nova master: Return Selection objects from the scheduler driver https://review.openstack.org/495854 | 17:00 |
| cfriesen | mriedem: efried: sweet...guess I should check the ini file first. :) | 17:00 |
| openstackgerrit | Ed Leafe proposed openstack/nova master: Move the to_dict() method to the Selection object https://review.openstack.org/523492 | 17:00 |
| openstackgerrit | Ed Leafe proposed openstack/nova master: Modify select_destinations() to return objects and alts https://review.openstack.org/510159 | 17:00 |
| openstackgerrit | Ed Leafe proposed openstack/nova master: Change RPC for select_destinations() https://review.openstack.org/516707 | 17:00 |
| openstackgerrit | Ed Leafe proposed openstack/nova master: Move the claim_resources method to scheduler utils https://review.openstack.org/511357 | 17:00 |
| openstackgerrit | Ed Leafe proposed openstack/nova master: Make conductor pass and use host_lists https://review.openstack.org/511358 | 17:00 |
| edleafe | mriedem: ^^ there ya go | 17:00 |
| mriedem | thanks | 17:01 |
| efried | cfriesen The other thing I do when I'm working on a series is: source .tox/pep8/bin/activate to get into the pep8 virtualenv, and leave that window open. Then from within that window I'll do: git diff --name-only HEAD~1 | grep '\.py$' | xargs flake8 | 17:01 |
| *** kmalloc has joined #openstack-nova | 17:01 | |
| efried | cfriesen HEAD~{whatever-number-here} to go back that many patches. | 17:01 |
| *** yamamoto has quit IRC | 17:01 | |
| *** slaweq_ has joined #openstack-nova | 17:01 | |
| *** thorst has joined #openstack-nova | 17:02 | |
| emccormick | Hey, if anyone has a sec for a possibly dumb question: Is there a reason that it's required to have the instance type in the domain xml match the instance type in nova.conf? | 17:02 |
| emccormick | I ask because I'm working on migrating backends from local to rbd, and while running instances continue to function after the change to nova.conf, and can also be migrated, they cannot be rebooted. | 17:03 |
| * efried also leaves venv windows open to shortcut test runs for pyXX, functional, docs, etc. | 17:03 | |
| emccormick | just wondering if that's something that could be changed or if it would break things horribly | 17:03 |
| *** lpetrut_ has quit IRC | 17:03 | |
| efried | cfriesen The only caveat to the permanent-venv deal is if you need to rebuild the venv for whatever reason; sometimes running via tox will figure that bit out for you without needing to use -r; but sitting in the venv you get nuffink. | 17:04 |
| *** imacdonn has joined #openstack-nova | 17:04 | |
| edleafe | efried: I use aliases to do roughly the same thing | 17:05 |
| cfriesen | efried: that's a useful trick and a good caveat. | 17:05 |
| efried | edleafe Aliases for what, running flake8 in the venv? | 17:05 |
| openstackgerrit | Chris Friesen proposed openstack/nova master: unreserve volumes after scheduling has failed https://review.openstack.org/525253 | 17:06 |
| edleafe | efried: like this: | 17:06 |
| edleafe | alias ptt='source .tox/py27/bin/activate; python -m testtools.run' | 17:06 |
| cfriesen | ^ we ran into this while testing Pike, looks like it's still an issue in master | 17:06 |
| edleafe | I have different aliases for functional, py3, etc. | 17:06 |
| efried | edleafe Ah, gotcha. | 17:06 |
| *** slaweq_ has quit IRC | 17:06 | |
| openstackgerrit | Stephen Finucane proposed openstack/nova master: zuul: Move legacy jobs to project https://review.openstack.org/514309 | 17:08 |
| openstackgerrit | Stephen Finucane proposed openstack/nova master: Resolve review comments from I41b03a347 https://review.openstack.org/525255 | 17:08 |
| cfriesen | The issue that the patch above ^^^ fixes is that if you do boot-from-vol and then fail scheduling, currently the volume remains reserved and is stuck in the "attaching" state. | 17:08 |
| *** slaweq_ has joined #openstack-nova | 17:08 | |
| *** felipemonteiro_ has quit IRC | 17:09 | |
| efried | mriedem The _ensure_resource_provider error handling cherry-pick for pike is looking green now (https://review.openstack.org/#/c/524618/) -- will proceed to backport to ocata unless some reason not to? | 17:09 |
| mriedem | go ahead | 17:11 |
| openstackgerrit | Surya Seetharaman proposed openstack/nova master: update_cell allows more than once cell to have the same db/transport url https://review.openstack.org/518556 | 17:12 |
| *** slaweq_ has quit IRC | 17:14 | |
| *** andreas_s has quit IRC | 17:14 | |
| *** andreas_s has joined #openstack-nova | 17:14 | |
| *** slaweq_ has joined #openstack-nova | 17:15 | |
| efried | urgh, merge conflict (which I guess isn't surprising). Will hit it after "lunch". | 17:16 |
| *** udesale has quit IRC | 17:17 | |
| *** slaweq_ has quit IRC | 17:20 | |
| *** chyka has quit IRC | 17:20 | |
| *** tesseract has quit IRC | 17:20 | |
| *** chyka has joined #openstack-nova | 17:21 | |
| *** andreas_s has quit IRC | 17:23 | |
| *** jpena|brb is now known as jpena | 17:25 | |
| *** slaweq_ has joined #openstack-nova | 17:26 | |
| *** dmsimard has joined #openstack-nova | 17:26 | |
| dmsimard | kashyap: o/ mriedem said you could help with a bug we're trying to nail down. Have you gotten familiar with https://bugs.launchpad.net/nova/+bug/1735823 ? | 17:26 |
| openstack | Launchpad bug 1735823 in OpenStack Compute (nova) "Nova can hang when creating a VM with disk injection" [Medium,In progress] - Assigned to Matt Riedemann (mriedem) | 17:26 |
| dmsimard | tl;dr, we thought we ran libguestfs with force_tcg for disk injection but we didn't, so in some environments (such as OVH nodepool cloud regions) with broken nested virtualization, this failed horribly | 17:27 |
| dmsimard | We believe the fix is here: https://review.openstack.org/#/c/524748/ -- however, when testing it here: https://review.openstack.org/#/c/524750/, we're seeing increased failures.. and outside of OVH too. | 17:28 |
| dmsimard | Before the fix (at OVH) http://logs.openstack.org/46/523646/1/check/legacy-tempest-dsvm-neutron-full-centos-7/5bf092c/job-output.txt#_2017-11-29_03_02_38_031436 -- After the fix (at RAX): http://logs.openstack.org/50/524750/1/check/legacy-tempest-dsvm-neutron-full-centos-7/a7f051e/job-output.txt.gz#_2017-12-04_15_25_13_211317 | 17:29 |
| dmsimard | I did a recheck in https://review.openstack.org/#/c/524750/ to make sure it wasn't a flap and it does look like it breaks things. | 17:29 |
| *** mvk has joined #openstack-nova | 17:30 | |
| *** slaweq_ has quit IRC | 17:31 | |
| stephenfin | efried: Interesting little way to speed up tox - 'touch nova.egg-ingo' | 17:32 |
| stephenfin | I read the tox source a while back to figure out why, but it skips the "check if any dependencies are out-of-date" step, which half of what takes so long | 17:32 |
| cdent | mriedem: I’m thinking that https://bugs.launchpad.net/nova/+bug/1734491 may have back port potential? | 17:32 |
| openstack | Launchpad bug 1734491 in OpenStack Compute (nova) "placement keystonemiddleware_authtoken ignores OS_PLACEMENT_CONFIG_DIR" [Undecided,In progress] - Assigned to Chris Dent (cdent) | 17:32 |
| stephenfin | Probably a bug in tox but I haven't figured out why yet | 17:32 |
| jaypipes | efried: let's go with get_provider_uuids() then. | 17:36 |
| *** andreas_s has joined #openstack-nova | 17:37 | |
| *** ralonsoh has quit IRC | 17:38 | |
| ericyoung | Any idea when the patch for ScaleIO ephemeral/image volume support will bubble to the top of the review list? https://review.openstack.org/#/c/407440/ | 17:38 |
| cdent | jaypipes: the point on the jsonschema comment was: webob is passing a string to jsonschema, one that looks like an int, but is not an int | 17:40 |
| cdent | just int will _not_ work | 17:41 |
| *** thorst_ has joined #openstack-nova | 17:41 | |
| jaypipes | cdent: if that's the case, then we shouldn't have "integer" in there at all, since we'll never be supplying actual numbers. | 17:41 |
| *** slaweq_ has joined #openstack-nova | 17:42 | |
| cdent | unless we change webob out from under it (this has been discussed further up the review stack) | 17:42 |
| cdent | I can make it just string if you like, but the way it is now is to be as flexible yet accurate as possible | 17:42 |
| cdent | it is also what the nova api does | 17:43 |
| cdent | so I copied it | 17:43 |
| *** thorst has quit IRC | 17:44 | |
| jaypipes | cdent: meh... | 17:44 |
| cdent | jaypipes: much meh made merry many months, mostly | 17:44 |
| *** andreas_s has quit IRC | 17:46 | |
| *** andreas_s has joined #openstack-nova | 17:46 | |
| *** slaweq_ has quit IRC | 17:47 | |
| *** felipemonteiro has joined #openstack-nova | 17:49 | |
| *** lucasagomes is now known as lucas-afk | 17:49 | |
| *** armax has quit IRC | 17:49 | |
| *** tidwellr has quit IRC | 17:50 | |
| *** felipemonteiro_ has joined #openstack-nova | 17:50 | |
| *** tidwellr has joined #openstack-nova | 17:50 | |
| *** thorst_ has quit IRC | 17:51 | |
| *** felipemonteiro has quit IRC | 17:54 | |
| *** armax has joined #openstack-nova | 17:54 | |
| *** tidwellr has quit IRC | 17:55 | |
| *** Apoorva has joined #openstack-nova | 17:55 | |
| *** moshele has joined #openstack-nova | 17:57 | |
| *** trozet has joined #openstack-nova | 17:58 | |
| *** andreas_s has quit IRC | 18:00 | |
| *** rabel has quit IRC | 18:00 | |
| *** thorst has joined #openstack-nova | 18:02 | |
| *** moshele has quit IRC | 18:03 | |
| *** slaweq_ has joined #openstack-nova | 18:04 | |
| *** thorst has quit IRC | 18:07 | |
| *** dtantsur is now known as dtantsur|afk | 18:08 | |
| *** slaweq_ has quit IRC | 18:09 | |
| mriedem | edleafe: ok i've made it through the code parts of https://review.openstack.org/#/c/510159/ | 18:11 |
| mriedem | edleafe: some things that need to be cleaned up in there, but we could arguably do those in a follow up, | 18:11 |
| mriedem | i will go through test changes after lunch | 18:11 |
| *** slaweq_ has joined #openstack-nova | 18:12 | |
| edleafe | mriedem: a follow-up is fine. | 18:15 |
| *** thorst has joined #openstack-nova | 18:15 | |
| *** suresh12 has joined #openstack-nova | 18:16 | |
| *** slaweq_ has quit IRC | 18:17 | |
| *** mriedem has quit IRC | 18:19 | |
| *** jpena is now known as jpena|away | 18:20 | |
| *** thorst has quit IRC | 18:20 | |
| *** suresh12 has quit IRC | 18:21 | |
| *** salv-orlando has quit IRC | 18:23 | |
| *** salv-orlando has joined #openstack-nova | 18:24 | |
| *** thorst has joined #openstack-nova | 18:24 | |
| openstackgerrit | Merged openstack/nova master: XenAPI: get vGPU stats from hypervisor https://review.openstack.org/512965 | 18:26 |
| *** trozet has quit IRC | 18:28 | |
| *** salv-orlando has quit IRC | 18:28 | |
| *** suresh12 has joined #openstack-nova | 18:30 | |
| *** sambetts is now known as sambetts|afk | 18:30 | |
| *** Apoorva has quit IRC | 18:33 | |
| *** Apoorva has joined #openstack-nova | 18:34 | |
| *** thorst has quit IRC | 18:38 | |
| *** Sanatics has joined #openstack-nova | 18:38 | |
| *** salv-orlando has joined #openstack-nova | 18:39 | |
| *** moshele has joined #openstack-nova | 18:40 | |
| *** Sanatics has quit IRC | 18:40 | |
| *** thorst has joined #openstack-nova | 18:41 | |
| *** moshele has quit IRC | 18:42 | |
| *** thorst has quit IRC | 18:43 | |
| *** tidwellr has joined #openstack-nova | 18:44 | |
| *** rwellum has quit IRC | 18:44 | |
| *** rwellum has joined #openstack-nova | 18:44 | |
| *** jpena|away is now known as jpena|off | 18:48 | |
| *** slaweq_ has joined #openstack-nova | 18:52 | |
| *** fragatina has quit IRC | 18:53 | |
| *** armax has quit IRC | 18:55 | |
| *** slaweq_ has quit IRC | 18:57 | |
| *** jmlowe has joined #openstack-nova | 19:00 | |
| *** erlon has joined #openstack-nova | 19:01 | |
| *** corey_ has joined #openstack-nova | 19:01 | |
| *** cleong has quit IRC | 19:03 | |
| *** sdague has quit IRC | 19:03 | |
| *** adisky_ has quit IRC | 19:10 | |
| *** andreas_s has joined #openstack-nova | 19:17 | |
| *** gouthamr has quit IRC | 19:20 | |
| *** auggy has quit IRC | 19:22 | |
| *** ttsiouts_ has joined #openstack-nova | 19:22 | |
| *** ttsiouts_ has quit IRC | 19:22 | |
| *** gszasz has quit IRC | 19:26 | |
| * edleafe runs off to grab lunch | 19:27 | |
| *** armax has joined #openstack-nova | 19:29 | |
| *** liusheng has quit IRC | 19:29 | |
| *** liusheng has joined #openstack-nova | 19:29 | |
| *** nore__ has joined #openstack-nova | 19:30 | |
| *** slaweq_ has joined #openstack-nova | 19:32 | |
| *** nore_rabel has quit IRC | 19:32 | |
| *** andreas_s has quit IRC | 19:35 | |
| *** fragatina has joined #openstack-nova | 19:37 | |
| *** slaweq_ has quit IRC | 19:37 | |
| *** gouthamr has joined #openstack-nova | 19:37 | |
| *** mriedem has joined #openstack-nova | 19:39 | |
| *** nore__ has quit IRC | 19:40 | |
| *** nore__ has joined #openstack-nova | 19:40 | |
| *** thorst has joined #openstack-nova | 19:42 | |
| efried | stephenfin That's a cool tip, thanks. I doubt it would be considered a "bug" since you're effectively hacking a metadata marker tox uses internally for exactly that purpose. | 19:43 |
| *** tssurya_ has joined #openstack-nova | 19:43 | |
| *** chyka_ has joined #openstack-nova | 19:44 | |
| *** tidwellr has quit IRC | 19:44 | |
| *** tidwellr_ has joined #openstack-nova | 19:44 | |
| *** gouthamr has quit IRC | 19:45 | |
| *** andreas_s has joined #openstack-nova | 19:45 | |
| *** dave-mccowan has joined #openstack-nova | 19:45 | |
| *** _pewp_ has quit IRC | 19:45 | |
| *** penick has joined #openstack-nova | 19:45 | |
| *** _pewp_ has joined #openstack-nova | 19:46 | |
| *** chyka has quit IRC | 19:47 | |
| *** artom has quit IRC | 19:47 | |
| *** artom has joined #openstack-nova | 19:48 | |
| *** chyka_ has quit IRC | 19:49 | |
| *** thorst has quit IRC | 19:51 | |
| *** trozet has joined #openstack-nova | 19:51 | |
| *** chyka has joined #openstack-nova | 19:54 | |
| *** chyka has quit IRC | 19:59 | |
| *** suresh12 has quit IRC | 19:59 | |
| mriedem | edleafe: ok, found a few more missing tests in https://review.openstack.org/#/c/510159/ | 19:59 |
| *** Apoorva has quit IRC | 19:59 | |
| mriedem | edleafe: should probably just add those in there rather than a follow up | 19:59 |
| mriedem | the comment updates could all be done in a follow up | 20:00 |
| *** itlinux has quit IRC | 20:01 | |
| *** slaweq_ has joined #openstack-nova | 20:02 | |
| *** andreas_s has quit IRC | 20:03 | |
| *** thorst has joined #openstack-nova | 20:04 | |
| *** slaweq__ has joined #openstack-nova | 20:05 | |
| *** slaweq_ has quit IRC | 20:07 | |
| mriedem | moving onto the next change in the series | 20:07 |
| *** priteau has joined #openstack-nova | 20:08 | |
| *** slaweq__ has quit IRC | 20:09 | |
| *** Apoorva has joined #openstack-nova | 20:11 | |
| *** sdague has joined #openstack-nova | 20:12 | |
| efried | edleafe jaypipes What about uuid_iter() ? (And make it an iterable) | 20:12 |
| *** slaweq__ has joined #openstack-nova | 20:13 | |
| efried | Ah, I think I remember why I didn't do that originally - don't want the caller holding (and possibly abandoning) the lock. | 20:13 |
| *** suresh12 has joined #openstack-nova | 20:14 | |
| openstackgerrit | Eric Fried proposed openstack/nova master: ProviderTree.get_provider_uuids() https://review.openstack.org/520243 | 20:20 |
| efried | edleafe jaypipes ^ | 20:20 |
| edleafe | mriedem: if I have to change that patch, might as well do 'em all there. | 20:28 |
| jaypipes | efried: +2 from me. nice. | 20:30 |
| mriedem | edleafe: well, if you're going to update all of those comments, | 20:30 |
| efried | jaypipes Thanks | 20:30 |
| jaypipes | efried: though you didn't update the make _provider really private patch. | 20:30 |
| mriedem | you should then do that in the change earlier in the series that actually makes the scheduler drivers return Selection objects back up to the SchedulerManager | 20:30 |
| efried | jaypipes Uh, was I supposed to? | 20:30 |
| jaypipes | efried: nah, just figured you would... rebase the entire thing to get em up to the same rev | 20:31 |
| *** moshele has joined #openstack-nova | 20:31 | |
| efried | jaypipes Hum, that's weird; I wouldn't have thought I needed to do that. | 20:31 |
| jaypipes | efried: since gerrit shows the get_provider_uuids() based on an old rev of the dep patch | 20:31 |
| efried | yeah - why? | 20:31 |
| efried | I rebased both of 'em earlier. | 20:31 |
| mriedem | edleafe: specifically https://review.openstack.org/#/c/495854/ made the comments invalid | 20:31 |
| mriedem | https://review.openstack.org/#/c/510159/ just needs more tests | 20:32 |
| edleafe | mriedem: ok | 20:32 |
| efried | jaypipes ohh, I know what happened. | 20:32 |
| mriedem | edleafe: comments in https://review.openstack.org/#/c/516707/ too - i think we need to not touch anything in the compute manager/claims/RT code there in that patch | 20:32 |
| efried | I manually edited two characters in the commit message of the bottom patch. | 20:32 |
| openstackgerrit | Eric Fried proposed openstack/nova master: ProviderTree.get_provider_uuids() https://review.openstack.org/520243 | 20:33 |
| efried | jaypipes ^ simple rebase from gerrit, kept your +2. Thanks for pointing it out. | 20:33 |
| *** slaweq_ has joined #openstack-nova | 20:33 | |
| *** awaugama has quit IRC | 20:35 | |
| jaypipes | efried: coolio. | 20:36 |
| *** gouthamr has joined #openstack-nova | 20:36 | |
| *** slaweq_ has quit IRC | 20:39 | |
| *** slaweq_ has joined #openstack-nova | 20:40 | |
| openstackgerrit | Dan Smith proposed openstack/nova master: Fix cellsv1 build request deletion if multiple api databases are used https://review.openstack.org/523187 | 20:40 |
| openstackgerrit | Dan Smith proposed openstack/nova master: Fix cellsv1 messaging test https://review.openstack.org/525304 | 20:40 |
| efried | dansmith Got some review bandwidth for these two? https://review.openstack.org/#/c/523932/ https://review.openstack.org/#/c/520243/ They're pretty simple. | 20:41 |
| dansmith | efried: no, and also, you're doing it wrong | 20:41 |
| efried | Well, that figures. What am I doing wrong? | 20:42 |
| dansmith | efried: acting like I have review bandwidth will never work.. playing to my guilt usually will | 20:42 |
| dansmith | watch and learn: | 20:42 |
| dansmith | dansmith: you haven't reviewed any of my patches in a while and I'm starting to feel like you're a bad person. prove me wrong and review these easy ones I have selected for you... | 20:42 |
| efried | okay, let me give that a try: | 20:43 |
| * dansmith waits | 20:43 | |
| mriedem | edleafe: some comments that can be updated in https://review.openstack.org/#/c/511357/ when you update the series | 20:43 |
| *** salv-orlando has quit IRC | 20:43 | |
| efried | dansmith: reviewing these simple patches will not only further the cause of Very Important blueprints, it will make you more handsome and popular. | 20:43 |
| *** salv-orlando has joined #openstack-nova | 20:43 | |
| dansmith | efried: weak, but passable | 20:43 |
| efried | Dangit, yeah, there's almost no guilt in there at all. | 20:43 |
| dansmith | but it plays to other things that work for me, so.. | 20:44 |
| mriedem | passive aggressive works too | 20:44 |
| efried | Like this: dansmith I think it's great how much time you have for cells stuff. | 20:44 |
| mriedem | no | 20:45 |
| efried | damn | 20:45 |
| *** salv-orlando has quit IRC | 20:45 | |
| efried | I really suck at this. | 20:45 |
| dansmith | yeah, that just makes me mad :) | 20:45 |
| *** salv-orlando has joined #openstack-nova | 20:45 | |
| mriedem | heh, yeah :) | 20:45 |
| *** slaweq_ has quit IRC | 20:45 | |
| dansmith | lol | 20:45 |
| mriedem | you need to be a true midwesterner to master passive aggressive | 20:45 |
| efried | My in-laws are from Wisconsin. That's as close as I can come. | 20:45 |
| mriedem | psh, wisconsin | 20:46 |
| efried | If I had my mother in law asking, it would be like, "dansmith, it's okay if you don't want to review my patches; I'll be dead soon and then it won't matter". | 20:46 |
| mriedem | cheese heads | 20:46 |
| mriedem | and serial killers | 20:46 |
| mriedem | efried: that's more like it | 20:46 |
| dansmith | yeah, getting there | 20:47 |
| * efried takes notes | 20:47 | |
| *** clutchbeyers has joined #openstack-nova | 20:50 | |
| *** jaosorior has quit IRC | 20:51 | |
| *** penick has quit IRC | 20:52 | |
| *** suresh12 has quit IRC | 20:52 | |
| *** penick has joined #openstack-nova | 20:52 | |
| *** suresh12 has joined #openstack-nova | 20:53 | |
| *** artom has quit IRC | 20:53 | |
| *** artom has joined #openstack-nova | 20:54 | |
| cfriesen | is it a bug that "nova quota-show" has "injected_file_path_bytes" but "nova limits" doesn't have an equivalent entry? | 20:54 |
| mriedem | cfriesen: meh | 20:54 |
| *** pchavva has quit IRC | 20:55 | |
| mriedem | isn't that maxPersonalitySize? | 20:55 |
| cfriesen | So I see "Personality" and "Personality Size" in limits (in Newton) but for quotas I see injected_files/injected_file_content_bytes/injected_file_path_bytes | 20:56 |
| *** thorst has quit IRC | 20:56 | |
| clutchbeyers | looking to achieve some type of multi tenancy exclusive isolation on host aggregates, I noticed there were some blueprints that were abandoned. Is anyone aware of any work being done or other mechanism to exclude tenants to an aggregate or global filter that wont provision a tenant to an aggregate that doesnt have such filter meta set? | 20:57 |
| *** awolf has joined #openstack-nova | 20:57 | |
| melwitt | cfriesen: did it use to show it in 'nova limits' but doesn't anymore? if so, that's a bug | 20:58 |
| melwitt | oh, newton. then not a bug. quota rewriting didn't happen until pike | 20:59 |
| melwitt | or rather, not a regression. one could argue whether it's a bug in general | 21:00 |
| cfriesen | melwitt: okay, will be upgrading soon. | 21:00 |
| cfriesen | why do we have both APIs anyway? | 21:00 |
| openstackgerrit | Eric Fried proposed openstack/nova stable/ocata: Proper error handling by _ensure_resource_provider https://review.openstack.org/525309 | 21:00 |
| cfriesen | qutoa and limits I mean | 21:01 |
| efried | mriedem ^ There's that cherry-pick. That was a pretty fun merge. | 21:01 |
| melwitt | that, I don't know | 21:01 |
| cfriesen | okay, thanks | 21:01 |
| mriedem | cfriesen: i'd bet it was likely due to v2.0 api extension stuff | 21:02 |
| *** chyka has joined #openstack-nova | 21:02 | |
| *** thorst has joined #openstack-nova | 21:02 | |
| *** chyka has quit IRC | 21:02 | |
| mriedem | clutchbeyers: https://review.openstack.org/#/c/381912/ | 21:04 |
| mriedem | clutchbeyers: no one is working on that | 21:05 |
| *** chyka has joined #openstack-nova | 21:05 | |
| mriedem | melwitt looked at something related recently, saying maybe it was a regression that we needed this anyway | 21:05 |
| *** itlinux has joined #openstack-nova | 21:05 | |
| melwitt | I think that was the lack of counting keypairs for the used limits API. it always returns 0 | 21:05 |
| *** itlinux has quit IRC | 21:06 | |
| * melwitt should actually read the link first | 21:06 | |
| *** thorst has quit IRC | 21:07 | |
| melwitt | this is the redhat bug where it's demonstrated to be a regression in mitaka https://bugzilla.redhat.com/show_bug.cgi?id=1435597 | 21:09 |
| openstack | bugzilla.redhat.com bug 1435597 in openstack-nova "AggegateImagePropertiesIsolatefiler not working" [High,New] - Assigned to sbauza | 21:09 |
| *** slaweq_ has joined #openstack-nova | 21:10 | |
| melwitt | given that, I was a bit lost about how it got turned into a blueprint-required feature | 21:11 |
| melwitt | I want to dig into it more to find out what actually happened back then | 21:11 |
| *** suresh12 has quit IRC | 21:13 | |
| melwitt | i.e. find what changed that made it stop working and whether it was intentional | 21:14 |
| *** suresh12 has joined #openstack-nova | 21:15 | |
| *** slaweq_ has quit IRC | 21:15 | |
| melwitt | I had done some looking while at the summit but didn't find anything yet | 21:15 |
| *** awaugama has joined #openstack-nova | 21:16 | |
| *** thorst has joined #openstack-nova | 21:17 | |
| *** nicolasbock has joined #openstack-nova | 21:19 | |
| *** suresh12 has quit IRC | 21:19 | |
| *** liverpooler has quit IRC | 21:19 | |
| *** tssurya_ has quit IRC | 21:20 | |
| *** thorst has quit IRC | 21:20 | |
| *** thorst has joined #openstack-nova | 21:21 | |
| *** AJaeger_ has left #openstack-nova | 21:21 | |
| *** smatzek has quit IRC | 21:23 | |
| mriedem | huh, how does this ever work? | 21:25 |
| mriedem | https://github.com/openstack/nova/blob/master/nova/compute/manager.py#L6595 | 21:25 |
| mriedem | ah oh nvm | 21:25 |
| *** yamahata has joined #openstack-nova | 21:26 | |
| *** corey_ has quit IRC | 21:30 | |
| *** yamahata has quit IRC | 21:32 | |
| *** tidwellr has joined #openstack-nova | 21:34 | |
| *** tidwellr_ has quit IRC | 21:34 | |
| *** penick has quit IRC | 21:36 | |
| *** thorst has quit IRC | 21:36 | |
| *** penick has joined #openstack-nova | 21:37 | |
| *** pcaruana has quit IRC | 21:39 | |
| mikal | mriedem: I commented on that bug, but mostly to say "you people confuse me" | 21:39 |
| mikal | In other news, Amazon shopping just launched in my country | 21:40 |
| *** moshele has quit IRC | 21:40 | |
| mriedem | mikal: the perf one? | 21:40 |
| mikal | Yep, https://bugs.launchpad.net/nova/+bug/1735663 | 21:46 |
| openstack | Launchpad bug 1735663 in OpenStack Compute (nova) "privsep.daemon takes 9 seconds during taking volume attaching operation" [Undecided,New] | 21:46 |
| *** slaweq_ has joined #openstack-nova | 21:46 | |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: Add regression test for rebuilding a volume-backed server https://review.openstack.org/521200 | 21:47 |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: Get original image_id from volume for volume-backed instance rebuild https://review.openstack.org/521391 | 21:47 |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: Fail fast if changing image on a volume-backed server rebuild https://review.openstack.org/520660 | 21:47 |
| mriedem | dansmith: think i addressed your comments in ^ | 21:48 |
| dansmith | slright | 21:48 |
| *** thorst has joined #openstack-nova | 21:50 | |
| *** slaweq_ has quit IRC | 21:51 | |
| *** thorst has quit IRC | 21:54 | |
| *** mriedem has quit IRC | 21:56 | |
| *** nore__ has quit IRC | 21:56 | |
| *** clutchbeyers has quit IRC | 21:56 | |
| *** slaweq__ has quit IRC | 21:58 | |
| *** Apoorva_ has joined #openstack-nova | 22:02 | |
| *** takashin has joined #openstack-nova | 22:02 | |
| *** tidwellr has quit IRC | 22:04 | |
| *** gouthamr has quit IRC | 22:04 | |
| *** Apoorva has quit IRC | 22:05 | |
| *** rcernin has joined #openstack-nova | 22:07 | |
| openstackgerrit | Takashi NATSUME proposed openstack/nova master: [placement] Add aggregate link note in API ref https://review.openstack.org/521502 | 22:07 |
| openstackgerrit | Takashi NATSUME proposed openstack/nova master: [placement] Add functional tests for traits API https://review.openstack.org/524094 | 22:07 |
| openstackgerrit | Takashi NATSUME proposed openstack/nova master: [placement] Add 'Location' parameters in API ref https://review.openstack.org/521541 | 22:08 |
| openstackgerrit | Takashi NATSUME proposed openstack/nova master: [cellv2] Improve getting BDMs in multiple cells https://review.openstack.org/521400 | 22:08 |
| openstackgerrit | Takashi NATSUME proposed openstack/nova master: Add a warning in 'nova-manage cell_v2 delete_cell' https://review.openstack.org/513771 | 22:08 |
| openstackgerrit | Takashi NATSUME proposed openstack/nova master: [placement] Add functional tests for resource class API https://review.openstack.org/524506 | 22:09 |
| *** thorst has joined #openstack-nova | 22:10 | |
| openstackgerrit | Takashi NATSUME proposed openstack/nova master: [placement] Fix foreign key constraint error https://review.openstack.org/516880 | 22:10 |
| openstackgerrit | Takashi NATSUME proposed openstack/nova master: [placement] Fix getting placement request ID https://review.openstack.org/523606 | 22:11 |
| openstackgerrit | Takashi NATSUME proposed openstack/nova master: Fix missing marker functions https://review.openstack.org/514579 | 22:11 |
| *** peter-hamilton has quit IRC | 22:11 | |
| openstackgerrit | Hongbin Lu proposed openstack/nova master: [WIP] Handle RPC timeout on get_console_output https://review.openstack.org/525335 | 22:12 |
| *** namnh has joined #openstack-nova | 22:14 | |
| *** thorst has quit IRC | 22:14 | |
| *** salv-orlando has quit IRC | 22:15 | |
| *** salv-orlando has joined #openstack-nova | 22:15 | |
| *** slaweq_ has joined #openstack-nova | 22:19 | |
| *** salv-orlando has quit IRC | 22:20 | |
| *** awolf has quit IRC | 22:20 | |
| *** namnh has quit IRC | 22:21 | |
| efried | jaypipes Been staring at this for a bit, can't figure out where 1.10 is coming from: http://logs.openstack.org/93/385693/83/check/legacy-grenade-dsvm-neutron/b513223/logs/screen-n-cpu.txt.gz#_Dec_04_15_47_11_988730 | 22:21 |
| *** slaweq_ has quit IRC | 22:24 | |
| *** awaugama has quit IRC | 22:25 | |
| *** salv-orlando has joined #openstack-nova | 22:27 | |
| *** BryanS68 has quit IRC | 22:30 | |
| gryf | efried, looks like you run pike version of placement against latest source | 22:31 |
| efried | gryf Wow, that would be... bad. How can you tell? | 22:31 |
| gryf | 1.10 was the max microversion in pike | 22:32 |
| *** Apoorva_ has quit IRC | 22:33 | |
| efried | cdent If you're still around -- how does legacy-grenade-dsvm-neutron spin up the placement API? Is there some chance we could be using the pike version in latest master? | 22:33 |
| *** armax has quit IRC | 22:33 | |
| efried | cdent And wouldn't lots of other stuff have broken by now if that were the case? | 22:33 |
| *** Apoorva has joined #openstack-nova | 22:33 | |
| cdent | efried: I assume you’re talking about the thing that’s claiming 1.10 is the max version? | 22:33 |
| efried | cdent yeah | 22:34 |
| cdent | was just reading that in my email | 22:34 |
| cdent | it could be that a new compute is talking to an older placement, which I think is something we _may_ do intentionally in some tests | 22:35 |
| efried | oy vay, that could be... complicated to fix. | 22:35 |
| cdent | and for the most part we’ve tried to make the report client be smart about that | 22:36 |
| * cdent looks at the code in the changes | 22:36 | |
| *** armax has joined #openstack-nova | 22:36 | |
| cdent | efried: hmmm. yeah, that code doesn’t have a fall back, but I’m not sure we want that or not. jaypipes, mriedem or dansmith will be able to say more on whether we need to be able to work with new compute/old placement | 22:37 |
| efried | cdent Okay, I gotcha. | 22:38 |
| cdent | If we don’t need to support that, the other chance is simply that the new placement hasn’t manage to get restarted yet | 22:38 |
| dansmith | old compute needs to work with new placement | 22:38 |
| cdent | dansmith: yes, but vice versa? | 22:38 |
| dansmith | not vice versa | 22:39 |
| efried | cdent "hasn't managed to get restarted yet" - so actually a recheck may clear it? | 22:39 |
| cdent | efried: probably, but it may be that the process mgt in grenade is insufficiently robust and could be robustified | 22:40 |
| efried | BTW, this was just exposed because of https://review.openstack.org/#/c/524263/ -- I think otherwise we would have blown right by this error and not noticed. So it may be a long-standing latent bug. | 22:40 |
| cdent | yup, I dig | 22:41 |
| *** lyan has quit IRC | 22:41 | |
| cdent | efried: if you haven’t got time to think about grenade right now (or ever), I’m happy to go digging instead, but wouldn’t be until tomorrow | 22:41 |
| efried | Well, I guess I'll try a recheck, but it seems like something that ought to be fixed *somewhere* | 22:42 |
| efried | cdent I've never heard of grenade. | 22:42 |
| efried | And it's jaypipes' patch anyway :P | 22:42 |
| cdent | code ownership is, like, _so_ 1990, but yeah, my curiosity is piqued | 22:43 |
| cdent | grenade is the sort of framework used to manage upgrade testing | 22:43 |
| cdent | a devstack in the previous version is started, verified a bit | 22:43 |
| cdent | then new code is installed, but old configruations kept | 22:44 |
| *** Yingxin has quit IRC | 22:44 | |
| *** Yingxin has joined #openstack-nova | 22:48 | |
| openstackgerrit | Ed Leafe proposed openstack/nova master: Return Selection objects from the scheduler driver https://review.openstack.org/495854 | 22:50 |
| openstackgerrit | Ed Leafe proposed openstack/nova master: Move the to_dict() method to the Selection object https://review.openstack.org/523492 | 22:50 |
| openstackgerrit | Ed Leafe proposed openstack/nova master: Modify select_destinations() to return objects and alts https://review.openstack.org/510159 | 22:50 |
| edleafe | mriedem: ^^ addressed your comments | 22:50 |
| edleafe | mriedem: will start on https://review.openstack.org/#/c/516707/ tomorrow | 22:51 |
| *** namnh has joined #openstack-nova | 22:51 | |
| openstackgerrit | Eric Fried proposed openstack/nova master: placement: allow filter providers in tree https://review.openstack.org/377215 | 22:52 |
| openstackgerrit | Eric Fried proposed openstack/nova master: placement: adds REST API for nested providers https://review.openstack.org/384807 | 22:52 |
| openstackgerrit | Eric Fried proposed openstack/nova master: placement: update client to set parent provider https://review.openstack.org/385693 | 22:52 |
| efried | jaypipes ^ this is just a rebase on top of _Provider-private so I can start rebasing the other stuff in that series (e.g. _set_inventory doing nested stuff) | 22:52 |
| *** felipemonteiro_ has quit IRC | 22:53 | |
| *** thorst has joined #openstack-nova | 22:53 | |
| *** priteau has quit IRC | 22:54 | |
| *** namnh has quit IRC | 22:56 | |
| *** slaweq_ has joined #openstack-nova | 22:57 | |
| *** thorst has quit IRC | 22:58 | |
| jaypipes | efried: roger | 22:59 |
| *** suresh12 has joined #openstack-nova | 23:00 | |
| *** slaweq_ has quit IRC | 23:02 | |
| *** namnh has joined #openstack-nova | 23:03 | |
| *** namnh has quit IRC | 23:08 | |
| *** gouthamr has joined #openstack-nova | 23:08 | |
| *** esberglu has quit IRC | 23:18 | |
| *** namnh has joined #openstack-nova | 23:19 | |
| *** esberglu has joined #openstack-nova | 23:19 | |
| *** mlavalle has quit IRC | 23:21 | |
| *** namnh has quit IRC | 23:23 | |
| *** esberglu has quit IRC | 23:23 | |
| openstackgerrit | Eric Fried proposed openstack/nova master: Scheduler set_inventory_for_provider does nested https://review.openstack.org/520643 | 23:24 |
| efried | jaypipes Like that ^ | 23:24 |
| efried | jaypipes That _validate_provider thing really comes into its own now. | 23:24 |
| efried | in the tests | 23:24 |
| *** gmann_afk is now known as gmann | 23:27 | |
| efried | jaypipes cdent et al: FYI I'm out tomorrow; working W, Th, and first half of Fr; then (officially) out til Jan. I'll probably be able to check in periodically to shepherd simple rebases/reviews, but not a lot of big stuff. | 23:28 |
| *** namnh has joined #openstack-nova | 23:28 | |
| cdent | efried: roger that. I’m fairly around throughout most of december, happy to mangle your stuff as required during that time | 23:29 |
| efried | cdent I really hope to get things in a state where that would be remotely possible before I bugger off. Right now most of my patches are kind of a mess. | 23:29 |
| efried | cdent And thank you for the offer. | 23:30 |
| *** esberglu has joined #openstack-nova | 23:31 | |
| *** thorst has joined #openstack-nova | 23:33 | |
| *** namnh has quit IRC | 23:33 | |
| *** thorst has quit IRC | 23:33 | |
| *** slaweq_ has joined #openstack-nova | 23:35 | |
| *** penick has quit IRC | 23:37 | |
| *** armax has quit IRC | 23:38 | |
| *** yamamoto has joined #openstack-nova | 23:38 | |
| *** smatzek has joined #openstack-nova | 23:38 | |
| *** esberglu has quit IRC | 23:39 | |
| *** slaweq_ has quit IRC | 23:40 | |
| *** slaweq_ has joined #openstack-nova | 23:41 | |
| *** efried is now known as efried_cya_wed | 23:41 | |
| *** namnh has joined #openstack-nova | 23:42 | |
| *** smatzek has quit IRC | 23:43 | |
| *** thorst has joined #openstack-nova | 23:45 | |
| *** slaweq_ has quit IRC | 23:46 | |
| *** namnh has quit IRC | 23:47 | |
| *** thorst has quit IRC | 23:49 | |
| *** thorst has joined #openstack-nova | 23:51 | |
| openstackgerrit | Hongbin Lu proposed openstack/nova master: Add support for listing hosts in cellv2 https://review.openstack.org/524755 | 23:52 |
| *** trinaths has joined #openstack-nova | 23:52 | |
| *** armax has joined #openstack-nova | 23:53 | |
| *** jmlowe has quit IRC | 23:55 | |
| *** thorst has quit IRC | 23:55 | |
| *** gouthamr has quit IRC | 23:56 | |
| *** gouthamr has joined #openstack-nova | 23:58 | |
| *** namnh has joined #openstack-nova | 23:58 | |
| *** hongbin has quit IRC | 23:59 | |
| *** ankit_ has quit IRC | 23:59 | |
| *** felipemonteiro has joined #openstack-nova | 23:59 | |
| *** danice has joined #openstack-nova | 23:59 | |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!