*** rcernin has joined #openstack-ironic | 00:02 | |
*** rcernin has quit IRC | 00:06 | |
*** diurnalist has quit IRC | 01:10 | |
*** xiaolin has joined #openstack-ironic | 01:29 | |
*** xiaolin has quit IRC | 01:45 | |
*** diurnalist has joined #openstack-ironic | 01:50 | |
*** xiaolin has joined #openstack-ironic | 01:51 | |
*** Qianbiao has joined #openstack-ironic | 01:56 | |
*** tzumainn has quit IRC | 02:27 | |
*** rcernin has joined #openstack-ironic | 02:34 | |
*** michchap has quit IRC | 02:35 | |
*** rcernin has quit IRC | 02:41 | |
*** ricolin has joined #openstack-ironic | 02:54 | |
*** k_mouza has joined #openstack-ironic | 03:06 | |
*** k_mouza has quit IRC | 03:10 | |
*** tkajinam is now known as tkajinam|lunch | 03:19 | |
*** tkajinam|lunch is now known as tkajinam | 03:19 | |
*** priteau has joined #openstack-ironic | 03:39 | |
*** priteau has quit IRC | 03:46 | |
*** uzumaki has quit IRC | 05:26 | |
*** diurnalist has quit IRC | 05:43 | |
*** yaawang has quit IRC | 05:47 | |
*** yaawang has joined #openstack-ironic | 05:47 | |
arne_wiebalck | Good morning, ironic! | 05:48 |
---|---|---|
*** yaawang has quit IRC | 06:01 | |
*** yaawang has joined #openstack-ironic | 06:02 | |
*** belmoreira has joined #openstack-ironic | 06:33 | |
*** dougsz has joined #openstack-ironic | 06:57 | |
*** rpittau|afk is now known as rpittau | 07:04 | |
rpittau | good morning ironic! o/ | 07:04 |
*** priteau has joined #openstack-ironic | 07:11 | |
*** yaawang has quit IRC | 07:18 | |
*** yaawang has joined #openstack-ironic | 07:19 | |
iurygregory | good morning arne_wiebalck rpittau and Ironic o/ | 07:22 |
rpittau | hey iurygregory :) | 07:22 |
arne_wiebalck | hey iurygregory rpittau o/ | 07:22 |
rpittau | hey arne_wiebalck :) | 07:22 |
*** yolanda has joined #openstack-ironic | 07:25 | |
*** priteau has quit IRC | 07:35 | |
Qianbiao | hello, the baremetal deploy blocked after loading deploy_ramdisk . | 07:46 |
Qianbiao | sometimes. not always, is there a known issue connect to this? | 07:47 |
Qianbiao | the console shows "Loading xxxxx/deploy_ramdisk... ok", and then stucked there. | 07:49 |
*** aedc_ has joined #openstack-ironic | 07:57 | |
*** zigo_ is now known as zigo | 07:58 | |
*** priteau has joined #openstack-ironic | 08:02 | |
*** jawad_axd has joined #openstack-ironic | 08:03 | |
iurygregory | Qianbiao, I would say to try to debug when doing the deploy, the troubleshooting may give some ideas https://docs.openstack.org/ironic/latest/admin/troubleshooting.html | 08:06 |
*** yaawang has quit IRC | 08:08 | |
Qianbiao | iurygregory i have tried to get deploy.log under /var/log/ironic/deploy | 08:08 |
Qianbiao | but it seems nothing related to this failed deploy | 08:08 |
iurygregory | your setup is configured to get the deploy logs? | 08:08 |
*** yaawang has joined #openstack-ironic | 08:09 | |
iurygregory | maybe you changed the dir where it should be stored | 08:09 |
Qianbiao | yes, collect log always | 08:09 |
Qianbiao | the dir is right. I can find other deploy log there. | 08:09 |
iurygregory | gotcha, strange .-. | 08:09 |
Qianbiao | it's not always reproduce | 08:09 |
Qianbiao | I do not how to reproduce it. | 08:10 |
*** lucasagomes has joined #openstack-ironic | 08:10 | |
*** ravsingh has joined #openstack-ironic | 08:19 | |
rpittau | Qianbiao: is that on real hardware? cause it smells like network issues to me | 08:20 |
Qianbiao | yes it's ibmc baremetal server. | 08:20 |
Qianbiao | after loading deploy_ramdisk image, it should start the tiny os, right? | 08:21 |
Qianbiao | not sure why it does not. | 08:21 |
*** priteau has quit IRC | 08:22 | |
rpittau | Qianbiao: yes, after loading the deploy_ramdisk it should load the kernel from tinycore | 08:23 |
*** priteau has joined #openstack-ironic | 08:24 | |
Qianbiao | rpittau the network for what may cause the issue, i will check it. | 08:24 |
openstackgerrit | Riccardo Pittau proposed openstack/ironic master: Enforce autospec in test_deploy_utils module https://review.opendev.org/733767 | 08:24 |
Qianbiao | it's a standalone env like bifrost. | 08:24 |
rpittau | Qianbiao: flat network? it could also be an issue with the memory on the server or a corrupted ramdisk, have you tried loading a different ramdisk in memory? | 08:26 |
Qianbiao | rpittau i am using tinyipa, and it works at most times. | 08:27 |
Qianbiao | network setting is noop | 08:28 |
Qianbiao | ip offered from dhcp. | 08:28 |
Qianbiao | dnsmasq | 08:28 |
Qianbiao | tinyos is syslinux-6.03.tgz | 08:29 |
rpittau | Qianbiao: are you using an official tinyipa ? | 08:29 |
Qianbiao | yes | 08:29 |
rpittau | Qianbiao: from master or from a stable branch? | 08:30 |
*** tkajinam has quit IRC | 08:31 | |
Qianbiao | it's ussuri for now. | 08:31 |
Qianbiao | stable ussuri. | 08:31 |
openstackgerrit | Arne Wiebalck proposed openstack/ironic-inspector master: [doc] Add misconfigured BMC channel to trouble shooting guide https://review.opendev.org/733770 | 08:32 |
*** dtantsur|afk is now known as dtantsur | 08:34 | |
rpittau | ok, a bit hard to exclude anything, but first I would still check potential network issues or problems with the memory of the server | 08:35 |
rpittau | if the ramdisk works most of the times it might be that sometimes it ended up in a corrupted state after the download | 08:35 |
dtantsur | morning ironic | 08:35 |
rpittau | hey dtantsur :) | 08:35 |
dtantsur | iurygregory: hey, if you have a second: https://review.opendev.org/#/c/733636/ | 08:37 |
patchbot | patch 733636 - bifrost - Set disable_dnsmasq_dns to True - 1 patch set | 08:37 |
dtantsur | rpittau: o/ after thinking more about bifrost-cli, I'd like to reduce its scope to NOT include enroll and deploy | 08:37 |
dtantsur | and build-images likely as well. actually, I'm considering deprecating the built-in DIB support in favour of people just calling IPA-builder or DIB | 08:37 |
rpittau | dtantsur: that sounds good, so basically just an install tool ? | 08:38 |
dtantsur | yep | 08:38 |
Qianbiao | rpittau yes, it's possible | 08:38 |
dtantsur | "How do I get from clean OS to ironic running?" | 08:38 |
Qianbiao | i will ask hardware engineer to have a look | 08:38 |
dtantsur | within the V timeframe I'd like to also have 'bifrost-cli update' which is something we don't quite support yet | 08:39 |
Qianbiao | dtantsur for personal, i think if we could get a simple config.yaml file and document it. | 08:39 |
Qianbiao | there are no need to create a cli | 08:39 |
Qianbiao | ansible-playbook is good enough | 08:39 |
rpittau | dtantsur: I think it makes sense, and the upgrade is something that we really need for testing | 08:39 |
dtantsur | ansible-playbooks has a downside of not making the options obvious | 08:40 |
Qianbiao | ansible-playbook xxxx -tag deploy|install|xxxx | 08:40 |
dtantsur | also there is an aspect of not having 'ansible-playbook' :) | 08:40 |
Qianbiao | :) ok | 08:41 |
rpittau | probably a new set of playbooks for the upgrade | 08:41 |
* rpittau second breakfast time | 08:42 | |
iurygregory | dtantsur, sure | 08:42 |
dtantsur | rpittau: I hope upgrade can be done by a new flag to the existing install playbooks | 08:43 |
dtantsur | like, allowing git to update checkouts, making sure we run online migrations first, etc | 08:44 |
Qianbiao | dtantsur it's not that easy.. if user update ironic.conf manually? or ironic update some config items. | 08:44 |
dtantsur | iurygregory: thanks! could you also check other outstanding bifrost patches when you have more time? some of them are required for the next ussuri release (also cc mgoddard) | 08:45 |
dtantsur | Qianbiao: I don't think we support manual updates | 08:45 |
iurygregory | dtantsur, will do o/ | 08:45 |
Qianbiao | dtantsur i see. | 08:47 |
Qianbiao | for me, the diffcult part for upgrade is the conf file. sometimes it's not compatible between versions. | 08:47 |
Qianbiao | upgrade itself is pretty simple, just install new code, and do db sync. | 08:48 |
iurygregory | dtantsur, also I was able to get ansible 2.9 working o/ will update the patch with a release note and the change that rpittau pointed in the review | 08:48 |
dtantsur | \o/ | 08:49 |
* iurygregory needs to say tks to my friend who helped me hehehe | 08:49 | |
*** uzumaki has joined #openstack-ironic | 08:53 | |
iurygregory | dtantsur, you may have a conflict on https://review.opendev.org/#/c/733075/ after CI merges https://review.opendev.org/#/c/733150/ | 09:08 |
patchbot | patch 733075 - bifrost - Switch to using virtual envs by default - 2 patch sets | 09:08 |
patchbot | patch 733150 - bifrost - Drop hard requirement on debootstrap and EPEL - 3 patch sets | 09:08 |
openstackgerrit | Andreas Jaeger proposed openstack/ironic-python-agent master: DNM Test ironic-python-agent-check-image-dib-centos8 https://review.opendev.org/733781 | 09:10 |
dtantsur | iurygregory: not strictly necessary, but I can try rebasing in advance | 09:10 |
openstackgerrit | Merged openstack/bifrost master: Set disable_dnsmasq_dns to True https://review.opendev.org/733636 | 09:10 |
dtantsur | iurygregory: nope, it rebases cleanly | 09:11 |
iurygregory | dtantsur, oh awesome! | 09:12 |
iurygregory | I will finish the review | 09:12 |
*** uzumaki has quit IRC | 09:12 | |
openstackgerrit | Dmitry Tantsur proposed openstack/bifrost master: bifrost-create-vm-nodes: generate a normal nodes.json https://review.opendev.org/733783 | 09:14 |
openstackgerrit | Dmitry Tantsur proposed openstack/bifrost stable/ussuri: Set disable_dnsmasq_dns to True https://review.opendev.org/733784 | 09:14 |
*** yaawang has quit IRC | 09:16 | |
openstackgerrit | Merged openstack/bifrost master: Enable manual-management and the noop management interface https://review.opendev.org/733156 | 09:16 |
*** yaawang has joined #openstack-ironic | 09:17 | |
*** rcernin has joined #openstack-ironic | 09:18 | |
openstackgerrit | Dmitry Tantsur proposed openstack/bifrost master: Quick start Bifrost CLI https://review.opendev.org/733559 | 09:18 |
openstackgerrit | Dmitry Tantsur proposed openstack/bifrost master: Quick start Bifrost CLI https://review.opendev.org/733559 | 09:20 |
*** rcernin has quit IRC | 09:22 | |
*** AJaeger has joined #openstack-ironic | 09:24 | |
openstackgerrit | Merged openstack/ironic master: Enable Basic HTTP authentication middleware https://review.opendev.org/727467 | 09:25 |
openstackgerrit | Merged openstack/ironic master: Add troubleshooting docs on -ing state failures. https://review.opendev.org/733294 | 09:25 |
AJaeger | I see the job ironic-python-agent-builder failing everywhere since yesterday, see https://zuul.opendev.org/t/openstack/builds?job_name=ironic-python-agent-check-image-dib-centos8 | 09:25 |
AJaeger | anybody an idea or fix for this? | 09:26 |
rpittau | AJaeger: yeah, I saw that too, we're mising a dep | 09:26 |
AJaeger | libffi-devel? | 09:26 |
rpittau | AJaeger: it looks like it, I'm testing locally directly on ipa-builder | 09:26 |
AJaeger | rpittau: thanks, I'll wait ;) | 09:27 |
rpittau | ok :) | 09:27 |
openstackgerrit | Merged openstack/bifrost master: Install uWSGI from PyPI to repair CentOS 8 support https://review.opendev.org/733076 | 09:27 |
openstackgerrit | Dmitry Tantsur proposed openstack/bifrost stable/ussuri: Install uWSGI from PyPI to repair CentOS 8 support https://review.opendev.org/733789 | 09:27 |
dtantsur | rpittau: we can post a fix and watch it in the CI as well | 09:32 |
openstackgerrit | Merged openstack/bifrost master: Support Ubuntu Focal (20.04) https://review.opendev.org/733060 | 09:32 |
openstackgerrit | Dmitry Tantsur proposed openstack/bifrost stable/ussuri: Support Ubuntu Focal (20.04) https://review.opendev.org/733793 | 09:34 |
openstackgerrit | Riccardo Pittau proposed openstack/ironic-python-agent-builder master: Fix centos8 job https://review.opendev.org/733794 | 09:37 |
rpittau | kind of weird seeing that pacakge suddenly missing :/ | 09:41 |
openstackgerrit | Merged openstack/bifrost master: Drop hard requirement on debootstrap and EPEL https://review.opendev.org/733150 | 09:42 |
openstackgerrit | Dmitry Tantsur proposed openstack/bifrost stable/ussuri: Drop hard requirement on debootstrap and EPEL https://review.opendev.org/733797 | 09:43 |
openstackgerrit | Dmitry Tantsur proposed openstack/bifrost master: Switch to using virtual envs by default https://review.opendev.org/733075 | 09:46 |
arne_wiebalck | Ops question: when the motherboard of an active node has been replaced, the address column in the ports table will be wrong (since the MAC has changed), so will all subsequent Ironic actions on this node fail since Ironic will not be able to find the node? | 09:46 |
dtantsur | rpittau, iurygregory, could you re-review https://review.opendev.org/#/c/733075 ? no code changes, gerrit reported a bogus merge conflict :( | 09:47 |
patchbot | patch 733075 - bifrost - Switch to using virtual envs by default - 3 patch sets | 09:47 |
dtantsur | arne_wiebalck: lookup and neutron DHCP won't work any more | 09:48 |
arne_wiebalck | dtantsur: do we have a way to cure this? | 09:48 |
iurygregory | dtantsur, bad gerrit =( | 09:48 |
dtantsur | arne_wiebalck: you can update a MAC on an active node (I don't remember if you need maintenance mode for that) | 09:48 |
dtantsur | iurygregory: right? it rebased locally without conflict.. | 09:48 |
iurygregory | black magic of our CI \o/ | 09:49 |
rpittau | pretty sure it's aliens | 09:50 |
arne_wiebalck | dtantsur: oh, rly? which command would that be? can't see anything rn | 09:51 |
dtantsur | arne_wiebalck: $ openstack baremetal port set --address=<new> <port UUID> | 09:51 |
arne_wiebalck | dtantsur: yep, that's what I was looking for, thanks! | 09:52 |
openstackgerrit | Dmitry Tantsur proposed openstack/bifrost master: bifrost-create-vm-nodes: generate a normal nodes.json https://review.opendev.org/733783 | 09:53 |
openstackgerrit | Dmitry Tantsur proposed openstack/bifrost master: Quick start Bifrost CLI https://review.opendev.org/733559 | 09:54 |
*** yaawang has quit IRC | 09:56 | |
*** yaawang has joined #openstack-ironic | 09:57 | |
rpittau | the theory of "everything else fails in a fix patch" seems getting more and more confirmations | 09:58 |
dtantsur | +++ | 09:58 |
openstackgerrit | Iury Gregory Melo Ferreira proposed openstack/bifrost master: Moving to ansible 2.9 https://review.opendev.org/733479 | 10:01 |
iurygregory | and we don't have the "one patch to fix them all" | 10:01 |
*** k_mouza has joined #openstack-ironic | 10:03 | |
*** rpittau is now known as rpittau|bbl | 10:08 | |
rpittau|bbl | sadness amplifies -> https://storyboard.openstack.org/#!/story/2007759 | 10:13 |
* rpittau|bbl milanese and fries needed, ttyl | 10:13 | |
iurygregory | damm | 10:16 |
*** k_mouza has quit IRC | 10:24 | |
*** k_mouza has joined #openstack-ironic | 10:25 | |
dtantsur | sigh | 10:28 |
*** k_mouza has quit IRC | 10:29 | |
openstackgerrit | Iury Gregory Melo Ferreira proposed openstack/bifrost master: Moving to ansible 2.9 https://review.opendev.org/733479 | 10:34 |
*** diurnalist has joined #openstack-ironic | 10:34 | |
*** jawad_axd has quit IRC | 10:35 | |
AJaeger | rpittau|bbl: thansk, your change fixed the job! Enjoy lunch! | 10:38 |
*** diurnalist has quit IRC | 10:39 | |
* dtantsur wonders where libffi-dev disappeared overnight | 10:41 | |
AJaeger | dtantsur: new package update that does not require it anymore? | 10:42 |
*** k_mouza has joined #openstack-ironic | 10:43 | |
dtantsur | yeah, likely | 10:44 |
*** derekh has joined #openstack-ironic | 10:56 | |
*** jawad_axd has joined #openstack-ironic | 10:57 | |
openstackgerrit | Dmitry Tantsur proposed openstack/ironic master: [WIP] Use the new extension call for getting partition UUIDs https://review.opendev.org/731745 | 10:58 |
openstackgerrit | Merged openstack/bifrost master: Switch to using virtual envs by default https://review.opendev.org/733075 | 10:58 |
openstackgerrit | Dmitry Tantsur proposed openstack/ironic stable/ussuri: Fix fast track when exiting cleaning https://review.opendev.org/733803 | 10:59 |
*** jawad_axd has quit IRC | 11:02 | |
openstackgerrit | Dmitry Tantsur proposed openstack/bifrost stable/ussuri: Switch to using virtual envs by default https://review.opendev.org/733808 | 11:03 |
*** k_mouza has quit IRC | 11:04 | |
*** k_mouza has joined #openstack-ironic | 11:05 | |
openstackgerrit | Merged openstack/bifrost master: Install metalsmith by default https://review.opendev.org/733095 | 11:13 |
openstackgerrit | Merged openstack/bifrost master: Documentation: minor fixes in the user guide https://review.opendev.org/733486 | 11:13 |
openstackgerrit | Dmitry Tantsur proposed openstack/bifrost stable/ussuri: Documentation: minor fixes in the user guide https://review.opendev.org/733811 | 11:15 |
dtantsur | mgoddard: we can release bifrost ussuri when we merge https://review.opendev.org/#/q/status:open+project:openstack/bifrost+branch:stable/ussuri (help appreciated) | 11:17 |
mgoddard | dtantsur: not stable core | 11:17 |
dtantsur | mgoddard: I think bifrost doesn't follow the stable policy and thus allows any core to merge changes | 11:17 |
* dtantsur cannot check it since he's a stable core | 11:17 | |
mgoddard | dtantsur: would like to release stables ASAP due to the DIB breakage | 11:18 |
mgoddard | dtantsur: seems you are right | 11:18 |
dtantsur | mgoddard: yep, but the breakage didn't (IIRC) affect Ussuri | 11:18 |
dtantsur | so let's release it separately | 11:18 |
iurygregory | dtantsur, we do consider https://review.opendev.org/733793 important for stable/ussuri? | 11:18 |
patchbot | patch 733793 - bifrost (stable/ussuri) - Support Ubuntu Focal (20.04) - 1 patch set | 11:18 |
dtantsur | iurygregory: yep, why not? | 11:18 |
mgoddard | right, that's why I removed it from the patch | 11:18 |
iurygregory | dtantsur, well in theory focal will be something for victoria | 11:19 |
iurygregory | but since it works why not =) | 11:19 |
mgoddard | as usual, the first release of bifrost ussuri doesn't pin to stable/ussuri. Would be nice to release to get that fix | 11:19 |
dtantsur | mgoddard: oh! good call. I can do the change. | 11:19 |
mgoddard | dtantsur: I think it was fixed, just not released | 11:19 |
* iurygregory facepalm after reading what mgoddard said =X | 11:19 | |
mgoddard | iurygregory: happens every time | 11:20 |
dtantsur | ah, right | 11:20 |
mgoddard | I suggested pinning on master then reverting but didn't get traction | 11:20 |
* dtantsur is considering implications | 11:21 | |
mgoddard | it means bifrost has to release after other ironic projects | 11:21 |
dtantsur | yeah, this is probably the only implication | 11:21 |
dtantsur | dunno, let's start an ML thread? | 11:21 |
dtantsur | iurygregory: "since it works" is my take on it :) we had to kick out Xenial in Ussuri, so I thought adding support for the latest LTS Ubuntu won't hurt. | 11:22 |
iurygregory | dtantsur, yeah =) | 11:22 |
dtantsur | if we consider bifrost as a way for people to get into ironic, supporting latest versions makes a lot of sense | 11:23 |
* iurygregory goes eat, meating in 37 min | 11:23 | |
*** belmoreira has quit IRC | 11:36 | |
*** k_mouza has quit IRC | 11:43 | |
*** k_mouza has joined #openstack-ironic | 11:46 | |
openstackgerrit | Merged openstack/bifrost stable/ussuri: Set disable_dnsmasq_dns to True https://review.opendev.org/733784 | 11:56 |
openstackgerrit | Dmitry Tantsur proposed openstack/bifrost master: Restore installing DIB-specific dependencies https://review.opendev.org/733815 | 11:56 |
dtantsur | mgoddard: good catch on ^^ (will backport to ussuri as well) | 11:56 |
*** rpittau|bbl is now known as rpittau | 11:59 | |
*** dougsz has quit IRC | 12:04 | |
openstackgerrit | Merged openstack/ironic master: If the "[conductor]XXX_timeout" is less than 0,disable periodic task https://review.opendev.org/723795 | 12:05 |
openstackgerrit | Merged openstack/bifrost stable/ussuri: Install uWSGI from PyPI to repair CentOS 8 support https://review.opendev.org/733789 | 12:06 |
openstackgerrit | Merged openstack/bifrost stable/ussuri: Documentation: minor fixes in the user guide https://review.opendev.org/733811 | 12:06 |
openstackgerrit | Merged openstack/bifrost stable/ussuri: Drop hard requirement on debootstrap and EPEL https://review.opendev.org/733797 | 12:08 |
*** k_mouza has quit IRC | 12:13 | |
*** k_mouza has joined #openstack-ironic | 12:14 | |
*** rh-jelabarre has joined #openstack-ironic | 12:14 | |
*** derekh has quit IRC | 12:20 | |
*** rh-jelabarre has quit IRC | 12:22 | |
openstackgerrit | Merged openstack/ironic master: Follow-up patch for ilo out-of-band `erase_devices` https://review.opendev.org/679927 | 12:22 |
dtantsur | OUCH | 12:29 |
dtantsur | ironic master also fails with missing ffi-dev? | 12:29 |
rpittau | lol | 12:29 |
dtantsur | ah, no, my patch uses IPA-builder | 12:30 |
* rpittau puts the rum bottle back in the shelf | 12:31 | |
dtantsur | rpittau: let's also put ffi-dev to https://opendev.org/openstack/ironic-python-agent/src/branch/master/bindep.txt | 12:31 |
rpittau | yup | 12:31 |
dtantsur | rpittau: ugh, https://review.opendev.org/#/c/733794/ is incomplete | 12:32 |
patchbot | patch 733794 - ironic-python-agent-builder - Fix centos8 job - 1 patch set | 12:32 |
dtantsur | iurygregory: can you remove W+1 from ^^ | 12:33 |
dtantsur | rpittau: sorry for missing it, we need to update pkg-map to account for distro differences. libffi-devel is an RH name. | 12:33 |
iurygregory | dtantsur, done | 12:33 |
dtantsur | thx! | 12:33 |
rpittau | right... | 12:33 |
dtantsur | as an aside, we should move most of package-installs to bindep in IPA itself | 12:34 |
openstackgerrit | Riccardo Pittau proposed openstack/ironic-python-agent master: Add ffi development lib to bindep https://review.opendev.org/733821 | 12:35 |
iurygregory | dtantsur, you pinged right after i finished my meeting hehe | 12:36 |
*** derekh has joined #openstack-ironic | 12:36 | |
dtantsur | good :) | 12:36 |
openstackgerrit | Dmitry Tantsur proposed openstack/ironic master: Use the new extension call for getting partition UUIDs https://review.opendev.org/731745 | 12:39 |
openstackgerrit | Merged openstack/bifrost stable/ussuri: Support Ubuntu Focal (20.04) https://review.opendev.org/733793 | 12:40 |
openstackgerrit | Riccardo Pittau proposed openstack/ironic-python-agent-builder master: Install missing ffi development library https://review.opendev.org/733794 | 12:41 |
openstackgerrit | Dmitry Tantsur proposed openstack/ironic master: Use the new extension call for getting partition UUIDs https://review.opendev.org/731745 | 12:41 |
rpittau | 5th recheck for the stable/ussuri memory fix | 12:43 |
dtantsur | \o/ | 12:43 |
*** rh-jelabarre has joined #openstack-ironic | 12:43 | |
* rpittau takes the rum bottle back on the desk | 12:43 | |
iurygregory | wow | 12:43 |
*** michchap has joined #openstack-ironic | 12:48 | |
openstackgerrit | Merged openstack/bifrost master: Moving to ansible 2.9 https://review.opendev.org/733479 | 12:51 |
openstackgerrit | Riccardo Pittau proposed openstack/ironic master: Enforce autospec in test_ipxe module https://review.opendev.org/733824 | 12:51 |
*** rloo has joined #openstack-ironic | 12:52 | |
openstackgerrit | Dmitry Tantsur proposed openstack/ironic master: WIP: Decompose the core deploy step of the direct deploy https://review.opendev.org/698309 | 12:52 |
openstackgerrit | Dmitry Tantsur proposed openstack/ironic master: DNM testing deploy steps https://review.opendev.org/733825 | 12:52 |
TheJulia | good morning | 13:05 |
rpittau | good morning TheJulia :) | 13:05 |
dtantsur | morning TheJulia | 13:05 |
iurygregory | good morning TheJulia =) | 13:07 |
* TheJulia sips coffee and tries to wake up | 13:09 | |
openstackgerrit | Riccardo Pittau proposed openstack/ironic master: Enforce autospec in test_ipxe and test_pxe https://review.opendev.org/733824 | 13:16 |
* TheJulia looks a patch and wonders if it is too early to enjoy some tuaca | 13:19 | |
*** rcernin has joined #openstack-ironic | 13:19 | |
dtantsur | depends on which patch (and what tuaca is) | 13:20 |
* iurygregory googles tuaca | 13:20 | |
TheJulia | sushy-tools | 13:20 |
TheJulia | amd google :) | 13:20 |
iurygregory | omg liqueur wow | 13:20 |
iurygregory | 70%? | 13:20 |
TheJulia | I don't think we can get it that strong here | 13:21 |
dtantsur | iurygregory: wikipedia says 70 proof (35% ABV). | 13:21 |
dtantsur | dunno what proof is, but 35% sounds reasonable | 13:21 |
rpittau | is tuaca good with sushy ? :) | 13:21 |
iurygregory | yeah | 13:21 |
TheJulia | rpittau: good question | 13:21 |
iurygregory | rpittau, probably with sushy-tools | 13:21 |
dtantsur | :D | 13:21 |
iurygregory | or sushi =) | 13:21 |
dtantsur | well, a sweet liqueur with fish.. | 13:22 |
dtantsur | probably when TheJulia will start reviewing my deploy steps patches? :D | 13:23 |
TheJulia | those are a fine whiskey sort of patches | 13:23 |
iurygregory | hehehe | 13:23 |
*** rcernin has quit IRC | 13:24 | |
dtantsur | mgoddard: not sure about you, but every time I read has_decomposed_deploy_steps, I interpret "decomposed" in its another meaning, and it's pretty ironic | 13:25 |
iurygregory | decomposed sounds like you have a huge step and you split in many that in the end will do the same thing of the big step ... | 13:26 |
dtantsur | this is the reality. my brain substitutes something like "rotten" though | 13:27 |
iurygregory | wpw | 13:27 |
iurygregory | wow* | 13:27 |
TheJulia | There is one thing that worries me about deploy steps which is the comment that dtantsur left on the oom prevention fix I've been working on that apparently we won't be able to stall out a step until the next heartbeat? | 13:28 |
dtantsur | mmm, not sure I understand | 13:29 |
dtantsur | I think it's a good thing about deploy steps that you can stop one and get back to it in 2 mins? | 13:29 |
dtantsur | (or how often do we heartbeat?) | 13:29 |
TheJulia | 120-300 seconds by default | 13:30 |
TheJulia | I guess for deploy steps we would need to instead set appropriate flags and then return nothing? | 13:31 |
*** bnemec is now known as beekneemech | 13:32 | |
dtantsur | TheJulia: DEPLOYWAIT+flags, I think | 13:32 |
dtantsur | and I may be wrong since I still fail to keep the whole picture in my head | 13:32 |
* dtantsur had to draw the interaction inside agent_base.py and agent.py and his whiteboard | 13:33 | |
iurygregory | wow | 13:42 |
iurygregory | dtantsur, you took a picture of it? | 13:43 |
dtantsur | yeah | 13:50 |
TheJulia | dtantsur: that is my problem right now with stress, I'm second guessing myself too :( | 13:52 |
TheJulia | on a plus side, it is a perfect time to build a compatability chart | 13:52 |
openstackgerrit | Riccardo Pittau proposed openstack/bifrost master: [WIP] Convert with_items to loop https://review.opendev.org/733847 | 13:53 |
TheJulia | rpioso: I'm nowhere near done on that chart, and we may need several so it might be a few days before I can cycle to the questions | 13:53 |
iurygregory | ironic-python-agent-check-image-dib-fedora it's broken for a while right? | 13:54 |
iurygregory | at least looking at https://zuul.openstack.org/builds?job_name=ironic-python-agent-check-image-dib-fedora | 13:54 |
openstackgerrit | Dmitry Tantsur proposed openstack/ironic master: WIP: Decompose the core deploy step of the direct deploy https://review.opendev.org/698309 | 14:14 |
openstackgerrit | Dmitry Tantsur proposed openstack/ironic master: DNM testing deploy steps https://review.opendev.org/733825 | 14:14 |
openstackgerrit | Merged openstack/sushy-tools master: Read current domain state from XML on filesystem https://review.opendev.org/733152 | 14:17 |
openstackgerrit | Dmitry Tantsur proposed openstack/sushy-tools master: Simplify the CI jobs and add a vmedia+UEFI job https://review.opendev.org/733495 | 14:18 |
*** ricolin_ has joined #openstack-ironic | 14:26 | |
*** ricolin has quit IRC | 14:29 | |
openstackgerrit | Merged openstack/bifrost stable/ussuri: Switch to using virtual envs by default https://review.opendev.org/733808 | 14:32 |
*** Qianbiao has quit IRC | 14:36 | |
*** Qianbiao has joined #openstack-ironic | 14:37 | |
openstackgerrit | Dmitry Tantsur proposed openstack/bifrost master: Set default deploy/rescue kernel/ramdisk in ironic.conf https://review.opendev.org/733860 | 14:41 |
*** diurnalist has joined #openstack-ironic | 14:52 | |
*** rcernin has joined #openstack-ironic | 15:20 | |
openstackgerrit | Iury Gregory Melo Ferreira proposed openstack/ironic master: Add api-ref for indicators API https://review.opendev.org/732149 | 15:24 |
*** rcernin has quit IRC | 15:24 | |
*** gyee has joined #openstack-ironic | 15:44 | |
*** Qianbiao has quit IRC | 15:50 | |
dtantsur | curl | sudo, lol JayF | 16:02 |
iurygregory | LOL | 16:03 |
*** tkajinam has joined #openstack-ironic | 16:08 | |
*** dtantsur is now known as dtantsur|afk | 16:10 | |
dtantsur|afk | have a great weekend! | 16:10 |
*** rpittau is now known as rpittau|afk | 16:10 | |
rpittau|afk | bye all! o/ | 16:10 |
*** lucasagomes has quit IRC | 16:15 | |
TheJulia | so random thought, it felt like we tried to compress things a little too much, thoughts? | 16:22 |
JayF | I don't think I have a wide enough lens on the state of the project to judge that. I do know if the PTG had been much more time consuming, I'd have had to "pick and choose" what sessions I participated in. | 16:23 |
*** alexmcleod has quit IRC | 16:25 | |
*** shadower has quit IRC | 16:35 | |
TheJulia | this is true | 16:37 |
* TheJulia suspects today is going to require a nap | 16:37 | |
*** shadower has joined #openstack-ironic | 16:41 | |
*** tkajinam has quit IRC | 16:49 | |
openstackgerrit | Riccardo Pittau proposed openstack/ironic master: Enforce autospec in test_snmp https://review.opendev.org/733891 | 16:51 |
TheJulia | \o/ no more calls this week | 17:07 |
*** derekh has quit IRC | 17:09 | |
*** ricolin_ has quit IRC | 17:11 | |
openstackgerrit | Merged openstack/ironic-python-agent-builder master: Install missing ffi development library https://review.opendev.org/733794 | 17:12 |
*** AJaeger has left #openstack-ironic | 17:13 | |
arne_wiebalck | TheJulia: If that means you're bored, I have put up multiple doc patches for Ironic and the Inspector re troubleshooting :-P | 17:15 |
openstackgerrit | Merged openstack/bifrost stable/stein: CI: reduce memory and disk for testing nodes https://review.opendev.org/733560 | 17:17 |
*** rcernin has joined #openstack-ironic | 17:21 | |
*** priteau has quit IRC | 17:21 | |
TheJulia | arne_wiebalck: Heh, I'm going to first go get some tasty lunch and then likely work on the capabilities doc | 17:25 |
*** rcernin has quit IRC | 17:26 | |
arne_wiebalck | TheJulia: ;) | 17:28 |
arne_wiebalck | TheJulia: Enjoy the call free rest of Friday and have a good week-end! | 17:28 |
TheJulia | you too arne_wiebalck | 17:28 |
arne_wiebalck | TheJulia: Thanks again for managing the PTG! | 17:28 |
arne_wiebalck | Bye all o/ | 17:29 |
TheJulia | maelk: your boot image requirements, have you had a chance to look at the ramdisk deploy interface? | 17:29 |
TheJulia | have a wonderful weekend! | 17:29 |
TheJulia | maelk: also, w/r/t mtls, it might be good to also discuss during one of our weekly meetings | 17:35 |
*** ociuhandu has joined #openstack-ironic | 18:25 | |
*** ociuhandu has quit IRC | 18:26 | |
*** ociuhandu_ has joined #openstack-ironic | 18:26 | |
*** ociuhandu_ has quit IRC | 18:27 | |
*** ociuhandu has joined #openstack-ironic | 18:27 | |
*** ociuhandu has quit IRC | 18:32 | |
openstackgerrit | Iury Gregory Melo Ferreira proposed openstack/ironic-lib master: DNM - Testing CI https://review.opendev.org/733909 | 18:33 |
*** k_mouza has quit IRC | 18:35 | |
openstackgerrit | Verification of a change to openstack/ironic failed: CI: raise DIB jobs memory to 3 GiB https://review.opendev.org/733519 | 18:35 |
*** ociuhandu has joined #openstack-ironic | 18:35 | |
TheJulia | ci hates us | 18:36 |
*** ociuhandu has quit IRC | 18:40 | |
iurygregory | yes =( | 18:46 |
iurygregory | and ironic-lib seems broken | 18:46 |
iurygregory | the unit tests at least | 18:46 |
larsks | If I'm using ironic without keystone (and w/o auth), how to I tell the cli (openstack baremetal ...) what endpoint to use? | 18:52 |
*** dustinc has joined #openstack-ironic | 19:01 | |
*** aedc has joined #openstack-ironic | 19:07 | |
*** aedc_ has quit IRC | 19:08 | |
*** ravsingh has quit IRC | 19:11 | |
*** dougsz has joined #openstack-ironic | 19:16 | |
TheJulia | larsks: one moment | 19:18 |
TheJulia | https://github.com/openstack/bifrost/blob/master/env-vars | 19:19 |
larsks | TheJulia: perfect, thanks! | 19:21 |
*** aedc has quit IRC | 19:23 | |
*** jtomasek has quit IRC | 19:27 | |
*** dougsz has quit IRC | 19:58 | |
*** xiaolin has quit IRC | 20:06 | |
iurygregory | seems zeroconf 0.27.1 broke us hehe | 20:30 |
TheJulia | :( | 20:30 |
TheJulia | how bad/ | 20:30 |
TheJulia | ? | 20:30 |
iurygregory | 3 tests failling in ironic-lib | 20:34 |
iurygregory | only the mdns | 20:34 |
iurygregory | I think it was zeroconf trying to check locally | 20:34 |
iurygregory | oh ok ci install 0.27.0 | 20:34 |
* iurygregory looks at the changes in code for zeroconf | 20:35 | |
iurygregory | Backwards incompatible: | 20:43 |
iurygregory | Removed previously deprecated ServiceInfo address constructor parameter and property | 20:43 |
iurygregory | Aha! | 20:44 |
iurygregory | either we should update our code to fix sicne we don't have address anymore or add a limit <0.27.0 in our requirements for now | 20:46 |
iurygregory | 0.26.3 works fine but 0.27.0 or 0.27.1 boom | 20:47 |
TheJulia | if you don't mind, go ahead and post a requirements limit for it | 20:47 |
iurygregory | sure, do we need a release note? | 20:48 |
TheJulia | is it in openstack/requirements/upper-constraints.txt? | 20:48 |
iurygregory | well I think we can fix by adding <0.27.0 in the requirements.txt for ironic-lib | 20:50 |
TheJulia | The requirements tests may file a complaint with that, but always worth a try :) | 20:50 |
iurygregory | https://github.com/openstack/requirements/blob/master/upper-constraints.txt#L104 | 20:50 |
iurygregory | oh truth | 20:51 |
TheJulia | :) | 20:51 |
iurygregory | let me test the requirements locally | 20:51 |
TheJulia | oik | 20:51 |
TheJulia | ok | 20:51 |
* TheJulia goes back to the compatabilit ymatrix | 20:51 | |
iurygregory | ERROR: Requirement for package zeroconf excludes a version not excluded in the global list | 20:53 |
iurygregory | yay =( | 20:53 |
TheJulia | we do it so projects can co-exist without breaking | 20:53 |
iurygregory | yeah | 20:54 |
iurygregory | Im going to check with requirements to see what we can do | 20:54 |
TheJulia | post a chance to the upper constraints | 20:54 |
TheJulia | err | 20:54 |
TheJulia | requirements | 20:54 |
TheJulia | upper-constraints in the requirements repo is the latest gate tested version | 20:55 |
TheJulia | since we're likely the only ones who run it, it can be problematic | 20:55 |
iurygregory | ack | 20:57 |
* TheJulia plays rammstein and wonders how many columns this table is going to end up with | 20:58 | |
iurygregory | ++ | 21:01 |
iurygregory | https://review.opendev.org/733920 done | 21:11 |
patchbot | patch 733920 - requirements - Pin zeroconf to 0.26.3 - 1 patch set | 21:11 |
TheJulia | getting there, looks like I've got everything that is not workflow related or highly vendor specific | 21:15 |
TheJulia | or network/deploy interface related | 21:15 |
TheJulia | I need to go lay down for a little bit | 21:15 |
*** rcernin has joined #openstack-ironic | 21:23 | |
*** rcernin has quit IRC | 21:27 | |
*** goldyfruit has joined #openstack-ironic | 21:37 | |
openstackgerrit | Merged openstack/ironic master: Add `get_node_network_data` method to NetworkInterface https://review.opendev.org/688184 | 21:39 |
*** dsneddon has quit IRC | 21:40 | |
*** bfournie has quit IRC | 21:40 | |
*** dsneddon has joined #openstack-ironic | 21:40 | |
*** goldyfruit has quit IRC | 21:41 | |
iurygregory | wow CI merged something =O | 21:46 |
*** bfournie has joined #openstack-ironic | 21:53 | |
*** rloo has quit IRC | 22:00 | |
*** dustinc has quit IRC | 22:01 | |
*** rh-jelabarre has quit IRC | 22:22 | |
*** ociuhandu has joined #openstack-ironic | 22:31 | |
*** ociuhandu has quit IRC | 22:36 | |
TheJulia | gasp! Maybe it doesn't hate us | 22:38 |
TheJulia | 100% | 22:38 |
*** jawad_axd has joined #openstack-ironic | 22:42 | |
*** rloo has joined #openstack-ironic | 22:45 | |
*** jawad_axd has quit IRC | 22:47 | |
*** rcernin has joined #openstack-ironic | 23:02 | |
*** ab-a has quit IRC | 23:20 | |
*** ab-a has joined #openstack-ironic | 23:21 | |
*** rcernin has quit IRC | 23:27 |
Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!