Skip to content
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

wsk activation poll needs more robust name matching #2274

Closed
cfjedimaster opened this issue May 22, 2017 · 4 comments
Closed

wsk activation poll needs more robust name matching #2274

cfjedimaster opened this issue May 22, 2017 · 4 comments

Comments

@cfjedimaster
Copy link
Contributor

I filed a bug report on "wsk activation poll -h" not making it clear that it can match by name as well, but I want to expand on the feature in general. It feels like the name matching isn't very robust. It's also possible I'm not doing this right, but this is where better docs would help.

I've got an action called makeresult that lives in the serverless_superman package. The only way I can see to poll for it is to poll JUST for 'makeresult'. If I try /serverless_superman/makeresult, it will not match. If I've got 10 packages with makeresult in it, then I'll get a lot of noise back.

If I need to also provide a namespace, than I'd consider that a bug as well. It would work, but the CLI should just default to my namespace anyway. (And again, it should be documented.)

Also, there seems to be no way to get everything called within one package. (Again, unless I'm running it wrong.)

@rabbah
Copy link
Member

rabbah commented May 22, 2017

confirmed bug - you can name an action in the default package but not an action in a named package.

it may be possible to support polling on all actions in a package as part of the same fix.

@dubee
Copy link
Member

dubee commented Jun 22, 2017

I had a PR open to update the activation poll usage, but was informed that activation IDs may not apply to rules and triggers in the future. I can update the usage to just mention actions though. See: #2398.

@dubee
Copy link
Member

dubee commented Jun 22, 2017

Linked the PR to the proper issue.

@csantanapr
Copy link
Member

Issue moved to apache/incubator-openwhisk-cli #133 via ZenHub

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants