Skip to content
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

regexes with curlies stopped to work #432

Closed
1 of 3 tasks
ignaqui opened this issue Jul 24, 2017 · 1 comment
Closed
1 of 3 tasks

regexes with curlies stopped to work #432

ignaqui opened this issue Jul 24, 2017 · 1 comment

Comments

@ignaqui
Copy link

ignaqui commented Jul 24, 2017

I'm submitting a ...

  • Bug report
  • Feature requestdd
  • Question / Support request

If you don't send bug report, remove sections below

For bug report please include this information:

Software Versions

Grafana Zabbix Grafana-Zabbix Plugin
4.4.1 3.2.4 3.5.1

What did you do?

Once I updated grafana and grafana-zabbix, variables in templates stopped to work.
It appeared that dot separator has been phased out from data source in favor of curlies.
However, I think this broke my regexes which have curlies inside.

What was the expected result?

The following query used to return proper values:
/alu/./tw-(nyc|que|brx|dwt|brk)-sta_(\w|\d)*-alu-[0-9{2}/

What happened instead?

This query does not work:
{alu}{/tw-(nyc|que|brx|dwt|brk)-sta_(\w|\d)*-alu-[0-9]{2}/}

and this does work:
{alu}{/tw-(nyc|que|brx|dwt|brk)-sta_(\w|\d)*-alu-[0-9]*/}

I tried to mask curlies with backslash and use double curlies without success

@alexanderzobnin
Copy link
Collaborator

@ignaqui this is a great test case! I'll add it to unit tests to ensure things are working in next releases.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants