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

sqlite 3.35.4 #74485

Closed
wants to merge 1 commit into from
Closed

sqlite 3.35.4 #74485

wants to merge 1 commit into from

Conversation

cho-m
Copy link
Member

@cho-m cho-m commented Apr 3, 2021

Created with brew bump-formula-pr.

@carlocab
Copy link
Member

carlocab commented Apr 5, 2021

Only ARM failure is

Error: 1 failed step!
brew test --retry --verbose cash-cli

However, this looks to be pre-existing, as cash-cli's test fails even before the sqlite version bump.

@carlocab carlocab mentioned this pull request Apr 5, 2021
5 tasks
carlocab added a commit to carlocab/homebrew-core that referenced this pull request Apr 5, 2021
buku keeps getting stuck in long-running PRs (e.g. Homebrew#74485, Homebrew#74538).
This change fixes that.
carlocab added a commit that referenced this pull request Apr 6, 2021
buku keeps getting stuck in long-running PRs (e.g. #74485, #74538).
This change fixes that.
@carlocab carlocab force-pushed the bump-sqlite-3.35.4 branch from f8ea374 to 166a4d3 Compare April 6, 2021 12:45
@carlocab
Copy link
Member

carlocab commented Apr 6, 2021

We're about to hit the 3-day timeout, and it doesn't look like the CI run is going to finish. However, it's taken so long because of two formulae with problematic tests that I think I've fixed now (#74592, #74567).

Rebased against master to pull in those fixes.

@carlocab
Copy link
Member

carlocab commented Apr 7, 2021

ARM:

Error: 1 failed step!
brew test --retry --verbose cash-cli

Big Sur:

Error: 12 failed steps!
brew test --retry --verbose asuka
brew test --retry --verbose caffe
brew test --retry --verbose cash-cli
brew test --retry --verbose creduce
brew test --retry --verbose dynare
brew test --retry --verbose howdoi
brew test --retry --verbose hyperkit
brew install libtensorflow
brew install pypy
brew install pypy3
brew test --retry --verbose wasm-pack
brew test --retry --verbose youtube-dlc

Catalina:

Error: 6 failed steps!
brew test --retry --verbose caffe
brew test --retry --verbose cash-cli
brew test --retry --verbose howdoi
brew test --retry --verbose hyperkit
brew test --retry --verbose wasm-pack
brew test --retry --verbose youtube-dlc

Mojave:

Error: 7 failed steps!
brew test --retry --verbose cash-cli
brew test --retry --verbose howdoi
brew test --retry --verbose hyperkit
brew test --retry --verbose opencascade
brew test --retry --verbose swift
brew test --retry --verbose wasm-pack
brew test --retry --verbose youtube-dlc

The Catalina runner looks like it encountered a pretty bad failure. So did Big Sur.

@carlocab
Copy link
Member

carlocab commented Apr 7, 2021

The logs are a lot to get through right now, but these failures need investigating:

  • caffe
  • dynare
  • libtensorflow

If dynare is just a test timeout, then it should be safe to ignore.

@cho-m
Copy link
Member Author

cho-m commented Apr 7, 2021

Just looking at log output...

  • caffe test seems to be timing out.
    In 11.0 logs, it gets to 41 out of 50 iterations before Homebrew test times out.

    2021-04-06T19:53:00.4956750Z ==> /usr/local/Cellar/caffe/1.0_31/bin/caffe test -model /usr/local/Cellar/caffe/1.0_31/share/caffe/models/bvlc_reference_caffenet/deploy.prototxt -solver /usr/local/Cellar/caffe/1.0_31/share/caffe/models/bvlc_reference_caffenet/solver.prototxt -weights bvlc_reference_caffenet.caffemodel
    ...
    2021-04-06T19:53:00.6702660Z I0406 20:43:28.632457 227286528 caffe.cpp:290] Running for 50 iterations.
    ...
    2021-04-06T19:54:12.1914030Z I0406 20:52:57.932752 168484352 caffe.cpp:313] Batch 41, prob = 0.000966262
    2021-04-06T19:54:12.1916750Z Error: caffe: failed
    2021-04-06T19:54:12.1918070Z An exception occurred within a child process:
    2021-04-06T19:54:12.1925640Z   Timeout::Error: execution expired
    2021-04-06T19:54:12.1929010Z /usr/local/Homebrew/Library/Homebrew/formula.rb:2102:in `write'
    
  • libtensorflow install runs out of disk space.

    2021-04-07T09:13:13.3160150Z ==> Pouring libtensorflow-2.4.1.big_sur.bottle.tar.gz
    2021-04-07T09:13:13.3162260Z cp: /usr/local/Cellar/libtensorflow/2.4.1/bin/transform_graph: No space left on device
    2021-04-07T09:13:13.3164340Z cp: /usr/local/Cellar/libtensorflow/2.4.1/.brew: No space left on device
    
  • cash-cli possible related issue: https://api.exchangeratesapi.io/latest requires access key xxczaki/cash-cli#79
    https://exchangeratesapi.io/ may require API key. An alternative may be to just test cash --help

@carlocab
Copy link
Member

carlocab commented Apr 8, 2021

Thanks for looking into this, @cho-m. It would be good to test caffe and libtensorflow locally just to be sure. I'll try to find time to do this soon.

Copy link
Member

@carlocab carlocab left a comment

Choose a reason for hiding this comment

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

Managed to test caffe and libtensorflow on Big Sur locally. They're fine. dynare was an execution timeout, which happens sometimes.

Thanks, @cho-m!

@BrewTestBot
Copy link
Member

:shipit: @carlocab has triggered a merge.

@cho-m cho-m deleted the bump-sqlite-3.35.4 branch April 15, 2021 04:24
@github-actions github-actions bot added the outdated PR was locked due to age label May 17, 2021
@github-actions github-actions bot locked as resolved and limited conversation to collaborators May 17, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
outdated PR was locked due to age
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants