Wednesday, 2023-09-27

opendevreviewNatz cheng proposed openstack/os-vif master: cleanup py27 support  https://review.opendev.org/c/openstack/os-vif/+/89659801:57
*** ralonsoh_afk is now known as ralonsoh06:21
opendevreviewSahid Orentino Ferdjaoui proposed openstack/nova master: scheduler: AggregateMultitenancyIsolation to support unlimited tenant  https://review.opendev.org/c/openstack/nova/+/89651207:08
dvo-plvHello, All08:47
dvo-plvMaybe you will have a chance to take a look on the reproposed spec file ? https://review.opendev.org/c/openstack/nova-specs/+/89592408:48
dvo-plvand check dependent patch for it https://review.opendev.org/c/openstack/nova/+/87607508:48
opendevreviewSahid Orentino Ferdjaoui proposed openstack/nova master: scheduler: AggregateMultitenancyIsolation to support unlimited tenant  https://review.opendev.org/c/openstack/nova/+/89651209:28
opendevreviewAmit Uniyal proposed openstack/nova master: Adds device tagging functional tests  https://review.opendev.org/c/openstack/nova/+/89516210:09
opendevreviewAmit Uniyal proposed openstack/nova master: Adds device tagging functional tests  https://review.opendev.org/c/openstack/nova/+/89516210:20
opendevreviewAmit Uniyal proposed openstack/nova master: Adds device tagging functional tests  https://review.opendev.org/c/openstack/nova/+/89516211:02
opendevreviewAmit Uniyal proposed openstack/nova master: Device tags: don't pass pf_interface=True to get_mac_by_pci_address  https://review.opendev.org/c/openstack/nova/+/67059311:02
opendevreviewAmit Uniyal proposed openstack/nova master: WIP: Device tagging: expose target PCI address, not source  https://review.opendev.org/c/openstack/nova/+/67212711:02
stephenfinbauzas: Can you look at https://review.opendev.org/c/openstack/placement/+/886229 and https://review.opendev.org/c/openstack/placement/+/888281 when you get a chance? (SQLAlchemy 2.x stuff)12:32
bauzasstephenfin: yeah I'll try12:32
opendevreviewStephen Finucane proposed openstack/placement master: db: Workaround SQLAlchemy 2.x bug  https://review.opendev.org/c/openstack/placement/+/88828112:37
opendevreviewStephen Finucane proposed openstack/placement master: Add job to test with SQLAlchemy master (2.x)  https://review.opendev.org/c/openstack/placement/+/88622912:37
opendevreviewStephen Finucane proposed openstack/placement master: db: Wrap raw SQL query in sqlalchemy.text  https://review.opendev.org/c/openstack/placement/+/89665012:37
opendevreviewPavlo Shchelokovskyy proposed openstack/nova master: Fix the libvirt.images_type value in KVM docs  https://review.opendev.org/c/openstack/nova/+/89665112:58
bauzasstephenfin: ack, thanks for the quick respins, will check13:00
sahido/ I have proposed a change for the filter AggregateMultiTenancyFilter, I have reused what we do in placement14:33
sahidhttps://review.opendev.org/c/openstack/nova/+/89651214:33
sahidcan you just tell whether that looks to be in the right direction for you guys?14:34
bauzasI'll try 14:35
sahidthank you14:36
opendevreviewSylvain Bauza proposed openstack/nova-specs master: explain why ironic-shards was reverted in the spec  https://review.opendev.org/c/openstack/nova-specs/+/89667914:53
opendevreviewStephen Finucane proposed openstack/placement master: Add job to test with SQLAlchemy master (2.x)  https://review.opendev.org/c/openstack/placement/+/88622914:58
opendevreviewDan Smith proposed openstack/nova-specs master: Add libvirt-dev-alias spec  https://review.opendev.org/c/openstack/nova-specs/+/89280614:59
opendevreviewStephen Finucane proposed openstack/placement master: db: Wrap raw SQL query in sqlalchemy.text  https://review.opendev.org/c/openstack/placement/+/89665015:01
opendevreviewStephen Finucane proposed openstack/placement master: Add job to test with SQLAlchemy master (2.x)  https://review.opendev.org/c/openstack/placement/+/88622915:01
*** blarnath is now known as d34dh0r5315:06
*** ralonsoh is now known as ralonsoh_ooo17:02
*** ykarel is now known as ykarel|away17:16
mnaserhttps://bugs.launchpad.net/nova/+bug/2037585 -- does deleting neutron ports in a specific order sound like a reasonable fix here?18:51
mnaserif so i can push that proposal18:51
greatgatsbyHi.  I was asking before about our compute aggregates become out of sync with the resource provider aggregates.  We had it happen again last night, and I've found a PUT /resource_providers/<uuid>/aggregates in the logs at the time we detected them out of sync.  What service or state might trigger this update on its own?  I've been grepping through logs most of the day, but still haven't found what 19:30
greatgatsbytriggers this to happen.19:30
greatgatsbythanks *a lot* for any help19:31
JayFgreatgatsby: just curious, are the resources in question Ironic nodes? 19:31
greatgatsbythanks for the reply.  No ironic, we pxe boot and use ansible for config19:33
JayFack; sorry I won't be much help then as that's my specialty; good luck!19:33
greatgatsbythanks :-)19:34
dansmithgreatgatsby: nova-api will do that in response to creating or deleting aggregates on the nova side19:34
dansmithnothing else should be doing that, AFAIK19:35
dansmithcan you trace the origin with a req-id or source ip?19:35
greatgatsbysource ip in one case came from the controller that lost the resource provider aggs.  The nova aggs have never changed (or at least, we haven't changed them, and they stay consistent)19:36
greatgatsbyso, for example, compute 1 will still have the correct linux agg on the nova side, but it disappears on the resource provider side19:36
greatgatsbythis happened late last night19:37
greatgatsbyin one PUT message I see "nova-compute keystoneauth1/4.5.0 python-requests...."19:38
dansmith"from the controller that lost the aggregates" doesn't mean anything to me19:41
dansmithdo you mean from a compute node?19:41
greatgatsbyoops, sorry, yes, the compute19:41
dansmiththere is some aggregate-manipulation that happens in the provider tree stuff now that I'm not familiar enough to speak about without some study19:42
dansmithso must be something related to that I guess19:42
dansmithbest option would be to chat with gibi in his morning (central EU)19:42
dansmithis the uuid in the PUT /resource_providers/uuid/aggregate the uuid of that same compute node?19:43
greatgatsbyyes, exactly19:43
greatgatsbyit's the only PUT to the resources providers and only to that specific node19:44
dansmithI really didn't think that any of the computes would be adjusting their own aggregate membership, so that seems weird to me19:45
dansmithis there something in the compute log that matches that timestamp?19:45
greatgatsbythere probably is, but I haven't found it.  I've been grepping through logs all day.  I just check the nova_api aggregates table in the database, the creation time is still months old, so they didn't get recreated.  Updated is NULL though.19:47
*** JayF is now known as Guest150520:10
*** JasonF is now known as jayf20:10
*** jayf is now known as JayF20:11
*** kopecmartin is now known as kopecmartin|off20:57

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