Repository navigation
{AKS} Add framework for interactive AI agent tool #8990
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
Changes from 5 commits
d207e30
3e93c18
feeff45
dba652e
9fd3d09
8265cd1
2d8590e
15fbd99
d006fc1
5dee708
041e0fc
05a818d
d1efab2
a3bbc45
8e1c379
06ad10c
fd8ee72
bb7b57e
c0d2ee4
3b498cf
3e750fe
5da8c0f
d473c41
1bfdf24
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -23,6 +23,7 @@ | |
| validate_nat_gateway_idle_timeout, | ||
| validate_nat_gateway_managed_outbound_ip_count, | ||
| ) | ||
| from azure.cli.core.api import get_config_dir | ||
| from azure.cli.core.commands.parameters import ( | ||
| edge_zone_type, | ||
| file_type, | ||
|
|
@@ -2772,6 +2773,47 @@ def load_arguments(self, _): | |
| action="store_true", | ||
| ) | ||
|
|
||
| with self.argument_context("aks agent") as c: | ||
| c.positional("prompt", help="Ask any question and answer using available tools.") | ||
| c.argument( | ||
| "max_steps", | ||
| type=int, | ||
| default=10, | ||
| required=False, | ||
| help="Maximum number of steps the LLM can take to investigate the issue.", | ||
| ) | ||
| c.argument( | ||
| "config_file", | ||
| type=str, | ||
|
mainred marked this conversation as resolved.
Resolved
|
||
| default=os.path.join(get_config_dir(), "aksAgent.config"), | ||
| required=False, | ||
|
Member
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. same as above
Member
Author
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Sorry, it has no type and no default value conflict defined in aks_agent I think? |
||
| help="Path to the config file.", | ||
| ) | ||
| c.argument( | ||
| "no_interactive", | ||
| help="Disable interactive mode. When set, the agent will not prompt for input and will run in batch mode.", | ||
| required=False, | ||
|
mainred marked this conversation as resolved.
Resolved
|
||
| action='store_true', | ||
| ) | ||
| c.argument( | ||
| "no_echo_request", | ||
| help="Disable echoing back the question provided to AKS Agent in the output.", | ||
| required=False, | ||
|
mainred marked this conversation as resolved.
Resolved
|
||
| action='store_true', | ||
| ) | ||
| c.argument( | ||
| "show_tool_output", | ||
| help="Show the output of each tool that was called.", | ||
| required=False, | ||
|
mainred marked this conversation as resolved.
Resolved
|
||
| action='store_true', | ||
| ) | ||
| c.argument( | ||
| "refresh_toolsets", | ||
| help="Refresh the toolsets status.", | ||
| required=False, | ||
|
mainred marked this conversation as resolved.
Resolved
|
||
| action='store_true', | ||
| ) | ||
|
|
||
|
|
||
| def _get_default_install_location(exe_name): | ||
| system = platform.system() | ||
|
|
||
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -7,7 +7,7 @@ | |
|
|
||
| from codecs import open as open1 | ||
|
|
||
| from setuptools import setup, find_packages | ||
| from setuptools import find_packages, setup | ||
|
|
||
| VERSION = "18.0.0b21" | ||
|
Member
Author
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. FumingZhang I don't need to new version for this change yet until some other changes are merged. How should I update the change into the next release's change log?
Member
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. To release a new version,
|
||
|
|
||
|
|
@@ -23,7 +23,9 @@ | |
| "License :: OSI Approved :: MIT License", | ||
| ] | ||
|
|
||
| DEPENDENCIES = [] | ||
| DEPENDENCIES = [ | ||
| "holmesgpt==0.12.3" | ||
|
Member
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. this does not always take effect, better test this thoroughly
Member
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more.
|
||
| ] | ||
|
|
||
| with open1("README.rst", "r", encoding="utf-8") as f: | ||
| README = f.read() | ||
|
|
||
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.
You does set a default value of the corresponding parameter max_steps in function aks_agent, which is contrary to your declaration here. If you want to mark it as optional, please set a default value (like None or "") for the parameter.
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.
I did not find the default value of max_steps in aks_agent.
Is it suggested to defaults a variable in this way?