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.

Sign up for GitHub

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

Jump to bottom

Enable stable-rt email report #565

Merged
merged 3 commits into from
Sep 4, 2024
Merged

Enable stable-rt email report #565

merged 3 commits into from
Sep 4, 2024

Conversation

JenySadadia
Copy link
Collaborator

@JenySadadia JenySadadia commented Aug 23, 2024 •
edited
Loading

For starters, stable-rt email reports will list down build, boot, and rt-tests failures.

@JenySadadia JenySadadia marked this pull request as ready for review August 24, 2024 05:12
@JenySadadia JenySadadia force-pushed the stable-rt-report branch 2 times, most recently from c4b268a to 81e78da Compare August 27, 2024 05:30
@JenySadadia JenySadadia mentioned this pull request Aug 28, 2024
Merged
spbnick
spbnick requested changes Aug 28, 2024
View reviewed changes
Copy link
Collaborator

@spbnick spbnick left a comment

Choose a reason for hiding this comment

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

That's a good start, thank you. However, could you please rename common.j2 to stable.j2. It's only "common" to stable templates, not to everything.

@spbnick
Copy link
Collaborator

spbnick commented Aug 28, 2024

Ah, I see you're using it for "fluster" reports too. Could you name it maestro.j2 perhaps, then?

@JenySadadia
Copy link
Collaborator Author

That's a good start, thank you. However, could you please rename common.j2 to stable.j2. It's only "common" to stable templates, not to everything.

I wanted to use it for codec summary report as well. That's why I named it common.j2.

@JenySadadia
Copy link
Collaborator Author

Ah, I see you're using it for "fluster" reports too. Could you name it maestro.j2 perhaps, then?

We are taking into account results from broonie as well.

@JenySadadia
Copy link
Collaborator Author

Here is the sample stable-rt report:

Subject: KernelCI report for stable-rt: [email protected]

OVERVIEW

        Builds: 0 passed, 2 failed

    Boot tests: 0 passed, 1 failed

      RT tests: 0 passed, 2 failed

    CI systems: maestro

REVISION

    Commit
        name: v6.1.105-rt38
        hash: 8fe1d58eff313241a0c3187db014fbccaad0b680
    Checked out from
        https://git.kernel.org/pub/scm/linux/kernel/git/rt/linux-stable-rt.git v6.1-rt


BUILDS

    Failures
      -arm64 (x86_64_defconfig)
      Build detail: https://kcidb.kernelci.org/d/build/build?orgId=1&var-id=maestro:66969c5faa07c494f8d22ac1
      -arm64 (defconfig)
      Build detail: https://kcidb.kernelci.org/d/build/build?orgId=1&var-id=maestro:66bf585a8ac59a075e6975f2
      Build error: make: *** [arch/arm64/Makefile:170: vdso_prepare] Error 2
      CI system: maestro


BOOT TESTS

    Failures

      arm64:(x86_64_defconfig)
      -acer-cbv514-1h-34uz-brya
      CI system: maestro

RT TESTS

    Failures

      rt-tests.cyclicdeadline:
      arm64:(defconfig)
      -bcm2711-rpi-4-b
      CI system: maestro

      rt-tests.cyclicdeadline.t0-avg-latency:
      arm64:(defconfig)
      -imx6q-sabrelite
      CI system: maestro

See complete and up-to-date report at:

    https://kcidb.kernelci.org/d/revision/revision?orgId=1&var-git_commit_hash=8fe1d58eff313241a0c3187db014fbccaad0b680&var-patchset_hash=


Tested-by: kernelci.org bot <[email protected]>

Thanks,
KernelCI team

Cc: @helen-fornazier

@helen-fornazier
Copy link
Collaborator

@JenySadadia the sample is looking good to me, thanks!

@JenySadadia
Copy link
Collaborator Author

As per suggestion from @spbnick , created common templates for stable-rc and stable-rt reports.

@JenySadadia
Copy link
Collaborator Author

Here is the updated sample report with test links:

BUILDS

    Failures
      -arm64 (x86_64_defconfig)
      Build detail: https://kcidb.kernelci.org/d/build/build?orgId=1&var-id=maestro:66969c5faa07c494f8d22ac1
      -arm64 (defconfig)
      Build detail: https://kcidb.kernelci.org/d/build/build?orgId=1&var-id=maestro:66bf585a8ac59a075e6975f2
      Build error: make: *** [arch/arm64/Makefile:170: vdso_prepare] Error 2
      CI system: maestro


BOOT TESTS

    Failures

      arm64:(x86_64_defconfig)
      -acer-cbv514-1h-34uz-brya
      Test details:
      - https://kcidb.kernelci.org/d/test/test?orgId=1&var-id=maestro:6696c76faa07c494f8d2828c
      - https://kcidb.kernelci.org/d/test/test?orgId=1&var-id=maestro:6696c76faa07c494f8d2828f

      CI system: maestro

RT TESTS

    Failures

      rt-tests.cyclicdeadline:
      arm64:(defconfig)
      -bcm2711-rpi-4-b
      Test details:
      - https://kcidb.kernelci.org/d/test/test?orgId=1&var-id=maestro:6696c76faa07c494f8d2828d

      CI system: maestro

      rt-tests.cyclicdeadline.t0-avg-latency:
      arm64:(defconfig)
      -imx6q-sabrelite
      Test details:
      - https://kcidb.kernelci.org/d/test/test?orgId=1&var-id=maestro:6696c76faa07c494f8d2828e

      CI system: maestro

See complete and up-to-date report at:

    https://kcidb.kernelci.org/d/revision/revision?orgId=1&var-git_commit_hash=8fe1d58eff313241a0c3187db014fbccaad0b680&var-patchset_hash=


Tested-by: kernelci.org bot <[email protected]>

Thanks,
KernelCI team

Jeny Sadadia added 3 commits September 4, 2024 11:56
kcidb/templates: add templates for creating stable-rt reports
219dd78 
Add templates for generating `stable-rt` report.

Signed-off-by: Jeny Sadadia <[email protected]>
kcidb/monitor/subscriptions: add 'linux_stable_rt' subscription
f7bd2b4 
Add a subscription script for generating notifications
for Linux `stable-rt` tree.

Signed-off-by: Jeny Sadadia <[email protected]>
stable-rc: use common templates created for stable
c403609 
Use common templates such as `stable_build.j2`, `stable_revision.j2`,
`stable_revision_summary.j2`, and `common.j2` to avoid
duplication in `stable-rc` and `stable-rt` report templates.

Signed-off-by: Jeny Sadadia <[email protected]>
@helen-fornazier
Copy link
Collaborator

The template looks good to me! Thanks!

@JenySadadia
Copy link
Collaborator Author

@spbnick Can we merge this?

@spbnick spbnick merged commit 6d67964 into kernelci:main Sep 4, 2024
5 of 6 checks passed
@spbnick
Copy link
Collaborator

spbnick commented Sep 4, 2024

Sure!

@JenySadadia JenySadadia deleted the stable-rt-report branch September 4, 2024 14:42
@JenySadadia JenySadadia mentioned this pull request Sep 5, 2024
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@spbnick spbnick spbnick requested changes

@helen-fornazier helen-fornazier Awaiting requested review from helen-fornazier

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@JenySadadia @spbnick @helen-fornazier