Skip to content

KEH-2058 - ECS SCP Fixes#63

Merged
TotalDwarf03 merged 3 commits intomainfrom
KEH-2058-ecs-scp-fixes
Mar 18, 2026
Merged

KEH-2058 - ECS SCP Fixes#63
TotalDwarf03 merged 3 commits intomainfrom
KEH-2058-ecs-scp-fixes

Conversation

@TotalDwarf03
Copy link
Copy Markdown
Contributor

What type of PR is this? (check all applicable)

  • Refactor
  • Feature
  • Bug Fix
  • Optimization
  • Documentation Update

What

This PR amends the policy dashboard to apply with new AWS Organisational settings.

This PR does the following:

  • Change dashboard container to run as non-root
  • Read only file system for dashboard container
  • Remove ECS public IP since NAT gateway and routing is in place.

Testing

Have any new tests been added as part of this issue? If not, try to explain why test coverage is not needed here.

  • Yes
  • No
    Please write a brief description of why test coverage is not necessary here.
  • Not as part of this ticket. (Could be done at a later point)

Documentation

Has any new documentation been written as part of this issue? We should try to keep documentation up to date
as new code is added, rather than leaving it for the future.

  • Yes
  • No
    Please write a brief description of why documentation is not necessary here.
  • Not as part of this ticket. (Could be done at a later point)

Related issues

KEH-2058 (Jira)

How to review

N/A

Without these changes I couldn't deploy so the fact it deploys means it works 🎉

- Need to install deps after switching user
- Complies with ONS standards
@TotalDwarf03 TotalDwarf03 self-assigned this Mar 18, 2026
@TotalDwarf03 TotalDwarf03 added the bug Something isn't working label Mar 18, 2026
@TotalDwarf03 TotalDwarf03 changed the title Keh 2058 ecs scp fixes KEH-2058 - ECS SCP Fixes Mar 18, 2026
@TotalDwarf03 TotalDwarf03 merged commit eaa9601 into main Mar 18, 2026
4 checks passed
@TotalDwarf03 TotalDwarf03 deleted the KEH-2058-ecs-scp-fixes branch March 18, 2026 14:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants