Operator-sdk: Default ansible operator Travisfile doesn't work - jmespath error

[geerlingguy]

Bug Report

What did you do?

1. I created a new operator (using the operator-sdk cli command in the documentation).
2. I pushed the generated code to a new GitHub repository.
3. I waited for the Travis CI build to finish.

What did you expect to see?

I expected the default test would pass.

What did you see instead? Under which circumstances?

I received the error:

TASK [Wait 60s for reconciliation to run] **************************************
fatal: [localhost]: FAILED! => {"msg": "The conditional check ''Successful' in (cr | json_query('resources[].status.conditions[].reason'))' failed. The error was: You need to install \"jmespath\" prior to running json_query filter"}

Environment

• operator-sdk version:

$ operator-sdk version
operator-sdk version: v0.10.0, commit: ff80b17737a6a0aade663e4827e8af3ab5a21170

• go version:

N/A

• Kubernetes version information:

N/A

• Kubernetes cluster kind:

N/A

Possible Solution

The default .travis.yml file included with the Ansible operator scaffold should include jmespath in its dependency installation, e.g.:

install:
  - pip install docker molecule openshift jmespath

Currently it does not include jmespath.

Additional context

N/A

[camilamacedo86]

/assign @camilamacedo86

Hi @geerlingguy,

Thank you for raise it. We will be looking at that. Did you try your solution? Did it work?
Anyway, I will be checking it as soon as possible.

[geerlingguy]

@camilamacedo86 - Just pushed up a PR to fix: https://github.com/operator-framework/operator-sdk/pull/2027

I can confirm that adding that dependency fixes the issue, as seen in the difference between the #2 and #3 builds in Travis CI here: https://travis-ci.com/geerlingguy/mariadb-operator/builds

[camilamacedo86]

it is great @geerlingguy !! Really tks for your contribution.

[camilamacedo86]

Also, wyt about doing a PR to fix the sample here as well?

[geerlingguy]

@camilamacedo86 - sure, will do!

[geerlingguy]

See: https://github.com/operator-framework/operator-sdk-samples/pull/76