Commit Graph

89 Commits

Author SHA1 Message Date
OpenDev Sysadmins 454b4165c5 OpenDev Migration Patch
This commit was bulk generated and pushed by the OpenDev sysadmins
as a part of the Git hosting and code review systems migration
detailed in these mailing list posts:

http://lists.openstack.org/pipermail/openstack-discuss/2019-March/003603.html
http://lists.openstack.org/pipermail/openstack-discuss/2019-April/004920.html

Attempts have been made to correct repository namespaces and
hostnames based on simple pattern matching, but it's possible some
were updated incorrectly or missed entirely. Please reach out to us
via the contact information listed at https://opendev.org/ with any
questions you may have.
2019-04-19 19:50:47 +00:00
ervin f5e7875552 Release v1.5.3
Synergy service updates

Change-Id: Id550365a7f7498d5c2f7424706a025a32b0373a6
2017-09-20 10:28:51 +02:00
Lisa Zangrando 40b56dd689 Shell version updated
This commit updates the Synergy shell version to 1.5.3

Change-Id: I4bee786179d6fc9a41b5fc50f445e3e3b00a9e74
Sem-Ver: bugfix
2017-09-12 12:48:13 +02:00
ervin 94a6072d7f Authorization section updated
Change-Id: If0640165649adb039846443535fc94c1eaa17e22
2017-09-08 11:23:18 +02:00
Vincent Llorens 35165c4241 Update RPM package dependencies
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
2017-08-23 10:44:53 +02:00
Lisa Zangrando c631a38fe6 Release v1.5.2
Synergy service updates.

Change-Id: Ice34496114b3e4e333a9af724ad48e3bb39f4676
2017-07-18 11:35:11 +02:00
Lisa Zangrando 51e6dde4e5 Fixes on the authorization mechanism
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
2017-07-18 07:49:05 +00:00
Lisa Zangrando 3b568b70a0 Enhancement in handling the parameters defined in the user request
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
2017-05-24 14:41:25 +00:00
Lisa Zangrando b99f2078a1 Missing security support
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
2017-05-23 12:57:50 +00:00
Lisa Zangrando a8c06a001c Synergy should never raise Exception directly
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
2017-05-17 06:21:33 +00:00
Jenkins b542db3beb Merge "manager.notify() doesn't handle the NotImplementedError exceptions" 2017-05-11 12:08:14 +00:00
Lisa Zangrando a2ba511da3 manager.notify() doesn't handle the NotImplementedError exceptions
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
2017-05-11 12:24:26 +02:00
Lisa Zangrando bba3767276 The synergy.log doesn't contain all logged messages
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
2017-05-11 10:11:40 +00:00
ervin 16d1843455 Release v1.5.1
Update the required version of synergy-service.

Change-Id: I5425090cd7b9d896718523974c748e6c54375f8f
2017-03-21 12:24:48 +01:00
ervin 0d84f532d8 Update synergy service packaging
This fix updates the packaging of synergy service making it compliant with the indigo-2 requirements.

Change-Id: Ide701fe90c23eb4375bcdf9ff9bfd010cd7290ac
Sem-Ver: bugfix
2017-03-20 18:13:15 +01:00
ervin 88e56e5d50 Release v1.5.0
Update the required version of synergy-service.

Change-Id: Ie2b80bc0e20ac6cab9d8018c9d53da92fce10f4c
2017-03-20 12:38:47 +01:00
Vincent Llorens 5ec5c9dc42 add some unit tests for Trust
Sem-Ver: bugfix
Change-Id: Ie4ca5c7ff5e8ae0a835650751571319ffc4a93ba
2017-02-27 08:52:27 +00:00
Vincent Llorens f31e39b9ea CLI: remove auth_token, unused vars and add tests
Remove unused variables related to token and synergy_url.

Also add some unit tests for the shell.

Change-Id: I101a77e478b95534ac04e1ae73d26fef98898a5c
Sem-Ver: bugfix
2017-02-24 10:37:21 +01:00
Vincent Llorens adf0f27de6 add some unit tests for ExecuteCommand
Sem-Ver: bugfix
Change-Id: Ia87cf70b1b120f01d0d430c5d00f2a2120ee1890
2017-02-22 16:32:46 +00:00
Jenkins 5dcea60830 Merge "add some unit tests for the ManagerCommand class" 2017-02-21 14:35:52 +00:00
Vincent Llorens e500611dfc add some unit tests for the ManagerCommand class
Also make small adjustements to command help texts and removed an
unused variable.

Change-Id: Id899b561cee8301be809136ad101d4a824d7e62a
Sem-Ver: bugfix
2017-02-21 14:00:59 +00:00
Vincent Llorens bf3f9e6895 simplify packaging with docker
Small tweak to the docker CLI invokation when packaging to make it less
verbose.

Related: I6a9bc7d3686a1b116d664db77f072b004b30fd25
Sem-Ver: bugfix
Change-Id: Ida07b3b439949fe83066039350647901693d4816
2017-02-21 14:58:14 +01:00
Vincent Llorens 449f53f687 fix one instance of "except Exception"
Sem-Ver: bugfix
Change-Id: Ic8f557c99e32990816fe0bbc8f6fa1dbcaa8593c
2017-02-21 14:26:20 +01:00
Vincent Llorens ea955b7405 add some unit tests for HTTPCommand
Sem-Ver: bugfix
Change-Id: Ibf9d255478869126da313ab54767d3d76126b899
2017-02-20 14:03:49 +00:00
Jenkins 6485fa60e7 Merge "fix missing elements in string formatting" 2017-02-20 11:17:31 +00:00
Vincent Llorens f1e951c927 fix missing elements in string formatting
Sem-Ver: bugfix
Change-Id: I70de0cd4d5c4cded7e9c3a303d178b3f03333fa6
2017-02-20 09:34:12 +00:00
Vincent Llorens cad7bca301 remove support for SysV init and Upstart
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
2017-02-14 16:26:33 +01:00
Lisa Zangrando 78eb46a608 Removed all managers parameters in synergy.conf
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
2017-02-14 15:36:28 +01:00
Vincent Llorens cb935dbcbc fix synergy erasing log file on upgrade
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
2017-02-14 12:52:25 +01:00
ervin 2edc9e2ef2 Release v1.4.0
Update the required version of synergy-service.

Change-Id: I8fccb21a7b36b7f1fd7142ea0f08c42feb2ff342
2017-01-30 15:51:00 +01:00
Lisa Zangrando 9a96dcd6d5 Update of the links to the Synergy documentation
Change-Id: I8a390f1610d30bdc7bd15f60a058c71c078c8803
2017-01-30 14:33:56 +01:00
Lisa Zangrando 1708739fd5 Update of the Synergy configuration file
This update includes all new parameters

Change-Id: I7ce871c6e9e4d3f1e99559aeda0b573519164c5e
Sem-Ver: feature
2017-01-30 11:30:52 +00:00
Lisa Zangrando 56d28fa54b The Synergy CLI is not SSL-enabled
Added support to HTTPS connections needed for interacting with Keystone

Change-Id: I1de2a862e054ac0954c9e4a5c6666b9e6e18822f
Sem-Ver: feature
2017-01-16 12:06:21 +01:00
Vincent Llorens aebc4aef10 Release v1.3.0 with packaging files
Change-Id: I2bf3516b2c59b484bd738b0d1caf820084fc9f89
2016-12-07 13:14:25 +01:00
Luong Anh Tuan 0b2d254ff5 Replaces uuid.uuid4 with uuidutils.generate_uuid()
Openstack common has a wrapper for generating uuids.We should
use that function when generating uuids for consistency.

Change-Id: I9e7d8a1fcfb3b378b2ab397a0ba932d7aac17e26
Closes-Bug: #1082248
2016-12-07 09:21:44 +00:00
Vincent Llorens a1657004eb [packaging] make docker aware of PKG_VERSION
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
2016-12-06 15:15:04 +00:00
Lisa Zangrando d82f0e3135 Added support for OpenStack DOMAIN to shell.py
- added new shell options:
    --os-user-domain-id, --os-ser-domain-name,
    --os-project-domain-id, --os-project-domain-name

Change-Id: Id110010a325989c3b6370647aa3cf8afebeb2657
Sem-Ver: feature
2016-11-28 20:14:14 +01:00
Vincent Llorens fbf53d25ad Update changelogs and system package versions
Change-Id: If18e6decc4b21ddaa230c4be3819dfe1c24cf5f4
2016-11-09 15:37:35 +01:00
Vincent Llorens 30a67fac85 Clean up oslo imports
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
2016-11-08 11:18:01 +01:00
Vincent Llorens 06f36e5e51 Update the Sphinx documentation
Point to the more up to date gitbook documentation.

Change-Id: I8e40ade71b8e5ab5104b99e20b25da42d7fca753
2016-10-28 14:41:48 +02:00
Vincent Llorens cc9eaff9b5 fix packaging with docker and its documentation
- pbr was not installed, but is now required.
- clean .eggs directory
- update packaging doc for CentOS

Sem-Ver: bugfix
Change-Id: I6a1ec486847a28f76283b8acb96c56af6a5e4e4c
2016-10-27 14:30:24 +00:00
Vincent Llorens b75d85315c Distribute tabulate as part of Synergy
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
2016-10-27 12:17:46 +02:00
Vincent Llorens e1ed6abec2 Remove versions for required packages
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
2016-10-27 11:20:04 +02:00
Vincent Llorens fd03bffe99 fix missing "requests" from the requirements
Also add it the spec file for RPM packaging.

Sem-Ver: bugfix
Change-Id: Ie32a7a9ce4b5d653410993dc14b4953dd3535a38
2016-10-25 15:37:06 +02:00
Iswarya_Vakati b1a3d44988 Updated coverage configuration file
removed unneccassary directories in .coveragerc file
  synergy/openstack/*

it is no longer valid, we no longer use content from oslo-incubator

Change-Id: I57290b998bd014ee00d5f98110133cf6773e1a03
2016-10-24 18:40:55 +05:30
Jenkins b50e3fddbe Merge "fix wrong version of eventlet" 2016-10-21 12:08:39 +00:00
Vincent Llorens b0f1c13b9e fix docker packaging for CentOS
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
2016-10-21 13:40:07 +02:00
Vincent Llorens 906ab1f7fc fix wrong version of eventlet
The required version for CentOS was 0.18, but only 0.17 is available
in the repositories.

Change-Id: I94ced14b0f0c520e431c7ef2fd94c11d6721af9c
Sem-Ver: bugfix
2016-10-21 11:21:26 +00:00
Vincent Llorens 524918a027 fix docs for packaging with Ubuntu
Change-Id: I9ab22416c8302e340bef027761a4538c8501bc98
Sem-Ver: bugfix
2016-10-20 15:58:59 +02:00
Vincent Llorens 4755163104 fix to get the synergy version when packaging
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
2016-10-18 11:48:14 +00:00