Skip to content

Correct mariadb restart handlers#676

Merged
priteau merged 1 commit intostackhpc/2024.1from
mariadb-clustercheck-handler
Feb 3, 2025
Merged

Correct mariadb restart handlers#676
priteau merged 1 commit intostackhpc/2024.1from
mariadb-clustercheck-handler

Conversation

@priteau
Copy link
Member

@priteau priteau commented Jan 31, 2025

This was removed in 65325d9 but it is still called by the config playbook.

Change-Id: Idfb5b814e16056d439cf1e1318f86bbcea82d981

@priteau priteau self-assigned this Jan 31, 2025
@priteau priteau requested a review from a team as a code owner January 31, 2025 09:59
@priteau priteau closed this Jan 31, 2025
@priteau priteau reopened this Jan 31, 2025
This behaviour was changed 65325d9 but
does not work for mariadb and mariadb-clustercheck. A fix was included
in master as part of 23413d4, let's do
the same in our fork.

Change-Id: Idfb5b814e16056d439cf1e1318f86bbcea82d981
@MoteHue MoteHue force-pushed the mariadb-clustercheck-handler branch from fb644fa to 30fa3fa Compare January 31, 2025 15:43
@MoteHue MoteHue changed the title Restore mariadb-clustercheck handler Correct mariadb restart handlers Jan 31, 2025
@MoteHue
Copy link

MoteHue commented Jan 31, 2025

Test failed with:

TASK [service-check-containers : mariadb | Notify handlers to restart containers] ***
Friday 31 January 2025 10:37:01 +0000 (0:00:01.210) 0:02:02.145 ********
ERROR! The requested handler 'Restart mariadb container' was not found in either the main handlers list nor in the listening handlers list

Looks like this was fixed as part of a different patch, I've extracted that out here

@priteau priteau merged commit 44f58ae into stackhpc/2024.1 Feb 3, 2025
3 checks passed
@priteau priteau deleted the mariadb-clustercheck-handler branch February 3, 2025 09:08
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.

3 participants