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

Adds rbx-19mode and jruby-19mode to Travis CI #6264

Closed
wants to merge 1 commit into from

Conversation

parndt
Copy link
Contributor

@parndt parndt commented May 11, 2012

I've added the rubies to allow_failures so that the build status is not interrupted. I would love to see Rails run CI against these rubies as I believe it will help to fix bugs in the rubies themselves and also Rails.

Both are running only in 1.9 mode as master does not support 1.8 anymore.

Thanks for the time taken to review this.

…res on these rubies so that the build status is not interrupted.
@michaelklishin
Copy link
Contributor

@parndt Rails VM image does not have Rubinius as far as I remember. That needs to be solved first.

@guilleiguaran
Copy link
Member

@michaelklishin what about jruby?

@michaelklishin
Copy link
Contributor

Actually, Rubinius and JRuby are there but not aliased exactly the same way as on Ruby images and there's only one mode. It's not a big deal to provision both modes and add aliases and deploy new images but it will take a day or few.

@joshk
Copy link
Contributor

joshk commented May 11, 2012

Do we even know if Rails passes on JRuby? Or what the test times are in comparison to 1.9.3? It might be best waiting until we have our new VM architecture up and running as we can then allocate extra VMs with next to no work.

@arunagw
Copy link
Member

arunagw commented May 11, 2012

I think we cannot do it for JRuby right now. I faced some issue running it locally for openssl doing bundle install.

I think for JRuby we should wait for JRuby 1.7.

@joshk
Copy link
Contributor

joshk commented May 11, 2012

We could add one or two Rails bits into an ignore matrix (in case it fails), but step one is to get it passing locally.

Feel free to ping me on IRC or in Campfire and we can talk more about this.

On 11/05/2012, at 12:16 PM, Arun Agrawal wrote:

I think we cannot do it for JRuby right now. I faced some issue running it locally for openssl doing bundle install.

I think for JRuby we should wait for JRuby 1.7.


Reply to this email directly or view it on GitHub:
#6264 (comment)

@parndt
Copy link
Contributor Author

parndt commented May 14, 2012

Should I split this up into rbx and jruby separately?

@michaelklishin
Copy link
Contributor

I think the consensus is that Rails core needs to make everything pass on JRuby and Rubinius locally first.

@parndt
Copy link
Contributor Author

parndt commented May 14, 2012

I see your point however with CI and allow_failures it means that everyone can see the failures and help out without affecting the build status.

@joshk
Copy link
Contributor

joshk commented May 14, 2012

The main issue right now is that we only have 5 VMs for Rails, adding rbx or jruby will kill the test times. Also, have anyone benchmarked how long rbx or jruby take for rails?

On 14/05/2012, at 4:05 AM, Philip Arndt wrote:

I see your point however with CI and allow_failures it means that everyone can see the failures and help out without affecting the build status.


Reply to this email directly or view it on GitHub:
#6264 (comment)

@arunagw
Copy link
Member

arunagw commented May 14, 2012

Agree that it will kill test times.

@parndt
Copy link
Contributor Author

parndt commented May 14, 2012

Yes, unfortunately it will always be inconvenient to test extra rubies
:-) not sure how to get around that.

@arunagw
Copy link
Member

arunagw commented May 14, 2012

In my opinion try to fix them locally first. if there are not much errors then we can think of adding them. Running them as a failures is not a good option.

Suggestions ??

@joshk
Copy link
Contributor

joshk commented May 14, 2012

Lets put this on hold until we move to a true VM setup, that way we can allocate more VMs for Rails easily and test times won't explode so much.

Until then, why not have someone work on making sure various parts of Rails pass on rbx and jruby and work with the respective teams on making the rails test suite run faster.

On 14/05/2012, at 11:25 AM, Philip Arndt wrote:

Yes, unfortunately it will always be inconvenient to test extra rubies
:-) not sure how to get around that.


Reply to this email directly or view it on GitHub:
#6264 (comment)

@joshk
Copy link
Contributor

joshk commented Jul 24, 2012

A little update for everyone.

I am all for adding jruby and rbx to Rails CI, but we need to wait a few more months while we sort out the new VM setup.

Has anyone tried running CI tests for jruby or rbx locally and benchmarked how long they take?

@arunagw
Copy link
Member

arunagw commented Jul 24, 2012

We have 3k fails with Rails master and JRuby 1.9 mode. Not sure about the time. I will post here the time from my machine.

@joshk
Copy link
Contributor

joshk commented Jul 24, 2012

Until the new VM setup is ready, why not try to reduce the failures first and maybe we can look at turning on testing for certain parts of Rails (eg. AR and RBX) if they fall within the timeout limits :)

@steveklabnik
Copy link
Member

Hey there! @joshk, would this be ready to be merged in, you think?

@joshk
Copy link
Contributor

joshk commented Oct 11, 2012

Sadly no, still working on the infrastructure change, find me in IRC tomorrow and we can talk more about it :)

On 12/10/2012, at 12:33 AM, Steve Klabnik wrote:

Hey there! @joshk, would this be ready to be merged in, you think?


Reply to this email directly or view it on GitHub.

@steveklabnik
Copy link
Member

Okay :D ❤️

@steveklabnik
Copy link
Member

We were wondering about times for rbx/jruby to run the test suite. The tests do not run at all on rbx (see the linked bug above) but on Jruby:

noglob rake  752.45s user 37.07s system 230% cpu 5:42.64 total

This is on my year-old MacBook Air.

@steveklabnik
Copy link
Member

The rbx 'bug' was not a bug, and while the ActiveModel tests seem to hang on my machine, the suite overall does this:

rbx -X19 -S bundle exec rake  805.08s user 34.56s system 113% cpu 12:17.11 total

@schneems
Copy link
Member

schneems commented Mar 9, 2013

Ping 4 months stale, what's the status of testing RBX and JRuby? Needs a rebase.

@steveklabnik
Copy link
Member

@schneems last time we discussed this, it was 'do they even run?'

I cannot currently compile rbx on my machine, so I can't tell you the answer these days :/

@schneems
Copy link
Member

schneems commented Mar 9, 2013

I've deployed Rails apps on JRuby to Heroku before so the tests should pass for that one. What if we just add JRuby today, and RBX another time?

@joshk
Copy link
Contributor

joshk commented Mar 9, 2013

Running on RBX and JRuby is very different to testing on RBX and JRuby.

That said, Travis is ready, we just recommend adding component by component, and making sure they pass locally before adding to Travis.

:)

On 10/03/2013, at 8:05 AM, Richard Schneeman notifications@github.com wrote:

I've deployed Rails apps on JRuby to Heroku before so the tests should pass for that one. What if we just add JRuby today, and RBX another time?


Reply to this email directly or view it on GitHub.

@arunagw
Copy link
Member

arunagw commented Mar 9, 2013

AR-JDBC is not ready with Rails master right now!

@arunagw
Copy link
Member

arunagw commented Apr 12, 2013

Hey Guys,

I have pushed this into a branch to see fails for JRuby. 0f138e02167c

Will be working more to fix these tests.

@joshk I actually wanted to see fails and I saw travis was not running any rails tests at moment so pushed.

@joshk
Copy link
Contributor

joshk commented Apr 12, 2013

WAT? Travis was not running any Rails builds?

What happened?

On 12/04/2013, at 2:11 AM, Arun Agrawal notifications@github.com wrote:

Hey Guys,

I have pushed this into a branch to see fails for JRuby. 0f138e0

Will be working more to fix these tests.

@joshk I actually wanted to see fails and I saw travis was not running any rails tests at moment so pushed.


Reply to this email directly or view it on GitHub.

@arunagw
Copy link
Member

arunagw commented Apr 12, 2013

Sorry about my bad english. I meant not at the moment. They are idle :-)

@joshk
Copy link
Contributor

joshk commented Apr 12, 2013

ahhhhhhh, no worries, I was worried for a second there :)

On 12/04/2013, at 2:24 AM, Arun Agrawal notifications@github.com wrote:

Sorry about my bad english. I meant not at the moment. They are idle :-)


Reply to this email directly or view it on GitHub.

@rafaelfranca
Copy link
Member

@parndt
Copy link
Contributor Author

parndt commented Aug 1, 2013

Wahooo! Next step: rbx-19mode?

@rafaelfranca
Copy link
Member

Done too.

@parndt
Copy link
Contributor Author

parndt commented Aug 1, 2013

Excellent.. I'll just go ahead and delete my patch branch then 😄

@parndt parndt deleted the patch-1 branch August 1, 2013 19:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

8 participants