| opendevreview | yatin proposed openstack/devstack master: DNM: Testing only jammy job https://review.opendev.org/c/openstack/devstack/+/839389 | 05:14 |
|---|---|---|
| opendevreview | yatin proposed openstack/devstack master: DNM: Testing only jammy job https://review.opendev.org/c/openstack/devstack/+/839389 | 06:17 |
| elodilles | melwitt: ack, thanks for the info | 06:31 |
| elodilles | meanwhile i see lot of tox-docs job failure in zuul (in several projects) which seems to be a result of the latest oslo.policy release. I'll try to look into it, but first I've pinged the oslo team on #openstack-oslo | 06:33 |
| *** pojadhav- is now known as pojadhav | 06:44 | |
| *** jpena|off is now known as jpena | 06:58 | |
| bhagyashris|ruck | jrosser, hey | 07:13 |
| *** pojadhav is now known as pojadhav|lunch | 08:21 | |
| *** pojadhav|lunch is now known as pojadhav | 09:11 | |
| jrosser | bhagyashris|ruck: hello | 09:37 |
| bhagyashris|ruck | jrosser, resolved the issue. actually we tripleo team are facing one issue https://bugs.launchpad.net/tripleo/+bug/1970710 due to change in openstack-ansible-os_tempest https://review.opendev.org/c/openstack/openstack-ansible-os_tempest/+/803477 . now we have fix at our end https://review.opendev.org/c/openstack/tripleo-quickstart-extras/+/839694 | 09:41 |
| bhagyashris|ruck | so no worries | 09:41 |
| bhagyashris|ruck | thanks | 09:41 |
| opendevreview | Victoria Martinez de la Cruz proposed openstack/devstack-plugin-ceph master: Deploy with cephadm https://review.opendev.org/c/openstack/devstack-plugin-ceph/+/826484 | 10:25 |
| opendevreview | yatin proposed openstack/devstack master: [DNM] Test OVN grenade job https://review.opendev.org/c/openstack/devstack/+/837303 | 10:27 |
| opendevreview | Victoria Martinez de la Cruz proposed openstack/devstack-plugin-ceph master: Deploy with cephadm https://review.opendev.org/c/openstack/devstack-plugin-ceph/+/826484 | 10:32 |
| *** pojadhav is now known as pojadhav|afk | 11:42 | |
| opendevreview | Merged openstack/tempest master: Fix the integrated-gate-compute job template https://review.opendev.org/c/openstack/tempest/+/839647 | 11:49 |
| *** pojadhav|afk is now known as pojadhav | 12:45 | |
| opendevreview | yatin proposed openstack/devstack master: Collect status of all services https://review.opendev.org/c/openstack/devstack/+/839752 | 12:49 |
| *** pojadhav is now known as pojadhav|afk | 13:42 | |
| opendevreview | Dan Smith proposed openstack/devstack master: Performance monitor counting improvements https://review.opendev.org/c/openstack/devstack/+/839067 | 13:56 |
| opendevreview | Dan Smith proposed openstack/devstack master: WIP: Test static perfdata comparisons https://review.opendev.org/c/openstack/devstack/+/838947 | 13:56 |
| dansmith | unfortunately, we're generating more than 100k rows in the query log, which means we need to bump that high ^ enough to capture them all. When I set it for 1m rows, the jobs OOM, so ... :/ | 13:56 |
| *** jpena is now known as jpena|off | 14:14 | |
| opendevreview | yatin proposed openstack/devstack master: DNM: Testing only jammy job https://review.opendev.org/c/openstack/devstack/+/839389 | 14:26 |
| opendevreview | yatin proposed openstack/devstack master: DNM: Testing only jammy job https://review.opendev.org/c/openstack/devstack/+/839389 | 15:01 |
| opendevreview | yatin proposed openstack/devstack master: Collect status of all services https://review.opendev.org/c/openstack/devstack/+/839752 | 15:03 |
| *** pojadhav|afk is now known as pojadhav | 15:17 | |
| opendevreview | Dan Smith proposed openstack/devstack master: Performance monitor counting improvements https://review.opendev.org/c/openstack/devstack/+/839067 | 15:58 |
| dansmith | elodilles: this https://review.opendev.org/c/openstack/glance/+/839786 should fix glance's broken docs build due to the oslo policy thing? | 16:06 |
| elodilles | dansmith: yes, but actually oslo.policy release just showed that the above fix is needed on stable branches for some repos :) | 16:08 |
| elodilles | dansmith: i mean... not directly fixing the oslo.policy thing, just the same errors on stable branches where the error should not happen at all (as oslo.policy is released on master / zed) | 16:09 |
| dansmith | elodilles: ack | 16:10 |
| elodilles | dansmith: on the other hand since u-c still contains 'oslo.policy===3.11.0', as a side effect it now fixes that, too o:) | 16:10 |
| dansmith | elodilles: so point is, we don't *need* this: https://review.opendev.org/c/openstack/releases/+/839775/2/deliverables/zed/oslo.policy.yaml to fix it if we have your tox patch yeah? | 16:11 |
| elodilles | dansmith: that is needed for master | 16:11 |
| dansmith | which? | 16:12 |
| elodilles | * 3.12.1 release from oslo.policy for master branch | 16:12 |
| dansmith | okay | 16:12 |
| elodilles | * and my patch for stable branches to avoid similar issues in the future on stable branches o:) | 16:13 |
| elodilles | dansmith: note that (what i was trying to say above) my docs patch also fixes master until this patch does not merge: https://review.opendev.org/c/openstack/requirements/+/839577 | 16:16 |
| elodilles | i hope it makes sense what i'm trying to say :D | 16:16 |
| dansmith | elodilles: right, I'm trying to figure out if quick merging your tox patch gets glance master happy again, since I have the power to do that :) | 16:16 |
| dansmith | sounds like it does | 16:16 |
| elodilles | yes, it should :) | 16:17 |
| dansmith | cool | 16:18 |
| elodilles | dansmith: btw, if you could also review the nova version of the same patch that would be awesome :D >>> https://review.opendev.org/c/openstack/nova/+/839781 | 16:26 |
| dansmith | elodilles: ack I was just waiting to see it work in glance, but I guess if gibi is already good, I guess no harm | 16:27 |
| gibi | yeah, I tested it locally, it works | 16:27 |
| dansmith | man, like 30k queries to the keystone DB before we even get to the first non-keystone project setup | 19:10 |
| dansmith | no wonder 300k query log entries isn't enough | 19:12 |
| dansmith | over 40k before we're done creating service accounts | 19:14 |
| dansmith | this can't be right | 19:16 |
| *** artom__ is now known as artom | 19:32 | |
| opendevreview | Dan Smith proposed openstack/devstack master: Change DB counting mechanism https://review.opendev.org/c/openstack/devstack/+/839820 | 19:39 |
| opendevreview | Dan Smith proposed openstack/devstack master: Change DB counting mechanism https://review.opendev.org/c/openstack/devstack/+/839820 | 21:30 |
| opendevreview | Dan Smith proposed openstack/devstack master: Improve API log parsing https://review.opendev.org/c/openstack/devstack/+/839067 | 21:36 |
| opendevreview | Dan Smith proposed openstack/devstack master: Change DB counting mechanism https://review.opendev.org/c/openstack/devstack/+/839820 | 23:13 |
| opendevreview | Dan Smith proposed openstack/devstack master: Improve API log parsing https://review.opendev.org/c/openstack/devstack/+/839067 | 23:16 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!