Friday, 2023-11-24

-@gerrit:opendev.org- Roman Kuznecov proposed: [zuul/zuul-jobs] 901072: tox: Do not concat stdout and stderr in getting siblings https://review.opendev.org/c/zuul/zuul-jobs/+/90107206:32
@cidlik:matrix.orgHello.06:45
I found, IMHO, the issue in tox job. In the determination of sibling packages names stderr concated with stdout. Due to this package names maybe incorrect, if some warnings were printed. Example:
```
$ python setup.py --name <PACKAGE_NAME>
...
_DeprecatedInstaller: setuptools.installer and fetch_build_eggs are deprecated.
!!
********************************************************************************
Requirements should be satisfied by a PEP 517 installer.
If you are using pip, you can try `pip install --use-pep517`.
********************************************************************************
!!
dist.fetch_build_eggs(dist.setup_requires)
WARNING setuptools_scm.pyproject_reading toml section missing 'pyproject.toml does not contain a tool.setuptools_scm section'
<PACKAGE_NAME>
```
So tox cannot reinstall sibling packages. I posted change ([link](https://review.opendev.org/c/zuul/zuul-jobs/+/901072)) into zuul/zuul-jobs project. Could someone review this change?
@cidlik:matrix.org * Hello.06:45
I found, IMHO, the issue in tox job. In the determination of sibling packages names stderr concated with stdout. Due to this package names maybe incorrect, if some warnings were printed. Example:
```
$ python setup.py --name <PACKAGE_NAME>
...
_DeprecatedInstaller: setuptools.installer and fetch_build_eggs are deprecated.
!!
********************************************************************************
Requirements should be satisfied by a PEP 517 installer.
If you are using pip, you can try `pip install --use-pep517`.
********************************************************************************
!!
dist.fetch_build_eggs(dist.setup_requires)
WARNING setuptools_scm.pyproject_reading toml section missing 'pyproject.toml does not contain a tool.setuptools_scm section'
<PACKAGE_NAME>
```
So tox cannot reinstall sibling packages. I posted change ([link](https://review.opendev.org/c/zuul/zuul-jobs/+/901072)) into zuul/zuul-jobs project. Can someone review this change?
-@gerrit:opendev.org- Zuul merged on behalf of Matthieu Huin https://matrix.to/#/@mhuin:matrix.org: [zuul/zuul] 901673: zuul-web: ensure HTTPError is invoked with 4xx status code https://review.opendev.org/c/zuul/zuul/+/90167308:13
-@gerrit:opendev.org- Simon Westphahl proposed: [zuul/zuul] 901829: Set remote head refs in workspace repo clone https://review.opendev.org/c/zuul/zuul/+/90182913:04

Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!