Skip to content

Commit b65489c

Browse files
author
Jeff McCormick
committed
fix backup to use master secret password
1 parent 0cfa561 commit b65489c

File tree

4 files changed

+30
-4
lines changed

4 files changed

+30
-4
lines changed

client/cmd/backup.go

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -268,8 +268,9 @@ func getBackupParams(name string) (*tpr.PgBackup, error) {
268268
Into(&cluster)
269269
if err == nil {
270270
spec.BACKUP_HOST = cluster.Spec.Name
271-
spec.BACKUP_USER = cluster.Spec.PG_MASTER_USER
272-
spec.BACKUP_PASS = cluster.Spec.PG_MASTER_PASSWORD
271+
//spec.BACKUP_USER = cluster.Spec.PG_MASTER_USER
272+
//spec.BACKUP_PASS = cluster.Spec.PG_MASTER_PASSWORD
273+
spec.BACKUP_PASS = GetMasterSecretPassword(cluster.Spec.Name)
273274
spec.BACKUP_PORT = cluster.Spec.Port
274275
} else if errors.IsNotFound(err) {
275276
log.Debug(name + " is not a cluster")

client/cmd/util.go

Lines changed: 24 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -142,3 +142,27 @@ func PrintSecrets(db string) {
142142
}
143143

144144
}
145+
146+
func GetMasterSecretPassword(db string) string {
147+
148+
lo := v1.ListOptions{LabelSelector: "pg-database=" + db}
149+
secrets, err := Clientset.Secrets(Namespace).List(lo)
150+
if err != nil {
151+
log.Error("error getting list of secrets" + err.Error())
152+
return "error"
153+
}
154+
155+
log.Debug("secrets for " + db)
156+
secretName := db + "-pgmaster-secret"
157+
for _, s := range secrets.Items {
158+
log.Debug("secret : " + s.ObjectMeta.Name)
159+
if s.ObjectMeta.Name == secretName {
160+
log.Debug("pgmaster password found")
161+
return string(s.Data["password"][:])
162+
}
163+
}
164+
165+
log.Error("master secret not found for " + db)
166+
return "error"
167+
168+
}

docs/build.asciidoc

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -197,7 +197,7 @@ the following:
197197
* create a cluster (*pgo create cluster testcluster*)
198198
* show a cluster (*pgo show cluster testcluster*)
199199
* show all clusters (*pgo show cluster all*)
200-
* backup a cluster (*pgo create backup testcluster*)
200+
* backup a cluster (*pgo backup testcluster*)
201201
* show backup of cluster (*pgo show backup testcluster*)
202202
* delete a cluster (*pgo delete cluster testcluster*)
203203
* restore a cluster (*pgo create cluster restoredcluster --backup-pvc=testcluster-backup-pvc --backup-path=testcluster-backups/2017-01-01-01-01-01*)

examples/operator/create-pv.sh

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -18,7 +18,8 @@ DIR="$( cd "$( dirname "${BASH_SOURCE[0]}" )" && pwd )"
1818
echo "create the test PV and PVC using the HostPath dir"
1919
for i in {1..60}
2020
do
21-
echo "creating PV crunchy-pv-$i"
21+
echo "creating PV crunchy-pv$i"
2222
export COUNTER=$i
23+
kubectl --namespace=$NAMESPACE delete pv crunchy-pv$i
2324
envsubst < $DIR/crunchy-pv.json | kubectl --namespace=$NAMESPACE create -f -
2425
done

0 commit comments

Comments
 (0)