Branch: refs/heads/master
  Home:   https://github.com/jenkinsci/acceptance-test-harness
  Commit: d4544922e0793b02530a51a1c663a90e51fa412a
      
https://github.com/jenkinsci/acceptance-test-harness/commit/d4544922e0793b02530a51a1c663a90e51fa412a
  Author: Andrew Bayer <[email protected]>
  Date:   2016-06-21 (Tue, 21 Jun 2016)

  Changed paths:
    M 
src/main/java/org/jenkinsci/test/acceptance/docker/DockerContainerHolder.java
    M src/main/java/org/jenkinsci/test/acceptance/docker/DockerImage.java

  Log Message:
  -----------
  [JENKINS-36045] Retry docker spin up if needed.

Also effectively rolls back 74ec1821ced1d2ef2b2567e92e699117370ff1bf
as unneeded.


  Commit: 4ffe962bd325fe946b162fcabee7bad090a8eacb
      
https://github.com/jenkinsci/acceptance-test-harness/commit/4ffe962bd325fe946b162fcabee7bad090a8eacb
  Author: Andrew Bayer <[email protected]>
  Date:   2016-06-22 (Wed, 22 Jun 2016)

  Changed paths:
    M src/main/java/org/jenkinsci/test/acceptance/docker/Docker.java
    M src/main/java/org/jenkinsci/test/acceptance/docker/DockerImage.java

  Log Message:
  -----------
  [JENKINS-36045] Major change to how we get cids and docker logs

More testing needed to verify this actually does the trick
consistently, but it does *work* in the non-failure case for
sure. We're now detaching the container, getting the container ID from
the "docker run" output (since that's the output from a detached run)
and running a side process piping "docker logs --follow (cid)" to the
log file. That side process gets automatically killed at the end of
the test when we kill and rm the container as well.


  Commit: 414b486518f17f429fd4c9166df26a800048e348
      
https://github.com/jenkinsci/acceptance-test-harness/commit/414b486518f17f429fd4c9166df26a800048e348
  Author: Andrew Bayer <[email protected]>
  Date:   2016-06-22 (Wed, 22 Jun 2016)

  Changed paths:
    M 
src/main/java/org/jenkinsci/test/acceptance/docker/DockerContainerHolder.java
    M src/main/java/org/jenkinsci/test/acceptance/docker/DockerImage.java

  Log Message:
  -----------
  Reworded an exception, got rid of now silly retry.


  Commit: 8058b01720303655a503724157e215e2f740216d
      
https://github.com/jenkinsci/acceptance-test-harness/commit/8058b01720303655a503724157e215e2f740216d
  Author: Andrew Bayer <[email protected]>
  Date:   2016-06-22 (Wed, 22 Jun 2016)

  Changed paths:
    M src/main/java/org/jenkinsci/test/acceptance/docker/DockerImage.java

  Log Message:
  -----------
  Clean up waiting for cid


  Commit: 498945b2f94a32090185c98993ff537288d6c3c6
      
https://github.com/jenkinsci/acceptance-test-harness/commit/498945b2f94a32090185c98993ff537288d6c3c6
  Author: Andrew Bayer <[email protected]>
  Date:   2016-06-22 (Wed, 22 Jun 2016)

  Changed paths:
    M src/main/java/org/jenkinsci/test/acceptance/docker/DockerImage.java

  Log Message:
  -----------
  Clarifying fallback exception on fetching cid


  Commit: 1a9f3ede9c7ee5853bf137810d8ddcec1952ca75
      
https://github.com/jenkinsci/acceptance-test-harness/commit/1a9f3ede9c7ee5853bf137810d8ddcec1952ca75
  Author: James Nord <[email protected]>
  Date:   2016-06-23 (Thu, 23 Jun 2016)

  Changed paths:
    M src/main/java/org/jenkinsci/test/acceptance/docker/Docker.java
    M src/main/java/org/jenkinsci/test/acceptance/docker/DockerImage.java

  Log Message:
  -----------
  The stream needs to be drained before waiting for the exit.

If the stream is not drained then the process can actually block wrting
output, and in this case the process will never exit.

https://www.securecoding.cert.org/confluence/display/java/FIO07-J.+Do+not+let+external+processes+block+on+IO+buffers


  Commit: 1c50c963fa09da13de184e1df822c9046f2c8881
      
https://github.com/jenkinsci/acceptance-test-harness/commit/1c50c963fa09da13de184e1df822c9046f2c8881
  Author: James Nord <[email protected]>
  Date:   2016-06-23 (Thu, 23 Jun 2016)

  Changed paths:
    M src/main/java/org/jenkinsci/test/acceptance/docker/DockerImage.java

  Log Message:
  -----------
  add return code in case of error


  Commit: ecb48b82590fe93d2b45af5226d9687e58df52f9
      
https://github.com/jenkinsci/acceptance-test-harness/commit/ecb48b82590fe93d2b45af5226d9687e58df52f9
  Author: Andrew Bayer <[email protected]>
  Date:   2016-06-23 (Thu, 23 Jun 2016)

  Changed paths:
    M src/main/java/org/jenkinsci/test/acceptance/docker/Docker.java
    M src/main/java/org/jenkinsci/test/acceptance/docker/DockerImage.java

  Log Message:
  -----------
  Merge pull request #1 from jtnord/jenkins-36045-mk2-plus

The stream needs to be drained before waiting for the exit.


  Commit: f12d28f4058e1b3c8e2d5da3a36a8ccb7b340d98
      
https://github.com/jenkinsci/acceptance-test-harness/commit/f12d28f4058e1b3c8e2d5da3a36a8ccb7b340d98
  Author: Andrew Bayer <[email protected]>
  Date:   2016-06-23 (Thu, 23 Jun 2016)

  Changed paths:
    M src/main/java/org/jenkinsci/test/acceptance/docker/Docker.java
    M 
src/main/java/org/jenkinsci/test/acceptance/docker/DockerContainerHolder.java
    M src/main/java/org/jenkinsci/test/acceptance/docker/DockerImage.java

  Log Message:
  -----------
  Merge pull request #138 from abayer/jenkins-36045-mk2

[JENKINS-36045] Rework acquiring Docker CID.


Compare: 
https://github.com/jenkinsci/acceptance-test-harness/compare/fa8937131185...f12d28f4058e

-- 
You received this message because you are subscribed to the Google Groups 
"Jenkins Commits" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to [email protected].
For more options, visit https://groups.google.com/d/optout.

Reply via email to