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

chore: adding new kpi report workflow #26922

Merged
merged 7 commits into from
Jun 9, 2023
Merged

Conversation

nagash77
Copy link
Contributor

@nagash77 nagash77 commented Jun 2, 2023

This KPI report workflow begins by generating reports on open vs closed issues and triage throughput metrics

Additional details

Steps to test

I don't think there is a way to test this easily. I have been doing testing in my private repos to verify it works.

How has the user experience changed?

PR Tasks

@nagash77
Copy link
Contributor Author

nagash77 commented Jun 2, 2023

relies on this PR in the release-automations repo - https://github.com/cypress-io/release-automations/pull/85

@nagash77 nagash77 marked this pull request as ready for review June 2, 2023 19:57
@nagash77 nagash77 requested a review from mschile June 2, 2023 19:57
@cypress
Copy link

cypress bot commented Jun 2, 2023

28 flaky tests on run #47248 ↗︎

0 27323 1308 0 Flakiness 28

Details:

pr updates
Project: cypress Commit: 3fba6baee6
Status: Passed Duration: 20:47 💡
Started: Jun 7, 2023 9:42 PM Ended: Jun 7, 2023 10:03 PM
Flakiness  commands/net_stubbing.cy.ts • 1 flaky test • 5x-driver-firefox

View Output Video

Test Artifacts
network stubbing > intercepting request > can delay and throttle a StaticResponse Output
Flakiness  e2e/origin/config_env.cy.ts • 1 flaky test • 5x-driver-firefox

View Output Video

Test Artifacts
cy.origin- Cypress.config() > serializable > overwrites different values in secondary, even if the Cypress.config() value does not exist in the primary Output
Flakiness  cypress/cypress.cy.js • 3 flaky tests • 5x-driver-firefox

View Output Video

Test Artifacts
... > correctly returns currentRetry Output
... > correctly returns currentRetry Output
... > correctly returns currentRetry Output
Flakiness  create-from-component.cy.ts • 2 flaky tests • app-e2e

View Output Video

Test Artifacts
... > runs generated spec Output Screenshots Video
... > runs generated spec Output Screenshots Video
Flakiness  specs_list_latest_runs.cy.ts • 1 flaky test • app-e2e

View Output Video

Test Artifacts
App/Cloud Integration - Latest runs and Average duration > when no runs are recorded > shows placeholders for all visible specs Output Screenshots Video

The first 5 flaky specs are shown, see all 16 specs in Cypress Cloud.

This comment has been generated by cypress-bot as a result of this project's GitHub integration settings.

.github/workflows/report_weekly_app_kpis.yml Outdated Show resolved Hide resolved
@@ -0,0 +1,71 @@
name: 'KPI Report Generation'
Copy link
Contributor

@mschile mschile Jun 7, 2023

Choose a reason for hiding this comment

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

Suggested change
name: 'KPI Report Generation'
name: 'Triage Report'

Copy link
Contributor

Choose a reason for hiding this comment

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

Suggest renaming to app_triage_weekly_report.yml

Copy link
Contributor Author

Choose a reason for hiding this comment

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

the end goal is to report on the overall health of the repo and not just firewatch specific. That just happens to be the first metric I was generating.

@nagash77 nagash77 merged commit 806b74e into develop Jun 9, 2023
@nagash77 nagash77 deleted the benm/kpi-report-workflow-v1 branch June 9, 2023 13:05
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.

4 participants