Skip to content

Conversation

hossainemruz
Copy link
Contributor

@hossainemruz hossainemruz commented May 31, 2019

Task:

  • Add chart
  • Add documentation

@hossainemruz hossainemruz requested a review from tamalsaha May 31, 2019 06:59
@hossainemruz hossainemruz requested a review from the-redback June 16, 2019 08:14
@hossainemruz hossainemruz marked this pull request as ready for review June 16, 2019 08:15
@hossainemruz hossainemruz changed the title Add chart for Function and Task Add Chart for Function and Task and documentation Jun 19, 2019
@hossainemruz hossainemruz changed the title Add Chart for Function and Task and documentation Add Chart and documentation Jul 9, 2019
@@ -44,8 +59,8 @@ BIN_PLATFORMS := $(DOCKER_PLATFORMS)
OS := $(if $(GOOS),$(GOOS),$(shell go env GOOS))
ARCH := $(if $(GOARCH),$(GOARCH),$(shell go env GOARCH))

BASEIMAGE_PROD ?= postgres:9.5-alpine
BASEIMAGE_DBG ?= postgres:9.5
BASEIMAGE_PROD ?= postgres:11.2-alpine
Copy link
Member

Choose a reason for hiding this comment

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

The first code should be for 9.x version. Let's talk on Monday. @hossainemruz

@tamalsaha
Copy link
Member

  • --enable-analytics ?


- Install Stash in your cluster following the steps [here](https://appscode.com/products/stash/0.8.3/setup/install/).

- Install [KubeDB](https://kubedb.com) in your cluster following the steps [here](https://kubedb.com/docs/0.12.0/setup/install/).
Copy link
Member

Choose a reason for hiding this comment

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

Clarify that it does not have to be provisioned by KubeDB

name: pg-backup-{{- .Chart.Version }}
labels:
{{- include "postgres-stash.labels" . | nindent 4 }}
spec:
Copy link
Member

Choose a reason for hiding this comment

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

How do we handle resources?

service:
name: my-database-service
port: 5432
secret:
Copy link
Member

Choose a reason for hiding this comment

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

We need to define the secret format.

@tamalsaha tamalsaha merged commit 1c99280 into master Jul 21, 2019
@tamalsaha tamalsaha deleted the e1 branch July 21, 2019 12:24
tamalsaha pushed a commit to stashed/elasticsearch that referenced this pull request Jul 21, 2019
- [x] Fixed Elasticdump
- [x] Added chart
- [x] added persistence support in chart
- [x] Added docs
- [x] use `es-restore-6.5` style task name instead of `es-restore-6-5` [wait for changes in stashed/postgres#2 ]
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants