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

Migration for external Audit DB #3824

Closed
plettich opened this issue Nov 29, 2023 · 3 comments · Fixed by #3962
Closed

Migration for external Audit DB #3824

plettich opened this issue Nov 29, 2023 · 3 comments · Fixed by #3962
Assignees
Milestone

Comments

@plettich
Copy link
Member

In case the audit log is send to a different database using PI_AUDIT_SQL_URI, possible migrations of the audit table will not be applied.
We should check for the existence of that configuration in the pi.cfg and either apply the migrations to the external table as well (need to check how to use a different db in alembic) or give a note to the user to apply the changes themselves.

@cornelinux
Copy link
Member

@plettich Which CC is it?

@plettich
Copy link
Member Author

@cornelinux

@plettich Which CC is it?

You can search for the issue number in the ticketing system. I try to avoid posting internal data here.

@cornelinux
Copy link
Member

We add docs:

  • in the description within pi.cfg
  • in the READ-BEFORE-UPDATE
    • a generic comment
    • comments in the upgrade steps, where the audit log schema was changed

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants