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 to rails 6.1 #3193

Merged
merged 3 commits into from May 13, 2021
Merged

Update to rails 6.1 #3193

merged 3 commits into from May 13, 2021

Conversation

tomhughes
Copy link
Member

Finally all the pieces are in place to upgrade to rails 6.1 so here's my offering...

A test instance is running at https://rails61.apis.dev.openstreetmap.org/.

Copy link
Collaborator

@gravitystorm gravitystorm left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to me. I worked on the same stuff for a different project yesterday, so I recognise the app:update changes in all the config files!

@gravitystorm
Copy link
Collaborator

I think we'll be able to enable most of the 6.1 framework defaults without any hassle, but we may as well wait until after this is merged since those are not blocking anything in particular - and will make bisecting easier if we do run into any issues.

@mmd-osm
Copy link
Contributor

mmd-osm commented May 12, 2021

Is the Docker build failure expected? I'm asking b/c some potential contributors to this site explicitly mentioned they're intending to use Docker to bootstrap their development process.

@tomhughes
Copy link
Member Author

tomhughes commented May 12, 2021

The docker build passes or fails based on the output of some random number generator so I just ignore those failures.

@gravitystorm
Copy link
Collaborator

Is the Docker build failure expected?

I've found the docker tests to be both slow and flaky, ever since it was introduced. I must say that I'm in the bad habit of ignoring the failures already.

@gravitystorm
Copy link
Collaborator

Is the Docker build failure expected?

I've found the docker tests to be both slow and flaky, ever since it was introduced. I must say that I'm in the bad habit of ignoring the failures already.

I should expand on that and say that most times I see it happen, I rerun the tests and it passes. So for my own PRs I do this (some times a couple of times) so that we get green lights across the board, and to save anyone else from diving into it or to remove any reluctance to merge. But when I see that everything is green and the docker test is red, I pretty much assume it's a false alarm.

@mmd-osm
Copy link
Contributor

mmd-osm commented May 12, 2021

I was just wondering if there's some change that still needs to be reflected in the Dockerfile and related files somehow. If that's not the case, then all is fine from my pov.

@tomhughes tomhughes merged commit 534c92e into openstreetmap:master May 13, 2021
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.

None yet

3 participants