Skip to content

Add new provider configurations and update server ports#308

Closed
Abdullakala wants to merge 2 commits into
vxcontrol:mainfrom
Abdullakala:main
Closed

Add new provider configurations and update server ports#308
Abdullakala wants to merge 2 commits into
vxcontrol:mainfrom
Abdullakala:main

Conversation

@Abdullakala

Copy link
Copy Markdown

Description of the Change

Problem

Solution

Closes #

Type of Change

  • 🐛 Bug fix (non-breaking change which fixes an issue)
  • 🚀 New feature (non-breaking change which adds functionality)
  • 💥 Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • 📚 Documentation update
  • 🔧 Configuration change
  • 🧪 Test update
  • 🛡️ Security update

Areas Affected

  • Core Services (Frontend UI/Backend API)
  • AI Agents (Researcher/Developer/Executor)
  • Security Tools Integration
  • Memory System (Vector Store/Knowledge Base)
  • Monitoring Stack (Grafana/OpenTelemetry)
  • Analytics Platform (Langfuse)
  • External Integrations (LLM/Search APIs)
  • Documentation
  • Infrastructure/DevOps

Testing and Verification

Test Configuration

PentAGI Version:
Docker Version:
Host OS:
LLM Provider:
Enabled Features: [Langfuse/Grafana/etc]

Test Steps

Test Results

Security Considerations

Performance Impact

Documentation Updates

  • README.md updates
  • API documentation updates
  • Configuration documentation updates
  • GraphQL schema updates
  • Other:

Deployment Notes

Checklist

Code Quality

  • My code follows the project's coding standards
  • I have added/updated necessary documentation
  • I have added tests to cover my changes
  • All new and existing tests pass
  • I have run go fmt and go vet (for Go code)
  • I have run npm run lint (for TypeScript/JavaScript code)

Security

  • I have considered security implications
  • Changes maintain or improve the security model
  • Sensitive information has been properly handled

Compatibility

  • Changes are backward compatible
  • Breaking changes are clearly marked and documented
  • Dependencies are properly updated

Documentation

  • Documentation is clear and complete
  • Comments are added for non-obvious code
  • API changes are documented

Additional Notes

Copilot AI review requested due to automatic review settings May 9, 2026 05:07

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

Adds new example LLM provider configuration files and updates Docker Compose default port mappings for the PentAGI and scraper services.

Changes:

  • Added example.custom.provider.yml (custom/OpenAI-style) and example.ollama.provider.yml (Ollama) provider configuration templates.
  • Updated docker-compose.yml to use 8444 (PentAGI) and 9444 (scraper) as the new default host/container ports.
  • Adjusted SERVER_PORT default in Compose to match the new exposed/mapped port.

Reviewed changes

Copilot reviewed 3 out of 4 changed files in this pull request and generated 3 comments.

File Description
example.ollama.provider.yml Adds an Ollama-focused agent model configuration template.
example.custom.provider.yml Adds a custom-provider agent model configuration template (includes pricing/reasoning fields).
docker-compose.yml Changes default published/exposed ports for PentAGI and the scraper service.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread docker-compose.yml
Comment on lines 28 to 34
container_name: pentagi
hostname: pentagi
expose:
- 8443/tcp
- 8444/tcp
ports:
- ${PENTAGI_LISTEN_IP:-127.0.0.1}:${PENTAGI_LISTEN_PORT:-8443}:8443
- ${PENTAGI_LISTEN_IP:-127.0.0.1}:${PENTAGI_LISTEN_PORT:-8444}:8444
depends_on:
Comment thread docker-compose.yml
- 443/tcp
ports:
- ${SCRAPER_LISTEN_IP:-127.0.0.1}:${SCRAPER_LISTEN_PORT:-9443}:443
- ${SCRAPER_LISTEN_IP:-127.0.0.1}:${SCRAPER_LISTEN_PORT:-9444}:443
model: "llama3.1:8b-instruct-q8_0"
temperature: 0.0
top_p: 1.0
n: 1
@sickwell

Copy link
Copy Markdown

Thank you for the PR.

The main configuration files are intended to be customized by users for their own environments, and we don’t consider these changes necessary for the upstream repository.

We already maintain the provider configurations that have been tested on our side, so we don’t want to include additional untested config changes.

Also, this PR includes unrelated changes, such as default port changes and a compiled backend/pentagi binary, which we can’t accept in a source PR.

Closing as not planned.

@sickwell sickwell closed this May 15, 2026
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