-
Notifications
You must be signed in to change notification settings - Fork 10
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
Cron-Connector only looks for functions in openfaas-fn namespace #6
Comments
Thanks for your interest Are you willing to work on this feature? If it's for a commercial user of openfaas, check out this thread -> openfaas/faas#1591 Alex |
Sure.
This might be a bigger change than expected. I already looked at the
dependencies, and it looks like several of the sdk dependencies are
outdated. So might take some more work than just adding a loop going
through all the namespaces.
Should the upgrade of the dependencies be a separate issue?
Thanks,
Joris
…On Fri, Nov 13, 2020 at 2:05 PM Alex Ellis ***@***.***> wrote:
Thanks for your interest
Are you willing to work on this feature?
If it's for a commercial user of openfaas, check out this thread ->
openfaas/faas#1591 <openfaas/faas#1591>
Alex
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
<#6 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AAMBRNCUYRFZNU5LKFH6M3DSPWUSBANCNFSM4TVC3U4A>
.
|
See above PR. Open for feedback! |
Thanks for contributing. I've asked @viveksyngh to take a look and help you. |
Hey @jdewinne, is this still in progress? |
Should be working. @alexellis can this be merged? |
@jdewinne this is an issue, please comment on the linked PR |
Signed-off-by: Joris De Winne <[email protected]>
PR #7 raised for this |
The cron-connector should get all the functions for all the namespaces, by first invoking
getNamespaces
and for each namespace getting all the functions.https://github.com/openfaas/cron-connector/blob/master/main.go#L66
Related to alpha feature namespaces
The text was updated successfully, but these errors were encountered: