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

Use IP of new Ubuntu 20 server for IE #951

Merged
merged 1 commit into from
Oct 28, 2024

Conversation

cillian
Copy link
Contributor

@cillian cillian commented Oct 27, 2024

openfoodnetwork.ie has been upgraded to Ubuntu 20 now by following the instructions at https://github.com/openfoodfoundation/ofn-install/wiki/Migrating-a-Production-Server which were a great help.

I noticed that the pull_assets.yml and push_assets.yml playbooks don't include the storage directory here. If an instance is configured to use local ActiveStorage this can be an issue, because I had to manually copy over the storage directory to the new server, although most instances probably don't use local.

Let me know if you need any more details.

Copy link
Contributor

@rioug rioug left a comment

Choose a reason for hiding this comment

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

👍

Copy link
Member

@dacook dacook left a comment

Choose a reason for hiding this comment

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

Great work! FYI I'm also working on upgrading some servers to Ubuntu 20.

I noticed that the pull_assets.yml and push_assets.yml playbooks don't include the storage directory here.

I came across this too, because there's at least one other server using local storage. I aim to update the playbook to handle it better in future.

Just wondering if you have any additional backup strategy for the local files?

@dacook dacook merged commit cd06576 into openfoodfoundation:master Oct 28, 2024
1 check passed
@cillian
Copy link
Contributor Author

cillian commented Oct 28, 2024

@dacook Thanks. I backup up the local files periodically but I should probably automate it at some point.

@dacook
Copy link
Member

dacook commented Oct 28, 2024

Fair enough. I think that is already more comprehensive than another server I was looking at recently..

@dacook dacook mentioned this pull request Oct 31, 2024
9 tasks
@mkllnk
Copy link
Member

mkllnk commented Oct 31, 2024

although most instances probably don't use local.

Some files like reports are always stored locally because their lifetime is relatively short and we want a very fast response once the file is generated.

@cillian
Copy link
Contributor Author

cillian commented Nov 1, 2024

👍 Good to know.

@dacook dacook mentioned this pull request Nov 4, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

4 participants