| opendevreview | Jon Bernard proposed openstack/releases master: Cinder team releases for stable/yoga https://review.opendev.org/c/openstack/releases/+/848794 | 00:01 |
|---|---|---|
| opendevreview | Dr. Jens Harbott proposed openstack/releases master: Tag monthly kolla stable releases https://review.opendev.org/c/openstack/releases/+/848811 | 07:04 |
| frickler | elodilles: how's the situation with the kayobe eol stack now? can we merge the first ones and see if that changes test results for the others? | 07:07 |
| *** amoralej|off is now known as amoralej | 07:22 | |
| elodilles | frickler: yes, i've +2'd them if i remember well | 07:56 |
| elodilles | hberaud ttx : when you have time could you please review the following patches: | 08:01 |
| elodilles | last remaining yoga trailing release: https://review.opendev.org/c/openstack/releases/+/847598 | 08:01 |
| elodilles | some EOL'ing stuff: https://review.opendev.org/c/openstack/releases/+/848386 && https://review.opendev.org/c/openstack/releases/+/848386 | 08:02 |
| elodilles | and any opinion about this patch? https://review.opendev.org/c/openstack/releases/+/848628 | 08:02 |
| opendevreview | Merged openstack/releases master: Release OpenStack-Ansible Yoga https://review.opendev.org/c/openstack/releases/+/847598 | 08:13 |
| frickler | elodilles: ah, sorry, wasn't sure what this is waiting for, but then it's fine I guess | 08:15 |
| ttx | elodilles: looking | 09:04 |
| hberaud | elodilles: concerning tap as a service, I think we don't want to add new deliverables files on stable branches | 09:06 |
| ttx | good point | 09:06 |
| hberaud | I remember similar previous situation, where, if I remember correctly, we rejected this kind of request | 09:07 |
| ttx | We usually do not modify the "content" of an old release | 09:07 |
| ttx | Xena did not contain tap-as-a-service | 09:07 |
| ttx | it's too late to change history | 09:07 |
| hberaud | +1 | 09:07 |
| elodilles | hberaud ttx : thanks! sounds logical | 09:10 |
| elodilles | i guess the eol tags are fine though, aren't they? | 09:10 |
| elodilles | as we did that already (which means we added deliverable files, but without releases) | 09:12 |
| opendevreview | Merged openstack/releases master: EOL Kayobe Pike https://review.opendev.org/c/openstack/releases/+/848386 | 09:12 |
| elodilles | like this ^^^ o:) | 09:12 |
| opendevreview | Hervé Beraud proposed openstack/releases master: Some rules about adding new deliverable file https://review.opendev.org/c/openstack/releases/+/848822 | 09:16 |
| hberaud | I think the kayobe story is a bit different, I think this file was forgot, Kayobe's deliverable was there with ocata https://opendev.org/openstack/releases/src/branch/master/deliverables/ocata/kayobe.yaml | 09:19 |
| hberaud | Whereas tap as a service became a new deliverable with yoga | 09:22 |
| elodilles | but from EOL perspective they are in the same situation i think | 09:23 |
| elodilles | so i mean: no release (tags) should be added to tap-as-a-service, but we can add the yamls with EOL tag only | 09:24 |
| hberaud | Hm, given that t-a-a-s already have tags that were generated under the x namespace then yes I think you are right we could EOL them by adding this yamls file. It's not as the first release was during Yoga. | 09:26 |
| hberaud | However, as you said, we should reject new tags (other than EOL tags) on the all series where this deliverable file is added retrospectively. | 09:28 |
| elodilles | hberaud: ack, i agree | 09:29 |
| elodilles | sounds good to me | 09:29 |
| hberaud | I'm not aware of any other similar previous situation ^, ttx do you agree with that? | 09:30 |
| hberaud | (where a project was moved from the x namespace (with existing releases), to the openstack namespace) | 09:30 |
| elodilles | not that usual situation, i would say :) | 09:51 |
| elodilles | meanwhile it seems kayobe's pike eol tagging job has failed: https://zuul.opendev.org/t/openstack/build/2d7b1b3b7748425b9c33c3b79d277729 | 09:52 |
| elodilles | the log does not say much | 09:53 |
| elodilles | but checking the repos i see that there's no pike-eol @ kayobe-config | 09:53 |
| elodilles | for kayobe and kayobe-config-dev the pike-eol tags exist | 09:54 |
| elodilles | hmm, i think there is some general problem with kayobe-config repository, as the ocata-eol tag is missing the same way. do our scripts have tagging rights for kayobe-config? | 10:01 |
| elodilles | strange, because the same gerrit acls are used for kayobe-config and kayobe-config-dev + kayobe. while kayobe-config tagging fails, tagging of kayobe-config-dev and kayobe succeeds... | 10:21 |
| *** amoralej is now known as amoralej|lunch | 11:05 | |
| *** dviroel|out is now known as dviroel | 11:25 | |
| elodilles | fungi: btw, do you have any idea why tagging could fail for kayobe-config (compared to kayobe and kayobe-config-dev, which share the same gerrit acls)? see the above 6 msg from me for context ^^^ | 12:09 |
| *** amoralej|lunch is now known as amoralej | 12:39 | |
| fungi | elodilles: i think it's this: https://zuul.opendev.org/t/openstack/build/2d7b1b3b7748425b9c33c3b79d277729/log/job-output.txt#765-767 | 12:45 |
| elodilles | fungi: oh, i did not notice that! :S | 12:47 |
| elodilles | thanks! | 12:47 |
| fungi | yw | 12:47 |
| elodilles | should we "force-merge" .gitreview for kayobe-config? or what could be the best solution? | 12:48 |
| elodilles | or simply manual tagging should do the job? :/ | 12:48 |
| opendevreview | Jon Bernard proposed openstack/releases master: Cinder team releases for stable/yoga https://review.opendev.org/c/openstack/releases/+/848794 | 13:05 |
| opendevreview | Jon Bernard proposed openstack/releases master: Cinder team releases for stable/xena https://review.opendev.org/c/openstack/releases/+/848841 | 13:05 |
| opendevreview | Jon Bernard proposed openstack/releases master: Cinder team releases for stable/wallaby https://review.opendev.org/c/openstack/releases/+/848842 | 13:05 |
| fungi | elodilles: i'd push tags directly in that case, and try to get them to update their remaining branches | 13:46 |
| fungi | but also the release scripts could perhaps check for missing .gitreview files with a clearer error message or even try to guess when there's not one | 13:47 |
| elodilles | fungi: ack, thanks for the ideas! | 14:30 |
| elodilles | for now i see that .gitreview files are there for queens+ branches on kayobe-config, so that won't be a problem for newer branches | 14:30 |
| elodilles | \o/ | 14:30 |
| *** dviroel is now known as dviroel|lunch | 14:59 | |
| *** marios is now known as marios|out | 15:54 | |
| *** dviroel|lunch is now known as dviroel | 16:08 | |
| *** amoralej is now known as amoralej|off | 16:28 | |
| *** dviroel is now known as dviroel|out | 20:50 | |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!