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

Fixed typo #2

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

AlbertoSH
Copy link

@AlbertoSH AlbertoSH commented Jul 2, 2019

Whenever you try to discover the istio probes & actions, there's a typo that causes an error

$ # before fix
$ chaos discover chaostoolkit-istio
[2019-07-02 12:14:42 INFO] Attempting to download and install package 'chaostoolkit-istio'
[2019-07-02 12:14:43 INFO] Package downloaded and installed in current environment
[2019-07-02 12:14:44 INFO] Discovering capabilities from chaostoolkit-istio
[2019-07-02 12:14:44 INFO] Searching for actions
[2019-07-02 12:14:44 INFO] Searching for probes
[2019-07-02 12:14:44 CRITICAL] could not import extension module 'chaosisio.fault.probes'

$ # after fix
$ chaos discover chaostoolkit-istio
[2019-07-02 12:15:53 INFO] Attempting to download and install package 'chaostoolkit-istio'
[2019-07-02 12:15:54 INFO] Package downloaded and installed in current environment
[2019-07-02 12:15:54 INFO] Discovering capabilities from chaostoolkit-istio
[2019-07-02 12:15:54 INFO] Searching for actions
[2019-07-02 12:15:54 INFO] Searching for probes
[2019-07-02 12:15:55 INFO] Discovery outcome saved in ./discovery.json

Signed-off-by: AlbertoSH [email protected]

Signed-off-by: AlbertoSH <[email protected]>
@Lawouach
Copy link
Contributor

Gah! I forgot to include your PR :'(

Do you mind if I simply fix the typo myself?

Lawouach added a commit that referenced this pull request Nov 13, 2019
From #2 by @AlbertoSH

Signed-off-by: Sylvain Hellegouarch <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants