Skip to content

Conversation

@jimallman
Copy link
Member

@jimallman jimallman commented Jun 1, 2022

This is necessary for now, since we need to deploy old web-apps etc. on new Ubuntu servers.

NB This is a work in progress, so probably best not to merge at this point. But I'm interested in reviews and (esp.) comments where I might have misunderstood the intent of some actions, or failed to consider the consequences of different deployment scenarios. Thanks!

This script has an annoying tendency to fail silently, with no user
feedback. I've adapted the logging feature used by other scripts to
write a log to ~/log/messages (currently /home/ubuntu/log/messages).
Apparently this was formerly a manual step? Now we make it official,
setting the `hostname` and `/etc/hostname` to survive reboots of the
deployed server. This is necessary for apache to trust some
configuration.

Any chance this goes wrong if we have deployed multiple systems to
the same server? e.g. `devtree.opentreeoflife.org` and
`devapi.opentreeoflife.org`. Comments welcome!
Many of the packages we need for web2py apps (still on python2.*) are
no longer provided in modern Ubuntu. Add the necessary repositories to
fetch these anyway, alongside their python3 counterparts for our newer
code.
Later steps in as-admin.sh would be skipped (w/o warning!) if
some of the files we want to backup are missing, which is entirely
likely if only some systems are installed.
Apache (rightly) complained about the missing user, but would also have
failed with the old paths to /home/opentree/...
@jimallman jimallman self-assigned this Jun 1, 2022
@jimallman
Copy link
Member Author

Hm, it looks like I accidentally picked up an unrelated commit b9a97a5 This should probably be merged, but apologies for the clutter here.

jimallman added 7 commits June 8, 2022 13:21
Stashing this for reference, but it seems to "leak" newer packages and
cause trouble. Backing it out next...
This reverts commit 009061e.
We'll use a single-file hot fix to venv `activate_this.py` instead
This is just adding the file and README, not yet patching...
Copy link
Member

@mtholder mtholder 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

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.

3 participants