Skip to content
GitLab
Projects Groups Topics Snippets
  • /
  • Help
    • Help
    • Support
    • Community forum
    • Submit feedback
  • Sign in
  • E ensembl
  • Project information
    • Project information
    • Activity
    • Labels
    • Members
  • Repository
    • Repository
    • Files
    • Commits
    • Branches
    • Tags
    • Contributor statistics
    • Graph
    • Compare revisions
    • Locked files
  • Issues 0
    • Issues 0
    • List
    • Boards
    • Service Desk
    • Milestones
    • Iterations
    • Requirements
  • Jira
    • Jira
  • Merge requests 0
    • Merge requests 0
  • CI/CD
    • CI/CD
    • Pipelines
    • Jobs
    • Schedules
    • Test cases
  • Deployments
    • Deployments
    • Environments
    • Releases
  • Packages and registries
    • Packages and registries
    • Package Registry
    • Container Registry
    • Terraform modules
  • Monitor
    • Monitor
    • Incidents
  • Analytics
    • Analytics
    • Value stream
    • CI/CD
    • Code review
    • Insights
    • Issue
    • Repository
  • Wiki
    • Wiki
  • Snippets
    • Snippets
  • Activity
  • Graph
  • Create a new issue
  • Jobs
  • Commits
  • Issue Boards
Collapse sidebar
  • ensembl-gh-mirror
  • ensembl
  • Merge requests
  • !488

[ENSWEB-5706] Allow multiple "ancestral" databases in the Registry

  • Review changes

  • Download
  • Patches
  • Plain diff
Merged Marek Szuba requested to merge bugfix/multi_ancestral_registry into release/100 Jun 05, 2020
  • Overview 2
  • Commits 1
  • Pipelines 0
  • Changes 1

Created by: muffato

Use case

See ENSWEB-5706 and related tickets for more information. The REST server sees all MySQL servers (Vertebrates and non-Vertebrates), which contain in total two ancestral databases, but the current Registry model only allows one. The Compara API then fails because it is not able to access the other one

Description

Ancestral databases are now registered with their division name, preventing a clash between the Plants and Vertebrates databases.

Note that for backwards compatibility and for consistency with the rest of the code, the name "Vertebrates" is currently omitted.

⚠ Do not merge until all these related pull-requests have been approved !: Ensembl/ensembl-compara#197, Ensembl/ensembl-production#430

Equivalent pull-requests: #491 for release/101 and #492 for master

Benefits

Now able to load all ancestral databases in the same Registry instance

Possible Drawbacks

Applications that load the NV server in the Registry won't be able to access the Plants ancestral database with "Ancestral sequences", but since this database has been introduced in e100, I think there is still time to change its Registry name and state that this is the nomenclature going forward.

Testing

Have you added/modified unit tests to test the changes?

No. The databases created by registry.t are prefixed by the user name and test_db. They wouldn't pass the regex used by load_registry_from_db

If so, do the tests pass/fail?

Have you run the entire test suite and no regression was detected?

Failures are in:

  • compara.t (because .travis.yml clones the master branch of Compara instead of the release/100 branch
  • housekeeping_apache2.t, because it is now 2020 and the files haven't had their copyright statement updated. I have only updated the one of Registry.pm
Assignee
Assign to
Reviewers
Request review from
Time tracking
Source branch: bugfix/multi_ancestral_registry