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

Remove katello-agent from entities and views #1005

Merged
merged 1 commit into from
Oct 19, 2023

Conversation

vsedmik
Copy link
Contributor

@vsedmik vsedmik commented Oct 13, 2023

Katello agent was deprecated and is being removed from 6.15, the functionality will be replaced by REX.
This PR removes references from entities and views.

@vsedmik vsedmik added entity Related to entity coverage view Issues related to Views coverage No-CherryPick PR doesnt need CherryPick to previous branches labels Oct 13, 2023
@vsedmik vsedmik self-assigned this Oct 13, 2023
@vsedmik
Copy link
Contributor Author

vsedmik commented Oct 13, 2023

trigger: test-robottelo
pytest: tests/foreman/ui/test_contenthost.py -k test_positive_end_to_end_bulk_update

Copy link
Contributor

@adarshdubey-star adarshdubey-star left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ack

@@ -102,8 +101,6 @@ def manage_packages(
view.apply_action(action, action_via=action_via)
view.flash.assert_no_error()
view.flash.dismiss()
if action_via == 'via Katello Agent':
view.done.click()
if action_via == 'via remote execution - customize first':
# After this step the user is redirected to remote execution job
# create view.
Copy link

@omkarkhatavkar omkarkhatavkar Oct 18, 2023

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

line number 114 or 111 conditional statement if action_via in ('via remote execution', 'via remote execution - customize first'): also can be removed.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yeah I think it could, good catch!

@omkarkhatavkar
Copy link

omkarkhatavkar commented Oct 18, 2023

some more places you are missing this @vsedmik will you covering them this PR ?

> grep -ir 'katello.*agent' --exclude="**/*/*.pyc" *
airgun/views/contenthost.py:    """Small icon indicating subscription or katello-agent status. Can be

Copy link
Contributor

@sambible sambible left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Approving pending removing even more as per omkar suggestion 😈

@vsedmik
Copy link
Contributor Author

vsedmik commented Oct 19, 2023

Addressed Omkar's comment, rebased.

@omkarkhatavkar omkarkhatavkar merged commit b626540 into SatelliteQE:master Oct 19, 2023
5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
entity Related to entity coverage No-CherryPick PR doesnt need CherryPick to previous branches Stream view Issues related to Views coverage
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants