| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2017-07-26 | |||
| 21:56:44 | bauzas | and again, sorry for not having been there for half-Pike | |
| 21:56:48 | bauzas | \o | |
| 21:58:36 | bauzas | mriedem: before I leave, I'm torn by https://review.openstack.org/#/c/408955/ | |
| 21:59:16 | mriedem | edleafe: hold the phone, it was happening before the migrate flavors thing http://logs.openstack.org/80/461480/5/check/gate-tempest-dsvm-ironic-ipa-wholedisk-bios-agent_ipmitool-tinyipa-ubuntu-xenial-nv/834477a/logs/screen-n-cpu.txt.gz?level=TRACE#_Jul_19_03_34_01_687299 | |
| 22:02:23 | mriedem | blows up in ironic-api http://logs.openstack.org/80/461480/5/check/gate-tempest-dsvm-ironic-ipa-wholedisk-bios-agent_ipmitool-tinyipa-ubuntu-xenial-nv/834477a/logs/screen-ir-api.txt.gz#_Jul_19_03_34_01_680460 | |
| 22:03:13 | jgriffith | mriedem ildikov added a note here https://review.openstack.org/#/c/330285/106 regarding the Trace showing up in the logs | |
| 22:07:16 | openstackgerrit | Merged openstack/nova master: Updated from global requirements https://review.openstack.org/487473 | |
| 22:09:39 | mriedem | bauzas: torn how | |
| 22:09:40 | mriedem | ? | |
| 22:10:36 | mriedem | there are 3 other patches for that: api, docs and novaclient | |
| 22:10:42 | mriedem | so it's pretty damn late | |
| 22:12:51 | edleafe | mriedem: ah, so _refresh_cache() is the culprit. The migration change just adds that call a bit earlier | |
| 22:13:12 | mriedem | jgriffith: ack | |
| 22:13:26 | mriedem | edleafe: right, like <1 sec earlier | |
| 22:13:28 | mriedem | but still blowing up | |
| 22:13:36 | mriedem | i've adjusted the bug and routed it to ironic | |
| 22:13:51 | edleafe | mriedem: yeah, but we should be more graceful about it | |
| 22:14:00 | dansmith | mriedem: edleafe well, the bug of not migrating nodes we get via hash rebalance is still a thing | |
| 22:14:27 | edleafe | dansmith: agreed | |
| 22:15:26 | openstackgerrit | Eric Fried proposed openstack/nova master: Centralize keystoneauth1 Adapter conf option setup https://review.openstack.org/487581 | |
| 22:19:23 | openstackgerrit | Jay Pipes proposed openstack/nova master: placement: account for move operations in claim https://review.openstack.org/487589 | |
| 22:19:23 | openstackgerrit | Jay Pipes proposed openstack/nova master: claim resources in placement API during schedule() https://review.openstack.org/483566 | |
| 22:19:25 | jaypipes | mriedem, bauzas, dansmith: ding ding ding. round 2. ^^ | |
| 22:19:51 | mriedem | heh, can i be glass joe? | |
| 22:22:48 | openstackgerrit | Nicolas Simonds proposed openstack/nova master: libvirt: add support for virtio-net rx/tx queue sizes https://review.openstack.org/484997 | |
| 22:23:17 | jaypipes | mriedem: I'll work on my eye tattoo. | |
| 22:24:02 | mriedem | heh i guess we forgot about safe_connect on claim_resources | |
| 22:24:40 | openstackgerrit | Merged openstack/nova master: Remove the code related to extension loading from APIRouterV21 https://review.openstack.org/486414 | |
| 22:25:03 | openstackgerrit | Eric Fried proposed openstack/nova master: nova.utils.get_service_url() https://review.openstack.org/458257 | |
| 22:30:05 | openstackgerrit | Matt Riedemann proposed openstack/nova master: Fix example in _serialize_allocations_for_consumer https://review.openstack.org/487614 | |
| 22:30:20 | mriedem | jaypipes: dansmith: so we don't need to handle anything different from the compute RT perspective for a move operation? | |
| 22:36:47 | openstackgerrit | Merged openstack/nova master: Suppress some test warnings https://review.openstack.org/486700 | |
| 22:37:32 | openstackgerrit | Merged openstack/nova master: Remove the useless FakeExt https://review.openstack.org/486415 | |
| 22:51:59 | openstackgerrit | Nicolas Simonds proposed openstack/nova master: libvirt: add support for virtio-net rx/tx queue sizes https://review.openstack.org/484997 | |
| 22:59:11 | dansmith | mriedem: seems like not | |
| 23:14:53 | mikal | melwitt: you still around? | |
| 23:15:34 | mikal | melwitt: tonyb and I are talking about making a quick list of things we'd like to see cleaned up in nova, because I keep forgetting things because I am an old man. tonyb suggested you had an etherpad like this already somewhere? | |
| 23:17:18 | melwitt | I think you probably mean mriedem :) ^ | |
| 23:17:37 | mikal | nah | |
| 23:17:40 | mikal | Its all about you | |
| 23:17:48 | melwitt | haha | |
| 23:18:29 | mikal | Well, it depends if tonyb is lying about you already having an etherpad | |
| 23:18:50 | melwitt | yeah, if I ever knew of one in the past, I've forgotten about it. I'm not aware of a general nova todos etherpad | |
| 23:19:04 | mikal | Ok, I'll just call tonyb a liar in public then | |
| 23:19:05 | melwitt | I wouldn't be surprised if we had one at some point though | |
| 23:19:07 | mikal | I'm ok with that | |
| 23:19:11 | melwitt | haha :) | |
| 23:19:30 | mikal | https://etherpad.openstack.org/p/nova-warts is my very long list of things I'd like to potter around fixing now that I am a hobbyist again | |
| 23:19:58 | melwitt | coolness | |
| 23:22:00 | mikal | melwitt: speaking of which, it would be cool to get https://review.openstack.org/#/c/486831 merged before pike releases too from that series. Its only comments but it gives us a cleanish slate for the cleanup in queens. | |
| 23:22:08 | openstackgerrit | Jay Pipes proposed openstack/nova master: claim resources in placement API during schedule() https://review.openstack.org/483566 | |
| 23:22:08 | openstackgerrit | Jay Pipes proposed openstack/nova master: placement: account for move operations in claim https://review.openstack.org/487589 | |
| 23:22:11 | mikal | melwitt: i.e. please make with the rubber stamping, kthxbye | |
| 23:25:11 | melwitt | mikal: k, will look | |
| 23:28:20 | jaypipes | edleafe: consolidated the dict construction to a single line for you. | |
| 23:29:30 | mikal | melwitt: ta | |
| 23:35:01 | openstackgerrit | Eric Fried proposed openstack/nova master: WIP: Use service catalog (and discovery) for cinder endpoints https://review.openstack.org/487621 | |
| 23:35:32 | efried | mordred ^^ *very* WIP, but could you look and see if this is a remotely sane approach to the cinder conversion? | |
| 23:35:44 | efried | mordred thanks in advance. | |
| 23:38:05 | smcginnis | Ooh, nice. | |
| 23:39:53 | openstackgerrit | Merged openstack/nova master: Netronome SmartNIC Enablement https://review.openstack.org/483459 | |
| 23:40:54 | openstackgerrit | Merged openstack/nova master: Remove the useless extension block_device_mapping_v1 object https://review.openstack.org/486069 | |
| 23:41:44 | openstackgerrit | Merged openstack/nova master: Remove the useless fake ExtensionManager from API unittests https://review.openstack.org/486416 | |
| 23:42:41 | openstackgerrit | Merged openstack/nova master: Skip boot from encrypted volume on Xen+libvirt https://review.openstack.org/481627 | |
| 23:43:24 | openstackgerrit | Merged openstack/nova master: Add helper method for waiting migrations in functional tests https://review.openstack.org/486537 | |
| 23:43:42 | tonyb | melwitt, mikal: let me see if I can find the one I was thinking of. | |
| 23:44:08 | openstackgerrit | Merged openstack/nova master: docstring and unused code removal https://review.openstack.org/487492 | |
| 23:44:49 | openstackgerrit | Merged openstack/nova master: Move the note about '/os-volume_boot' to the correct place https://review.openstack.org/486071 | |
| 23:49:37 | melwitt | tonyb: in rootwrap.d, do the comments do anything to scope the commands or are they just for humans to read? | |
| 23:51:08 | tonyb | melwitt: just for humans | |
| 23:51:18 | melwitt | tonyb: cool, thanks | |
| 23:52:02 | tonyb | melwitt: when doing the privsep stuff in later patches it was annoying to identify things that could be pruned, so I thought I'd mkae it a little easier | |
| 23:52:26 | melwitt | tonyb: yeah, I think I understand now | |
| 23:53:05 | tonyb | melwitt: I'll address your review feedback after the last_bytes patch lands to make the rebase slightly easier if that's cool with you | |
| 23:54:04 | melwitt | tonyb: okay. if the stuff I mentioned is going to be a later cleanup, that's cool. just wanted to make sure I understood | |
| 23:55:21 | tonyb | well one of you comments is taken care of in a later patch the other was on my mental todo list I haven't done more that that. | |
| 23:55:34 | tonyb | I'll reply and then you can make a call. | |
| 23:57:48 | melwitt | tonyb: cool thanks | |
| #openstack-nova - 2017-07-27 | |||
| 00:11:30 | openstackgerrit | Moshe Levi proposed openstack/nova master: hardware offload support for openvswitch https://review.openstack.org/398265 | |
| 00:13:01 | tonyb | mikal, melwitt: the etherpad I was thinking of was https://etherpad.openstack.org/p/nova-low-hanging-fruit which has been migrated to the wiki | |
| 00:13:27 | tonyb | So I guess start a new one? | |
| 00:13:51 | tonyb | I know low-hanging-fruit isn't exactly what we were thinking of but I'm easily confused | |
| 00:15:12 | mikal | tonyb: I have a tweak to the localfs patch I'd like to upload before you do any rebasing please | |
| 00:15:53 | mikal | tonyb: I might upload that now unless you shout no in the next ten seconds | |
| 00:16:00 | tonyb | mikal: I was kinda using the 'I' in the sense mikal or tony sense | |
| 00:16:00 | tonyb | mikal: Sure | |
| 00:16:05 | mikal | LOL | |
| 00:16:12 | mikal | We are one entity now? | |
| 00:16:18 | mikal | That must be very embarassing for you | |
| 00:16:25 | tonyb | mikal: as long as you don't touch the last_bytes review go nuts ;P | |
| 00:16:41 | moshele | mriedem: hi | |
| 00:16:52 | tonyb | mikal: only here where privsep is concerend | |
| 00:17:09 | mikal | Oh sigh. I need to rebase. Please hold. | |
| 00:17:13 | tonyb | mikal: It's more of that 'tonyb took my keyboard and wont give it back pair programming' | |
| 00:17:26 | mikal | tonyb: true that. tonyb doesn't know how to share. | |
| 00:17:40 | tonyb | mikal: that's what my teachers said a school | |
| 00:17:52 | mikal | Heh | |
| 00:18:09 | tonyb | mikal: you might want to hold off until last_bytes merged unless that also needs a fix | |
| 00:18:19 | mikal | That didn't | |
| 00:18:24 | mikal | But it seems to hate merging | |