You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@nuttx.apache.org by GitBox <gi...@apache.org> on 2020/04/01 03:50:40 UTC

[GitHub] [incubator-nuttx-testing] liuguo09 opened a new pull request #26: Refine arm testlists and save more jobs for later usage

liuguo09 opened a new pull request #26: Refine arm testlists and save more jobs for later usage
URL: https://github.com/apache/incubator-nuttx-testing/pull/26
 
 
   1. Refine arm testlists with 35 configs in average
   2. Remove arm-14 and arm-15 jobs to save more jobs for Windows and MacOS
   builds lately since github action free version limit up to 20 jobs.
   
   Signed-off-by: liuhaitao <li...@xiaomi.com>

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
users@infra.apache.org


With regards,
Apache Git Services

[GitHub] [incubator-nuttx-testing] btashton edited a comment on issue #26: Refine arm testlists and save more jobs for later usage

Posted by GitBox <gi...@apache.org>.
btashton edited a comment on issue #26: Refine arm testlists and save more jobs for later usage
URL: https://github.com/apache/incubator-nuttx-testing/pull/26#issuecomment-607025805
 
 
   I don't think we should worry too much. If we have more than 20 jobs they will just wait for others to free up. The startup time is fairly low now and there is still room for improvement by limiting the depth of the clones we do. 
   
   Also make distclean is kinda slow. We can be using git -xdf to reset state a lot faster. 
   
   As for the storage limits I think that only applies to private repos. 

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
users@infra.apache.org


With regards,
Apache Git Services

[GitHub] [incubator-nuttx-testing] xiaoxiang781216 merged pull request #26: Refine arm testlists and save more jobs for later usage

Posted by GitBox <gi...@apache.org>.
xiaoxiang781216 merged pull request #26: Refine arm testlists and save more jobs for later usage
URL: https://github.com/apache/incubator-nuttx-testing/pull/26
 
 
   

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
users@infra.apache.org


With regards,
Apache Git Services

[GitHub] [incubator-nuttx-testing] liuguo09 commented on a change in pull request #26: Refine arm testlists and save more jobs for later usage

Posted by GitBox <gi...@apache.org>.
liuguo09 commented on a change in pull request #26: Refine arm testlists and save more jobs for later usage
URL: https://github.com/apache/incubator-nuttx-testing/pull/26#discussion_r401509200
 
 

 ##########
 File path: testlist/arm-01.dat
 ##########
 @@ -6,6 +6,6 @@
 
 /arm/cxd56xx,CONFIG_ARMV7M_TOOLCHAIN_GNU_EABIL
 
-/arm/dm320,CONFIG_ARMV7M_TOOLCHAIN_GNU_EABIL
-
 /arm/efm32,CONFIG_ARMV7M_TOOLCHAIN_GNU_EABIL
+
+/arm/stm32f0l0g0,CONFIG_ARMV7M_TOOLCHAIN_GNU_EABIL
 
 Review comment:
   Now refine in  alphabetic order done.

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
users@infra.apache.org


With regards,
Apache Git Services

[GitHub] [incubator-nuttx-testing] liuguo09 commented on issue #26: Refine arm testlists and save more jobs for later usage

Posted by GitBox <gi...@apache.org>.
liuguo09 commented on issue #26: Refine arm testlists and save more jobs for later usage
URL: https://github.com/apache/incubator-nuttx-testing/pull/26#issuecomment-607022627
 
 
   > > github action free version limit up to 20 jobs.
   > 
   > i wasn't aware of the limit. where this kind of things are documented?
   
   https://help.github.com/en/actions/getting-started-with-github-actions/about-github-actions#usage-limits

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
users@infra.apache.org


With regards,
Apache Git Services

[GitHub] [incubator-nuttx-testing] liuguo09 commented on a change in pull request #26: Refine arm testlists and save more jobs for later usage

Posted by GitBox <gi...@apache.org>.
liuguo09 commented on a change in pull request #26: Refine arm testlists and save more jobs for later usage
URL: https://github.com/apache/incubator-nuttx-testing/pull/26#discussion_r401391841
 
 

 ##########
 File path: testlist/arm-01.dat
 ##########
 @@ -6,6 +6,6 @@
 
 /arm/cxd56xx,CONFIG_ARMV7M_TOOLCHAIN_GNU_EABIL
 
-/arm/dm320,CONFIG_ARMV7M_TOOLCHAIN_GNU_EABIL
-
 /arm/efm32,CONFIG_ARMV7M_TOOLCHAIN_GNU_EABIL
+
+/arm/stm32f0l0g0,CONFIG_ARMV7M_TOOLCHAIN_GNU_EABIL
 
 Review comment:
   If in the alphabetic order completely, we should use more level depth boards path instead.

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
users@infra.apache.org


With regards,
Apache Git Services

[GitHub] [incubator-nuttx-testing] yamt commented on issue #26: Refine arm testlists and save more jobs for later usage

Posted by GitBox <gi...@apache.org>.
yamt commented on issue #26: Refine arm testlists and save more jobs for later usage
URL: https://github.com/apache/incubator-nuttx-testing/pull/26#issuecomment-607046082
 
 
   the change looks good to me. (as far as it's merged to all repos at once)
   it covers the same set of configs as before.

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
users@infra.apache.org


With regards,
Apache Git Services

[GitHub] [incubator-nuttx-testing] btashton commented on issue #26: Refine arm testlists and save more jobs for later usage

Posted by GitBox <gi...@apache.org>.
btashton commented on issue #26: Refine arm testlists and save more jobs for later usage
URL: https://github.com/apache/incubator-nuttx-testing/pull/26#issuecomment-607025805
 
 
   I don't think we should worry too much. If we have more than 20 jobs they will just wait for others to free up. The startup time is fairly low now and there is still room for improvement by limiting the depth of the clones we do. 
   
   Also make distclean is kinda slow. We can be using git -xdf to reset state a lot faster. 

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
users@infra.apache.org


With regards,
Apache Git Services

[GitHub] [incubator-nuttx-testing] yamt commented on issue #26: Refine arm testlists and save more jobs for later usage

Posted by GitBox <gi...@apache.org>.
yamt commented on issue #26: Refine arm testlists and save more jobs for later usage
URL: https://github.com/apache/incubator-nuttx-testing/pull/26#issuecomment-607024437
 
 
   thank you.
   
   do this Storage/Minutes limits apply as well?
   https://help.github.com/en/github/setting-up-and-managing-billing-and-payments-on-github/about-billing-for-github-actions#about-billing-for-github-actions

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
users@infra.apache.org


With regards,
Apache Git Services

[GitHub] [incubator-nuttx-testing] xiaoxiang781216 commented on a change in pull request #26: Refine arm testlists and save more jobs for later usage

Posted by GitBox <gi...@apache.org>.
xiaoxiang781216 commented on a change in pull request #26: Refine arm testlists and save more jobs for later usage
URL: https://github.com/apache/incubator-nuttx-testing/pull/26#discussion_r401389203
 
 

 ##########
 File path: testlist/arm-01.dat
 ##########
 @@ -6,6 +6,6 @@
 
 /arm/cxd56xx,CONFIG_ARMV7M_TOOLCHAIN_GNU_EABIL
 
-/arm/dm320,CONFIG_ARMV7M_TOOLCHAIN_GNU_EABIL
-
 /arm/efm32,CONFIG_ARMV7M_TOOLCHAIN_GNU_EABIL
+
+/arm/stm32f0l0g0,CONFIG_ARMV7M_TOOLCHAIN_GNU_EABIL
 
 Review comment:
   Could we order the list? so we can find the untested board quickly.

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
users@infra.apache.org


With regards,
Apache Git Services

[GitHub] [incubator-nuttx-testing] yamt commented on issue #26: Refine arm testlists and save more jobs for later usage

Posted by GitBox <gi...@apache.org>.
yamt commented on issue #26: Refine arm testlists and save more jobs for later usage
URL: https://github.com/apache/incubator-nuttx-testing/pull/26#issuecomment-607018941
 
 
   > github action free version limit up to 20 jobs.
   
   i wasn't aware of the limit. where this kind of things are documented?

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
users@infra.apache.org


With regards,
Apache Git Services