Commit Graph

48 Commits

Author SHA1 Message Date
Dmitry Tantsur 95b69e2bbc Fix two CI issues
* Fix lower-constraints with the new pip resolver

  First, remove pep8 dependencies from test-requirements, they're not
  needed there and are hard to constraint properly.

  Second, bump keystoneauth to the version required by openstacksdk.

* Force updating libgcrypt to work around CentOS/RHEL 8.3 bug
  https://bugzilla.redhat.com/show_bug.cgi?id=1840485

Change-Id: I65379a3d1b42e3607118bd785f580604bef7d10d
2020-12-08 17:09:05 +01:00
Riccardo Pittau 35998b780a Set safe version of hacking
Versions of hacking from 3.1.0 until 4.0.0 NOT included are supposed
to require a safe version of flake8, so besides normale issues related
to minor version upgrades, we can safely use those versions.
Also forcing pycodestyle versions to be compatible with flake8 installed
by hacking.

Change-Id: I3573f9d360450eaa204f5e951fc9e56592c77af2
2020-10-21 15:40:30 +00:00
wu.shiming 4c588187e3 Fix hacking min version to 3.1.0
flake8 new release 3.8.0 added new checks and gate pep8
job start failing. hacking 3.0.1 fix the pinning of flake8 to
avoid bringing in a new version with new checks.

Though it is fixed in latest hacking but 2.0 and 3.0 has cap for
flake8 as <4.0.0 which mean flake8 new version 3.9.0 can also
break the pep8 job if new check are added.

To avoid similar gate break in future, we need to bump the hacking min
version.

- http://lists.openstack.org/pipermail/openstack-discuss/2020-May/014828.html

Change-Id: I214fef0669c63ce3476fe872ab2a91b55120cccf
2020-10-20 09:21:27 +08:00
Dmitry Tantsur da3593244f Clean up requirements
Paramiko used to be required by ironic, now it's only used by ansible
and should be handled by its requirements.

Simplify the setuptools requirement to use the version from
Bionic as a minimum and pull in the latest exclusions from g-r.

Move PyYAML to requirements since it's used in production code.

Remove test requirements that are not referenced anywhere.

Change-Id: Idc9f1cd66b123c9c9325d7fd7a557475bfdbdd68
2020-07-30 18:19:48 +02:00
Dmitry Tantsur 8b43bf00f3 Get rid of the shade dependency
Shade is being phased out in favour of openstacksdk, which also allows
to simplify the authentication code.

Change-Id: I14210a720792c3f7f0669330ccb6d2f67d624246
2020-06-03 15:57:15 +02:00
Riccardo Pittau 556ce37df6 Upgrade flake8-import-order version to 0.17.1
Full py3 compatible version.
Add all Python3 modules to stdlib list.
Also includes fix to an enum34 dependency bug.

Change-Id: Ie260ace9bc1d8d5bc89e346c5eb4007fbbed2d14
2020-04-21 16:20:29 +02:00
Sean McGinnis af465031f8
Use unittest.mock instead of third party mock
Now that we no longer support py27, we can use the standard library
unittest.mock module instead of the third party mock lib.

Change-Id: I219e2386aa957de3aafca7d76f3d0923e5fef1a9
Signed-off-by: Sean McGinnis <sean.mcginnis@gmail.com>
2020-04-18 11:51:06 -05:00
Andreas Jaeger fdcbfb8365 Update hacking for Python3
The repo is Python 3 now, so update hacking to version 3.0 which
supports Python 3.

Fix problems found.

Change-Id: I2707a1f2047f89d9f12ea282cec98411454e1f79
2020-03-30 10:18:15 +00:00
Riccardo Pittau db3d46a7d3 Switch to stestr
Use stestr for tests.

Also clean tox.ini and add check for flake8 H106:
Don't put vim configuration in source files.

Change-Id: Ia76961a091688372c6df363d5e1dcb29ff0f39bf
2020-03-17 07:48:42 +01:00
Nguyen Van Trung 460a8f6be3 Follow the new PTI for document build
For compliance with the Project Testing Interface as described in:
https://governance.openstack.org/tc/reference/project-testing-interface.html

For more detials information, please refer to:
http://lists.openstack.org/pipermail/openstack-dev/2017-December/125710.html

Change-Id: I3eb92c968ece3c15eeae59bb6ef9492f3d03eeed
2018-07-10 16:33:42 +00:00
Zuul 4e65227c43 Merge "Allow to check pep8 along with D000" 2018-05-27 11:05:27 +00:00
Nguyen Van Trung fa4f095759 Fix incompatible requirements
A lot of packages in lower-constraints.txt and requirement.txt
must change to suitable version. Which raised fail in following logs:

 Validating requirements.txt
 Validating test-requirements.txt
 Validating lower constraints of requirements.txt
 Package u'oslo.log' is constrained to 3.37.0 which does not match the minimum version specifier 3.36.0 in requirements.txt
 Package u'paramiko' is constrained to 2.4.1 which does not match the minimum version specifier 2.0.0 in requirements.txt
 Package u'setuptools' is used in requirements.txt but not in lower-constraints.txt
 Package u'pbr' is constrained to 3.1.1 which does not match the minimum version specifier 2.0.0 in requirements.txt
 Validating lower constraints of test-requirements.txt
 Package u'sphinx' is constrained to 1.6.5 which does not match the minimum version specifier 1.6.2 in test-requirements.txt
 Package u'coverage' is constrained to 4.5.1 which does not match the minimum version specifier 4.0 in test-requirements.txt
 Package u'oslotest' is constrained to 3.3.0 which does not match the minimum version specifier 3.2.0 in test-requirements.txt
 Package u'openstackdocstheme' is constrained to 1.20.0 which does not match the minimum version specifier 1.18.1 in test-requirements.txt
 Package u'doc8' is constrained to 0.8.0 which does not match the minimum version specifier 0.6.0 in test-requirements.txt
 Package u'reno' is constrained to 2.7.0 which does not match the minimum version specifier 2.5.0 in test-requirements.txt
 Package u'testrepository' is constrained to 0.0.20 which does not match the minimum version specifier 0.0.18 in test-requirements.txt
 Package u'testscenarios' is constrained to 0.5.0 which does not match the minimum version specifier 0.4 in test-requirements.txt
 Package u'testtools' is constrained to 2.3.0 which does not match the minimum version specifier 2.2.0 in test-requirements.txt
 *** Incompatible requirement found!
 *** See http://docs.openstack.org/developer/requirements
 ERROR
 {
   "delta": "0:00:09.290821",
   "end": "2018-05-23 16:24:17.040257",
   "failed": true,
   "rc": 1,
   "start": "2018-05-23 16:24:07.749436"
 }

Change-Id: I5bcc2b3523c30126e88c0396a3ea080a70bb66bd
2018-05-24 14:22:47 +07:00
Nguyen Van Trung ac079bfa37 Allow to check pep8 along with D000
This commit going to remove --ignore D000 which had raised
following issues:

"D000 Cannot analyze code. Pygments package not found."

And they also fixed in https://review.openstack.org/#/c/568729/
Depends-On: https://review.openstack.org/#/c/570236/

Change-Id: I3f35c65d385dc65ccefbac608498ad85ff4a4d71
2018-05-24 01:31:51 +00:00
OpenStack Proposal Bot 2dc8b6be8f Updated from global requirements
Change-Id: Iee4a9c66442d9dcefce32f6bfb4bf3100aa82e9b
2018-03-15 06:46:22 +00:00
OpenStack Proposal Bot 985c3a9dfa Updated from global requirements
Change-Id: I453b03b6495afa44dde1510fb929fd471e91d792
2018-03-13 06:47:39 +00:00
OpenStack Proposal Bot 872df741ff Updated from global requirements
Change-Id: Ia81d0bef64e3a0fc866934964f5e4bc96b3d8666
2018-03-10 13:07:15 +00:00
OpenStack Proposal Bot 9578e9743d Updated from global requirements
Change-Id: Ifc42bb9014967b0233541e8479fe253a906123c7
2018-03-04 09:59:28 +00:00
OpenStack Proposal Bot b4629da82b Updated from global requirements
Change-Id: I2962676f663684141f614cf808ee5e14db4cf4ab
2018-01-24 00:49:24 +00:00
OpenStack Proposal Bot 2129f9264c Updated from global requirements
Change-Id: I7ff3ddb441fd1433b4626d86be71713f48edbf2b
2018-01-17 20:28:15 +00:00
OpenStack Proposal Bot f61ab7a044 Updated from global requirements
Change-Id: Ic2b5ef32015c8021bc601b81bf7b5dabf0a7cdc9
2018-01-16 04:02:33 +00:00
Nguyen Van Trung 7047a329b5 Add doc8 into pep8 check to scan rst files
This change will help fix pep8 issues of all *.rst files
in the project and checking for new files.

Change-Id: Ib06aeb7fa76bd387d76c23bc3f8ecca89332d989
Co-authored-By: Hoang Trung Hieu <hieuht@vn.fujitsu.com>
2017-11-29 13:13:15 +07:00
OpenStack Proposal Bot cc46b9280c Updated from global requirements
Change-Id: I846dc9c1a0410012881cc90cdcb573105015c696
2017-11-16 10:59:19 +00:00
zhangyangyang 50b7cd4c70 Cleanup test-requirements
python-subunit is not used directly anywhere
and it is dependency of both testrepository
and os-testr
(probably was used by some tox wrapper script before)

Change-Id: Ibcb61cc05e59cf201354cefeb717c3e869487174
2017-09-20 23:22:02 +08:00
OpenStack Proposal Bot 2752410bad Updated from global requirements
Change-Id: I15fd0cf614517eeaf4fb7c7e1fad86560ebc977c
2017-09-13 12:48:36 +00:00
OpenStack Proposal Bot 2af8f4166d Updated from global requirements
Change-Id: Ia3b12b5f5a5dc63888073b1e895d1ef0afdbac54
2017-08-18 11:28:14 +00:00
OpenStack Proposal Bot 2d62bdbedb Updated from global requirements
Change-Id: Iee8e838435063d0d8bba7c669f4548edf2ff1f8d
2017-08-18 04:38:40 +00:00
OpenStack Proposal Bot 8cffdf93a4 Updated from global requirements
Change-Id: Ifb9e836dbc9f438333e5c430b1c2216de6d1b226
2017-07-27 19:07:42 +00:00
Julia Kreger 531c695a74 Switch from oslosphinx to openstackdocstheme
As part of the docs migration work[0] for Pike we need to switch to use
the openstackdocstheme.

[0]https://review.openstack.org/#/c/472275/

Change-Id: Ifcfdaf8247ed52e57ed8ccb31338eb492df10e0e
2017-07-11 19:55:53 +00:00
OpenStack Proposal Bot f5320d0e20 Updated from global requirements
Change-Id: I2299be4a7aaedbdd7e1b74eba1dd9eef51229ffa
2017-06-27 12:08:13 +00:00
OpenStack Proposal Bot 0f8d5029e4 Updated from global requirements
Change-Id: I320db7a4dfae2e25b475286b224fcfa2d1e6d2a8
2017-06-02 21:53:36 +00:00
OpenStack Proposal Bot 905e9cae12 Updated from global requirements
Change-Id: Idc567b7f22d406a2c0ef1285c2516a6988cc04c6
2017-05-17 03:46:11 +00:00
OpenStack Proposal Bot e199a3abfd Updated from global requirements
Change-Id: Ia910a4429224f71982b3a6478a2eb238afdd8b12
2017-05-15 16:44:21 +00:00
Mark Goddard faecdb4557 Ironic inventory source duplicates MACs
When using the 'ironic' Bifrost inventory source to obtain an Ansible
inventory from ironic, if any nodes have multiple ports then the
resulting inventory contains a 'nics' list variable where each item has
the same MAC address.

This change fixes this issue by reinitialising each new_nic variable
such that all items in the nics list do not reference the same dict.
We also introduce a couple of unit tests for the ironic inventory
source - one to test the simple case of a single node with a single NIC,
another to test the broken case of a single node with multiple NICS.

Change-Id: I173fbbdbebafc05536a153dcf59e4b2f37aebdbc
Closes-Bug: #1671014
2017-03-08 09:42:16 +00:00
OpenStack Proposal Bot 567e9486c4 Updated from global requirements
Change-Id: I107529b69802cdabe7e886c41914741788d175fe
2017-03-02 23:53:04 +00:00
Julia Kreger cc0f846f07 Update hacking in test-requirements.txt
Change-Id: Ia520381d4e3fbfea1c2d4be4eb3e6e70f82cd64c
2017-03-02 17:04:30 +00:00
OpenStack Proposal Bot 1a262cf0b8 Updated from global requirements
Change-Id: If3da5580048f1e9229ebe0a61d72df9459db22c6
2017-02-14 05:49:32 +00:00
Pavlo Shchelokovskyy 1a04c6abf1 Cleanup python requirements
there are a number of requirements listed in requirements.txt that can
be dropped or moved around, especially since bifrost begins to support
installing/operating ironic on a remote host:

- six - not used directly. Is listed as dependency for os_ironic_facts
  module, but so is shade which requires it anyway.

- Jinja2 - not used directly, is a dependency for Ansible

- Babel - not used directly as no strings use i18n in bifrost inventory,
  and even when they begin to, this should be done through oslo.i18n instead.
  It is also pulled in as oslo.log->oslo.i18n->Babel dependency anyway.

- PyMySQL - not used by bifrost inventory itself, is installed as part of
  bifrost-ironic-install role.

- pycrypto - not used by bifrost itself, but will be installed as Ansible
  dependency

- PyYAML - although used by bifrost inventory, in real usage we
  should better rely on Ansible's dependency on PyYAML for that and use
  whatever version Ansible developers declare as required.
  Moved to test-requirements for running tests without installing
  Ansible.

Change-Id: I8021ec10f82b17590dd86780da6e27589fdea1ba
2017-02-09 14:14:17 +02:00
OpenStack Proposal Bot 5970763475 Updated from global requirements
Change-Id: Ia82dc2b174fd73b92ef05652d113245409ca0c0c
2016-11-04 05:53:49 +00:00
OpenStack Proposal Bot 6e14419d42 Updated from global requirements
Change-Id: I67d5384004c8104ee5a3deb1eca980b9649c35d8
2016-09-30 19:57:18 +00:00
Jenkins 12b926ab4e Merge "Remove discover from test-requirements" 2016-07-25 16:36:51 +00:00
Swapnil Kulkarni (coolsvap) 353a1c028a Remove discover from test-requirements
It's only needed for python < 2.7 which is not supported

Change-Id: Idbf1e7416422e47f170a7743acfe816dc6a4336c
2016-07-21 07:28:48 +00:00
OpenStack Proposal Bot 155bb48714 Updated from global requirements
Change-Id: Ice87efc77b17a18008a4c41947ba57f95a488ef7
2016-06-24 03:10:03 +00:00
OpenStack Proposal Bot c630eb5117 Updated from global requirements
Change-Id: I8b95f4418c7d65affad59ed06b373d23062491fa
2016-04-07 17:06:47 +00:00
Julia Kreger c52cf302c1 Sync from global requirements
Change-Id: Ice48f30a818ee2d745078e17a58be021090e529a
2016-01-22 11:39:17 -05:00
Julia Kreger 3c62f730a1 Initial support for Reno
Since everyone is going to reno, why not bifrost as well.

Adds basic substrate for reno, which may be incorrect, however
that can be addressed as time goes on.

Change-Id: I36caf7682de56917be90bed14e290c5509c7fa86
2015-12-17 05:17:27 -05:00
Julia Kreger aa39db29c7 Resync with openstack/requirements
Change-Id: I76b198b29f1a97a6c74ab0443238c5a4864279fb
2015-08-27 08:56:40 -04:00
Julia Kreger d5526e37bd Sync with global requirements
Change-Id: I031b442ad3b5b4c2f7726c9a93acf3cfd5c38f55
2015-07-15 17:04:11 -04:00
Jenkins 2a9d49d9bc Add cookiecutter-generated files for project
Use cookiecutter (https://git.openstack.org/openstack-dev/cookiecutter)
to generate config for tox so we can generate docs, run pep8, etc.
Also move CONTRIBUTING.rst to the root of the repository in keeping
with what seems to be the standard location. This move allows us to
easily generate the docs. Also include a tiny fix to allow pep8 to
run cleanly.

Change-Id: Ifbfc6d85c7b02bf4ab989974b491a3a1ae6f0900
2015-06-10 08:40:19 -07:00