Skip to content
This repository has been archived by the owner on Oct 11, 2023. It is now read-only.

ros config merge broken from stdin on v1.5.1 #2689

Closed
riaan53 opened this issue Feb 26, 2019 · 2 comments
Closed

ros config merge broken from stdin on v1.5.1 #2689

riaan53 opened this issue Feb 26, 2019 · 2 comments

Comments

@riaan53
Copy link

riaan53 commented Feb 26, 2019

RancherOS Version: (ros os version)
ros v1.5.1

Where are you running RancherOS? (docker-machine, AWS, GCE, baremetal, etc.)
virtualbox

On ros v1.5.0 I can pipe in ros config like this:

cat somefile.yaml | ros config merge

But on ros v1.5.1 I get the following error for the exact same command:

FATAL input parameter cannot be empty

It still works when loading the file via the -i argument but we are doing it programatically and writing a file first is not ideal.

@niusmallnan
Copy link
Contributor

@kingsd041 Please add this function to our validation tests.

@rootwuj
Copy link

rootwuj commented May 17, 2019

Tested with rancher/os:v1.5.2-rc1 from May 17
This version supports the ros config merge command.
Verified fixed

@rootwuj rootwuj closed this as completed May 17, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

5 participants