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

.travis.yml: force xenial for docker builds #46

Merged
merged 1 commit into from
Jul 25, 2019
Merged

Conversation

commodo
Copy link
Contributor

@commodo commodo commented Jul 25, 2019

Apparently, even though Travis-CI is moving to use Xenial (16.04) dockers,
some Trusty dockers are still lurking around and randomly being assigned to
builds.

For this repo in particular, Trusty seems to be allocated mostly.
Force Xenial, since that will be the next default docker image in a few
months. This fails because of EXTRA_SSH=-oHostKeyAlgorithms=+ssh-dss
where the build complains that:

sftp -oHostKeyAlgorithms=+ssh-dss [secure]@[secure]
command-line line 0: Bad protocol 2 host key algorithms '+ssh-dss'.

We could remove the EXTRA_SSH stuff in the build, but we'd get the error
that the EXTRA_SSH option is fixing.

So, just default to Xenial docker images (for docker builds), add a note
about it in the .travis.yml and change later. We will need to re-update
builds at some point in time anyway.

Signed-off-by: Alexandru Ardelean [email protected]

Apparently, even though Travis-CI is moving to use Xenial (16.04) dockers,
some Trusty dockers are still lurking around and randomly being assigned to
builds.

For this repo in particular, Trusty seems to be allocated mostly.
Force Xenial, since that will be the next default docker image in a few
months. This fails because of `EXTRA_SSH=-oHostKeyAlgorithms=+ssh-dss`
where the build complains that:
```
sftp -oHostKeyAlgorithms=+ssh-dss [secure]@[secure]
command-line line 0: Bad protocol 2 host key algorithms '+ssh-dss'.
```

We could remove the EXTRA_SSH stuff in the build, but we'd get the error
that the EXTRA_SSH option is fixing.

So, just default to Xenial docker images (for docker builds), add a note
about it in the `.travis.yml` and change later. We will need to re-update
builds at some point in time anyway.

Signed-off-by: Alexandru Ardelean <[email protected]>
@commodo commodo requested review from tfcollins, mhennerich and a team July 25, 2019 06:17
@commodo commodo merged commit e7bb5cd into master Jul 25, 2019
@commodo commodo deleted the fix-centos-build3 branch July 25, 2019 13:47
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.

2 participants