Skip to content

Conversation

tarasmadan
Copy link
Collaborator

@tarasmadan tarasmadan commented May 23, 2025

It allows to manually specify multiple target managers as a &manager=...&manager=... GET params.
Closes #6044

Clang only managers are &manager=ci-qemu-native-arm64-kvm&manager=ci-snapshot-upstream-root&manager=ci-upstream-bpf-next-kasan-gce&manager=ci-upstream-gce-arm64&manager=ci-upstream-kasan-gce-root&manager=ci-upstream-kasan-gce-smack-root&manager=ci-upstream-kmsan-gce-root&manager=ci-upstream-linux-next-kasan-gce-root&manager=ci-upstream-net-kasan-gce&manager=ci-upstream-net-this-kasan-gce&manager=ci2-upstream-fs&manager=ci2-upstream-kcsan-gce.

@tarasmadan tarasmadan force-pushed the multiple-managers-coverage branch from 4e44ee6 to 7e1a54d Compare May 23, 2025 10:16
@tarasmadan tarasmadan marked this pull request as ready for review May 23, 2025 10:39
@tarasmadan tarasmadan force-pushed the multiple-managers-coverage branch 4 times, most recently from 4a0e369 to 800b2f8 Compare May 23, 2025 13:15
Use '&manager=...&manager=...' GET params on the /ns/coverage page.
@tarasmadan tarasmadan force-pushed the multiple-managers-coverage branch from 800b2f8 to 4cc8cfe Compare May 23, 2025 13:37
@a-nogikh
Copy link
Collaborator

a-nogikh commented Jul 2, 2025

Is this PR still relevant?

@tarasmadan
Copy link
Collaborator Author

70 yes / 30 no. There is a need to get coverage from clang only instances because mixing clang and gcc inputs we're getting lower coverage numbers.
It is needed only for the KVM fuzzing thus getting the coverage from the single instance may work (that's why 30 no).

@ramosian-glider wdyt?

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.

dashboard/app: merged coverage numbers are too low when 2 compilers are used
2 participants