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

Bug 1504957 - Broker should use recreate strategy #511

Merged
merged 1 commit into from Oct 26, 2017

Conversation

djzager
Copy link
Member

@djzager djzager commented Oct 20, 2017

Use the recreate strategy for the broker deployment.

@openshift-ci-robot openshift-ci-robot added the size/XS Denotes a PR that changes 0-9 lines, ignoring generated files. label Oct 20, 2017
@djzager
Copy link
Member Author

djzager commented Oct 20, 2017

Before Fix

$ oc logs asb-2-z4kg8 -c asb
Using config file mounted to /etc/ansible-service-broker/config.yaml
============================================================
==           Starting Ansible Service Broker...           ==
============================================================
[2017-10-20T15:09:13.108Z] [NOTICE] Initializing clients...
[2017-10-20T15:09:13.108Z] [DEBUG] Trying to connect to etcd
[2017-10-20T15:09:13.108Z] [INFO] == ETCD CX ==
[2017-10-20T15:09:13.108Z] [INFO] EtcdHost: 0.0.0.0
[2017-10-20T15:09:13.108Z] [INFO] EtcdPort: 2379
[2017-10-20T15:09:13.108Z] [INFO] Endpoints: [http://0.0.0.0:2379]
[2017-10-20T15:09:14.108Z] [ERROR] client: etcd cluster is unavailable or misconfigured; error #0: client: endpoint http://0.0.0.0:2379 exceeded header timeout

$ oc logs asb-2-deploy
--> Scaling up asb-2 from 0 to 1, scaling down asb-1 from 1 to 0 (keep 1 pods available, don't exceed 2 pods)
    Scaling asb-2 up to 1

This always fails because (from etcd's logs):

2017-10-20 14:49:08.881731 W | etcdserver: another etcd process is using "/data/member/snap/db" and holds the file lock.
2017-10-20 14:49:08.882677 W | etcdserver: waiting for it to exit before starting...

After Fix

$ oc rollout latest asb
deploymentconfig "asb" rolled out

$ oc logs -f dc/asb
--> Scaling asb-1 down to zero
--> Scaling asb-2 to 1 before performing acceptance check
--> Success

Copy link
Contributor

@jmrodri jmrodri left a comment

Choose a reason for hiding this comment

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

ACK

@jmrodri jmrodri merged commit 701d51f into openshift:master Oct 26, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
needs-review size/XS Denotes a PR that changes 0-9 lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants