Skip to content

Comments

Fix Homebrew deployment on CircleCI#3564

Merged
ulope merged 2 commits intoraiden-network:masterfrom
ulope:ci/fix-homebrew
Feb 28, 2019
Merged

Fix Homebrew deployment on CircleCI#3564
ulope merged 2 commits intoraiden-network:masterfrom
ulope:ci/fix-homebrew

Conversation

@ulope
Copy link
Collaborator

@ulope ulope commented Feb 27, 2019

No description provided.

Copy link
Contributor

@rakanalh rakanalh left a comment

Choose a reason for hiding this comment

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

Looks good... so you're pretty much:

  1. Adding github.com to known_hosts because of not in known_hosts (IIRC) error
  2. Adding support to add RCs to homebrew

Edit: Weird error in circle CI lint step... wondering how that happend 🤔

rm $FORMULA_FILE.bak
export UPDATED_SHA256=$(openssl dgst -sha256 dist/archive/raiden-${ARCHIVE_TAG}-macOS-x86_64.zip)
export FORMULA_FILE="homebrew-raiden/raiden.rb"
sed -i.bak -r "s/[0-9]+\.[0-9]+\.[0-9]+(-?rc.)?/${ARCHIVE_TAG/v/}/g" ${FORMULA_FILE}
Copy link
Contributor

Choose a reason for hiding this comment

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

Was removing the space in -i.bak done on purpose?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Yes it was. Previously (on Travis) this ran on macOS which has BSD sed. On Circle this is now running in a linux container with gnu sed which (for whatever reason) doesn't accept a space after the -i option.

@ulope
Copy link
Collaborator Author

ulope commented Feb 28, 2019

  1. Adding github.com to known_hosts because of not in known_hosts (IIRC) error

Yes, with additional checking of the key fingerprint to prevent MitM attacks.

  1. Adding support to add RCs to homebrew

Also yes. Previously the -rcX suffixes would 'pile up' after multiple rc releases (e.g. raiden-v0.100.2-rc2-rc3-macOS.zip) since the regex didn't capture them.

Edit: Weird error in circle CI lint step... wondering how that happend 🤔

👀

That line hasn't been touched since early 2018...

@ulope
Copy link
Collaborator Author

ulope commented Feb 28, 2019

Ok found the reason for the lint failure. On Circle we get astroid==2.2.0 (as a dependency of pylint) which seems to cause this error.

Related: pylint-dev/astroid#651, pylint-dev/pylint#2774

/edit: @konradkonrad already fixed it another way in #3557. Still we should pin astroid since it seems to break other stuff as well.

@codecov
Copy link

codecov bot commented Feb 28, 2019

Codecov Report

Merging #3564 into master will decrease coverage by 0.02%.
The diff coverage is 100%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #3564      +/-   ##
==========================================
- Coverage   77.15%   77.12%   -0.03%     
==========================================
  Files         103      103              
  Lines       13631    13631              
  Branches     1913     1913              
==========================================
- Hits        10517    10513       -4     
- Misses       2449     2450       +1     
- Partials      665      668       +3
Impacted Files Coverage Δ
raiden/accounts.py 47.56% <100%> (ø) ⬆️
raiden/network/proxies/secret_registry.py 82.71% <0%> (-1.24%) ⬇️
raiden/tasks.py 73.8% <0%> (-0.8%) ⬇️
raiden/network/transport/matrix/client.py 69.2% <0%> (-0.77%) ⬇️
raiden/network/transport/matrix/transport.py 75.41% <0%> (-0.76%) ⬇️
raiden/transfer/events.py 77.55% <0%> (+0.39%) ⬆️
raiden/raiden_service.py 89.7% <0%> (+0.54%) ⬆️
raiden/transfer/mediated_transfer/events.py 84.11% <0%> (+0.93%) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update a6e95b9...4ab8b9d. Read the comment docs.

@ulope ulope merged commit 8281602 into raiden-network:master Feb 28, 2019
@ulope ulope deleted the ci/fix-homebrew branch February 28, 2019 13:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants