? ?
avatar C-Lodder
C-Lodder
18 Apr 2017

Not sure if this is possible, but is there a way to speed up the Drone tests at all? Currently they're quite slow as we're running npm install each time (https://github.com/joomla/joomla-cms/blob/staging/.drone.yml#L24)

Maybe using something like: https://www.npmjs.com/package/npm-cache

@mbabker @dgt41 ?

avatar C-Lodder C-Lodder - open - 18 Apr 2017
avatar joomla-cms-bot joomla-cms-bot - change - 18 Apr 2017
Labels Added: ?
avatar joomla-cms-bot joomla-cms-bot - labeled - 18 Apr 2017
avatar mbabker mbabker - change - 18 Apr 2017
Labels Added: ?
avatar mbabker mbabker - labeled - 18 Apr 2017
avatar rdeutz
rdeutz - comment - 18 Apr 2017

Just to confirm we are speaking about the same, under 1 minute is slow?

avatar C-Lodder
C-Lodder - comment - 18 Apr 2017

@rdeutz - When you've got PR's that have changes being made to them on the fly, it all adds up. If there is a way to use cached packages rather than download and install them each time, do you not deem that a better approach?

avatar zero-24
zero-24 - comment - 18 Apr 2017

@yvesh told me there is something on his todo list 👍

avatar rdeutz
rdeutz - comment - 18 Apr 2017

I made a quick check, drone can't cache (it was able to do it in the past but it seems they removed this function for whatever reason). The install takes 16 seconds so there is not so much to optimise.

avatar franz-wohlkoenig franz-wohlkoenig - change - 18 Apr 2017
Category Unit Tests
avatar franz-wohlkoenig
franz-wohlkoenig - comment - 22 Apr 2017

@C-Lodder close this Issue?

avatar franz-wohlkoenig franz-wohlkoenig - change - 22 Apr 2017
Status New Discussion
avatar C-Lodder C-Lodder - change - 22 Apr 2017
Status Discussion Closed
Closed_Date 0000-00-00 00:00:00 2017-04-22 08:37:08
Closed_By C-Lodder
avatar C-Lodder C-Lodder - close - 22 Apr 2017
avatar yvesh
yvesh - comment - 22 Apr 2017

@rdeutz before we forget it, there is a way for caching, it's just different

See https://github.com/appleboy/drone-sftp-cache

Add a Comment

Login with GitHub to post a comment