| opendevreview | OpenStack Proposal Bot proposed openstack/requirements master: update constraint for os-refresh-config to new release 14.1.0 https://review.opendev.org/c/openstack/requirements/+/1008251 | 11:40 |
|---|---|---|
| opendevreview | OpenStack Proposal Bot proposed openstack/requirements master: update constraint for os-apply-config to new release 14.1.0 https://review.opendev.org/c/openstack/requirements/+/1008252 | 11:44 |
| opendevreview | OpenStack Proposal Bot proposed openstack/requirements master: update constraint for os-collect-config to new release 14.1.0 https://review.opendev.org/c/openstack/requirements/+/1008253 | 11:46 |
| mnasiadka | Ok then, feature freeze done, release done, time to get back into business I guess | 11:56 |
| frickler | mnasiadka: I was hoping for a few more quiet days, but yeah ... could you take a look at https://review.opendev.org/c/openstack/requirements/+/996441 ? IMO it would be good to get this done soon to unblock some of the waiting changes | 12:08 |
| mnasiadka | (Or make them redundant :) ) | 12:08 |
| frickler | unblock changes in other projects I meant | 12:08 |
| mnasiadka | right | 12:10 |
| opendevreview | Michal Nasiadka proposed openstack/requirements master: Do not fail on optional dependencies https://review.opendev.org/c/openstack/requirements/+/996441 | 12:18 |
| mnasiadka | frickler: we could merge https://review.opendev.org/c/openstack/requirements/+/1000078 now? | 12:19 |
| frickler | mnasiadka: I'd prefer to wait for the whole stack to be ready, so that we don't have to wait too long for the git blame ignore update | 12:27 |
| mnasiadka | Ok I think that would address the current ruff ,,failure’’ which is not failing - but I’m fine with waiting | 12:30 |
| frickler | yeah, it might semi-conflict with 996441, but otherwise I don't think we have relevant code changes pending, or did I miss some? I'd still suggest to go for 996441 first | 12:52 |
| frickler | and then I'd be interested in your and others' opinion on https://review.opendev.org/c/openstack/requirements/+/1007742 | 12:52 |
| mnasiadka | stephenfin: if you could check if changes on 996441 make you happier - I guess we could merge it ;-) | 13:09 |
| stephenfin | sure. Looking now. 1000078 doesn't conflict and should be mergeable: conflicts are later in the series, and whether we want to do those is still TBD (per my discussions with frickler on same) | 13:15 |
| stephenfin | mnasiadka: Left two questions on https://review.opendev.org/c/openstack/requirements/+/996441 | 13:27 |
| mnasiadka | Will have a look later today, thanks | 13:28 |
| stephenfin | mnasiadka: I'm leaving a few suggestions on the change. Would you be offened if I just pushed an alternative with your co-authored-by on it? | 13:29 |
| stephenfin | *offended | 13:30 |
| stephenfin | I can just leave the suggestions if so | 13:30 |
| mnasiadka | I’m happy for you to fix it, doing too many things at the same time, so no brainer I mixed up something :) | 13:30 |
| opendevreview | Stephen Finucane proposed openstack/requirements master: Remove dead code https://review.opendev.org/c/openstack/requirements/+/1008270 | 14:13 |
| opendevreview | Stephen Finucane proposed openstack/requirements master: Remove unused property https://review.opendev.org/c/openstack/requirements/+/1008271 | 14:13 |
| opendevreview | Stephen Finucane proposed openstack/requirements master: Remove support for legacy requirements files https://review.opendev.org/c/openstack/requirements/+/1008274 | 14:16 |
| opendevreview | Stephen Finucane proposed openstack/requirements master: Do not fail on optional dependencies https://review.opendev.org/c/openstack/requirements/+/996441 | 14:16 |
| mnasiadka | frickler: I’d say it would be nice if you could review https://review.opendev.org/c/openstack/requirements/+/996441 after stephenfin’s changes | 16:00 |
| opendevreview | Stephen Finucane proposed openstack/requirements master: Denylist yet more types-* packages https://review.opendev.org/c/openstack/requirements/+/1007890 | 17:11 |
| opendevreview | Merged openstack/requirements master: Remove dead code https://review.opendev.org/c/openstack/requirements/+/1008270 | 17:25 |
| opendevreview | Merged openstack/requirements master: Remove unused property https://review.opendev.org/c/openstack/requirements/+/1008271 | 17:25 |
| opendevreview | Takashi Kajinami proposed openstack/requirements master: Ignore sys_platform!='win32' https://review.opendev.org/c/openstack/requirements/+/1008323 | 18:16 |
Generated by irclog2html.py 4.1.0 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!