| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2018-12-11 | |||
| 09:11:07 | kashyap | naichuans: First of all...that's a red flag to me: you should not run 'qemu-monitor-command' behind libvirt's back. | |
| 09:11:16 | kashyap | (At least not on production VMs.) | |
| 09:11:37 | naichuans | Ah, just for a test :) | |
| 09:12:05 | kashyap | naichuans: Ah, OK. Even for test, I'm curious if you've tried this: | |
| 09:12:07 | kashyap | $ virsh migrate --verbose --copy-storage-all --p2p --live cvm1 qemu+tcp://user@dest/system | |
| 09:12:27 | kashyap | Under the hood, the above is using QEMU's 'dirve-mirror'. | |
| 09:13:44 | naichuans | Oh, then it should be the cmd I needed. I'm checking the qemu3.0 new feature: copy_mode for live mirror. | |
| 09:14:29 | naichuans | How to specify it to use `write-blocking` option in the command? | |
| 09:15:13 | kashyap | naichuans: I think you're referring to: "Support for "active" mirror using the new option "copy-mode" to the "drive-mirror" monitor option." | |
| 09:15:22 | naichuans | yes | |
| 09:16:23 | kashyap | Again, 'write-blocking' is a deep QMP (QEMU Monitor Protocol) option; if you want to try that, you have to use QMP commands by shell | |
| 09:16:44 | kashyap | This is probably better discussed on #qemu, OFTC IRC network. | |
| 09:17:27 | naichuans | I see, thanks. If I use the cmd you provide above, it would use 'write-blocking' by default, right? I saw there is `--live` flag | |
| 09:18:52 | kashyap | naichuans: "It depends". I.e. -- | |
| 09:19:41 | naichuans | Thank you very much, Kashyap. Will check it. | |
| 09:19:41 | openstackgerrit | Lee Yarwood proposed openstack/nova master: libvirt: Add workaround to cleanup instance dir during evac with rbd https://review.openstack.org/618478 | |
| 09:20:02 | kashyap | naichuans: If you're using 3.0 or above, I don't think 'write-blocking' will be use by default; you have to explicitly specify it. | |
| 09:20:17 | kashyap | naichuans: And with 3.0 or above, 'copy-mode' will default to "background". | |
| 09:21:42 | naichuans | I see, the release log also said that. passive or backgroud is default. Will check how to specify it to use `write-blocking` | |
| 09:44:54 | openstackgerrit | Brin Zhang proposed openstack/nova master: Remove the unused parameter from os-hypervisors https://review.openstack.org/624333 | |
| 10:06:15 | fanzhang | hi kashyap if you have some time, please help review this patch https://review.openstack.org/#/c/612272/ | |
| 10:07:44 | fanzhang | kashyap I discussed with melwitt , and she said she would ask you to take a look at your earliest convenience. I just notice you here, so... | |
| 10:07:46 | kashyap | fanzhang: Hi, will look. I'm troubleshooting something else at the moment. | |
| 10:08:09 | fanzhang | kashyap thanks a lot, no need to hurry, :) | |
| 10:48:46 | openstackgerrit | Chris Dent proposed openstack/nova master: Delete the placement code https://review.openstack.org/618215 | |
| 10:50:52 | kashyap | fanzhang: I'll be off from next week until 01-Jan, I'll do my best to get to it this week | |
| 11:08:30 | lyarwood | mdbooth_: https://review.openstack.org/#/c/618478/ - can you hit this again? I've added some additional context around migrations after melwitt reviewed yesterday, would love to close this out this week if possible. | |
| 11:09:24 | mdbooth | lyarwood: ack | |
| 11:38:03 | openstackgerrit | Chris Dent proposed openstack/os-resource-classes master: Initial automated testing for os-resource-classes https://review.openstack.org/624366 | |
| 11:42:47 | openstackgerrit | Yikun Jiang proposed openstack/nova master: Add live migration timeout action https://review.openstack.org/619143 | |
| 11:42:47 | openstackgerrit | Yikun Jiang proposed openstack/nova master: Remove live_migration_progress_timeout config https://review.openstack.org/619142 | |
| 11:56:06 | openstackgerrit | Chris Dent proposed openstack/os-resource-classes master: Initial automated testing for os-resource-classes https://review.openstack.org/624366 | |
| 11:56:07 | openstackgerrit | Chris Dent proposed openstack/os-resource-classes master: Rename ORDERED_CLASSES to STANDARDS https://review.openstack.org/624369 | |
| 11:56:08 | openstackgerrit | Chris Dent proposed openstack/os-resource-classes master: Sync STANDARDS with placement/nova rc_fields https://review.openstack.org/624370 | |
| 12:05:22 | openstackgerrit | Yikun Jiang proposed openstack/nova master: Add live migration timeout action https://review.openstack.org/619143 | |
| 12:05:22 | openstackgerrit | Yikun Jiang proposed openstack/nova master: Remove live_migration_progress_timeout config https://review.openstack.org/619142 | |
| 12:08:06 | lbragstad | johnthetubaguy sometime soon we should go through https://review.openstack.org/#/c/596520/ again? | |
| 12:08:20 | lbragstad | i'll have to ping jaypipes, too | |
| 12:08:59 | lbragstad | looks like wxy-xiyuan worked some changes into the latest revision | |
| 12:50:49 | openstackgerrit | Chris Dent proposed openstack/os-resource-classes master: Tune up documentation to be more useful https://review.openstack.org/624384 | |
| 13:12:03 | sean-k-mooney | anyone know how to use logstash and write querries | |
| 13:12:38 | sean-k-mooney | i cant get the message filtering to work | |
| 13:12:40 | sean-k-mooney | tags:"screen-n-cpu.txt" and message:"os_vif error: [Errno 24] Too many open files" and project:"openstack/neutron" | |
| 13:13:39 | cdent | sean-k-mooney: sorry, I'm no help. I generally just flail around, mostly using message: and that's it | |
| 13:14:24 | sean-k-mooney | for some reason its ignoreing the message section and retruning message strings without the message in it | |
| 13:15:11 | cdent | maybe it is confused by [ or ]? | |
| 13:15:56 | sean-k-mooney | maybe i tried excaping them ill try a few more things | |
| 14:18:28 | openstackgerrit | Matt Riedemann proposed openstack/nova master: Handle tags in _bury_in_cell0 https://review.openstack.org/621856 | |
| 14:20:35 | openstackgerrit | Matt Riedemann proposed openstack/nova stable/queens: Create BDMs/tags in cell with instance when over-quota https://review.openstack.org/623934 | |
| 14:22:10 | mriedem | lyarwood: ready for you https://review.openstack.org/#/q/topic:bug/1806064+branch:stable/queens | |
| 14:22:52 | mriedem | gibi: do you remember why we don't include tags in the instance payload in versioned notifications? | |
| 14:23:26 | mriedem | i guess it's on instance create and update, but not the regular action paylodas | |
| 14:23:28 | mriedem | *payloads | |
| 14:25:36 | gibi | mriedem: maybe because they doesn't change in other cases | |
| 14:35:44 | gibi | mriedem: maybe this helps remebering https://review.openstack.org/#/c/407228/36/nova/notifications/objects/instance.py@119 | |
| 14:36:00 | jangutter | mriedem: thanks, and it's Kandarian, not Greek. Sincere sympathies and hope your soul remains intact. | |
| 14:36:34 | gibi | mriedem: I think we wanted to avoid some lazy load issues | |
| 14:53:01 | mriedem | gibi: yeah i figured that was the reason | |
| 14:53:07 | mriedem | and we could add it in later if someone needed it | |
| 14:53:22 | gibi | mriedem: yeah, it is not hard to add | |
| 14:53:26 | mriedem | reminds me i have an old patch to optimize the instance tags creation stuff since we do a lot of unnecessary db stuff | |
| 14:53:51 | gibi | mriedem: btw vitrage has a patch up to consume nova versioned notifications | |
| 14:54:08 | gibi | mriedem: so we will have one modul that trasition from legacy to versioned | |
| 14:54:14 | mriedem | cool - are they migrating from legacy or adding versioned from scratch? | |
| 14:54:17 | mriedem | ah ok | |
| 14:54:31 | gibi | mriedem: https://review.openstack.org/#/c/623274/ | |
| 15:06:07 | openstackgerrit | Matt Riedemann proposed openstack/nova stable/pike: Add functional regression test for bug 1806064 https://review.openstack.org/623935 | |
| 15:06:07 | openstack | bug 1806064 in OpenStack Compute (nova) queens "Volume remains in attaching/reserved status, if the instance is deleted after TooManyInstances exception in nova-conductor" [Medium,In progress] https://launchpad.net/bugs/1806064 - Assigned to Matt Riedemann (mriedem) | |
| 15:10:26 | openstackgerrit | Matt Riedemann proposed openstack/nova stable/pike: Create BDMs/tags in cell with instance when over-quota https://review.openstack.org/623937 | |
| 15:10:39 | openstackgerrit | Matt Riedemann proposed openstack/nova stable/pike: Create BDMs/tags in cell with instance when over-quota https://review.openstack.org/623937 | |
| 15:12:15 | mriedem | dansmith: looks like https://review.openstack.org/#/c/623282/ last failed on the test added, | |
| 15:12:24 | mriedem | so we likely need to reset the global somewhere more....global | |
| 15:12:31 | mriedem | maybe in the test setUp? | |
| 15:12:54 | mriedem | or in the base TestCase setup where we reset other globals? | |
| 15:13:10 | dansmith | mriedem: ah, I applied a fix for that to the next patch by accident I think | |
| 15:13:22 | dansmith | I don't think we need to reset it anywhere other than just in that test | |
| 15:14:49 | stephenfin | sean-k-mooney: RE: https://bugs.launchpad.net/nova/+bug/1805891, this is where things break down https://github.com/openstack/nova/blob/master/nova/compute/resource_tracker.py#L209-L210 | |
| 15:14:49 | openstack | Launchpad bug 1805891 in OpenStack Compute (nova) "pci numa polices are not followed" [High,Triaged] - Assigned to sean mooney (sean-k-mooney) | |
| 15:15:15 | stephenfin | Looks like we're not persisting that field in the DB :-\ | |
| 15:16:34 | openstackgerrit | Dan Smith proposed openstack/nova master: Only warn about not having computes nodes once in rpcapi https://review.openstack.org/623282 | |
| 15:16:35 | openstackgerrit | Dan Smith proposed openstack/nova master: Make compute rpcapi version calculation check all cells https://review.openstack.org/623284 | |
| 15:16:35 | openstackgerrit | Dan Smith proposed openstack/nova master: Make service.get_minimum_version_all_cells() cache the results https://review.openstack.org/623283 | |
| 15:16:41 | dansmith | mriedem: moved that down a patch, mah bad | |
| 15:18:47 | mriedem | ack | |
| 15:26:21 | openstackgerrit | Chris Dent proposed openstack/nova master: Add python 3.7 unit and functional tox jobs https://review.openstack.org/624055 | |
| 15:26:54 | cdent | gibi: there are some functional test failures related to waiting for notifications in ^ | |
| 15:27:22 | cdent | dansmith, mriedem: that ^ fixes some unit test failures in py37 and a real issue in the multi cell query stuff with StopIteration | |
| 15:28:07 | gibi | cdent: ack | |
| 15:28:34 | cdent | gibi: I haven't investigate the failures closely yet, but I'm assuming some kind of deadlock | |
| 15:29:02 | gibi | cdent: I made a note to look at it at some point | |
| 15:29:10 | mriedem | moar jubz | |
| 15:29:41 | gibi | cdent: is it in PS3 as PS2 passed the functional jobs | |
| 15:29:41 | mriedem | is there a tl;dr from the tc on when we're going to stop running 20 different python unit test/functional jobs per change? | |
| 15:29:46 | cdent | mriedem: i have it on good authority from clarkb that unit test jobs are not "the problem" | |
| 15:30:03 | mriedem | we still have race failures in py jobs that reset the gate | |
| 15:30:16 | mriedem | http://status.openstack.org/elastic-recheck/#1793364 | |
| 15:30:25 | mriedem | http://status.openstack.org/elastic-recheck/#1800472 | |
| 15:30:48 | mriedem | just wondering if at some point py27 goes to experimental queue or something, and py35 gets dropped | |
| 15:30:48 | cdent | mriedem: heh, on the tl;dr: I'm not really sure as I lost interest when "stop running 20 different..." was taken off the table | |
| 15:31:08 | cdent | I, perhaps, wanted somethign a bit too aggressive: one distro, one python version | |