Earlier  
Posted Nick Remark
#openstack-nova - 2021-03-29
10:28:14 openstackgerrit Merged openstack/nova stable/wallaby: [stable-only]Update .gitreview for stable/wallaby https://review.opendev.org/c/openstack/nova/+/783195
10:28:27 openstackgerrit Merged openstack/nova stable/wallaby: [stable-only]Update TOX_CONSTRAINTS_FILE for stable/wallaby https://review.opendev.org/c/openstack/nova/+/783216
10:55:55 openstackgerrit Xinran WANG proposed openstack/nova-specs master: Repropose smartnic support spec https://review.opendev.org/c/openstack/nova-specs/+/783632
11:25:28 stephenfin kashyap: Can you revisit https://review.opendev.org/c/openstack/nova/+/781438/?
11:25:41 stephenfin I've added a follow-up addressing your comments here https://review.opendev.org/c/openstack/nova/+/782410/
12:56:09 kashyap stephenfin: Queued (I see it's the CPU models doc)
13:38:08 sean-k-mooney stephenfin: did you see my comments on the sev one that is already merged https://review.opendev.org/c/openstack/nova/+/781437/2/doc/source/admin/sev.rst
14:09:04 openstackgerrit Lee Yarwood proposed openstack/nova stable/victoria: Add regression test for bug #1908075 https://review.opendev.org/c/openstack/nova/+/783653
14:09:05 openstack bug 1908075 in OpenStack Compute (nova) "Nova allows a non-multiattach volume to be attached to multiple instances *if* its volume state is reset by an admin" [Low,In progress] https://launchpad.net/bugs/1908075 - Assigned to Lee Yarwood (lyarwood)
14:09:05 openstackgerrit Lee Yarwood proposed openstack/nova stable/victoria: api: Reject volume attach requests when an active bdm exists https://review.opendev.org/c/openstack/nova/+/783654
14:10:34 openstackgerrit Lee Yarwood proposed openstack/nova stable/ussuri: Add regression test for bug #1908075 https://review.opendev.org/c/openstack/nova/+/783655
14:10:35 openstackgerrit Lee Yarwood proposed openstack/nova stable/ussuri: api: Reject volume attach requests when an active bdm exists https://review.opendev.org/c/openstack/nova/+/783656
14:10:36 openstack bug 1908075 in OpenStack Compute (nova) "Nova allows a non-multiattach volume to be attached to multiple instances *if* its volume state is reset by an admin" [Low,In progress] https://launchpad.net/bugs/1908075 - Assigned to Lee Yarwood (lyarwood)
14:30:56 openstackgerrit Artom Lifshitz proposed openstack/nova master: WIP: Move port ID verification to REST API https://review.opendev.org/c/openstack/nova/+/675418
14:39:53 openstackgerrit Artom Lifshitz proposed openstack/nova master: Follow up from bp/pci-socket-affinity series https://review.opendev.org/c/openstack/nova/+/779556
14:47:05 openstackgerrit Stephen Finucane proposed openstack/nova master: Remove 'create_db_entry_for_new_instance' https://review.opendev.org/c/openstack/nova/+/778545
14:47:06 openstackgerrit Stephen Finucane proposed openstack/nova master: Remove references to 'instance_type' https://review.opendev.org/c/openstack/nova/+/778546
14:47:07 openstackgerrit Stephen Finucane proposed openstack/nova master: Remove references to 'inst_type' https://review.opendev.org/c/openstack/nova/+/778548
14:47:07 openstackgerrit Stephen Finucane proposed openstack/nova master: Remove (almost) all references to 'instance_type' https://review.opendev.org/c/openstack/nova/+/778547
14:47:09 openstackgerrit Stephen Finucane proposed openstack/nova master: tests: Remove useless mocks https://review.opendev.org/c/openstack/nova/+/778730
14:47:10 openstackgerrit Stephen Finucane proposed openstack/nova master: tests: Remove duplicate policy tests https://review.opendev.org/c/openstack/nova/+/778731
14:47:15 openstackgerrit Stephen Finucane proposed openstack/nova master: tests: Speed up 'servers' API tests https://review.opendev.org/c/openstack/nova/+/778732
14:47:19 openstackgerrit Stephen Finucane proposed openstack/nova master: tests: Merge flavor tests https://review.opendev.org/c/openstack/nova/+/783662
14:51:42 openstackgerrit Artom Lifshitz proposed openstack/nova master: Follow up from bp/pci-socket-affinity series https://review.opendev.org/c/openstack/nova/+/779556
15:08:14 openstackgerrit melanie witt proposed openstack/nova stable/rocky: Use subqueryload() instead of joinedload() for (system_)metadata https://review.opendev.org/c/openstack/nova/+/761813
15:35:44 openstackgerrit Balazs Gibizer proposed openstack/nova master: Bumping min os-brick ver to 4.3.1 https://review.opendev.org/c/openstack/nova/+/783674
15:41:37 gibi lyarwood: os-brick min version bump due to critical bug https://bugs.launchpad.net/os-brick/+bug/1921381
15:41:40 openstack Launchpad bug 1921381 in os-brick xena "iSCSI: Flushing issues when multipath config has changed" [Critical,In progress] - Assigned to Gorka Eguileor (gorka)
15:43:36 gibi lyarwood: if you have time then I appreaciate your review
15:44:09 lyarwood ack just on a call
15:44:30 gibi ack, thanks
15:51:04 kopecmartin hi, can you confirm last comments in this devstack change please? https://review.opendev.org/c/openstack/devstack/+/768363 (just to be sure.. ) - if we add gawk to general, can we remove it from files/*/nova ?
16:22:00 lyarwood gibi: looking at this now, I think it's missing a few things
16:22:32 gibi could be
16:22:47 gibi I only bumped where pip asked for it
16:23:59 lyarwood gibi: which version of pip?
16:24:17 gibi 20.3.4
16:24:46 gibi hm
16:25:54 gibi I tried with 20.3.4 and that showed the eventlet bump, but eventually failed as eventlet 0.30.1 and greenlet does not like py39. So I went to another machine where I have py38
16:26:19 gibi and there it is pip 20.2.4 so that explains why I might missed something /o\
16:26:46 lyarwood gibi: no issues, mind if I push an update that works for me?
16:27:07 gibi lyarwood: I'm glad if you can fix it :)
16:27:10 gibi thanks
16:27:37 lyarwood gibi: np
16:28:24 openstackgerrit Archit Modi proposed openstack/nova master: [Docs] Use 'resize' instead of 'migrate' in migration confirm/revert https://review.opendev.org/c/openstack/nova/+/783705
16:36:42 openstackgerrit Lee Yarwood proposed openstack/nova master: Bumping min os-brick ver to 4.3.1 https://review.opendev.org/c/openstack/nova/+/783674
16:39:00 lyarwood gibi: ^ lets see how that gets on
16:39:35 gibi ack
17:05:25 sean-k-mooney stephenfin: havent fully read your review but ill try and adress it today
17:05:59 sean-k-mooney stephenfin: the main reason i did not remove the object fild in the patch was backports by the way but i assume you know that
17:06:19 stephenfin yeah, you can't remove it without a major version bump
17:06:39 sean-k-mooney do i need a major bump ok ok
17:06:58 sean-k-mooney am well i guess we should look at that
17:07:31 sean-k-mooney there are a few object that are long overdue one but ill stop working on my loacl patch to do that so until we deciide that is what we will do
17:08:06 sean-k-mooney i need to have 2 copies of the class to do a major bump
17:08:13 sean-k-mooney which is a lot more work
17:30:16 melwitt lyarwood, elod: looks like we have a new gate failure on stable/rocky ":stderr: alembic requires Python '!=3.0.*,!=3.1.*,!=3.2.*,!=3.3.*,!=3.4.*,!=3.5.*,>=2.7' but the running Python is 3.5.2" https://zuul.opendev.org/t/openstack/build/c68ff42e5a2347febc6912f0f950ac1d/console#4/0/31/controller
17:48:17 lyarwood melwitt: ack, smells like it needs to be capped right?
17:49:18 melwitt lyarwood: yeah, let me see if there was a new update recently. don't yet know what changed
17:51:04 elod melwitt: uh, that looks bad :S as that stackviz task is part of 'pinned' tempest code which cannot be changed (that task was moved to devstack in newer branches: https://review.opendev.org/q/I0ba1fd05c6391cd8bb978119ecfbb891def2d497 but it's not possible in rocky and older :/ )
17:51:17 melwitt no changes to openstack/requirements...
17:51:40 melwitt elod: ugh
17:52:10 elod maybe if we hack somehow alembic before we get to the stackviz task. (if it doesn't run in a venv :S)
17:59:46 melwitt last successful run used alembic-1.4.3 and now "Collecting alembic>=0.8.10 (from subunit2sql>=0.8.0->stackviz==0.0.1.dev320)" fails
18:00:33 melwitt successful run from march 28 https://zuul.opendev.org/t/openstack/build/e393cf5224974eac8b4cde8756ed711a/console#4/0/31/controller
18:14:50 openstackgerrit Stephen Finucane proposed openstack/nova master: docs: Address review feedback on nova-manage docs https://review.opendev.org/c/openstack/nova/+/783729
18:15:43 elod that's interesting in multiple points... as alembic 1.5.8 was released 23 Mar. so it either should have failed earlier, otherwise I don't know why 1.4.3 was installed before :S
18:24:58 melwitt same :\
18:30:04 elod ok, I've checked, under py35 pip installs the correct alembic version (1.4.3). so I think this error was due to some intermittent pypi mirror failure... at least i remember something like this happened in the past
18:31:12 elod it might worth a recheck
18:32:15 melwitt elod: will try it, thanks!
18:33:40 elod fingers crossed :X
18:33:48 melwitt :)
19:02:34 sean-k-mooney stephenfin: ok so the patch i pushed are not what i tought i pushed
19:03:01 sean-k-mooney stephenfin: as noted inline i lost all my work because i forgot to save before proceedign to the next patch in my interactive rebase
19:03:19 sean-k-mooney so i quickly did them again but i guess i did not put the change in the right place
19:03:29 sean-k-mooney ill respin them both again
19:05:08 sean-k-mooney that kind of annoying but i guess i should have check them more closely when i was trying to redoo it quickly after loosing my work
19:05:36 sean-k-mooney ill fix it tomorrow and ping you when its done
21:00:54 openstackgerrit Merged openstack/nova master: zuul: Add nova-live-migration-ceph job https://review.opendev.org/c/openstack/nova/+/768466
21:06:36 openstackgerrit Merged openstack/nova master: Add Python3 xena unit tests https://review.opendev.org/c/openstack/nova/+/783218
#openstack-nova - 2021-03-30
02:39:58 openstackgerrit Xinran WANG proposed openstack/nova-specs master: Repropose smartnic support spec https://review.opendev.org/c/openstack/nova-specs/+/783632
05:46:18 openstackgerrit Xinran WANG proposed openstack/nova-specs master: Repropose smartnic support spec https://review.opendev.org/c/openstack/nova-specs/+/783632
05:54:10 openstackgerrit Brin Zhang proposed openstack/nova-specs master: Re-propose Remove tenant_id https://review.opendev.org/c/openstack/nova-specs/+/783783
08:26:34 openstackgerrit Yongli He proposed openstack/nova master: Smartnic support - cyborg drive https://review.opendev.org/c/openstack/nova/+/771362
08:26:36 openstackgerrit Yongli He proposed openstack/nova master: smartnic support - new vnic type https://review.opendev.org/c/openstack/nova/+/771363
08:26:37 openstackgerrit Yongli He proposed openstack/nova master: smartnic support - functional tests https://review.opendev.org/c/openstack/nova/+/780147
08:26:37 openstackgerrit Yongli He proposed openstack/nova master: smartnic support - reject server move and suspend https://review.opendev.org/c/openstack/nova/+/779913
08:26:37 openstackgerrit Yongli He proposed openstack/nova master: smartnic support https://review.opendev.org/c/openstack/nova/+/758944
08:29:51 openstackgerrit Lee Yarwood proposed openstack/nova-specs master: Repropose flavour and image defined ephemeral storage encryption https://review.opendev.org/c/openstack/nova-specs/+/783806
09:12:01 openstackgerrit Lee Yarwood proposed openstack/nova-specs master: Repropose flavour and image defined ephemeral storage encryption https://review.opendev.org/c/openstack/nova-specs/+/783806
09:24:39 openstackgerrit Merged openstack/nova master: Bumping min os-brick ver to 4.3.1 https://review.opendev.org/c/openstack/nova/+/783674
09:28:19 openstackgerrit Lee Yarwood proposed openstack/nova stable/wallaby: Bumping min os-brick ver to 4.3.1 https://review.opendev.org/c/openstack/nova/+/783822
09:28:34 lyarwood gibi: ^
09:40:18 bauzas lyarwood: adding nova to the bug
09:40:38 bauzas lyarwood: and then I'll add the correct tag
09:41:14 bauzas done.
09:50:24 lyarwood bauzas: thanks
09:53:09 openstackgerrit Stephen Finucane proposed openstack/nova-specs master: Repropose configurable-instance-hostnames https://review.opendev.org/c/openstack/nova-specs/+/783827
09:55:04 gibi lyarwood: ack, thanks

Earlier   Later