| *** yingjun has joined #openstack-nova | 00:01 | |
| *** devkulkarni1 has quit IRC | 00:03 | |
| *** annegentle has quit IRC | 00:05 | |
| *** harlowja has quit IRC | 00:06 | |
| *** devkulkarni has joined #openstack-nova | 00:06 | |
| *** ayoung has quit IRC | 00:08 | |
| *** devkulkarni has quit IRC | 00:11 | |
| *** esberglu has joined #openstack-nova | 00:12 | |
| *** esberglu has quit IRC | 00:16 | |
| *** raunak has quit IRC | 00:17 | |
| *** portdirect is now known as portdirect_away | 00:18 | |
| *** raunak has joined #openstack-nova | 00:19 | |
| *** portdirect_away has quit IRC | 00:19 | |
| *** raunak has quit IRC | 00:19 | |
| *** siva_krish1 has joined #openstack-nova | 00:20 | |
| *** siva_krish has quit IRC | 00:20 | |
| *** marst has quit IRC | 00:20 | |
| openstackgerrit | melanie witt proposed openstack/nova: Add cell_uuid with target_cell to save DB lookups https://review.openstack.org/411543 | 00:21 |
|---|---|---|
| *** mtanino has quit IRC | 00:23 | |
| *** mlavalle has quit IRC | 00:24 | |
| *** raunak has joined #openstack-nova | 00:24 | |
| *** raunak has quit IRC | 00:25 | |
| *** dtp has joined #openstack-nova | 00:25 | |
| melwitt | dansmith: you can check how much you hate that ^ at your leisure. it took awhile because I got tripped up by SingleCellSimple's patching of target_cell | 00:25 |
| *** efried has quit IRC | 00:27 | |
| *** efried1 has joined #openstack-nova | 00:27 | |
| *** sdague has quit IRC | 00:28 | |
| *** Jeffrey4l_ is now known as Jeffrey4l | 00:28 | |
| *** gouthamr has joined #openstack-nova | 00:30 | |
| *** yingjun has quit IRC | 00:31 | |
| *** yingjun has joined #openstack-nova | 00:32 | |
| openstackgerrit | Michael Still proposed openstack/nova: libvirt: fix vCPU usage reporing for LXC/QEMU guests https://review.openstack.org/393254 | 00:33 |
| tonyb | andreykurilin: ping? | 00:33 |
| andreykurilin | tonyb: pong | 00:34 |
| *** harlowja has joined #openstack-nova | 00:34 | |
| tonyb | andreykurilin: Can you give me more deatils on https://review.openstack.org/#/c/411518/ ? | 00:34 |
| tonyb | andreykurilin: I'm confused | 00:34 |
| tonyb | andreykurilin: I can't find the gate job you mention and the that bug isn't supposted to affect trusty | 00:35 |
| *** raildo_ has quit IRC | 00:35 | |
| andreykurilin | tonyb: all novaclient's patches started fail today after 12UTC. for example - http://logs.openstack.org/06/350106/14/check/gate-novaclient-dsvm-functional-mitaka-ubuntu-trusty-nv/7ce219e/ | 00:37 |
| andreykurilin | tonyb: updating virtualenv resolves an issue - https://review.openstack.org/#/c/411519/ | 00:37 |
| tonyb | andreykurilin: ok I'll look at it. | 00:38 |
| andreykurilin | tonyb: I'm no sure that virtualenv is a root of this issue, since that version was used for a long time and it started fail only today, but as I said before, updating helped | 00:38 |
| tonyb | andreykurilin: I didn't look on master for the job | 00:38 |
| tonyb | andreykurilin: Yeah it's very strange | 00:39 |
| tonyb | andreykurilin: It's a non-voting job so IIUC it isn't blocking that gate butit is training you to ignore that job | 00:39 |
| andreykurilin | tonyb: I had a talk with clarkb an hour ago at #openstack-infra, he made an assumption that issue relates to new release of setuptools, which were done earlier today | 00:41 |
| tonyb | andreykurilin: Ahh okay that's good info to have. | 00:42 |
| andreykurilin | tonyb: nv jobs doesn't train to ignore it. nv just allows to continue main development regardless of external issues :) | 00:50 |
| andreykurilin | tonyb: I think this job has nv suffix by historical reasons. it was added to not block gates while adding new job and we forgot to remove nv suffix. | 00:50 |
| tonyb | andreykurilin: :) | 00:50 |
| andreykurilin | tonyb: and I'm happy that it has now nv suffix, it allowed to merge a half of my patches from long chain(14 patches) :) | 00:51 |
| tonyb | :) | 00:51 |
| *** ayoung has joined #openstack-nova | 00:53 | |
| tonyb | andreykurilin: Can we try with 15.0.3 ? | 00:55 |
| tonyb | andreykurilin: otherwise the version on mitaka would be higher than newton | 00:55 |
| *** jamielennox is now known as jamielennox|away | 00:57 | |
| andreykurilin | tonyb: sure. But I will be near PC only tomorrow, so if you have free time now, please go ahead and update my patch :) | 00:58 |
| tonyb | andreykurilin: Thanks I will | 00:59 |
| andreykurilin | tonyb: thank you | 00:59 |
| *** jamielennox|away is now known as jamielennox | 01:01 | |
| tonyb | andreykurilin: np I'll watch them | 01:02 |
| *** unicell has quit IRC | 01:04 | |
| *** Zer0Byte__ has quit IRC | 01:05 | |
| *** tuanluong has joined #openstack-nova | 01:05 | |
| *** phuongnh has joined #openstack-nova | 01:06 | |
| *** esberglu has joined #openstack-nova | 01:06 | |
| *** raunak has joined #openstack-nova | 01:08 | |
| *** asettle has joined #openstack-nova | 01:08 | |
| *** kevinz has joined #openstack-nova | 01:10 | |
| *** esberglu has quit IRC | 01:11 | |
| *** claudiub has joined #openstack-nova | 01:12 | |
| *** asettle has quit IRC | 01:13 | |
| *** markvoelker has joined #openstack-nova | 01:14 | |
| *** mriedem has joined #openstack-nova | 01:17 | |
| *** esberglu has joined #openstack-nova | 01:17 | |
| *** markvoelker has quit IRC | 01:19 | |
| *** unicell has joined #openstack-nova | 01:20 | |
| *** zhangjl has joined #openstack-nova | 01:25 | |
| *** harlowja has quit IRC | 01:25 | |
| *** tuanluong has quit IRC | 01:26 | |
| openstackgerrit | Huan Xie proposed openstack/nova: XenAPI Use os-xenapi lib for nova https://review.openstack.org/406059 | 01:28 |
| alex_xu | edleafe: I'm wakeup now :) | 01:32 |
| alex_xu | s/I'm/I/ | 01:33 |
| openstackgerrit | Huan Xie proposed openstack/nova: XenAPI Remove useless files https://review.openstack.org/411241 | 01:33 |
| *** yuntongjin has joined #openstack-nova | 01:38 | |
| *** jamielennox is now known as jamielennox|away | 01:41 | |
| mnaser | Would anyone know where is the best place to look at this sort of bug (to troubleshoot), maybe olso.messaging? Single RabbitMQ node, "something" happens and we see a ton of these in RabbitMQ log: "operation basic.publish caused a channel exception not_found" and the cloud is basically stuck (things are all sitting on "scheduling") | 01:42 |
| mnaser | I know this isn't a support channel but I'm trying to track down the bug to resolve and see what's going on exactly. This has hit us a few times in production and it brings down the whole control plane to a halt | 01:43 |
| mnaser | The way we can get things to recover is by restarting all openstack-nova-compute services (however, I suspect that there is one of the compute nodes that is messing things up) | 01:44 |
| mnaser | because the reply_<uuid> is the same that was repeating. | 01:44 |
| *** claudiub|2 has joined #openstack-nova | 01:44 | |
| *** mvk has quit IRC | 01:46 | |
| *** jamielennox|away is now known as jamielennox | 01:47 | |
| *** claudiub has quit IRC | 01:47 | |
| *** elynn has joined #openstack-nova | 01:48 | |
| *** yuntongjin has quit IRC | 01:53 | |
| *** elynn_ has joined #openstack-nova | 01:53 | |
| *** elynn has quit IRC | 01:55 | |
| *** yufei has joined #openstack-nova | 01:57 | |
| *** yuntongjin has joined #openstack-nova | 01:57 | |
| melwitt | mnaser: you might try #openstack-operators channel and openstack-operators mailing list where people often get leads from others who might have seen the same problem before | 01:59 |
| mnaser | thanks melwitt .. we're trying to deal with nova-scheduler not being able to keep up once we reboot * | 02:00 |
| *** claudiub|2 has quit IRC | 02:00 | |
| dtp | is there a doc outlining state transitions for instance migration? | 02:01 |
| *** mdnadeem has joined #openstack-nova | 02:03 | |
| melwitt | dtp: not that I know of. you might ask tdurakov as he's done a lot of work on live migration, if he knows of any docs that might exist | 02:03 |
| dtp | thanks for the tip | 02:03 |
| *** unicell has quit IRC | 02:05 | |
| *** rwmjones has quit IRC | 02:05 | |
| *** rwmjones has joined #openstack-nova | 02:07 | |
| *** mdnadeem has quit IRC | 02:07 | |
| *** asettle has joined #openstack-nova | 02:09 | |
| *** raunak has quit IRC | 02:11 | |
| *** nic has quit IRC | 02:11 | |
| *** tuanluong has joined #openstack-nova | 02:13 | |
| *** asettle has quit IRC | 02:13 | |
| *** sreeram_v has joined #openstack-nova | 02:17 | |
| *** tjones has quit IRC | 02:18 | |
| openstackgerrit | int32bit proposed openstack/nova: Add soft delete wrinkle to api-ref https://review.openstack.org/411440 | 02:25 |
| *** armax has quit IRC | 02:27 | |
| openstackgerrit | int32bit proposed openstack/nova: Add soft delete wrinkle to api-ref https://review.openstack.org/411440 | 02:30 |
| *** Jeffrey4l has quit IRC | 02:31 | |
| *** Jeffrey4l has joined #openstack-nova | 02:31 | |
| *** markvoelker has joined #openstack-nova | 02:31 | |
| *** Jeffrey4l has quit IRC | 02:33 | |
| *** Jeffrey4l has joined #openstack-nova | 02:33 | |
| *** markvoelker has quit IRC | 02:36 | |
| *** sreeram_v has quit IRC | 02:36 | |
| *** gongysh has joined #openstack-nova | 02:37 | |
| *** sreeram_v has joined #openstack-nova | 02:37 | |
| *** dtp has quit IRC | 02:40 | |
| *** yingjun has quit IRC | 02:40 | |
| *** sreeram_v has quit IRC | 02:41 | |
| *** gcb has joined #openstack-nova | 02:45 | |
| *** esberglu has quit IRC | 02:50 | |
| *** esberglu has joined #openstack-nova | 02:50 | |
| *** yingjun has joined #openstack-nova | 02:51 | |
| openstackgerrit | Zhenyu Zheng proposed openstack/nova: Fix misleading port delete description https://review.openstack.org/410582 | 02:53 |
| *** esberglu has quit IRC | 02:55 | |
| *** yamahata has quit IRC | 02:58 | |
| *** ducttape_ has joined #openstack-nova | 02:58 | |
| *** nherciu has quit IRC | 03:02 | |
| *** ducttape_ has quit IRC | 03:02 | |
| *** zhiyan has quit IRC | 03:02 | |
| *** nherciu has joined #openstack-nova | 03:03 | |
| *** ducttape_ has joined #openstack-nova | 03:04 | |
| *** fragatina has quit IRC | 03:07 | |
| *** fragatina has joined #openstack-nova | 03:08 | |
| *** gouthamr has quit IRC | 03:10 | |
| *** asettle has joined #openstack-nova | 03:10 | |
| *** fragatina has quit IRC | 03:12 | |
| *** fragatina has joined #openstack-nova | 03:12 | |
| openstackgerrit | ChangBo Guo(gcb) proposed openstack/nova: Port api.openstack.compute.test_disk_config to Python 3 https://review.openstack.org/403536 | 03:14 |
| *** asettle has quit IRC | 03:14 | |
| *** s1061123 has quit IRC | 03:15 | |
| *** fragatin_ has joined #openstack-nova | 03:15 | |
| *** fragatina has quit IRC | 03:15 | |
| *** fragatin_ has quit IRC | 03:18 | |
| *** sudipto has joined #openstack-nova | 03:19 | |
| *** fragatina has joined #openstack-nova | 03:19 | |
| *** tyrola has quit IRC | 03:20 | |
| *** fragatina has quit IRC | 03:24 | |
| *** ducttape_ has quit IRC | 03:26 | |
| *** yuntongjin has quit IRC | 03:28 | |
| *** tyrola has joined #openstack-nova | 03:29 | |
| *** dimtruck is now known as zz_dimtruck | 03:29 | |
| *** ducnc has joined #openstack-nova | 03:31 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/nova: Updated from global requirements https://review.openstack.org/411069 | 03:31 |
| *** ducnc has quit IRC | 03:31 | |
| openstackgerrit | ChangBo Guo(gcb) proposed openstack/nova: Port security group related tests to Python 3 https://review.openstack.org/401241 | 03:31 |
| openstackgerrit | ChangBo Guo(gcb) proposed openstack/nova: Port compute unit tests to Python 3 https://review.openstack.org/402010 | 03:31 |
| *** s1061123 has joined #openstack-nova | 03:33 | |
| *** amotoki has quit IRC | 03:33 | |
| *** amotoki has joined #openstack-nova | 03:34 | |
| *** amotoki has quit IRC | 03:35 | |
| *** amotoki has joined #openstack-nova | 03:36 | |
| openstackgerrit | Huan Xie proposed openstack/nova: XenAPI Remove useless files https://review.openstack.org/411241 | 03:37 |
| openstackgerrit | Zhenyu Zheng proposed openstack/nova: Add query parameters white list for server list/detail https://review.openstack.org/408571 | 03:40 |
| *** amotoki has quit IRC | 03:42 | |
| *** baoli has joined #openstack-nova | 03:44 | |
| *** yamahata has joined #openstack-nova | 03:45 | |
| *** diogogmt has quit IRC | 03:54 | |
| openstackgerrit | melanie witt proposed openstack/nova: Make NovaException format errors fatal for tests https://review.openstack.org/411615 | 03:55 |
| *** portdirect_away has joined #openstack-nova | 03:57 | |
| *** portdirect_away is now known as portdirect | 03:57 | |
| *** psachin has joined #openstack-nova | 03:58 | |
| *** catintheroof has quit IRC | 04:06 | |
| *** catintheroof has joined #openstack-nova | 04:06 | |
| *** catintheroof has quit IRC | 04:06 | |
| *** asettle has joined #openstack-nova | 04:10 | |
| *** yingjun has quit IRC | 04:13 | |
| *** zz_dimtruck is now known as dimtruck | 04:14 | |
| *** asettle has quit IRC | 04:15 | |
| *** baoli has quit IRC | 04:17 | |
| *** thorst has joined #openstack-nova | 04:18 | |
| *** ducttape_ has joined #openstack-nova | 04:26 | |
| *** dave-mccowan has quit IRC | 04:26 | |
| *** slaweq_ has quit IRC | 04:31 | |
| *** ducttape_ has quit IRC | 04:32 | |
| *** ducttape_ has joined #openstack-nova | 04:33 | |
| *** mvk has joined #openstack-nova | 04:34 | |
| *** nicolasbock has quit IRC | 04:34 | |
| *** yingjun has joined #openstack-nova | 04:38 | |
| *** diogogmt has joined #openstack-nova | 04:40 | |
| *** bjolo_ has joined #openstack-nova | 04:42 | |
| *** thorst has quit IRC | 04:45 | |
| *** GB21 has joined #openstack-nova | 04:47 | |
| *** thorst has joined #openstack-nova | 04:49 | |
| *** thorst has quit IRC | 04:57 | |
| *** GB21 has quit IRC | 04:59 | |
| *** ducttape_ has quit IRC | 05:00 | |
| *** mdnadeem has joined #openstack-nova | 05:02 | |
| *** sreeram_v has joined #openstack-nova | 05:08 | |
| *** asettle has joined #openstack-nova | 05:11 | |
| *** prateek has joined #openstack-nova | 05:13 | |
| *** unicell has joined #openstack-nova | 05:15 | |
| *** ratailor has joined #openstack-nova | 05:15 | |
| *** asettle has quit IRC | 05:15 | |
| *** nandal has joined #openstack-nova | 05:21 | |
| *** links has joined #openstack-nova | 05:21 | |
| openstackgerrit | ChangBo Guo(gcb) proposed openstack/nova: Port network.test_manager.LdapDNSTestCase to Python 3 https://review.openstack.org/406425 | 05:22 |
| openstackgerrit | ChangBo Guo(gcb) proposed openstack/nova: Port compute.test_user_data.ServersControllerCreateTest to Python 3 https://review.openstack.org/406435 | 05:22 |
| openstackgerrit | ChangBo Guo(gcb) proposed openstack/nova: Port test_matchers.TestDictMatches.test__str__ to Python 3 https://review.openstack.org/406437 | 05:22 |
| *** fragatina has joined #openstack-nova | 05:26 | |
| *** diga has joined #openstack-nova | 05:27 | |
| *** nandal has quit IRC | 05:28 | |
| *** tlian has quit IRC | 05:35 | |
| *** fragatina has quit IRC | 05:36 | |
| *** fragatina has joined #openstack-nova | 05:36 | |
| *** GB21 has joined #openstack-nova | 05:38 | |
| *** wlhc has joined #openstack-nova | 05:39 | |
| *** diogogmt has quit IRC | 05:39 | |
| *** thorst has joined #openstack-nova | 05:44 | |
| *** thorst has quit IRC | 05:52 | |
| *** ducttape_ has joined #openstack-nova | 06:00 | |
| *** preethipy has joined #openstack-nova | 06:00 | |
| *** jaosorior has joined #openstack-nova | 06:02 | |
| *** yuntongjin has joined #openstack-nova | 06:02 | |
| *** ducttape_ has quit IRC | 06:05 | |
| *** dikonoor has joined #openstack-nova | 06:12 | |
| *** rcernin has quit IRC | 06:13 | |
| openstackgerrit | Zhenyu Zheng proposed openstack/nova: Add query parameters white list for server list/detail https://review.openstack.org/408571 | 06:23 |
| *** sridharg has joined #openstack-nova | 06:25 | |
| *** sridharg has quit IRC | 06:26 | |
| *** sridharg has joined #openstack-nova | 06:26 | |
| *** phuongnh has quit IRC | 06:31 | |
| *** phuongnh has joined #openstack-nova | 06:32 | |
| *** rcernin has joined #openstack-nova | 06:37 | |
| *** armax has joined #openstack-nova | 06:38 | |
| *** thorst has joined #openstack-nova | 06:39 | |
| *** markus_z has joined #openstack-nova | 06:42 | |
| *** rcernin has quit IRC | 06:43 | |
| *** thorst has quit IRC | 06:46 | |
| *** bks has joined #openstack-nova | 06:49 | |
| *** mjura has joined #openstack-nova | 06:53 | |
| *** rcernin has joined #openstack-nova | 06:54 | |
| *** wlhc_ has joined #openstack-nova | 06:55 | |
| *** wlhc has quit IRC | 06:55 | |
| *** Oku_OS-away is now known as Oku_OS | 06:55 | |
| *** mjura has quit IRC | 07:00 | |
| *** mjura has joined #openstack-nova | 07:00 | |
| *** ducttape_ has joined #openstack-nova | 07:01 | |
| *** phuongnh has quit IRC | 07:02 | |
| *** phuongnh has joined #openstack-nova | 07:03 | |
| *** andreas_s has joined #openstack-nova | 07:03 | |
| *** preethipy has quit IRC | 07:05 | |
| *** wlhc_ has quit IRC | 07:05 | |
| *** wlhc has joined #openstack-nova | 07:06 | |
| *** ducttape_ has quit IRC | 07:07 | |
| *** preethipy has joined #openstack-nova | 07:07 | |
| *** wlhc has quit IRC | 07:07 | |
| *** wlhc has joined #openstack-nova | 07:08 | |
| openstackgerrit | Pawel Koniszewski proposed openstack/nova: WIP: testing LM grenade job https://review.openstack.org/411662 | 07:08 |
| openstackgerrit | yunfeng zhou proposed openstack/nova: decode_as_bytes can not used when oslo.serialization < 2.15.0 https://review.openstack.org/411663 | 07:09 |
| *** siva_krish1 has quit IRC | 07:10 | |
| *** yuntongjin has quit IRC | 07:11 | |
| *** preethi has joined #openstack-nova | 07:12 | |
| *** asettle has joined #openstack-nova | 07:12 | |
| openstackgerrit | yunfeng zhou proposed openstack/nova: decode_as_bytes can not be used when oslo.serialization < 2.15.0 https://review.openstack.org/411663 | 07:14 |
| *** preethipy has quit IRC | 07:15 | |
| preethi | Please review https://review.openstack.org/#/c/410713/ - it is a minor change of adding a variable PRSM to HVType class to enable the out-of-tree driver "nova-dpm". | 07:16 |
| *** asettle has quit IRC | 07:17 | |
| *** adreznec has quit IRC | 07:21 | |
| *** bnemec has quit IRC | 07:21 | |
| *** gongysh has quit IRC | 07:22 | |
| *** mvk has quit IRC | 07:22 | |
| *** mvk has joined #openstack-nova | 07:31 | |
| *** dimtruck is now known as zz_dimtruck | 07:34 | |
| *** bnemec has joined #openstack-nova | 07:37 | |
| openstackgerrit | chenyingnan proposed openstack/nova: Port pipelib to Python 3 https://review.openstack.org/411676 | 07:37 |
| *** thorst has joined #openstack-nova | 07:38 | |
| *** derekh has joined #openstack-nova | 07:40 | |
| *** thorst has quit IRC | 07:44 | |
| *** lpetrut has joined #openstack-nova | 07:46 | |
| *** derekh has quit IRC | 07:47 | |
| *** gszasz has joined #openstack-nova | 07:48 | |
| *** GB21 has quit IRC | 07:51 | |
| openstackgerrit | Alex Xu proposed openstack/nova-specs: Add whitelist for filter and sort query parameters for server list API https://review.openstack.org/393205 | 07:55 |
| *** moshele has joined #openstack-nova | 07:56 | |
| *** pcaruana has joined #openstack-nova | 07:56 | |
| *** tesseract has joined #openstack-nova | 07:57 | |
| *** tesseract is now known as Guest31304 | 07:58 | |
| *** yuntongjin has joined #openstack-nova | 07:58 | |
| *** bjolo_ has quit IRC | 08:01 | |
| openstackgerrit | Pawel Koniszewski proposed openstack/nova: WIP: testing LM grenade job https://review.openstack.org/411662 | 08:02 |
| openstackgerrit | Huan Xie proposed openstack/nova: XenAPI Use os-xenapi lib for nova https://review.openstack.org/406059 | 08:03 |
| *** ihrachys has joined #openstack-nova | 08:04 | |
| openstackgerrit | Huan Xie proposed openstack/nova: XenAPI Remove useless files https://review.openstack.org/411241 | 08:04 |
| *** sacharya has quit IRC | 08:06 | |
| *** jamielennox is now known as jamielennox|away | 08:07 | |
| *** sacharya has joined #openstack-nova | 08:08 | |
| *** fragatina has quit IRC | 08:09 | |
| *** adreznec has joined #openstack-nova | 08:09 | |
| *** sacharya has quit IRC | 08:12 | |
| *** jamielennox|away is now known as jamielennox | 08:14 | |
| *** ralonsoh has joined #openstack-nova | 08:15 | |
| *** GB21 has joined #openstack-nova | 08:19 | |
| openstackgerrit | chenyingnan proposed openstack/nova: Port nova.cloudpipe.pipelib and nova.tests.unit.virt.xenapi.test_agent to Python 3 https://review.openstack.org/411676 | 08:21 |
| *** matrohon has joined #openstack-nova | 08:23 | |
| *** yufei has quit IRC | 08:30 | |
| *** amoralej|off is now known as amoralej | 08:31 | |
| *** yufei has joined #openstack-nova | 08:32 | |
| *** jaosorior has quit IRC | 08:32 | |
| *** jaosorior has joined #openstack-nova | 08:33 | |
| *** asettle has joined #openstack-nova | 08:34 | |
| *** wlhc has quit IRC | 08:36 | |
| *** wlhc has joined #openstack-nova | 08:37 | |
| openstackgerrit | Zhenyu Zheng proposed openstack/nova: Add query parameters white list for server list/detail https://review.openstack.org/408571 | 08:37 |
| *** asettle has quit IRC | 08:39 | |
| *** thorst has joined #openstack-nova | 08:39 | |
| *** diga has quit IRC | 08:39 | |
| *** ihrachys has quit IRC | 08:39 | |
| *** diga has joined #openstack-nova | 08:40 | |
| *** claudiub|2 has joined #openstack-nova | 08:40 | |
| *** hogepodge has quit IRC | 08:42 | |
| openstackgerrit | chenyingnan proposed openstack/nova: Port nova.cloudpipe.pipelib and nova.tests.unit.virt.xenapi.test_agent to Python 3 https://review.openstack.org/411676 | 08:43 |
| *** lpetrut has quit IRC | 08:43 | |
| *** moshele has quit IRC | 08:44 | |
| *** thorst has quit IRC | 08:44 | |
| *** hogepodge has joined #openstack-nova | 08:44 | |
| *** dikonoor has quit IRC | 08:46 | |
| openstackgerrit | jichenjc proposed openstack/nova: remove keypair server extension https://review.openstack.org/396611 | 08:49 |
| *** gongysh has joined #openstack-nova | 08:49 | |
| *** xinliang has quit IRC | 08:51 | |
| *** med_ has quit IRC | 08:53 | |
| *** gcb has quit IRC | 08:54 | |
| openstackgerrit | ChangBo Guo(gcb) proposed openstack/nova: Port network.test_manager.LdapDNSTestCase to Python 3 https://review.openstack.org/406425 | 08:55 |
| openstackgerrit | ChangBo Guo(gcb) proposed openstack/nova: Port compute.test_user_data.ServersControllerCreateTest to Python 3 https://review.openstack.org/406435 | 08:55 |
| openstackgerrit | ChangBo Guo(gcb) proposed openstack/nova: Port test_matchers.TestDictMatches.test__str__ to Python 3 https://review.openstack.org/406437 | 08:55 |
| *** jaosorior has quit IRC | 08:55 | |
| *** wlhc has quit IRC | 08:55 | |
| *** med_ has joined #openstack-nova | 08:57 | |
| *** med_ is now known as Guest67717 | 08:57 | |
| *** rwmjones is now known as rwmjones|hols | 08:59 | |
| *** zzzeek has quit IRC | 09:00 | |
| *** ihrachys has joined #openstack-nova | 09:00 | |
| *** zzzeek has joined #openstack-nova | 09:02 | |
| *** lucas-afk is now known as lucasagomes | 09:04 | |
| alex_xu | johnthetubaguy: the spec is ready https://review.openstack.org/#/c/393205/, and Kevin_Zheng is working on the patch | 09:04 |
| *** ducttape_ has joined #openstack-nova | 09:04 | |
| alex_xu | I will catch up Sean when he wake up | 09:04 |
| *** xinliang has joined #openstack-nova | 09:04 | |
| *** ihrachys has quit IRC | 09:05 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/nova: Updated from global requirements https://review.openstack.org/411069 | 09:05 |
| *** sacharya has joined #openstack-nova | 09:09 | |
| *** ducttape_ has quit IRC | 09:09 | |
| *** ihrachys has joined #openstack-nova | 09:10 | |
| *** kaisers1 has joined #openstack-nova | 09:13 | |
| *** sacharya has quit IRC | 09:13 | |
| *** rmart04 has joined #openstack-nova | 09:13 | |
| openstackgerrit | Mikhail Feoktistov proposed openstack/nova: raise exception if create Virtuozzo container with swap disk https://review.openstack.org/397829 | 09:15 |
| *** yuntongjin has quit IRC | 09:16 | |
| *** gcb has joined #openstack-nova | 09:17 | |
| *** lucasagomes is now known as lucas-brb | 09:19 | |
| *** ntt has joined #openstack-nova | 09:20 | |
| ntt | Hi, I'm using swift as a backend for glance. Is there a way to download images on compute nodes using swift or at least without using http with glance? | 09:20 |
| *** mlakat has quit IRC | 09:21 | |
| *** GB21 has quit IRC | 09:24 | |
| *** yamahata has quit IRC | 09:24 | |
| *** yassine has joined #openstack-nova | 09:29 | |
| *** yassine is now known as Guest68574 | 09:29 | |
| *** mlakat has joined #openstack-nova | 09:30 | |
| *** markus_z has quit IRC | 09:31 | |
| *** aloga has quit IRC | 09:31 | |
| *** aloga has joined #openstack-nova | 09:32 | |
| *** trananhkma has quit IRC | 09:32 | |
| *** asettle has joined #openstack-nova | 09:33 | |
| *** GB21 has joined #openstack-nova | 09:35 | |
| *** yingjun has quit IRC | 09:36 | |
| *** thorst has joined #openstack-nova | 09:39 | |
| *** diga has quit IRC | 09:41 | |
| *** derekh has joined #openstack-nova | 09:41 | |
| *** thorst has quit IRC | 09:44 | |
| *** markus_z has joined #openstack-nova | 09:46 | |
| openstackgerrit | Gábor Antal proposed openstack/nova: Transform instance.reboot notifications https://review.openstack.org/382959 | 09:47 |
| *** zenoway has joined #openstack-nova | 09:47 | |
| *** jamielennox is now known as jamielennox|away | 09:48 | |
| *** zenoway has quit IRC | 09:48 | |
| *** zenoway has joined #openstack-nova | 09:49 | |
| *** mjura_ has joined #openstack-nova | 09:53 | |
| *** jamielennox|away is now known as jamielennox | 09:55 | |
| *** mvk has quit IRC | 09:56 | |
| *** kevinz has quit IRC | 10:02 | |
| *** markus_z has quit IRC | 10:03 | |
| *** kylek3h has quit IRC | 10:05 | |
| *** ducttape_ has joined #openstack-nova | 10:05 | |
| *** rfolco has joined #openstack-nova | 10:07 | |
| *** mvk has joined #openstack-nova | 10:09 | |
| *** ducttape_ has quit IRC | 10:09 | |
| *** mjura_ has quit IRC | 10:10 | |
| *** yufei has quit IRC | 10:13 | |
| *** markus_z has joined #openstack-nova | 10:15 | |
| *** openstackgerrit has quit IRC | 10:18 | |
| *** siva_krish has joined #openstack-nova | 10:21 | |
| *** lucas-brb is now known as lucasagomes | 10:31 | |
| *** lpetrut has joined #openstack-nova | 10:32 | |
| *** thorst has joined #openstack-nova | 10:33 | |
| *** zhangjl has left #openstack-nova | 10:34 | |
| *** thorst has quit IRC | 10:41 | |
| *** cdent has joined #openstack-nova | 10:44 | |
| *** kylek3h has joined #openstack-nova | 10:47 | |
| *** phuongnh has quit IRC | 10:48 | |
| *** GB21 has quit IRC | 10:52 | |
| *** sreeram_v has quit IRC | 10:54 | |
| *** GB21 has joined #openstack-nova | 10:54 | |
| *** openstackgerrit has joined #openstack-nova | 10:55 | |
| openstackgerrit | Spencer Yu proposed openstack/nova: Replace six.iteritems() with .items() https://review.openstack.org/411752 | 10:55 |
| *** ociuhandu has joined #openstack-nova | 10:55 | |
| johnthetubaguy | alex_xu: sorry, slow this morning, taking a look at that soon | 10:56 |
| *** prateek has quit IRC | 10:59 | |
| alex_xu | johnthetubaguy: thanks | 11:01 |
| *** sreeram_v has joined #openstack-nova | 11:02 | |
| *** ducttape_ has joined #openstack-nova | 11:05 | |
| *** sreeram_v has quit IRC | 11:06 | |
| openstackgerrit | Roman Podoliaka proposed openstack/python-novaclient: Add profiling support to novaclient https://review.openstack.org/254699 | 11:06 |
| *** sacharya has joined #openstack-nova | 11:06 | |
| openstackgerrit | Roman Podoliaka proposed openstack/nova: Integrate OSProfiler and Nova https://review.openstack.org/254703 | 11:08 |
| johnthetubaguy | alex_xu: whats the difference between the filter and sort list? | 11:08 |
| alex_xu | johnthetubaguy: a little difference. the sort list is all about the db column. the filter includes all_tenant, pagination params, and few paras like 'name', 'image', those mapping to 'display_name' and 'image_ref' | 11:10 |
| *** ducttape_ has quit IRC | 11:10 | |
| *** sacharya has quit IRC | 11:11 | |
| *** bks has quit IRC | 11:12 | |
| openstackgerrit | Alex Xu proposed openstack/nova: Use JSON-Schema to validate query parameters for keypairs API https://review.openstack.org/398755 | 11:12 |
| openstackgerrit | Alex Xu proposed openstack/nova: Adds support for versioned schema validation for query parameters https://review.openstack.org/389003 | 11:12 |
| gcb | johnthetubaguy, can you help look at a python 3 porting patch, just remove items from blacklist in https://review.openstack.org/#/c/410542/ | 11:14 |
| *** siva_krish has quit IRC | 11:16 | |
| *** cdent has quit IRC | 11:16 | |
| *** gongysh has quit IRC | 11:19 | |
| *** siva_krish has joined #openstack-nova | 11:20 | |
| *** ratailor has quit IRC | 11:20 | |
| *** sdague has joined #openstack-nova | 11:21 | |
| *** dgonzalez has quit IRC | 11:22 | |
| *** preethi has quit IRC | 11:23 | |
| alex_xu | sdague: good morning, the spec is ready https://review.openstack.org/#/c/393205/ | 11:23 |
| *** dgonzalez has joined #openstack-nova | 11:24 | |
| *** roeyc has joined #openstack-nova | 11:25 | |
| *** bdeschenes has joined #openstack-nova | 11:26 | |
| *** prateek has joined #openstack-nova | 11:27 | |
| *** cdent has joined #openstack-nova | 11:27 | |
| *** thorst has joined #openstack-nova | 11:29 | |
| openstackgerrit | Alvaro Lopez Garcia proposed openstack/nova: Ensure that periodic reclaim cleans DB deleted instances https://review.openstack.org/323250 | 11:33 |
| *** roeyc has quit IRC | 11:35 | |
| *** thorst has quit IRC | 11:36 | |
| *** jaosorior has joined #openstack-nova | 11:38 | |
| *** tbachman has quit IRC | 11:42 | |
| *** snikitin has quit IRC | 11:42 | |
| *** nicolasbock has joined #openstack-nova | 11:46 | |
| *** elynn_ has quit IRC | 11:49 | |
| *** GB21 has quit IRC | 11:50 | |
| openstackgerrit | xianming.mao proposed openstack/nova: Add wsgi response for attach method https://review.openstack.org/411773 | 11:53 |
| *** paul-carlton has joined #openstack-nova | 12:00 | |
| *** paul-carlton has left #openstack-nova | 12:00 | |
| openstackgerrit | int32bit proposed openstack/nova: Add soft delete wrinkle to api-ref https://review.openstack.org/411440 | 12:06 |
| *** ducttape_ has joined #openstack-nova | 12:06 | |
| *** lpetrut1 has joined #openstack-nova | 12:07 | |
| *** lpetrut has quit IRC | 12:09 | |
| *** lpetrut1 is now known as lpetrut | 12:09 | |
| *** ducttape_ has quit IRC | 12:11 | |
| pkholkin | mriedem: alex_xu: andreykurilin: hey folks, review wanted for novaclient patch https://review.openstack.org/#/c/408166/ :) thanks! | 12:19 |
| pkholkin | andreykurilin: answered to you about decorator, I think we can't do like this | 12:19 |
| pkholkin | we just deprecate the command so we downgrade current version to 2.35, do the action, return to 2.39 (but with api_versions.wraps we will get exception) | 12:20 |
| *** siva_krish has quit IRC | 12:20 | |
| openstackgerrit | jichenjc proposed openstack/nova: Catch VolumeEncryptionNotSupported during spawn https://review.openstack.org/406109 | 12:22 |
| *** thorst has joined #openstack-nova | 12:25 | |
| openstackgerrit | jichenjc proposed openstack/python-novaclient: Add version pin for image list function https://review.openstack.org/411783 | 12:29 |
| *** bvanhav has joined #openstack-nova | 12:29 | |
| openstackgerrit | John Garbutt proposed openstack/nova: WIP: create fake ordered_nets for get_instance_nw_info https://review.openstack.org/405452 | 12:30 |
| openstackgerrit | jichenjc proposed openstack/nova: no extension for multiple create https://review.openstack.org/396622 | 12:32 |
| *** dave-mccowan has joined #openstack-nova | 12:33 | |
| *** thorst has quit IRC | 12:33 | |
| *** catintheroof has joined #openstack-nova | 12:34 | |
| *** dikonoor has joined #openstack-nova | 12:34 | |
| *** tuanluong has quit IRC | 12:35 | |
| *** lucasagomes is now known as lucas-hungry | 12:35 | |
| *** zhangyujun has quit IRC | 12:40 | |
| *** links has quit IRC | 12:42 | |
| *** markvoelker has joined #openstack-nova | 12:45 | |
| *** dikonoor has quit IRC | 12:45 | |
| *** gouthamr has joined #openstack-nova | 12:46 | |
| *** jamesdenton has joined #openstack-nova | 12:47 | |
| *** thorst has joined #openstack-nova | 12:49 | |
| *** jamesden_ has joined #openstack-nova | 12:50 | |
| alex_xu | johnthetubaguy: thanks for the review, just replied | 12:50 |
| *** lpetrut1 has joined #openstack-nova | 12:51 | |
| *** jamesdenton has quit IRC | 12:52 | |
| *** lpetrut has quit IRC | 12:52 | |
| *** lpetrut1 is now known as lpetrut | 12:52 | |
| *** tbachman has joined #openstack-nova | 12:52 | |
| wznoinsk | sfinucan, mdbooth, mriedem re intel nfv CI - it's fine since https://review.openstack.org/#/c/410563/ landed last night (GMT) | 12:53 |
| mdbooth | wznoinsk: Thanks | 12:53 |
| *** gcb has quit IRC | 12:54 | |
| openstackgerrit | Andrey Kurilin proposed openstack/python-novaclient: Transmit all auth related vars from cli to inner methods https://review.openstack.org/408704 | 12:55 |
| openstackgerrit | Andrey Kurilin proposed openstack/python-novaclient: Deprecate connection_pool variable https://review.openstack.org/408705 | 12:55 |
| openstackgerrit | Andrey Kurilin proposed openstack/python-novaclient: Make SessionClient interface similar to HTTPClient https://review.openstack.org/350140 | 12:55 |
| openstackgerrit | jichenjc proposed openstack/nova: Add document update for get console usage https://review.openstack.org/411789 | 12:56 |
| openstackgerrit | Andrey Kurilin proposed openstack/python-novaclient: Create keystone session instance if not present https://review.openstack.org/304035 | 12:56 |
| openstackgerrit | Andrey Kurilin proposed openstack/python-novaclient: Remove all code related to HTTPClient https://review.openstack.org/408728 | 12:56 |
| openstackgerrit | Andrey Kurilin proposed openstack/python-novaclient: Deprecate volume_service_name argument https://review.openstack.org/408745 | 12:56 |
| *** thorst has quit IRC | 12:56 | |
| *** thorst has joined #openstack-nova | 12:57 | |
| sdague | bauzas: https://review.openstack.org/#/c/409951/ is updated hopefully is the thing that gets us there with placement by default in CI | 12:58 |
| *** zhs_ has quit IRC | 12:58 | |
| *** weshay_bbiab is now known as weshay | 13:01 | |
| openstackgerrit | Gábor Antal proposed openstack/nova: Transform instance.reboot.error notification https://review.openstack.org/411791 | 13:04 |
| *** thorst has quit IRC | 13:05 | |
| *** esberglu has joined #openstack-nova | 13:06 | |
| *** ducttape_ has joined #openstack-nova | 13:07 | |
| *** sacharya has joined #openstack-nova | 13:07 | |
| alex_xu | johnthetubaguy: agree with you, "OS-EXT-SRV-ATTR:hostname" != "hostname" confuse people | 13:08 |
| *** baoli has joined #openstack-nova | 13:09 | |
| *** mdnadeem has quit IRC | 13:10 | |
| alex_xu | johnthetubaguy: maybe we can use alias instead of the hostname. for now, really not sure there is better way | 13:10 |
| *** esberglu has quit IRC | 13:10 | |
| *** sacharya has quit IRC | 13:11 | |
| johnthetubaguy | alex_xu: yeah, I think they all basically become an alias | 13:12 |
| *** jamesden_ has quit IRC | 13:12 | |
| alex_xu | johnthetubaguy: yea, with a microversion | 13:12 |
| johnthetubaguy | alex_xu: agreed we should allow "name", the alias, but I don't think we should allow the raw DB name "display_name" | 13:12 |
| johnthetubaguy | alex_xu: hmm, I guess it should be a microversion, but I think that means we disallow "hostname" for now, as it doesn't match anything in the API response | 13:13 |
| alex_xu | johnthetubaguy: display_name only available for the admin user. and name is one we said supported. so we can remove display_name, that sounds reasonable? | 13:13 |
| johnthetubaguy | alex_xu: yeah | 13:13 |
| johnthetubaguy | I think thats the sort of thing we agreed to remove, to make sure we don't expose DB implementation details | 13:14 |
| alex_xu | johnthetubaguy: disallow hostname make sense for me | 13:14 |
| alex_xu | johnthetubaguy: got it, that is good point | 13:15 |
| *** tblakes has joined #openstack-nova | 13:15 | |
| *** xyang1 has joined #openstack-nova | 13:15 | |
| *** sreeram_v has joined #openstack-nova | 13:18 | |
| *** pradk has joined #openstack-nova | 13:25 | |
| openstackgerrit | Alex Xu proposed openstack/nova-specs: Add whitelist for filter and sort query parameters for server list API https://review.openstack.org/393205 | 13:27 |
| alex_xu | johnthetubaguy: ^ updated | 13:27 |
| *** mkoderer has joined #openstack-nova | 13:27 | |
| *** armax has quit IRC | 13:28 | |
| markus_z | pkoniszewski: Any input for me what I can do for the backport of the fix of bug 1455252? https://review.openstack.org/#/c/406898/2 | 13:29 |
| openstack | bug 1455252 in OpenStack Compute (nova) "enabling serial console breaks live migration" [High,Fix released] https://launchpad.net/bugs/1455252 - Assigned to Markus Zoeller (markus_z) (mzoeller) | 13:29 |
| *** siva_krish has joined #openstack-nova | 13:29 | |
| *** fragatina has joined #openstack-nova | 13:29 | |
| *** amotoki has joined #openstack-nova | 13:32 | |
| *** cdent_ has joined #openstack-nova | 13:32 | |
| *** cdent has quit IRC | 13:32 | |
| *** cdent_ is now known as cdent | 13:32 | |
| johnthetubaguy | alex_xu: not sure about missing off the pre-fix bits for the filter properties | 13:35 |
| johnthetubaguy | alex_xu: I think the filter thing should match what you get in the response | 13:35 |
| johnthetubaguy | like include "OS-EXT-SRV-ATTR:hostname" if we can, I think | 13:36 |
| johnthetubaguy | for the moment that maybe means no addition of hostname | 13:36 |
| *** mnasiadka has joined #openstack-nova | 13:39 | |
| sdague | johnthetubaguy: ug, could we please not encode the terrible names forever? :) | 13:39 |
| mriedem | yeah i thought we were deprecating that | 13:39 |
| sdague | I really think the filter/sort name should be hostname not OS-EXT-SRV-ATTR:hostname | 13:40 |
| mriedem | there is a note in the code somewhere on the extended server attributes extension | 13:40 |
| johnthetubaguy | sdague: mriedem: agreed with killing that longer term | 13:40 |
| sdague | right, but the issue is, if they become official in this other list, they become harder to get rid of | 13:40 |
| johnthetubaguy | just technically we don't have hostname in the API response right now | 13:40 |
| johnthetubaguy | sdague: true | 13:40 |
| sdague | johnthetubaguy: sure, that's fine | 13:40 |
| *** ducttape_ has quit IRC | 13:40 | |
| johnthetubaguy | probably worth a microversion to remove the prefix horridness, and add the matching filter and sorting stuff in | 13:41 |
| pkoniszewski | markus_z: hey, sure, just let me take a deep look at this | 13:41 |
| sdague | johnthetubaguy: what you are saying is that we add OS-EXT-SRV-ATTR:hostname as a working filter, which has never been one before, then deprecate it right away? | 13:42 |
| sdague | that just doesn't make any sense to me | 13:42 |
| openstackgerrit | Gábor Antal proposed openstack/nova: Added instance.reboot.error to the legacy notifications https://review.openstack.org/411816 | 13:42 |
| johnthetubaguy | sdague: sorry, what I really mean is, hostname isn't a valid filter, as its not currently in the API | 13:42 |
| johnthetubaguy | display_description is a better example | 13:43 |
| johnthetubaguy | its called description in the API | 13:43 |
| johnthetubaguy | same with name and display_name, although that already has the correct alias defined | 13:43 |
| sdague | johnthetubaguy: ok, I guess I'm confused what you are proposing | 13:43 |
| sdague | alex_xu has a filter list | 13:44 |
| sdague | are you proposing we take those out? | 13:44 |
| johnthetubaguy | yes | 13:44 |
| sdague | then add them in in Pike with a microversion? | 13:45 |
| mriedem | i think i'd prefer starting with a hostname filter IN, and then microversion "OS-EXT-SRV-ATTR:" out of the response | 13:45 |
| johnthetubaguy | sdague: with their correct names that match the API | 13:45 |
| sdague | doesn't that do the thing you are most concerned about, which is breaking people's working filters? | 13:45 |
| mriedem | do the new filters work across all microversions? | 13:45 |
| gabor_antal | Hi there! Would someone please check and approve my last patch? https://review.openstack.org/411816 | 13:45 |
| johnthetubaguy | sdague: yeah | 13:46 |
| johnthetubaguy | mriedem: so these are existing ones, I think | 13:46 |
| sdague | so I do get that hostname doesn't match our REST api | 13:46 |
| *** gongysh has joined #openstack-nova | 13:46 | |
| *** jaypipes has joined #openstack-nova | 13:46 | |
| sdague | however, it's one of those I would expect actually get used | 13:46 |
| *** amoralej is now known as amoralej|lunch | 13:47 | |
| sdague | I think with the rest of this going down, I'd rather leave those in to break less people, and clean up the inconsistency later | 13:47 |
| sdague | also, are those 2 attributes the last issue on agreement here? | 13:47 |
| johnthetubaguy | sdague: yeah, thats more sensible | 13:47 |
| johnthetubaguy | sdague: honestly, speaking, I been fighting off a nasty cold since Friday, and struggling to do a sensible review of the spec right now :( | 13:49 |
| *** lucas-hungry is now known as lucasagomes | 13:49 | |
| johnthetubaguy | sdague: from a pragmatic point of view, I don't see any glaring admissions, which is the main thing | 13:50 |
| sdague | johnthetubaguy: ok, no problem. I just want to make sure that we figure out what if anything holds us back from committing here, especially with the holidays riding up on us | 13:50 |
| johnthetubaguy | just not really sure on the best route to consistency, but at least these are all things you get back in the API under some name, so thats way better than the status quo | 13:50 |
| johnthetubaguy | sdague: totally | 13:50 |
| *** mnasiadka has quit IRC | 13:51 | |
| sdague | I think at this point I'm happy to let a few extra things in here with sensible names that we should adjust the REST api to later | 13:51 |
| johnthetubaguy | sdague: yeah, thats probably the right trade off, these are all things we accept today (for admins) which is probably the key part | 13:52 |
| mnaser | is it expected behaviour that every nova-scheduler instance handles every update that's sent out from compute nodes? | 13:55 |
| openstackgerrit | int32bit proposed openstack/nova: Add soft delete wrinkle to api-ref https://review.openstack.org/411440 | 13:55 |
| mnaser | its basically amplifying load on our db by N schedulers | 13:55 |
| mnaser | not too much of an issue except when we do a mass restart of all of our compute nodes | 13:55 |
| *** bvanhav_ has joined #openstack-nova | 13:58 | |
| *** bvanhav has quit IRC | 13:59 | |
| *** ralonsoh_ has joined #openstack-nova | 14:00 | |
| openstackgerrit | jichenjc proposed openstack/python-novaclient: Add version pin for image list function https://review.openstack.org/411783 | 14:01 |
| *** sudipto has quit IRC | 14:01 | |
| *** snikitin has joined #openstack-nova | 14:02 | |
| openstackgerrit | Béla Vancsics proposed openstack/nova: Reduce code duplication https://review.openstack.org/342187 | 14:03 |
| *** ralonsoh__ has joined #openstack-nova | 14:03 | |
| *** ralonsoh has quit IRC | 14:03 | |
| openstackgerrit | Diana Clarke proposed openstack/nova: Failing test (mac osx) - test_cache_ephemeral https://review.openstack.org/411827 | 14:04 |
| *** ralonsoh has joined #openstack-nova | 14:04 | |
| *** baoli has quit IRC | 14:04 | |
| *** ralonsoh_ has quit IRC | 14:06 | |
| *** ralonsoh__ has quit IRC | 14:06 | |
| *** jsheeren has joined #openstack-nova | 14:06 | |
| sdague | mriedem: I thought you were not going to use the nova objects | 14:06 |
| jsheeren | hi all, i've got a question about the nova spice console | 14:07 |
| jsheeren | when using a windows instance, the cursor inside the instance does not follow the mouse cursor of my pc | 14:07 |
| jsheeren | also, the keymapping is off | 14:07 |
| jsheeren | letters aren't where the're supposed to be :-) | 14:08 |
| jsheeren | Is there a way to fix this? | 14:08 |
| *** amotoki has quit IRC | 14:08 | |
| openstackgerrit | int32bit proposed openstack/nova: Page the db query when filter instances with IP https://review.openstack.org/405171 | 14:09 |
| *** gongysh has quit IRC | 14:09 | |
| mriedem | sdague: i'm not | 14:13 |
| mriedem | sdague: just using a constant | 14:13 |
| alex_xu | johnthetubaguy: sdague so we also want to keep 'display_name'? the final goal keep the one mapping to db column and the alias? | 14:14 |
| sdague | ok, when I try to run this in a venv I get an issue around CellMapping not being defined | 14:14 |
| *** dave-mccowan has quit IRC | 14:14 | |
| *** baoli has joined #openstack-nova | 14:15 | |
| * alex_xu guess 'yes' from the chat log | 14:16 | |
| *** GB21 has joined #openstack-nova | 14:16 | |
| *** ducttape_ has joined #openstack-nova | 14:17 | |
| mriedem | sdague: ok i was wondering if that would be an issue since it's using the nova.objects.__init__ magic to load up the classes. i can just fully qualify the import, that should fix it. | 14:17 |
| sdague | mriedem: did you run the nova-status code locally? | 14:17 |
| sdague | mriedem: yeh, or just honestly lift the constant over | 14:17 |
| sdague | it's not like we're going to change that | 14:17 |
| mriedem | famous last words | 14:17 |
| *** mnestratov has quit IRC | 14:17 | |
| sdague | well if we do, then we change it here | 14:18 |
| openstackgerrit | Roman Podoliaka proposed openstack/nova: Integrate OSProfiler and Nova https://review.openstack.org/254703 | 14:18 |
| sdague | mriedem: I also think we probably want to print the full stack trace if things go wrong with this tool, at least for now | 14:18 |
| *** edleafe is now known as figleaf | 14:18 | |
| openstackgerrit | Julian proposed openstack/nova: Create script that prints JSON Schema of Notification Payload Classes https://review.openstack.org/411836 | 14:19 |
| sdague | because: | 14:19 |
| sdague | (.venv) os4:~/nova(review/matt_riedemann/bp/nova-status-ready-for-upgrade)> nova-status upgrade check | 14:19 |
| sdague | result = self._query(query) | 14:19 |
| sdague | Error: 'module' object has no attribute 'CellMapping' | 14:19 |
| sdague | not supper informative :) | 14:19 |
| *** gszasz has quit IRC | 14:21 | |
| *** ducttape_ has quit IRC | 14:21 | |
| openstackgerrit | int32bit proposed openstack/nova: Fix wrong disk size for sparse flat files https://review.openstack.org/396873 | 14:22 |
| *** mrsoul has quit IRC | 14:22 | |
| figleaf | alex_xu: I'm awake now too :) | 14:23 |
| alex_xu | figleaf: :) | 14:23 |
| figleaf | alex_xu: I have a question about microversion statuses | 14:24 |
| figleaf | alex_xu: http://specs.openstack.org/openstack/api-wg/guidelines/microversion_specification.html | 14:24 |
| alex_xu | figleaf: yea | 14:25 |
| figleaf | alex_xu: Under 'Version Discovery', the response JSON shows a key of 'status', with the value 'CURRENT' | 14:25 |
| *** amotoki has joined #openstack-nova | 14:25 | |
| figleaf | alex_xu: What other values for status are possible? | 14:25 |
| alex_xu | figleaf: CURRENT, SUPPORTED, the legacy v2 API is SUPPORTED status | 14:26 |
| *** vladikr has joined #openstack-nova | 14:26 | |
| figleaf | alex_xu: is DEPRECATED also possible? | 14:26 |
| alex_xu | figleaf: yes, just check this doc http://developer.openstack.org/api-guide/compute/versions.html | 14:27 |
| alex_xu | and 'EXPERIMENTAL' | 14:27 |
| *** tlian has joined #openstack-nova | 14:27 | |
| figleaf | alex_xu: ah, thanks for the link | 14:28 |
| alex_xu | figleaf: np | 14:28 |
| *** jamesdenton has joined #openstack-nova | 14:28 | |
| figleaf | alex_xu: I was looking at http://developer.openstack.org/api-guide/quick-start/index.html, which doesn't mention EXPERIMENTAL | 14:28 |
| alex_xu | figleaf: really not sure we have chance to use EXPERIMENTAL, then really don't sure when we have chance to use it, whether people still think it is right | 14:29 |
| alex_xu | figleaf: there is one in the fourth | 14:30 |
| figleaf | alex_xu: I can see it for alpha versions where you want user feedback, knowing it could change or go away | 14:30 |
| alex_xu | figleaf: yea, probably you think about placement API? | 14:30 |
| *** prateek has quit IRC | 14:31 | |
| figleaf | alex_xu: well, we're already microversioning that, so placement wouldn't be experimental | 14:31 |
| alex_xu | figleaf: yea | 14:31 |
| *** amotoki has quit IRC | 14:32 | |
| figleaf | alex_xu: but maybe a particular deployment might try out some things before figuring a good path forward | 14:32 |
| *** Dinesh_Bhor has quit IRC | 14:32 | |
| figleaf | alex_xu: that could be a way to clearly state "use this at your own risk" :) | 14:32 |
| alex_xu | figleaf: I remember there is expermental flag in the microversion discussion. finally people don't like it | 14:33 |
| figleaf | alex_xu: understood. It's basically an alias for 'alpha' for APIs | 14:34 |
| alex_xu | figleaf: yea | 14:35 |
| *** cdent has quit IRC | 14:35 | |
| *** amoralej|lunch is now known as amoralej | 14:36 | |
| *** prateek has joined #openstack-nova | 14:37 | |
| *** dansmith is now known as superdan | 14:38 | |
| *** prateek has quit IRC | 14:38 | |
| *** claudiub|2 has quit IRC | 14:38 | |
| *** prateek has joined #openstack-nova | 14:39 | |
| openstackgerrit | Alex Xu proposed openstack/nova-specs: Add whitelist for filter and sort query parameters for server list API https://review.openstack.org/393205 | 14:41 |
| alex_xu | johnthetubaguy: sdague mriedem ^ updated | 14:41 |
| *** thorst_ has joined #openstack-nova | 14:41 | |
| *** marst has joined #openstack-nova | 14:41 | |
| alex_xu | johnthetubaguy: really thanks for your review even you have cold, hope you get well soon | 14:41 |
| johnthetubaguy | alex_xu: no worries, its just the time of year for that over here | 14:42 |
| alex_xu | johnthetubaguy: :) | 14:42 |
| *** baoli has quit IRC | 14:46 | |
| *** thorst_ has quit IRC | 14:46 | |
| *** ducttape_ has joined #openstack-nova | 14:47 | |
| *** liverpooler has joined #openstack-nova | 14:47 | |
| *** sreeram_v has quit IRC | 14:51 | |
| *** thorst_ has joined #openstack-nova | 14:51 | |
| *** sreeram_v has joined #openstack-nova | 14:52 | |
| *** dave-mccowan has joined #openstack-nova | 14:52 | |
| *** andymaier has joined #openstack-nova | 15:02 | |
| openstackgerrit | Spencer Yu proposed openstack/nova: Replace six.iteritems() with .items() https://review.openstack.org/411752 | 15:02 |
| *** hongbin has joined #openstack-nova | 15:03 | |
| mriedem | wasn't there already a big ML blowout about patches like ^? | 15:04 |
| figleaf | mriedem: yeah, but the gist of the ML was not to use six.iteritems() | 15:05 |
| *** GB21 has quit IRC | 15:05 | |
| *** portdirect has quit IRC | 15:06 | |
| mriedem | until next month | 15:07 |
| *** portdirect has joined #openstack-nova | 15:07 | |
| *** sacharya has joined #openstack-nova | 15:08 | |
| *** jaypipes is now known as leakypipes | 15:09 | |
| *** Dave has quit IRC | 15:09 | |
| superdan | mriedem: well, I was looking for a video clip for you | 15:09 |
| superdan | mriedem: I think it was in the last castle, a bare-hairy-chested robert redford moving rocks from one pile to the other and then back again | 15:10 |
| superdan | but can't find a good one | 15:10 |
| mriedem | i know that movie, with tony soprano | 15:11 |
| mriedem | it's odd how tony took a break from the garbage mob thing to be a prison warden for awhile | 15:11 |
| *** Dave has joined #openstack-nova | 15:11 | |
| superdan | heh | 15:12 |
| *** sacharya has quit IRC | 15:13 | |
| *** thorst_ has quit IRC | 15:14 | |
| *** jaosorior has quit IRC | 15:15 | |
| *** derekh has quit IRC | 15:15 | |
| *** esberglu has joined #openstack-nova | 15:15 | |
| *** eharney has joined #openstack-nova | 15:15 | |
| *** mlavalle has joined #openstack-nova | 15:16 | |
| *** prateek has quit IRC | 15:17 | |
| *** tbachman has quit IRC | 15:21 | |
| *** mtanino has joined #openstack-nova | 15:22 | |
| *** jgriffith is now known as jgriffith_AutoAw | 15:22 | |
| *** tbachman has joined #openstack-nova | 15:22 | |
| *** jgriffith_AutoAw is now known as jgriffith | 15:22 | |
| *** tbachman has quit IRC | 15:25 | |
| *** armax has joined #openstack-nova | 15:25 | |
| *** zz_dimtruck is now known as dimtruck | 15:26 | |
| *** dave-mccowan has quit IRC | 15:27 | |
| *** mjura has quit IRC | 15:29 | |
| mriedem | superdan: does it matter if we've run db sync for ocata before or after we run nova-status upgrade check? | 15:29 |
| mriedem | i think you said you expected it to be run after db sync | 15:30 |
| mriedem | but i'm not sure why we need it | 15:30 |
| superdan | mriedem: not if we're not using the models.. it should be able to be run either before or after I'd say | 15:30 |
| mriedem | ok yeah that's what i was thinking, cool | 15:31 |
| *** jsheeren has quit IRC | 15:31 | |
| *** rmart04 has quit IRC | 15:32 | |
| *** GB21 has joined #openstack-nova | 15:34 | |
| sdague | mriedem: you want me to try to hack in the placement check? | 15:34 |
| sdague | or are you working on that already? | 15:35 |
| *** thorst_ has joined #openstack-nova | 15:35 | |
| *** markus_z has quit IRC | 15:35 | |
| mriedem | sdague: hold up | 15:36 |
| mriedem | sdague: i'm nearly ready to push up, then you can rebase on top of that | 15:36 |
| sdague | mriedem: are you doing placement check yet? I can do local hacking on that | 15:37 |
| mriedem | on an unrelated note, the french press i made this morning is entirely too strong | 15:37 |
| sdague | and rebase once you have new stuff | 15:37 |
| mriedem | sdague: no not yet | 15:37 |
| superdan | sdague: the way mriedem said that, it sounded like he was planning to do one push-up and have us be impressed | 15:37 |
| *** brault has joined #openstack-nova | 15:37 | |
| leakypipes | mriedem: you must have let it stew for >4 mins. superdan could have schooled you on that. | 15:38 |
| mriedem | leakypipes: it was probably 10 min, but also too many beans | 15:38 |
| superdan | leakypipes: no jay. not >4, not "almost 4", not "near 4". 4 minutes exactly. | 15:38 |
| sdague | mriedem: if I didn't already have proper coffee callibration on you, I'd say you were just being a midwesterner | 15:38 |
| leakypipes | :) | 15:38 |
| *** sreeram_v has quit IRC | 15:38 | |
| mriedem | i got excited to be working from home and actually make french press | 15:38 |
| openstackgerrit | Sergey Nikitin proposed openstack/nova: Change parent of diagnostics objects to NovaObject class https://review.openstack.org/394480 | 15:38 |
| openstackgerrit | Sergey Nikitin proposed openstack/nova: Standardization of VM diagnostics info API. https://review.openstack.org/355540 | 15:38 |
| openstackgerrit | Sergey Nikitin proposed openstack/nova: Completed implementation of instance diagnostics for Xen https://review.openstack.org/399613 | 15:38 |
| leakypipes | the number shall be 4. not 2, not 3, certainly not 5, but 4. | 15:39 |
| mriedem | i didn't realize that was a thing, i knew about it for tea | 15:39 |
| superdan | it's 4.00000000000 minutes | 15:39 |
| leakypipes | :) | 15:39 |
| superdan | variance beyond those number of places is allowed | 15:39 |
| sdague | heh | 15:39 |
| mriedem | reminds me of 'if it's brown flush it down, if it's yellow let it mellow' | 15:39 |
| mriedem | for some reason | 15:39 |
| leakypipes | mao | 15:39 |
| leakypipes | or lmao.. | 15:40 |
| *** dave-mccowan has joined #openstack-nova | 15:40 | |
| leakypipes | https://www.youtube.com/watch?v=xOrgLj9lOwk | 15:40 |
| openstackgerrit | Sergey Nikitin proposed openstack/nova: Standardization of VM diagnostics info API. https://review.openstack.org/355540 | 15:41 |
| *** ihrachys has quit IRC | 15:42 | |
| *** thorst_ has quit IRC | 15:46 | |
| *** whenry has joined #openstack-nova | 15:46 | |
| *** tbachman has joined #openstack-nova | 15:47 | |
| mriedem | sdague: i think i'm going to rework the test structure a bit so the basic patch (first in the series) is just using mocks to test the logic, | 15:47 |
| mriedem | and then when we build in new checks, like cells v2 and placement, i'll add separate test classes for those | 15:48 |
| mriedem | as they require different fixtures | 15:48 |
| *** whenry has quit IRC | 15:48 | |
| sdague | ok | 15:48 |
| *** siva_krish has quit IRC | 15:51 | |
| *** armax has quit IRC | 15:53 | |
| *** thorst_ has joined #openstack-nova | 15:57 | |
| *** andreas_s has quit IRC | 15:58 | |
| *** dtp has joined #openstack-nova | 16:01 | |
| *** thorst_ has quit IRC | 16:01 | |
| *** Guest31304 has quit IRC | 16:02 | |
| *** ducttape_ has quit IRC | 16:06 | |
| diana_clarke | sdague: Can I get your +W back on this one, please? (not my patch, just cleaning my review queue) | 16:06 |
| diana_clarke | https://review.openstack.org/#/c/350140/ | 16:07 |
| openstackgerrit | Sergey Nikitin proposed openstack/nova: Standardization of VM diagnostics info API. https://review.openstack.org/355540 | 16:07 |
| diana_clarke | sdague: and this one: https://review.openstack.org/#/c/408705/ Thanks! | 16:07 |
| *** mlakat has quit IRC | 16:10 | |
| *** baoli has joined #openstack-nova | 16:10 | |
| mriedem | sdague: incoming | 16:10 |
| openstackgerrit | Sean Dague proposed openstack/nova: Add basic placement status checking https://review.openstack.org/411883 | 16:10 |
| openstackgerrit | Matt Riedemann proposed openstack/nova: WIP: nova-status: implement _check_cellsv2 https://review.openstack.org/411525 | 16:10 |
| openstackgerrit | Matt Riedemann proposed openstack/nova: Add nova-status upgrade check command framework https://review.openstack.org/411517 | 16:10 |
| sdague | mriedem: ok, I'll rebase that | 16:10 |
| mriedem | sdague: cool. the basic framework should be a bit more solid now. | 16:11 |
| mriedem | superdan: also addressed your comments in that first patch | 16:11 |
| sdague | mriedem: I'm not adding unit tests right now, I'm stepping through partial reconstruction of a placement api real enviroment at the issues you might hit | 16:11 |
| openstackgerrit | Sarafraj Singh proposed openstack/nova: Add service_token for nova-cinder interaction https://review.openstack.org/397399 | 16:11 |
| superdan | mriedem: cool | 16:11 |
| openstackgerrit | Sarafraj Singh proposed openstack/nova: Add service_token for nova-neutron interaction https://review.openstack.org/410394 | 16:12 |
| mriedem | sdague: yeah i think once i get the cells v2 check negative tests written i'm going to bring up a newton devstack with no cells v2, install this in a venv and then tease it out | 16:12 |
| *** raunak has joined #openstack-nova | 16:12 | |
| mriedem | cells v2 is a bit easier to unit test though given the fixtures we have, and the db | 16:13 |
| *** ducttape_ has joined #openstack-nova | 16:13 | |
| mriedem | and it's not a separate service | 16:13 |
| superdan | mriedem: so you're thinking that you barf errors to stdout and return a pass/warn/fail constant for the summary at the end? | 16:14 |
| mriedem | superdan: yeah | 16:14 |
| mriedem | that's in the 2nd patch for cells v2 | 16:14 |
| mriedem | i could also print something before or after the summary table saying, there are warnings and/or errors, check the output of this command for details | 16:15 |
| superdan | I was kinda thinking/hoping we'd get something like zfs, which always describes a problem, justification, solution kind of thing | 16:15 |
| superdan | https://docs.oracle.com/cd/E18752_01/html/819-5461/gbbuw.html#gbcve | 16:15 |
| superdan | all zfs commands behave like this ^ which is super handy | 16:15 |
| sdague | superdan: I think that's all pretty doable | 16:16 |
| sdague | the important part at this point is just identifying all the failure modes | 16:16 |
| superdan | yeah | 16:16 |
| *** raunak_ has joined #openstack-nova | 16:16 | |
| *** raunak has quit IRC | 16:18 | |
| *** raunak_ is now known as raunak | 16:18 | |
| mriedem | superdan: so you want something like this? http://paste.openstack.org/show/592658/ | 16:19 |
| superdan | mriedem: that looks cool | 16:20 |
| *** siva_krish has joined #openstack-nova | 16:20 | |
| * mriedem sees himself dicking with prettytable for the next 2 hours | 16:20 | |
| mriedem | ok, i can see how that's more usable | 16:20 |
| *** cmartin has joined #openstack-nova | 16:20 | |
| *** rcernin has quit IRC | 16:23 | |
| *** raunak_ has joined #openstack-nova | 16:23 | |
| *** raunak_ has quit IRC | 16:24 | |
| *** andymaier has quit IRC | 16:25 | |
| sdague | mriedem: also, ideally, a link to what they need to do to move forward | 16:25 |
| mriedem | gonna take a break to review pkholkin's 2.39 novaclient patch | 16:25 |
| *** pcaruana has quit IRC | 16:25 | |
| mriedem | sdague: actual hyperlink? | 16:25 |
| *** mlakat has joined #openstack-nova | 16:25 | |
| sdague | mriedem: yeh | 16:25 |
| mriedem | well, i figure that'd be part of the details | 16:25 |
| mriedem | e.g. "No cell mappings found. Run the nova-manage cell_v2 simple_cell_setup command and retry." | 16:26 |
| pkholkin | (y) | 16:26 |
| sdague | so, a bunch of these like "install placement service" are probably better served with links to existing docs | 16:26 |
| mriedem | yeah that's fine | 16:26 |
| mriedem | for placement, build this into the details message http://docs.openstack.org/developer/nova/placement.html | 16:26 |
| mriedem | we don't have docs for cells v2 upgrades because superdan hates them remember? :P | 16:27 |
| mriedem | i also forgot to mention in my ML recap of the call yesterday that sdague said we haven't done anything for 3 releases | 16:27 |
| *** raunak has quit IRC | 16:27 | |
| superdan | mriedem: :| | 16:27 |
| mriedem | and that leakypipes is going to have to give a talk in boston about the placement stuff | 16:27 |
| mriedem | like, actually in front of people and everything | 16:27 |
| sdague | well, that should definitely be a thing for sure | 16:28 |
| mriedem | yeah it probably should have been in BCN | 16:28 |
| mriedem | i don't think there were any nova talks | 16:28 |
| mriedem | laski did cells v2 in austin | 16:28 |
| sdague | Cells v2 and Placement API should both end up in conference program in Boston | 16:28 |
| mriedem | yeah agreed | 16:28 |
| mriedem | assuming we have devs at the summit :) | 16:28 |
| *** bvanhav__ has joined #openstack-nova | 16:29 | |
| mriedem | funny thing is the PTG is where cross project teams are going to care about placement and hooking into it | 16:29 |
| mriedem | and those devs probably won't be at the summit | 16:29 |
| *** bvanhav_ has quit IRC | 16:29 | |
| mriedem | oh well videos | 16:29 |
| *** swamireddy has quit IRC | 16:30 | |
| *** Oku_OS is now known as Oku_OS-away | 16:32 | |
| superdan | sdague: yeah, I was thinking I should keep an eye out for the CFP and submit something | 16:33 |
| superdan | since things are getting pretty real now | 16:33 |
| *** slaweq_ has joined #openstack-nova | 16:33 | |
| mriedem | pkholkin: did you see my question here? https://review.openstack.org/#/c/392523/6/novaclient/v2/shell.py@1310 | 16:34 |
| mriedem | pkholkin: stack@filters:~$ nova image-list | 16:34 |
| mriedem | WARNING: Command image-list is deprecated and will be removed after Nova 15.0.0 is released. Use python-glanceclient or openstackclient instead | 16:34 |
| mriedem | ERROR (VersionNotFoundForAPIMethod): API version 'API Version Major: 2, Minor: 39' is not supported on 'list' method. | 16:34 |
| *** claudiub|2 has joined #openstack-nova | 16:35 | |
| *** zenoway has quit IRC | 16:36 | |
| mriedem | https://bugs.launchpad.net/python-novaclient/+bug/1650617 | 16:37 |
| openstack | Launchpad bug 1650617 in python-novaclient "Can't run nova image-list after microversion 2.35" [Undecided,New] | 16:37 |
| pkholkin | mriedem: I saw exception inside images.list, I am not sure why it is there, but yes I saw, I think decorator should be on 'list' | 16:38 |
| *** sridharg has quit IRC | 16:38 | |
| mriedem | yup, it was just misssed | 16:38 |
| mriedem | *missed | 16:38 |
| *** tjones has joined #openstack-nova | 16:38 | |
| *** preethipy has joined #openstack-nova | 16:40 | |
| openstackgerrit | jichenjc proposed openstack/python-novaclient: Add version pin for image list function https://review.openstack.org/411783 | 16:40 |
| pkholkin | mriedem: and what about this? https://github.com/openstack/python-novaclient/blob/master/novaclient/v2/images.py#L118-L125 | 16:40 |
| *** baoli has quit IRC | 16:40 | |
| pkholkin | if we add decorator on list command in shell I think we don't need this | 16:40 |
| *** zenoway has joined #openstack-nova | 16:40 | |
| pkholkin | mriedem: in general, 'nova image-list' don't fully related to my patch :) | 16:42 |
| *** browne has joined #openstack-nova | 16:44 | |
| mriedem | pkholkin: i know, but you refere to the other change that deprecated the CLIs in your patch | 16:44 |
| mriedem | anyway, jichenjc has a patch up to fix image-list now anyway | 16:44 |
| *** baoli has joined #openstack-nova | 16:44 | |
| *** baoli has quit IRC | 16:45 | |
| *** zenoway has quit IRC | 16:45 | |
| mriedem | pkholkin: i don't think you can use the decorator on https://github.com/openstack/python-novaclient/blob/master/novaclient/v2/images.py#L118-L125 | 16:45 |
| mriedem | which is why i had to hack that in | 16:45 |
| pkholkin | mriedem: I didn't try, but I read the message) | 16:46 |
| mriedem | yeah, ManagerWithFind.findall looks for detailed in the argspec | 16:46 |
| mriedem | and when you decorate the method it doesn't find it because it's looking in the decorator | 16:46 |
| *** bdeschenes has quit IRC | 16:46 | |
| pkholkin | found mentioned patch https://review.openstack.org/#/c/411783/ | 16:46 |
| mriedem | i wasted quite a bit of time trying to sort that out, and eventually gave up because we're going to drop this code soon anyway | 16:46 |
| pkholkin | mriedem: I think we can do it later | 16:47 |
| pkholkin | mriedem: I think we should fix 'image-list' in mentioned patch, not my | 16:48 |
| pkholkin | they are not fully related | 16:49 |
| pkholkin | im my patch I was doing everything about image-meta and what was said in the spec about new microversion | 16:49 |
| pkholkin | what do you think? | 16:50 |
| *** baoli has joined #openstack-nova | 16:50 | |
| *** swamireddy has joined #openstack-nova | 16:50 | |
| mriedem | pkholkin: i wasn't suggesting we fix image-list in your patch | 16:51 |
| mriedem | jichenjc has the fix for that | 16:51 |
| mriedem | pkholkin: i've commented on your change, it's all good except for the release note | 16:51 |
| pkholkin | mriedem: ok, looking | 16:51 |
| *** matrohon has quit IRC | 16:51 | |
| openstackgerrit | Sean Dague proposed openstack/nova: Add basic placement status checking https://review.openstack.org/411883 | 16:52 |
| openstackgerrit | Roman Podoliaka proposed openstack/nova-specs: Ironic: add support for instance RAID configuration https://review.openstack.org/408151 | 16:52 |
| sdague | mriedem: ok, you changed a bunch more there then I expected | 16:52 |
| sdague | mriedem: so... instead of the embedded prints everywhere | 16:52 |
| pkholkin | mriedem: ok, will fix in few minutes | 16:53 |
| sdague | what about well defined exceptions with payloads | 16:53 |
| sdague | then do the print unroll at the end | 16:53 |
| *** ralonsoh has quit IRC | 16:53 | |
| *** rpodolyaka is now known as rpodolyaka_afk | 16:54 | |
| sdague | especially as that could take all the describing of failures out of the main logic flow | 16:55 |
| openstackgerrit | Spencer Yu proposed openstack/nova: Replace six.iteritems() with .items() https://review.openstack.org/411752 | 16:55 |
| mriedem | sdague: did you see this? http://paste.openstack.org/show/592658/ | 16:55 |
| preethipy | https://review.openstack.org/#/c/410713/ - Please review the change- added PRSM type in HVType to support nova-dpm out of tree driver | 16:55 |
| sdague | mriedem: yeh | 16:55 |
| mriedem | so you want to raise exceptions? | 16:56 |
| mriedem | i was going to basically change the inner checks to return a 2 item tuple of (result, details) | 16:56 |
| mriedem | result is the enum, success/warning/failure | 16:57 |
| mriedem | details would be the message | 16:57 |
| mriedem | i don't see why we need exceptions for this | 16:57 |
| sdague | mriedem: http://paste.openstack.org/show/592663/ | 16:59 |
| sdague | mostly as a separation of conerns | 16:59 |
| sdague | concerns | 16:59 |
| sdague | you end up with blocks of text up from about what the failure modes are and the help around them | 16:59 |
| *** Swami has joined #openstack-nova | 16:59 | |
| sdague | which are outside the flow logic | 16:59 |
| sdague | anyway, just a thougt | 17:01 |
| sdague | I'm going away from keyboard for a bit | 17:01 |
| mriedem | but then how do you know what's a warning vs a failure? unless you build that into the exception class itself | 17:01 |
| mriedem | idk, i just don't really care for using exceptions for this | 17:01 |
| sdague | mriedem: right, class types | 17:01 |
| mriedem | i could create global namedtuples if you care | 17:01 |
| mriedem | UpgradeCheck = namedtuple(msg, code) | 17:02 |
| mriedem | something something | 17:02 |
| sdague | UpgradeCheckFail vs. UpgradeCheckWarning | 17:02 |
| superdan | log exceptions to a file if we want them, | 17:02 |
| superdan | but never to the console, IMHO | 17:02 |
| openstackgerrit | STEW TY proposed openstack/nova: Transform instance.rescue notifications https://review.openstack.org/385644 | 17:02 |
| mriedem | i for sure don't want tracebacks for this | 17:02 |
| *** preethipy has quit IRC | 17:02 | |
| sdague | superdan: this isn't about exception dumping | 17:03 |
| mriedem | i personally don't like exceptions for flow control | 17:03 |
| sdague | is the fact that a failure in a check is | 17:03 |
| mriedem | because you miss something and oops shit is all busted | 17:03 |
| sdague | 1) status it failed | 17:03 |
| superdan | sdague: you mean instead of returning PASS, WARN, FAIL ? | 17:03 |
| sdague | 2) basic summary | 17:03 |
| superdan | I don't like exceptions for that | 17:03 |
| sdague | 3) more details | 17:04 |
| *** rcernin has joined #openstack-nova | 17:04 | |
| sdague | and the inline print + specific return feels weird to me | 17:04 |
| mriedem | i'm not going to inline print anymore | 17:04 |
| superdan | sure, but he's not doing that now | 17:04 |
| mriedem | can i just do http://paste.openstack.org/show/592658/ and then we can over design this? | 17:05 |
| superdan | return a CheckResult class or something with those bits filled out | 17:05 |
| sdague | ok, I just rebased | 17:05 |
| sdague | and it was still there I thought | 17:05 |
| sdague | mriedem: sure | 17:05 |
| mriedem | sdague: because i haven't done it yet :) | 17:05 |
| mriedem | i was reviewing pkholkin's change | 17:05 |
| sdague | mriedem: oh, ok | 17:05 |
| mriedem | now that's done, so i'll do this | 17:05 |
| sdague | it went from return the message, to inline print + status code, I didn't realize it was going back towards the first thing | 17:06 |
| *** raunak has joined #openstack-nova | 17:06 | |
| mriedem | i'll do the thing here and then we can chat | 17:06 |
| openstackgerrit | Pavel Kholkin proposed openstack/python-novaclient: [proxy-api] microversion 2.39 deprecates image-metadata proxy API https://review.openstack.org/408166 | 17:06 |
| mriedem | shouldn't take long | 17:06 |
| pkholkin | mriedem: updated releasenote ^ | 17:06 |
| sdague | ok, well, I'm walking away from a bit. The core logic for placement should be in a state to be adapted to however the flow control ends up | 17:07 |
| mriedem | yup i glanced, looks like a nice start | 17:07 |
| mriedem | pkholkin: lgtm | 17:08 |
| mriedem | btw our next release of novaclient is going to be a monster | 17:08 |
| pkholkin | mriedem: thanks you ! | 17:08 |
| pkholkin | yes) | 17:09 |
| *** sacharya has joined #openstack-nova | 17:09 | |
| *** nic has joined #openstack-nova | 17:09 | |
| *** nic has quit IRC | 17:09 | |
| *** nic has joined #openstack-nova | 17:10 | |
| pkholkin | mriedem: thanks for giving that spec for implementing! | 17:10 |
| mriedem | np | 17:10 |
| pkholkin | can do something else | 17:10 |
| pkholkin | mriedem: feel free to propose smth to me, will look | 17:11 |
| pkholkin | also I am still interested in this spec https://review.openstack.org/#/c/381912/, for P release | 17:11 |
| pkholkin | this needed to solve some issue before | 17:12 |
| *** bvanhav_ has joined #openstack-nova | 17:13 | |
| *** bvanhav_ is now known as bvanhav | 17:13 | |
| *** jgriffith is now known as jgriffith_AutoAw | 17:13 | |
| *** bvanhav__ has quit IRC | 17:14 | |
| *** jgriffith_AutoAw is now known as jgriffith | 17:14 | |
| *** yamahata has joined #openstack-nova | 17:18 | |
| *** psachin has quit IRC | 17:20 | |
| *** cdent has joined #openstack-nova | 17:22 | |
| *** preethipy has joined #openstack-nova | 17:22 | |
| openstackgerrit | jichenjc proposed openstack/python-novaclient: Add version pin for image list function https://review.openstack.org/411783 | 17:28 |
| *** armax has joined #openstack-nova | 17:29 | |
| *** tblakes has quit IRC | 17:30 | |
| *** esberglu has quit IRC | 17:30 | |
| *** esberglu has joined #openstack-nova | 17:30 | |
| *** unicell has quit IRC | 17:33 | |
| *** lamt has joined #openstack-nova | 17:34 | |
| *** esberglu has quit IRC | 17:35 | |
| *** ayoung has quit IRC | 17:35 | |
| *** Zer0Byte__ has joined #openstack-nova | 17:44 | |
| mriedem | how ironic, the tables in the prettytable docs are not pretty https://code.google.com/archive/p/prettytable/wikis/Tutorial.wiki | 17:44 |
| *** xyang1 has quit IRC | 17:44 | |
| BobBall | That's awesomly bad. | 17:47 |
| BobBall | and can't really be called a table, let alone pretty. | 17:48 |
| openstackgerrit | Neil Jerram proposed openstack/nova: libvirt: avoid generating script with empty path https://review.openstack.org/411936 | 17:51 |
| *** bdeschenes has joined #openstack-nova | 17:51 | |
| *** eharney has quit IRC | 17:51 | |
| mriedem | i don't really know how to format this http://paste.openstack.org/show/592658/ | 17:53 |
| mriedem | it's really like tables within tables | 17:53 |
| superdan | mriedem: as long as the first two are just single lines, | 17:53 |
| superdan | then I think what you have is fine | 17:53 |
| superdan | no need to get more complex with formatting in there, IMHO | 17:53 |
| mriedem | well the thing is when you add the row if you provide a list it formats those into columns, which isn't what we want, we really have a single-column table, with rows that are their own cells | 17:54 |
| mriedem | i guess i can just munge stuff together with newlines | 17:54 |
| BobBall | Write out HTML with an embedded style sheet. That will make it easier to read | 17:55 |
| superdan | yeah, just format the string in multiple lines and hand just that string to add_row | 17:55 |
| *** bdeschenes has quit IRC | 17:55 | |
| *** bdeschenes has joined #openstack-nova | 17:58 | |
| openstackgerrit | STEW TY proposed openstack/nova: Transform instance.resize_revert notification https://review.openstack.org/396811 | 17:59 |
| *** lamt has quit IRC | 18:00 | |
| *** asettle has quit IRC | 18:01 | |
| *** asettle has joined #openstack-nova | 18:01 | |
| *** bdeschenes has quit IRC | 18:03 | |
| *** unicell has joined #openstack-nova | 18:05 | |
| *** asettle has quit IRC | 18:06 | |
| *** eharney has joined #openstack-nova | 18:07 | |
| openstackgerrit | Feodor Tersin proposed openstack/nova: libvirt: Use config types to parse XML for root disk https://review.openstack.org/411941 | 18:09 |
| openstackgerrit | Feodor Tersin proposed openstack/nova: libvirt: Improve get_disk_type_from_path https://review.openstack.org/411942 | 18:09 |
| *** preethipy has quit IRC | 18:17 | |
| *** GB21 has quit IRC | 18:18 | |
| smcginnis | mriedem: Know anything about this? https://review.openstack.org/#/c/357679/13 | 18:22 |
| smcginnis | mriedem: I'm just confused. | 18:22 |
| *** lucasagomes is now known as lucas-afk | 18:23 | |
| *** ociuhandu has quit IRC | 18:23 | |
| mriedem | smcginnis: mnestratov would be the person to ask probably | 18:25 |
| mriedem | smcginnis: we have parallels and ploop all over the libvirt code in nova | 18:25 |
| mriedem | there are random renames based on which versoins of things you're using or something, i never fully keep track of it | 18:26 |
| smcginnis | I hate it when ploop gets all over the place. | 18:26 |
| mriedem | unless that's a thing of yours | 18:26 |
| mriedem | fetish wise | 18:26 |
| * smcginnis slowly backs away from the nova channel | 18:26 | |
| mriedem | remember, we keep it classy here until mikal shows up | 18:27 |
| smcginnis | :) | 18:27 |
| *** slaweq_ has quit IRC | 18:28 | |
| cdent | mriedem: I'm making some changes to the placement-dev wip I've started at https://review.openstack.org/#/c/408313/ | 18:30 |
| cdent | would you prefer that I submit that doc all as one patch, or multiple patches, perhaps per section or something like that? | 18:30 |
| *** slaweq_ has joined #openstack-nova | 18:30 | |
| mriedem | cdent: i haven't had a chance to go through the current thing yet so up to you, multiple patches might be nice for reviewers | 18:30 |
| cdent | roger, thanks | 18:31 |
| *** asettle has joined #openstack-nova | 18:31 | |
| *** asettle has quit IRC | 18:32 | |
| *** jgriffith is now known as jgriffith_AutoAw | 18:33 | |
| *** kfarr has joined #openstack-nova | 18:33 | |
| *** esberglu has joined #openstack-nova | 18:33 | |
| *** ducttape_ has quit IRC | 18:34 | |
| mriedem | smcginnis: note the vz exe ci failed on that change | 18:35 |
| mriedem | it's nv so not sure if it matters | 18:35 |
| mriedem | http://openstack-3rd-party-storage-ci-logs.virtuozzo.com/79/357679/13/check/dsvm-tempest-exe/685a5c2/logs/devstacklog.txt.gz | 18:35 |
| mriedem | openstack image create: error: argument --disk-format: invalid choice: u'ploop' (choose from 'ami', 'ari', 'aki', 'vhd', 'vmdk', 'raw', 'qcow2', 'vhdx', 'vdi', 'iso') | 18:36 |
| mriedem | seems like a legit failure | 18:36 |
| smcginnis | mriedem: Oops, missed that. | 18:38 |
| smcginnis | Too far down in the list. | 18:39 |
| *** sihan has joined #openstack-nova | 18:39 | |
| openstackgerrit | Chris Dent proposed openstack/nova: [placement] start a placement_dev doc https://review.openstack.org/408313 | 18:40 |
| openstackgerrit | Chris Dent proposed openstack/nova: [placement] placement_dev info for microversion handling https://review.openstack.org/411946 | 18:40 |
| *** bdeschenes has joined #openstack-nova | 18:41 | |
| *** cdent has quit IRC | 18:41 | |
| *** yamahata has quit IRC | 18:44 | |
| *** haplo37_ has quit IRC | 18:44 | |
| openstackgerrit | John Garbutt proposed openstack/nova: WIP: Objectify _validate_requested_network_ids https://review.openstack.org/411950 | 18:47 |
| openstackgerrit | John Garbutt proposed openstack/nova: WIP: _validate_requested_network_ids takes objects https://review.openstack.org/411951 | 18:47 |
| openstackgerrit | John Garbutt proposed openstack/nova: WIP: _validate_requested_network_ids returns objects https://review.openstack.org/411952 | 18:47 |
| openstackgerrit | John Garbutt proposed openstack/nova: WIP: pass objects into _create_ports_for_instance https://review.openstack.org/411953 | 18:47 |
| openstackgerrit | John Garbutt proposed openstack/nova: WIP: Modify allocate_for_instance to skip create_ports https://review.openstack.org/411954 | 18:47 |
| *** fragatina has quit IRC | 18:47 | |
| *** osnewbie has joined #openstack-nova | 18:47 | |
| osnewbie | hello, has anyone here been successful with setting up openstack with controller and compute nodes using vagrantfile? | 18:48 |
| *** kfarr has quit IRC | 18:49 | |
| *** baoli has quit IRC | 18:52 | |
| mriedem | great, so now prettytable doesn't give me row separators for some reason | 18:52 |
| *** haplo37_ has joined #openstack-nova | 18:54 | |
| *** jgriffith_AutoAw is now known as jgriffith | 18:55 | |
| superdan | mriedem: we don't really need prettytable with one column for this | 18:56 |
| superdan | so if it presents any resistance at all, I'd just move to wrapping to 60 cols or something sane | 18:56 |
| mriedem | no | 18:57 |
| mriedem | NO | 18:57 |
| mriedem | baby jesus wants a gd pretty table | 18:57 |
| mriedem | and that's what he's going to get | 18:57 |
| mriedem | i think i figured it out anyway | 18:57 |
| mriedem | hrules defaults to only the frame | 18:57 |
| mriedem | rather than after each row | 18:57 |
| mriedem | yup, kabam! | 18:57 |
| * melwitt slowly backs away | 18:57 | |
| mriedem | melwitt: you can't reuse smcginnis's line | 18:57 |
| melwitt | I didn't know that was smcginnis's line | 18:58 |
| melwitt | :) | 18:58 |
| *** liusheng has quit IRC | 18:58 | |
| mriedem | melwitt: i've already scraed someone off today | 18:58 |
| mriedem | *scared | 18:58 |
| melwitt | haha, awesome | 18:58 |
| *** tblakes has joined #openstack-nova | 18:59 | |
| smcginnis | mriedem: You're on a roll today | 19:00 |
| *** clif_h has quit IRC | 19:01 | |
| mriedem | i hope textwrap can handle _() message objects | 19:03 |
| mriedem | melwitt: while you're around https://review.openstack.org/#/c/408166/ | 19:14 |
| mriedem | ^ would get that bp closed out | 19:14 |
| *** lpetrut has quit IRC | 19:14 | |
| melwitt | okay | 19:15 |
| *** baoli has joined #openstack-nova | 19:15 | |
| openstackgerrit | Matt Riedemann proposed openstack/nova: WIP: nova-status: implement _check_cellsv2 https://review.openstack.org/411525 | 19:15 |
| openstackgerrit | Matt Riedemann proposed openstack/nova: Add nova-status upgrade check command framework https://review.openstack.org/411517 | 19:15 |
| mriedem | sdague: superdan: here we go ^ | 19:15 |
| diana_clarke | Anyone want to +2 this nova client patch? It already has a +2 & I'de love to rebase on top of it once it's merged. Thanks! https://review.openstack.org/#/c/408166/ | 19:15 |
| melwitt | I'm looking at it | 19:18 |
| diana_clarke | melwitt: thanks a bunch!! | 19:18 |
| diana_clarke | (oops, maybe you were talking about the other one, lol.) | 19:19 |
| melwitt | the numbers look to be the same, 408166 | 19:20 |
| melwitt | that is, you and mriedem mentioned the same patch I thought | 19:20 |
| *** thorst_ has joined #openstack-nova | 19:20 | |
| diana_clarke | oh, wow.... friday brain ;) | 19:20 |
| *** liusheng has joined #openstack-nova | 19:21 | |
| mriedem | heh, well Vek just +Wed | 19:21 |
| *** yamahata has joined #openstack-nova | 19:22 | |
| diana_clarke | Oh, now I finally know Kevin's IRC nic :) | 19:22 |
| melwitt | yeah, I just saw. oh well | 19:22 |
| *** thorst_ has quit IRC | 19:27 | |
| diana_clarke | melwitt: I bunch of these need +W back, not sure if it's kosher to do that if you weren't the original +2 but it would be great to keep the ball rolling. https://review.openstack.org/#/q/owner:akurilin%2540mirantis.com+status:open | 19:27 |
| diana_clarke | better link: https://review.openstack.org/#/q/status:open+project:openstack/python-novaclient+branch:master+topic:session | 19:28 |
| *** slaweq_ has quit IRC | 19:28 | |
| diana_clarke | (he's not around today to ask himself) | 19:29 |
| *** slaweq_ has joined #openstack-nova | 19:30 | |
| *** READ10 has joined #openstack-nova | 19:31 | |
| *** asettle has joined #openstack-nova | 19:33 | |
| mriedem | leakypipes: are you going to rebase this series today? https://review.openstack.org/#/c/398470/ | 19:36 |
| *** asettle has quit IRC | 19:36 | |
| *** asettle has joined #openstack-nova | 19:36 | |
| *** lamt has joined #openstack-nova | 19:36 | |
| *** yamahata has quit IRC | 19:36 | |
| *** yamahata has joined #openstack-nova | 19:37 | |
| *** asettle has quit IRC | 19:39 | |
| *** owalsh has quit IRC | 19:40 | |
| *** ducttape_ has joined #openstack-nova | 19:42 | |
| *** ducttape_ has quit IRC | 19:47 | |
| *** ducttape_ has joined #openstack-nova | 19:47 | |
| *** amoralej is now known as amoralej|off | 19:48 | |
| *** lamt has quit IRC | 19:48 | |
| leakypipes | mriedem: gah. rebased it last night. didn't realize I needed to rebase again. yeah, I'll do it right away. | 19:49 |
| leakypipes | mriedem: or Wednesday night I guess... either way, yeah, working on it now. | 19:50 |
| *** lamt has joined #openstack-nova | 19:50 | |
| *** tbachman has quit IRC | 19:50 | |
| *** jamesdenton has quit IRC | 19:52 | |
| openstackgerrit | Jay Pipes proposed openstack/nova: rt: pass the nodename to public methods https://review.openstack.org/398470 | 19:52 |
| openstackgerrit | Jay Pipes proposed openstack/nova: rt: remove fluff from test_resource_tracker https://review.openstack.org/398471 | 19:52 |
| openstackgerrit | Jay Pipes proposed openstack/nova: rt: pass nodename to internal methods https://review.openstack.org/398469 | 19:52 |
| openstackgerrit | Jay Pipes proposed openstack/nova: placement: create aggregate map in report client https://review.openstack.org/407309 | 19:52 |
| openstackgerrit | Jay Pipes proposed openstack/nova: placement: correct improper test case inheritance https://review.openstack.org/406363 | 19:52 |
| openstackgerrit | Jay Pipes proposed openstack/nova: rt: explicitly pass compute node to _update() https://review.openstack.org/398472 | 19:52 |
| leakypipes | mriedem: done. | 19:52 |
| openstackgerrit | Jay Pipes proposed openstack/nova: rt: use a single ResourceTracker object instance https://review.openstack.org/398473 | 19:52 |
| *** jamesdenton has joined #openstack-nova | 19:53 | |
| *** fragatina has joined #openstack-nova | 19:55 | |
| *** clenimar has quit IRC | 19:55 | |
| *** claudiub|2 has quit IRC | 19:58 | |
| *** jamesdenton has quit IRC | 19:58 | |
| *** bvanhav has quit IRC | 19:58 | |
| openstackgerrit | sean mooney proposed openstack/os-vif: introduces MTU support for vhost-user https://review.openstack.org/410135 | 19:58 |
| *** READ10 has quit IRC | 19:59 | |
| openstackgerrit | Akshil Verma proposed openstack/python-novaclient: Fixed the __ne__ implementation in base.Resource https://review.openstack.org/409549 | 20:00 |
| *** tbachman has joined #openstack-nova | 20:06 | |
| *** lpetrut has joined #openstack-nova | 20:07 | |
| mriedem | sdague: superdan: so...we don't want to use the objects in the actual nova-status command, but i can use them for convenience in the tests right? for creating things in the db | 20:10 |
| superdan | yeah | 20:11 |
| mriedem | great | 20:11 |
| melwitt | mriedem: do we have a schedule for the PTG for friday, like is it a whole day? | 20:11 |
| mriedem | melwitt: working on your travel budget request huh? | 20:11 |
| melwitt | ... no | 20:11 |
| melwitt | yes | 20:11 |
| mriedem | so wed-fri are really like the old midcycle meetup | 20:11 |
| mriedem | so i imagine the 3rd and final day will be a half day | 20:12 |
| superdan | mriedem: not request.. we're approved, just booking now | 20:12 |
| mriedem | sort of like the summit probably | 20:12 |
| mriedem | we have the meetup on friday at the summit but we're usually done by 1 or 2 | 20:12 |
| melwitt | yeah, sorry. I mean I'm picking flights | 20:12 |
| mriedem | if yo'ure leaving friday afternoon you wouldn't probably miss anything if you left late morning or after lunch on friday | 20:12 |
| *** sdague has quit IRC | 20:13 | |
| melwitt | okay, thanks | 20:13 |
| mriedem | i don't have travel approval so i can't really say :) but i assume | 20:13 |
| superdan | for what it's worth, I'm leaving saturday just because leaving late flying west sucks and I figure we don't fully know what the schedule will be like with the new stuff | 20:13 |
| mriedem | yeah if i go i'm going to try and leave saturday morning | 20:13 |
| mriedem | so i can enjoy hijinx on friday night | 20:13 |
| mriedem | s/hijinx/team building/ | 20:13 |
| melwitt | :) | 20:13 |
| mriedem | msp is a delta hub so it's all direct flight for me, and only like 3 hours i think | 20:14 |
| mriedem | so not bad for me | 20:14 |
| * leakypipes driving up... | 20:14 | |
| superdan | it's a pretty bad flight for me, hence not wanting to squash it into friday night | 20:14 |
| *** rfolco has quit IRC | 20:15 | |
| *** rfolco has joined #openstack-nova | 20:15 | |
| *** rfolco has quit IRC | 20:15 | |
| *** sdague has joined #openstack-nova | 20:16 | |
| openstackgerrit | Sarafraj Singh proposed openstack/nova: Objectify requests_and_created_ports https://review.openstack.org/405398 | 20:17 |
| openstackgerrit | Sarafraj Singh proposed openstack/nova: WIP: remove uses of network in _update_ports_for_instance https://review.openstack.org/405399 | 20:19 |
| *** jgriffith is now known as jgriffith_AutoAw | 20:20 | |
| *** diegows has joined #openstack-nova | 20:22 | |
| *** jgriffith_AutoAw is now known as jgriffith | 20:23 | |
| *** stvnoyes has quit IRC | 20:24 | |
| *** stvnoyes has joined #openstack-nova | 20:25 | |
| mriedem | hmmm, so i wrote a test that uses the database fixture with the api db, | 20:26 |
| mriedem | creates a cell mapping in there | 20:26 |
| mriedem | but when the code queries the db, there is nothing there, | 20:26 |
| mriedem | so i'm guessing db_session.get_api_engine() is returning something else | 20:27 |
| mriedem | do i need to mock get_api_engine() and return the engine from the DatabaseFixture? | 20:28 |
| melwitt | that's odd. I've never had a problem with the api db before | 20:28 |
| superdan | yeah should be no problem with api db | 20:28 |
| *** ayoung has joined #openstack-nova | 20:29 | |
| *** tbachman has quit IRC | 20:30 | |
| *** eharney has quit IRC | 20:33 | |
| mriedem | heh, don't worry, | 20:33 |
| mriedem | it was a much stupider problem | 20:33 |
| mriedem | tests generally work better when you don't delete your assertions | 20:34 |
| superdan | *nod* | 20:35 |
| *** ducttape_ has quit IRC | 20:41 | |
| mriedem | blarg | 20:43 |
| openstackgerrit | Diana Clarke proposed openstack/python-novaclient: Microversion 2.40 - Simple tenant usage pagination https://review.openstack.org/407825 | 20:43 |
| mriedem | on a clean install this would fail because there won't be compute nodes | 20:43 |
| mriedem | so no host mappings | 20:43 |
| *** gouthamr has quit IRC | 20:44 | |
| mriedem | _map_cell_and_hosts in simple_cell_setup just exits if there are no compute nodes yet | 20:44 |
| mriedem | which makes me wonder, | 20:44 |
| mriedem | on a fresh install, if you don't have any compute nodes, at what point do you get the host mappings if you only ran simple_cell_setup before starting everything? | 20:44 |
| mriedem | didn't we talk about automatically creating host mappings at some point? | 20:45 |
| melwitt | hmm | 20:46 |
| *** slaweq_ has quit IRC | 20:47 | |
| melwitt | didn't a command having to do with discovering hosts get merged already? looking ... | 20:47 |
| *** gouthamr has joined #openstack-nova | 20:48 | |
| melwitt | https://review.openstack.org/#/c/328491/ | 20:48 |
| *** slaweq_ has joined #openstack-nova | 20:48 | |
| superdan | yeah | 20:50 |
| mriedem | and bauzas had the same concern | 20:52 |
| superdan | what concern | 20:52 |
| superdan | ? | 20:52 |
| mriedem | that you have to run this other thing afterward, or in a cron or sometihng | 20:53 |
| mriedem | so what is devstack doing today? running simple_cell_setup after we've already started n-cpu? | 20:53 |
| superdan | yeah | 20:53 |
| melwitt | shouldn't we just remove the requirement and instruct to use discover hosts? | 20:53 |
| *** fragatina has quit IRC | 20:53 | |
| superdan | what are you going to have do the automatic registration? | 20:54 |
| mriedem | well, the 030 api db migrations is ok i think for new installs | 20:54 |
| mriedem | because on a fresh install, there are no flavors yet, and the 030 migration checks for that | 20:54 |
| mriedem | i was going to only fail nova-status upgrade check if there are no host mappings AND there are no compute nodes | 20:54 |
| mriedem | which, no compute nodes is going to get tricky once we have multiple cells | 20:54 |
| mriedem | well, maybe not, you're in a cell | 20:55 |
| mriedem | so nova.conf has the cell db connection | 20:55 |
| mriedem | for that cell | 20:55 |
| superdan | but this command runs from elsewhere | 20:55 |
| mriedem | we could also take an arg if needed for the cell db you're going to upgrade | 20:55 |
| superdan | so it can iterate cells and count | 20:55 |
| *** Guest68574 has quit IRC | 20:55 | |
| mriedem | via cellmapping i suppose | 20:55 |
| superdan | right | 20:55 |
| mriedem | i don't know what would do the auto registration, we don't want the computes doing it when creating the compute node | 20:56 |
| superdan | we can't have them do it | 20:56 |
| superdan | and apis don't (and shouldn't) have periodica, | 20:56 |
| *** mkoderer has quit IRC | 20:56 | |
| mriedem | periodic in the api? | 20:56 |
| superdan | er, periodics, | 20:56 |
| mriedem | gah | 20:56 |
| *** esberglu has quit IRC | 20:56 | |
| superdan | and you wouldn't want to do it as part of a request | 20:56 |
| mriedem | yeah multiple workers | 20:56 |
| superdan | we will have conductors at the top level, | 20:57 |
| superdan | but that means they'll all have to be checking and randomly distributed to avoid a storm | 20:57 |
| superdan | cron seems like a better choice to me | 20:57 |
| *** esberglu has joined #openstack-nova | 20:57 | |
| superdan | if you want auto-registration | 20:57 |
| *** eharney has joined #openstack-nova | 20:57 | |
| mriedem | what tries to use the host mapping today? | 20:58 |
| mriedem | like, what looks up the host mapping and errors out if it's not htere? | 20:58 |
| superdan | any rpc call | 20:58 |
| superdan | the scheduler needs them | 20:58 |
| mriedem | and those can't create the host mapping at that point? | 20:58 |
| superdan | no, because they don't know what cell they're going to without the host mapping, | 20:59 |
| superdan | and you don't want them scanning all cells presumably :) | 20:59 |
| *** asettle has joined #openstack-nova | 20:59 | |
| *** asettle has quit IRC | 21:00 | |
| superdan | the scheduler has periodics, I guess, so maybe you could put it in there | 21:00 |
| superdan | it's not a great assignment of new scope to the scheduler though | 21:01 |
| *** esberglu has quit IRC | 21:01 | |
| superdan | given the scheduler is single threaded and often quite busy | 21:01 |
| mriedem | ok, so do we want the upgrade check command iterating non-cell0 cell mappings to look for host mappings? | 21:02 |
| superdan | it's the kind of thing that you end up taking a lot of load all the time for something that happens infrequently (adding a physical host) | 21:02 |
| mriedem | right now it's just checking for at least 1 cell mapping, one cell0 mapping, and any host mappings | 21:02 |
| *** tblakes has quit IRC | 21:02 | |
| superdan | I dunno, you could put that in a non-upgrade check, because it's more relevant for "is my system operating at 100%" | 21:02 |
| mriedem | do we instead want that to be, for each non-cell0 mapping, find all host mappings and fail if there are none for a given cell? | 21:03 |
| superdan | because it would be real relevant to say "92% of your computes are mapped, so only those can be utilized" | 21:03 |
| superdan | as a general health thing | 21:03 |
| *** tjones has quit IRC | 21:03 | |
| mriedem | yeah...i guess we're just doing a 'do you have at least one gd host mapping to schedule things to at least 1 gd compute node? because if not, you're f'ed' | 21:03 |
| mriedem | is that a fair assessment? | 21:04 |
| superdan | for upgrade? yeah | 21:04 |
| mriedem | ok | 21:04 |
| *** dtp has quit IRC | 21:04 | |
| mriedem | let me push this up now that i have tests passing | 21:04 |
| superdan | I literally cannot wait | 21:05 |
| openstackgerrit | Matt Riedemann proposed openstack/nova: WIP: nova-status: implement _check_cellsv2 https://review.openstack.org/411525 | 21:05 |
| openstackgerrit | Matt Riedemann proposed openstack/nova: Add nova-status upgrade check command framework https://review.openstack.org/411517 | 21:05 |
| mriedem | wait no more! | 21:05 |
| mriedem | i have a fixme in there for the no host mapping / no compute node thing | 21:05 |
| *** esberglu has joined #openstack-nova | 21:06 | |
| *** liverpooler has quit IRC | 21:07 | |
| *** ihrachys has joined #openstack-nova | 21:07 | |
| mriedem | i'm rather pleased with how the table summary has turned out | 21:08 |
| *** catintheroof has quit IRC | 21:10 | |
| superdan | yeah, I think it's going to rock | 21:10 |
| *** lpetrut has quit IRC | 21:10 | |
| superdan | 2016: the year nova stops punching people in the dick | 21:10 |
| leakypipes | lol | 21:10 |
| openstackgerrit | sean mooney proposed openstack/nova: bump os-vif min version https://review.openstack.org/411991 | 21:11 |
| *** esberglu has quit IRC | 21:11 | |
| *** catintheroof has joined #openstack-nova | 21:11 | |
| mriedem | superdan: we're still going to punch people in the dick | 21:12 |
| superdan | you're right.. dick punching for upgrade readiness checks stops in 2016 | 21:13 |
| superdan | but that's all. | 21:13 |
| *** tlian has quit IRC | 21:13 | |
| superdan | other dick punching will continue | 21:13 |
| diana_clarke | This is ready for review now that microversion 2.39 has landed in the client. Monday is my last day before heading on holiday. Thanks! https://review.openstack.org/#/c/407825/ | 21:13 |
| superdan | specifically that thing about nova suddenly requiring 48 databases to respond to a single api query | 21:13 |
| mriedem | yeah, i mean, e.g. want to know if you can snapshot your instance or live migrate it? sorry, lack of capabilities just punched your dick off. | 21:14 |
| superdan | haha | 21:14 |
| diana_clarke | lol | 21:14 |
| mriedem | superdan: i was going to reply to that ML thread about resolutions and say no new nova dbs | 21:14 |
| superdan | mriedem: lol | 21:14 |
| mriedem | but....separate placement db maybe | 21:14 |
| superdan | mriedem: but then you realized that probably wasn't accurate | 21:14 |
| superdan | yeah | 21:14 |
| superdan | heh | 21:14 |
| mriedem | <=1 new nova dbs | 21:14 |
| mriedem | as a resolution | 21:14 |
| superdan | haha | 21:15 |
| superdan | so sad. | 21:15 |
| *** catintheroof has quit IRC | 21:15 | |
| mriedem | there, sent that one | 21:16 |
| mriedem | i have a feeling the guy that made that post immediately regretted it | 21:16 |
| *** lamt has quit IRC | 21:16 | |
| superdan | I was kinda hoping it was a joke | 21:16 |
| openstackgerrit | Matthew Edmonds proposed openstack/nova: fix os-volume_attachments policy checks https://review.openstack.org/389314 | 21:17 |
| mriedem | jesus you double dipped on that typo | 21:18 |
| mriedem | what an ass | 21:18 |
| *** ducttape_ has joined #openstack-nova | 21:20 | |
| superdan | dude, I'm out for a long time, I gotta pad my stats up a little | 21:20 |
| *** lamt has joined #openstack-nova | 21:21 | |
| *** siva_krish has quit IRC | 21:21 | |
| *** tbachman has joined #openstack-nova | 21:22 | |
| diana_clarke | superdan: I have some easy points for you: https://review.openstack.org/#/c/411444/ & https://review.openstack.org/#/c/411827/ :) | 21:22 |
| superdan | +2s don't count | 21:22 |
| *** ihrachys has quit IRC | 21:24 | |
| *** gouthamr has quit IRC | 21:25 | |
| *** thorst_ has joined #openstack-nova | 21:26 | |
| *** gouthamr has joined #openstack-nova | 21:26 | |
| *** gouthamr has quit IRC | 21:27 | |
| mriedem | diana_clarke: +2 on that first one but left something to ponder | 21:28 |
| mriedem | if you're using the master api-ref but on a liberty cloud or something that will be wrong | 21:28 |
| mriedem | i guess you could qualify, by saying something like '(or osapi_max_limit before 15.0.0)' | 21:29 |
| diana_clarke | mriedem: yeah... a couple of people have mentioned similar concerns, but I've yet to find an example that mentions both old & new. | 21:29 |
| mriedem | probably because no one cared to distinguish them before | 21:30 |
| mriedem | but the person on liberty reading the ocata api-ref might appreciate the distinction | 21:30 |
| mriedem | on the other hand, pointing out config things in the API ref is kind of silly anyway | 21:30 |
| mriedem | as the end user shouldn't know or care about the config of the cloud | 21:31 |
| mriedem | it's just we leak so much garbage out of the API you kind of have to know | 21:31 |
| *** bdeschenes has quit IRC | 21:31 | |
| mriedem | goto: previous capabilities dick punching discussion | 21:32 |
| *** Jeffrey4l has quit IRC | 21:35 | |
| *** thorst_ has quit IRC | 21:35 | |
| *** Jeffrey4l has joined #openstack-nova | 21:36 | |
| *** thorst_ has joined #openstack-nova | 21:36 | |
| *** tjones has joined #openstack-nova | 21:39 | |
| *** tlian has joined #openstack-nova | 21:46 | |
| *** eharney has quit IRC | 21:57 | |
| *** kiran-r has joined #openstack-nova | 21:58 | |
| *** slaweq_ has quit IRC | 22:01 | |
| *** esberglu has joined #openstack-nova | 22:01 | |
| mriedem | superdan: i've even already got the nova-status man page written | 22:02 |
| mriedem | you'll love it | 22:02 |
| leakypipes | mriedem: does it involve dick punching? | 22:02 |
| mriedem | no, but there may be dick fingers | 22:03 |
| *** slaweq_ has joined #openstack-nova | 22:03 | |
| *** unicell has quit IRC | 22:03 | |
| leakypipes | :) | 22:04 |
| *** leakypipes has quit IRC | 22:04 | |
| *** esberglu has quit IRC | 22:05 | |
| *** kiran-r has quit IRC | 22:07 | |
| *** unicell has joined #openstack-nova | 22:08 | |
| *** slaweq_ has quit IRC | 22:09 | |
| *** jgriffith is now known as jgriffith_AutoAw | 22:11 | |
| *** jgriffith_AutoAw is now known as jgriffith | 22:12 | |
| *** cmartin has quit IRC | 22:16 | |
| *** smatzek has joined #openstack-nova | 22:17 | |
| openstackgerrit | Matt Riedemann proposed openstack/nova: WIP: nova-status: implement _check_cellsv2 https://review.openstack.org/411525 | 22:19 |
| openstackgerrit | Matt Riedemann proposed openstack/nova: Add nova-status upgrade check command framework https://review.openstack.org/411517 | 22:19 |
| *** dave-mccowan has quit IRC | 22:25 | |
| *** esberglu has joined #openstack-nova | 22:27 | |
| *** figleaf is now known as edleafe | 22:29 | |
| *** unicell has quit IRC | 22:32 | |
| *** weshay is now known as weshay_lata | 22:32 | |
| mriedem | sdague: nice the subnode is now registered with the placement service in your d-g change http://logs.openstack.org/71/409871/2/check/gate-tempest-dsvm-multinode-full-ubuntu-xenial-nv/97edf24/logs/subnode-2/screen-n-cpu.txt.gz#_2016-12-16_11_57_33_416 | 22:34 |
| *** esberglu has quit IRC | 22:34 | |
| *** toure has quit IRC | 22:34 | |
| *** esberglu has joined #openstack-nova | 22:34 | |
| *** tbachman_ has joined #openstack-nova | 22:34 | |
| *** tbachman has quit IRC | 22:35 | |
| *** tbachman_ is now known as tbachman | 22:35 | |
| *** roeyc has joined #openstack-nova | 22:36 | |
| *** toure has joined #openstack-nova | 22:36 | |
| *** jamesden_ has joined #openstack-nova | 22:37 | |
| *** unicell has joined #openstack-nova | 22:37 | |
| *** jgriffith is now known as jgriffith_AutoAw | 22:38 | |
| *** esberglu has quit IRC | 22:39 | |
| *** mtanin___ has joined #openstack-nova | 22:40 | |
| *** lpetrut has joined #openstack-nova | 22:40 | |
| *** mtanino has quit IRC | 22:42 | |
| *** smatzek has quit IRC | 22:48 | |
| *** fragatin_ has joined #openstack-nova | 22:51 | |
| *** dave-mccowan has joined #openstack-nova | 22:51 | |
| *** fragatin_ has quit IRC | 22:53 | |
| *** dave-mccowan has quit IRC | 22:56 | |
| *** fragatina has joined #openstack-nova | 22:56 | |
| *** burt has quit IRC | 22:57 | |
| *** Swami has quit IRC | 22:57 | |
| *** fragatina has quit IRC | 23:00 | |
| *** hongbin has quit IRC | 23:02 | |
| *** unicell has quit IRC | 23:05 | |
| *** liangy has quit IRC | 23:09 | |
| *** sdague has quit IRC | 23:11 | |
| *** dtp has joined #openstack-nova | 23:13 | |
| openstackgerrit | Matt Riedemann proposed openstack/nova: nova-status: check for cells v2 upgrade readiness https://review.openstack.org/411525 | 23:18 |
| *** marst has quit IRC | 23:21 | |
| *** lpetrut has quit IRC | 23:25 | |
| mriedem | superdan: what was your warning scenario for comparing resource providers to compute nodes again? i can't find that in the chat logs. the etherpad says if we have providers < computes, it's a fail, but i thought that was more of a warning as in you're going to be underutilized | 23:29 |
| mriedem | and 0 RPs meant it's a fail | 23:29 |
| mriedem | is that right? | 23:29 |
| *** jgriffith_AutoAw is now known as jgriffith | 23:31 | |
| mriedem | jgriffith: you've gotta get that auto-away boto stop :) | 23:33 |
| mriedem | *bot | 23:34 |
| jgriffith | mriedem oh, thought I turned that sucker off | 23:34 |
| jgriffith | mriedem sorry | 23:34 |
| *** baoli has quit IRC | 23:34 | |
| mriedem | np | 23:35 |
| openstackgerrit | Matt Riedemann proposed openstack/nova: WIP: nova-status: add basic placement status checking https://review.openstack.org/411883 | 23:35 |
| jgriffith | mriedem hollar if you notice it again, I *think* it's off in my bouncer now | 23:35 |
| mriedem | will do | 23:36 |
| *** lamt has quit IRC | 23:37 | |
| superdan | mriedem: warning would be fewer RPs than compute nodes | 23:40 |
| *** Lehtu has quit IRC | 23:40 | |
| mriedem | ok that's what i thought | 23:40 |
| *** ducttape_ has quit IRC | 23:40 | |
| mriedem | i'll also have to iterate the cell mappings to get the compute node count from each cell | 23:40 |
| mriedem | since placement is global | 23:40 |
| *** Lehtu has joined #openstack-nova | 23:42 | |
| *** armax has quit IRC | 23:42 | |
| openstackgerrit | Matt Riedemann proposed openstack/nova: WIP: nova-status: add basic placement status checking https://review.openstack.org/411883 | 23:43 |
| *** dimtruck is now known as zz_dimtruck | 23:44 | |
| melwitt | mriedem: BobBall responded to the question on the xen xapi<n> config question from that patch https://review.openstack.org/#/c/299092/ | 23:45 |
| *** kiran-r has joined #openstack-nova | 23:54 | |
| *** unicell has joined #openstack-nova | 23:58 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!