Skip to content

fix: deploy test report error #1703

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

Merged
merged 7 commits into from
Nov 27, 2024
Merged

Conversation

astandrik
Copy link
Collaborator

@astandrik astandrik commented Nov 26, 2024

Closes #1702

Should run sequentially or

Run peaceiris/actions-gh-pages@v3
[INFO] Usage https://github.com/peaceiris/actions-gh-pages#readme
Dump inputs
Setup auth token
Prepare publishing assets
Setup Git config
Create a commit
Push the commit or tag
  /usr/bin/git push origin --force gh-pages
  To https://github.com/ydb-platform/ydb-embedded-ui.git
   ! [remote rejected] gh-pages -> gh-pages (cannot lock ref 'refs/heads/gh-pages': is at aac9b855b126a1fb0630e87cf89be6e597a8b33c but expected 42c0e40f[13](https://github.com/ydb-platform/ydb-embedded-ui/actions/runs/12030274326/job/33538148088#step:6:14)fb6ced9aec921cce88e8cacd07b72d)
  error: failed to push some refs to 'https://github.com/ydb-platform/ydb-embedded-ui.git'
  Error: Action failed with "The process '/usr/bin/git' failed with exit code 1"

Solution:

The concurrency field manages how concurrent workflow runs are handled:

group: deploy_report

Creates a concurrency group named "deploy_report"

  • Only one job in this group can run at a time
  • Jobs from the same workflow with the same group name will be queued

cancel-in-progress: false
When set to false, new workflow runs will wait for in-progress runs to complete
If set to true, it would cancel any currently running workflow and start the new one

Real-world example: If multiple PRs are merged quickly:

  1. First workflow run starts deploying
  2. Second PR triggers another workflow
  3. Instead of running both simultaneously (which could cause conflicts), the second run waits for the first to finish

Proof that it actually works:

image

CI Results

Test Status: ✅ PASSED

📊 Full Report

Total Passed Failed Flaky Skipped
208 208 0 0 0

😟 No changes in tests. 😕

Bundle Size: ✅

Current: 66.09 MB | Main: 66.09 MB
Diff: 0.00 KB (0.00%)

✅ Bundle size unchanged.

ℹ️ CI Information
  • Test recordings for failed tests are available in the full report.
  • Bundle size is measured for the entire 'dist' directory.
  • 📊 indicates links to detailed reports.
  • 🔺 indicates increase, 🔽 decrease, and ✅ no change in bundle size.

@astandrik astandrik added this pull request to the merge queue Nov 27, 2024
Merged via the queue into main with commit efb0b9c Nov 27, 2024
7 checks passed
@astandrik astandrik deleted the astandrik.deploy-test-report-error-1702 branch November 27, 2024 08:25
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.

fix: deploy test report error
4 participants