-
Notifications
You must be signed in to change notification settings - Fork 223
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Question: generating image tags from Git #955
Comments
When deploying from the store, any --env variables passed or defined for a function were being ignored due to a bug. This has been fixed and tested. Signed-off-by: Alex Ellis (OpenFaaS Ltd) <[email protected]>
/set title: Question: generating image tags from Git |
Hi thanks for your interest in OpenFaaS Community Edition. I think this may be the first time we've heard about you using OpenFaaS CE as part of your business for your customers. Could you send us a note to the ADOPTERS.md file? One thing that looks inefficient is your use of the faas-cli as a container instead of as a binary on the host. It's probably why the tag is failing to resolve, because the .git directory isn't present or in context. You can find our recommended GitHub Action here: https://docs.actuated.dev/examples/openfaas-publish/#try-out-the-action-on-your-agent I'd also suggest that since you're using OpenFaaS CE commercially, and I assume want us to continue working on it for your benefit and your customers' benefit, that you review the sponsorship tiers to see what makes sense for you, custom amounts are also enabled. Also it's worth noting that you may to be leaking the Thanks Alex |
One other thing I noticed is that you don't use faas-cli to generate the hash, so I'm going to close this issue. You're using GitHub Actions to get the hash, and then you're also using the deprecated set-output directive. Let us know if we can help further. |
Thank you @alexellis for the prompt response! Sorry I wasn't able to reply earlier. Please find my comments below.
What you say makes total sense however, the "docker way" used to work before moving from docker hub to GHCR. I really don't know why 🤷♂️ . Having said that, I did follow your recommendation and now I'm installing Bottomline all is working now! Thanks again!! |
There wasn't a change with the CLI, it's still working the same way. There was a deprecation in GitHub Actions in which they've removed "set-output" as a valid way to set an environment variable. You need to migrate it to use a similar pattern to what we do in the linked docs. Take a look at the sponsorship options, even if you think 25 USD / mo is the only value you get from our continued work, it all helps contribute to help all the free users like yourself who use the community version to make a business profit. Alex |
Expected Behaviour
I'm expecting to run
faas-cli deploy
to successfully deploy my function (pods, replicaset, etc)Current Behaviour
My pod is not deployed and instead I get the following error
Why do you need this?
I need to be able to deploy newer versions of my functions. Something to note is that this all started happening a few days ago after
faas-cli
docker image was removed from Docker Hub and we started pulling the image from ghcr.io instead.It seems that, for some reason, the commit SHA is not properly determined or read and ends up rendering something as
-fatal
instead of-<COMMIT-SHA>
Who is this for?
What company is this for? Are you listed in the ADOPTERS.md file?
I'm using this with my customers.
And no, I don't seem to be listed in the adopters.md file
Are you a GitHub Sponsor (Yes/No?)
Check at: https://github.com/sponsors/openfaas
List All Possible Solutions and Workarounds
I'm not sure yet 🤷♂️
Which Solution Do You Recommend?
Honestly I've been using
faas-cli
without an issue. I'm definitely not an expert user. I have never committed any code to the project. I'm only using the "basic" out-of-the-box commandslogin
,build
,deploy
Steps to Reproduce (for bugs)
I'm using
faas-cli
inside a Github Action but not sure it is really a bug onfaas-cli
.Context
Your Environment
I'm using this within Github Actions using
ubuntu-latest
(see here)faas-cli version
):0.16.1
docker version
):docker://20.10.13
No
Linux
The text was updated successfully, but these errors were encountered: