| *** ykarel|away is now known as ykarel | 04:14 | |
| *** marios is now known as marios|ruck | 05:44 | |
| opendevreview | Takashi Kajinami proposed openstack/releases master: [Puppet OpenStack] Fix some bugs in the last train release https://review.opendev.org/c/openstack/releases/+/796770 | 07:05 |
|---|---|---|
| opendevreview | Takashi Kajinami proposed openstack/releases master: [Puppet OpenStack] Fix some bugs in the last train release https://review.opendev.org/c/openstack/releases/+/796770 | 07:06 |
| *** rpittau|afk is now known as rpittau | 07:22 | |
| *** akekane__ is now known as abhishekk | 08:04 | |
| *** ykarel is now known as ykarel|lunch | 08:38 | |
| *** ykarel|lunch is now known as ykarel | 09:45 | |
| opendevreview | Hervé Beraud proposed openstack/releases master: retire puppet-panko https://review.opendev.org/c/openstack/releases/+/796690 | 11:32 |
| opendevreview | Hervé Beraud proposed openstack/releases master: Fix validation to ensure that first release of series isn't a bugfix https://review.opendev.org/c/openstack/releases/+/796456 | 11:36 |
| opendevreview | Hervé Beraud proposed openstack/releases master: Fix validation to ensure that first release of series isn't a bugfix https://review.opendev.org/c/openstack/releases/+/796456 | 11:40 |
| opendevreview | Merged openstack/releases master: Release rally 3.3.0 https://review.opendev.org/c/openstack/releases/+/796634 | 11:41 |
| opendevreview | Hervé Beraud proposed openstack/releases master: Fix validation to ensure that first release of series isn't a bugfix https://review.opendev.org/c/openstack/releases/+/796456 | 11:48 |
| opendevreview | Merged openstack/releases master: Push validations-libs 1.2.0 version https://review.opendev.org/c/openstack/releases/+/796242 | 11:56 |
| opendevreview | Merged openstack/releases master: ovsdbapp 1.11.0 https://review.opendev.org/c/openstack/releases/+/796473 | 11:56 |
| hberaud | ttx, elodilles: Just to make it explicit, the latest job failure is related to puppet-panko, so, we face again the same problem of retired repo with reno. Worth ignoring it. | 12:19 |
| elodilles | hberaud: ack | 12:23 |
| elodilles | hberaud, another question (regarding ocata-eol): i don't know how many similar repositories are, but there is for example dragonflow, that is 'abandoned' but has stable/ocata branch seemingly: https://opendev.org/openstack/dragonflow | 12:49 |
| elodilles | and the question is: what should we do with it? | 12:49 |
| elodilles | it's not even listed in deliverables/ocata/ | 12:50 |
| elodilles | so it's a bit strange | 12:50 |
| hberaud | Good question, I would suggest to transition it to eol if the stable branch exist | 12:50 |
| elodilles | hberaud: but how? :) there is no deliverable/ocata/dragonflow :) | 12:51 |
| elodilles | or should we create a yaml there? | 12:51 |
| hberaud | Ah I see, sorry | 12:51 |
| hberaud | :) | 12:51 |
| hberaud | Really good question | 12:51 |
| hberaud | maybe we could simply drop the existing branch | 12:52 |
| hberaud | and ignore the rest | 12:52 |
| hberaud | Thoughts? | 12:52 |
| elodilles | actually, as it seems, even stable/newton branch still exists for dragonflow | 12:53 |
| elodilles | the question is whether someone consumes dragonflow's ocata branches... | 12:54 |
| hberaud | Do we want to spend time on it? if the project is abandoned I don't expect lot of people complaining about the remaining branches | 12:55 |
| elodilles | let's ask noonedeadpunk as he asked me about what to do with dragonflow | 12:55 |
| elodilles | noonedeadpunk: opinion? ^^^ | 12:55 |
| hberaud | Maybe we could simply trigger a ML thread to warn about the deletion and if nobody complain then start removing these stale branches | 12:56 |
| noonedeadpunk | But I think we would need to drop them sooner or later anyway? | 12:56 |
| noonedeadpunk | Or at least have some workflow to follow? | 12:56 |
| hberaud | Right time to drop them | 12:56 |
| noonedeadpunk | as it's not single project that has been deprecated | 12:56 |
| elodilles | noonedeadpunk: + a comment: retired projects are usually also listed under deliverables/ , and those are treated the same way when EOL'ing as active projects | 12:57 |
| elodilles | noonedeadpunk: so here the problem seems that it is not even listed under deliverables/ | 12:57 |
| noonedeadpunk | yes, retired, not deprecatred, sorry | 12:58 |
| noonedeadpunk | yeah, got it, so em can't be set without it being in deliverables. | 13:00 |
| noonedeadpunk | which means that the code will eventually be just droped without ability to track down progress between releases... | 13:01 |
| noonedeadpunk | but I can imagine it's being used in some P or Q deployment easily... | 13:02 |
| elodilles | well... hmmm... maybe we could tag the latest state of those branches as '$series-eol' (by adding them now to deliverables/$series/dragonflow.yaml) and then delete | 13:03 |
| elodilles | hberaud: what do you think about this ^^^ ? | 13:03 |
| noonedeadpunk | I like this plan | 13:03 |
| elodilles | so that we won't lose history | 13:04 |
| hberaud | +1 | 13:04 |
| hberaud | Can't hurt | 13:05 |
| elodilles | hberaud: cool, then I'll try to propose a patch :) | 13:05 |
| elodilles | noonedeadpunk: by the way, do you need that to OSA, or why did you asked about dragonflow in the first place? :) | 13:06 |
| elodilles | * ask | 13:06 |
| noonedeadpunk | I needed to set some sha/branch/tag to EOL patch so it would be a valid one | 13:07 |
| noonedeadpunk | ie https://review.opendev.org/c/openstack/openstack-ansible/+/795690/2/playbooks/defaults/repo_packages/openstack_services.yml | 13:07 |
| hberaud | elodilles: thanks | 13:08 |
| elodilles | noonedeadpunk: I see. Do you see other than dragonflow that has ocata branch but not listed under deliverables/ocata/ ? | 13:11 |
| noonedeadpunk | neutron-lbaas-dashboard ? | 13:13 |
| noonedeadpunk | also not sure about nova-powervm, nova-lxd | 13:15 |
| noonedeadpunk | and swift3 | 13:15 |
| noonedeadpunk | but we didn't have all projects available for deployement, so there might be more | 13:16 |
| elodilles | I'll check those, thanks | 13:16 |
| opendevreview | Elod Illes proposed openstack/releases master: WIP: Set eol to Dragonflow https://review.opendev.org/c/openstack/releases/+/796841 | 13:36 |
| elodilles | hberaud: meanwhile, I've checked neutron-lbaas-dashboard, it has similar (but not the same!) situation: it has stable/ocata, but has only neutron-lbaas-dashboard.yaml under pike-queens (and branches up till stable/stein) | 13:38 |
| elodilles | hberaud: and now I'm a bit confused :/ | 13:38 |
| elodilles | hberaud: whether we really need to add this kind of 'fake' deliverables for all of these (maybe there are even more repos like this?) | 13:39 |
| hberaud | elodilles: AFAIK until now nobody complained, maybe we could continue to live with them | 13:41 |
| hberaud | if it's not an isolated case, then I would suggest to not add these deliverables files | 13:41 |
| hberaud | and maybe go straight to branch deletion by warning before | 13:42 |
| hberaud | Stable branches seems to be the poor parent of our release management process | 13:44 |
| hberaud | I don't know if we faced similar use case during previous series (oldest than ocata) | 13:45 |
| elodilles | hberaud: the 1st part is OK with me: to live with them :) but in that case I would not even delete those branches as who knows whether those are consumed downstream and we would lose 'history' o:) | 13:45 |
| hberaud | LGTM | 13:46 |
| elodilles | definitely there is stable/newton still open (at least for dragonflow) | 13:46 |
| elodilles | and there can be others as well | 13:46 |
| hberaud | I see | 13:47 |
| hberaud | I think that you're right we should leave these branches opened | 13:47 |
| elodilles | maybe smcginnis and ttx have some historical knowledge :) | 13:48 |
| tosky | or maybe just people forgot to close them and they should be deleted... | 13:50 |
| elodilles | tosky: i'm not sure it's simply just forgotten. | 13:51 |
| elodilles | tosky: and deleting without tagging them would mean losing history | 13:51 |
| tosky | sure, so just tag the latest | 13:52 |
| tosky | or just ask again the people responsible for that project (I'm 99% sure they will say to kill anything <=ocata) | 13:52 |
| elodilles | well, maybe they are not even around, so no answers will come :) (I know, in that case we could simply delete them... and what if we delete them and someone comes with the question 'where is it?' later on :)) | 13:54 |
| elodilles | maybe I'm overthinking it, but.... :) | 13:55 |
| elodilles | anyway, if we just simply leave those stable/ocata branches open, then OSA should use hashes for them, as there won't be ocata-eol tag | 13:57 |
| elodilles | and I will go back to approve the nice and clean ocata-eol patches... (and think more about this later on) | 13:59 |
| *** marios|ruck is now known as marios|call | 14:02 | |
| opendevreview | Merged openstack/releases master: [watcher] Transition Ocata to End of Life https://review.opendev.org/c/openstack/releases/+/795684 | 14:31 |
| opendevreview | Merged openstack/releases master: [winstackers] Transition Ocata to End of Life https://review.opendev.org/c/openstack/releases/+/795685 | 14:36 |
| opendevreview | Merged openstack/releases master: [horizon] Transition Ocata to End of Life https://review.opendev.org/c/openstack/releases/+/795655 | 14:43 |
| *** marios|call is now known as marios|ruck | 14:44 | |
| opendevreview | Merged openstack/releases master: [magnum] Transition Ocata to End of Life https://review.opendev.org/c/openstack/releases/+/795658 | 14:46 |
| opendevreview | Merged openstack/releases master: [monasca] Transition Ocata to End of Life https://review.opendev.org/c/openstack/releases/+/795661 | 14:56 |
| opendevreview | Merged openstack/releases master: [glance] Transition Ocata to End of Life https://review.opendev.org/c/openstack/releases/+/795653 | 15:01 |
| hberaud | +1 to leave the branches opened for now. I think that they just have been forgotten but in the doubt that can't hurt to leave them opened. The checks against old branches have been introduced with Wallaby, so, it doesn't shock me if they have been forgotten. Also EM have been introduced around the ocata period too and so ocata have been surely impcated by the grey area around the | 15:05 |
| opendevreview | Merged openstack/releases master: [neutron] Transition Ocata to End of Life https://review.opendev.org/c/openstack/releases/+/795663 | 15:05 |
| hberaud | stable branch deletion policy brought by EM. | 15:05 |
| *** ykarel is now known as ykarel|away | 15:10 | |
| opendevreview | Merged openstack/releases master: [ironic] Transition Ocata to End of Life https://review.opendev.org/c/openstack/releases/+/795656 | 15:14 |
| elodilles | well, these old open branches are not related to EM. It's more like repositories without deliverables, I think. | 15:17 |
| opendevreview | Merged openstack/releases master: [nova] Transition Ocata to End of Life https://review.opendev.org/c/openstack/releases/+/795664 | 15:18 |
| elodilles | hmmm, and there is a release job failure: https://zuul.opendev.org/t/openstack/build/cbeed3bf86b749c49ddac9ae708a4c36 | 15:18 |
| elodilles | related to this patch: https://review.opendev.org/c/openstack/releases/+/795655 | 15:20 |
| elodilles | "No '.gitreview' file found in this repository. We don't know where | 15:20 |
| elodilles | your gerrit is." | 15:21 |
| elodilles | (but .gitreview is there: https://opendev.org/openstack/django_openstack_auth/src/branch/stable/ocata/.gitreview ) | 15:22 |
| opendevreview | Merged openstack/releases master: [Telemetry] Transition Ocata to End of Life https://review.opendev.org/c/openstack/releases/+/795679 | 15:26 |
| *** rpittau is now known as rpittau|afk | 16:09 | |
| *** marios|ruck is now known as marios|out | 16:28 | |
| *** akekane_ is now known as abhishekk | 16:44 | |
| opendevreview | Jeremy Stanley proposed openstack/releases master: Correct the tox option for skipping sdist generation https://review.opendev.org/c/openstack/releases/+/796916 | 17:09 |
| elodilles | and a 2nd release job failure, this time it's the SSH connection timeout again: https://zuul.opendev.org/t/openstack/build/69fc52bff367468f9de9ad4713baf339 | 17:47 |
| hberaud | The tag have been create https://opendev.org/openstack/django_openstack_auth/src/tag/ocata-eol | 17:58 |
| hberaud | s/has/have/ | 17:58 |
| hberaud | I would argue that we can ignore it | 18:00 |
| opendevreview | Elod Illes proposed openstack/releases master: [Puppet OpenStack] Transition Train to EM https://review.opendev.org/c/openstack/releases/+/790767 | 18:37 |
Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!