Skip to content

Conversation

@victorusu
Copy link
Contributor

This test checks the ssh host keys age. The list of host keys and the maximum age allowed are parameters and variables of the test.

If the key file is not found, the test is skipped

This test checks the ssh host keys age. The list of host keys and the
maximum age allowed are parameters and variables of the test.

If the key file is not found, the test is skipped
@victorusu victorusu added this to the ReFrame 4.4 milestone Jul 21, 2023
@victorusu victorusu requested review from ekouts, teojgo and vkarak July 21, 2023 12:06
@victorusu victorusu self-assigned this Jul 21, 2023
@codecov
Copy link

codecov bot commented Jul 21, 2023

Codecov Report

Patch and project coverage have no change.

Comparison is base (6a8ee3a) 87.36% compared to head (20201d7) 87.36%.

Additional details and impacted files
@@           Coverage Diff            @@
##           develop    #2957   +/-   ##
========================================
  Coverage    87.36%   87.36%           
========================================
  Files           60       60           
  Lines        11687    11687           
========================================
  Hits         10210    10210           
  Misses        1477     1477           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@vkarak vkarak changed the title Add host_keys_check test [testlib] Add host_keys_check test Jul 25, 2023
Copy link
Contributor

@vkarak vkarak left a comment

Choose a reason for hiding this comment

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

Lgtm

@vkarak vkarak changed the title [testlib] Add host_keys_check test [testlib] New test for testing SSH host keys Aug 22, 2023
@vkarak
Copy link
Contributor

vkarak commented Aug 22, 2023

You should also add the corresponding docs, but this is also mentioned in #2956, which can include the rest of the docs. I will merge this.

@vkarak vkarak merged commit eda6b23 into reframe-hpc:develop Aug 22, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants