GitHub user kwmonroe opened a pull request:
https://github.com/apache/bigtop/pull/156
BIGTOP-2575: zk charm test updates
Update the Juju charm for Zookeeper:
- ensure we deploy correct zk charm and watch for correct status during
tests
- use new syntax for executing smoke tests
- include the network interface binding test (previously skipped)
- report the application version for status output
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/juju-solutions/bigtop
bug/zookeeper-action-fixes
Alternatively you can review and apply these changes as the patch at:
https://github.com/apache/bigtop/pull/156.patch
To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:
This closes #156
----
commit fc21e2c9aebc948246f1e1f8399eabf38c316d63
Author: Kevin W Monroe <[email protected]>
Date: 2016-10-27T21:55:18Z
check for correct action key in amulet tests
commit 9f6e1f6508c16defa1a1fd7c7aa839c0bbf0d695
Author: Kevin W Monroe <[email protected]>
Date: 2016-10-28T04:11:38Z
tweak suboptimal message
commit 20d58db4cc2600d1b0c61e0dc72ac7944935e357
Author: Kevin W Monroe <[email protected]>
Date: 2016-10-28T19:45:10Z
update amulet status message to match zk charm, wait for status message,
increase bigtop smoke-test timeout
commit f94be547798a7947c4ea751def7ed9012bd7805e
Author: Kevin W Monroe <[email protected]>
Date: 2016-10-28T20:43:54Z
net binding test updates
- don't watch for status (we've missed it randomly); instead, sleep 2
minutes
to let config-changed settle
- binding to 0.0.0.0 works, so stop skipping the test
commit 95d45da6203098deb4c32d92dcdf970c5136b7e6
Author: Kevin W Monroe <[email protected]>
Date: 2016-10-28T20:49:33Z
deploy separate zk when testing config changed
Our charms and bundles use reset:false. To avoid potentially changing a
desirable config without an admin knowing, make sure we deploy a 'zk-test'
charm when testing config-changed events.
commit 72101288eb4868b18d9da1e0ef997b96052133b2
Author: Kevin W Monroe <[email protected]>
Date: 2016-10-28T21:37:03Z
missed the zookeeper->zk-test rename
commit 9293b065f2511b9d7432d0b59b38997ce8b3a2d8
Author: Kevin W Monroe <[email protected]>
Date: 2016-11-07T17:07:57Z
lint, add app version for status, add .py suffix to tests
----
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at [email protected] or file a JIRA ticket
with INFRA.
---
