Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix: Add missing indexes in the vfolders table #1937

Merged
merged 1 commit into from
Feb 28, 2024

Conversation

achimnol
Copy link
Member

@achimnol achimnol commented Feb 28, 2024

Checklist: (if applicable)

  • Milestone metadata specifying the target backport version
  • Mention to the original issue

- It is a follow-up to #1892 and #1936.
- Adds `index=True` to the `kernels.role` column to avoid mismatch of DB
  and migrations.
@achimnol achimnol added this to the 24.03 milestone Feb 28, 2024
@achimnol achimnol added comp:manager Related to Manager component skip:changelog Make the action workflow to skip towncrier check type:enhance Enhance component, behavior, internals without user-facing features labels Feb 28, 2024
@achimnol achimnol self-assigned this Feb 28, 2024
@github-actions github-actions bot added require:db-migration Automatically set when alembic migrations are added or updated size:M 30~100 LoC labels Feb 28, 2024
@achimnol achimnol enabled auto-merge February 28, 2024 08:32
@achimnol achimnol added this pull request to the merge queue Feb 28, 2024
Merged via the queue into main with commit c16a71b Feb 28, 2024
43 of 50 checks passed
@achimnol achimnol deleted the fix/add-missing-vfolders-table-indexes branch February 28, 2024 08:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
comp:manager Related to Manager component require:db-migration Automatically set when alembic migrations are added or updated size:M 30~100 LoC skip:changelog Make the action workflow to skip towncrier check type:enhance Enhance component, behavior, internals without user-facing features
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant