-
Notifications
You must be signed in to change notification settings - Fork 62
chore: update config vars #85
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
|
@copilot Review the variables to see if any thing is mismatched. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull request overview
This PR updates configuration variables for the xtramcp server deployment, renaming from xtragpt_* to xtramcp_* naming convention and adding new configuration options.
Changes:
- Renamed configuration variables from
xtragpt_*toxtramcp_*prefix for consistency - Added new configuration options for MongoDB, arXiv metadata DB, and email addresses
- Updated the Docker image tag for the xtramcp server
Reviewed changes
Copilot reviewed 4 out of 4 changed files in this pull request and generated 4 comments.
| File | Description |
|---|---|
| helm-chart/values.yaml | Updates Docker image tag for paperdebugger-xtramcp-server |
| helm-chart/templates/paperdebugger-xtramcp-server.yaml | Renames config variables from xtragpt to xtramcp, adds new env vars (OpenAI base URL, max retries, timeout, MongoDB, arXiv DB, email addresses) |
| hack/prd.sh | Updates environment variable names from XTRAGPT to XTRAMCP and adds new variables for production deployment script |
| .github/workflows/release.yml | Updates GitHub Actions secrets references from XTRAGPT to XTRAMCP for production deployment |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is the final PR Bugbot will review for you during this billing cycle
Your free Bugbot reviews will reset on February 6
Details
You are on the Bugbot Free tier. On this plan, Bugbot will review limited PRs each billing cycle.
To receive Bugbot reviews on all of your PRs, visit the Cursor dashboard to activate Pro and start your 14-day free trial.
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
…rDebugger/paperdebugger into chore/update-xtramcp-config
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull request overview
Copilot reviewed 4 out of 4 changed files in this pull request and generated no new comments.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Update config vars for deployment.
Note
Migrates deployment configuration to use
XTRAMCP_*variables and expands the Xtra MCP server settings.XTRAGPT_*withXTRAMCP_*in.github/workflows/release.ymlandhack/prd.sh, passing new values to Helmpaperdebugger-xtramcp-serverConfigMapwithOPENAI_BASE_URL/API_KEY,OPENAI_MAX_RETRIES,OPENAI_TIMEOUT, OpenReview creds,DOI_EMAIL_ADDRESS,CROSSREF_EMAIL_ADDRESS,MONGODB_CONNECTION_STRING,MONGODB_DATABASE, andARXIV_METADATA_DB_URLhelm-chart/values.yamlto usextragpt-mcp-server:sha-2f1404cimageWritten by Cursor Bugbot for commit 48d4e35. This will update automatically on new commits. Configure here.