Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2019-01-31
15:41:21 ShilpaSD sean-k-mooney: ok
15:43:35 jangutter why do I, even when working on simple things, always end up with a minimum of 10 gerrit tabs open? (only about 3 unique reviews in them). Should I seek medical attention?
15:44:36 sean-k-mooney if you use chome you should maybe seek more ram for you laptop
15:44:41 openstackgerrit Eric Fried proposed openstack/os-resource-classes master: Add normalize_name utility https://review.openstack.org/634258
16:16:34 openstackgerrit Jack Ding proposed openstack/nova master: Flavor extra spec and image properties validation https://review.openstack.org/620706
16:17:12 openstackgerrit Jack Ding proposed openstack/nova master: Correct instance port binding for rebuilds https://review.openstack.org/603844
16:19:51 openstackgerrit Jack Ding proposed openstack/nova master: Improve libvirt image and snapshot handling https://review.openstack.org/616692
16:20:31 openstackgerrit Eric Fried proposed openstack/os-resource-classes master: Add normalize_name utility https://review.openstack.org/634258
16:20:32 openstackgerrit Jack Ding proposed openstack/nova master: Preserve UEFI NVRAM variable store https://review.openstack.org/621646
17:25:37 sean-k-mooney jaypipes: mriedem: stephenfin: looks like the ci is finally happy with https://review.openstack.org/#/c/626381 if yet have time to take a look.
17:25:38 openstackgerrit Chris Dent proposed openstack/nova master: Switch to using os-resource-classes https://review.openstack.org/628278
17:26:34 mriedem in the middle of something but i'll leave a tab open
17:27:23 sean-k-mooney no worries.
18:17:17 openstackgerrit Matt Riedemann proposed openstack/nova master: WIP: Add PrepResizeAtDestTask https://review.openstack.org/627890
18:17:17 openstackgerrit Matt Riedemann proposed openstack/nova master: Add prep_snapshot_based_resize_at_dest compute method https://review.openstack.org/633293
18:17:18 openstackgerrit Matt Riedemann proposed openstack/nova master: WIP: Add PrepResizeAtSourceTask https://review.openstack.org/627891
18:19:00 openstackgerrit Eric Fried proposed openstack/nova master: Move retry from _update to _update_to_placement https://review.openstack.org/634252
19:40:44 mriedem sean-k-mooney: comments in https://review.openstack.org/#/c/626381/
19:40:58 mriedem fix is good, just needs polish
19:41:10 sean-k-mooney mriedem: thanks
19:41:30 sean-k-mooney ill fix it up today/tommorow
19:45:05 sean-k-mooney im just leavening of the day but ya all the comments make sense to me
19:53:41 mriedem dansmith: you want to send this backport on (it's your fix) https://review.openstack.org/#/c/633086/
19:55:00 dansmith yeah
20:05:55 openstackgerrit Hang Yang proposed openstack/nova stable/queens: Fix port dns_name reset https://review.openstack.org/633807
20:39:05 openstackgerrit Hang Yang proposed openstack/nova stable/queens: Fix port dns_name reset https://review.openstack.org/633807
20:40:41 openstackgerrit Hang Yang proposed openstack/nova stable/queens: Fix port dns_name reset https://review.openstack.org/633807
20:45:19 mriedem nova meeting in 15 minutes
21:19:42 lbragstad mriedem when is nova specification freeze?
21:19:50 lbragstad or did that already happen?
21:20:48 edleafe lbragstad: that was in early January
21:20:56 mriedem lbragstad: jan 10
21:21:19 lbragstad aha - gotcha, thanks edleafe and mriedem
21:21:39 mriedem lbragstad: i assume you're asking because of https://review.openstack.org/#/c/602201/
21:22:16 mriedem which seemed to enter the summit blackhole and was vaporized
21:22:34 lbragstad correct
21:22:52 mriedem sounds like you need a pop up team mr
21:23:00 mriedem a formal one anyway
21:23:02 lbragstad :)
21:23:25 mnaser someone actually reads [tc] tagged ml posts?
21:23:37 lbragstad i'll see if i can catch johnthetubaguy or jaypipes sometime so we can sync on it at least
21:23:47 lbragstad but i assume the ship has said for stein
21:23:52 lbragstad sailed*
21:23:55 mriedem mnaser: i had a whole reply written and everything
21:23:56 smcginnis mriedem: Glance is very actively battling the subunit parser issues right now too.
21:24:06 mriedem smcginnis: ok i knew it hit glance as well
21:24:25 mriedem i'm hoping one of our fine contributors will pick up the investigation as to what changes are needed in nova for that subnit thing
21:24:29 mriedem *subunit
21:24:33 smcginnis If your interested, can probably ping rosmaita for any latest findings.
21:24:43 mriedem i'm interested in someone else being interested
21:24:54 mriedem i spent enough time on the udhcp..pid bug the last 2 weeks
21:25:01 smcginnis The only way I was able to "fix" it in Cinder was to try to get rid of as much log output as possible from unit test runs.
21:25:33 mriedem so my guess is there is a correlation between py3 deprecate/user warnings and our console logs
21:25:41 mriedem which we've worked on stifling over time
21:25:49 smcginnis Seems likely.
21:25:50 mriedem hell i have a glance patch for that as well...
21:25:57 mriedem but some joker nacked it
21:26:06 mriedem https://review.openstack.org/#/c/625092/
21:26:35 smcginnis Part of the Cinder fix, IIRC, was getting rid of all those warnings.
21:27:26 mriedem ok that's a good tip though, thanks
21:34:35 mriedem b"/home/zuul/src/git.openstack.org/openstack/nova/.tox/py35/lib/python3.5/site-packages/sqlalchemy/orm/evaluator.py:99: SAWarning: Evaluating non-mapped column expression 'id' onto ORM instances; this is a deprecated use case. Please make use of the actual mapped columns in ORM-evaluated UPDATE / DELETE expressions."
21:34:35 mriedem this is probably the worst warning that shows up
21:34:41 mriedem 866 hits in one py35 unit test run
21:36:47 stephenfin mriedem: I'm gone to FOSDEM tomorrow, but I should be able to take on https://blueprints.launchpad.net/nova/+spec/remove-force-flag-from-live-migrate-and-evacuate if alex_xu and gibi aren't available. Mention it to me on Monday?
21:37:16 mriedem sure
22:01:12 openstackgerrit Merged openstack/nova stable/rocky: Fix incompatible version handling in BuildRequest https://review.openstack.org/631731
22:01:21 openstackgerrit Matt Riedemann proposed openstack/nova master: Add microversion to expose virtual device tags https://review.openstack.org/631948
22:39:11 openstackgerrit Matt Riedemann proposed openstack/nova master: Fix race in test_volume_swap_server_with_error https://review.openstack.org/634334
22:39:12 mriedem easy gate bug fix ^
22:58:21 openstackgerrit Jack Ding proposed openstack/nova master: Flavor extra spec and image properties validation https://review.openstack.org/620706
#openstack-nova - 2019-02-01
00:19:00 openstackgerrit Merged openstack/nova master: Make 'plugin' a required argument for '_get_vif_instance' https://review.openstack.org/608279
02:17:05 openstackgerrit Merged openstack/nova stable/rocky: Fix using template cell urls with nova-manage https://review.openstack.org/633086
02:24:50 openstackgerrit Zhenyu Zheng proposed openstack/nova master: Don't change volume status to `in-use` when attach to shelved vm. https://review.openstack.org/627096
09:29:56 openstackgerrit Zhenyu Zheng proposed openstack/nova master: Per-instance serial number https://review.openstack.org/619953
12:07:15 openstackgerrit Lee Yarwood proposed openstack/nova master: WIP libvirt: Avoid premature calls to disconnect_volume during LM https://review.openstack.org/634398
12:07:53 lyarwood mdbooth: ^ if you wouldn't mind taking a look at this when you're back, from a downstream triage bug.
12:15:30 mdbooth lyarwood: Looking
12:17:42 mdbooth lyarwood: Just reading the commit message makes me sad :( That's so racy.
12:17:52 mdbooth Not your patch, the one you reference.
12:18:48 mdbooth That reference counting can only happen on the host, while holding a lock.
12:21:55 lyarwood mdbooth: ack yeah we've spoken about lock many many times in this area
12:22:00 lyarwood locks*
12:22:21 sean-k-mooney os-bricks?
12:22:21 lyarwood mdbooth: tbh this is just a backportable fix, I'd like to follow up and refactor this entire method after this.
12:22:40 lyarwood sean-k-mooney: up a level in n-cpu before we call down.
12:23:27 lyarwood sean-k-mooney: there's reference/connection counting going on before we call disconnect_volume in os-brick that could race in this example.
12:23:35 sean-k-mooney i was just asking as when you mentiond talking about locks several times previously the os-brick interaction conde was what jumped to mind
12:23:56 lyarwood yeah that's part of it, I don't think anything has happened there tbh
12:24:41 sean-k-mooney there was talk of intoducing a context manager to hanel the locking if i remmebre. not sure if mdbooth ever had time to work on it
12:26:58 mdbooth sean-k-mooney: That particular instance requires a lock across connect and disconnect
12:27:46 mdbooth So, for eg, a concurrent connect can't interfere with attachments during the check in disconnect
12:27:58 sean-k-mooney mdbooth: is this related to the down stream issue from yesterday?
12:28:09 mdbooth No. It's Lee's bug.
12:28:22 sean-k-mooney oh ok
12:28:48 lyarwood yeah just a triage bug
12:30:08 sean-k-mooney lyarwood: well it sould like it a valid one if ye are dicussing adding new locking
12:30:38 mdbooth Yeah, pretty sure it's valid.
12:32:27 sean-k-mooney lyarwood: mdbooth im not sure if we have any gate jobs that test livemigration with multiatached volumes
12:33:05 mdbooth Probably not
12:33:28 lyarwood yeah I don't think there's any multihost multiattach testing at the moment

Earlier   Later