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

Fix to resolve a broken master build #15

Merged
merged 2 commits into from
May 31, 2019
Merged

Fix to resolve a broken master build #15

merged 2 commits into from
May 31, 2019

Conversation

jvanderhoof
Copy link
Contributor

What does this PR do? Is there any background context you want to provide?

This PR:

  • Fixes the broken build on the master branch
  • Updates the Ansible version to one without a major security vulnerability (and insure we're testing with the modern version of Ansible)

What ticket does this PR close?

No ticket filed.

Where should the reviewer start? How should this functionality be validated?

The changes are simple. test/docker-compose.yml has the most change.

Tests are now passing: https://jenkins.conjur.net/job/cyberark--ansible-conjur-host-identity/job/cli-fix/, which resolves the primary issue.

Links to open issues for related documentation (in READMEs, docs, etc)

N/A

Links to open issues for related automated integration and unit tests

N/A

Have the Version and Changelog been updated?

N/A - there is no functional change.

@garymoon garymoon merged commit babac72 into master May 31, 2019
@garymoon garymoon deleted the cli-fix branch May 31, 2019 17:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants