Skip to content

Conversation

@thatguyinabeanie
Copy link
Collaborator

@thatguyinabeanie thatguyinabeanie commented Sep 5, 2025

Description

Context, Consequences, & Considerations

Required: Please step through the following list, pausing at each item to consider your change in relation to the item's context.
Check the box to mark that it applies, and enter your relevant notes under the item.

  • Security: This has security implications. This includes (but not limited to) adding users, modifying user/app permissions, network rules/policies, changing a system interconnection, or changing an authorization strategy.
    • This PR does not require security review. These changes are part of a project plan that has already undergone security review. The link is provided below.
    • This PR requires security review. Add the security label to this PR then request a review from the Security Code Reviewers Team.
  • Execution: This change requires commands to be run outside of the normal merge.
  • Impact: This change may cause service interruptions.
  • Testing: How did you test this change (unit tests, acceptance tests, etc.)? Did you do any manual testing?

manual

  • Testing: How will you confirm this change once it's merged?
  • Documentation: Documentation to reflect this change has been added to Confluence or Zendesk.
  • All items of the checklist have been considered and this PR description is complete.

services:
docker-linuxserver-ubuntu-fips:
image: ${ECR_ACCOUNT_ID:-0123456789012}.dkr.ecr-fips.us-east-1.amazonaws.com/docker-linuxserver-ubuntu-fips:latest
image: ${ECR_ACCOUNT_ID:-0123456789012}.dkr.ecr-fips.us-east-1.amazonaws.com/docker-linuxserver-ubuntu-fips:jammy-latest
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

updated to use the actual production tag.

Copy link

@leanne73 leanne73 left a comment

Choose a reason for hiding this comment

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

Nice that we can re-enable this now! LGTM

@thatguyinabeanie thatguyinabeanie merged commit 14902f7 into jammy Sep 5, 2025
1 check passed
@civis-robot
Copy link

Pull Request Merged!!! This build is running now.

@thatguyinabeanie thatguyinabeanie deleted the civis-11444-lsiown-config branch September 5, 2025 15:05
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