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

Update otp_release list in .travis.yml #421

Merged
merged 1 commit into from
Dec 12, 2013

Conversation

mbbx6spp
Copy link
Contributor

Changes:

  • Removed R14* Erlang version targets from .travis.yml
  • Added R15B* and R16B Erlang version targets to .travis.yml

Background:
Until this change Travis CI was building against Erlang R14 which is not
supported for Riak (KV) according to Basho documentation,
which states, "Riak requires Erlang R15B01". The R14 builds consistently
failed since the ring resizing change was merged.

Since .travis.yml was updated last (a year ago) R15B0{2-3} and R16B
were released. R16B01 and R16B02 do not appear to be offered as choices
in Travis CI yet (according to Travis CI's documentation).

Changes:
* Removed R14* Erlang version targets from `.travis.yml`
* Added R15B* and R16B Erlang version targets to `.travis.yml`

Background:
Until this change Travis CI was building against Erlang R14 which is not
supported for Riak (KV) according to [Basho documentation]
(http://docs.basho.com/riak/latest/ops/building/installing/erlang/),
which states, *"Riak requires Erlang R15B01"*. These consistently failed
since the ring resizing change was merged.

Since `.travis.yml` was updated last (a year ago) R15B0{2-3} and R16B
were released. R16B01 and R16B02 to not appear to be offered as choices
in Travis CI yet ([according to Travis CI's documentation]
(http://about.travis-ci.org/docs/user/languages/erlang/)).
@slfritchie
Copy link
Contributor

+1, thanks, and many apologies for the delay!

slfritchie added a commit that referenced this pull request Dec 12, 2013
Update otp_release list in .travis.yml
@slfritchie slfritchie merged commit a725d33 into basho:develop Dec 12, 2013
@mbbx6spp mbbx6spp deleted the clean-up-.travis.yml branch December 12, 2013 11:32
@jrwest
Copy link
Contributor

jrwest commented Dec 12, 2013

I didn't realize we were actually relying on travis for builds anymore. They rarely seem to reflect the correct results (or well, yea its probably failing but its the wrong failure). Is there are reason we just don't remove the .yml file?

@seancribbs
Copy link
Contributor

@jrwest We have already disabled the hook, they do not currently run.

@jrwest
Copy link
Contributor

jrwest commented Dec 12, 2013

any reason to keep the file then? would be happy to open a quick PR to remove it (although i guess we should do the other repos too then?).

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.

4 participants