Skip to content

Conversation

@SergioGasquez
Copy link
Member

No description provided.

@bugadani bugadani added the skip-changelog No changelog modification needed label Apr 7, 2025
@jessebraham jessebraham enabled auto-merge April 7, 2025 11:09
@SergioGasquez SergioGasquez disabled auto-merge April 7, 2025 11:09
@SergioGasquez
Copy link
Member Author

I dont think its viable to have it this way (25-30 mins of CI runs), probably we should revert this changes and just open an issue when a scheduled CI run fails

@jessebraham
Copy link
Member

jessebraham commented Apr 7, 2025

How did we go from 5-6 minutes to 25+? 🤔

@SergioGasquez
Copy link
Member Author

SergioGasquez commented Apr 7, 2025

How did we go from 5-6 minutes to 25+? 🤔

I guess 6 minutes was the time it took to fail. cc @bugadani

@bugadani
Copy link
Contributor

bugadani commented Apr 7, 2025

No, CI used to finish in 5 minutes (https://github.com/esp-rs/esp-generate/actions/runs/13724008064), I guess caching works differently when it's always working with the main branch.

@bjoernQ
Copy link
Collaborator

bjoernQ commented Apr 7, 2025

At least for S3 there is a cache miss:

image

For C6 there wasn't

image

🤔

@bjoernQ
Copy link
Collaborator

bjoernQ commented Apr 7, 2025

But the check time increased a lot during the last few weeks

image

image

@bjoernQ
Copy link
Collaborator

bjoernQ commented Apr 7, 2025

The bump from 20 minutes to almost 3h was on 7th of March apparently

@bjoernQ
Copy link
Collaborator

bjoernQ commented Apr 7, 2025

From these

image

I can only imagine #141 to do something relevant - i.e. sometimes using nightly

Thing I don't really understand however: The last build here took 42 minutes while the last successful nightly was 4h and 17 minutes - these is always some randomness is how long it takes but that looks wild

@SergioGasquez
Copy link
Member Author

Maybe we should open an issue to investigate this, in the meantime I've created #159

@bjoernQ
Copy link
Collaborator

bjoernQ commented Apr 7, 2025

Maybe we should open an issue to investigate this, in the meantime I've created #159

Opened #160 for the investigation

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

skip-changelog No changelog modification needed

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants