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

chore: ansible improvements #254

Merged
merged 11 commits into from
Aug 24, 2021
Merged

chore: ansible improvements #254

merged 11 commits into from
Aug 24, 2021

Conversation

strophy
Copy link
Collaborator

@strophy strophy commented Aug 20, 2021

Issue being fixed or feature implemented

Errors and deprecation warnings were being thrown in ansible:

  • Error processing new output from Terraform 1.0.5 update
  • Error while testing Insight container status
  • General difficulty reading errors
  • Deprecation warning about Ansible docker_container community plugin syntax
  • Deprecated package warnings
  • Linter warnings

What was done?

  • Fix Terraform output
  • Register variable before test and test correct variable
  • Format output
  • Specify future behaviour after default behaviour deprecation
  • Update some packages and linter to resolve build warnings

How Has This Been Tested?

Tested deploying devnet-schnapps with latest Terraform (1.0.5) and Ansible (2.11.4)

Breaking Changes

None

Checklist:

  • I have performed a self-review of my own code
  • I have commented my code, particularly in hard-to-understand areas
  • I have added or updated relevant unit/integration/functional/e2e tests
  • I have made corresponding changes to the documentation

For repository code-owners and collaborators only

  • I have assigned this pull request to a milestone

@strophy strophy marked this pull request as ready for review August 21, 2021 09:32
@strophy strophy requested a review from shumkov August 21, 2021 09:32
Copy link
Member

@shumkov shumkov left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM! 👍

Copy link
Member

@shumkov shumkov left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We need to update docker file and probably ansible/requirements

@strophy
Copy link
Collaborator Author

strophy commented Aug 23, 2021

Dockerfile does not need updating because Ansible is installed from pip, which always has the latest versions. Ansible itself will update rapidly (mostly due to rolling release on 3 week schedule to always have up-to-date included community plugins) but ansible-core follows a more conservative release schedule and will warn us through warnings and deprecations for many months/years before failing.

Ansible's requirements.txt has been updated and tested on devnet-schnapps. Elastic was not updated yet due to licensing uncertainty and because 7.11.1 (last fully open source version) is still working fine, maybe we can do this separately if it becomes necssary.

@strophy strophy requested a review from shumkov August 23, 2021 11:49
Copy link
Member

@shumkov shumkov left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I see. Thank you. Then it looks good to me!

@strophy strophy added this to the v0.21.0 milestone Aug 24, 2021
@strophy strophy merged commit d155b41 into v0.21-dev Aug 24, 2021
@strophy strophy deleted the ansible-improvements branch August 24, 2021 12:39
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.

2 participants