| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2018-09-30 | |||
| 06:04:32 | openstackgerrit | Brin Zhang proposed openstack/nova master: Add API samples test for ``volume_type`` https://review.openstack.org/606758 | |
| 07:54:08 | openstackgerrit | Brin Zhang proposed openstack/nova master: Add microversion 2.67 to support volume_type https://review.openstack.org/606398 | |
| 11:48:37 | openstackgerrit | Ghanshyam Mann proposed openstack/nova master: Merge used_limits extension response into limit view builder https://review.openstack.org/606031 | |
| 12:00:59 | openstackgerrit | Ghanshyam Mann proposed openstack/nova master: Merge used_limits extension response into limit view builder https://review.openstack.org/606031 | |
| 13:38:26 | openstackgerrit | Ghanshyam Mann proposed openstack/nova master: Merge image_size extension response into image view builder https://review.openstack.org/606845 | |
| 15:25:49 | openstackgerrit | Matt Riedemann proposed openstack/nova master: Add hide server address tests in test_serversV21.py https://review.openstack.org/605435 | |
| 20:50:04 | openstackgerrit | Merged openstack/nova master: Use INFO for logging no allocation candidates https://review.openstack.org/605765 | |
| 23:08:06 | openstackgerrit | Takashi NATSUME proposed openstack/python-novaclient master: Update the CLI reference https://review.openstack.org/606871 | |
| #openstack-nova - 2018-10-01 | |||
| 00:43:49 | openstackgerrit | Takashi NATSUME proposed openstack/nova master: Add API ref guideline for body text https://review.openstack.org/605628 | |
| 06:24:07 | openstackgerrit | Brin Zhang proposed openstack/nova master: Add microversion 2.67 to support volume_type https://review.openstack.org/606398 | |
| 06:45:54 | openstackgerrit | Sam Morrison proposed openstack/python-novaclient master: Fix up userdata argument to rebuild. https://review.openstack.org/605341 | |
| 07:01:30 | bauzas | good morning Nova | |
| 07:50:54 | openstackgerrit | Christoph Manns proposed openstack/nova stable/rocky: Fix stacktraces with redis caching backend https://review.openstack.org/606895 | |
| 07:52:09 | openstackgerrit | Christoph Manns proposed openstack/nova stable/queens: Fix stacktraces with redis caching backend https://review.openstack.org/606896 | |
| 07:53:46 | openstackgerrit | Brin Zhang proposed openstack/nova master: Add microversion 2.67 to support volume_type https://review.openstack.org/606398 | |
| 09:07:47 | openstackgerrit | Matthew Booth proposed openstack/nova master: Raise error on timeout in wait_for_versioned_notifications https://review.openstack.org/604859 | |
| 09:08:06 | openstackgerrit | Matthew Booth proposed openstack/nova master: Don't delete disks on shared storage during evacuate https://review.openstack.org/578846 | |
| 09:08:06 | openstack | bug 1550919 in OpenStack Compute (nova) "[Libvirt]Evacuate fail may cause disk image be deleted" [Medium,In progress] https://launchpad.net/bugs/1550919 - Assigned to Matthew Booth (mbooth-9) | |
| 09:08:06 | openstackgerrit | Matthew Booth proposed openstack/nova master: Add regression test for bug 1550919 https://review.openstack.org/591733 | |
| 09:11:58 | kashyap | testone nova.tests.unit.virt.libvirt.test_driver.LibvirtConnTestCase.test_next_min_qemu_version_ok | |
| 09:12:47 | kashyap | Err, disregard that | |
| 09:13:14 | openstackgerrit | Theodoros Tsioutsias proposed openstack/nova-specs master: Add PENDING vm state https://review.openstack.org/554212 | |
| 09:13:47 | ttsiouts | johnthetubaguy: I tried to refresh the spec ^ | |
| 10:03:21 | sean-k-mooney | o/ | |
| 10:04:17 | openstackgerrit | Takashi NATSUME proposed openstack/python-novaclient master: Update the contributor guide https://review.openstack.org/606926 | |
| 10:05:40 | sean-k-mooney | stephenfin: when you get a chance can you run your eyes over https://review.openstack.org/#/c/604992/3. looks like an easy addtion to me. | |
| 10:05:48 | stephenfin | ack | |
| 10:06:23 | sean-k-mooney | stephenfin: i think we need to port other vif types from kuyr-kubernetes too but they can be in different patches | |
| 10:35:08 | sean-k-mooney | ralonsoh: hi are you around today? | |
| 10:35:47 | ralonsoh | sean-k-mooney: very well, fighting with DVR and FIP (and SRIOV) | |
| 10:36:52 | stephenfin | sean-k-mooney: Also, "i prefer explcit else clauses" | |
| 10:37:00 | stephenfin | sean-k-mooney: Just wanted you to know you're wrong | |
| 10:37:04 | stephenfin | sean-k-mooney: That is all. | |
| 10:37:05 | stephenfin | :) | |
| 10:37:17 | sean-k-mooney | ralonsoh: if you bussy i can respin https://review.openstack.org/#/c/605422/2 | |
| 10:38:05 | ralonsoh | sean-k-mooney: sure, I'll do this now | |
| 10:38:40 | sean-k-mooney | stephenfin: hehe well we can agree to disagree on that. it can help prevent some types of errors when you are maintaing the fucntion in the future but also consumes indentation space so is not allways desireable | |
| 10:48:35 | openstackgerrit | Stephen Finucane proposed openstack/nova master: conf: Allow 'nova-xvpvncproxy' to be called with CLI args https://review.openstack.org/606929 | |
| 10:48:35 | openstackgerrit | Stephen Finucane proposed openstack/nova master: doc: Rewrite the console doc https://review.openstack.org/606148 | |
| 10:51:27 | openstackgerrit | Rodolfo Alonso Hernandez proposed openstack/os-vif master: Remove IPTools deprecated implementation https://review.openstack.org/605422 | |
| 10:51:45 | ralonsoh | sean-k-mooney: done | |
| 10:55:27 | sean-k-mooney | ralonsoh: thanks :) | |
| 11:11:38 | openstackgerrit | Theodoros Tsioutsias proposed openstack/nova-specs master: Enable rebuild for instances in cell0 https://review.openstack.org/554218 | |
| 11:15:21 | openstackgerrit | Theodoros Tsioutsias proposed openstack/nova-specs master: Add PENDING vm state https://review.openstack.org/554212 | |
| 11:28:36 | openstackgerrit | Merged openstack/nova stable/rocky: Use INFO for logging no allocation candidates https://review.openstack.org/605771 | |
| 11:48:27 | openstackgerrit | Merged openstack/os-vif master: Add support for Windows network commands https://review.openstack.org/487405 | |
| 12:05:59 | kashyap | stephenfin: On that version bump thing for libvirt, I just looked at "what's involved" to clean up | |
| 12:06:15 | kashyap | stephenfin: I need to address at least 10 unit tests. Will get to it this week. Currently buried in something a bit more urgent | |
| 12:30:43 | stephenfin | kashyap: Yup, it's low priority and can be done anytime this cycle. No rush! | |
| 12:31:56 | kashyap | Yep. | |
| 12:32:40 | kashyap | Then, I might even wait until I'm through all the CPU modelling stuff that I'm looking into (until 27 Oct) | |
| 12:40:36 | mdbooth | bauzas: LM failure on https://review.openstack.org/#/c/604400/ | |
| 12:41:05 | mdbooth | So, the exact failure is unlikely to be relevant. Chances are that this is just the first thing which touched that host, and the host was broken by the evacuate test we just ran. | |
| 12:41:35 | bauzas | mdbooth: so, you got a failure for req-dd88c142-4710-49d0-b6e9-594c6a1e89f6 (live-migrate action) because of http://logs.openstack.org/00/604400/6/check/nova-live-migration/4ba65e4/logs/screen-n-sch.txt.gz#_Sep_28_20_39_55_712058 | |
| 12:41:52 | mdbooth | bauzas: Yes | |
| 12:42:01 | bauzas | rather http://logs.openstack.org/00/604400/6/check/nova-live-migration/4ba65e4/logs/screen-n-sch.txt.gz#_Sep_28_20_39_55_711740 | |
| 12:42:07 | mdbooth | So, the question is: why does the computefilter think the host is down? | |
| 12:42:26 | bauzas | mdbooth: because the db record is old | |
| 12:42:45 | bauzas | mdbooth: I mean the updated_at filed | |
| 12:42:47 | bauzas | field* | |
| 12:42:52 | mdbooth | bauzas: Right, but in the compute logs I see the host joining the service group within the last 60 secs | |
| 12:42:58 | bauzas | https://github.com/openstack/nova/blob/master/nova/scheduler/filters/compute_filter.py#L44 | |
| 12:43:12 | mdbooth | However, I note that it doesn't update the record at that point, just schedules it to be updated in 5 seconds | |
| 12:43:40 | mdbooth | So I then got to wondering why it doesn't seem to have been updated 5 seconds later. | |
| 12:44:00 | mdbooth | I was wondering if perhaps a long running init_host might be related? | |
| 12:44:16 | mdbooth | Because that's feasible following the evacuate test | |
| 12:45:01 | bauzas | https://github.com/openstack/nova/blob/master/nova/servicegroup/drivers/db.py#L73 | |
| 12:45:09 | mdbooth | However, threadgroup.add_timer seems to be unconditional | |
| 12:45:21 | bauzas | and https://github.com/openstack/nova/blob/master/nova/servicegroup/drivers/db.py#L60 | |
| 12:45:59 | mdbooth | bauzas: Right, so it hasn't updated its record within the last 60 seconds | |
| 12:46:20 | bauzas | just checking, it's 60sec http://logs.openstack.org/00/604400/6/check/nova-live-migration/4ba65e4/logs/screen-n-sch.txt.gz#_Sep_28_20_25_02_439642 | |
| 12:46:27 | bauzas | yup | |
| 12:46:32 | mdbooth | But given that it joined the servicegroup http://logs.openstack.org/00/604400/6/check/nova-live-migration/4ba65e4/logs/subnode-2/screen-n-cpu.txt.gz?#_Sep_28_20_39_12_277355 | |
| 12:46:38 | bauzas | now, I'm checking the compute logs | |
| 12:46:53 | mdbooth | I can't see why not. | |
| 12:47:08 | bauzas | it's a periodic task IIRC | |
| 12:47:41 | mdbooth | Looks like it's a threadgroup timer | |
| 12:47:57 | mdbooth | Which would be eventlet-fu | |
| 12:48:18 | mdbooth | But I wonder if its execution could be blocked by, eg, a long-running periodic task | |
| 12:49:51 | sean-k-mooney | mdbooth: we had a downstream bugs(3) related to osp7 upgrades that saw similar behavior. compute filter elimated host becase host was still down after upgrade. in the downstream case it was causeing tempset test to fail as the heartbeat had not been recived yet and the host was down. this seams similar. | |
| 12:50:28 | mdbooth | sean-k-mooney: The immediate cause would certainly be the same. I wonder if the underlying cause is, though. | |
| 12:50:33 | mdbooth | Did you get to the bottom of it? | |
| 12:51:40 | sean-k-mooney | mdbooth: no in the down stream case we saw that subsequent schduling attempts suceeded as the heatbeat had been processed so we closed as not a but as the cloud had not stablised after upgrade before running the tests | |
| 12:52:05 | sean-k-mooney | e.g. it corrected it self a few seconds after teh tempest test was started | |
| 12:57:49 | bauzas | mdbooth: honestly, I don't see the problem with the service group .join() call | |
| 12:58:44 | bauzas | mdbooth: is this only a transient issue ? | |
| 12:59:10 | bauzas | mdbooth: if that's more than that, try to have a DNM patch adding some log like herehttps://github.com/openstack/nova/blob/master/nova/servicegroup/drivers/db.py#L91 | |
| 13:11:01 | gryf | is anyone know what is happening with this blueprint: https://blueprints.launchpad.net/nova/+spec/numa-aware-live-migration ? | |
| 13:12:16 | mnaser | a little idle, simple clean backport if someone has a second or two: https://review.openstack.org/#/q/I811e84af46d678c3fdbf94ee400eabe659fc3d4e | |
| 13:14:19 | openstackgerrit | Takashi NATSUME proposed openstack/python-novaclient master: WIP: Fix test_instance_action functional test failure https://review.openstack.org/606947 | |
| 13:17:30 | mdbooth | Folks, anybody else hitting the bug in parallel evacuate in CI I have a fix up for it here: https://review.openstack.org/#/c/605436/ | |
| 13:18:08 | mdbooth | It's not a one line fix, though, so you'll need to block out some time to review it. However, probably not as much time as we're losing to this test failing continuously. | |
| 13:29:30 | mdbooth | Hmm, I just had to recheck all 3 commits from this morning for that bug :/ | |
| 13:53:22 | efried | n-sch/placement meeting in 7 minutes in #openstack-meeting-alt | |
| 14:45:33 | openstackgerrit | Takashi NATSUME proposed openstack/python-novaclient master: Fix test_instance_action functional test failure https://review.openstack.org/606947 | |
| 14:57:57 | openstackgerrit | Chris Dent proposed openstack/nova master: Do not encode server create location reponse header https://review.openstack.org/606967 | |
| 15:25:04 | mriedem | bauzas: let's not run the linuxbridge job on all nova changes https://review.openstack.org/#/c/606616/ | |
| 15:25:08 | mriedem | ^ fixes a regression | |
| 15:25:23 | bauzas | mriedem: /me looks | |
| 15:26:13 | bauzas | mriedem: oh shit, reviewing the py3-first changes was horribly difficult, my bad if I made a regression :( | |