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

driver "sqlite" not supported #13859

Closed
xtcw255 opened this issue Sep 4, 2023 · 1 comment · Fixed by #13865
Closed

driver "sqlite" not supported #13859

xtcw255 opened this issue Sep 4, 2023 · 1 comment · Fixed by #13865
Labels
bug unexpected problem or unintended behavior

Comments

@xtcw255
Copy link

xtcw255 commented Sep 4, 2023

Relevant telegraf.conf

# # Read metrics from SQL queries
[[inputs.sql]]
#   ## Database Driver
#   ## See https://github.com/influxdata/telegraf/blob/master/docs/SQL_DRIVERS_INPUT.md for
#   ## a list of supported drivers.
driver = "sqlite"
#

Logs from Telegraf

2023-09-04T11:13:32Z E! [telegraf] Error running agent: could not initialize input inputs.sql: driver "sqlite" not supported use one of [clickhouse cockroach flightsql maria mssql mysql pgx pgx/v4 postgres snowflake sqlserver tidb]

System info

Telegraf 1.27.4, Winserver 2019

Docker

No response

Steps to reproduce

  1. Read https://github.com/influxdata/telegraf/blob/master/docs/SQL_DRIVERS_INPUT.md
  2. Set driver to sqlite
  3. Observe the error
    ...

Expected behavior

Expect to not get an error about sqlite being unsupported

Actual behavior

Get an error about it being unsupported

Additional info

No response

@xtcw255 xtcw255 added the bug unexpected problem or unintended behavior label Sep 4, 2023
powersj added a commit to powersj/telegraf that referenced this issue Sep 5, 2023
@powersj
Copy link
Contributor

powersj commented Sep 5, 2023

Hmm in #11260 I thought we allowed windows. I did confirm that upstream does support windows as well. It appears #11664 constrained things further, in order to "sync build constraints found in the sql output for the sqlite"

I have put up #13865 which removes the restrictions for everything except the mips builds. Let's see what CI says and if there are artifacts produced I'll need you to confirm that this works again on Windows.

Thanks!

powersj added a commit to powersj/telegraf that referenced this issue Sep 5, 2023
powersj added a commit to powersj/telegraf that referenced this issue Sep 7, 2023
powersj added a commit to powersj/telegraf that referenced this issue Sep 7, 2023
powersj added a commit to powersj/telegraf that referenced this issue Sep 7, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug unexpected problem or unintended behavior
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants