Skip to content

feat(dbm): add 'agent integration setup postgres' CLI command #51751

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.

Sign up for GitHub

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

Draft
sangeetashivaji wants to merge 11 commits into
base: main
Choose a base branch
from
Draft

feat(dbm): add 'agent integration setup postgres' CLI command #51751

Show file tree
Hide file tree
Changes from 6 commits
Commits
Show all changes
11 commits
Select commit Hold shift + click to select a range
befc375
feat(dbm): add 'agent db setup postgres' CLI command
sangeetashivaji Jun 3, 2026
18e7c0f
refactor(dbm): move 'db setup postgres' under 'agent integration setu…
sangeetashivaji Jun 11, 2026
0e48c1b
chore: fix Python lint and add release note for integration setup pos…
sangeetashivaji Jun 11, 2026
4f0af9b
fix: ship postgres_setup.py with agent assets and fix script lookup path
sangeetashivaji Jun 11, 2026
a96eba2
Merge branch 'main' into sangeeta/dbm-setup-cli-postgres
sangeetashivaji Jun 11, 2026
bc5c2ea
refactor(dbm): use integration's configure_connection instead of mirr…
sangeetashivaji Jun 15, 2026
f042e91
Tunnel integration setup to the Postgres integration's setup module
sangeetashivaji Jun 30, 2026
a90ce41
Merge remote-tracking branch 'origin/main' into sangeeta/dbm-setup-cl…
sangeetashivaji Jun 30, 2026
b8e2672
Add unit tests for the integration setup CLI's Go plumbing
sangeetashivaji Jul 14, 2026
e00dd28
Fix CI build failures: regenerate Bazel BUILD files and lint issues
sangeetashivaji Jul 14, 2026
42d6686
Merge remote-tracking branch 'origin/main' into sangeeta/dbm-setup-cl…
sangeetashivaji Jul 14, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
1 change: 1 addition & 0 deletions cmd/agent/dist/BUILD.bazel
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -57,6 +57,7 @@ pkg_filegroup(
":python_files",
"//cmd/agent/dist/checks:python_files",
"//cmd/agent/dist/utils:python_files",
"//cmd/agent/dist/setup:python_files",
],
visibility = [
"//cmd/agent:__subpackages__",
Expand Down
15 changes: 15 additions & 0 deletions cmd/agent/dist/setup/BUILD.bazel
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -0,0 +1,15 @@
"""DBM setup script assets."""

load("@rules_pkg//pkg:mappings.bzl", "pkg_attributes", "pkg_files", "strip_prefix")

pkg_files(
name = "python_files",
srcs = glob(
["**"],

@chatgpt-codex-connector chatgpt-codex-connector Bot Jun 15, 2026

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

P3 Badge Avoid a recursive glob in the new package

The Bazel review guideline for this repo forbids recursive glob(["**"]) patterns because they silently skip files under future subpackages and hurt incremental/remote-cache behavior. This package currently only ships postgres_setup.py, so listing that file explicitly keeps the package hermetic and avoids the recursive-glob trap.

Useful? React with 👍 / 👎.

exclude = ["BUILD.bazel"],
),
attributes = pkg_attributes(mode = "0644"),
prefix = "bin/agent/dist/setup",
strip_prefix = strip_prefix.from_pkg(),
visibility = ["//cmd/agent/dist:__pkg__"],
)
Loading
Loading