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

[ci] Add polling to download-build-artifacts #30515

Merged
merged 9 commits into from
Jul 29, 2024
Merged

Conversation

poteto
Copy link
Member

@poteto poteto commented Jul 29, 2024

Stack from ghstack (oldest at bottom):

Several CI workflows depend on the runtime_build_and_test.yml workflow
to complete before it can successfully download the build artifact.
However it is possible to encounter a race condition where the build
hasn't completed when the new workflow is started.

This PR adds a simple polling mechanism that waits up to 6 minutes for
the build for that revision to complete.

[ghstack-poisoned]
Copy link

vercel bot commented Jul 29, 2024

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
react-compiler-playground ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jul 29, 2024 10:57pm

@facebook-github-bot facebook-github-bot added CLA Signed React Core Team Opened by a member of the React Core Team labels Jul 29, 2024
poteto added a commit that referenced this pull request Jul 29, 2024
Several CI workflows depend on the runtime_build_and_test.yml workflow
to complete before it can successfully download the build artifact.
However it is possible to encounter a race condition where the build
hasn't completed when the new workflow is started.

This PR adds a simple polling mechanism that waits up to 6 minutes for
the build for that revision to complete.

ghstack-source-id: 2176d7f49ed94121a19ed2a5a037af989ee25eda
Pull Request resolved: #30515
@react-sizebot
Copy link

react-sizebot commented Jul 29, 2024

Comparing: 6b82f3c...7108e0a

Critical size changes

Includes critical production bundles, as well as any change greater than 2%:

Name +/- Base Current +/- gzip Base gzip Current gzip
oss-stable/react-dom/cjs/react-dom.production.js = 6.68 kB 6.68 kB = 1.83 kB 1.83 kB
oss-stable/react-dom/cjs/react-dom-client.production.js = 501.46 kB 501.46 kB = 89.98 kB 89.98 kB
oss-experimental/react-dom/cjs/react-dom.production.js = 6.69 kB 6.69 kB = 1.83 kB 1.83 kB
oss-experimental/react-dom/cjs/react-dom-client.production.js = 508.59 kB 508.59 kB = 91.15 kB 91.15 kB
facebook-www/ReactDOM-prod.classic.js = 596.53 kB 596.53 kB = 105.80 kB 105.80 kB
facebook-www/ReactDOM-prod.modern.js = 572.82 kB 572.82 kB = 102.01 kB 102.01 kB

Significant size changes

Includes any change greater than 0.2%:

Expand to show
Name +/- Base Current +/- gzip Base gzip Current gzip
react-native/implementations/ReactFabric-profiling.fb.js = 391.88 kB 390.86 kB = 68.44 kB 68.23 kB
react-native/implementations/ReactFabric-prod.fb.js = 364.38 kB 363.34 kB = 64.14 kB 63.98 kB

Generated by 🚫 dangerJS against 99acd4f

[ghstack-poisoned]
poteto added a commit that referenced this pull request Jul 29, 2024
Several CI workflows depend on the runtime_build_and_test.yml workflow
to complete before it can successfully download the build artifact.
However it is possible to encounter a race condition where the build
hasn't completed when the new workflow is started.

This PR adds a simple polling mechanism that waits up to 6 minutes for
the build for that revision to complete.

ghstack-source-id: f7a48547663055f993d5c0464b154e95796e093b
Pull Request resolved: #30515
[ghstack-poisoned]
poteto added a commit that referenced this pull request Jul 29, 2024
Several CI workflows depend on the runtime_build_and_test.yml workflow
to complete before it can successfully download the build artifact.
However it is possible to encounter a race condition where the build
hasn't completed when the new workflow is started.

This PR adds a simple polling mechanism that waits up to 6 minutes for
the build for that revision to complete.

ghstack-source-id: 6e0b68587b7a44b98857739691f87a11933d798b
Pull Request resolved: #30515
[ghstack-poisoned]
poteto added a commit that referenced this pull request Jul 29, 2024
Several CI workflows depend on the runtime_build_and_test.yml workflow
to complete before it can successfully download the build artifact.
However it is possible to encounter a race condition where the build
hasn't completed when the new workflow is started.

This PR adds a simple polling mechanism that waits up to 10 minutes for
the build for that revision to complete.

ghstack-source-id: 513e1335f152c8dc5e46274fa49214bf661ec508
Pull Request resolved: #30515
[ghstack-poisoned]
poteto added a commit that referenced this pull request Jul 29, 2024
Several CI workflows depend on the runtime_build_and_test.yml workflow
to complete before it can successfully download the build artifact.
However it is possible to encounter a race condition where the build
hasn't completed when the new workflow is started.

This PR adds a simple polling mechanism that waits up to 10 minutes for
the build for that revision to complete.

ghstack-source-id: 409c79c1ec74834f17c6fe9a3f114b767f45b722
Pull Request resolved: #30515
[ghstack-poisoned]
poteto added a commit that referenced this pull request Jul 29, 2024
Several CI workflows depend on the runtime_build_and_test.yml workflow
to complete before it can successfully download the build artifact.
However it is possible to encounter a race condition where the build
hasn't completed when the new workflow is started.

This PR adds a simple polling mechanism that waits up to 10 minutes for
the build for that revision to complete.

ghstack-source-id: f95640dc641958cfe1c97b068f06d60694f3c0f3
Pull Request resolved: #30515
[ghstack-poisoned]
poteto added a commit that referenced this pull request Jul 29, 2024
Several CI workflows depend on the runtime_build_and_test.yml workflow
to complete before it can successfully download the build artifact.
However it is possible to encounter a race condition where the build
hasn't completed when the new workflow is started.

This PR adds a simple polling mechanism that waits up to 10 minutes for
the build for that revision to complete.

ghstack-source-id: c6d3d0e047fde6bf1b26b9df0f1b8c82edb94fea
Pull Request resolved: #30515
[ghstack-poisoned]
poteto added a commit that referenced this pull request Jul 29, 2024
Several CI workflows depend on the runtime_build_and_test.yml workflow
to complete before it can successfully download the build artifact.
However it is possible to encounter a race condition where the build
hasn't completed when the new workflow is started.

This PR adds a simple polling mechanism that waits up to 10 minutes for
the build for that revision to complete.

ghstack-source-id: 886c6b89d8b9d6e09b87239b749536798501b2fe
Pull Request resolved: #30515
[ghstack-poisoned]
poteto added a commit that referenced this pull request Jul 29, 2024
Several CI workflows depend on the runtime_build_and_test.yml workflow
to complete before it can successfully download the build artifact.
However it is possible to encounter a race condition where the build
hasn't completed when the new workflow is started.

This PR adds a simple polling mechanism that waits up to 10 minutes for
the build for that revision to complete.

ghstack-source-id: 6a954638a800fbea8081e6fba35ee4b4437731c5
Pull Request resolved: #30515
@poteto poteto merged commit 99acd4f into gh/poteto/8/base Jul 29, 2024
185 checks passed
poteto added a commit that referenced this pull request Jul 29, 2024
Several CI workflows depend on the runtime_build_and_test.yml workflow
to complete before it can successfully download the build artifact.
However it is possible to encounter a race condition where the build
hasn't completed when the new workflow is started.

This PR adds a simple polling mechanism that waits up to 10 minutes for
the build for that revision to complete.

ghstack-source-id: 6a954638a800fbea8081e6fba35ee4b4437731c5
Pull Request resolved: #30515
@poteto poteto deleted the gh/poteto/8/head branch July 29, 2024 23:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CLA Signed React Core Team Opened by a member of the React Core Team
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants