feat(admin): Add resolver cache flush command #1349
No reviewers
Labels
No labels
Blocked
Bug
Cherry-picking
Database
Dependencies
Dependencies/Renovate
Difficulty
Easy
Difficulty
Hard
Difficulty
Medium
Documentation
Enhancement
Good first issue
Help wanted
Inherited
Matrix/Administration
Matrix/Appservices
Matrix/Auth
Matrix/Client
Matrix/Core
Matrix/Federation
Matrix/Hydra
Matrix/MSC
Matrix/Media
Meta
Meta/CI
Meta/Packaging
Priority
Blocking
Priority
High
Priority
Low
Security
Status
Confirmed
Status
Duplicate
Status
Invalid
Status
Needs Investigation
Support
To-Merge
Wont fix
old/ci/cd
old/rust
No project
No assignees
5 participants
Notifications
Due date
No due date set.
Dependencies
No dependencies set.
Reference
continuwuation/continuwuity!1349
Loading…
Add table
Add a link
Reference in a new issue
No description provided.
Delete branch "Omar007/continuwuity:feature/flush-cache"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
This command allows an admin to flush a specific server from the resolver caches or flush the whole cache.
Pull request checklist:
mainbranch, and the branch is named something other thanmain.myself, if applicable. This includes ensuring code compiles.
e9a92f128937a66e5f80Looks good to me
@ -114,1 +114,4 @@### `!admin query resolver flush-cache`Flush a given server from the resolver caches or flush them completelycan this include an example with all, or a domain, specified
Done.
37a66e5f80e9c93c12a0e9c93c12a0ae16a45515@ -7,6 +7,7 @@ use crate::{admin_command, admin_command_dispatch};#[admin_command_dispatch]#[derive(Debug, Subcommand)]#[allow(clippy::enum_variant_names)]Ran tests and pre-commit checks locally and those were all good but saw the clippy workflow fail. I assume we don't want to rename all commands to avoid a suffix or prefix so I've added
#[allow(clippy::enum_variant_names)]here.Is this OK?
Yeah that's fine. Lots of the admin commands could do with renaming tbh but that would break muscle memory, and this case seems fine to me