You cannot select more than 25 topics Topics must start with a letter or number, can include dashes ('-') and can be up to 35 characters long.
 
 
 
 
 
Go to file
Maykel Moya 5da6fcae4d Add support for string values
The SET GLOBAL statement requires properly quoting of values. For example, the
following correct queries will fail if quotes are toggled:

mysql> SET GLOBAL innodb_lru_scan_depth = 2000;
mysql> SET GLOBAL master_info_repository = "TABLE";

`mysql_variable` module doesn't quote the value argument, therefore
string values will fail.

  # this task will pass, 2000 is passed without quotes
  - name: set a numeric value
    mysql_variable: variable=innodb_lru_scan_depth value=2000

  # this task will fail, TABLE is passed without quotes
  - name: set a string value
    mysql_variable: variable=master_info_repository value=TABLE

With this patch prepared statements are used. Proper quoting will be
done automatically based on the type of the variables thus an attempt
to convert to int, then to float is done in first place.

Booleans values, ie: ON, OFF, are not specially handled because they
can be quoted. For example, the following queries are correct and
equivalent, they all set _innodb_file_per_table_ to logical _True_:

mysql> SET GLOBAL innodb_file_per_table = "ON";
mysql> SET GLOBAL innodb_file_per_table = ON;
mysql> SET GLOBAL innodb_file_per_table = 1;

Tested in mysql 5.5 and 5.6.
12 years ago
cloud Merge pull request #5962 from sivel/rax-facts-docs-update 12 years ago
commands Adding no_log: capability for tasks 12 years ago
database Add support for string values 12 years ago
files Merge pull request #5363 from bcoca/acl_improved 12 years ago
internal ansible -> ansibleworks 12 years ago
inventory Make add_host clear the inventory pattern cache, add some more aliases. Fixes #4442. 12 years ago
messaging Use new syntax for module snippets in rabbitmq_policy and update changelog 12 years ago
monitoring module(nagios): don't catch `SystemExit` 12 years ago
net_infrastructure All modules should be using /usr/bin/python in the shebang as ansible_python_interpreter processes this (see FAQ). 12 years ago
network Merge pull request #5826 from romeotheriault/fix_follow_redirects 12 years ago
notification ansible -> ansibleworks 12 years ago
packaging small change to docs 12 years ago
source_control (#5822) Use `mkstemp` to open a temporary file (as NamedTemporaryFile doesn't take a `delete` argument in Python < 2.6) 12 years ago
system Merge pull request #5554 from ttarabul/devel 12 years ago
utilities Detect remote_user change in accelerate daemon and allow a restart 12 years ago
web_infrastructure Clean up stray space at the end of a line after a period. 12 years ago