Skip to content

Conversation

@PVince81
Copy link
Contributor

@PVince81 PVince81 commented Oct 5, 2016

Thanks to the phpseclib update, the last remaining failing test is gone !

@DeepDiver1975

@mention-bot
Copy link

@PVince81, thanks for your PR! By analyzing the history of the files in this pull request, we identified @DeepDiver1975 to be a potential reviewer.

@DeepDiver1975
Copy link
Member

Own repo? Move to ftp repo?

@PVince81
Copy link
Contributor Author

PVince81 commented Oct 5, 2016

Own repo? Move to ftp repo?

Yes, at some point. But I don't want to start yet another construction site here until #25422 is merged. Without that PR, every separate ext storage app would need to auto-enable files_external to work properly.

@DeepDiver1975
Copy link
Member

Pretty long run time .....

00:54:45.489 Waiting for sftp initialization......
00:54:46.834 No coverage
00:54:46.834 PHPUnit 5.5.4 by Sebastian Bergmann and contributors.
00:54:46.835 
00:54:46.835 Runtime:       PHP 7.0.10 with Xdebug 2.4.1
00:54:46.836 Configuration: /var/lib/jenkins/workspace/owncloud-core_core_PR-26283-SNFCZ5D7XTG5URIGDACQIMBGTEJ73TXNN55PZGBXKIPDWNSXVHZA/tests/phpunit-autotest-external.xml
00:54:46.837 
02:54:19.758 ............................................................Sending interrupt signal to process
02:54:30.625 Makefile:121: recipe for target 'test-external' failed
02:54:30.634 make: *** [test-external] Terminated
02:54:30.638 Terminated

@PVince81
Copy link
Contributor Author

PVince81 commented Oct 6, 2016

Wow... on my local machine it might take 5 mins or so

@DeepDiver1975
Copy link
Member

Wow... on my local machine it might take 5 mins or so

let me rerun it ....

@DeepDiver1975
Copy link
Member

Wow... on my local machine it might take 5 mins or so

did hang again - also hangs on my machine ....

@PVince81 PVince81 added this to the 9.2 milestone Oct 7, 2016
@PVince81
Copy link
Contributor Author

Let's resume this when we move SFTP out into its own repo.

@PVince81 PVince81 closed this Feb 10, 2017
@PVince81 PVince81 deleted the tests-add-sftp-to-ci branch February 10, 2017 15:45
@lock
Copy link

lock bot commented Aug 3, 2019

This thread has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.

@lock lock bot locked as resolved and limited conversation to collaborators Aug 3, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants