Add new nextcloud chart #5180

affix · 2018-04-21T19:11:24Z

What this PR does / why we need it:
This PR adds a nextcloud chart to incubator

Nextcloud offers industry-leading on-premises file sync and online collaboration technology. Our expertise is in combining the convenience and ease of use of consumer-grade solutions like Dropbox and Google Drive with the security, privacy and control business needs.

Special notes for your reviewer:
This is loosely based around the owncloud chart, However I as many other prefer to use nextcloud for some of its optimisations.

Uses the official Nextcloud Docker image

affix · 2018-04-21T19:23:33Z

/assign @mattfarina

bbriggs · 2018-04-23T15:32:22Z

I was literally writing a chart for that this weekend. Glad to see someone else took an interest too.

unguiculus · 2018-04-23T16:52:57Z

/assign

mattfarina

Thanks for submitting the pull request. I know of several people who want to use nextcloud on Kubernetes. This is great.

I did a quick review to provide some feedback.

mattfarina · 2018-04-26T14:55:53Z

incubator/nextcloud/Chart.yaml

+sources:
+- https://github.com/nextcloud/server
+maintainers:
+- name: Keiran Smith


The name should be your github username. That helps us to know who to contact when there's an issue with the chart.

mattfarina · 2018-04-26T15:03:46Z

incubator/nextcloud/OWNERS

+- tompizmor
+- sameersbn
+- carrodher
+- affix


For the time you can drop this file. It looks like you copied this from a chart managed by Bitnami. Since you're not a member of the Kubernetes org at the moment this isn't going to enable you to review or merge for the chart. Once you have some more contributions you can apply for membership and then a file like this will make sense.

mattfarina · 2018-04-26T15:05:50Z

incubator/nextcloud/values.yaml

+resources:
+  requests:
+    memory: 512Mi
+    cpu: 300m


It is generally not advisable to provide hard resource limits to workloads and leave it configurable unless the workload requires such quantity bare minimum to function.

This is from the reviewers guide.

Can you comment these out unless this is the minimum to function.

mattfarina · 2018-04-26T15:07:29Z

incubator/nextcloud/values.yaml

+##
+ingress:
+  enabled: false
+  servicePort: 80


Does the container image allow for both http and https? It might be nice to allow for secure out of the box. That doesn't need to be part of the first PR but would be nice to add.

mattfarina · 2018-04-26T15:11:15Z

incubator/nextcloud/values.yaml

+image:
+  registry: docker.io
+  repository: nextcloud
+  tag: latest


Can you update the image information to follow our standard layout? See the guide for more information and an example.

Also, please use a tagged version rather than latest. This will enable consistency between environments.

mattfarina · 2018-04-26T15:14:13Z

incubator/nextcloud/templates/ingress.yaml

@@ -0,0 +1,28 @@
+{{- if .Values.ingress.enabled }}
+apiVersion: {{ required "A valid .Values.networkPolicyApiVersion entry required!" .Values.networkPolicyApiVersion }}


Why is the API version a values parameter?

mattfarina · 2018-04-26T15:14:40Z

incubator/nextcloud/templates/deployment.yaml

@@ -0,0 +1,103 @@
+{{- if include "nextcloud.host" . -}}
+apiVersion: extensions/v1beta1


Can you use the apps/v1beta2 version instead?

mattfarina · 2018-04-26T15:15:23Z

incubator/nextcloud/requirements.yaml

@@ -0,0 +1,5 @@
+dependencies:
+- name: mariadb
+  version: 0.7.0


The version supports ranges. What about ~0.7.0 so that it can pull in patch level differences?

unguiculus · 2018-05-02T19:36:52Z

Please compare with what you get from helm create using Helm 2.8+ and update accordingly.

pierreozoux · 2018-05-16T19:44:31Z

I think it would make more sense to PR against stable folder. It is a pretty straight forward addition.

Also I can help maintain this chart.

Another question: how could we keep this under /nextcloud github namespace, and sync it with the official charts? This would help a lot the pace of development.

billimek · 2018-06-24T18:08:20Z

@affix are you still planning to finish this? I was about to write a chart for nextcloud as well, but will wait for this to complete - unless you want someone else to pick up where you left off?

affix · 2018-06-24T19:49:59Z

@billimek I am planning to, just been a hectic few weeks. I will get it sorted and address comments

- Change maintainer to affix - Remove OWNERS file - Comment out resources in values.yml - Default to nextcloud 13.0.4-apache - Specify Ingress api version - Make Deployment v1beta2 - Support patch level MariaDB Signed-off-by: Keiran Smith <contact@keiran.scot>

affix · 2018-06-24T20:05:46Z

@mattfarina I have made the comments you requested

- Change maintainer to affix - Remove OWNERS file - Comment out resources in values.yml - Default to nextcloud 13.0.4-apache - Specify Ingress api version - Make Deployment v1beta2 - Support patch level MariaDB Signed-off-by: Keiran Smith <contact@keiran.scot>

Signed-off-by: Keiran Smith <contact@keiran.scot>

unguiculus

Again, please compare with what you get from helm create using Helm 2.8+ and update accordingly (_helpers.tpl, standard labels, etc.).

Also, you'll need to merge in the latest master to make CI happy.

unguiculus · 2018-06-25T18:28:04Z

stable/nextcloud/README.md

+
+## Prerequisites
+
+- Kubernetes 1.4+ with Beta APIs enabled


Should be 1.9+

unguiculus · 2018-06-25T18:31:53Z

stable/nextcloud/templates/deployment.yaml

@@ -0,0 +1,103 @@
+{{- if include "nextcloud.host" . -}}
+apiVersion: extensions/v1beta1


unguiculus · 2018-06-25T18:32:52Z

stable/nextcloud/.helmignore

@@ -0,0 +1 @@
+.git


Use .helmignore you get from helm create

unguiculus · 2018-06-25T18:34:23Z

stable/nextcloud/values.yaml

+    # - secretName: nextcloud.cluster.local
+    #   hosts:
+    #     - nextcloud.cluster.local
+


I'd suggest you add one level of nesting:

nextcloud: host: loadBalancerIP: username: password: ...

billimek · 2018-06-25T19:24:29Z

In the values.yaml there are references to,

persistence:
  enabled: true
  nextcloud:
    # storageClass: "-"
    # existingClaim:
    accessMode: ReadWriteOnce
    size: 8Gi

... however it doesn't appear that anything is referencing these values. From what I gather, this should be referencing where the nextcloud data itself resides. Perhaps the following should be added:

in deployment.yaml:

        resources:
{{ toYaml .Values.resources | indent 10 }}
        volumeMounts:
        - name: apache-data
          mountPath: /var/www/html
        - name: nextcloud-data
          mountPath: /var/www/html/data
      volumes:
      - name: apache-data
      {{- if .Values.persistence.apache.enabled }}
        persistentVolumeClaim:
          claimName: {{ if .Values.persistence.apache.existingClaim }}{{ .Values.persistence.apache.existingClaim }}{{- else }}{{ template "nextcloud.fullname" . }}-apache{{- end }}
      {{- else }}
        emptyDir: {}
      {{- end }}
      - name: nextcloud-data
      {{- if .Values.persistence.nextcloud.enabled }}
        persistentVolumeClaim:
          claimName: {{ if .Values.persistence.nextcloud.existingClaim }}{{ .Values.persistence.nextcloud.existingClaim }}{{- else }}{{ template "nextcloud.fullname" . }}-nextcloud{{- end }}
      {{- else }}
        emptyDir: {}
      {{- end }}

(see new references to nextcloud-data)

A new file, nextcloud-pvc.yaml created containing,

{{- if .Values.persistence.nextcloud.enabled -}}
{{- if not .Values.persistence.nextcloud.existingClaim -}}
kind: PersistentVolumeClaim
apiVersion: v1
metadata:
  name: {{ template "nextcloud.fullname" . }}-nextcloud
spec:
  accessModes:
    - {{ .Values.persistence.nextcloud.accessMode | quote }}
  resources:
    requests:
      storage: {{ .Values.persistence.nextcloud.size | quote }}
{{- if .Values.persistence.nextcloud.storageClass }}
{{- if (eq "-" .Values.persistence.nextcloud.storageClass) }}
  storageClassName: ""
{{- else }}
  storageClassName: "{{ .Values.persistence.nextcloud.storageClass }}"
{{- end }}
{{- end }}
{{- end -}}
{{- end -}}

Also suggest moving the enabled stanza inside the respective apache and nextcloud sections so that users may choose which of the two (apache and/or nextcloud data) they want to enable.

int128 · 2018-06-27T02:21:04Z

@affix Just FYI,
I have yet another Nextcloud chart using in our team.
https://github.com/int128/devops-kompose/tree/master/nextcloud
If you don't mind me, I am happy to contribute it and help you.

billimek · 2018-06-27T03:55:26Z

The following also needs to change in order for the nextcloud autoconfiguration to work properly,

        - name: nextcloud_DATABASE_NAME
          value: {{ .Values.mariadb.mariadbDatabase | quote }}
        - name: nextcloud_DATABASE_USER
          value: {{ .Values.mariadb.mariadbUser | quote }}
        - name: nextcloud_DATABASE_PASSWORD

should be:

        - name: MYSQL_DATABASE
          value: {{ .Values.mariadb.mariadbDatabase | quote }}
        - name: MYSQL_USER
          value: {{ .Values.mariadb.mariadbUser | quote }}
        - name: MYSQL_PASSWORD

billimek · 2018-06-27T04:14:01Z

affix#1 created with the changes I suggested.

Also added the ability to set existing claims for use in NextCloud via the 'external storage' plugin.

giacomoguiulfo · 2018-09-21T21:25:31Z

Oh, I see what you are saying. Well, for development purposes you could have "drop" commits that only bump the version number. Once you are confident with the PR and everything has been addressed, just drop those. Not sure if this is the right workflow in helm/charts though.

billimek · 2018-10-15T19:48:30Z

Is there anything we are waiting on to merge this one?

giacomoguiulfo · 2018-10-15T20:45:37Z

stable/nextcloud/Chart.yaml

@@ -0,0 +1,19 @@
+name: nextcloud
+version: 0.1.1


So apparently, charts going to stable need to start at 1.0.0

Suggested change

version: 0.1.1

version: 1.0.0

anarcat

so the bitnami docker image is gone, as others have noted here. i've tried to fix all the 404s refering to that image and point instead at the official docker image managed by the nextcloud organization. i am not sure it's sufficient here because the newer image might not work the same way esp. WRT the mariadb bitnami image.

anarcat · 2018-10-24T19:18:28Z

stable/nextcloud/README.md

@@ -0,0 +1,128 @@
+# nextcloud
+
+[nextcloud](https://nextcloud.org/) is a file sharing server that puts the control and security of your own data back into your hands.


making this formal with GH's new proposed change gizmo:

Suggested change

[nextcloud](https://nextcloud.org/) is a file sharing server that puts the control and security of your own data back into your hands.

[nextcloud](https://nextcloud.com/) is a file sharing server that puts the control and security of your own data back into your hands.

anarcat · 2018-10-24T19:20:30Z

stable/nextcloud/values.yaml

@@ -0,0 +1,170 @@
+## Bitnami nextcloud image version
+## ref: https://hub.docker.com/r/bitnami/nextcloud/tags/


Suggested change

## ref: https://hub.docker.com/r/bitnami/nextcloud/tags/

## Official nextcloud image version

## ref: https://hub.docker.com/r/library/nextcloud/tags/

anarcat · 2018-10-24T19:21:55Z

stable/nextcloud/values.yaml

+  email: user@example.com
+
+## Set to `yes` to allow the container to be started with blank passwords
+## ref: https://github.com/bitnami/bitnami-docker-nextcloud#environment-variables


Suggested change

## ref: https://github.com/bitnami/bitnami-docker-nextcloud#environment-variables

## ref: https://github.com/nextcloud/docker#auto-configuration-via-environment-variables

anarcat · 2018-10-24T19:22:48Z

stable/nextcloud/values.yaml

+  enabled: true
+
+  ## Create a database
+  ## ref: https://github.com/bitnami/bitnami-docker-mariadb/blob/master/README.md#creating-a-database-on-first-run


another broken link:

Suggested change

## ref: https://github.com/bitnami/bitnami-docker-mariadb/blob/master/README.md#creating-a-database-on-first-run

## ref: https://github.com/nextcloud/docker#using-an-external-database

anarcat · 2018-10-24T19:23:12Z

stable/nextcloud/values.yaml

+  mariadbDatabase: nextcloud
+
+  ## Create a database user
+  ## ref: https://github.com/bitnami/bitnami-docker-mariadb/blob/master/README.md#creating-a-database-user-on-first-run


just remove this duplicate link

Suggested change

## ref: https://github.com/bitnami/bitnami-docker-mariadb/blob/master/README.md#creating-a-database-user-on-first-run

anarcat · 2018-10-24T19:23:41Z

stable/nextcloud/values.yaml

+  mariadbUser: nextcloud
+
+  ## Password for mariadbUser
+  ## ref: https://github.com/bitnami/bitnami-docker-mariadb/blob/master/README.md#creating-a-database-user-on-first-run


just remove this duplicate link

Suggested change

## ref: https://github.com/bitnami/bitnami-docker-mariadb/blob/master/README.md#creating-a-database-user-on-first-run

anarcat · 2018-10-24T19:24:00Z

stable/nextcloud/values.yaml

+  # mariadbPassword:
+
+  ## MariaDB admin password
+  ## ref: https://github.com/bitnami/bitnami-docker-mariadb/blob/master/README.md#setting-the-root-password-on-first-run


just remove this duplicate link

Suggested change

## ref: https://github.com/bitnami/bitnami-docker-mariadb/blob/master/README.md#setting-the-root-password-on-first-run

varac · 2018-11-21T22:04:05Z

Hej, what's the state of this ? Would be great to have it meged soon!

iMartyn · 2018-11-22T13:12:43Z

FYI the official nextcloud docker image lacks php-imap support because a maintainer deemed it to be "too big". This makes it useless for my setup, even if the helm chart works. :'(

anarcat · 2018-11-22T14:30:29Z

On 2018-11-22 05:12:51, Martyn Ranyard wrote: FYI the official nextcloud docker image lacks php-imap support because a maintainer deemed it to be "too big". This makes it useless for my setup, even if the helm chart works. :'(

Well we have to stat somewhere. There can be a billion variations for any PHP-hosted project but hopefully the chart would serve as a good foundation for any of those...

kfox1111 · 2018-12-13T15:29:01Z

Any updates?

joshuacox · 2018-12-27T05:30:06Z

+1 on getting this merged

doodlemania2 · 2018-12-31T03:16:40Z

Pretty please?

kfox1111 · 2018-12-31T16:30:15Z

If we can get something merged, then it will be easier for people to contribute bugfixes/features.

joshuacox · 2018-12-31T16:56:48Z

perhaps @affix abandoned this PR? What is left from @unguiculus review that is undone? I agree with @kfox1111 let's wrap this up and get it merged so it can get some wider testing and then new PRs.

ngortheone · 2019-01-14T21:54:28Z

Also interested in this chart. I think it is reasonable to merge this to incubating so then interested people can contribute fixes

desaintmartin · 2019-01-15T09:32:44Z

stable/nextcloud/Chart.yaml

@@ -0,0 +1,19 @@
+name: nextcloud
+version: 0.1.1


Suggested change

version: 0.1.1

version: 1.0.0

k8s-ci-robot · 2019-01-15T10:14:53Z

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: affix
To fully approve this pull request, please assign additional approvers.
We suggest the following additional approver: mattfarina

If they are not already assigned, you can assign the PR to them by writing /assign @mattfarina in a comment when ready.

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

anarcat · 2019-01-16T20:16:34Z

if someone is really interested in getting this off the ground, the way forward is to just make a new PR, because the proposed changes here do not seem to be going anywhere...

guglez · 2019-01-23T15:04:33Z

/assign @mattfarina

EmperorArthur · 2019-01-24T01:04:37Z

Came looking for a Nextcloud chart and found this PR. I see a few things haven't been fixed from the changes requested. I'd fix it, but am still learning. Hope it's still being actively worked on.

PS: The Details button on "tide" goes to a 404.

chrisingenhaag · 2019-01-27T00:01:53Z

Hey together,
I created a new, bit refactored and updated the chart. PR is #10922 . I hope I´ve got all of your feedback and that we can get this chart into stable.

Signed-off-by: Christian Ingenhaag <christian.ingenhaag@googlemail.com>

evanstucker-hates-2fa · 2019-02-05T00:01:14Z

I'm looking forward to this getting merged.

unguiculus · 2019-02-08T16:33:02Z

I'd suggest we move forward with #10922.

* add nextcloud chart Signed-off-by: Christian Ingenhaag <christian.ingenhaag@googlemail.com> * insert suggestions from reviews in #5180 Signed-off-by: Christian Ingenhaag <christian.ingenhaag@googlemail.com> * disable ingress per default Signed-off-by: Christian Ingenhaag <christian.ingenhaag@googlemail.com> * fix nextcloud e2e tests Signed-off-by: Christian Ingenhaag <christian.ingenhaag@googlemail.com>

unguiculus · 2019-02-11T18:27:16Z

Closing in favor of #10922.

* add nextcloud chart Signed-off-by: Christian Ingenhaag <christian.ingenhaag@googlemail.com> * insert suggestions from reviews in #5180 Signed-off-by: Christian Ingenhaag <christian.ingenhaag@googlemail.com> * disable ingress per default Signed-off-by: Christian Ingenhaag <christian.ingenhaag@googlemail.com> * fix nextcloud e2e tests Signed-off-by: Christian Ingenhaag <christian.ingenhaag@googlemail.com> * [nextcloud] add owners file for further contribution Signed-off-by: Christian Ingenhaag <christian.ingenhaag@googlemail.com>

* add nextcloud chart Signed-off-by: Christian Ingenhaag <christian.ingenhaag@googlemail.com> * insert suggestions from reviews in helm#5180 Signed-off-by: Christian Ingenhaag <christian.ingenhaag@googlemail.com> * disable ingress per default Signed-off-by: Christian Ingenhaag <christian.ingenhaag@googlemail.com> * fix nextcloud e2e tests Signed-off-by: Christian Ingenhaag <christian.ingenhaag@googlemail.com>

* add nextcloud chart Signed-off-by: Christian Ingenhaag <christian.ingenhaag@googlemail.com> * insert suggestions from reviews in helm#5180 Signed-off-by: Christian Ingenhaag <christian.ingenhaag@googlemail.com> * disable ingress per default Signed-off-by: Christian Ingenhaag <christian.ingenhaag@googlemail.com> * fix nextcloud e2e tests Signed-off-by: Christian Ingenhaag <christian.ingenhaag@googlemail.com> * [nextcloud] add owners file for further contribution Signed-off-by: Christian Ingenhaag <christian.ingenhaag@googlemail.com>

k8s-ci-robot added cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. needs-ok-to-test Indicates a PR that requires an org member to verify it is safe to test. size/XL Denotes a PR that changes 500-999 lines, ignoring generated files. labels Apr 21, 2018

Add new nextcloud chart

f2ac2da

affix force-pushed the nextcloud branch from 4cfae77 to f2ac2da Compare April 21, 2018 19:14

k8s-ci-robot assigned mattfarina Apr 21, 2018

k8s-ci-robot assigned unguiculus Apr 23, 2018

mattfarina reviewed Apr 26, 2018

View reviewed changes

tcassaert mentioned this pull request May 16, 2018

Kubernetes Support nextcloud/docker#87

Closed

affix added 2 commits June 24, 2018 20:55

Move nextcloud to stable

5be83c6

Changes from Code Review

5371b96

- Change maintainer to affix - Remove OWNERS file - Comment out resources in values.yml - Default to nextcloud 13.0.4-apache - Specify Ingress api version - Make Deployment v1beta2 - Support patch level MariaDB Signed-off-by: Keiran Smith <contact@keiran.scot>

affix added 4 commits June 24, 2018 21:08

Changes from Code Review

923a044

- Change maintainer to affix - Remove OWNERS file - Comment out resources in values.yml - Default to nextcloud 13.0.4-apache - Specify Ingress api version - Make Deployment v1beta2 - Support patch level MariaDB Signed-off-by: Keiran Smith <contact@keiran.scot>

Merge branch 'nextcloud' of github.com:affix/charts into nextcloud

62a1466

remove network policy values

2a476a1

Signed-off-by: Keiran Smith <contact@keiran.scot>

Minor Fixes to Deployment + Service

bac458d

unguiculus suggested changes Jun 25, 2018

View reviewed changes

changed referenced in original pull request

6adeaf8

billimek mentioned this pull request Jun 27, 2018

changed referenced in original pull request affix/charts#1

Merged

giacomoguiulfo reviewed Oct 15, 2018

View reviewed changes

anarcat suggested changes Oct 24, 2018

View reviewed changes

billimek mentioned this pull request Jan 8, 2019

nextcloud chart doesn't work anymore billimek/k8s-gitops#7

Closed

desaintmartin suggested changes Jan 15, 2019

View reviewed changes

chrisingenhaag mentioned this pull request Jan 26, 2019

add nextcloud chart #10922

Merged

3 tasks

chrisingenhaag added a commit to chrisingenhaag/charts that referenced this pull request Jan 27, 2019

insert suggestions from reviews in helm#5180

3fb125f

Signed-off-by: Christian Ingenhaag <christian.ingenhaag@googlemail.com>

unguiculus closed this Feb 11, 2019

		@@ -0,0 +1,28 @@
		{{- if .Values.ingress.enabled }}
		apiVersion: {{ required "A valid .Values.networkPolicyApiVersion entry required!" .Values.networkPolicyApiVersion }}

		@@ -0,0 +1,103 @@
		{{- if include "nextcloud.host" . -}}
		apiVersion: extensions/v1beta1

		@@ -0,0 +1,128 @@
		# nextcloud

		[nextcloud](https://nextcloud.org/) is a file sharing server that puts the control and security of your own data back into your hands.

	[nextcloud](https://nextcloud.org/) is a file sharing server that puts the control and security of your own data back into your hands.
	[nextcloud](https://nextcloud.com/) is a file sharing server that puts the control and security of your own data back into your hands.

		@@ -0,0 +1,170 @@
		## Bitnami nextcloud image version
		## ref: https://hub.docker.com/r/bitnami/nextcloud/tags/

	## ref: https://hub.docker.com/r/bitnami/nextcloud/tags/
	## Official nextcloud image version
	## ref: https://hub.docker.com/r/library/nextcloud/tags/

	## ref: https://github.com/bitnami/bitnami-docker-nextcloud#environment-variables
	## ref: https://github.com/nextcloud/docker#auto-configuration-via-environment-variables

		@@ -0,0 +1 @@
		.git

Add new nextcloud chart #5180

Add new nextcloud chart #5180

Conversation

affix commented Apr 21, 2018

affix commented Apr 21, 2018

bbriggs commented Apr 23, 2018

unguiculus commented Apr 23, 2018

mattfarina 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

unguiculus commented May 2, 2018

pierreozoux commented May 16, 2018

billimek commented Jun 24, 2018

affix commented Jun 24, 2018

affix commented Jun 24, 2018

unguiculus left a comment • 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

billimek commented Jun 25, 2018 • edited Loading

int128 commented Jun 27, 2018

billimek commented Jun 27, 2018

billimek commented Jun 27, 2018 • edited Loading

giacomoguiulfo commented Sep 21, 2018

billimek commented Oct 15, 2018

Choose a reason for hiding this comment

Choose a reason for hiding this comment

anarcat 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

varac commented Nov 21, 2018

iMartyn commented Nov 22, 2018

anarcat commented Nov 22, 2018 via email

kfox1111 commented Dec 13, 2018

joshuacox commented Dec 27, 2018

doodlemania2 commented Dec 31, 2018

kfox1111 commented Dec 31, 2018

joshuacox commented Dec 31, 2018 • edited Loading

ngortheone commented Jan 14, 2019

Choose a reason for hiding this comment

k8s-ci-robot commented Jan 15, 2019

anarcat commented Jan 16, 2019

guglez commented Jan 23, 2019

EmperorArthur commented Jan 24, 2019

chrisingenhaag commented Jan 27, 2019

evanstucker-hates-2fa commented Feb 5, 2019

unguiculus commented Feb 8, 2019

unguiculus commented Feb 11, 2019

unguiculus left a comment •

edited

Loading

billimek commented Jun 25, 2018 •

edited

Loading

billimek commented Jun 27, 2018 •

edited

Loading

joshuacox commented Dec 31, 2018 •

edited

Loading