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

sqlserver 2008 r2 can not connect #23

Closed
qw4285632q opened this issue Mar 29, 2021 · 3 comments · Fixed by #34
Closed

sqlserver 2008 r2 can not connect #23

qw4285632q opened this issue Mar 29, 2021 · 3 comments · Fixed by #34

Comments

@qw4285632q
Copy link

use the lastest prometheus-mssql-exporter in docker.can not connect to sqlserver 2008 r2. the dynamic port is closed.i have try more than three machine throw same errors like :Failed to connect to database: Failed to connect to ...:1433 in 15000m; but i also try sqlserver 2016 and sqlserver 2019.they are ok!!

@awaragi
Copy link
Owner

awaragi commented Mar 21, 2022

I do not have access to mssql 2008 r2 but in version 1.1 coming out soon, I have updated the library that I use to connect to sql server. Perhaps that will fix the problem.

@awaragi awaragi linked a pull request Mar 21, 2022 that will close this issue
@buddynotshotme
Copy link

buddynotshotme commented Apr 25, 2022

when i use the latest version,i get an error.

mssql_up.set(0);
^

TypeError: mssql_up.set is not a function
at D:\k8s\prometheus-mssql-exporter\src\index.js:133:14
at processTicksAndRejections (node:internal/process/task_queues:96:5)

@awaragi
Copy link
Owner

awaragi commented Apr 25, 2022

Version 1.2.1 is out and should have fixed the mssql_up.set(0) issue crashing the server.

The root cause of the probleme however is that the exporter is unable to connect the database. Can you please let me know what kind of error you are getting if that is the case.

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 a pull request may close this issue.

3 participants