Skip to content

CI: Avoid 32 bits failures and improve behavior when failures happen#8283

Merged
bcardiff merged 5 commits intomasterfrom
ci/update
Oct 8, 2019
Merged

CI: Avoid 32 bits failures and improve behavior when failures happen#8283
bcardiff merged 5 commits intomasterfrom
ci/update

Conversation

@bcardiff
Copy link
Member

@bcardiff bcardiff commented Oct 7, 2019

It seems the CI is able to run the std_specs for 32 bits fine enough when using a released compiler. There is some heavy memory consuming specs. But building the compiler is the thing that often fails. For example when running the run macro to embed tool init .ecr templates. (See #8279 for a potential fix idea)

Meanwhile, we are near to drop 32bits packages (and CI). LLVM 8 is not even available in their apt repo for 32 bits.

Before dropping 32 bits CI (maybe temporarily) this PR introduce a couple of changes and improvements in the CI.

  • For 32 bits only std_specs are run
  • On nightlies, maintenance, and tagged releases the std_specs are run using the recently built compiler.
    • this recently built compiler fails less often, but there build process is different from the one used to run the per-commit workflow.
  • The distributions jobs can start before the test jobs finished.
    • Darwin builds usually took longer to start, delaying the build queue.
    • Failures along the way might cause longer delays.
  • How build output is stored in artifacts also changed a bit
    • docker images dump is no longer persisted (it was never actually used)
    • steps that produce files for the final artifacts are also available in each independent steps.

This PR is marked as draft until the changes in distributions-scripts are merged in master.
A sample of the final workflow is already available at https://circleci.com/workflow-run/2fbeef10-7614-4805-aedf-702ab979477b

The CI is no longer able to build the compiler in the 32bits build images. Yet the release process is still able probably since the process is different.
In case some steps fails, it will be possible to extract artifacts along de way
Since darwin jobs are too slow, this will reduce the overall build time.
@bcardiff bcardiff marked this pull request as ready for review October 8, 2019 13:29
@bcardiff bcardiff merged commit 0505d1c into master Oct 8, 2019
@bcardiff bcardiff added this to the 0.32.0 milestone Oct 8, 2019
@bcardiff bcardiff mentioned this pull request Jun 19, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant