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

Lighthouse performance metrics setup #9304

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

Conversation

ruchidh
Copy link

@ruchidh ruchidh commented Jan 30, 2025

Description

This workflow runs Lighthouse CI to measure the performance of OpenSearch Dashboards. It logs performance warnings without failing the CI when failing the metrics like first time load and speed of page. Additionally, it provides a hosted Lighthouse report URL in PR comments for tracking performance regressions over time.

Screenshot

Screenshot 2025-02-20 at 12 35 07 PM

Testing the changes

  • yarn lhci autorun

Changelog

  • feat: Lighthouse Page Performance Metrics CI workflow

Check List

  • All tests pass
    • yarn test:jest
    • yarn test:jest_integration
  • New functionality includes testing.
  • New functionality has been documented.
  • Update CHANGELOG.md
  • Commits are signed per the DCO using --signoff

Signed-off-by: Ruchi Sharma <ruchi492@gmail.com>
Signed-off-by: Ruchi Sharma <ruchi492@gmail.com>
Signed-off-by: Ruchi Sharma <ruchi492@gmail.com>
Signed-off-by: Ruchi Sharma <ruchi492@gmail.com>
Signed-off-by: Ruchi Sharma <ruchi492@gmail.com>
Signed-off-by: Ruchi Sharma <ruchi492@gmail.com>
Signed-off-by: Ruchi Sharma <ruchi492@gmail.com>
Signed-off-by: Ruchi Sharma <ruchi492@gmail.com>
Signed-off-by: Ruchi Sharma <ruchi492@gmail.com>
Signed-off-by: Ruchi Sharma <ruchi492@gmail.com>
Signed-off-by: Ruchi Sharma <ruchi492@gmail.com>
Signed-off-by: Ruchi Sharma <ruchi492@gmail.com>
Signed-off-by: Ruchi Sharma <ruchi492@gmail.com>
Signed-off-by: Ruchi Sharma <ruchi492@gmail.com>
Signed-off-by: Ruchi Sharma <ruchi492@gmail.com>
Copy link
Contributor

❌ Invalid Prefix For Manual Changeset Creation

Invalid description prefix. Found "feature". Only "skip" entry option is permitted for manual commit of changeset files.

If you were trying to skip the changelog entry, please use the "skip" entry option in the ##Changelog section of your PR description.

@github-actions github-actions bot added failed changeset and removed Skip-Changelog PRs that are too trivial to warrant a changelog or release notes entry labels Feb 25, 2025
Copy link
Contributor

❌ Invalid Prefix For Manual Changeset Creation

Invalid description prefix. Found "feature". Only "skip" entry option is permitted for manual commit of changeset files.

If you were trying to skip the changelog entry, please use the "skip" entry option in the ##Changelog section of your PR description.

1 similar comment
Copy link
Contributor

❌ Invalid Prefix For Manual Changeset Creation

Invalid description prefix. Found "feature". Only "skip" entry option is permitted for manual commit of changeset files.

If you were trying to skip the changelog entry, please use the "skip" entry option in the ##Changelog section of your PR description.

Signed-off-by: Ruchi Sharma <ruchi492@gmail.com>
Signed-off-by: Ruchi Sharma <ruchi492@gmail.com>
Signed-off-by: Ruchi Sharma <ruchi492@gmail.com>
Signed-off-by: Ruchi Sharma <ruchi492@gmail.com>
Copy link
Contributor

❌ Invalid Prefix For Manual Changeset Creation

Invalid description prefix. Found "feat". Only "skip" entry option is permitted for manual commit of changeset files.

If you were trying to skip the changelog entry, please use the "skip" entry option in the ##Changelog section of your PR description.

Signed-off-by: Ruchi Sharma <ruchi492@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants