Skip to content

Conversation

@keranbyge
Copy link

While investigating issue #2359, I ran into the same
"permission denied for table ticket" error when running
update_metrics locally.

This PR documents the required local setup steps
(database creation, Trac schema import, and permissions)
to ensure update_metrics runs successfully.

Fixes #2359

@ulgens ulgens self-requested a review December 31, 2025 13:32
Copy link
Member

@ulgens ulgens left a comment

Choose a reason for hiding this comment

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

High level issues:

  • Please install pre-commit hooks locally, using prek install. Then you can run the hooks againist all files using prek run --all-files and you'll see a failure.
  • Please only include the related commit.

Coming from

This seems like a documentation / setup gap rather than a code bug. I’d be happy to open a PR to clarify the local setup steps if that would be helpful.

#2359 (comment)

I was expecting this PR to be an update in Setting up database access section. I'm not sure a new section is needed.

@keranbyge keranbyge closed this Dec 31, 2025
@keranbyge keranbyge reopened this Dec 31, 2025
@keranbyge
Copy link
Author

Thanks for the review!

I’ve now:

Installed and verified the pre-commit hooks locally (CI is passing).

Cleaned up the PR so it only includes the relevant documentation change.

Moved the update_metrics local setup details into the existing “Setting up database access” section instead of introducing a new section.

Please let me know if this looks good or if you’d prefer the guidance structured differently.

@ulgens
Copy link
Member

ulgens commented Jan 1, 2026

I'm missing why you deleted the readme. Also, please try to use rebasing to update your branch, not "merge 'main' into <your_branch>" approach. Rebase yields a much cleaner diff and git history.

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.

update_metrics fails with "permission denied for table ticket" even after importing trac.sql

2 participants