Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

[ansible]: upgrade to use ansible 2.8.7 #1213

Merged
merged 8 commits into from
Nov 20, 2019
Merged

Conversation

lguohan
Copy link
Contributor

@lguohan lguohan commented Nov 18, 2019

Description of PR

Summary:
Fixes # (issue)

Type of change

  • [] Bug fix
  • Testbed and Framework(new/improvement)
  • [] Test case(new/improvement)

Approach

How did you do it?

How did you verify/test it?

I am not able to verify every test. Here are things I verified.

  • start-vms/stop-vms/add-topo/remove-top/refrersh-dut
  • deploy-mg
  • test_bgp_facts

depends on sonic-net/sonic-buildimage#3776

Any platform specific information?

Supported testbed topology if it's a new test case?

Documentation

- remove docker-py 1.7.2
- remove library/docker.py
- use docker-login and docker-container module from ansible
- adapt to docker 4.0.2 in vm_topology.py

requires ansible 2.8.7
Signed-off-by: Guohan Lu <gulv@microsoft.com>
to_unicode is deprecated and removed starting ansible 2.5
Signed-off-by: Guohan Lu <gulv@microsoft.com>
use listen for "Update VM state"

You cannot notify a handler that is defined inside of an include.

Signed-off-by: Guohan Lu <gulv@microsoft.com>
ansible changes

Signed-off-by: Guohan Lu <gulv@microsoft.com>
Signed-off-by: Guohan Lu <gulv@microsoft.com>
@lguohan lguohan marked this pull request as ready for review November 20, 2019 05:51
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant