| *** salv-orlando has joined #openstack-nova | 00:00 | |
| *** jwnx has quit IRC | 00:03 | |
| *** tbachman has joined #openstack-nova | 00:03 | |
| *** salv-orlando has quit IRC | 00:05 | |
| openstackgerrit | Artom Lifshitz proposed openstack/nova master: Run api sample tests against 2.latest https://review.openstack.org/430352 | 00:10 |
|---|---|---|
| openstackgerrit | Artom Lifshitz proposed openstack/nova master: Add skip_latest_microversion decorator https://review.openstack.org/433585 | 00:10 |
| mriedem | comstud: only when it's about babies | 00:11 |
| *** ckopper has quit IRC | 00:12 | |
| mriedem | mikal: heh that reminds me we said at the ptg we were going to kill the localfs driver | 00:13 |
| *** mtanino_ has quit IRC | 00:13 | |
| * mriedem makes a todo | 00:13 | |
| comstud | mriedem: i see | 00:14 |
| mikal | mriedem: that code is bonkers, I'd love it to die. That said, its still used by xenapi. | 00:14 |
| mriedem | mikal: yeah https://review.openstack.org/#/c/324720/ | 00:14 |
| mriedem | we're leaving the xenapi using stuff in | 00:14 |
| *** mtanino_ has joined #openstack-nova | 00:15 | |
| mikal | If you're going to do that it would be nice to move the localfs stuff that xenapi into their driver and then remove the central thing so its clear its not a thing | 00:16 |
| mriedem | all good comments | 00:17 |
| mriedem | mikal: maybe note that in the review since it's old and we forgot about it at least a few times | 00:17 |
| mikal | Sure, I can add that in the commit message | 00:18 |
| mikal | Or I could just do it I suppose | 00:18 |
| mikal | Is it cheating to put that at the end of the chain? | 00:18 |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: api-ref: fix type for hypervisor_marker https://review.openstack.org/471970 | 00:19 |
| *** mtanino_ has quit IRC | 00:20 | |
| *** r-daneel has quit IRC | 00:26 | |
| openstackgerrit | Artom Lifshitz proposed openstack/nova master: api-ref: Add tags to server(s) detail response https://review.openstack.org/431149 | 00:27 |
| *** masber has quit IRC | 00:32 | |
| *** lyan has joined #openstack-nova | 00:32 | |
| *** masber has joined #openstack-nova | 00:32 | |
| openstackgerrit | Michael Still proposed openstack/nova master: First attempt at adding a privsep user to nova itself. https://review.openstack.org/459166 | 00:34 |
| openstackgerrit | Michael Still proposed openstack/nova master: Move libvirt usages of chown to privsep. https://review.openstack.org/471972 | 00:34 |
| openstackgerrit | melanie witt proposed openstack/nova master: Add FixedIPList.get_count_by_project() https://review.openstack.org/446246 | 00:36 |
| openstackgerrit | melanie witt proposed openstack/nova master: Add FloatingIPList.get_count_by_project() https://review.openstack.org/446247 | 00:36 |
| openstackgerrit | melanie witt proposed openstack/nova master: Add get_count_by_vm_state() to InstanceList object https://review.openstack.org/446244 | 00:36 |
| openstackgerrit | melanie witt proposed openstack/nova master: Add SecurityGroupList.get_counts() https://review.openstack.org/446245 | 00:36 |
| openstackgerrit | melanie witt proposed openstack/nova master: Remove 'reserved' count from used limits https://review.openstack.org/446242 | 00:36 |
| openstackgerrit | melanie witt proposed openstack/nova master: Remove useless quota_usage_refresh from nova-manage https://review.openstack.org/446243 | 00:36 |
| openstackgerrit | melanie witt proposed openstack/nova master: Count server groups to check quota https://review.openstack.org/446240 | 00:36 |
| openstackgerrit | melanie witt proposed openstack/nova master: Count networks to check quota https://review.openstack.org/446241 | 00:36 |
| openstackgerrit | melanie witt proposed openstack/nova master: Count instances to check quota https://review.openstack.org/416521 | 00:36 |
| openstackgerrit | melanie witt proposed openstack/nova master: Add InstanceGroup._remove_members_in_db https://review.openstack.org/457858 | 00:36 |
| openstackgerrit | melanie witt proposed openstack/nova master: Count server group members to check quota https://review.openstack.org/457859 | 00:36 |
| openstackgerrit | melanie witt proposed openstack/nova master: Count security groups to check quota https://review.openstack.org/457860 | 00:36 |
| openstackgerrit | melanie witt proposed openstack/nova master: Make Quotas object favor the API database https://review.openstack.org/410945 | 00:36 |
| openstackgerrit | melanie witt proposed openstack/nova master: Count fixed ips to check quota https://review.openstack.org/457861 | 00:36 |
| openstackgerrit | melanie witt proposed openstack/nova master: Add online migration to move quotas to API database https://review.openstack.org/410946 | 00:36 |
| openstackgerrit | melanie witt proposed openstack/nova master: Count floating ips to check quota https://review.openstack.org/457862 | 00:36 |
| openstackgerrit | melanie witt proposed openstack/nova master: Default to 0 when merging values in limit check https://review.openstack.org/471973 | 00:36 |
| *** lbragstad has joined #openstack-nova | 00:37 | |
| openstackgerrit | Merged openstack/nova master: Remove unused CONF import from placement/auth.py https://review.openstack.org/471716 | 00:38 |
| melwitt | grr why doesn't the infra user update bugs for me anymore | 00:39 |
| melwitt | of course, right after I say that it does it. several minutes after I uploaded the patch | 00:40 |
| *** rfolco has quit IRC | 00:44 | |
| *** armax has quit IRC | 00:48 | |
| *** Shunli has joined #openstack-nova | 00:48 | |
| *** lyan has quit IRC | 00:49 | |
| *** mdnadeem has joined #openstack-nova | 00:52 | |
| *** nic1 has quit IRC | 00:53 | |
| *** gjayavelu has joined #openstack-nova | 00:54 | |
| openstackgerrit | Artom Lifshitz proposed openstack/nova master: api-ref: Add tags to server(s) detail response https://review.openstack.org/431149 | 00:55 |
| *** armax has joined #openstack-nova | 00:56 | |
| openstackgerrit | Shunli Zhou proposed openstack/nova master: Transform servergroup.create notification https://review.openstack.org/454023 | 00:57 |
| openstackgerrit | Artom Lifshitz proposed openstack/nova master: Test InstanceNotFound handling in 'nova usage' https://review.openstack.org/468514 | 00:58 |
| * artom got flooded with emails from Jenkins informing him of apparent merge conflicts | 00:58 | |
| * artom is going through his patches to rebase and fix those | 00:58 | |
| artom | Except they don't actually exist | 00:58 |
| artom | Everything rebases cleanly | 00:59 |
| *** armax has quit IRC | 00:59 | |
| *** salv-orlando has joined #openstack-nova | 01:01 | |
| *** liujiong has joined #openstack-nova | 01:04 | |
| *** salv-orlando has quit IRC | 01:06 | |
| *** jamesdenton has joined #openstack-nova | 01:08 | |
| *** cNilesh has joined #openstack-nova | 01:08 | |
| *** rfolco has joined #openstack-nova | 01:08 | |
| *** gjayavelu has quit IRC | 01:13 | |
| *** smatzek has joined #openstack-nova | 01:14 | |
| *** thorst_afk has joined #openstack-nova | 01:14 | |
| mriedem | artom: yeah some known issue from earlier today | 01:15 |
| *** sree has joined #openstack-nova | 01:18 | |
| *** litao__ has joined #openstack-nova | 01:19 | |
| *** thorst_afk has quit IRC | 01:19 | |
| *** mriedem has quit IRC | 01:22 | |
| *** sree has quit IRC | 01:22 | |
| *** rfolco has quit IRC | 01:25 | |
| *** jamesdenton has quit IRC | 01:25 | |
| *** crushil has joined #openstack-nova | 01:31 | |
| *** smatzek has quit IRC | 01:33 | |
| *** winston-d_ has joined #openstack-nova | 01:35 | |
| *** Apoorva has joined #openstack-nova | 01:36 | |
| *** wlfightup has joined #openstack-nova | 01:36 | |
| *** Apoorva_ has quit IRC | 01:40 | |
| *** Apoorva has quit IRC | 01:40 | |
| *** yamahata has quit IRC | 01:40 | |
| openstackgerrit | Zhenyu Zheng proposed openstack/nova master: Add tags fields for server list detail/show api-ref https://review.openstack.org/465842 | 01:42 |
| *** lbragstad has quit IRC | 01:42 | |
| *** phuongnh has joined #openstack-nova | 01:43 | |
| *** thorst_afk has joined #openstack-nova | 01:47 | |
| *** thorst_afk has quit IRC | 01:47 | |
| *** zhurong has joined #openstack-nova | 01:49 | |
| *** jichen has joined #openstack-nova | 01:58 | |
| *** xinliang has quit IRC | 02:00 | |
| *** armax has joined #openstack-nova | 02:02 | |
| *** jwnx has joined #openstack-nova | 02:03 | |
| *** jamielennox is now known as jamielennox|away | 02:04 | |
| *** jaypipes has joined #openstack-nova | 02:09 | |
| *** gomarivera has joined #openstack-nova | 02:09 | |
| openstackgerrit | Huan Xie proposed openstack/nova master: XenAPI: Remove bittorrent.py which is already deprecated https://review.openstack.org/468273 | 02:10 |
| *** bkopilov has quit IRC | 02:12 | |
| *** coreywright has quit IRC | 02:12 | |
| *** thorst_afk has joined #openstack-nova | 02:12 | |
| *** thorst_afk has quit IRC | 02:12 | |
| *** xinliang has joined #openstack-nova | 02:12 | |
| *** yamamoto_ has quit IRC | 02:14 | |
| *** jamielennox|away is now known as jamielennox | 02:15 | |
| *** yamamoto_ has joined #openstack-nova | 02:17 | |
| openstackgerrit | Zhenyu Zheng proposed openstack/nova master: Support tag instances when boot(3/4) https://review.openstack.org/459593 | 02:17 |
| *** lei-zh has joined #openstack-nova | 02:17 | |
| openstackgerrit | falseuser proposed openstack/nova master: Delete the instance files after revert a resize task https://review.openstack.org/469814 | 02:25 |
| *** gomarivera has quit IRC | 02:27 | |
| *** gyee has quit IRC | 02:27 | |
| *** jaypipes has quit IRC | 02:27 | |
| *** coreywright has joined #openstack-nova | 02:30 | |
| *** hongbin has quit IRC | 02:36 | |
| *** Sukhdev has joined #openstack-nova | 02:36 | |
| *** gomarivera has joined #openstack-nova | 02:44 | |
| *** liusheng has quit IRC | 02:46 | |
| *** liusheng has joined #openstack-nova | 02:46 | |
| *** crushil_ has joined #openstack-nova | 02:47 | |
| *** liuqing has quit IRC | 02:47 | |
| *** liuqing has joined #openstack-nova | 02:47 | |
| *** slaweq has quit IRC | 02:53 | |
| *** ckopper has joined #openstack-nova | 02:54 | |
| *** ckopper has quit IRC | 02:55 | |
| *** edmondsw has joined #openstack-nova | 02:55 | |
| *** edmondsw has quit IRC | 02:59 | |
| *** wlfightup has quit IRC | 03:04 | |
| *** yamamoto_ has quit IRC | 03:04 | |
| *** crushil has quit IRC | 03:05 | |
| *** crushil has joined #openstack-nova | 03:06 | |
| *** rfolco has joined #openstack-nova | 03:11 | |
| *** rfolco has quit IRC | 03:12 | |
| *** Shashi has joined #openstack-nova | 03:12 | |
| *** thorst_afk has joined #openstack-nova | 03:13 | |
| *** yamahata has joined #openstack-nova | 03:15 | |
| *** sridharg has joined #openstack-nova | 03:15 | |
| *** dikonoor has joined #openstack-nova | 03:17 | |
| openstackgerrit | Zhenyu Zheng proposed openstack/nova master: Add tags fields for server list detail/show api-ref https://review.openstack.org/465842 | 03:20 |
| *** dimtruck is now known as zz_dimtruck | 03:23 | |
| *** rfolco has joined #openstack-nova | 03:27 | |
| *** rfolco has quit IRC | 03:27 | |
| *** gomarivera has quit IRC | 03:30 | |
| *** gomarivera has joined #openstack-nova | 03:31 | |
| *** gomarivera has quit IRC | 03:31 | |
| *** gomarivera has joined #openstack-nova | 03:32 | |
| *** bkopilov has joined #openstack-nova | 03:32 | |
| *** thorst_afk has quit IRC | 03:32 | |
| *** links has joined #openstack-nova | 03:34 | |
| *** zsli_ has joined #openstack-nova | 03:36 | |
| *** Shunli has quit IRC | 03:39 | |
| *** vks1 has joined #openstack-nova | 03:42 | |
| *** zhurong has quit IRC | 03:45 | |
| *** dikonoor has quit IRC | 03:48 | |
| openstackgerrit | Zhenyu Zheng proposed openstack/nova master: Support tag instances when boot(3/4) https://review.openstack.org/459593 | 03:49 |
| *** tojuvone has quit IRC | 03:52 | |
| *** crushil_ has quit IRC | 03:54 | |
| *** crushil_ has joined #openstack-nova | 03:54 | |
| *** gomarivera has quit IRC | 03:56 | |
| *** gomarivera has joined #openstack-nova | 03:57 | |
| *** tojuvone has joined #openstack-nova | 03:57 | |
| *** yingjun_1 has joined #openstack-nova | 03:58 | |
| *** gomarivera has quit IRC | 04:01 | |
| *** psachin has joined #openstack-nova | 04:02 | |
| *** itlinux has joined #openstack-nova | 04:03 | |
| *** slaweq has joined #openstack-nova | 04:25 | |
| *** dave-mccowan has quit IRC | 04:26 | |
| openstackgerrit | Takashi NATSUME proposed openstack/nova master: Enable cold migration with target host(1/2) https://review.openstack.org/408955 | 04:26 |
| openstackgerrit | Takashi NATSUME proposed openstack/nova master: Enable cold migration with target host(2/2) https://review.openstack.org/408964 | 04:27 |
| openstackgerrit | Takashi NATSUME proposed openstack/nova master: Use PCIAddressField in oslo.versionedobjects https://review.openstack.org/466201 | 04:28 |
| openstackgerrit | Takashi NATSUME proposed openstack/nova master: api-ref: Add parameters in cold migrate action https://review.openstack.org/410042 | 04:28 |
| openstackgerrit | Takashi NATSUME proposed openstack/nova master: Add functional tests for cold migration to same host https://review.openstack.org/414926 | 04:29 |
| *** thorst_afk has joined #openstack-nova | 04:29 | |
| openstackgerrit | Takashi NATSUME proposed openstack/nova master: Avoid forcing translation on logging calls https://review.openstack.org/413876 | 04:29 |
| *** pcaruana has joined #openstack-nova | 04:31 | |
| *** mamandle has joined #openstack-nova | 04:32 | |
| *** zhurong has joined #openstack-nova | 04:33 | |
| *** thorst_afk has quit IRC | 04:37 | |
| *** slaweq has quit IRC | 04:38 | |
| *** salv-orlando has joined #openstack-nova | 04:39 | |
| *** crushil_ has quit IRC | 04:41 | |
| *** yamamoto has joined #openstack-nova | 04:48 | |
| *** sree has joined #openstack-nova | 04:51 | |
| *** elady has quit IRC | 04:54 | |
| *** adisky__ has joined #openstack-nova | 04:55 | |
| *** pcaruana has quit IRC | 04:57 | |
| *** jaosorior_away is now known as jaosorior | 05:05 | |
| *** salv-orlando has quit IRC | 05:05 | |
| *** salv-orlando has joined #openstack-nova | 05:06 | |
| *** ltomasbo|away is now known as ltomasbo | 05:09 | |
| openstackgerrit | Huan Xie proposed openstack/nova master: XenAPI: Create linux bridge in dest host during live migration https://review.openstack.org/451657 | 05:11 |
| *** jamielennox is now known as jamielennox|away | 05:12 | |
| *** prateek has joined #openstack-nova | 05:12 | |
| *** jamielennox|away is now known as jamielennox | 05:19 | |
| *** itlinux has quit IRC | 05:21 | |
| *** PM_ has joined #openstack-nova | 05:25 | |
| *** claudiub has joined #openstack-nova | 05:26 | |
| *** gomarivera has joined #openstack-nova | 05:27 | |
| *** udesale has joined #openstack-nova | 05:28 | |
| *** ratailor has joined #openstack-nova | 05:30 | |
| *** Sukhdev has quit IRC | 05:35 | |
| *** kwathore has joined #openstack-nova | 05:35 | |
| *** ratailor has quit IRC | 05:44 | |
| *** ratailor has joined #openstack-nova | 05:45 | |
| *** trinaths has joined #openstack-nova | 05:46 | |
| *** ayogi has joined #openstack-nova | 05:56 | |
| *** jamielennox is now known as jamielennox|away | 05:56 | |
| *** dikonoor has joined #openstack-nova | 05:58 | |
| *** jamielennox|away is now known as jamielennox | 06:03 | |
| *** armax has quit IRC | 06:08 | |
| *** armax has joined #openstack-nova | 06:08 | |
| *** armax has quit IRC | 06:09 | |
| *** diga has joined #openstack-nova | 06:09 | |
| *** Oku_OS-away is now known as Oku_OS | 06:10 | |
| *** rcernin has joined #openstack-nova | 06:15 | |
| *** oanson has quit IRC | 06:20 | |
| openstackgerrit | Béla Vancsics proposed openstack/nova master: Use more specific asserts in tests https://review.openstack.org/466609 | 06:21 |
| *** oanson has joined #openstack-nova | 06:21 | |
| openstackgerrit | jichenjc proposed openstack/nova-specs master: add z/VM Driver https://review.openstack.org/464915 | 06:22 |
| *** thorst_afk has joined #openstack-nova | 06:29 | |
| *** zhurong has quit IRC | 06:32 | |
| *** thorst_afk has quit IRC | 06:34 | |
| *** PM_ has quit IRC | 06:34 | |
| *** gomarivera has quit IRC | 06:41 | |
| *** gomarivera has joined #openstack-nova | 06:42 | |
| openstackgerrit | Chris Friesen proposed openstack/nova master: Add validation for cpu_realtime_mask https://review.openstack.org/468203 | 06:42 |
| openstackgerrit | Chris Friesen proposed openstack/nova master: Tweak the cpu_realtime_mask handling slightly https://review.openstack.org/461456 | 06:42 |
| openstackgerrit | Béla Vancsics proposed openstack/nova master: Remove reimported https://review.openstack.org/466658 | 06:43 |
| openstackgerrit | Alex Xu proposed openstack/nova-specs master: Fix to return server groups quota in the quota-classes API https://review.openstack.org/468619 | 06:45 |
| *** gomarivera has quit IRC | 06:47 | |
| *** esberglu has joined #openstack-nova | 06:49 | |
| openstackgerrit | Zhenyu Zheng proposed openstack/nova master: Add tags fields for server list detail/show api-ref https://review.openstack.org/465842 | 06:50 |
| *** esberglu has quit IRC | 06:54 | |
| *** pcaruana has joined #openstack-nova | 06:57 | |
| *** jwnx has quit IRC | 06:58 | |
| openstackgerrit | Bhagyashri Shewale proposed openstack/nova master: Don't create instance backup image if rotation is 0 https://review.openstack.org/409644 | 06:58 |
| *** Hosam has quit IRC | 07:03 | |
| *** damien_r has joined #openstack-nova | 07:07 | |
| openstackgerrit | sahid proposed openstack/nova master: libvirt: update the logic to configure volume with scsi controller https://review.openstack.org/459741 | 07:12 |
| openstackgerrit | sahid proposed openstack/nova master: libvirt: update logic to configure device for scsi controller https://review.openstack.org/459740 | 07:12 |
| *** tesseract has joined #openstack-nova | 07:12 | |
| openstackgerrit | Béla Vancsics proposed openstack/nova master: Transform instance.trigger_crash_dump notification https://review.openstack.org/396225 | 07:12 |
| *** mingyu has joined #openstack-nova | 07:13 | |
| *** markus_z has joined #openstack-nova | 07:14 | |
| openstackgerrit | Chris Friesen proposed openstack/python-novaclient master: 2.47: Show flavor info in server details https://review.openstack.org/435141 | 07:15 |
| *** oanson has quit IRC | 07:20 | |
| *** Hosam has joined #openstack-nova | 07:20 | |
| *** diga has quit IRC | 07:20 | |
| *** oanson has joined #openstack-nova | 07:21 | |
| *** mingyu has quit IRC | 07:22 | |
| *** klipka has joined #openstack-nova | 07:29 | |
| openstackgerrit | melanie witt proposed openstack/nova master: Add periodic task to clean expired console tokens https://review.openstack.org/325381 | 07:30 |
| openstackgerrit | melanie witt proposed openstack/nova master: Use ConsoleConnection object to generate authorizations https://review.openstack.org/325414 | 07:30 |
| openstackgerrit | melanie witt proposed openstack/nova master: Add console connection object https://review.openstack.org/320063 | 07:30 |
| openstackgerrit | melanie witt proposed openstack/nova master: Add access_url_base to console_auth_tokens table https://review.openstack.org/334614 | 07:30 |
| openstackgerrit | melanie witt proposed openstack/nova master: Convert websocketproxy to use db for token validation https://review.openstack.org/333990 | 07:30 |
| *** mingyu has joined #openstack-nova | 07:30 | |
| *** thorst_afk has joined #openstack-nova | 07:30 | |
| *** oanson has quit IRC | 07:31 | |
| *** oanson has joined #openstack-nova | 07:31 | |
| *** mlakat has joined #openstack-nova | 07:32 | |
| *** jpena|off is now known as jpena | 07:34 | |
| *** thorst_afk has quit IRC | 07:35 | |
| *** alexchadin has joined #openstack-nova | 07:36 | |
| *** gcb has joined #openstack-nova | 07:39 | |
| *** mingyu has quit IRC | 07:40 | |
| *** jaosorior has quit IRC | 07:42 | |
| *** nicolasbock has joined #openstack-nova | 07:45 | |
| *** zhurong has joined #openstack-nova | 07:47 | |
| *** bkopilov has quit IRC | 07:47 | |
| *** ralonsoh has joined #openstack-nova | 07:48 | |
| *** mnestratov has joined #openstack-nova | 07:49 | |
| *** ralonsoh has quit IRC | 07:49 | |
| openstackgerrit | zhangyanxian proposed openstack/nova-specs master: Sriov-bond https://review.openstack.org/463526 | 07:52 |
| *** jaosorior has joined #openstack-nova | 07:54 | |
| *** ralonsoh has joined #openstack-nova | 07:56 | |
| *** tuanluong has joined #openstack-nova | 07:57 | |
| *** zzzeek has quit IRC | 08:00 | |
| *** zzzeek has joined #openstack-nova | 08:01 | |
| *** yingjun_1 has quit IRC | 08:07 | |
| *** zhurong has quit IRC | 08:13 | |
| *** salv-orlando has quit IRC | 08:13 | |
| *** litao__ has quit IRC | 08:13 | |
| *** salv-orlando has joined #openstack-nova | 08:14 | |
| *** zhurong has joined #openstack-nova | 08:15 | |
| *** qwertyco has joined #openstack-nova | 08:17 | |
| *** salv-orlando has quit IRC | 08:17 | |
| *** lucas-afk is now known as lucasagomes | 08:18 | |
| *** litao__ has joined #openstack-nova | 08:23 | |
| *** links has quit IRC | 08:25 | |
| *** prateek_ has joined #openstack-nova | 08:28 | |
| *** prateek has quit IRC | 08:30 | |
| *** thorst_afk has joined #openstack-nova | 08:31 | |
| *** prateek_ has quit IRC | 08:32 | |
| *** prateek has joined #openstack-nova | 08:33 | |
| *** prateek has quit IRC | 08:33 | |
| *** prateek has joined #openstack-nova | 08:34 | |
| *** prateek has quit IRC | 08:35 | |
| *** prateek has joined #openstack-nova | 08:35 | |
| *** frickler_ is now known as frickler | 08:36 | |
| *** esberglu has joined #openstack-nova | 08:37 | |
| *** links has joined #openstack-nova | 08:38 | |
| *** esberglu has quit IRC | 08:42 | |
| *** fragatina has joined #openstack-nova | 08:44 | |
| *** prateek has quit IRC | 08:44 | |
| *** salv-orlando has joined #openstack-nova | 08:45 | |
| openstackgerrit | Stephen Finucane proposed openstack/nova master: Add PCIWeigher https://review.openstack.org/379524 | 08:45 |
| openstackgerrit | Stephen Finucane proposed openstack/nova master: Prefer non-PCI host nodes for non-PCI instances https://review.openstack.org/379625 | 08:45 |
| openstackgerrit | Pooja Jadhav proposed openstack/nova master: Nova flavor-list swap should be 0 when no value is specified https://review.openstack.org/471715 | 08:46 |
| *** efoley has joined #openstack-nova | 08:47 | |
| *** xiefp has joined #openstack-nova | 08:48 | |
| *** thorst_afk has quit IRC | 08:50 | |
| openstackgerrit | Stephen Finucane proposed openstack/nova master: Remove **kwargs passing in payload __init__ https://review.openstack.org/418489 | 08:50 |
| openstackgerrit | Stephen Finucane proposed openstack/nova master: remove get_nw_info_for_instance from compute.utils https://review.openstack.org/451387 | 08:53 |
| openstackgerrit | Stephen Finucane proposed openstack/nova master: Remove useless API tests https://review.openstack.org/461011 | 08:55 |
| openstackgerrit | Stephen Finucane proposed openstack/nova master: conf: Move availability zones opts to a group https://review.openstack.org/462469 | 08:55 |
| openstackgerrit | Mikhail Feoktistov proposed openstack/nova master: Remove fstype param from ploop init https://review.openstack.org/444970 | 08:56 |
| *** yamamoto has quit IRC | 08:58 | |
| *** roeyc has joined #openstack-nova | 08:58 | |
| openstackgerrit | Stephen Finucane proposed openstack/nova master: Convert additional disassociate tests to mock https://review.openstack.org/471408 | 08:59 |
| *** mvk has joined #openstack-nova | 09:02 | |
| openstackgerrit | Mikhail Feoktistov proposed openstack/nova master: Add is_vz_container function https://review.openstack.org/445947 | 09:03 |
| *** gszasz has joined #openstack-nova | 09:04 | |
| openstackgerrit | Rodolfo Alonso Hernandez proposed openstack/nova master: Read Neutron port 'binding_profile' during boot https://review.openstack.org/449257 | 09:06 |
| openstackgerrit | Balazs Gibizer proposed openstack/nova master: Improve assertJsonEqual error reporting https://review.openstack.org/428199 | 09:09 |
| openstackgerrit | Béla Vancsics proposed openstack/nova master: Transform aggregate.update_metadata notification https://review.openstack.org/460625 | 09:11 |
| *** ftersin has joined #openstack-nova | 09:13 | |
| openstackgerrit | Balazs Gibizer proposed openstack/nova master: Add separate instance.create payload type https://review.openstack.org/463001 | 09:15 |
| openstackgerrit | Balazs Gibizer proposed openstack/nova master: Add keypairs field to InstanceCreatePayload https://review.openstack.org/419730 | 09:17 |
| *** qwertyco has quit IRC | 09:23 | |
| *** prateek has joined #openstack-nova | 09:25 | |
| openstackgerrit | Balazs Gibizer proposed openstack/nova master: Add key_name field to InstancePayload https://review.openstack.org/463002 | 09:27 |
| *** Hosam has quit IRC | 09:29 | |
| Kevin_Zheng | Hi, I'm working on https://review.openstack.org/#/c/465842/ which adds tags field to instance list details/update/show API and related functional tests, and tests for list details/show already exists in https://github.com/openstack/nova/tree/master/nova/tests/functional/api_sample_tests/api_samples/os-server-tags/v2.26 under os-server-tags tests, as it is tests for servers API, I'd think it should be under servers | 09:30 |
| Kevin_Zheng | tests, so, should I add test for PUT also to os-server-tags or move them all to servers? | 09:30 |
| *** Hosam has joined #openstack-nova | 09:30 | |
| *** cNilesh has quit IRC | 09:30 | |
| *** cNilesh has joined #openstack-nova | 09:30 | |
| *** alexchadin has quit IRC | 09:31 | |
| *** alexchadin has joined #openstack-nova | 09:31 | |
| *** zsli_ has quit IRC | 09:32 | |
| *** gongysh has joined #openstack-nova | 09:32 | |
| *** crushil has quit IRC | 09:33 | |
| openstackgerrit | Balazs Gibizer proposed openstack/nova master: Add BDM to InstancePayload https://review.openstack.org/448779 | 09:33 |
| *** crushil has joined #openstack-nova | 09:33 | |
| *** Hosam has quit IRC | 09:34 | |
| *** efoley has quit IRC | 09:34 | |
| openstackgerrit | jichenjc proposed openstack/nova master: propagate OSError to MigrationPreCheckError https://review.openstack.org/469779 | 09:34 |
| *** esberglu has joined #openstack-nova | 09:35 | |
| *** moshele has joined #openstack-nova | 09:35 | |
| *** cNilesh has quit IRC | 09:36 | |
| *** cNilesh has joined #openstack-nova | 09:36 | |
| openstackgerrit | Andrey Volkov proposed openstack/nova master: [WIP] [placement] Add api-ref for allocations https://review.openstack.org/470933 | 09:37 |
| openstackgerrit | Andrey Volkov proposed openstack/nova master: [placement] Add api-ref for aggregates https://review.openstack.org/470932 | 09:37 |
| *** edand has joined #openstack-nova | 09:37 | |
| *** sambetts|afk is now known as sambetts | 09:38 | |
| *** alexchadin has quit IRC | 09:40 | |
| *** esberglu has quit IRC | 09:40 | |
| openstackgerrit | Balazs Gibizer proposed openstack/nova master: Transform aggregate.add_host notification https://review.openstack.org/396210 | 09:43 |
| openstackgerrit | Balazs Gibizer proposed openstack/nova master: Transform aggregate.remove_host notification https://review.openstack.org/396211 | 09:43 |
| *** cNilesh has quit IRC | 09:47 | |
| *** thorst_afk has joined #openstack-nova | 09:47 | |
| *** cNilesh has joined #openstack-nova | 09:47 | |
| openstackgerrit | Merged openstack/nova master: Remove useless API tests https://review.openstack.org/461011 | 09:49 |
| *** rmart04 has joined #openstack-nova | 09:49 | |
| *** mdnadeem has quit IRC | 09:49 | |
| *** thorst_afk has quit IRC | 09:51 | |
| *** diga has joined #openstack-nova | 09:52 | |
| openstackgerrit | Zhenyu Zheng proposed openstack/nova master: Add tags fields for server list detail/show api-ref https://review.openstack.org/465842 | 09:53 |
| *** cdent has joined #openstack-nova | 09:54 | |
| openstackgerrit | zhangyanxian proposed openstack/nova-specs master: Sriov-bond https://review.openstack.org/463526 | 09:54 |
| *** cNilesh has quit IRC | 09:57 | |
| openstackgerrit | Chris Dent proposed openstack/nova master: [placement] Disambiguate resource provider conflict message https://review.openstack.org/468928 | 09:57 |
| *** klipka has quit IRC | 09:57 | |
| openstackgerrit | Chris Dent proposed openstack/nova master: [placement] Update placement devref to modern features https://review.openstack.org/469047 | 09:59 |
| openstackgerrit | Chris Dent proposed openstack/nova master: [placement] Use util.extract_json in allocations handler https://review.openstack.org/471067 | 10:00 |
| *** kaisers has quit IRC | 10:00 | |
| openstackgerrit | Chris Dent proposed openstack/nova master: Optional separate database for placement API https://review.openstack.org/362766 | 10:01 |
| *** jichen has quit IRC | 10:02 | |
| *** lei-zh has quit IRC | 10:03 | |
| *** klipka has joined #openstack-nova | 10:03 | |
| *** liujiong has quit IRC | 10:06 | |
| *** sdague has joined #openstack-nova | 10:11 | |
| openstackgerrit | sahid proposed openstack/nova master: libvirt: update the logic to configure volume with scsi controller https://review.openstack.org/459741 | 10:12 |
| *** kaisers has joined #openstack-nova | 10:16 | |
| *** kwathore_ has joined #openstack-nova | 10:21 | |
| *** kwathore has quit IRC | 10:21 | |
| *** phuongnh has quit IRC | 10:24 | |
| openstackgerrit | Mikhail Feoktistov proposed openstack/nova master: raise exception if create Virtuozzo container with swap disk https://review.openstack.org/397829 | 10:24 |
| *** markvoelker has quit IRC | 10:27 | |
| *** markvoelker has joined #openstack-nova | 10:27 | |
| *** mnestratov|3 has joined #openstack-nova | 10:29 | |
| *** dtantsur|afk is now known as dtantsur | 10:31 | |
| *** med_ has quit IRC | 10:31 | |
| *** cristicalin has joined #openstack-nova | 10:31 | |
| *** mnestratov has quit IRC | 10:32 | |
| *** cristicalin has quit IRC | 10:33 | |
| *** med_ has joined #openstack-nova | 10:35 | |
| *** med_ is now known as Guest72612 | 10:35 | |
| openstackgerrit | Rodolfo Alonso Hernandez proposed openstack/nova master: Add Neutron port capabilities to devspec in request https://review.openstack.org/451777 | 10:38 |
| *** sree has quit IRC | 10:39 | |
| *** Gayi has joined #openstack-nova | 10:39 | |
| Gayi | Hello everyone.. | 10:40 |
| *** Gayi has quit IRC | 10:40 | |
| *** dikonoor has quit IRC | 10:43 | |
| *** salv-orlando has quit IRC | 10:44 | |
| *** salv-orlando has joined #openstack-nova | 10:44 | |
| openstackgerrit | falseuser proposed openstack/nova master: Clear instance root_device_name in rebuild operation https://review.openstack.org/464991 | 10:48 |
| *** smatzek has joined #openstack-nova | 10:53 | |
| *** winston-d_ has quit IRC | 10:55 | |
| openstackgerrit | Ritesh proposed openstack/nova master: rbd: flatten images when unshelving https://review.openstack.org/457886 | 10:55 |
| *** kaisers has quit IRC | 11:06 | |
| *** udesale has quit IRC | 11:06 | |
| *** jpena is now known as jpena|lunch | 11:07 | |
| mikal | mdbooth: You around? Not sure what timezone you're in. | 11:08 |
| *** carthaca_ has joined #openstack-nova | 11:10 | |
| *** mvk has quit IRC | 11:16 | |
| mdbooth | mikal: I'm around. UK. | 11:19 |
| mdbooth | What's up? | 11:19 |
| * cdent will see mdbooth at the polls | 11:20 | |
| mikal | I was confused about some "pty flush" code in the libvirt driver... | 11:20 |
| *** Hosam has joined #openstack-nova | 11:21 | |
| mdbooth | mikal: I'll probably be similarly confused :) | 11:21 |
| mikal | But I think I've got it now. I'm fiddling with file permissions while I sneak up with privsep and I didn't understand why the flush needed a chown first | 11:21 |
| mikal | It turns out it doesn't its the call _after_ that which needs the chown | 11:21 |
| mikal | So yeah, I think I just wasted your time | 11:21 |
| mikal | Sorry | 11:21 |
| mdbooth | cdent: Too late: https://goo.gl/photos/31jQijVTf5wsAuFi6 :) | 11:21 |
| cdent | +many | 11:21 |
| mdbooth | mikal: Hehe, np. | 11:22 |
| *** dikonoor has joined #openstack-nova | 11:23 | |
| *** kaisers has joined #openstack-nova | 11:23 | |
| *** Hosam_ has joined #openstack-nova | 11:25 | |
| *** lucasagomes is now known as lucas-hungry | 11:26 | |
| *** Hosam has quit IRC | 11:28 | |
| openstackgerrit | Merged openstack/nova master: Fix html_last_updated_fmt for Python3 https://review.openstack.org/470676 | 11:28 |
| *** Hosam has joined #openstack-nova | 11:29 | |
| *** Hosam_ has quit IRC | 11:32 | |
| *** trinaths has left #openstack-nova | 11:32 | |
| *** dikonoor has quit IRC | 11:33 | |
| *** mingyu has joined #openstack-nova | 11:35 | |
| *** dikonoor has joined #openstack-nova | 11:36 | |
| *** liuqing has quit IRC | 11:36 | |
| *** liuqing has joined #openstack-nova | 11:36 | |
| *** gongysh has quit IRC | 11:37 | |
| *** CristinaPauna has quit IRC | 11:37 | |
| *** dtantsur is now known as dtantsur|sick | 11:37 | |
| *** edmondsw has joined #openstack-nova | 11:40 | |
| *** mikal has quit IRC | 11:41 | |
| *** gomarivera has joined #openstack-nova | 11:43 | |
| *** mikal has joined #openstack-nova | 11:43 | |
| *** dikonoor has quit IRC | 11:45 | |
| *** mingyu has quit IRC | 11:45 | |
| *** mingyu has joined #openstack-nova | 11:46 | |
| *** thorst_afk has joined #openstack-nova | 11:46 | |
| *** mvk has joined #openstack-nova | 11:47 | |
| *** gomarivera has quit IRC | 11:47 | |
| *** dikonoor has joined #openstack-nova | 11:49 | |
| *** kwathore_ has quit IRC | 11:51 | |
| *** abalutoiu has joined #openstack-nova | 11:52 | |
| *** mingyu has quit IRC | 11:53 | |
| *** litao__ has quit IRC | 11:53 | |
| openstackgerrit | Merged openstack/nova master: Remove **kwargs passing in payload __init__ https://review.openstack.org/418489 | 11:54 |
| *** hwoarang_ is now known as hwoarang | 11:56 | |
| *** tbachman has quit IRC | 11:57 | |
| *** jpena|lunch is now known as jpena | 11:57 | |
| *** mingyu has joined #openstack-nova | 11:58 | |
| *** kwathore has joined #openstack-nova | 11:58 | |
| *** Shashi has quit IRC | 12:05 | |
| *** jwnx has joined #openstack-nova | 12:07 | |
| openstackgerrit | Michael Still proposed openstack/nova master: Move the last_bytes util method to the libvirt driver. https://review.openstack.org/472228 | 12:08 |
| openstackgerrit | Michael Still proposed openstack/nova master: Avoid chowing console logs in libvirt https://review.openstack.org/472229 | 12:08 |
| *** lucas-hungry is now known as lucasagomes | 12:10 | |
| *** mingyu has quit IRC | 12:11 | |
| *** mingyu has joined #openstack-nova | 12:11 | |
| *** vks1 has quit IRC | 12:12 | |
| openstackgerrit | Mikhail Feoktistov proposed openstack/nova master: Add is_vz_container function https://review.openstack.org/445947 | 12:13 |
| *** mriedem has joined #openstack-nova | 12:13 | |
| *** lyan has joined #openstack-nova | 12:14 | |
| mriedem | it's p-2 day | 12:16 |
| *** mingyu has quit IRC | 12:16 | |
| mriedem | any burning regressions or bugs we need to get fixed before we tag it? | 12:16 |
| *** damien_r has left #openstack-nova | 12:18 | |
| *** jamesdenton has joined #openstack-nova | 12:22 | |
| *** tuanluong has quit IRC | 12:23 | |
| bauzas | mriedem: honestly, I'm still looking at all the bugs | 12:23 |
| bauzas | for the moment, none | 12:23 |
| bauzas | but that's a today triage day for me | 12:23 |
| mriedem | lyarwood: i just noticed the git author thing again while rebasing https://review.openstack.org/#/c/436094/ | 12:25 |
| mriedem | i pulled that down with git review -d | 12:25 |
| mriedem | rebased it to master: git rebase -i master | 12:25 |
| *** krtaylor has quit IRC | 12:25 | |
| mriedem | resolved merge conflicts, added the resolved files and did git commit, then git rebase --continue | 12:26 |
| mriedem | and then i was the author | 12:26 |
| *** jwcroppe has joined #openstack-nova | 12:26 | |
| mriedem | i do'nt know what's going on there | 12:27 |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: Cleanup update_instance cell mapping handling https://review.openstack.org/467380 | 12:29 |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: DNM: Test changes with multiple cells https://review.openstack.org/467383 | 12:29 |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: Remove cells v2 transition code from update_instance https://review.openstack.org/467382 | 12:29 |
| kashyap | cfriesen: Have you booted an UEFI image lately? | 12:29 |
| kashyap | (Or if ever) | 12:29 |
| openstackgerrit | Matthew Edmonds proposed openstack/nova master: [WIP] Deactivate powervm compute until ready https://review.openstack.org/471773 | 12:29 |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: Pull out code that builds VIF in _build_network_info_model https://review.openstack.org/466449 | 12:30 |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: WIP: Handle network-changed event for a specific port https://review.openstack.org/465787 | 12:30 |
| *** derekh has joined #openstack-nova | 12:31 | |
| *** catintheroof has joined #openstack-nova | 12:32 | |
| *** karimb has joined #openstack-nova | 12:34 | |
| *** sree has joined #openstack-nova | 12:35 | |
| *** zhurong has quit IRC | 12:36 | |
| sdague | mriedem: https://bugs.launchpad.net/nova/+bug/1687581 seems like something we should probably fix sooner rather than later | 12:36 |
| openstack | Launchpad bug 1687581 in OpenStack Compute (nova) "Newton > Ocata upgrade. secret_uuid empty" [High,New] | 12:36 |
| sdague | people upgrading newton -> ocata ceph apparently can't start newton guests | 12:37 |
| *** tbachman has joined #openstack-nova | 12:37 | |
| *** liverpooler has joined #openstack-nova | 12:37 | |
| mriedem | i know what caused that | 12:37 |
| mriedem | https://review.openstack.org/#/c/389399/ | 12:38 |
| mriedem | i think | 12:38 |
| *** krtaylor has joined #openstack-nova | 12:38 | |
| *** ratailor has quit IRC | 12:40 | |
| mriedem | so it looks like in newton they had rbd auth enabled on the cinder side, but didn't have the secret uuid coming from cinder, it was coming from nova.conf | 12:40 |
| *** rfolco has joined #openstack-nova | 12:41 | |
| sdague | right | 12:42 |
| mriedem | i can work up a patch | 12:42 |
| sdague | mriedem: there is a patch in the bug | 12:42 |
| sdague | they just didn't gerrit it | 12:42 |
| sdague | http://paste.ubuntu.com/24497744/ | 12:43 |
| mriedem | i know, but that needs tests and comments, etc | 12:45 |
| mriedem | i can co-author them | 12:45 |
| *** jroll has left #openstack-nova | 12:47 | |
| *** ayogi has quit IRC | 12:48 | |
| bauzas | claudiub: see https://bugs.launchpad.net/nova/+bug/1696685 | 12:48 |
| openstack | Launchpad bug 1696001 in OpenStack Compute (nova) ocata "duplicate for #1696685 nova-manage cell_v2 map_cell0 can create invalid connection URL" [Undecided,In progress] - Assigned to Sylvain Bauza (sylvain-bauza) | 12:48 |
| bauzas | claudiub: you can review the stable/ocata change | 12:49 |
| openstackgerrit | Lee Yarwood proposed openstack/nova master: libvirt: Reconnect volumes and encryptors during a hard reboot https://review.openstack.org/400384 | 12:50 |
| bauzas | mriedem: we have a super weird issue for stable/newton with db migrations when you have different TZs https://bugs.launchpad.net/nova/+bug/1696417 | 12:50 |
| openstack | Launchpad bug 1696417 in OpenStack Compute (nova) "nova-manage db online_data_migrations can fail when upgrading to newton under certain conditions" [Undecided,Incomplete] | 12:50 |
| bauzas | mriedem: for sure we could just say "meh" | 12:50 |
| lyarwood | mriedem: huh, are you using an alias for git commit or something? | 12:50 |
| *** Hosam has quit IRC | 12:51 | |
| *** Hosam has joined #openstack-nova | 12:52 | |
| sdague | mriedem: git author or git committer? | 12:52 |
| lyarwood | sdague: both, https://review.openstack.org/#/c/441204/ is an example. | 12:54 |
| mriedem | lyarwood: nope | 12:55 |
| mriedem | just git commit | 12:55 |
| mriedem | sdague: author | 12:55 |
| sdague | mriedem: author shouldn't change unless you --reset-author | 12:55 |
| *** yamamoto has joined #openstack-nova | 12:55 | |
| mriedem | which i didn't do | 12:55 |
| mriedem | just git review | 12:55 |
| mriedem | git rebase | 12:55 |
| mriedem | git commit | 12:55 |
| lyarwood | yeah that's weird | 12:55 |
| mriedem | sdague: somehow stole author from lyarwood on https://review.openstack.org/#/c/441204/ | 12:55 |
| mriedem | and noticed the same this morning when rebasing a change of dan's in devstack | 12:55 |
| mriedem | but caught it this time | 12:55 |
| sdague | mriedem: and nothing in the ~/.gitconfig? | 12:56 |
| *** cleong has joined #openstack-nova | 12:56 | |
| mriedem | user@ubuntu:~/git/nova$ cat ~/.gitconfig | 12:56 |
| mriedem | [user] | 12:56 |
| mriedem | name = Matt Riedemann | 12:56 |
| mriedem | email = mriedem.os@gmail.com | 12:56 |
| mriedem | [gitreview] | 12:56 |
| mriedem | username = mriedem | 12:56 |
| mriedem | [core] | 12:56 |
| mriedem | excludesfile = /home/user/.gitignore | 12:56 |
| *** crushil has quit IRC | 12:56 | |
| sdague | and this is all at the cli and not using some other tool? | 12:56 |
| bauzas | mriedem: possibly merging https://bugs.launchpad.net/nova/+bug/1696605 before pike-2 too | 12:56 |
| openstack | Launchpad bug 1696605 in OpenStack Compute (nova) "TypeError: unorderable types: int() < NoneType() possible during quota limit check" [High,In progress] - Assigned to melanie witt (melwitt) | 12:56 |
| *** Hosam has quit IRC | 12:56 | |
| *** mriedem1 has joined #openstack-nova | 12:59 | |
| *** udesale has joined #openstack-nova | 12:59 | |
| *** mriedem has quit IRC | 13:00 | |
| *** mriedem1 is now known as mriedem | 13:02 | |
| *** baoli has joined #openstack-nova | 13:02 | |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: Log a warning if there is only one cell when listing instances https://review.openstack.org/440652 | 13:03 |
| *** cristicalin has joined #openstack-nova | 13:03 | |
| *** gongysh has joined #openstack-nova | 13:03 | |
| *** gongysh has quit IRC | 13:03 | |
| *** CristinaPauna has joined #openstack-nova | 13:03 | |
| *** mdrabe has joined #openstack-nova | 13:04 | |
| *** esberglu has joined #openstack-nova | 13:06 | |
| bauzas | does the VM really supports the order of NICs that you provided when booting ? | 13:07 |
| *** crushil has joined #openstack-nova | 13:07 | |
| bauzas | looks to me hard to respect | 13:07 |
| *** pchavva has joined #openstack-nova | 13:07 | |
| bauzas | https://bugs.launchpad.net/nova/+bug/1696664 | 13:07 |
| openstack | Launchpad bug 1696664 in OpenStack Compute (nova) "Order of the interfaces is not honored when using virt and SR-IOV interfaces" [Undecided,New] | 13:07 |
| bauzas | sfinucan: any idea of the above ? ^ | 13:07 |
| * sfinucan looking | 13:07 | |
| bauzas | I mean, I suspect that as a non-issue | 13:07 |
| bauzas | but it's not really related to SR-IOV | 13:08 |
| sfinucan | bauzas: I concur - we don't say anywhere that the order is important | 13:08 |
| sfinucan | /respected | 13:09 |
| *** jamesdenton has quit IRC | 13:10 | |
| bauzas | well, we don't say anything in https://developer.openstack.org/api-ref/compute/?expanded=create-server-detail#create-server | 13:10 |
| bauzas | and when looking at code, we're just getting the dict | 13:10 |
| claudiub | bauzas: ah, cool, ty. It's in merge conflict though. :) https://review.openstack.org/#/c/471790/ | 13:10 |
| *** yamamoto has quit IRC | 13:10 | |
| bauzas | sfinucan: so I'm pretty sure we're not exactly verifying the order | 13:10 |
| sfinucan | bauzas: Agreed. You can close it as invalid, IMO | 13:10 |
| bauzas | sfinucan: will do | 13:11 |
| bauzas | if the user cares about the order, that would be a feature to me | 13:11 |
| bauzas | and good luck with that. | 13:11 |
| bauzas | claudiub: oh good point, will rebase it | 13:11 |
| *** dave-mccowan has joined #openstack-nova | 13:12 | |
| *** lei-zh has joined #openstack-nova | 13:12 | |
| *** yamamoto has joined #openstack-nova | 13:14 | |
| *** crushil has quit IRC | 13:15 | |
| mriedem | bauzas: sfinucan: there is some network sorting in the neutronv2.api code | 13:15 |
| *** links has quit IRC | 13:15 | |
| mriedem | https://github.com/openstack/nova/blob/master/nova/network/neutronv2/api.py#L2522 | 13:15 |
| *** lucasxu has joined #openstack-nova | 13:16 | |
| mriedem | that's for networks, but when given ports we get the networks for those ports too so maybe that's what they are thinking | 13:16 |
| bauzas | humpg | 13:19 |
| bauzas | mriedem: so there are two possible outcomes | 13:20 |
| bauzas | mriedem: either we say "sorry, but we don't sort NICs when you have both network and port requests" | 13:20 |
| bauzas | mriedem: or we say it's a valid bug that requires resolution | 13:21 |
| mriedem | i don't want to say anything | 13:22 |
| bauzas | mriedem: so I'll keep the bug open | 13:24 |
| bauzas | :) | 13:24 |
| bauzas | anyone can just look at it | 13:24 |
| *** sbezverk has joined #openstack-nova | 13:25 | |
| *** smatzek has quit IRC | 13:29 | |
| *** diga has quit IRC | 13:29 | |
| *** igordcard has quit IRC | 13:31 | |
| mriedem | i replied with some details too and code links | 13:31 |
| mriedem | looks like we do only preserve the order of requested networks, not the networks associated with a requested port | 13:31 |
| mriedem | adding the port.network_id to the list of ordered_networks would be easy, but would probably break something else, | 13:31 |
| mriedem | like i bet that would end up having it create another port on that other port's network or something | 13:32 |
| mriedem | it just gets very complicated | 13:32 |
| mriedem | we could build a 3rd list of networks :) | 13:32 |
| *** igordcard has joined #openstack-nova | 13:34 | |
| *** awaugama has joined #openstack-nova | 13:34 | |
| *** sbezverk has quit IRC | 13:34 | |
| *** gomarivera has joined #openstack-nova | 13:36 | |
| *** gomarivera has quit IRC | 13:36 | |
| *** gomarivera has joined #openstack-nova | 13:37 | |
| *** udesale has quit IRC | 13:38 | |
| *** lucasxu has quit IRC | 13:38 | |
| *** tbachman has quit IRC | 13:38 | |
| *** prateek has quit IRC | 13:40 | |
| *** lucasxu has joined #openstack-nova | 13:40 | |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: libvirt: handle missing rbd_secret_uuid from old connection info https://review.openstack.org/472266 | 13:41 |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: libvirt: handle missing rbd_secret_uuid from old connection info https://review.openstack.org/472266 | 13:42 |
| mriedem | sdague: here is that fix for the rbd secret uuid ^ | 13:42 |
| mriedem | sdague: which reminds me of something i've thought about for awhile, | 13:42 |
| mriedem | which is adding an admin api to force a bdm.connection_info refresh | 13:43 |
| openstackgerrit | Alex Szarka proposed openstack/nova master: Add method for verify multiple versioned notifications https://review.openstack.org/465526 | 13:43 |
| mriedem | since if you changed things somewhere on the cinder side, or in your underlying storage, i think you have to detach the volume and re-attach it, or reboot the instance to refresh the connection info | 13:43 |
| mriedem | this has come up when people recycle secrets in ceph | 13:43 |
| mriedem | and the only way to update the existing bdm connection info is via straight db hackery | 13:44 |
| mriedem | so we could do something as simple as an rpc cast to the compute, call os-initialize_connection in cinder to get the connection_info and then update the bdm record | 13:45 |
| mriedem | melwitt: haven't you and i talked about something like this before? ^ or at least a periodic task | 13:45 |
| *** Jack_Iv has joined #openstack-nova | 13:46 | |
| *** jwnx has quit IRC | 13:46 | |
| mriedem | the problem is i bet initialize_connection in cinder isn't the same (idempotent) for all volume backends | 13:46 |
| *** roeyc has quit IRC | 13:47 | |
| *** tbachman has joined #openstack-nova | 13:48 | |
| *** burt has joined #openstack-nova | 13:48 | |
| sdague | mriedem: could that come from cinder in the admin event interface? | 13:49 |
| *** edand has quit IRC | 13:49 | |
| sdague | it would be best if cinder was the one that said to do that when needed | 13:49 |
| *** smatzek has joined #openstack-nova | 13:50 | |
| *** lucasxu has quit IRC | 13:51 | |
| mriedem | it could | 13:52 |
| mriedem | but, | 13:52 |
| mriedem | send what? just telling us to do it? because that api doesn't take something like a big json blob | 13:52 |
| mriedem | we could get a "refresh-connection-info" event from cinder though yeah, like we do for network-changed in neutron | 13:53 |
| *** igordcard has quit IRC | 13:54 | |
| *** gbarros has joined #openstack-nova | 13:54 | |
| *** igordcard has joined #openstack-nova | 13:55 | |
| sdague | yeh, that's kind of what I was thinking | 13:57 |
| sdague | I guess it should be instance oriented? | 13:57 |
| *** igordcard has quit IRC | 13:57 | |
| mriedem | yes it would be | 13:58 |
| mriedem | just sent to the ML | 13:58 |
| mriedem | might be a duplicate idea i've already brought up before | 13:58 |
| mriedem | anyway, bbiab | 13:58 |
| *** igordcard has joined #openstack-nova | 13:58 | |
| *** READ10 has joined #openstack-nova | 13:59 | |
| *** jamesdenton has joined #openstack-nova | 13:59 | |
| *** tidwellr has joined #openstack-nova | 14:00 | |
| *** krtaylor has quit IRC | 14:01 | |
| *** igordcard has quit IRC | 14:01 | |
| *** igordcard has joined #openstack-nova | 14:02 | |
| *** lucasxu has joined #openstack-nova | 14:02 | |
| *** ZZelle has quit IRC | 14:03 | |
| *** zz_dimtruck is now known as dimtruck | 14:03 | |
| *** ZZelle has joined #openstack-nova | 14:03 | |
| *** crushil has joined #openstack-nova | 14:06 | |
| *** mtanino has joined #openstack-nova | 14:06 | |
| *** sbezverk has joined #openstack-nova | 14:07 | |
| *** dikonoor has quit IRC | 14:09 | |
| *** ZZelle has quit IRC | 14:12 | |
| *** jamesdenton has quit IRC | 14:12 | |
| *** ZZelle has joined #openstack-nova | 14:12 | |
| openstackgerrit | Stephen Finucane proposed openstack/nova master: objects: remove cpuset_reserved from __init__ of InstanceNUMATopology https://review.openstack.org/466030 | 14:13 |
| openstackgerrit | Stephen Finucane proposed openstack/nova master: objects: remove related pinning from __init__ of InstanceNUMATopology https://review.openstack.org/408242 | 14:13 |
| openstackgerrit | Stephen Finucane proposed openstack/nova master: objects: remove pagesize from __init__ of InstanceNUMATopology https://review.openstack.org/408241 | 14:13 |
| *** lbragstad has joined #openstack-nova | 14:14 | |
| *** ZZelle has quit IRC | 14:14 | |
| *** eharney has joined #openstack-nova | 14:16 | |
| *** r-daneel has joined #openstack-nova | 14:17 | |
| *** felipemonteiro_ has joined #openstack-nova | 14:18 | |
| sfinucan | dansmith: I think I finally figured out the remaining kinks with the above ^^^ patches, if you want to take a look at some point | 14:18 |
| * sfinucan goes to implement an '__eq__' method for 'nova.objects.base.NovaObject' | 14:19 | |
| openstackgerrit | Hironori Shiina proposed openstack/nova master: Ironic: Check a last error of power action https://review.openstack.org/470754 | 14:20 |
| bauzas | sfinucan: I think the consensus is not to have __eq__ | 14:20 |
| *** Yangyapeng has joined #openstack-nova | 14:20 | |
| dansmith | sfinucan: nak on the __eq__ yeah :) | 14:20 |
| sfinucan | bauzas: Oh? Any particular reason? | 14:20 |
| bauzas | sfinucan: when you want to verify o.vo objects, you need to verify the fields | 14:20 |
| *** burt has quit IRC | 14:21 | |
| sfinucan | bauzas: Yeah - we do that in nova/objects/pci_device.py and nova/objects/numa.py | 14:21 |
| *** burt has joined #openstack-nova | 14:21 | |
| sfinucan | ...and we could also do with it in nova/objects/instance_numa_topology.py | 14:21 |
| bauzas | sfinucan: there are multiple reasons, one being that objects can be semantically identical without being the same, like if they're passed over RPC | 14:22 |
| *** tbachman has quit IRC | 14:22 | |
| sfinucan | bauzas: You mean there'd be differences other than timestamps and the likes? | 14:24 |
| bauzas | well, take an object, pass it as an argument to a remote method, you won't get the same object at the end | 14:25 |
| *** jamesdenton has joined #openstack-nova | 14:25 | |
| *** sbezverk has quit IRC | 14:26 | |
| *** awaugama_ has joined #openstack-nova | 14:27 | |
| *** lei-zh has quit IRC | 14:28 | |
| *** lei-zh has joined #openstack-nova | 14:28 | |
| *** hieulq_ has joined #openstack-nova | 14:29 | |
| *** awaugama has quit IRC | 14:29 | |
| sfinucan | hmm, well I'm going to try something to see if it makes sense | 14:29 |
| sfinucan | it might not, but then I'll know :) | 14:30 |
| *** felipemonteiro__ has joined #openstack-nova | 14:30 | |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: WIP/POC Implement new attach Cinder flow https://review.openstack.org/330285 | 14:31 |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: Skip microversion discovery check for update/delete volume attachments https://review.openstack.org/469668 | 14:31 |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: Use 3.27 microversion when creating new style volume attachments https://review.openstack.org/469609 | 14:31 |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: Use microversions for new style volume attachments https://review.openstack.org/469579 | 14:31 |
| mriedem | sdague: could you add the bottom 3 changes here to your review queue? ^ | 14:31 |
| mriedem | the new cinder stuff depends on that | 14:31 |
| *** moshele has quit IRC | 14:32 | |
| sdague | yeh, can do | 14:33 |
| sdague | it will be a minute, in the middle of unrelated stuff | 14:33 |
| *** felipemonteiro_ has quit IRC | 14:34 | |
| mriedem | sure, thanks | 14:34 |
| *** jwnx has joined #openstack-nova | 14:34 | |
| mriedem | dansmith: looks like your devstack fleet week patch needs to depend on the check min service version in all cells patch | 14:34 |
| mriedem | http://logs.openstack.org/94/436094/32/check/gate-tempest-dsvm-neutron-full-ubuntu-xenial/0d3a7e2/console.html#_2017-06-08_13_33_37_461957 | 14:34 |
| mriedem | i think... | 14:34 |
| *** gbarros has quit IRC | 14:34 | |
| *** coreywright has quit IRC | 14:35 | |
| *** jamesdenton has quit IRC | 14:35 | |
| bauzas | sfinucan: the consensus is basically about providing a general __eq__ rule | 14:38 |
| bauzas | sfinucan: semantically, the equivalence really depends on the object type | 14:38 |
| bauzas | sfinucan: like for example, you could assume some fields are irrelevant for that | 14:38 |
| sfinucan | Yeah, that sounds fair. It does seem pretty contextual | 14:38 |
| bauzas | the same goes with __hash__ | 14:39 |
| bauzas | since objects can be very different, we try hard to not define __hash__ on a global basis | 14:39 |
| bauzas | sfinucan: that said, you know we have an obj_equal_prims() right? | 14:40 |
| sfinucan | bauzas: I've just spotted that, alright | 14:41 |
| bauzas | cool | 14:41 |
| bauzas | that's the only thing we can really do on a general basis | 14:41 |
| bauzas | if primitives are equal, fine | 14:42 |
| bauzas | but standing that obj_1 == obj_2 is a total different deal | 14:42 |
| bauzas | same goes for obj1 in an objectlist | 14:42 |
| *** sree has quit IRC | 14:43 | |
| *** claudiub has quit IRC | 14:43 | |
| *** salv-orl_ has joined #openstack-nova | 14:44 | |
| dansmith | mriedem: aye, not sure why I wasn't hitting that before | 14:46 |
| *** salv-orlando has quit IRC | 14:47 | |
| mriedem | dansmith: i think the test was recently-ish merged | 14:47 |
| mriedem | well, | 14:47 |
| dansmith | ah | 14:47 |
| *** armax has joined #openstack-nova | 14:47 | |
| mriedem | and it required the ssh validation stuff being turned on, which happened globally about 2 weeks ago | 14:47 |
| dansmith | mriedem: thanks for fixing the bottom patch, I just realized I needed to get back to it | 14:47 |
| dansmith | is the merge conflict because of all the infra drama yesterday? | 14:48 |
| mriedem | yes, i've rechecked | 14:48 |
| mriedem | well, i assume it is, i haven't tried rebasing | 14:48 |
| dansmith | cool | 14:48 |
| dansmith | it's running it so I think it must've been | 14:48 |
| dansmith | otherwise it'd kick right out | 14:48 |
| *** coreywright has joined #openstack-nova | 14:48 | |
| cfriesen | kashyap: I haven't, one of our people might have and I'm pinging for confirmation. We're just going GA with Newton though, so we're not exactly bleeding edge. | 14:49 |
| *** hongbin has joined #openstack-nova | 14:50 | |
| johnthetubaguy | mriedem: was just taking a peak at the cinder patches (https://review.openstack.org/#/c/469668/3) | 14:52 |
| johnthetubaguy | mriedem: any reason not to cache the availablity of a microversion, once we find its available? | 14:53 |
| *** abalutoiu has quit IRC | 14:53 | |
| *** jaypipes has joined #openstack-nova | 14:54 | |
| cfriesen | johnthetubaguy: couldn't they reboot the cinder service at any time? | 14:54 |
| cfriesen | johnthetubaguy: or are you expecting monotonic increases | 14:55 |
| johnthetubaguy | cfriesen: yeah, but we don't really support downgrades | 14:55 |
| johnthetubaguy | cfriesen: your right in general though, just wondering if we can be simpler | 14:55 |
| johnthetubaguy | actually, yeah, downgrading past this point just isn't going to work, as far as I understand it | 14:56 |
| *** psachin has quit IRC | 14:57 | |
| openstackgerrit | Stephen Finucane proposed openstack/nova master: objects: Add NovaComparableObject mixin https://review.openstack.org/472285 | 14:58 |
| sfinucan | bauzas, dansmith: What about that? ^ | 14:59 |
| mriedem | johnthetubaguy: replied inline | 14:59 |
| sfinucan | johnthetubaguy: Oh hai | 14:59 |
| mriedem | johnthetubaguy: in general because caching of versions gets complicated | 14:59 |
| mriedem | especially when nova as the client might need to request multiple different versions based on which operation we're doing | 14:59 |
| *** sree has joined #openstack-nova | 14:59 | |
| mriedem | like if we need 3.27 today for something but 3.33 tomorrow for something else | 14:59 |
| * sfinucan feels like he hasn't seen johnthetubaguy in aaaages | 14:59 | |
| dansmith | sfinucan: why not use equal_prims for that? | 14:59 |
| mriedem | johnthetubaguy: for now i'm just keeping it simple | 14:59 |
| dansmith | sfinucan: more "is this equal" helpers seems bad to me | 15:00 |
| johnthetubaguy | mriedem: I was thinking caching was simpler, but you are right | 15:00 |
| *** jmlowe has quit IRC | 15:00 | |
| mriedem | it's simple in the simple case :) | 15:00 |
| johnthetubaguy | mriedem: what you have is clearly correct | 15:00 |
| sfinucan | dansmith: I was hoping you'd tell me :) I'm not confident enough in that code to say they're interchangeable | 15:00 |
| * johnthetubaguy waves at sfinucan as he comes back from the dead | 15:00 | |
| sfinucan | If they are, I can take that route | 15:00 |
| mriedem | johnthetubaguy: when talking about this last week, mordred was also saying that at some point i think ksa can provide this info, so we don't need the extra call to cinder | 15:01 |
| mriedem | to get the version doc | 15:01 |
| mriedem | ksa would just provide it | 15:01 |
| sfinucan | (by "that code", I mean "all things object'y") | 15:01 |
| * edleafe runs away from zombie johnthetubaguy | 15:01 | |
| johnthetubaguy | mriedem: good point, he is cooking things in that area, they can do the caching | 15:01 |
| mriedem | johnthetubaguy: i.e. when we get the ksa session (which is cached...) | 15:01 |
| dansmith | sfinucan: I'm saying if you want to have a mixin that implements __eq__, use the existing equal_prims thing to determine equality in your handler, don't define another one | 15:01 |
| *** Yangyape_ has joined #openstack-nova | 15:02 | |
| mriedem | johnthetubaguy: but thanks for reviewing, and welcome back | 15:02 |
| *** sree has quit IRC | 15:02 | |
| *** Yangyapeng has quit IRC | 15:02 | |
| mordred | mriedem: yes | 15:02 |
| mordred | mriedem: this is correct | 15:02 |
| *** gbarros has joined #openstack-nova | 15:06 | |
| openstackgerrit | Rick Bartra proposed openstack/nova master: Add policy granularity to the Flavors API https://review.openstack.org/449288 | 15:07 |
| *** lbragstad has quit IRC | 15:08 | |
| *** awaugama_ has quit IRC | 15:08 | |
| openstackgerrit | Chris Friesen proposed openstack/python-novaclient master: 2.47: Show flavor info in server details https://review.openstack.org/435141 | 15:10 |
| openstackgerrit | Stephen Finucane proposed openstack/nova master: objects: Add NovaComparableObject mixin https://review.openstack.org/472285 | 15:10 |
| mordred | johnthetubaguy, mriedem: the ksa patches for all of this are under review, so hopefully should be able to take legit forward aciton in the next couple of weeks | 15:10 |
| sfinucan | dansmith: Like that? | 15:10 |
| *** gbarros_ has joined #openstack-nova | 15:11 | |
| *** yamahata has quit IRC | 15:11 | |
| *** slaweq has joined #openstack-nova | 15:11 | |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: Add docstring for test_limit_check_project_and_user_zero_values https://review.openstack.org/472287 | 15:12 |
| mriedem | cdent: bauzas: melwitt: ^ | 15:12 |
| *** gbarros has quit IRC | 15:14 | |
| *** awaugama has joined #openstack-nova | 15:15 | |
| cfriesen | kashyap: apparently we have booted UEFI on Newton by specifying ‘--property hw_firmware_type=uefi” on the image and then booting as normal. | 15:16 |
| *** rcernin has quit IRC | 15:16 | |
| kashyap | cfriesen: Thank you for confirming | 15:16 |
| kashyap | cfriesen: I just checked with Fedora 25, and current Nova Git | 15:16 |
| kashyap | cfriesen: I thought the Q35 machine type was _mandatory_ for OVMF | 15:17 |
| kashyap | But, apparently it worked even with stock F25; which was QEMU 2.7 or something | 15:17 |
| cfriesen | kashyap: we're using CentOS 7's qemu-kvm-ev-2.6.0-28.el7_3.9.1, so who knows what all they've done. :) | 15:19 |
| openstackgerrit | sean mooney proposed openstack/os-vif master: set mtu on all code paths https://review.openstack.org/469951 | 15:19 |
| mnaser | stable/newton is broken with the new release of oslo.db that drops sqlite_db (it was removed in stable/ocata and beyond but not in stable/newton apparently): https://review.openstack.org/#/c/472290/ | 15:20 |
| mnaser | not sure why a notification didn't pop up here but if a small eye could be kept on that :) | 15:20 |
| kashyap | cfriesen: Heh | 15:20 |
| *** jmlowe has joined #openstack-nova | 15:21 | |
| mriedem | mnaser: i'm not sure why something dropped in ocata would break newton, | 15:21 |
| mriedem | since we should be using upper-constraints | 15:21 |
| mnaser | mriedem: maybe it was my mistake, but pretty much i did a pip install from the stable/newton repo and there doesn't seem to be a limit for oslo.db in requirements.txt | 15:22 |
| mnaser | so it downloaded the latest version of the library which has this commit from a few days ago => https://github.com/openstack/oslo.db/commit/cc64f7010522e812a696159009a72e59e2b017a8 | 15:23 |
| mriedem | mnaser: it'd be in tox.ini | 15:23 |
| mnaser | mriedem ok i see it | 15:23 |
| *** rmart04 has quit IRC | 15:23 | |
| mriedem | mnaser: yeah that's 4.19.0, but we're not using that in newton https://github.com/openstack/requirements/blob/stable/newton/upper-constraints.txt#L215 | 15:23 |
| mriedem | https://github.com/openstack/nova/blob/stable/newton/tox.ini#L12 should have you covered | 15:24 |
| mnaser | mriedem: sorry about that then, should i abandon that patch in that case? | 15:24 |
| openstackgerrit | Alex Szarka proposed openstack/nova master: Refactor create_delete_server_with_instance_update https://review.openstack.org/466296 | 15:24 |
| mriedem | mnaser: yes please | 15:25 |
| *** tbachman has joined #openstack-nova | 15:25 | |
| mnaser | mriedem: done, thanks for the lesson in requirements and sorry for noise | 15:25 |
| mriedem | not a problem | 15:25 |
| cfriesen | speaking of requirements...I'm trying to run current master devstack and it's giving me a series of version issues: ContextualVersionConflict: (os-brick 1.12.0 (/usr/local/lib/python2.7/dist-packages), Requirement.parse('os-brick>=1.13.1'), set(['cinder'])) | 15:28 |
| cfriesen | isn't it supposed to handle upgrading packages as needed automatically? | 15:28 |
| *** yamamoto has quit IRC | 15:30 | |
| *** moshele has joined #openstack-nova | 15:31 | |
| openstackgerrit | Andrey Volkov proposed openstack/nova master: [placement] Add api-ref for aggregates https://review.openstack.org/470932 | 15:32 |
| *** lei-zh1 has joined #openstack-nova | 15:33 | |
| *** lei-zh has quit IRC | 15:34 | |
| *** tbachman has quit IRC | 15:34 | |
| *** yamamoto has joined #openstack-nova | 15:35 | |
| *** jgriffith_ is now known as jgriffith | 15:36 | |
| *** moshele has quit IRC | 15:37 | |
| *** tbachman has joined #openstack-nova | 15:38 | |
| *** yamamoto has quit IRC | 15:39 | |
| mriedem | melwitt: can we use an IN clause to delete all server group members using the list of instance uuids at once? https://review.openstack.org/#/c/457858/16 | 15:39 |
| *** armax has quit IRC | 15:40 | |
| *** chyka has joined #openstack-nova | 15:40 | |
| *** Jack_Iv has quit IRC | 15:41 | |
| *** Jack_Iv has joined #openstack-nova | 15:41 | |
| *** Jack_Iv has quit IRC | 15:41 | |
| *** Jack_Iv has joined #openstack-nova | 15:42 | |
| *** Jack_Iv has quit IRC | 15:42 | |
| *** gyee has joined #openstack-nova | 15:42 | |
| *** slaweq has quit IRC | 15:43 | |
| *** Jack_Iv has joined #openstack-nova | 15:43 | |
| *** mdrabe has quit IRC | 15:44 | |
| *** lei-zh1 has quit IRC | 15:45 | |
| *** Jack_Iv has quit IRC | 15:47 | |
| *** mdrabe has joined #openstack-nova | 15:50 | |
| *** clayton has quit IRC | 15:51 | |
| *** markus_z has quit IRC | 15:51 | |
| *** Apoorva has joined #openstack-nova | 15:52 | |
| *** Apoorva has quit IRC | 15:52 | |
| sfinucan | melwitt: Could you take another look at https://review.openstack.org/#/c/379524/ today, please? :) | 15:52 |
| *** hshiina has joined #openstack-nova | 15:52 | |
| openstackgerrit | Hironori Shiina proposed openstack/nova master: Ironic: Support boot from Cinder volume https://review.openstack.org/215385 | 15:54 |
| *** dikonoor has joined #openstack-nova | 15:54 | |
| *** dtp has joined #openstack-nova | 15:54 | |
| *** mlakat has quit IRC | 15:54 | |
| *** clayton has joined #openstack-nova | 15:54 | |
| openstackgerrit | Merged openstack/python-novaclient master: Better handle key=value pair not being key=value https://review.openstack.org/469988 | 15:56 |
| *** eharney has quit IRC | 15:57 | |
| *** lbragstad has joined #openstack-nova | 15:57 | |
| *** gyee has quit IRC | 15:58 | |
| *** jaosorior has quit IRC | 16:00 | |
| *** Oku_OS is now known as Oku_OS-away | 16:00 | |
| *** Apoorva has joined #openstack-nova | 16:01 | |
| *** gomarivera has quit IRC | 16:01 | |
| cfriesen | re: my comment above, I just hit another one...neutron wants os-xenapi>=0.2.0. Shouldn't devstack handle the pip upgrade itself? | 16:04 |
| sdague | johnthetubaguy: https://dogpilecache.readthedocs.io/en/latest/api.html#memory-backends if we wanted to go down that path | 16:08 |
| *** eharney has joined #openstack-nova | 16:08 | |
| sdague | cfriesen: RECLONE=True ? | 16:08 |
| sdague | because if not you can hit requirements files out of sync with upper-constraints | 16:09 |
| *** nic1 has joined #openstack-nova | 16:09 | |
| cfriesen | sdague: thanks, will give that a try. I thought that was the default, for some reason. | 16:12 |
| sdague | no, because then it hard resets people's code | 16:13 |
| cfriesen | wonder if I should try PIP_UPGRADE=True | 16:13 |
| *** gyee has joined #openstack-nova | 16:16 | |
| sdague | I don't think that does anything any more with constraints in play | 16:17 |
| *** jamesdenton has joined #openstack-nova | 16:17 | |
| mriedem | kashyap: is there any reason that nova "needs" to set the 'serial' field in the disk config for the libvirt domain xml? | 16:19 |
| mriedem | reading https://libvirt.org/formatdomain.html#elementsDisks | 16:19 |
| mriedem | it says, | 16:19 |
| mriedem | "If present, this specify serial number of virtual hard drive. For example, it may look like <serial>WD-WMAP9A966149</serial>. Not supported for scsi-block devices, that is those using disk type 'block' using device 'lun' on bus 'scsi'. Since 0.7.1" | 16:19 |
| mriedem | however, we make no distinction of when we set that in the disk config, block type or not | 16:20 |
| mriedem | and it's always the cinder volume id i believe | 16:20 |
| mriedem | does qemu actually care about it? | 16:20 |
| johnthetubaguy | sdague: yeah, good call, spied your comment on that | 16:21 |
| mriedem | except avoid pickle | 16:22 |
| *** lucasagomes is now known as lucas-afk | 16:26 | |
| *** slaweq has joined #openstack-nova | 16:27 | |
| cfriesen | sdague: with RECLONE=True I still hit it: ContextualVersionConflict: (pyroute2 0.4.13 (/usr/local/lib/python2.7/dist-packages), Requirement.parse('pyroute2>=0.4.15'), set(['neutron'])) | 16:27 |
| sdague | cfriesen: I don't know | 16:28 |
| sdague | definitely going to need more than a single error log to know how you got there | 16:28 |
| sdague | go through the log and see why there is a disconnect there | 16:29 |
| *** sbezverk has joined #openstack-nova | 16:29 | |
| *** mingyu has joined #openstack-nova | 16:29 | |
| cfriesen | sdague: http://paste.openstack.org/show/612007/ | 16:29 |
| sdague | cfriesen: and the value in /opt/stack/requirements/upper-constraints.txt | 16:30 |
| cfriesen | sdague: pyroute2===0.4.15 | 16:30 |
| sdague | hmmm... idk | 16:31 |
| sdague | I thought that was always supposed to override | 16:32 |
| *** tesseract has quit IRC | 16:32 | |
| cfriesen | sdague: it choked on pyroute2 again even with PIP_UPGRADE=True | 16:35 |
| cfriesen | I'm tempted to just blow everything away and start from scratch | 16:36 |
| *** yamamoto has joined #openstack-nova | 16:37 | |
| sdague | yeh, I'm rerunning locally to try to figure out if I see anything | 16:37 |
| sdague | there could be a pip regression | 16:38 |
| *** Yangyape_ has quit IRC | 16:42 | |
| *** derekh has quit IRC | 16:42 | |
| *** rajathagasthya has joined #openstack-nova | 16:42 | |
| *** armax has joined #openstack-nova | 16:43 | |
| *** yamamoto has quit IRC | 16:43 | |
| *** clayton has quit IRC | 16:43 | |
| openstackgerrit | Merged openstack/nova master: Default to 0 when merging values in limit check https://review.openstack.org/471973 | 16:45 |
| *** smatzek has quit IRC | 16:46 | |
| *** clayton has joined #openstack-nova | 16:46 | |
| *** mamandle has quit IRC | 16:48 | |
| *** itlinux has joined #openstack-nova | 16:48 | |
| openstackgerrit | Alex Szarka proposed openstack/nova master: Refactor create_delete_server_with_instance_update https://review.openstack.org/466296 | 16:49 |
| *** rajathagasthya has quit IRC | 16:52 | |
| dansmith | mriedem: looks like that service version set is good now | 16:55 |
| mriedem | dansmith: did you see artom's questions in the top change? | 16:56 |
| dansmith | nope | 16:56 |
| dansmith | oh, yeah, I was just fixing the device metadata thing here | 16:57 |
| mriedem | the min version is checked in 2 places in conductor | 16:57 |
| dansmith | I can do the others now | 16:57 |
| dansmith | artom: ^ | 16:57 |
| mriedem | one is safely top-level super conductor, | 16:57 |
| *** nic1 is now known as nic | 16:57 | |
| mriedem | the other is rpc, so it's not possibly (i think) | 16:57 |
| dansmith | yeah, the rpc one he points to needs to stay single-cell, but I'll look to see if maybe we should do it higher | 16:57 |
| cfriesen | sdague: I have no idea what's going on. I blew away ~/devstack and cloned it again, and blew away /opt/stack, and I'm still hitting version conflicts. | 16:57 |
| *** hshiina has quit IRC | 16:58 | |
| dansmith | mriedem: we may also be able to drop lots of these | 16:58 |
| *** sambetts is now known as sambetts|afk | 16:58 | |
| mriedem | i'd like to drop paging from the hypervisors api | 16:59 |
| dansmith | heh | 16:59 |
| *** pcaruana has quit IRC | 17:00 | |
| *** cdent has quit IRC | 17:00 | |
| *** rajathagasthya has joined #openstack-nova | 17:01 | |
| clarkb | cfriesen: sdague fwiw a trivial attempt at reproducing using a venv with 0.4.13 installed then an install of 0.4.15 using constraints works ok | 17:01 |
| mriedem | btw i'm out tomorrow | 17:02 |
| clarkb | so whatever it is seems more complicated than just pyroute2 and constraints | 17:03 |
| openstackgerrit | Alex Szarka proposed openstack/nova master: Transform instance.exists notification https://review.openstack.org/403660 | 17:03 |
| *** ralonsoh has quit IRC | 17:03 | |
| openstackgerrit | Alex Szarka proposed openstack/nova master: Refactor instance-exists docstring and unit test https://review.openstack.org/462738 | 17:03 |
| *** dtp has quit IRC | 17:04 | |
| *** baoli has quit IRC | 17:04 | |
| clarkb | cfriesen: maybe try with pip -v and/or --log ? | 17:05 |
| *** baoli has joined #openstack-nova | 17:05 | |
| cfriesen | clarkb: I can upgrade this manually no problem....but I expect devstack to be doing the upgrading and I don't understand why it isn't | 17:05 |
| openstackgerrit | Alex Szarka proposed openstack/nova master: Transform instance.resize.error notifications https://review.openstack.org/417882 | 17:06 |
| clarkb | cfriesen: so if you run the command yourself it works fine? | 17:06 |
| cfriesen | clarkb: I'm hitting this with multiple packages. If I manually upgrade one package it continues on until it hits another one that needs upgrading. | 17:06 |
| mriedem | dansmith: are you ok with this now after the cells v1 rpc version bump? https://review.openstack.org/#/c/461532/ | 17:06 |
| cfriesen | clarkb: something like this works fine: sudo pip install --upgrade ovsdbapp==0.4.0 | 17:07 |
| dansmith | mriedem: with that approach yeah, I need to look at the rest of the patch though | 17:07 |
| sdague | cfriesen: I for installed pyroute2==0.4.13 then ran ./clean.sh && ./stack.sh and everthing works fine | 17:07 |
| clarkb | perhaps something with the -H then and pip caching things? But also getting output from pip -v and/or --log would likely be helpful | 17:07 |
| sdague | this is a 16.04 box | 17:08 |
| cfriesen | sdague: I hadn't tried clean.sh, will give that a shot | 17:08 |
| cfriesen | sdague: same here | 17:09 |
| *** mamandle has joined #openstack-nova | 17:10 | |
| openstackgerrit | Merged openstack/nova master: code comments incorrectness https://review.openstack.org/449416 | 17:11 |
| cfriesen | mriedem: if you've got suggestions on how to properly mock the api_version in https://review.openstack.org/#/c/435141/ I'd be happy to make changes. I couldn't figure it out short of modifying each testcase using the "list" command. | 17:12 |
| *** lucasxu has quit IRC | 17:14 | |
| *** ltomasbo is now known as ltomasbo|away | 17:14 | |
| *** sridharg has quit IRC | 17:15 | |
| *** ericyoung has quit IRC | 17:16 | |
| cfriesen | sdague: ./clean.sh wasn't enough. just hit ContextualVersionConflict with neutron-lib | 17:16 |
| cfriesen | sdague: clarkb: http://paste.openstack.org/show/612034/ | 17:18 |
| *** rmart04 has joined #openstack-nova | 17:18 | |
| *** jpena is now known as jpena|off | 17:23 | |
| *** baoli has quit IRC | 17:23 | |
| *** baoli has joined #openstack-nova | 17:24 | |
| *** mamandle has quit IRC | 17:24 | |
| *** fragatina has quit IRC | 17:24 | |
| *** baoli has quit IRC | 17:25 | |
| *** amotoki is now known as amotoki_away | 17:28 | |
| *** rmart04 has quit IRC | 17:30 | |
| *** dtp has joined #openstack-nova | 17:31 | |
| artom | dansmith, huh, didn't notice the ping - so basically the other places where get_min_service_version is called will be fixed in different patches? | 17:32 |
| dansmith | artom: yeah, I was just fixing that one place in device metadata because of mriedem's bug he opened | 17:32 |
| dansmith | but I | 17:32 |
| dansmith | will take a look at the rest in a few | 17:32 |
| artom | dansmith, makes sense :) | 17:33 |
| artom | Keep commits topical | 17:33 |
| *** cdent has joined #openstack-nova | 17:33 | |
| *** mkucia has quit IRC | 17:33 | |
| *** baoli has joined #openstack-nova | 17:34 | |
| *** amotoki_away is now known as amotoki | 17:35 | |
| *** ihrachys has joined #openstack-nova | 17:35 | |
| ihrachys | seems like port detach issues still happens even with https://review.openstack.org/#/c/441204/ in | 17:36 |
| mriedem | i like my creams just like i like my commits | 17:36 |
| ihrachys | mriedem, ^ | 17:36 |
| ihrachys | example: http://logs.openstack.org/63/471763/2/check/gate-tempest-dsvm-neutron-linuxbridge-ubuntu-xenial/dccf187/logs/screen-n-cpu.txt.gz?level=TRACE#_Jun_08_16_58_01_395989 | 17:36 |
| mriedem | hmm, that's not fun | 17:37 |
| *** nicolasbock has quit IRC | 17:37 | |
| mriedem | ihrachys: it's always this same test right http://logs.openstack.org/63/471763/2/check/gate-tempest-dsvm-neutron-linuxbridge-ubuntu-xenial/dccf187/console.html#_2017-06-08_17_13_16_844927 ? | 17:38 |
| mriedem | the hotpluc nic one? | 17:38 |
| mriedem | and usually on the linuxbridge job? | 17:38 |
| artom | mriedem, the word you're looking for is "ointments" - it's up there with "moist" as the most shudder-inducing word in the English language | 17:38 |
| mriedem | yeah it's like 99% LB job | 17:38 |
| ihrachys | I think yes | 17:38 |
| mriedem | i wonder what's different about LB in thiscase | 17:38 |
| mriedem | maybe the vif isn't unplugging how we'd expect before we detach from the guest? | 17:38 |
| mriedem | artom: i think "topical cream" | 17:39 |
| mriedem | not "topical ointment" | 17:39 |
| ihrachys | afaik for linuxbridge case, neutron now manages the whole plugging cycle ? | 17:39 |
| ihrachys | since late switch to tap device type | 17:39 |
| *** hieulq_ has quit IRC | 17:39 | |
| mriedem | ihrachys: i don't know anything about that | 17:39 |
| ihrachys | I mean https://review.openstack.org/#/c/447150/ | 17:39 |
| *** hieulq_ has joined #openstack-nova | 17:39 | |
| *** cdent has quit IRC | 17:39 | |
| artom | mriedem, because of you my search history is weird now | 17:39 |
| mriedem | artom: search on your phone, not your work system | 17:40 |
| *** hieulq_ has quit IRC | 17:40 | |
| mriedem | nothing weird about grown man pain reliefe | 17:40 |
| mriedem | *relief | 17:40 |
| mriedem | just ask shaq | 17:40 |
| * artom wonders if we need a DNM patch that shows us the actual libvirt error | 17:41 | |
| mriedem | should be in the libvirtd logs wouldn't it? | 17:41 |
| mriedem | i also don't understand why libvirt considers this a transient domain | 17:43 |
| * ihrachys doesn't even know what transient domain is | 17:43 | |
| mriedem | https://wiki.libvirt.org/page/VM_lifecycle#Transient_guest_domains_vs_Persistent_guest_domains | 17:43 |
| artom | I *think* it does both? | 17:43 |
| mriedem | me neither | 17:43 |
| artom | As in, a running domain has the persistent part, and the transient part | 17:44 |
| artom | kashyap would help here, but I think he's gone | 17:44 |
| melwitt | I think it's something like it creates a transient domain on-the-fly and uses that for the migration? something like that? | 17:44 |
| ihrachys | oh it's just a in-ram domain? | 17:44 |
| mriedem | reading ^ makes me think all of our guests are transient | 17:44 |
| mriedem | this isn't a migration though | 17:44 |
| melwitt | oh | 17:45 |
| *** jamesdenton has quit IRC | 17:45 | |
| mriedem | http://git.openstack.org/cgit/openstack/tempest/tree/tempest/scenario/test_network_basic_ops.py#n475 | 17:45 |
| *** itlinux has quit IRC | 17:45 | |
| cfriesen | mriedem: I think we generally make the xml file and tell libvirt to make a guest from it, making them persistent | 17:46 |
| artom | Wait, that's not a libvirt error | 17:48 |
| melwitt | one thing to know is a detach from transient domain is optional, that is, the guest can ignore it | 17:49 |
| artom | https://github.com/openstack/nova/blob/master/nova/virt/libvirt/guest.py#L442-L449 | 17:50 |
| cfriesen | melwitt: that sounds like a terrible idea. | 17:50 |
| mriedem | artom: oh nice spot | 17:50 |
| mriedem | we should be checking the persistent flag there | 17:50 |
| ihrachys | ehm, why in the snippet do we raise unconditionally? | 17:50 |
| mriedem | ihrachys: it's part of a retry loop | 17:51 |
| mriedem | see the decorator | 17:51 |
| mriedem | it retries on that expected error until the device conf is gone or we timeout | 17:51 |
| artom | It retries until it stops raising? | 17:51 |
| ihrachys | ah I see | 17:51 |
| cfriesen | melwitt: mriedem: so under what circumstances would nova create a transient domain? | 17:51 |
| melwitt | I don't think nova does it, I thought it was something with qemu but I can't remember what kashyap told me. maybe I'm totally off on that | 17:52 |
| cfriesen | looks like maybe while doing a snapshot | 17:52 |
| mriedem | we're not using os-vif http://logs.openstack.org/63/471763/2/check/gate-tempest-dsvm-neutron-linuxbridge-ubuntu-xenial/dccf187/logs/screen-n-cpu.txt.gz?level=DEBUG#_Jun_08_16_56_19_922445 | 17:53 |
| mriedem | but that probably doesn't matter | 17:53 |
| *** nicolasbock has joined #openstack-nova | 17:53 | |
| artom | Hrmm, that could be a libvirt error though | 17:56 |
| artom | If we've failed to detach and reach the max retry count | 17:56 |
| * artom is confused | 17:56 | |
| mriedem | it's got to be something with linuxbridge and this tap vif type | 17:56 |
| *** jwcroppe has quit IRC | 17:56 | |
| mriedem | not a problem for ovs | 17:56 |
| mriedem | and a race, since it's not 100% fail | 17:57 |
| *** jwcroppe has joined #openstack-nova | 17:57 | |
| openstackgerrit | Merged openstack/nova master: Use microversions for new style volume attachments https://review.openstack.org/469579 | 17:57 |
| kfox1111 | so... whats the easiest way to list out all vms in all tenants that use a particular flavor? | 17:58 |
| mriedem | Successfully detached device <nova.virt.libvirt.config.LibvirtConfigGuestInterface object at 0x7f86a0051650> from guest. Persistent? 1. Live? True {{(pid=24466) _try_detach_device /opt/stack/new/nova/nova/virt/libvirt/guest.py:407}} | 17:58 |
| *** gjayavelu has joined #openstack-nova | 17:58 | |
| melwitt | yeah, initially I was thinking we must have reached the max retry count because the guest wasn't responding, but I'm not sure how to tell from the log, if we can look at a timestamp delta at all to see how long it took before the error was raised | 17:58 |
| mriedem | kfox1111: nova list --all-tenants --flavor x ? | 17:58 |
| ihrachys | mriedem, you think we may try to revert the tap switch patch for the time being? | 17:59 |
| kfox1111 | doh. there it was, right in front of me. | 17:59 |
| ihrachys | will it possibly help | 17:59 |
| kfox1111 | thanks. :) | 17:59 |
| mriedem | ihrachys: the neutron one? | 17:59 |
| *** jwcroppe_ has joined #openstack-nova | 17:59 | |
| mriedem | ihrachys: we could see if that's the cause by rechecking that a few times | 17:59 |
| ihrachys | at this point, we can't land much in neutron, and we have a line of gate fixes, so anything that could help.. | 17:59 |
| ihrachys | ok lemme revert and see where it goes | 17:59 |
| mriedem | i don't know enough about the low level networking stuff here to know why this would cause issues | 17:59 |
| mriedem | i'm going to go libvirtd log diving | 17:59 |
| *** lucasxu has joined #openstack-nova | 18:00 | |
| *** jwcroppe has quit IRC | 18:01 | |
| mriedem | right before we detach in the driver, we get the network-vif-unplugged event from neutron | 18:03 |
| *** gszasz has quit IRC | 18:04 | |
| mriedem | but we don't do anything with that | 18:05 |
| mriedem | "Successfully detached device <nova.virt.libvirt.config.LibvirtConfigGuestInterface object at 0x7f86a0051650> from guest. Persistent? 1. Live? True" | 18:06 |
| mriedem | so i guess the domain is persistent on the first detach_device call to the guest | 18:06 |
| *** smatzek has joined #openstack-nova | 18:07 | |
| openstackgerrit | Merged openstack/nova master: Use 3.27 microversion when creating new style volume attachments https://review.openstack.org/469609 | 18:08 |
| mriedem | hmm, could we be hitting one of these libvirt errors and raising DeviceNotFound? | 18:08 |
| mriedem | https://github.com/openstack/nova/blob/master/nova/virt/libvirt/guest.py#L409 | 18:09 |
| melwitt | compute manager ignores DeviceNotFound though, just considers detach already done, right? | 18:09 |
| openstackgerrit | Merged openstack/nova master: Skip microversion discovery check for update/delete volume attachments https://review.openstack.org/469668 | 18:09 |
| mriedem | for volumes | 18:09 |
| mriedem | the driver ignores DeviceNotFound during detach_interface | 18:09 |
| mriedem | so that's probably not it | 18:10 |
| *** jamesdenton has joined #openstack-nova | 18:10 | |
| mriedem | wtf, i don't think it's even retrying | 18:10 |
| *** pewp has quit IRC | 18:10 | |
| mriedem | so we detach the guest here http://logs.openstack.org/63/471763/2/check/gate-tempest-dsvm-neutron-linuxbridge-ubuntu-xenial/dccf187/logs/screen-n-cpu.txt.gz?level=DEBUG#_Jun_08_16_56_25_266814 | 18:11 |
| mriedem | s/guest/device | 18:11 |
| mriedem | that's https://github.com/openstack/nova/blob/master/nova/virt/libvirt/guest.py#L405 | 18:11 |
| mriedem | on the first attempt | 18:11 |
| mriedem | where persistent=1 | 18:11 |
| mriedem | then we start our retry loop http://logs.openstack.org/63/471763/2/check/gate-tempest-dsvm-neutron-linuxbridge-ubuntu-xenial/dccf187/logs/screen-n-cpu.txt.gz?level=DEBUG#_Jun_08_16_56_25_267223 | 18:11 |
| mriedem | then we try detach again http://logs.openstack.org/63/471763/2/check/gate-tempest-dsvm-neutron-linuxbridge-ubuntu-xenial/dccf187/logs/screen-n-cpu.txt.gz?level=DEBUG#_Jun_08_16_56_30_278317 | 18:11 |
| mriedem | this time persistent=False | 18:11 |
| mriedem | because https://github.com/openstack/nova/blob/master/nova/virt/libvirt/guest.py#L445 | 18:12 |
| *** pewp has joined #openstack-nova | 18:12 | |
| mriedem | oh nvm it does retry | 18:13 |
| ihrachys | it may be interesting that tap3e81295f-c1 never shows in l2 agent log since 08 16:56:20.783179 which is long before the detach. should we have seen the agent to e.g. get a detach event from neutron-server and untangling in from the integration bridge, or smth? I assume nova talks to neutron when detaching? | 18:13 |
| mriedem | the retry decorator doesn't have the logging context for the request id | 18:13 |
| mriedem | ihrachys: nova will only update the port to unbind it | 18:13 |
| mriedem | otherwise nova just unplugs the vif and relies on neutron agent picking that up and processing it | 18:14 |
| mriedem | and then neutron sends the network-vif-unplugged event | 18:14 |
| *** Sukhdev has joined #openstack-nova | 18:16 | |
| mriedem | i wonder if we're not checking the right things here | 18:17 |
| mriedem | https://github.com/openstack/nova/blob/master/nova/virt/libvirt/guest.py#L247-L256 | 18:17 |
| mriedem | this is our xml | 18:18 |
| mriedem | http://paste.openstack.org/show/612041/ | 18:18 |
| mriedem | from LibvirtConfigGuestInterface | 18:18 |
| ihrachys | mriedem, what do you mean unplugging the VIF? from the domain? because with tap vif type, everything else is probably on neutron to do right? meaning, you request unbind and then we handle it all? | 18:19 |
| ihrachys | kevinbenton, are you up already? | 18:20 |
| mriedem | ihrachys: i mean this https://github.com/openstack/nova/blob/master/nova/virt/libvirt/driver.py#L1450 | 18:20 |
| ihrachys | which is empty for lb | 18:22 |
| mriedem | ihrachys: no it's going here https://github.com/openstack/nova/blob/master/nova/virt/libvirt/vif.py#L906 | 18:23 |
| ihrachys | oh. I was looking at unplug in os-vif | 18:24 |
| *** mvk has quit IRC | 18:24 | |
| ihrachys | for lb plugin | 18:24 |
| mriedem | yeah it's not using os-vif | 18:24 |
| mriedem | because there is no os-vif tap conversion in nova yet | 18:24 |
| mriedem | http://logs.openstack.org/63/471763/2/check/gate-tempest-dsvm-neutron-linuxbridge-ubuntu-xenial/dccf187/logs/screen-n-cpu.txt.gz?level=DEBUG#_Jun_08_16_56_19_922445 | 18:25 |
| mriedem | No conversion for VIF type tap yet | 18:25 |
| mriedem | so nova does: sudo nova-rootwrap /etc/nova/rootwrap.conf ip link delete tap3e81295f-c1 | 18:25 |
| *** salv-orl_ has quit IRC | 18:25 | |
| *** salv-orlando has joined #openstack-nova | 18:26 | |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: libvirt: fix alternative_device_name for detaching interfaces https://review.openstack.org/472371 | 18:30 |
| *** salv-orlando has quit IRC | 18:30 | |
| mriedem | ihrachys: i also noticed this comment which is probably from danpb https://github.com/openstack/nova/blob/master/nova/virt/libvirt/designer.py#L56 | 18:31 |
| mriedem | but that's the method called to build the device config for this type of tap device | 18:31 |
| ihrachys | btw I misinterpreted the logs before. I now see that neutron logs operations on the tap device at the same time nova starts the decorated func | 18:31 |
| mriedem | this is all i see about it in the libvirt docs https://libvirt.org/formatdomain.html#elementsNICSEthernet | 18:32 |
| mriedem | i assume it's discouraged because "Provides a means for the administrator to execute an arbitrary script to connect the guest's network to the LAN." | 18:32 |
| mriedem | as far as i can tell though nova doesn't pass through any script path | 18:33 |
| mriedem | https://github.com/openstack/nova/blob/master/nova/virt/libvirt/designer.py#L69 | 18:34 |
| mriedem | oh i think that means if not provided, it defaults to /etc/qemu-ifup | 18:34 |
| *** jamesdenton has quit IRC | 18:34 | |
| *** jamesdenton has joined #openstack-nova | 18:35 | |
| *** gbarros_ has quit IRC | 18:35 | |
| *** dikonoor has quit IRC | 18:35 | |
| *** itlinux_ has joined #openstack-nova | 18:38 | |
| mriedem | ihrachys: i'm going to push a change that catches the final exception, dumps a bunch of debug info to the logs, and then re-raises | 18:38 |
| mriedem | to see what's actually in the guest xml when we fail compared to the device xml we're trying to remove | 18:38 |
| *** moshele has joined #openstack-nova | 18:44 | |
| *** nicolasbock has quit IRC | 18:45 | |
| *** nicolasbock has joined #openstack-nova | 18:48 | |
| *** egonzalez has joined #openstack-nova | 18:49 | |
| *** crushil has quit IRC | 18:50 | |
| *** nicolasbock has quit IRC | 18:50 | |
| *** nicolasbock has joined #openstack-nova | 18:50 | |
| *** cdent has joined #openstack-nova | 18:53 | |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: libvirt: dump debug info when interface detach times out https://review.openstack.org/472377 | 18:54 |
| mriedem | ihrachys: i'll push a neutron change that depends on this ^ to debug, i see you pushed a revert | 18:54 |
| *** amotoki is now known as amotoki_away | 18:55 | |
| openstackgerrit | Corey Bryant proposed openstack/nova master: Make nova-placement-api default config less strict https://review.openstack.org/472378 | 18:56 |
| *** mnestratov|3 has quit IRC | 18:56 | |
| *** dtp has quit IRC | 18:58 | |
| *** jwcroppe_ has quit IRC | 18:58 | |
| *** dave-mccowan has quit IRC | 18:58 | |
| *** eharney has quit IRC | 18:58 | |
| cfriesen | if instances have attached volumes, that information would be in the block_device_mapping table of the nova db, right? | 18:59 |
| mriedem | cfriesen: yes | 19:00 |
| *** cristicalin has quit IRC | 19:00 | |
| *** nicolasbock has quit IRC | 19:00 | |
| *** gbarros has joined #openstack-nova | 19:01 | |
| *** dave-mccowan has joined #openstack-nova | 19:02 | |
| *** jwcroppe has joined #openstack-nova | 19:03 | |
| *** cdent has quit IRC | 19:04 | |
| *** jwcroppe has quit IRC | 19:04 | |
| *** jwcroppe has joined #openstack-nova | 19:05 | |
| mriedem | we have an intermittent KeyError in functional tests https://bugs.launchpad.net/nova/+bug/1696834 | 19:05 |
| openstack | Launchpad bug 1696834 in OpenStack Compute (nova) "Intermittent "KeyError: 'allocations'" in functional tests" [Medium,Confirmed] | 19:05 |
| *** nicolasbock has joined #openstack-nova | 19:05 | |
| melwitt | yeah, seems to hit pretty often too | 19:05 |
| mriedem | 14 in 7 days | 19:05 |
| *** jmlowe_ has joined #openstack-nova | 19:07 | |
| *** jmlowe has quit IRC | 19:08 | |
| *** Jack_Iv has joined #openstack-nova | 19:09 | |
| *** pewp has quit IRC | 19:13 | |
| *** yamahata has joined #openstack-nova | 19:17 | |
| mriedem | dansmith: melwitt: can one of you put this through so i don't have to work it into my hypervisors uuid api series? https://review.openstack.org/#/c/471970/ | 19:17 |
| mriedem | i have to build on that one | 19:17 |
| mriedem | to change the type | 19:17 |
| dansmith | done | 19:17 |
| openstackgerrit | Corey Bryant proposed openstack/nova master: Make nova-placement-api default config less strict https://review.openstack.org/472378 | 19:17 |
| mriedem | <3 | 19:17 |
| *** pewp has joined #openstack-nova | 19:17 | |
| mriedem | fleet week failed again, | 19:18 |
| mriedem | but something different this time http://logs.openstack.org/94/436094/33/check/gate-tempest-dsvm-neutron-full-ubuntu-xenial/bed0e6a/console.html#_2017-06-08_18_39_03_929168 | 19:18 |
| *** nicolasbock has quit IRC | 19:18 | |
| *** nicolasbock has joined #openstack-nova | 19:18 | |
| dansmith | o.O | 19:19 |
| mordred | efried: https://review.openstack.org/#/q/project:openstack/keystoneauth+topic:version-discovery is updated from reviews if you wanna poke | 19:21 |
| efried | mordred Roger wilco. | 19:21 |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: Handle uuid in HostAPI.compute_node_get https://review.openstack.org/461532 | 19:21 |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: WIP: Handle uuids in os-hypervisors API https://review.openstack.org/471899 | 19:21 |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: WIP: Use uuid for id in Services API https://review.openstack.org/464280 | 19:21 |
| *** rmart04 has joined #openstack-nova | 19:21 | |
| mordred | efried: still missing interface lists and service type aliases - but we're getting real close | 19:21 |
| efried | neat | 19:21 |
| mriedem | dansmith: unrelated | 19:22 |
| mriedem | the test is posting a password to the metadata server | 19:23 |
| dansmith | good, because....... | 19:23 |
| *** nicolasbock has quit IRC | 19:23 | |
| mriedem | and then checks to see it shows up | 19:23 |
| mriedem | which seems super racy | 19:23 |
| mriedem | that's totally async so tempest should probably be polling here | 19:23 |
| mriedem | maybe i'm not sure, | 19:26 |
| mriedem | it sets a password field in the instance sys meta | 19:26 |
| mriedem | it's a POST, | 19:26 |
| mriedem | but i'm not sure where the metadata service returns | 19:26 |
| openstackgerrit | Merged openstack/nova master: api-ref: fix type for hypervisor_marker https://review.openstack.org/471970 | 19:26 |
| kevinbenton | ihrachys: hey | 19:26 |
| *** gbarros has quit IRC | 19:27 | |
| kevinbenton | ihrachys: looking at backlog. l2-agent never deletes a tap device, that's left up to nova | 19:28 |
| kevinbenton | mriedem: ^^ | 19:28 |
| kevinbenton | but it looks like you already found that | 19:28 |
| kevinbenton | anything i can help with? | 19:28 |
| *** itlinux_ has quit IRC | 19:28 | |
| *** awaugama has quit IRC | 19:31 | |
| *** sbezverk has quit IRC | 19:32 | |
| mriedem | dansmith: i guess the metadata POST is synchronous | 19:32 |
| *** rajathagasthya has quit IRC | 19:34 | |
| *** moshele has quit IRC | 19:35 | |
| mriedem | dansmith: aha http://logs.openstack.org/94/436094/33/check/gate-tempest-dsvm-neutron-full-ubuntu-xenial/bed0e6a/logs/screen-n-api.txt.gz#_Jun_08_18_36_06_549748 | 19:36 |
| mriedem | the cache fs us again | 19:36 |
| mriedem | we update inst.system_metadata, then tempest calls the rest api to get the password, and it looks like things are going through the metadata cache | 19:37 |
| mriedem | not sure why we wouldn't update the cache when we update/set the password... | 19:37 |
| dansmith | but that's still unrelated right? | 19:38 |
| dansmith | but yeah, that metadata stuff needs fixin' | 19:38 |
| mriedem | wtf, we're getting an InstanceNotFound | 19:38 |
| mriedem | oooooo | 19:38 |
| mriedem | it is cells | 19:38 |
| mriedem | we're not targeted | 19:38 |
| mriedem | mofo | 19:38 |
| mriedem | opening a bug | 19:38 |
| dansmith | we shouldn't be targeted | 19:39 |
| mriedem | actually the instance should be targeted when we look it up | 19:39 |
| mriedem | i mean, we look up the mapping and then target the instance don't we? | 19:40 |
| dansmith | metadata should be a cell thing so it shouldn't be using target_cell, is my point | 19:40 |
| dansmith | maybe that's the problem | 19:40 |
| dansmith | does metadata run as a separate process in devstack? | 19:40 |
| * dansmith looks | 19:40 | |
| mriedem | not here, it's part of nova-api | 19:41 |
| openstackgerrit | Merged openstack/nova master: Add docstring for test_limit_check_project_and_user_zero_values https://review.openstack.org/472287 | 19:41 |
| *** mvk has joined #openstack-nova | 19:41 | |
| mriedem | https://bugs.launchpad.net/nova/+bug/1696843 | 19:42 |
| openstack | Launchpad bug 1696843 in OpenStack Compute (nova) "metadata: POST /openstack/2013-10-17/password fails with InstanceNotFound" [Undecided,New] | 19:42 |
| mriedem | kevinbenton: i've got a debug patch up to see if i can recreate in a neutron patch and get some dumps of the related domain xml | 19:43 |
| dansmith | mriedem: well, that's going to be a problem then | 19:43 |
| *** claudiub has joined #openstack-nova | 19:44 | |
| mriedem | and ihar has a patch up with a revert of the lb tap change to see if that is any different | 19:44 |
| *** itlinux_ has joined #openstack-nova | 19:44 | |
| *** yamahata has quit IRC | 19:44 | |
| *** gbarros has joined #openstack-nova | 19:44 | |
| mriedem | dansmith: right here https://github.com/openstack/nova/blob/b535c7432d02366f11fad7d1b1a3e3a8b8ac5415/nova/api/metadata/password.py#L70 | 19:45 |
| mriedem | why wouldn't we lookup the instance mapping via that uuid | 19:45 |
| mriedem | and then target the get_by_uuid to the cell | 19:45 |
| dansmith | mriedem: because metadata is in the cell and we can't do that | 19:45 |
| mriedem | i thought it was global | 19:45 |
| mriedem | actually i don't think i've ever heard of anyone ever talk about the metadata service wrt cells v2 | 19:45 |
| dansmith | not sure why it would be global, | 19:46 |
| dansmith | but also, | 19:46 |
| dansmith | not sure from a neutron perspective | 19:46 |
| dansmith | I always think of the world like nova net multihost | 19:46 |
| dansmith | maybe we can ask kevinbenton since he's here | 19:46 |
| mriedem | why wouldn't it be global if the api is global? | 19:46 |
| mriedem | i guess because you want the guests confined to the cell | 19:47 |
| mriedem | ? | 19:47 |
| dansmith | hmm, we already target in metadata one place | 19:47 |
| kevinbenton | ask me what? | 19:47 |
| mriedem | yeah... | 19:47 |
| mriedem | if metadata can't be global then it's going to have to know what cell it's in | 19:47 |
| dansmith | mriedem: in multihost you run one per compute node so you scale 1:1 with computes | 19:47 |
| *** gszasz has joined #openstack-nova | 19:47 | |
| dansmith | mriedem: I don't think I understand why that is | 19:48 |
| *** jaypipes has quit IRC | 19:48 | |
| dansmith | mriedem: however, question would be how the neutron proxy thing finds the metadata api | 19:48 |
| dansmith | kevinbenton: ^ | 19:48 |
| mriedem | it's configured in neutron.conf i though | 19:48 |
| mriedem | *thought | 19:48 |
| kevinbenton | dansmith: ah, it's just a hardcoded config | 19:48 |
| kevinbenton | dansmith: of a single http endpoint | 19:48 |
| dansmith | but neutron config per compute node right? | 19:48 |
| mriedem | no | 19:48 |
| mriedem | well i can't answer that | 19:49 |
| kevinbenton | it's on the network node that runs the proxy | 19:49 |
| *** gbarros has quit IRC | 19:49 | |
| kevinbenton | so not compute node | 19:49 |
| dansmith | kevinbenton: so is it a single config for the whole deployment? | 19:49 |
| kevinbenton | dansmith: essentially yes | 19:49 |
| kevinbenton | dansmith: you could have different configs per network node | 19:49 |
| dansmith | okay, not sure that's a good thing, although it does make this easier | 19:49 |
| kevinbenton | but i don't think that would do anything useful | 19:49 |
| mriedem | kevinbenton: where is that in http://logs.openstack.org/94/436094/33/check/gate-tempest-dsvm-neutron-full-ubuntu-xenial/bed0e6a/logs/etc/neutron/neutron.conf.txt.gz ? | 19:49 |
| kevinbenton | since networks nodes aren't correlated with cells | 19:49 |
| mriedem | oh nvm http://logs.openstack.org/94/436094/33/check/gate-tempest-dsvm-neutron-full-ubuntu-xenial/bed0e6a/logs/etc/neutron/metadata_agent.ini.txt.gz | 19:49 |
| mriedem | nova_metadata_ip = 10.40.184.140 | 19:50 |
| kevinbenton | mriedem: +1 | 19:50 |
| dansmith | from a scale point of view I mean, but since we already effectively have it global I guess that's a totally different conversation | 19:50 |
| *** rmart04 has quit IRC | 19:50 | |
| mriedem | don't people already disable metadata service b/c of perf issues, and security | 19:50 |
| mriedem | i know that's irrelevant | 19:51 |
| dansmith | depends on their goals, but some do yeah | 19:51 |
| dansmith | alright anyway, I'll fix that up | 19:51 |
| openstackgerrit | Dan Smith proposed openstack/nova master: Remove old service version check for mitaka https://review.openstack.org/472395 | 19:51 |
| *** mingyu has quit IRC | 19:52 | |
| mriedem | i suppose this meta bug has to be backported | 19:52 |
| mriedem | although i'm only seeing it in the multi-cell devstack change | 19:52 |
| *** rajathagasthya has joined #openstack-nova | 19:56 | |
| kevinbenton | dansmith, mriedem: is there a way we can know which cell to send something to? | 19:56 |
| kevinbenton | we could alter the proxy logic to route in some manner because it does do a port lookup to inject headers | 19:57 |
| dansmith | mriedem: I don't think we need to backport it | 19:57 |
| *** salv-orlando has joined #openstack-nova | 19:57 | |
| dansmith | kevinbenton: you don't and shouldn't know any of that | 19:57 |
| kevinbenton | dansmith: ok, just checking if this is an issue we need to fix on our metadata proxy logic | 19:58 |
| mriedem | i wonder how cells v1 people are doing the metadata service | 19:59 |
| dansmith | kevinbenton: no it sounds like you're effectively forcing a global metadata server now, which makes a fix for this easy on our end | 19:59 |
| dansmith | mriedem: not sure it works in cellsv1, or they just run it in a cell I'd imagine | 19:59 |
| mriedem | i'll ask in ops | 19:59 |
| kevinbenton | just put a load balancer in front of it that fans every request out to a cell until it doesn't get a 404 :) | 19:59 |
| *** liangy has joined #openstack-nova | 19:59 | |
| *** lyan has quit IRC | 20:01 | |
| mriedem | you don't get a 404 here, | 20:02 |
| mriedem | you get a 500 :( | 20:02 |
| mriedem | we probably also need to fix that | 20:02 |
| mriedem | http://logs.openstack.org/94/436094/33/check/gate-tempest-dsvm-neutron-full-ubuntu-xenial/bed0e6a/logs/screen-n-api.txt.gz#_Jun_08_18_36_06_549748 | 20:02 |
| mriedem | 10.1.0.11,10.40.184.140 "POST /openstack/2013-10-17/password HTTP/1.1" status: 500 len: 139 time: 0.0445061: InstanceNotFound: Instance 6cd26f78-9c8a-4b37-8c62-38ad003a095f could not be found. | 20:03 |
| mriedem | i'll open a separate bug for that | 20:03 |
| mriedem | which is backportable | 20:04 |
| *** pchavva has quit IRC | 20:04 | |
| mriedem | https://bugs.launchpad.net/nova/+bug/1696848 | 20:05 |
| openstack | Launchpad bug 1696848 in OpenStack Compute (nova) "POST /openstack/2013-10-17/password returns 500 if instance is not found" [Medium,Triaged] | 20:05 |
| *** harlowja has quit IRC | 20:08 | |
| mriedem | claudiub: are you doing anything with this? https://blueprints.launchpad.net/nova/+spec/hyperv-sriov | 20:08 |
| mriedem | else i'm going to defer it to queens | 20:08 |
| *** liverpooler has quit IRC | 20:09 | |
| mriedem | gdi, you know what's awesome, | 20:13 |
| mriedem | overriding a method and tests start failing, you spend too much time trying to figure out why, | 20:13 |
| mriedem | and realize you're not returning the results of the method you're now delegating too | 20:13 |
| mriedem | *to | 20:13 |
| *** pchavva has joined #openstack-nova | 20:17 | |
| *** jmlowe_ has quit IRC | 20:17 | |
| *** Jack_Iv has quit IRC | 20:17 | |
| *** Jack_Iv has joined #openstack-nova | 20:18 | |
| openstackgerrit | Maciej Kucia proposed openstack/nova master: libvirt: Extract method _guest_add_spice_channel https://review.openstack.org/470787 | 20:18 |
| openstackgerrit | Maciej Kucia proposed openstack/nova master: libvirt: Extract method _guest_add_video_device https://review.openstack.org/470783 | 20:18 |
| openstackgerrit | Maciej Kucia proposed openstack/nova master: libvirt: Extract method _guest_add_memory_balloon https://review.openstack.org/470786 | 20:18 |
| openstackgerrit | Maciej Kucia proposed openstack/nova master: libvirt: Extract method _guest_add_watchdog_action https://review.openstack.org/470785 | 20:18 |
| openstackgerrit | Maciej Kucia proposed openstack/nova master: libvirt: Extract method _guest_add_pci_devices https://review.openstack.org/470784 | 20:18 |
| melwitt | does anyone know if it matter whether you give a sqlalchemy in_() call a list or a set? is there any benefit in a set vs a list? | 20:18 |
| melwitt | *matters | 20:19 |
| *** sbezverk has joined #openstack-nova | 20:19 | |
| *** Jack_Iv has quit IRC | 20:22 | |
| *** cheran has joined #openstack-nova | 20:23 | |
| *** owalsh is now known as owalsh-pto | 20:23 | |
| bauzas | melwitt: AFAIK, it just needs a container | 20:25 |
| bauzas | ie. anything having an __in__ | 20:26 |
| melwitt | bauzas: yeah, I know it accepts list, tuple, set. I was just curious if there was a benefit of any over others. I didn't see anything when I googled so far | 20:26 |
| dansmith | isn't it generating a text query? | 20:27 |
| dansmith | such that you need iter() not __in__ ? | 20:27 |
| dansmith | in that case, I can't imagine it would matter, other than that a set avoids potential dupes | 20:27 |
| bauzas | mmmm, http://docs.sqlalchemy.org/en/latest/core/sqlelement.html#sqlalchemy.sql.operators.ColumnOperators.in_ | 20:28 |
| melwitt | yeah, that would make sense why there no mention about it mattering | 20:28 |
| bauzas | it just says lists and tuples | 20:28 |
| melwitt | *there's. guh, I can't type today | 20:28 |
| bauzas | honestly, I think it's not a big deal | 20:28 |
| melwitt | I assumed it's not a big deal, I was just curious about it | 20:28 |
| * bauzas shrugs | 20:29 | |
| bauzas | melwitt: if it accepts sets, then cool, as dansmith said | 20:30 |
| melwitt | I know. I know what it accepts. I was curious how it worked and decided to ask if anyone happened to know | 20:30 |
| bauzas | for dupes I mean | 20:30 |
| *** mnestratov|3 has joined #openstack-nova | 20:30 | |
| bauzas | unrelated, could someone tell me the cases where when we have a live-migration, the instance can be shutoff ? | 20:31 |
| bauzas | AFAIK, when it's timing out, it's not shut the instance off, right? | 20:32 |
| *** Apoorva_ has joined #openstack-nova | 20:33 | |
| *** cleong has quit IRC | 20:33 | |
| *** jmlowe has joined #openstack-nova | 20:35 | |
| dansmith | bauzas: afaik live migration only works on a running instance, and if you want to live migrate it you need to start it paused to do so | 20:35 |
| *** Apoorva has quit IRC | 20:36 | |
| openstackgerrit | Dan Smith proposed openstack/nova master: Fix lookup of instance mapping in metadata set-password https://review.openstack.org/472405 | 20:36 |
| *** purplerbot has quit IRC | 20:36 | |
| bauzas | dansmith: sorry, the question is about the output of a live-migration | 20:37 |
| bauzas | dansmith: I'm trying to understand https://bugs.launchpad.net/nova/+bug/1696025 | 20:37 |
| openstack | Launchpad bug 1696025 in OpenStack Compute (nova) "live-migration sometimes failed" [Undecided,New] | 20:37 |
| bauzas | but I'll ask for more details | 20:37 |
| * dansmith shrugs | 20:37 | |
| *** purplerbot has joined #openstack-nova | 20:38 | |
| *** hamzy has quit IRC | 20:38 | |
| *** Apoorva_ has quit IRC | 20:41 | |
| *** Apoorva has joined #openstack-nova | 20:41 | |
| *** karimb has quit IRC | 20:42 | |
| *** jamesdenton has quit IRC | 20:42 | |
| *** Dksad has joined #openstack-nova | 20:44 | |
| openstackgerrit | Merged openstack/nova master: Add PCIWeigher https://review.openstack.org/379524 | 20:44 |
| *** salv-orl_ has joined #openstack-nova | 20:44 | |
| openstackgerrit | Steve Noyes proposed openstack/nova master: WIP: update live migration to use v3 cinder api https://review.openstack.org/463987 | 20:46 |
| *** Dksad has left #openstack-nova | 20:47 | |
| *** pchavva has quit IRC | 20:47 | |
| *** salv-orlando has quit IRC | 20:47 | |
| *** cdent has joined #openstack-nova | 20:51 | |
| *** slaweq has quit IRC | 20:52 | |
| *** slaweq has joined #openstack-nova | 20:53 | |
| *** takashin has joined #openstack-nova | 20:54 | |
| *** tidwellr has quit IRC | 20:54 | |
| melwitt | this is weird, api sample test fail because of the v2.1 in the url? http://logs.openstack.org/59/457859/17/check/gate-nova-tox-functional-py35-ubuntu-xenial/6e6bd37/testr_results.html.gz | 20:56 |
| *** tidwellr has joined #openstack-nova | 20:56 | |
| mriedem | dansmith: you can live migrate a paused instance | 20:57 |
| mriedem | we have a tempest test for it | 20:57 |
| dansmith | mriedem: right | 20:57 |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: WIP: Handle uuids in os-hypervisors API https://review.openstack.org/471899 | 20:58 |
| *** slaweq has quit IRC | 20:58 | |
| sdague | melwitt: that actually looks like the template didn't fill for some reason | 20:58 |
| *** Apoorva_ has joined #openstack-nova | 20:59 | |
| melwitt | sdague: oh, I see. so it shouldn't be showing the placeholders | 20:59 |
| sdague | no, or they should match | 20:59 |
| *** dtp_ has joined #openstack-nova | 21:00 | |
| openstackgerrit | Corey Bryant proposed openstack/nova master: Pass CLI options to nova-placement-api https://review.openstack.org/472378 | 21:00 |
| mriedem | meeting time | 21:00 |
| melwitt | yeah. everything looks to match, except possibly the "v2.1" part. but that parameter is called "versioned_compute_endpoint" so it seems like it would encompass the v2.1 | 21:00 |
| *** tidwellr has quit IRC | 21:01 | |
| *** Apoorva has quit IRC | 21:02 | |
| openstackgerrit | melanie witt proposed openstack/nova master: Add FixedIPList.get_count_by_project() https://review.openstack.org/446246 | 21:02 |
| openstackgerrit | melanie witt proposed openstack/nova master: Add FloatingIPList.get_count_by_project() https://review.openstack.org/446247 | 21:02 |
| openstackgerrit | melanie witt proposed openstack/nova master: Add get_count_by_vm_state() to InstanceList object https://review.openstack.org/446244 | 21:02 |
| openstackgerrit | melanie witt proposed openstack/nova master: Add SecurityGroupList.get_counts() https://review.openstack.org/446245 | 21:02 |
| openstackgerrit | melanie witt proposed openstack/nova master: Remove 'reserved' count from used limits https://review.openstack.org/446242 | 21:02 |
| openstackgerrit | melanie witt proposed openstack/nova master: Remove useless quota_usage_refresh from nova-manage https://review.openstack.org/446243 | 21:02 |
| *** thorst_afk has quit IRC | 21:02 | |
| openstackgerrit | melanie witt proposed openstack/nova master: Count networks to check quota https://review.openstack.org/446241 | 21:02 |
| openstackgerrit | melanie witt proposed openstack/nova master: Count instances to check quota https://review.openstack.org/416521 | 21:02 |
| openstackgerrit | melanie witt proposed openstack/nova master: Add InstanceGroup._remove_members_in_db https://review.openstack.org/457858 | 21:02 |
| openstackgerrit | melanie witt proposed openstack/nova master: Count server group members to check quota https://review.openstack.org/457859 | 21:02 |
| openstackgerrit | melanie witt proposed openstack/nova master: Count security groups to check quota https://review.openstack.org/457860 | 21:02 |
| openstackgerrit | melanie witt proposed openstack/nova master: Make Quotas object favor the API database https://review.openstack.org/410945 | 21:02 |
| openstackgerrit | melanie witt proposed openstack/nova master: Count fixed ips to check quota https://review.openstack.org/457861 | 21:02 |
| openstackgerrit | melanie witt proposed openstack/nova master: Add online migration to move quotas to API database https://review.openstack.org/410946 | 21:02 |
| openstackgerrit | melanie witt proposed openstack/nova master: Count floating ips to check quota https://review.openstack.org/457862 | 21:03 |
| openstackgerrit | Balazs Gibizer proposed openstack/nova master: Transform instance.live_migration_rollback notification https://review.openstack.org/402124 | 21:03 |
| *** thorst_afk has joined #openstack-nova | 21:04 | |
| openstackgerrit | Balazs Gibizer proposed openstack/nova master: Transform instance.soft_delete notifications https://review.openstack.org/460029 | 21:04 |
| *** xyang1 has joined #openstack-nova | 21:05 | |
| bauzas | cdent: you could be interested in https://bugs.launchpad.net/nova/+bug/1696830 | 21:06 |
| openstack | Launchpad bug 1696830 in OpenStack Compute (nova) "nova-placement-api default config files is too strict" [Low,In progress] - Assigned to Corey Bryant (corey.bryant) | 21:06 |
| bauzas | cdent: I'm honestly not opiniated but given oslo.config supports this, placement should accept that too | 21:07 |
| cdent | bauzas: I don't have much of an opinion either. sdague wrote that code if I remember right. | 21:07 |
| * cdent looks at the patch | 21:08 | |
| cdent | hmm, not clear on how that fix will work | 21:08 |
| *** thorst_afk has quit IRC | 21:09 | |
| *** lucasxu has quit IRC | 21:11 | |
| *** Sukhdev has quit IRC | 21:14 | |
| bauzas | cdent: seems like the patch is not fully done yet | 21:17 |
| bauzas | cdent: just pointed you the bug report rather | 21:17 |
| *** Sukhdev has joined #openstack-nova | 21:17 | |
| openstackgerrit | Merged openstack/nova master: Add get_minimum_version_all_cells() helper for service https://review.openstack.org/471431 | 21:17 |
| openstackgerrit | Merged openstack/nova master: Fix device metadata service version check for multiple cells https://review.openstack.org/471432 | 21:18 |
| openstackgerrit | Merged openstack/nova master: Move rebuild notification tests into separate method https://review.openstack.org/451789 | 21:19 |
| *** Sukhdev has quit IRC | 21:21 | |
| *** tbachman has quit IRC | 21:25 | |
| *** rfolco has quit IRC | 21:26 | |
| *** esberglu has quit IRC | 21:27 | |
| *** thorst_afk has joined #openstack-nova | 21:27 | |
| *** esberglu has joined #openstack-nova | 21:27 | |
| efried | Can anyone explain why e.g. https://review.openstack.org/#/c/471796/ is showing a merge conflict? | 21:29 |
| efried | It appears to be right at the tip of the newton branch. | 21:29 |
| bauzas | efried: I need to rebase it | 21:29 |
| efried | bauzas I tried that. It's already where it should be, as far as I can tell. | 21:30 |
| efried | There are several like this. | 21:30 |
| bauzas | efried: lemme test again | 21:31 |
| *** thorst_afk has quit IRC | 21:31 | |
| *** esberglu has quit IRC | 21:32 | |
| mikal | The CI system was telling lies yesterday | 21:32 |
| mriedem | yes it was | 21:33 |
| efried | Yeah, but how do we get it to straighten up? | 21:33 |
| bauzas | mikal: mmm, so recheck then ? | 21:33 |
| mriedem | efried: just needs a recheck | 21:34 |
| mriedem | which i just did | 21:34 |
| efried | k, cool | 21:34 |
| * bauzas facepalms | 21:34 | |
| bauzas | just did too | 21:34 |
| *** jwnx has quit IRC | 21:34 | |
| mriedem | NOOOOOOO | 21:34 |
| mriedem | double recheck | 21:34 |
| mriedem | sends it to the black hole | 21:35 |
| bauzas | recheck recheck recheck | 21:35 |
| bauzas | (candyman) | 21:35 |
| jgriffith | mriedem FYI, it appears that cinder-migrate is broken anyway and would seem that it has been for some time now. | 21:36 |
| mriedem | jgriffith: great | 21:36 |
| mriedem | jgriffith: not sure if scottda's tempest changes to test retype in a multinode job ever landed | 21:36 |
| mriedem | but he was working on that | 21:37 |
| jgriffith | mriedem not sure.. but anyway, I started on this: https://docs.google.com/document/d/10IfIeZ4luPbPOd2oi57FByQ5ByN223KN4j81EskOIM0/edit# | 21:37 |
| jgriffith | ildikov ^^ | 21:37 |
| ildikov | jgriffith: thank you! | 21:38 |
| jgriffith | and I was thinking.. "hey, we don't update the bdm on the volume-id swap nonsense it seems". so tried it out and, well... it's borked | 21:38 |
| jgriffith | anyway... back at it, let ya know if/when I make some progress | 21:38 |
| *** tbachman has joined #openstack-nova | 21:38 | |
| *** fragatina has joined #openstack-nova | 21:38 | |
| jgriffith | it may work using the retype path... I'll try that next | 21:39 |
| mriedem | melwitt: so the IN thing is all good? | 21:40 |
| mriedem | jgriffith: i figured retype was the same from a nova pov | 21:40 |
| melwitt | mriedem: looks like it, yeah. worked in the db func test. the only thing I wasn't 100% about is what to use for the synchronize_session parameter, most of the bulk deletes I found use False | 21:40 |
| mriedem | maybe a zzzeek question | 21:41 |
| melwitt | has to do with whether anything else in the session expects to use those objects after deletion (whether to remove them from session after delete) but I thought that shouldn't be the case that anything else is using the session | 21:41 |
| mriedem | s/maybe/definitely/ s/question/conjuring/ | 21:41 |
| jgriffith | mriedem nope, sadly... the retype uses a different rpc call to get to the manager | 21:42 |
| jgriffith | anyway | 21:42 |
| *** smatzek has quit IRC | 21:42 | |
| jgriffith | f'ing shit code | 21:43 |
| jgriffith | just incase I haven't mentioned my distaste a hundred times already | 21:43 |
| mriedem | heh | 21:45 |
| *** mriedem is now known as mriedem_afk | 21:48 | |
| *** mriedem_afk has quit IRC | 21:48 | |
| *** catintheroof has quit IRC | 21:50 | |
| *** tbachman has quit IRC | 21:53 | |
| *** slaweq has joined #openstack-nova | 21:54 | |
| *** xyang1 has quit IRC | 21:57 | |
| *** slaweq has quit IRC | 21:59 | |
| *** xyang1 has joined #openstack-nova | 21:59 | |
| *** adisky__ has quit IRC | 22:00 | |
| *** edmondsw has quit IRC | 22:02 | |
| *** baoli has quit IRC | 22:02 | |
| *** baoli has joined #openstack-nova | 22:02 | |
| *** luos has quit IRC | 22:04 | |
| *** lyarwood has quit IRC | 22:04 | |
| *** luos has joined #openstack-nova | 22:04 | |
| *** lyarwood has joined #openstack-nova | 22:04 | |
| *** mnestratov|3 has quit IRC | 22:05 | |
| *** tbachman has joined #openstack-nova | 22:05 | |
| *** dave-mccowan has quit IRC | 22:05 | |
| scottda | mriedem this tempest test merged: https://review.openstack.org/#/c/326681/ | 22:06 |
| scottda | and this project-config: https://review.openstack.org/#/c/330678/ | 22:06 |
| *** FL1SK has quit IRC | 22:07 | |
| *** baoli has quit IRC | 22:07 | |
| *** mdrabe has quit IRC | 22:11 | |
| *** salv-orl_ has quit IRC | 22:19 | |
| *** salv-orlando has joined #openstack-nova | 22:20 | |
| *** Apoorva_ has quit IRC | 22:21 | |
| *** jdurgin has quit IRC | 22:21 | |
| *** cheran has quit IRC | 22:21 | |
| *** Apoorva has joined #openstack-nova | 22:21 | |
| *** cdent has quit IRC | 22:22 | |
| cfriesen | how do I restart services in devstack now? | 22:22 |
| cfriesen | never mind, I was looking for the wrong name in the systemd service list | 22:23 |
| *** salv-orlando has quit IRC | 22:24 | |
| *** esberglu has joined #openstack-nova | 22:29 | |
| *** jdurgin has joined #openstack-nova | 22:30 | |
| *** dtp_ has quit IRC | 22:32 | |
| *** esberglu has quit IRC | 22:33 | |
| *** xyang1 has quit IRC | 22:39 | |
| *** jwcroppe has quit IRC | 22:39 | |
| *** liangy has quit IRC | 22:51 | |
| *** mingyu has joined #openstack-nova | 22:53 | |
| *** tonygunk has quit IRC | 22:53 | |
| *** slaweq has joined #openstack-nova | 22:55 | |
| openstackgerrit | melanie witt proposed openstack/nova master: Use ConsoleConnection object to generate authorizations https://review.openstack.org/325414 | 22:57 |
| openstackgerrit | melanie witt proposed openstack/nova master: Convert websocketproxy to use db for token validation https://review.openstack.org/333990 | 22:57 |
| *** mingyu has quit IRC | 22:58 | |
| *** slaweq has quit IRC | 23:00 | |
| *** rajathagasthya has quit IRC | 23:00 | |
| *** tonygunk has joined #openstack-nova | 23:01 | |
| *** itlinux_ has quit IRC | 23:03 | |
| *** rajathagasthya has joined #openstack-nova | 23:04 | |
| *** felipemonteiro__ has quit IRC | 23:05 | |
| *** fragatina has quit IRC | 23:07 | |
| *** sdague has quit IRC | 23:08 | |
| openstackgerrit | Merged openstack/nova master: Count server groups to check quota https://review.openstack.org/446240 | 23:09 |
| *** armax_ has joined #openstack-nova | 23:12 | |
| *** egonzalez has quit IRC | 23:12 | |
| *** Hosam has joined #openstack-nova | 23:14 | |
| *** armax has quit IRC | 23:14 | |
| *** armax_ is now known as armax | 23:14 | |
| *** claudiub has quit IRC | 23:15 | |
| *** d9k has quit IRC | 23:16 | |
| *** d9k has joined #openstack-nova | 23:17 | |
| *** Jack_Iv has joined #openstack-nova | 23:18 | |
| *** pewp has quit IRC | 23:21 | |
| *** pewp has joined #openstack-nova | 23:22 | |
| *** Jack_Iv has quit IRC | 23:22 | |
| *** rajathagasthya has quit IRC | 23:25 | |
| *** Apoorva has quit IRC | 23:25 | |
| *** rajathagasthya has joined #openstack-nova | 23:28 | |
| *** awaugama has joined #openstack-nova | 23:28 | |
| *** mhenkel has quit IRC | 23:32 | |
| *** armax_ has joined #openstack-nova | 23:33 | |
| *** armax has quit IRC | 23:33 | |
| *** armax_ is now known as armax | 23:33 | |
| *** markvoelker has quit IRC | 23:34 | |
| *** sbezverk has quit IRC | 23:38 | |
| *** Apoorva has joined #openstack-nova | 23:40 | |
| *** rajathagasthya has quit IRC | 23:43 | |
| *** jaypipes has joined #openstack-nova | 23:48 | |
| *** jaypipes has quit IRC | 23:48 | |
| *** tonygunk has quit IRC | 23:53 | |
| *** chyka has quit IRC | 23:56 | |
| *** slaweq has joined #openstack-nova | 23:56 | |
| *** harlowja has joined #openstack-nova | 23:57 | |
| *** jamielennox is now known as jamielennox|away | 23:57 | |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!