| *** pmannidi is now known as pmannidi|Lunch | 02:34 | |
| NobodyCam | so for those following along at home.. what I am seeing on the nova side, once a node finishes cleaning after a and is in available state it is seen on `allocation candidate list` however any previous instances may still exist on the nova side; what I have found that works is: Node listed on ACL, Node has no allocations listed on its resource class, there are no reserved instances on `resource provider inventory list` | 03:05 |
|---|---|---|
| TheJulia | NobodyCam: what is the state the prior instance is in? | 03:43 |
| TheJulia | and where is the exception going sideways in nova exactly? | 03:44 |
| TheJulia | i ask because it sounds like it is failing | 03:44 |
| NobodyCam | No error. All timing | 03:45 |
| TheJulia | so why can't nova use the node if it is in the candidate list | 03:45 |
| TheJulia | ? | 03:45 |
| NobodyCam | oh sorry miss understood. No valid host. Is the error | 03:46 |
| TheJulia | when it goes to dispatch the request? | 03:46 |
| NobodyCam | This is for back to back deployments | 03:46 |
| TheJulia | on the message bus? | 03:46 |
| NobodyCam | Log trace back | 03:47 |
| NobodyCam | But I have stepped away from the laptop | 03:47 |
| TheJulia | NobodyCam: hmm | 03:48 |
| NobodyCam | Can we pick up in the morning | 03:48 |
| TheJulia | sure | 03:48 |
| NobodyCam | :) | 03:48 |
| TheJulia | I thought the candidate list came from placement | 03:48 |
| NobodyCam | they do | 03:50 |
| TheJulia | oO | 03:50 |
| *** pmannidi|Lunch is now known as pmannidi | 04:18 | |
| opendevreview | Merged openstack/ironic master: Silence debug logging from oslo_policy https://review.opendev.org/c/openstack/ironic/+/805875 | 05:58 |
| opendevreview | Merged openstack/ironic bugfix/18.0: Suppress policy deprecation and default change warnings https://review.opendev.org/c/openstack/ironic/+/799479 | 05:58 |
| iurygregory | good morning Ironic o/ | 06:01 |
| cenne | Good morning iurygregory! | 06:06 |
| * cenne is still away. Will catch up in bit. | 06:06 | |
| cenne | :p | 06:06 |
| iurygregory | morning cenne o/ | 06:23 |
| opendevreview | Merged openstack/ironic master: [doc] Cross-reference the updated power sync docs from tuning https://review.opendev.org/c/openstack/ironic/+/805847 | 06:46 |
| *** rpittau|afk is now known as rpittau | 07:06 | |
| rpittau | good morning ironic! o/ | 07:06 |
| iurygregory | morning rpittau o/ | 07:07 |
| rpittau | hey iurygregory :) | 07:07 |
| rpittau | we're looking into centos-stream and I can see centos-9 behind the corner :/ | 07:21 |
| iurygregory | oh wow | 07:22 |
| opendevreview | Riccardo Pittau proposed openstack/ironic-python-agent-builder master: [WIP] Use native Centos 8 Stream for dib images https://review.opendev.org/c/openstack/ironic-python-agent-builder/+/805957 | 07:41 |
| rpittau | ^ WIP for now as we'll have to do a double backflip with ipa | 07:43 |
| iurygregory | double backflip OMG | 07:43 |
| rpittau | mmmm and DIB | 07:46 |
| *** mgoddard- is now known as mgoddard | 07:47 | |
| opendevreview | Riccardo Pittau proposed openstack/ironic-python-agent-builder master: [WIP] Use native Centos 8 Stream for dib images https://review.opendev.org/c/openstack/ironic-python-agent-builder/+/805957 | 07:50 |
| opendevreview | Riccardo Pittau proposed openstack/ironic-python-agent master: Use Centos 8 Stream jobs to check and publish ipa DIB images https://review.opendev.org/c/openstack/ironic-python-agent/+/805958 | 07:52 |
| opendevreview | Aija Jauntēva proposed openstack/ironic master: Fix iDRAC import configuration job with errors https://review.opendev.org/c/openstack/ironic/+/805431 | 09:01 |
| opendevreview | Tadeas Kot proposed openstack/ironic master: Add support for fields in drivers API https://review.opendev.org/c/openstack/ironic/+/804416 | 09:09 |
| opendevreview | Tadeas Kot proposed openstack/python-ironicclient master: Add support for fields in drivers CLI This commit add support for fields selector to the driver CLI. https://review.opendev.org/c/openstack/python-ironicclient/+/805978 | 09:50 |
| opendevreview | Tadeas Kot proposed openstack/python-ironicclient master: Add support for fields in drivers CLI https://review.opendev.org/c/openstack/python-ironicclient/+/805978 | 09:52 |
| dtantsur | good afternoon ironic | 10:42 |
| dtantsur | stream-9 omg. let's wait until 2024 with that. | 10:42 |
| iurygregory | good afternoon dtantsur | 10:43 |
| iurygregory | well I don't think they will wait till 2024 lol | 10:43 |
| dtantsur | 2024 is centos-8 EOL, that's why :) | 10:43 |
| iurygregory | c8s no? | 10:43 |
| dtantsur | yeah, I meant stream-8 | 10:46 |
| opendevreview | Dmitry Tantsur proposed openstack/ironic bugfix/18.1: Silence debug logging from oslo_policy https://review.opendev.org/c/openstack/ironic/+/805986 | 12:00 |
| opendevreview | Dmitry Tantsur proposed openstack/bifrost master: Use TLS for virtual media when TLS is enabled https://review.opendev.org/c/openstack/bifrost/+/805874 | 12:13 |
| * dtantsur errands, brb | 12:20 | |
| rpittau | and the centos8-stream patches arefailing because of... the debian jobs! It didn't take our choice very well apparently | 12:41 |
| rpittau | some weird issue with one of the repos | 12:43 |
| iurygregory | rpittau, let the fight begin Debian VS C8S | 12:51 |
| iurygregory | round 2, Fight! | 12:51 |
| rpittau | Recheck! | 12:52 |
| iurygregory | yes! | 12:52 |
| iurygregory | you probably need to include the cookies :D | 12:52 |
| rpittau | the size is less terrible than I expecteed | 12:52 |
| rpittau | lol | 12:52 |
| iurygregory | let's open a beer to celebrate :D | 13:01 |
| iurygregory | 27min for the midcycle =) | 13:33 |
| * TheJulia CAFFINATES | 13:33 | |
| TheJulia | https://www.amazon.com/Unique-Doctor-Caffeinate-birthday-colleague/dp/B08ZC79QGB | 13:34 |
| iurygregory | Nice one! | 13:34 |
| dtantsur | :D | 13:37 |
| dtantsur | good morning TheJulia | 13:37 |
| dtantsur | and now, completely accidentally, I've discovered the Bifrost's os_ironic_node_info does not work with TLS. sigh. | 13:40 |
| iurygregory | wow | 13:40 |
| TheJulia | dtantsur: wut?!? | 13:53 |
| dtantsur | yeah. I suspect it ignores ca_cert. | 13:53 |
| * TheJulia blinks | 13:53 | |
| dtantsur | I'll post a patch and we'll see | 13:55 |
| iurygregory | https://meetpad.opendev.org/ironic 3min | 13:57 |
| opendevreview | Dmitry Tantsur proposed openstack/bifrost master: os_ironic_node_info: fix TLS and potentially other issues https://review.opendev.org/c/openstack/bifrost/+/806001 | 13:57 |
| opendevreview | Dmitry Tantsur proposed openstack/bifrost master: Use TLS for virtual media when TLS is enabled https://review.opendev.org/c/openstack/bifrost/+/805874 | 13:58 |
| iurygregory | lmcgann, https://etherpad.opendev.org/p/ironic-xena-midcycle | 14:55 |
| iurygregory | sam_z, fyi https://wiki.openstack.org/wiki/Meetings/Ironic in this wiki we have information about our weekly meeting / review-jams =) | 15:04 |
| TheJulia | sam_z: I am available later this afternoon | 15:08 |
| TheJulia | and Thursday looks fairly open | 15:08 |
| TheJulia | sam_z: What time are you thinking? I'm in US Pacific, fwiw | 15:10 |
| jeeva | unlucky: for some of us the day is done | 15:12 |
| iurygregory | TheJulia, https://twitter.com/_Skolt_Iury_/status/1384501771817660416/ | 15:12 |
| TheJulia | jeeva: happy to provide insight into internals earlier in my day. I typically start my workday at 6 AM local. | 15:14 |
| jeeva | i try 6am, but it its winter now, so then its like 7:30, clock off @ 3pm | 15:15 |
| TheJulia | As we shift to winter here, it is likely I will start getting up more towards 6:30 or 7 AM as well | 15:15 |
| TheJulia | Getting up when it is dark is just *really* hard for me | 15:16 |
| sam_z | TheJulia maybe like 2pm your time? | 15:17 |
| sam_z | PST is 3 hrs behind EST yea? | 15:17 |
| TheJulia | sam_z: yeah, that works | 15:17 |
| jeeva | got our first ironic team meeting tomorrow, to see if we can start deploying this | 15:17 |
| TheJulia | sam_z: it will be the end of my day, fwiw | 15:17 |
| TheJulia | jeeva: awesome! | 15:18 |
| sam_z | oh shoot does an earlier time work better? | 15:18 |
| TheJulia | sam_z: no, really, that time is fine for me | 15:18 |
| sam_z | aight i'll plan on it then :) | 15:18 |
| TheJulia | sam_z: send me a calendar invite | 15:21 |
| NobodyCam | good morning Ironic folks... Happy hump day | 15:27 |
| TheJulia | good morning NobodyCam | 15:28 |
| NobodyCam | :) | 15:28 |
| iurygregory | good morning NobodyCam o/ Happy hump day! | 15:30 |
| NobodyCam | :) oh ya | 15:30 |
| sam_z | TheJulia what calendar system do you use? | 15:30 |
| TheJulia | sam_z: send an invite to jkreger@redhat.com | 15:31 |
| TheJulia | gmail, fwiw | 15:31 |
| opendevreview | Merged openstack/ironic bugfix/18.1: Silence debug logging from oslo_policy https://review.opendev.org/c/openstack/ironic/+/805986 | 15:41 |
| opendevreview | Aija Jauntēva proposed openstack/ironic master: Add RAID mode conversion for idrac-redfish https://review.opendev.org/c/openstack/ironic/+/804112 | 15:50 |
| dtantsur | omg, opensuse is broken in bifrost again | 16:03 |
| rpittau | \o/ | 16:03 |
| TheJulia | now what? | 16:03 |
| iurygregory | <insert surprise gif > | 16:04 |
| rpittau | bye everyone! o/ | 16:04 |
| *** rpittau is now known as rpittau|afk | 16:04 | |
| iurygregory | bye rpittau|afk | 16:04 |
| dtantsur | surprised_pikatchu.jpg | 16:05 |
| dtantsur | I've pinged infra | 16:05 |
| TheJulia | surprised pixie boots is surprised | 16:05 |
| * dtantsur is wondering if pixie boots is a pokemon | 16:06 | |
| TheJulia | why not? | 16:06 |
| iurygregory | ++ makes sense to me | 16:06 |
| dtantsur | and the PTL is the one who has collected the pokemon, right? | 16:07 |
| iurygregory | is probably an evolution of Teddiursa | 16:07 |
| dtantsur | :D | 16:07 |
| iurygregory | https://pokemon.fandom.com/wiki/Teddiursa | 16:07 |
| dtantsur | looks kinda similar, although a bit friendlier | 16:08 |
| iurygregory | dtantsur, so maybe Ursaring? https://pokemon.fandom.com/wiki/Ursaring | 16:09 |
| iurygregory | intimidation ++ | 16:09 |
| dtantsur | neeeeat! | 16:10 |
| dtantsur | surprisingly enough, https://review.opendev.org/c/openstack/bifrost/+/806001/ seems to work from the first try (copy-paste succeeded!), please review | 16:13 |
| iurygregory | WHAT? O.o | 16:14 |
| *** sshnaidm is now known as sshnaidm|afk | 16:14 | |
| iurygregory | I love the magic (code removal) you did with openstack_cloud_from_module O.o | 16:15 |
| iurygregory | I will look after dinner =) | 16:15 |
| dtantsur | yeah, it uses https://opendev.org/openstack/ansible-collections-openstack/src/branch/master/plugins/module_utils/ironic.py that I wrote ~ a year ago | 16:15 |
| dtantsur | (and other helpers from before) | 16:15 |
| sam_z | IDENTIFY mGv3y7PAqGSOUjU sam_z | 16:23 |
| sam_z | wow oop | 16:23 |
| sam_z | time to change that | 16:23 |
| iurygregory | totally! | 16:24 |
| dtantsur | that's why I usually create a chat buffer with NickServ before issuing any command :) | 16:25 |
| dtantsur | now to something completely different: voices in my head hint me that we may want to make a round of wallaby releases after https://review.opendev.org/c/openstack/ironic/+/798316 merges | 16:28 |
| sam_z | thought i was clever for having keepass auto-type the identify command out... didn't realize the danger that entailed | 16:29 |
| dtantsur | :) | 16:30 |
| dtantsur | ironic, inspector, IPA, sushy, ironicclient, ironic-lib - all have meaningful wallaby changes | 16:31 |
| dtantsur | I'll probably start working from the bottom of this list | 16:32 |
| TheJulia | dtantsur: yes, releases ++ | 16:42 |
| dtantsur | TheJulia: https://review.opendev.org/c/openstack/ironic-inspector/+/804167 doesn't seem correct at first glance, at least its commit message | 16:46 |
| dtantsur | the SDK does support retries, maybe we simply don't tell it to.. | 16:46 |
| TheJulia | does it know to automatically retry on conflict lock errors like python-ironicclient ? | 16:47 |
| dtantsur | yep | 16:47 |
| TheJulia | because, locking errors I spotted in logs indicated it hard failed immediately | 16:47 |
| dtantsur | I'm not sure why it didn't work for you... | 16:47 |
| dtantsur | but I seem to recall that the number of retries is 0 by default | 16:48 |
| TheJulia | oh, that would do it | 16:48 |
| dtantsur | TheJulia: https://review.opendev.org/c/openstack/ironic/+/725883/1/ironic/conf/json_rpc.py#42 is for connection, there is a similar option for status codes | 16:49 |
| dtantsur | that being said, I'm not sure it's worth spending more time on it | 16:49 |
| TheJulia | I'd be cautious changing it at this point since apparently it did clear up issues for stevebaker's tripleo changes since it was blowing up there | 16:49 |
| TheJulia | ack | 16:49 |
| opendevreview | Nisha Agarwal proposed openstack/ironic master: Clean step to remove CA certificates from iLO https://review.opendev.org/c/openstack/ironic/+/799562 | 16:49 |
| dtantsur | yeah, it was more of JFYI | 16:49 |
| dtantsur | okay, all wallaby releases requested except for ironic and bifrost which have outstanding changes (cc iurygregory) | 16:50 |
| iurygregory | ack | 16:51 |
| iurygregory | bifrost doesn't seem to have any open changes in wallaby https://review.opendev.org/q/project:openstack/bifrost+branch:stable/wallaby | 16:52 |
| dtantsur | it will :) | 16:52 |
| iurygregory | ohhhh | 16:52 |
| *** dprince is now known as Guest5389 | 16:52 | |
| dtantsur | okay, maybe it's a weak argument for holding a release. but at the same time, I don't think anybody is using bifrost from pypi. | 16:52 |
| iurygregory | now we know dtantsur has the power to see the future :D | 16:53 |
| dtantsur | \o/ | 16:53 |
| iurygregory | yeah bifrost can wait for sure | 16:53 |
| opendevreview | Dmitry Tantsur proposed openstack/sushy-tools master: Basic support for vmedia TLS certificates from version 1.4.0 https://review.opendev.org/c/openstack/sushy-tools/+/805621 | 17:26 |
| opendevreview | Nisha Agarwal proposed openstack/ironic master: Clean step to remove CA certificates from iLO https://review.opendev.org/c/openstack/ironic/+/799562 | 17:28 |
| opendevreview | Dmitry Tantsur proposed openstack/bifrost master: Use TLS for virtual media when TLS is enabled https://review.opendev.org/c/openstack/bifrost/+/805874 | 18:04 |
| dtantsur | have a good night folks | 18:10 |
| lmcgann | Hey TheJulia, should i add reviewing our keylime patches to the review jam agenda here https://wiki.openstack.org/wiki/Meetings/Ironic ? | 18:24 |
| TheJulia | goodnight dtantsur | 18:25 |
| TheJulia | lmcgann: yes please | 18:25 |
| TheJulia | lmcgann: also update the next meeting agenda annoucement so people are made aware | 18:25 |
| lmcgann | update it to say we have an agenda for review jams next week? | 18:31 |
| TheJulia | lmcgann: yes | 18:41 |
| opendevreview | Merged openstack/ironic stable/wallaby: Remove redundant/legacy is_admin logic https://review.opendev.org/c/openstack/ironic/+/798316 | 18:48 |
| stevebaker | good morning | 19:44 |
| TheJulia | good morning | 19:50 |
Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!