cetic / helm-nifi

Helm Chart for Apache Nifi
Apache License 2.0
215 stars 228 forks source link

At test time, confirm web service is started and responding with expe… #210

Closed wknickless closed 2 years ago

wknickless commented 2 years ago

What this PR does / why we need it:

The master branch push and pull_request workflows will now actually watch the logs to confirm the NiFi web application has started, and that it returns the expected content.

Special notes for your reviewer:

@banzo @zakaria2905 please merge this soon; tomorrow I'm going to work on another pull request to address issue #208 and hope to include tests specific to confirming those fixes. This PR does NOT resolve that issue, but it would be nice to have these changes in the master branch so I can build the next pull requests' additional tests on top of it.

Checklist

[Place an '[x]' (no spaces) in all applicable fields. Please remove unrelated fields.]