Skip to content

Edvardwd/book search#1894

Open
edvardwd wants to merge 10 commits into
masterfrom
edvardwd/book-search
Open

Edvardwd/book search#1894
edvardwd wants to merge 10 commits into
masterfrom
edvardwd/book-search

Conversation

@edvardwd
Copy link
Copy Markdown
Contributor

This PR includes a simple Book-search / library application with a Streamlit frontend.
It is made to showcase how Testcontainers can be used to test applications by starting actual docker containers. This will not work before a Vespa's Testcontainers implementation has been published.

hmusum
hmusum previously approved these changes Apr 16, 2026
Copy link
Copy Markdown
Member

@hmusum hmusum left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Great!

hmusum
hmusum previously approved these changes Apr 23, 2026
johansolbakken
johansolbakken previously approved these changes Apr 23, 2026
Copy link
Copy Markdown
Contributor

@johansolbakken johansolbakken left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Great 👍🏻

<dependency>
<groupId>ai.vespa</groupId>
<artifactId>vespa-testcontainers</artifactId>
<version>8-SNAPSHOT</version>
Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Works when installing to local Maven repository as of now, but may need to be changed based on Testcontainers publishing to Maven Central

@edvardwd edvardwd temporarily deployed to Vespa Cloud CD May 21, 2026 14:27 — with GitHub Actions Inactive

The tests use `VespaContainer` to spin up an isolated Vespa instance automatically — no manual setup needed.

First, install `vespa-testcontainers` to your local Maven repository:
Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Should not be necessary after a proper publishing pipeline has been set up I guess (ref. this )

@edvardwd edvardwd marked this pull request as ready for review May 21, 2026 14:47
@edvardwd
Copy link
Copy Markdown
Contributor Author

@hmusum please see comments above. Not sure if this is OK to merge atm, or if a proper publishing pipeline should be set up for vespa-testcontainers first (and some lines would need to be changed here). One or two lines in the README here would also need to be changed (version number specifically).

@edvardwd edvardwd requested a review from hmusum May 21, 2026 14:51
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.

3 participants