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.

Sign up for GitHub

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

Jump to bottom

[sqlqueryreceiver] TestOracleDBIntegration test times out #12332

Closed
codeboten opened this issue Jul 12, 2022 · 5 comments
Closed

[sqlqueryreceiver] TestOracleDBIntegration test times out #12332

codeboten opened this issue Jul 12, 2022 · 5 comments
Labels
flaky test a test is flaky good first issue Good for newcomers help wanted Extra attention is needed priority:p2 Medium receiver/sqlquery SQL query receiver

Comments

@codeboten
Copy link
Contributor

See an example here:

https://github.com/open-telemetry/opentelemetry-collector-contrib/runs/7303422652?check_suite_focus=true#step:5:1016
@codeboten codeboten added flaky test a test is flaky receiver/sqlquery SQL query receiver labels Jul 12, 2022
@TylerHelmuth TylerHelmuth added receiver/sqlquery SQL query receiver and removed receiver/sqlquery SQL query receiver labels Jul 16, 2022
@github-actions
Copy link
Contributor

Pinging code owners: @dmitryax @pmcollins

@crobert-1
Copy link
Member

crobert-1 commented Aug 3, 2022 •
edited
Loading

This is referenced in #12202 and #12203. It looks like we may need to keep increasing the timeout level, or figure out another way to implement the TestOracleDBIntegration test that requires the longer timeout.

@dmitryax
Copy link
Member

dmitryax commented Aug 3, 2022

It looks like we may need to keep increasing the timeout level, or figure out another way to implement the TestOracleDBIntegration test that requires the longer timeout.

2 option would be the best to try first. This takes currently takes too much time

@dmitryax dmitryax added help wanted Extra attention is needed good first issue Good for newcomers labels Aug 3, 2022
@evan-bradley evan-bradley added the priority:p2 Medium label Sep 14, 2022
@atoulme
Copy link
Contributor

atoulme commented Sep 21, 2022 •
edited
Loading

This is because the docker image itself used for this is quite large (11.2 GB), and it takes a long time for it to download.

@atoulme atoulme mentioned this issue Oct 4, 2022
Merged
@dmitryax
Copy link
Member

dmitryax commented Oct 5, 2022

Should be fixed by #14717

@dmitryax dmitryax closed this as completed Oct 5, 2022
@crobert-1 crobert-1 mentioned this issue Jan 19, 2023
Closed
6 tasks
@crobert-1 crobert-1 mentioned this issue Oct 11, 2023
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
flaky test a test is flaky good first issue Good for newcomers help wanted Extra attention is needed priority:p2 Medium receiver/sqlquery SQL query receiver
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
6 participants
@atoulme @codeboten @dmitryax @evan-bradley @TylerHelmuth @crobert-1