| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2022-02-08 | |||
| 16:19:12 | sean-k-mooney | not local to the current doc? | |
| 16:19:14 | gibi | sean-k-mooney: something like that | |
| 16:19:22 | sean-k-mooney | that would be a problem ya | |
| 16:19:48 | stephenfin | shouldn't have happened in minor release or arguably at all. Context in the upstream bug https://github.com/sphinx-doc/sphinx/issues/10177 | |
| 16:19:58 | bauzas | just to make it clear, that means we have homework before accepting 4.4.x, right? | |
| 16:19:58 | sean-k-mooney | ack | |
| 16:20:11 | sean-k-mooney | well or we can see if we can get this reverted | |
| 16:20:14 | sean-k-mooney | or made configurable | |
| 16:20:15 | bauzas | stephenfin: ah I see | |
| 16:20:25 | gibi | sean-k-mooney: yepp, stephenfin proposed the revert :) | |
| 16:20:42 | sean-k-mooney | if we need too its proably scriptable to make them unique | |
| 16:20:55 | sean-k-mooney | but ya lets see what happens with stephenfin pr | |
| 16:21:23 | gmann | another gate failure we discussed in QA meeting is about failure in devstack-platform-centos-9-stream job which seems 100 % #link https://zuul.openstack.org/builds?job_name=devstack-platform-centos-9-stream&skip=0 | |
| 16:21:38 | gmann | rescue server test failing on volume detach #link https://a886e0e70a23f464643f-7cd608bf14cafb686390b86bc06cde2a.ssl.cf1.rackcdn.com/827576/6/check/devstack-platform-centos-9-stream/53de74e/testr_results.html | |
| 16:21:57 | gmann | I have not checked log yet, in case anyone aware of it? | |
| 16:22:42 | gibi | that feels new | |
| 16:22:48 | gibi | at least to me | |
| 16:24:11 | gmann | ok, let me see if I get any info in logs and then will report bug | |
| 16:24:22 | gibi | thanks | |
| 16:24:22 | bauzas | the error seems weird | |
| 16:24:59 | bauzas | InvalidVolume | |
| 16:25:20 | bauzas | but a timeout | |
| 16:25:46 | bauzas | which indicates something bad happening on the cinder side | |
| 16:26:50 | gibi | affraid it is libvirt device detach issue again | |
| 16:26:51 | gibi | https://zuul.openstack.org/build/3e24d977991d4536b6279afd7f3b5d56/log/controller/logs/screen-n-cpu.txt?severity=4#49433 | |
| 16:28:21 | bauzas | gibi: on centos then ? | |
| 16:28:53 | gibi | for some reason it is hitting only that centos job but hitting it hard | |
| 16:29:03 | gmann | yeah | |
| 16:29:59 | bauzas | ok, I guess we need to investigate more, next | |
| 16:30:07 | bauzas | #topic Release Planning | |
| 16:30:12 | bauzas | #link https://releases.openstack.org/yoga/schedule.html#y-ff FeatureFreeze in 2.5 weeks | |
| 16:30:17 | bauzas | #link https://etherpad.opendev.org/p/nova-yoga-blueprint-status Etherpad for blueprints tracking | |
| 16:30:30 | bauzas | thanks gibi for having made a couple of insighful reviews | |
| 16:30:36 | bauzas | I started to review as well | |
| 16:31:14 | bauzas | folks, just a reminder to use this etherpad for coordinating review requests and comments | |
| 16:31:41 | bauzas | any particular BP someone wanting to address by now ? (provided this is not about asking reviews) | |
| 16:32:32 | gmann | added link for policy refresh 2 BP and moved up for review section #link https://etherpad.opendev.org/p/nova-yoga-blueprint-status#L40 | |
| 16:32:49 | gmann | It is not complete yet, I am working on system_reader conversion | |
| 16:33:00 | gmann | but existing patches is up for review | |
| 16:33:12 | gmann | dansmith: ^^ I am +2 on your patches for server policy | |
| 16:33:17 | opendevreview | Merged openstack/nova-specs master: Avoid Sphinx 4.4.0 to fix specs doc build https://review.opendev.org/c/openstack/nova-specs/+/828368 | |
| 16:33:24 | bauzas | gmann: thanks | |
| 16:33:36 | dansmith | gmann: ack, I've been wondering when that was going to go | |
| 16:33:41 | bauzas | anyone wanting to discuss now about priorities ? | |
| 16:33:47 | dansmith | bauzas: want a note on the quotas stuff? | |
| 16:33:54 | gmann | dansmith: lot of test to fix so taking time but I am working on remaining policies | |
| 16:33:58 | bauzas | dansmith: I started looking at your comment | |
| 16:34:07 | bauzas | on the next patch I wanted to review | |
| 16:34:22 | dansmith | bauzas: well, | |
| 16:34:28 | bauzas | dansmith: those are reasonable concerns but I started reviewing this patch too | |
| 16:34:41 | dansmith | melwitt and I discussed a few refactoring changes | |
| 16:34:50 | dansmith | after my first review, and we're still working on hammering those out | |
| 16:35:20 | bauzas | dansmith: given the series, do you feel she needs to rebase his change or can she work on a follow-up patch ? | |
| 16:35:30 | bauzas | (well, can ping melwitt actually for the answer) | |
| 16:35:33 | dansmith | I'm working on the rebase now, | |
| 16:35:45 | dansmith | but yeah, we're actually going to drop the latest PS, back to the previous and iterate from there | |
| 16:35:52 | dansmith | I'm almost done with that actually, | |
| 16:36:03 | dansmith | but want to sync with her when she's around this morning before I push that major change up :) | |
| 16:36:14 | bauzas | dansmith: ok, then I'll look at other bps tonight and I'll look back at the unified limits one tomorrow | |
| 16:36:28 | bauzas | thanks for the notice | |
| 16:36:56 | dansmith | aye | |
| 16:37:14 | bauzas | fwiw, I thought about a simple thing for giving ourselves visibility during the last 3-week sprint | |
| 16:37:46 | bauzas | if cores don't disagree, I'd recommend cores to mark their IRC nicks on the BPs they'd like to review this week | |
| 16:37:52 | bauzas | no obligation | |
| 16:38:07 | bauzas | just a matter of giving a bit of visibility for the people who want | |
| 16:38:37 | bauzas | feel it like it's runways or whatever name it is | |
| 16:39:12 | bauzas | it's just about adding a bullet point below each blueprint on the blueprint etherpad for people willing to mark their names | |
| 16:39:30 | bauzas | another option could be the Review-Priority flag :) | |
| 16:40:11 | bauzas | what folks think about it ? maybe this is a bit early ? | |
| 16:40:35 | gibi | I can do both | |
| 16:41:38 | gmann | either is fine, I will review tenant-id series this week | |
| 16:41:38 | bauzas | I don't want to ask for duplicate things | |
| 16:42:12 | bauzas | I feel we can use the review-prio flag then | |
| 16:42:24 | bauzas | but that means we need to clean it up first | |
| 16:43:41 | gibi | there is not much marked with review priority at the moment | |
| 16:43:56 | bauzas | ok, then let's jump straight to the next topîc | |
| 16:44:22 | bauzas | #topic Review priorities | |
| 16:44:28 | bauzas | #link https://review.opendev.org/q/status:open+(project:openstack/nova+OR+project:openstack/placement+OR+project:openstack/os-traits+OR+project:openstack/os-resource-classes+OR+project:openstack/os-vif+OR+project:openstack/python-novaclient+OR+project:openstack/osc-placement)+label:Review-Priority%252B1 | |
| 16:45:03 | gibi | pack and spread, remote-manged and unified limits are all has core review attention so they are good having the prio label | |
| 16:45:19 | bauzas | right | |
| 16:45:24 | gibi | the other patches in that list feels less important now | |
| 16:45:58 | bauzas | pack-n-spread doesn't match a bp | |
| 16:46:35 | bauzas | but I'm ok with treating it as some kind of feature request | |
| 16:46:50 | bauzas | anyway, let's overthink that | |
| 16:47:24 | bauzas | sean-k-mooney: can you drop r-p flag on https://review.opendev.org/c/openstack/os-vif/+/765912 and https://review.opendev.org/c/openstack/os-vif/+/765970 | |
| 16:47:41 | bauzas | those are old patches that fail CI, hence meaningless to be 'priorities' | |
| 16:48:17 | sean-k-mooney | yes | |
| 16:48:20 | bauzas | https://review.opendev.org/c/openstack/os-resource-classes/+/819207 will be tonight's priority for me | |
| 16:48:29 | bauzas | so hopefully we could merge it | |
| 16:48:46 | sean-k-mooney | done | |
| 16:48:56 | bauzas | that would leave the other changes having r-p matching blueprints | |
| 16:49:00 | opendevreview | Dmitrii Shcherbakov proposed openstack/nova master: Bump os-traits to 2.7.0 https://review.opendev.org/c/openstack/nova/+/826675 | |
| 16:49:01 | opendevreview | Dmitrii Shcherbakov proposed openstack/nova master: Add supports_remote_managed_ports capability https://review.opendev.org/c/openstack/nova/+/827839 | |
| 16:49:01 | opendevreview | Dmitrii Shcherbakov proposed openstack/nova master: Filter computes without remote-managed ports early https://review.opendev.org/c/openstack/nova/+/812111 | |
| 16:49:02 | opendevreview | Dmitrii Shcherbakov proposed openstack/nova master: [yoga] Add support for VNIC_REMOTE_MANAGED https://review.opendev.org/c/openstack/nova/+/824835 | |
| 16:50:22 | bauzas | let's finish this discussion next week and make the r-p flag be the tool for coordinating review priorities for this feature freeze | |
| 16:50:34 | bauzas | agreed ? | |
| 16:50:48 | gibi | works for me | |
| 16:51:22 | bauzas | anyway, moving on, time is flying | |
| 16:51:37 | bauzas | #topic Stable Branches | |