| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2020-10-14 | |||
| 09:05:49 | tosky | you tell me | |
| 09:09:59 | aarents | lyarwood: I know that nova-compute restart on source host with live-migration will reset task_state, no other idea.. | |
| 09:13:06 | lyarwood | ouch | |
| 09:13:08 | lyarwood | TIL | |
| 09:14:21 | hemanth_n | stephenfin: thanks for your review on https://review.opendev.org/#/c/749175/, i left a comment can you please check when you get sometime | |
| 09:26:29 | songwenping_ | stephenfin: hi, this patch https://review.opendev.org/#/c/569386/27 has completed set/unset tag for server, so my patch https://review.opendev.org/#/c/756664/3 isnot need anymore. | |
| 09:28:06 | stephenfin | songwenping_: indeed; I can abandon that for you now so :) | |
| 09:28:57 | lyarwood | hmm do ~./ | |
| 09:29:08 | lyarwood | gah weechat-- | |
| 09:31:19 | songwenping_ | stephenfin: thanks, but seems it lack 'server-tag-delete-all' and 'server-tag-list', should we supply? | |
| 09:32:26 | stephenfin | songwenping_: I would expect the output of 'nova server-tag-list' to be included in 'openstack server show', no? I'm not sure if 'server-tag-delete-all' is all that useful either | |
| 09:35:28 | songwenping_ | right, maybe it's `openstack server show --tag`, i'll verify if merged once, and not to consider `server-tag-delete-all` temporary. :) | |
| 09:41:41 | stephenfin | songwenping_: If you really wanted to add an equivalent to the 'server-tag-delete-all' command, you could copy what we do in 'openstack flavor set' and add a '--no-property' flag | |
| 09:42:52 | songwenping_ | stephenfin: ok, cool. | |
| 09:48:37 | lyarwood | huh fun | |
| 09:49:57 | lyarwood | stephenfin: so the task_state thing from earlier, something called stop_instance during the live migration, this failed as libvirt had locked the domain and as it's wrapped with reverts_task_state moved the state to None | |
| 09:50:24 | lyarwood | but the request-id doesn't map to an instance action | |
| 09:50:26 | lyarwood | /o\ | |
| 12:52:03 | openstackgerrit | Mamduh proposed openstack/os-vif master: Fix - os-vif fails to get the correct UpLink Representor https://review.opendev.org/745895 | |
| 12:57:06 | openstackgerrit | Takashi Natsume proposed openstack/nova stable/victoria: Fix a hacking test https://review.opendev.org/758112 | |
| 13:16:10 | lyarwood | stephenfin: any idea why my test would end up using the nova.virt.fake.SmallFakeDriver when I've used nova.tests.functional.libvirt.base.ServersTestBase ? | |
| 13:16:27 | stephenfin | are you using a mixin? | |
| 13:17:39 | lyarwood | stephenfin: nope, just nova.tests.functional.libvirt.base.ServersTestBase | |
| 13:17:57 | lyarwood | ah wait | |
| 13:18:03 | lyarwood | I used _start_compute | |
| 13:18:06 | lyarwood | *facepalm* | |
| 13:20:46 | lyarwood | cool that's working now | |
| 13:21:04 | lyarwood | might want to stub out _start_compute in ServersTestBase to avoid confusion tbh | |
| 14:08:04 | openstackgerrit | Balazs Gibizer proposed openstack/nova master: Set instance host and drop migration under lock https://review.opendev.org/754815 | |
| 14:08:04 | openstack | bug 1896463 in OpenStack Compute (nova) rocky "evacuation failed: Port update failed : Unable to correlate PCI slot " [Low,In progress] https://launchpad.net/bugs/1896463 | |
| 14:08:04 | openstackgerrit | Balazs Gibizer proposed openstack/nova master: Reproduce bug 1896463 in func env https://review.opendev.org/754100 | |
| 14:09:16 | gibi | stephenfin: fixed and replied in these ^^ | |
| 14:14:47 | bauzas | gibi: go to my house | |
| 14:15:32 | gibi | bauzas: need to mow the lawn? | |
| 14:15:59 | bauzas | actually, yes :p | |
| 14:16:10 | gibi | I have to be more specific I would like to work on some greenfield software project :) | |
| 14:16:20 | gibi | but a bit of gardening would be nice too | |
| 14:16:34 | bauzas | hehe | |
| 14:17:02 | bauzas | gibi: btw, I'm not around this week in this chan, but I should be back here tomorrow morning | |
| 14:17:38 | gibi | bauzas: ack thanks for the heads up | |
| 14:17:57 | gibi | ooh, did you tried AMD vGPUs? | |
| 14:18:13 | bauzas | gibi: nope, nvidia :p | |
| 14:18:27 | gibi | nvidia and sriov? I thought nvidia used mdevs | |
| 14:18:35 | gibi | OK | |
| 14:18:42 | gibi | o/ | |
| 14:18:49 | bauzas | gibi: https://docs.nvidia.com/grid/latest/grid-vgpu-user-guide/index.html#mig-gpu-support-vgpu-software fyi | |
| 14:20:09 | gibi | thanks | |
| 14:31:16 | openstackgerrit | Merged openstack/nova master: Add a hacking rule for assert_has_calls https://review.opendev.org/754714 | |
| 15:25:03 | openstackgerrit | Stephen Finucane proposed openstack/nova master: db: Merge in unique constraints https://review.opendev.org/758156 | |
| 15:25:03 | openstackgerrit | Stephen Finucane proposed openstack/nova master: db: Clean up base migration https://review.opendev.org/758155 | |
| 15:25:04 | openstackgerrit | Stephen Finucane proposed openstack/nova master: db: Drop add/remove of index https://review.opendev.org/758157 | |
| 15:26:26 | gibi | does somebody know why don't we have release notes link on the release page for nova? https://releases.openstack.org/victoria/index.html | |
| 15:29:40 | elod | it's interesting because, the release notes page is there: https://docs.openstack.org/releasenotes/nova/victoria.html | |
| 15:29:46 | sean-k-mooney | https://docs.openstack.org/releasenotes/nova/victoria.html | |
| 15:33:53 | gibi | sean-k-mooney: yeah, it is there but not linked to the release page | |
| 15:34:17 | sean-k-mooney | i think kolla/osa had that same issue | |
| 15:34:30 | sean-k-mooney | mgoddard: ^ | |
| 15:34:45 | sean-k-mooney | did you fix the progjec being missing recently for kolla | |
| 15:35:03 | mgoddard | we haven't branched yet | |
| 15:35:10 | mgoddard | so no renos | |
| 15:35:25 | sean-k-mooney | right but dont you normally have to fix it with a patch when you do | |
| 15:35:26 | mgoddard | the link gets added to the releases repo | |
| 15:35:30 | mgoddard | yes | |
| 15:35:39 | sean-k-mooney | ya so i think we need to do that for nova | |
| 15:35:49 | sean-k-mooney | do you have an example | |
| 15:40:35 | mgoddard | sean-k-mooney: 1b78be1a90f61e69a07720fb73fd665588186976 | |
| 15:41:50 | sean-k-mooney | ok so release-notes: https://docs.openstack.org/releasenotes/os-vif/ussuri.html | |
| 15:42:04 | sean-k-mooney | or in this case release-notes: https://docs.openstack.org/releasenotes/nova/victoria.html | |
| 15:42:15 | sean-k-mooney | need to be added to the nova deliverable for victoria | |
| 15:42:22 | sean-k-mooney | that seams pretty simple | |
| 15:42:24 | gibi | mgoddard, sean-k-mooney: this added most of the reno links https://review.opendev.org/#/c/755476/ | |
| 15:42:57 | sean-k-mooney | https://github.com/openstack/releases/blob/master/deliverables/victoria/nova.yaml | |
| 15:43:01 | sean-k-mooney | ya nova does not have it set | |
| 15:43:04 | mgoddard | there may be a script someone can run to pick up the late branchers | |
| 15:43:13 | mgoddard | try #openstack-release | |
| 15:43:19 | sean-k-mooney | in this case nova branced on time but ya | |
| 15:43:19 | gibi | ack | |
| 15:45:07 | sean-k-mooney | gibi: are you goign to update the release repo? if not i can do it after my call finishes | |
| 15:45:39 | gibi | sean-k-mooney: I pinge smcginnis about it as it is not just nova that is missing | |
| 15:45:52 | gibi | maybe there is just a script that needs to be run | |
| 15:46:06 | smcginnis | gibi: We have a script to add them, but it checks that the actual landing page exists first. So if a project hadn't merged the bot patches to add it, that would explain why it doesn't show up. (from releases channel) | |
| 15:51:53 | gibi | sean-k-mooney: smcginnis solved the issue with https://review.opendev.org/#/c/758197/ | |
| 15:54:55 | sean-k-mooney | ok cool | |
| 15:55:12 | sean-k-mooney | looks like half of puppet was skipped | |
| 15:55:59 | melwitt | https://zuul.opendev.org/t/openstack/build/f2608672b6c74108ad07d8cdd0ec63ae/log/logs/screen-n-cpu.txt#17674 | |
| 15:55:59 | melwitt | I've been noticing quite a few libvirtError during live migration in the gate, recent example: "Live Migration failure: internal error: missing block job data for disk 'vda': libvirt.libvirtError: internal error: missing block job data for disk 'vda'" in ova-grenade-multinode | |
| 15:56:31 | melwitt | I'm not aware there's anything we can do about it but it's a pain | |
| 16:04:32 | gibi | melwitt: the only recent libvirt driver change I'm aware of is https://review.opendev.org/#/q/Ib9ed7069cef5b73033351f7a78a3fb566753970d but I don't know if it is related | |
| 16:06:43 | openstackgerrit | Mark Goddard proposed openstack/nova master: Fix PCI passthrough race on reschedule (claims) https://review.opendev.org/710847 | |
| 16:07:19 | melwitt | thanks gibi. I doubt it's related but I will have a look in logstash to see if there's any time correlation. also noting that e-r indexing is behind by 198hrs http://status.openstack.org/elastic-recheck/ I've discussed the e-r indexing stuff with -infra and I'm going to try a change there to help with that | |
| 16:08:53 | gibi | melwitt: other change is the Focal migration that brought in new libvirt version | |
| 16:09:55 | melwitt | ohh right. that might be more likely | |
| 16:38:21 | openstackgerrit | Balazs Gibizer proposed openstack/nova master: Use _evacuate_server helper in func test https://review.opendev.org/758210 | |
| 16:38:21 | openstackgerrit | Balazs Gibizer proposed openstack/nova master: Remove self.server dependency form _evacuate_server https://review.opendev.org/758209 | |
| 16:38:29 | gibi | stephenfin: ^^ these might be interesting for you | |
| 18:02:27 | openstackgerrit | Merged openstack/nova master: libvirt: Log exception when unable to import rbd or rados https://review.opendev.org/746873 | |
| 20:17:05 | openstackgerrit | Lee Yarwood proposed openstack/nova master: CinderFixture: Return a driver_type of `fake` https://review.opendev.org/758238 | |
| 20:17:07 | openstack | bug 1899835 in OpenStack Compute (nova) "disconnect_volume called during rollback from pre_live_migration failure on destination" [Undecided,New] https://launchpad.net/bugs/1899835 - Assigned to Lee Yarwood (lyarwood) | |
| 20:17:07 | openstackgerrit | Lee Yarwood proposed openstack/nova master: Add regression test for bug #1899835 https://review.opendev.org/758239 | |
| 20:18:19 | openstackgerrit | Lee Yarwood proposed openstack/nova stable/victoria: libvirt: Log exception when unable to import rbd or rados https://review.opendev.org/758240 | |