Skip to content

Conversation

@gmarciani
Copy link
Contributor

@gmarciani gmarciani commented Dec 12, 2025

Description of changes

Mock cookbook virtualenv for Docker tests on AL 2023.
This is required to execute system tests on Docker for AL 2023 that depend on the cookbook virtualenv, such as the CloudWatch Agent tests.

This change fixes the PR check failure on AL23 (example:

       [2025-12-12T20:22:32+00:00] FATAL: Errno::ENOENT: cloudwatch[Configure CloudWatch] (aws-parallelcluster-environment::init line 16) had an error: Errno::ENOENT: execute[cloudwatch-config-validation] (aws-parallelcluster-environment::init line 156) had an error: Errno::ENOENT: No such file or directory - /opt/parallelcluster/pyenv/versions/3.12.11/envs/cookbook_virtualenv/bin/python
>>>>>> ------Exception-------
>>>>>> Class: Kitchen::ActionFailed
>>>>>> Message: 1 actions failed.
>>>>>>     Converge failed on instance <slurm-config-head-node-x86-64-docker-alinux-2023>.  Please see .kitchen/logs/slurm-config-head-node-x86-64-docker-alinux-2023.log for more details

NOTE The recursive-deletion-check was reporting a false positive, so it is safe to skip it. It was complaining because in the Docker mock I am creating the directory tree with option recursive true. The check complains about it because it thinks it is used for deletion, but in this case it is used for creation, which is fine.

Tests

  • PR checks.

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

@gmarciani gmarciani requested review from a team as code owners December 12, 2025 20:58
@gmarciani gmarciani force-pushed the wip/mgiacomo/3150/fix-pr-system-tests-al2023-1212-1 branch from 1cf7dfb to 4976649 Compare December 12, 2025 21:07
@gmarciani gmarciani changed the title [Test] Skip CloudWatch config on AL2023 Docker containers to prevent … [Test] Mock cookbook virtualenv for Docker tests on AL 2023. Dec 12, 2025
@gmarciani gmarciani added the skip-recursive-deletion-check Skip the checks regarding the use of recursive deletion. label Dec 12, 2025
@gmarciani gmarciani force-pushed the wip/mgiacomo/3150/fix-pr-system-tests-al2023-1212-1 branch 4 times, most recently from eb7620b to 3ad7d3f Compare December 12, 2025 22:40
This is required to execute system tests on Docker for AL 2023 that depend on the cookbook virtualenv, such as the CloudWatch Agent tests.
@gmarciani gmarciani force-pushed the wip/mgiacomo/3150/fix-pr-system-tests-al2023-1212-1 branch from 3ad7d3f to a7d270b Compare December 12, 2025 23:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

3.x skip-changelog-update skip-recursive-deletion-check Skip the checks regarding the use of recursive deletion. Test

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant