Skip to content
This repository has been archived by the owner on Aug 5, 2019. It is now read-only.

DDPB 2645 rollback procedure #550

Merged
merged 2 commits into from
May 28, 2019
Merged

DDPB 2645 rollback procedure #550

merged 2 commits into from
May 28, 2019

Conversation

alexeves
Copy link
Contributor

Purpose

An initial document which proposes a suitable rollback procedure for infrastructure, application, and database.

Addresses DDPB-2645

Approach

This is an initial outline which has yet to be fully replicated. The database migration proposal has been verified against the develop database. The rollback of code uses existing Jenkins tools and scripts.

Learning

Given that the only variable when applying any Doctrine down migrations is the filename of the migration file to revert to, it seems feasible that we could automate the migration command execution via a parameterised Jenkins task.

Checklist

  • I have performed a self-review of my own code
  • I have updated documentation (Confluence/GitHub wiki) where relevant
  • [N/A] I have added tests to prove my work, and they follow our best practices
  • [N/A] I have successfully built my branch to a feature environment
  • [N/A] New and existing unit tests pass locally with my changes (docker-compose run --rm api sh scripts/apiunittest.sh)
  • [N/A] There are no Composer security issues (docker-compose run api php app/console security:check)
  • [N/A] The product team have tested these changes

Copy link
Contributor

@hansnqyr hansnqyr left a comment

Choose a reason for hiding this comment

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

As an aside I'd like to see us tidy up our migrations to remove those unnecessary create schema statements

@gregtyler
Copy link
Contributor

As an aside I'd like to see us tidy up our migrations to remove those unnecessary create schema statements

You're in luck: #551

@alexeves alexeves merged commit 7651f6b into master May 28, 2019
@alexeves alexeves deleted the DDPB-2645-rollback-procedure branch May 28, 2019 17:36
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants