Skip to content

Conversation

@edhall-nhs
Copy link
Contributor

@edhall-nhs edhall-nhs commented Jan 28, 2026

Summary

  • Routine Change
  • 🤖 Operational or Infrastructure Change
  • ⚠️ Potential issues that might be caused by this change

Main objectives of this ticket are:

  • Allow rollback capability for lambda deployments via updating image tagging logic
  • Only trigger builds if relevant files are changed (e.g. ignore README.md updates
  • Potential change to ECR setup
  • General terraform tidy up

Reviews Required

  • Dev
  • Test
  • Tech Author
  • Product Owner

Review Checklist

ℹ️ This section is to be filled in by the reviewer.

  • I have reviewed the changes in this PR and they fill all of the acceptance criteria of the ticket.
  • If there were infrastructure, operational, or build changes, I have made sure there is sufficient evidence that the changes will work.
  • If there were changes that are outside of the regular release processes e.g. account infrastructure to setup, manual setup for external API integrations, secrets to set, then I have checked that the developer has flagged this to the Tech Lead as release steps.
  • I have checked that no Personal Identifiable Data (PID) is logged as part of the changes.

@github-actions
Copy link
Contributor

This branch is working on a ticket in the NHS England VED JIRA Project. Here's a handy link to the ticket:

VED-1040

@sonarqubecloud
Copy link

@edhall-nhs
Copy link
Contributor Author

Closing this PR as the desired functionality is likely not possible with the current deployment strategy of building and pushing Docker images via terraform. Please see the accompanying ticket for more details.

I've created a separate PR for some general terraform tidy up which supersedes this one

@edhall-nhs edhall-nhs closed this Jan 29, 2026
@edhall-nhs edhall-nhs deleted the feature/VED-1040-docker-build-improvements branch January 29, 2026 16:54
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.

1 participant