Wednesday, 2018-02-14

« Tuesday, 2018-02-13 Index Thursday, 2018-02-15 »
*** claudiub has quit IRC00:00
*** kumarmn has joined #openstack-release00:11
*** pbandark has quit IRC00:13
*** kumarmn has quit IRC00:15
*** edmondsw has quit IRC00:18
*** lbragstad has quit IRC00:22
*** wolverineav has quit IRC00:32
*** wolverineav has joined #openstack-release00:33
*** wolverineav has quit IRC00:37
*** esberglu has quit IRC01:00
openstackgerritZhipeng Huang proposed openstack/releases master: cyborg 0.2.0 release  https://review.openstack.org/54412701:03
*** armstrong has quit IRC01:23
*** hongbin has joined #openstack-release01:24
*** ekcs has quit IRC01:38
*** edmondsw has joined #openstack-release01:46
*** edmondsw has quit IRC01:51
*** mriedem_afk has quit IRC02:34
*** lbragstad has joined #openstack-release02:35
*** notmyname has quit IRC02:38
*** iyamahat__ has quit IRC02:40
*** notmyname has joined #openstack-release02:41
*** yamahata has quit IRC02:42
*** d0ugal has quit IRC02:49
openstackgerritZhipeng Huang proposed openstack/releases master: cyborg 0.2.0 release  https://review.openstack.org/54412702:51
openstackgerritZhipeng Huang proposed openstack/releases master: cyborg 0.2.0 release  https://review.openstack.org/54412702:55
openstackgerritZhipeng Huang proposed openstack/releases master: cyborg 0.2.0 release  https://review.openstack.org/54412702:59
*** d0ugal has joined #openstack-release03:00
openstackgerritMerged openstack/releases master: cyborg 0.2.0 release  https://review.openstack.org/54412703:12
*** harlowja has quit IRC03:30
*** edmondsw has joined #openstack-release03:34
*** edmondsw has quit IRC03:39
*** yamamoto has joined #openstack-release03:43
*** d0ugal has quit IRC03:55
*** sree has joined #openstack-release03:59
*** udesale has joined #openstack-release03:59
*** d0ugal has joined #openstack-release04:00
*** lbragstad has quit IRC04:02
*** dave-mccowan has quit IRC04:10
*** harlowja has joined #openstack-release04:12
*** kumarmn has joined #openstack-release04:12
*** ykarel_ has joined #openstack-release04:22
*** harlowja has quit IRC04:30
*** kumarmn has quit IRC04:42
*** hongbin has quit IRC04:55
*** harlowja has joined #openstack-release05:14
*** claudiub has joined #openstack-release05:21
*** iyamahat has joined #openstack-release05:37
*** lbragstad has joined #openstack-release05:42
*** iyamahat has quit IRC05:45
*** coolsvap has joined #openstack-release06:17
*** harlowja has quit IRC06:24
openstackgerritMasahito Muroi proposed openstack/releases master: Add queens release note link  https://review.openstack.org/54417706:25
openstackgerritMasahito Muroi proposed openstack/releases master: Add queens release note link for Blazar  https://review.openstack.org/54417706:25
*** ykarel_ is now known as ykarel06:31
*** kumarmn has joined #openstack-release06:43
*** kumarmn has quit IRC06:47
*** rosmaita has quit IRC06:51
*** ianychoi_ is now known as ianychoi07:08
*** edmondsw has joined #openstack-release07:11
*** edmondsw has quit IRC07:15
*** lbragstad has quit IRC07:17
*** alexchadin has joined #openstack-release07:17
*** lvdombrkr has joined #openstack-release07:31
*** ykarel_ has joined #openstack-release07:47
*** ykarel has quit IRC07:48
*** ykarel__ has joined #openstack-release07:52
*** ykarel has joined #openstack-release07:54
*** ykarel_ has quit IRC07:55
*** pcaruana has joined #openstack-release07:56
*** ykarel__ has quit IRC07:57
*** eumel8 has joined #openstack-release07:57
openstackgerritChandan Kumar proposed openstack/releases master: Release python-tempestconf-1.1.4  https://review.openstack.org/53884008:13
openstackgerritChandan Kumar proposed openstack/releases master: Release python-tempestconf-1.1.4  https://review.openstack.org/53884008:14
openstackgerritJean-Philippe Evrard proposed openstack/releases master: Release OpenStack-Ansible queens/17.0.0.0rc1  https://review.openstack.org/54420708:17
*** amoralej|off is now known as amoralej08:20
*** yamamoto has quit IRC08:21
evrardjphey, I'd be super pleased if we can branch osa soon :)08:21
evrardjphttps://review.openstack.org/54420708:21
*** yamamoto has joined #openstack-release08:22
*** lvdombrkr89 has joined #openstack-release08:23
*** lvdombrkr has quit IRC08:26
*** armax has quit IRC08:28
*** armax has joined #openstack-release08:29
*** armax has quit IRC08:29
*** ttx has joined #openstack-release08:50
*** jpich has joined #openstack-release08:59
*** iyamahat has joined #openstack-release09:05
*** yamahata has joined #openstack-release09:06
*** pbandark has joined #openstack-release09:08
*** slaweq has joined #openstack-release09:18
*** ykarel is now known as ykarel|lunch09:19
*** slaweq has quit IRC09:23
*** iyamahat has quit IRC09:23
*** yamahata has quit IRC09:24
*** slaweq_ has joined #openstack-release09:27
*** yamahata has joined #openstack-release09:31
*** alexchadin has quit IRC09:56
*** alexchadin has joined #openstack-release09:57
*** ykarel|lunch is now known as ykarel10:00
*** shardy has joined #openstack-release10:02
*** pcaruana has quit IRC10:05
*** sree has quit IRC10:09
*** sree has joined #openstack-release10:10
*** sree has quit IRC10:15
*** slaweq_ is now known as slaweq10:16
*** pcaruana has joined #openstack-release10:20
*** ykarel_ has joined #openstack-release10:26
*** ykarel has quit IRC10:29
*** armax has joined #openstack-release10:30
*** lucas-pto is now known as lucasagomes10:30
*** yamahata has quit IRC10:34
*** ykarel_ is now known as ykarel10:37
*** edmondsw has joined #openstack-release10:47
*** ykarel_ has joined #openstack-release10:52
*** edmondsw has quit IRC10:52
*** ykarel has quit IRC10:54
*** pcaruana has quit IRC11:30
*** udesale_ has joined #openstack-release11:36
*** sree has joined #openstack-release11:36
*** alexchadin has quit IRC11:38
*** udesale has quit IRC11:38
*** sree has quit IRC11:40
*** udesale has joined #openstack-release11:41
*** udesale_ has quit IRC11:41
*** armstrong has joined #openstack-release11:42
*** pcaruana has joined #openstack-release11:42
*** alexchadin has joined #openstack-release11:47
*** udesale has quit IRC11:47
*** sree has joined #openstack-release11:51
*** sree has quit IRC11:56
*** dave-mccowan has joined #openstack-release11:56
*** sree has joined #openstack-release11:58
*** sree_ has joined #openstack-release12:00
*** sree_ is now known as Guest1896812:01
*** sree has quit IRC12:03
*** wolverineav has joined #openstack-release12:17
*** dtantsur|afk is now known as dtantsur12:21
dtantsurtonyb: ack, thanks12:22
*** yamamoto has quit IRC12:23
*** yamamoto has joined #openstack-release12:24
*** edmondsw has joined #openstack-release12:36
*** edmondsw has quit IRC12:40
*** chandankumar has joined #openstack-release12:43
chandankumarsmcginnis: Hello12:43
chandankumarsmcginnis: please have a look at this review https://review.openstack.org/#/c/538840/ when you are free, Thanks :-)12:44
*** edmondsw has joined #openstack-release12:50
*** udesale has joined #openstack-release12:51
*** ianychoi has quit IRC13:00
*** ykarel__ has joined #openstack-release13:03
*** ianychoi has joined #openstack-release13:04
*** ykarel_ has quit IRC13:05
*** ianychoi has quit IRC13:09
*** pkovar has joined #openstack-release13:09
*** Guest18968 has quit IRC13:09
*** sree has joined #openstack-release13:10
*** rosmaita has joined #openstack-release13:14
dirkwhats the normal way to circument new dependencies in a stable release?13:15
dirkhttp://logs.openstack.org/41/543241/2/check/openstack-tox-validate/f7c06d2/job-output.txt.gz#_2018-02-14_10_44_21_24451213:15
*** sree has quit IRC13:15
dirkits the entire point of the release that we want to switch to that dependency13:15
dirkwould it work if its not marked as a bugfix release?13:15
smcginnisdirk: Yeah, usually if there are requirements changes in a stable release we ask that they bump the minor rev and not just the bugfix rev.13:15
smcginnisdirk: If possible.13:16
smcginnisSometimes it isn't depending if there are newer releases that would cause issues with using a minor rev bump.13:16
dirkso what do we do here?13:16
fungiright, our take on semver is that a change in what version of dependencies you support makes it more than a patch rev13:16
dirkit seems a 1.60 doesn't exist here yet13:17
dirkso we could retag it as 1.6013:17
dirkshould I do that?13:17
*** yamamoto has quit IRC13:17
smcginnisLooking...13:17
*** yamamoto has joined #openstack-release13:18
smcginnisdirk: Ah, yep. That's trying to tag it as .1. I believe it will need to be 1.60.0.13:18
openstackgerritDirk Mueller proposed openstack/releases master: New tooz release for msgpack rename  https://review.openstack.org/54324113:18
dirkdone13:19
smcginnisdirk: That's in global-requirements, right?13:20
ttxsmcginnis: yes13:22
smcginnisSo we need a requirements FFE for that one.13:22
*** yamamoto has quit IRC13:22
ttxsmcginnis: the change is already in there13:22
smcginnisttx: Not sure I follow.13:23
ttxor do you mean that tooz needs bump13:23
*** eumel8 has quit IRC13:23
ttxI thought you were saying msgpack bump13:23
*** lbragstad has joined #openstack-release13:23
smcginnisThat tooz would need a bump.13:23
ttxok, u-c only I gather?13:24
smcginnisI would think so. https://github.com/openstack/requirements/blob/master/upper-constraints.txt#L32813:24
smcginnisUnless there is a reason the lower releases are really broken.13:24
ttxsmcginnis: they are not broken really, if you manage to install them you can probably use them alright iiuc13:25
smcginnisEither way, we need a requirements FFE to allow that in stable/queens u-c.13:26
ttxsmcginnis: do you think that those FFEs block the branching work we planned for today/now?13:26
ttxMy guess is not13:27
smcginnisI don't think this will impact that.13:27
ttxsmcginnis: you OK with me approving the missing branches ?13:27
smcginnisttx: Yep, I think we are ready. We gave enough warning at this point.13:28
ttxAlright, +2a13:29
smcginnisThe devstack and grenade branching should be good to go too I think.13:30
*** amoralej is now known as amoralej|lunch13:30
ttxyeah, but let's do it one at a time13:30
smcginnis++13:30
ttxI'm around for the next two hours.13:31
smcginnisOK, great.13:31
openstackgerritMerged openstack/releases master: Add missing stable/queens branches  https://review.openstack.org/54360113:38
*** kumarmn has joined #openstack-release13:44
*** kumarmn has quit IRC13:48
*** yamamoto has joined #openstack-release13:48
*** mriedem has joined #openstack-release13:57
*** alexchadin has quit IRC14:08
ttxsmcginnis: looks like it ran without error14:08
*** alexchadin has joined #openstack-release14:08
*** alexchadin has quit IRC14:08
ttxsmcginnis: ready to approve https://review.openstack.org/#/c/543611/14:09
smcginnisShould be good to go.14:09
*** alexchadin has joined #openstack-release14:09
*** alexchadin has quit IRC14:09
*** alexchadin has joined #openstack-release14:10
*** alexchadin has quit IRC14:10
*** alexchadin has joined #openstack-release14:11
*** alexchadin has quit IRC14:11
*** alexchadin has joined #openstack-release14:11
ttx+2a14:12
*** alexchadin has quit IRC14:12
*** alexchadin has joined #openstack-release14:12
*** alexchadin has quit IRC14:12
smcginnisGave a heads up in the -qa channel too.14:13
openstackgerritMerged openstack/releases master: Create stable/queens for devstack  https://review.openstack.org/54361114:18
*** lucasagomes is now known as lucas-hungry14:21
*** alexchadin has joined #openstack-release14:22
*** kumarmn has joined #openstack-release14:25
*** ykarel__ is now known as ykarel14:26
*** alexchadin has quit IRC14:26
smcginnisttx: Looks like all the release jobs were fine for devstack.14:27
*** udesale has quit IRC14:29
*** udesale has joined #openstack-release14:29
ttx\o/14:30
ttxsmcginnis: can I approve https://review.openstack.org/#/c/543614/ ? Looks like we are in a lucky day14:30
smcginnisttx: Yes, let's take advantage of our luck before it runs out. ;)14:31
*** amoralej|lunch is now known as amoralej14:31
ttxalright +2a14:33
*** udesale has quit IRC14:36
*** coolsvap has quit IRC14:39
openstackgerritMerged openstack/releases master: Create stable/queens branch for grenade  https://review.openstack.org/54361414:43
*** ianychoi has joined #openstack-release14:47
*** lbragsta_ has joined #openstack-release14:50
*** lbragstad has quit IRC14:51
*** yamamoto has quit IRC14:53
*** yamahata has joined #openstack-release14:54
*** yamamoto has joined #openstack-release14:54
*** yamamoto has quit IRC14:57
*** yamamoto has joined #openstack-release14:57
*** iyamahat has joined #openstack-release14:58
*** alexchadin has joined #openstack-release15:00
*** coolsvap has joined #openstack-release15:03
*** alexchadin has quit IRC15:03
-openstackstatus- NOTICE: Due to a race in stable/queens branch creation and some job removals, Zuul has reported syntax errors for the past hour; if you saw a syntax error reported for "Job tripleo-ci-centos-7-ovb-containers-oooq not defined" you can safely recheck now15:05
*** bobh has joined #openstack-release15:05
*** iyamahat has quit IRC15:07
ttxsmcginnis: I'm working to propose the next change15:09
ttxsmcginnis: hmm, no stable/queens created for devstack yet ?15:11
evrardjpsmcginnis: mmm15:11
ttxmaybe those jobs weren't as successful as we thought15:12
evrardjpsmcginnis: I've answered here: https://review.openstack.org/#/c/544207/115:16
*** lbragsta_ is now known as lbragstad15:19
ttxsmcginnis: looks like there is a shortcircuit in the tag-releases job that does the branching15:21
ttxhttp://logs.openstack.org/3c/3c12523c4df58db02c19418009d92f81de5bd965/release-post/tag-releases/4795c2e/job-output.txt.gz#_2018-02-14_14_20_54_46293815:21
ttxfails if there is no release at all in the file, which is the case for devstack and grenade15:22
ttxso the job completes successfully, but no branch is created15:22
ttxweird that it worked for stable/pike though, so maybe I misinterpret15:25
smcginnisHmm, I wonder if something changed since then. :/15:27
ttxwe might have "optomized" it15:28
ttxoptimized15:28
ttxsmcginnis: we should revert those changes for now since they were NOOPs15:28
ttx(the devstack and the grenade ones)15:28
smcginnisI'll do that now.15:28
ttxI have a meeting tunnel starting now15:29
openstackgerritSean McGinnis proposed openstack/releases master: Revert "Create stable/queens for devstack"  https://review.openstack.org/54438015:29
openstackgerritSean McGinnis proposed openstack/releases master: Revert "Create stable/queens branch for grenade"  https://review.openstack.org/54438115:29
smcginnisttx: I'll try to track down where that changed.15:29
ttx+2ed both, feel free to approve if they pass tests15:30
ttxsmcginnis: alternatively...15:30
ttxwe could push the branches by hand and not revert anything15:30
*** alexchadin has joined #openstack-release15:30
ttxand punt on debugging it for a calmer future15:30
*** lbragstad has quit IRC15:31
smcginnisttx: I haven't done that manually before, but if you can give me a pointer, that might be the most expedient thing to do at this point.15:31
smcginnisStill need to fix this for next time around, but at least that piece would be done.15:31
*** lucas-hungry is now known as lucasagomes15:31
*** hongbin has joined #openstack-release15:31
ttxunless you find quickly where it fails we'll do that. Hold on reverting the patches for now15:32
smcginnisttx: Digging in now. If it's not a quick and easy fix, let's go that manual route.15:32
*** tellesnobrega has quit IRC15:33
*** slaweq has quit IRC15:33
*** lbragstad has joined #openstack-release15:33
smcginnisttx: At least found the change: https://review.openstack.org/#/c/514780/15:34
smcginnisJust a move though.15:34
openstackgerritMerged openstack/releases master: Release OpenStack-Ansible queens/17.0.0.0rc1  https://review.openstack.org/54420715:37
prometheanfiremriedem: can your -W be removed from https://review.openstack.org/53851515:43
mriedemdone15:47
prometheanfiresmcginnis: about mistralclient, I don't think the FFE needs my approval, (though I do approve)15:51
smcginnisprometheanfire: Oh, I thought it was in u-c/15:52
prometheanfiresmcginnis: it is15:52
smcginnisprometheanfire: So wouldn't that need a requirements FFE?15:53
prometheanfireI guess it kinda needs my approval, it just seems so minor lol15:53
prometheanfireI'll respond15:53
smcginnis:)15:53
smcginnisThanks15:53
prometheanfirewe also were not tagged in the email15:53
smcginnisShoot, I didn't notice that.15:54
prometheanfire:P15:55
*** ykarel is now known as ykarel|away15:57
*** lvdombrkr89 has quit IRC15:58
*** alexchadin has quit IRC16:02
smcginnisttx: I think I have it fixed, but would be good to get your eyes on it.16:03
smcginnisttx: https://review.openstack.org/#/c/544513/16:03
*** ykarel|away has quit IRC16:05
openstackgerritMerged openstack/releases master: Release Mistral client 3.3.0  https://review.openstack.org/54340216:11
*** pcaruana has quit IRC16:13
*** yamamoto has quit IRC16:13
*** esberglu has joined #openstack-release16:15
*** electrofelix has quit IRC16:20
*** annabelleB has joined #openstack-release16:23
openstackgerritMerged openstack/releases master: Revert "Create stable/queens for devstack"  https://review.openstack.org/54438016:26
*** yamamoto has joined #openstack-release16:27
openstackgerritMerged openstack/releases master: Revert "Create stable/queens branch for grenade"  https://review.openstack.org/54438116:29
*** yamamoto has quit IRC16:31
*** mriedem is now known as mriedem_afk16:34
*** iyamahat has joined #openstack-release16:35
*** slaweq_ has joined #openstack-release16:36
*** alexchadin has joined #openstack-release16:40
*** slaweq_ has quit IRC16:40
*** armax has quit IRC16:42
*** alexchadin has quit IRC16:44
*** tellesnobrega has joined #openstack-release16:48
ttxlooking16:52
* ttx is out of the meeting tunnel16:52
ttxLGTM16:54
smcginnisGreat, I will ask in infra again.16:54
*** slaweq has joined #openstack-release16:56
*** slaweq has quit IRC17:00
*** rosmaita has quit IRC17:01
*** dtantsur is now known as dtantsur|afk17:09
*** yamamoto has joined #openstack-release17:12
*** rosmaita has joined #openstack-release17:13
*** alexchadin has joined #openstack-release17:16
*** pcaruana has joined #openstack-release17:17
*** yamamoto has quit IRC17:17
*** iyamahat has quit IRC17:23
*** jpich has quit IRC17:24
*** yamahata has quit IRC17:31
openstackgerritSean McGinnis proposed openstack/releases master: Create stable/queens for devstack  https://review.openstack.org/54456317:31
openstackgerritSean McGinnis proposed openstack/releases master: Create stable/queens branch for grenade  https://review.openstack.org/54456417:32
*** armstrong has quit IRC17:42
*** iyamahat has joined #openstack-release17:46
*** alexchadin has quit IRC17:50
openstackgerritZhipeng Huang proposed openstack/releases master: cyborg 0.3.0 release  https://review.openstack.org/54457617:53
openstackgerritMerged openstack/releases master: Create stable/queens for devstack  https://review.openstack.org/54456317:55
ttxsmcginnis: it worked!17:59
ttxLet's approve the other one17:59
smcginnis\o/18:01
ttxI don't have enough time left to do the devstack patch18:02
smcginnisttx: Which one?18:02
ttx"    Update the default branch for devstack in the new stable branch. For example, https://review.openstack.org/#/c/493208/"18:03
*** armax has joined #openstack-release18:03
*** armax has left #openstack-release18:03
smcginnisttx: OK, got it.18:03
ttxOK will talk to you tomorrow18:04
ttxneed to run now18:04
*** armax has joined #openstack-release18:04
smcginnisttx: Thanks18:05
*** yamahata has joined #openstack-release18:06
openstackgerritMerged openstack/releases master: Create stable/queens branch for grenade  https://review.openstack.org/54456418:08
*** pkovar has quit IRC18:22
*** yamamoto has joined #openstack-release18:27
*** alexchadin has joined #openstack-release18:32
*** yamamoto has quit IRC18:32
*** mriedem_afk is now known as mriedem18:32
*** openstackgerrit has quit IRC18:33
*** alexchadin has quit IRC18:36
*** shardy has quit IRC18:38
*** yamamoto has joined #openstack-release18:52
*** yamamoto has quit IRC18:52
*** lucasagomes is now known as lucas-afk18:56
*** bobh has quit IRC18:58
*** claudiub has quit IRC19:02
*** alexchadin has joined #openstack-release19:07
EmilienMwho created stable/queens branches for tripleo????19:20
*** harlowja has joined #openstack-release19:21
EmilienMsmcginnis: ^19:21
EmilienMI see stable/queens for paunch, instack-undercloud19:22
EmilienMand more19:22
mwhahahathe tripleo-heat-templates don't19:22
EmilienMwe're not ready to have a branch19:22
EmilienMyeah but no usually it does't work like this19:22
EmilienMdoesn't19:22
EmilienMat least someone could have asked me before19:22
EmilienMour CI is just broken on stable/queens19:22
EmilienMso now we have to backport to branch where CI is broken :(19:23
clarkbor delete the branch19:23
clarkbI wonder if this is a side effect of the hcnage that was made to branch devstack and grenade19:23
EmilienMthat's brutal but yes, I'm probably going to request a branch removal19:23
prometheanfireI remember it being part of a review (and commented that it needed to be removed) not sure why it was created19:23
EmilienMin any case for anything related to tripleo I need to be in the loop19:23
prometheanfireya, branch removal is what I'd reccommend, it's not so bad really, not loosing data19:24
mwhahahahttps://review.openstack.org/#/c/543601/19:24
mwhahahathere it was19:24
EmilienMI'll need to partially revert some releases patches19:24
EmilienMrelease-managers: for any tripleo project, please next time get my ack before19:25
EmilienMme or alex19:25
EmilienMmwhahaha: I'm removing branches now for tripleo projects19:25
mwhahahak19:26
EmilienMwell I can't19:26
mwhahahayea i thought only infra can on that19:26
EmilienMI guess I need some infra-root19:26
clarkbno you need teh release team19:26
clarkbat least what we've been doing is temporarily bumping release team acls permissions so they can do this stuff19:27
*** ekcs has joined #openstack-release19:27
fungismcginnis: ttx: dhellmann: dims: see above ^19:27
clarkbif that doesn't work here for some reason infra-root can do it19:27
*** openstackgerrit has joined #openstack-release19:28
openstackgerritEmilien Macchi proposed openstack/releases master: Partial revert: remove stable/queens for TripleO projects  https://review.openstack.org/54461019:28
EmilienMsmcginnis: I think we'll be ready to branch by next week, hopefully19:30
EmilienMI proposed ^ and you'll probably have to remove the branches. Thanks for your help!19:30
fungibut yeah, it looks like any cycle-with-intermediary deliverables which were past the deadline to have stable/queens branches got them created from their most recent release tags19:31
prometheanfireI'll let the osa team know19:33
prometheanfirepuppet probably needs to know too19:33
*** bobh has joined #openstack-release19:38
*** alexchadin has quit IRC19:39
*** coolsvap has quit IRC19:41
smcginnisEmilienM: Do those deliverables need to be changed?19:42
smcginnisEmilienM: They were branched (as advertised) because they were past the required date to do so.19:43
EmilienMsmcginnis: past the date?19:43
EmilienMwe're trailing no?19:43
smcginnisEmilienM: The deadline for cycle-with-intermediary projects.19:43
smcginnisEmilienM: Those are not cycle-trailing according to how they were set up.19:44
EmilienMbut we're not ready to have this branch, like I said19:44
smcginnisSo they were branched.19:44
EmilienMall tripleo should be cycle trailing19:44
smcginnisI get that. I'm just saying with how they are set up now, they should be and they were past the deadline, so as we stated mutliple times we would do, we created that branch.19:44
smcginnisEmilienM: So if that is not correct, those deliverables need to be updated to be cycle-trailing.19:45
smcginnisInstead of what they are now.19:45
EmilienMsure, I can do it now19:45
smcginnisI don't believe just reverting those changes in the deliverable files will get rid of the branches. fungi, corvus, clarkb - any of you know how that can be done now?19:46
clarkbsmcginnis: someone with gerrit permissions will have to go and delete the branches in gerrit once the revert is in place (so we don't auto recreate or however that happens)19:47
clarkbsmcginnis: for the last two cycles we've given tonyb escalated privs to do that on stable branches19:47
clarkb(temporarily)19:47
prometheanfireosa is fine btw, it seems they branched on purpose19:47
smcginnisclarkb: Thanks. So if I merge that patch removing those branches, should I just ping infra-core to do that?19:47
clarkbsmcginnis: ya unless you want to get the escalated privs tempoararily and do it. An infra root can do it too19:48
openstackgerritEmilien Macchi proposed openstack/releases master: Switch all TripleO projects to cycle-trailing  https://review.openstack.org/54461919:48
smcginnisclarkb: Either way works for me.19:48
openstackgerritEmilien Macchi proposed openstack/releases master: Partial revert: remove stable/queens for TripleO projects  https://review.openstack.org/54461019:48
prometheanfireso just puppet and tripleo are currently unbranched cycle-trailing19:49
EmilienMmwhahaha, smcginnis: ^ done19:49
smcginnisprometheanfire: That sounds right.19:50
clarkbsmcginnis: I can do branch deletion if you want in about 15 minutes19:51
smcginnisclarkb: Great, works for me. I approved the patch removing them from the deliverable files, but that's still working its way through check.19:52
*** yamamoto has joined #openstack-release19:53
*** weshay has joined #openstack-release19:58
EmilienMsmcginnis: please let me know when branches are dropped, thanks a lot19:58
smcginnisEmilienM: Patches are going through the gate queue now. Then we just need clarkb to do his magic.19:58
EmilienM:) ok thanks19:59
EmilienMsmcginnis: and sorry if I missed the milestone etc19:59
EmilienMin my mind, we were all trailing19:59
EmilienMI'll be more careful next time19:59
smcginnisEmilienM: Glad we got it straightened out. Now the Rocky setup will pick up those changes and it should all be good.19:59
*** yamamoto has quit IRC20:02
*** amoralej is now known as amoralej|off20:07
*** apevec has joined #openstack-release20:07
openstackgerritMerged openstack/releases master: Switch all TripleO projects to cycle-trailing  https://review.openstack.org/54461920:07
openstackgerritMerged openstack/releases master: Partial revert: remove stable/queens for TripleO projects  https://review.openstack.org/54461020:12
smcginnisclarkb: OK, should be in the clear to delete those branches now. ^^20:12
clarkbsmcginnis: cool almost ready to do that. Can we make a list of the projects that need it deleted just so I don't over or under delete?20:13
EmilienMI can prepare that for sure20:14
EmilienMbut they are all listed in https://review.openstack.org/#/c/544610/20:14
prometheanfireEmilienM: shouldn't it just match the review?20:14
EmilienMlet me pastbin it20:14
smcginnisGood, probably safest to have them all listed out in one place.20:14
EmilienMhttp://paste.openstack.org/show/672632/20:15
EmilienMarf typo20:15
EmilienMclarkb: http://paste.openstack.org/show/672635/20:16
EmilienMand thanks20:16
EmilienMweshay: once it's done ^ you can close the alert about stable/queens issue.20:17
clarkband I'm removing stable/queens right?20:17
clarkbjust to be double sure I do this correctly20:17
EmilienMclarkb: yes, removing stable/queens branch20:17
clarkbok20:17
EmilienMyou got it20:17
EmilienMI'll proceed to a release next weekish20:17
clarkbEmilienM: there are open changes at least on paunch20:17
EmilienMI'll clsoe them right now20:17
clarkbthanks20:17
clarkbcan you check the other projects for open changes too?20:17
EmilienMclarkb: all closed too20:18
clarkbEmilienM: instack-undercloud has open changes20:20
clarkbthe rest are done20:20
EmilienMdang20:20
EmilienMdone20:20
clarkband should be all cleaned up now20:21
*** alexchadin has joined #openstack-release20:21
EmilienMclarkb: thanks20:21
smcginnisclarkb: Thanks for your help!20:21
EmilienMweshay: ^ done, branches removed20:21
EmilienMTIL I need to be careful next cycle on the release model20:21
smcginnis:]20:22
*** alexchadin has quit IRC20:25
openstackgerritSean McGinnis proposed openstack/releases master: Add osprofiler release for stable/queens  https://review.openstack.org/54164520:27
openstackgerritMerged openstack/releases master: Add osprofiler release for stable/queens  https://review.openstack.org/54164520:40
corvusit looks like paunch stable/queens has been recreated20:43
corvusos-net-config also20:45
corvusand os-refresh-config20:45
corvusbut not the others20:45
EmilienMhum20:45
EmilienMclarkb: can you remove them again please? ^20:46
corvuspabelanger is trying to find out where they were recreated20:46
clarkbwell I think we need to figure out why they were recreated first20:46
pabelangerclarkb: do you have approx time when you deleted them?20:50
corvushttps://etherpad.openstack.org/p/jrLJuZcrm520:51
corvusi'm trying to dig up more info, but those are the 3 times today that zuul saw something on that branch20:51
pabelangerthanks20:51
smcginnisNew release patches get automatically rebased, right?20:52
clarkbpabelanger: 20:21:01          clarkb | and should be all cleaned up now thats utc from scrollback20:53
smcginnisSo it couldn't be the merging of other patches that still had those branches in them.20:53
clarkband I started about 3 minutes before that20:53
corvusit's looking a lot like 20:19:34 was the delete, and 20:20:15 is the recreate20:54
pabelanger2018-02-14 20:24:25,779 +0000 was the first failure for zuul syntax error20:54
corvuswhat job creates branches?20:54
pabelangertrying to locate that, but I think I see it in gerrit logs @ 2018-02-14 20:20:00,041 +0000]20:55
smcginnistag-releases20:56
smcginnisIf this is due to some rebase issue with older release patches, I can go through and rebase all outstanding patches to make sure they are not created again if we can manually delete those again.20:57
pabelangerhttp://logs.openstack.org/d1/d1b72774b21be15fd885cd26b0a84fa1ae397657/release-post/tag-releases/037722c/job-output.txt.gz is too late it looks like20:57
corvushttp://logs.openstack.org/9f/9f1fcc0602890f8c2a6409ae48ef3e0cb24c3b05/release-post/tag-releases/0e54468/job-output.txt.gz#_2018-02-14_20_20_05_89989320:58
corvusthere it is20:58
pabelangeryah, just found it too20:58
smcginnisSo I do need to rebase everything?20:59
corvusthat was triggered by https://git.openstack.org/cgit/openstack/releases/commit/?id=9f1fcc0602890f8c2a6409ae48ef3e0cb24c3b0520:59
pabelangerlooks like a race, because following commit removes the branches from yaml21:00
corvusthat's the change before the change which removed.. yeah21:00
clarkbso the race was with post queue processing things?21:00
smcginnisHmm, the branches weren't deleted until those landed.21:00
corvusso i think the branches were deleted while we still had post jobs running with the old config21:00
smcginnisMaybe we did miss it in release-post.21:00
pabelangeragree with corvus21:00
corvussmcginnis: yeah, the patches landed, but tag-releases runs in release-post21:01
corvusso we should have waited until they finished running21:01
smcginnisOK, so nothing needed with the existing outstanding patches.21:01
clarkbat this point it should be safe to remove the branches again?21:01
smcginnisSo we should be safe to just manually delete again?21:01
corvusyep.  good news is, we can just delete the branches again and it's not going to be goundhog day :)21:01
smcginnisHah21:01
pabelanger++21:01
smcginnisDid that trigger patches to those branches for .gitreview and such?21:01
corvussmcginnis: yes, so we'll probably need to abandon those21:02
corvusclarkb: you going to re-delete?21:02
clarkbI can21:02
smcginnisclarkb: I just checked and there are no open reviews left.21:06
clarkbok should be done with rbanch removal too21:06
smcginnisGood, mischief managed then.21:07
pabelangeryah for automation21:07
smcginnis;)21:07
*** esberglu has quit IRC21:08
*** esberglu has joined #openstack-release21:09
*** esberglu has quit IRC21:13
openstackgerritMerged openstack/releases master: pymod2pkg: Release version 0.13.2  https://review.openstack.org/54342421:19
*** inc0 has quit IRC21:21
*** esberglu has joined #openstack-release21:22
*** inc0 has joined #openstack-release21:22
*** pcaruana has quit IRC21:28
EmilienMThanks folks for the help21:28
pabelangersmcginnis: dhellmann: just a heads up, we need to restart zuul to clear our syntax error from the recreated tripleo queens branches21:35
pabelangerI'll wait until existing release post jobs are finished before doing so21:36
smcginnispabelanger: OK, that release-post should be the last thing from me.21:43
*** esberglu has quit IRC21:43
pabelangersmcginnis: thanks, reload in process21:43
pabelangerwon't be long21:43
smcginnisI don't plan on approving anything else today unless something urgent comes up, so should be all clear to restart as needed.21:43
pabelangerwfm21:43
*** wolverineav has quit IRC22:05
*** wolverineav has joined #openstack-release22:06
*** wolverineav has quit IRC22:10
*** bobh has quit IRC22:17
*** dave-mccowan has quit IRC22:20
*** claudiub has joined #openstack-release22:43
*** claudiub has quit IRC22:43
*** mriedem has quit IRC22:55
*** apevec has quit IRC23:03
*** hongbin has quit IRC23:06
*** edmondsw has quit IRC23:12
dirksmcginnis: so I need to post a FFE before https://review.openstack.org/#/c/543241/3 is getting merged?23:15
*** ajo is now known as ajo\pto23:35
*** mriedem has joined #openstack-release23:38
*** kumarmn has quit IRC23:43
*** kzaitsev_pi has quit IRC23:54
*** kzaitsev_pi has joined #openstack-release23:55
« Tuesday, 2018-02-13 Index Thursday, 2018-02-15 »

Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!