GoogleContainerTools/skaffold

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

Open

#6,137 opened on Jul 6, 2021

View on GitHubContributor guide
 (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
 (PR metrics pending)

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