openstackgerrit | Michael Still proposed openstack/nova master: Move nbd commands to privsep. https://review.openstack.org/500351 | 00:00 |
---|---|---|
openstackgerrit | Michael Still proposed openstack/nova master: Move lvm handling to privsep. https://review.openstack.org/495516 | 00:00 |
openstackgerrit | Michael Still proposed openstack/nova master: Move xend existence probes to privsep. https://review.openstack.org/495538 | 00:00 |
openstackgerrit | Michael Still proposed openstack/nova master: Move shred to privsep. https://review.openstack.org/495537 | 00:00 |
openstackgerrit | Michael Still proposed openstack/nova master: Cleanup mount / umount and associated rmdir calls https://review.openstack.org/494423 | 00:00 |
openstackgerrit | Michael Still proposed openstack/nova master: WIP / Aspirational: we don't need rootwrap any more. https://review.openstack.org/495542 | 00:00 |
openstackgerrit | Michael Still proposed openstack/nova master: Move loopback setup and removal to privsep. https://review.openstack.org/495664 | 00:00 |
openstackgerrit | Michael Still proposed openstack/nova master: Move libvirt usages of chown to privsep. https://review.openstack.org/471972 | 00:00 |
openstackgerrit | Michael Still proposed openstack/nova master: Don't shell out to mkdir, use ensure_tree() https://review.openstack.org/492326 | 00:00 |
openstackgerrit | Michael Still proposed openstack/nova master: Move ploop commands to privsep. https://review.openstack.org/492325 | 00:00 |
openstackgerrit | Michael Still proposed openstack/nova master: Read from console ptys using privsep. https://review.openstack.org/489486 | 00:00 |
openstackgerrit | Michael Still proposed openstack/nova master: Move the idmapshift binary into privsep. https://review.openstack.org/495541 | 00:01 |
openstackgerrit | Michael Still proposed openstack/nova master: Move kpartx calls to privsep. https://review.openstack.org/500354 | 00:01 |
openstackgerrit | Michael Still proposed openstack/nova master: Move execs of touch to privsep. https://review.openstack.org/489190 | 00:01 |
openstackgerrit | Michael Still proposed openstack/nova master: Move libvirts dmcrypt support to privsep. https://review.openstack.org/490737 | 00:01 |
openstackgerrit | Michael Still proposed openstack/nova master: Move blkid calls to privsep. https://review.openstack.org/500398 | 00:01 |
openstackgerrit | Michael Still proposed openstack/nova master: Move execs of tee to privsep. https://review.openstack.org/489438 | 00:01 |
*** armax has quit IRC | 00:02 | |
*** priteau has quit IRC | 00:04 | |
*** gyee has quit IRC | 00:04 | |
*** yangyapeng has joined #openstack-nova | 00:06 | |
*** yangyapeng has quit IRC | 00:11 | |
*** felipemonteiro has quit IRC | 00:11 | |
*** rajathagasthya has quit IRC | 00:12 | |
mriedem | melwitt: i'm pretty close to having a test now | 00:12 |
mriedem | ok got it | 00:18 |
*** thorst has quit IRC | 00:18 | |
*** thorst has joined #openstack-nova | 00:18 | |
*** itlinux has joined #openstack-nova | 00:18 | |
*** catintheroof has joined #openstack-nova | 00:18 | |
openstackgerrit | Brianna Poulos proposed openstack/nova master: Add trusted_certificates to REST API https://review.openstack.org/486204 | 00:20 |
openstackgerrit | Matt Riedemann proposed openstack/nova master: Track which cell each instance is created in and use it consistently https://review.openstack.org/501452 | 00:22 |
openstackgerrit | Matt Riedemann proposed openstack/nova master: Ensure instance mapping is updated in case of quota recheck fails https://review.openstack.org/501408 | 00:22 |
mriedem | melwitt: here we go, ready for review ^ | 00:22 |
mriedem | alex_xu: ^ these fix a couple of serious bugs for cells v2 in pike that we'll want to get backported soon | 00:22 |
*** thorst has quit IRC | 00:23 | |
mikal | sdague: that upload should address your concerns with the next patch in the series of doom | 00:26 |
*** hshiina has joined #openstack-nova | 00:26 | |
*** suresh12 has quit IRC | 00:29 | |
*** suresh12 has joined #openstack-nova | 00:29 | |
*** Swami has quit IRC | 00:29 | |
*** catintheroof has quit IRC | 00:31 | |
*** thorst has joined #openstack-nova | 00:32 | |
*** sdague has quit IRC | 00:36 | |
openstackgerrit | Chris Dent proposed openstack/nova master: WIP: [placement] POST /allocations to set allocations for >1 consumers https://review.openstack.org/500073 | 00:37 |
openstackgerrit | Chris Dent proposed openstack/nova master: [placement] Allow _set_allocations to delete allocations https://review.openstack.org/501051 | 00:37 |
*** jichen has joined #openstack-nova | 00:38 | |
*** slaweq has joined #openstack-nova | 00:39 | |
*** thorst has quit IRC | 00:39 | |
*** thorst has joined #openstack-nova | 00:40 | |
*** slaweq has quit IRC | 00:44 | |
*** thorst has quit IRC | 00:44 | |
*** harlowja has quit IRC | 00:48 | |
*** claudiub has quit IRC | 00:49 | |
*** esberglu has quit IRC | 00:50 | |
*** Apoorva_ has quit IRC | 00:51 | |
*** kiennt has joined #openstack-nova | 00:54 | |
*** dixiaoli has joined #openstack-nova | 00:57 | |
*** phuongnh has joined #openstack-nova | 01:01 | |
*** baoli has joined #openstack-nova | 01:03 | |
*** litao__ has joined #openstack-nova | 01:05 | |
*** thorst has joined #openstack-nova | 01:08 | |
*** thorst has quit IRC | 01:09 | |
*** gbarros has joined #openstack-nova | 01:12 | |
*** zhurong has joined #openstack-nova | 01:15 | |
*** yangyapeng has joined #openstack-nova | 01:15 | |
*** tetsuro has joined #openstack-nova | 01:17 | |
openstackgerrit | jichenjc proposed openstack/nova master: WIP:Remove quota_networks and enable_network_quota https://review.openstack.org/501068 | 01:19 |
openstackgerrit | jichenjc proposed openstack/nova master: WIP: remove vnedordata_driver https://review.openstack.org/501507 | 01:19 |
dansmith | melwitt: I didn't see it before I left, did you do it? | 01:24 |
dansmith | mriedem: heh, that is almost exactly what I did on the plane | 01:25 |
dansmith | but didn't get a full test run to see the failures before I landed | 01:25 |
mriedem | dansmith: i've got ConductorTaskTestCase working locally with multiple cells, but that made some other tests fail, so i'm splitting that out in the series | 01:26 |
mriedem | to ungorpify it | 01:26 |
dansmith | cool | 01:26 |
melwitt | dansmith: no, I assumed you were doing it bc I assumed you didn't see my message | 01:28 |
dansmith | melwitt: ack, and I did, but sounds like mriedem is on it | 01:29 |
melwitt | didn't want to redo the doing | 01:29 |
melwitt | okay | 01:29 |
*** gbarros has quit IRC | 01:31 | |
openstackgerrit | jichenjc proposed openstack/nova master: WIP: remove CONF.vendordata_driver https://review.openstack.org/501510 | 01:31 |
openstackgerrit | jichenjc proposed openstack/nova master: Check root disk size when doing resize in API https://review.openstack.org/484664 | 01:32 |
mriedem | ok i'm about ready to push | 01:34 |
mriedem | just running fast8 | 01:34 |
mriedem | PUSH! | 01:35 |
openstackgerrit | Matt Riedemann proposed openstack/nova master: Track which cell each instance is created in and use it consistently https://review.openstack.org/501452 | 01:35 |
openstackgerrit | Matt Riedemann proposed openstack/nova master: Ensure instance mapping is updated in case of quota recheck fails https://review.openstack.org/501408 | 01:35 |
openstackgerrit | Matt Riedemann proposed openstack/nova master: Allow setting up multiple cells in the base TestCase https://review.openstack.org/501513 | 01:35 |
openstackgerrit | Matt Riedemann proposed openstack/nova master: Make ConductorTaskTestCase run with 2 cells https://review.openstack.org/501514 | 01:35 |
mriedem | dansmith: melwitt: ^ | 01:35 |
mriedem | i have not run a full py27 run on that, only the conductor task test case tests | 01:35 |
mriedem | and now i go to enjoy marital bliss for 30 minutes | 01:38 |
dansmith | wow 30 whole minutes? | 01:38 |
mriedem | in a purely non-physical way... | 01:38 |
openstackgerrit | jichenjc proposed openstack/nova master: check query param for used_limits function https://review.openstack.org/499091 | 01:39 |
dansmith | oh | 01:39 |
dansmith | then | 01:39 |
dansmith | oh | 01:39 |
*** krtaylor has joined #openstack-nova | 01:39 | |
*** slaweq has joined #openstack-nova | 01:40 | |
*** slaweq has quit IRC | 01:45 | |
*** suresh12 has quit IRC | 01:49 | |
*** Tom has joined #openstack-nova | 01:52 | |
*** Tom has quit IRC | 01:52 | |
*** Tom has joined #openstack-nova | 01:52 | |
*** Tom has quit IRC | 01:52 | |
*** hongbin has joined #openstack-nova | 01:54 | |
openstackgerrit | jichenjc proposed openstack/nova master: check query param for used_limits function https://review.openstack.org/499091 | 01:57 |
*** trungnv has joined #openstack-nova | 01:58 | |
*** abhi89 has joined #openstack-nova | 01:58 | |
*** trungnv has quit IRC | 02:04 | |
openstackgerrit | jichenjc proposed openstack/nova master: check query param for service's index function https://review.openstack.org/489492 | 02:06 |
*** abhi89 has quit IRC | 02:08 | |
*** yamamoto_ has joined #openstack-nova | 02:09 | |
*** bkopilov has quit IRC | 02:10 | |
*** markvoelker has quit IRC | 02:11 | |
openstackgerrit | jichenjc proposed openstack/nova master: Add quiesce and unquiesce in support matrix https://review.openstack.org/482390 | 02:12 |
openstackgerrit | jichenjc proposed openstack/nova master: fix warning: unknown option enable-extensions https://review.openstack.org/497170 | 02:14 |
*** swamireddy has quit IRC | 02:14 | |
*** yamahata has quit IRC | 02:15 | |
*** thorst has joined #openstack-nova | 02:15 | |
*** thorst has quit IRC | 02:15 | |
*** moshele has joined #openstack-nova | 02:16 | |
*** trungnv has joined #openstack-nova | 02:19 | |
mnaser | oh ey | 02:22 |
mnaser | i found new ways of breaking nova | 02:22 |
mnaser | delete an instance during a live migration | 02:22 |
mriedem | what does that break? | 02:23 |
openstackgerrit | Merged openstack/nova master: Provide hints when nova-manage db sync fails to sync cell0 https://review.openstack.org/486660 | 02:23 |
mriedem | besides knowing we'll be hitting weird InstanceNotFound things | 02:23 |
mriedem | and a half migrated guest | 02:23 |
mnaser | let me show you the fun log | 02:24 |
mnaser | mriedem http://paste.openstack.org/show/620587/ i think its also a combo involving libvirt too | 02:25 |
*** suresh12 has joined #openstack-nova | 02:27 | |
*** vladikr has quit IRC | 02:28 | |
mriedem | UnexpectedDeletingTaskStateError is expected | 02:29 |
mriedem | libvirt crashing is not | 02:29 |
mnaser | the process clearly crashed too cause of the start up time for the proc | 02:29 |
mnaser | systemd[1]: libvirtd.service: main process exited, code=killed, status=11/SEGV | 02:30 |
mnaser | well at least nova's in the claer | 02:30 |
mnaser | but that cant be bueno | 02:30 |
openstackgerrit | Tetsuro Nakamura proposed openstack/nova master: disable numa feature when virt_type is not kvm https://review.openstack.org/465160 | 02:30 |
mriedem | what version of libvirt? | 02:31 |
openstackgerrit | Hironori Shiina proposed openstack/nova master: Ironic: Get IP address for volume connector https://review.openstack.org/468353 | 02:31 |
openstackgerrit | jichenjc proposed openstack/nova master: Add quiesce and unquiesce in support matrix https://review.openstack.org/482390 | 02:31 |
*** suresh12 has quit IRC | 02:31 | |
mnaser | libvirt-daemon-2.0.0-10.el7_3.9.x86_64 | 02:32 |
mnaser | quite old, but its what the qemu-ev ships with | 02:33 |
mnaser | http://mirror.centos.org/centos/7/virt/x86_64/libvirt-latest/ looks like centos sig ships this, but dunno what sort of testing it gets | 02:34 |
*** adisky__ has joined #openstack-nova | 02:34 | |
*** ijw has joined #openstack-nova | 02:35 | |
*** itlinux has quit IRC | 02:37 | |
*** trungnv has quit IRC | 02:38 | |
*** zxy has quit IRC | 02:39 | |
*** itlinux has joined #openstack-nova | 02:40 | |
*** ijw has quit IRC | 02:40 | |
*** vladikr has joined #openstack-nova | 02:41 | |
*** itlinux has quit IRC | 02:41 | |
*** slaweq has joined #openstack-nova | 02:41 | |
openstackgerrit | Jeremy Liu proposed openstack/nova master: Implement keymgr list() method https://review.openstack.org/501533 | 02:42 |
*** zxy has joined #openstack-nova | 02:42 | |
*** slaweq has quit IRC | 02:46 | |
*** baoli has quit IRC | 02:50 | |
*** baoli has joined #openstack-nova | 02:50 | |
*** litao__ has quit IRC | 02:53 | |
*** litao__ has joined #openstack-nova | 02:53 | |
*** Tom has joined #openstack-nova | 02:58 | |
openstackgerrit | Jeremy Liu proposed openstack/nova master: Implement keymgr list() method https://review.openstack.org/501533 | 02:59 |
*** baoli has quit IRC | 03:01 | |
*** gvrangan has joined #openstack-nova | 03:03 | |
openstackgerrit | jichenjc proposed openstack/nova master: Add quiesce and unquiesce in support matrix https://review.openstack.org/482390 | 03:03 |
*** gvrangan has quit IRC | 03:04 | |
*** gvrangan has joined #openstack-nova | 03:04 | |
*** rajathagasthya has joined #openstack-nova | 03:06 | |
*** ssmith has joined #openstack-nova | 03:08 | |
*** brault has quit IRC | 03:09 | |
*** markvoelker has joined #openstack-nova | 03:11 | |
openstackgerrit | Ildiko Vancsa proposed openstack/nova master: Add attachment_complete call to volume/cinder.py https://review.openstack.org/493323 | 03:12 |
openstackgerrit | Ildiko Vancsa proposed openstack/nova master: Tweak connection_info translation for the new Cinder attach/detach API https://review.openstack.org/493324 | 03:12 |
openstackgerrit | Ildiko Vancsa proposed openstack/nova master: Implement new attach Cinder flow https://review.openstack.org/330285 | 03:12 |
*** abhi89 has joined #openstack-nova | 03:15 | |
*** vladikr has quit IRC | 03:16 | |
*** vladikr has joined #openstack-nova | 03:16 | |
*** TuanLA has joined #openstack-nova | 03:17 | |
*** itlinux has joined #openstack-nova | 03:19 | |
*** bkopilov has joined #openstack-nova | 03:28 | |
*** ssmith has quit IRC | 03:30 | |
*** trungnv has joined #openstack-nova | 03:32 | |
*** yangyape_ has joined #openstack-nova | 03:34 | |
*** yingjun has joined #openstack-nova | 03:36 | |
*** yangyapeng has quit IRC | 03:37 | |
*** vladikr has quit IRC | 03:40 | |
*** slaweq has joined #openstack-nova | 03:42 | |
*** vladikr has joined #openstack-nova | 03:44 | |
*** markvoelker has quit IRC | 03:46 | |
*** hshiina has quit IRC | 03:47 | |
*** slaweq has quit IRC | 03:47 | |
*** yangyape_ has quit IRC | 03:47 | |
*** yangyapeng has joined #openstack-nova | 03:48 | |
*** udesale has joined #openstack-nova | 03:50 | |
*** manasm has joined #openstack-nova | 03:50 | |
*** Tom has quit IRC | 03:52 | |
*** swamireddy has joined #openstack-nova | 03:53 | |
*** takashin has joined #openstack-nova | 03:56 | |
*** vladikr has quit IRC | 04:00 | |
*** hshiina has joined #openstack-nova | 04:01 | |
*** priteau has joined #openstack-nova | 04:01 | |
*** yangyape_ has joined #openstack-nova | 04:05 | |
*** priteau has quit IRC | 04:05 | |
*** harlowja has joined #openstack-nova | 04:06 | |
*** yangyapeng has quit IRC | 04:06 | |
*** trungnv has quit IRC | 04:07 | |
*** Tom has joined #openstack-nova | 04:08 | |
*** armax has joined #openstack-nova | 04:08 | |
*** Tom has quit IRC | 04:12 | |
*** thorst has joined #openstack-nova | 04:16 | |
*** swamireddy has quit IRC | 04:19 | |
*** yamamot__ has joined #openstack-nova | 04:20 | |
*** armax_ has joined #openstack-nova | 04:20 | |
*** spotz has quit IRC | 04:20 | |
*** jistr|off has quit IRC | 04:20 | |
*** edleafe has quit IRC | 04:20 | |
openstackgerrit | Takashi NATSUME proposed openstack/nova master: Update contributor guide for Queens https://review.openstack.org/500454 | 04:21 |
*** thorst has quit IRC | 04:21 | |
*** armax has quit IRC | 04:21 | |
*** armax_ is now known as armax | 04:21 | |
*** xinliang has quit IRC | 04:21 | |
*** cshen has quit IRC | 04:21 | |
*** cshen has joined #openstack-nova | 04:21 | |
*** xinliang has joined #openstack-nova | 04:21 | |
*** yamamoto_ has quit IRC | 04:21 | |
*** kevinbenton has quit IRC | 04:21 | |
*** spotz has joined #openstack-nova | 04:22 | |
*** edleafe has joined #openstack-nova | 04:22 | |
*** kevinbenton has joined #openstack-nova | 04:23 | |
*** swamireddy has joined #openstack-nova | 04:23 | |
openstackgerrit | Takashi NATSUME proposed openstack/nova-specs master: List/show all server migration types https://review.openstack.org/489029 | 04:23 |
*** gvrangan has quit IRC | 04:24 | |
*** vvargaszte has joined #openstack-nova | 04:24 | |
*** jistr has joined #openstack-nova | 04:24 | |
*** hongbin has quit IRC | 04:26 | |
*** vvargaszte has quit IRC | 04:28 | |
*** gouthamr has quit IRC | 04:31 | |
*** vvargaszte has joined #openstack-nova | 04:32 | |
*** vvargaszte has quit IRC | 04:36 | |
*** zhurong has quit IRC | 04:38 | |
*** moshele has quit IRC | 04:39 | |
*** dims has quit IRC | 04:40 | |
*** markvoelker has joined #openstack-nova | 04:43 | |
openstackgerrit | Jackie Truong proposed openstack/python-novaclient master: Microversion 2.54 - Add trusted_certificates param https://review.openstack.org/500396 | 04:43 |
openstackgerrit | Jackie Truong proposed openstack/python-novaclient master: Microversion 2.54 - Add trusted_certificates param https://review.openstack.org/500396 | 04:46 |
*** rmcall has quit IRC | 04:48 | |
*** ekuris has joined #openstack-nova | 04:51 | |
*** psachin has joined #openstack-nova | 04:51 | |
*** rmcall has joined #openstack-nova | 04:51 | |
*** boris_42 has joined #openstack-nova | 04:52 | |
*** Sree_ has joined #openstack-nova | 05:01 | |
*** Sree_ is now known as Guest75181 | 05:01 | |
*** diga has joined #openstack-nova | 05:03 | |
*** rajathagasthya has quit IRC | 05:07 | |
*** yamamot__ has quit IRC | 05:07 | |
*** yamamoto_ has joined #openstack-nova | 05:07 | |
*** zhurong has joined #openstack-nova | 05:08 | |
openstackgerrit | Andreas Jaeger proposed openstack/nova master: Fix broken URLs https://review.openstack.org/501402 | 05:10 |
*** Tom has joined #openstack-nova | 05:10 | |
*** harlowja has quit IRC | 05:11 | |
*** pcaruana has joined #openstack-nova | 05:15 | |
*** claudiub has joined #openstack-nova | 05:16 | |
*** markvoelker has quit IRC | 05:16 | |
*** thorst has joined #openstack-nova | 05:17 | |
*** ssmith has joined #openstack-nova | 05:17 | |
*** shan has joined #openstack-nova | 05:18 | |
*** ratailor has joined #openstack-nova | 05:18 | |
*** Guest75181 has quit IRC | 05:21 | |
*** thorst has quit IRC | 05:22 | |
*** ssmith has quit IRC | 05:23 | |
*** moshele has joined #openstack-nova | 05:24 | |
*** trinaths has joined #openstack-nova | 05:29 | |
*** gvrangan has joined #openstack-nova | 05:32 | |
*** suresh12 has joined #openstack-nova | 05:33 | |
*** jaosorior has joined #openstack-nova | 05:39 | |
*** udesale__ has joined #openstack-nova | 05:40 | |
*** udesale has quit IRC | 05:42 | |
*** udesale has joined #openstack-nova | 05:42 | |
*** slaweq has joined #openstack-nova | 05:44 | |
*** udesale__ has quit IRC | 05:45 | |
*** rybridges1 has quit IRC | 05:47 | |
*** slaweq has quit IRC | 05:49 | |
*** oanson has quit IRC | 05:58 | |
*** oanson has joined #openstack-nova | 05:59 | |
*** diga has quit IRC | 06:00 | |
*** priteau has joined #openstack-nova | 06:02 | |
*** gvrangan has quit IRC | 06:03 | |
*** gvrangan has joined #openstack-nova | 06:04 | |
openstackgerrit | Rajesh Tailor proposed openstack/nova master: Host addition host-aggregate should be case-sensitive https://review.openstack.org/498334 | 06:06 |
*** priteau has quit IRC | 06:06 | |
*** gvrangan has quit IRC | 06:10 | |
*** Oku_OS-away is now known as Oku_OS | 06:12 | |
*** yangyape_ has quit IRC | 06:12 | |
*** yangyapeng has joined #openstack-nova | 06:13 | |
*** markvoelker has joined #openstack-nova | 06:13 | |
*** sridharg has joined #openstack-nova | 06:14 | |
*** jianghuaw has joined #openstack-nova | 06:14 | |
*** thorst has joined #openstack-nova | 06:18 | |
*** hieulq has quit IRC | 06:19 | |
*** phuongnh has quit IRC | 06:19 | |
*** TuanLA has quit IRC | 06:19 | |
*** gvrangan has joined #openstack-nova | 06:19 | |
*** phuongnh has joined #openstack-nova | 06:19 | |
*** TuanLA has joined #openstack-nova | 06:19 | |
*** hieulq has joined #openstack-nova | 06:19 | |
*** thorst has quit IRC | 06:23 | |
*** jmlowe has quit IRC | 06:29 | |
*** jaosorior has quit IRC | 06:33 | |
*** suresh12 has quit IRC | 06:41 | |
*** cfriesen has quit IRC | 06:42 | |
*** slaweq has joined #openstack-nova | 06:45 | |
*** brault has joined #openstack-nova | 06:46 | |
*** markvoelker has quit IRC | 06:47 | |
openstackgerrit | jichenjc proposed openstack/nova master: remove CONF.vendordata_driver https://review.openstack.org/501510 | 06:49 |
*** slaweq has quit IRC | 06:50 | |
*** rcernin has joined #openstack-nova | 06:53 | |
*** andreas_s has joined #openstack-nova | 06:56 | |
*** ababich has joined #openstack-nova | 06:58 | |
*** armax has quit IRC | 07:02 | |
*** hshiina2 has joined #openstack-nova | 07:02 | |
Dinesh_Bhor | stephenfin: Hi, you there? | 07:02 |
*** alexchadin has joined #openstack-nova | 07:04 | |
*** hshiina3 has joined #openstack-nova | 07:05 | |
*** hshiina has quit IRC | 07:05 | |
*** sahid has joined #openstack-nova | 07:07 | |
*** hshiina2 has quit IRC | 07:08 | |
*** slaweq has joined #openstack-nova | 07:11 | |
*** tesseract has joined #openstack-nova | 07:13 | |
*** slaweq has quit IRC | 07:16 | |
*** moshele has quit IRC | 07:18 | |
openstackgerrit | AJAY KALAMBUR proposed openstack/nova master: Fix incorrect port unbind request in nova on instance delete https://review.openstack.org/387687 | 07:20 |
*** zenoway has joined #openstack-nova | 07:22 | |
*** moshele has joined #openstack-nova | 07:26 | |
*** hshiina2 has joined #openstack-nova | 07:29 | |
*** hshiina3 has quit IRC | 07:32 | |
*** ekuris has quit IRC | 07:36 | |
*** hshiina has joined #openstack-nova | 07:37 | |
*** jpena|off is now known as jpena | 07:38 | |
*** hshiina2 has quit IRC | 07:41 | |
*** markvoelker has joined #openstack-nova | 07:44 | |
*** moshele has quit IRC | 07:46 | |
*** ralonsoh has joined #openstack-nova | 07:49 | |
*** sridharg has quit IRC | 07:50 | |
*** ekuris has joined #openstack-nova | 07:52 | |
*** sridharg has joined #openstack-nova | 07:52 | |
*** lajoskatona has joined #openstack-nova | 07:57 | |
*** tetsuro has quit IRC | 07:58 | |
*** tbh_ has joined #openstack-nova | 07:59 | |
*** trungnv has joined #openstack-nova | 08:02 | |
*** moshele has joined #openstack-nova | 08:02 | |
*** liverpooler has joined #openstack-nova | 08:02 | |
*** andreas_s_ has joined #openstack-nova | 08:09 | |
*** omnipresent has joined #openstack-nova | 08:09 | |
*** zphir has joined #openstack-nova | 08:10 | |
*** slaweq has joined #openstack-nova | 08:12 | |
*** andreas_s has quit IRC | 08:12 | |
*** zenoway has quit IRC | 08:12 | |
*** slaweq has quit IRC | 08:17 | |
*** markvoelker has quit IRC | 08:17 | |
*** jaosorior has joined #openstack-nova | 08:17 | |
*** thorst has joined #openstack-nova | 08:20 | |
*** gvrangan has quit IRC | 08:21 | |
*** priteau has joined #openstack-nova | 08:23 | |
openstackgerrit | Rodolfo Alonso Hernandez proposed openstack/os-vif master: Add abstract OVSDB API https://review.openstack.org/476612 | 08:23 |
*** thorst has quit IRC | 08:24 | |
*** liverpooler has quit IRC | 08:25 | |
*** liverpooler has joined #openstack-nova | 08:25 | |
*** lucas-afk is now known as lucasagomes | 08:31 | |
*** rmart04 has joined #openstack-nova | 08:31 | |
*** rmcall has quit IRC | 08:33 | |
*** rmcall has joined #openstack-nova | 08:33 | |
*** rmcall has quit IRC | 08:34 | |
openstackgerrit | Merged openstack/nova master: Update contributor guide for Queens https://review.openstack.org/500454 | 08:38 |
*** diga has joined #openstack-nova | 08:40 | |
*** omnipresent has quit IRC | 08:43 | |
*** derekh has joined #openstack-nova | 08:45 | |
*** zenoway has joined #openstack-nova | 08:46 | |
bauzas | good morning Novaers | 08:46 |
bauzas | a bit late due to paperwork... | 08:47 |
openstackgerrit | Rodolfo Alonso Hernandez proposed openstack/os-vif master: Add native implementation OVSDB API https://review.openstack.org/482226 | 08:49 |
*** jaosorior has quit IRC | 08:50 | |
openstackgerrit | jichenjc proposed openstack/nova master: remove CONF.vendordata_driver https://review.openstack.org/501510 | 08:51 |
*** oomichi has quit IRC | 08:53 | |
*** oomichi has joined #openstack-nova | 08:54 | |
*** mriedem has quit IRC | 08:55 | |
stephenfin | Dinesh_Bhor: I am now (forgot to open HexChat). What's up? | 08:57 |
*** gvrangan has joined #openstack-nova | 08:58 | |
stephenfin | mikal: RE: removing UML support, I removed it from the docs because it didn't work, hadn't been tested in years, and we saw that it was unused by anyone and everyone (was originally added to test some esoteric features in the gate, iirc) | 08:59 |
*** hferenc has joined #openstack-nova | 08:59 | |
stephenfin | mikal: tl;dr: It's dead. We can remove all references to it | 08:59 |
Dinesh_Bhor | stephenfin: about this: https://review.openstack.org/#/c/466799/2/nova/tests/unit/api/openstack/test_wsgi.py | 09:03 |
Dinesh_Bhor | stephenfin: Earlier the behaviors was in response headers the integers values were allowed. | 09:04 |
Dinesh_Bhor | stephenfin: and I can see it RFC as well that response headers can contain integer values: https://tools.ietf.org/html/rfc2616#section-6.2 | 09:04 |
Dinesh_Bhor | stephenfin: https://tools.ietf.org/html/rfc2616#section-14.6 | 09:05 |
*** ekuris has quit IRC | 09:05 | |
Dinesh_Bhor | stephenfin: Are we restricting integer values now? IF yes, then is it going to be impact from user's point of view. | 09:06 |
stephenfin | Ohhhh... | 09:06 |
stephenfin | Dinesh_Bhor: So I'm pretty sure I found the plain-text bit in another RFC (which I should have linked). Clearly it was either wrong or had been supplanted though | 09:07 |
stephenfin | Dinesh_Bhor: Fortunately that didn't merge, so you can leave your -1 and I'll address the comments if possible. If not, I'll abandon the patch | 09:08 |
Dinesh_Bhor | stephenfin: If we decide to not allow integers then it's ok. sure I will add my comment. | 09:09 |
stephenfin | Dinesh_Bhor: Yeah, I'll investigate and see why I had to change that test. If nova spits the dummy out on integers, which are clearly valid, that would not be good :) | 09:12 |
stephenfin | Thanks for bringing that up too. Good catch | 09:12 |
Dinesh_Bhor | stephenfin: Okay, np, | 09:13 |
*** slaweq has joined #openstack-nova | 09:13 | |
*** avolkov has joined #openstack-nova | 09:14 | |
*** markvoelker has joined #openstack-nova | 09:14 | |
*** mvk has joined #openstack-nova | 09:16 | |
*** slaweq has quit IRC | 09:18 | |
*** ekuris has joined #openstack-nova | 09:20 | |
openstackgerrit | Rajesh Tailor proposed openstack/nova master: Host addition host-aggregate should be case-sensitive https://review.openstack.org/498334 | 09:20 |
*** yamahata has joined #openstack-nova | 09:22 | |
stephenfin | mikal: Do you think this change could be restored now? https://review.openstack.org/#/c/397835/ | 09:22 |
openstackgerrit | Jianghua Wang proposed openstack/nova master: XenAPI: deprecate image handler class path option https://review.openstack.org/497201 | 09:23 |
openstackgerrit | Jianghua Wang proposed openstack/nova master: XenAPI: define a new image handler to use vdi streaming https://review.openstack.org/486475 | 09:23 |
*** sambetts|afk is now known as sambetts | 09:24 | |
*** yamamoto_ has quit IRC | 09:28 | |
*** Tom has quit IRC | 09:36 | |
*** Tom has joined #openstack-nova | 09:37 | |
*** jichen has quit IRC | 09:41 | |
*** psachin has quit IRC | 09:43 | |
*** Tom has quit IRC | 09:43 | |
*** yamamoto has joined #openstack-nova | 09:46 | |
*** jaosorior has joined #openstack-nova | 09:47 | |
*** yingjun has quit IRC | 09:47 | |
*** markvoelker has quit IRC | 09:48 | |
*** rabel has joined #openstack-nova | 09:49 | |
*** chyka has joined #openstack-nova | 09:51 | |
openstackgerrit | Rodolfo Alonso Hernandez proposed openstack/os-vif master: Add abstract OVSDB API https://review.openstack.org/476612 | 09:53 |
*** edmondsw has joined #openstack-nova | 09:54 | |
*** chyka has quit IRC | 09:56 | |
*** edmondsw has quit IRC | 09:58 | |
openstackgerrit | Naichuan Sun proposed openstack/nova master: xenapi: Exception Error logs shown in Citrix XenServer CI https://review.openstack.org/478437 | 09:58 |
*** kiennt has quit IRC | 09:58 | |
openstackgerrit | Rodolfo Alonso Hernandez proposed openstack/os-vif master: Add memoize function using oslo.cache https://review.openstack.org/472773 | 10:00 |
*** yassine has quit IRC | 10:00 | |
*** yangyapeng has quit IRC | 10:01 | |
openstackgerrit | Rodolfo Alonso Hernandez proposed openstack/os-vif master: Add native implementation OVSDB API https://review.openstack.org/482226 | 10:02 |
*** Summer has joined #openstack-nova | 10:02 | |
*** dtantsur|afk is now known as dtantsur | 10:05 | |
*** TuanLA has quit IRC | 10:07 | |
*** jmlowe has joined #openstack-nova | 10:08 | |
*** zenoway has quit IRC | 10:08 | |
*** zenoway has joined #openstack-nova | 10:09 | |
*** jmlowe_ has joined #openstack-nova | 10:11 | |
*** zenoway has quit IRC | 10:13 | |
*** wlfightup has quit IRC | 10:14 | |
*** slaweq has joined #openstack-nova | 10:14 | |
*** yamamoto has quit IRC | 10:14 | |
*** wlfightup has joined #openstack-nova | 10:14 | |
*** jmlowe has quit IRC | 10:14 | |
*** oanson has quit IRC | 10:16 | |
*** oanson has joined #openstack-nova | 10:18 | |
*** zenoway has joined #openstack-nova | 10:18 | |
*** yamamoto has joined #openstack-nova | 10:19 | |
*** slaweq has quit IRC | 10:19 | |
*** thorst has joined #openstack-nova | 10:21 | |
*** zenoway has quit IRC | 10:23 | |
*** zenoway has joined #openstack-nova | 10:23 | |
*** zhurong has quit IRC | 10:23 | |
*** abhi89 has quit IRC | 10:26 | |
*** thorst has quit IRC | 10:26 | |
*** sridharg has quit IRC | 10:26 | |
*** moshele has quit IRC | 10:27 | |
*** sridharg has joined #openstack-nova | 10:27 | |
gibi | What is using compute.api.HostAPI? I see that xen and vmware implements the backend but I don't see any usage of this compute.api | 10:28 |
*** moshele has joined #openstack-nova | 10:32 | |
*** ekuris has quit IRC | 10:33 | |
*** markvoelker has joined #openstack-nova | 10:45 | |
*** bkopilov has quit IRC | 10:48 | |
*** ekuris has joined #openstack-nova | 10:49 | |
*** Summer has quit IRC | 10:51 | |
openstackgerrit | Andreas Jaeger proposed openstack/nova master: Fix broken URLs https://review.openstack.org/501402 | 10:53 |
*** jpena is now known as jpena|lunch | 10:57 | |
*** amuresan has joined #openstack-nova | 11:01 | |
*** udesale has quit IRC | 11:04 | |
*** ekuris has quit IRC | 11:05 | |
*** sdague has joined #openstack-nova | 11:06 | |
*** szaher has quit IRC | 11:07 | |
*** yassine has joined #openstack-nova | 11:13 | |
*** slaweq has joined #openstack-nova | 11:15 | |
*** psachin has joined #openstack-nova | 11:16 | |
*** markvoelker has quit IRC | 11:17 | |
*** tbh_ has quit IRC | 11:19 | |
*** zhurong has joined #openstack-nova | 11:20 | |
*** slaweq has quit IRC | 11:20 | |
*** ekuris has joined #openstack-nova | 11:21 | |
*** thorst has joined #openstack-nova | 11:22 | |
*** avolkov has quit IRC | 11:22 | |
*** avolkov has joined #openstack-nova | 11:23 | |
*** smatzek has joined #openstack-nova | 11:25 | |
*** zphir has quit IRC | 11:28 | |
*** zphir has joined #openstack-nova | 11:29 | |
*** thorst has quit IRC | 11:30 | |
*** zphir has quit IRC | 11:31 | |
*** lucasagomes is now known as lucas-hungry | 11:32 | |
*** sean-k-mooney has joined #openstack-nova | 11:41 | |
*** liverpooler has quit IRC | 11:44 | |
*** dixiaoli has quit IRC | 11:45 | |
*** lpetrut has joined #openstack-nova | 11:46 | |
*** lpetrut has quit IRC | 11:46 | |
*** litao__ has quit IRC | 11:48 | |
*** szaher has joined #openstack-nova | 11:59 | |
*** thorst has joined #openstack-nova | 12:01 | |
*** thorst_ has joined #openstack-nova | 12:03 | |
*** jpena|lunch is now known as jpena | 12:05 | |
*** thorst__ has joined #openstack-nova | 12:05 | |
*** thorst has quit IRC | 12:05 | |
*** amuresan has quit IRC | 12:05 | |
*** bkopilov has joined #openstack-nova | 12:06 | |
*** thorst_ has quit IRC | 12:07 | |
*** dims has joined #openstack-nova | 12:07 | |
*** lucas-hungry is now known as lucasagomes | 12:10 | |
*** phuongnh has quit IRC | 12:11 | |
*** catintheroof has joined #openstack-nova | 12:11 | |
*** vladikr has joined #openstack-nova | 12:14 | |
*** diga has quit IRC | 12:14 | |
*** gszasz has joined #openstack-nova | 12:14 | |
*** markvoelker has joined #openstack-nova | 12:15 | |
*** jaosorior has quit IRC | 12:15 | |
*** slaweq has joined #openstack-nova | 12:16 | |
*** manasm has quit IRC | 12:16 | |
*** dave-mccowan has joined #openstack-nova | 12:19 | |
*** slaweq has quit IRC | 12:21 | |
*** markvoelker has quit IRC | 12:24 | |
*** markvoelker has joined #openstack-nova | 12:24 | |
*** alexchadin has quit IRC | 12:24 | |
*** zhurong has quit IRC | 12:25 | |
*** jamesdenton has quit IRC | 12:27 | |
*** abhi89 has joined #openstack-nova | 12:27 | |
*** edmondsw has joined #openstack-nova | 12:32 | |
*** rabel has quit IRC | 12:32 | |
*** catintheroof has quit IRC | 12:33 | |
*** jamesdenton has joined #openstack-nova | 12:33 | |
*** ratailor has quit IRC | 12:34 | |
*** catintheroof has joined #openstack-nova | 12:38 | |
*** awaugama has joined #openstack-nova | 12:39 | |
awaugama | t | 12:39 |
*** dave-mcc_ has joined #openstack-nova | 12:39 | |
*** esberglu has joined #openstack-nova | 12:40 | |
openstackgerrit | Takashi NATSUME proposed openstack/nova-specs master: Enable cold migration with target host https://review.openstack.org/489031 | 12:41 |
*** dave-mccowan has quit IRC | 12:42 | |
*** edmondsw has quit IRC | 12:43 | |
*** edmondsw has joined #openstack-nova | 12:43 | |
*** jaosorior has joined #openstack-nova | 12:44 | |
*** shan has quit IRC | 12:46 | |
*** edmondsw has quit IRC | 12:48 | |
*** jaypipes has joined #openstack-nova | 12:48 | |
*** abhi89 has quit IRC | 12:53 | |
*** edmondsw has joined #openstack-nova | 12:54 | |
*** liverpooler has joined #openstack-nova | 12:54 | |
*** Oku_OS is now known as Oku_OS-away | 12:56 | |
*** lyan has joined #openstack-nova | 12:57 | |
*** vladikr has quit IRC | 12:57 | |
*** zhenq has joined #openstack-nova | 12:59 | |
openstackgerrit | Stephen Finucane proposed openstack/nova master: doc: Add documentation for emulator_thread_policy https://review.openstack.org/501721 | 12:59 |
stephenfin | sahid, bauzas, jaypipes: Could ye look at that ^ simple doc fix | 12:59 |
* stephenfin moves onto realtime flavor extra spec doco | 13:00 | |
bauzas | ack | 13:00 |
*** baoli has joined #openstack-nova | 13:02 | |
bauzas | stephenfin: https://en.wikipedia.org/wiki/RRRrrrr!!! | 13:03 |
bauzas | stephenfin: https://review.openstack.org/#/c/501721/1 | 13:03 |
openstackgerrit | Balazs Gibizer proposed openstack/nova master: Send soft_delete from context manager https://review.openstack.org/476459 | 13:03 |
openstackgerrit | Balazs Gibizer proposed openstack/nova master: Transform missing delete notifications https://review.openstack.org/410297 | 13:03 |
stephenfin | Ha! | 13:03 |
*** felipemonteiro has joined #openstack-nova | 13:03 | |
openstackgerrit | Stephen Finucane proposed openstack/nova master: doc: Add documentation for emulator_thread_policy https://review.openstack.org/501721 | 13:04 |
stephenfin | bauzas: Done :D | 13:04 |
*** felipemonteiro_ has joined #openstack-nova | 13:05 | |
bauzas | stephenfin: it's a cult film y'know | 13:06 |
*** udesale has joined #openstack-nova | 13:07 | |
stephenfin | bauzas: I figured as much. Had a Monty Python feel about | 13:07 |
*** felipemonteiro has quit IRC | 13:09 | |
*** mriedem has joined #openstack-nova | 13:09 | |
*** pchavva has joined #openstack-nova | 13:10 | |
mriedem | this is blocked an upper-constraints change, can we get this in https://review.openstack.org/#/c/501400/ ? | 13:10 |
mriedem | needs another +2 | 13:10 |
*** vladikr has joined #openstack-nova | 13:10 | |
* stephenfin looks | 13:10 | |
*** yassine has quit IRC | 13:12 | |
stephenfin | mriedem: Yup, looks grand to me | 13:12 |
efried | 'cept for the typos | 13:12 |
*** yassine has joined #openstack-nova | 13:12 | |
stephenfin | efried: If you're brave enough to -1 it for that, I'll give you a cookie :P | 13:13 |
*** rabel has joined #openstack-nova | 13:13 | |
stephenfin | efried: mriedem might give you something else though | 13:13 |
efried | I only -1 for typos in docs | 13:14 |
*** edmondsw_ has joined #openstack-nova | 13:14 | |
efried | ("typo's in doc's"?) | 13:14 |
stephenfin | Yup, likewise. I kid :) | 13:14 |
*** edmondsw has quit IRC | 13:15 | |
*** slaweq has joined #openstack-nova | 13:17 | |
*** smatzek has quit IRC | 13:18 | |
mikal | stephenfin: yeah, I think we could take another pass at that now | 13:18 |
sahid | stephenfin: https://review.openstack.org/#/c/501721/2 the explnation does not make a lot of sense | 13:18 |
*** itlinux has quit IRC | 13:19 | |
stephenfin | sahid: Yup, good call. I'll rework shortly | 13:20 |
bauzas | sahid: I tend to fast-approve doc changes but it helps, but feel free to open a follow-up and I'll review it | 13:20 |
*** yassine has quit IRC | 13:20 | |
*** Tom has joined #openstack-nova | 13:20 | |
bauzas | s/but/because | 13:20 |
stephenfin | bauzas, sahid: It's actually blocked by a base patch so it won't merge :/ | 13:20 |
bauzas | stephenfin: ack, then push another rev and I'll +2 | 13:21 |
stephenfin | +2 to fast-approving docs too. Fast iteration is really helpful | 13:21 |
bauzas | I mean, doc bugs don't hurt | 13:21 |
bauzas | while doc amendments really help | 13:21 |
stephenfin | Well, not as much as no docs does | 13:21 |
bauzas | yup that | 13:21 |
bauzas | unless of course if you state in the doc something like "rm -rf /" | 13:22 |
*** slaweq has quit IRC | 13:22 | |
sdague | mikal: ok, looking | 13:22 |
*** gvrangan has quit IRC | 13:23 | |
stephenfin | mikal: Sweet. I'll revive that shortly so | 13:23 |
bauzas | mriedem: new school year, new constraints. I'll need to bail out from the nova meeting around 1420UTC (ie. 20 mins after you start= | 13:24 |
bauzas | but I should be back 20 mins after | 13:24 |
bauzas | hopefully the daylight shift will solve the problem in a couple of weeks :) | 13:25 |
mriedem | ok | 13:25 |
*** catintheroof has quit IRC | 13:27 | |
*** catintheroof has joined #openstack-nova | 13:27 | |
*** manasm has joined #openstack-nova | 13:28 | |
*** yassine has joined #openstack-nova | 13:28 | |
*** edmondsw_ is now known as edmondsw | 13:29 | |
*** gouthamr has joined #openstack-nova | 13:29 | |
*** felipemonteiro_ has quit IRC | 13:30 | |
*** smatzek has joined #openstack-nova | 13:31 | |
*** gbarros has joined #openstack-nova | 13:32 | |
*** slaweq has joined #openstack-nova | 13:33 | |
openstackgerrit | Andreas Jaeger proposed openstack/nova master: Fix broken URLs https://review.openstack.org/501402 | 13:33 |
*** slaweq has quit IRC | 13:33 | |
sdague | mikal: https://review.openstack.org/#/c/489486 ... that needs some more explaining, the the cryptsetup add isn't clear to me | 13:33 |
*** catintheroof has quit IRC | 13:37 | |
*** catintheroof has joined #openstack-nova | 13:37 | |
dansmith | mriedem: you going to fix the typo in that second one or just steam on? | 13:37 |
mriedem | doing it now | 13:38 |
openstackgerrit | Matt Riedemann proposed openstack/nova master: Track which cell each instance is created in and use it consistently https://review.openstack.org/501452 | 13:38 |
openstackgerrit | Matt Riedemann proposed openstack/nova master: Make ConductorTaskTestCase run with 2 cells https://review.openstack.org/501514 | 13:38 |
openstackgerrit | Matt Riedemann proposed openstack/nova master: Ensure instance mapping is updated in case of quota recheck fails https://review.openstack.org/501408 | 13:38 |
mriedem | alex raised a good question here https://review.openstack.org/#/c/501408/8/nova/conductor/manager.py@1049 | 13:39 |
*** catintheroof has quit IRC | 13:39 | |
*** catintheroof has joined #openstack-nova | 13:40 | |
mriedem | in that if the build request is already deleted after we create the instance in the cell but before we actually send the instance to the host to build, we cleanup the instance (local delete) but don't cleanup dest node allocations | 13:40 |
mriedem | separate bug, but a bug nonetheless | 13:40 |
mikal | sdague: so I need a reno when I remove a command from rootwrap's filters? Or only when I add one? | 13:41 |
openstackgerrit | Merged openstack/nova master: Fix test_rpc_consumer_isolation for oslo.messaging 5.31.0 https://review.openstack.org/501400 | 13:41 |
*** udesale has quit IRC | 13:41 | |
sdague | the adds are the critical bit. That's a config file which won't work if you don't add those lines | 13:42 |
sdague | if people left old things around, meh | 13:42 |
openstackgerrit | Andreas Jaeger proposed openstack/nova master: Fix broken URLs https://review.openstack.org/501402 | 13:43 |
mriedem | who wants to look at https://review.openstack.org/#/c/501514/ and up? regression in pike that needs to be backported | 13:43 |
mriedem | vexxhost hit it yesterday | 13:43 |
jaypipes | mriedem: just FYI, we're evacuating this evening so I'll be OOO tomorrow and part of this afternoon. | 13:44 |
*** hshiina has quit IRC | 13:44 | |
mriedem | jaypipes: ok, good luck | 13:44 |
jaypipes | mriedem: and I do mean evacuate, not rebuild. | 13:44 |
sdague | mikal: the general rule of thumb, assume someone is upgrading with pip install -U on the commit. Do they need to do something besides that to have usable environment | 13:44 |
mriedem | jaypipes: i hear ohio still has room for you | 13:44 |
mriedem | if you'll take it back | 13:44 |
jaypipes | mriedem: that's where we're headed. | 13:44 |
mriedem | nice | 13:45 |
mriedem | just in time for your beloved browns season to begin | 13:45 |
sdague | jaypipes: well, that means jeni's icecream when you arrive | 13:45 |
dansmith | jaypipes: good luck | 13:45 |
jaypipes | sdague: heh :) | 13:45 |
*** Oku_OS-away is now known as Oku_OS | 13:45 | |
jaypipes | dansmith: thx. still smoldering where you are? | 13:45 |
dansmith | jaypipes: yeah | 13:45 |
sdague | jaypipes: we stopped in columbus north market on our drive to TN for the eclipse. Could not leave without some jeni's | 13:45 |
sdague | jaypipes: safe travels | 13:46 |
jaypipes | sdague: indeed. Jeni's is awesome. Graeters is also very good :) | 13:46 |
jaypipes | sdague: thx | 13:46 |
jaypipes | dansmith: how close to you are the fires? | 13:46 |
*** jangutter has joined #openstack-nova | 13:46 | |
sdague | jaypipes: we going to see you in CO still? | 13:46 |
dansmith | jaypipes: the fires are at portland's watershed, if that tells you, but that's like 15 miles by the crow I think | 13:47 |
kashyap | dansmith: "by the crow" == shortest path I presume? | 13:49 |
dansmith | kashyap: yes, as in "as the crow flies" | 13:49 |
dansmith | apparently crows fly in straight lines | 13:49 |
openstackgerrit | Matt Riedemann proposed openstack/nova stable/pike: Provide hints when nova-manage db sync fails to sync cell0 https://review.openstack.org/501745 | 13:49 |
kashyap | Yeah, I recall hearing that, just wanted to double-check :-) | 13:49 |
openstackgerrit | Matt Riedemann proposed openstack/nova stable/ocata: Provide hints when nova-manage db sync fails to sync cell0 https://review.openstack.org/501746 | 13:49 |
kashyap | Oh dear, the wildfire looks insane | 13:49 |
*** thegreenhundred has joined #openstack-nova | 13:50 | |
gibi | mriedem: hi! Is it OK for you If I rebase your patch https://review.openstack.org/#/c/498596 as mine top of it need to be adapted to a test case from master ? | 13:50 |
*** tidwellr has joined #openstack-nova | 13:53 | |
mriedem | gibi: sure | 13:56 |
mriedem | gibi: you were +1 on https://review.openstack.org/#/c/499718/ are you OK with it? sylvain is +2 and i'd like to move that series along as it's a regression that needs to be backported | 13:57 |
gibi | mriedem: looking... | 13:57 |
*** dave-mcc_ is now known as dave-mccowan | 13:57 | |
mriedem | looks like we need to talk about the other comments higher up the stack at some point too | 13:57 |
mriedem | after the meeting | 13:58 |
gibi | mriedem: yes, let's agree on the way forward with that series | 13:58 |
jaypipes | sdague: on line with Delta right now trying to redo flights | 14:00 |
bauzas | jaypipes: crossing fingers, mate | 14:00 |
*** cfriesen has joined #openstack-nova | 14:01 | |
mikal | sdague: yeah in that case I think a few of the renos need tweaking | 14:02 |
*** trinaths has left #openstack-nova | 14:02 | |
mikal | sdague: I will do that during the TWO HOURS OF KEYNOTES THIS ORNING | 14:02 |
sdague | mikal: but, you could miss some critical detail about synergizing! | 14:10 |
*** baoli has quit IRC | 14:19 | |
*** marst has joined #openstack-nova | 14:19 | |
openstackgerrit | Balazs Gibizer proposed openstack/nova master: Moving more utils to ServerResourceAllocationTestBase https://review.openstack.org/499539 | 14:19 |
openstackgerrit | Balazs Gibizer proposed openstack/nova master: Test resource allocation during soft delete https://review.openstack.org/495159 | 14:19 |
openstackgerrit | Balazs Gibizer proposed openstack/nova master: Refactor ServerMovingTests for non-move tests https://review.openstack.org/498596 | 14:19 |
*** baoli has joined #openstack-nova | 14:19 | |
*** andreas_s_ has quit IRC | 14:23 | |
mriedem | gibi: i think you can remove your -1 on https://review.openstack.org/#/c/499237/ | 14:26 |
*** ssmith has joined #openstack-nova | 14:26 | |
mriedem | gibi: i also replied to https://review.openstack.org/#/c/499874/4/nova/tests/functional/test_servers.py@1817 | 14:26 |
gibi | mriedem: reading... | 14:27 |
gibi | mriedem: what can we do with your dislike about rebuild after failed evac? | 14:28 |
mriedem | not sure i understand the question | 14:29 |
mriedem | gibi: i tend to agree with your point that we should handle a failure in the conductor (NoValidHost) the same as it's being handled in the compute when a claim fails or the spawn fails on the dest node, | 14:30 |
gibi | mriedem: I agree that rebuild after failed evac is not a major use case but I know about users depending on it. So I don't really want to prevent it | 14:30 |
mriedem | which is set the migration status to a state such that _destroy_evacuated_instances on the source node doesn't process it, | 14:30 |
mriedem | and delete the dest node allocation | 14:30 |
*** edmondsw has quit IRC | 14:30 | |
gibi | mriedem: OK. then I think we can set the migration to failed, and then nova will not try to delete source allocation either after the source compute is brought up | 14:31 |
mriedem | gibi: given that rebuild does not allocate resources or make a claim, i think in that case it's actually preferrable to *not* attempt to delete the source node allocations if the evacuate failed, | 14:31 |
gibi | mriedem: totally agree | 14:31 |
mriedem | because if the evac fails and you want to rebuild the instance on the source node after it comes back up, then the allocations are still there | 14:31 |
gibi | I think that we are on the same page now :) | 14:31 |
mriedem | and if the evac fails on the dest node, regardless of location (conductor or compute), we put the migration status into a state such that the source node won't process it | 14:31 |
mriedem | when it's restarted | 14:31 |
gibi | mriedem: agreee | 14:32 |
mriedem | yeah, so drop your -1 on https://review.openstack.org/#/c/499237/ and i'll +1 that | 14:32 |
gibi | mriedem: OK | 14:32 |
*** ekuris has quit IRC | 14:33 | |
gibi | mriedem: done | 14:33 |
gibi | mriedem: this also means I have nothing agains the force evacuate test cases any more | 14:33 |
mriedem | yay | 14:33 |
gibi | mriedem: so I'm going to drop -1 there too | 14:33 |
jaypipes | ffs, delta wants $3700 to change my wife's flight. | 14:33 |
*** slaweq has joined #openstack-nova | 14:34 | |
mnaser | that's outrageous. | 14:34 |
jaypipes | mnaser: welcome to price gouging. :) | 14:34 |
mnaser | don't know if it's any help but apparently jetblue's doing $99 flights -- dont know how much of those are still available (and im sure you heard of them) | 14:35 |
mriedem | when i was looking at flights to china in may, the price changed in one hour and went up like $7K | 14:35 |
jaypipes | mnaser: they want to charge me $800 for a single 50-minute flight from Columbus, OH to Atlanta, GA. | 14:35 |
jaypipes | looks like there's a possibility I might miss Denver... :( | 14:36 |
*** lucasxu has joined #openstack-nova | 14:36 | |
jaypipes | latest European ensemble is terrible... https://twitter.com/cajunweatherman/status/905792235656138753/photo/1 | 14:36 |
*** edmondsw has joined #openstack-nova | 14:37 | |
alex_xu | do we have wiki or etherpad for all the sessions and schedule in the PTG? I remember we have something in previous PTG | 14:38 |
jaypipes | alex_xu: I think ttx was putting together an ethercalc. | 14:38 |
*** slaweq has quit IRC | 14:39 | |
*** sapcc-bot1 has quit IRC | 14:39 | |
*** carthaca_ has quit IRC | 14:39 | |
*** carthaca_ has joined #openstack-nova | 14:39 | |
*** tpatzig_ has joined #openstack-nova | 14:39 | |
*** david_1 has joined #openstack-nova | 14:39 | |
*** sapcc-bot has joined #openstack-nova | 14:39 | |
*** dgonzalez_ has joined #openstack-nova | 14:39 | |
*** seife_ has joined #openstack-nova | 14:39 | |
openstackgerrit | Takashi NATSUME proposed openstack/nova master: List/show all server migration types (1/2) https://review.openstack.org/430608 | 14:40 |
alex_xu | jaypipes: ah, let me try to find it | 14:40 |
mriedem | alex_xu: https://etherpad.openstack.org/p/nova-ptg-queens | 14:40 |
mriedem | oh http://ptg.openstack.org/ | 14:40 |
mriedem | ^ is the home page | 14:40 |
*** lucasxu has quit IRC | 14:41 | |
*** david_1 has quit IRC | 14:41 | |
*** tpatzig_ has quit IRC | 14:41 | |
*** dgonzalez_ has quit IRC | 14:41 | |
alex_xu | mriedem: ah, thanks, surprise the home page change like that | 14:41 |
*** seife_ has quit IRC | 14:41 | |
mriedem | well, it's the home page for devs | 14:41 |
mriedem | it's not the marketing glitzy homepage | 14:41 |
mriedem | gibi: don't forget this one https://review.openstack.org/#/c/499877/ | 14:42 |
*** hongbin has joined #openstack-nova | 14:42 | |
gibi | mriedem: I'm in the process of changing my vote there :) | 14:43 |
mriedem | whew | 14:43 |
mriedem | ok | 14:43 |
mriedem | i have to modify something in the 2nd to bottom patch for sylvain so i was waiting to do that | 14:43 |
*** cleong has joined #openstack-nova | 14:46 | |
gibi | I'm OK with the first 4 patches in that series and just started reading the 5th | 14:47 |
*** moshele has quit IRC | 14:50 | |
*** Swami has joined #openstack-nova | 14:50 | |
dtantsur | hi folks! I'm afraid to imagine how often you've heard this question, but.. what's the difference between nova migrate and nova evacuate? | 14:55 |
dtantsur | I'm figuring out which one fits better into https://review.openstack.org/449155 | 14:55 |
*** gbarros has quit IRC | 14:55 | |
*** gbarros has joined #openstack-nova | 14:56 | |
edleafe | dtantsur: http://www.danplanet.com/blog/2016/03/03/evacuate-in-nova-one-command-to-confuse-us-all/ | 14:56 |
dtantsur | thanks! | 14:57 |
edleafe | dtantsur: I keep that one handy, because after all these years it's still confusing | 14:58 |
* dtantsur bookmarks | 14:58 | |
*** felipemonteiro has joined #openstack-nova | 14:58 | |
*** felipemonteiro_ has joined #openstack-nova | 14:59 | |
openstackgerrit | Stephen Finucane proposed openstack/nova master: doc: Add documentation for emulator_thread_policy https://review.openstack.org/501721 | 15:00 |
stephenfin | sahid, bauzas, gibi: ^ addressed sahid's comments | 15:00 |
stephenfin | I think... | 15:00 |
gibi | stephenfin: I think so too | 15:03 |
*** felipemonteiro has quit IRC | 15:03 | |
*** takashin has left #openstack-nova | 15:04 | |
bauzas | stephenfin: +Wiiii | 15:04 |
*** suresh12 has joined #openstack-nova | 15:06 | |
bauzas | mriedem: so, 2 points | 15:08 |
bauzas | mriedem: just remove the upgrade reno and then I'll +2 https://review.openstack.org/#/c/499399/6 | 15:09 |
bauzas | mriedem: also, about your skip_filters thoughts, just remember that we already have something called force_hosts | 15:09 |
bauzas | of course, that field is terribly named | 15:09 |
*** Oku_OS is now known as Oku_OS-away | 15:09 | |
bauzas | but we could just not add yet again a new RPC API parameter, and rather just use the RequestSpec object for that | 15:10 |
gibi | stephenfin: I made an answer to your comment in https://review.openstack.org/#/c/463946/9/nova/tests/functional/notification_sample_tests/test_keypair.py@20 | 15:10 |
openstackgerrit | Merged openstack/nova-specs master: Convert consoles code to use objects framework https://review.openstack.org/500975 | 15:10 |
bauzas | mriedem: like, we could create a new field (and deprecate the legacy one) called forced_destination | 15:11 |
bauzas | having a Destination value | 15:11 |
bauzas | and so, if you pass a forced host, then the scheduler would claim | 15:11 |
mriedem | efried: ok is the first half of wednesday to your liking? https://etherpad.openstack.org/p/nova-ptg-queens | 15:13 |
mriedem | bauzas: i prefer an rpc parameter to select_destinations rather than hiding things in the already super confusing RequestSpec object | 15:13 |
mriedem | because force_hosts, requested_destination, and then a forced_destination would be even more complicate | 15:14 |
mriedem | *complicated | 15:14 |
mriedem | skip_filters in the rpc api parameters is pretty clear | 15:14 |
*** rmart04 has quit IRC | 15:15 | |
gibi | cores: there is a notification transformation patch only needs a second +2 if you are bored https://review.openstack.org/#/c/417882/ | 15:18 |
*** armax has joined #openstack-nova | 15:19 | |
bauzas | mriedem: I tend to prefer passing versioned objects than parameters on RPC APIs because I think it's clearer, you know | 15:20 |
bauzas | mriedem: the fact is that I agree with you, we should get rid of forced_hosts | 15:20 |
*** armax has quit IRC | 15:20 | |
bauzas | mriedem: but I was just proposing to only use 2 fields : one for requesting the scheduler filters, one for not | 15:20 |
bauzas | that could be a comment in the object | 15:21 |
mriedem | i guess it will probably come down to whether or not we need to persist that information for later | 15:21 |
bauzas | we already have a long list of explicit parameters in the API method, and I'm not sure it's cool | 15:21 |
bauzas | mriedem: we have an helper method for that in the object | 15:21 |
bauzas | mriedem: and there are fields that aren't persisted already | 15:21 |
*** penick has joined #openstack-nova | 15:22 | |
bauzas | so, maybe the work consists of making the line clearer in between what requires to be persisted and what's not | 15:22 |
bauzas | I could do that | 15:22 |
stephenfin | sdague: Nice little doc cleanup here, if you're interested https://review.openstack.org/#/c/501342/ | 15:22 |
*** manasm has quit IRC | 15:22 | |
mriedem | def select_destinations(self, ctxt, spec_obj, instance_uuids): | 15:23 |
mriedem | bauzas: ^ is a long list? | 15:23 |
*** lajoskatona has quit IRC | 15:24 | |
bauzas | mriedem: you know we had a problem with instance_uuids being different from what we have in the spec object, so I'd say one parameter is maybe too much | 15:24 |
*** gmann has quit IRC | 15:24 | |
*** gmann has joined #openstack-nova | 15:24 | |
openstackgerrit | Matt Riedemann proposed openstack/nova master: Remove dest node allocation if evacuate MoveClaim fails https://review.openstack.org/499878 | 15:25 |
bauzas | mriedem: the real crux of the problem is that I never explicited whether the RequestSpec object you pass is either the original spec or the amended spec in case of a move | 15:25 |
mriedem | gdi now my git review push failed | 15:26 |
openstackgerrit | Matt Riedemann proposed openstack/nova master: Remove dest node allocation if evacuate MoveClaim fails https://review.openstack.org/499878 | 15:26 |
openstackgerrit | Matt Riedemann proposed openstack/nova master: Pass migration from API to conductor for evacuate https://review.openstack.org/500176 | 15:26 |
openstackgerrit | Matt Riedemann proposed openstack/nova master: Add a test to make sure failed evacuate cleans up dest allocation https://review.openstack.org/499877 | 15:26 |
openstackgerrit | Matt Riedemann proposed openstack/nova master: Add recreate test for evacuate claim failure https://review.openstack.org/499874 | 15:26 |
openstackgerrit | Matt Riedemann proposed openstack/nova master: Create allocations against forced dest host during evacuate https://review.openstack.org/499399 | 15:26 |
openstackgerrit | Matt Riedemann proposed openstack/nova master: Refactor out claim_resources_on_destination into a utility https://review.openstack.org/499718 | 15:26 |
openstackgerrit | Matt Riedemann proposed openstack/nova master: Modernize set_vm_state_and_notify https://review.openstack.org/499799 | 15:26 |
bauzas | do folks agree with the fact that we should disable reschedules if someone is requesting a destination when cold migrating ? | 15:26 |
*** nicolasbock has joined #openstack-nova | 15:26 | |
mriedem | the bottom change there is going to need to be re-approved | 15:26 |
bauzas | mriedem: ack will do | 15:26 |
*** itlinux has joined #openstack-nova | 15:27 | |
mriedem | don't we already restrict reschedules like that in some other case? | 15:27 |
bauzas | mriedem: we don't reschedule for both evacuate and live-migrate | 15:28 |
bauzas | we only reschedule for two cases : boot and resize | 15:28 |
mriedem | well, we do reschedule for live migrate | 15:28 |
mriedem | from conductor | 15:28 |
mriedem | if the pre-checks fails | 15:28 |
bauzas | well, and cold migrate since resize shares the same codepath | 15:28 |
mriedem | *fail | 15:28 |
bauzas | mriedem: you mean at the conductor level ? | 15:29 |
mriedem | yes | 15:29 |
bauzas | mriedem: ah yeah | 15:29 |
bauzas | that's recent | 15:29 |
bauzas | I was talking of the classical reschedule situation where a compute raises an exception and then calls out conductor by amending the retry dict | 15:29 |
mriedem | evacuate in the conductor doesn't reschedule | 15:29 |
*** gbarros has quit IRC | 15:30 | |
mriedem | if you specify a host for cold migrate and that host fails, we shouldn't reschedule IMO | 15:30 |
bauzas | the situation I mention in takashi's spec is that if you allow 3 reschedules by default, you could have the scenario of a requested destination call, and then if the compute fails, then ending up finding another compute node | 15:30 |
bauzas | mriedem: that's my point | 15:30 |
mriedem | yes i realize, it's ComputeManager._reschedule_resize_or_reraise | 15:30 |
bauzas | I just wanted to make sure we all agree | 15:31 |
mriedem | now, live migration allows you to specify a host, not force it, and it will retry migrate_max_retries times | 15:32 |
mriedem | which defaults to -1 for unlimited | 15:32 |
mriedem | although i think in the case of live migration, if you specify a host (without force) and the scheduler raises NoValidHost, we're done | 15:34 |
mriedem | there is no retry | 15:34 |
mriedem | otherwise if you don't specify a host, we just keep trying until we find a host or run out of hosts and the scheduler raises NoValidHost | 15:34 |
bauzas | that's fine then | 15:34 |
bauzas | for live migrate | 15:34 |
mriedem | yeah so for cold migrate if i specify a host and the scheduler kicks it out with NoValidHost, you're done | 15:35 |
*** slaweq has joined #openstack-nova | 15:35 | |
*** chyka has joined #openstack-nova | 15:35 | |
mriedem | if the scheduler says it's ok and the compute fails, we don't reschedule to another host via prep_resize | 15:35 |
mriedem | you're done | 15:35 |
bauzas | okay, so I +1d the spec | 15:35 |
bauzas | once takashi amends the spec for mentioning that, I +2 | 15:35 |
mriedem | did he remove the stuff about adding the force flag to the cold migrate api? | 15:35 |
bauzas | he deserves more attention than last cycle :) | 15:35 |
bauzas | he did | 15:35 |
mriedem | well, what i'd also like to see with this is to not do the same underhanded request spec stuff that live migrate and evacuate do when a host is specified, | 15:36 |
mriedem | i.e. the API does some logic and shoves stuff in the request spec, which passes through conductor to the scheduler | 15:36 |
mriedem | i'd prefer if that was all more explicit between api->conductor-scheduler | 15:36 |
mriedem | because whenever i look at the conductor code i have to know exactly how the api code works | 15:37 |
*** manasm has joined #openstack-nova | 15:37 | |
bauzas | mriedem: when I wrote the original blueprint, there was a reason | 15:38 |
bauzas | mriedem: I necessarly needed to set the Spec fields by the API because the force or not force logic is there | 15:38 |
bauzas | mriedem: plus the fact that you can call again the conductor by the compute if you resize | 15:39 |
bauzas | mriedem: those reasons led me to set the fields there in the API | 15:39 |
bauzas | but I agree with the fact that is spaghetti code | 15:40 |
*** slaweq has quit IRC | 15:40 | |
mriedem | i've tried to document the hell out of all of these areas that i've had to touch for live migration and evacuate to help clarify some of it | 15:42 |
mriedem | but we still need some detailed documentation in the request spec object itself per my ML email | 15:42 |
mriedem | plus the fact that we don't convert some things to primitive in the request spec | 15:42 |
mriedem | which seems like a bug | 15:42 |
mriedem | but, i don't really know enough about it or if it's something we should care about | 15:43 |
mriedem | i.e. if you only have to convert to a primitive for legacy stuff due to an old scheduler client, then we probably don't care | 15:43 |
bauzas | when you say primitives, you mean legacy dictionaries ? | 15:45 |
bauzas | not the primitived dictionary in terms of o.vo ? | 15:46 |
*** baoli has quit IRC | 15:46 | |
bauzas | honestly, the problem is that we still have lots of back-and-forths between those legacy dicts and the new object, which is errorprone | 15:46 |
*** baoli has joined #openstack-nova | 15:47 | |
bauzas | so, my goal is to reduce those by changing the scheduler helper methods to accept a spec object | 15:47 |
mriedem | the legacy dict | 15:51 |
*** zenoway has quit IRC | 15:53 | |
mriedem | gibi: in https://review.openstack.org/#/c/417882/23/doc/notification_samples/instance-resize-error.json it looks weird in a few places where there are empty dicts or lists and there is a blank line in between | 15:53 |
*** gbarros has joined #openstack-nova | 15:54 | |
bauzas | by reading the news, all my thoughts go to folks in Florida, Georgia and South Carolina. Hope jaypipes will be all good | 15:54 |
jaypipes | bauzas: it's a logistical nightmare :( | 15:54 |
bauzas | jaypipes: you're far from me, buddy so I can hardly help | 15:55 |
jaypipes | bauzas: heh, I know :) it's cool, dude. | 15:55 |
*** itlinux has quit IRC | 15:55 | |
bauzas | jaypipes: but in case you need anything I can do, just lemme know | 15:55 |
jaypipes | bauzas: thx :) | 15:55 |
bauzas | jaypipes: you're temporarly relocating, as you said ? | 15:56 |
*** owalsh is now known as owalsh_afk | 15:56 | |
jaypipes | bauzas: yeah, though plans have changed... now looking to got to Asheville, NC for a night, drop the girls off with my parents and then catch flights out of ATL. | 15:57 |
jaypipes | bauzas: it's a clusterf**k | 15:57 |
*** itlinux has joined #openstack-nova | 15:58 | |
bauzas | :( | 15:59 |
*** rmart04 has joined #openstack-nova | 16:02 | |
*** rabel has quit IRC | 16:03 | |
*** itlinux has quit IRC | 16:06 | |
*** cdent has joined #openstack-nova | 16:06 | |
*** Apoorva has joined #openstack-nova | 16:08 | |
*** suresh12 has quit IRC | 16:10 | |
*** yassine has quit IRC | 16:13 | |
*** sahid has quit IRC | 16:15 | |
openstackgerrit | Stephen Finucane proposed openstack/nova master: conf: Remove 'vendordata_driver' opt https://review.openstack.org/397835 | 16:17 |
openstackgerrit | John Garbutt proposed openstack/nova master: Enable test_iscsi_volume in live migration job https://review.openstack.org/459316 | 16:17 |
stephenfin | mriedem: Per mikal's comment earlier, could you remove the -2 on this, please? https://review.openstack.org/397835 | 16:17 |
mriedem | stephenfin: what was mikal's comment earlier? | 16:18 |
* stephenfin scrolls up | 16:18 | |
stephenfin | <stephenfin> mikal: Do you think this change could be restored now? https://review.openstack.org/#/c/397835/ | 16:19 |
stephenfin | <mikal> stephenfin: yeah, I think we could take another pass at that now | 16:19 |
stephenfin | mriedem: I assume that means the vendordata v2 stuff is done? idk for sure, personally | 16:19 |
mriedem | mikal made improvements in ocata | 16:20 |
mriedem | https://specs.openstack.org/openstack/nova-specs/specs/ocata/implemented/vendordata-reboot-ocata.html | 16:20 |
stephenfin | Ultimately though, does that mean configurability of vendordata driver is no longer required? | 16:21 |
*** annegentle has joined #openstack-nova | 16:22 | |
openstackgerrit | Stephen Finucane proposed openstack/nova-specs master: DNM! Make "reproposals" more obvious to readers https://review.openstack.org/501797 | 16:24 |
openstackgerrit | Stephen Finucane proposed openstack/nova-specs master: DNM: Unformatted file https://review.openstack.org/501798 | 16:24 |
efried | mriedem Bitter. So bitter. | 16:24 |
mriedem | gdi i can't find the vendordata v2 stuff in the docs nova | 16:27 |
mriedem | *now | 16:27 |
mriedem | that should be in admin, or user, or reference, or something | 16:27 |
*** jaosorior has quit IRC | 16:30 | |
sdague | it's in user/ | 16:30 |
sdague | but, I'm trying to figure out if it's linked in anywhere | 16:31 |
sdague | https://docs.openstack.org/nova/latest/#maintenance | 16:31 |
sdague | Exposing custom metadata to compute instances: How and when you might want to extend the basic metadata exposed to compute instances (either via metadata server or config drive) for your specific purposes. | 16:32 |
mriedem | ok, not really maintenance | 16:32 |
mriedem | but at least it's somewhere | 16:32 |
sdague | well... putting things in buckets is hard | 16:32 |
mriedem | i know, | 16:33 |
mriedem | i'm just really gd grumpy today | 16:33 |
*** rajathagasthya has joined #openstack-nova | 16:33 | |
* cdent gives mriedem a cookie | 16:33 | |
*** rmart04 has quit IRC | 16:33 | |
*** rmart04 has joined #openstack-nova | 16:34 | |
*** annegentle has quit IRC | 16:34 | |
sdague | need some salad with marshmellows in it | 16:35 |
*** itlinux has joined #openstack-nova | 16:36 | |
*** slaweq has joined #openstack-nova | 16:37 | |
sean-k-mooney | mriedem: oh speaking of maintenance if you were wondering why the intel nfv ci has not been posting for a few days its because it did "maintenance" | 16:38 |
sean-k-mooney | mriedem: they completely blocked ssh connection out of the datacenter and we are arguing with them to whitelist review.openstack.org and our logserver... | 16:39 |
*** thorst__ has quit IRC | 16:39 | |
*** annegentle has joined #openstack-nova | 16:40 | |
*** slaweq has quit IRC | 16:41 | |
*** thorst has joined #openstack-nova | 16:43 | |
cdent | edleafe, jaypipes : do either of you have opinions on the two different body formats described in https://review.openstack.org/#/c/499259/ ? dansmith and I both have a bit of a preference for the dict-ish style. | 16:43 |
*** armax has joined #openstack-nova | 16:44 | |
*** rcernin has quit IRC | 16:45 | |
openstackgerrit | Matt Riedemann proposed openstack/nova master: DNM: test new style volume attach with live migration https://review.openstack.org/481290 | 16:45 |
openstackgerrit | Matt Riedemann proposed openstack/nova master: DNM: Run test_iscsi_volume with new style volume attachments https://review.openstack.org/501805 | 16:45 |
*** rmart04_ has joined #openstack-nova | 16:46 | |
*** rmart04 has quit IRC | 16:46 | |
*** rmart04_ is now known as rmart04 | 16:46 | |
*** marst has quit IRC | 16:46 | |
dansmith | mriedem: so we're good on that whole stack now yeah? and you'll propose backports when they land? | 16:46 |
mriedem | dansmith: yup | 16:47 |
mriedem | bauzas: were you ok with this now? https://review.openstack.org/#/c/499399/ | 16:47 |
mriedem | i updated the reno | 16:47 |
*** thorst has quit IRC | 16:48 | |
*** thorst_ has joined #openstack-nova | 16:48 | |
*** itlinux has quit IRC | 16:48 | |
*** edmondsw has quit IRC | 16:49 | |
*** armax has quit IRC | 16:50 | |
*** junbo has quit IRC | 16:50 | |
*** itlinux has joined #openstack-nova | 16:51 | |
*** lucasagomes is now known as lucas-afk | 16:51 | |
*** edmondsw has joined #openstack-nova | 16:51 | |
efried | stephenfin Re neutron VIF NUMA affinity - jaypipes had some general ideas about how to model this generically using placement aggregates. | 16:51 |
*** dtantsur is now known as dtantsur|afk | 16:52 | |
stephenfin | efried: Yeah, I figured placement would come into it on the nova side. rn I'm more curious about how we get this information from neutron though | 16:52 |
efried | stephenfin It's noted in the 'generic device management' etherpad we're teeing up for the PTG: https://etherpad.openstack.org/p/nova-ptg-queens-generic-device-management | 16:52 |
efried | stephenfin See around line 69 if you want to add some notes for specific stuff you're thinking for this case. | 16:53 |
jaypipes | cdent: will try to review in a bit. currently still trying to arrange hotels and flight changes... :( | 16:53 |
mriedem | efried: does https://etherpad.openstack.org/p/nova-ptg-queens-generic-device-management supersede the block of stuff in https://etherpad.openstack.org/p/nova-ptg-queens starting at L74? | 16:54 |
cdent | jaypipes: good luck with all that jaypipes | 16:54 |
efried | stephenfin And the mess starting at line 77 talks about how nova & neutron are going to play together. | 16:54 |
mriedem | because if so that would be nice to replace | 16:54 |
efried | mriedem Yes | 16:54 |
mriedem | as the main etherpad is getting huge | 16:54 |
efried | mriedem I've been working on moving content over. | 16:54 |
mriedem | ok, please remove from the main page when yo'ure done | 16:54 |
mriedem | i'm doing the same with cells | 16:54 |
efried | mriedem It's actually all represented - I didn't delete from the original because then we lose all the attributions. Do we care about that? | 16:54 |
*** edmondsw_ has joined #openstack-nova | 16:55 | |
*** crushil has quit IRC | 16:55 | |
*** junbo has joined #openstack-nova | 16:55 | |
stephenfin | efried: Oh, now that's interesting | 16:55 |
stephenfin | Fancy replying to that mail with those notes? | 16:55 |
mriedem | i don't really care about the colors, this is why i tell people to put their name/nick next to an item | 16:55 |
cburgess | Yeah jaypipes are you the wife and bugs GTFOing I hope? | 16:55 |
stephenfin | (for anyone note on #openstack-nova) | 16:55 |
cburgess | pugs.. not bugs | 16:55 |
cburgess | You can take the bugs with you too I suppose. | 16:55 |
*** rajathagasthya has quit IRC | 16:56 | |
*** edmondsw has quit IRC | 16:56 | |
*** Swami has quit IRC | 16:56 | |
*** rajathagasthya has joined #openstack-nova | 16:56 | |
*** rmcall has joined #openstack-nova | 16:57 | |
*** edmondsw_ has quit IRC | 16:57 | |
*** edmondsw has joined #openstack-nova | 16:57 | |
*** ralonsoh has quit IRC | 16:58 | |
openstackgerrit | Matt Riedemann proposed openstack/nova master: DNM: Run test_iscsi_volume with new style volume attachments https://review.openstack.org/501805 | 16:58 |
jaypipes | cburgess: yeah, we're heading up to ATL by car on Saturday early morning (like 4am) | 16:59 |
jaypipes | cburgess: all pugs aboard the pug train. | 16:59 |
cburgess | jaypipes Oh good, stay safe. You coming to denver or going to be riding it in out in ATL? | 16:59 |
*** baoli has quit IRC | 17:00 | |
openstackgerrit | Chris Dent proposed openstack/nova master: WIP: [placement] POST /allocations to set allocations for >1 consumers https://review.openstack.org/500073 | 17:00 |
jaypipes | cburgess: I got delta to put me on the monday flight instead of the sunday flight so I'll be in Denver Monday evening through Friday night | 17:00 |
openstackgerrit | Stephen Finucane proposed openstack/nova-specs master: DNM: Unformatted file https://review.openstack.org/501798 | 17:00 |
openstackgerrit | Stephen Finucane proposed openstack/nova-specs master: DNM! Make "reproposals" more obvious to readers https://review.openstack.org/501797 | 17:00 |
*** edmondsw has quit IRC | 17:00 | |
*** baoli has joined #openstack-nova | 17:00 | |
*** cdent has quit IRC | 17:00 | |
cburgess | jaypipes Cool, well hope the drive is smooth and your house gets through it. | 17:01 |
sean-k-mooney | stephenfin: are you +1 or -3 on the idea of using the sriov nic agent to create the vf resouce providers and apply traits such as physnets? | 17:01 |
*** david_ has joined #openstack-nova | 17:01 | |
sean-k-mooney | stephenfin: im assuming most people will not have an in between on that idea | 17:01 |
*** crushil has joined #openstack-nova | 17:01 | |
jaypipes | cburgess: hotel with the pugs for Sat and Sun nights, Julie flies to Italy from ATL on Sunday afternoon, parents driving down from Maine to meet me in ATL on Sunday night to take the girls and then I fly out Monday to Denver... | 17:01 |
jaypipes | cburgess: total logistical nightmare. | 17:01 |
cburgess | OMG wow.. thats insane. Poor pugs. | 17:02 |
jaypipes | cburgess: everything basically needs to work flawlessly for this to pull off successfully... | 17:02 |
*** itlinux has quit IRC | 17:02 | |
jaypipes | cburgess: no accidents, etc | 17:02 |
jaypipes | so, crossing my fingers | 17:02 |
cburgess | jaypipes I feel for you. We just went through similar levels of insanity in my family. We have 2 95+ year old grandparents in our family that were in the direct path of Harvey. So.. been there, done that. | 17:03 |
*** david_ has quit IRC | 17:03 | |
openstackgerrit | Merged openstack/nova master: Allow setting up multiple cells in the base TestCase https://review.openstack.org/501513 | 17:04 |
openstackgerrit | Merged openstack/nova master: Make ConductorTaskTestCase run with 2 cells https://review.openstack.org/501514 | 17:05 |
*** gvrangan has joined #openstack-nova | 17:05 | |
openstackgerrit | Merged openstack/nova master: Track which cell each instance is created in and use it consistently https://review.openstack.org/501452 | 17:06 |
dansmith | woo | 17:06 |
openstackgerrit | Merged openstack/nova master: Ensure instance mapping is updated in case of quota recheck fails https://review.openstack.org/501408 | 17:07 |
*** ssmith has quit IRC | 17:08 | |
jaypipes | cburgess: damn :( | 17:08 |
cburgess | jaypipes Good news, no property damage to either home. bad news, 98 year old grandmother still can't go home to Beaumont because no clean water in the city. SO she is still in ATL with my folks. | 17:08 |
jaypipes | cburgess: ugh. well, at least there's no prop damage. | 17:09 |
*** gbarros has quit IRC | 17:09 | |
cburgess | jaypipes yeah hope you fare as well | 17:10 |
*** edmondsw has joined #openstack-nova | 17:10 | |
*** shan has joined #openstack-nova | 17:14 | |
*** edmondsw has quit IRC | 17:14 | |
*** itlinux has joined #openstack-nova | 17:15 | |
*** suresh12 has joined #openstack-nova | 17:16 | |
*** vvargaszte has joined #openstack-nova | 17:17 | |
*** jpena is now known as jpena|off | 17:17 | |
*** vvargaszte has quit IRC | 17:17 | |
*** vvargaszte has joined #openstack-nova | 17:18 | |
*** harlowja has joined #openstack-nova | 17:19 | |
*** baoli has quit IRC | 17:19 | |
*** baoli has joined #openstack-nova | 17:20 | |
*** suresh12 has quit IRC | 17:20 | |
*** suresh12 has joined #openstack-nova | 17:20 | |
*** edmondsw has joined #openstack-nova | 17:21 | |
*** itlinux has quit IRC | 17:21 | |
*** gszasz has quit IRC | 17:22 | |
*** tesseract has quit IRC | 17:23 | |
*** edmondsw has quit IRC | 17:25 | |
*** rajathagasthya has quit IRC | 17:25 | |
*** psachin has quit IRC | 17:31 | |
openstackgerrit | Michael Still proposed openstack/nova master: Move nbd commands to privsep. https://review.openstack.org/500351 | 17:31 |
openstackgerrit | Michael Still proposed openstack/nova master: Move lvm handling to privsep. https://review.openstack.org/495516 | 17:31 |
openstackgerrit | Michael Still proposed openstack/nova master: Move xend existence probes to privsep. https://review.openstack.org/495538 | 17:31 |
openstackgerrit | Michael Still proposed openstack/nova master: Move shred to privsep. https://review.openstack.org/495537 | 17:31 |
openstackgerrit | Michael Still proposed openstack/nova master: Cleanup mount / umount and associated rmdir calls https://review.openstack.org/494423 | 17:31 |
openstackgerrit | Michael Still proposed openstack/nova master: WIP / Aspirational: we don't need rootwrap any more. https://review.openstack.org/495542 | 17:31 |
openstackgerrit | Michael Still proposed openstack/nova master: Move loopback setup and removal to privsep. https://review.openstack.org/495664 | 17:31 |
openstackgerrit | Michael Still proposed openstack/nova master: Move libvirt usages of chown to privsep. https://review.openstack.org/471972 | 17:31 |
openstackgerrit | Michael Still proposed openstack/nova master: Don't shell out to mkdir, use ensure_tree() https://review.openstack.org/492326 | 17:31 |
openstackgerrit | Michael Still proposed openstack/nova master: Move ploop commands to privsep. https://review.openstack.org/492325 | 17:31 |
openstackgerrit | Michael Still proposed openstack/nova master: Read from console ptys using privsep. https://review.openstack.org/489486 | 17:32 |
openstackgerrit | Michael Still proposed openstack/nova master: Move the idmapshift binary into privsep. https://review.openstack.org/495541 | 17:32 |
openstackgerrit | Michael Still proposed openstack/nova master: Move kpartx calls to privsep. https://review.openstack.org/500354 | 17:32 |
openstackgerrit | Michael Still proposed openstack/nova master: Move execs of touch to privsep. https://review.openstack.org/489190 | 17:32 |
openstackgerrit | Michael Still proposed openstack/nova master: Move libvirts dmcrypt support to privsep. https://review.openstack.org/490737 | 17:32 |
openstackgerrit | Michael Still proposed openstack/nova master: Move blkid calls to privsep. https://review.openstack.org/500398 | 17:32 |
openstackgerrit | Michael Still proposed openstack/nova master: Move execs of tee to privsep. https://review.openstack.org/489438 | 17:32 |
openstackgerrit | Matt Riedemann proposed openstack/nova stable/pike: Allow setting up multiple cells in the base TestCase https://review.openstack.org/501818 | 17:32 |
openstackgerrit | Matt Riedemann proposed openstack/nova stable/pike: Make ConductorTaskTestCase run with 2 cells https://review.openstack.org/501819 | 17:32 |
openstackgerrit | Matt Riedemann proposed openstack/nova stable/pike: Track which cell each instance is created in and use it consistently https://review.openstack.org/501820 | 17:32 |
openstackgerrit | Matt Riedemann proposed openstack/nova stable/pike: Ensure instance mapping is updated in case of quota recheck fails https://review.openstack.org/501821 | 17:32 |
mikal | sdague: this round of gate smashing is all updates of reno to document adds and removes | 17:32 |
*** edmondsw has joined #openstack-nova | 17:33 | |
*** gouthamr has quit IRC | 17:33 | |
*** edmondsw has quit IRC | 17:37 | |
*** edmondsw has joined #openstack-nova | 17:37 | |
*** slaweq has joined #openstack-nova | 17:38 | |
*** rajathagasthya has joined #openstack-nova | 17:38 | |
*** baoli has quit IRC | 17:39 | |
*** baoli has joined #openstack-nova | 17:41 | |
*** slaweq has quit IRC | 17:43 | |
*** rmcall has quit IRC | 17:45 | |
*** rmcall has joined #openstack-nova | 17:46 | |
*** tidwellr has quit IRC | 17:48 | |
*** edmondsw has quit IRC | 17:49 | |
*** vvargaszte has quit IRC | 17:49 | |
*** gbarros has joined #openstack-nova | 17:55 | |
*** gvrangan has quit IRC | 17:55 | |
*** jmlowe has joined #openstack-nova | 17:55 | |
*** jmlowe_ has quit IRC | 17:57 | |
*** penick has quit IRC | 17:58 | |
*** edmondsw has joined #openstack-nova | 17:59 | |
*** tidwellr has joined #openstack-nova | 18:00 | |
*** rmart04 has quit IRC | 18:01 | |
*** penick has joined #openstack-nova | 18:01 | |
*** gouthamr has joined #openstack-nova | 18:01 | |
*** vvargaszte has joined #openstack-nova | 18:02 | |
*** vvargaszte has quit IRC | 18:02 | |
*** marst has joined #openstack-nova | 18:03 | |
mikal | mriedem: you seem to have forgotten to schedule a three hours session on privsep for first thing Wednesday. I can fix the etherpad if you want. | 18:05 |
*** derekh has quit IRC | 18:05 | |
*** jmlowe has quit IRC | 18:06 | |
*** jmlowe has joined #openstack-nova | 18:06 | |
mriedem | you can have 4pm on friday | 18:06 |
*** Swami has joined #openstack-nova | 18:06 | |
*** liverpooler has quit IRC | 18:07 | |
*** suresh12 has quit IRC | 18:07 | |
*** rmcall has quit IRC | 18:10 | |
*** Apoorva has quit IRC | 18:11 | |
sean-k-mooney | stephenfin: by the way when are you flying out | 18:13 |
mikal | mriedem: don't make me break the gate more | 18:13 |
sean-k-mooney | stephenfin: i get in late saturday night will you be around deven on sunday/monday. | 18:14 |
sean-k-mooney | stephenfin: if you are interested in the nova neutron negociation work i can bring you up to speed on where we are and how i want to evolve it to support minium band with gurrentess,neutron provided traits and several other usecases. | 18:15 |
sean-k-mooney | stephenfin: got to go buy a suitcase so ttyl | 18:16 |
*** edmondsw has quit IRC | 18:18 | |
*** egonzalez has joined #openstack-nova | 18:21 | |
*** manasm has quit IRC | 18:23 | |
*** edmondsw has joined #openstack-nova | 18:24 | |
openstackgerrit | Matt Riedemann proposed openstack/nova master: Transform instance.resize.error notifications https://review.openstack.org/417882 | 18:24 |
*** jangutter has quit IRC | 18:26 | |
*** chyka has quit IRC | 18:28 | |
*** chyka has joined #openstack-nova | 18:28 | |
*** penick has quit IRC | 18:30 | |
*** chyka_ has joined #openstack-nova | 18:32 | |
*** chyka has quit IRC | 18:33 | |
*** suresh12 has joined #openstack-nova | 18:33 | |
*** dims has quit IRC | 18:34 | |
*** dims has joined #openstack-nova | 18:36 | |
*** slaweq has joined #openstack-nova | 18:39 | |
mtreinish | sdague: hmm, the libvirt tests all fail: http://logs.openstack.org/76/501776/1/check/gate-cross-nova-python27-ubuntu-xenial/a161f12/testr_results.html.gz | 18:40 |
mtreinish | I guess it's not the group regex like I though | 18:40 |
*** Nisha_Agarwal has joined #openstack-nova | 18:41 | |
Nisha_Agarwal | jaypipes, hi | 18:41 |
*** dims has quit IRC | 18:41 | |
*** sdague has quit IRC | 18:42 | |
*** yamahata has quit IRC | 18:42 | |
*** coreywright has quit IRC | 18:43 | |
*** slaweq has quit IRC | 18:43 | |
openstackgerrit | Matt Riedemann proposed openstack/nova master: Transform keypair.delete notification https://review.openstack.org/463946 | 18:43 |
mriedem | Nisha_Agarwal: probably not a good time for jay, he's trying to avoid a hurricane | 18:44 |
Nisha_Agarwal | mriedem, ohk. | 18:44 |
*** edmondsw has quit IRC | 18:44 | |
Nisha_Agarwal | mriedem, i was proposing https://review.openstack.org/#/c/500429/ for ironic. Wanted discussion on that aspect from nova side. | 18:44 |
*** dims has joined #openstack-nova | 18:45 | |
Nisha_Agarwal | mriedem, Are u fine to discuss? or anyone else i should ask? | 18:45 |
openstackgerrit | Jackie Truong proposed openstack/nova master: Implement certificate_utils https://review.openstack.org/479949 | 18:46 |
mriedem | we plan on having some time at the ptg to talk about nova/ironic things on wednesday afternoon, | 18:46 |
mriedem | are you going to be at the ptg? | 18:46 |
*** Apoorva has joined #openstack-nova | 18:46 | |
Nisha_Agarwal | mriedem, no :( thats why i want to leave a spec before PTG starts | 18:46 |
mtreinish | mriedem: it the intent of: https://github.com/openstack/nova/blob/master/tox.ini#L30 to only run that 1 test module? | 18:46 |
mtreinish | s/it/is/g | 18:46 |
mriedem | mtreinish: idk | 18:46 |
mriedem | it was added with the osprofiler stuff | 18:47 |
mtreinish | ok, because that test path doesn't do anything, and we've been running unit tests twice... | 18:47 |
mriedem | Nisha_Agarwal: it looks like dtantsur|afk has reviewed the spec at least | 18:47 |
Nisha_Agarwal | mriedem, yes | 18:47 |
*** coreywright has joined #openstack-nova | 18:47 | |
mriedem | Nisha_Agarwal: does it fall under anything around L90 here? https://etherpad.openstack.org/p/nova-ptg-queens | 18:48 |
Nisha_Agarwal | mriedem, dtantsur|afk, wants that first we should solve the issue on the nova side before we fix on ironic side | 18:48 |
* Nisha_Agarwal checks | 18:48 | |
*** edmondsw has joined #openstack-nova | 18:48 | |
*** edmondsw has quit IRC | 18:48 | |
Nisha_Agarwal | mriedem, yes and no both | 18:48 |
Nisha_Agarwal | mriedem, yes in the sense that the topic is to avoid such ugly hack, but not with how nova supports the ironci node today | 18:49 |
*** boris_42 has quit IRC | 18:50 | |
mriedem | Nisha_Agarwal: ok can you add something under that "Ironic problems" section then and link in your spec? | 18:50 |
*** rajathagasthya has quit IRC | 18:51 | |
Nisha_Agarwal | mriedem, i was looking forward from the aspect that nova supports the 1. Multiple resource classes for ironic node. 2. Atleast the basic flavor inventroy should get scheduled from nova for ironic node, how it used to work before pike | 18:51 |
Nisha_Agarwal | mriedem, yes :) | 18:51 |
Nisha_Agarwal | mriedem, so on nova side both above points can be covered in single blueprint or u see them as two blueprints? | 18:52 |
Nisha_Agarwal | and hence one/two specs? | 18:52 |
mriedem | i don't understand the problem yet since i haven't dug into the spec, so i can't really say | 18:52 |
Nisha_Agarwal | mriedem, currently ironic node just gets scheduled based on custom resource class | 18:53 |
Nisha_Agarwal | and even basic inventory like ram, disk and cpus are not used for the ironic node scheduling | 18:53 |
*** edmondsw has joined #openstack-nova | 18:54 | |
Nisha_Agarwal | since nova now treats each of these resources as different resource class, hence ironic shud also support multiple resource class so that it can be used for scheduling by nova | 18:55 |
*** vvargaszte has joined #openstack-nova | 18:55 | |
mriedem | right, be the instance in nova consumes the entire ironic node | 18:55 |
mriedem | that's why the min/max units are 1 | 18:55 |
Nisha_Agarwal | yes, i understand the resourceclass structure | 18:56 |
mriedem | sorry, i should say, the max unit is the full amount of whatever resource class on the node | 18:56 |
Nisha_Agarwal | thats what ironic should have for any resource it wants to use for scheduling IMO | 18:56 |
Nisha_Agarwal | yes, min is 1, max= total | 18:57 |
Nisha_Agarwal | and step-size shud also be as what is total | 18:57 |
*** edmondsw has quit IRC | 18:57 | |
Nisha_Agarwal | allocation ratio also 1:0 | 18:57 |
*** edmondsw has joined #openstack-nova | 18:57 | |
*** edmondsw has quit IRC | 18:58 | |
Nisha_Agarwal | if we have this kind of structure for ironic resource, then ironic resources can be claimed fully at the placement | 18:58 |
*** vvargaszte has quit IRC | 18:59 | |
mriedem | right this https://github.com/openstack/nova/blob/master/nova/virt/ironic/driver.py#L801 | 18:59 |
mriedem | min/max/step/total/allocation are all 1 | 18:59 |
Nisha_Agarwal | mriedem, total and max should be the value of the resource | 19:00 |
Nisha_Agarwal | rest all shud be 1 | 19:00 |
Nisha_Agarwal | and if we schedule it from placement it can be fully claimed | 19:00 |
mriedem | ^ is for the custom resource class | 19:01 |
mriedem | total and max being the value of the resource is only for vcpus/ram/disk | 19:01 |
mriedem | https://github.com/openstack/nova/blob/master/nova/virt/ironic/driver.py#L769-L794 | 19:01 |
Nisha_Agarwal | mriedem, yes. i was looking for scheduling to happen based on vcpus/ram/disk | 19:01 |
mriedem | yeah i see the argument in the spec about not using the Exact filters | 19:02 |
Nisha_Agarwal | we have customers who in their production(not on master branch), use non-exact filter | 19:02 |
Nisha_Agarwal | for ironic node scheduling | 19:02 |
Nisha_Agarwal | with the custom resource class that scheduling doesnt happen | 19:03 |
Nisha_Agarwal | mriedem, thats the main reason we want atleast scheduling to happen for these resources. Eventually we would want to leverage the same resource class structure in ironic for any other quantitative resources | 19:04 |
*** edmondsw_ has joined #openstack-nova | 19:05 | |
Nisha_Agarwal | which cannot be handled by capabilities whenever they are deprecated and taken over by traits | 19:05 |
openstackgerrit | Merged openstack/nova master: Refactor out claim_resources_on_destination into a utility https://review.openstack.org/499718 | 19:06 |
mriedem | Nisha_Agarwal: so you're aware of these release notes in pike right? http://docs-draft.openstack.org/77/501477/1/check/gate-nova-releasenotes/1dc7513//releasenotes/build/html/unreleased.html#id2 | 19:07 |
Nisha_Agarwal | mriedem, yes. | 19:08 |
Nisha_Agarwal | but the same filter_scheduler could be used for scheduling ironic nodes as well, isnt it? | 19:08 |
openstackgerrit | Matthew Treinish proposed openstack/nova master: Set regex flag on ostestr command for osprofiler tests https://review.openstack.org/501842 | 19:09 |
Nisha_Agarwal | mriedem, s/filter_scheduler/enabled_filters | 19:09 |
*** edmondsw_ has quit IRC | 19:09 | |
*** slaweq has joined #openstack-nova | 19:12 | |
mriedem | i think that depends on if you're using the IronicHostManager doesn't it? | 19:13 |
mtreinish | mriedem: it also looks like that second command breaks openstack-health. We're only capturing the subunit for the last run | 19:14 |
mtreinish | so we only ever track that one test: http://status.openstack.org/openstack-health/#/job/gate-nova-python27-ubuntu-xenial | 19:14 |
*** cdent has joined #openstack-nova | 19:14 | |
mriedem | guh | 19:14 |
mriedem | idk, what are other projects doing for the osprofiler test? | 19:15 |
mriedem | running it in a separate job? | 19:15 |
mtreinish | I have no idea | 19:15 |
mriedem | cinder doesn't run it | 19:15 |
* mtreinish checks cinder | 19:15 | |
mtreinish | oh, well that answers that | 19:15 |
mriedem | nor glance | 19:15 |
mriedem | i'm not sure why nova does | 19:15 |
*** xyang1 has joined #openstack-nova | 19:15 | |
Nisha_Agarwal | mriedem, yeah but could we configure that? | 19:15 |
Nisha_Agarwal | means IronicHostManager is not configurable IMO, and Nova use it for ironic nodes | 19:16 |
*** Nel1x has joined #openstack-nova | 19:16 | |
mriedem | Nisha_Agarwal: yes it looks like if use_baremetal_filters is False, which is the default, it uses the normal enabled_filters | 19:16 |
Nisha_Agarwal | mriedem, yes thats true. ^^ | 19:16 |
mtreinish | mriedem: should I just drop that from the tox.ini? | 19:17 |
mriedem | and as of pike, enabled_filters no longer has ram/disk filters in there by default | 19:17 |
mriedem | because of placement | 19:17 |
mriedem | idk, is DinaBelova around? | 19:17 |
Nisha_Agarwal | mriedem, even non-exact filters like RamFilter, CoreFilter, diskFilter are deprecated? | 19:18 |
mriedem | Nisha_Agarwal: those weren't deprecated, just removed from the default enabled_filters list for new installs | 19:18 |
mriedem | since they are redundant with placement | 19:18 |
Nisha_Agarwal | ok. that makes sense | 19:18 |
mtreinish | mriedem: if we need to keep it around I can switch nova over to stestr which would let us combine the results into a single subunit stream | 19:19 |
Nisha_Agarwal | but they are replaced with resource classes n schedulin still happen based on them | 19:19 |
mriedem | stestr is your testrepository fork? | 19:19 |
mtreinish | yeah | 19:19 |
mtreinish | we just switched ostestr over to use that internally which is how I found all of this | 19:20 |
mriedem | reading the note at the top of https://review.openstack.org/#/c/254703/60/nova/tests/unit/test_profiler.py i don't know if we could just make the osprofiler stuff run first or not | 19:21 |
*** derekh has joined #openstack-nova | 19:22 | |
mriedem | but whatever, if we need to combine results then yeah do whatever | 19:22 |
mtreinish | mriedem: I mean yeah we could swap the order too, you'd lose the capture on the osprofiler test, but that's much lower impact than losing all the unit tests | 19:23 |
mriedem | i don't care about the capture on that | 19:23 |
mriedem | but i'm not sure if swapping it causes problems | 19:23 |
mriedem | because it instruments the classes | 19:23 |
Nisha_Agarwal | mriedem, so if we want scheduling for non-exact filters we need the resource classes for disk/vcpus/ram in place | 19:24 |
Nisha_Agarwal | for ironic nodes too | 19:24 |
mtreinish | mriedem: it sholud be isolated between runs because python exits after the tests run | 19:24 |
mtreinish | things definitely fail if you run all the tests with OS_PROFILER set to true | 19:24 |
mriedem | Nisha_Agarwal: "if we want scheduling for non-exact filters" is the major question | 19:25 |
mriedem | and it's been my limited understanding that is not what people have wanted long-term | 19:25 |
Nisha_Agarwal | mriedem, Atleast thats done in customers env | 19:25 |
mriedem | i'm really not the best person to talk about this though, it was jroll before and dtantsur|afk now, plus people running ironic deployments | 19:26 |
mriedem | Nisha_Agarwal: it would be interesting to know how many operators are running w/o the exact filters | 19:26 |
mriedem | and why | 19:26 |
mriedem | like, in the operators ML | 19:26 |
*** penick has joined #openstack-nova | 19:27 | |
Nisha_Agarwal | ironic team says nova should agree to make changes for all this then its fine to do corresponding changes in ironic | 19:27 |
Nisha_Agarwal | mriedem, i cannot get such list but i do know one customer atleast(cannot name the customer) who does have such env | 19:28 |
mriedem | Nisha_Agarwal: can you start a thread about this in the openstack-operators mailing list? | 19:28 |
Nisha_Agarwal | and they just have ironic nodes in their datacenter | 19:28 |
mriedem | tag it with [nova][ironic] | 19:29 |
Nisha_Agarwal | is it not our usual mailing list? | 19:29 |
mriedem | you could cross-post to the openstack-dev mailing list | 19:29 |
mriedem | but i'm mostly interested in hearing from ironic operators | 19:29 |
Nisha_Agarwal | ok. will just start a thread. Never wrote in this mailing list though :) | 19:30 |
mriedem | i have faith that you can do it | 19:30 |
openstackgerrit | Matthew Treinish proposed openstack/nova master: Set regex flag on ostestr command for osprofiler tests https://review.openstack.org/501842 | 19:30 |
mtreinish | mriedem: ^^^ lets see if that does it | 19:30 |
*** yamahata has joined #openstack-nova | 19:30 | |
*** pcaruana has quit IRC | 19:31 | |
mriedem | i will hold my breath in anticipation | 19:32 |
Nisha_Agarwal | mriedem, :) | 19:33 |
*** penick has quit IRC | 19:33 | |
*** sridharg has quit IRC | 19:34 | |
Nisha_Agarwal | mriedem, it would be good if you could suggest the subject for the mail....i dont know what to write actually | 19:36 |
Nisha_Agarwal | in the subject | 19:37 |
mriedem | hmm | 19:37 |
Nisha_Agarwal | body i can fill up | 19:37 |
*** edmondsw has joined #openstack-nova | 19:37 | |
*** thorst_ has quit IRC | 19:37 | |
mriedem | [nova][ironic] Concerns over rigid resource class-only ironic scheduling | 19:37 |
mriedem | ? | 19:37 |
Nisha_Agarwal | mriedem, looks good :) | 19:39 |
mriedem | stvnoyes: well looky here http://logs.openstack.org/05/501805/2/check/gate-tempest-dsvm-multinode-live-migration-ubuntu-xenial/069e82e/console.html#_2017-09-07_17_45_00_706868 | 19:42 |
*** thorst has joined #openstack-nova | 19:43 | |
*** derekh has quit IRC | 19:45 | |
Nisha_Agarwal | mriedem, do i need to subscribe to openstack-operators@lists.openstack.org for sending an email? | 19:45 |
mriedem | yeah | 19:45 |
Nisha_Agarwal | ok | 19:45 |
mriedem | ildikov: why does https://review.openstack.org/#/c/330285/ need to depend on https://review.openstack.org/#/c/498828/ ? | 19:46 |
ildikov | mriedem: uWSGI changes in Cinder blew up | 19:47 |
*** thorst has quit IRC | 19:47 | |
mriedem | why would that matter here? | 19:48 |
*** Apoorva has quit IRC | 19:48 | |
*** derekh has joined #openstack-nova | 19:48 | |
Nisha_Agarwal | mriedem, i sent an email just now. if you would want to add to any thing to the body of the mail please add it. | 19:49 |
ildikov | mriedem: because the url wasn't working with the microversion checks in the client | 19:49 |
*** armax has joined #openstack-nova | 19:50 | |
Nisha_Agarwal | mriedem, meanwhile i would open a blueprint at nova side or i should do it after responses on the email and probably after the PTG? | 19:50 |
*** rajathagasthya has joined #openstack-nova | 19:50 | |
Nisha_Agarwal | just asking | 19:50 |
*** itlinux has joined #openstack-nova | 19:50 | |
ildikov | mriedem: smcginnis might be able to describe it in details | 19:50 |
*** ijw has joined #openstack-nova | 19:51 | |
*** cdent has quit IRC | 19:52 | |
smcginnis | Wha? | 19:52 |
*** itlinux has quit IRC | 19:53 | |
*** edmondsw has quit IRC | 19:53 | |
smcginnis | Ah | 19:53 |
*** edmondsw has joined #openstack-nova | 19:53 | |
smcginnis | mriedem: There's an issue right now with the way the public endpoint is being set and how that works for microversions. | 19:53 |
smcginnis | So that needs to get fixed first. | 19:54 |
*** itlinux has joined #openstack-nova | 19:54 | |
stvnoyes | mriedem: nice to see it pass. I also figured out why I put the change in the iscsi test. the other live migrate block test was skipped dur a bug. I just removed that skip decorator - https://review.openstack.org/#/c/501828/ . If you're OK with that, I'll add the new attach checks to that test too. | 19:56 |
*** cdent has joined #openstack-nova | 19:56 | |
stvnoyes | dur = 'due to' | 19:56 |
mriedem | Nisha_Agarwal: replied with some more details from our irc discussion | 19:57 |
mriedem | thanks for starting that thread | 19:57 |
mriedem | stvnoyes: probably less worthwhile to add things to a skipped test in tempest | 19:58 |
mriedem | fwiw this does test that volume-backed live migration test https://review.openstack.org/#/c/481290/ | 19:58 |
mriedem | because it depends on https://review.openstack.org/#/c/481346/ | 19:58 |
*** edmondsw has quit IRC | 19:58 | |
Nisha_Agarwal | mriedem, thanks for the reply there | 19:58 |
stvnoyes | ah you already did it. ok I'll abandon my review since it does the same thing | 19:59 |
Nisha_Agarwal | mriedem, with respect to "eventually lead to scheduling failures later because of the scheduler thinking a node is available for more than one instance when it's really not." doesnt happen in the customer env where he uses non-exact filters | 19:59 |
mriedem | mtreinish: i'm +2 on your ostestr change, maybe you want to review this now https://review.openstack.org/#/c/498828/ | 20:00 |
Nisha_Agarwal | mriedem, the ironic node doesnt get scheduled again if use non-exact filters | 20:00 |
Nisha_Agarwal | atleast with this customer we have never heard of this issue | 20:02 |
*** suresh12 has quit IRC | 20:02 | |
mtreinish | mriedem: +2 | 20:02 |
mtreinish | although I wonder how much the tls-proxy is used now. As things move to use uwsgi the standalone proxy isn't used anymore | 20:03 |
*** suresh12 has joined #openstack-nova | 20:03 | |
*** smatzek has quit IRC | 20:05 | |
*** vvargaszte has joined #openstack-nova | 20:10 | |
*** pchavva has quit IRC | 20:11 | |
*** owalsh_afk is now known as owalsh | 20:12 | |
openstackgerrit | Matt Riedemann proposed openstack/nova-specs master: Add spec to use cinder's new attachment API https://review.openstack.org/497552 | 20:12 |
*** Nisha_Agarwal has quit IRC | 20:13 | |
*** edmondsw has joined #openstack-nova | 20:14 | |
*** itlinux has quit IRC | 20:15 | |
*** thorst has joined #openstack-nova | 20:16 | |
ildikov | mriedem: thanks ^^ | 20:16 |
mriedem | ildikov: johnthetubaguy found the same thing i did in https://review.openstack.org/#/c/493323/ | 20:16 |
*** thorst_ has joined #openstack-nova | 20:18 | |
ildikov | mriedem: I bumped the version for all the calls to 3.44 in the attach patch | 20:18 |
mriedem | that's not clear here, it wasn't called out in the commit message, and it would probably just be cleaner if we did it in this change | 20:19 |
ildikov | mriedem: I got lost on the newer version note in the review though | 20:20 |
*** thorst has quit IRC | 20:20 | |
mriedem | mine? | 20:20 |
ildikov | I was debating where to do it and thought to try to keep the patches clean | 20:20 |
mriedem | i noticed the skip_version_check=True thing, | 20:20 |
mriedem | which is wrong here, as i noted | 20:20 |
ildikov | because I haven't bumped all versions? | 20:21 |
mriedem | i'd just bump the version for attachment_create to be 3.44 as john pointed out, update whatever unit tests and make a comment about it in the commit message that we need the minimum microversion for creating the attachment to equal the microversoin needed for complete | 20:21 |
mnaser | abaranov https://review.openstack.org/#/c/501820/1 looks lik ethis failed somewhere along the line | 20:21 |
mnaser | it doesnt seem unrelated but does anyone know the bug # offhand | 20:22 |
ildikov | mriedem: that's what I planned to do | 20:22 |
*** thorst_ has quit IRC | 20:23 | |
mriedem | mnaser: unrelated, just recheck it | 20:23 |
ildikov | mriedem: as I said, I had two choices on where to bump and I chose what seemed cleaner to me, but agree with this variation too so will change when I get there later today | 20:23 |
mnaser | done, ty | 20:23 |
mriedem | mnaser: btw, wanted to ask, how do you pick up changes from stable branches? do you wait for a release or just pull from the branch? | 20:23 |
*** thorst has joined #openstack-nova | 20:24 | |
mnaser | mriedem rdo trunk stable builds | 20:24 |
mriedem | ok so don't wait for release | 20:24 |
mnaser | in cases like this, no | 20:24 |
mriedem | do you patch rpms until the changes are merged upstream then? | 20:24 |
mnaser | in very urgent scenarios yes, with puppet after the rpm is installed (not pretty) | 20:24 |
mnaser | but for the most we try to get it into stable branch asap and then deploy those packages | 20:25 |
mriedem | yeah | 20:25 |
*** tbachman has quit IRC | 20:25 | |
mriedem | dansmith: these stable backports are up when you've got the time https://review.openstack.org/#/q/topic:bug/1715462+branch:stable/pike | 20:25 |
mriedem | mtreinish: could you also go through these ^ ? | 20:26 |
mriedem | nasty regressions for pike | 20:26 |
mriedem | and we're short on stable cores atm | 20:26 |
mriedem | that aren't in europe anyway | 20:26 |
*** crushil has quit IRC | 20:26 | |
*** vvargaszte has quit IRC | 20:26 | |
openstackgerrit | Merged openstack/nova-specs master: Add spec to use cinder's new attachment API https://review.openstack.org/497552 | 20:29 |
*** Apoorva has joined #openstack-nova | 20:31 | |
*** moshele has joined #openstack-nova | 20:31 | |
*** moshele has quit IRC | 20:37 | |
*** crushil has joined #openstack-nova | 20:38 | |
*** cleong has quit IRC | 20:38 | |
*** armax has quit IRC | 20:40 | |
*** suresh12 has quit IRC | 20:40 | |
*** armax has joined #openstack-nova | 20:40 | |
*** armax has quit IRC | 20:40 | |
mtreinish | mriedem: sure, I'll take a look | 20:41 |
*** suresh12 has joined #openstack-nova | 20:41 | |
*** ijw has quit IRC | 20:45 | |
*** vvargaszte has joined #openstack-nova | 20:47 | |
*** vvargaszte has quit IRC | 20:48 | |
*** hamzy has quit IRC | 20:48 | |
*** catintheroof has quit IRC | 20:49 | |
*** catintheroof has joined #openstack-nova | 20:49 | |
*** shan has quit IRC | 20:50 | |
*** catintheroof has quit IRC | 20:50 | |
*** armax has joined #openstack-nova | 20:52 | |
*** chyka_ has quit IRC | 20:53 | |
*** chyka has joined #openstack-nova | 20:53 | |
*** cdent has quit IRC | 20:54 | |
*** dave-mccowan has quit IRC | 20:55 | |
*** itlinux has joined #openstack-nova | 20:58 | |
*** smatzek has joined #openstack-nova | 20:58 | |
*** scottda_ has joined #openstack-nova | 20:59 | |
*** cdent has joined #openstack-nova | 21:00 | |
openstackgerrit | Merged openstack/nova master: Make eventlet hub use a monotonic clock https://review.openstack.org/434327 | 21:00 |
*** thorst has quit IRC | 21:01 | |
*** cdent_ has joined #openstack-nova | 21:01 | |
*** itlinux has quit IRC | 21:01 | |
*** thorst has joined #openstack-nova | 21:01 | |
*** cdent has quit IRC | 21:05 | |
*** cdent_ is now known as cdent | 21:05 | |
*** itlinux has joined #openstack-nova | 21:05 | |
*** itlinux has quit IRC | 21:05 | |
*** thorst has quit IRC | 21:05 | |
*** egonzalez has quit IRC | 21:06 | |
*** ijw has joined #openstack-nova | 21:06 | |
*** devananda has joined #openstack-nova | 21:08 | |
*** devananda has quit IRC | 21:08 | |
*** devananda has joined #openstack-nova | 21:09 | |
*** devananda has quit IRC | 21:10 | |
*** cdent_ has joined #openstack-nova | 21:10 | |
*** itlinux has joined #openstack-nova | 21:10 | |
*** devananda has joined #openstack-nova | 21:10 | |
*** cdent has quit IRC | 21:13 | |
*** cdent_ is now known as cdent | 21:13 | |
*** devananda has quit IRC | 21:14 | |
*** devananda has joined #openstack-nova | 21:14 | |
openstackgerrit | Matt Riedemann proposed openstack/nova master: Drop support for the Cinder v2 API https://review.openstack.org/501874 | 21:14 |
smcginnis | Woot! ^ | 21:15 |
*** devananda has quit IRC | 21:15 | |
*** armax has quit IRC | 21:22 | |
*** smatzek has quit IRC | 21:23 | |
*** tbachman has joined #openstack-nova | 21:24 | |
*** baoli has quit IRC | 21:25 | |
*** armax has joined #openstack-nova | 21:25 | |
*** slaweq has quit IRC | 21:27 | |
*** annegentle has quit IRC | 21:27 | |
*** slaweq has joined #openstack-nova | 21:27 | |
*** tbachman has quit IRC | 21:30 | |
*** tbachman has joined #openstack-nova | 21:31 | |
*** slaweq has quit IRC | 21:32 | |
*** edmondsw has quit IRC | 21:32 | |
*** tidwellr has quit IRC | 21:34 | |
*** edmondsw has joined #openstack-nova | 21:34 | |
*** edmondsw has quit IRC | 21:38 | |
*** slaweq has joined #openstack-nova | 21:41 | |
*** awaugama has quit IRC | 21:42 | |
*** slaweq has quit IRC | 21:43 | |
*** slaweq has joined #openstack-nova | 21:44 | |
*** slaweq has quit IRC | 21:44 | |
*** slaweq has joined #openstack-nova | 21:46 | |
*** felipemonteiro_ has quit IRC | 21:46 | |
*** itlinux has quit IRC | 21:51 | |
*** ijw has quit IRC | 21:51 | |
*** takashin has joined #openstack-nova | 21:53 | |
*** tbachman has quit IRC | 21:53 | |
*** armax has quit IRC | 21:53 | |
*** armax has joined #openstack-nova | 21:54 | |
*** esberglu has quit IRC | 21:55 | |
*** smatzek has joined #openstack-nova | 21:55 | |
*** slaweq has quit IRC | 21:56 | |
*** derekh has quit IRC | 21:57 | |
*** armax has quit IRC | 21:59 | |
*** chyka has quit IRC | 21:59 | |
openstackgerrit | Eric Fried proposed openstack/nova master: Use ksa adapter for placement conf & requests https://review.openstack.org/492247 | 22:05 |
openstackgerrit | Eric Fried proposed openstack/nova master: nova.utils.get_ksa_adapter() https://review.openstack.org/488137 | 22:05 |
*** jaypipes has quit IRC | 22:05 | |
*** smatzek has quit IRC | 22:05 | |
*** tbachman has joined #openstack-nova | 22:06 | |
*** priteau has quit IRC | 22:07 | |
openstackgerrit | Ildiko Vancsa proposed openstack/nova master: Add attachment_complete call to volume/cinder.py https://review.openstack.org/493323 | 22:09 |
openstackgerrit | Ildiko Vancsa proposed openstack/nova master: Tweak connection_info translation for the new Cinder attach/detach API https://review.openstack.org/493324 | 22:09 |
openstackgerrit | Ildiko Vancsa proposed openstack/nova master: Implement new attach Cinder flow https://review.openstack.org/330285 | 22:09 |
*** tbachman_ has joined #openstack-nova | 22:10 | |
*** tbachman has quit IRC | 22:11 | |
*** tbachman_ is now known as tbachman | 22:11 | |
*** thegreenhundred has quit IRC | 22:11 | |
*** cdent has quit IRC | 22:12 | |
*** derekh has joined #openstack-nova | 22:17 | |
openstackgerrit | Merged openstack/nova master: Transform keypair.delete notification https://review.openstack.org/463946 | 22:19 |
*** cdent has joined #openstack-nova | 22:25 | |
openstackgerrit | Eric Fried proposed openstack/nova master: Use ksa adapter for placement conf & requests https://review.openstack.org/492247 | 22:26 |
cdent | efried: I don’t think you are being sufficiently critical of yourself in your comments | 22:27 |
*** armax has joined #openstack-nova | 22:28 | |
efried | cdent I know what I mean, and I'm VICIOUS. | 22:28 |
cdent | :) | 22:28 |
*** annegentle has joined #openstack-nova | 22:33 | |
*** tbachman has quit IRC | 22:36 | |
openstackgerrit | Chris Dent proposed openstack/nova master: Optional separate database for placement API https://review.openstack.org/362766 | 22:38 |
openstackgerrit | Eric Fried proposed openstack/nova master: Get auth from context for glance endpoint https://review.openstack.org/490057 | 22:39 |
*** annegentle has quit IRC | 22:40 | |
*** priteau has joined #openstack-nova | 22:43 | |
*** priteau has quit IRC | 22:43 | |
*** priteau has joined #openstack-nova | 22:44 | |
*** ijw has joined #openstack-nova | 22:44 | |
*** suresh12 has quit IRC | 22:45 | |
*** priteau has quit IRC | 22:48 | |
*** marst has quit IRC | 22:49 | |
openstackgerrit | Ildiko Vancsa proposed openstack/nova master: Add attachment_complete call to volume/cinder.py https://review.openstack.org/493323 | 22:51 |
openstackgerrit | Ildiko Vancsa proposed openstack/nova master: Tweak connection_info translation for the new Cinder attach/detach API https://review.openstack.org/493324 | 22:51 |
openstackgerrit | Ildiko Vancsa proposed openstack/nova master: Implement new attach Cinder flow https://review.openstack.org/330285 | 22:51 |
*** thorst has joined #openstack-nova | 22:54 | |
*** suresh12 has joined #openstack-nova | 22:57 | |
*** thorst has quit IRC | 22:59 | |
*** Apoorva_ has joined #openstack-nova | 22:59 | |
*** claudiub has quit IRC | 23:00 | |
*** ijw has quit IRC | 23:01 | |
*** tbachman has joined #openstack-nova | 23:01 | |
*** Apoorva has quit IRC | 23:02 | |
*** itlinux has joined #openstack-nova | 23:04 | |
*** itlinux has quit IRC | 23:04 | |
*** tbachman_ has joined #openstack-nova | 23:05 | |
*** armax has quit IRC | 23:05 | |
*** tbachman has quit IRC | 23:05 | |
*** tbachman_ is now known as tbachman | 23:05 | |
*** xyang1 has quit IRC | 23:06 | |
*** gouthamr has quit IRC | 23:09 | |
*** lyan has quit IRC | 23:12 | |
*** gbarros has quit IRC | 23:18 | |
*** yassine has joined #openstack-nova | 23:20 | |
*** chyka has joined #openstack-nova | 23:20 | |
*** hongbin has quit IRC | 23:24 | |
*** chyka has quit IRC | 23:25 | |
*** zhenq has quit IRC | 23:30 | |
*** Apoorva has joined #openstack-nova | 23:32 | |
*** Apoorva_ has quit IRC | 23:32 | |
*** vladikr has quit IRC | 23:38 | |
*** tbachman has quit IRC | 23:41 | |
*** tbachman has joined #openstack-nova | 23:45 | |
*** vladikr has joined #openstack-nova | 23:52 | |
*** slaweq has joined #openstack-nova | 23:58 | |
*** gouthamr has joined #openstack-nova | 23:59 |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!