Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2019-05-29
16:23:56 sean-k-mooney yes that is for hyperv i think
16:23:57 openstackgerrit Stephen Finucane proposed openstack/nova master: Stop handling 'InstanceUnknownCell' exception https://review.opendev.org/651307
16:23:58 openstackgerrit Stephen Finucane proposed openstack/nova master: db: Remove cell APIs https://review.opendev.org/651309
16:23:58 openstackgerrit Stephen Finucane proposed openstack/nova master: Remove unnecessary wrapper https://review.opendev.org/651308
16:23:59 openstackgerrit Stephen Finucane proposed openstack/nova master: Remove 'ComputeManager._reschedule' https://review.opendev.org/660774
16:23:59 openstackgerrit Stephen Finucane proposed openstack/nova master: conf: Remove cells v1 options, group https://review.opendev.org/651310
16:24:00 openstackgerrit Stephen Finucane proposed openstack/nova master: Stop passing 'kwargs' to 'rebuild_instance' https://review.opendev.org/651312
16:24:00 openstackgerrit Stephen Finucane proposed openstack/nova master: Remove cells v1 parameter from 'ComputeTaskAPI.resize_instance' https://review.opendev.org/651311
16:24:01 openstackgerrit Stephen Finucane proposed openstack/nova master: filters: Stop handling cells v1 https://review.opendev.org/651314
16:24:01 openstackgerrit Stephen Finucane proposed openstack/nova master: Stop passing 'delete_type' to 'terminate_instance' https://review.opendev.org/651313
16:24:02 openstackgerrit Stephen Finucane proposed openstack/nova master: Ensure controllers all call super https://review.opendev.org/660950
16:24:02 openstackgerrit Stephen Finucane proposed openstack/nova master: Remove nova.compute.*API() shims https://review.opendev.org/660527
16:24:04 sean-k-mooney for libvirt it already works
16:24:41 phillw that's cool, I'm on nodding terms with libvirt as I use it for my VMs :)
16:25:13 mriedem sean-k-mooney: that patch is for making nvidia work with windows guests in libvirt
16:25:19 mriedem the existing support is for libvirt guests only
16:25:33 sean-k-mooney * linux guests
16:26:05 mriedem i never really understood the opposition to that patch since it's just doing the same thing for windows guests on kvm that we allow for linux guests
16:26:12 mriedem i.e. we've already opened pandora's box there
16:26:23 sean-k-mooney we nack it however as we did not want to add a feature to specically workaound a driver check that is there to prevent you breaching nvidas eula
16:26:38 mriedem *we already do*
16:26:54 sean-k-mooney the hypervior hidden flag was not added for that use however
16:26:54 mriedem https://blueprints.launchpad.net/nova/+spec/hide-hypervisor-id-flavor-extra-spec
16:27:00 sean-k-mooney it was already there
16:27:06 mriedem yes it was
16:27:14 mriedem https://blueprints.launchpad.net/nova/+spec/add-kvm-hidden-feature
16:27:40 mriedem https://review.opendev.org/#/c/459753/18//COMMIT_MSG@14
16:28:03 mriedem so jay was +W on that change, but not this other one to allow the same thing for windows guests
16:28:06 mriedem which i don't understand
16:28:14 sean-k-mooney huh in that case i guess i dont know why we have not made it work for windows guests
16:28:42 sean-k-mooney i had under stood this was not sepcifcally for the nvida case but i guess it was
16:28:45 openstackgerrit Stephen Finucane proposed openstack/nova master: Add reno for removed cells v1 policies https://review.opendev.org/662031
16:30:48 mriedem that said i don't see any cern people around to rev that patch
16:30:49 mriedem or ovh people
16:31:07 mriedem amorin: you're ovh right? do you guys want to revive this? https://review.opendev.org/#/c/579897/
16:31:27 mriedem looks like Maciej and Piotr cared about it at one point
16:32:44 phillw on a not totally dissimilar topic, I was actually paying attention on my last C7 installation and it offers to install a compute instance. Am I correct in my reading that this is a nova instance?
16:33:14 mriedem C7 = centos7?
16:33:23 phillw yes, sorry
16:33:25 mriedem "it offers to install a compute instance. Am I correct in my reading that this is a nova instance?" no idea
16:33:34 jcosmao hi mriedem, i'll forward info to my teammate (i work at ovh also)
16:33:43 mriedem jcosmao: thanks
16:34:05 phillw mriedem: home work for me, then :)
16:34:14 mriedem just make tim do it
16:35:20 openstackgerrit Matt Riedemann proposed openstack/nova master: Refresh instance network info on deletion https://review.opendev.org/660761
16:39:01 openstackgerrit Stephen Finucane proposed openstack/nova master: hacking: Resolve E731 (do not assign a lambda expression, use a def) https://review.opendev.org/651554
16:39:02 openstackgerrit Stephen Finucane proposed openstack/nova master: hacking: Resolve E741 (ambiguous variable name) https://review.opendev.org/652103
16:39:02 openstackgerrit Stephen Finucane proposed openstack/nova master: hacking: Resolve W503 (line break occurred before a binary operator) https://review.opendev.org/651555
16:39:03 openstackgerrit Stephen Finucane proposed openstack/nova master: hacking: Resolve W605 (invalid escape sequence) https://review.opendev.org/652104
16:40:20 aspiers mriedem: does this look familiar to you? b'/home/zuul/src/opendev.org/openstack/nova/.tox/lower-constraints/lib/python3.6/site-packages/pymysql/cursors.py:167: Warning: (1831, "Duplicate index \'block_device_mapping_instance_uuid_virtual_name_device_name_idx\' defined on the table \'fjisjekoia.block_device_mapping\'. This is deprecated and will be disallowed in a future release.")'
16:40:40 aspiers seems completely unrelated to my patch which is failing in the gate
16:40:50 aspiers and I think I've seen it a few times in the last 24 hours
16:43:00 aspiers oh, that's just a warning
16:43:34 aspiers still trying to find the real issue in http://logs.openstack.org/66/661266/4/gate/openstack-tox-lower-constraints/7af4be7/job-output.txt.gz
16:43:42 aspiers I see a bunch of timeouts
16:45:26 openstackgerrit Merged openstack/os-vif stable/stein: Prevent "qbr" Linux Bridge from replying to ARP messages https://review.opendev.org/655678
16:47:06 kashyap aspiers: Remind me again, in 'os-traits' repo, what to do when 'tox' complains too long lines for a URL?
16:47:23 aspiers kashyap: isn't that just a warning?
16:47:36 aspiers if not, I guess # noqa or similar
16:48:05 kashyap aspiers: It moans an ERROR -- "E501 line too long (108 > 79 characters)"
16:48:28 kashyap aspiers: Before I go off shortly, I'm pushing the change. For convenience, I'm including a short "changelog" noting explicitly what I addressed
16:48:55 kashyap ... so that if you want to rebase on it tomm.
16:49:01 aspiers kashyap: http://flake8.pycqa.org/en/3.1.1/user/ignoring-errors.html#in-line-ignoring-errors
16:49:09 aspiers # noqa: E501
16:49:42 kashyap Right, thanks!
16:53:52 mriedem aspiers: that duplicate index thing is super old yes
17:05:10 aspiers mriedem: the failures are very similar to https://bugs.launchpad.net/nova/+bug/1823251, but without the InterfaceError
17:05:11 openstack Launchpad bug 1823251 in OpenStack Compute (nova) "Spike in TestNovaMigrationsMySQL.test_walk_versions/test_innodb_tables failures since April 1 2019 on limestone-regionone" [High,Confirmed]
17:06:12 aspiers I wonder if the InterfaceError is just one symptom of a timeout
17:06:24 aspiers My timeouts show stuff like IndexError: tuple index out of range
17:06:46 aspiers and 'sqlalchemy.exc.ResourceClosedError: This result object does not return rows. It has been closed automatically.'
17:08:26 aspiers ah, that latter one looks similar to https://bugs.launchpad.net/nova/+bug/1793364
17:08:28 openstack Launchpad bug 1793364 in Manila "mysql db opportunistic unit tests timing out intermittently in the gate (bad thread switch?)" [Undecided,In progress] - Assigned to Goutham Pacha Ravi (gouthamr)
17:09:00 mriedem right
17:09:05 mriedem i've been down that road
17:10:41 aspiers I guess elastic recheck will tell me as much when the other gate jobs finish
17:11:20 openstackgerrit Kashyap Chamarthy proposed openstack/os-traits master: hw: cpu: Rework the directory layout; add missing traits https://review.opendev.org/655193
17:12:42 kashyap aspiers: When you get a moment, does the short change log address all things you want --^
17:12:50 aspiers kashyap: already looking at it
17:13:00 kashyap Thank you very much, sir.
17:14:06 kashyap aspiers: Yeah, I wondered, too. Just cargo-culted it.
17:15:04 aspiers +1
17:15:50 kashyap aspiers: Ah, thanks. My focus was in Gerrit, and I saw the "Update" pop-up (your comment). I was expecting a firm -1 :D
17:16:02 aspiers Oh ye of little faith :)
17:16:03 kashyap Okay, now I cook food
17:16:07 aspiers Good plan
17:35:27 ganso lyarwood, melwitt, dansmith: Hello! When you have a minute, could you please take a look at those backports that are missing just a final +2? https://review.opendev.org/#/c/652153 , https://review.opendev.org/#/c/658136 and https://review.opendev.org/#/c/661349 ? Thanks in advance! =)
17:35:30 ganso tonyb: thanks a lot =D
17:49:23 openstackgerrit melanie witt proposed openstack/nova master: Follow up for counting quota usage from placement https://review.opendev.org/662056
18:24:03 melwitt mriedem: replied to your comments on https://review.opendev.org/#/c/638073
18:24:22 melwitt and I dunno if you saw, but it looks like https://review.opendev.org/661391 fixes the novnc tests on the multi-cell job
18:36:34 openstackgerrit Artom Lifshitz proposed openstack/nova master: Run revert resize tests in nova-live-migration https://review.opendev.org/653498
18:36:34 openstackgerrit Artom Lifshitz proposed openstack/nova master: Revert resize: wait for events according to hybrid plug https://review.opendev.org/644881
18:36:35 openstackgerrit Artom Lifshitz proposed openstack/nova master: [DNM] use iptables in nova-multinode https://review.opendev.org/660782
18:37:07 artom mriedem, ^^ when you get some time, assuming unit tests pass (I've fixed all of them, normally). I think sean-k-mooney and I have finally figured it out.
18:37:49 artom The DNM patch is... optional? I'm not sure what we want to do upstream, but it's good to have it on top to show that both hybrid and normal plug work
18:47:53 sean-k-mooney the DNM patch change the job to use iptables. if we wanted to merge it we should add a second job that use iptable rather then changing multinode job to use it instead of the defualt contrack driver
18:49:11 konetzed will the scheduler honor setting ram_weight_multiplier as extra metadata on a host aggregate? The goal is just to have one host aggregate stack vs spread.
18:50:52 sean-k-mooney konetzed: i dont think so https://github.com/openstack/nova/blob/master/nova/scheduler/weights/ram.py
18:51:33 sean-k-mooney https://github.com/openstack/nova/blob/master/nova/scheduler/utils.py#L1089

Earlier   Later