| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2018-08-24 | |||
| 17:31:51 | mnaser | i dont think things that break after unrelated nova issues would be priority number 1, imho | |
| 17:32:28 | melwitt | thanks for the understanding | |
| 17:32:49 | melwitt | dansmith: that's the first time I've seen that | |
| 17:33:39 | melwitt | hm, weird FileNotFoundError: [Errno 2] No such file or directory: '/home/zuul/src/git.openstack.org/openstack/nova/instances/.directio.test' | |
| 17:34:22 | melwitt | it tries to write a file to test for directio support | |
| 17:35:38 | melwitt | and that failed. does that mean the nova/instances/ dir missing from the test tree probably? hm | |
| 17:36:15 | dansmith | just not sure what would have changed | |
| 17:36:32 | melwitt | I can't remember if that was one of those wonky directories that get created during test runs, to put temp files in for tests | |
| 17:37:08 | melwitt | like, what is that instances/ directory, I can't remember | |
| 17:37:59 | melwitt | oh wait, it's that the actual code is trying to write that file in a real life scenario and in the test env there is no nova/instances dir. but I don't know how that used to be working if it was working before | |
| 17:38:29 | melwitt | in func test land | |
| 17:46:38 | melwitt | CONF.instances_path | |
| 18:05:36 | jrock_cfdg | could someone please help out? Trying to add a serial port with source mode='connect' to do low-level kernel debugging in windows instance; everytime i add device to instance it shuts down - is this even possible with nova? i can do this in libvirt w/out issue | |
| 18:08:20 | Sundar_ | fried_rice, efried: Do you have a moment? | |
| 18:08:41 | fried_rice | Sundar_: sure. | |
| 18:08:51 | openstackgerrit | Merged openstack/nova master: Make CELL_TIMEOUT a constant https://review.openstack.org/594570 | |
| 18:08:55 | fried_rice | Sundar_: I *am* trying to get to reviewing your spec really soon | |
| 18:09:09 | fried_rice | Sundar_: I've just been inundated with other things. | |
| 18:09:21 | Sundar_ | In the scheduling spec, we defined a way to extend the granular resource request syntax to support Cyborg: https://git.openstack.org/cgit/openstack/cyborg/tree/doc/specs/rocky/cyborg-nova-sched.rst?h=refs/changes/17/554717/18#n315 | |
| 18:09:43 | Sundar_ | Just want to make sure you have reviewed it. | |
| 18:10:30 | Sundar_ | fied_rice: NP :) I understand. We all have to roll on different tracks at the same time. | |
| 18:11:18 | Sundar_ | And I may have added yet another track for you to roll on. | |
| 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 | |