Skip to content

refactor(a2a): Remove A2A_SERVICE_DISCOVERY_ENDPOINTS environment variable#34

Merged
edenreich merged 2 commits intomainfrom
claude/issue-29-20250729-1647
Jul 29, 2025
Merged

refactor(a2a): Remove A2A_SERVICE_DISCOVERY_ENDPOINTS environment variable#34
edenreich merged 2 commits intomainfrom
claude/issue-29-20250729-1647

Conversation

@edenreich
Copy link
Copy Markdown
Collaborator

Since this environment variable doesn't exist on the Inference Gateway and auto service discovery is being used, it's not needed.

Changes

  • Removed A2A_SERVICE_DISCOVERY_ENDPOINTS env var from gateway controller
  • Removed endpoints discovery logic and unused discoverA2AEndpoints function
  • Updated test expectations to remove A2A_SERVICE_DISCOVERY_ENDPOINTS
  • Fixed unused logger variable

Closes #29

Generated with Claude Code

claude bot and others added 2 commits July 29, 2025 16:52
Since this environment variable doesn't exist on the Inference Gateway and 
auto service discovery is being used, it's not needed.

Changes:
- Removed A2A_SERVICE_DISCOVERY_ENDPOINTS env var from gateway controller
- Removed endpoints discovery logic and unused discoverA2AEndpoints function  
- Updated test expectations to remove A2A_SERVICE_DISCOVERY_ENDPOINTS
- Fixed unused logger variable

Closes #29

Co-authored-by: Eden Reich <edenreich@users.noreply.github.com>
@edenreich edenreich merged commit 987171c into main Jul 29, 2025
5 checks passed
@edenreich edenreich deleted the claude/issue-29-20250729-1647 branch July 29, 2025 17:07
ig-semantic-release-bot bot added a commit that referenced this pull request Jul 29, 2025
## [0.12.0](v0.11.1...v0.12.0) (2025-07-29)

### ✨ Features

* **gateway:** Add ServiceAccount for RBAC configuration to Gateway CRD ([#33](#33)) ([dda8cc6](dda8cc6))

### ♻️ Improvements

* **a2a:** Remove A2A_SERVICE_DISCOVERY_ENDPOINTS environment variable ([#34](#34)) ([987171c](987171c)), closes [#29](#29)
* **controller:** Rename ENABLE_AUTH environment variable to AUTH_ENABLE ([#30](#30)) ([86fe434](86fe434)), closes [#28](#28)
* **gateway:** Rename ENABLE_TELEMETRY to TELEMETRY_ENABLE ([#31](#31)) ([adbaefc](adbaefc)), closes [#26](#26)

### 🔨 Miscellaneous

* **deps:** Bump the go_modules group across 1 directory with 2 updates ([#24](#24)) ([76d5a51](76d5a51))
@ig-semantic-release-bot
Copy link
Copy Markdown
Contributor

🎉 This PR is included in version 0.12.0 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

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.

[TASK] Remove A2A_SERVICE_DISCOVERY_ENDPOINTS

1 participant