| *** markmcclain has quit IRC | 00:13 | |
| *** markmcclain has joined #openstack-requirements | 00:15 | |
| *** markmcclain has quit IRC | 00:24 | |
| *** smcginnis has quit IRC | 02:05 | |
| *** timburke has quit IRC | 02:05 | |
| *** prometheanfire has quit IRC | 02:06 | |
| *** smcginnis has joined #openstack-requirements | 02:06 | |
| *** prometheanfire has joined #openstack-requirements | 02:07 | |
| *** timburke has joined #openstack-requirements | 02:11 | |
| *** vishalmanchanda has joined #openstack-requirements | 03:30 | |
| *** evrardjp has quit IRC | 04:33 | |
| *** evrardjp has joined #openstack-requirements | 04:33 | |
| *** dmellado has quit IRC | 06:11 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/requirements master: Updated from generate-constraints https://review.opendev.org/746837 | 06:17 |
|---|---|---|
| openstackgerrit | Matthew Thode proposed openstack/requirements master: Updated from generate-constraints https://review.opendev.org/746837 | 06:28 |
| *** dmellado has joined #openstack-requirements | 06:41 | |
| *** dmellado has quit IRC | 07:07 | |
| *** hberaud has quit IRC | 07:36 | |
| *** hberaud has joined #openstack-requirements | 07:37 | |
| *** e0ne has joined #openstack-requirements | 07:51 | |
| *** dmellado has joined #openstack-requirements | 07:53 | |
| *** dtantsur|afk is now known as dtantsur | 08:16 | |
| *** tosky has joined #openstack-requirements | 08:28 | |
| yoctozepto | morning; do requirements-checks differ between stable and master branches? asking because https://review.opendev.org/746852 hit 'yamllint not in lower' error but the same state on master did not | 08:41 |
| yoctozepto | as in: is master ignoring l-c checks? | 08:41 |
| yoctozepto | (on purpose?) | 08:41 |
| smcginnis | yoctozepto: There are stable branches for requirements since they differ over time. But lower-constraints is something that is in-repo, so that wouldn't be a global requirements thing. | 11:49 |
| smcginnis | yoctozepto: Not sure how it was able to get to that state, but it does look like yamllint needs to be added to lower-constraints.txt. | 11:50 |
| smcginnis | Checks are only run when one of the requirements files are modified, so that may explain why it wasn't found before now. Could be it was like that for a long time, but there haven't been any requirements modifications until now to find that. | 11:51 |
| yoctozepto | smcginnis: that could be true, except it's like this on master and it was happy an hour before that :-) | 12:49 |
| yoctozepto | smcginnis: ok, I found the culprit: https://opendev.org/openstack/requirements/src/branch/master/blacklist.txt vs https://opendev.org/openstack/requirements/src/branch/stable/ussuri/blacklist.txt | 12:50 |
| yoctozepto | smcginnis: but still wonder how that came to be :-) | 12:50 |
| yoctozepto | smcginnis: ok, I think I now (quick thinker) - we cut ussuri late, it was probably cut off like this but we had req-checks already with victoria | 12:51 |
| yoctozepto | know* | 12:51 |
| yoctozepto | we=kolla | 12:51 |
| openstackgerrit | Radosław Piliszek proposed openstack/requirements stable/ussuri: Add yamllint to blacklist https://review.opendev.org/746916 | 12:53 |
| yoctozepto | smcginnis: https://review.opendev.org/746916 - would you accept that? I know it's unusual but so is kolla | 12:53 |
| smcginnis | yoctozepto: Yep, that looks right. | 12:55 |
| smcginnis | Yeah, that's the risk with cycle-trailing I guess. Once requirements unfreezes at the start of a new cycle, trailing repos are no longer actually testing with the official requirements for their target branch. | 12:56 |
| smcginnis | I might have to think about how we could handle that better. It's actually kind of scary now that I think of it. | 12:56 |
| yoctozepto | smcginnis: we try to release early but this time we got this mixed in | 12:57 |
| jrosser | for openstack-ansible we branch as soon as possible and then accept that we just have to backport tons and tons of stuff to that branch | 12:57 |
| yoctozepto | smcginnis: true that, osa had a very long delay so they were on ice | 12:57 |
| smcginnis | yoctozepto: Yeah. There are other cycle-trailing ones that could potentially hit this. | 12:57 |
| yoctozepto | jrosser: but it still goes via master | 12:57 |
| jrosser | right, so it's the usual pattern of merge master and backport/fix as necessary | 12:58 |
| jrosser | which is work | 12:58 |
| jrosser | but it's less work than not branching and ending up in a huge mess when you do | 12:58 |
| smcginnis | Now that we have redirects in place, I could see having bot proposed patches to cycle-trailing deliverables that sets the upper-constraints path to the new series. Then when requirements branches, the redirect would automatically pick that up and it would always test against the correct set of requirements. | 12:58 |
| smcginnis | Then you wouldn't have to branch early and backport a lot. | 12:58 |
| yoctozepto | jrosser: haha, true that; we could try that next time, we don't like the 'release candidate' thing though, but with new release model we could get away from that | 12:58 |
| smcginnis | But yeah, until we get that, branching early may be the best bet. | 12:59 |
| jrosser | in particular this cycle we have only just released, basically at the deadline | 12:59 |
| jrosser | so it's been pretty bad tbh | 12:59 |
| yoctozepto | jrosser: I'm glad you did, so that it's no longer only kolla vs tripleo on centos8 | 13:00 |
| yoctozepto | diversity is nice | 13:00 |
| jrosser | sure yes - it was centos8 that delayed things, we had no choice but to land it for U as we didnt for T | 13:01 |
| jrosser | and it was messy | 13:01 |
| jrosser | also made worse that we had to support centos7 for U in order to give our centos users a transition release | 13:01 |
| yoctozepto | jrosser: oh my, py3 on centos7? ;d | 13:02 |
| yoctozepto | jrosser: kolla kept train as dual support release | 13:02 |
| jrosser | yep, we have the whole thing running py3 on centos7 | 13:02 |
| yoctozepto | jrosser: and we kept py2 in it | 13:02 |
| jrosser | which is "suprising" | 13:02 |
| yoctozepto | jrosser: yup, but whatever works | 13:02 |
| yoctozepto | jrosser: it's only temporary anyhow | 13:02 |
| yoctozepto | (or at least we all hope!) | 13:03 |
| jrosser | yes it is, and thankfully we can take that all out for V becasue it's introduced a lot of tech debt keeping the centos7 support in | 13:03 |
| yoctozepto | jrosser: you don't say :D doubled images for Train are mind-boggling as well; I'm glad we finally harnessed the thing though; and now only py3 and only modern (2019+) distros | 13:06 |
| yoctozepto | prometheanfire, hberaud: if you could have a look at: https://review.opendev.org/746916 :-) | 13:10 |
| hberaud | yoctozepto: ack | 13:57 |
| *** KeithMnemonic has joined #openstack-requirements | 14:09 | |
| yoctozepto | hberaud: thanks | 14:48 |
| *** kberger_ has joined #openstack-requirements | 15:08 | |
| *** KeithMnemonic has quit IRC | 15:10 | |
| yoctozepto | prometheanfire: thanks | 15:51 |
| *** dtantsur is now known as dtantsur|afk | 17:01 | |
| *** e0ne has quit IRC | 17:44 | |
| openstackgerrit | Merged openstack/requirements stable/ussuri: Add yamllint to blacklist https://review.opendev.org/746916 | 18:01 |
| openstackgerrit | Radosław Piliszek proposed openstack/requirements stable/ussuri: Add ansible to blacklist https://review.opendev.org/747012 | 18:31 |
| *** vishalmanchanda has quit IRC | 18:49 | |
| prometheanfire | #startmeeting requirements | 20:32 |
| openstack | Meeting started Wed Aug 19 20:32:10 2020 UTC and is due to finish in 60 minutes. The chair is prometheanfire. Information about MeetBot at http://wiki.debian.org/MeetBot. | 20:32 |
| openstack | Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. | 20:32 |
| *** openstack changes topic to " (Meeting topic: requirements)" | 20:32 | |
| openstack | The meeting name has been set to 'requirements' | 20:32 |
| prometheanfire | #topic rollcall | 20:32 |
| *** openstack changes topic to "rollcall (Meeting topic: requirements)" | 20:32 | |
| prometheanfire | tonyb, prometheanfire, dirk, smcginnis ping | 20:32 |
| prometheanfire | o/ | 20:32 |
| smcginnis | o/ | 20:33 |
| prometheanfire | #topic Any controversies in the Queue? | 20:33 |
| *** openstack changes topic to "Any controversies in the Queue? (Meeting topic: requirements)" | 20:33 | |
| prometheanfire | https://review.opendev.org/746434 | 20:33 |
| prometheanfire | once that is fixed (move the entry to the unapproved section) I think it's fine | 20:34 |
| prometheanfire | I personally don't like adding another config parser and would think that openstack projects would use oslo.config only (for this review https://review.opendev.org/746436 ) | 20:34 |
| smcginnis | I would hope so too. I saw adjutant was proposing an alternative config package though. | 20:35 |
| smcginnis | I would much rather they work with oslo to try to see if they can get what they need in oslo.config rather than adding their own custom library. | 20:35 |
| prometheanfire | yep | 20:36 |
| smcginnis | FWIW, I agree with your comments on the toml lib thing. | 20:36 |
| prometheanfire | anything else before open floor | 20:36 |
| smcginnis | No, I think that's it. | 20:36 |
| prometheanfire | #topic Open Floor | 20:36 |
| *** openstack changes topic to "Open Floor (Meeting topic: requirements)" | 20:37 | |
| smcginnis | There was some discussion earlier about cycle-trailing projects not branching, and therefore testing with the wrong branch requirements. | 20:37 |
| smcginnis | Which then causes things to suddenly break when they do branch and start using the correct set of requirements. | 20:38 |
| smcginnis | I haven't been able to focus on that yet, but I think we might be able to have some sort of trigger that pre-updates cycle-trailing repos, rather than the post-branching we do for normal things. | 20:39 |
| prometheanfire | well... they should branch, or temporarilly change their master to point to what they are deving against | 20:39 |
| dirk | o/ | 20:39 |
| smcginnis | Yeah. | 20:39 |
| smcginnis | But I was thinking when they branch, we could probably have a bot proposed patch to update their master to the next branch name. Since we have the redirect in place, that would mean they are testing against master. | 20:39 |
| smcginnis | Then when requirements actually does branch, months later, they automatically will still be testing against the right set of constraints. | 20:40 |
| prometheanfire | ya | 20:40 |
| smcginnis | Then when they branch again, update master to the next series name, rinse, lather, repeat. | 20:40 |
| smcginnis | Anyway, just something I was thinking about. If I can focus on it for more than 2 minutes, I might see how much it would take to get something like that in place. | 20:41 |
| prometheanfire | anything else? | 20:41 |
| prometheanfire | :D | 20:41 |
| prometheanfire | ya, focus... | 20:41 |
| smcginnis | Hey dirk! o/ | 20:41 |
| smcginnis | Nothing else from me. | 20:41 |
| prometheanfire | #endmeeting | 20:41 |
| *** openstack changes topic to "OpenStack Requirements - IRC meetngs on Wednesdays @ 07:00 UTC in here in #openstack-requirements - See agenda @ http://tinyurl.com/h44ryuw - IRC channel is *LOGGED* @ http://tinyurl.com/j38rk24" | 20:42 | |
| openstack | Meeting ended Wed Aug 19 20:41:59 2020 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 20:42 |
| openstack | Minutes: http://eavesdrop.openstack.org/meetings/requirements/2020/requirements.2020-08-19-20.32.html | 20:42 |
| openstack | Minutes (text): http://eavesdrop.openstack.org/meetings/requirements/2020/requirements.2020-08-19-20.32.txt | 20:42 |
| openstack | Log: http://eavesdrop.openstack.org/meetings/requirements/2020/requirements.2020-08-19-20.32.log.html | 20:42 |
| prometheanfire | also, hi dirk | 20:42 |
| smcginnis | If you haven't seen Dirk Gently on Hulu, I would recommend it. Didn't read the books, but it's got a good Hitchhikers Guide vibe to it. | 20:43 |
| dirk | Hi all :-) | 20:43 |
| dirk | smcginnes: I think I have ran across advertisement about it already | 20:45 |
| dirk | Its also on netflix. And I have vacation upcoming. Dangerous combination | 20:46 |
| smcginnis | ;) | 20:46 |
| *** dustinc has quit IRC | 20:49 | |
| *** tosky has quit IRC | 21:47 | |
Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!