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

Add missing configuration options and fix ansible lint errors #22

Draft
wants to merge 44 commits into
base: trunk
Choose a base branch
from

Conversation

Lun4m
Copy link
Collaborator

@Lun4m Lun4m commented Jul 3, 2024

Fixes:

  • Add deploy role → moved to Add ansible files for deployment #33
  • Add security group for Obsinn during configuration
  • Add table partitions during configuration
  • Fix postgres data directory (closes Make Postgres use the correct partition for data storage #36)
  • Make style conform to ansible-lint
  • Figure out how to deal with changed_when and ignore_errors
  • Check if lard service should be stopped/restarted during switchover
  • Add automatic teardown of instances and volumes
  • Upgrade to postgres 17 and Ubuntu Noble

TODO (when Louise comes back):

  • Add a way to handle ssh on teardown? (I don't remember what this means 🙈)
  • Figure out why lard_ingestion can't connect to postgres on the standby node
    "failed to lookup address information: Temporary failure in name resolution"

Actually the problem is that we can't connect to stinfosys, because the standby is not routed through the
ipalias interface, therefore it drops most of the external connections.
Probably we want to have two floating IPs and switch them if the primary goes down.

@Lun4m Lun4m linked an issue Jul 8, 2024 that may be closed by this pull request
@Lun4m Lun4m force-pushed the deploy_lard branch 3 times, most recently from ef5290b to e906d07 Compare July 12, 2024 14:30
@Lun4m Lun4m changed the title Add Ansible files for deployment Add missing configuration options and fix ansible lint errors Oct 10, 2024
@Lun4m Lun4m removed a link to an issue Oct 10, 2024
@Lun4m Lun4m force-pushed the deploy_lard branch 2 times, most recently from f97cc1b to f9e515b Compare January 23, 2025 13:02
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.

Make Postgres use the correct partition for data storage
1 participant