-
Notifications
You must be signed in to change notification settings - Fork 3
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
ocp13 replacement #518
base: main
Are you sure you want to change the base?
ocp13 replacement #518
Conversation
@RobHooper Is it that you observed that e.g. collect.data.open-contracting.org lists no projects? That's because we need to do something like
|
Yeah, I have a note about that in the docs https://ocdsdeploy.readthedocs.io/en/latest/develop/update/apache.html
I've now also added:
|
Yes, we also need to move It contains the Django media files for that project. It has a
For the data support server, we have:
|
In terms of migration process, is this the plan?
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good! Just one comment.
@@ -5,6 +5,8 @@ awscli: | |||
- name: python3-pip | |||
pip.installed: | |||
- name: awscli | |||
- extra_args: | |||
- --break-system-packages |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can you add a comment about why this option is necessary?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is required on systems with Python 3.11 and newer.
I have added a comment and a Jinja if statement so it only runs where needed.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Is this flag also needed on the pip.installed state in kingfisher/init.sls?
I added a documentation page that covers most of what I wrote above (except where I still have a question). Please add the steps to restore the We should add a subheading on this other page, and explain in general how to restore from individual database backups versus the existing docs about pgBackrest backups: https://ocdsdeploy.readthedocs.io/en/latest/maintain/databases.html#restore-from-backup We can then just link to that from the registry page. |
The Lets Encrypt staging environment won't generate SSL certificates without live DNS, so this doesn't help in this situation.
This was already done in a previous PR :)
Yes, I have ran |
Probably at an appropriate step in create_server.rst, and maybe as a note at https://ocdsdeploy.readthedocs.io/en/latest/maintain/hosting.html#rescale-a-server |
It helps to not get temporarily blocked by Let's Encrypt :) |
Resolves #494 and #504.