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

[MANOPD-81458] - update paramiko version #316

Merged
merged 1 commit into from
Dec 23, 2022
Merged

[MANOPD-81458] - update paramiko version #316

merged 1 commit into from
Dec 23, 2022

Conversation

Imadzuma
Copy link
Contributor

Description

Connect to Ubuntu 22.04 via old paramiko version follows exception:

2022-12-21 14:30:24,212 test-mame-local-k8s.com DEBUG Start detecting nodes context...
2022-12-21 14:30:54,848 test-mame-local-k8s.com CRITICAL FAILURE!
2022-12-21 14:30:54,849 test-mame-local-k8s.com CRITICAL Failed to proceed inventory file
2022-12-21 14:30:54,849 test-mame-local-k8s.com CRITICAL KME0001: Unexpected exception
Traceback (most recent call last):
  File "/mnt/c/Users/mame0719/Documents/Projects/PaaS/github/KubeMarine/kubemarine/core/resources.py", line 172, in _get_nodes_context
    self._nodes_context = light_cluster.detect_nodes_context()
  File "/mnt/c/Users/mame0719/Documents/Projects/PaaS/github/KubeMarine/kubemarine/core/cluster.py", line 175, in detect_nodes_context
    self._check_accessible_nodes()
  File "/mnt/c/Users/mame0719/Documents/Projects/PaaS/github/KubeMarine/kubemarine/core/cluster.py", line 210, in _check_accessible_nodes
    raise Exception(f"{not_accessible_online.get_hosts()} are not accessible through ssh. "
Exception: ['192.168.56.201', '192.168.56.202'] are not accessible through ssh. Check ssh credentials.

Solution

Worked paramiko version was added in requirements.

Test Cases

Please describe the tests that you ran to verify your changes. Provide instructions so we can reproduce. Please also list any relevant details for your test configuration.

TestCase 1

Test Configuration:

  • OS: Ubuntu 22.04
  • Paramiko version: 2.6.0

Steps:

  1. kubemarine install or other procedure with connection to nodes.

Results:

Before After
Procedure fails because of ssh connectivity Connection to node is successful

Checklist

  • I have commented my code, particularly in hard-to-understand areas
  • I have made corresponding changes to the documentation
  • Integration CI passed
  • Unit tests. If Yes list of new/changed tests with brief description
  • There is no merge conflicts

@Imadzuma Imadzuma requested a review from koryaga December 21, 2022 13:22
@koryaga koryaga self-assigned this Dec 22, 2022
@koryaga koryaga added this to the 0.11.1 milestone Dec 22, 2022
@koryaga koryaga requested a review from alku89 December 22, 2022 08:32
@koryaga koryaga merged commit 996039a into main Dec 23, 2022
@koryaga koryaga deleted the update-paramiko branch December 23, 2022 08:55
dmyar21 pushed a commit that referenced this pull request Jan 19, 2023
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.

3 participants