Skip to content

Agent templates: advanced openai template#166

Merged
bbqiu merged 3 commits intodatabricks:mainfrom
bbqiu:bbqiu/openai-advanced
Apr 14, 2026
Merged

Agent templates: advanced openai template#166
bbqiu merged 3 commits intodatabricks:mainfrom
bbqiu:bbqiu/openai-advanced

Conversation

@bbqiu
Copy link
Copy Markdown
Contributor

@bbqiu bbqiu commented Mar 20, 2026

stacked on top of #166 | just review the last commit

Caution

requires changes from databricks/databricks-ai-bridge#397

testing:

  • querying via the UI with a msg that prompts storage creates both checkpoints
  • ran the long running agent tests

@bbqiu bbqiu force-pushed the bbqiu/openai-advanced branch 4 times, most recently from 5116438 to 81d9973 Compare March 26, 2026 23:20
@bbqiu bbqiu requested review from dhruv0811 and jennsun March 26, 2026 23:31
Comment thread agent-openai-advanced/pyproject.toml Outdated
Copy link
Copy Markdown
Contributor

@jennsun jennsun left a comment

Choose a reason for hiding this comment

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

can you make sure to update the old template references as part of this restructuring? will take another pass after langgraph templates are updated
Image

Comment thread agent-openai-advanced/databricks.yml
@bbqiu bbqiu force-pushed the bbqiu/openai-advanced branch 4 times, most recently from 2263604 to 6776dd2 Compare March 31, 2026 01:40
@bbqiu bbqiu requested a review from jennsun March 31, 2026 04:44
@bbqiu bbqiu force-pushed the bbqiu/openai-advanced branch 3 times, most recently from a976ac9 to df77315 Compare March 31, 2026 05:23
Copy link
Copy Markdown
Contributor

@dhruv0811 dhruv0811 left a comment

Choose a reason for hiding this comment

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

Tried this template out, works smoothly!

Left a few nits. Will re-visit after rebasing for a final pass.

Looks good!

Comment thread agent-openai-advanced/.claude/skills/deploy/SKILL.md Outdated
Comment thread agent-openai-advanced/agent_server/start_server.py Outdated
Comment thread agent-openai-advanced/pyproject.toml Outdated
Comment thread agent-openai-advanced/agent_server/utils.py Outdated
@bbqiu bbqiu force-pushed the bbqiu/openai-advanced branch from df77315 to 4fdabe0 Compare April 10, 2026 23:45
bbqiu added 2 commits April 13, 2026 16:06
.
Signed-off-by: Bryan Qiu <bryan.qiu@databricks.com>
.
Signed-off-by: Bryan Qiu <bryan.qiu@databricks.com>
@bbqiu bbqiu force-pushed the bbqiu/openai-advanced branch from 4fdabe0 to f7da003 Compare April 13, 2026 23:23
@bbqiu bbqiu requested review from dhruv0811 and jennsun April 13, 2026 23:24
Comment thread agent-openai-advanced/agent_server/utils.py
Comment thread agent-openai-advanced/scripts/grant_lakebase_permissions.py
Comment thread agent-openai-advanced/scripts/grant_lakebase_permissions.py
Comment thread agent-openai-advanced/agent_server/utils.py Outdated
Comment thread agent-openai-advanced/pyproject.toml Outdated
Copy link
Copy Markdown
Contributor

@jennsun jennsun left a comment

Choose a reason for hiding this comment

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

stamping to unblock, had some cleanup comments :)

.
Signed-off-by: Bryan Qiu <bryan.qiu@databricks.com>
@bbqiu bbqiu changed the title Agent templates: one openai advanced Agent templates: advanced openai template Apr 14, 2026
@bbqiu bbqiu merged commit 861de21 into databricks:main Apr 14, 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