| *** zxiiro has joined #openstack-release | 00:23 | |
| openstackgerrit | Sean McGinnis proposed openstack/releases master: Release final osc-placement for ussuri https://review.opendev.org/718239 | 00:35 |
|---|---|---|
| openstackgerrit | Merged openstack/releases master: Release final python-magnumclient for ussuri https://review.opendev.org/718253 | 00:40 |
| *** ricolin has joined #openstack-release | 01:58 | |
| openstackgerrit | wes hayutin proposed openstack/releases master: tripleo ussuri milestone 3 release https://review.opendev.org/718297 | 02:37 |
| *** zxiiro has quit IRC | 03:27 | |
| *** vishalmanchanda has joined #openstack-release | 03:53 | |
| *** ykarel|away is now known as ykarel | 04:24 | |
| *** evrardjp has quit IRC | 04:36 | |
| *** evrardjp has joined #openstack-release | 04:37 | |
| *** udesale has joined #openstack-release | 05:22 | |
| *** udesale has quit IRC | 05:23 | |
| *** udesale has joined #openstack-release | 05:23 | |
| *** armax has quit IRC | 05:27 | |
| *** armax has joined #openstack-release | 05:28 | |
| *** armax has quit IRC | 05:33 | |
| *** armax has joined #openstack-release | 06:13 | |
| *** armax has quit IRC | 06:18 | |
| *** slaweq_ has joined #openstack-release | 06:57 | |
| *** rpittau|afk is now known as rpittau | 07:17 | |
| *** e0ne has joined #openstack-release | 07:38 | |
| *** dtantsur|afk is now known as dtantsur | 07:41 | |
| *** tosky has joined #openstack-release | 08:00 | |
| *** dmellado has quit IRC | 08:01 | |
| *** dmellado has joined #openstack-release | 08:05 | |
| openstackgerrit | Eyal proposed openstack/releases master: Bump vitrage version to 6.0.2 https://review.opendev.org/718338 | 08:12 |
| *** slaweq_ is now known as slaweq | 08:36 | |
| *** ykarel is now known as ykarel|lunch | 08:53 | |
| *** ricolin_ has quit IRC | 09:29 | |
| *** ricolin_ has joined #openstack-release | 09:30 | |
| *** ricolin has quit IRC | 09:30 | |
| *** ricolin_ has quit IRC | 09:57 | |
| *** ricolin_ has joined #openstack-release | 09:57 | |
| *** jbadiapa has joined #openstack-release | 09:59 | |
| *** ricolin has joined #openstack-release | 10:00 | |
| *** ricolin_ has quit IRC | 10:02 | |
| *** ykarel|lunch is now known as ykarel | 10:12 | |
| openstackgerrit | Merged openstack/releases master: Add Ussuri cycle highlights for Octavia. https://review.opendev.org/715014 | 10:16 |
| openstackgerrit | Merged openstack/releases master: Ussuri: update cycle highlights for Kolla https://review.opendev.org/718185 | 10:20 |
| *** slaweq has quit IRC | 10:23 | |
| *** rpittau is now known as rpittau|bbl | 10:24 | |
| *** slaweq has joined #openstack-release | 10:25 | |
| openstackgerrit | Dmitry Tantsur proposed openstack/releases master: Release final python-ironicclient for ussuri https://review.opendev.org/718250 | 10:25 |
| *** ricolin has quit IRC | 10:39 | |
| openstackgerrit | Slawek Kaplonski proposed openstack/releases master: Add Ussuri cycle highlights for Neutron https://review.opendev.org/718382 | 10:40 |
| slaweq | amotoki: ^^ please take a look at it if You will have some time :) | 10:41 |
| amotoki | slaweq: sure. thanks | 10:42 |
| *** rakhmerov has joined #openstack-release | 10:50 | |
| *** jtomasek has joined #openstack-release | 10:55 | |
| *** e0ne has quit IRC | 10:58 | |
| *** e0ne has joined #openstack-release | 10:58 | |
| openstackgerrit | Eyal proposed openstack/releases master: Bump vitrage version to 7.0.0 https://review.opendev.org/718338 | 11:17 |
| openstackgerrit | Jean-Philippe Evrard proposed openstack/releases master: Release final python-ironic-inspector-client for ussuri https://review.opendev.org/718249 | 11:21 |
| openstackgerrit | Merged openstack/releases master: Release final sushy-cli for ussuri https://review.opendev.org/718274 | 11:33 |
| openstackgerrit | Merged openstack/releases master: Release final python-neutronclient for ussuri https://review.opendev.org/718259 | 11:52 |
| openstackgerrit | Merged openstack/releases master: Release final python-senlinclient for ussuri https://review.opendev.org/718266 | 11:53 |
| openstackgerrit | Merged openstack/releases master: Release final osc-placement for ussuri https://review.opendev.org/718239 | 12:06 |
| amotoki | slaweq: just posted comments on the neutron cycle highlights. I believe enough topics are covered. my comments are mainly on improvements as cycle-highlights. | 12:13 |
| openstackgerrit | Merged openstack/releases master: Release final python-pankoclient for ussuri https://review.opendev.org/718262 | 12:17 |
| openstackgerrit | Merged openstack/releases master: Release final python-muranoclient for ussuri https://review.opendev.org/718258 | 12:22 |
| smcginnis | evrardjp: Thanks for the reviews! | 12:22 |
| *** rpittau|bbl is now known as rpittau | 12:23 | |
| evrardjp | hey I got a few minutes between fires, I tried to use them. Sorry for being quite away nowadays. | 12:23 |
| openstackgerrit | Merged openstack/releases master: Release final python-aodhclient for ussuri https://review.opendev.org/718240 | 12:24 |
| smcginnis | evrardjp: Absolutely no need to apologize. | 12:26 |
| openstackgerrit | Merged openstack/releases master: Bump vitrage version to 7.0.0 https://review.opendev.org/718338 | 12:32 |
| slaweq | amotoki: thx | 12:51 |
| *** ykarel is now known as ykarel|afk | 12:56 | |
| *** ricolin has joined #openstack-release | 12:57 | |
| openstackgerrit | Julia Kreger proposed openstack/releases master: Cycle highlights for ironic https://review.opendev.org/718436 | 13:08 |
| *** udesale_ has joined #openstack-release | 13:18 | |
| openstackgerrit | Sean McGinnis proposed openstack/releases master: Release final python-solumclient for ussuri https://review.opendev.org/718267 | 13:19 |
| *** udesale has quit IRC | 13:20 | |
| *** sshnaidm has joined #openstack-release | 13:34 | |
| sshnaidm | hi, I'd like to delete tripleo-ansible stein branch, which we don't need. Which check should be done from releasing perspective? | 13:35 |
| *** ykarel|afk is now known as ykarel | 13:37 | |
| smcginnis | sshnaidm: What is the reason for wanting to delete a cycle's stable branch? That's normally not done. | 13:47 |
| sshnaidm | it doesn't contain anything meaningful since all work started after stein, but this also breaks our CI jobs by its existence | 13:48 |
| *** dtantsur is now known as dtantsur|brb | 13:50 | |
| openstackgerrit | Arne Wiebalck proposed openstack/releases master: Cycle highlights for ironic-inspector https://review.opendev.org/718456 | 13:52 |
| *** weshay_ has joined #openstack-release | 13:53 | |
| weshay_ | 0/ | 13:53 |
| sshnaidm | smcginnis, tripleo ptl ^ | 13:54 |
| corvus | sshnaidm: how does the stein branch break the ci builds? | 13:55 |
| sshnaidm | corvus, we use some roles from tripleo-ansible in all roles, something that is not in stein and never been there | 13:56 |
| sshnaidm | since stein doesn't contain anything meaningful for this project | 13:56 |
| smcginnis | Yeah, this is probably more of an infra/opendev question. It would be a little unusual. But from the releases side, it would be a matter of remove the branch entry from the deliverables/stein/tripleo-ansible.yaml file. | 13:56 |
| sshnaidm | smcginnis, infra is good while you're good | 13:57 |
| corvus | sshnaidm: i still don't understand how that breaks things -- if there never wase a "tripleo-ansible stein" what's using those roles? do you have a link to a failed job or something? | 13:59 |
| sshnaidm | corvus, yes https://review.opendev.org/#/c/717774/ | 14:00 |
| sshnaidm | corvus, stein is quite "empty" | 14:00 |
| corvus | sshnaidm: it's the third-party ci you're concerned about? | 14:01 |
| sshnaidm | as I understand the branch was created much before actual work started | 14:01 |
| sshnaidm | corvus, yes | 14:01 |
| sshnaidm | corvus, these roles can be used in a usual CI as well, 3party is just an example | 14:02 |
| openstackgerrit | Merged openstack/releases master: Release final python-vitrageclient for ussuri https://review.opendev.org/718271 | 14:02 |
| openstackgerrit | Julia Kreger proposed openstack/releases master: Cycle highlights for ironic https://review.opendev.org/718436 | 14:03 |
| * weshay_ looks | 14:04 | |
| sshnaidm | smcginnis, should I remove these 3 lines? https://github.com/openstack/releases/blob/master/deliverables/stein/tripleo-ansible.yaml#L14-L16 | 14:05 |
| sshnaidm | or complete file | 14:05 |
| weshay_ | sshnaidm, I missed a bit before joining the channel.. is the context removing the stein branch for tripleo-ansible? | 14:05 |
| smcginnis | sshnaidm: Yes, that would be what is needed on the releases end. | 14:05 |
| sshnaidm | weshay_, yes | 14:05 |
| openstackgerrit | Julia Kreger proposed openstack/releases master: Cycle highlights for ironic https://review.opendev.org/718436 | 14:06 |
| corvus | (i mean, if you want to delete it because it never should have existed in the first place, i don't really have an opinion; but if the motivation is something ci-related, i'd at least like to understand the issue, so i'm looking into it) | 14:06 |
| sshnaidm | corvus, it's expecting to use roles that exist in tripleo-ansible, but doesn't exist in its stein branch | 14:07 |
| corvus | sshnaidm: ack; it's just taking me a few minutes to trace back through the jobs | 14:09 |
| openstackgerrit | Sagi Shnaidman proposed openstack/releases master: Remove stein from tripleo-ansible in scope of removing stein branch https://review.opendev.org/718468 | 14:10 |
| sshnaidm | weshay_, smcginnis ^ | 14:10 |
| * weshay_ looking | 14:10 | |
| corvus | sshnaidm: iiuc, the issue is this line https://review.rdoproject.org/r/gitweb?p=config.git;a=blob;f=zuul.d/tripleo-rdo-base.yaml;h=df3287c3e59d06a2a3810577a4f18b95e06f77c0;hb=HEAD#l63 refers to this repo state: https://opendev.org/openstack/tripleo-ansible/src/branch/stable/stein which has nothing that looks like an ansible role? | 14:13 |
| sshnaidm | corvus, yes | 14:14 |
| corvus | sshnaidm: that's interesting -- you override-checkout to master for tripleo-ansible -- should tripleo-ansible not have any branches? | 14:15 |
| corvus | sshnaidm: the role selection should also follow override-checkout, so i would expect that job to check out the master branch of tripleo-ansible | 14:19 |
| sshnaidm | corvus, thanks, you discovered a bug :) https://review.rdoproject.org/r/26334 | 14:19 |
| sshnaidm | <weshay_> 0/ | 14:19 |
| sshnaidm | corvus, I thought I remove this, but some remained.. | 14:19 |
| corvus | oh, ha, how about that :) | 14:20 |
| corvus | i'm still a little confused though -- if that really was checking out the master branch, shouldn't it have found the role and not raised the error? | 14:20 |
| sshnaidm | corvus, the problem with override is when the job actually runs on train branch of tripleo-ansible, then this overrides and pull master instead of tested train | 14:21 |
| corvus | sshnaidm: yeah, i agree that would be a problem | 14:21 |
| sshnaidm | corvus, it was working with override, and then I removed it, then stein jobs started to fail.. | 14:21 |
| corvus | sshnaidm: so the failure in https://review.opendev.org/717774 happened with override-checkout was not in place? | 14:22 |
| sshnaidm | corvus, this is after override-checkout was removed in these jobs | 14:23 |
| sshnaidm | corvus, I removed in part, just in your link saw another jobs to remove there | 14:23 |
| corvus | sshnaidm: right, but my link is for a job in the hierarchy of the tripleo-ci-centos-7-ovb-3ctlr_1comp-featureset001 which failed in https://review.opendev.org/717774 -- and i'm trying to understand whether tripleo-ansible was set to "override-checkout: master" in that job when it ran. are you saying it was or was not? | 14:25 |
| sshnaidm | corvus, no, it wasn't set | 14:26 |
| sshnaidm | corvus, when it's set, it runs ok | 14:26 |
| corvus | sshnaidm: okay, that all makes sense then. :) | 14:26 |
| corvus | sshnaidm: so i think our options are: 1) delete the stein branch of tripleo-ansible [do this if it's bogus and never should have existed in the first place]. 2) add a roles directory to the stein branch of tripleo-ansible. 3) add a branch-variant of the tripleo-ci-base-ovb-notebroker-centos-8 job which does not include the tripleo-ansible project as a role. | 14:28 |
| corvus | sshnaidm: thanks for your patience while i walked through that :) | 14:28 |
| corvus | smcginnis: ^ fyi | 14:29 |
| sshnaidm | corvus, thanks for your time too, I was thinking about option 2, but it adds maintenance of unused branch, and option 3 is adding some more effort in jobs configuration. I think deleting unused branch will be the most clean one | 14:30 |
| sshnaidm | just to get rid off something we don't use | 14:31 |
| smcginnis | corvus: I'm fine with whatever the team decides, but it sounds like maybe just removing that is the best option since it sounds like there wasn't actually anything there in stein. | 14:31 |
| sshnaidm | corvus, but in general seems like it was non so good architectural decision to use roles from tested repo.. | 14:31 |
| sshnaidm | smcginnis, so, should we remove the whole file then..? mwhahaha weshay_ are you aware of any possible issues with that? | 14:36 |
| smcginnis | sshnaidm: I am leaning that way, but I think I need to mull that over a little more to make sure there isn't an issue with doing that. | 14:37 |
| smcginnis | But it does sound like that may be the most logical step. | 14:37 |
| mwhahaha | could we just em tag it and remove the branch? | 14:37 |
| mwhahaha | as an alternative | 14:37 |
| weshay_ | I don't think it would be a problem.. probably want to keep the patch up for a bit.. to make sure people are aware of it | 14:37 |
| smcginnis | True. Or even bend the stable rules a bit and make it -eol. | 14:37 |
| smcginnis | ttx: If you happen to be around, would love your opinion on this issue. ^ | 14:38 |
| *** mlavalle has joined #openstack-release | 14:41 | |
| *** udesale_ has quit IRC | 14:46 | |
| openstackgerrit | Sean McGinnis proposed openstack/releases master: Drop email content weekly link from tracking template https://review.opendev.org/718483 | 14:54 |
| *** weshay_ is now known as weshay|ruck | 14:55 | |
| *** armax has joined #openstack-release | 15:01 | |
| *** zxiiro has joined #openstack-release | 15:04 | |
| *** ykarel is now known as ykarel|away | 15:04 | |
| *** priteau has joined #openstack-release | 15:06 | |
| ttx | smcginnis: do you have a tl;dr? I'm on calls until the end of times | 15:15 |
| ttx | If nothing in that branch I would just remove it | 15:15 |
| ttx | will review patch tomorrow | 15:17 |
| smcginnis | ttx: Sorry, tl;dr is a release was done and a branch was made, but no *real* content in the repo yet. Question of whether to delete the branch, or if the entire deliverable file should just be removed and the 0.1.0 release that was included there should just be considered non-official, since there really wasn't anything that was part of the stein release. | 15:24 |
| *** dtantsur|brb is now known as dtantsur | 15:48 | |
| *** sshnaidm is now known as sshnaidm|afk | 16:04 | |
| *** rpittau is now known as rpittau|afk | 16:23 | |
| *** evrardjp has quit IRC | 16:37 | |
| *** evrardjp has joined #openstack-release | 16:37 | |
| *** dtantsur is now known as dtantsur|afk | 17:07 | |
| *** priteau has quit IRC | 17:15 | |
| *** diablo_rojo has quit IRC | 18:02 | |
| *** diablo_rojo has joined #openstack-release | 18:06 | |
| *** e0ne has quit IRC | 18:21 | |
| openstackgerrit | Sean McGinnis proposed openstack/releases master: Release final python-cinderclient for ussuri https://review.opendev.org/718243 | 18:45 |
| openstackgerrit | Brian Rosmaita proposed openstack/releases master: Release final python-cinderclient for ussuri https://review.opendev.org/718243 | 18:51 |
| *** diablo_rojo has quit IRC | 18:53 | |
| *** slaweq_ has joined #openstack-release | 18:59 | |
| *** slaweq has quit IRC | 19:00 | |
| openstackgerrit | Merged openstack/releases master: Release final python-cinderclient for ussuri https://review.opendev.org/718243 | 19:31 |
| *** vishalmanchanda has quit IRC | 19:32 | |
| openstackgerrit | Brian Rosmaita proposed openstack/releases master: Release final python-brick-cinderclient-ext for ussuri https://review.opendev.org/718242 | 20:21 |
| openstackgerrit | Brian Rosmaita proposed openstack/releases master: Release final python-brick-cinderclient-ext for ussuri https://review.opendev.org/718242 | 20:22 |
| *** jtomasek has quit IRC | 21:25 | |
| *** slaweq_ has quit IRC | 21:38 | |
| *** slaweq_ has joined #openstack-release | 21:40 | |
| openstackgerrit | Merged openstack/releases master: Release final python-brick-cinderclient-ext for ussuri https://review.opendev.org/718242 | 21:43 |
| *** slaweq_ has quit IRC | 21:45 | |
| openstackgerrit | Brian Rosmaita proposed openstack/releases master: Cycle highlights for cinder https://review.opendev.org/718563 | 21:53 |
| openstackgerrit | Brian Rosmaita proposed openstack/releases master: Cycle highlights for cinder https://review.opendev.org/718563 | 22:08 |
| *** slaweq_ has joined #openstack-release | 22:13 | |
| smcginnis | Release managers - please hold off on approving anything with branching until https://review.opendev.org/#/c/718564/ merges. | 22:23 |
| smcginnis | And even then, maybe start with just one and make sure everything works as it should before doing any more. | 22:24 |
| *** tosky has quit IRC | 23:20 | |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!