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.

Sign up for GitHub

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

Jump to bottom

[6.13.z cp] variable value should del after update #17236

Open
wants to merge 1 commit into
base: 6.13.z
Choose a base branch
from
Open

[6.13.z cp] variable value should del after update #17236

wants to merge 1 commit into from

Conversation

amolpati30
Copy link
Contributor

@amolpati30 amolpati30 commented Jan 2, 2025 •
edited
Loading

I extended the existing test case to include the following steps:

Create an Ansible variable with a default value and an overridden key.
Update the value of the Ansible variable at the host level.
After updating, delete the value, ensuring that it reverts to the default value.

Dependent PR: SatelliteQE/airgun#1689

Fixes - #17130

@amolpati30 amolpati30 added No-CherryPick PR doesnt need CherryPick to previous branches 6.13.z Introduced in or relating directly to Satellite 6.13 labels Jan 2, 2025
@amolpati30 amolpati30 force-pushed the update_ansiblevar6.13 branch from 35c8a65 to 6127deb Compare January 2, 2025 09:49
@amolpati30 amolpati30 mentioned this pull request Jan 2, 2025
Open
@amolpati30 amolpati30 changed the title [6.13.z cp] Update ansiblevar [6.13.z cp] variable value should del after update Jan 2, 2025
@amolpati30 amolpati30 force-pushed the update_ansiblevar6.13 branch 2 times, most recently from ad46ba6 to 70e8089 Compare January 2, 2025 11:24
@Satellite-QE
Copy link
Collaborator

PRT Result

Build Number: 9726
Build Status: UNSTABLE
PRT Comment: pytest tests/foreman/ui/test_ansible.py::TestAnsibleCfgMgmt -k test_positive_assign_ansible_role_variable_on_host --external-logging
Test Result : ====== 1 failed, 11 deselected, 22 warnings, 1 error in 744.73s (0:12:24) ======

@Satellite-QE Satellite-QE added the PRT-Failed Indicates that latest PRT run is failed for the PR label Jan 2, 2025
@Satellite-QE
Copy link
Collaborator

PRT Result

Build Number: 9729
Build Status: UNSTABLE
PRT Comment: pytest tests/foreman/ui/test_ansible.py::TestAnsibleCfgMgmt -k test_positive_assign_ansible_role_variable_on_host --external-logging
Test Result : ========== 1 failed, 11 deselected, 23 warnings in 773.59s (0:12:53) ===========

@Satellite-QE
Copy link
Collaborator

PRT Result

Build Number: 9733
Build Status: UNSTABLE
PRT Comment: pytest tests/foreman/ui/test_ansible.py::TestAnsibleCfgMgmt -k test_positive_assign_ansible_role_variable_on_host --external-logging
Test Result : ========== 1 failed, 11 deselected, 23 warnings in 769.87s (0:12:49) ===========

@amolpati30
Copy link
Contributor Author

trigger: test-robottelo
pytest: tests/foreman/ui/test_ansible.py::TestAnsibleCfgMgmt -k test_positive_assign_ansible_role_variable_on_host
airgun: 1689

@Satellite-QE
Copy link
Collaborator

PRT Result

Build Number: 9734
Build Status: UNSTABLE
PRT Comment: pytest tests/foreman/ui/test_ansible.py::TestAnsibleCfgMgmt -k test_positive_assign_ansible_role_variable_on_host --external-logging
Test Result : ========== 1 failed, 11 deselected, 23 warnings in 786.46s (0:13:06) ===========

@amolpati30
Copy link
Contributor Author

trigger: test-robottelo
pytest: tests/foreman/ui/test_ansible.py::TestAnsibleCfgMgmt -k test_positive_assign_ansible_role_variable_on_host
airgun: 1689

@Satellite-QE
Copy link
Collaborator

PRT Result

Build Number: 9737
Build Status: UNSTABLE
PRT Comment: pytest tests/foreman/ui/test_ansible.py::TestAnsibleCfgMgmt -k test_positive_assign_ansible_role_variable_on_host --external-logging
Test Result : ====== 1 failed, 11 deselected, 22 warnings, 1 error in 776.74s (0:12:56) ======

@amolpati30
Copy link
Contributor Author

trigger: test-robottelo
pytest: tests/foreman/ui/test_ansible.py::TestAnsibleCfgMgmt -k test_positive_assign_ansible_role_variable_on_host
airgun: 1689

@Satellite-QE
Copy link
Collaborator

PRT Result

Build Number: 9739
Build Status: UNSTABLE
PRT Comment: pytest tests/foreman/ui/test_ansible.py::TestAnsibleCfgMgmt -k test_positive_assign_ansible_role_variable_on_host --external-logging
Test Result : ====== 1 failed, 11 deselected, 22 warnings, 1 error in 737.56s (0:12:17) ======

@amolpati30
Copy link
Contributor Author

trigger: test-robottelo
pytest: tests/foreman/ui/test_ansible.py::TestAnsibleCfgMgmt -k test_positive_assign_ansible_role_variable_on_host
airgun: 1689

@Satellite-QE
Copy link
Collaborator

PRT Result

Build Number: 9753
Build Status: UNSTABLE
PRT Comment: pytest tests/foreman/ui/test_ansible.py::TestAnsibleCfgMgmt -k test_positive_assign_ansible_role_variable_on_host --external-logging
Test Result : ========== 1 failed, 11 deselected, 23 warnings in 807.28s (0:13:27) ===========

@amolpati30 amolpati30 force-pushed the update_ansiblevar6.13 branch from 70e8089 to 224ed3d Compare January 13, 2025 04:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees

@amolpati30 amolpati30

Labels
6.13.z Introduced in or relating directly to Satellite 6.13 No-CherryPick PR doesnt need CherryPick to previous branches PRT-Failed Indicates that latest PRT run is failed for the PR
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@amolpati30 @Satellite-QE