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 Paramiko version #170

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

alex-pimperton
Copy link

@alex-pimperton alex-pimperton commented May 26, 2022

SUMMARY

Paramiko 2.9+ breaks SSH connectivity to older hosts, cisco.ios collection has already downgraded Paramiko to fix:

https://github.com/ansible-collections/cisco.ios/blob/main/requirements.txt

Bug here:

ansible/ansible#76737 (comment)

ISSUE TYPE
  • Bugfix Pull Request
COMPONENT NAME
ADDITIONAL INFORMATION

@softwarefactory-project-zuul
Copy link

Build succeeded.

✔️ ansible-galaxy-importer SUCCESS in 3m 57s
✔️ build-ansible-collection SUCCESS in 3m 41s
ansible-test-network-integration-asa-python39 FAILURE in 7m 35s (non-voting)
ansible-test-network-integration-asa-python38-stable212 FAILURE in 8m 37s (non-voting)
✔️ ansible-test-network-integration-asa-python38-stable211 SUCCESS in 14m 33s (non-voting)
ansible-test-network-integration-asa-python38-stable29 FAILURE in 8m 46s (non-voting)
✔️ ansible-test-network-integration-asa-libssh-python39 SUCCESS in 15m 44s
✔️ ansible-test-network-integration-asa-libssh-python38-stable212 SUCCESS in 14m 01s
✔️ ansible-test-network-integration-asa-libssh-python38-stable211 SUCCESS in 15m 05s
ansible-test-network-integration-asa-libssh-python38-stable29 FAILURE in 14m 57s (non-voting)
✔️ ansible-test-sanity-docker-devel SUCCESS in 9m 48s (non-voting)
✔️ ansible-test-sanity-docker-milestone SUCCESS in 11m 15s
✔️ ansible-test-sanity-docker-stable-2.9 SUCCESS in 10m 45s
✔️ ansible-test-sanity-docker-stable-2.10 SUCCESS in 9m 44s
✔️ ansible-test-sanity-docker-stable-2.11 SUCCESS in 8m 19s
✔️ ansible-test-sanity-docker-stable-2.12 SUCCESS in 8m 38s
✔️ ansible-test-units-asa-python39 SUCCESS in 5m 56s
✔️ ansible-test-units-asa-python36 SUCCESS in 5m 43s
✔️ ansible-test-units-asa-python37 SUCCESS in 7m 10s
✔️ ansible-tox-linters SUCCESS in 4m 56s

@KB-perByte KB-perByte requested a review from justjais June 1, 2022 13:08
@justjais justjais closed this Oct 12, 2022
@justjais justjais reopened this Oct 12, 2022
@justjais
Copy link
Member

@alex-pimperton Thanks for raising the PR, can you plz fix the test failures for us to go ahead with the PR merge

@softwarefactory-project-zuul
Copy link

Build succeeded.

✔️ ansible-galaxy-importer SUCCESS in 4m 05s
✔️ build-ansible-collection SUCCESS in 3m 58s
ansible-test-network-integration-asa-python39 FAILURE in 8m 13s (non-voting)
ansible-test-network-integration-asa-python38-stable212 FAILURE in 8m 19s (non-voting)
ansible-test-network-integration-asa-python38-stable214 FAILURE in 7m 36s (non-voting)
✔️ ansible-test-network-integration-asa-python38-stable211 SUCCESS in 12m 35s (non-voting)
ansible-test-network-integration-asa-python38-stable29 FAILURE in 8m 59s (non-voting)
✔️ ansible-test-network-integration-asa-libssh-python39 SUCCESS in 12m 43s
✔️ ansible-test-network-integration-asa-libssh-python38-stable214 SUCCESS in 12m 53s
✔️ ansible-test-network-integration-asa-libssh-python38-stable212 SUCCESS in 12m 26s
✔️ ansible-test-network-integration-asa-libssh-python38-stable211 SUCCESS in 12m 34s
ansible-test-network-integration-asa-libssh-python38-stable29 FAILURE in 13m 25s (non-voting)
ansible-test-sanity-docker-devel FAILURE in 10m 30s (non-voting)
✔️ ansible-test-sanity-docker-milestone SUCCESS in 8m 22s
✔️ ansible-test-sanity-docker-stable-2.9 SUCCESS in 11m 41s
✔️ ansible-test-sanity-docker-stable-2.10 SUCCESS in 13m 01s
✔️ ansible-test-sanity-docker-stable-2.11 SUCCESS in 10m 35s
✔️ ansible-test-sanity-docker-stable-2.12 SUCCESS in 9m 02s
ansible-test-sanity-docker-stable-2.14 FAILURE in 9m 17s (non-voting)
✔️ ansible-test-units-asa-python39 SUCCESS in 6m 25s
✔️ ansible-test-units-asa-python36 SUCCESS in 6m 05s
✔️ ansible-test-units-asa-python37 SUCCESS in 6m 34s
✔️ ansible-tox-linters SUCCESS in 5m 22s

Paramiko 2.9+ breaks SSH connectivity to older hosts, cisco.ios collection has already downgraded Paramiko to fix:

https://github.com/ansible-collections/cisco.ios/blob/main/requirements.txt

Bug here:

ansible/ansible#76737 (comment)
@softwarefactory-project-zuul
Copy link

Build succeeded.
https://ansible.softwarefactory-project.io/zuul/buildset/5a49fd746fdc43cc8921468717e68df1

✔️ ansible-galaxy-importer SUCCESS in 3m 18s
✔️ build-ansible-collection SUCCESS in 8m 36s
ansible-test-network-integration-asa-python39 FAILURE in 7m 34s (non-voting)
ansible-test-network-integration-asa-python38-stable212 FAILURE in 7m 29s (non-voting)
ansible-test-network-integration-asa-python38-stable214 FAILURE in 6m 41s (non-voting)
✔️ ansible-test-network-integration-asa-python38-stable211 SUCCESS in 11m 53s (non-voting)
ansible-test-network-integration-asa-python38-stable29 FAILURE in 10m 17s (non-voting)
✔️ ansible-test-network-integration-asa-libssh-python39 SUCCESS in 10m 59s
✔️ ansible-test-network-integration-asa-libssh-python38-stable214 SUCCESS in 13m 01s
✔️ ansible-test-network-integration-asa-libssh-python38-stable212 SUCCESS in 12m 44s
✔️ ansible-test-network-integration-asa-libssh-python38-stable211 SUCCESS in 12m 14s
ansible-test-network-integration-asa-libssh-python38-stable29 FAILURE in 12m 19s (non-voting)
✔️ ansible-test-sanity-docker-devel SUCCESS in 7m 56s (non-voting)
✔️ ansible-test-sanity-docker-milestone SUCCESS in 10m 01s (non-voting)
✔️ ansible-test-sanity-docker-stable-2.9 SUCCESS in 10m 04s
✔️ ansible-test-sanity-docker-stable-2.10 SUCCESS in 8m 29s
✔️ ansible-test-sanity-docker-stable-2.11 SUCCESS in 9m 43s
✔️ ansible-test-sanity-docker-stable-2.12 SUCCESS in 7m 38s
✔️ ansible-test-sanity-docker-stable-2.14 SUCCESS in 9m 39s
✔️ ansible-test-units-asa-python39 SUCCESS in 7m 12s
✔️ ansible-test-units-asa-python36 SUCCESS in 5m 58s
✔️ ansible-test-units-asa-python37 SUCCESS in 6m 24s
✔️ ansible-tox-linters SUCCESS in 10m 25s

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.

None yet

2 participants