Skip to content

ci: collect metrics from canary CI build #1199

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
Open
wants to merge 46 commits into
base: canaries/dev
Choose a base branch
from
Open

ci: collect metrics from canary CI build #1199

wants to merge 46 commits into from

Conversation

@booskander
Copy link

@booskander booskander commented May 29, 2025 •
edited
Loading

GitHub Issue (If applicable): closes #

PR Type

What kind of change does this PR introduce?

  • Build or CI related changes

What is the current behavior?

Currently, we collect performance metrics by hand

What is the new behavior?

Metrics will now be collected directly through the CI pipelines while building canaries

PR Checklist

Please check if your PR fulfills the following requirements:

  • Tested on iOS.
  • Tested on Wasm.
  • Tested on Android.
  • Tested on UWP.
  • Tested in both Light and Dark themes.
  • Associated with an issue (GitHub or internal)

Other information

Internal Issue (If applicable):

@github-actions
Copy link

github-actions bot commented May 29, 2025

Azure Static Web Apps: Your stage site is ready! Visit it here: https://white-field-08c5cf00f-1199.eastus2.3.azurestaticapps.net

2 similar comments
@github-actions
Copy link

github-actions bot commented May 29, 2025

Azure Static Web Apps: Your stage site is ready! Visit it here: https://white-field-08c5cf00f-1199.eastus2.3.azurestaticapps.net

@github-actions
Copy link

github-actions bot commented May 29, 2025

Azure Static Web Apps: Your stage site is ready! Visit it here: https://white-field-08c5cf00f-1199.eastus2.3.azurestaticapps.net

@unodevops
Copy link

unodevops commented May 29, 2025

⚠️⚠️ The build 167137 has failed on Uno.Gallery - CI.

@github-actions
Copy link

github-actions bot commented May 29, 2025

Azure Static Web Apps: Your stage site is ready! Visit it here: https://white-field-08c5cf00f-1199.eastus2.3.azurestaticapps.net

@booskander booskander changed the base branch from master to canaries/dev May 29, 2025 19:25
@github-actions
Copy link

github-actions bot commented May 29, 2025

Azure Static Web Apps: Your stage site is ready! Visit it here: https://white-field-08c5cf00f-1199.eastus2.3.azurestaticapps.net

@unodevops
Copy link

unodevops commented May 29, 2025

⚠️⚠️ The build 167156 has failed on Uno.Gallery - Canary.

Xiaoy312
Xiaoy312 approved these changes May 29, 2025
View reviewed changes
build/stage-build-skia-desktop.yml Outdated
cat <<EOF > $(Build.ArtifactStagingDirectory)/bin/metrics.json
{
"buildId": "$(Build.BuildId)",
Copy link
Contributor

@Xiaoy312 Xiaoy312 May 29, 2025

Choose a reason for hiding this comment

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

maybe add a timestamp here?

Copy link
Author

@booskander booskander May 30, 2025

Choose a reason for hiding this comment

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

thank xiao, should be added

@booskander booskander changed the title chore: collect metrics from canary CI build ci: collect metrics from canary CI build May 30, 2025
@unodevops
Copy link

unodevops commented May 30, 2025

⚠️⚠️ The build 167326 has failed on Uno.Gallery - Canary.

booskander added 29 commits June 2, 2025 17:53
@booskander
ci: adding quotes around script path
d77abc2
@booskander
ci: publishing the metrics artifact
91f3bdf
@booskander
ci: missing steps keyword
6519bcb
@booskander
ci: adding windows build metrics
79a0d01
@booskander
ci: correcting typo in bash command
9058b18
@booskander
ci: fixing date and path on windows build
22e0060
@booskander
ci: excluding bin-logs from package size
5814afd
@booskander
ci: excluding bin-log from package size
c436e47
@booskander
ci: windows works, changing path for skia
b7b98fe
@booskander
ci: changing du command to avoid rounding
8dc1832
@booskander
ci: adding metrics for other stages
febd368
@booskander
ci: fixing wasm path not found
687d6a9
@booskander
ci: wasm works, fixing mobile
e0a8f7b
@booskander
ci: testing webhook
ac44e90
@booskander
ci: https request on web server
32662a8
@booskander
ci: fix artifact directory
42d1a08
@booskander
ci: testing dotnet tools
7fcc0bb
@booskander
ci: testing dotnet tools
afbdbd4
@booskander
ci: dotnet counter
b840aa1
@booskander
ci: dotnet counters
0018696
@booskander
ci: typo in yml
1df410b
@booskander
ci: changing directory for diagnostic stage
3f3deef
@booskander
ci: removing templates import
c636e3b
@booskander
ci: adding profiling step inside build stage (skia)
3a51824
@booskander
ci: removing line in yml
1ba2d97
@booskander
ci: changing for bash instead of script
0b2e02f
@booskander
ci: fixing exe path
36d46b3
@booskander
ci: changing csv path to artifact staging dir
2a536d4
@booskander
ci: update publish artifact
da3625d
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@Xiaoy312 Xiaoy312 Xiaoy312 approved these changes

@ebariche ebariche Awaiting requested review from ebariche

@jeromelaban jeromelaban Awaiting requested review from jeromelaban

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

5 participants

@booskander @unodevops @Xiaoy312 @jeromelaban