evolix
/
ansible-roles
22
2
Fork 2
Code Issues 61 Pull Requests 18 Releases 32 Wiki Activity

WIP: Added option to prepare mysql servers for replication #94

Closed
Ghost wants to merge 8 commits from mysql_replication into unstable
pull from: mysql_replication
merge into: evolix:unstable
Conversation 3 Commits 8 Files Changed 11 +153 -17
Ghost commented 2019-10-17 23:38:04 +02:00
First-time contributor
Copy Link

Doesnt start the replication, but prepares everything for it.

Doesnt start the replication, but prepares everything for it.
jlecour was assigned by Ghost 2019-10-17 23:38:08 +02:00
Ghost added the
enhancement
 label 2019-10-17 23:38:08 +02:00
lpoujol commented 2019-11-25 10:42:43 +01:00
Owner
Copy Link

Hey :)

A quick question : Do we always want to install xinetd ?

For the check done via xinetd, wouldn't it be better to rely on the mysql check as used by nagios ? There's a simplier check that we use. It's documented on the public wiki on the HAProxy page that doesn't even write temporary files in /tmp

Also :

  • You don't need to set the check_mysql_slave check command as it's aleady defined here

  • I don't get how dbadmin.sh is related to replication.

Hey :) A quick question : Do we always want to install xinetd ? For the check done via xinetd, wouldn't it be better to rely on the mysql check as used by nagios ? There's a simplier check that we use. It's documented on the [public wiki on the HAProxy page](https://wiki.evolix.org/HowtoHaproxy#exemple-pour-mysql) that doesn't even write temporary files in /tmp Also : - You don't need to set the check_mysql_slave check command as it's aleady defined [here](https://gitea.evolix.org/evolix/ansible-roles/src/branch/stable/nagios-nrpe/templates/evolix.cfg.j2#L27) - I don't get how dbadmin.sh is related to replication.
lpoujol requested changes 2019-11-25 10:49:53 +01:00
mysql/tasks/replication.yml
@ -0,0 +50,4 @@
dest: '/usr/share/scripts/'
mode: '0755'
- name: 'Copy dbadmin script'
lpoujol commented 2019-11-25 10:49:26 +01:00
Owner
Copy Link

This is not related to replication.
I'm not sure it's the right place to put it

This is not related to replication. I'm not sure it's the right place to put it
Ghost commented 2019-11-25 17:47:46 +01:00
Author
First-time contributor
Copy Link

You're right. Is this script even necessary ? I think I can remove it completely.

You're right. Is this script even necessary ? I think I can remove it completely.
mysql/templates/replication_check.cfg.j2
@ -0,0 +1,3 @@
# ansible managed
command[check_mysql_slave]=/usr/lib/nagios/plugins/check_mysql --check-slave -H localhost -f ~nagios/.my.cnf -w 1800 -c 3600
lpoujol commented 2019-11-25 10:47:11 +01:00
Owner
Copy Link

Dumplicate with our default nrpe config

Dumplicate with our default [nrpe config](https://gitea.evolix.org/evolix/ansible-roles/src/branch/stable/nagios-nrpe/templates/evolix.cfg.j2#L27)
Ghost commented 2019-11-25 17:40:45 +01:00
Author
First-time contributor
Copy Link

You're right, I'll remove it.

You're right, I'll remove it.
gcolpart commented 2019-11-25 13:23:13 +01:00
Owner
Copy Link

A quick question : Do we always want to install xinetd ?

xinetd is for advanced haproxy check, it's always better for critical replication, but not necessary in somes cases.

> A quick question : Do we always want to install xinetd ? xinetd is for advanced haproxy check, it's always better for critical replication, but not necessary in somes cases.
Ghost commented 2019-11-25 17:42:52 +01:00
Author
First-time contributor
Copy Link

This was based on a former client case. Does the simple nagios check make sure the database is synced ? Or does it only check if the database is up ? And do we want to keep the xinetd version as on option ?

This was based on a former client case. Does the simple nagios check make sure the database is synced ? Or does it only check if the database is up ? And do we want to keep the xinetd version as on option ?
Ghost closed this pull request 2020-06-01 18:06:02 +02:00
Some checks failed
continuous-integration/drone/push Build is passing
Details
continuous-integration/drone/pr Build is failing
Details

Pull request closed

This pull request cannot be reopened because the branch was deleted.
Sign in to join this conversation.
Reviewers
No reviewers
Labels
Clear labels   
bug

Something is not working

  
duplicate

This issue or pull request already exists

  
enhancement

New feature

  
help wanted

Need some help

  
invalid

Something is wrong

  
question

More information is needed

  
security

Concern a security issue

  
wontfix

This won't be fixed

No Label
bug
duplicate
enhancement
help wanted
invalid
question
security
wontfix
Milestone
Clear milestone
No items
No Milestone
Assignees
Clear assignees
No Assignees
jlecour
3 Participants
Notifications
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: evolix/ansible-roles#94
No description provided.
Delete Branch "mysql_replication"

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?