Skip to content

feat: auto-fetch-collections-from-s3#109

Merged
tobias-pfaffelmoser-aa merged 4 commits into
mainfrom
feat/auto-fetch-collections-from-s3
May 19, 2026
Merged

feat: auto-fetch-collections-from-s3#109
tobias-pfaffelmoser-aa merged 4 commits into
mainfrom
feat/auto-fetch-collections-from-s3

Conversation

@bennotieno
Copy link
Copy Markdown
Contributor

No description provided.

@github-actions github-actions Bot added the feat label May 18, 2026
@bennotieno bennotieno force-pushed the feat/auto-fetch-collections-from-s3 branch 3 times, most recently from 841aab8 to 78c8df9 Compare May 19, 2026 06:36
@bennotieno bennotieno force-pushed the feat/auto-fetch-collections-from-s3 branch from 25bfdd1 to f0553f4 Compare May 19, 2026 06:46
github-actions Bot and others added 2 commits May 19, 2026 06:46
Follow-up to feat/auto-fetch-collections-from-s3 with three small fixes
surfaced during review.

- track_collection: skip the read loop when the jq result is empty. Bash
  here-strings always append a newline, so `done <<< ""` would otherwise
  iterate once with an empty item and write a bogus `host,` line to
  $QDRANT_COLLECTIONS_FILE. Affects `get_coll` against an empty source.
- generate_snapshot_file_from_s3: fix word order in the non-error exit
  message so the filename lands in the correct position.
- UPGRADE-OPERATIONS.md restore example: set QDRANT_SOURCE_HOSTS="" to
  match the new restore-from-S3-only contract used by the k8s manifests;
  move the port-forward note to QDRANT_RESTORE_HOSTS where it belongs.
- Sync configmap-script.yaml via config_map_updater.sh.
@tobias-pfaffelmoser-aa tobias-pfaffelmoser-aa force-pushed the feat/auto-fetch-collections-from-s3 branch from 5e35d27 to 3193a80 Compare May 19, 2026 09:09
@tobias-pfaffelmoser-aa tobias-pfaffelmoser-aa merged commit 06f8d9e into main May 19, 2026
6 checks passed
@tobias-pfaffelmoser-aa tobias-pfaffelmoser-aa deleted the feat/auto-fetch-collections-from-s3 branch May 19, 2026 09:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants