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

Add new Refresh button in capsule content #1605

Merged
merged 1 commit into from
Nov 29, 2024

Conversation

vsedmik
Copy link
Contributor

@vsedmik vsedmik commented Oct 25, 2024

A new kebab for the content counts update has been added in 6.17 to granularize the counts calculation.

ccc

This PR just adds the ability to refresh content counts for particular CV under a LCE.

Tested locally with packit instance for Katello/katello#11187 and some populated capsule and a dummie like this worked

def test_dummie(target_sat, module_capsule_configured):
    with target_sat.ui_session() as session:
        session.organization.select(org_name='Any organization')
        details = session.capsule.read_details(module_capsule_configured.hostname)
        session.capsule.refresh_lce_counts(module_capsule_configured.hostname, lce_name='etPJDb')
        session.capsule.refresh_lce_counts(
            module_capsule_configured.hostname, lce_name='etPJDb', cv_name='AzrxdmTifn'
        )

@vsedmik vsedmik added No-CherryPick PR doesnt need CherryPick to previous branches Stream labels Oct 25, 2024
@vsedmik vsedmik self-assigned this Oct 25, 2024
Copy link
Contributor

@damoore044 damoore044 left a comment

Choose a reason for hiding this comment

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

ack looks good, is it possible to open a draft PR and run w/ packit 6.17 in PRT?

To make sure of no navigation or element exceptions, when it runs with different timing in CI? No worries if that is a headache, nonblocking

Copy link
Contributor

@LadislavVasina1 LadislavVasina1 left a comment

Choose a reason for hiding this comment

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

Great addition! ACK

@vsedmik
Copy link
Contributor Author

vsedmik commented Oct 30, 2024

ack looks good, is it possible to open a draft PR and run w/ packit 6.17 in PRT?

Sure it is, currently drafted since we need more-than-one Katello PR to be used to tune the assertions: SatelliteQE/robottelo#16820

@vsedmik
Copy link
Contributor Author

vsedmik commented Nov 26, 2024

packit/PRT passed here SatelliteQE/robottelo#16820 (comment)

@pondrejk pondrejk merged commit 65065f5 into SatelliteQE:master Nov 29, 2024
8 checks passed
@vsedmik vsedmik deleted the extend-counts-refresh branch November 29, 2024 09:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
No-CherryPick PR doesnt need CherryPick to previous branches Stream
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants