| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2017-08-01 | |||
| 10:54:20 | bauzas | cdent: ah | |
| 10:54:32 | bauzas | cdent: yeah, I know that but I do wonder if we test those multihost | |
| 10:54:46 | cdent | yes, multihost | |
| 10:54:51 | bauzas | ok | |
| 10:55:36 | bauzas | cdent: I see your comment now, you mean those test exist but aren't really stress-testing at limits so we can't really verify capacity issues | |
| 10:55:42 | bauzas | tests* | |
| 10:56:09 | bauzas | so, not a big deal for functional testing, but not for placement-specific concerns | |
| 10:56:15 | bauzas | gotcha | |
| 10:58:23 | gibi | cdent, sdague: I'm still intended to understand and remove the last time.sleep(1) left in the setUp() of that tests as that is ugly and might not be necessary | |
| 11:05:32 | openstackgerrit | Merged openstack/nova master: fix test_rebuild_server_exc instability https://review.openstack.org/487382 | |
| 11:08:54 | gibi | OK I think that sleep was only needed before the fake virt driver was updated to keep a local nodes copy and the periodic tasks was running every second | |
| 11:09:21 | gibi | that two things together caused a race on setting the nodes of the fake virt driver | |
| 11:09:25 | sdague | gibi: yeh, I just went through with a fine toothed comb there and provided some import | |
| 11:09:28 | sdague | input | |
| 11:09:57 | gibi | sdague: thanks. checking... | |
| 11:10:13 | sdague | gibi: mostly also bringing fresh eyes of not being familiar with some of this, so some code restructure for clarity that might help as well | |
| 11:11:46 | gibi | sdague: sure, fresh eyes helps a lot | |
| 11:12:39 | openstackgerrit | Merged openstack/nova master: Fix example in _serialize_allocations_for_consumer https://review.openstack.org/487614 | |
| 11:16:22 | openstackgerrit | Merged openstack/nova master: rootwrap.d cleanup mislabeled files https://review.openstack.org/486831 | |
| 11:30:38 | sdague | gibi / cdent - https://review.openstack.org/#/c/487327 | |
| 11:30:47 | sdague | can we fix wsgi_intercept instead? | |
| 11:31:12 | sdague | because if we don't pass those vars, people can't build their venvs | |
| 11:34:51 | openstackgerrit | Gábor Antal proposed openstack/nova master: Transform instance.rebuild_scheduled notification https://review.openstack.org/473929 | |
| 11:35:09 | gibi | sdague: I have an env that is behind proxy and I can build the env there without passing the PROXY wars | |
| 11:35:54 | gibi | sdague: But if this can be fixed in wsgi_intercept then I'm happy with that solution | |
| 11:36:08 | openstackgerrit | Merged openstack/nova master: Instance remains in migrating state forever https://review.openstack.org/483911 | |
| 11:36:10 | sdague | gibi: not everyone can - https://review.openstack.org/#/c/189569/ | |
| 11:36:29 | sdague | it was added because without it people were blocked from running tests | |
| 11:37:07 | openstackgerrit | Gábor Antal proposed openstack/nova master: Transform instance.rebuild_scheduled notification https://review.openstack.org/473929 | |
| 11:37:09 | sdague | unless tox itself changed since then | |
| 11:37:32 | jaypipes | cdent: morning. | |
| 11:37:42 | cdent | sdague: I think tox has changed | |
| 11:38:10 | cdent | it’s difficult to fix in the wsgi_intercept because of the way urllib3 manages proxy variables very early in its handling | |
| 11:38:15 | cdent | (at least last time I checked) | |
| 11:38:18 | cdent | jaypipes: morning | |
| 11:38:20 | jaypipes | cdent: unfortunately, trying to fix bug #1707071 has been excruciating. Looks like I'm going to need to go back to the drawing board and rewrite my patch pretty much from scratch. | |
| 11:38:20 | openstack | bug 1707071 in OpenStack Compute (nova) "Compute nodes will fight over allocations during migration" [Medium,In progress] https://launchpad.net/bugs/1707071 - Assigned to Jay Pipes (jaypipes) | |
| 11:38:29 | cdent | jaypipes: oh noes! | |
| 11:38:30 | sdague | ok, so we need to figure out when it changed, because our minimum is 2.0 | |
| 11:38:40 | cdent | sdague: i’ll look in the changelogs | |
| 11:38:59 | sdague | gibi: if you can confirm when tox changed, and bump the minimum at the same time, I'm +2 | |
| 11:39:00 | gibi | cdent: thanks for taking that | |
| 11:39:10 | gibi | sdague: good point | |
| 11:39:14 | sdague | I just don't want to break folks | |
| 11:39:44 | cdent | jaypipes: is there a crucial bit, or is it many things combined? | |
| 11:41:30 | cdent | sdague, gibi: 2.1.0: http://tox.readthedocs.io/en/latest/changelog.html#id12 | |
| 11:42:12 | gibi | cdent: thanks! I will update the patch with the mimimum bump soon | |
| 11:43:04 | jaypipes | cdent: the latter | |
| 11:43:18 | jaypipes | cdent: and the fact that we need to deal with Ocata computes migrating to Pike computes. | |
| 11:43:55 | cdent | jaypipes: do you have an idea/plan or still cogitating? anything I can do to help? | |
| 11:43:56 | sdague | gibi: I can actually bump it if you like, you are working on more important patches | |
| 11:44:54 | jaypipes | cdent: not on this bug, no. but in debugging, I did notice that the aggregate refresh rate in the resource tracker/report client was super high. I know you'd submitted a bug on that (or related) some time ago. might be nice to look at reducing the number of those calls. | |
| 11:44:57 | openstackgerrit | Sean Dague proposed openstack/nova master: do not pass proxy env variables by tox https://review.openstack.org/487327 | |
| 11:45:08 | sdague | gibi: fixed + approved | |
| 11:46:13 | openstackgerrit | Merged openstack/python-novaclient master: Remove run_tests.sh https://review.openstack.org/488276 | |
| 11:46:46 | cdent | jaypipes: it’s on the cycle of the periodic job isn’t it? Or is it happening even more than that? | |
| 11:47:09 | jaypipes | cdent: it happens every time reportclient._ensure_resource_provider() runs | |
| 11:47:14 | jaypipes | cdent: so.. a lot. | |
| 11:47:17 | gibi | sdague: thanks a lot! | |
| 11:47:46 | cdent | jaypipes: ah, okay, yeah. these days we do that even more. will poke. | |
| 11:51:40 | openstackgerrit | Merged openstack/python-novaclient master: Update URLs in docs, comments and setup.cfg https://review.openstack.org/488271 | |
| 12:00:14 | openstackgerrit | Michael Still proposed openstack/nova master: Move execs of tee to privsep. https://review.openstack.org/489438 | |
| 12:00:14 | openstackgerrit | Michael Still proposed openstack/nova master: Read from console ptys using privsep. https://review.openstack.org/489486 | |
| 12:26:11 | openstackgerrit | Balazs Gibizer proposed openstack/nova master: Test resize with placement api https://review.openstack.org/487958 | |
| 12:27:26 | gibi | sdague: I've fixed your comments in https://review.openstack.org/487958 | |
| 12:32:49 | mriedem | gibi: want to skip the notification meeting today? | |
| 12:33:17 | gibi | mriedem: I can chair it but I don't see any pressing issue to discuss | |
| 12:33:57 | gibi | mriedem: therefore I suggested to skip | |
| 12:34:05 | mriedem | seconded | |
| 12:34:34 | gibi | mriedem: Should I send a second mail to the ML about the skip? | |
| 12:34:43 | openstackgerrit | OpenStack Proposal Bot proposed openstack/nova master: Imported Translations from Zanata https://review.openstack.org/477091 | |
| 12:35:21 | mriedem | gibi: up to you | |
| 12:35:54 | gibi | mriedem: then I won't. I don't think anybody besides you and me could be interested | |
| 12:42:10 | jaypipes | gibi: gah. I was just in the process of reworking that resize tests... | |
| 12:45:42 | sdague | mriedem: ... http://logs.openstack.org/81/488381/3/experimental/gate-tempest-dsvm-cells-ubuntu-xenial/b5f4fa1/logs/devstacklog.txt.gz#_2017-08-01_10_56_05_959 ... oh cells munging hostnames | |
| 12:46:10 | sdague | cells v1 that is | |
| 12:47:18 | sdague | we should rename cells v1 to apoptosis | |
| 12:47:52 | mriedem | it's not deprecated yet | |
| 12:47:59 | mriedem | cells v1 is alive and well | |
| 12:48:06 | sdague | alive... sure | |
| 12:48:22 | sdague | mriedem: we really haven't deprecated it yet? | |
| 12:48:51 | sdague | because it's definitely in freeze for everything bug critical fixes, it seems like deprecation should be a thing | |
| 12:48:57 | sdague | even if we don't have a remove date | |
| 12:51:17 | gibi | jaypipes: sorry for that. I read back and only see an opportunity to refactor but not that somebody already started that | |
| 12:52:17 | mriedem | i'm +2 on gibi's test change | |
| 12:52:53 | mriedem | sdague: yes i've been thinking about deprecating cells v1 before we release pike, in case we want to drop it in queens | |
| 12:53:04 | mriedem | b/c we can't drop nova-network until we drop cells v1 and i don't want to lose another cycle on that | |
| 12:54:31 | jaypipes | mriedem: hold up. I'm fixing up a couple more things. | |
| 12:57:43 | openstackgerrit | Merged openstack/nova master: provide interface-scoped nameserver information https://review.openstack.org/467699 | |
| 13:02:32 | bauzas | jaypipes: gibi: I'm confused by the current situation for reviewing, how can I help by looking at first? | |
| 13:03:10 | jaypipes | bauzas: please give me 15 mins to push up another patch on top of gibi's latest test resize patch | |
| 13:03:18 | openstackgerrit | Rodolfo Alonso Hernandez proposed openstack/os-vif master: Read datapath_type from VIF object https://review.openstack.org/474914 | |
| 13:04:16 | bauzas | jaypipes: okay | |
| 13:04:32 | bauzas | jaypipes: I missed your convo yesterday with dan, hoping to understand more the issues | |
| 13:06:41 | mriedem | hooray for neighborhood construction taking out all the power | |
| 13:07:46 | dims | mriedem : yikes | |
| 13:09:11 | openstackgerrit | Matt Riedemann proposed openstack/nova master: Don't warn on expected network-vif-unplugged events https://review.openstack.org/465794 | |
| 13:10:31 | jaypipes | bauzas: you didn't miss much other than me getting super raging. | |
| 13:13:52 | sdague | mriedem: https://review.openstack.org/#/c/487860/ that nova manage change, most of the urls you suggested in the test aren't valid urls | |
| 13:15:26 | openstackgerrit | Ildiko Vancsa proposed openstack/nova master: Implement new attach Cinder flow https://review.openstack.org/330285 | |