| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2018-08-24 | |||
| 17:25:40 | melwitt | dansmith: oh, geez, okay | |
| 17:25:42 | dansmith | so it's behaving like the combined global api service would have previously when we smashed them all together, | |
| 17:26:02 | dansmith | but with wsgi you yank them into life a little differently, without the nova-api service bootstrapping code | |
| 17:26:19 | dansmith | which means it should be looking up the mapping for instances and talking to the right cell, like in global mode | |
| 17:26:57 | melwitt | cray cray | |
| 17:27:24 | dansmith | calling it "non-standalone" is a bit of a misnomer, but compared to our old combined service bootstrapper thing, it's in "non-standalone" mode of operation | |
| 17:27:33 | dansmith | but.. it's fairly standalone in that it's running all by itself and has its own log file | |
| 17:27:39 | melwitt | dansmith: okay, so what is this then here? this looks like it means if service enabled n-api-meta, then remove the metadata from nova_enabled_apis and run it is _our_ standalone service. right? https://review.openstack.org/#/c/564978/2/lib/nova@470 | |
| 17:27:54 | melwitt | *as | |
| 17:28:07 | dansmith | that's probably now confused, | |
| 17:28:51 | dansmith | as that used to be how devstack referred to *our* standalone service, and may still in some cases, idk, but now post-systemd and post-wsgi probably refers to the, uh, other one :) | |
| 17:29:04 | melwitt | mnaser: we think so, but we also thought we'd only have one RC and now we have 3 | |
| 17:29:22 | mnaser | maybe ican help you make it 4, hah :P | |
| 17:29:24 | dansmith | melwitt: why are you even reviewing this with no details in the bug or commit message? | |
| 17:29:46 | mnaser | thats reasonable. ill proceed being careful and give input :) | |
| 17:30:53 | dansmith | melwitt: mriedem_afk: is this known? http://logs.openstack.org/71/594571/2/gate/nova-tox-functional-py35/fd2d9ac/testr_results.html.gz | |
| 17:31:01 | melwitt | mnaser: haha :) always happy to get your testing of the code, and sorry we've been behind on the cells orphaned records patch reviews. things have been just really crazy | |
| 17:31:16 | dansmith | failed on both jobs of one of my patches | |
| 17:31:32 | mnaser | melwitt: nah its cool, my workaround for that was make sure our underlying infra doesnt break and thats ok enough :p | |
| 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 | |