-
Notifications
You must be signed in to change notification settings - Fork 23.7k
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
mysql_info: add support for global status #63189
Conversation
Thank you! I’ll try to look at this tomorrow morning |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@sayap , good job! thank you for the PR!
I tried it manually - works as expected.
IMO global_status
looks better that just status
, more obvious and it corresponds to the SQL.
Would you mind to change it, please ?
@@ -25,8 +25,8 @@ | |||
filter: | |||
description: | |||
- Limit the collected information by comma separated string or YAML list. | |||
- Allowable values are C(version), C(databases), C(settings), C(users), | |||
C(slave_status), C(slave_hosts), C(master_status), C(engines). | |||
- Allowable values are C(version), C(databases), C(settings), C(status), |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
- Allowable values are C(version), C(databases), C(settings), C(status), | |
- Allowable values are C(version), C(databases), C(settings), C(global_status), |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Agree, I have renamed it to global_status
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
good job! thank you
Signed-off-by: Yap Sok Ann <sokann@gmail.com>
3804b62
to
ae9948d
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
shipit
shipit |
thanks all! |
SUMMARY
This allows the mysql_info module to also include the output of
SHOW GLOBAL STATUS
.ISSUE TYPE
COMPONENT NAME
mysql_info
ADDITIONAL INFORMATION
For my specific use case, we are checking whether the global status
wsrep_local_state_comment
is equal toSynced
, after adding a node to galera replication.