Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2020-04-17
15:13:49 gibi but I haven't checked the code
15:13:55 sean-k-mooney lyarwood: where you looking into https://bugs.launchpad.net/nova/+bug/1860913 downstream
15:14:24 gibi I let you folks to handle it :)
15:14:42 bauzas the 'you folks' is important :)
15:14:47 bauzas :D
15:15:02 lyarwood sean-k-mooney: nope I don't recall seeing this
15:15:14 sean-k-mooney gibi: well this is basically an issue with how we handel the snapshot
15:15:35 openstack bugzilla.redhat.com bug 1757691 in openstack-nova "Instance uses base image file when it is rebooted after snapshot creation if cinder nfs backend is used" [Urgent,New] - Assigned to nova-maint
15:15:35 sean-k-mooney https://bugzilla.redhat.com/show_bug.cgi?id=1757691#c1
15:15:38 lyarwood looks like Matt triaged it a while ago
15:15:52 sean-k-mooney we discussed it on the triage call
15:16:44 sean-k-mooney whats happening is during the snapshot we cate a path form cinder to use a a temp file or something which updates the xml but if you do a hard reboot we regenerate it and use the one it had beofre
15:17:03 sean-k-mooney the real fix is to not allow reboots while we do snapshots
15:17:47 bauzas either way, the upstream bug is triaged with no assignee
15:17:55 bauzas feel free to pick it, teammates
15:18:03 lyarwood I'll take it
15:18:16 bauzas lyarwood: thanks
15:18:18 lyarwood we should be able to fix this easily by updating the connection_info
15:19:28 sean-k-mooney maybe i still think we need to lock the instance when you intialise the snapshot of the volume via cinder somehow
15:23:13 stephenfin bauzas: RE: the integrated_helpers duplication, I'm okay with you doing that as a follow-up, so long as it's actually done and you stick a TODO in in your patch saying that it will be done :)
15:23:45 stephenfin Will happily review the follow-up too
15:23:50 sean-k-mooney ya i guss i would be fine with that as well.
15:24:06 stephenfin need the TODOs though
15:37:39 openstack Launchpad bug 1858402 in OpenStack Compute (nova) "In-use volume does not support the same ceph cluster retype" [Undecided,New]
15:37:39 gibi lyarwood: do you think https://bugs.launchpad.net/nova/+bug/1858402 is a duplicate of https://bugs.launchpad.net/nova/+bug/1868996 ?
15:37:41 openstack Launchpad bug 1868996 in OpenStack Compute (nova) "Unable to swap between RBD volumes using virDomainBlockRebase" [High,Fix released] - Assigned to Lee Yarwood (lyarwood)
15:46:04 bauzas stephenfin: nah, no worries, I'm pretty done with the new revision, since I need to update it for the main revert resize issue that dansmith said, I won't provide a FUP
15:49:07 lyarwood gibi: hey sorry yes that's a duplicate
15:49:10 bauzas gibi: any idea why I got this weird exception : http://paste.openstack.org/show/792339/
15:51:28 gibi lyarwood: tahnks
15:51:32 lyarwood gibi: updated the bug
15:51:58 gibi lyarwood: thanks a lot
15:52:19 lyarwood np
15:52:20 gibi bauzas: looking
15:52:32 bauzas thanks, I know it's late
15:53:27 bauzas gibi: and that's what I wait for :
15:53:28 bauzas 'instance.resize_revert.end')
15:53:28 bauzas fake_notifier.wait_for_versioned_notifications(
15:54:30 bauzas oh, but I understand the error, I was expecting 'instance.resize_revert' but i never got it
15:54:49 bauzas weirdo
15:54:50 gibi that is my guess yes
15:54:58 gibi you can print all the notification that you received
15:55:09 bauzas gibi: the paste says []
15:55:17 gibi fake_notifier.VERSIONED_NOTIFICATIONS
15:55:58 bauzas do I need to setup something specific in setUp ?
15:56:36 gibi you need to call stub_notifier()
15:56:52 gibi from fake_notifier
15:57:07 bauzas gibi: I moved https://github.com/openstack/nova/blob/cd16ae2/nova/tests/functional/integrated_helpers.py#L1009-L1019 to https://github.com/openstack/nova/blob/cd16ae2/nova/tests/functional/integrated_helpers.py#L75
15:57:25 gibi self.addCleanup(fake_notifier.reset)
15:57:25 gibi fake_notifier.stub_notifier(self)
15:57:39 bauzas ah, thanks
15:57:41 bauzas TIL
15:58:22 gibi cool
16:02:44 gibi OK, I'm leaving for today. See you on Monday
16:07:48 openstackgerrit Lee Yarwood proposed openstack/nova master: WIP block_device: Use the attachment update API to refresh connection_info https://review.opendev.org/720769
16:08:34 bauzas gibi: \o
16:09:32 openstackgerrit Sylvain Bauza proposed openstack/nova master: Allocate mdevs when resizing or reverting resize https://review.opendev.org/712741
16:09:55 bauzas stephenfin: ^ updated the last change for the vgpu resize thing
16:10:42 stephenfin ack
18:26:18 artom *snerk* _get_all_rp_uuids_in_a_tree()
18:26:23 artom Why does that make me giggle
18:26:31 artom Maybe because I'm imagining an actual tree
18:26:33 openstackgerrit Ghanshyam Mann proposed openstack/nova master: Add docs and releasenotes for BP policy-defaults-refresh https://review.opendev.org/720129
18:26:34 artom Or that song...
18:29:53 openstackgerrit melanie witt proposed openstack/nova stable/pike: Fix os-simple-tenant-usage result order https://review.opendev.org/709600
18:36:44 openstackgerrit melanie witt proposed openstack/nova stable/ocata: Fix os-simple-tenant-usage result order https://review.opendev.org/709603
18:37:24 openstackgerrit melanie witt proposed openstack/nova stable/ocata: Fix os-simple-tenant-usage result order https://review.opendev.org/709603
19:25:55 artom Oh, I forgot func tests are broken on queens
19:26:15 artom sqlalchemy.exc.NoSuchTableError: migration_tmp
19:27:54 openstack Launchpad bug 1807262 in nova (Ubuntu) "stein unit tests fail with sqlalchemy.exc.NoSuchTableError: migration_tmp" [High,Fix released]
19:27:54 artom https://bugs.launchpad.net/ubuntu/+source/nova/+bug/1807262 ?
21:07:59 openstackgerrit Artom Lifshitz proposed openstack/nova stable/train: DNM: Add a placement audit command https://review.opendev.org/720838
21:10:58 openstackgerrit Artom Lifshitz proposed openstack/nova stable/stein: DNM: Add a placement audit command https://review.opendev.org/720839
21:21:52 openstackgerrit Artom Lifshitz proposed openstack/nova stable/rocky: DNM: Partial cherry-pick of assertRequestMatchesUsage() https://review.opendev.org/720840
21:21:53 openstackgerrit Artom Lifshitz proposed openstack/nova stable/rocky: DNM: Add a placement audit command https://review.opendev.org/720842
21:21:53 openstackgerrit Artom Lifshitz proposed openstack/nova stable/rocky: DNM: Partial cherry-pick of _check_allocation_during_evacuate() https://review.opendev.org/720841
21:22:15 artom I'm going to let CI chew on that as an extra layer of safety before even thinking about the rocky -> queens backport
21:22:39 artom But it's all going downstream-only eventually
21:29:45 gmann dansmith: lbragstad melwitt artom updated the review comments. this is ready for another re-review - https://review.opendev.org/#/c/720129/
21:30:14 artom gmann, will take a look after supper, I guess
21:30:30 artom gmann, btw, if my initial comments came off as flippant or "agressive" that was not at all the intention
21:30:41 artom I guess I was having a bad say, or was annoyed at something else
21:31:25 artom Maybe I could get a screen reader so I can listen to that doc while making supper...
21:31:25 gmann artom: :) those were good feedback.
21:33:10 gmann hehe or invite me for dinner, i can read for you :)
21:33:54 artom gmann, hahaha
22:11:55 openstackgerrit Ghanshyam Mann proposed openstack/nova master: Add docs and releasenotes for BP policy-defaults-refresh https://review.opendev.org/720129
22:40:02 openstackgerrit Sylvain Bauza proposed openstack/nova master: Allocate mdevs when resizing or reverting resize https://review.opendev.org/712741
23:23:09 openstackgerrit melanie witt proposed openstack/nova stable/pike: Fix os-simple-tenant-usage result order https://review.opendev.org/709600
23:23:10 openstackgerrit melanie witt proposed openstack/nova stable/pike: Require stestr < 3.0.0 in test-requirements https://review.opendev.org/720849
23:25:03 openstackgerrit melanie witt proposed openstack/nova stable/pike: Require stestr < 3.0.0 in test-requirements https://review.opendev.org/720849
23:25:04 openstackgerrit melanie witt proposed openstack/nova stable/pike: Fix os-simple-tenant-usage result order https://review.opendev.org/709600
#openstack-nova - 2020-04-18
04:22:53 openstackgerrit Merged openstack/python-novaclient stable/ussuri: Update .gitreview for stable/ussuri https://review.opendev.org/719164
05:23:45 openstackgerrit Takashi Natsume proposed openstack/python-novaclient stable/ussuri: Update TOX/UPPER_CONSTRAINTS_FILE for stable/ussuri https://review.opendev.org/719165
07:44:42 openstackgerrit Merged openstack/python-novaclient stable/ussuri: Update TOX/UPPER_CONSTRAINTS_FILE for stable/ussuri https://review.opendev.org/719165
12:50:36 openstackgerrit Merged openstack/nova master: Follow-ups for host_status:unknown-only policy rule https://review.opendev.org/713295
12:50:43 openstackgerrit Merged openstack/nova master: fix scsi disk unit number of the attaching volume when cdrom bus is scsi https://review.opendev.org/712607
14:55:09 openstackgerrit Lee Yarwood proposed openstack/nova master: block_device: Use the attachment update API to refresh connection_info https://review.opendev.org/720769
15:24:17 openstackgerrit Merged openstack/nova stable/train: Avoid spurious error logging in _get_compute_nodes_in_db https://review.opendev.org/702902
16:16:15 openstackgerrit Lee Yarwood proposed openstack/nova stable/pike: Init HostState.failed_builds https://review.opendev.org/669538

Earlier   Later