| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2021-03-29 | |||
| 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 https://review.opendev.org/c/openstack/nova/+/758944 | |
| 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 - functional tests https://review.opendev.org/c/openstack/nova/+/780147 | |
| 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 | |
| 09:58:54 | gibi | I don't have the +2 rights to push the wallaby backport through | |
| 09:59:03 | gibi | elod: could you check https://review.opendev.org/c/openstack/nova/+/783822 please? | |
| 10:14:03 | elod | +2+W'd ! | |
| 10:14:11 | gibi | thanks | |
| 10:14:31 | lyarwood | cheers | |
| 10:15:34 | elod | now it just has to merge. and then, release. i guess :X | |
| 10:17:05 | openstackgerrit | Stephen Finucane proposed openstack/nova master: docs: Follow up for SEV doc https://review.opendev.org/c/openstack/nova/+/783830 | |
| 10:17:39 | lyarwood | elod: it's a new rc but yeah | |
| 10:18:43 | stephenfin | kashyap: polite reminder to revisit https://review.opendev.org/c/openstack/nova/+/781438/ today, if you can :) | |
| 10:18:56 | elod | sorry, yes, another release candidate :) | |
| 10:19:49 | stephenfin | gibi: lyarwood: I think I've addressed kashyap's feedback on ^ (via a follow-up), so hopefully we can keep the docs train rolling forward? https://review.opendev.org/q/topic:%22docs%22+status:open | |
| 10:21:22 | lyarwood | yup I'll try to hit it again today, likely this evening if that's okay? | |
| 10:22:07 | stephenfin | for sure, any time in the next month would be okay, so long as they eventually merge and I can move on with my life :) | |
| 10:22:34 | lyarwood | yarp sounds good | |
| 10:28:05 | sean-k-mooney | i know people hate reviewing this patch but it has not passed grenade twice in the last 2 rebases so it appears to be working | |
| 10:28:07 | sean-k-mooney | https://review.opendev.org/c/openstack/nova/+/602432 | |
| 10:56:44 | kashyap | stephenfin: Sorry, I had it open yesterday but got distracted | |
| 10:58:39 | kashyap | stephenfin: Thanks; I see you've worked in almost all the comments here: https://review.opendev.org/c/openstack/nova/+/782410/1/doc/source/admin/cpu-models.rst | |
| 11:00:51 | kashyap | stephenfin: Looks fine to me; just looked | |
| 11:25:07 | kashyap | stephenfin: I'm somewhat ambivalent about the folding in here: https://review.opendev.org/c/openstack/nova/+/782411 | |
| 11:26:06 | kashyap | I'm not against it; but just that sometimes having the separate doc w/ a single focus can be less distracting. One can link to the other, generic CPU modelss doc, though. | |
| 11:31:48 | sean-k-mooney | kashyap: too much linking can make reading them in git/github or pdf form so its a trade off but ya topic docs can be useful. i dont partically like the extream we take it to downstream | |