Added Fabric8 Maven Plugin in Integration #836

piyush-garg · 2017-10-04T08:59:16Z

Changes for #732

added fabric8-maven-plugin kompose integration links and basic description

k8s-ci-robot · 2017-10-04T08:59:19Z

Thanks for your pull request. Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA).

📝 Please follow instructions at https://github.com/kubernetes/kubernetes/wiki/CLA-FAQ to sign the CLA.

It may take a couple minutes for the CLA signature to be fully registered; after that, please reply here with a new comment and we'll verify. Thanks.

If you've already signed a CLA, it's possible we don't have your GitHub username or you're using a different email address. Check your existing CLA data and verify that your email is set on your git commits.
If you signed the CLA as a corporation, please sign in with your organization's credentials at https://identity.linuxfoundation.org/projects/cncf to be authorized.
If you have done the above and are still having issues with the CLA being reported as unsigned, please email the CNCF helpdesk: helpdesk@rt.linuxfoundation.org

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

surajnarwade · 2017-10-04T09:12:59Z

hey @piyush1594 , you have to sign CLA first

aditya-konarde · 2017-10-04T09:20:40Z

docs/integrations.md

+
+### Fabric8 Maven Plugin by Red Hat
+
+__Description:__  "To bring docker-compose based JAVA application to Kubernetes/Openshift Cluster using Maven Plugin"


Use 'Java' instead of 'JAVA'. It is a registered trademark.

cdrage · 2017-10-04T15:33:01Z

@piyush1594 Ideally it would be nice to add a longer description (paragraph length) as well as quick tutorial on how to use this. Since ideally, this would be one of the "top" integrations for Kompose that's most widely used.

cdrage

Hey all!

Good work so far.

I'm pretty straight-forward when doing reviews, but please elaborate on the installation instructions. Was difficult to understand from someone who has never used Maven or Fabric8 before, and that's what most of our users will be experiencing too.

I've gone ahead and made comments!

cdrage · 2017-10-05T15:37:21Z

docs/integrations.md

+
+### Fabric8 Maven Plugin by Red Hat
+
+__Description:__  "The fabric8-maven-plugin is used to bring your Java applications onto Kubernetes or Openshift. The main task of this plugin is to build Docker Images and generate Kubernetes or  Openshift resource descriptors. A docker-compose configuration is one of the method to bring up docker compose deployments on a Kubernetes/Openshift cluster. Fabric8-maven Plugin processes the external docker-compose file and generates Kubernetes/Openshift resources using Kompose."


Openshift -> OpenShift
Images -> images
There's an extra spec between or Openshift
A Docker Compose configuration is one of the methods to bring up deployments on a Kubernetes or OpenShift cluster
Fabric8-maven -> Fabric8 Maven Plugin
The Fabric8-maven plugin processes the external docker-compose.yaml file and generates Kubernetes or OpenSHift resources via Kompose.

Please separate sentences too, it's quite long. Markdown doesn't cut-off sentences if there is a newline.

cdrage · 2017-10-05T15:37:32Z

docs/integrations.md

+__Links:__
+
+* [Quickstart](http://kompose.io/maven-example/)
+


remove newline

cdrage · 2017-10-05T15:39:05Z

docs/maven-example.md

+# Fabric8 Maven Plugin + Kompose:
+
+Lets deploy a Springboot Java application with Docker Compose file using fabric8 maven plugin to Kubernetes/Openshift.
+


Would be nice to add a "requirements" section.
ex:
Required:

Linux/macOS/Windows (or whatever platform Maven runs on)

Kompose

etc..

Figure out the requirements.

Also have installation instructions for maven (just a link would do).

Try to make it so that if someone has NO IDEA how to use Maven / Fabric8, they can easily follow this tutorial (example: me)

cdrage · 2017-10-05T15:40:18Z

docs/maven-example.md

+
+__1. Clone the example project from github__
+```bash
+$ git clone https://github.com/piyush1594/Kompose-Example.git


nitpick, but would be better having Kompose-Example lowercase, could you change the name of your repo?

cdrage · 2017-10-05T15:40:39Z

docs/maven-example.md

+
+Lets deploy a Springboot Java application with Docker Compose file using fabric8 maven plugin to Kubernetes/Openshift.
+
+__1. Clone the example project from github__


GitHub needs to be capitalized

cdrage · 2017-10-05T15:41:06Z

docs/maven-example.md

+```
+$ mvn fabric8:install
+```
+This command installs the `kompose` and other binaries on the host.


what other binaries? link?

this command: mvn fabric8:install has no description... What are we doing here? I believe the sentence "This command installs.." should be above it. Also needs to be changed to "This command installs Fabric8 and Kompose" or something similar.

cdrage · 2017-10-05T15:41:18Z

docs/maven-example.md

+This command installs the `kompose` and other binaries on the host.
+
+
+__3. Add the fabric8 maven plugin to project__ 


Fabric8 needs to be capitalized?

cdrage · 2017-10-05T15:43:38Z

docs/maven-example.md

+```
+$ mvn io.fabric8:fabric8-maven-plugin:3.5.28:setup
+```
+This command adds the fabric8 maven plugin to `pom.xml` of project. 


No need to start sentences with "This command".

You can simply say: "Add the Fabric8 maven plugin to the pom.xml of the project.

Speaking of pom.xml, as someone who has never used Maven, can you describe what this does? Need to elaborate on the instructions.

cdrage · 2017-10-05T15:43:53Z

docs/maven-example.md

+
+__5. Deploy application on Kubernetes/OpenShift__
+
+ Make sure that Kubernetes/OpenShift cluster or Minikube/MiniShift is running.


MiniShift is suppose to be minishift

Perhaps add instructions to "start" minishift or minikube.

cdrage · 2017-10-05T15:44:26Z

docs/maven-example.md

+$ mvn fabric8:deploy  
+```
+
+Now that your service has been deployed, let's access it by querying `pod`, `service` from Kubernetes/OpenShift.


Where are the instructions on how to access it?

cdrage · 2017-10-05T15:46:00Z

docs/integrations.md

+
+__Links:__
+
+* [Quickstart](http://kompose.io/maven-example/)


change link to /docs/maven-example

piyush-garg · 2017-10-06T08:53:19Z

@cdrage updated integrations.md and maven-example.md

surajnarwade · 2017-10-06T11:23:10Z

docs/integrations.md

+### Fabric8 Maven Plugin by Red Hat
+
+__Description:__  "Maven is one of the widely used build tool for Java applications. The Fabric8 Maven Plugin is a maven extension that simplifies the deployment of Java applications to Kubernetes or OpenShift cluster. The main task of this plugin is to build Docker images, generate Kubernetes or OpenShift resource descriptors and run/deploy the application on Kubernetes or OpenShift cluster. Plugin has wide range of configuration options and Docker Compose is one of them to bring up deployments on Kubernetes or OpenShift cluster. The Fabric8 Maven Plugin processes the external `docker-compose.yml` file and generates Kubernetes or OpenShift resources via Kompose."
+__Links:__


add blank line before __link:__

surajnarwade · 2017-10-06T11:25:22Z

docs/integrations.md

+
+* [Quickstart](/docs/maven-example)
+
+* [Documentation](https://maven.fabric8.io/)


it will be better if you point towards part of maven docs where kompose is being used.
like, https://maven.fabric8.io/#fabric8:resource

surajnarwade · 2017-10-06T11:26:37Z

docs/maven-example.md

+
+Now that your service has been deployed, let's access it by querying `pod`, `service` from Kubernetes or OpenShift.
+```bash
+$ kubectl get pods


show output of these commands as well

surajnarwade · 2017-10-06T11:26:52Z

docs/maven-example.md

+$ kubectl get pods
+```
+```bash
+$ kubectl get svc


same here, show output of command

surajnarwade · 2017-10-06T11:27:23Z

docs/maven-example.md

+```bash
+$ kubectl get svc
+```
+


remove extra line

surajnarwade · 2017-10-06T11:28:32Z

docs/maven-example.md

+
+__1. Clone the example project from GitHub__
+```bash
+$ git clone https://github.com/piyush1594/kompose-example.git


name of repo kompose-maven-example will be good, I think

surajnarwade · 2017-10-06T11:29:36Z

docs/maven-example.md

+Lets deploy a Springboot Java application with Docker Compose file using Fabric8 Maven Plugin to Kubernetes or OpenShift.
+##### Requirements
+* Linux or MacOS or Windows
+* JDK 1.7+


openjdk installation link will be better here, http://openjdk.java.net/install/

surajnarwade · 2017-10-06T11:30:29Z

docs/maven-example.md

+```bash
+$ git clone https://github.com/piyush1594/kompose-example.git
+```
+Navigate (cd) to `kompose-example` directory.


mention command

$ cd kompose-example

piyush-garg · 2017-10-06T14:02:02Z

@surajnarwade @cdrage Updates done.

cdrage

Still needs improvement on how to use Maven.. Add tutorial on how to use Maven + Kompose from a total beginners perspective.

cdrage · 2017-10-06T14:01:18Z

docs/maven-example.md

+$ kubectl get pods
+NAME                                    READY     STATUS      RESTARTS   AGE
+springboot-docker-compose-1-vk80j       1/1       Running     0          5m
+springboot-docker-compose-s2i-1-build   0/1       Completed   0          7m


this still hasn't completed? 0/1

you need to add instructions on how to actually access these services.

hint: minikube service sprintboot would possibly bring it up.

Its s2i-1-build pod as I have observed numerous time shows readiness like this only
But status column shows Completed

May be we can remove the build pod entry or give the hint: minikube service sprintboot

Okay.

No. It wouldn't be wise to remove an entry or else people would be wondering what in the world it this other container.

Yes, adding an instruction on how to access via minikube service would be wise!

cdrage · 2017-10-06T14:02:22Z

docs/integrations.md

+
+### Fabric8 Maven Plugin by Red Hat
+
+__Description:__  "Maven is one of the widely used build tool for Java applications. The Fabric8 Maven Plugin is a maven extension that simplifies the deployment of Java applications to Kubernetes or OpenShift cluster. The main task of this plugin is to build Docker images, generate Kubernetes or OpenShift resource descriptors and run/deploy the application on Kubernetes or OpenShift cluster. Plugin has wide range of configuration options and Docker Compose is one of them to bring up deployments on Kubernetes or OpenShift cluster. The Fabric8 Maven Plugin processes the external `docker-compose.yml` file and generates Kubernetes or OpenShift resources via Kompose."


again, please separate this into different sentences... add some newlines.

cdrage · 2017-10-06T14:02:30Z

docs/maven-example.md

@@ -0,0 +1,74 @@
+# Fabric8 Maven Plugin + Kompose:
+Lets deploy a Springboot Java application with Docker Compose file using Fabric8 Maven Plugin to Kubernetes or OpenShift.
+##### Requirements


need newline

cdrage · 2017-10-06T14:03:07Z

docs/maven-example.md

+* Linux or MacOS or Windows
+* JDK 1.7+ - [JDK Quick Installation Guide](http://openjdk.java.net/install/)
+* Maven 3.x+ - [Maven Installation Guide](https://maven.apache.org/install.html) 
+* Kompose


no link to kompose

Is it really needed? as the tutorial is on Kompose website only.

Yes. Other ones have links. May as well add one. It doesn't hurt.

cdrage · 2017-10-06T14:04:09Z

can you add a WIP (work in progress) into your title since you're still working on this @piyush1594

cdrage · 2017-10-06T14:04:45Z

you also need to merge your commits (one commit)

cdrage · 2017-10-06T14:09:29Z

docs/maven-example.md

+* JDK 1.7+ - [JDK Quick Installation Guide](http://openjdk.java.net/install/)
+* Maven 3.x+ - [Maven Installation Guide](https://maven.apache.org/install.html) 
+* Kompose
+


Perhaps add a previous step here saying that you must install Maven, here are the links to install to windows, etc. Add some commands here, etc.

+* Maven 3.x+ - [Maven Installation Guide] here we have given the link for installation guide. Isn't it sufficient?
In that way it would stay consistent with Java installation link in the same section.

Alright. 👍 Let's just keep it like this for-now.

cdrage · 2017-10-06T14:09:57Z

docs/maven-example.md

+$ cd kompose-maven-example
+```
+__2. Install Kompose through Maven__ 
+```


needs some spacing here, try to always keep a newline between ``` and new headers.

cdrage · 2017-10-06T14:10:18Z

docs/maven-example.md

+```
+$ mvn fabric8:install
+```
+This command installs the `kompose` and 'fabric8' on the host.


fabric8 should be

`fabric8`

cdrage · 2017-10-06T14:10:35Z

docs/maven-example.md

+```
+This command installs the `kompose` and 'fabric8' on the host.
+
+__3. Add Fabric8 Maven Plugin to project__     


to your project

cdrage · 2017-10-06T14:10:46Z

docs/maven-example.md

+```
+$ mvn io.fabric8:fabric8-maven-plugin:3.5.28:setup
+```
+Adds the Fabric8 Maven Plugin configuration to `pom.xml` of project. `pom.xml` is manifest or deployment descriptor file of maven project.


of a maven project

cdrage · 2017-10-06T14:10:56Z

docs/maven-example.md

+```
+Adds the Fabric8 Maven Plugin configuration to `pom.xml` of project. `pom.xml` is manifest or deployment descriptor file of maven project.
+
+__4. Configure Fabric8 Maven Plugin to use Docker Compose file__


to use a Docker Compose file

cdrage · 2017-10-06T14:11:02Z

docs/maven-example.md

+  <groupId>io.fabric8</groupId>
+  <artifactId>fabric8-maven-plugin</artifactId>
+
+  <!-- ... -->


what's this?

cdrage · 2017-10-06T14:11:31Z

docs/maven-example.md

+Add the `<configuration>` and `<executions>` sections to `pom.xml` as shown in above `pom.xml` snippet. Update the `<composeFile>` to provide the relative path of Docker Compose file from `pom.xml`
+
+__5. Deploy application on Kubernetes or OpenShift__
+ Make sure that Kubernetes/OpenShift cluster or Minikube/minishift is running. In case, if anything of this is not running, you can run Minikube to test this application by using following command. 


should be minishift

Make a newline here for separated sentences.

cdrage · 2017-10-06T14:11:54Z

docs/maven-example.md

+$ minikube start
+```
+ Below command deploys this application on Kubernetes or OpenShift.
+```bash


again, add some newlines between "```" and sentences

cdrage · 2017-10-06T14:12:39Z

docs/maven-example.md

+$ kubectl get svc
+NAME                        CLUSTER-IP       EXTERNAL-IP   PORT(S)    AGE
+springboot-docker-compose   172.30.205.137   <none>        8080/TCP   6m
+```


you need to add a tutorial on how to actually access these... bringing up the web interface aka minikube service foobar or minishift openshift service foobar if a route has been made.

piyush-garg · 2017-10-06T16:11:55Z

@cdrage done changes.

surajnarwade · 2017-10-09T05:37:11Z

docs/maven-example.md

+Created the new window in existing browser session.
+```
+
+It will open your application endpoint in default browser.


can you add screenshot of output screen ?

surajnarwade · 2017-10-09T05:38:31Z

docs/integrations.md

+
+### Fabric8 Maven Plugin by Red Hat
+
+__Description:__  "Maven is one of the widely used build tool for Java applications. The Fabric8 Maven Plugin is a maven extension that simplifies the deployment of Java applications to Kubernetes or OpenShift cluster. 


one of the widely used build tools

surajnarwade · 2017-10-09T06:25:38Z

docs/maven-example.md

+
+Add the `<configuration>` and `<executions>` sections to `pom.xml` as shown in above `pom.xml` snippet. Update the `<composeFile>` to provide the relative path of Docker Compose file from `pom.xml`
+
+__5. Deploy application on Kubernetes or OpenShift__


add blank line here

@piyush1594 please fix this comment then LGTM.

@cdrage I already fixed it in the last commit.

hrishin · 2017-10-09T11:46:00Z

docs/maven-example.md

+$ git clone https://github.com/piyush1594/kompose-maven-example.git
+```
+
+Change current directory to `kompose-maven-example` directory. Use command


It would better if we remove 'use command'

Fixed typo Updated documents Updated integrations.md and maven-example.md Added output for commands and some minor updates Added coomand to access service and minor updates Fixed minor typing errors Added screenshot of output Added output image in image folder and minor typing errors Fixed typo Updated minor typos

piyush-garg · 2017-10-10T12:38:07Z

@cdrage Review needed.

cdrage · 2017-10-10T16:22:25Z

LGTM after minor comment fix.

cdrage · 2017-10-10T19:57:20Z

LGTM.

k8s-ci-robot added cncf-cla: no Indicates the PR's author has not signed the CNCF CLA. size/S Denotes a PR that changes 10-29 lines, ignoring generated files. labels Oct 4, 2017

aditya-konarde reviewed Oct 4, 2017

View reviewed changes

piyush-garg force-pushed the docs-update branch from 00cce64 to 2fad6be Compare October 5, 2017 15:25

k8s-ci-robot added size/M Denotes a PR that changes 30-99 lines, ignoring generated files. and removed size/S Denotes a PR that changes 10-29 lines, ignoring generated files. labels Oct 5, 2017

cdrage reviewed Oct 5, 2017

View reviewed changes

docs/integrations.md Outdated

__Links:__

* [Quickstart](http://kompose.io/maven-example/)

Copy link

Member

cdrage Oct 5, 2017

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

change link to /docs/maven-example

piyush-garg force-pushed the docs-update branch from 67a0bac to ab9daea Compare October 6, 2017 09:28

k8s-ci-robot added cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. and removed cncf-cla: no Indicates the PR's author has not signed the CNCF CLA. labels Oct 6, 2017

surajnarwade reviewed Oct 6, 2017

View reviewed changes

docs/maven-example.md Outdated

$ kubectl get pods

```

```bash

$ kubectl get svc

Copy link

Contributor

surajnarwade Oct 6, 2017

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

same here, show output of command

surajnarwade reviewed Oct 6, 2017

View reviewed changes

docs/maven-example.md Outdated

```bash

$ kubectl get svc

```

Copy link

Contributor

surajnarwade Oct 6, 2017

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

remove extra line

surajnarwade reviewed Oct 6, 2017

View reviewed changes

cdrage requested changes Oct 6, 2017

View reviewed changes

piyush-garg changed the title ~~Added Fabric8 Maven Plugin in Integration~~ [WIP] Added Fabric8 Maven Plugin in Integration Oct 6, 2017

k8s-ci-robot added the do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. label Oct 6, 2017

piyush-garg force-pushed the docs-update branch from 0c152ae to 89667cc Compare October 6, 2017 16:10

surajnarwade reviewed Oct 9, 2017

View reviewed changes

k8s-ci-robot added size/L Denotes a PR that changes 100-499 lines, ignoring generated files. and removed size/M Denotes a PR that changes 30-99 lines, ignoring generated files. labels Oct 9, 2017

surajnarwade reviewed Oct 9, 2017

View reviewed changes

hrishin reviewed Oct 9, 2017

View reviewed changes

piyush-garg force-pushed the docs-update branch from b7385b9 to d0fff4c Compare October 9, 2017 12:41

surajnarwade changed the title ~~[WIP] Added Fabric8 Maven Plugin in Integration~~ Added Fabric8 Maven Plugin in Integration Oct 10, 2017

k8s-ci-robot removed the do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. label Oct 10, 2017

cdrage merged commit 903acfd into kubernetes:master Oct 10, 2017

surajnarwade mentioned this pull request Oct 16, 2017

Add documentation on Kompose integrations #732

Closed


		### Fabric8 Maven Plugin by Red Hat

		__Description:__ "To bring docker-compose based JAVA application to Kubernetes/Openshift Cluster using Maven Plugin"


		### Fabric8 Maven Plugin by Red Hat

		__Description:__ "The fabric8-maven-plugin is used to bring your Java applications onto Kubernetes or Openshift. The main task of this plugin is to build Docker Images and generate Kubernetes or Openshift resource descriptors. A docker-compose configuration is one of the method to bring up docker compose deployments on a Kubernetes/Openshift cluster. Fabric8-maven Plugin processes the external docker-compose file and generates Kubernetes/Openshift resources using Kompose."

		# Fabric8 Maven Plugin + Kompose:

		Lets deploy a Springboot Java application with Docker Compose file using fabric8 maven plugin to Kubernetes/Openshift.


		Lets deploy a Springboot Java application with Docker Compose file using fabric8 maven plugin to Kubernetes/Openshift.

		__1. Clone the example project from github__

		This command installs the `kompose` and other binaries on the host.


		__3. Add the fabric8 maven plugin to project__


		__5. Deploy application on Kubernetes/OpenShift__

		Make sure that Kubernetes/OpenShift cluster or Minikube/MiniShift is running.


		* [Quickstart](/docs/maven-example)

		* [Documentation](https://maven.fabric8.io/)


		### Fabric8 Maven Plugin by Red Hat

		__Description:__ "Maven is one of the widely used build tool for Java applications. The Fabric8 Maven Plugin is a maven extension that simplifies the deployment of Java applications to Kubernetes or OpenShift cluster. The main task of this plugin is to build Docker images, generate Kubernetes or OpenShift resource descriptors and run/deploy the application on Kubernetes or OpenShift cluster. Plugin has wide range of configuration options and Docker Compose is one of them to bring up deployments on Kubernetes or OpenShift cluster. The Fabric8 Maven Plugin processes the external `docker-compose.yml` file and generates Kubernetes or OpenShift resources via Kompose."


		Add the `<configuration>` and `<executions>` sections to `pom.xml` as shown in above `pom.xml` snippet. Update the `<composeFile>` to provide the relative path of Docker Compose file from `pom.xml`

		__5. Deploy application on Kubernetes or OpenShift__

Added Fabric8 Maven Plugin in Integration #836

Added Fabric8 Maven Plugin in Integration #836

Conversation

piyush-garg commented Oct 4, 2017

k8s-ci-robot commented Oct 4, 2017

surajnarwade commented Oct 4, 2017

Choose a reason for hiding this comment

cdrage commented Oct 4, 2017 • edited Loading

cdrage left a comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

piyush-garg commented Oct 6, 2017

surajnarwade Oct 6, 2017 • edited Loading

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

piyush-garg commented Oct 6, 2017

cdrage left a comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

piyush-garg Oct 6, 2017 • edited Loading

Choose a reason for hiding this comment

cdrage Oct 6, 2017 • edited Loading

Choose a reason for hiding this comment

cdrage commented Oct 6, 2017

cdrage commented Oct 6, 2017

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

piyush-garg commented Oct 6, 2017

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

piyush-garg commented Oct 10, 2017

cdrage commented Oct 10, 2017

cdrage commented Oct 10, 2017

cdrage commented Oct 4, 2017 •

edited

Loading

surajnarwade Oct 6, 2017 •

edited

Loading

piyush-garg Oct 6, 2017 •

edited

Loading

cdrage Oct 6, 2017 •

edited

Loading