Skip to content

Conversation

@dhalbert
Copy link
Collaborator

First try: group by category. Ten jobs, which seems like too many, but let's see how long each one takes to run.

@dhalbert
Copy link
Collaborator Author

I think I'd like to remove trinket_m0_haxpress, feather_m0_supersized, and feather_radiofruit_zigbee from regular builds. @dastels, comments on the first two? @ladyada zigbee board is not a product?

@jerryneedell
Copy link
Collaborator

jerryneedell commented Aug 18, 2019

@dhalbert will they still be available for manual build. That is, will the board definitions remain in the repo?

@dhalbert
Copy link
Collaborator Author

dhalbert commented Aug 18, 2019

@jerryneedell Yes, I just am not sure there are many physical instances of these in the real world. Do you have some?

It's just an idea.

@jerryneedell
Copy link
Collaborator

I have a feather_m0_supersized, but I'm fine with it not be part of the build process. As long as I can build it myself.

@jerryneedell
Copy link
Collaborator

Also, if you want to stop maintaining it completely, that is OK with me - Jus t so I know to keep what I need to build it locally in the future.

@dhalbert dhalbert requested a review from tannewt August 19, 2019 02:06
@dhalbert
Copy link
Collaborator Author

@tannewt the build time for the first subjob went to 1.5 hours after I accepted the -flto merge for nRF52. I rearranged things to this. Would you be willing to accept this for now to get the jobs more balanced again (and shorter) for now until GitHub actions is working? Here's the latest build: https://travis-ci.com/adafruit/circuitpython/builds/123654081. There's a short test-only job, a shorter nRF job, and the 5 others are similar to before. So not just 5 like before, but still some empty slots for other stuff to get a chance to run.

@dhalbert dhalbert requested a review from jepler August 19, 2019 12:54
@dhalbert
Copy link
Collaborator Author

@jepler if you are willing to review this now we can speed up the builds before @tannewt wakes up :)

Copy link

@jepler jepler left a comment

Choose a reason for hiding this comment

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

I semi-manually checked that the list of boards is the same "before" and "after", and didn't spot any differences.

@dhalbert dhalbert merged commit 9343d09 into adafruit:master Aug 19, 2019
@dhalbert dhalbert deleted the travis-jobs-reorg branch August 19, 2019 13:02
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.

3 participants