| *** tosky has quit IRC | 00:05 | |
| *** prometheanfire has quit IRC | 02:24 | |
| *** prometheanfire has joined #openstack-release | 02:25 | |
| *** armax has quit IRC | 02:47 | |
| *** tbarron has quit IRC | 02:52 | |
| *** diablo_rojo has quit IRC | 03:28 | |
| *** ykarel has joined #openstack-release | 04:26 | |
| *** evrardjp has quit IRC | 05:33 | |
| *** evrardjp has joined #openstack-release | 05:33 | |
| *** whoami-rajat___ has joined #openstack-release | 05:36 | |
| *** ricolin has joined #openstack-release | 06:59 | |
| *** ricolin has quit IRC | 07:18 | |
| *** rpittau|afk is now known as rpittau | 07:33 | |
| *** slaweq has joined #openstack-release | 07:42 | |
| *** sboyron has joined #openstack-release | 07:53 | |
| *** tosky has joined #openstack-release | 08:39 | |
| hberaud | fungi: o/ FYI finally I think we need some update on zuul's side, indeed some projects doesn't have docs but use reno but this rule isn't reflected by zuul, cf. this example => https://review.opendev.org/c/openstack/tripleo-heat-templates/+/769263 | 08:53 |
|---|---|---|
| hberaud | fungi: so to match our need I voluntarily added a doc/requirements.txt even if this project doesn't contains doc, but this is not an elegant solution | 08:55 |
| hberaud | I think it would be preferable to properly handle this case in https://opendev.org/zuul/zuul-jobs/src/branch/master/roles/ensure-sphinx/tasks/main.yaml#L36 | 08:55 |
| hberaud | by adding "releasenotes/requirements.txt" before the default case | 08:56 |
| *** sboyron_ has joined #openstack-release | 09:04 | |
| *** sboyron has quit IRC | 09:04 | |
| *** sboyron_ has quit IRC | 09:06 | |
| *** sboyron_ has joined #openstack-release | 09:07 | |
| hberaud | fungi: the releasenotes playbook will call ensure-sphinx ^ so we will pull our requirements from there even with reno => https://opendev.org/zuul/zuul-jobs/src/branch/master/playbooks/releasenotes/pre.yaml#L6 | 09:14 |
| hberaud | fungi: I just proposed a related patch on zuul => https://review.opendev.org/c/zuul/zuul-jobs/+/769292 | 09:28 |
| *** jbadiapa has joined #openstack-release | 09:35 | |
| *** tkajinam has quit IRC | 09:42 | |
| *** e0ne has joined #openstack-release | 10:00 | |
| *** rpittau is now known as rpittau|bbl | 10:32 | |
| *** e0ne has quit IRC | 10:54 | |
| openstackgerrit | Mark Goddard proposed openstack/releases master: Release kolla deliverables Victoria RC2 https://review.opendev.org/c/openstack/releases/+/769322 | 11:17 |
| openstackgerrit | Mark Goddard proposed openstack/releases master: Release kolla deliverables for Ussuri https://review.opendev.org/c/openstack/releases/+/769324 | 11:32 |
| openstackgerrit | Mark Goddard proposed openstack/releases master: Release kolla deliverables for Train https://review.opendev.org/c/openstack/releases/+/769325 | 11:43 |
| openstackgerrit | Mark Goddard proposed openstack/releases master: Release kolla deliverables for Victora https://review.opendev.org/c/openstack/releases/+/769328 | 11:48 |
| *** e0ne has joined #openstack-release | 11:57 | |
| *** sboyron_ has quit IRC | 12:07 | |
| *** sboyron_ has joined #openstack-release | 12:07 | |
| *** tbarron has joined #openstack-release | 12:32 | |
| *** rpittau|bbl is now known as rpittau | 12:44 | |
| *** sboyron_ has quit IRC | 13:08 | |
| *** sboyron_ has joined #openstack-release | 13:09 | |
| *** whoami-rajat___ is now known as whoami-rajat__ | 13:16 | |
| yoctozepto | release-core: dears, please kindly review our kolla releases, thank you! ^ | 14:12 |
| *** tosky has quit IRC | 14:15 | |
| *** tosky_ has joined #openstack-release | 14:16 | |
| *** tosky_ is now known as tosky | 14:16 | |
| ttx | hberaud: see https://review.opendev.org/c/openstack/project-config/+/769353 following our discussion Monday | 14:30 |
| hberaud | ttx: thanks | 14:31 |
| fungi | hberaud: got it, so the release tools were trying to build release notes for projects with no docs? | 14:34 |
| hberaud | yoctozepto: ack, we will have a look, but we currently face some issues with doc/reno jobs, however kolla isn't in the list of impacted projects (c.f http://lists.openstack.org/pipermail/openstack-discuss/2021-January/019611.html). ttx: FYI I think we can go ahead with these projects, any opinions? | 14:35 |
| hberaud | fungi: it's depend... some projects yes some other not | 14:36 |
| hberaud | fungi: I found few projects that build release notes without any docs | 14:37 |
| hberaud | so I think this use case should be handled too | 14:37 |
| fungi | and also have broken test-requirements.txt files? | 14:37 |
| hberaud | nope | 14:37 |
| fungi | okay, so they're not affected by the current problem then? | 14:38 |
| hberaud | normally yes | 14:38 |
| hberaud | but I didn't tested all of them | 14:38 |
| hberaud | however I think it could be worth to allow to split reqs for reno like we already do for docs | 14:41 |
| hberaud | else projects need to pull everything just to build release notes and that increase the likelihood to face a resolver issue | 14:42 |
| yoctozepto | hberaud: thanks :-) | 14:42 |
| hberaud | and with that we limit the chances to get simialr side effects on our release pipeline | 14:44 |
| openstackgerrit | Thierry Carrez proposed openstack/releases master: Website: Add link to the git repository https://review.opendev.org/c/openstack/releases/+/769355 | 14:44 |
| fungi | hberaud: sure, i agree being able to override and specify releasenotes/requirements.txt might be helpful to projects who just build release notes | 14:44 |
| fungi | so it would be searched in this order: doc/requirements.txt releasenotes/requirements.txt test-requirements.txt | 14:44 |
| hberaud | yes, that what I proposed in my patch | 14:46 |
| hberaud | https://review.opendev.org/c/zuul/zuul-jobs/+/769292 | 14:47 |
| hberaud | also another option could be to override the "Get requirements files" only for the releasenotes playbook instead of inherit everything from the ensure-sphinx role | 14:49 |
| fungi | ahh, cool, for some reason i didn't see gerritbot echo that one | 14:49 |
| hberaud | following this way could allow us to limit the side effect on the doc side | 14:49 |
| hberaud | but for now I think that these changes solve 98% of our problem | 14:50 |
| hberaud | my main concern is what will happen if doc/requirements doesn't exist but if the project build doc anyway... | 14:52 |
| *** armax has joined #openstack-release | 14:57 | |
| fungi | well, as you noted, 769292 isn't going to fix any of the failing release notes builds because you say the projects building release notes with no doc/requirements.txt files don't have broken test-requirements.txt files | 14:57 |
| fungi | if the project builds docs then doc/requirements.txt *should* exist because the pti says so | 14:57 |
| hberaud | yes good point | 14:57 |
| fungi | i don't think the release team needs to worry about alternatives to that case, if projects don't follow the pti then the release jobs can't support them | 14:58 |
| fungi | and the release jobs shouldn't get unnecessarily complicated in order to support projects not following guidelines | 14:58 |
| hberaud | I was not sure to cover 100% of the case so I prefered to think about this particular use case | 14:59 |
| hberaud | ok | 14:59 |
| hberaud | s/case/cases/ | 14:59 |
| *** yoctozepto has quit IRC | 15:43 | |
| *** yoctozepto has joined #openstack-release | 15:44 | |
| *** sboyron_ has quit IRC | 15:51 | |
| openstackgerrit | Mark Goddard proposed openstack/releases master: Release kolla deliverables for Victoria https://review.opendev.org/c/openstack/releases/+/769328 | 15:52 |
| *** e0ne has quit IRC | 15:56 | |
| *** e0ne has joined #openstack-release | 15:57 | |
| *** sboyron has joined #openstack-release | 16:02 | |
| *** dave-mccowan has quit IRC | 16:03 | |
| *** dave-mccowan has joined #openstack-release | 16:07 | |
| *** sboyron has quit IRC | 16:17 | |
| *** sboyron has joined #openstack-release | 16:17 | |
| *** e0ne has quit IRC | 16:28 | |
| *** e0ne has joined #openstack-release | 17:00 | |
| *** rpittau is now known as rpittau|afk | 17:08 | |
| *** sboyron has quit IRC | 17:22 | |
| *** sboyron has joined #openstack-release | 17:24 | |
| *** ykarel has quit IRC | 17:36 | |
| ttx | https://review.opendev.org/c/openstack/project-config/+/769353 was merged, so keep an eye open in the very unlikely case it creates a regression. | 18:01 |
| hberaud | ack | 18:06 |
| *** e0ne has quit IRC | 18:30 | |
| *** whoami-rajat__ has quit IRC | 18:39 | |
| *** mgoddard has quit IRC | 18:45 | |
| *** sboyron has quit IRC | 20:06 | |
| *** slaweq has quit IRC | 20:33 | |
| *** dave-mccowan has quit IRC | 20:37 | |
| *** tosky_ has joined #openstack-release | 22:12 | |
| *** tosky has quit IRC | 22:13 | |
| openstackgerrit | Pierre Riteau proposed openstack/releases master: [Tempest Plugins] Tag stein EM https://review.opendev.org/c/openstack/releases/+/768538 | 22:16 |
| *** tosky_ is now known as tosky | 22:17 | |
| *** tkajinam has joined #openstack-release | 23:01 | |
| *** dave-mccowan has joined #openstack-release | 23:15 | |
Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!