Skip to content
This repository was archived by the owner on Feb 25, 2025. It is now read-only.

[Impeller] Disable impeller_unittests. #38307

Merged
merged 1 commit into from
Dec 15, 2022

Conversation

chinmaygarde
Copy link
Member

Copy link
Member

@jason-simmons jason-simmons left a comment

Choose a reason for hiding this comment

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

Another option would be wrapping this run_engine_executable call inside a try/catch block and logging the exception

With that we could continue running this test and checking whether the error is still happening without causing the CI job to fail

@chinmaygarde chinmaygarde added the autosubmit Merge PR when tree becomes green via auto submit App label Dec 15, 2022
@chinmaygarde
Copy link
Member Author

Good call. I'm timeboxing an investigation right now and if I don't have a firm fix, I'll apply my speculative fixes with the try-catch.

@chinmaygarde chinmaygarde self-assigned this Dec 15, 2022
@auto-submit auto-submit bot merged commit a04997c into flutter:main Dec 15, 2022
engine-flutter-autoroll added a commit to engine-flutter-autoroll/flutter that referenced this pull request Dec 15, 2022
auto-submit bot pushed a commit to flutter/flutter that referenced this pull request Dec 15, 2022
…117148)

* df430c4fd Revert "[Impeller] Speculatively attempt to fix Metal PSO construction errors on host targets." (flutter/engine#38292)

* 41dd4f5e1 Revert "Revert "[Impeller] Speculatively attempt to fix Metal PSO construction errors on host targets." (#38292)" (flutter/engine#38301)

* 8ce9a3f71 Roll Skia from 3171deabd88a to b368746d696a (13 revisions) (flutter/engine#38294)

* 4881fe25e Revert "Revert "reland "Migrate darwin common "framework_shared" target to ARC #37049" (#37219)" (#37320)" (flutter/engine#37883)

* 3b2302c8d [Impeller] Remove validation log when the pipeline library is collected before pipeline is setup. (flutter/engine#38306)

* a04997c81 [Impeller] Disable impeller_unittests. (flutter/engine#38307)

* fc71faad0 License script improvements (flutter/engine#38148)

* 6a2560c35 [Windows] Synthesize modifier keys events on pointer events (flutter/engine#38138)

* b1d407563 Roll Skia from b368746d696a to 3f81f95176ce (11 revisions) (flutter/engine#38312)

* b25fcf748 Roll Skia from 3f81f95176ce to 46e8f2a18a3d (3 revisions) (flutter/engine#38314)

* 948699bba Collapse bounds calculations into DisplayListBuilder (flutter/engine#34365)

* 38367de84 Roll Fuchsia Mac SDK from u-tC0QEGUT4xQ4KOo... to VEOIaacOA75U7PYyz... (flutter/engine#38316)

* 29196519c Roll Skia from 46e8f2a18a3d to 9f728d78f10d (1 revision) (flutter/engine#38317)
loic-sharma pushed a commit to loic-sharma/flutter-engine that referenced this pull request Dec 16, 2022
loic-sharma pushed a commit to loic-sharma/flutter-engine that referenced this pull request Jan 3, 2023
gspencergoog pushed a commit to gspencergoog/flutter that referenced this pull request Jan 19, 2023
…lutter#117148)

* df430c4fd Revert "[Impeller] Speculatively attempt to fix Metal PSO construction errors on host targets." (flutter/engine#38292)

* 41dd4f5e1 Revert "Revert "[Impeller] Speculatively attempt to fix Metal PSO construction errors on host targets." (flutter#38292)" (flutter/engine#38301)

* 8ce9a3f71 Roll Skia from 3171deabd88a to b368746d696a (13 revisions) (flutter/engine#38294)

* 4881fe25e Revert "Revert "reland "Migrate darwin common "framework_shared" target to ARC flutter#37049" (flutter#37219)" (flutter#37320)" (flutter/engine#37883)

* 3b2302c8d [Impeller] Remove validation log when the pipeline library is collected before pipeline is setup. (flutter/engine#38306)

* a04997c81 [Impeller] Disable impeller_unittests. (flutter/engine#38307)

* fc71faad0 License script improvements (flutter/engine#38148)

* 6a2560c35 [Windows] Synthesize modifier keys events on pointer events (flutter/engine#38138)

* b1d407563 Roll Skia from b368746d696a to 3f81f95176ce (11 revisions) (flutter/engine#38312)

* b25fcf748 Roll Skia from 3f81f95176ce to 46e8f2a18a3d (3 revisions) (flutter/engine#38314)

* 948699bba Collapse bounds calculations into DisplayListBuilder (flutter/engine#34365)

* 38367de84 Roll Fuchsia Mac SDK from u-tC0QEGUT4xQ4KOo... to VEOIaacOA75U7PYyz... (flutter/engine#38316)

* 29196519c Roll Skia from 46e8f2a18a3d to 9f728d78f10d (1 revision) (flutter/engine#38317)
@chinmaygarde chinmaygarde deleted the disable_impeller_unittests branch March 9, 2024 00:26
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
autosubmit Merge PR when tree becomes green via auto submit App e: impeller
Projects
No open projects
Archived in project
Development

Successfully merging this pull request may close these issues.

[Impeller] Some try jobs are failing on segfaults internal to the Metal shader compiler.
2 participants