The repo is Python 3 now, so update hacking to version 3.0 which
supports Python 3.
Fix problems found.
Update local hacking checks for new flake8.
Remove hacking and friends from lower-constraints, it's not needed
there.
Change-Id: Iadf750a1e6d3181b645ceccdf75cf910bc23adfd
With changeset [1], releasenotes are added to monasca-persister. To
make the release notes available, the notes can be build now with
tox ("tox -e releasenotes") and the standard zuul
job (release-notes-jobs-python3) was added to build and publish the
notes.
Also adjust elasticsearch requirement to "<3.0.0" to be inline with
the global-requirements list. This fixes:
Requirement for package elasticsearch excludes a version not excluded
in the global list.
[1] Ida11c326b3e6771b5ccf994205c55874db05bc5c
Change-Id: I16268797151a6b997610c8fbded4c9eeb9265597
Using "python setup.py check -r -s" method of checking the package has
been deprecated with the new recommendation to build the sdist and
wheel, then running "twine check" against the output.
Luckily, there is already a job that covers this that only runs when the
README, setup.py, or setup.cfg files change, making running this in the
pep8 job redundant. This covered by the test-release-openstack-python3
that is defined in the publish-to-pypi-python3 template.
More details can be found in this mailing list post:
http://lists.openstack.org/pipermail/openstack-dev/2018-October/136136.html
Change-Id: Ie86513569c5a25c2de5eca94e0479f03806a45fb
Signed-off-by: Sean McGinnis <sean.mcginnis@gmail.com>
Quotes around {posargs} cause the entire string to be combined into one
arg that gets passed to stestr. This prevents passing multiple args
(e.g. '--concurrency=16 some-regex')
Change-Id: I5cf12e8db1bbf6215ad67524b80ad52e8b9eff4d
We want to default to running all tox environments under python 3, so
set the basepython value in each environment.
We do not want to specify a minor version number, because we do not
want to have to update the file every time we upgrade python.
We do not want to set the override once in testenv, because that
breaks the more specific versions used in default environments like
py35 and py36.
Change-Id: Iff786d273a1511b32fbdae4b5ad647c859df349f
According to Openstack summit session [1] stestr is maintained
project to which all Openstack projects should migrate.
Let's switch it then.
[1] https://etherpad.openstack.org/p/YVR-python-pti
Also fix the lower-requiremenst.txt to pass the gate.
Change-Id: I498282defd87c4ea16979b729600a179cdd3d01f
Signed-off-by: Charles Short <zulcss@gmail.com>
* Update max line length to 100
* Clean up codes for pep8 checks from tox.ini
Change-Id: I974c0c31dc51784506cbf54b87bc450f2334845e
Signed-off-by: Jui Chandwaskar <jchandwaskar@op5.com>
Create a tox environment for running the unit tests against the lower
bounds of the dependencies.
Create a lower-constraints.txt to be used to enforce the lower bounds
in those tests.
Add openstack-tox-lower-constraints job to the zuul configuration.
See http://lists.openstack.org/pipermail/openstack-dev/2018-March/128352.html
for more details.
Change-Id: I2cafc4f53f65c5f0c3fd18eff6600cca59b585ac
Depends-On: https://review.openstack.org/555034
Signed-off-by: Doug Hellmann <doug@doughellmann.com>
This change extends the persister to read OpenStack
event objects from kafka topic and then stores
them into ElasticSearch.
Story: 2001112
Task: 4843
Change-Id: I35b0db67dc088e56ad281c84fc4b50fa7a064e44
To generate the sample config file, do what other OpenStack projects do
and use the oslo.config provided mechanism.
Story: 2001009
Task: 4201
Change-Id: I490de4110843f7dfd618a6a8cc8d6a4e35db65ce
Enforce following codestyle rules:
* no xrange
* no LOG.warn usage (deprecated in favour of LOG.warning)
* usage of assertTrue(x) instead assertEqual(True, x)
* usage of assertIsNone(x) instead assertEqual(None, x)
* usage of assertIsNotNone(x) instead assertNotEqual(None, x)
or assertIsNot(None, x)
Change-Id: Ib5b61816faae95937c99b1e4f651c6390b888070
Both are optional requirements (but one must be used) so add both to the
"extras" section instead of a commented entry in requirements.txt .
Change-Id: I1f0eca2474ca275f64314a7b30b18e534afefdee
Add bandit job as part of pep8 in tox.ini
Had to fix one issue and mark two instances of try except pass as OK so that
bandit will pass
Change-Id: Ia1c96e27d1bae360c6ae0d4131665e2b712f573f
Add unit tests for persister.py in root directory.
Additionally:
* switched to testr as it seems to be used in many
openstack projects
* enabled coverage
Change-Id: I429ef6a900808c192ad5613c13808583d33fde24
There is no this directory openstack/common which was used
to keep codes from oslo-incubator, we have retired oslo-incubator.
Removing openstack/commonfrom all OpenStack code in favor of the
Oslo libraries is a project wide goal for the Ocata release. So
don't use this directory any more. We should drop it for improving
searching efficiency.
Change-Id: I65e1a75c65d44ff50c9ab0c2ff8c66faa99c227b
I unlock the hacking requirements in test-requirements.txt
Change-Id: I2a26845f16064be8c082d9cbe2e2d2bc48339314
Signed-off-by: Micu Matei-Marius <mmicu@cloudbasesolutions.com>
This review is to prepare for adding monasca-persister into openstack
global-requirements. After this project gets into global requirements,
it will be in upper-constraints.txt. If we make changes on this
specific project, we need to replace it in the upper-constraints with
source based specifier before applying it to pip install.
Change-Id: Ife8c77d41815b262300068497f7197af35ef2f5a
Add upper-constraints and min tox version.
This will prevent taking in packages that
are greater then what is defined in the
upper-constraints
Change-Id: I4a7aea0900efa767a4a557d20d601caa9938dfeb
Bring over the cleaning line from run_tests.sh for the pyc files to
all the tox runs.
This should eliminate the need to clean -x -i to kill pyc files in
your local directory to get tests to pass.
Change-Id: Ifc8df1634336fdfd298c353ad808c77f48b87b68
Closes-Bug: #1368661