search

amihaiemil / docker-java-api

Lightweight Java Docker client
BSD 3-Clause "New" or "Revised" License
270 stars 53 forks source link

Added implementation of network method for class RtDocker and test for it #330

Closed lumimul closed 4 years ago

lumimul commented 4 years ago

PR for #167 Implemented RtDocker.networks()

amihaiemil commented 4 years ago

@rultor merge it please

rultor commented 4 years ago

@rultor merge it please

@amihaiemil OK, I'll try to merge now. You can check the progress of the merge here

rultor commented 4 years ago

@rultor merge it please

@amihaiemil @lumimul Oops, I failed. You can see the full log here (spent 2min)

    at org.junit.runners.ParentRunner.run(ParentRunner.java:363)
    at org.apache.maven.surefire.junit4.JUnit4Provider.execute(JUnit4Provider.java:252)
    at org.apache.maven.surefire.junit4.JUnit4Provider.executeTestSet(JUnit4Provider.java:141)
    at org.apache.maven.surefire.junit4.JUnit4Provider.invoke(JUnit4Provider.java:112)
    at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
    at sun.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:62)
    at sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
    at java.lang.reflect.Method.invoke(Method.java:498)
    at org.apache.maven.surefire.util.ReflectionUtils.invokeMethodWithArray(ReflectionUtils.java:189)
    at org.apache.maven.surefire.booter.ProviderFactory$ProviderProxy.invoke(ProviderFactory.java:165)
    at org.apache.maven.surefire.booter.ProviderFactory.invokeProvider(ProviderFactory.java:85)
    at org.apache.maven.surefire.booter.ForkedBooter.runSuitesInProcess(ForkedBooter.java:115)
    at org.apache.maven.surefire.booter.ForkedBooter.main(ForkedBooter.java:75)

Running com.amihaiemil.docker.RtLogsTestCase
Tests run: 6, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 0.02 sec
Running com.amihaiemil.docker.UnexpectedResponseExceptionTestCase
Tests run: 7, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 0.003 sec
Running com.amihaiemil.docker.MatchStatusTestCase
Tests run: 3, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 0.002 sec
Running com.amihaiemil.docker.ListedImagesTestCase
Tests run: 3, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 0.006 sec
Running com.amihaiemil.docker.RtImageTestCase
Tests run: 13, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 0.028 sec
Running com.amihaiemil.docker.RegistryConfigAuthTestCase
Tests run: 3, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 0.003 sec
Running com.amihaiemil.docker.ListedNetworksTestCase
Tests run: 2, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 0.005 sec
Running com.amihaiemil.docker.RtPluginsTestCase
Tests run: 5, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 0.009 sec
Running com.amihaiemil.docker.AuthHttpClientTestCase
Tests run: 2, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 0.003 sec
Running com.amihaiemil.docker.RtNetworksTestCase
Tests run: 7, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 0.01 sec

Results :

Failed tests:   unsupportedOperationNetworks(com.amihaiemil.docker.UnixDockerTestCase): Expected exception: java.lang.UnsupportedOperationException
  unsupportedOperationNetworks(com.amihaiemil.docker.LocalDockerTestCase): Expected exception: java.lang.UnsupportedOperationException

Tests run: 229, Failures: 2, Errors: 0, Skipped: 0

[\u001b[1;34mINFO\u001b[m] \u001b[1m------------------------------------------------------------------------\u001b[m
[\u001b[1;34mINFO\u001b[m] \u001b[1;31mBUILD FAILURE\u001b[m
[\u001b[1;34mINFO\u001b[m] \u001b[1m------------------------------------------------------------------------\u001b[m
[\u001b[1;34mINFO\u001b[m] Total time:  24.090 s
[\u001b[1;34mINFO\u001b[m] Finished at: 2020-04-15T15:21:35Z
[\u001b[1;34mINFO\u001b[m] \u001b[1m------------------------------------------------------------------------\u001b[m
[\u001b[1;31mERROR\u001b[m] Failed to execute goal \u001b[32morg.apache.maven.plugins:maven-surefire-plugin:2.12.4:test\u001b[m \u001b[1m(default-test)\u001b[m on project \u001b[36mdocker-java-api\u001b[m: \u001b[1;31mThere are test failures.\u001b[m
[\u001b[1;31mERROR\u001b[m] \u001b[1;31m\u001b[m
[\u001b[1;31mERROR\u001b[m] \u001b[1;31mPlease refer to /home/r/repo/target/surefire-reports for the individual test results.\u001b[m
[\u001b[1;31mERROR\u001b[m] -> \u001b[1m[Help 1]\u001b[m
[\u001b[1;31mERROR\u001b[m] 
[\u001b[1;31mERROR\u001b[m] To see the full stack trace of the errors, re-run Maven with the \u001b[1m-e\u001b[m switch.
[\u001b[1;31mERROR\u001b[m] Re-run Maven using the \u001b[1m-X\u001b[m switch to enable full debug logging.
[\u001b[1;31mERROR\u001b[m] 
[\u001b[1;31mERROR\u001b[m] For more information about the errors and possible solutions, please read the following articles:
[\u001b[1;31mERROR\u001b[m] \u001b[1m[Help 1]\u001b[m http://cwiki.apache.org/confluence/display/MAVEN/MojoFailureException
container aff1bc6178d8df9de9fe24e78ebbbbc4a65294050aa56b89982e6af70233acfe is dead
Wed Apr 15 17:22:54 CEST 2020
amihaiemil commented 4 years ago

@lumimul there are some test failures, can you please fix them? Make sure the build mvn clean install -Pcheckstyle passes :D

lumimul commented 4 years ago

@lumimul there are some test failures, can you please fix them? Make sure the build mvn clean install -Pcheckstyle passes :D

yes, sorry, fix it

amihaiemil commented 4 years ago

@rultor merge it please

rultor commented 4 years ago

@rultor merge it please

@amihaiemil OK, I'll try to merge now. You can check the progress of the merge here

rultor commented 4 years ago

@rultor merge it please

@amihaiemil @lumimul Oops, I failed. You can see the full log here (spent 2min)

+ set -e
+ set -o pipefail
++ dirname ./run.sh
+ cd .
+ echo 24019
+ echo '1.68.13 bf40c2027'
1.68.13 bf40c2027
+ date
Wed Apr 15 21:03:08 CEST 2020
+ uptime
 21:03:08 up 181 days, 17:12,  0 users,  load average: 4.71, 5.03, 5.06
+ ff=default
+ image=g4s8/rultor:alpine3.10
+ rebase=false
+ head_branch=master
+ pull_title='Added implementation of network method for class RtDocker and test for it'
+ author=amihaiemil
+ directory=
+ head=git@github.com:amihaiemil/docker-java-api.git
+ pull_id=330
+ fork=git@github.com:lumimul/docker-java-api.git
+ fork_branch=167
+ squash=false
+ vars=('--env=MAVEN_OPTS=-XX:MaxPermSize=256m -Xmx1g' '--env=head=git@github.com:amihaiemil/docker-java-api.git' '--env=pull_id=330' '--env=fork=git@github.com:lumimul/docker-java-api.git' '--env=fork_branch=167' '--env=head_branch=master' '--env=pull_title=Added implementation of network method for class RtDocker and test for it' '--env=author=amihaiemil' '--env=scripts=( '\''export '\''\'\'''\''MAVEN_OPTS=-XX:MaxPermSize=256m -Xmx1g'\''\'\'''\'''\'' '\'';'\'' '\''export '\''\'\'''\''head=git@github.com:amihaiemil/docker-java-api.git'\''\'\'''\'''\'' '\'';'\'' '\''export '\''\'\'''\''pull_id=330'\''\'\'''\'''\'' '\'';'\'' '\''export '\''\'\'''\''fork=git@github.com:lumimul/docker-java-api.git'\''\'\'''\'''\'' '\'';'\'' '\''export '\''\'\'''\''fork_branch=167'\''\'\'''\'''\'' '\'';'\'' '\''export '\''\'\'''\''head_branch=master'\''\'\'''\'''\'' '\'';'\'' '\''export '\''\'\'''\''pull_title=Added implementation of network method for class RtDocker and test for it'\''\'\'''\'''\'' '\'';'\'' '\''export '\''\'\'''\''author=amihaiemil'\''\'\'''\'''\'' '\'';'\'' '\''mvn clean install -Pcheckstyle'\'' '\'';'\'' )')
+ scripts=('export '\''MAVEN_OPTS=-XX:MaxPermSize=256m -Xmx1g'\''' ';' 'export '\''head=git@github.com:amihaiemil/docker-java-api.git'\''' ';' 'export '\''pull_id=330'\''' ';' 'export '\''fork=git@github.com:lumimul/docker-java-api.git'\''' ';' 'export '\''fork_branch=167'\''' ';' 'export '\''head_branch=master'\''' ';' 'export '\''pull_title=Added implementation of network method for class RtDocker and test for it'\''' ';' 'export '\''author=amihaiemil'\''' ';' 'mvn clean install -Pcheckstyle' ';')
+ container=amihaiemil_docker-java-api_330
+ as_root=false
+ mkdir -p /home/rultor/.ssh
+ echo -e 'Host github.com\n\tStrictHostKeyChecking no\n'
+ chmod 600 /home/rultor/.ssh/config
+ git clone git@github.com:amihaiemil/docker-java-api.git repo
Cloning into 'repo'...
+ cd repo
+ git config user.email me@rultor.com
+ git config user.name rultor
+ '[' -z 'export '\''MAVEN_OPTS=-XX:MaxPermSize=256m -Xmx1g'\''' ']'
+ cd ..
+ cat
+ '[' false = true ']'
+ cat
+ chmod a+x entry.sh
+ cat
+ echo 'export '\''MAVEN_OPTS=-XX:MaxPermSize=256m -Xmx1g'\''' ';' 'export '\''head=git@github.com:amihaiemil/docker-java-api.git'\''' ';' 'export '\''pull_id=330'\''' ';' 'export '\''fork=git@github.com:lumimul/docker-java-api.git'\''' ';' 'export '\''fork_branch=167'\''' ';' 'export '\''head_branch=master'\''' ';' 'export '\''pull_title=Added implementation of network method for class RtDocker and test for it'\''' ';' 'export '\''author=amihaiemil'\''' ';' 'mvn clean install -Pcheckstyle' ';'
+ sensitive=()
+ gpgconf --reload gpg-agent
+ gpg --keyserver hkp://pool.sks-keyservers.net --verbose --recv-keys 9AF0FA4C
gpg: requesting key 9AF0FA4C from hkp server pool.sks-keyservers.net
gpgkeys: key 9AF0FA4C can't be retrieved
gpg: no valid OpenPGP data found.
gpg: Total number processed: 0
gpg: keyserver communications error: keyserver helper general error
gpg: keyserver communications error: unknown pubkey algorithm
gpg: keyserver receive failed: unknown pubkey algorithm
'cid' file is absent, container wasn't started correctly
amihaiemil commented 4 years ago

@rultor try to merge again

rultor commented 4 years ago

@rultor try to merge again

@amihaiemil OK, I'll try to merge now. You can check the progress of the merge here

rultor commented 4 years ago

@rultor try to merge again

@amihaiemil Done! FYI, the full log is here (took me 2min)

0crat commented 4 years ago

Job gh:amihaiemil/docker-java-api#330 is not assigned, can't get performer

amihaiemil commented 4 years ago

@lumimul Thanks for your contribution!