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.

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

[6.14.z] hosts export selectable columns #15069

Conversation

Satellite-QE
Copy link
Collaborator

Cherrypick of PR: #14977

Problem Statement

The "Export" button in "All Hosts" page does not exports the visible columns as expected.

(https://bugzilla.redhat.com/show_bug.cgi?id=2167146)

Solution

New test for exporting only the selected columns in the web UI.

Related Issues

SAT-21626 (component evaluation)

@Satellite-QE Satellite-QE added 6.14.z Introduced in or relating directly to Satellite 6.14 Auto_Cherry_Picked Automatically cherrypicked PR using GHA No-CherryPick PR doesnt need CherryPick to previous branches labels May 15, 2024
@Satellite-QE
Copy link
Collaborator Author

trigger: test-robottelo
pytest: tests/foreman/ui/test_host.py -k test_positive_export_selected_columns

@Satellite-QE Satellite-QE added the AutoMerge_Cherry_Picked The cherrypicked PRs of master PR would be automerged if all checks passing label May 15, 2024
@Satellite-QE
Copy link
Collaborator Author

PRT Result

Build Number: 6945
Build Status: UNSTABLE
PRT Comment: pytest tests/foreman/ui/test_host.py -k test_positive_export_selected_columns --external-logging
Test Result : ========== 1 failed, 41 deselected, 51 warnings in 1048.31s (0:17:28) ==========

@Satellite-QE Satellite-QE added the PRT-Failed Indicates that latest PRT run is failed for the PR label May 15, 2024
@rplevka
Copy link
Member

rplevka commented May 15, 2024

@pnovotny
apparently, 6.14 has problems with the RHEL Lifecycle status field:

2024-05-15 21:09:45 - widgetastic_null - ERROR - [ManageColumnsView/checkbox_group]: An exception happened during fill({'Power': True, 'Recommendations': True, 'Name': True, 'IPv4': True, 'IPv6': True, 'MAC': True, 'OS': True, 'Owner': True, 'Host group': True, 'Boot time': True, 'Last report': True, 'Comment': True, 'Model': True, 'Sockets': True, 'Cores': True, 'RAM': True, 'Virtual': True, 'Disks space': True, 'Kernel version': True, 'BIOS vendor': True, 'BIOS release date': True, 'BIOS version': True, 'RHEL Lifecycle status': True, 'Installable updates': False, 'Lifecycle environment': True, 'Content view': True, 'Registered': True, 'Last checkin': True}) call (elapsed 8258 ms)
2024-05-15 21:09:45 - widgetastic_null - ERROR - [ManageColumnsView/checkbox_group]: 'RHEL Lifecycle status'

    Traceback (most recent call last):
      File "/opt/app-root/lib64/python3.12/site-packages/widgetastic/log.py", line 137, in wrapped
        result = f(self, *args, **kwargs)
                 ^^^^^^^^^^^^^^^^^^^^^^^^
      File "/opt/app-root/lib64/python3.12/site-packages/widgetastic/widget/base.py", line 41, in wrapped
        return method(self, Fillable.coerce(value), *args, **kwargs)
               ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
      File "/opt/app-root/lib64/python3.12/site-packages/airgun/widgets.py", line 1523, in fill
        self.checkboxes[name].fill(value)
        ~~~~~~~~~~~~~~~^^^^^^
    KeyError: 'RHEL Lifecycle status'
...

@pnovotny
Copy link
Contributor

trigger: test-robottelo
pytest: tests/foreman/ui/test_host.py -k test_positive_export_selected_columns

@pnovotny
Copy link
Contributor

@pnovotny apparently, 6.14 has problems with the RHEL Lifecycle status field:
Right, thanks for the check. The column has a different name in 6.14.
Fixed.

@pnovotny
Copy link
Contributor

trigger: test-robottelo
pytest: tests/foreman/ui/test_host.py -k test_positive_export_selected_columns

pnovotny added 2 commits May 27, 2024 10:46
hosts: export selected columns test

New test for hosts for exporting only columns that are selected in the web UI.

BZ: https://bugzilla.redhat.com/2167146
(cherry picked from commit 31b25db)
@pnovotny pnovotny force-pushed the cherry-pick-6.14.z-31b25db8dcbc76373f3829a0e653c3a896365cb5 branch from eccf501 to 63529f9 Compare May 27, 2024 08:46
@Satellite-QE
Copy link
Collaborator Author

PRT Result

Build Number: 7125
Build Status: SUCCESS
PRT Comment: pytest tests/foreman/ui/test_host.py -k test_positive_export_selected_columns --external-logging
Test Result : ========== 1 passed, 41 deselected, 51 warnings in 760.14s (0:12:40) ===========

@Satellite-QE Satellite-QE added PRT-Passed Indicates that latest PRT run is passed for the PR and removed PRT-Failed Indicates that latest PRT run is failed for the PR labels May 27, 2024
@Satellite-QE Satellite-QE merged commit 91a8a09 into 6.14.z May 27, 2024
12 of 13 checks passed
@Satellite-QE Satellite-QE deleted the cherry-pick-6.14.z-31b25db8dcbc76373f3829a0e653c3a896365cb5 branch May 27, 2024 09:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
6.14.z Introduced in or relating directly to Satellite 6.14 Auto_Cherry_Picked Automatically cherrypicked PR using GHA AutoMerge_Cherry_Picked The cherrypicked PRs of master PR would be automerged if all checks passing No-CherryPick PR doesnt need CherryPick to previous branches PRT-Passed Indicates that latest PRT run is passed for the PR
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants