Skip to content

Conversation

@PredictiveManish
Copy link
Contributor

Description

  • Fixed two locations where OWNER TO "augur" was hardcoded:
  • 0_legacy.py: Added dynamic replacement of OWNER TO "augur" with OWNER TO CURRENT_USER before executing legacy SQL migration files
  • 3_oauth_and_user_groups.py: Changed hardcoded OWNER TO "augur" to OWNER TO CURRENT_USER in the user_groups table creation SQL

Technical Details

This PR fixes #3417

Notes for Reviewers

Signed commits

  • Yes, I signed my commits.

Signed-off-by: PredictiveManish <[email protected]>
@MoralCode
Copy link
Contributor

unfortunately, I don't think retroactively changing the migrations is the answer here. My ultimate hope is that, once #3435 is good to merge, we can switch the database initialization over to a different method that doesnt require replaying migrations from very old versions of augur

@MoralCode MoralCode marked this pull request as draft December 30, 2025 19:42
@MoralCode
Copy link
Contributor

Closing this PR since the only changes it makes are to old migrations that should not be changed (because they are how older instances get themselves up to date with the latest augur versions.

@MoralCode MoralCode closed this Jan 10, 2026
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.

Database initialization (via alembic) assumes a user called "augur" is present

2 participants