-
Notifications
You must be signed in to change notification settings - Fork 1.2k
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
Azure Container Apps session execute new api-version #8249
Changes from all commits
8fa3228
c12450a
9a5113e
9fa414d
ee37284
988d0ac
511181f
57b148b
5aa7e8f
d15e3ea
d6889ba
33e1f71
cc0d1df
dd6dc3d
cb122e6
be3d72d
6b65c01
71ca21c
18bf7af
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 |
---|---|---|
|
@@ -321,13 +321,10 @@ | |
} | ||
|
||
SessionCodeInterpreterExecution = { | ||
"properties": { | ||
"identifier": None, | ||
"codeInputType": None, | ||
"executionType": None, | ||
"code": None, | ||
"timeoutInSeconds": None | ||
} | ||
"codeInputType": None, | ||
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 one property 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. The backend Api is switched to use 2024-10-02-preview in this PR, in which the identifier property is removed in the payload, but still exists in the query parameter. |
||
"executionType": None, | ||
"code": None, | ||
"timeoutInSeconds": None | ||
} | ||
|
||
DaprComponentResiliency = { | ||
|
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -444,7 +444,7 @@ def load_arguments(self, _): | |
with self.argument_context('containerapp session code-interpreter', arg_group='file') as c: | ||
c.argument('filename', help="The file to delete or show from the session") | ||
c.argument('filepath', help="The local path to the file to upload to the session") | ||
c.argument('path', help="The path to list files from the session") | ||
c.argument('path', help="The path of files in the session") | ||
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. Recommend to add an example for 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. Fixed |
||
|
||
with self.argument_context('containerapp session code-interpreter', arg_group='execute') as c: | ||
c.argument('code', help="The code to execute in the code interpreter session") | ||
|
This comment was marked as resolved.
Sorry, something went wrong.
This comment was marked as resolved.
Sorry, something went wrong.