Skip to content

fix docker-compose.yml singularity_init container env var#526

Merged
Sankara-Jefferson merged 1 commit intodata-preservation-programs:mainfrom
lanzafame:fix/docker-compose
Jul 9, 2025
Merged

fix docker-compose.yml singularity_init container env var#526
Sankara-Jefferson merged 1 commit intodata-preservation-programs:mainfrom
lanzafame:fix/docker-compose

Conversation

@lanzafame
Copy link
Collaborator

No description provided.

@Sankara-Jefferson Sankara-Jefferson self-requested a review July 9, 2025 03:28
Copy link
Contributor

@Sankara-Jefferson Sankara-Jefferson left a comment

Choose a reason for hiding this comment

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

Verified locally.
The singularity_init container now picks up DATABASE_CONNECTION_STRING,
runs migrations, and exits successfully.
Ready to merge.

@Sankara-Jefferson Sankara-Jefferson merged commit 22e67ce into data-preservation-programs:main Jul 9, 2025
9 checks passed
ianconsolata pushed a commit that referenced this pull request Jul 9, 2025
Verified locally.
The singularity_init container now picks up DATABASE_CONNECTION_STRING,
runs migrations, and exits successfully.
@ianconsolata
Copy link
Collaborator

Manually pushed this change to develop since it accidentally got merged to main

@lanzafame
Copy link
Collaborator Author

Sorry didn't realise PRs were against develop.

@lanzafame lanzafame deleted the fix/docker-compose branch July 10, 2025 00:07
parkan pushed a commit that referenced this pull request Oct 22, 2025
Verified locally.
The singularity_init container now picks up DATABASE_CONNECTION_STRING,
runs migrations, and exits successfully.
parkan pushed a commit that referenced this pull request Nov 6, 2025
Verified locally.
The singularity_init container now picks up DATABASE_CONNECTION_STRING,
runs migrations, and exits successfully.
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