[solaechea]

Written with Claude Opus 5.5

Description

Moves the "Restarting PostgreSQL now so it can use the new locales" flush_handlers from the end of setup-Debian.yml to just after initialize.yml in main.yml.

Before this change, the restart that #29 added ran before initialize.yml. On an image where PostgreSQL is installed but its data directory has been removed, there was no cluster yet at that point, so the restart failed with Unable to start service postgresql@15-main. The cluster only gets rebuilt later, in the "Rebuilding a scrubbed PostgreSQL cluster (Debian)" task. Now the restart runs after the rebuild. On a stock install it is still a restart after the locales are generated and before any database is created, which is what #29 needed.

Closes #40

Type of Change

• Bug fix (non-breaking change which fixes an issue)
• New role or feature (non-breaking change which adds functionality)
• Breaking change (variable renames, removed roles, changed behavior)
• Documentation update
• CI/workflow change

Role Convention Checklist

• ansible-lint passes with zero failures and zero warnings (production profile, strict): not run locally, left to CI
• Tasks use FQCN modules, gerund names, quoted modes, changed_when on commands
• meta/argument_specs.yml and the role README updated for any variable or behavior change: no variable changes
• No secrets in tracked files
• Checked startcloud.startcloud_roles for overlap
• Commit messages follow Conventional Commits

Testing performed

The role's molecule/default converge and verify playbooks, run in privileged Debian 12 systemd containers (geerlingguy/docker-debian12-ansible, ansible-core 2.19). en_US.UTF-8 was not generated beforehand in any run.

Starting state	main (v0.4.3)	This branch
PostgreSQL 15 installed, stopped, data directory removed (the #40 case)	converge fails: Unable to start service postgresql@15-main	converge ok=37 failed=0, verify ok=24 failed=0, startclouddb collation en_US.UTF-8
Only locales installed (the #29 case)	converge and verify pass	converge ok=35 failed=0, verify ok=24 failed=0, startclouddb collation en_US.UTF-8

Additional Context

This was found on a SwitchBoard image whose PostgreSQL data directory is removed when the image is built.

[solaechea]

@MarkProminic I am merging this as it seems to be just a fix associated to our own update on #29.
The Molecule error seems to be external and a ticket has been opened for that:
#42