Skip to content
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

fix(server): Add Azure-related environment variables for server #7770

Draft
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

andrewhooker2
Copy link
Contributor

Included new environment variables for Azure AD token, OpenAI API type, version, and endpoint in the .env.template. This will enable support for Azure services in the AutoGPT server configuration.

Background

Resolves: #7746

Our current server configuration lacks support for Azure OpenAI services, preventing users from leveraging Azure-deployed models in the AutoGPT Builder, particularly when working with LLM agent blocks.
This PR adds the necessary environment variables to support Azure services in the AutoGPT server configuration, enabling seamless integration with Azure-deployed OpenAI models.

Changes 🏗️

Updated .env.template to include new environment variables:

Addition of the following required items to our .env.template

  • Azure AD token
  • OpenAI API type
  • API version
  • API endpoint

Next Steps:

  • One call out here is that with these added we are still requiring the user to add their api key to each Block. This will be fixed in a following PR and does not align with being atomic here.

PR Quality Scorecard ✨

  • Have you used the PR description template?   +2 pts
  • Is your pull request atomic, focusing on a single change?   +5 pts
  • Have you linked the GitHub issue(s) that this PR addresses?   +5 pts
  • Have you documented your changes clearly and comprehensively?   +5 pts
  • Have you changed or added a feature?   -4 pts
    • Have you added/updated corresponding documentation?   +4 pts
    • Have you added/updated corresponding integration tests?   +5 pts
  • Have you changed the behavior of AutoGPT?   -5 pts
    • Have you also run agbenchmark to verify that these changes do not regress performance?   +10 pts

Included new environment variables for Azure AD token, OpenAI API type, version, and endpoint in the .env.template. This will enable support for Azure services in the AutoGPT server configuration.
Copy link

netlify bot commented Aug 8, 2024

Deploy Preview for auto-gpt-docs canceled.

Name Link
🔨 Latest commit 4907d33
🔍 Latest deploy log https://app.netlify.com/sites/auto-gpt-docs/deploys/66b4de7c2a6626000822df94

@ntindle
Copy link
Member

ntindle commented Aug 8, 2024

Can these be set via block?

@github-actions github-actions bot added the conflicts Automatically applied to PRs with merge conflicts label Sep 4, 2024
Copy link
Contributor

github-actions bot commented Sep 4, 2024

This pull request has conflicts with the base branch, please resolve those so we can evaluate the pull request.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
conflicts Automatically applied to PRs with merge conflicts Server size/s
Projects
Status: 🆕 Needs initial review
Development

Successfully merging this pull request may close these issues.

[Server]: Deployed Azure instances with endpoints do not run agents
2 participants