| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2018-03-28 | |||
| 20:38:34 | openstackgerrit | Dan Smith proposed openstack/nova master: WIP: Honor availability_zone hint via placement https://review.openstack.org/546282 | |
| 20:40:36 | mikal | mriedem: yeah, I can be around to interate on privsep for the next couple of weeks. Noting that both of the next two weeks are short because of easter. | |
| 20:41:26 | openstackgerrit | Merged openstack/nova stable/queens: Avoid exploding if guest refuses to detach a volume https://review.openstack.org/551948 | |
| 20:41:27 | mikal | mriedem: what's blocking removing nova-net? The last couple of privsep patches aren't mergable until nova-net is gone, but I can WIP those or something. It would be nice to get them landed this cycle though. | |
| 20:43:34 | mriedem | mikal: nothing is blocking it | |
| 20:43:44 | mriedem | mikal: but, | |
| 20:43:57 | mriedem | i imagine it's not trivial since a ton of the test code in-tree assumes nova-net | |
| 20:44:05 | mriedem | and that's including non-nova-net things | |
| 20:44:20 | mikal | Is anyone working on it? | |
| 20:44:25 | mriedem | so all of that has to be unwound | |
| 20:44:28 | mriedem | mikal: not that i know of, | |
| 20:44:36 | mriedem | i got the cells v1 job running with neutron and that was step 1 | |
| 20:46:53 | mriedem | which reminds me, | |
| 20:47:01 | mriedem | melwitt: dansmith: efried: i'm out on monday and tuesday | |
| 20:47:22 | dansmith | orly | |
| 20:47:22 | melwitt | k, thanks for the heads up | |
| 20:47:23 | dansmith | I'm out friday for "GENERIC SPRING HOLIDAY" | |
| 20:47:34 | melwitt | me too | |
| 20:47:40 | efried | roger that, mriedem. Means you have to review this bug fix tonight/tomorrow I guess. | |
| 20:49:18 | openstackgerrit | Matt Riedemann proposed openstack/nova master: Log a warning and add nova-status check for old API service versions https://review.openstack.org/557506 | |
| 20:49:19 | mriedem | SamYaple: ^ for your bug from yesterday, i hope i get a bag of jelly beans for this | |
| 20:50:05 | SamYaple | oh snap! | |
| 20:50:33 | SamYaple | that was quick. i promise you regular flavored jelly beans | |
| 20:52:18 | mriedem | is there such a thing as unflavored jelly beans? | |
| 20:52:52 | SamYaple | i said "regular". as opposed to.... other flavors | |
| 20:53:00 | mriedem | oh right | |
| 20:53:01 | mriedem | yeah brachs | |
| 20:53:07 | mriedem | not that new fangled starburts stuff | |
| 20:53:12 | mriedem | *starbursts | |
| 20:53:19 | SamYaple | haha | |
| 20:53:42 | dansmith | starburts? | |
| 20:53:58 | openstackgerrit | Dan Smith proposed openstack/nova master: Make get_allocation_candidates() honor aggregate restrictions https://review.openstack.org/547990 | |
| 20:53:59 | openstackgerrit | Dan Smith proposed openstack/nova master: Add an index on aggregate_metadata.value https://review.openstack.org/555851 | |
| 20:53:59 | openstackgerrit | Dan Smith proposed openstack/nova master: Add AggregateList.get_by_metadata() query method https://review.openstack.org/544728 | |
| 20:54:00 | openstackgerrit | Dan Smith proposed openstack/nova master: Add require_tenant_aggregate request filter https://review.openstack.org/545002 | |
| 20:54:00 | openstackgerrit | Dan Smith proposed openstack/nova master: WIP: Documentation for tenant isolation with placement https://review.openstack.org/557490 | |
| 20:54:01 | openstackgerrit | Dan Smith proposed openstack/nova master: WIP: Honor availability_zone hint via placement https://review.openstack.org/546282 | |
| 20:54:01 | openstackgerrit | Eric Fried proposed openstack/nova master: Use ksa session for cinder microversion check https://review.openstack.org/557508 | |
| 20:54:02 | dansmith | (man I suck) | |
| 20:54:05 | efried | mriedem: ^ | |
| 20:54:08 | efried | Hahaha | |
| 20:54:26 | mriedem | ? | |
| 20:54:38 | mriedem | starburts are burt reynolds flavored candies | |
| 20:54:46 | mriedem | mostly taste like oil and stache | |
| 20:55:02 | mikal | Does anyone else see pep8 failures with master by the way? | |
| 20:55:05 | efried | mriedem: I was pointing out the bug fix patch I just posted. But crossed streams with dansmith, so it looked like I was bringing your attention to the fact that dansmith, by his own admission, sucks. | |
| 20:55:13 | mikal | Running flake8 on all files | |
| 20:55:13 | mikal | ./nova/tests/unit/virt/libvirt/volume/test_quobyte.py:230:18: E211 whitespace before '(' | |
| 20:55:16 | mikal | ./nova/virt/libvirt/guest.py:664:68: F821 undefined name 'unicode' | |
| 20:55:34 | mriedem | mikal: nope | |
| 20:55:41 | mikal | Herm, how odd. | |
| 20:55:41 | dansmith | efried: I ain't swordfightin' with you dude.. ever. | |
| 20:55:58 | openstackgerrit | Merged openstack/nova stable/pike: Avoid exploding if guest refuses to detach a volume https://review.openstack.org/551950 | |
| 20:55:58 | efried | Promises promises. | |
| 20:56:07 | openstackgerrit | Merged openstack/nova stable/queens: Add functional regression test for bug 1746509 https://review.openstack.org/555914 | |
| 20:56:07 | openstack | bug 1746509 in OpenStack Compute (nova) pike "TypeError: Can't upgrade a READER transaction to a WRITER mid-transaction" [High,In progress] https://launchpad.net/bugs/1746509 - Assigned to melanie witt (melwitt) | |
| 20:56:13 | openstackgerrit | Merged openstack/nova stable/queens: Move _make_instance_list call outside of DB transaction context https://review.openstack.org/555915 | |
| 20:56:22 | openstackgerrit | Merged openstack/nova master: tox: Make everything work with Python 3 https://review.openstack.org/556894 | |
| 20:56:52 | openstackgerrit | Merged openstack/nova master: tests: fixes mock autospec usage https://review.openstack.org/447505 | |
| 20:57:28 | openstackgerrit | Merged openstack/nova stable/queens: Save admin password to sysmeta in libvirt driver https://review.openstack.org/550489 | |
| 20:57:51 | mriedem | mordred must have just flushed the zuul toilet | |
| 20:59:57 | melwitt | mriedem, dansmith: pike backports ready2rock https://review.openstack.org/#/c/555058 | |
| 21:00:04 | efried | dansmith: Did you mean to do this or not? https://review.openstack.org/#/c/544730/15..16/nova/scheduler/manager.py | |
| 21:00:19 | efried | Am I just failing to detect sarcasm via gerrit? | |
| 21:00:37 | dansmith | efried: no, I meant I meant to do what mriedem was suggesting | |
| 21:00:43 | efried | ight | |
| 21:00:58 | efried | cause he seems to have taken you at face value also. | |
| 21:02:25 | dansmith | melwitt: why is that patch strikeouted | |
| 21:02:25 | dansmith | ? | |
| 21:03:09 | melwitt | I dunno. usually that means it merged but it didn't merge | |
| 21:03:23 | melwitt | or no, that's the dot that means it merged. | |
| 21:04:05 | melwitt | it's not strikeouted anymore now. not sure what that was | |
| 21:04:35 | efried | Were you looking at the cherry-picks tabbydoo? | |
| 21:04:42 | melwitt | yes | |
| 21:04:50 | melwitt | Related Changes | |
| 21:05:06 | melwitt | so no, but similar | |
| 21:05:41 | efried | The Cherry Picks one (rightly) shows black-dotted patches from the other branches. I don't see black dots under Related Changes. | |
| 21:05:55 | mriedem | dansmith: one more time through the ringer https://review.openstack.org/#/c/547990/ | |
| 21:06:03 | efried | You say temporary glitch. I say it never happened. Need new glasses. | |
| 21:06:10 | melwitt | earlier before mriedem +2ed it, it was strikeouted in the Related Changes tabbydoo | |
| 21:06:27 | mriedem | i saw it | |
| 21:09:21 | mriedem | efried: since we don't have that https failure in devstack, we'll likely need divya to assert it fixes the bug | |
| 21:09:25 | mriedem | i don't have the setup for it | |
| 21:09:50 | efried | mriedem: Roger that. If it looks okay to you at a glance, I'll send it that way. | |
| 21:10:15 | mikal | mriedem: its not just unit tests, look at nova.virt.xenapi.firewall for example. | |
| 21:11:15 | mriedem | mikal: yeah linux_net is going to be mingled into everything | |
| 21:11:51 | mriedem | so i'm not really sure what the best way is to unwind the nova-net stuff since it's mixed into everything | |
| 21:12:00 | mikal | Orbital nukes? | |
| 21:12:25 | mikal | For a start, do we even know if xen people still use the dom0 firewalling stuff? Its copyright 8 years ago now. | |
| 21:12:27 | mriedem | well, what i know i don't want to see is a single giant patch that just deletes everything | |
| 21:12:40 | mriedem | mikal: would have to ask BobBall or jianghuaw_ | |
| 21:12:49 | mikal | I think you just did? | |
| 21:13:56 | cdent | dansmith or mriedem : were either of you aware of jaypipes-away asking about some tweaks to oslo config to know if an option's value was coming from a default or elsewhere (even if the value was the same)? dhellman implemented a feature to support that in oslo_config but it seems to have bad mojo and maybe it can die: https://bugs.launchpad.net/oslo.config/+bug/1759689 | |
| 21:13:56 | openstack | Launchpad bug 1759689 in oslo.config "In 1.6.0 startup can be extremely slow when checking many config items" [Undecided,New] | |
| 21:14:27 | mikal | This nova.virt.firewall thing is news to me and is going to be an issue I think | |
| 21:14:32 | mikal | Its used quite widely. | |
| 21:14:36 | mriedem | cdent: nope | |
| 21:14:39 | cdent | (sorry that's 6.0.0 above, not 1.6.0) | |
| 21:14:52 | mriedem | i saw the ML thread | |
| 21:15:02 | mikal | Oh, its security groups | |