| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2021-03-29 | |||
| 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 | |
| 12:01:55 | sean-k-mooney | stephenfin: do you have a partical wording you would like me to add for the release note in the cpu_max_* patch | |
| 12:28:13 | gibi | stephenfin: I guess it is OK that these docs are only merged for Xena now | |
| 12:33:32 | sean-k-mooney | stephenfin: https://review.opendev.org/c/openstack/nova/+/769614/2//COMMIT_MSG#18 is that better? | |
| 12:34:32 | kashyap | sean-k-mooney: Yeah; I hear ya | |
| 12:50:39 | stephenfin | gibi: I think so, yeah | |
| 12:51:17 | gibi | stephenfin: OK, cool, I'm working through it | |
| 12:54:38 | stephenfin | sean-k-mooney: Yeah, that makes sense. So iiuc what we want to say is that previously we would attempt to best map the guest NUMA topology of pinned instances to the host NUMA topology, but that was buggy and wasn't updated e.g. during live migration so it's been removed? | |