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

fixes #4021 - upgrade to Rails 3.2.16 #1154

Closed
wants to merge 1 commit into from

Conversation

isratrade
Copy link
Member

No description provided.

@isratrade
Copy link
Member Author

@skottler
Copy link

The packages are going to need to be built in Koji, too.

@isratrade do you want to take a shot at doing that? I think this PR should wait until after the first RC drops because it might hold up getting RC's out the door.

@domcleal
Copy link
Contributor

We won't build these in koji because we use RHSCL, that'll remain on 3.2.8.

@skottler
Copy link

Whoops, right. There are some failing tests and I'm not sure whether they're related to the PR, but I see activerecord in the stacktrace so I'm rerunning the tests to know for sure.

[test]

@isratrade
Copy link
Member Author

no problem to wait on merging this PR

@domcleal
Copy link
Contributor

Merged as eccf81a for Foreman 1.4, thanks @isratrade.

@domcleal domcleal closed this Jan 16, 2014
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
3 participants