stevebaker | (I'm back) | 00:16 |
---|---|---|
openstackgerrit | Merged openstack/ironic stable/ussuri: Update number of VM on ironic-base https://review.opendev.org/744538 | 00:39 |
openstackgerrit | Verification of a change to openstack/ironic failed: Pin ironic-tempest-plugin https://review.opendev.org/744547 | 01:01 |
openstackgerrit | Verification of a change to openstack/ironic failed: Auto extend the timeout for RAX hosts https://review.opendev.org/741853 | 01:07 |
tzumainn | janders, thanks (sorry, was afk)! the multi-tenant stuff was fun to do, and we're slooooowly preparing a demo presentation for devconf.us in september | 01:36 |
tzumainn | if you have a link to any of your work, I'd definitely be interested in taking a look! | 01:37 |
*** diurnalist has joined #openstack-ironic | 02:12 | |
*** tzumainn has quit IRC | 02:37 | |
*** mkrai has joined #openstack-ironic | 03:02 | |
*** rh-jelabarre has quit IRC | 03:22 | |
*** diurnalist has quit IRC | 03:38 | |
*** patchbot has quit IRC | 04:17 | |
openstackgerrit | Merged openstack/ironic stable/stein: Pin ironic-tempest-plugin https://review.opendev.org/744547 | 04:18 |
*** patchbot has joined #openstack-ironic | 04:28 | |
*** gyee has quit IRC | 04:48 | |
openstackgerrit | Verification of a change to openstack/ironic failed: Auto extend the timeout for RAX hosts https://review.opendev.org/741853 | 05:16 |
*** janders has quit IRC | 05:28 | |
*** tkajinam has quit IRC | 05:34 | |
*** tkajinam has joined #openstack-ironic | 05:35 | |
*** tkajinam has quit IRC | 05:50 | |
*** tkajinam has joined #openstack-ironic | 05:51 | |
*** yedongcan has joined #openstack-ironic | 05:58 | |
*** yaawang has quit IRC | 06:01 | |
*** yaawang has joined #openstack-ironic | 06:01 | |
arne_wiebalck | Good morning, ironic! | 06:20 |
openstackgerrit | Kaifeng Wang proposed openstack/ironic-inspector master: Identify accelerator devices during introspection https://review.opendev.org/745289 | 06:21 |
*** Zempashi has joined #openstack-ironic | 06:38 | |
*** mkrai has quit IRC | 06:48 | |
*** mkrai_ has joined #openstack-ironic | 06:48 | |
*** janders has joined #openstack-ironic | 06:49 | |
*** ntt__ has joined #openstack-ironic | 06:50 | |
*** yedongcan has quit IRC | 06:52 | |
*** mkrai_ has quit IRC | 06:57 | |
*** janders has quit IRC | 07:12 | |
*** janders has joined #openstack-ironic | 07:21 | |
rpittau | good morning ironic! o/ | 07:23 |
rpittau | happy friday! | 07:26 |
janders | g'day rpittau | 07:27 |
janders | same to you! :) | 07:27 |
*** k_mouza has joined #openstack-ironic | 07:27 | |
*** k_mouza has quit IRC | 07:27 | |
rpittau | hey janders :) | 07:29 |
rpittau | how's it going on the other side of the terraqueous globe ? | 07:30 |
*** Lucas_Gray has joined #openstack-ironic | 07:31 | |
janders | depends which part I suppose | 07:31 |
janders | I think New Zealanders are doing well | 07:31 |
janders | with us Aussies - the Victorians aren't too happy, peeps in New South Wales are a little worried with covid but okay, the rest are doing well for now | 07:32 |
rpittau | :) | 07:32 |
janders | I need to fix up my mountainbike cause it started doing this: | 07:32 |
*** dougsz has joined #openstack-ironic | 07:33 | |
janders | https://photos.app.goo.gl/AzPZ8djFzgcZP2z97 | 07:33 |
janders | which is rather annoying while trying to ride anything that isn't downhill | 07:33 |
rpittau | it doesn't sound good :/ | 07:34 |
janders | it happened a fair bit away from home last weekend but some bush mechanics got me home | 07:34 |
janders | (think along the lines of hitting metal with large rocks to make things lock in place) | 07:34 |
rpittau | oh interesting, sounds like standard way to fix a baremetal node | 07:35 |
janders | probably needs a new freewheel assembly but I will take it to a shop, these things rarely break so I don't have the tools | 07:35 |
janders | haha! :) | 07:35 |
janders | so yeah that's me | 07:35 |
janders | how are you doing? Any exciting weekend plans? | 07:36 |
rpittau | I think I will slowly melt, temperatures are getting higher and higher | 07:37 |
openstackgerrit | Merged openstack/ironic master: agent_client: support custom TLS certificates https://review.opendev.org/744731 | 07:37 |
rpittau | plan for the weekend, finish ordering all the bathroom furnitures for the new apartment :) | 07:37 |
janders | nice! :) | 07:38 |
janders | nothing like a new jacuzzi | 07:39 |
rpittau | I was tempted to, but it would take the entire bathroom! | 07:40 |
janders | yeah that's the trouble with these things | 07:40 |
janders | I once thought about putting one in the backyard but neighbours come and go, some are great, some much less so, that can cause issues | 07:41 |
rpittau | I did the design for the renovations and things fit at the millimiter :D | 07:41 |
janders | nice! :) | 07:41 |
janders | I've done a partial bathroom renovation with my wife a couple years back... challenging but rewarding | 07:42 |
rpittau | hardest part so far: tiles | 07:42 |
*** yaawang has quit IRC | 07:45 | |
*** yaawang has joined #openstack-ironic | 07:46 | |
janders | these are quite fun or quite frustrating depending on the quality of walls I find | 07:48 |
iurygregory | good morning everyone Happy Friday! | 07:59 |
rpittau | hey iurygregory, Happy Friday! :) | 08:03 |
*** Qianbiao has joined #openstack-ironic | 08:07 | |
openstackgerrit | Iury Gregory Melo Ferreira proposed openstack/ironic master: Add tempest default_boot_option and altflavor https://review.opendev.org/741886 | 08:07 |
janders | good morning iurygregory | 08:07 |
*** Qianbiao has quit IRC | 08:08 | |
*** Qianbiao has joined #openstack-ironic | 08:09 | |
Qianbiao | good morning ironic. | 08:09 |
*** dmellado has quit IRC | 08:09 | |
Qianbiao | hi all, where can i get the latest ironic victoria release rpm info. | 08:09 |
Qianbiao | i want to query where ibmc raid config code is released or not. | 08:10 |
rpittau | Qianbiao: you can probably ask on #rdo | 08:12 |
Qianbiao | ok thanks rpittau | 08:13 |
*** dmellado has joined #openstack-ironic | 08:14 | |
iurygregory | https://trunk.rdoproject.org/centos8/component/baremetal/a9/ed/a9ed390f08c0dd8263a0050a12b36efc36c70c48_224e9ee2/commit.yaml | 08:14 |
iurygregory | maybe this will help | 08:15 |
iurygregory | oh its python-agent nvm | 08:15 |
Qianbiao | <iurygregory> should be some url like that. but i did not find the entry index page to search ironic package. | 08:15 |
iurygregory | rdo people will know better =) | 08:16 |
*** lucasagomes has joined #openstack-ironic | 08:16 | |
Qianbiao | sure | 08:18 |
*** sshnaidm|afk is now known as sshnaidm|off | 08:19 | |
openstackgerrit | Aija Jaunteva proposed openstack/ironic master: Add import, export configuration steps https://review.opendev.org/745300 | 08:20 |
iurygregory | sad the change to generate PEM on ngs didn't help the multinode grenade =( | 08:25 |
* iurygregory cries | 08:26 | |
openstackgerrit | Merged openstack/ironic-python-agent stable/queens: Pin Ironic Tempest Plugin https://review.opendev.org/744969 | 08:27 |
*** yaawang has quit IRC | 08:32 | |
*** yaawang has joined #openstack-ironic | 08:33 | |
rpittau | iurygregory: same error ? | 08:34 |
iurygregory | yep .-. | 08:34 |
iurygregory | I thought that adding the -m PEM would help | 08:35 |
iurygregory | https://review.opendev.org/#/c/739186/2/devstack/plugin.sh | 08:35 |
patchbot | patch 739186 - networking-generic-switch - Generate key on PEM format - 2 patch sets | 08:35 |
iurygregory | wondering if not only generating the key is enough and we need to do something more | 08:36 |
*** Lucas_Gray has quit IRC | 08:40 | |
*** Lucas_Gray has joined #openstack-ironic | 08:44 | |
openstackgerrit | Kaifeng Wang proposed openstack/ironic-inspector master: Identify accelerator devices during introspection https://review.opendev.org/745289 | 08:48 |
*** derekh has joined #openstack-ironic | 08:52 | |
*** ociuhandu has joined #openstack-ironic | 08:56 | |
*** ociuhandu_ has quit IRC | 08:57 | |
*** mkrai has joined #openstack-ironic | 09:12 | |
*** tkajinam has quit IRC | 09:17 | |
openstackgerrit | Merged openstack/ironic master: Enforce autospec in test_cinder https://review.opendev.org/745178 | 09:19 |
*** dtantsur|afk is now known as dtantsur | 09:30 | |
*** k_mouza has joined #openstack-ironic | 09:33 | |
*** cp- has joined #openstack-ironic | 09:37 | |
openstackgerrit | Iury Gregory Melo Ferreira proposed openstack/ironic-tempest-plugin master: Update basic_ops https://review.opendev.org/741445 | 09:42 |
*** janders has quit IRC | 09:52 | |
*** ricolin has quit IRC | 09:59 | |
dtantsur | stbenjam, TheJulia, re HTTP 500 in inspector: it's old Flask downstream (1.0 vs 1.1) | 10:01 |
dtantsur | rpittau is looking into updating it | 10:01 |
dtantsur | and good morning ironic | 10:01 |
dtantsur | TGIF! | 10:01 |
iurygregory | morning dtantsur | 10:01 |
openstackgerrit | Dmitry Tantsur proposed openstack/bifrost master: Support default_boot_mode and prepare the CI for different boot modes https://review.opendev.org/744678 | 10:04 |
openstackgerrit | Dmitry Tantsur proposed openstack/bifrost master: Support redfish emulation and run the keystone jobs with redfish https://review.opendev.org/744421 | 10:10 |
openstackgerrit | Dmitry Tantsur proposed openstack/bifrost master: Add CI jobs with Redfish and UEFI https://review.opendev.org/744434 | 10:10 |
openstackgerrit | Dmitry Tantsur proposed openstack/bifrost master: [WIP] Configure redfish virtual media https://review.opendev.org/744459 | 10:11 |
openstackgerrit | Dmitry Tantsur proposed openstack/ironic-inspector master: Listen on :: by default https://review.opendev.org/745308 | 10:18 |
iurygregory | dtantsur, sushy-tools patch reviewed =) | 10:20 |
dtantsur | thanks! | 10:20 |
iurygregory | anything else from your side that need attention? | 10:20 |
* iurygregory looks at whiteboard | 10:20 | |
dtantsur | I have 4 bifrost patches that could use some attention. and the RAID CI job. | 10:22 |
dtantsur | https://review.opendev.org/#/c/743169/ https://review.opendev.org/#/c/741964/ https://review.opendev.org/#/c/744678/ фт | 10:22 |
patchbot | patch 743169 - bifrost - Dynamic roles: consolidate auth parameters in one ... - 5 patch sets | 10:22 |
patchbot | patch 741964 - bifrost - Support HTTP basic auth and switch bifrost-cli to it - 25 patch sets | 10:22 |
patchbot | patch 744678 - bifrost - Support default_boot_mode and prepare the CI for d... - 2 patch sets | 10:22 |
dtantsur | ehh and https://review.opendev.org/#/c/744421/ | 10:22 |
patchbot | patch 744421 - bifrost - Support redfish emulation and run the keystone job... - 11 patch sets | 10:22 |
iurygregory | ack | 10:22 |
dtantsur | would be cool to clear them before I go on PTO | 10:22 |
dtantsur | the RAID work is https://review.opendev.org/741227 and https://review.opendev.org/744009 | 10:23 |
patchbot | patch 741227 - ironic-tempest-plugin - Test software RAID in deploy-time on Victoria and ... - 12 patch sets | 10:23 |
patchbot | patch 744009 - ironic - Enable deploy-time software RAID in standalone jobs - 1 patch set | 10:23 |
dtantsur | many thanks! | 10:23 |
iurygregory | I'm going for lunch and look at them when I come back o/ | 10:24 |
openstackgerrit | Dmitry Tantsur proposed openstack/bifrost master: Validate that the services are running after installation https://review.opendev.org/743569 | 10:25 |
*** k_mouza has quit IRC | 10:43 | |
openstackgerrit | Merged openstack/bifrost master: Change to "collections" where needed https://review.opendev.org/743186 | 10:44 |
*** k_mouza has joined #openstack-ironic | 10:51 | |
openstackgerrit | Verification of a change to openstack/ironic-python-agent failed: Import example hardware managers from ipa-example-hardware-managers https://review.opendev.org/743794 | 10:58 |
openstackgerrit | Verification of a change to openstack/ironic-python-agent failed: Examples: add deploy_steps examples https://review.opendev.org/744705 | 10:58 |
*** janders has joined #openstack-ironic | 11:04 | |
*** Zempashi has quit IRC | 11:06 | |
janders | wrapping up for the day | 11:19 |
janders | sorry I've been on and off, I had to chase up a couple things related to our house move. Couple things missing etc. | 11:20 |
janders | have a great weekend everyone | 11:20 |
janders | dtantsur enjoy your break, have a great time in Switzerland :) | 11:20 |
dtantsur | thank you, have a nice weekend janders :) | 11:20 |
janders | iurygregory if you have some time on Monday I might ask for some help clean-steps priorities, unless I manage to fix the issues with my patch Monday morning :) | 11:22 |
janders | thanks dtantsur :) | 11:22 |
rpittau | bye janders o/ | 11:24 |
janders | good luck with the renovations rpittau | 11:25 |
openstackgerrit | Dmitry Tantsur proposed openstack/ironic-inspector master: Normalize formatting in the API versions reference https://review.opendev.org/745312 | 11:25 |
dtantsur | arne_wiebalck: ^^ | 11:25 |
rpittau | thanks! | 11:27 |
openstackgerrit | Dmitry Tantsur proposed openstack/ironic-python-agent master: Revert "Switch the inspector job to tinyIPA until DIB is fixed" https://review.opendev.org/739180 | 11:29 |
openstackgerrit | Merged openstack/sushy-tools master: Get rid of Resources cache, store resources on the Application https://review.opendev.org/744911 | 11:35 |
*** k_mouza has quit IRC | 11:46 | |
*** ociuhandu has quit IRC | 11:47 | |
*** ociuhandu has joined #openstack-ironic | 11:48 | |
iurygregory | janders, sure! | 11:51 |
janders | thanks iurygregory :) | 11:54 |
*** janders has quit IRC | 11:55 | |
*** vmud213 has joined #openstack-ironic | 11:56 | |
*** k_mouza has joined #openstack-ironic | 11:56 | |
openstackgerrit | Merged openstack/ironic stable/ussuri: Wipe agent token and URL on rescue and unrescue https://review.opendev.org/745083 | 11:59 |
iurygregory | Does it ring a bell `An unknown error has occurred when trying to get the list of nodes from the Ironic inventory. Error: The baremetal service for 158.69.73.199:RegionOne exists but does not have any supported versions.: openstack.exceptions.NotSupported: The baremetal service for 158.69.73.199:RegionOne exists but does not have any supported versions.` ? | 12:00 |
*** k_mouza has quit IRC | 12:00 | |
iurygregory | Seems like the flavor is not matching the capabilities .-. | 12:01 |
*** mkrai has quit IRC | 12:02 | |
dtantsur | iurygregory: nova cannot reach ironic? maybe it crashed? | 12:07 |
iurygregory | dtantsur, ironic seems to be running fine https://zuul.opendev.org/t/openstack/build/96553882a5054a32b8c45eb933a0795b/log/controller/logs/screen-ir-api.txt | 12:09 |
iurygregory | but nova sched couldn't match capabilities https://zuul.opendev.org/t/openstack/build/96553882a5054a32b8c45eb933a0795b/log/controller/logs/screen-n-sch.txt | 12:10 |
iurygregory | so maybe I need to update something on the tempest test to ensure that | 12:10 |
openstackgerrit | Merged openstack/sushy-tools master: Flatten the emulator resources https://review.opendev.org/745092 | 12:14 |
*** Zempashi has joined #openstack-ironic | 12:20 | |
openstackgerrit | Verification of a change to openstack/ironic failed: Fix idrac-wsman RAID apply_configuration https://review.opendev.org/743992 | 12:22 |
*** JamesBenson has joined #openstack-ironic | 12:23 | |
*** vmud213 has quit IRC | 12:27 | |
*** rh-jelabarre has joined #openstack-ironic | 12:30 | |
*** rcernin has quit IRC | 12:32 | |
*** vmud213 has joined #openstack-ironic | 12:36 | |
dtantsur | iurygregory: yeah, I was responding to "baremetal service .. exits but does not have any supported versions" | 13:07 |
openstackgerrit | Merged openstack/bifrost master: Support default_boot_mode and prepare the CI for different boot modes https://review.opendev.org/744678 | 13:11 |
iurygregory | dtantsur, should we set the api version on the tests? | 13:13 |
iurygregory | I'm wondering if it would help | 13:14 |
dtantsur | iurygregory: which exactly API version? | 13:14 |
iurygregory | well seems to me it failed to get the API version | 13:14 |
dtantsur | Note that this "does not have any support versions" is a terrible way of saying "I could not reach the service" | 13:14 |
*** rh-jelabarre has quit IRC | 13:14 | |
dtantsur | which is the only reason an ironic instances would not have "any supported versions" | 13:15 |
* rpittau wondering if we should add some info to that message | 13:15 | |
iurygregory | well its from openstacksdk | 13:15 |
iurygregory | https://opendev.org/openstack/openstacksdk/src/branch/master/openstack/service_description.py#L265 | 13:15 |
dtantsur | I know this bit much better than I'd want to :) | 13:16 |
dtantsur | This is the thing that has been killing bifrost with HTTP basic auth | 13:16 |
dtantsur | (but for a different reason) | 13:16 |
iurygregory | ouch | 13:16 |
dtantsur | In a keystone-based environment (I assume you're talking about such), I can think of two reasons for this error: | 13:17 |
openstackgerrit | Merged openstack/bifrost master: Dynamic roles: consolidate auth parameters in one place https://review.opendev.org/743169 | 13:17 |
openstackgerrit | Verification of a change to openstack/bifrost failed: Support HTTP basic auth and switch bifrost-cli to it https://review.opendev.org/741964 | 13:17 |
dtantsur | 1. keystone returns non-sense, 2. the service is unavailable | 13:17 |
* iurygregory cheks keystone logs | 13:18 | |
iurygregory | dtantsur, ouch merge conflict https://review.opendev.org/#/c/741964/ | 13:18 |
patchbot | patch 741964 - bifrost - Support HTTP basic auth and switch bifrost-cli to it - 25 patch sets | 13:18 |
dtantsur | yep :( looking into it now | 13:18 |
dtantsur | I suspect it'll also conflict with the redfish patch | 13:18 |
iurygregory | I thought it was a chain of patches =( | 13:18 |
dtantsur | I'll rebase on top of https://review.opendev.org/#/c/744421/ | 13:19 |
patchbot | patch 744421 - bifrost - Support redfish emulation and run the keystone job... - 11 patch sets | 13:19 |
dtantsur | or can I really? I guess I need to rebase both of them :( | 13:20 |
openstackgerrit | Dmitry Tantsur proposed openstack/bifrost master: Support redfish emulation and run the keystone jobs with redfish https://review.opendev.org/744421 | 13:21 |
openstackgerrit | Dmitry Tantsur proposed openstack/bifrost master: Support HTTP basic auth and switch bifrost-cli to it https://review.opendev.org/741964 | 13:21 |
dtantsur | iurygregory, rpittau, a re-review would be appreciated ^^ | 13:21 |
iurygregory | keystone can't find baremetal service | 13:21 |
iurygregory | wtf .-. | 13:22 |
dtantsur | the conflict with in playbooks/ci | 13:22 |
rpittau | ok :) | 13:22 |
iurygregory | dtantsur, will do in a few =) | 13:22 |
* rpittau should close at least one messaging platform | 13:22 | |
iurygregory | rpittau, you can also try to make one platform silent =) | 13:23 |
rpittau | iurygregory: my problem right now is that I have multiple conversations open in multiple platforms :/ | 13:24 |
dtantsur | weechat with its `/buffer notify` command is a life-saver | 13:24 |
iurygregory | rpittau, ouch =( good luck | 13:25 |
dtantsur | could someone please request Yet Another (tm) sushy-tools request? | 13:29 |
dtantsur | ehmmm, release request, of course | 13:29 |
rpittau | request a release request! | 13:29 |
dtantsur | REQUEST ALL ZE REQUESTS \o/ | 13:30 |
rpittau | dtantsur: from master ? | 13:30 |
dtantsur | btw if counterpart of python-request in the rust world is called reqwest, and it messes with my brain each time | 13:30 |
dtantsur | rpittau: sushy-tools only has master | 13:30 |
dtantsur | it's _indepedent | 13:30 |
rpittau | right | 13:30 |
TheJulia | good morning | 13:31 |
dtantsur | morning TheJulia, are you ready to conquer the world this Friday? | 13:31 |
TheJulia | reqwest!?!? Did someone in a little headspace name it? | 13:32 |
TheJulia | dtantsur: I _think_ so... just need coffee first | 13:32 |
*** rcernin has joined #openstack-ironic | 13:32 | |
dtantsur | heh, no idea :D but it's the primary HTTP library in the rust world, we all have to live with it | 13:32 |
iurygregory | good morning TheJulia =) | 13:32 |
iurygregory | rpittau, I can do if you are too busy =) | 13:33 |
TheJulia | dtantsur: I'm not saying it is bad or wrong.... but yeah, that does kind of make ones brain go "wut" | 13:33 |
dtantsur | yep, especially when you work on python and rust at the same time | 13:33 |
TheJulia | if we ever see dtantsur setting request = reqwest in the code, we'll know why | 13:33 |
dtantsur | precisely | 13:33 |
* TheJulia looks to try and figure out what the latest information is on the nearby fires before coffee | 13:34 | |
TheJulia | or... as the coffee brews | 13:34 |
dtantsur | fires doesn't sound good :( with +37 during the weekend we may end up with some as well | 13:35 |
TheJulia | okay, we're 9 km from the latest zone that has gone into evacuation warning | 13:37 |
*** rcernin has quit IRC | 13:37 | |
rpittau | thanks iurygregory, I just did it, so I took a break from the chats :D | 13:39 |
*** tzumainn has joined #openstack-ironic | 13:42 | |
openstackgerrit | Julia Kreger proposed openstack/ironic-python-agent stable/rocky: Pin Ironic Tempest Plugin https://review.opendev.org/744773 | 13:45 |
TheJulia | https://review.opendev.org/#/c/745236/ super confuses me since I guess I completely forgot to put the pin in | 13:47 |
patchbot | patch 745236 - ironic (stable/queens) - Pin ironic-tempest-plugin - 1 patch set | 13:47 |
TheJulia | ahh, the override is already there, that is why | 13:49 |
* TheJulia blinks | 13:49 | |
dtantsur | heh | 13:50 |
openstackgerrit | Julia Kreger proposed openstack/ironic stable/queens: Disable the multinode job for queens https://review.opendev.org/745335 | 13:52 |
TheJulia | okay, that is more accurate and a separate change because it is entirely different from the prior backport. | 13:53 |
iurygregory | TheJulia, tks =) | 13:54 |
iurygregory | TheJulia, Ironic queens doesn't need the pin https://github.com/openstack/ironic/blob/stable/queens/zuul.d/legacy-ironic-jobs.yaml#L23 =) | 13:56 |
*** Qianbiao has quit IRC | 13:56 | |
*** k_mouza has joined #openstack-ironic | 13:56 | |
TheJulia | yeah | 13:57 |
TheJulia | I guess since it is identical when I backported it it disappeared | 13:57 |
iurygregory | yeah and the commit message was the same =) | 13:58 |
* iurygregory brb meeting | 13:58 | |
*** ociuhandu_ has joined #openstack-ironic | 14:00 | |
*** ociuhandu has quit IRC | 14:01 | |
*** k_mouza has quit IRC | 14:14 | |
*** k_mouza has joined #openstack-ironic | 14:15 | |
*** Qianbiao has joined #openstack-ironic | 14:17 | |
*** vmud213 has quit IRC | 14:21 | |
*** Qianbiao has quit IRC | 14:28 | |
dking | mgoddard: If you have a moment, I'd like to discuss your ticket https://storyboard.openstack.org/#!/story/1678134 | 14:30 |
mgoddard | dking: sure, fire away | 14:30 |
dking | So, I was about to put in a similar ticket, and I actually already made a patch, to collect sys-name, and was told that it should be placed in switch_info. However, I see that your ticket says that would not be a good place for it. | 14:31 |
openstackgerrit | Julia Kreger proposed openstack/ironic bugfix/15.1: Extend base build timeouts https://review.opendev.org/745347 | 14:31 |
mgoddard | dking: well it depends on the neutron driver | 14:32 |
mgoddard | dking: networking-generic-switch assumes switch_info is a string containing the switch name | 14:33 |
dking | I had wondered if the Neutron mechanism drivers you mentioned could have the information they needed added later during introspection, or perhaps if there might be a good reason to see if there could be a separate field added to local_link_connection | 14:33 |
openstackgerrit | Julia Kreger proposed openstack/ironic stable/ussuri: Extend base build timeouts https://review.opendev.org/745348 | 14:33 |
TheJulia | okay, that should make hte ussuri branch... a littl emore successful | 14:33 |
mgoddard | dking: I think the plugin still has merit - it's useful in some cases, and it's optional | 14:34 |
TheJulia | it also allows the retry logic to trigger too... | 14:34 |
mgoddard | dking: it was probably one of those cases where there is a solution that does not solve every problem, so someone asked if it could solve every problem, then I went back to my day job | 14:34 |
mgoddard | dking: now it's 3 years on and it still hasn't been done | 14:35 |
dking | mgoddard: Oh, wow. I didn't even check the date. It has been a while. | 14:36 |
mgoddard | dking: alternatively, you can just do what we do - install our package | 14:36 |
mgoddard | dking: https://github.com/stackhpc/stackhpc-inspector-plugins/ | 14:36 |
mgoddard | I don't really see myself writing a spec for it | 14:37 |
TheJulia | mgoddard: you know... we could likely take them into ironic-inspector if you guys want | 14:37 |
TheJulia | just floating that idea out there | 14:38 |
dking | mgoddard: My situation at the moment is that we will absolutely need TLV 5 for our setup, and I already patched the code for our use, but don't like stuff not being upstream when it can. But I wasn't aware you already had a plugin for it. | 14:38 |
TheJulia | just glacning at the names, they look like they would be helpful for folks | 14:38 |
mgoddard | TheJulia: I'm happy to propose it. hjensas already helped bring one of those plugins into inspector | 14:38 |
openstackgerrit | Verification of a change to openstack/ironic failed: Auto extend the timeout for RAX hosts https://review.opendev.org/741853 | 14:39 |
mgoddard | TheJulia: would be nice to share them, but the effort/reward of writing a spec, adding functionality and pushing upstream didn't work out :) | 14:39 |
mgoddard | dking: inspector is built to be plugin based, it works fairly well with out of tree plugins like this | 14:40 |
mgoddard | they just lose a bit of visibility | 14:40 |
TheJulia | mgoddard: I'm not too concerned about writing a spec because we've become less focused on them in general when smalling more focused problems | 14:40 |
*** Zempashi has quit IRC | 14:41 | |
mgoddard | dking: is this something you would be interested in picking up? | 14:41 |
mgoddard | i.e. pushing our plugin upstream | 14:42 |
dtantsur | a well written RFE >> a spec | 14:43 |
TheJulia | very much so | 14:43 |
dtantsur | (a caveat: many ignore the "well written" bit) | 14:43 |
dtantsur | but the same caveat applies to specs | 14:43 |
mgoddard | good words, somewhere | 14:43 |
TheJulia | and specs are really designed to make people ask questions with larger scope changes | 14:43 |
TheJulia | "did you think about scaling? what is the scaling impact of this design change?" | 14:44 |
mgoddard | the specs I've written were very helpful for designing features | 14:44 |
mgoddard | helpful for me at least | 14:44 |
TheJulia | absolutely | 14:45 |
dking | mgoddard: I haven't yet looked into anything about writing specs, but I suppose that it's a skill I might need to learn. And I'm still somewhat new to pushing code to the project in general. But if nobody wants to pick it up before I have some time, I might be willing to give it a go. I probably won't have time to do that before September, though. | 14:45 |
dtantsur | stuff like deploy steps - sure. nothing in stackhps-inspector-plugins seems to require a spec to me | 14:45 |
TheJulia | dtantsur: agreed | 14:46 |
mgoddard | dking: as TheJulia mentioned - no spec required. Bars have been lowered since 2017 :) | 14:46 |
TheJulia | ++ | 14:46 |
TheJulia | ohhh, could we do a "loweing the bare metal bar" sticker? | 14:46 |
TheJulia | or maybe a pin?!? | 14:46 |
dking | Neat. | 14:46 |
dking | "Ironic: Lowering the bare metal bar since 2017" ? :) | 14:47 |
mgoddard | lowered bars are good unless you're doing the limbo | 14:47 |
dtantsur | dking: that's an awesome idea! :) | 14:47 |
TheJulia | well, more like 2015 timeframe | 14:47 |
TheJulia | 2014 to early 2015 was lots of code being refactored | 14:48 |
* TheJulia looked at the commit log recently | 14:48 | |
mgoddard | I think I might have just used ironic in 2014 | 14:48 |
mgoddard | if not, early 2015 | 14:48 |
TheJulia | would folks be interested in enamal pins? | 14:49 |
TheJulia | #seriousquestion | 14:49 |
dking | I do think the Ironic project is really good about lowering the bar for folks to manage baremetal. | 14:49 |
dking | Sure, I'd wear a pin. | 14:50 |
TheJulia | I need to see if I can get ahold of some of the guardian of the gate pins... totally forgot to ask last time I was in that area of the country | 14:50 |
larsks | TheJulia: do you have a few minutes of free time today to chat with tzumainn and I about some networking questions? Follow up to what I was asking the other day. | 14:52 |
TheJulia | larsks: I was actually thinking that I hadn't heard from you guys around 8 PM last night | 14:52 |
TheJulia | I will gladly make some time when would be good? | 14:52 |
*** diurnalist has joined #openstack-ironic | 14:53 | |
larsks | My schedule is mostly open today, except for noon and 4-5 (us/eastern). It *looks* like tzumainn is free after noon. How does 1pm look for you? | 14:54 |
TheJulia | larsks: in about 2 hours? | 14:54 |
tzumainn | 2 hours, yep! | 14:55 |
larsks | Yes! | 14:55 |
TheJulia | double checking, but I believe I'm free then | 14:55 |
TheJulia | Yeah, that works | 14:56 |
larsks | Awesome. I'll throw together a calendar invite. | 14:56 |
TheJulia | and this reminds me, I need to call my mechanic to see if he can put new pads/rotors on my car | 14:57 |
openstackgerrit | Merged openstack/ironic stable/train: Update number of VM on ironic-base https://review.opendev.org/744539 | 14:57 |
openstackgerrit | Merged openstack/ironic-inspector master: Remove built-in API documentation in favour of API-REF https://review.opendev.org/743506 | 14:57 |
openstackgerrit | Verification of a change to openstack/ironic-tempest-plugin failed: Ramdisk iso boot tempest scenario test https://review.opendev.org/736378 | 14:57 |
openstackgerrit | Merged openstack/ironic stable/rocky: Remove locks before RPC bus is started https://review.opendev.org/743811 | 14:57 |
TheJulia | I ordered fast stop pads/rotors back in feburary.... | 14:57 |
TheJulia | the rotors finally arrived this week | 14:57 |
*** sdanni has joined #openstack-ironic | 14:58 | |
larsks | Are "fast stop" rotors a specialty item (like for a fancy or offroad car or something)? | 14:58 |
TheJulia | I have a ford focus ST and these are designed for maximum stopping power. My rotors are in "okay" shape, but the end plan is to be able to flat tow and we then want maximum stopping power for the car. | 15:01 |
TheJulia | That and they do look kind of cool, they are slotted but not drillecf | 15:02 |
TheJulia | drilled | 15:02 |
* arne_wiebalck goes on holidays, see you all in 2 weeks o/ | 15:03 | |
rpittau | arne_wiebalck: enjoy! | 15:03 |
arne_wiebalck | rpittau: with the heat in France ... let's see if that is possible :) | 15:04 |
arne_wiebalck | rpittau: but thanks :) | 15:04 |
rpittau | arne_wiebalck: though you were going somewhere :) | 15:04 |
* iurygregory is back | 15:05 | |
arne_wiebalck | rpittau: no need, my sicilian mother in law is here and plans some cooking sessions :-D | 15:05 |
rpittau | arne_wiebalck: oh wow! That sounds great! :) | 15:05 |
arne_wiebalck | rpittau: started with parmigiana de melanzane today :) | 15:05 |
rpittau | arne_wiebalck: post videos with recipes! :D | 15:06 |
arne_wiebalck | rpittau: I follow her very closely, but there something beyond the sheer mechanics which I haven't figured out yet ... | 15:07 |
arne_wiebalck | rpittau: ... some special magic :) | 15:07 |
rpittau | arne_wiebalck: if it's like with my grandma, yeah, it's the magic touch :) | 15:07 |
arne_wiebalck | rpittau: there is! | 15:07 |
arne_wiebalck | bye everyone o/ | 15:08 |
rpittau | o/ | 15:08 |
openstackgerrit | Merged openstack/bifrost master: Support redfish emulation and run the keystone jobs with redfish https://review.opendev.org/744421 | 15:10 |
openstackgerrit | Merged openstack/bifrost master: Support HTTP basic auth and switch bifrost-cli to it https://review.opendev.org/741964 | 15:10 |
*** yolanda has quit IRC | 15:12 | |
openstackgerrit | Merged openstack/ironic-tempest-plugin master: Test software RAID in deploy-time on Victoria and newer https://review.opendev.org/741227 | 15:13 |
*** yolanda has joined #openstack-ironic | 15:13 | |
TheJulia | arne_wiebalck: have fun! | 15:25 |
*** mkrai has joined #openstack-ironic | 15:28 | |
*** rcernin has joined #openstack-ironic | 15:34 | |
openstackgerrit | Merged openstack/ironic master: Enable deploy-time software RAID in standalone jobs https://review.opendev.org/744009 | 15:37 |
TheJulia | \o/ | 15:37 |
iurygregory | zuul gets a cookie | 15:38 |
iurygregory | 4 merges in a row | 15:38 |
*** rcernin has quit IRC | 15:38 | |
TheJulia | A chocolate chip cookie at that | 15:42 |
TheJulia | if it gets to 6, https://wishesndishes.com/dark-chocolate-chip-skillet-cookie/ | 15:43 |
iurygregory | dtantsur, the creation of the flavors look correct to you https://review.opendev.org/#/c/741886/9/devstack/lib/ironic ? | 15:43 |
patchbot | patch 741886 - ironic - Add tempest default_boot_option and altflavor - 9 patch sets | 15:43 |
iurygregory | TheJulia, ++ | 15:43 |
TheJulia | If it gets to 8, I'll make a pizza in it's honor | 15:43 |
iurygregory | 10 | 15:44 |
TheJulia | You think pizza at 10? | 15:44 |
iurygregory | otherwise it will think we are easy | 15:44 |
TheJulia | lol | 15:44 |
TheJulia | true | 15:44 |
TheJulia | I'm just glad the gate is much happier now | 15:44 |
iurygregory | grenade multinode still sad even with the -m PEM | 15:44 |
TheJulia | ugh | 15:44 |
TheJulia | same error? | 15:45 |
iurygregory | yeah =( | 15:45 |
TheJulia | maybe try putting the change on the stable branch? | 15:45 |
TheJulia | and see what happens? | 15:45 |
iurygregory | I did | 15:45 |
TheJulia | oh... | 15:45 |
TheJulia | well config is also supposed to be updated | 15:45 |
iurygregory | ussuri - https://review.opendev.org/745169 master - https://review.opendev.org/739186 | 15:45 |
patchbot | patch 745169 - networking-generic-switch (stable/ussuri) - Generate key on PEM format - 1 patch set | 15:45 |
patchbot | patch 739186 - networking-generic-switch - Generate key on PEM format - 2 patch sets | 15:45 |
iurygregory | both are Depends-On in the grenade patch | 15:45 |
TheJulia | uggggh | 15:46 |
iurygregory | now I'm back fighting why keystone doesn't find Ironic and nova BOOM | 15:47 |
TheJulia | wtf | 15:47 |
iurygregory | this is in the partition wholedisk testing | 15:47 |
iurygregory | we are trying to have a flavor with netboot and another with local | 15:47 |
TheJulia | but still... | 15:47 |
iurygregory | to improve things .-. | 15:47 |
TheJulia | you'd think that failure woudl be across all of our jobs | 15:47 |
iurygregory | yeah | 15:48 |
openstackgerrit | Riccardo Pittau proposed openstack/ironic master: Enforce autospec in xclarity tests https://review.opendev.org/745364 | 15:48 |
iurygregory | I did something wrong for sure, just trying to figure out what I did | 15:48 |
openstackgerrit | Merged openstack/ironic stable/ussuri: Make the final deploy step validation actually fail deploy https://review.opendev.org/743587 | 15:55 |
openstackgerrit | Merged openstack/ironic-python-agent master: Import example hardware managers from ipa-example-hardware-managers https://review.opendev.org/743794 | 15:55 |
openstackgerrit | Merged openstack/ironic-python-agent master: Examples: add deploy_steps examples https://review.opendev.org/744705 | 15:55 |
openstackgerrit | Merged openstack/ironic bugfix/15.1: Wipe agent token and URL on rescue and unrescue https://review.opendev.org/745222 | 15:55 |
* TheJulia blinks | 15:55 | |
TheJulia | 2 from a pizza | 15:55 |
rpittau | not bad zuul, not bad | 15:56 |
iurygregory | way to go | 15:56 |
*** JamesBenson has quit IRC | 15:57 | |
*** lucasagomes has quit IRC | 16:01 | |
*** dougsz has quit IRC | 16:02 | |
dtantsur | on this positive note I'm wishing you a great weekend | 16:04 |
dtantsur | see you in a bit more than a week | 16:04 |
TheJulia | have a wonderful weekend and good PTO dtantsur | 16:04 |
dtantsur | oh, and my IRC bouncer will be done during this time | 16:04 |
dtantsur | s/done/down/ | 16:04 |
TheJulia | okay | 16:04 |
TheJulia | enjoy the disconnected time :) | 16:04 |
*** dtantsur is now known as dtantsur|afk | 16:04 | |
dtantsur|afk | :) | 16:04 |
TheJulia | we'll try not to burn down irc | 16:04 |
*** ntt__ has quit IRC | 16:05 | |
iurygregory | enjoy the PTO | 16:06 |
TheJulia | where is aeva when I have a mysql question.... | 16:07 |
*** mkrai has quit IRC | 16:07 | |
*** dtantsur|afk has quit IRC | 16:11 | |
JayF | enjoy the... and he's gone | 16:13 |
*** alexmcleod has quit IRC | 16:13 | |
TheJulia | c'est la vie | 16:15 |
JayF | TheJulia: I can attempt to answer a mysql question. Probably not successfully but I know a thing or threeve | 16:24 |
openstackgerrit | Merged openstack/ironic-python-agent master: Fix TypeError on agent lookup failure https://review.opendev.org/744077 | 16:32 |
iurygregory | wooot 1 to go | 16:32 |
TheJulia | \o/ | 16:42 |
TheJulia | JayF: https://review.opendev.org/#/c/652811/ <-- I just sent it over to Aeva as well. I don't remember the structure/interaction with text fields in mysql anymore versus how that translates to IO operations | 16:43 |
patchbot | patch 652811 - ironic-specs - Support node history - 2 patch sets | 16:43 |
JayF | Ah; that comment is well outside my knowledge | 16:44 |
JayF | I know some esoteric mysql stuff from working in POS integration between ironic-y gigs, but that does not extend to scaling | 16:45 |
TheJulia | Aha! a failing ironic patch, Ironic won't make it to 10! | 16:48 |
TheJulia | no pizza for ironic! | 16:48 |
openstackgerrit | Verification of a change to openstack/ironic failed: Use TLS for json_rpc when configured https://review.opendev.org/744131 | 16:48 |
iurygregory | TheJulia, =O | 16:48 |
iurygregory | WOW | 16:48 |
TheJulia | lol | 16:48 |
TheJulia | 9 patches in a row has not happened in a while | 16:49 |
iurygregory | see I said 10 was a nice number XD | 16:49 |
*** Lucas_Gray has quit IRC | 16:51 | |
*** sradhakrishna has joined #openstack-ironic | 16:58 | |
*** derekh has quit IRC | 17:01 | |
*** k_mouza has quit IRC | 17:06 | |
*** sradhakrishna has quit IRC | 17:08 | |
*** rcernin has joined #openstack-ironic | 17:35 | |
*** rcernin has quit IRC | 17:39 | |
*** k_mouza has joined #openstack-ironic | 17:44 | |
*** k_mouza has quit IRC | 17:49 | |
openstackgerrit | Merged openstack/ironic stable/ussuri: Fix idrac-wsman RAID apply_configuration https://review.opendev.org/743992 | 18:27 |
openstackgerrit | Merged openstack/ironic stable/queens: Disable the multinode job for queens https://review.opendev.org/745335 | 18:27 |
openstackgerrit | Merged openstack/ironic-tempest-plugin master: Ramdisk iso boot tempest scenario test https://review.opendev.org/736378 | 18:27 |
openstackgerrit | Merged openstack/ironic bugfix/15.1: Extend base build timeouts https://review.opendev.org/745347 | 18:39 |
tzumainn | TheJulia, hi! quick question - is the CLI for starting ironic managed inspection different than 'openstack baremetal introspection start'? | 18:49 |
tzumainn | oh, nevermind - I set the provision state to 'inspect', I guess? | 18:49 |
TheJulia | yeah | 19:05 |
TheJulia | let ironic drive it all | 19:05 |
tzumainn | it's a liberating feeling :) | 19:07 |
*** k_mouza has joined #openstack-ironic | 19:45 | |
*** k_mouza has quit IRC | 19:50 | |
TheJulia | :) | 20:02 |
*** sdanni has quit IRC | 20:53 | |
*** rcernin has joined #openstack-ironic | 21:47 | |
openstackgerrit | Merged openstack/ironic stable/ussuri: Use TLS for json_rpc when configured https://review.opendev.org/744131 | 22:21 |
*** rcernin has quit IRC | 22:26 | |
openstackgerrit | Julia Kreger proposed openstack/ironic-python-agent bugfix/6.2: Fix TypeError on agent lookup failure https://review.opendev.org/745422 | 22:29 |
openstackgerrit | Julia Kreger proposed openstack/ironic-python-agent stable/ussuri: Fix TypeError on agent lookup failure https://review.opendev.org/745423 | 22:29 |
openstackgerrit | Julia Kreger proposed openstack/ironic-python-agent stable/train: Fix TypeError on agent lookup failure https://review.opendev.org/745424 | 22:30 |
TheJulia | well, through stein with a button cclick is not horrible | 22:30 |
*** rcernin has joined #openstack-ironic | 22:31 | |
*** stevebaker has quit IRC | 22:34 | |
openstackgerrit | Julia Kreger proposed openstack/ironic-python-agent stable/stein: Fix TypeError on agent lookup failure https://review.opendev.org/745425 | 22:34 |
openstackgerrit | Julia Kreger proposed openstack/ironic-python-agent stable/rocky: Fix TypeError on agent lookup failure https://review.opendev.org/745426 | 22:35 |
openstackgerrit | Julia Kreger proposed openstack/ironic-python-agent stable/queens: Fix TypeError on agent lookup failure https://review.opendev.org/745427 | 22:36 |
*** diurnalist has quit IRC | 22:47 | |
*** k_mouza has joined #openstack-ironic | 23:10 | |
*** k_mouza has quit IRC | 23:14 | |
*** diurnalist has joined #openstack-ironic | 23:20 | |
*** rcernin has quit IRC | 23:38 | |
*** rcernin has joined #openstack-ironic | 23:59 |
Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!