oc describe build output improvements #8293

php-coder · 2016-03-30T13:25:02Z

Trello card: https://trello.com/c/l8CyWprA/863-3-clean-up-the-build-describe-output
Fix #5247

Note: build config describer mostly unchanged.

Example output:

$ oc describe build ruby-hello-world-1 
Name:           ruby-hello-world-1
Created:        58 seconds ago
Labels:         app=ruby-hello-world
                buildconfig=ruby-hello-world
                openshift.io/build-config.name=ruby-hello-world
Annotations:    openshift.io/build.number=1
                openshift.io/build.pod-name=ruby-hello-world-1-build

Status:         Complete
Started:        Fri, 01 Apr 2016 14:22:36 UTC
Duration:       46s
Build Config:   ruby-hello-world
Build Pod:      ruby-hello-world-1-build

Strategy:           Source
URL:                https://github.com/openshift/ruby-hello-world.git
Commit:             bd94cbb (Merge pull request #52 from bparees/stdout)
Author/Committer:   Ben Parees
From Image:         DockerImage centos/ruby-22-centos7@sha256:990326b8ad8c4ae2619b24d019b7871bb10ab08c41e9d5b19d0b72cb0200e28c
Output to:          ImageStreamTag ruby-hello-world:latest
Push Secret:        builder-dockercfg-m9x7z

Events:
  FirstSeen LastSeen    Count   From                SubobjectPath           Type        Reason      Message
  --------- --------    -----   ----                -------------           --------    ------      -------
  58s       58s     1   {default-scheduler }                        Normal      Scheduled   Successfully assigned ruby-hello-world-1-build to localhost.localdomain
  57s       57s     1   {kubelet localhost.localdomain} spec.containers{sti-build}  Normal      Pulled      Container image "openshift/origin-sti-builder:v1.1.4" already present on machine
  56s       56s     1   {kubelet localhost.localdomain} spec.containers{sti-build}  Normal      Created     Created container with docker id 328f9f39f368
  56s       56s     1   {kubelet localhost.localdomain} spec.containers{sti-build}  Normal      Started     Started container with docker id 328f9f39f368

$ oc describe bc/ruby-hello-world
Name:           ruby-hello-world
Created:        11 minutes ago
Labels:         app=ruby-hello-world
Annotations:    openshift.io/generated-by=OpenShiftNewApp
Latest Version: 1

Strategy:           Source
URL:                https://github.com/openshift/ruby-hello-world.git
From Image:         ImageStreamTag ruby-22-centos7:latest
Output to:          ImageStreamTag ruby-hello-world:latest
Triggered by:       Config, ImageChange
Webhook GitHub:     https://localhost:8443/oapi/v1/namespaces/test/buildconfigs/ruby-hello-world/webhooks/Fbklht-OP-w1EMEWOJXY/github
Webhook Generic:    https://localhost:8443/oapi/v1/namespaces/test/buildconfigs/ruby-hello-world/webhooks/xshVl6kq2OYcdLJYabPl/generic

Build               Status      Duration    Creation Time
ruby-hello-world-1  complete    46s         2016-04-01 14:22:33 +0000 UTC

jwforres · 2016-04-01T15:08:35Z

This is looking a lot better, the only thing i'm not sure about is that all the source details are before the Strategy: Source line.

php-coder · 2016-04-01T15:13:53Z

@jwforres Yes, I can re-order them:

$ oc describe build ruby-hello-world-1 
...
Strategy:       Source
From Image:     DockerImage centos/ruby-22-centos7@sha256:990326b8ad8c4ae2619b24d019b7871bb10ab08c41e9d5b19d0b72cb0200e28c
Output to:      ImageStreamTag ruby-hello-world:latest
Push Secret:    builder-dockercfg-m9x7z

URL:                https://github.com/openshift/ruby-hello-world.git
Commit:             bd94cbb (Merge pull request #52 from bparees/stdout)
Author/Committer:   Ben Parees

$ oc describe bc/ruby-hello-world
...
Strategy:           Source
From Image:         ImageStreamTag ruby-22-centos7:latest
Output to:          ImageStreamTag ruby-hello-world:latest
Triggered by:       Config, ImageChange
Webhook GitHub:     https://localhost:8443/oapi/v1/namespaces/test/buildconfigs/ruby-hello-world/webhooks/Fbklht-OP-w1EMEWOJXY/github
Webhook Generic:    https://localhost:8443/oapi/v1/namespaces/test/buildconfigs/ruby-hello-world/webhooks/xshVl6kq2OYcdLJYabPl/generic

URL:    https://github.com/openshift/ruby-hello-world.git

bparees · 2016-04-01T15:21:44Z

i'm not convinced we need a line break between the strategy/from/etc block and the url/commit/etc block.

all those items are config/input into the buildconfig or build that was run, so i think they can be grouped together ("From Image" has just as much impact on your build result as your "URL", for example, and either one can change for a given build)

putting the URL right under the strategy seems desirable since for most cases we'd then have:

Strategy: Source
URL: https://foo.bar/foo.git

which has a very natural flow to it, imo

bparees · 2016-04-01T15:22:06Z

otherwise i second @jwforres, this is a huge improvement, thanks!

bparees · 2016-04-01T15:22:49Z

will Author/Committer show two values if they are different? (under what circumstances could they be different? i'm not up on my git terminology :) )

jwforres · 2016-04-01T15:42:40Z

Author is the original author of the commit (I think) and then in cases
where someone does a rebase that includes someone else's commits, then the
committer can be different, example see the 1.3 console branch
https://github.com/openshift/origin/pull/8214/commits

On Fri, Apr 1, 2016 at 11:22 AM, Ben Parees [email protected]
wrote:

will Author/Committer show two values if they are different? (under what
circumstances could they be different? i'm not up on my git terminology :) )

—
You are receiving this because you were mentioned.
Reply to this email directly or view it on GitHub
#8293 (comment)

php-coder · 2016-04-01T15:47:42Z

will Author/Committer show two values if they are different?

@bparees Yes, it will show 2 values on the same line. For example:
Author/Committer: Ben Parees / Slava Semushin

(under what circumstances could they be different? i'm not up on my git terminology :) )

When you're cherry-pick-ing someone's commit or applying it with git am AFAIR.

bparees · 2016-04-01T16:15:18Z

@bparees Yes, it will show 2 values on the same line. For example:
Author/Committer: Ben Parees / Slava Semushin

cool, sounds good.

php-coder · 2016-04-01T17:02:19Z

i'm not convinced we need a line break between the strategy/from/etc block and the url/commit/etc block.

I've combine them together.

bparees · 2016-04-01T17:13:02Z

@php-coder can you paste a sample of the new output for a BC and build? I think we're in good shape but it'd be easier to confirm w/ a sample.

php-coder · 2016-04-01T17:18:55Z

@bparees I updated the example in the description.

bparees · 2016-04-01T17:44:31Z

I hate that we called our build sequence id "last version" but that's part of the api, so not your problem :)

can I see an example with a BC/build that declares 2+ image input sources, some with 1 path and some with multiple paths? this template contains the skeleton for such a BC:
https://github.com/openshift/origin/blob/master/test/extended/fixtures/test-imagesource-build.yaml

php-coder · 2016-04-04T12:28:18Z

can I see an example with a BC/build that declares 2+ image input sources, some with 1 path and some with multiple paths?

@bparees Yes, sure:

$ oc describe bc/imagesourcebuild
Name:           imagesourcebuild
Created:        6 minutes ago
Labels:         build=imagesourcebuild
Annotations:    <none>
Latest Version: 1

Strategy:       Source
URL:            https://github.com/openshift/ruby-hello-world.git
From Image:     ImageStreamTag openshift/ruby:latest
Force Pull:     yes
Output to:      ImageStreamTag imagesourceapp:latest
Image Source:   copies /usr/lib/jenkins/jenkins.war from openshift/jenkins:latest to injected/dir
Triggered by:   <none>

Build               Status      Duration            Creation Time
imagesourcebuild-1  running     running for 3m4s    2016-04-04 12:21:05 +0000 UTC


$ oc describe build/imagesourcebuild-1
Name:           imagesourcebuild-1
Created:        3 minutes ago
Labels:         build=imagesourcebuild
                buildconfig=imagesourcebuild
                openshift.io/build-config.name=imagesourcebuild
Annotations:    openshift.io/build.number=1
                openshift.io/build.pod-name=imagesourcebuild-1-build

Status:         Complete
Started:        Mon, 04 Apr 2016 12:21:08 UTC
Duration:       3m5s
Build Config:   imagesourcebuild
Build Pod:      imagesourcebuild-1-build

Strategy:           Source
URL:                https://github.com/openshift/ruby-hello-world.git
Commit:             bd94cbb (Merge pull request #52 from bparees/stdout)
Author/Committer:   Ben Parees
From Image:         DockerImage centos/ruby-22-centos7@sha256:990326b8ad8c4ae2619b24d019b7871bb10ab08c41e9d5b19d0b72cb0200e28c
Force Pull:         yes
Output to:          ImageStreamTag imagesourceapp:latest
Image Source:       copies /usr/lib/jenkins/jenkins.war from openshift/jenkins-1-centos7@sha256:be195f14530529244b45995aa7220b21b0eb407a37739c43ba9aaa6a5b4f0732 to injected/dir
Push Secret:        builder-dockercfg-y92fy

Events:
  FirstSeen LastSeen    Count   From                SubobjectPath           Type        Reason      Message
  --------- --------    -----   ----                -------------           --------    ------      -------
  3m        3m      1   {default-scheduler }                        Normal      Scheduled   Successfully assigned imagesourcebuild-1-build to localhost.localdomain
  3m        3m      1   {kubelet localhost.localdomain} spec.containers{sti-build}  Normal      Pulled      Container image "openshift/origin-sti-builder:v1.1.4" already present on machine
  3m        3m      1   {kubelet localhost.localdomain} spec.containers{sti-build}  Normal      Created     Created container with docker id 25acb47b78a6
  3m        3m      1   {kubelet localhost.localdomain} spec.containers{sti-build}  Normal      Started     Started container with docker id 25acb47b78a6

smarterclayton · 2016-04-04T15:08:54Z

Looking pretty good. Can we use the short name printer for the images
(there should be a helper for that, if there isn't we should create it and
open an issue to use it most places)

On Mon, Apr 4, 2016 at 8:28 AM, Vyacheslav Semushin <
[email protected]> wrote:

can I see an example with a BC/build that declares 2+ image input sources,
some with 1 path and some with multiple paths?

@bparees https://github.com/bparees Yes, sure:

$ oc describe bc/imagesourcebuildName: imagesourcebuildCreated: 6 minutes agoLabels: build=imagesourcebuildAnnotations: Latest Version: 1
Strategy: SourceURL: https://github.com/openshift/ruby-hello-world.gitFrom Image: ImageStreamTag openshift/ruby:latestForce Pull: yesOutput to: ImageStreamTag imagesourceapp:latestImage Source: copies /usr/lib/jenkins/jenkins.war from openshift/jenkins:latest to injected/dirTriggered by:
Build Status Duration Creation Timeimagesourcebuild-1 running running for 3m4s 2016-04-04 12:21:05 +0000 UTC

$ oc describe build/imagesourcebuild-1Name: imagesourcebuild-1Created: 3 minutes agoLabels: build=imagesourcebuild buildconfig=imagesourcebuild openshift.io/build-config.name=imagesourcebuildAnnotations: openshift.io/build.number=1 openshift.io/build.pod-name=imagesourcebuild-1-build
Status: CompleteStarted: Mon, 04 Apr 2016 12:21:08 UTCDuration: 3m5sBuild Config: imagesourcebuildBuild Pod: imagesourcebuild-1-build
Strategy: SourceURL: https://github.com/openshift/ruby-hello-world.gitCommit: bd94cbb (Merge pull request #52 from bparees/stdout)Author/Committer: Ben PareesFrom Image: DockerImage centos/ruby-22-centos7@sha256:990326b8ad8c4ae2619b24d019b7871bb10ab08c41e9d5b19d0b72cb0200e28cForce Pull: yesOutput to: ImageStreamTag imagesourceapp:latestImage Source: copies /usr/lib/jenkins/jenkins.war from openshift/jenkins-1-centos7@sha256:be195f14530529244b45995aa7220b21b0eb407a37739c43ba9aaa6a5b4f0732 to injected/dirPush Secret: builder-dockercfg-y92fy
Events: FirstSeen LastSeen Count From SubobjectPath Type Reason Message --------- -------- ----- ---- ------------- -------- ------ ------- 3m 3m 1 {default-scheduler } Normal Scheduled Successfully assigned imagesourcebuild-1-build to localhost.localdomain 3m 3m 1 {kubelet localhost.localdomain} spec.containers{sti-build} Normal Pulled Container image "openshift/origin-sti-builder:v1.1.4" already present on machine 3m 3m 1 {kubelet localhost.localdomain} spec.containers{sti-build} Normal Created Created container with docker id 25acb47b78a6 3m 3m 1 {kubelet localhost.localdomain} spec.containers{sti-build} Normal Started Started container with docker id 25acb47b78a6

—
You are receiving this because you were mentioned.
Reply to this email directly or view it on GitHub
#8293 (comment)

bparees · 2016-04-04T16:00:19Z

@smarterclayton i think @php-coder tried to shorten the names and found they weren't pullable once he truncated the hash... is there some other shortening mechanism you're referring to?

php-coder · 2016-04-04T16:10:22Z

@smarterclayton i think @php-coder tried to shorten the names and found they weren't pullable once he truncated the hash..

Yes, it doesn't work:

$ docker pull centos/ruby-22-centos7@sha256:990326b
Trying to pull repository docker.io/centos/ruby-22-centos7 ... failed
manifest unknown: manifest unknown
$ docker pull centos/ruby-22-centos7@sha256:990326b8ad8c4ae2619b24d019b7871bb10ab08c41e9d5b19d0b72cb0200e28c
Trying to pull repository docker.io/centos/ruby-22-centos7 ... sha256:990326b8ad8c4ae2619b24d019b7871bb10ab08c41e9d5b19d0b72cb0200e28c: Pulling from centos/ruby-22-centos7

Digest: sha256:990326b8ad8c4ae2619b24d019b7871bb10ab08c41e9d5b19d0b72cb0200e28c
Status: Image is up to date for docker.io/centos/ruby-22-centos7@sha256:990326b8ad8c4ae2619b24d019b7871bb10ab08c41e9d5b19d0b72cb0200e28c

smarterclayton · 2016-04-04T16:24:59Z

For image stream tags, the expectation is the default command is:

oc describe istag/ruby-22-centos@sha256:990326b

not docker pull.

The docker pull may be a bug that is fixable.

On Mon, Apr 4, 2016 at 12:10 PM, Vyacheslav Semushin <
[email protected]> wrote:

@smarterclayton https://github.com/smarterclayton i think @php-coder
https://github.com/php-coder tried to shorten the names and found they
weren't pullable once he truncated the hash..

Yes, it doesn't work:

$ docker pull centos/ruby-22-centos7@sha256:990326bTrying to pull repository docker.io/centos/ruby-22-centos7 ... failedmanifest unknown: manifest unknown
$ docker pull centos/ruby-22-centos7@sha256:990326b8ad8c4ae2619b24d019b7871bb10ab08c41e9d5b19d0b72cb0200e28cTrying to pull repository docker.io/centos/ruby-22-centos7 ... sha256:990326b8ad8c4ae2619b24d019b7871bb10ab08c41e9d5b19d0b72cb0200e28c: Pulling from centos/ruby-22-centos7
Digest: sha256:990326b8ad8c4ae2619b24d019b7871bb10ab08c41e9d5b19d0b72cb0200e28cStatus: Image is up to date for docker.io/centos/ruby-22-centos7@sha256:990326b8ad8c4ae2619b24d019b7871bb10ab08c41e9d5b19d0b72cb0200e28c

—
You are receiving this because you were mentioned.
Reply to this email directly or view it on GitHub
#8293 (comment)

bparees · 2016-04-04T16:28:07Z

ok but in this case it's a DockerImage, not an ImageStreamTag:

From Image: DockerImage centos/ruby-22-centos7@sha256:990326b8ad8c4ae2619b24d019b7871bb10ab08c41e9d5b19d0b72cb0200e28c

so we'd truncate when it's an imagestreamtag reference and not when it's a dockerimage reference?

smarterclayton · 2016-04-04T16:42:20Z

Yeah.

On Mon, Apr 4, 2016 at 12:28 PM, Ben Parees [email protected]
wrote:

ok but in this case it's a DockerImage, not an ImageStreamTag:

From Image: DockerImage centos/ruby-22-centos7@sha256
:990326b8ad8c4ae2619b24d019b7871bb10ab08c41e9d5b19d0b72cb0200e28c

so we'd truncate when it's an imagestreamtag reference and not when it's a
dockerimage reference?

—
You are receiving this because you were mentioned.
Reply to this email directly or view it on GitHub
#8293 (comment)

php-coder · 2016-04-04T17:14:22Z

so we'd truncate when it's an imagestreamtag reference and not when it's a dockerimage reference?

In case of imagestreamtag there is nothing to truncate because we're not showing this hash:

$ oc describe bc/s2i-test | grep ImageStreamTag
From Image:     ImageStreamTag java:8
Output to:      ImageStreamTag s2i-test:latest
$ oc get istag
NAME      DOCKER REF                                                                     UPDATED       IMAGENAME
java:8    java@sha256:cedc6a5dcba447e77a300351faeae6f926d2b22cd384d35d102fc4bfb117abd2   4 hours ago   sha256:cedc6a5dcba447e77a300351faeae6f926d2b22cd384d35d102fc4bfb117abd2

smarterclayton · 2016-04-05T04:26:02Z

The docker pull issue is fixable, but only if we can patch the registry to
redirect from the short digest to the long digest on a specific error
code. It might be worth carrying, I'll open an issue.

On Mon, Apr 4, 2016 at 12:24 PM, Clayton Coleman [email protected]
wrote:

For image stream tags, the expectation is the default command is:

oc describe istag/ruby-22-centos@sha256:990326b

not docker pull.

The docker pull may be a bug that is fixable.

On Mon, Apr 4, 2016 at 12:10 PM, Vyacheslav Semushin <
[email protected]> wrote:

@smarterclayton https://github.com/smarterclayton i think @php-coder
https://github.com/php-coder tried to shorten the names and found they
weren't pullable once he truncated the hash..

Yes, it doesn't work:

$ docker pull centos/ruby-22-centos7@sha256:990326bTrying to pull repository docker.io/centos/ruby-22-centos7 ... failedmanifest unknown: manifest unknown
$ docker pull centos/ruby-22-centos7@sha256:990326b8ad8c4ae2619b24d019b7871bb10ab08c41e9d5b19d0b72cb0200e28cTrying to pull repository docker.io/centos/ruby-22-centos7 ... sha256:990326b8ad8c4ae2619b24d019b7871bb10ab08c41e9d5b19d0b72cb0200e28c: Pulling from centos/ruby-22-centos7
Digest: sha256:990326b8ad8c4ae2619b24d019b7871bb10ab08c41e9d5b19d0b72cb0200e28cStatus: Image is up to date for docker.io/centos/ruby-22-centos7@sha256:990326b8ad8c4ae2619b24d019b7871bb10ab08c41e9d5b19d0b72cb0200e28c

—
You are receiving this because you were mentioned.
Reply to this email directly or view it on GitHub
#8293 (comment)

smarterclayton · 2016-04-05T04:34:12Z

Spawned #8354 for image pull by
short ID

On Tue, Apr 5, 2016 at 12:25 AM, Clayton Coleman [email protected]
wrote:

The docker pull issue is fixable, but only if we can patch the registry to
redirect from the short digest to the long digest on a specific error
code. It might be worth carrying, I'll open an issue.

On Mon, Apr 4, 2016 at 12:24 PM, Clayton Coleman [email protected]
wrote:

For image stream tags, the expectation is the default command is:

oc describe istag/ruby-22-centos@sha256:990326b

not docker pull.

The docker pull may be a bug that is fixable.

On Mon, Apr 4, 2016 at 12:10 PM, Vyacheslav Semushin <
[email protected]> wrote:

@smarterclayton https://github.com/smarterclayton i think @php-coder
https://github.com/php-coder tried to shorten the names and found
they weren't pullable once he truncated the hash..

Yes, it doesn't work:

$ docker pull centos/ruby-22-centos7@sha256:990326bTrying to pull repository docker.io/centos/ruby-22-centos7 ... failedmanifest unknown: manifest unknown
$ docker pull centos/ruby-22-centos7@sha256:990326b8ad8c4ae2619b24d019b7871bb10ab08c41e9d5b19d0b72cb0200e28cTrying to pull repository docker.io/centos/ruby-22-centos7 ... sha256:990326b8ad8c4ae2619b24d019b7871bb10ab08c41e9d5b19d0b72cb0200e28c: Pulling from centos/ruby-22-centos7
Digest: sha256:990326b8ad8c4ae2619b24d019b7871bb10ab08c41e9d5b19d0b72cb0200e28cStatus: Image is up to date for docker.io/centos/ruby-22-centos7@sha256:990326b8ad8c4ae2619b24d019b7871bb10ab08c41e9d5b19d0b72cb0200e28c

—
You are receiving this because you were mentioned.
Reply to this email directly or view it on GitHub
#8293 (comment)

php-coder · 2016-04-05T10:04:30Z

@smarterclayton What else I should improve as part of this PR? Add truncation for (unimplemented yet) docker's sums? Show truncated sum for ImageStreamTags?

smarterclayton · 2016-04-05T15:27:23Z

we can avoid truncation for now, are the examples up to date?

On Tue, Apr 5, 2016 at 6:04 AM, Vyacheslav Semushin <
[email protected]> wrote:

@smarterclayton https://github.com/smarterclayton What else I should
improve as part of this PR? Add truncation for (unimplemented yet) docker's
sums? Show truncated sum for ImageStreamTags?

—
You are receiving this because you were mentioned.
Reply to this email directly or view it on GitHub
#8293 (comment)

php-coder · 2016-04-05T15:30:09Z

@smarterclayton yes, they are up to date.

bparees · 2016-04-07T15:48:20Z

@jwforres @smarterclayton i'm good w/ this going into 3.2, does it have api/ux approval?

smarterclayton · 2016-04-07T16:08:08Z

Approved for 3.2

bparees · 2016-04-07T17:23:14Z

[merge]

openshift-bot · 2016-04-07T17:25:24Z

continuous-integration/openshift-jenkins/merge SUCCESS (https://ci.openshift.redhat.com/jenkins/job/merge_pull_requests_origin/5538/) (Image: devenv-rhel7_3927)

openshift-bot · 2016-04-07T17:25:25Z

Evaluated for origin merge up to b11459d

openshift-bot · 2016-04-07T17:25:26Z

[Test]ing while waiting on the merge queue

openshift-bot · 2016-04-07T17:30:24Z

Evaluated for origin test up to b11459d

openshift-bot · 2016-04-07T19:05:24Z

continuous-integration/openshift-jenkins/test FAILURE (https://ci.openshift.redhat.com/jenkins/job/test_pr_origin/2793/)

bparees · 2016-04-07T19:20:59Z

woo. nice work @php-coder

php-coder · 2016-05-09T11:46:04Z

PR kubernetes/kubernetes#23988 for showing multiple labels/annotations on multiple lines was merged.

php-coder force-pushed the gh5247_build_and_build_config_ux_improvements branch from de43648 to b75b543 Compare April 1, 2016 14:30

oc describe build: improve output.

b11459d

php-coder force-pushed the gh5247_build_and_build_config_ux_improvements branch from b75b543 to b11459d Compare April 1, 2016 16:59

php-coder changed the title ~~[WIP] oc describe build output improvements~~ oc describe build output improvements Apr 7, 2016

openshift-bot merged commit 20967ed into openshift:master Apr 7, 2016

php-coder deleted the gh5247_build_and_build_config_ux_improvements branch April 7, 2016 21:47

oc describe build output improvements #8293

oc describe build output improvements #8293

Conversation

php-coder commented Mar 30, 2016

jwforres commented Apr 1, 2016

php-coder commented Apr 1, 2016

bparees commented Apr 1, 2016

bparees commented Apr 1, 2016

bparees commented Apr 1, 2016

jwforres commented Apr 1, 2016

php-coder commented Apr 1, 2016

bparees commented Apr 1, 2016

php-coder commented Apr 1, 2016

bparees commented Apr 1, 2016

php-coder commented Apr 1, 2016

bparees commented Apr 1, 2016

php-coder commented Apr 4, 2016

smarterclayton commented Apr 4, 2016

bparees commented Apr 4, 2016

php-coder commented Apr 4, 2016

smarterclayton commented Apr 4, 2016

bparees commented Apr 4, 2016

smarterclayton commented Apr 4, 2016

php-coder commented Apr 4, 2016

smarterclayton commented Apr 5, 2016

smarterclayton commented Apr 5, 2016

php-coder commented Apr 5, 2016

smarterclayton commented Apr 5, 2016

php-coder commented Apr 5, 2016

bparees commented Apr 7, 2016

smarterclayton commented Apr 7, 2016

bparees commented Apr 7, 2016

openshift-bot commented Apr 7, 2016

openshift-bot commented Apr 7, 2016

openshift-bot commented Apr 7, 2016

openshift-bot commented Apr 7, 2016

openshift-bot commented Apr 7, 2016

bparees commented Apr 7, 2016

php-coder commented May 9, 2016