Skip to content

ci(packit): fix extra parameters for TF #75

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

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

mfocko
Copy link

@mfocko mfocko commented May 5, 2025

context is nested under the tmt key. Noticed in Packit monitoring where it trips on failed requests to the Testing Farm.

Enhancement:

Fixed the tf_extra_params in the Packit config.

Reason:

Failing CI, also spams our Sentry backlog.

Result:

Green CI, doesn't spam our Sentry.

Issue Tracker Tickets (Jira or BZ if any):

`context` is nested under the `tmt` key. Noticed in Packit monitoring
where it trips on failed requests to the Testing Farm.

Signed-off-by: Matej Focko <[email protected]>
@mfocko mfocko requested review from spetrosi and richm as code owners May 5, 2025 09:30
@mfocko mfocko force-pushed the ci/fix-tmt-config branch from 99c2109 to 7c7a2c5 Compare May 5, 2025 09:34
@richm
Copy link
Contributor

richm commented May 5, 2025

@mfocko still failing?

@mfocko
Copy link
Author

mfocko commented May 5, 2025

@mfocko still failing?

I've just noticed, it looks like there's an issue with installing the beakerlib dependency:

Cause number 1:

    prepare step failed

    The exception was caused by 1 earlier exceptions

    Cause number 1:

        Command 'rpm -q --whatprovides beakerlib /usr/bin/flock /usr/bin/awk || dnf install -y  beakerlib /usr/bin/flock /usr/bin/awk' returned 1.

        # stdout (6 lines)
        # ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
        no package provides beakerlib
        util-linux-core-2.37.4-21.el9.x86_64
        gawk-5.1.0-6.el9.x86_64
        No match for argument: beakerlib
        Package util-linux-core-2.37.4-21.el9.x86_64 is already installed.
        Package gawk-5.1.0-6.el9.x86_64 is already installed.
        # ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~

        # stderr (1 lines)
        # ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
        Error: Unable to find a match: beakerlib
        # ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~


        The exception was caused by 1 earlier exceptions

        Cause number 1:

            Command 'rpm -q --whatprovides beakerlib /usr/bin/flock /usr/bin/awk || dnf install -y  beakerlib /usr/bin/flock /usr/bin/awk' returned 1.

            # stdout (9 lines)
            # ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
            no package provides beakerlib
            util-linux-core-2.37.4-21.el9.x86_64
            gawk-5.1.0-6.el9.x86_64
            CentOS Stream 9 - BaseOS                         35 MB/s | 8.7 MB     00:00    
            CentOS Stream 9 - AppStream                      75 MB/s |  23 MB     00:00    
            CentOS Stream 9 - Extras packages               152 kB/s |  20 kB     00:00    
            No match for argument: beakerlib
            Package util-linux-core-2.37.4-21.el9.x86_64 is already installed.
            Package gawk-5.1.0-6.el9.x86_64 is already installed.
            # ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~

            # stderr (1 lines)
            # ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
            Error: Unable to find a match: beakerlib
            # ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~

@richm
Copy link
Contributor

richm commented May 15, 2025

@mfocko any update on the testing farm failures?

@mfocko
Copy link
Author

mfocko commented May 15, 2025

@mfocko any update on the testing farm failures?

IIUC we need to enable EPEL here, cause beakerlib is not provided in the CentOS repos.

@richm
Copy link
Contributor

richm commented Jul 2, 2025

@mfocko what should we do with this PR?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants