| opendevreview | Takashi Kajinami proposed openstack/telemetry-tempest-plugin master: Fix stable/wallaby job definition without python 3 https://review.opendev.org/c/openstack/telemetry-tempest-plugin/+/841062 | 03:55 |
|---|---|---|
| opendevreview | Merged openstack/ceilometer stable/yoga: Debug log libvirt metadata version fails https://review.opendev.org/c/openstack/ceilometer/+/838531 | 06:44 |
| mrunge | tobias-urdin, hey, good morning, did you try gnocchi and ceilometer from master lately? E.g by using devstack? | 07:38 |
| mrunge | it seems the gates are fine, while I can not see any metrics getting consumed by gnocchi | 07:39 |
| mrunge | it always reports "archive-policy already exists" (which is correct). but that's it, the request is aborted | 07:39 |
| tobias-urdin | o/ | 08:10 |
| tobias-urdin | i have not tested that recently atleast | 08:10 |
| tobias-urdin | i can help check logs if you have link | 08:11 |
| tobias-urdin | but i haven't checked gnocchi since a month back iirc, so no changes there recently either | 08:11 |
| mrunge | it looks like gnocchiclient exceptions are changed/different now? | 08:12 |
| mrunge | I need to debug this further | 08:13 |
| mrunge | and I also think that some of the patches around sqlachelmy 1.4 are not included in the 4.4.1 tag, but sqlalchemy is now unpinned there | 08:13 |
| tobias-urdin | iirc 4.4.1 should include sqlalchemy 1.4 support, not sure if devstack pulls a tagged release though, doesn't it use branch directly? | 08:15 |
| tobias-urdin | anyhow there is nothing new on the gnocchi side but might be dependency changes that is giving issues | 08:16 |
| tobias-urdin | let me know if you need any help | 08:17 |
| mrunge | tobias-urdin, so e.g https://github.com/gnocchixyz/gnocchi/commit/5ed9cc9a042d2b5717bd935956a99a2ba58d2e4a is missing in 4.4.1 | 08:44 |
| mrunge | tobias-urdin, or here: https://github.com/openstack/ceilometer/blob/master/ceilometer/publisher/gnocchi.py#L274 the exception thrown is a gnocchiclient exception named Conflict. | 08:46 |
| mrunge | so. *something* has changed | 08:46 |
| mrunge | and I wonder when this broke and how we didn't catch any of this in CI | 08:46 |
| opendevreview | Merged openstack/telemetry-tempest-plugin master: setup.cfg: Replace dashes with underscores https://review.opendev.org/c/openstack/telemetry-tempest-plugin/+/789456 | 09:39 |
| tobias-urdin | mrunge: ok so all of the sqlalchemy 1.4 stuff wasn't released, i can investigate releasing it, but i thought the devstack stuff went directly to master branch when testing | 14:19 |
| mrunge | it does not | 14:20 |
| mrunge | it installs from pip? | 14:20 |
| mrunge | at least I could cherry-pick the change | 14:20 |
| mrunge | however, I am not sure how related the sqlalchemy change is to what I see otherwise | 14:21 |
| tobias-urdin | mrunge: i've created a new 4.4.2 release https://github.com/gnocchixyz/gnocchi/releases/tag/4.4.2, need jd to push to pypi though https://github.com/gnocchixyz/gnocchi/issues/1230 | 14:29 |
| mrunge | thank you tobias-urdin | 14:30 |
| tobias-urdin | dont see anything interesting in there though, mostly CI fixes and smaller bugfixes, so nothing for sqlalchemy there | 14:31 |
| mrunge | I don't see the sqlalchemy change in there | 14:31 |
| tobias-urdin | $git tag --contain 58c78bf063f9ca4ef1f320d880ed4cf3fe156f62 | 14:33 |
| tobias-urdin | 4.4.1 | 14:33 |
| tobias-urdin | it should already be there, just checking the master commit, another commit id for backports | 14:34 |
| tobias-urdin | doesn't hurt releasing 4.4.2 i guess, will just leave it there | 14:38 |
| mrunge | the patch I linked was not included in 4.4.1 | 14:39 |
| mrunge | maybe double-check? | 14:39 |
| tobias-urdin | it becomes a separate PR when backporting, so it's not the same git commit, 58c78bf063f9ca4ef1f320d880ed4cf3fe156f62 is same change but for stable/4.4 branch and that one was included in 4.4.1 | 15:02 |
| opendevreview | Takashi Kajinami proposed openstack/python-aodhclient master: Remove six https://review.opendev.org/c/openstack/python-aodhclient/+/841139 | 16:04 |
| opendevreview | Takashi Kajinami proposed openstack/python-aodhclient master: Remove six https://review.opendev.org/c/openstack/python-aodhclient/+/759178 | 16:37 |
| opendevreview | Takashi Kajinami proposed openstack/python-aodhclient master: DNM: Debugging test failure https://review.opendev.org/c/openstack/python-aodhclient/+/841150 | 16:57 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!