*** ysandeep|away is now known as ysandeep | 01:19 | |
*** ysandeep is now known as ysandeep|afk | 02:00 | |
*** marios is now known as marios|ruck | 05:12 | |
noonedeadpunk | hberaud: thanks, answered | 05:18 |
---|---|---|
hberaud | noonedeadpunk: thank you | 05:19 |
*** ysandeep|afk is now known as ysandeep | 05:51 | |
noonedeadpunk | hberaud: smth stops us from +W? | 06:47 |
hberaud | noonedeadpunk: Just done. | 07:40 |
opendevreview | Merged openstack/releases master: Create Wallaby branch for OpenStack-Ansible Roles https://review.opendev.org/c/openstack/releases/+/798181 | 07:58 |
noonedeadpunk | awesome, thanks! | 08:11 |
opendevreview | James Page proposed openstack/releases master: osc-lib release 2.4.1 https://review.opendev.org/c/openstack/releases/+/798520 | 08:34 |
*** ykarel is now known as ykarel|lunch | 08:38 | |
hberaud | ttx, elodilles: o/. Any opinion about this trick? => https://review.opendev.org/c/openstack/releases/+/798144 | 09:15 |
elodilles | hberaud: ouch :S that doesn't look good :/ isn't version 1.1.1 good for train for them? | 09:22 |
hberaud | elodilles: no idea | 09:22 |
hberaud | I think that they try to emulate "stable" branches with that | 09:23 |
elodilles | yes, but why? :) | 09:26 |
hberaud | Else if you've some spare time it could be worth to validate these 2 patches too, they have been proposed one month ago and they just have been validated by release liaison => https://review.opendev.org/c/openstack/releases/+/792913 https://review.opendev.org/c/openstack/releases/+/792914 | 09:26 |
elodilles | hberaud: looking | 09:27 |
hberaud | "Why?" I don't know. let me ask them why :) | 09:27 |
elodilles | I've added the question already :) | 09:28 |
elodilles | i'll recheck those month-old patches to have some logs :X | 09:29 |
hberaud | Hm... I don't see your question | 09:29 |
hberaud | thanks | 09:29 |
elodilles | hberaud: well, then I've sent it now o:) | 09:30 |
hberaud | ack | 09:30 |
ttx | hberaud: re that 1.1.2 patch I think that's a bad idea... | 10:08 |
hberaud | ttx: ok thanks for your confirmation | 10:09 |
ttx | how is that even... | 10:09 |
ttx | Independent model just has a single branch, and you release from it | 10:09 |
ttx | How do they do a 1.1.2 on a branch that is at 1.2.0? If that's maintaining a dangling subbranch within the branch... | 10:10 |
ttx | If the goal is to maintain stable branches, we have a model for that and independent is not it | 10:11 |
ttx | If that's some exceptional case, I'd rather create a bugfix branch, then do a out-of-band release for that (ironic style) | 10:11 |
ttx | Here it's like creating an invisible stable branch and releasing from it, within the independent model | 10:12 |
opendevreview | Merged openstack/releases master: [wallaby] Release mistral-dashboard 12.0.1 https://review.opendev.org/c/openstack/releases/+/792913 | 10:31 |
*** ykarel|lunch is now known as ykarel | 10:32 | |
opendevreview | Merged openstack/releases master: [victoria] Release mistral-dashboard 11.0.1 https://review.opendev.org/c/openstack/releases/+/792914 | 10:37 |
hberaud | ttx: I agree, it's like stable branch emulation | 11:14 |
*** amoralej|off is now known as amoralej | 13:05 | |
smcginnis | Something must be broken in our validation job again. There was a check specifically to make sure new release numbers were higher than previous release numbers, so that validations-common 1.1.2 patch should have failed. | 14:09 |
hberaud | I can see a lot of "ERROR: Could not find 1.1.2: Command '['git', 'show', '1.1.2']' returned non-zero exit status 1" | 14:47 |
hberaud | I'm not yet sure if we should expect them where they are raised | 14:48 |
hberaud | I think that they are surely triggered by the checks where we decide to ignore existing tags | 14:50 |
hberaud | (the `skip_if_tag_exist` decorator) | 14:50 |
*** ysandeep is now known as ysandeep|dinner | 14:56 | |
hberaud | smcginnis: Apparently the only check on previous versions increment is a check on the pre release progressions order. | 14:57 |
hberaud | https://opendev.org/openstack/releases/src/branch/master/openstack_releases/cmds/validate.py#L284 | 14:58 |
hberaud | Else we check ancestors with the given hash on the branch membership checks https://opendev.org/openstack/releases/src/branch/master/openstack_releases/cmds/validate.py#L1435 | 14:59 |
hberaud | Without too digging either, apparently this is the closest check that we have | 15:00 |
*** ykarel is now known as ykarel|away | 15:28 | |
*** ysandeep|dinner is now known as ysandeep | 15:41 | |
*** ysandeep is now known as ysandeep|away | 15:44 | |
*** amoralej is now known as amoralej|off | 15:57 | |
*** marios|ruck is now known as marios|out | 16:17 | |
ttx | I need to code up progression checks, that was on my plate | 16:41 |
ttx | but anyone should feel free to beat me to it :) | 16:41 |
hberaud | No enough bandwidth on my side :( | 16:44 |
opendevreview | Brian Rosmaita proposed openstack/releases master: Release cinderlib for wallaby https://review.opendev.org/c/openstack/releases/+/797422 | 21:05 |
dhellmann | ttx, hberaud : there were a few bounces to the release-announce mailing list today, which reminded me to remove myself from the administrator email list. | 22:28 |
Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!