Skip to content

Revert: Grant migrator privileges even when the user already exists#8313

Open
CarolineDenis wants to merge 1 commit into
v7_12_0_7_basefrom
issue-8305
Open

Revert: Grant migrator privileges even when the user already exists#8313
CarolineDenis wants to merge 1 commit into
v7_12_0_7_basefrom
issue-8305

Conversation

@CarolineDenis

@CarolineDenis CarolineDenis commented Jul 15, 2026

Copy link
Copy Markdown
Contributor

Fixes #8305

Checklist

  • Self-review the PR after opening it to make sure the changes look good and
    self-explanatory (or properly documented)
  • Add relevant issue to release milestone
  • Add pr to documentation list
  • Add automated tests
  • Add a reverse migration if a migration is present in the PR
  • Add migration function to
    def fix_schema_config(stdout: WriteToStdOut | None = None):

Testing instructions

  1. Check that you have a specify_migrator* and specify_user* already created in MySQL.
  • *The users may not be of those names. They need to match the MIGRATOR_NAME and APP_USER_NAME in the .env file.
  1. Edit the .env file so that the DATABASE_NAME is one that does not exist yet.
  2. Build/rebuild your Docker containers.
  • Check the specify7 container logs and make sure there are no migration errors.
  1. Open the application in your browser.
  • Make sure that it's opening the Specify 7 initial database setup screen.
  1. Stop the containers.
  2. Edit the .env file so that the DATABASE_NAME is one that does exist and was not previously opened in v7.12.
  • You may run into migration errors while testing an old database.
  1. Build/rebuild your Docker containers.
  • Check the specify7 container logs and make sure there are no migration errors.
  1. Open the application in your browser.
  • Make sure that it's opening the Specify 7 login page.

@CarolineDenis CarolineDenis added this to the 7.12.0.7 milestone Jul 15, 2026
@github-project-automation github-project-automation Bot moved this to 📋Back Log in General Tester Board Jul 15, 2026
@CarolineDenis
CarolineDenis requested a review from combs-a July 15, 2026 10:23
@CarolineDenis
CarolineDenis changed the base branch from main to v7_12_0_7_base July 15, 2026 10:24
@coderabbitai

coderabbitai Bot commented Jul 15, 2026

Copy link
Copy Markdown
Contributor

Important

Review skipped

Auto reviews are disabled on base/target branches other than the default branch.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro Plus

Run ID: 8127a034-c951-4639-9bad-e592744da3c3

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch issue-8305

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@CarolineDenis CarolineDenis changed the title Issue 8305 Revert: Grant migrator privileges even when the user already exists Jul 15, 2026

@combs-a combs-a left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

I wrote these testing instructions, but they may need to be added onto if anyone spots any additional coverage needed!


New database:
  • Check the specify7 container logs and make sure there are no migration errors.
  • Make sure that it's opening the Specify 7 initial database setup screen.

Previously existing database (used pre v7.12):

  • Check the specify7 container logs and make sure there are no migration errors.
  • Make sure that it's opening the Specify 7 login page.

New database worked as expected.
Image

Was able to create a new database with no issues, but ran into an issue with the Specify app user when attempting to open an old database. Used the same database as I did in the originally reported issue.

Image

@github-project-automation github-project-automation Bot moved this from 📋Back Log to Dev Attention Needed in General Tester Board Jul 16, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Dev Attention Needed

Development

Successfully merging this pull request may close these issues.

Cannot setup a new Specify 7 database locally / cannot access Specify 7 databases not opened in v7.12 locally

2 participants