Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

release: v19.1.5 #40796

Closed
17 of 18 tasks
ajwerner opened this issue Sep 16, 2019 · 14 comments
Closed
17 of 18 tasks

release: v19.1.5 #40796

ajwerner opened this issue Sep 16, 2019 · 14 comments
Assignees

Comments

@ajwerner
Copy link
Contributor

ajwerner commented Sep 16, 2019

Candidate SHA: 6b14c0a
Deployment status: Deployed to ajwerner-release-1914 and ajwerner-release-1915 at 2:32EDT on 9/23/2019
Nightly Suite: https://teamcity.cockroachdb.com/viewLog.html?buildId=1502389&buildTypeId=Cockroach_Nightlies_NightlySuite

Release process checklist

Prep date: 9/23/2019

Release date: 9/30/2019

@ajwerner ajwerner self-assigned this Sep 16, 2019
@bdarnell
Copy link
Contributor

For the homebrew formula of this version, we want to add a message to the caveats section about the upcoming move to our own tap:

Due to a license change, the cockroach package in homebrew-core will no longer be updated when CockroachDB 19.2 is released. Please switch to https://github.com/cockroachdb/homebrew-tap instead.

We also need to make a version bump in the homebrew-tap repo in addition to homebrew-core. Since you're not a mac user @ajwerner I can handle the brew PRs on monday.

@knz
Copy link
Contributor

knz commented Sep 20, 2019

@rmloveland and I found out the SHA may have been picked in error. It's just 2 commits beyond provisional_201903251931_v19.1-rc.1, a commit from March 2019.

@ajwerner
Copy link
Contributor Author

🤦‍♂️ I'm restarting all of this now. Sorry all!

Restarting at d6a7e59.

@ajwerner
Copy link
Contributor Author

Rebooting this now at 6b14c0a targetting a 9/30 release date.

@ajwerner
Copy link
Contributor Author

@bdarnell I took a stab at typing up the brew PR. Here's the commit:

ajwerner/homebrew-core@ce081d9

If that looks right I'll PR it and follow the same patterns on our tap.

@bdarnell
Copy link
Contributor

👍 on submitting that PR upstream.

On our tap it'll be simpler (no caveat change) but a different SHA because it downloads the binary tarball instead of the source one.

@ajwerner
Copy link
Contributor Author

ajwerner commented Oct 3, 2019

This is still open due to the not-yet-merged Homebrew PR. I've gotten my hands on a mac and am working on it now.

@ajwerner
Copy link
Contributor Author

ajwerner commented Oct 7, 2019

The homebrew build seems to be broken due to rocksdb compilation problems:

In file included from /tmp/cockroach-20191005-36246-1n1kmgg/cockroach-v19.1.5/src/github.com/cockroachdb/cockroach/c-deps/rocksdb/db/builder.cc:19:
/tmp/cockroach-20191005-36246-1n1kmgg/cockroach-v19.1.5/src/github.com/cockroachdb/cockroach/c-deps/rocksdb/db/internal_stats.h:111:5: error: declaration shadows a variable in namespace 'rocksdb' [-Werror,-Wshadow]
    WAL_FILE_BYTES,
    ^
...

https://jenkins.brew.sh/job/Homebrew%20Core%20Pull%20Requests/49277/version=catalina/testReport/junit/brew-test-bot/catalina/install___build_bottle_cockroach/

@ajkr could I have some help with this later today?

@bdarnell
Copy link
Contributor

bdarnell commented Oct 7, 2019

The upstream fix is facebook/rocksdb#5779.

There's a workaround in https://github.com/cockroachdb/rocksdb/pull/41/files that would be easier to patch in via the homebrew formula.

@ajwerner
Copy link
Contributor Author

ajwerner commented Oct 7, 2019

Thanks @bdarnell!

The build was also broken due to the use of modules:

bin/prereqs: go/build: importGo golang.org/x/net/html: exit status 1
error writing go.mod: open /private/tmp/cockroach-20191007-76155-1qpr9si/cockroach-v19.1.5/pkg/mod/github.com/andybalholm/[email protected]/go.mod298498081.tmp: permission denied

I've updated https://github.com/Homebrew/homebrew-core/pull/44732/files to patch rocksdb/CMakeLists.txt and to set GO111MODULE=off.

I've verified that it now builds on a mac.

@ajwerner
Copy link
Contributor Author

ajwerner commented Oct 9, 2019

Ugh the build failed to build the bottle on Mojave and Catalina though not on High Sierra.

https://jenkins.brew.sh/job/Homebrew%20Core%20Pull%20Requests/49436/

/tmp/cockroach-20191009-18802-1c75inf/cockroach-v19.1.5/src/github.com/cockroachdb/cockroach/c-deps/rocksdb/utilities/persistent_cache/block_cache_tier_file.h:265:9: error: explicitly defaulted copy assignment operator is implicitly deleted [-Werror,-Wdefaulted-function-deleted]
    IO& operator=(const IO&) = default;
        ^

I'm happy to add -Wno-defaulted-function-deleted to the patch which I suspect will fix the error.

@bdarnell or @ajkr thoughts on whether this is surprising or expected?

@bdarnell
Copy link
Contributor

bdarnell commented Oct 9, 2019

That is surprising; it was building on mojave before (and i don't see any reference to this in the rocksdb tracker). Maybe this is from the xcode update that was just released? It'll be nice to get back to our own tap and isolated build.

You could set the warning flag, or apply the upstream fix from facebook/rocksdb#5095

@ajwerner
Copy link
Contributor Author

ajwerner commented Oct 9, 2019

There was indeed a major version upgrade to AppleClang. I'm just going to suppress the error.

@ajwerner
Copy link
Contributor Author

Due to my own confusion about how to safely upgrade the reg cluster from 19.1.4 to 19.1.5 I'm going to close this. If somebody tells me I need to do it I will go and figure it out. In the meantime I've asked for some guidance elsewhere.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants