From owner-freebsd-git@freebsd.org Mon Sep 23 20:44:18 2019 Return-Path: Delivered-To: freebsd-git@mailman.nyi.freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2610:1c1:1:606c::19:1]) by mailman.nyi.freebsd.org (Postfix) with ESMTP id 7E162FF8F0 for ; Mon, 23 Sep 2019 20:44:18 +0000 (UTC) (envelope-from wlosh@bsdimp.com) Received: from mail-qk1-x72b.google.com (mail-qk1-x72b.google.com [IPv6:2607:f8b0:4864:20::72b]) (using TLSv1.3 with cipher TLS_AES_128_GCM_SHA256 (128/128 bits) server-signature RSA-PSS (4096 bits) client-signature RSA-PSS (2048 bits) client-digest SHA256) (Client CN "smtp.gmail.com", Issuer "GTS CA 1O1" (verified OK)) by mx1.freebsd.org (Postfix) with ESMTPS id 46cbsF6c4vz4W46 for ; Mon, 23 Sep 2019 20:44:17 +0000 (UTC) (envelope-from wlosh@bsdimp.com) Received: by mail-qk1-x72b.google.com with SMTP id u186so16937248qkc.5 for ; Mon, 23 Sep 2019 13:44:17 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=bsdimp-com.20150623.gappssmtp.com; s=20150623; h=mime-version:references:in-reply-to:from:date:message-id:subject:to :cc; bh=wNYiBp+rr5XZj5oNVrHHUyDTm5DbGPxpvwZPtHhqXEA=; b=BAv2jWEZMbE/zwn8igPJEm5stw4tsrCU335TWQby6wdkJc+vMPK9iv/XM3EfeuKHaI RQBOsDjfwooeTCSMvK0Yc9Uco395aiPKExdTpwr2or10JA6cCz1aVSAN7CeWw/Ivx/Kh LsXOq4VhV+8+89CDX+C2qwDzqL9y0535Yr7+iHz7Ew96DgYwwSLTa2ewRhX+oEmGfPqN K5vmzqvmjc8F3VvgNp9OWVm+Y/fZejhzfka3cKoZMy7pVj6TlB3KF5WB91zKgZeq6mpK rf3SuND1BmlZrwyUmFtKhRGSCc0So1j/dzmWxW4ueE7YOdWb95K9aaYdkwfuFD76nOAX iF8A== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:mime-version:references:in-reply-to:from:date :message-id:subject:to:cc; bh=wNYiBp+rr5XZj5oNVrHHUyDTm5DbGPxpvwZPtHhqXEA=; b=unUrFNX2jxDN0nWrAbHs7UTKhvv6qYnF8U6RgVj9K+XL6e8MA6MKi26JnqrHYE/nx0 guWjDkF5jzjO+35IUmjEkoQhKVF/TBWarjUCHvU+EGg1P1jdRXoaOqqemag5KvLt/vFY MgSvwQCPjefZuqYnYIHCd2OteJaaAHy98MNmhzbPvOJFmgsSgjptstzW2aoQd+X83NRc //12DXOXx2YGkbbmMmXPl7Doq5aXBwJaNHHWzvXOO2W9ExQJtU5S3lw/WoiGtiv6zCm4 M+5ddgkAQNcVQZr/IelOQ4VWPaPDQgWQZIKq0GV26vGpjxrfJYCQ2GrBvZj983pACNE5 iM+w== X-Gm-Message-State: APjAAAUVhS8GzChayZ4ElRulkgTm9m673qfUEWrofwiHmdz4pPxxnaaB RIERIEIDwg4A/WFjkX2PeRpZd3JT02ZGxhLCZIvk/J6u51Q= X-Google-Smtp-Source: APXvYqwmO3FYuZJ5MRjSjU1cP1DBiEI3MbrOzpwfXp7EZhVdretsVTE1VMWVZhpszudz/1xWVDkG/qCo1CL22+qM57g= X-Received: by 2002:a37:7bc7:: with SMTP id w190mr2019524qkc.215.1569271456565; Mon, 23 Sep 2019 13:44:16 -0700 (PDT) MIME-Version: 1.0 References: <20190923183424.ebnghzf67mx56aom@mutt-hbsd> <20190923185113.dyvxxn36gvj4dtu5@mutt-hbsd> In-Reply-To: From: Warner Losh Date: Mon, 23 Sep 2019 22:44:04 +0200 Message-ID: Subject: Re: Service disruption: git converter currently down To: =?UTF-8?Q?Ulrich_Sp=C3=B6rlein?= Cc: Shawn Webb , freebsd-git@freebsd.org X-Rspamd-Queue-Id: 46cbsF6c4vz4W46 X-Spamd-Bar: ---- Authentication-Results: mx1.freebsd.org; dkim=pass header.d=bsdimp-com.20150623.gappssmtp.com header.s=20150623 header.b=BAv2jWEZ; dmarc=none; spf=none (mx1.freebsd.org: domain of wlosh@bsdimp.com has no SPF policy when checking 2607:f8b0:4864:20::72b) smtp.mailfrom=wlosh@bsdimp.com X-Spamd-Result: default: False [-4.87 / 15.00]; ARC_NA(0.00)[]; NEURAL_HAM_MEDIUM(-1.00)[-1.000,0]; R_DKIM_ALLOW(-0.20)[bsdimp-com.20150623.gappssmtp.com:s=20150623]; FROM_HAS_DN(0.00)[]; RCPT_COUNT_THREE(0.00)[3]; TO_DN_SOME(0.00)[]; NEURAL_HAM_LONG(-1.00)[-1.000,0]; MIME_GOOD(-0.10)[multipart/alternative,text/plain]; PREVIOUSLY_DELIVERED(0.00)[freebsd-git@freebsd.org]; DMARC_NA(0.00)[bsdimp.com]; TO_MATCH_ENVRCPT_SOME(0.00)[]; DKIM_TRACE(0.00)[bsdimp-com.20150623.gappssmtp.com:+]; RCVD_IN_DNSWL_NONE(0.00)[b.2.7.0.0.0.0.0.0.0.0.0.0.0.0.0.0.2.0.0.4.6.8.4.0.b.8.f.7.0.6.2.list.dnswl.org : 127.0.5.0]; R_SPF_NA(0.00)[]; FORGED_SENDER(0.30)[imp@bsdimp.com,wlosh@bsdimp.com]; FREEMAIL_TO(0.00)[gmail.com]; MIME_TRACE(0.00)[0:+,1:+,2:~]; IP_SCORE(-2.87)[ip: (-9.46), ipnet: 2607:f8b0::/32(-2.64), asn: 15169(-2.20), country: US(-0.05)]; ASN(0.00)[asn:15169, ipnet:2607:f8b0::/32, country:US]; FROM_NEQ_ENVFROM(0.00)[imp@bsdimp.com,wlosh@bsdimp.com]; RCVD_TLS_ALL(0.00)[]; RCVD_COUNT_TWO(0.00)[2] Content-Type: text/plain; charset="UTF-8" Content-Transfer-Encoding: quoted-printable X-Content-Filtered-By: Mailman/MimeDel 2.1.29 X-BeenThere: freebsd-git@freebsd.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: Discussion of git use in the FreeBSD project List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Mon, 23 Sep 2019 20:44:18 -0000 On Mon, Sep 23, 2019, 10:38 PM Ulrich Sp=C3=B6rlein = wrote: > Am Mo., 23. Sept. 2019 um 20:51 Uhr schrieb Shawn Webb > : > > > > On Mon, Sep 23, 2019 at 08:42:10PM +0200, Ulrich Sp??rlein wrote: > > > Am Mo., 23. Sept. 2019 um 20:34 Uhr schrieb Shawn Webb > > > : > > > > > > > > Hey Ulrich, > > > > > > > > I appreciate your hard work in maintaining the git mirror. Work lik= e > > > > this can sometimes go unthanked. I want to take a moment to show > > > > appreciation for you and the FreeBSD project in maintaining the git > > > > mirror. > > > > > > > > I do have a few concerns with what was stated in your email. I've > > > > written my concerns inline. I hope this discussion is a positive on= e, > > > > wherein upstream and downstream can effectively come to a conclusio= n. > > > > > > > > On Mon, Sep 23, 2019 at 08:16:25PM +0200, Ulrich Sp??rlein wrote: > > > > > Am Mo., 23. Sept. 2019 um 19:51 Uhr schrieb Sean Chittenden > > > > > : > > > > > >> > > > > > >> Please note however, that more "garbage" metadata escaped from > SVN into > > > > > >> github, meaning 3rd parties have a hard time re-running the > conversion and > > > > > >> making sure that it matches SVN down to the metadata (i.e. > timestamps). > > > > > >> > > > > > >> Eventually, this will have to be re-rolled and a new "master" > branch will > > > > > >> be force-pushed into github. There's no timeline for this yet. > > > > > > > > > > > > > > > > > > Wait, what? Can you elaborate? > > > > > > > > > > > > Discussion of a force-push to github has occurred a few times > and been explicitly ruled out because most of our corporate citizens use > github to integrate changes from FreeBSD. Rerolling master was universal= ly > rejected when we socialized wanting to do this due to the level of > disruption this would cause. The feedback was that this would be a > high-cost, low-value operation. In the tradeoffs of purity vs pragmatism= , > pragmatism wins every time (that is the FreeBSD way). > > > > > > > > > > > > -sc > > > > > > > > > > > > > > > This is not just about pragmatism and the disruption it would > cause is > > > > > vastly overblown by people who don't seem to know much about the > git > > > > > storage model. > > > > > > > > > > There *is* garbage metadata in the published version on github, > there > > > > > *is* a disclaimer on https://wiki.freebsd.org/GitWorkflow since > > > > > forever, and the cost of switching from 1 published branch to > another > > > > > is literally: > > > > > > > > > > - git diff origin/broken_master mybranch > mybranch.patch > > > > > - git checkout -b fixed_branch origin/fixed_master > > > > > - patch < mybranch.patch > > > > > > > > Such a workflow breaks historical accuracy. Instead of `git annotat= e` > > > > showing the history properly, it's now based on an "epoch commit". > > > > Sure such a commit brings the branch to a working condition, but at > > > > the cost of history. > > > > > > Is there really that much value in having "git blame" work in that > environment? > > > My mental model is of short-lived branches that get upstreamed, so I'= m > biased > > > towards this not being all that big of a problem (for some at least). > > > > > > > > > > > > > It should also be possible to merge both broken and fixed master > into > > > > > your branch (at the exact same SVN revision in time) and then you > can > > > > > follow fixed_master going forward. You'll schlepp around double t= he > > > > > commit history, but not tree objects. > > > > > If you want to retain history, you can upstream the changes prior > to > > > > > the switch > > > > > > > > I so wish that were possible for certain downstream projects. We're > > > > unable to upstream the majority of our work. To argue "upstream you= r > > > > work and you won't be affected" is to choose an argument that does > not > > > > reflect the reality of a growing portion of FreeBSD's downstream > > > > consumers: the inability to work effectively with upstream. > > > > > > :/ > > > > > > I'm 80% sure that you can just merge both branches and things will be > fine > > > (though the exact incantation will surely be black magic). I'd love t= o > > > try this on > > > an actual repo though, I don't have the time to craft some test repo > to verify > > > this assumption, and then find out that other repos are different). > > > > HardenedBSD's github repo has existed since 2013, with branches > > stemming from that work existing still today. Perhaps HardenedBSD is > > somewhat in a special case: we aim to provide the BSD community with a > > clean-room reimplementation of publicly-documented parts of the > > grsecurity patchset. > > > > With FreeBSD not taking the same approach, we will have very > > long-lived branches. For example, our hardened/current/master branch > > follows FreeBSD's HEAD and syncs every six hours. Meaning, we maintain > > our patches, resolving whatever few merge conflicts arrive. The > > hardened/current/master branch was created so many years ago, I've > > forgotten when it was actually created (perhaps in 2013?) > > > > Though HardenedBSD's cause for existence may be a special case, this > > problem can be viewed in a general fashion. I'm confident HardenedBSD > > is not alone in facing issues of these types. > > > > Thanks, > > What I don't understand is how a security focused project can trust a > random source for the svn2git conversion. I could have planted a bunch > of backdoors and then come up with some SVN metadata corruption > conspiracy as to why the commit hashes are different. Why would you > trust me > The metadata has been independently audited. otherwise I'd be worried... Git use has taken off to such an extent we'd need a well tested easy button to do migrations that has good messaging and clear, easy to follow instructions that are as automated as possible. Warner HardenedBSD of all people should be running the converter themselves > and check that the content really matches what is in SVN (which it > currently doesn't for metadata). > > Cheers, > Uli > _______________________________________________ > freebsd-git@freebsd.org mailing list > https://lists.freebsd.org/mailman/listinfo/freebsd-git > To unsubscribe, send any mail to "freebsd-git-unsubscribe@freebsd.org" >