~woutervb/charm-glance-sync:functional

Last commit made on 2020-02-06
Get this branch:
git clone -b functional https://git.launchpad.net/~woutervb/charm-glance-sync
Only Wouter van Bommel can upload to this branch. If you are Wouter van Bommel please log in for upload directions.

Branch merges

Branch information

Name:
functional
Repository:
lp:~woutervb/charm-glance-sync

Recent commits

4cfd9f2... by Wouter van Bommel

Small fixes in functionaltest

ebd49f9... by Wouter van Bommel

Added initial code for functional testing

Attempt 1 to add functional testing code in order to get the charm
tested, so we can release with confidence.

bb5512e... by Wouter van Bommel

Add a todo list

The code is defective on some parts, make it more clear what should be
done to rectify these problems.

cfb0959... by Wouter van Bommel

Cleanup

This commit added .gitignore file
* Updates to the README.md (More documentation,
  documentation on setting the charm up etc)
* Updates to metadata.yaml (stripping trusty as target)
* Updates to Makefile to reflect latest charm build requirements

df0a09c... by James Hilling

[j.hilling] fixes bug lp#1812855 and updated Makefile

The 'Makefile' has been updated to use the latest environment
variables for Juju. The charm now uses the CHARM_LAYERS_DIR and
CHARM_INTERFACES_DIR variables, instead of the now deprecated
LAYER_PATH and INTERFACE_PATH environment variables respectively.

ccd6cf0... by James Hilling

[j.hilling] charm cleanup and preparation for tests

Fixed a mistake within the 'README.md' file. Note: the
'README.md' file is still very much a WIP.

Inverted the series order within 'metadata.yaml', as
per aluria's suggestion.

Cleaned up the directory structure somewhat, in preparation
to add tests. Attempted to keep in line with the 'template-
python-pytest' reactive template.

52ec591... by James Hilling

[j.hilling] updated README.md with more content

The 'README.md' now has additional content added.

Also performed a small refactoring within the charm.

f2d9d67... by James Hilling

[j.hilling] fix: now using correct glance-sync slave scripts

Fetched and ported the correct glance-sync slave scripts from:

https://code.launchpad.net/~glance-sync-charmers/glance-sync-slave-layer/+git/glance-sync-slave-layer/+ref/community-image

Removed all traces of unit and functional tests. Will begin
porting the relevant tests from 'glance-sync-slave-layer' and
'glance-sync-master-layer' repositories properly.

Refactored handlers file after changing the glance-sync slave
script within the charm.

TODO: complete 'README.md'.
TODO: Unit/Functional tests.

e7efff9... by James Hilling

[j.hilling] fixed bug within ./src/reactive/glance_sync.py

Found a bug within the charm, when the charm gets deployed with 'master_mode'
initially set to 'true', i.e:

 juju deploy --config master_mode=true \
 ~/charms/glance-sync-layer/xenial/glance-sync glance-sync

Code within the 'configure_cron()' handler intended to run when 'master_mode'
is set to 'false', was accidentally being run when 'master_mode' was set to
'true', causing the charm to become stuck within a blocked state.

cdfc7d7... by James Hilling

[j.hilling] ./src/README.md partially completed