Skip to content

Conversation

@daxian-dbw
Copy link
Member

This is an attempt to fix the 'brew update' failure that happens a lot in your macOS CI runs: RPC failed; curl 56 SSLRead() return error -36.

With this change, we will try run brew update twice. The first run is the same as before, but if it fails, then we increase the Git buffer size before the second attempt.

Copy link
Member

@adityapatwardhan adityapatwardhan left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

With minor comment.

# A potential solution is to increase the Git buffer size to a larger number, say 150 mb. The default buffer size is 1 mb.
echo "First attempt of update failed. Increase Git buffer size and try again ..."
git config --global http.postBuffer 157286400
sleep 5
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Why is this required? Can you put a comment.

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Sure, will add a comment.

@daxian-dbw daxian-dbw closed this Nov 10, 2017
@daxian-dbw daxian-dbw reopened this Nov 10, 2017
# Try the update twice if the first time fails
brew update > /dev/null && break

# If the update fails again after increasing the Git buffer size, exit with error.
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

caching the brew cache in travis-ci might help:

cache:
  directories:
    - $HOME/Library/Caches/Homebrew

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks Travis! I will add this.

@adityapatwardhan
Copy link
Member

@daxian-dbw Can you rebase and get #5249

@daxian-dbw
Copy link
Member Author

@adityapatwardhan Sorry that I don't understand your comment -- the PR #5249 has been merged for over 2 weeks.

@daxian-dbw
Copy link
Member Author

Travis-CI finally passed: https://travis-ci.org/PowerShell/PowerShell/builds/300376520?utm_source=github_status&utm_medium=notification

I will not add code that caches the brew cache yet (Travis suggested that) because I want to verify if the fix can resolve the "Curl error" in brew update. We can add that cache optimization later. Issue #5436 opened to track it.

@adityapatwardhan
Copy link
Member

It seems the fix in #5429 does not fix all the webhook issues. The tests seems to have passed but GitHub doesnot seem it as passed. I will merge.

@adityapatwardhan adityapatwardhan merged commit da0b384 into PowerShell:master Nov 13, 2017
@daxian-dbw daxian-dbw deleted the oxsbootstrap branch November 13, 2017 20:34
@daxian-dbw
Copy link
Member Author

@adityapatwardhan The webhook fix in that PR is for the build report in our Team channel, not for the Github package. This should be something wrong with the Travis CI.

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

Successfully merging this pull request may close these issues.

3 participants