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

[sqlquery]: Command invoked during init statement #32420

Open
MovieStoreGuy opened this issue Apr 16, 2024 · 5 comments
Open

[sqlquery]: Command invoked during init statement #32420

MovieStoreGuy opened this issue Apr 16, 2024 · 5 comments
Labels
bug Something isn't working receiver/sqlquery SQL query receiver

Comments

@MovieStoreGuy
Copy link
Contributor

MovieStoreGuy commented Apr 16, 2024

Component(s)

receiver/sqlquery

What happened?

Description

On invocation of the binary, the function https://github.com/snowflakedb/gosnowflake/blob/3b14581d78684b783c0b2c86ae5873cf338fdec4/driver.go#L70 is called which can cause lock contention within the kernel if binary is invoked frequently enough. (Ie, via a docker health command.

Steps to Reproduce

  • Have github.com/open-telemetry/opentelemetry-collector-contrib/receiver/sqlqueryreceiver v0.98.0 installed
  • Invoke the binary
  • Error message should be reported
# Err Message
level=warning msg=\"DBUS_SESSION_BUS_ADDRESS envvar looks to be not set, this can lead to runaway dbus-daemon processes. To avoid this, set envvar DBUS_SESSION_BUS_ADDRESS=$XDG_RUNTIME_DIR/bus (if it exists) or DBUS_SESSION_BUS_ADDRESS=/dev/null.\" func=\"gosnowflake.(*defaultLogger).Warn\" file=\"log.go:228\"\nPassed healthcheck\n

Expected Result

Ideally not having this error reported or function invoked.

Actual Result

High amount of invoked called to dbus

Collector version

v0.97.0

Environment information

Environment

Compiler: v1.21.2 + cgo

@MovieStoreGuy MovieStoreGuy added bug Something isn't working needs triage New item requiring triage labels Apr 16, 2024
@github-actions github-actions bot added the receiver/sqlquery SQL query receiver label Apr 16, 2024
Copy link
Contributor

Pinging code owners:

See Adding Labels via Comments if you do not have permissions to add labels yourself.

@MovieStoreGuy MovieStoreGuy changed the title [sqlquery]: Kernel lock contention [sqlquery]: Command invoked during init statement Apr 16, 2024
@MovieStoreGuy
Copy link
Contributor Author

The ideal situation is that the upstream dep moves their init function to an optional watchdog process that can called manually by the service.

@crobert-1
Copy link
Member

Thanks for filing @MovieStoreGuy! Looks like this is 99designs/keyring#103 and snowflakedb/gosnowflake#773. Unfortunately from those issues it looks like there's nothing that can be done on our side at this time, from what I understand. I'll leave it open though in case others run into this issue and someone has an idea on how to resolve. 👍

@crobert-1 crobert-1 removed the needs triage New item requiring triage label Apr 16, 2024
@crobert-1
Copy link
Member

crobert-1 commented Apr 16, 2024

Note that this is not a sqlquery receiver issue alone, but also impacts the Snowflake receiver, Pulsar receiver, and docsgen and configschema packages as well, if not more.

Copy link
Contributor

This issue has been inactive for 60 days. It will be closed in 60 days if there is no activity. To ping code owners by adding a component label, see Adding Labels via Comments, or if you are unsure of which component this issue relates to, please ping @open-telemetry/collector-contrib-triagers. If this issue is still relevant, please ping the code owners or leave a comment explaining why it is still relevant. Otherwise, please close it.

Pinging code owners:

See Adding Labels via Comments if you do not have permissions to add labels yourself.

@github-actions github-actions bot added the Stale label Jun 17, 2024
@crobert-1 crobert-1 removed the Stale label Jun 17, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working receiver/sqlquery SQL query receiver
Projects
None yet
Development

No branches or pull requests

2 participants