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

[do NOT merge] Just a draft to run packit PRT #16656

Closed

Conversation

vsedmik
Copy link
Contributor

@vsedmik vsedmik commented Oct 11, 2024

Problem Statement

Just a draft PR to test possible regressions for Katello/katello#11058

PRT test Cases example

This run should pass, we can start here and trigger more related stuff later.

trigger: test-robottelo
pytest: tests/foreman/api/test_contentview.py
Katello:
    katello: 11058

@vsedmik vsedmik added No-CherryPick PR doesnt need CherryPick to previous branches Do Not Merge labels Oct 11, 2024
@vsedmik
Copy link
Contributor Author

vsedmik commented Oct 11, 2024

trigger: test-robottelo
pytest: tests/foreman/api/test_contentview.py
Katello:
    katello: 11058

@Satellite-QE Satellite-QE added the PRT-Passed Indicates that latest PRT run is passed for the PR label Oct 11, 2024
@vsedmik
Copy link
Contributor Author

vsedmik commented Oct 11, 2024

trigger: test-robottelo
pytest: tests/foreman/cli/test_contentviewfilter.py tests/foreman/cli/test_docker.py
Katello:
    katello: 11058

@Satellite-QE Satellite-QE added PRT-Failed Indicates that latest PRT run is failed for the PR and removed PRT-Passed Indicates that latest PRT run is passed for the PR labels Oct 11, 2024
@vsedmik
Copy link
Contributor Author

vsedmik commented Oct 11, 2024

=========== 3 failed, 138 passed, 316 warnings in 4150.27s (1:09:10) ===========

2x failure on known issue of AK update:

12:59:39  E           robottelo.exceptions.CLIReturnCodeError: CLIReturnCodeError(status=64, stderr="Could not update the activation key:\n  Missing arguments for '--name'.\n", msg='Command "activation-key update" finished with status 64\nstderr contains:\nCould not update the activation key:\n  Missing arguments for \'--name\'.\n'

1x assertion failure on hammer product info here

assert repo_names.issubset({repo_['repo-name'] for repo_ in product['content']})

looks like it did not return the expected repos for that product

@ianballou
Copy link

It looks like the failing docker repo test has a very similar one right after it in robottelo. If that later test is passing, perhaps the failure is a fluke.

@vsedmik
Copy link
Contributor Author

vsedmik commented Oct 16, 2024

trigger: test-robottelo
pytest: tests/foreman/cli/test_contentviewfilter.py tests/foreman/cli/test_docker.py -k create_repos_using_same_product
Katello:
    katello: 11058

@vsedmik
Copy link
Contributor Author

vsedmik commented Oct 16, 2024

Or I wonder if repos creation failed silently and thus they weren't found in the module-scoped product (where repos from other tests within the module were present).
Anyway, let's see how it runs standalone ^^

@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 Oct 16, 2024
@@ -12,6 +12,8 @@

"""

# This file fully passed in snaps 72-75, let's start here.
Copy link
Collaborator

Choose a reason for hiding this comment

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

Just for awareness, If you would like to run an upstream PRT, the PRT comment can be added directly to the Jenkins job while build with parameters, without requiring robottelo PRs, and you can also post results on any GH PRs where you wish, such as here on upstream Katello PR

@vsedmik
Copy link
Contributor Author

vsedmik commented Oct 16, 2024

Standalone it passed @ianballou ^

=== 1 passed, 140 deselected, 149 warnings in 1361.42s (0:22:41) ===

@Gauravtalreja1 @omkarkhatavkar Not sure why the PRT results comment is not visible here. The curl -L -X POST from CI was sent ok. Any idea?

@omkarkhatavkar
Copy link

omkarkhatavkar commented Oct 16, 2024

Standalone it passed @ianballou ^

=== 1 passed, 140 deselected, 149 warnings in 1361.42s (0:22:41) ===

@Gauravtalreja1 @omkarkhatavkar Not sure why the PRT results comment is not visible here. The curl -L -X POST from CI was sent ok. Any idea?

Thanks for raising the issue and fix is here #16684

@vsedmik vsedmik closed this Nov 13, 2024
@vsedmik vsedmik deleted the prt-for-structured-publisher branch November 13, 2024 11:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Do Not Merge 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.

5 participants