Skip to content

Conversation

@markets
Copy link
Collaborator

@markets markets commented Jan 8, 2021

🎺 🎺 Living on the edge 🎺 🎺

Now that we are on Rails 6, let's try to be as much updated as we can.

NOTE After reviewed the release changelog and played a bit locally (tests are also passing ✔️ ), this seems safe to merge

@markets
Copy link
Collaborator Author

markets commented Jan 8, 2021

Tested ✅ merging

@markets markets merged commit de4a629 into develop Jan 8, 2021
@markets markets deleted the rails_6_1 branch January 8, 2021 18:16
@sseerrggii
Copy link
Contributor

Ok, I will test a little bit also in staging and deploy to production 💪

@sseerrggii
Copy link
Contributor

sseerrggii commented Jan 9, 2021

@markets I have a problem with unicorn when I tried to deploy it to staging

When I did sudo systemctl restart timeoverflow seams that this unit cannot restart

2021-01-09-065248_1960x2160_scrot

When I check on log/unicorn.err.log

I [2021-01-09T06:43:01.612635 #4249]  INFO -- : reaped #<Process::Status: pid 4328 exit 0> worker=0
I, [2021-01-09T06:43:01.613216 #4249]  INFO -- : reaped #<Process::Status: pid 4329 exit 0> worker=1
I, [2021-01-09T06:43:01.613345 #4249]  INFO -- : master complete
I, [2021-01-09T06:43:01.613257 #4966]  INFO -- : reaped #<Process::Status: pid 5039 exit 0> worker=0
I, [2021-01-09T06:43:01.613600 #4966]  INFO -- : reaped #<Process::Status: pid 5040 exit 0> worker=1
I, [2021-01-09T06:43:01.613859 #4966]  INFO -- : master complete
I, [2021-01-09T06:43:02.203674 #5414]  INFO -- : Refreshing Gem list
bundler: failed to load command: unicorn (/var/www/timeoverflow/shared/bundle/ruby/2.6.0/bin/unicorn)
RuntimeError: Couldn't find Active Storage configuration in /var/www/timeoverflow/releases/20210109053957/config/storage.yml

When I come back to master I could start this unit without problem with sudo systemctl start timeoverflow

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.

2 participants