Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add SFTP tests to CI #26283

Closed
wants to merge 1 commit into from
Closed

Add SFTP tests to CI #26283

wants to merge 1 commit into from

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.

3 participants