Skip to content

Agent templates: e2e tests + remove unnecessary app.yaml files#143

Merged
bbqiu merged 33 commits intodatabricks:mainfrom
bbqiu:bbqiu/e2e-tests
Mar 5, 2026
Merged

Agent templates: e2e tests + remove unnecessary app.yaml files#143
bbqiu merged 33 commits intodatabricks:mainfrom
bbqiu:bbqiu/e2e-tests

Conversation

@bbqiu
Copy link
Contributor

@bbqiu bbqiu commented Feb 27, 2026

review the last commit only

stacked on top of #131

  • remove app.yaml when unnecessary (transfer to databricks.yml file)
  • NEED to keep app.yaml for templates that can be created through the UI (the UI install doesn't run databricks bundle commands)
  • update relevant skills
  • added + ran e2e tests (all passed in the https://db-ml-models-dev-us-west.cloud.databricks.com workspace)
All 7 templates passed (local + deploy), completed in 6 minutes 2 seconds.

  ┌───────────────────────────────────────────┬────────┐
  │                 Template                  │ Result │
  ├───────────────────────────────────────────┼────────┤
  │ agent-langgraph                           │ PASSED │
  ├───────────────────────────────────────────┼────────┤
  │ agent-langgraph-short-term-memory         │ PASSED │
  ├───────────────────────────────────────────┼────────┤
  │ agent-langgraph-long-term-memory          │ PASSED │
  ├───────────────────────────────────────────┼────────┤
  │ agent-openai-agents-sdk                   │ PASSED │
  ├───────────────────────────────────────────┼────────┤
  │ agent-openai-agents-sdk-short-term-memory │ PASSED │
  ├───────────────────────────────────────────┼────────┤
  │ agent-openai-agents-sdk-multiagent        │ PASSED │
  ├───────────────────────────────────────────┼────────┤
  │ agent-non-conversational                  │ PASSED │
  └───────────────────────────────────────────┴────────┘

@bbqiu bbqiu force-pushed the bbqiu/e2e-tests branch from 2a6f6f6 to 781220a Compare March 2, 2026 06:52
@bbqiu bbqiu changed the title Agent templates: e2e tests Agent templates: e2e tests + remove all app.yaml files Mar 3, 2026
@bbqiu bbqiu force-pushed the bbqiu/e2e-tests branch from 059311c to f3861f8 Compare March 3, 2026 21:12
@bbqiu bbqiu requested a review from dhruv0811 March 4, 2026 01:19
@bbqiu bbqiu changed the title Agent templates: e2e tests + remove all app.yaml files Agent templates: e2e tests + remove unnecessary app.yaml files Mar 4, 2026
Copy link
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.

Thanks for adding testing on these templates! Will make our own devex much better for sure :)

Left a few comments on how we can make the test infra a bit lighter. Also some other minor nits and clarifications.

I tried running the tests locally, apart perm errors on lakebase and genie resources, everything passed!

bbqiu added 23 commits March 5, 2026 01:18
Signed-off-by: Bryan Qiu <bryan.qiu@databricks.com>

synced skills

Signed-off-by: Bryan Qiu <bryan.qiu@databricks.com>
Signed-off-by: Bryan Qiu <bryan.qiu@databricks.com>
.
Signed-off-by: Bryan Qiu <bryan.qiu@databricks.com>
.
Signed-off-by: Bryan Qiu <bryan.qiu@databricks.com>
Signed-off-by: Bryan Qiu <bryan.qiu@databricks.com>
Signed-off-by: Bryan Qiu <bryan.qiu@databricks.com>
.
Signed-off-by: Bryan Qiu <bryan.qiu@databricks.com>
.
Signed-off-by: Bryan Qiu <bryan.qiu@databricks.com>
.
Signed-off-by: Bryan Qiu <bryan.qiu@databricks.com>
.
Signed-off-by: Bryan Qiu <bryan.qiu@databricks.com>
.
Signed-off-by: Bryan Qiu <bryan.qiu@databricks.com>
.
Signed-off-by: Bryan Qiu <bryan.qiu@databricks.com>
.
Signed-off-by: Bryan Qiu <bryan.qiu@databricks.com>
.
Signed-off-by: Bryan Qiu <bryan.qiu@databricks.com>
.
Signed-off-by: Bryan Qiu <bryan.qiu@databricks.com>
Signed-off-by: Bryan Qiu <bryan.qiu@databricks.com>
Signed-off-by: Bryan Qiu <bryan.qiu@databricks.com>
.
Signed-off-by: Bryan Qiu <bryan.qiu@databricks.com>
Signed-off-by: Bryan Qiu <bryan.qiu@databricks.com>
.
Signed-off-by: Bryan Qiu <bryan.qiu@databricks.com>
.
Signed-off-by: Bryan Qiu <bryan.qiu@databricks.com>
Signed-off-by: Bryan Qiu <bryan.qiu@databricks.com>
.
Signed-off-by: Bryan Qiu <bryan.qiu@databricks.com>
bbqiu added 10 commits March 5, 2026 01:19
.
Signed-off-by: Bryan Qiu <bryan.qiu@databricks.com>
.
Signed-off-by: Bryan Qiu <bryan.qiu@databricks.com>
.
Signed-off-by: Bryan Qiu <bryan.qiu@databricks.com>
.
Signed-off-by: Bryan Qiu <bryan.qiu@databricks.com>
.
Signed-off-by: Bryan Qiu <bryan.qiu@databricks.com>
.
Signed-off-by: Bryan Qiu <bryan.qiu@databricks.com>
.
Signed-off-by: Bryan Qiu <bryan.qiu@databricks.com>
.
Signed-off-by: Bryan Qiu <bryan.qiu@databricks.com>
Signed-off-by: Bryan Qiu <bryan.qiu@databricks.com>
.
Signed-off-by: Bryan Qiu <bryan.qiu@databricks.com>
@bbqiu bbqiu force-pushed the bbqiu/e2e-tests branch from 7ea5ad0 to 969ffaa Compare March 5, 2026 09:19
@bbqiu bbqiu merged commit f07ec37 into databricks:main Mar 5, 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.

2 participants