During a recent installation of Synergy on a standard CentOS 7 cluster
it appeared that python dependencies present in both EPEL and OpenStack
repositories were not installed correctly.
This commit makes the dependencies more explicit and result in a correct
installation procedure.
Note that since the Synergy code has not changed since the last version,
only the package "dist" version is changed.
Change-Id: I1d10fdc7e98788ce83c245aa3606351e358603e1
This commit fixes some errors introduced with the support for the
authorization in Synergy. The files involved are: service.py and
shell.py and related unit tests.
Change-Id: I84587e48927307d85b9a3f96ea49154805da7bcd
Sem-Ver: bugfix
The Synergy's methods startManager() stopManager() getManagerStatus() and
executeCommand() require the parsing of the parameters included into the
user request and make some checks for the mandatory parameters.
This commit enhances this parsing and checking steps by using the Python
decorators. Moreover it simplifies the execution of the methods startManager()
stopManager() and getManagerStatus() by handling requests for only one manager
at a time.
Change-Id: Ic295ad20f03459cf3e512e795529c35bd6969ed3
Sem-Ver: bugfix
This fix provides to Synergy a security mechanism highly configurable.
The security policies are pluggable so that it is possible to define any
kind of authorization checks.
This commit includes a very simple authorization plugin (i.e.
synergy.auth.plugin.LocalHostAuthorization) which denies any command
coming from clients having IP address different from the Synergy's one.
Bug: #1691352
Change-Id: I2535b2a3edeea5e56cd8918d01070a6f8a534c3e
Sem-Ver: bugfix
The basic Exception can be used anywhere in the code but however, no
program nor library should ever raise Exception directly: it's not
specific enough to be helpful. This fix better handles the exceptions by
removing the Exception occurencies and adding a new SynergyError type.
Bug: #1690795
Change-Id: I202e063198ee9aef7397bad9b8398c24d52b5fe1
Sem-Ver: bugfix
In case a manager doesn't implement the method doOnEvent() the
NotImplementedError exception is raised. The manager.notify()
doesn't handle such exceptions and fails. This fix solves the problem.
Bug: #1690117
Change-Id: I005658c4341106b54f6757ee4dfcca08a2405605
Sem-Ver: bugfix
Not all messages are logged in synergy.log (e.g. the ones logged by the
modules in synergy_scheduler_manager/common). This fix solves the problem.
Bug: 1690113
Change-Id: I861f7281a1e2cc2a49567ab97482234f834919f4
Sem-Ver: bugfix
This fix updates the packaging of synergy service making it compliant with the indigo-2 requirements.
Change-Id: Ide701fe90c23eb4375bcdf9ff9bfd010cd7290ac
Sem-Ver: bugfix
Remove unused variables related to token and synergy_url.
Also add some unit tests for the shell.
Change-Id: I101a77e478b95534ac04e1ae73d26fef98898a5c
Sem-Ver: bugfix
Also make small adjustements to command help texts and removed an
unused variable.
Change-Id: Id899b561cee8301be809136ad101d4a824d7e62a
Sem-Ver: bugfix
Small tweak to the docker CLI invokation when packaging to make it less
verbose.
Related: I6a9bc7d3686a1b116d664db77f072b004b30fd25
Sem-Ver: bugfix
Change-Id: Ida07b3b439949fe83066039350647901693d4816
Synergy is now expected to work on either CentOS 7 and Ubuntu 16.04.
Both of these OS ship with Systemd so we can get rid of the SysV init
and Upstart scripts we had.
Sem-Ver: bugfix
Change-Id: I8e51acc4c86ee82c2484ee09036f902156580ce0
The synergy.conf must configure just the service whereas the
managers must be configured with separate configuration files.
- removed all scheduler managers sections
- added support to read all configuration files in /etc/synergy
- removed service automatic restart after installation or update
bug #1663581
Depends-On: I7ceb140182db306567d7f7a6b5bfcf0871249a05
Sem-Ver: feature
Change-Id: I4491606402314d29363420aeb5cb943ae3a9090f
On CentOS 7, upgrading Synergy would cause the Synergy log file to be
overwritten. We fix this behaviour in this commit.
Note that now the Synergy log file is created during Synergy first run,
not during the first installation of Synergy.
fixes bug #1664502
Sem-Ver: bugfix
Change-Id: Id1a4fc3871a141f1d5fd0f2c95c10464e86dda8e
Openstack common has a wrapper for generating uuids.We should
use that function when generating uuids for consistency.
Change-Id: I9e7d8a1fcfb3b378b2ab397a0ba932d7aac17e26
Closes-Bug: #1082248
When packaging, we can now pass a PKG_VERSION environment variable to
docker to force which version is being packaged.
If this variable is not passed to docker, then the version is the latest
git tag.
Change-Id: I8863d6a101e8d85573d8547bab1a667060648018
We used to put `try / except ImportError` around oslo lib imports
to support both the `oslo_lib` and `oslo.lib` name patterns.
It is now useless since the newer versions of these libs all support
the `import oslo_lib` pattern.
Tested on CentOS 7 with the OpenStack Liberty repository.
Tested on Ubuntu 14.04 with OpenStack Liberty CloudArchive repository.
Change-Id: I7a147586d7a933506df62e34424e5fc57b3ff503
Sem-Ver: bugfix
- pbr was not installed, but is now required.
- clean .eggs directory
- update packaging doc for CentOS
Sem-Ver: bugfix
Change-Id: I6a1ec486847a28f76283b8acb96c56af6a5e4e4c
Synergy depends on the python tabulate package. However, it is not
available as a system package for both CentOS 7 and Ubuntu 14.04.
In this commit we opt for the following solution: don't mark
tabulate as a dependency when building synergy system packages,
instead we provide the tabulate package as part of Synergy.
DocImpact
Change-Id: I76018e1ba27de8ad47b59c85baad2c2d06f03398
Sem-Ver: bugfix
Requirement versions were set in the first place to limit the
buginess. However, after some time it appears that it made more
harm than good, as discussed in:
https://review.openstack.org/#/c/388003/
In this commit we remove the versions for all the requirements.
In the future, if a specific dependency version if causing some issue
we should reintroduce the version for this dependency to block the
unstable version.
Sem-Ver: bugfix
Change-Id: Ia42509bf2bf9012cc03c8bb25b84f465425e2aad
removed unneccassary directories in .coveragerc file
synergy/openstack/*
it is no longer valid, we no longer use content from oslo-incubator
Change-Id: I57290b998bd014ee00d5f98110133cf6773e1a03
It was missing git, which is necessary since we now rely on it to get
the current version of synergy-service.
Change-Id: Ifd334dd646ca9e9629e9224163f01380c2e5f1b6
Sem-Ver: bugfix
The required version for CentOS was 0.18, but only 0.17 is available
in the repositories.
Change-Id: I94ced14b0f0c520e431c7ef2fd94c11d6721af9c
Sem-Ver: bugfix
Commit 670f608252 introduced this bug.
We previously relied on the "version: " line of setup.cfg, but this
was removed due to using pbr.
We now get the synergy version from the git tag.
Change-Id: I31fc9215269b05acabba8c4288af540f4de9c0c4
Sem-Ver: bugfix