| opendevreview | OpenStack Release Bot proposed openstack/masakari-monitors master: Update master for stable/xena https://review.opendev.org/c/openstack/masakari-monitors/+/809031 | 01:57 |
|---|---|---|
| opendevreview | OpenStack Release Bot proposed openstack/masakari-monitors master: Add Python3 yoga unit tests https://review.opendev.org/c/openstack/masakari-monitors/+/809032 | 01:58 |
| *** bhagyashris is now known as bhagyashris|off | 05:35 | |
| zigo | Hi there! I saw the comment on libvirt-python in Masakari's requirements.txt | 11:48 |
| zigo | Could you guys please leave the required version there, even commented out, so at least we (distro package maintainers) can have a clue what's required? | 11:49 |
| opendevreview | Radosław Piliszek proposed openstack/python-masakariclient master: Replacing the link in HACKING.rst https://review.opendev.org/c/openstack/python-masakariclient/+/618288 | 12:00 |
| opendevreview | Radosław Piliszek proposed openstack/python-masakariclient master: Fix url to docs https://review.opendev.org/c/openstack/python-masakariclient/+/721200 | 12:02 |
| yoctozepto | zigo: hi; I discussed this and it is best to follow nova which just tests against what ubuntu and centos provide; tbh, we use very little of the libvirt api; if you need a specific version, I think it makes most sense to just copy this information from nova (if you have it there) | 12:06 |
| zigo | yoctozepto: This doesn't mean anything. In Debian, we're talking about what? Stable ? OldStable ? Unstable ?!? | 12:13 |
| zigo | Also, if that's the case, then you must also write this in the requirements.txt (ie: please follow what's in nova...) | 12:14 |
| opendevreview | Radosław Piliszek proposed openstack/python-masakariclient master: Fix home_page in setup.cfg https://review.opendev.org/c/openstack/python-masakariclient/+/809176 | 12:14 |
| zigo | In a release from now, I will forget what you just wrote to me ! :) | 12:14 |
| yoctozepto | zigo: that's not bad because it's for masakari-instancemonitor which runs alongside nova-compute which requires python-libvirt; I know it sounds bad but you can just assume it works with the version you have :D | 12:16 |
| yoctozepto | the libvirt API does not change really in the relevant scope at least | 12:16 |
| yoctozepto | similar thing happened to ceilometer btw | 12:16 |
| zigo | yoctozepto: We're talking about packaging here, where I should write some versions in debian/control, so what you just wrote unfortunately makes very little sense. I will *NOT* build masakari in an environment where Nova is installed, for example ... | 12:17 |
| opendevreview | Merged openstack/python-masakariclient master: Fix url to docs https://review.opendev.org/c/openstack/python-masakariclient/+/721200 | 12:21 |
| yoctozepto | zigo: ah, so first it's only for masakari-instancemonitor and it is not required at build time, only at run time if you want this particular monitor :-) I hope it helps | 12:21 |
| zigo | What would help for the packaging is what I suggest, but nv mind ... :) | 12:22 |
| yoctozepto | ok, sorry, I seem unable to help here | 12:23 |
| opendevreview | Thomas Goirand proposed openstack/masakari-monitors master: Add note for package maintainers in requirements.txt https://review.opendev.org/c/openstack/masakari-monitors/+/809178 | 12:28 |
| opendevreview | Merged openstack/python-masakariclient master: Fix home_page in setup.cfg https://review.opendev.org/c/openstack/python-masakariclient/+/809176 | 12:34 |
| zigo | yoctozepto: This! ^ :) | 12:37 |
| yoctozepto | zigo: ah, you mean to copy this info from the reno, got it; thanks :-) | 13:11 |
| zigo | Thanks for merging, and sorry for not expressing myself well enough. | 13:27 |
| yoctozepto | no problem | 13:51 |
| opendevreview | Merged openstack/masakari-monitors master: Add note for package maintainers in requirements.txt https://review.opendev.org/c/openstack/masakari-monitors/+/809178 | 14:04 |
| opendevreview | Radosław Piliszek proposed openstack/masakari-monitors master: Fix home_page in setup.cfg https://review.opendev.org/c/openstack/masakari-monitors/+/809237 | 16:42 |
| opendevreview | Merged openstack/masakari-monitors master: Update master for stable/xena https://review.opendev.org/c/openstack/masakari-monitors/+/809031 | 17:06 |
| opendevreview | Merged openstack/masakari-monitors master: Add Python3 yoga unit tests https://review.opendev.org/c/openstack/masakari-monitors/+/809032 | 17:06 |
| opendevreview | Merged openstack/masakari master: Update master for stable/xena https://review.opendev.org/c/openstack/masakari/+/808830 | 17:14 |
| opendevreview | Merged openstack/masakari master: Add Python3 yoga unit tests https://review.opendev.org/c/openstack/masakari/+/808831 | 17:14 |
| opendevreview | Merged openstack/masakari-dashboard master: Update master for stable/xena https://review.opendev.org/c/openstack/masakari-dashboard/+/808826 | 17:19 |
| opendevreview | Merged openstack/masakari-dashboard master: Add Python3 yoga unit tests https://review.opendev.org/c/openstack/masakari-dashboard/+/808827 | 17:19 |
| opendevreview | Merged openstack/masakari-monitors master: Fix home_page in setup.cfg https://review.opendev.org/c/openstack/masakari-monitors/+/809237 | 18:14 |
| *** osmanlicilegi is now known as Guest0 | 21:20 | |
Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!