Navigation Menu

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

Add informational diagnostic for archive invalidations table. #17110

Merged
merged 13 commits into from Mar 16, 2021

Conversation

diosmosis
Copy link
Member

Description:

Adds a new informational diagnostic to get information about rows in the invalidations table. @tsteur do you see value in this? If so I will add a test.

Review

  • Functional review done
  • Usability review done (is anything maybe unclear or think about anything that would cause people to reach out to support)
  • Security review done see checklist
  • Code review done
  • Tests were added if useful/possible
  • Reviewed for breaking changes
  • Developer changelog updated if needed
  • Documentation added if needed
  • Existing documentation updated if needed

@diosmosis
Copy link
Member Author

@tsteur does this look useful to you?

@tsteur
Copy link
Member

tsteur commented Jan 31, 2021

Yes that sounds good. Anything that will help us troubleshoot and understand things easier/quick is good 👍 Especially around archiving which is usually the most difficult one

@tsteur tsteur added the Needs Review PRs that need a code review label Feb 9, 2021
@tsteur tsteur added this to the 4.2.0 milestone Feb 9, 2021
@mattab mattab modified the milestones: 4.2.0, 4.3.0 Feb 22, 2021
@sgiehl
Copy link
Member

sgiehl commented Feb 24, 2021

@diosmosis UI test needs to be updated before merging

@sgiehl
Copy link
Member

sgiehl commented Mar 15, 2021

@diosmosis the UI file you've added here is not on LFS

@diosmosis diosmosis merged commit 4493012 into 4.x-dev Mar 16, 2021
@diosmosis diosmosis deleted the archive-invalidations-diagnostic branch March 16, 2021 03:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Needs Review PRs that need a code review
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants