Skip to content

Conversation

@anderslindho
Copy link
Contributor

No description provided.

@anderslindho anderslindho requested a review from jacomago March 13, 2025 10:20
@anderslindho anderslindho self-assigned this Mar 13, 2025
@anderslindho anderslindho marked this pull request as draft March 13, 2025 11:28
@anderslindho
Copy link
Contributor Author

anderslindho commented Mar 13, 2025

Looks like issues w elastic affecting both this and #173

org.springframework.web.server.ResponseStatusException: 500 INTERNAL_SERVER_ERROR "Count failed for {~tag=[group4_10]} Cause [es/count] failed: [index_not_found_exception] no such index [test_443_channelfinder]"; nested exception is co.elastic.clients.elasticsearch._types.ElasticsearchException: [es/count] failed: [index_not_found_exception] no such index [test_443_channelfinder]
	at org.phoebus.channelfinder.ChannelRepository.count(ChannelRepository.java:577) ~[classes/:na]
	at org.phoebus.channelfinder.ChannelRepository.countByTag(ChannelRepository.java:602) ~[classes/:na]
	at org.phoebus.channelfinder.MetricsService.updateMetrics(MetricsService.java:84) ~[classes/:na]
	at jdk.internal.reflect.GeneratedMethodAccessor166.invoke(Unknown Source) ~[na:na]
	at java.base/jdk.internal.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43) ~[na:na]
	at java.base/java.lang.reflect.Method.invoke(Method.java:569) ~[na:na]
	at org.springframework.scheduling.support.ScheduledMethodRunnable.run(ScheduledMethodRunnable.java:84) ~[spring-context-5.3.22.jar:5.3.22]
	at org.springframework.scheduling.support.DelegatingErrorHandlingRunnable.run(DelegatingErrorHandlingRunnable.java:54) ~[spring-context-5.3.22.jar:5.3.22]
	at java.base/java.util.concurrent.Executors$RunnableAdapter.call(Executors.java:539) ~[na:na]
	at java.base/java.util.concurrent.FutureTask.runAndReset(FutureTask.java:305) ~[na:na]
	at java.base/java.util.concurrent.ScheduledThreadPoolExecutor$ScheduledFutureTask.run(ScheduledThreadPoolExecutor.java:305) ~[na:na]
	at java.base/java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1136) ~[na:na]
	at java.base/java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:635) ~[na:na]
	at java.base/java.lang.Thread.run(Thread.java:840) ~[na:na]
Caused by: co.elastic.clients.elasticsearch._types.ElasticsearchException: [es/count] failed: [index_not_found_exception] no such index [test_443_channelfinder]
	at co.elastic.clients.transport.ElasticsearchTransportBase.getApiResponse(ElasticsearchTransportBase.java:345) ~[elasticsearch-java-8.11.2.jar:na]
	at co.elastic.clients.transport.ElasticsearchTransportBase.performRequest(ElasticsearchTransportBase.java:147) ~[elasticsearch-java-8.11.2.jar:na]
	at co.elastic.clients.elasticsearch.ElasticsearchClient.count(ElasticsearchClient.java:467) ~[elasticsearch-java-8.11.2.jar:na]
	at org.phoebus.channelfinder.ChannelRepository.count(ChannelRepository.java:571) ~[classes/:na]
	... 13 common frames omitted

Error:  Tests run: 1, Failures: 1, Errors: 0, Skipped: 0, Time elapsed: 65.01 s <<< FAILURE! -- in org.phoebus.channelfinder.ChannelRepositorySearchIT
Error:  org.phoebus.channelfinder.ChannelRepositorySearchIT.searchTest -- Time elapsed: 64.64 s <<< FAILURE!
Error:  Failures: 
Error:    ChannelRepositorySearchIT.searchTest:84 expected: <1> but was: <0>
[INFO] 
Error:  Tests run: 218, Failures: 1, Errors: 0, Skipped: 0

@anderslindho anderslindho marked this pull request as ready for review March 13, 2025 13:20
@jacomago
Copy link
Contributor

Since you are improving things, what about adding the build step into the dockerfile. That way you don't have this awkward copy of the jar file, which means you need to have run build before you can run the docker compose?

@sonarqubecloud
Copy link

@jacomago jacomago merged commit 6176003 into ChannelFinder:master Mar 17, 2025
6 checks passed
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 this pull request may close these issues.

2 participants