Client: High CPU usage on Sierra

[gabriel]

From @ivanoats:
I am also seeing very high CPU usage on Sierra. I frequently have to kill keybase.
Log ID: 1494504486d5132a59beea1c

cc @cjb @maxtaco @strib

[gabriel]

cc @ivanoats

[maxtaco]

Which process is showing up in activity monitor?

On Wednesday, October 19, 2016, Gabriel Handford [email protected]
wrote:

cc @ivanoats https://github.com/ivanoats

—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/keybase/client/issues/4671#issuecomment-254963139,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AA05_6wTw9olVU1Lkh79nrxSeaFR65_Oks5q1p3zgaJpZM4Kbida
.

[ivanoats]

keybase

[maxtaco]

@patrickxb can you look into whether it's the same issue we've seen previously with lots of identify calls?

[maxtaco]

cc: @mmaxim there might be a gregor bug here with repeated delivery of user changes notifications

[mgaffney]

+1

[alex]

I appear to be experiencing this as well, running dtruss on the process (full CLI, /usr/local/Cellar/keybase/1.0.17/bin/keybase -d --log-file=/Users/alex_gaynor/Library/Logs/keybase.service.log service): all I see is select(0x0, 0x0, 0x0, 0x0, 0x70000722DDE8) = 0 0 over and over again.

[maxtaco]

@alex you're running an old version, and it looks like you got it via brew. We started release our on .dmg files.

This is probably an issue that the old binaries don't support Sierra well due to bugs/limitations in the old Go complier and libraries.

[alex]

@maxtaco that is correct that I got it via brew -- which indicates that I have the latest release, will a PR be sent to brew to upgrade the version they have?

[maxtaco]

We're low on bandwidth and are having trouble supporting the brew release in addition to our main release. I'm not sure what we decided on it, but we were hoping that users gravitate to our DMG release.

[alex]

@maxtaco I'm a bit confused; the version in brew appears to match the latest tag in this repo. Are releases not tagged here?

[maxtaco]

We've cut a whole bunch of dailies, but you're right, it's time to move up a version

Cc: @gabriel and @cjb

[alex]

This is a bit confusing for me, because the version homebrew has (1.0.17)
is the same as the version listed on https://github.com/keybase/client/tags,
are you not tagging releases in git?

On Mon, Oct 24, 2016 at 10:51 AM, Maxwell Krohn [email protected]
wrote:

We're low on bandwidth and are having trouble supporting the brew release
in addition to our main release. I'm not sure what we decided on it, but we
were hoping that users gravitate to our DMG release.

—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/keybase/client/issues/4671#issuecomment-255763028,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAADBLcU7epmqHo5ftMSXYF5pvfkv6y_ks5q3MXrgaJpZM4Kbida
.

"I disapprove of what you say, but I will defend to the death your right to
say it." -- Evelyn Beatrice Hall (summarizing Voltaire)
"The people's good is the highest law." -- Cicero
GPG Key fingerprint: D1B3 ADC0 E023 8CA6

[alex]

Great, I'm more than happy to handle sending the PR to brew once there's a
release tagged.

On Mon, Oct 24, 2016 at 8:47 PM, Maxwell Krohn [email protected]
wrote:

We've cut a whole bunch of dailies, but you're right, it's time to move up
a version

Cc: @gabriel https://github.com/gabriel and @cjb
https://github.com/cjb

—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/keybase/client/issues/4671#issuecomment-255906005,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAADBCCeODkgfRzEVUEDIdy-ZDSw3K10ks5q3VGrgaJpZM4Kbida
.

"I disapprove of what you say, but I will defend to the death your right to
say it." -- Evelyn Beatrice Hall (summarizing Voltaire)
"The people's good is the highest law." -- Cicero
GPG Key fingerprint: D1B3 ADC0 E023 8CA6

[gabriel]

https://github.com/Homebrew/homebrew-core/pull/6303

[gabriel]

Brew update is available