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

[dynamodb] Re-Enable tests after Jetty upgrade #16535

Closed
holgerfriedrich opened this issue Mar 17, 2024 · 0 comments · Fixed by #16554
Closed

[dynamodb] Re-Enable tests after Jetty upgrade #16535

holgerfriedrich opened this issue Mar 17, 2024 · 0 comments · Fixed by #16554
Assignees
@holgerfriedrich
Labels
test

Comments

@holgerfriedrich
Copy link
Member

Karaf and Jetty were updated in openhab/openhab-core#4133. Jetty is now at 9.4.53.

DynamoDB tests relay on DynamodDBLocal 1.15, which has a transitive dependency to jetty 9.4.18. There are newer packages available, from 2.x uses Jetty 11 or 12.
https://mvnrepository.com/artifact/com.amazonaws/DynamoDBLocal

Jetty team has changed the packaging of HuffmanEncoder files, leading to an incompatibility of DynamoDBLocal with current Jetty release.

#16493 was merged with tests for DynamoDB disabled in pom.xml, as add-ons could no longer be built after the update in core.

A solution should be found how to resolve this issue and to re-enable the tests.
@holgerfriedrich holgerfriedrich self-assigned this Mar 17, 2024
@holgerfriedrich holgerfriedrich mentioned this issue Mar 17, 2024
Merged
holgerfriedrich added a commit to holgerfriedrich/openhab-addons that referenced this issue Mar 21, 2024
@holgerfriedrich
[dynamodb] Enable tests.
d474325 
Add missing dependencies after Karaf update openhab#16493.

Fixes openhab#16535.

Signed-off-by: Holger Friedrich <[email protected]>
holgerfriedrich added a commit to holgerfriedrich/openhab-addons that referenced this issue Mar 21, 2024
@holgerfriedrich
[dynamodb] Enable tests.
b950a9f 
Add missing dependencies after Karaf upgrade openhab#16493.

Fixes openhab#16535.

Signed-off-by: Holger Friedrich <[email protected]>
@holgerfriedrich holgerfriedrich mentioned this issue Mar 21, 2024
Merged
holgerfriedrich added a commit to holgerfriedrich/openhab-addons that referenced this issue Mar 21, 2024
@holgerfriedrich
[dynamodb] Enable tests
5073c77 
Add missing dependencies after Karaf upgrade openhab#16493.

Fixes openhab#16535.

Signed-off-by: Holger Friedrich <[email protected]>
lolodomo pushed a commit that referenced this issue Mar 29, 2024
@holgerfriedrich
[dynamodb] Enable tests (#16554)
d2050f4 
Add missing dependencies after Karaf upgrade #16493.

Fixes #16535.

Signed-off-by: Holger Friedrich <[email protected]>
lo92fr pushed a commit to lo92fr/openhab-addons that referenced this issue Apr 30, 2024
@holgerfriedrich @lo92fr
[dynamodb] Enable tests (openhab#16554)
c7e8f59 
Add missing dependencies after Karaf upgrade openhab#16493.

Fixes openhab#16535.

Signed-off-by: Holger Friedrich <[email protected]>
adr001db pushed a commit to adr001db/openhab-addons that referenced this issue May 12, 2024
@holgerfriedrich @adr001db
[dynamodb] Enable tests (openhab#16554)
ef2ea58 
Add missing dependencies after Karaf upgrade openhab#16493.

Fixes openhab#16535.

Signed-off-by: Holger Friedrich <[email protected]>
Signed-off-by: Alexander Drent <[email protected]>
pgfeller pushed a commit to pgfeller/openhab-addons that referenced this issue Sep 29, 2024
@holgerfriedrich @pgfeller
[dynamodb] Enable tests (openhab#16554)
03c8378 
Add missing dependencies after Karaf upgrade openhab#16493.

Fixes openhab#16535.

Signed-off-by: Holger Friedrich <[email protected]>
Signed-off-by: Patrik Gfeller <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@holgerfriedrich holgerfriedrich

Labels
test
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

[dynamodb] Enable tests holgerfriedrich/openhab-addons
1 participant
@holgerfriedrich