https://pantsbuild.org/ logo
#announce
Title
# announce
e

enough-analyst-54434

06/10/2020, 11:39 PM
Releasing green branches for many of the 1.2{1-8}.1 picks will be challenging. There appear to be several branches suffering from real test failures 😕.
h

hundreds-father-404

06/10/2020, 11:40 PM
I was afraid that would be the case
w

witty-crayon-22786

06/10/2020, 11:41 PM
hm, yea i wouldn’t cherrypick back that far… i think that the convention has been to do the last three stable releases.
sorry, i just realized what all those emails were… thought it was a github glitch
e

enough-analyst-54434

06/10/2020, 11:41 PM
The user with the issue is upgrading to 1.23
w

witty-crayon-22786

06/10/2020, 11:42 PM
sure. feels like they could still bump version at a time (without committing) while ignoring this particular issue?
i say convention though, because i don’t think we actually have rules about this
e

enough-analyst-54434

06/10/2020, 11:43 PM
Yeah. It feels pretty crappy to ask that when the fix is a fix for us failing our deprecation policy.
w

witty-crayon-22786

06/10/2020, 11:43 PM
k
e

enough-analyst-54434

06/10/2020, 11:45 PM
I guess I was fishing for the expected state of release branches. I found, for example, that 1.25.x has legit formatting errors. I.E.: It appears releases don't go through ci, just straight cherry-picks to the origin branch,
h

hundreds-father-404

06/10/2020, 11:45 PM
Yeah, that’s true, John. I think @witty-crayon-22786 we should stop straight cherry-picking. It’s convenient, but results in the bitrot John is seening
w

witty-crayon-22786

06/10/2020, 11:46 PM
first we would want to answer the question of how far back we support, i think.
e

enough-analyst-54434

06/10/2020, 11:46 PM
Its convenient in the way pushing straight to master would be! And its worse beacuse its for a stability guaranty we made with the community.
👍 1
w

witty-crayon-22786

06/10/2020, 11:46 PM
we don’t guarantee back that far though. not explicitly.
e

enough-analyst-54434

06/10/2020, 11:47 PM
@witty-crayon-22786 I think that's a bit not relevant when we broke our own rules. This wasn't a bug fix. It was restoring a feature that was yanked with no deprecation - and worse, with no need to.
👍 1
h

hundreds-father-404

06/10/2020, 11:48 PM
Its convenient in the way pushing straight to master would be! And its worse beacuse its for a stability guaranty we made with the community.
Full agreement.
w

witty-crayon-22786

06/10/2020, 11:48 PM
as a special case, sure. but we would say “no, sorry” at some point… ie, if someone identified a back compat break in 1.0.0
h

hundreds-father-404

06/10/2020, 11:49 PM
I see what Stu is saying, but I think the more important point is that our RC branches have horrible bit rot. And a major reason for this is that we directly cherry-pick, rather than opening proper PRs
w

witty-crayon-22786

06/10/2020, 11:49 PM
part of the reason we have been ok with cherry-picking without a PR is that we don’t try to go back more than a month or so, and large picks just … aren’t picked
e

enough-analyst-54434

06/10/2020, 11:49 PM
We do not apply the same logic to small master commits.
👍 1
I think we should just stop.
1
w

witty-crayon-22786

06/10/2020, 11:51 PM
we don’t apply them to small master commits because those changes haven’t had any CI… when cherry-picking to a recent branch, they have just finished CI.
h

hundreds-father-404

06/10/2020, 11:51 PM
Agreed. It’s also not that much more inconvenient to use a proper PR. About an extra hour due to CI The biggest inconvenience is when people don’t cherry-pick right away, so the releasor now has to wait for 5-6 cherry-picks to land. We should stop that too
e

enough-analyst-54434

06/10/2020, 11:51 PM
I'm pretty sure I'm going to have to become part of the problem to deliver a fix for Pinterest here and that's crappy.
h

hundreds-father-404

06/10/2020, 11:51 PM
when cherry-picking to a recent branch, they have just finished CI.
Finished CI against the master branch, but not against the stable branch
e

enough-analyst-54434

06/10/2020, 11:52 PM
I think convenience is not a valid thing to reference at all. We made a contract with the community in 1.0.0. We should keep the contract. Stability means a green release branch.
👍 1
Alright - I've got enough info - we can expect bitrot since direct cherry-picks have been a thing.
w

witty-crayon-22786

06/10/2020, 11:53 PM
you can expect bitrot regardless of direct picks. but yes.
the “must have green branch” thing applies to stable branches, absolutely. but the internet is changing beneath us, and i expect you will encounter other failures on code that old.
in the absence of 100% hermetic builds, “green at time of release” is the best we can do
e

enough-analyst-54434

06/10/2020, 11:54 PM
Yes. Those I can accept and expect. What I did not expect was broken for real branches. We have those.
w

witty-crayon-22786

06/10/2020, 11:55 PM
i can say that in the case of 1.25.x, twitter was thrashing pretty hard trying to get on 1.26.x, and backported a lot of stuff
they have since forked it.
e

enough-analyst-54434

06/10/2020, 11:56 PM
Afaict every release branch since shortly after 1.0 ws for / by Twitter. I can't recall the last stable branch to get a bug report / fix for / by Foursquare or Square.
w

witty-crayon-22786

06/10/2020, 11:57 PM
is that relevant to pinterest in some way…?
“contributing to producing” the stable branch is different from consuming
anyway. sorry that it’s not cleaner. let me know if i can help unblock you/them
i expect that “requiring a PR” is related to the “must run rcs” discussion from earlier today.
e

enough-analyst-54434

06/11/2020, 12:13 AM
I really hate this. The only thing I've been able to come up with to feel not evil is to release the older branches as rcs so that they are not picked up by normal resolves (ie pip install without --pre). So we'd have 1.21.1rc0 and never 1.21.1. Does something like that sound reasonable?
h

hundreds-father-404

06/11/2020, 12:13 AM
Yes, that does indeed sound reasonable
e

enough-analyst-54434

06/11/2020, 12:14 AM
Basically trying to communicate an unstable stable release.
w

witty-crayon-22786

06/11/2020, 12:14 AM
yea.
i missed the discussion with them, but they requested or need the patch in all intermediate releases…? if not, maybe we could ask them which version they actually plan on releasing/landing, and only pick there?
e

enough-analyst-54434

06/11/2020, 12:15 AM
Never again. OK, I'll also update releaser docs to steer folks away from direct cherry-picks / spell out PRs for picks.
👍 1
I did not ask about intermediates. Theyt are (attempting) 1.23. If they succeed and we don't pick the remainder - they just hit it on next upgrade. That seems bad.
h

hundreds-father-404

06/11/2020, 12:16 AM
Thanks, John. When editing the docs, if you update 1.29.x (the default version), be sure to update 1.30.x too, please. Or fine to only update 1.30.x (Reason we need 1.30.x is that it will be forked when creating 1.31.x)
w

witty-crayon-22786

06/11/2020, 12:17 AM
@enough-analyst-54434: i’m happy to make the doc update given that you’re already fighting this battle.
❤️ 1
had added it to my todo list after the rc discussion.
👍 1
e

enough-analyst-54434

06/11/2020, 12:18 AM
Thanks - that's be great. I was actually going to edit the "real" docs since the readme thing is still not standard until 2.0.
h

hundreds-father-404

06/11/2020, 12:18 AM
The “real” docs no longer exist for the release process - they live entirely on readme.io
e

enough-analyst-54434

06/11/2020, 12:18 AM
Damn.
OK - well, I'll happily let someone else do it since readme interaction is clear as mud to me at this point.
w

witty-crayon-22786

06/11/2020, 12:20 AM
you’re headed out tomorrow, right? whenever you need to disappear, put a blurb somewhere and one of us can finish tackling tomorrow (if need be)
👍 1
e

enough-analyst-54434

06/11/2020, 12:21 AM
Yeah. I'll have internet access for the next 4 days so I'll probably be hacking 2 or so nights to actually accomplish something. Its been a thrashy month.
h

happy-kitchen-89482

06/11/2020, 2:24 AM
Mea culpa re https://github.com/pantsbuild/pants/pull/8208: I guess I didn't consider changing console output as something that requires deprecation. I don't think we've ever formally committed to a specific form or content for pants's stdout/stderr, except for ConsoleTasks?
w

witty-crayon-22786

06/11/2020, 3:40 AM
i’m signing off the evening, but in case it saves time, one more comment re: https://pantsbuild.slack.com/archives/C18RRR4JK/p1591834598257800?thread_ts=1591832347.247700&cid=C18RRR4JK
realistically, i don’t expect that they will actually land all of the version bumps. so to be clear about what i’m suggesting, it’s that if they’re trying to actually land a particular version in their repository, that picking to that version and the following ones might save some time
👍 1
any versions for which they might just sanity check without actually landing the upgrade, we might be able to skip.
anyway, i’ll take a look at the state of it tomorrow
safe travels John!
❤️ 2
h

hundreds-father-404

06/15/2020, 9:33 PM
e

enough-analyst-54434

06/15/2020, 9:37 PM
Thanks. It's confusing to say not to use
gut cherry-pick
though. The PR branch really should be created with a cherry-pick. It's just that that cherry-pick should be done on a PR branch, not directly on the stable branch (and then pushed upstream w/o PR).
h

hundreds-father-404

06/15/2020, 9:41 PM
Oh, I see. So 1. create a new branch 2. git cherry-pick 3. open a PR Makes sense. Thanks
Okay, updated
e

enough-analyst-54434

06/16/2020, 12:50 AM
OK, and - unfortunately re-updated. Is there away to revert an edit? I meant to suggest an edit to get review, but clicked wrong button. I can;t seem to find a changelog / edit history to revert and re-apply as a suggestion.
There were two issues I think.
cherrypick
is actually
cherry-pick
, and from everything I know, checking out a branch and then creating a new branch are two operations that have nothing to do with each other.
So I changed the checkout of 1.28.x to a fetch to get latest commits.
h

hundreds-father-404

06/16/2020, 12:53 AM
Yes, in the right side, underneath all the boxes like “header”, there is a “page history” button
checking out a branch and then creating a new branch are two operations that have nothing to do with each other.
My thinking was so that you create the new branch with the release branch as your merge base, rather than
master
e

enough-analyst-54434

06/16/2020, 12:56 AM
Yeah, so just read up on git checkout / git branch. That is a way I've never used the two. So, that works iff your local 1.28.x is up to date. I'll see if I can find page history and revert / re-apply so you can do as you see fit.
h

hundreds-father-404

06/16/2020, 12:57 AM
Ah, yes. I think rather than trying to give the precise git commands, it’ll be better to say what I meant at a high level.
No worries on reverting. I can edit it. And 1.30.x is unedited for a diff. Thanks for taking a look!
e

enough-analyst-54434

06/16/2020, 12:58 AM
image.png
I'm not seeing history.
h

hundreds-father-404

06/16/2020, 12:59 AM
Screen Shot 2020-06-15 at 5.59.14 PM.png
e

enough-analyst-54434

06/16/2020, 12:59 AM
Huh - you must have more powers.
h

hundreds-father-404

06/16/2020, 12:59 AM
Ah, possibly because you’re on “Suggest edits”, rather than
<http://dash.readme.io|dash.readme.io>
e

enough-analyst-54434

06/16/2020, 1:00 AM
Am I being dumb - can I selve serve howto here?
h

hundreds-father-404

06/16/2020, 1:01 AM
I’m not sure what that means, but this is the URL I use for editing: https://dash.readme.io/project/pants/v1.29/docs. Your account has the same privileges as mine according to the settings
e

enough-analyst-54434

06/16/2020, 1:01 AM
Ah, admin panel. Got it.
👍 1
By self serve I mean how can I learn on my own from docs and not bug people, Tends to be my mode.
h

hundreds-father-404

06/16/2020, 1:04 AM
Got it. The only main gotcha with self-serving the docs is that you need to be careful which version you update. If you update 1.29.x (the default version), also update 1.30.x
e

enough-analyst-54434

06/16/2020, 1:06 AM
Oh wow. I mean, as a general concept having nothing to do with http servers. I like to self-serve learn. IE find documentation and figure things out from there, asking others as a last resort. Words!
h

hundreds-father-404

06/16/2020, 1:08 AM
Ah, yes, I interpreted your clarification that way. I did not read it as HTTP servers haha. Ah, sorry. Bad link. https://dash.readme.io/project/pants/v1.29/docs/welcome-to-pants I’ll handle this particular case - not at all a big deal and I appreciate the reviews!
e

enough-analyst-54434

06/16/2020, 1:13 AM
Ok - thanks, that did it. Bookmarked. Is the dual edit problem specific to this strange period of time or will that be with us as an ongoing problem?
💯 1
h

hundreds-father-404

06/16/2020, 1:15 AM
Double edit depends on the content we’re documenting. If we’re documenting a new feature, then it will only go in the newest dev docs (1.30.x) If changing docs that already exist in the main branch (1.29.x), then we need to update both the main and dev branches
e

enough-analyst-54434

06/16/2020, 1:42 AM
Ok, makes sense. I think that means in normal times we only edit / create dev docs since documentation would rarely rise to the level of a cherry-pick.