| *** ykarel|away is now known as ykarel | 04:53 | |
| *** dasm is now known as Guest1555 | 07:16 | |
| gibi | greatgatsby, dansmith: based on my serach the only code that writes aggregates to placement is the report client's set_aggregates_for_provider() https://github.com/openstack/nova/blob/87d4807848bb9546c1fca972da2eb2eda13eb08d/nova/scheduler/client/report.py#L1122 | 07:45 |
|---|---|---|
| gibi | we call that during nova aggregate manipulation but you said that is not happening | 07:46 |
| gibi | so I ignore those places now | 07:46 |
| gibi | we call that from update_from_provider_tree() as well that is the codepath dansmith refers to above | 07:47 |
| gibi | that will basically flush the nova-compute's internal caches (the provider tree) back to placement | 07:49 |
| gibi | so we need to track down what updates aggregates in that cache | 07:50 |
| sahid | o/ | 07:52 |
| sahid | any issue with our ci, it looks like zull did not pass completely for https://review.opendev.org/c/openstack/nova/+/896512 | 07:53 |
| sahid | ? | 07:53 |
| *** Continuity__ is now known as Continuity | 08:21 | |
| gibi | greatgatsby, dansmith: there is _refresh_associations in https://github.com/openstack/nova/blob/87d4807848bb9546c1fca972da2eb2eda13eb08d/nova/scheduler/client/report.py#L817-L818 that updates the placement aggregates in the caches | 08:29 |
| gibi | greatgatsby: there is a DEBUG log associated with it | 08:29 |
| gibi | "Refreshing aggregate associations for resource provider" | 08:30 |
| gibi | do you have nova-compute debug logs from that time period? do you see this logged? | 08:30 |
| sahid | let's try a recheck... | 08:38 |
| * zigo just booted and pinged his first VM running on top of Bobcat ! | 09:46 | |
| zigo | Thanks guys for all the work. | 09:47 |
| zigo | stephenfin: Especially you. Now I have to finish the work of backporting all the SQLA 2.x patches ... :P | 09:47 |
| sahid | all is green | 12:08 |
| *** Guest1555 is now known as dasm | 14:48 | |
| *** d34dh0r5- is now known as d34dh0r53 | 15:10 | |
| greatgatsby | gibi: thanks so much for the info! I'm in and out of the office today, so I'll try to take a look. | 16:50 |
| opendevreview | Merged openstack/placement master: Changed /tmp/migrate-db.rc to /root/migrate-db.rc https://review.opendev.org/c/openstack/placement/+/882436 | 17:19 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!