Skip to content

Add RCA for major freeze not allowed - #1252

Open
wayyoungboy wants to merge 1 commit into
oceanbase:masterfrom
wayyoungboy:fix-issue-713-major-freeze-not-allow
Open

wayyoungboy wants to merge 1 commit into
oceanbase:masterfrom
wayyoungboy:fix-issue-713-major-freeze-not-allow

Conversation

@wayyoungboy

Copy link
Copy Markdown
Member

Summary

  • Add major_freeze_not_allow RCA scene for ERROR 4217 / OB_MAJOR_FREEZE_NOT_ALLOW after manually executing ALTER SYSTEM MAJOR FREEZE tenant = ....
  • Query oceanbase.DBA_OB_TENANTS for tenant_role and explain the standby-tenant case called out in [Feature]: support analyzing error for OB_MAJOR_FREEZE_NOT_ALLOW when manually execute merge #713.
  • Optionally collect observer logs containing OB_MAJOR_FREEZE_NOT_ALLOW and add unit tests for error matching, tenant role handling, missing tenants, and SQL escaping.

Fixes #713

Verification

  • PYTHONPATH=. python -m pytest test/rca/test_major_freeze_not_allow.py -q
  • PYTHONPATH=. python -m pytest test/rca/test_major_freeze_not_allow.py test/common/test_scene.py -q
  • black --check -S -l 256 plugins/rca/major_freeze_not_allow.py test/rca/test_major_freeze_not_allow.py
  • python workflow_data/check_py_files.py .
  • git diff --check

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.

[Feature]: support analyzing error for OB_MAJOR_FREEZE_NOT_ALLOW when manually execute merge

1 participant