[MXS-1262] Mantenance bit(s) should persist after maxscale restart Created: 2017-05-10 Updated: 2017-08-16 Resolved: 2017-08-16 |
|
| Status: | Closed |
| Project: | MariaDB MaxScale |
| Component/s: | maxadmin |
| Affects Version/s: | 2.0.5, 2.1.2 |
| Fix Version/s: | 2.2.0 |
| Type: | Bug | Priority: | Major |
| Reporter: | Andrii Nikitin (Inactive) | Assignee: | markus makela |
| Resolution: | Fixed | Votes: | 0 |
| Labels: | None | ||
| Sprint: | 2017-39 |
| Description |
|
On practice it may be very unpleasant situation when maxscale service is restarted and server 'maintenance' bits are not preserved. Thus server 'maintenance' flag must survive service restart.
|
| Comments |
| Comment by Dipti Joshi (Inactive) [ 2017-05-10 ] | |||||||||||||||||||||||||||||||||||||
|
anikitinPlease test this with 2.1.2 - With dynamic configuration features of 2.1 this should have been fixed there. | |||||||||||||||||||||||||||||||||||||
| Comment by Andrii Nikitin (Inactive) [ 2017-05-10 ] | |||||||||||||||||||||||||||||||||||||
|
Thank you Dipti - I've tried 2.1.2 and it shows identical behavior.
| |||||||||||||||||||||||||||||||||||||
| Comment by markus makela [ 2017-05-11 ] | |||||||||||||||||||||||||||||||||||||
|
This has been partially fixed for 2.2 where the MySQL monitor module now stores a persistent copy of the server states on disk. This prevents the loss of information that happens when MaxScale is restarted. We'll probably extend this functionality to a generic monitor feature for 2.2. | |||||||||||||||||||||||||||||||||||||
| Comment by Andrii Nikitin (Inactive) [ 2017-05-11 ] | |||||||||||||||||||||||||||||||||||||
|
This feature is very rarely needed, but considering amount of damage it may lead to: I'd give it top priority in every future 2.0 / 2.1 release where possible . Also in my understanding 'Maintenance' status is conceptually different from other states like 'Master' and 'Running', etc. thus should be treated differently. I.e. we cannot be fully sure that Server is still 'Running': even if we just checked that it was OK - we should be ready that some problem may occur any moment. | |||||||||||||||||||||||||||||||||||||
| Comment by markus makela [ 2017-05-11 ] | |||||||||||||||||||||||||||||||||||||
|
With 2.1, this can be emulated by removing the server from the service. | |||||||||||||||||||||||||||||||||||||
| Comment by markus makela [ 2017-08-10 ] | |||||||||||||||||||||||||||||||||||||
|
Moved the code that persists the server state into the core and took it into use in all monitors. |