Document self-managed Search requirement and refresh checkout action#146
Document self-managed Search requirement and refresh checkout action#146dex-the-ai wants to merge 2 commits into
Conversation
|
Owner ping for this maintenance PR: @nithishr Requesting review from the manifest-listed owner(s) where GitHub does not already show a pending review request. |
There was a problem hiding this comment.
Code Review
This pull request updates the README.md to add a note about the Full Text Search requirement for the hotel autocomplete and filter endpoints. The review feedback correctly points out that the hotel_search index is a bucket-level index rather than a scope-level index, and provides a suggestion to clarify this for users setting up a self-managed cluster.
|
Dex update: I pushed follow-up changes.
|
|
I pushed two follow-ups on this branch: (1) clarified the README note so |
Summary
actions/checkout@v3toactions/checkout@v4Verification
cd src && uv venv --python 3.12 --seed --clear .venv && source .venv/bin/activate && uv pip install -r requirements.txtcd src && source .venv/bin/activate && python -m pip list --outdated --format=json→[]cd src && source .venv/bin/activate && uvx pip-audit -r requirements.txt→No known vulnerabilities foundcd src && source .venv/bin/activate && python -m pytest -q(Capella-backed.env) →47 passed in 21.99scurl -sS http://127.0.0.1:8080/curl -sS 'http://127.0.0.1:8080/api/v1/airport/list?country=France&limit=2'curl -sS 'http://127.0.0.1:8080/api/v1/hotel/autocomplete?name=sea'curl -sS -X POST 'http://127.0.0.1:8080/api/v1/hotel/filter?limit=2&offset=0' -H 'Content-Type: application/json' -d '{"description":"newly renovated"}'Evidence
requirements.txtwas already current: no direct package updates were available andpip-auditreported no known vulnerabilities..env: pytest passed end to end and the airport + hotel walkthrough requests returned expected travel-sample data./and captured a screenshot locally attutorial-maintenance/runs/python-quickstart/2026-05-26/screenshots/swagger-ui.png.tests/test_hotel.pyfails (6 failed, 41 passed).tutorial-maintenance/runs/python-quickstart/2026-05-26/verification.mdtutorial-maintenance/runs/python-quickstart/2026-05-26/walkthrough.txttutorial-maintenance/runs/python-quickstart/2026-05-26/pytest-capella.txtPytest result
Hotel walkthrough excerpt
[ { "name": "Seal View" }, { "name": "Seal Cove Inn" }, { "name": "Seal Rock Inn" } ]Notes
hotel_searchindex can be created intravel-sample.inventory.Media evidence
Notes
tutorial-maintenance/runs/python-quickstart/2026-05-26/screenshots/swagger-ui.pngNotes
tutorial-maintenance/runs/python-quickstart/2026-05-26/swagger-ui.webm