Skip to content

[NO-MERGE][CMPT-5391] feat(pipelines): add pipeline CRUD, versions, and graph commands - #2

Closed
sunny2get wants to merge 14 commits into
cmpt-5391/pipelines-basefrom
cmpt-5391/pipelines-crud
Closed

[NO-MERGE][CMPT-5391] feat(pipelines): add pipeline CRUD, versions, and graph commands#2
sunny2get wants to merge 14 commits into
cmpt-5391/pipelines-basefrom
cmpt-5391/pipelines-crud

Conversation

@sunny2get

Copy link
Copy Markdown
Owner

NO-MERGE — Requirements are still evolving. This PR is open for early review only.
Stack: PR 1 (base)PR 2 (this) → PR 3 → PR 4 → PR 5 → PR 6

Summary

  • Adds dr pipelines create/list/get/update/delete/lock — full pipeline lifecycle CRUD
  • Adds dr pipelines version list/get — read-only access to pipeline versions
  • Adds dr pipelines graph — renders the pipeline/task DAG (draft or locked scope)
  • Introduces cmd/pipelines/scopeflag/ — shared --pipeline/--scope/--version flag helpers used by graph (and reused by later PRs)
  • Extends internal/pipelines/pipeline.go (CRUD API calls) and version.go

Stacked on: cmpt-5391/pipelines-base — diff here shows only the delta.

Test plan

  • go build ./...
  • go test ./cmd/pipelines/... ./internal/pipelines/...
  • golangci-lint run ./cmd/pipelines/... ./internal/pipelines/...
  • DATAROBOT_CLI_FEATURE_PIPELINES=true go run main.go pipelines --help — shows create/list/get/update/delete/lock/version/graph
  • dr pipelines create ./my_pipeline.py --help / dr pipelines graph --help

🤖 Generated with Claude Code

sunny2get and others added 4 commits May 27, 2026 13:23
…ate with shared infrastructure (datarobot-oss#517)

Co-authored-by: Claude Sonnet 4.6 <noreply@anthropic.com>
Co-authored-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
Co-authored-by: Carson Gee <carson.gee@datarobot.com>
@sunny2get
sunny2get force-pushed the cmpt-5391/pipelines-crud branch 2 times, most recently from 9382396 to 3b4d4c4 Compare May 28, 2026 14:40
sunny2get and others added 8 commits May 28, 2026 10:44
Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Introduces the core pipeline management subcommands:

- create / list / get / update / delete / lock — full pipeline lifecycle
- version list / version get — inspect pipeline versions
- graph — render the pipeline/task DAG (draft or locked scope)
- scopeflag package — shared --pipeline/--scope/--version flag binding

Internal additions:
- internal/pipelines/pipeline.go: CreatePipeline, ListPipelines, GetPipeline,
  UpdatePipeline, DeletePipeline, LockPipeline, multipart upload helpers
- internal/pipelines/version.go: ListVersions, GetVersion, GetGraph

Documentation: pipelines.md CRUD sections, initial pipelines-reference.md.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
First used by the graph command on this branch.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Update all cmd/pipeline subcommand files to use internal/pipeline
(singular) package import and pipeline.X symbol references after the
internal/pipelines → internal/pipeline rename.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
…n, limit defaults, version prefix

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@sunny2get
sunny2get force-pushed the cmpt-5391/pipelines-crud branch from 3b4d4c4 to 6327eb4 Compare May 28, 2026 14:46
…up commands

Add: alias check, RunE=nil guard, full subcommand list (del/lock/version/graph),
version sub-subcommand registration, and version metadata assertions.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Replace all dr pipelines → dr pipeline and
DATAROBOT_CLI_FEATURE_PIPELINES → DATAROBOT_CLI_FEATURE_PIPELINE
to match the registered command name and feature gate key.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@sunny2get sunny2get closed this May 28, 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.

4 participants