Comment 12 for bug 794645

Revision history for this message
Salvatore Orlando (salvatore-orlando) wrote :

Hi Edouard,

I looked back at your initial bug report and I saw you found the error against rev 1158.
Looking at changes in nova revs, I found out that the string for building the filter was slightly changed, replacing single quotes with double quotes and vice versa.
This caused the failure as xapi backend only accepts filter tokens enclosed in double quotes.

I think the change was due to coding style consistency. I have reverted the string to the old style, and verified that the error does not occur anymore. The fix will be proposed for merge.

If the change gets rejected for coding style issues, I will propose a fix along the lines of the patch you supplied (btw, thanks for providing it).

Thanks,
Salvatore