-
Notifications
You must be signed in to change notification settings - Fork 99
Private mirrors2 #17
base: master
Are you sure you want to change the base?
Private mirrors2 #17
Conversation
prove an apt configuration string to use. Also it's possible to define own key id and/or keyserver to download the apt key, and/or to provide the apt key within a variable `clickhouse_repo_key_data`. Closes: AlexeySetevoi#15
@@ -24,6 +24,18 @@ matrix: | |||
dist: xenial | |||
sudo: true | |||
env: ANSIBLE_VERSION='ansible>=2.3,<2.4' | |||
- os: linux | |||
dist: bionic |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
As far as I took a look on: https://docs.travis-ci.com/user/reference/overview/ - this OS, same as Debian are not supported by Travis.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Wow. This is heavily outdated CI. I saw their comment on adding Xenial support in 2017... And still none was done.
What is your suggestion here? Remove debian/xenial/bionic from tests?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I don't think we have any direct impact on that CI, except making issue/request to them. Best would be to remove them from tests, if they're just running default image, which is tested anyway.
Travis supports only for trusty, so other distros couldn't be tested. Now there is a clickhouse_supported_distros list with list of supported versions in form of string: '{{ ansible_distribution }}_{{ ansible_distribution_release }}'
4c9cd02
to
5000990
Compare
Okay. I've trimmed all tests down to Trusty. Sad story. |
Falled builds for stretch, bionic and jessie. This distros have no support from travis, fallback to precise(sic!) and glibc errors)). Please remove this distros from matrix, then i ready to merge your improvements. |
I don't understand why tests are failing.
Is it related to mirror configuration? I don't feel so. |
clickhouse_supported
variable with explicit list ofclickhouse_supported_distros