summaryrefslogtreecommitdiff
Commit message (Collapse)AuthorAgeFilesLines
* Merge "Switch to Ussuri jobs"5.4.5Zuul2019-12-061-2/+1
|\
| * Switch to Ussuri jobsxuanyandong2019-12-031-2/+1
| | | | | | | | Change-Id: I059ae377aed10e5dcb0e418c6aa427aca88cf9e1
* | trivial: Use 'open' context manager5.4.4Stephen Finucane2019-08-271-5/+5
| | | | | | | | | | | | | | It's a little nicer to look at. Change-Id: If5bc9b7e3927fe3203ba9adadaa1185c4a1cc7ed Signed-off-by: Stephen Finucane <sfinucan@redhat.com>
* | Update to latest hackingStephen Finucane2019-08-2712-148/+141
| | | | | | | | | | Change-Id: If73238a39d0934c6f577fc52132fcbc63e8eb25f Signed-off-by: Stephen Finucane <sfinucan@redhat.com>
* | Mark strings as rawStephen Finucane2019-08-275-26/+26
|/ | | | | | | | | Resolves the following deprecation warning: DeprecationWarning: invalid escape sequence \. Change-Id: If0bc30e646a3bb35c02dc73cc31fb146b3ecd6b5 Signed-off-by: Stephen Finucane <sfinucan@redhat.com>
* Merge "Use more verbose logging for test failure"5.4.3Zuul2019-08-121-1/+3
|\
| * Use more verbose logging for test failureStephen Finucane2019-05-211-1/+3
| | | | | | | | | | Change-Id: Iae259fe064b8f88037894247585a8394c8368b22 Signed-off-by: Stephen Finucane <sfinucan@redhat.com>
* | Merge "Fix parsing on egg names with dashes from git URLs"5.4.2Zuul2019-07-182-15/+213
|\ \
| * | Fix parsing on egg names with dashes from git URLsHervé Beraud2019-06-262-15/+213
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | repository urls for non editables end up in the `install_requires` list causing setup to fail. In this commit, all usable external urls get inserted into the `dependency_links` list with the corresponding egg fragments parsed into package names and versions (maintaining dashes if any) and added to `install_requires` list. Introduce parsing by using urlparse to extract properly the url fragment. Change-Id: If19576869fdf38e1768190c2d9c33f32504f7c54 Closes-Bug: #1373623
* | | Add Python 3 Train unit testsCorey Bryant2019-07-172-3/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This is a mechanically generated patch to ensure unit testing is in place for all of the Tested Runtimes for Train. See the Train python3-updates goal document for details: https://governance.openstack.org/tc/goals/train/python3-updates.html Change-Id: Ie9765e198e1204f56976476c8cfc9cc008c23d8f Story: #2005924 Task: #34234
* | | Merge "Resolve some issue with tox.ini, setup.cfg"5.4.1Zuul2019-07-152-26/+17
|\ \ \
| * | | Resolve some issue with tox.ini, setup.cfgStephen Finucane2019-06-132-26/+17
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | - Use constraints for documentation targets - Indicate support for Python 3.6, 3.7 - Fix indentation Change-Id: I486b6f645fa71f0fa519464465bf26411ca4faf4 Signed-off-by: Stephen Finucane <sfinucan@redhat.com>
* | | | option to print only the version of a package5.4.0Gaetan Semet2019-06-242-2/+18
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Example: $ pbr info -s pkgname 1.2.3 Change-Id: Ic66848dbe44ed946800979fb73776ab6d00869f6 Signed-off-by: Gaetan Semet <gaetan@xeberon.net>
* | | | Merge "Fix Windows support"5.3.1Zuul2019-06-172-4/+21
|\ \ \ \
| * | | | Fix Windows supportLucian Petrut2019-05-312-4/+21
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | A recent commit [1] broke Windows support by using shlex to unquote paths. The issue with shlex.split is that it doesn't work with Windows paths, treating backslashes as escape characters. We'll just replace backslashes with slashes before using shlex.split, converting them back afterwards. Closes-Bug: #1831242 [1] Id2cc32e4e40c1f834b19756e922118d8526358d3 Change-Id: Icb3abca004a35ab9760db8116fedfa96d012d0d0
* | | | | Merge "Install more dependencies for integration testing"Zuul2019-06-171-1/+1
|\ \ \ \ \ | |_|/ / / |/| | | |
| * | | | Install more dependencies for integration testingBen Nemec2019-05-211-1/+1
| | |_|/ | |/| | | | | | | | | | | | | | | | | | | | | | | | | | When running this locally I needed to install a few more distro packages to provide build dependencies for pip packages. This change adds them to the list of things installed in tools/integration.sh. Change-Id: I2ae04b893c7dfddcb6d94b724f53ee08436bf577
* | | | Merge "Read description file as utf-8"5.3.0Zuul2019-06-102-1/+22
|\ \ \ \
| * | | | Read description file as utf-8Ben Nemec2019-05-242-1/+22
| | |/ / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Currently pbr fails if the description file contains unicode characters. To fix this we need to open the description file as utf-8 explicitly. Since open() in Python 2 doesn't support an encoding parameter, use io.open() which works on both 2 and 3. Co-Authored-By: Hervé Beraud<hberaud@redhat.com> Change-Id: I1bee502ac84b474cc9db5523d2437a8c0a861c00 Closes-Bug: 1704472
* | | | Merge "Allow git-tags to be SemVer compliant"Zuul2019-06-102-1/+19
|\ \ \ \
| * | | | Allow git-tags to be SemVer compliantMartin Domke2019-05-242-1/+19
| |/ / / | | | | | | | | | | | | | | | | | | | | | | | | This fix allows it to use git-tags with a pre-release suffix that follows the SemVer specification (e.g. 1.2.3-rc1). Change-Id: Ie2e3c0e4145a105aff0a35c8dc6ec2a0cc1fc04e
* | | | Stop using pbr sphinx integrationBen Nemec2019-06-078-23/+17
| | | | | | | | | | | | | | | | | | | | | | | | | | | | Switch to sphinx-build directly and add sphinxcontrib-apidoc for generated api docs. Change-Id: I089a7695986892dcb47c5b5e79a6a227891a2347
* | | | Switch to release.o.o for constraintsBen Nemec2019-06-071-3/+3
| | | | | | | | | | | | | | | | | | | | | | | | This is the preferred location to get constraints now. Change-Id: I5dc34e63f94ebf9c19ffb5caeeea24581eca658f
* | | | Make WSGI tests listen on localhostBen Nemec2019-06-072-4/+8
|/ / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Currently we rely on the wsgi server to choose its own bind address, which tends to result in it listening on a DNS name. This means we have a dependency on that name resolving as expected on the node, which seems to not always be the case. In the interest of removing that environment dependency, let's have the wsgi tests explicitly listen on 127.0.0.1, which should always work and still allows us to verify our part of the WSGI functionality. This also includes a fix to use constraints for building docs because that is also blocking ci on this repo. We need both of these changes for anything to merge without endless rechecks. Change-Id: I48438eb31ee9dd102f06e30b13c6d892a93aba3b Closes-Bug: 1830962
* | | Merge "Remove neutron-lbaas"5.2.1Zuul2019-05-243-3/+0
|\ \ \
| * | | Remove neutron-lbaasAndreas Jaeger2019-05-233-3/+0
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | neutron-lbaas is getting retired, see [1]. Therefore remove use of it from this repo. [1] http://lists.openstack.org/pipermail/openstack-discuss/2019-May/006142.html Change-Id: Icbff2f69cfea78e961a60ae7723ed7dc58f296a7
* | | | Merge "Set subparser argument required"Zuul2019-05-231-1/+3
|\ \ \ \
| * | | | Set subparser argument requiredHervé Beraud2019-05-231-1/+3
| |/ / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | When subparser argument is not provided by user argparse return an error message not really useful for user: 'Namespace' object has no attribute 'func' This is due to the fact that when we launch the pbr in cli mode the subparser argument is not mandatory (required) and directly we try to execute a undefined function. Set the subparser required is more helpful for users due to the fact that argparse display the helping message with the available sub-commands that users can use These changes provides the following output if the argument is not passed: usage: pbr [-h] [-v] {sha,info,freeze} ... main.py: error: too few arguments Change-Id: I7982f9d40cb0979ddb89d7bc53964167f8e4b269
* | | | Merge "Fix white space handling in file names"Zuul2019-05-235-15/+108
|\ \ \ \
| * | | | Fix white space handling in file namesWill Szumski2019-04-255-15/+108
| | |/ / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Previously, when using data_files with a glob that matched a file with whitespace in the name, pip would error with a message that the file does not exist, e.g: error: can't copy 'ansible/roles/yatesr.timezone/templates/timezone-Arch': doesn't exist or not a regular file The problem was that ansible/roles/yatesr.timezone/templates/timezone-Arch was a truncated form of the actual filename: ansible/roles/yatesr.timezone/templates/timezone-Arch Linux.j2 Note the space in the filename and that it has been split on this space. This change allows you to use a glob that matches files with whitespace in the name. It does this by quoting the path. Change-Id: Id2cc32e4e40c1f834b19756e922118d8526358d3 Fixes-Bug: 1810934
* | | | Add openstack-tox-py37 jobCorey Bryant2019-05-232-2/+5
| |/ / |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Enables py37 testing while also being more friendly with developer environments where not all python versions may be available. Runs the docs jobs last as that one is the least likely to be needed. Tox min version is needed for the new options used inside. Change-Id: Icc6cb4979b2523a0edfc6375c5c032bb8be76b9f Story: #2004073 Task: #27440
* | | Update Sphinx requirementHervé Beraud2019-05-162-2/+4
|/ / | | | | | | | | | | | | Sphinx 2.0 no longer works on python 2.7, so we need to start capping it there. Change-Id: Ia13b8a75f11ee997e00e817464829ce8b86d2e16
* | OpenDev Migration Patch5.2.0OpenDev Sysadmins2019-04-194-23/+23
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 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.
* | Typo fix: s/extract_mesages/extract_messages/Ben Nemec2019-03-251-1/+1
| | | | | | | | Change-Id: If4141dd8e8876fc3d9d3e7579177a44f7c7da563
* | Support provides_extra metadataBen Nemec2019-03-252-1/+15
|/ | | | | | | | | This was added to setuptools in [1] so we should support passing it through from pbr. Change-Id: I46493c18756bcb01c53575ab51ec5e5e97670fc3 1: https://github.com/pypa/setuptools/commit/b2ea3c4a20d008622caec445f5b6916ddd420d16 Closes-Bug: 1760938
* Replace openstack.org git:// URLs with https://Ian Wienand2019-03-242-2/+2
| | | | | | | | | | | | | | | | | This is a mechanically generated change to replace openstack.org git:// URLs with https:// equivalents. This is in aid of a planned future move of the git hosting infrastructure to a self-hosted instance of gitea (https://gitea.io), which does not support the git wire protocol at this stage. This update should result in no functional change. For more information see the thread at http://lists.openstack.org/pipermail/openstack-discuss/2019-March/003825.html Change-Id: I1d81f12d728b628f5f6efc0ffb812246c0c2fef8
* Merge "Fix nits and typos on release note message."Zuul2019-03-211-4/+4
|\
| * Fix nits and typos on release note message.Hervé Beraud2019-03-201-4/+4
| | | | | | | | | | | | Fixing some nits and typos. Change-Id: Ia6c2dbcd99d88bf23e5e8fda7ba9321c7f6cc3aa
* | Merge "Fix error when keywords are defined as a list in cfg"Zuul2019-03-204-3/+40
|\ \ | |/
| * Fix error when keywords are defined as a list in cfgHervé Beraud2019-03-044-3/+40
| | | | | | | | | | | | | | | | When keywords are defined as a list in cfg file the generated output have errors and breaks metadata generation. Change-Id: Ie8a5f30d6af1e81ecf3ca40bc94bc460cca38179 Closes-Bug: #1811475
* | Remove libzmq-dev from integration.sh package installBen Nemec2019-03-191-1/+1
| | | | | | | | | | | | | | | | | | It appears this no longer exists on bionic, and we shouldn't need it for pbr tests. I suspect this may have been some copy-pasta from another project's integration tests. Change-Id: Ife631f77a92ee0b34c19e77cad782d94d18f2e74 Closes-Bug: 1820855
* | Merge "Change openstack-dev to openstack-discuss"Zuul2019-03-132-2/+2
|\ \ | |/ |/|
| * Change openstack-dev to openstack-discussmelissaml2019-02-152-2/+2
| | | | | | | | | | | | Mailinglists have been updated. Openstack-discuss replaces openstack-dev. Change-Id: Ifc72c767e76283df4608da8d7097ef86c367a5cf
* | Merge "Resolve ``ValueError`` when mapping value contains a literal ``=``."5.1.3Zuul2019-02-253-14/+62
|\ \ | |/ |/|
| * Resolve ``ValueError`` when mapping value contains a literal ``=``.Brandon LeBlanc2019-02-253-14/+62
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Example ``setup.cfg``:: project_urls = Documentation = https://format-pipfile.readthedocs.io/en/latest/?badge=latest Current result:: $ ./setup.py ERROR:root:Error parsing Traceback (most recent call last): File "/usr/local/lib/python3.7/site-packages/pbr/core.py", line 96, in pbr attrs = util.cfg_to_args(path, dist.script_args) File "/usr/local/lib/python3.7/site-packages/pbr/util.py", line 258, in cfg_to_args kwargs = setup_cfg_to_setup_kwargs(config, script_args) File "/usr/local/lib/python3.7/site-packages/pbr/util.py", line 336, in setup_cfg_to_setup_kwargs k, v = i.split(=) ValueError: too many values to unpack (expected 2) error in setup command: Error parsing /Users/brandon/src/format-pipfile/setup.cfg: ValueError: too many values to unpack (expected 2) After changes:: $ ./setup.py egg_info [...] $ grep -i project-url *.egg-info/PKG-INFO [...] Project-URL: Documentation, https://format-pipfile.readthedocs.io/en/latest/?badge=latest [...] add unit tests {1} pbr.tests.test_util.TestMapFieldsParsingScenarios.test_project_url_parsing(simple_project_urls) [0.034230s] ... ok {1} pbr.tests.test_util.TestMapFieldsParsingScenarios.test_project_url_parsing(query_parameters) [0.029791s] ... ok Closes-Bug: #1817592 Change-Id: Ifd4c46111528d99dadee77d6aabed201d9e84bdb Signed-off-by: Brandon LeBlanc <brandon@leblanc.codes>
* | Merge "Ignore --find-links in requirements file"5.1.2Zuul2019-01-223-5/+14
|\ \
| * | Ignore --find-links in requirements fileBen Nemec2019-01-223-5/+14
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | We already skip things like --index-url, but --find-links can also be present and also shouldn't be included in install_requires. This also fixes some issues with the existing unit test for this filtering. Change-Id: Ie8eca8c19e955d52722feaa71d5843ccd74b0da0 Closes-Bug: 1716808
* | | Do not globally replace path prefixWill Szumski2019-01-113-2/+36
| |/ |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | If a subdirectory contained the source prefix in it's name, this was replaced globally e.g using share/ansible = ansible/*, with the following directory structure: ansible/roles/kolla-ansible/test would result in the files being installed as follows: share/ansible/roles/kolla-share/test whereas we expected: share/ansible/roles/kolla-ansible/test This patch changes the behavior so that only the first occurance is replaced. Change-Id: I0aab845315dab0aaccd5f67725d2ebcf0fd08aef Fixes-Bug: 1810804
* | Change openstack-dev to openstack-discussqingszhao2018-12-041-1/+1
| | | | | | | | | | | | Mailinglists have been updated. Openstack-discuss replaces openstack-dev. Change-Id: I18a5f9696659bce3c03018b636968d5c3c96dc73
* | Merge "Correct documentation hyperlink for environment-markers"5.1.1Zuul2018-10-301-1/+1
|\ \