Skip to content

Standardise config location - #994

Draft
costasmantzoukas wants to merge 1 commit into
mainfrom
costas/standardise-config-location
Draft

costasmantzoukas wants to merge 1 commit into
mainfrom
costas/standardise-config-location

Conversation

@costasmantzoukas

@costasmantzoukas costasmantzoukas commented Oct 2, 2026 •

Copy link
Copy Markdown
Contributor

What does this PR change?

As part of the ECS migration we'll be configuring apps to read S3 config files from a standardized /etc/gu/s3-sync directory.

This PR modifies the EC2 build to read from the new directory, utilizing symlinks to place the config on the expected location on disk.

@costasmantzoukas
costasmantzoukas requested review from a team as code owners October 2, 2026 13:26
@costasmantzoukas costasmantzoukas added the feature Departmental tracking: work on a new feature label Oct 2, 2026
@github-actions

github-actions Bot commented Oct 2, 2026 •

Copy link
Copy Markdown
Contributor

@AshCorr
AshCorr marked this pull request as draft October 2, 2026 14:11
@AshCorr
AshCorr added this pull request to stack #996 October 2, 2026 16:11
@AshCorr
AshCorr force-pushed the costas/standardise-config-location branch 2 times, most recently from fc48248 to 0c15729 Compare October 2, 2026 16:21
@aug24

aug24 commented Oct 5, 2026

Copy link
Copy Markdown
Contributor

Need to remove the google file location from the config if we do this.

@aug24 aug24 closed this Oct 5, 2026
@aug24 aug24 reopened this Oct 5, 2026
@aug24

aug24 commented Oct 5, 2026

Copy link
Copy Markdown
Contributor

Whoops, clicked the wrong button. Meant to just comment, not close with comment! Sorry.

Co-authored-by: Ashleigh Carr <ashcorr20@gmail.com>
Co-authored-by: Justin Rowles <justin.rowles.contractor@guardian.co.uk>
@costasmantzoukas
costasmantzoukas force-pushed the costas/standardise-config-location branch from 0c15729 to 05c57ba Compare October 8, 2026 17:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

feature Departmental tracking: work on a new feature

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants