Wednesday, 2023-11-29

@cidlik:matrix.org> <@cidlik:matrix.org> Hello.09:38
> 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?
Hello.
Can someone review this change, please?
@clarkb:matrix.org> <@cidlik:matrix.org> Hello.16:14
> Can someone review this change, please?
This change right: https://review.opendev.org/c/zuul/zuul-jobs/+/901072?
-@gerrit:opendev.org- Zuul merged on behalf of Benjamin Schanzel: [zuul/nodepool] 898109: Fix logging of failed node launches https://review.opendev.org/c/zuul/nodepool/+/89810916:19
-@gerrit:opendev.org- Zuul merged on behalf of Roman Kuznecov: [zuul/zuul-jobs] 901072: tox: Do not concat stdout and stderr in getting siblings https://review.opendev.org/c/zuul/zuul-jobs/+/90107216:35
-@gerrit:opendev.org- James E. Blair https://matrix.to/#/@jim:acmegating.com proposed: [zuul/zuul] 902173: Update tenant error pagination total https://review.opendev.org/c/zuul/zuul/+/90217316:49
-@gerrit:opendev.org- Zuul merged on behalf of James E. Blair https://matrix.to/#/@jim:acmegating.com:18:02
- [zuul/zuul] 901511: Use server-side filtering and pagination in config error page https://review.opendev.org/c/zuul/zuul/+/901511
- [zuul/zuul] 901512: Use tenant-status endpoint to show the bell https://review.opendev.org/c/zuul/zuul/+/901512
-@gerrit:opendev.org- James E. Blair https://matrix.to/#/@jim:acmegating.com proposed: [zuul/zuul] 902180: Add a filter icon to config error values https://review.opendev.org/c/zuul/zuul/+/90218018:42
@fungicide:matrix.orgcorvus: how are you previewing 902180? when i visit https://c209af3df1a4a20986e3-6d27f622786e2e5d7efb56feaf2b64dc.ssl.cf2.rackcdn.com/t/openstack/status clicking the bell icon does nothing20:04
@fungicide:matrix.orgnevermind, now it's working20:04
@fungicide:matrix.orgserver-side filtering from the parent obviously isn't previewable of course20:07
@jim:acmegating.comfungi: maybe slowness serving some required js bit on the log server  -- or, i've noticed during all my testing, that we have to click on the actual bell, not the blue area around it.  (i think that is an opportunity for improvement, but in the mean time...)20:07
@jim:acmegating.comfungi: opendev is running the server-side filtering, so everything is previewable now20:07
@fungicide:matrix.orgoh, yep i just realized that too ;)20:08
@fungicide:matrix.orgmore up to date than even i realized20:08
-@gerrit:opendev.org- James E. Blair https://matrix.to/#/@jim:acmegating.com proposed: [zuul/zuul] 902185: Try left alignment of zoomy icons https://review.opendev.org/c/zuul/zuul/+/90218520:11
-@gerrit:opendev.org- James E. Blair https://matrix.to/#/@jim:acmegating.com proposed: [zuul/zuul] 902118: Freeze job dependencies with job graph https://review.opendev.org/c/zuul/zuul/+/90211822:36
-@gerrit:opendev.org- Zuul merged on behalf of James E. Blair https://matrix.to/#/@jim:acmegating.com: [zuul/zuul] 901636: Further fixes to check deduplication https://review.opendev.org/c/zuul/zuul/+/90163623:54

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