Skip to content

Expose fedora-review results to users#32

Merged
LecrisUT merged 1 commit intopackit:mainfrom
FrostyX:fedora-review-2
Mar 24, 2026
Merged

Expose fedora-review results to users#32
LecrisUT merged 1 commit intopackit:mainfrom
FrostyX:fedora-review-2

Conversation

@FrostyX
Copy link
Contributor

@FrostyX FrostyX commented Feb 28, 2026

See #27
This is a follow-up after PR #28

@FrostyX
Copy link
Contributor Author

FrostyX commented Feb 28, 2026

You can see an example here:

$ testing-farm request
    --git-url https://github.com/FrostyX/tmt-plans.git  \
    --git-ref fedora-review-2 \
    --plan /plans/fedora-review \
    -e KOJI_TASK_ID=142668218 \
    --context initiator=fedora-ci \
    --context trigger=commit \
    --compose Fedora-latest

📦 repository https://github.com/FrostyX/tmt-plans.git ref fedora-review-2 test-type fmf
💻 Fedora-latest on x86_64
🔎 api https://api.dev.testing-farm.io/v0.1/requests/759f2516-8d45-492a-9426-c4e102441394
💡 waiting for request to finish, use ctrl+c to skip
👷 request is queued
🚀 request is running
🚢 artifacts https://artifacts.dev.testing-farm.io/759f2516-8d45-492a-9426-c4e102441394
❌ tests failed
┏━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━┳━━━━━━━━━━┳━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━┳━━━━━━━━━┳━━━━━━━━━━━━━━━━━━┳━━━━━━━━━━━━━━━┳━━━━━━━━━━━━━━┳━━━━━━━━━━━━━━━┳━━━━━━━━━━━━━━┓
┃ id                                   ┃ state    ┃ artifacts                                                                  ┃ overall ┃ arches_requested ┃ errored_plans ┃ failed_plans ┃ skipped_plans ┃ passed_plans ┃
┡━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━╇━━━━━━━━━━╇━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━╇━━━━━━━━━╇━━━━━━━━━━━━━━━━━━╇━━━━━━━━━━━━━━━╇━━━━━━━━━━━━━━╇━━━━━━━━━━━━━━━╇━━━━━━━━━━━━━━┩
│ 759f2516-8d45-492a-9426-c4e102441394 │ complete │ https://artifacts.dev.testing-farm.io/759f2516-8d45-492a-9426-c4e102441394 │ failed  │ x86_64           │ 0             │ 1            │ 0             │ 0            │
└──────────────────────────────────────┴──────────┴────────────────────────────────────────────────────────────────────────────┴─────────┴──────────────────┴───────────────┴──────────────┴───────────────┴──────────────┘
┏━━━━━━━━━━━━━━━━━━━━━━┳━━━━━━━━┓
┃ plan                 ┃ x86_64 ┃
┡━━━━━━━━━━━━━━━━━━━━━━╇━━━━━━━━┩
│ /plans/fedora-review │ fail   │
└──────────────────────┴────────┘

@FrostyX
Copy link
Contributor Author

FrostyX commented Mar 17, 2026

@LecrisUT I updated the PR and workarounded the weird viewer.html bug. Please see the copy_data_into_data function. It is definitely weird, and I consider this to be a Testing Farm bug but I don't want to get stuck on it.

A new example is here:

$ testing-farm request \
    --git-url https://github.com/FrostyX/tmt-plans.git \
    --git-ref fedora-review-2 \
    --plan /plans/fedora-review \
    -e KOJI_TASK_ID=143386800 \
    --context initiator=fedora-ci \
    --context trigger=commit \
    --compose Fedora-latest

📦 repository https://github.com/FrostyX/tmt-plans.git ref fedora-review-2 test-type fmf
💻 Fedora-latest on x86_64
🔎 api https://api.dev.testing-farm.io/v0.1/requests/272d925f-9b6b-4497-984e-9c321780af92
💡 waiting for request to finish, use ctrl+c to skip
👶 request is waiting to be queued
🚀 request is running
🚢 artifacts https://artifacts.dev.testing-farm.io/272d925f-9b6b-4497-984e-9c321780af92
❌ tests failed
┏━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━┳━━━━━━━━━━┳━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━┳━━━━━━━━━┳━━━━━━━━━━━━━━━━━━┳━━━━━━━━━━━━━━━┳━━━━━━━━━━━━━━┳━━━━━━━━━━━━━━━┳━━━━━━━━━━━━━━┓
┃ id                                   ┃ state    ┃ artifacts                                                                  ┃ overall ┃ arches_requested ┃ errored_plans ┃ failed_plans ┃ skipped_plans ┃ passed_plans ┃
┡━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━╇━━━━━━━━━━╇━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━╇━━━━━━━━━╇━━━━━━━━━━━━━━━━━━╇━━━━━━━━━━━━━━━╇━━━━━━━━━━━━━━╇━━━━━━━━━━━━━━━╇━━━━━━━━━━━━━━┩
│ 272d925f-9b6b-4497-984e-9c321780af92 │ complete │ https://artifacts.dev.testing-farm.io/272d925f-9b6b-4497-984e-9c321780af92 │ failed  │ x86_64           │ 0             │ 1            │ 0             │ 0            │
└──────────────────────────────────────┴──────────┴────────────────────────────────────────────────────────────────────────────┴─────────┴──────────────────┴───────────────┴──────────────┴───────────────┴──────────────┘
┏━━━━━━━━━━━━━━━━━━━━━━┳━━━━━━━━┓
┃ plan                 ┃ x86_64 ┃
┡━━━━━━━━━━━━━━━━━━━━━━╇━━━━━━━━┩
│ /plans/fedora-review │ fail   │
└──────────────────────┴────────┘

@FrostyX
Copy link
Contributor Author

FrostyX commented Mar 17, 2026

I don't understand what is pre-commit complaining about though.

Copy link
Collaborator

@LecrisUT LecrisUT left a comment

Choose a reason for hiding this comment

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

I don't understand what is pre-commit complaining about though.

Good question, seems to be an issue with the versions in pre-commit. I've opened #33 to hopefully fix it

Other than that all looks good to me

@FrostyX
Copy link
Contributor Author

FrostyX commented Mar 24, 2026

I don't understand what is pre-commit complaining about though.

I rebased from main and there are no CI problems anymore

@LecrisUT LecrisUT merged commit ccb8b0c into packit:main Mar 24, 2026
1 check passed
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.

3 participants