ci: update integration and coverage workflow python version#190
ci: update integration and coverage workflow python version#190brahmanim wants to merge 14 commits into
Conversation
077a5ae to
124720d
Compare
124720d to
cc939fe
Compare
135348c to
4d992dd
Compare
| cancel-in-progress: true | ||
|
|
||
| on: # yamllint disable-line rule:truthy | ||
| pull_request: |
There was a problem hiding this comment.
We decided that the coverage test should run on a PR push, not on each PR commit.
Please see the Slack chat: https://redhat-internal.slack.com/archives/CGUMLTNLS/p1783492350136339?thread_ts=1783440020.417639&cid=CGUMLTNLS
| @@ -1,5 +1,5 @@ | |||
| --- | |||
| deploy_flask_app_workers_ssh_private_key: /tmp/id_rsa | |||
There was a problem hiding this comment.
This appears to be a breaking change. Why are you changing this?
There was a problem hiding this comment.
Integration tests target test_deploy_flask_app, failed with error:
TASK [cloud.aws_ops.deploy_flask_app : Ensure SSH directory exists on bastion host] ***
[ERROR]: Task failed: Module failed: [Errno 1] Operation not permitted: b'/tmp'
Origin: /Users/brahmani/.ansible/collections/ansible_collections/cloud/aws_ops/roles/deploy_flask_app/tasks/setup_infra.yaml:40:3
38 register: deploy_flask_app__keypair_result
39
40 - name: Ensure SSH directory exists on bastion host
^ column 3
fatal: [localhost -> ec2-18-212-199-252-bastion(18.212.199.252)]: FAILED! => {"changed": false, "msg": "Task failed: Module failed: [Errno 1] Operation not permitted: b'/tmp'"}
AI suggested changing the deploy_flask_app_workers_ssh_private_key value to ~/.ssh/workers_id_rsa.
Do you have a solution that won't be a breaking change ( I will try to find one)?
There was a problem hiding this comment.
I override the integration test vars, tests/integration/targets/test_deploy_flask_app/roles/run_deploy_flask_app/vars/main.yaml, — not in role defaults — so existing consumers of /tmp/id_rsa are unchanged.
|



Resolves
ACA-6690
SUMMARY
Updates Python version to 3.13 in integration and coverage test workflows.
Changes:
Name: Fedora-Cloud-Base-.x86_64-4 (matches current Fedora 4x images, excludes ELN/Rawhide)
push, instead of each pull_request.test_backup_select_resources- update iam create role to include ansible-test prefix.test_deploy_flask_app- Gather information about RDS engine version instead of using a constant var value.test_deploy_flask_app- update bastion_host_iam_role iam create role to include ansible-test prefix.test_customized_ami- use setup_rsa_keys__path with ansible.builtin.file instead of tempfile.roles/deploy_flask_app- Move key path from /tmp/id_rsa → ~/.ssh/workers_id_rsatest_deploy_flask_app- Fixing bastion user_data in test_deploy_flask_app to use dnf and Fedora-appropriate packages.roles/customized_ami/tasks/create_ec2_resources.yamlandroles/deploy_flask_app/tasks/setup_infra.yaml- to install coverage on the bastion host for coverage collection.ngnix, instead of relying onpodmanbeing absent from the Fedora 4x base image.roles/manage_transit_gateway- updated debug message.ISSUE TYPE
Test Pull Request
COMPONENT NAME
CI/CD workflows
Reported CI Issues
Integration tests #184
Assisted-by: Claude Sonnet 4.5 (claude-sonnet-4-5@20250929)