Simon Li

Consider switching to argparse for nicer command line handling and help

4

### Proposed change Switch from `tornado.options` to `argparse` (or traitlets, though argparse is easier for a small utility like this) for handling command line arguments ### Alternative options Do nothing...

Support building on mybinder.org to ease testing

2

At the moment if you want to test a PR that affects the UI you need to clone the PR and install it locally. In many cases testing on mybinder.org...

Add binder-badge bot

1

This is an optional add-on to https://github.com/jupyterhub/nbgitpuller/pull/258 This will update a PR description (so avoids the noise of a new comment) with a mybinder.org badge that builds the PR on...

http://nbgitpuller.link/ is a shortcut link that redirects to the nbgitpuller docs. However the https version https://nbgitpuller.link/ isn't valid. Is there anything we can do to enable it, especially if we...

Add /heath endpoint for checking availability

2

To support horizontal scaling, e.g. in container orchestration systems, it would be helpful to have an unauthenticated endpoint to indicate whether the service was healthy. This could be fairly simple...

Galaxy roles should be prefixed with organisation

4

The dependencies of galaxy roles have the form `openmicroscopy.basedeps`, however the playbooks in this repository expect a role name without a prefix (just `basedeps`). As a result some roles are...

At the moment most of our playbooks use the default version (as defined in the role) of dependencies such as Ice and Postgres, which means bumping any of these dependencies...

See https://docs.docker.com/engine/reference/commandline/dockerd/#/linux-configuration-file This should be used in the `docker` role, so the upstream `docker.service` systemd file can be used unmodified. This should avoid problems with breaking changes which would otherwise...

Change environment vars to use `{{ }}` as reported in https://github.com/openmicroscopy/infrastructure/pull/53#issuecomment-192409290 ``` TASK [upgrade-distpackages : system packages | upgrade] ************************ [DEPRECATION WARNING]: Using bare variables is deprecated. Update your playbooks...

A few roles use separate tasks to control services due to previous problems with systemd in docker. These shouldn't be required any more https://github.com/ome/ome-docker/tree/master/omero-ssh-systemd