GoogleContainerTools/skaffold

skaffold apply - support manifests on stdin, similar to kubectl apply -f -

Open

#6,137 opened on Jul 6, 2021

View on GitHub
 (2 comments) (2 reactions) (1 assignee)Go (1,416 forks)batch import
area/applygood first issuehelp wantedkind/feature-requestpriority/p2triage/needs-information

Repository metrics

Stars
 (12,822 stars)
PR merge metrics
 (Avg merge 3d 6h) (16 merged PRs in 30d)

Description

Please add support to skaffold apply for using Kubernetes manifests from stdin, similar to what kubectl apply -f - does.

Obviously it'd be nice to support the same -f - are that kubectl does but I can't due to that already being taken for -f skaffold.yaml, but could still support accepting k8s manifests from stdin if no file args are given.

The current workaround of outputting to a file and then supplying a file arg is ok, but accepting from stdin would be nice to have too as a more unix-like behaviour to support CLI pipes.

Expected behavior

skaffold render |
skaffold apply

Actual behavior

skaffold render |
skaffold apply 
`apply` requires at least one manifest argument
write /dev/stdout: broken pipe

Information

  • Skaffold version: 1.27.0
  • Operating system: Mac
  • Installed via: Homebrew

Contributor guide

Research direction
Modify the CLI argument parsing for `skaffold apply` to accept manifests from stdin when no file arguments are provided. Look at how `kubectl apply f ` works and implement similar stdin reading logic in Go using `os.Stdin`. Ensure the change works with pipes and does not break existing file based usage.
Tech stack
go
Domain
backend
Issue type
Feature
DifficultyEstimated implementation difficulty for a new contributor, from 1 for very small changes to 5 for expert-level work.
3
Estimated timeA rough time range for an experienced contributor to investigate, implement, test, and prepare a pull request.
1-3 hours
Activity statusHow available the issue appears right now: fresh, active, stale, blocked, or waiting on maintainer input.
Active
ClarityHow clearly the issue explains the expected change, acceptance criteria, and next step.
Clear
Prerequisites
GoKubernetes
Newbie friendlinessA 1-100 score estimating how approachable this issue is for first-time contributors.
70