Skip to content

fix: added opensearch creds to search service#43

Merged
edinstance merged 1 commit intodevfrom
search-k8s-fixes
Nov 3, 2025
Merged

fix: added opensearch creds to search service#43
edinstance merged 1 commit intodevfrom
search-k8s-fixes

Conversation

@edinstance
Copy link
Owner

@edinstance edinstance commented Nov 3, 2025

Summary by CodeRabbit

  • Chores
    • Updated search-service Deployment configuration with new environment variables.

@edinstance edinstance merged commit 8eb82a9 into dev Nov 3, 2025
1 check passed
@coderabbitai
Copy link

coderabbitai bot commented Nov 3, 2025

Caution

Review failed

The pull request is closed.

Walkthrough

Two environment variables—OPENSEARCH_USERNAME and OPENSEARCH_PASSWORD, both set to "admin"—were added to the search-service Deployment configuration. No other modifications to existing variables or settings were made.

Changes

Cohort / File(s) Summary
Kubernetes Deployment Configuration
k8s/search/service.yml
Added two new environment variables (OPENSEARCH_USERNAME and OPENSEARCH_PASSWORD) to the search-service Deployment for OpenSearch authentication.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

  • Straightforward configuration addition with no logic changes.
  • Verify that credentials are appropriate for the environment (dev/staging/production distinction).

Poem

🐰 Hopping through configs with care,
Two secrets now float in the air,
Admin, admin, so simple and plain,
OpenSearch welcomes the search-service's domain! 🔐

✨ Finishing touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch search-k8s-fixes

📜 Recent review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between a65313a and 4a8d053.

📒 Files selected for processing (1)
  • k8s/search/service.yml (1 hunks)

Comment @coderabbitai help to get the list of available commands and usage tips.

@edinstance edinstance deleted the search-k8s-fixes branch November 3, 2025 22:42
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.

1 participant