openstackgerrit | Jacob Anders proposed openstack/bifrost master: Create symlink to ansible collections on install https://review.opendev.org/742192 | 00:25 |
---|---|---|
openstackgerrit | Jacob Anders proposed openstack/bifrost master: Create symlink to ansible collections on install https://review.opendev.org/742192 | 00:33 |
*** yaawang has quit IRC | 01:00 | |
*** yaawang has joined #openstack-ironic | 01:00 | |
*** ricolin has joined #openstack-ironic | 02:02 | |
*** yaawang has quit IRC | 02:04 | |
*** mkrai has joined #openstack-ironic | 02:20 | |
*** yaawang has joined #openstack-ironic | 02:24 | |
*** gyee has quit IRC | 02:33 | |
*** gyee has joined #openstack-ironic | 02:40 | |
*** rh-jelabarre has quit IRC | 02:45 | |
*** yaawang has quit IRC | 03:00 | |
*** yaawang has joined #openstack-ironic | 03:01 | |
*** Lucas_Gray has quit IRC | 03:16 | |
*** yaawang has quit IRC | 03:31 | |
*** yaawang has joined #openstack-ironic | 03:32 | |
*** yaawang has quit IRC | 04:09 | |
*** yaawang has joined #openstack-ironic | 04:09 | |
*** gyee has quit IRC | 04:14 | |
*** mkrai has quit IRC | 04:34 | |
*** mkrai has joined #openstack-ironic | 04:44 | |
*** yaawang has quit IRC | 05:11 | |
*** yaawang has joined #openstack-ironic | 05:12 | |
openstackgerrit | Steve Baker proposed openstack/ironic master: Convert root controller to plain controller https://review.opendev.org/741064 | 05:16 |
openstackgerrit | Steve Baker proposed openstack/ironic master: Convert v1 controller to plain, return JSON https://review.opendev.org/741065 | 05:16 |
openstackgerrit | Steve Baker proposed openstack/ironic master: WIP Remove Driver, DriverList types https://review.opendev.org/741865 | 05:16 |
openstackgerrit | Steve Baker proposed openstack/ironic master: WIP New argument validate decorator https://review.opendev.org/742795 | 05:16 |
*** cdearborn has quit IRC | 06:12 | |
*** dmellado has quit IRC | 06:27 | |
arne_wiebalck | Good morning, ironic! | 06:28 |
janders | g'day arne_wiebalck o/ | 06:36 |
arne_wiebalck | hey janders o/ | 06:45 |
*** yaawang has quit IRC | 06:59 | |
*** yaawang has joined #openstack-ironic | 06:59 | |
iurygregory | good morning arne_wiebalck janders and Ironic! Happy Friday | 07:21 |
rpittau | good morning ironic! Happy Friday! o/ | 07:25 |
openstackgerrit | Merged openstack/ironic-inspector master: Respond so Apache HTTPd doesn't think the request failed https://review.opendev.org/740344 | 07:29 |
arne_wiebalck | Hey iurygregory and rpittau, happy Friday o/ | 07:39 |
rpittau | hey arne_wiebalck :) | 07:39 |
*** yaawang has quit IRC | 07:40 | |
*** yaawang has joined #openstack-ironic | 07:40 | |
*** mkrai has quit IRC | 07:44 | |
openstackgerrit | vinay kumar muddu proposed openstack/ironic master: Decouple the ISO creation logic from redfish https://review.opendev.org/739174 | 07:45 |
rpittau | mmm so next open-infra will also be virtual | 07:46 |
janders | good call | 07:52 |
janders | safety is important but also getting any travel approvals seems near impossible these days even if things are relatively under control by then | 07:53 |
janders | I will disappear for an hour or two but will be back - it would be great to chat about the playbook fixes and the uninstall component rpittau iurygregory dtantsur|afk | 07:54 |
rpittau | janders: yup, I was looking at the changes and they seem good,, thanks! :) | 07:55 |
janders | great, thank you rpittau :) | 07:55 |
iurygregory | morning rpittau o/ | 07:56 |
rpittau | hey iurygregory :) | 07:56 |
* iurygregory will look in a few minutes | 07:56 | |
*** lucasagomes has joined #openstack-ironic | 08:09 | |
*** dmellado has joined #openstack-ironic | 08:20 | |
*** mkrai has joined #openstack-ironic | 08:32 | |
openstackgerrit | Riccardo Pittau proposed openstack/ironic-python-agent master: More refactoring of the image module https://review.opendev.org/742110 | 08:51 |
*** derekh has joined #openstack-ironic | 08:51 | |
openstackgerrit | Riccardo Pittau proposed openstack/ironic-python-agent master: More refactoring of the image module https://review.opendev.org/742110 | 08:51 |
*** dtantsur|afk is now known as dtantsur | 08:53 | |
dtantsur | morning ironic | 08:53 |
rpittau | hey dtantsur :) | 08:57 |
dtantsur | folks, can I get some reviews on https://review.opendev.org/#/c/742643/ please? | 09:01 |
patchbot | patch 742643 - ironic-lib - Return correct response on HTTP basic failure - 1 patch set | 09:01 |
dtantsur | and https://review.opendev.org/#/c/742306/ | 09:01 |
patchbot | patch 742306 - ironic-inspector - Add public api support to basic auth - 2 patch sets | 09:01 |
*** Lucas_Gray has joined #openstack-ironic | 09:01 | |
iurygregory | morning dtantsur | 09:03 |
openstackgerrit | Dmitry Tantsur proposed openstack/ironic master: DNM try sha256 for hash ring https://review.opendev.org/742828 | 09:05 |
*** ociuhandu has joined #openstack-ironic | 09:06 | |
*** ociuhandu has quit IRC | 09:09 | |
iurygregory | dtantsur, https://github.com/openstack/ironic-lib/tree/master/releasenotes/notes O.o | 09:13 |
iurygregory | that's why I asked =) | 09:13 |
dtantsur | yeah, but https://docs.openstack.org/releasenotes/ironic-lib | 09:14 |
iurygregory | gotcha | 09:14 |
dtantsur | which is sort of fair, because ironic-lib is only for internal usage | 09:14 |
iurygregory | yeah makes sense | 09:14 |
dtantsur | but I don't -1 people for having release notes with their patches, hence the ones you've found | 09:14 |
iurygregory | understood | 09:14 |
*** Lucas_Gray has quit IRC | 09:18 | |
*** Lucas_Gray has joined #openstack-ironic | 09:25 | |
*** mkrai has quit IRC | 09:27 | |
*** mkrai_ has joined #openstack-ironic | 09:27 | |
janders | back | 09:28 |
janders | good morning dtantsur rpittau iurygregory | 09:29 |
*** yaawang has quit IRC | 09:30 | |
rpittau | hey janders :) | 09:30 |
*** yaawang has joined #openstack-ironic | 09:30 | |
janders | thanks for reviewing https://review.opendev.org/#/c/742192/ guys :) | 09:30 |
patchbot | patch 742192 - bifrost - Create symlink to ansible collections on install - 5 patch sets | 09:30 |
iurygregory | yw =) | 09:31 |
janders | I'm thinking what to work on next. I ran into my bifrost issues while setting myself up to work on https://storyboard.openstack.org/#!/story/1618014 | 09:32 |
janders | while fixing up these issues the idea of uninstall playbook came up as well as refactoring the code to use collections instead of os_* modules | 09:32 |
janders | what's better done first? | 09:33 |
janders | uninstall, collections refactor or the steps RFE? | 09:33 |
dtantsur | uninstall is interesting, but a quite low priority | 09:34 |
dtantsur | the steps RFE ties into the deploy steps work and will give you hands-on experience with hacking on ironic | 09:35 |
dtantsur | I'd say go for it | 09:35 |
janders | ok! thank you dtantsur | 09:35 |
janders | now that I have bifrost deploying smoothly (and cleaning working) I'm set up to work on that | 09:35 |
janders | dtantsur would you have time to have a quick look at https://review.opendev.org/#/c/741611/ ? | 09:36 |
patchbot | patch 741611 - bifrost - Expand advanced install instructions - 9 patch sets | 09:36 |
dtantsur | yep | 09:36 |
janders | thanks! | 09:36 |
janders | it's that doco tweak we've done once but it ended up in merge conflict | 09:37 |
janders | I believe I fixed that up but I figure it may end up in the same state again as doco is quite dynamic | 09:37 |
janders | it's probably less concern for the other patches | 09:37 |
dtantsur | approved | 09:38 |
openstackgerrit | Dmitry Tantsur proposed openstack/bifrost master: [WIP] Support HTTP basic auth and switch bifrost-cli to it https://review.opendev.org/741964 | 09:48 |
openstackgerrit | Merged openstack/bifrost master: Create symlink to ansible collections on install https://review.opendev.org/742192 | 09:50 |
openstackgerrit | Merged openstack/bifrost master: Expand advanced install instructions https://review.opendev.org/741611 | 09:50 |
*** ricolin has quit IRC | 09:51 | |
openstackgerrit | Dmitry Tantsur proposed openstack/ironic master: Deprecate http_basic_username and http_basic_password in [json_rpc] https://review.opendev.org/742474 | 09:51 |
dtantsur | I think I know why this ^^ did not work, our devstack plugin was setting [json_rpc]auth_type=password unconditionally :) | 09:53 |
iurygregory | damm | 09:54 |
openstackgerrit | Dmitry Tantsur proposed openstack/ironic-tempest-plugin master: POC deploy-time RAID https://review.opendev.org/741227 | 09:56 |
openstackgerrit | Dmitry Tantsur proposed openstack/ironic stable/ussuri: Fixes to skip validation of in-band deploy steps before agent boot https://review.opendev.org/742840 | 09:58 |
*** vmud213 has joined #openstack-ironic | 09:59 | |
*** mkrai_ has quit IRC | 10:01 | |
janders | while testing the bifrost fixes I realised I don't fully understand how baremetal nodes get deployed with standalone ironic. I suppose that's what "baremetal node deploy" is for but that seems to only take config drive. Where do we set the image to be deployed? | 10:07 |
janders | bifrost playbooks abstract this away nicely but trying to educate myself in this regard. Hints welcome! :) | 10:07 |
*** k_mouza has joined #openstack-ironic | 10:08 | |
iurygregory | janders, https://github.com/openstack/bifrost/blob/dc3f7c3f39779558d10d03ba38103261d5d1277a/playbooks/inventory/group_vars/localhost | 10:08 |
iurygregory | If you want to change things you probably need to update your inventory | 10:09 |
janders | thank you iurygregory. How about if I wanted to do it via CLI? | 10:09 |
iurygregory | it shouldn't be a problem | 10:10 |
janders | there's decent ironic-standalone CLI reference online but it's light on examples | 10:10 |
janders | and CLI comes in handy when things don't work | 10:10 |
iurygregory | the workflow to deploy shouldn't change that much from (standalone vs nonstandalone) | 10:11 |
iurygregory | you need to set the client for noauth | 10:11 |
iurygregory | and use openstack baremetal ... | 10:11 |
janders | with non-standalone I'd use the nova abstraction to pick the image | 10:12 |
janders | that's the bit I'm not quite sure about for standalone | 10:12 |
janders | I see it appear in instance_info field but haven't yet figured out how it gets there | 10:12 |
dtantsur | usually, manually :) | 10:13 |
janders | is it somehow injected into the config drive? | 10:13 |
janders | (I mean image URL) | 10:13 |
dtantsur | nope, just set by a user https://docs.openstack.org/ironic/latest/install/standalone.html | 10:13 |
dtantsur | scroll down to #5 (we should really add subsections there) | 10:13 |
iurygregory | yeah | 10:14 |
*** Lucas_Gray has quit IRC | 10:14 | |
iurygregory | Configuration / Deployment | 10:15 |
janders | okay, got it! | 10:15 |
janders | thank you dtantsur iurygregory | 10:15 |
iurygregory | janders, yw =) | 10:15 |
janders | this is called "direct deploy" right? | 10:15 |
janders | or is that yet another thing? | 10:15 |
*** Lucas_Gray has joined #openstack-ironic | 10:17 | |
openstackgerrit | Dmitry Tantsur proposed openstack/ironic master: Add subsections to the standalone documentation https://review.opendev.org/742842 | 10:18 |
dtantsur | janders: no, it's different. see https://docs.openstack.org/ironic/latest/admin/interfaces/deploy.html | 10:18 |
janders | ok, got it! :) mixed up two different areas | 10:19 |
janders | ok I think I now have enough to figure it out end-to-end on Monday | 10:19 |
*** k_mouza has quit IRC | 10:19 | |
janders | thanks guys! | 10:20 |
dtantsur | janders: there is also this idea https://storyboard.openstack.org/#!/story/2006910 | 10:20 |
dtantsur | which will make deploying much more obvious | 10:20 |
dtantsur | but I don't have time for it :( | 10:20 |
*** mkrai_ has joined #openstack-ironic | 10:22 | |
dtantsur | may I get some reviews on https://review.opendev.org/#/c/741894/ and https://review.opendev.org/742203 please? | 10:25 |
patchbot | patch 741894 - bifrost - Support configuring credential-less deploy - 5 patch sets | 10:25 |
patchbot | patch 742203 - bifrost - Use random passwords by default - 11 patch sets | 10:25 |
*** k_mouza has joined #openstack-ironic | 10:29 | |
janders | dtantsur I very much like the RFE but I understand the time concerns. Big job. | 10:30 |
dtantsur | I've even started it. If somebody approves https://review.opendev.org/#/c/703778/ maybe it'll motivate me to dedicate some time to it ;) | 10:31 |
patchbot | patch 703778 - ironic - Add RPC objects for deployment API - 11 patch sets | 10:31 |
janders | I'm wrapping up, thank you for all your help this week and have a great weekend all! | 10:33 |
dtantsur | have a nice weekend janders | 10:33 |
openstackgerrit | Dmitry Tantsur proposed openstack/bifrost master: [WIP] Support HTTP basic auth and switch bifrost-cli to it https://review.opendev.org/741964 | 10:43 |
*** mkrai_ has quit IRC | 10:43 | |
*** mkrai__ has joined #openstack-ironic | 10:43 | |
*** k_mouza has quit IRC | 10:45 | |
openstackgerrit | Verification of a change to openstack/sushy failed: Update system schema to 1.10 https://review.opendev.org/742236 | 10:48 |
*** yaawang has quit IRC | 10:50 | |
*** yaawang has joined #openstack-ironic | 10:51 | |
*** k_mouza has joined #openstack-ironic | 10:51 | |
*** k_mouza has quit IRC | 10:53 | |
*** k_mouza has joined #openstack-ironic | 10:53 | |
*** ociuhandu has joined #openstack-ironic | 10:54 | |
*** ociuhandu has quit IRC | 10:59 | |
*** janders has quit IRC | 10:59 | |
openstackgerrit | Verification of a change to openstack/ironic-inspector failed: Add public api support to basic auth https://review.opendev.org/742306 | 10:59 |
*** k_mouza has quit IRC | 11:01 | |
openstackgerrit | Dmitry Tantsur proposed openstack/bifrost master: Make logging more uniform https://review.opendev.org/742854 | 11:06 |
*** k_mouza has joined #openstack-ironic | 11:12 | |
*** aedc has joined #openstack-ironic | 11:16 | |
*** k_mouza has quit IRC | 11:17 | |
dtantsur | TheJulia: JFYI: Sprint 2 releases are planned for the week of August 10th. I'll very likely be out that week. | 11:20 |
*** uzumaki has joined #openstack-ironic | 11:27 | |
vmud213 | Hello Ironic! | 11:27 |
uzumaki | hello ironic! \o/ | 11:27 |
vmud213 | I have a query | 11:28 |
vmud213 | When we use instance images for the deployment and the images are hosted in a HTTPS hosted webserver, what whould be the behavior | 11:29 |
vmud213 | iface.validate() fails for bot "boot" and "deploy" interfaces. | 11:29 |
vmud213 | The certification validation fails as there is no support yet in HttpImageService for HTTPS connection | 11:31 |
vmud213 | dtantsur, TheJulia could u help me to understand the behavior in such cases.Planning to raise a patch accordingly | 11:33 |
*** uzumaki has quit IRC | 11:39 | |
dtantsur | You mean, you have customer certificates? | 11:39 |
dtantsur | * custom | 11:39 |
*** uzumaki has joined #openstack-ironic | 11:41 | |
*** mkrai__ has quit IRC | 11:42 | |
vmud213 | Hi | 11:47 |
vmud213 | yes | 11:47 |
vmud213 | So not just in this case | 11:47 |
vmud213 | even in the case of redfish when we create deploy ISO on the fly, we also may need to support custom certificates. | 11:48 |
vmud213 | I see this support in ironic-python-agent | 11:49 |
vmud213 | so planning to add one | 11:49 |
vmud213 | u have any suggestions | 11:50 |
openstackgerrit | Dmitry Tantsur proposed openstack/bifrost master: Make logging more uniform https://review.opendev.org/742854 | 11:54 |
vmud213 | if we want to boot up a baremetal node with a deploy ISO hosted on a HTTPS webserver, we may require to upload the certificate on to the nodes for them to boot up from the URL | 11:55 |
dtantsur | vmud213: makes sense to me, but I'm not sure what the current status is. I guess you could start with a detailed RFE. | 11:56 |
vmud213 | ok.Sure. | 11:56 |
vmud213 | Thanks | 11:58 |
*** uzumaki has quit IRC | 12:01 | |
*** k_mouza has joined #openstack-ironic | 12:07 | |
*** rh-jelabarre has joined #openstack-ironic | 12:07 | |
dtantsur | iurygregory: do I get it right that ANSIBLE_COLLECTIONS_PATH=/opt/stack/bifrost/collections/ is NOT enough to find the collections? do you know what else should be done? | 12:10 |
dtantsur | ah, PATHS, not PATH | 12:11 |
*** k_mouza has quit IRC | 12:12 | |
*** k_mouza has joined #openstack-ironic | 12:18 | |
*** k_mouza has quit IRC | 12:23 | |
weshay|ruck | TheJulia, dtantsur iurygregory thanks for the reviews... updated https://review.opendev.org/#/c/742707/ | 12:24 |
patchbot | patch 742707 - tripleo-common - tripleo ipa image should use the new ipa-ramdisk - 2 patch sets | 12:24 |
iurygregory | dtantsur, sorry was in a meeting | 12:24 |
*** derekh has quit IRC | 12:24 | |
iurygregory | weshay|ruck, /me looks | 12:25 |
iurygregory | lgtm | 12:25 |
iurygregory | dtantsur, well you have figure out I think =) | 12:25 |
dtantsur | yeah.. now I just need to understand why ironic-enroll-dynamic works locally but fails in the CI >_< | 12:27 |
iurygregory | patch link? | 12:28 |
*** k_mouza has joined #openstack-ironic | 12:28 | |
openstackgerrit | Dmitry Tantsur proposed openstack/bifrost master: bifrost_inventory: use stderr for logging https://review.opendev.org/742869 | 12:28 |
dtantsur | this gave me grey hair ^^ | 12:28 |
dtantsur | iurygregory: https://review.opendev.org/741964 | 12:29 |
patchbot | patch 741964 - bifrost - [WIP] Support HTTP basic auth and switch bifrost-c... - 15 patch sets | 12:29 |
dtantsur | I'm not running it fully, that's maybe a problem | 12:29 |
dtantsur | when I just run playbooks/enroll-dynamic.yaml it works | 12:29 |
iurygregory | hummm | 12:30 |
*** k_mouza has quit IRC | 12:31 | |
*** k_mouza has joined #openstack-ironic | 12:31 | |
*** Lucas_Gray has quit IRC | 12:32 | |
*** Lucas_Gray has joined #openstack-ironic | 12:32 | |
openstackgerrit | Dmitry Tantsur proposed openstack/bifrost master: Make logging more uniform https://review.opendev.org/742854 | 12:34 |
openstackgerrit | Riccardo Pittau proposed openstack/ironic-python-agent master: More refactoring of the image module https://review.opendev.org/742110 | 12:34 |
dtantsur | aha, my local invocation had "auth_type": "http_basic" in the module arguments. it's not present in the CI. | 12:39 |
dtantsur | oh, wait, it is | 12:39 |
openstackgerrit | Riccardo Pittau proposed openstack/bifrost master: Consolidate env-setup https://review.opendev.org/742201 | 12:40 |
rpittau | if anyone has a minute, this should be quick https://review.opendev.org/741408 | 12:40 |
patchbot | patch 741408 - ironic - Enforce autospec in test_deployments - 3 patch sets | 12:40 |
*** ociuhandu has joined #openstack-ironic | 12:43 | |
*** ociuhandu has quit IRC | 12:47 | |
openstackgerrit | Dmitry Tantsur proposed openstack/bifrost master: [WIP] Support HTTP basic auth and switch bifrost-cli to it https://review.opendev.org/741964 | 12:50 |
*** derekh has joined #openstack-ironic | 12:51 | |
*** jj123 has joined #openstack-ironic | 12:54 | |
openstackgerrit | Verification of a change to openstack/ironic failed: Force RAX hosts to run tinyipa https://review.opendev.org/742502 | 12:55 |
*** rloo has joined #openstack-ironic | 12:55 | |
*** Lucas_Gray has quit IRC | 13:06 | |
*** bfournie has joined #openstack-ironic | 13:07 | |
TheJulia | good morning | 13:07 |
rpittau | good morning TheJulia :) | 13:07 |
TheJulia | some days I feel like CI just doesn't like me | 13:11 |
openstackgerrit | Dmitry Tantsur proposed openstack/bifrost master: Correct handling of inspect_nodes in test-bifrost.yaml https://review.opendev.org/742877 | 13:11 |
dtantsur | if anybody wants a good facepalm ^^ | 13:11 |
dtantsur | morning TheJulia. it likes no one. | 13:11 |
*** Lucas_Gray has joined #openstack-ironic | 13:12 | |
iurygregory | good morning TheJulia =) | 13:12 |
iurygregory | dtantsur, LOL | 13:13 |
openstackgerrit | Dmitry Tantsur proposed openstack/bifrost master: Enable debug logging in virtualbmc https://review.opendev.org/742878 | 13:13 |
TheJulia | dtantsur: well, as long as the parser happily groks it these days | 13:13 |
dtantsur | heh | 13:13 |
iurygregory | facepalm is not enough =( | 13:13 |
openstackgerrit | Dmitry Tantsur proposed openstack/bifrost master: Make logging more uniform https://review.opendev.org/742854 | 13:15 |
*** aedc has quit IRC | 13:15 | |
dtantsur | "The baremetal service for : exists but does not have any supported versions." | 13:16 |
dtantsur | oh ffs | 13:16 |
* dtantsur has had enough today | 13:16 | |
*** zigo has quit IRC | 13:19 | |
iurygregory | magic versions -.-' | 13:24 |
TheJulia | wut?!? | 13:27 |
*** ociuhandu has joined #openstack-ironic | 13:30 | |
*** zigo has joined #openstack-ironic | 13:31 | |
openstackgerrit | Julia Kreger proposed openstack/ironic stable/ussuri: Change non-tinyipa jobs to use multiple cores https://review.opendev.org/741577 | 13:33 |
openstackgerrit | Julia Kreger proposed openstack/ironic master: Guard conductor from consuming all of the ram https://review.opendev.org/726483 | 13:33 |
openstackgerrit | Julia Kreger proposed openstack/ironic master: ISO ramdisk virtual media test enablement https://review.opendev.org/736383 | 13:37 |
TheJulia | iurygregory: this is the job your looking for https://review.opendev.org/#/c/736383/ | 13:38 |
patchbot | patch 736383 - ironic - ISO ramdisk virtual media test enablement - 12 patch sets | 13:38 |
iurygregory | TheJulia, aha! | 13:38 |
iurygregory | tks! | 13:38 |
TheJulia | Because of the iso contents we can't see it past boot (which yes, is a problem, but we should be able to fix that eventually. What we do get is being able to ping the machine | 13:39 |
iurygregory | ack, I was wondering if we introduced in other patch just to see the result of tempest =) | 13:39 |
TheJulia | yeah, slightly counter-intuitive :\ | 13:43 |
openstackgerrit | Julia Kreger proposed openstack/ironic master: Add some tuning documentation https://review.opendev.org/739783 | 13:45 |
*** jj123 has quit IRC | 13:47 | |
openstackgerrit | Dmitry Tantsur proposed openstack/bifrost master: Make logging more uniform https://review.opendev.org/742854 | 13:53 |
*** tzumainn has joined #openstack-ironic | 13:56 | |
*** k_mouza has quit IRC | 14:00 | |
openstackgerrit | Dmitry Tantsur proposed openstack/ironic-tempest-plugin master: POC deploy-time RAID https://review.opendev.org/741227 | 14:04 |
*** ociuhandu has quit IRC | 14:07 | |
*** ociuhandu has joined #openstack-ironic | 14:07 | |
openstackgerrit | Dmitry Tantsur proposed openstack/bifrost master: Store inspector ramdisk logs by default https://review.opendev.org/742888 | 14:09 |
openstackgerrit | Dmitry Tantsur proposed openstack/bifrost master: Make ironic logging more in line with other services. https://review.opendev.org/742854 | 14:11 |
*** k_mouza has joined #openstack-ironic | 14:11 | |
TheJulia | wheee major upgrades to debian to get python3.8 hopefully running | 14:14 |
dtantsur | good luck :) | 14:14 |
TheJulia | thanks | 14:14 |
TheJulia | the follow-up ipxe_template change has me really worried that it is occassionally returning the ipxe template instead of the pxe template | 14:15 |
TheJulia | makes me think driver loading is matching pxe in ipxe | 14:15 |
TheJulia | and loading ipxe instead | 14:15 |
TheJulia | but it _shouldn't_ be | 14:15 |
dtantsur | hmmm | 14:16 |
*** ricolin has joined #openstack-ironic | 14:19 | |
TheJulia | interesting, it nuked my running network configuration | 14:23 |
*** Goneri has joined #openstack-ironic | 14:26 | |
dtantsur | iurygregory: it looks like supporting basic auth in bifrost may be blocked by the collections transition | 14:30 |
dtantsur | or, hmm, I'll try without it. maybe the old modules are good enough. | 14:30 |
iurygregory | dtantsur, I'm working today on the patch | 14:32 |
iurygregory | monday I will have it ready =) | 14:32 |
dtantsur | neat! I'll try to do without it, but it may end up needed.. | 14:34 |
openstackgerrit | Merged openstack/bifrost master: Correct handling of inspect_nodes in test-bifrost.yaml https://review.opendev.org/742877 | 14:36 |
openstackgerrit | Dmitry Tantsur proposed openstack/bifrost master: Find authentication from the 'bifrost' cloud instead of calling os_cloud_config https://review.opendev.org/742898 | 14:39 |
dtantsur | it may be this ^^^ | 14:39 |
dtantsur | TheJulia: your opinion is highly welcome | 14:39 |
TheJulia | ohhhh | 14:39 |
openstackgerrit | Dmitry Tantsur proposed openstack/bifrost master: [WIP] Support HTTP basic auth and switch bifrost-cli to it https://review.opendev.org/741964 | 14:39 |
dtantsur | yeah :) | 14:39 |
dtantsur | I've spent 3 days scratching my head | 14:40 |
dtantsur | we'll see if it fixes the HTTP auth | 14:40 |
*** k_mouza has quit IRC | 14:43 | |
TheJulia | ohhh, I might be rebuilding my desktop this weekend | 14:43 |
TheJulia | libc issues | 14:43 |
dtantsur | libc issues sounds like a lot of weird fun | 14:45 |
TheJulia | no... not at all | 14:47 |
TheJulia | ohh, an upgrade is 2GB of packages | 14:48 |
* dtantsur hands TheJulia a nice Fedora :D | 14:48 | |
* TheJulia fires it off and suspects when she gets back from the dr's office that it may be done | 14:48 | |
TheJulia | heh | 14:48 |
*** k_mouza has joined #openstack-ironic | 14:53 | |
*** k_mouza has quit IRC | 15:03 | |
*** derekh has quit IRC | 15:03 | |
openstackgerrit | Merged openstack/ironic master: Allow node lessee to see node's ports https://review.opendev.org/730366 | 15:06 |
openstackgerrit | Merged openstack/ironic-lib master: Return correct response on HTTP basic failure https://review.opendev.org/742643 | 15:06 |
openstackgerrit | Merged openstack/ironic master: Adds raid validation for in-band AgentRAID deploy step https://review.opendev.org/742217 | 15:06 |
openstackgerrit | Merged openstack/ironic master: Enforce autospec in test_deployments https://review.opendev.org/741408 | 15:06 |
openstackgerrit | Merged openstack/ironic master: Enforce autospec in test_notification_utils https://review.opendev.org/741419 | 15:06 |
openstackgerrit | Verification of a change to openstack/ironic failed: Enforce autospec in some api tests https://review.opendev.org/741468 | 15:06 |
iurygregory | 5/6 not bad Zuul | 15:07 |
openstackgerrit | Zygimantas Matonis proposed openstack/ironic-inspector master: More flexible definitions of conditions in rules https://review.opendev.org/731794 | 15:11 |
arne_wiebalck | iurygregory: be nice to Zuul, then Zuul is nice to you | 15:12 |
iurygregory | arne_wiebalck, I gave a cookie | 15:12 |
iurygregory | it was 5/6 | 15:12 |
arne_wiebalck | iurygregory: perfect! | 15:12 |
arne_wiebalck | Zuul likes cookies | 15:12 |
*** ociuhandu has quit IRC | 15:14 | |
openstackgerrit | Merged openstack/ironic-inspector master: Add public api support to basic auth https://review.opendev.org/742306 | 15:18 |
*** k_mouza has joined #openstack-ironic | 15:19 | |
openstackgerrit | Dmitry Tantsur proposed openstack/bifrost master: Find authentication from the 'bifrost' cloud instead of calling os_cloud_config https://review.opendev.org/742898 | 15:21 |
openstackgerrit | Dmitry Tantsur proposed openstack/bifrost master: [WIP] Support HTTP basic auth and switch bifrost-cli to it https://review.opendev.org/741964 | 15:21 |
dtantsur | we need an ironic-lib release now, anyone up for it? iurygregory? | 15:21 |
-openstackstatus- NOTICE: We are renaming projects in Gerrit and review.opendev.org will experience a short outage. Thank you for your patience. | 15:22 | |
rpittau | ehm | 15:22 |
openstackgerrit | Merged openstack/sushy master: Update system schema to 1.10 https://review.opendev.org/742236 | 15:23 |
*** k_mouza has quit IRC | 15:23 | |
iurygregory | dtantsur, sure | 15:26 |
dtantsur | thanks | 15:27 |
iurygregory | yay will do my first release \o/ | 15:27 |
*** k_mouza has joined #openstack-ironic | 15:32 | |
*** bfournie has quit IRC | 15:33 | |
*** k_mouza has quit IRC | 15:36 | |
*** k_mouza has joined #openstack-ironic | 15:37 | |
openstackgerrit | Richard G. Pioso proposed openstack/ironic-specs master: [WIP] Operationally efficient & fast-track provisioning https://review.opendev.org/740721 | 15:41 |
*** gyee has joined #openstack-ironic | 15:42 | |
*** k_mouza has quit IRC | 15:42 | |
iurygregory | dtantsur, 4.3.1 should be fine? | 15:43 |
dtantsur | I think so | 15:43 |
*** rloo has quit IRC | 15:43 | |
dtantsur | mmm, I'm not sure bifrost can be fixed without fixing the collections | 15:43 |
*** rloo has joined #openstack-ironic | 15:44 | |
iurygregory | dtantsur, fyi https://review.opendev.org/742931 | 15:45 |
patchbot | patch 742931 - releases - Release 4.3.1 of ironic-lib - 1 patch set | 15:45 |
iurygregory | monday I will have the collections patch up o/ | 15:45 |
*** bnemec is now known as beekneemech | 15:46 | |
*** k_mouza has joined #openstack-ironic | 15:47 | |
dtantsur | iurygregory: nice, but there is an issue there as well: https://review.opendev.org/742932 | 15:48 |
patchbot | patch 742932 - ansible-collections-openstack - Do not require ironic_url if cloud is provided - 1 patch set | 15:48 |
dtantsur | I guess I'll have to work around it.. | 15:48 |
iurygregory | =O | 15:48 |
*** rloo has quit IRC | 15:49 | |
*** rloo has joined #openstack-ironic | 15:50 | |
*** k_mouza has quit IRC | 15:51 | |
*** dtantsur is now known as dtantsur|afk | 15:53 | |
dtantsur|afk | have a great weekend! | 15:53 |
*** k_mouza has joined #openstack-ironic | 15:54 | |
*** k_mouza has quit IRC | 15:59 | |
*** lucasagomes has quit IRC | 16:01 | |
*** k_mouza has joined #openstack-ironic | 16:04 | |
openstackgerrit | vinay kumar muddu proposed openstack/ironic master: Allow HttpImageService to accept custom certificate https://review.opendev.org/742936 | 16:08 |
*** k_mouza has quit IRC | 16:08 | |
*** k_mouza has joined #openstack-ironic | 16:15 | |
*** k_mouza has quit IRC | 16:17 | |
*** k_mouza has joined #openstack-ironic | 16:18 | |
*** SpamapS has quit IRC | 16:27 | |
*** SpamapS has joined #openstack-ironic | 16:29 | |
openstackgerrit | Merged openstack/ironic master: Force RAX hosts to run tinyipa https://review.opendev.org/742502 | 16:30 |
*** ociuhandu has joined #openstack-ironic | 16:34 | |
*** ociuhandu has quit IRC | 16:39 | |
*** rloo has quit IRC | 16:39 | |
*** rloo has joined #openstack-ironic | 16:39 | |
TheJulia | yay coffee finally | 16:39 |
*** k_mouza has quit IRC | 16:59 | |
openstackgerrit | Verification of a change to openstack/ironic failed: Extend base build timeouts https://review.opendev.org/741818 | 17:13 |
*** k_mouza has joined #openstack-ironic | 17:17 | |
*** k_mouza has quit IRC | 17:21 | |
*** k_mouza has joined #openstack-ironic | 17:22 | |
*** k_mouza has quit IRC | 17:27 | |
openstackgerrit | Julia Kreger proposed openstack/ironic stable/ussuri: Force RAX hosts to run tinyipa https://review.opendev.org/742950 | 17:31 |
* iurygregory is wondering if we want to lowe the VM SPECS for rax hosts | 17:33 | |
iurygregory | s/lowe/decrease* | 17:34 |
*** ricolin has quit IRC | 17:43 | |
TheJulia | I'm not sure they will really matter much | 17:43 |
TheJulia | is anyone able to reproduce the python38 unit test failure on https://review.opendev.org/#/c/739841/ ? | 17:45 |
patchbot | patch 739841 - ironic - Add ``ipxe_template`` override option - 5 patch sets | 17:45 |
TheJulia | I've not been able to do so... yet | 17:45 |
iurygregory | let me test | 17:45 |
TheJulia | I've got a loop running now to try and see if it can break on my local machine. So far every test run has passed | 17:46 |
* TheJulia tries the old "fire up a movie stream" addition | 17:46 | |
iurygregory | TheJulia, green =) | 17:51 |
iurygregory | zuul is drunk since is friday night | 17:51 |
TheJulia | yeah, no failure yet | 17:51 |
TheJulia | is it?!? | 17:51 |
iurygregory | py38: commands succeeded congratulations :) | 17:52 |
iurygregory | it's friday night an in Japan is already Sat so Zuul is probably at some party | 17:52 |
TheJulia | boom, failed | 17:55 |
TheJulia | Well, that makes me feel a little better | 17:55 |
iurygregory | hehehe | 17:56 |
*** k_mouza has joined #openstack-ironic | 18:00 | |
*** k_mouza has quit IRC | 18:05 | |
TheJulia | looks like it is with-in the class/test runner | 18:08 |
*** gmann is now known as gmann_lunch | 18:13 | |
TheJulia | hmm the runner it seems | 18:18 |
*** Lucas_Gray has quit IRC | 18:20 | |
*** ociuhandu has joined #openstack-ironic | 18:22 | |
iurygregory | so it's random the failure? O.o | 18:25 |
*** ociuhandu has quit IRC | 18:27 | |
openstackgerrit | Merged openstack/bifrost master: Make ansible ask for sudo password https://review.opendev.org/742175 | 18:36 |
openstackgerrit | Merged openstack/bifrost master: Auto-set ansible_python_interpreter when in venv https://review.opendev.org/742335 | 18:36 |
openstackgerrit | Merged openstack/bifrost master: Enable debug logging in virtualbmc https://review.opendev.org/742878 | 18:36 |
TheJulia | iurygregory: yeah, it is | 18:37 |
iurygregory | sad =( | 18:38 |
TheJulia | it can happen with-in the test runners | 18:38 |
iurygregory | gotcha =( | 18:38 |
TheJulia | this is often why dmitry or I will grump and split apart one test class into two | 18:39 |
iurygregory | =( | 18:44 |
iurygregory | it only happens when we try to check if the lists are identical? | 18:45 |
openstackgerrit | Merged openstack/ironic master: Add subsections to the standalone documentation https://review.opendev.org/742842 | 18:52 |
*** rloo has quit IRC | 18:53 | |
*** rloo has joined #openstack-ironic | 18:54 | |
TheJulia | iurygregory: more like in-memory collision in the test runner and the class helps distribute things between runners. | 18:57 |
iurygregory | gotcha | 18:57 |
TheJulia | because each class gets distributed to different runners | 18:57 |
iurygregory | this is the first time I see this problem | 18:58 |
iurygregory | and we do have some classes with a TON of tests | 18:58 |
TheJulia | yup | 19:03 |
TheJulia | What is super odd is that we're even seeing the failure on such a small scale but it is likely just the alignment of the starts. | 19:04 |
TheJulia | And often this is weirdness around module loading | 19:04 |
TheJulia | unfortunately, what is failing is about as close as our tests get to that area | 19:04 |
TheJulia | so *shrugs* | 19:04 |
TheJulia | I changed things to explicitly set the interface for the test since... we have multiple so we're not trying to guess | 19:05 |
TheJulia | so far so good, on... run #27 or 28 now | 19:05 |
iurygregory | yeah =( | 19:06 |
TheJulia | I kind of view it like the mysql stuff we sometimes see in CI | 19:07 |
TheJulia | 31... | 19:08 |
TheJulia | Can't reproduce it, happens randomly | 19:08 |
TheJulia | and there is basically "no good reason" | 19:08 |
TheJulia | yet it also happens on specific tests | 19:08 |
TheJulia | if it hits 40 with it, I'll just put an inline note and push it back u p | 19:11 |
TheJulia | gah, and it just failed | 19:13 |
*** gmann_lunch is now known as gmann | 19:19 | |
iurygregory | damm =( | 19:25 |
TheJulia | gah | 19:26 |
TheJulia | gah, mock is not even working to prevent it from blowing up if I change the driver | 19:26 |
TheJulia | I'm going to go run into town, I have to do what will hopefully be the very last bit of estate paperwork for this year | 19:27 |
openstackgerrit | Verification of a change to openstack/ironic failed: Fixes to skip validation of in-band deploy steps before agent boot https://review.opendev.org/742840 | 19:33 |
openstackgerrit | Iury Gregory Melo Ferreira proposed openstack/bifrost master: Switch bifrost to openstack.cloud collections https://review.opendev.org/742983 | 19:37 |
iurygregory | good luck TheJulia =) | 19:37 |
iurygregory | good night o/ have a nice weekend =) | 19:38 |
*** bfournie has joined #openstack-ironic | 19:47 | |
*** SpamapS has quit IRC | 20:03 | |
*** SpamapS has joined #openstack-ironic | 20:05 | |
*** ociuhandu has joined #openstack-ironic | 20:10 | |
*** bfournie has quit IRC | 20:10 | |
*** ociuhandu has quit IRC | 20:15 | |
*** Lucas_Gray has joined #openstack-ironic | 20:21 | |
openstackgerrit | Iury Gregory Melo Ferreira proposed openstack/bifrost master: Switch bifrost to openstack.cloud collections https://review.opendev.org/742983 | 21:01 |
*** trandles has joined #openstack-ironic | 21:10 | |
*** vmud213 has quit IRC | 21:22 | |
openstackgerrit | Riccardo Pittau proposed openstack/ironic master: Enforce autospec in some api tests https://review.opendev.org/741468 | 21:22 |
openstackgerrit | Riccardo Pittau proposed openstack/ironic master: Enforce autospec in some api tests https://review.opendev.org/741468 | 21:23 |
trandles | I know it's getting late on Friday but I have a stubborn problem. I'm using Ironic (Train) in standalone mode but can't figure out why I can't get nodes to validate. I have automatic cleaning disabled but it's still telling me "UUID or name of cleaning network is not set in configuration " | 21:42 |
*** ociuhandu has joined #openstack-ironic | 22:02 | |
*** ociuhandu has quit IRC | 22:07 | |
TheJulia | trandles: what is network_interface set to? | 22:14 |
trandles | flat | 22:14 |
TheJulia | are you actually using neutron? | 22:14 |
trandles | nope | 22:15 |
TheJulia | then change it to noop and it should enroll | 22:15 |
TheJulia | you'll want to change the default in the config most likely | 22:15 |
trandles | ...yeah...it's been a loooooooooong week...now I feel dumb :P | 22:15 |
TheJulia | no worries | 22:15 |
TheJulia | doing alright? | 22:15 |
trandles | Thanks for that | 22:15 |
trandles | Yeah, thanks, you? | 22:15 |
TheJulia | no problem :) | 22:15 |
trandles | Busy week with OpenDev I assume. I listened in on the second day. | 22:16 |
TheJulia | yeah, I'm doing okay although I just found out I can't get a battery system for my house | 22:16 |
TheJulia | yeah, very busy | 22:16 |
trandles | :( I asked about a battery for the house here and got a very unsatisfying "dunno, ask some other time" from the power company | 22:16 |
trandles | companhy | 22:16 |
trandles | Good news is we're _finally_ taking a serious look at replacing all of our provisioning with something like Ironic | 22:17 |
TheJulia | trandles: :( about the power company. My father like 30-40 miles south of where your at just wants to go totally off the grid because of the frequent outages... then again they are right in a perfect spot for high winds | 22:18 |
TheJulia | trandles: that is awesome news! | 22:18 |
TheJulia | please submit something to the virtual summit :) | 22:18 |
TheJulia | if you can :) | 22:18 |
trandles | Utilities in our neighborhood are all underground so we don't get the power outages. I'd like to go as off-the-power-grid as possible, especially since our monthly usage is often less than 350kWh. Problem is, our usage is so low that none of the solar installers are keen to work with us for such a small install. | 22:19 |
trandles | I'll see if I can get something together for the virtual summit. I saw the CFP. | 22:20 |
TheJulia | Yeah, if you have the right meter you could do self-install with micro-inverters | 22:20 |
TheJulia | They should be more willing, but there is all the weird regulation around net metering | 22:21 |
trandles | Since we're pretty much 100% work from home, the desire to get solar, install a battery, and get an EV has dwindled. I don't think I've driven 500 miles total since mid-March. That's a big difference from my normal 90 mile round-trip to work. I'm not complaining. ;) | 22:23 |
trandles | TheJulia, deploying...thanks | 22:25 |
TheJulia | trandles: 90 miles!?!? | 22:28 |
TheJulia | wow, I thought you were closer to there | 22:28 |
trandles | Yeah, I live on the south end of Santa Fe | 22:28 |
trandles | Near the airport | 22:28 |
TheJulia | Ahh! | 22:28 |
*** rloo has quit IRC | 22:29 | |
TheJulia | Ahh, for some reason I thought you lived on the other side of the los alamos, but santa fe makes way more sense :) | 22:30 |
TheJulia | my mental map of new mexico might need a trek to roswell, and then a bouncing aroudn the state for a couple of weeks. | 22:32 |
trandles | :D | 22:32 |
*** rloo has joined #openstack-ironic | 22:41 | |
* TheJulia crosses her fingers hoping she finally has a unit test behaving | 22:46 | |
*** tonyb[m] has left #openstack-ironic | 23:21 | |
*** trandles has quit IRC | 23:21 | |
openstackgerrit | Iury Gregory Melo Ferreira proposed openstack/bifrost master: Switch bifrost to openstack.cloud collections https://review.opendev.org/742983 | 23:39 |
Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!