libremesh / lime-packages

LibreMesh packages configuring OpenWrt for wireless mesh networking
https://libremesh.org/
GNU Affero General Public License v3.0
277 stars 96 forks source link

travis after_success.sh asks for password #210

Closed aparcar closed 6 years ago

aparcar commented 7 years ago

My current PR ci build seems never to finish as after_success.sh asks for a password:

iv undefined
iv undefined
Warning: Permanently added the ECDSA host key for IP address '138.255.88.52' to the list of known hosts.
Enter passphrase for key '/home/travis/build/libremesh/lime-packages/.ci/travisci_rsa': 
gmarcos87 commented 7 years ago

This error occurs when the encrypted key is created with a password. @nicopace in this commit you created the key (5729b781fa3a0f9019cedcd733ff87ca390842de), did you use password?

nicopace commented 7 years ago

No, and this worked for the other commits... I will try later to see what is happening

El 13 de agosto de 2017 1:16:35 AM GMT+02:00, Marcos Gutierrez notifications@github.com escribió:

This error occurs when the encrypted key is created with a password. @nicopace in this commit you created the key (5729b781fa3a0f9019cedcd733ff87ca390842de), did you use password?

-- You are receiving this because you were mentioned. Reply to this email directly or view it on GitHub: https://github.com/libremesh/lime-packages/issues/210#issuecomment-322011704

-- Enviado desde mi dispositivo Android con K-9 Mail. Por favor, disculpa mi brevedad.

nicopace commented 7 years ago

For sure it is related wirh the fact that the branch is not from the libremesh repository... I will modify the script to be able to finish the test without doing the privileged tasks like uploading the firmware or commenting the image. Regards

El 13 de agosto de 2017 12:45:20 AM GMT+02:00, Paul Spooren notifications@github.com escribió:

My current PR ci build seems never to finish as after_success.sh asks for a password:

iv undefined
iv undefined
Warning: Permanently added the ECDSA host key for IP address
'138.255.88.52' to the list of known hosts.
Enter passphrase for key
'/home/travis/build/libremesh/lime-packages/.ci/travisci_rsa': 

-- You are receiving this because you are subscribed to this thread. Reply to this email directly or view it on GitHub: https://github.com/libremesh/lime-packages/issues/210

-- Enviado desde mi dispositivo Android con K-9 Mail. Por favor, disculpa mi brevedad.

gmarcos87 commented 7 years ago

I propouse a fix in #214

ilario commented 6 years ago

Why not to test for the presence of $encrypted_4ddb7c1a1584_key and $encrypted_4ddb7c1a1584_iv variables at the beginning of .ci/after_success.sh? @gmarcos87 @nicopace do you think it would be enough? If so, should I do a PR with this or you do this? By the way I don't get the check for the presence of $TRAVIS_BUILD_DIR in .ci/after_success.sh added in dab1d57, I would remove it.

ilario commented 6 years ago

This change was implemented in 8f017dc as part of #214 which closes this issue :D I still miss the meaning of

if [ $TRAVIS_BUILD_DIR ]; then

but doesn't matter, it doesn't hurt.

gmarcos87 commented 6 years ago

@ilario In the hurry to restore continuous integration I forgot that line. Now I see it and I can not find the meaning. Surely this must have a motive and is not doing it. I think @nicopace should know the intent of that.