Skip to content

[Test] Handle exceptions from non-blocking blocks#3136

Merged
himani2411 merged 2 commits intoaws:developfrom
himani2411:develop-fsx
Mar 10, 2026
Merged

[Test] Handle exceptions from non-blocking blocks#3136
himani2411 merged 2 commits intoaws:developfrom
himani2411:develop-fsx

Conversation

@himani2411
Copy link
Contributor

@himani2411 himani2411 commented Mar 6, 2026

Description of changes

The rescue nil means the before block still attempts to warm up the yum cache, but if it fails, the test proceeds anyway and lets InSpec's own internal yum -v repolist all call handle it.

Our tests fails with below


2026-03-06 00:34:13.640000  Stdout: �[38;5;9m  ×  tag:install_lustre_client_installed: Verify that lustre client is installed (1 failed)�[0m
2026-03-06 00:34:13.640000  Stdout: �[38;5;41m     ✔  System Package kmod-lustre-client is expected to be installed�[0m
2026-03-06 00:34:13.640000  Stdout: �[38;5;41m     ✔  System Package lustre-client is expected to be installed�[0m
2026-03-06 00:34:13.640000  Stdout: �[38;5;41m     ✔  System Package kmod-lustre-client version is expected to cmp >= "2.12"�[0m
2026-03-06 00:34:13.640000  Stdout: �[38;5;41m     ✔  System Package lustre-client version is expected to cmp >= "2.12"�[0m
2026-03-06 00:34:13.640000  Stdout: �[38;5;9m     ×  YumRepo aws-fsx is expected to exist
2026-03-06 00:34:13.640000  Stdout:      Command `yum -v repolist all` timed out after  seconds�[0m
2026-03-06 00:34:13.640000  Stdout: �[38;5;41m     ✔  YumRepo aws-fsx is expected to be enabled�[0m
2026-03-06 00:34:13.640000  Stdout: �[38;5;41m     ✔  YumRepo aws-fsx baseurl is expected to include "fsx-lustre-client-repo.s3.amazonaws.com"�[0m
2026-03-06 00:34:13.640000  Stdout: �[38;5;41m  ✔  tag:install_lustre_lnet_kernel_module_enabled: Verify that lnet kernel module is enabled�[0m
2026-03-06 00:34:13.640000  Stdout: �[38;5;41m     ✔  Kernel Module lnet is expected to be loaded�[0m
2026-03-06 00:34:13.640000  Stdout: �[38;5;41m     ✔  Kernel Module lnet is expected not to be disabled�[0m

References

  • Link to impacted open issues.
  • Link to related PRs in other packages (i.e. cookbook, node).
  • Link to documentation useful to understand the changes.

Checklist

  • Make sure you are pointing to the right branch.
  • If you're creating a patch for a branch other than develop add the branch name as prefix in the PR title (e.g. [release-3.6]).
  • Check all commits' messages are clear, describing what and why vs how.
  • Make sure to have added unit tests or integration tests to cover the new/modified code.
  • Check if documentation is impacted by this change.

Please review the guidelines for contributing and Pull Request Instructions.

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

@himani2411 himani2411 marked this pull request as ready for review March 6, 2026 19:48
@himani2411 himani2411 requested review from a team as code owners March 6, 2026 19:48
…p the yum cache, but if it fails, the test proceeds anyway and lets InSpec's own

internal yum -v repolist all call handle it.
@himani2411 himani2411 enabled auto-merge (rebase) March 6, 2026 21:55
@himani2411 himani2411 merged commit bde54d9 into aws:develop Mar 10, 2026
28 of 30 checks passed
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