Skip to content

Commit 8ffe4ed

Browse files
author
jmccormick2001
committed
fix pgo status not ready reporting, update dockerfiles to latest pgdg and backrest versions
1 parent 7e867eb commit 8ffe4ed

15 files changed

+60
-62
lines changed

‎apiserver/statusservice/statusimpl.go‎

Lines changed: 8 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -16,11 +16,11 @@ limitations under the License.
1616
*/
1717

1818
import (
19-
log "github.com/sirupsen/logrus"
2019
"github.com/crunchydata/postgres-operator/apiserver"
2120
msgs "github.com/crunchydata/postgres-operator/apiservermsgs"
2221
"github.com/crunchydata/postgres-operator/kubeapi"
2322
"github.com/crunchydata/postgres-operator/util"
23+
log "github.com/sirupsen/logrus"
2424
"k8s.io/api/core/v1"
2525
"k8s.io/apimachinery/pkg/api/resource"
2626
"k8s.io/client-go/kubernetes"
@@ -149,9 +149,13 @@ func getNotReady(ns string) []string {
149149
return agg
150150
}
151151
for _, p := range pods.Items {
152-
for _, stat := range p.Status.ContainerStatuses {
153-
if !stat.Ready {
154-
agg = append(agg, p.ObjectMeta.Name)
152+
if p.ObjectMeta.Labels["job-name"] != "" {
153+
log.Debugf("status - skipping job pod in getNotReady %s", p.ObjectMeta.Name)
154+
} else {
155+
for _, stat := range p.Status.ContainerStatuses {
156+
if !stat.Ready {
157+
agg = append(agg, p.ObjectMeta.Name)
158+
}
155159
}
156160
}
157161
}

‎apiservermsgs/common.go‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -17,7 +17,7 @@ limitations under the License.
1717

1818
import ()
1919

20-
const PGO_VERSION = "3.5.2"
20+
const PGO_VERSION = "3.5.3"
2121

2222
// Ok status
2323
const Ok = "ok"

‎centos7/Dockerfile.pgo-apiserver.centos7‎

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -4,11 +4,11 @@ LABEL Vendor="Crunchy Data Solutions" \
44
PostgresVersion="11" \
55
PostgresFullVersion="11.2" \
66
Version="7.6" \
7-
Release="3.5.2" \
7+
Release="3.5.3" \
88
summary="Crunchy Data PostgreSQL Operator - Apiserver" \
99
description="Crunchy Data PostgreSQL Operator - Apiserver"
1010

11-
ENV PGVERSION="11" PGDG_REPO="pgdg-centos11-11-2.noarch.rpm"
11+
ENV PGVERSION="11" PGDG_REPO="pgdg-redhat-repo-latest.noarch.rpm"
1212

1313
# PGDG PostgreSQL Repository
1414

‎centos7/Dockerfile.pgo-backrest-repo.centos7‎

Lines changed: 7 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -4,19 +4,21 @@ LABEL Vendor="Crunchy Data Solutions" \
44
PostgresVersion="11" \
55
PostgresFullVersion="11.2" \
66
Version="7.6" \
7-
Release="3.5.2" \
7+
Release="3.5.3" \
88
summary="Crunchy Data PostgreSQL Operator - Apiserver" \
99
description="Crunchy Data PostgreSQL Operator - Apiserver"
1010

11-
ENV PGVERSION="11" PGDG_REPO="pgdg-centos11-11-2.noarch.rpm" BACKREST_VERSION="2.10"
11+
ENV PGVERSION="11" PGDG_REPO="pgdg-redhat-repo-latest.noarch.rpm" PGDG_REPO_DISABLE="pgdg10,pgdg96,pgdg95,pgdg94" \
12+
BACKREST_VERSION="2.13"
1213

1314
# PGDG PostgreSQL Repository
1415

1516
RUN rpm -Uvh https://download.postgresql.org/pub/repos/yum/${PGVERSION}/redhat/rhel-7-x86_64/${PGDG_REPO}
1617

1718
RUN yum -y update && \
1819
#yum -y install epel-release && \
19-
yum -y install psmisc openssh-server openssh-clients pgbackrest-"${BACKREST_VERSION}" hostname pgocps-ng && \
20+
yum -y install --disablerepo="${PGDG_REPO_DISABLE}" \
21+
psmisc openssh-server openssh-clients pgbackrest-"${BACKREST_VERSION}" hostname pgocps-ng && \
2022
yum -y clean all
2123

2224
RUN groupadd pgbackrest -g 2000 && useradd pgbackrest -u 2000 -g 2000
@@ -25,6 +27,8 @@ RUN chmod +x /usr/local/bin/pgo-backrest-repo.sh && mkdir -p /opt/cpm/bin && cho
2527

2628
ADD bin/uid_pgbackrest.sh /opt/cpm/bin
2729

30+
ADD conf/pgo-backrest-repo/.bashrc /home/pgbackrest
31+
2832
RUN chmod g=u /etc/passwd && \
2933
chmod g=u /etc/group
3034

‎centos7/Dockerfile.pgo-backrest-restore.centos7‎

Lines changed: 6 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -4,15 +4,18 @@ LABEL Vendor="Crunchy Data Solutions" \
44
PostgresVersion="11" \
55
PostgresFullVersion="11.2" \
66
Version="7.6" \
7-
Release="3.5.2" \
7+
Release="3.5.3" \
88
summary="Crunchy Data PostgreSQL Operator - pgBackRest" \
99
description="pgBackRest image that is integrated for use with Crunchy Data's PostgreSQL Operator."
1010

11-
ENV PGVERSION="11" PGDG_REPO="pgdg-centos11-11-2.noarch.rpm" BACKREST_VERSION="2.10"
11+
ENV PGVERSION="11" PGDG_REPO="pgdg-redhat-repo-latest.noarch.rpm" PGDG_REPO_DISABLE="pgdg10,pgdg96,pgdg95,pgdg94" \
12+
BACKREST_VERSION="2.13"
1213

1314
RUN rpm -Uvh https://download.postgresql.org/pub/repos/yum/${PGVERSION}/redhat/rhel-7-x86_64/${PGDG_REPO}
1415

15-
RUN yum -y update && yum -y install psmisc openssh-server openssh-clients pgbackrest-"${BACKREST_VERSION}" postgresql11-server procps-ng && yum -y clean all
16+
RUN yum -y update && yum -y install --disablerepo="${PGDG_REPO_DISABLE}" \
17+
psmisc openssh-server openssh-clients pgbackrest-"${BACKREST_VERSION}" postgresql11-server procps-ng && yum -y clean all
18+
1619

1720
VOLUME ["/sshd", "/pgdata"]
1821

‎centos7/Dockerfile.pgo-backrest.centos7‎

Lines changed: 6 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -4,15 +4,18 @@ LABEL Vendor="Crunchy Data Solutions" \
44
PostgresVersion="11" \
55
PostgresFullVersion="11.2" \
66
Version="7.6" \
7-
Release="3.5.2" \
7+
Release="3.5.3" \
88
summary="Crunchy Data PostgreSQL Operator - pgBackRest" \
99
description="pgBackRest image that is integrated for use with Crunchy Data's PostgreSQL Operator."
1010

11-
ENV PGVERSION="11" PGDG_REPO="pgdg-centos11-11-2.noarch.rpm" BACKREST_VERSION="2.10"
11+
ENV PGVERSION="11" PGDG_REPO="pgdg-redhat-repo-latest.noarch.rpm" PGDG_REPO_DISABLE="pgdg10,pgdg96,pgdg95,pgdg94" \
12+
BACKREST_VERSION="2.13"
1213

1314
RUN rpm -Uvh https://download.postgresql.org/pub/repos/yum/${PGVERSION}/redhat/rhel-7-x86_64/${PGDG_REPO}
1415

15-
RUN yum -y update && yum -y install postgresql11-server && yum -y install pgbackrest-"${BACKREST_VERSION}" && yum -y clean all
16+
RUN yum -y update && yum -y install --disablerepo="${PGDG_REPO_DISABLE}" \
17+
postgresql11-server && yum -y install pgbackrest-"${BACKREST_VERSION}" && yum -y clean all
18+
1619

1720
RUN mkdir -p /opt/cpm/bin /pgdata /backrestrepo && chown -R 26:26 /opt/cpm
1821
ADD bin/pgo-backrest/ /opt/cpm/bin

‎centos7/Dockerfile.pgo-load.centos7‎

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -5,7 +5,7 @@ LABEL name="crunchydata/pgo-load" \
55
PostgresVersion="11" \
66
PostgresFullVersion="11.2" \
77
Version="7.6" \
8-
Release="3.5.2" \
8+
Release="3.5.3" \
99
url="https://crunchydata.com" \
1010
summary="Crunchy Data PostgreSQL Operator - Load Data" \
1111
description="Loads a CSV or JSON file into the database." \
@@ -14,7 +14,7 @@ LABEL name="crunchydata/pgo-load" \
1414
io.openshift.expose-services="" \
1515
io.openshift.tags="crunchy,database"
1616

17-
ENV PGVERSION="11" PGDG_REPO="pgdg-centos11-11-2.noarch.rpm"
17+
ENV PGVERSION="11" PGDG_REPO="pgdg-redhat-repo-latest.noarch.rpm"
1818

1919
# PGDG PostgreSQL Repository
2020

‎centos7/Dockerfile.pgo-lspvc.centos7‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -2,7 +2,7 @@ FROM centos:7
22

33
LABEL Vendor="Crunchy Data Solutions" \
44
Version="7.6" \
5-
Release="3.5.2" \
5+
Release="3.5.3" \
66
summary="Crunchy Data PostgreSQL Operator - Display PVC" \
77
description="Crunchy Data PostgreSQL Operator - Display contents of a PVC"
88

‎centos7/Dockerfile.pgo-rmdata.centos7‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -2,7 +2,7 @@ FROM centos:7
22

33
LABEL Vendor="Crunchy Data Solutions" \
44
Version="7.6" \
5-
Release="3.5.2" \
5+
Release="3.5.3" \
66
summary="Crunchy Data PostgreSQL Operator - Remove Data" \
77
description="Crunchy Data PostgreSQL Operator - Remove Data"
88

‎centos7/Dockerfile.pgo-scheduler.centos7‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -2,7 +2,7 @@ FROM centos:7
22

33
LABEL Vendor="Crunchy Data Solutions" \
44
Version="7.6" \
5-
Release="3.5.2" \
5+
Release="3.5.3" \
66
summary="Crunchy Data PostgreSQL Operator - Scheduler" \
77
description="Crunchy Data PostgreSQL Operator - Scheduler"
88

0 commit comments

Comments
 (0)