| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2018-08-24 | |||
| 18:12:36 | fried_rice | Sundar_: I remember seeing this and being okay with it, but I would have to do a major context switch to be able to discuss it intelligently at this point. | |
| 18:14:16 | fried_rice | I noticed you said the extended keys aren't interpreted by nova. Presumably that means they're interpreted by cyborg. How does that information get to cyborg? And does it translate to some new/different placement syntax? I assume not, that it just tells cyborg how to do certain things. | |
| 18:17:32 | Sundar_ | fried_rice: Good to know that it is accepted. Yes, they will be interpreted by Cyborg (not sure if os-acc will also interpret it). The extra specs in the request are passed from Nova virt to os-acc to Cyborg. | |
| 18:17:44 | Sundar_ | No new placement APIs needed. | |
| 18:19:18 | melwitt | I have to run to an appointment, will be back later | |
| 18:34:16 | openstackgerrit | Ghanshyam Mann proposed openstack/nova master: Merge security groups extension response into server view builder https://review.openstack.org/585475 | |
| 18:35:49 | openstackgerrit | Ghanshyam Mann proposed openstack/nova master: Merge extended_status extension response into server view builder https://review.openstack.org/592092 | |
| 18:41:41 | openstackgerrit | Merged openstack/nova master: tests: Create functional libvirt test base class https://review.openstack.org/407055 | |
| 18:50:31 | mriedem | dansmith: i haven't seen that functional test failure | |
| 19:00:55 | mriedem | dansmith: umm, could https://review.openstack.org/#/c/407055/ that just merged be a regression? | |
| 19:02:44 | mriedem | or the change below it | |
| 19:02:55 | mriedem | http://logstash.openstack.org/#dashboard/file/logstash.json?query=message%3A%5C%22FileNotFoundError%3A%20%5BErrno%202%5D%20No%20such%20file%20or%20directory%3A%20'%2Fhome%2Fzuul%2Fsrc%2Fgit.openstack.org%2Fopenstack%2Fnova%2Finstances%2F.directio.test'%5C%22%20AND%20tags%3A%5C%22console%5C%22&from=7d | |
| 19:03:28 | mriedem | just started | |
| 19:03:32 | mriedem | so those are probably it | |
| 19:05:16 | mriedem | https://bugs.launchpad.net/nova/+bug/1788922 | |
| 19:05:17 | openstack | Launchpad bug 1788922 in OpenStack Compute (nova) "SRIOVServersTest.test_create_server_with_VF intermittently fails due to "FileNotFoundError: [Errno 2] No such file or directory: '/home/zuul/src/git.openstack.org/openstack/nova/instances/.directio.test'"" [Undecided,New] | |
| 19:07:15 | mriedem | -1ed https://review.openstack.org/#/c/564978/ which it looks like you and melwitt were talking about - the change has no context | |
| 19:07:20 | mriedem | "this fixes something" | |
| 19:14:40 | mriedem | oh you know, this func test might only be failing on one node provider | |
| 19:14:52 | mriedem | but that means we aren't mocking something out properly and accidentally passing on other hosts | |
| 19:14:55 | mriedem | maybe | |
| 19:23:52 | openstackgerrit | Eric Fried proposed openstack/nova master: [placement] split gigantor SQL query, add logging https://review.openstack.org/590041 | |
| 19:49:35 | mriedem | melwitt: question in https://review.openstack.org/#/c/540258/ | |
| 20:09:30 | openstackgerrit | Merged openstack/nova master: Stash the cell uuid on the context when targeting https://review.openstack.org/594571 | |
| 20:11:44 | mriedem | melwitt: have you thought about an agenda/schedule for the ptg? just thought of it since gorka was asking in the thread on cross-cell migration | |
| 20:23:53 | mriedem | dansmith: i must not be understanding this comment https://review.openstack.org/#/c/592698/12/nova/tests/unit/compute/test_multi_cell_list.py@118 | |
| 20:24:34 | dansmith | mriedem: copied from L89 above, and yes | |
| 20:25:35 | dansmith | it means that since key0 is the same on both it should actually sort the two instances by key1 | |
| 20:25:49 | dansmith | but since I copied it for this test and only have one instance it obviously doesn't need to be htere | |
| 20:26:01 | mriedem | yeah... | |
| 20:26:02 | mriedem | i was like, | |
| 20:26:07 | mriedem | but it's only 1 record... | |
| 20:26:37 | mriedem | cleanup patch on top of the series? | |
| 20:27:15 | dansmith | whatever you want | |
| 20:28:17 | dansmith | if we're blocked on that functional test, I might as well just fix it | |
| 20:29:56 | openstackgerrit | Dan Smith proposed openstack/nova master: Batch results per cell when doing cross-cell listing https://review.openstack.org/592698 | |
| 20:29:57 | openstackgerrit | Dan Smith proposed openstack/nova master: List instances from all cells explicitly https://review.openstack.org/593717 | |
| 20:29:58 | openstackgerrit | Dan Smith proposed openstack/nova master: Make instance_list perform per-cell batching https://review.openstack.org/593131 | |
| 20:29:59 | openstackgerrit | Dan Smith proposed openstack/nova master: Record cell success/failure/timeout in CrossCellLister https://review.openstack.org/594265 | |
| 20:30:00 | openstackgerrit | Dan Smith proposed openstack/nova master: Optimize global marker re-lookup in multi_cell_list https://review.openstack.org/594577 | |
| 20:36:11 | mriedem | shouldn't be blocked http://status.openstack.org/elastic-recheck/#1788922 | |
| 20:36:17 | mriedem | but logstash might be behind | |
| 20:36:39 | dansmith | I had both functional jobs fail in one thing so it seemed like it was high frequency but okay | |
| 20:40:48 | mriedem | yeah it might be higher fail rate than logstash/e-r is seeing right now | |
| 20:40:52 | mriedem | or my query is missing something | |
| 20:41:38 | mriedem | hmm, looking at this i think my query is only going to hit py27 jobs | |
| 20:42:57 | mriedem | no i guess its hitting the opposite, only py35 | |
| 20:57:18 | mriedem | +2s up the board | |
| 21:13:49 | openstackgerrit | Eric Fried proposed openstack/nova master: Report client: Real get_allocs_for_consumer https://review.openstack.org/584599 | |
| 21:13:50 | openstackgerrit | Eric Fried proposed openstack/nova master: Report client: get_allocations_for_provider_tree https://review.openstack.org/584648 | |
| 21:13:51 | openstackgerrit | Eric Fried proposed openstack/nova master: Report client: _reshape helper, placement min bump https://review.openstack.org/585034 | |
| 21:13:52 | openstackgerrit | Eric Fried proposed openstack/nova master: Report client: update_from_provider_tree w/reshape https://review.openstack.org/585049 | |
| 21:13:53 | openstackgerrit | Eric Fried proposed openstack/nova master: Compute: Handle reshaped provider trees https://review.openstack.org/576236 | |
| 21:29:16 | openstackgerrit | Matt Riedemann proposed openstack/nova master: api-ref: fix volume attachment update policy note https://review.openstack.org/596489 | |
| 21:47:32 | openstackgerrit | Matt Riedemann proposed openstack/nova master: api-ref: add a warning about calling swap volume directly https://review.openstack.org/596492 | |
| 21:49:34 | openstackgerrit | Eric Fried proposed openstack/nova master: Document no content on POST /reshaper 204 https://review.openstack.org/596494 | |
| 21:53:15 | openstackgerrit | Eric Fried proposed openstack/nova master: Compute: Handle reshaped provider trees https://review.openstack.org/576236 | |
| 21:53:52 | openstackgerrit | Matt Riedemann proposed openstack/nova master: Add functional test for live migrate with anti-affinity group https://review.openstack.org/588935 | |
| 22:07:44 | openstackgerrit | Eric Fried proposed openstack/nova master: Fix race condition in reshaper handler https://review.openstack.org/596497 | |
| 22:15:23 | melwitt | mriedem: thanks. I was going to make a similar comment on the devstack change but now I don't have to | |
| 22:16:07 | mriedem | troll proxy | |
| 22:16:21 | melwitt | haha | |
| 22:17:09 | melwitt | mriedem: yeah, thinking monday I'll try and schedule some times slots for nova/cinder, nova/ironic, nova/neutron (assuming we need that to) | |
| 22:17:11 | melwitt | *too | |
| 22:18:11 | melwitt | originally had in mind to figure it out this week, but this week was too busy with RC3 and other things | |
| 22:18:40 | mriedem | i didn't think we had any nova/ironic stuff except finalizing on resource classes and dropping the old flavor stuff, but that seems to have hit a fan so will probably be a bigger discussion i guess | |
| 22:18:44 | mriedem | i thought we could just drop a lot of code | |
| 22:19:31 | mriedem | jroll: btw do you have that tripleo bug handy? i want to throw it in the ptg etherpad | |
| 22:19:48 | melwitt | I was thinking we'd do thursday as cross project meet day again but I need to find out if mlavalle and jungleboyj are ok with thursday. TheJulia said thursday afternoon is ok for ironic. she told me they have a thing where they want to discuss the possibility of being able to configure subsets of ironic nodes per nova-compute | |
| 22:20:07 | melwitt | failure domain type stuff | |
| 22:20:29 | mlavalle | melwitt: yes, Thursday works for us, as usual | |
| 22:20:39 | melwitt | cool, thanks | |
| 22:21:06 | melwitt | we'll tune what time of day. I haven't scheduled the nova team photo yet | |
| 22:21:13 | mriedem | subsets of ironic nodes per compute... | |
| 22:21:28 | mriedem | so like a mini az? | |
| 22:21:41 | jroll | mriedem: https://bugs.launchpad.net/tripleo/+bug/1787910/ | |
| 22:21:41 | openstack | Launchpad bug 1787910 in tripleo "OVB overcloud deploy fails on nova placement errors" [Critical,Triaged] - Assigned to Marios Andreou (marios-b) | |
| 22:22:06 | jroll | mriedem: I don't think it's a problem for master, as people should be dropping those filters anyway | |
| 22:23:17 | openstackgerrit | Merged openstack/nova-specs master: Repropose reshaper spec for Stein https://review.openstack.org/592650 | |
| 22:23:24 | jungleboyj | melwitt: hey. Thanks for bringing this up. Then to follow up with you today in the day got away from me. | |
| 22:23:25 | mriedem | jroll: ok, added the bug anyway - would be nice to know what the summary of that issue was | |
| 22:23:49 | jungleboyj | melwitt: Thursday is fine for us. In the past we've done it in the morning but can be flexible. | |
| 22:23:51 | jroll | mriedem: agree, I think the override is fine but the filter is doing something funky | |
| 22:23:52 | mriedem | jroll: was tripleo still using the Exact* filters? | |
| 22:24:02 | jroll | mriedem: I think just regular RamFilter | |
| 22:24:19 | jroll | https://review.openstack.org/#/c/596093/5/environments/undercloud.yaml | |
| 22:24:20 | mriedem | ok, did the flavor values have garbage in them? | |
| 22:24:41 | jroll | they had flavor.memory_mb set, with flavor.properties[whatever:memory_mb]=0 | |
| 22:24:44 | melwitt | mriedem: something like that, I think. just sounded like something good for a discussion to get on the same page and figure out what's the best way to approach it | |
| 22:24:56 | melwitt | I don't know that much about it yet | |
| 22:25:08 | mriedem | jroll: ah ok, so the filter say 0 ram on the HostState, | |
| 22:25:10 | mriedem | and the filter rejected the host | |
| 22:25:14 | mriedem | *saw | |
| 22:25:18 | jroll | right | |
| 22:25:50 | mriedem | i bet our docs on those filters don't say to stop using them... | |
| 22:26:03 | mriedem | https://docs.openstack.org/nova/latest/admin/configuration/schedulers.html#ramfilter | |
| 22:26:16 | melwitt | jungleboyj: cool, that's right. on monday I'll schedule our team photo and then guess a time slot for nova/cinder and adjust with you if needed | |
| 22:26:17 | mriedem | nope | |
| 22:27:11 | melwitt | don't you still need them if CachingScheduler? or no? | |
| 22:28:24 | mriedem | yes | |