Faas: Rename sample function names to lowercase

[alexellis]

The current set of function names in the samples.yml file are invalid, so need to be lowercase to be deployable on Swarm/K8s. Also replace _ for - for DNS validation.

Testing:

This should yield no errors:

faas-cli build -f samples.yml
faas-cli deploy -f samples.yml

https://github.com/openfaas/faas/blob/master/sample-functions/samples.yml

[alexellis]

Derek add label: helpwanted

[riktimmondal]

i am taking it

[vijaygomathinayagam]

don't know how to run these commands

faas-cli build -f samples.yml
faas-cli deploy -f samples.yml

[rgee0]

@VijayGomathi Can you give us a bit more info? How have you tried so far? What message do you get when trying to run them?

[vijaygomathinayagam]

@rgee0 I tried installing faas-cli via npm. but i could not run the command. I am getting the following error message.
"'faas-cli' is not recognized as an internal or external command,
operable program or batch file."

I also tried installing faas-cli package globally in npm. Still getting the same error

[rgee0]

@VijayGomathi please try $ curl -sSL https://cli.openfaas.com | sudo sh

[vijaygomathinayagam]

@rgee0 I am using windows operating system

[rgee0]

https://github.com/openfaas/faas-cli/releases

[vijaygomathinayagam]

@rgee0 now I'm getting this error. Please help.

[alexellis]

You can't use Windows containers for this script.. switch to Linux containers.

[GrantSheehan]

I'm not sure what was meant by Also replace _ for - for DNS validation., maybe those replacements were made in an earlier merged PR? If my branch is missing what's meant by that, just let me know what exactly needs to change and I'll fix it as well.

[alexellis]

Fixed by @GrantSheehan - thank you. @VijayGomathi thank you for taking a shot at this.