Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

#10 fix make docker #11

Merged
merged 1 commit into from
May 14, 2017
Merged

#10 fix make docker #11

merged 1 commit into from
May 14, 2017

Conversation

hjacobs
Copy link
Contributor

@hjacobs hjacobs commented May 14, 2017

Fixes #10

@hjacobs
Copy link
Contributor Author

hjacobs commented May 14, 2017

I also did:

IMAGE=registry-write.opensource.zalan.do/acid/postgres-operator make docker
IMAGE=registry-write.opensource.zalan.do/acid/postgres-operator make push
docker run -it registry.opensource.zalan.do/acid/postgres-operator:8c7becc
2017/05/14 16:00:01 Spilo operator 946f623-dirty

@hjacobs
Copy link
Contributor Author

hjacobs commented May 14, 2017

Only after this change I could get the complete README instructions to work (with Minikube):

kubectl get pods
NAME                                  READY     STATUS    RESTARTS   AGE
acid-testcluster-0                    1/1       Running   0          7m
acid-testcluster-1                    1/1       Running   0          6m
etcd0                                 1/1       Running   0          29m
etcd1                                 1/1       Running   0          29m
etcd2                                 1/1       Running   0          29m
fake-teams-api-3898472216-jg0rd       1/1       Running   0          29m
kube-ops-view-3644896577-66nf1        1/1       Running   2          1d
kube-ops-view-redis-775453491-s87x4   1/1       Running   2          1d
postgres-operator-3843529673-d0kl4    1/1       Running   112        28m

@hjacobs
Copy link
Contributor Author

hjacobs commented May 14, 2017

👍

@mkabilov
Copy link
Contributor

LGTM

@mkabilov
Copy link
Contributor

👍

@hjacobs hjacobs merged commit c08e83b into master May 14, 2017
@erthalion erthalion deleted the fix-docker-build branch September 13, 2018 09:15
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.

None yet

2 participants