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

tracking the highest observed timeline in the cluster #59

Closed
wants to merge 2 commits into from

Conversation

YanChii
Copy link
Contributor

@YanChii YanChii commented Jan 16, 2017

Hi,
as I promised in #57, I've implemented the new data security check.

Now it is probably time to discuss one issue. For tracking the highest reached timeline, I've created a unique variable name that includes a postgresql DB system identifier in its name (so the name of the attribute is e.g. "postgres-6354064916801227111-highest-timeline").
I did this because:
1), the minor issue: if you delete all datadirs and re-init the whole cluster, you start from timeline 1 again and we don't use the old parameter.
2), the major issue: if you have two databases with two configured HA resources, they will interfere as they use the same parameter names.

An the second issue is what I'm concerned for also with the other parameters you already have.
My question:
If you have two pgsqlms resources within one pacemaker cluster, can attributes (lsn_location, nodes, cancel_switchover) interfere between them? For example, if master node fails and both resources start master voting simultaneously, will they write to the same attributes and therefore master voting can work with wrong data?

Thank you for your consideration & reply.

Jan

@ioguix
Copy link
Member

ioguix commented Jan 19, 2017

Answered in #57

@ioguix
Copy link
Member

ioguix commented Feb 16, 2017

I pushed another patch in master in regard with the TL check during standby election.

See b75971d

I'll close this pull request for now. Discussion following on #57

@ioguix ioguix closed this Feb 16, 2017
@YanChii YanChii mentioned this pull request Sep 16, 2020
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.

2 participants