Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2020-10-13
16:22:30 stephenfin lyarwood: done
16:22:46 stephenfin lyarwood: some nits but nothing blocking
17:31:32 mnaser sean-k-mooney: so far i have made it to the conclusion that my issue is that both vfs and pfs are all in the same iommu group, therefore is not possible to attach a single vf..
20:46:33 openstackgerrit Merged openstack/nova master: Fix a hacking test https://review.opendev.org/754713
#openstack-nova - 2020-10-14
04:47:08 openstackgerrit Hemanth Nakkina proposed openstack/nova master: Update pci stat pools based on PCI device changes https://review.opendev.org/749175
08:00:28 openstackgerrit Lee Yarwood proposed openstack/nova master: Add regression test for bug #1899649 https://review.opendev.org/757893
08:00:28 openstack bug 1899649 in OpenStack Compute (nova) "Volume marked as available after a failure to build" [Undecided,In progress] https://launchpad.net/bugs/1899649 - Assigned to Lee Yarwood (lyarwood)
08:00:28 openstackgerrit Lee Yarwood proposed openstack/nova master: compute: Don't detach volumes when RescheduledException raised without retry https://review.opendev.org/757894
08:00:28 lyarwood stephenfin: ^ Morning, sorry borked the unit test change with that fix.
08:00:49 lyarwood stephenfin: would you also mind taking a look at https://review.opendev.org/#/c/754695/ today? Another super trivial change.
08:55:39 lyarwood Does the following trace ring any bells for anyone?
08:56:18 lyarwood 2020-10-08 00:40:06.508 8 ERROR oslo_messaging.rpc.server [req-d1269d5e-18c3-410b-87f1-b36a71ec79b9 c6015c4d46694de2bb99411a5fdb388e 301e3b6a1fc941f2897d00a657a22482 - default default] Exception during message handling: UnexpectedTaskStateError_Remote: Conflict updating instance 558b87b7-dbb4-4a3a-9a63-8d873dc4bf33. Expected: {'task_state': [u'migrating']}. Actual: {'task_state': None}
08:56:28 lyarwood during post_live_migration_at_destination
08:57:23 lyarwood on stable/queens
08:58:32 stephenfin lyarwood: not me, but sounds like a race with something else setting task_state. Possibly multiple instance operations happening at the same time and poor locking in place?
09:00:28 lyarwood stephenfin: yeah there's no overlapping instance actions that I can see, just looks like poor error handling, was hoping someone had seen it
09:04:20 tosky as you are around, may I ask how it's going with the zuul v3 jobs?
09:05:15 lyarwood tosky: I've not made any progress with https://review.opendev.org/#/c/711625/
09:05:24 lyarwood tosky: slightly slammed downstream this week
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: Reproduce bug 1896463 in func env https://review.opendev.org/754100
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: Set instance host and drop migration under lock https://review.opendev.org/754815
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: Clean up base migration https://review.opendev.org/758155
15:25:03 openstackgerrit Stephen Finucane proposed openstack/nova master: db: Merge in unique constraints https://review.opendev.org/758156
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 gibi ack
15:43:19 sean-k-mooney in this case nova branced on time but ya
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)

Earlier   Later