*** eandersson has quit IRC | 00:35 | |
*** ccstone has quit IRC | 00:35 | |
*** eandersson has joined #openstack-ironic | 00:36 | |
openstackgerrit | Kaifeng Wang proposed openstack/ironic-python-agent-builder master: Fix python-dev package mapping https://review.opendev.org/733462 | 00:51 |
---|---|---|
openstackgerrit | Merged openstack/ironic stable/ussuri: Add the noop management interface to the redfish hardware type https://review.opendev.org/731539 | 01:02 |
*** xiaolin has joined #openstack-ironic | 01:04 | |
openstackgerrit | Verification of a change to openstack/ironic failed: change devstack ipa-builder default to BRANCHLESS_TARGET_BRANCH https://review.opendev.org/730887 | 01:10 |
*** yaawang has quit IRC | 01:11 | |
*** rcernin has quit IRC | 02:28 | |
*** diurnalist has joined #openstack-ironic | 02:28 | |
*** Qianbiao has joined #openstack-ironic | 02:56 | |
*** rcernin has joined #openstack-ironic | 02:58 | |
*** ociuhandu has joined #openstack-ironic | 03:12 | |
*** ociuhandu has quit IRC | 03:17 | |
*** diurnalist has quit IRC | 03:43 | |
openstackgerrit | Verification of a change to openstack/ironic failed: Increase callback timeout https://review.opendev.org/731599 | 03:47 |
*** yaawang has joined #openstack-ironic | 03:51 | |
*** ricolin has joined #openstack-ironic | 04:14 | |
*** dhellmann has quit IRC | 04:17 | |
*** dhellmann has joined #openstack-ironic | 04:27 | |
openstackgerrit | Merged openstack/ironic master: Add validate decorator to expose https://review.opendev.org/704487 | 04:39 |
*** jtomasek has joined #openstack-ironic | 04:58 | |
*** k_mouza has joined #openstack-ironic | 05:07 | |
*** k_mouza has quit IRC | 05:11 | |
*** jtomasek has quit IRC | 05:37 | |
arne_wiebalck | Good morning, ironic! | 05:38 |
*** jawad_axd has joined #openstack-ironic | 06:07 | |
*** jawad_axd has quit IRC | 06:12 | |
*** xiaolin has quit IRC | 06:35 | |
*** jtomasek has joined #openstack-ironic | 06:41 | |
*** yolanda has joined #openstack-ironic | 06:43 | |
*** iurygregory has joined #openstack-ironic | 07:10 | |
iurygregory | good morning Ironic | 07:10 |
*** rpittau|afk is now known as rpittau | 07:14 | |
rpittau | good morning ironic! o/ | 07:14 |
iurygregory | good morning rpittau o/ | 07:17 |
rpittau | hey iurygregory :) | 07:17 |
maelk | Hello! noob question! How can I map an ironic version X.Y.Z-dev... to a specific commit in the master branch ? | 07:19 |
*** rcernin has quit IRC | 07:21 | |
rpittau | hey maelk, not sure I understand correctly what you're trying to do | 07:25 |
*** dougsz has joined #openstack-ironic | 07:33 | |
*** jawad_axd has joined #openstack-ironic | 07:34 | |
Qianbiao | hello, maelk X.Y.Z-dev${int} means X.Y.Z => major version, dev(int) means the int-th commit of major version | 07:34 |
Qianbiao | mostly dev-int is a version for development like master | 07:35 |
Qianbiao | ironic version | 07:37 |
rpittau | oh ok, that :) | 07:37 |
Qianbiao | <rpittau> i remember someone gave a git command that can get the commit id. | 07:42 |
Qianbiao | but i forget it. | 07:42 |
rpittau | Qianbiao: git rev-parse HEAD will give the commit id | 07:45 |
Qianbiao | rpittau no, i mean can get the *th commit. | 07:47 |
Qianbiao | x.y.z-dev(*th) ^ | 07:47 |
Qianbiao | get the *th commit of x.y.z | 07:48 |
rpittau | mmmmm | 07:48 |
iurygregory | maybe the CI scripts have something | 07:50 |
rpittau | so mmm if you know the number of the commit, I can think only doing some math | 07:50 |
iurygregory | not ours but devstack/ infra | 07:50 |
Qianbiao | rpittau someone on this channel gives the answer :). there are someone ask same question as maelk asked. | 07:51 |
Qianbiao | I just can not search it. forget the keyword | 07:51 |
maelk | Qianbiao thanks for the answer. I already knew that it is the nth commit since the last release, but I don't know how to map that to an actualy commit without just counting. I was wondering if there's a command or a tool to do that | 07:56 |
Qianbiao | <maelk> ok, there must have. I just forget it :) | 07:57 |
Qianbiao | <iurygregory> any online bakup of this channel chat records. | 07:57 |
iurygregory | Qianbiao, http://eavesdrop.openstack.org/irclogs/%23openstack-ironic/ | 07:58 |
Qianbiao | oh it's grouped by days. ~~ | 07:58 |
*** lucasagomes has joined #openstack-ironic | 07:59 | |
iurygregory | yeah | 08:01 |
*** tkajinam has quit IRC | 08:07 | |
*** ravsingh has joined #openstack-ironic | 08:07 | |
dtantsur | morning ironic | 08:13 |
rpittau | hey dtantsur :) | 08:13 |
rpittau | iurygregory: are you working on fixing ironic-lib for the latest version of zeroconf ? | 08:16 |
iurygregory | rpittau, https://review.opendev.org/733920 | 08:22 |
patchbot | patch 733920 - requirements - Pin zeroconf to 0.26.3 - 3 patch sets | 08:22 |
iurygregory | the fix atm | 08:22 |
rpittau | iurygregory: I mean the actual fix in ironic-lib | 08:22 |
iurygregory | I talked with Julia before pushing | 08:22 |
iurygregory | rpittau, havent touched | 08:22 |
rpittau | ok | 08:22 |
dtantsur | iurygregory: is it hard to fix? pinning is a very problematic action | 08:24 |
iurygregory | dtantsur, I didn't dig that much it was friday at midnight when I noticed | 08:25 |
dtantsur | do we have a story for this? I'm trying to understand if disabling the 2 releases will actually help | 08:25 |
dtantsur | what will happen when 0.27.2 is released? | 08:25 |
rpittau | dtantsur: what I see in the code is just tests that need fixing so far | 08:26 |
iurygregory | we will have fixed I hope | 08:26 |
dtantsur | iurygregory: could you at least point me at a breakage? | 08:26 |
iurygregory | 0.27.0 | 08:26 |
iurygregory | Removed previously deprecated ServiceInfo address constructor parameter and property | 08:26 |
iurygregory | Backwards incompatible | 08:26 |
rpittau | dtantsur: I have a patch with the fixed tests, do we want to go from there ? | 08:26 |
dtantsur | rpittau: yep | 08:26 |
dtantsur | iurygregory: if this is what breaks us, it will NOT get ever fixed | 08:26 |
iurygregory | dtantsur, I know, it was just a workaround for the issue so we could fix when I talked with TheJulia | 08:27 |
dtantsur | yeah, but tomorrow (say) they'll release 0.27.2 and we're broken again | 08:27 |
openstackgerrit | Riccardo Pittau proposed openstack/ironic-lib master: Fix for latest zeroconf version https://review.opendev.org/734033 | 08:28 |
openstackgerrit | Riccardo Pittau proposed openstack/ironic-lib master: Fix for latest zeroconf version https://review.opendev.org/734033 | 08:29 |
*** xiaolin has joined #openstack-ironic | 08:36 | |
*** alexmcleod has joined #openstack-ironic | 08:42 | |
*** sshnaidm|afk is now known as sshnaidm | 08:44 | |
* dtantsur shakes fist at vodafone | 08:45 | |
*** jtomasek has quit IRC | 08:48 | |
*** jtomasek has joined #openstack-ironic | 08:50 | |
openstackgerrit | Dmitry Tantsur proposed openstack/ironic-specs master: Amend the in-band deploy steps spec https://review.opendev.org/734039 | 08:55 |
dtantsur | mgoddard: morning, if you have a minute ^^ | 08:55 |
mgoddard | dtantsur: sure | 08:57 |
dtantsur | iurygregory: I think rpioso's patch should do it wrt zeroconf | 08:59 |
dtantsur | * rpittau's (sorry rpioso) | 08:59 |
iurygregory | hehe | 08:59 |
*** ravsingh has quit IRC | 08:59 | |
iurygregory | dtantsur, yep ++ | 08:59 |
dtantsur | now to something completely different: could I get some reviews on https://review.opendev.org/#/c/733957/ please? | 09:00 |
patchbot | patch 733957 - ironic - devstack: increase concurrency and worker timeout ... - 1 patch set | 09:00 |
dtantsur | it fixes Yet Another transient failure in our CI (hopefully) | 09:00 |
*** ravsingh has joined #openstack-ironic | 09:00 | |
iurygregory | dtantsur, done | 09:02 |
dtantsur | thx! | 09:02 |
*** k_mouza has joined #openstack-ironic | 09:03 | |
*** kaiokmo has joined #openstack-ironic | 09:06 | |
openstackgerrit | Merged openstack/ironic-lib master: Fix for latest zeroconf version https://review.opendev.org/734033 | 09:12 |
dtantsur | mgoddard: could you release ^^ please? | 09:14 |
dtantsur | actually, no, it's tests only | 09:14 |
dtantsur | rpittau: could you add backports to ussuri and train (I think)? | 09:14 |
rpittau | dtantsur: yeah, working on that :) | 09:15 |
dtantsur | awesome | 09:15 |
openstackgerrit | Riccardo Pittau proposed openstack/ironic-lib stable/ussuri: Fix for latest zeroconf version https://review.opendev.org/734044 | 09:15 |
openstackgerrit | Riccardo Pittau proposed openstack/ironic-lib stable/train: Fix for latest zeroconf version https://review.opendev.org/734045 | 09:15 |
*** ricolin has quit IRC | 09:18 | |
openstackgerrit | Merged openstack/bifrost master: Remove empty files from the roles https://review.opendev.org/733956 | 09:33 |
openstackgerrit | Dmitry Tantsur proposed openstack/ironic master: agent: poll long-running commands till completion https://review.opendev.org/733985 | 09:34 |
openstackgerrit | Dmitry Tantsur proposed openstack/ironic master: WIP: Decompose the core deploy step of the direct deploy https://review.opendev.org/698309 | 09:34 |
openstackgerrit | Dmitry Tantsur proposed openstack/ironic master: DNM testing deploy steps https://review.opendev.org/733825 | 09:34 |
dtantsur | mgoddard, rpittau: could you check https://review.opendev.org/#/c/732515/ ? I think it's the last patch before we can release stable/ussuri | 09:38 |
patchbot | patch 732515 - bifrost (stable/ussuri) - Correct handling enable_venv in pip_install.yml - 1 patch set | 09:38 |
iurygregory | I missed that one, sorry dtantsur | 09:42 |
dtantsur | no worries, I made too many patches last week :D | 09:43 |
iurygregory | hehehe | 09:45 |
iurygregory | funny libselinux-python is not available in F32 hehe | 10:12 |
iurygregory | maybe there is libselinux-python3... | 10:12 |
*** xiaolin has quit IRC | 10:12 | |
openstackgerrit | Merged openstack/bifrost stable/ussuri: Correct handling enable_venv in pip_install.yml https://review.opendev.org/732515 | 10:14 |
*** rpittau is now known as rpittau|bbl | 10:15 | |
rpittau|bbl | iurygregory: python3-libselinux | 10:16 |
iurygregory | rpittau|bbl, yeah I just found =) | 10:16 |
*** k_mouza has quit IRC | 10:17 | |
iurygregory | I just had to remove libselinux-python3 from https://github.com/openstack/bifrost/blob/master/playbooks/roles/bifrost-ironic-install/defaults/required_defaults_Fedora.yml#L19 to run things locally | 10:17 |
*** k_mouza has joined #openstack-ironic | 10:22 | |
openstackgerrit | Arne Wiebalck proposed openstack/ironic master: [doc] Extend trouble shooting docs for node stuck in wait states https://review.opendev.org/733655 | 10:23 |
*** jtomasek has quit IRC | 10:41 | |
openstackgerrit | Verification of a change to openstack/ironic failed: devstack: increase concurrency and worker timeout for sushy-tools https://review.opendev.org/733957 | 10:41 |
*** jtomasek has joined #openstack-ironic | 10:43 | |
*** derekh has joined #openstack-ironic | 10:46 | |
*** k_mouza has quit IRC | 10:59 | |
openstackgerrit | Pierre Riteau proposed openstack/bifrost stable/queens: CI: fetch all required projects in bifrost-base https://review.opendev.org/734059 | 11:03 |
*** yaawang has quit IRC | 11:05 | |
*** yaawang has joined #openstack-ironic | 11:05 | |
*** k_mouza has joined #openstack-ironic | 11:17 | |
openstackgerrit | Rachit Kapadia proposed openstack/ironic master: DRAC: Fix a failure to create virtual disk https://review.opendev.org/730725 | 11:18 |
dtantsur | TheJulia, JayF, rpittau|bbl, the deployment without power credentials idea we talked about on the PTG: https://storyboard.openstack.org/#!/story/2007771 | 11:19 |
openstackgerrit | Merged openstack/ironic stable/ussuri: Increase callback timeout https://review.opendev.org/731599 | 11:41 |
openstackgerrit | Merged openstack/ironic stable/ussuri: Increase timeout for introspection in ironic-inspector-tempest https://review.opendev.org/731942 | 11:52 |
openstackgerrit | Verification of a change to openstack/ironic failed: change devstack ipa-builder default to BRANCHLESS_TARGET_BRANCH https://review.opendev.org/730887 | 11:59 |
*** uzumaki has joined #openstack-ironic | 12:00 | |
uzumaki | hey ironic! o/ | 12:00 |
uzumaki | i have a quick question, in the metal3 ironic deployment I see a parameter called "PROVISIONING_INTERFACE" with value "provisioning" and I'm unsure what that exactly means | 12:02 |
uzumaki | shouldn't that look like one of the conductor node provisioning interfaces? like we have in devstack? like eno3 eno4 or something? | 12:03 |
iurygregory | uzumaki, it will probably depend on your deployment | 12:06 |
iurygregory | you can override the PROVISIONING_INTERFACE | 12:06 |
iurygregory | its the interface for dnsmasq config | 12:06 |
uzumaki | i mean, the default flavor upstream in their env file has a peculiar value of "provisioning" which is weird | 12:06 |
iurygregory | they probably have a good reason for that I would say | 12:07 |
uzumaki | so my real concern is, will there be problems in provisioning if I let it setup ironic that way? I actually want to attempt to reconfigure the DHCP range in dnsmasq (which I know how) i'm just wondering if changing provisioning interface will also be required | 12:07 |
openstackgerrit | Verification of a change to openstack/ironic failed: New configuration parameter to use ipmitool retries https://review.opendev.org/731676 | 12:11 |
*** jtwill98 has quit IRC | 12:12 | |
*** jtwill98 has joined #openstack-ironic | 12:12 | |
*** rpittau|bbl is now known as rpittau | 12:13 | |
*** rh-jelabarre has joined #openstack-ironic | 12:18 | |
openstackgerrit | Merged openstack/ironic-lib stable/ussuri: Fix for latest zeroconf version https://review.opendev.org/734044 | 12:24 |
openstackgerrit | Pierre Riteau proposed openstack/bifrost stable/queens: CI: Use WORKSPACE for all dependencies https://review.opendev.org/734059 | 12:25 |
openstackgerrit | Merged openstack/ironic master: Enforce autospec in test_deploy_utils module https://review.opendev.org/733767 | 12:32 |
openstackgerrit | Merged openstack/ironic master: Enforce autospec in test_ipxe and test_pxe https://review.opendev.org/733824 | 12:33 |
openstackgerrit | Merged openstack/ironic master: Enforce autospec in test_snmp https://review.opendev.org/733891 | 12:33 |
openstackgerrit | Merged openstack/ironic-inspector master: [doc] Add misconfigured BMC channel to trouble shooting guide https://review.opendev.org/733770 | 12:33 |
*** uzumaki has quit IRC | 12:37 | |
*** deivid has joined #openstack-ironic | 12:37 | |
deivid | Hi! I'm trying to use pyghmi and I'm having some issues.. is this the right place to ask about it? | 12:37 |
dtantsur | deivid: hi, unfortunately not really. you may need to email maintainer directly. | 12:39 |
dtantsur | unless you want to talk about virtualbmc. | 12:39 |
deivid | Nope, I'll email the maintainer then | 12:40 |
deivid | thank you | 12:40 |
openstackgerrit | Riccardo Pittau proposed openstack/ironic-lib stable/train: Fix for latest zeroconf version https://review.opendev.org/734045 | 12:43 |
openstackgerrit | Riccardo Pittau proposed openstack/bifrost master: [WIP] Convert with_items to loop https://review.opendev.org/733847 | 13:00 |
*** ociuhandu has joined #openstack-ironic | 13:05 | |
*** jtwill98 has quit IRC | 13:08 | |
Qianbiao | <uzumaki> you may fire an issue on metal3, i have the same question indeed. I want to make metal3 work with an exists ironic env. but did not find a document. | 13:11 |
openstackgerrit | Dmitry Tantsur proposed openstack/ironic-python-agent master: Make the install_bootloader command asynchronous https://review.opendev.org/734076 | 13:11 |
*** ociuhandu has quit IRC | 13:12 | |
openstackgerrit | Pierre Riteau proposed openstack/bifrost stable/queens: CI: Use WORKSPACE for all dependencies https://review.opendev.org/734059 | 13:12 |
*** iurygregory is now known as iurygregory|afk | 13:20 | |
openstackgerrit | Pierre Riteau proposed openstack/bifrost stable/queens: CI: Use WORKSPACE for all dependencies https://review.opendev.org/734059 | 13:25 |
*** rloo has joined #openstack-ironic | 13:25 | |
TheJulia | good morning | 13:26 |
dtantsur | morning TheJulia | 13:26 |
rpittau | good morning TheJulia :) | 13:28 |
TheJulia | Qianbiao: When they started metal3, they explicitly chose not to support that path... even though it is possibly the most efficent path for many using ironic already. They may be willing to accept it, but the baremetal-operator may need some logic changes because it thinks it completely owns everything in the ironic it speaks to | 13:28 |
*** uzumaki has joined #openstack-ironic | 13:28 | |
*** ociuhandu has joined #openstack-ironic | 13:30 | |
*** tzumainn has joined #openstack-ironic | 13:32 | |
Qianbiao | good morning TheJulia. | 13:33 |
Qianbiao | Do u mean metal3 does not support a standalone ironic deployment env now? | 13:33 |
dtantsur | It has ironic embedded inside of it | 13:33 |
Qianbiao | dtantsur it's an ironic vm. so i thought a standalone env should be support too. | 13:34 |
Qianbiao | But i can find nothing talked about this. | 13:34 |
dtantsur | Qianbiao: they have an ML, I think metal3-dev@googlegroups.com | 13:35 |
dtantsur | you may ask there | 13:35 |
Qianbiao | ok | 13:36 |
*** ociuhandu has quit IRC | 13:38 | |
uzumaki | I'm trying to craft my own dev environment, because I want to work with bare metal machines, that's why I'm taking the painful route | 13:45 |
maelk | uzumaki the value "provisioning" in "PROVISIONING_INTERFACE" is the name of the bridge created by libvirt in this case, on which the VMs are (i)PXE booting. so you should change it if you are not using the libvirt based env, and set it to the interface you have on the provisioning network of your Baremetal machines. It might be better to discussth | 13:49 |
maelk | is in the channel cluster-api-baremetal of Kubernetes slack | 13:49 |
*** ociuhandu has joined #openstack-ironic | 13:49 | |
maelk | Qianbiao and TheJulia we have added the possibility to configure BMO to talk to an external Ironic instance. TheJulia's point still stand true though. So you can use external ironic instances with metal3, but with restrictions | 13:52 |
dtantsur | oh, TIL | 13:52 |
Qianbiao | maelk thanks for the answer. Is there a document that could help me to setup metal3 with standalone ironic env. | 13:55 |
Qianbiao | currently, i try to deploy bm with metal3-dev-env. it seems that metal3 could not talk to real baremetal due to network issue. | 13:56 |
maelk | not really :/ I was guiding the Dell guys through that, and realized that we really miss this kind of docs, but didn't have time to start writing anything | 13:57 |
*** ociuhandu has quit IRC | 13:57 | |
maelk | however, with the BMO integration in CAPM3, it will become straightforward. you'll be able to use clusterctl to setup CAPM# and BMO at once, just passing the ironic and inspector APIs urls | 13:58 |
maelk | I'll put a reminder to myself to write that bit of docs when we complete the BMO integration in CAPM3. | 13:59 |
uzumaki | maelk, oh, just as I suspected, thank you! :) Yes, I just switches laptops so I'm setting up slack here. Thanks again! | 13:59 |
openstackgerrit | Tzu-Mainn Chen proposed openstack/ironic master: Allow node vif attach to specify port_uuid or portgroup_uuid https://review.opendev.org/731780 | 13:59 |
rpioso | Good morning, ironic | 13:59 |
Qianbiao | maelk that would be great. | 14:00 |
Qianbiao | hello uzumaki, you can join metal3 slack ? | 14:00 |
Qianbiao | when i try to join, it ask for a email that i could not provide. | 14:00 |
TheJulia | maelk: awesome, just hoping some of those issues and the whole conceptual ownership of everything becomes a setting at some point | 14:00 |
rpioso | dtantsur: When you have a chance, please review https://review.opendev.org/#/c/731644/ | 14:01 |
patchbot | patch 731644 - ironic - Fix Redfish handle no continuous override boot src - 4 patch sets | 14:01 |
uzumaki | I think you need the k8s SIG invite link for that, once you're in, simply join the channel | 14:01 |
maelk | Qianbiao https://slack.k8s.io/ you need to first add your email here | 14:01 |
Qianbiao | thanks. will try. | 14:01 |
Qianbiao | I am not familiar with slack. It's blocked in china. | 14:02 |
rpioso | dtantsur: In addition to fixing an issue in the original workaround, when it's backported to stable/train, it will fix the reference to optional import of sushy issue pas-ha reported. | 14:02 |
*** Qianbiao has quit IRC | 14:03 | |
*** Qianbiao has joined #openstack-ironic | 14:05 | |
*** priteau has joined #openstack-ironic | 14:05 | |
dtantsur | I will, trying to get there | 14:06 |
*** deivid has quit IRC | 14:07 | |
*** diurnalist has joined #openstack-ironic | 14:11 | |
rpioso | dtantsur: Thank you so much! | 14:12 |
rpioso | And entirely understand :-) | 14:12 |
*** ravsingh has quit IRC | 14:16 | |
* TheJulia wonders if more items are going to appear on the meeting agenda this morning https://wiki.openstack.org/wiki/Meetings/Ironic#Agenda_for_next_meeting | 14:20 | |
* TheJulia sees efnet is down and blinks | 14:24 | |
*** ociuhandu has joined #openstack-ironic | 14:25 | |
dtantsur | - Failed: 35 | 14:30 |
dtantsur | ugh, a lot of fun ahead | 14:30 |
TheJulia | inband deploy steps? | 14:30 |
dtantsur | yeaaah | 14:30 |
dtantsur | I got it working, now I need to fix ALL THE UNIT TESTS | 14:31 |
TheJulia | Yeaaaah | 14:31 |
TheJulia | ugh | 14:31 |
*** jawad_axd has quit IRC | 14:31 | |
dtantsur | I think I'm on the final approach though | 14:32 |
dtantsur | just need to make some bits come together (and fix unit tests for them) | 14:32 |
*** ociuhandu has quit IRC | 14:33 | |
*** ociuhandu has joined #openstack-ironic | 14:34 | |
TheJulia | priteau: Greetings, regarding https://review.opendev.org/#/c/734059/ It looks like it is directly against stable/queens. Is this something that is not applicable in other branches? | 14:34 |
patchbot | patch 734059 - bifrost (stable/queens) - CI: Use WORKSPACE for all dependencies - 4 patch sets | 14:34 |
dtantsur | mount: /tmp/tmp1gcfuru0/dev: mount point does not exist | 14:36 |
*** iurygregory|afk is now known as iurygregory | 14:36 | |
dtantsur | do we have any clues? | 14:36 |
dtantsur | wait, why is the partition job trying local boot? Oo | 14:37 |
priteau | Hi TheJulia. rocky and newer have been migrated to Zuul v3 native jobs and are already using a similar config | 14:37 |
TheJulia | dtantsur: partition jobs should always local boot, partition no longer implies network booting by default | 14:37 |
priteau | I'll tune my patch to make it similar to the current bifrost-jobs.yaml | 14:37 |
dtantsur | TheJulia: not in our CI where we use cirros | 14:37 |
TheJulia | this is true, it shouldn't a fatal failure | 14:38 |
TheJulia | We're always going to fail on that not existing but it should otherwise be bootable | 14:38 |
openstackgerrit | Dmitry Tantsur proposed openstack/ironic-lib stable/ussuri: Add netboot to all partition jobs https://review.opendev.org/734102 | 14:38 |
TheJulia | I thought we fixed that | 14:38 |
dtantsur | TheJulia: ^^^ | 14:38 |
TheJulia | oh, That would do it | 14:38 |
dtantsur | we cannot fix it, cirros doesn't have anything in its root partition | 14:38 |
dtantsur | unless we switch away from cirros.. | 14:38 |
TheJulia | no, I thought we had code to also see "oh, this is bootable" and boot it too | 14:38 |
* iurygregory is back | 14:38 | |
TheJulia | did you bring coffee? | 14:39 |
TheJulia | maybe coffee cake or cheesecake? :) | 14:39 |
dtantsur | TheJulia: this is for whole disk images, I think? | 14:39 |
TheJulia | dtantsur: yeah... it is | 14:39 |
* TheJulia orders a new brain | 14:39 | |
dtantsur | oh, do they have them on amazon? I'd get one myself as well | 14:40 |
* uzumaki thinks the new brain delivery might be delayed because or Covid-19 | 14:40 | |
iurygregory | I have coffee XD | 14:40 |
uzumaki | dtantsur, I think you should be more concerned about the guy/girl who will be installing that new brain in your head | 14:41 |
TheJulia | lol | 14:41 |
dtantsur | uzumaki: my wife is a chemist, she can manage it | 14:41 |
uzumaki | i REALLY hope so | 14:41 |
dtantsur | *shrug* you won't notice much of a difference if she fails | 14:41 |
uzumaki | that is just so sad and true | 14:42 |
openstackgerrit | Verification of a change to openstack/ironic failed: devstack: increase concurrency and worker timeout for sushy-tools https://review.opendev.org/733957 | 14:43 |
iurygregory | dtantsur, lol | 14:43 |
dtantsur | c'mon, unstable unit tests jobs is just too much for my already quite limited sanity | 14:43 |
openstackgerrit | Pierre Riteau proposed openstack/bifrost stable/queens: CI: Use WORKSPACE for all dependencies https://review.opendev.org/734059 | 14:44 |
* dtantsur git rm -r ironic/tests && git commit -m "Go away" && git push -f | 14:44 | |
*** k_mouza has quit IRC | 14:44 | |
TheJulia | oh my | 14:44 |
* uzumaki git rebase dtantsur/master --fast-forward dtantsur/new-brain --force --please-do-it-quickly | 14:45 | |
dtantsur | :D | 14:45 |
* iurygregory agrees with dtantsur | 14:45 | |
*** cdearborn has joined #openstack-ironic | 14:46 | |
*** ociuhandu has quit IRC | 14:49 | |
*** kaifeng has joined #openstack-ironic | 14:51 | |
*** beekneemech is now known as bnemec | 14:54 | |
*** k_mouza has joined #openstack-ironic | 14:56 | |
openstackgerrit | Julia Kreger proposed openstack/ironic master: WIP Add a driver compatability matrix https://review.opendev.org/733716 | 14:57 |
iurygregory | o/ | 15:00 |
iurygregory | ops too fast | 15:00 |
TheJulia | #startmeeting ironic | 15:00 |
openstack | Meeting started Mon Jun 8 15:00:18 2020 UTC and is due to finish in 60 minutes. The chair is TheJulia. Information about MeetBot at http://wiki.debian.org/MeetBot. | 15:00 |
openstack | Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. | 15:00 |
iurygregory | o/ | 15:00 |
*** openstack changes topic to " (Meeting topic: ironic)" | 15:00 | |
openstack | The meeting name has been set to 'ironic' | 15:00 |
ajya | o/ | 15:00 |
dtantsur | o/ | 15:00 |
rpittau | o/ | 15:00 |
TheJulia | 1 point goes to iurygregory for being ready to start the meeting :) | 15:00 |
kaifeng | o/ | 15:00 |
TheJulia | \o | 15:00 |
erbarr | o/ | 15:00 |
rloo | o/ | 15:00 |
dtantsur | iurygregory++ :D | 15:00 |
cdearborn | o/ | 15:00 |
iurygregory | =) | 15:00 |
TheJulia | Good morning, time for our ironic meeting! | 15:00 |
TheJulia | Our agenda is on the wiki, you can find it at the next link | 15:01 |
TheJulia | #link https://wiki.openstack.org/wiki/Meetings/Ironic#Agenda_for_next_meeting | 15:01 |
rajinir | o/ | 15:01 |
TheJulia | #topic Announcements / Reminders | 15:01 |
*** openstack changes topic to "Announcements / Reminders (Meeting topic: ironic)" | 15:01 | |
TheJulia | I have two items on the agenda for annoucements/reminders | 15:01 |
TheJulia | The first is the priorities change set in ironic-specs. I'd like to get this wrapped up this week, so reviews would be super helpful. | 15:02 |
TheJulia | #link https://review.opendev.org/#/c/720100/ | 15:02 |
patchbot | patch 720100 - ironic-specs - WIP - Victoria Cycle Priorit(y|ies) - 6 patch sets | 15:02 |
TheJulia | The other is PTG notes, which can be found at... | 15:02 |
TheJulia | #link https://meetpad.opendev.org/etherpad/p/ironic-ptg | 15:02 |
*** stendulker has joined #openstack-ironic | 15:02 | |
TheJulia | Does anyone have anything else to announce or remind us of? | 15:02 |
stendulker | o/ | 15:02 |
TheJulia | \o | 15:02 |
dtantsur | we need to do some math | 15:03 |
dtantsur | and figure out when we're doing the next release | 15:03 |
TheJulia | dtantsur: on the priorities document | 15:03 |
TheJulia | already | 15:03 |
TheJulia | :) | 15:03 |
dtantsur | wow | 15:03 |
dtantsur | okay, I've publicly admitted that I haven't read it in a whole. oops. | 15:03 |
TheJulia | lol | 15:03 |
* dtantsur will correct that | 15:03 | |
TheJulia | reviews ++ | 15:04 |
rloo | wrt priorities. it was discussed at ptg, about 'lazy consensus' and some time period. Did we decide? If not, shall we decide now? 1 week after PTG or something like that? | 15:04 |
dtantsur | in API SIG we did it this way: | 15:04 |
TheJulia | rloo: I was thinking by the next meeting at worst, tbh | 15:04 |
dtantsur | when the core team (2-4 ppl) is pleased with a document, we announce it and the deadline | 15:04 |
dtantsur | after the deadline the change is merged if no objections | 15:05 |
TheJulia | Regarding our next topic, we have no action items from the prior meeting | 15:05 |
dtantsur | for us it could be something like once 3x +2 are there, merge in 1 week | 15:05 |
TheJulia | dtantsur: That seems reasonable | 15:05 |
rloo | dtantsur: is taht the entire core team or majority or ? | 15:05 |
dtantsur | rloo: API SIG has had 4 cores top during its whole history | 15:05 |
dtantsur | only 2 nowdays | 15:06 |
TheJulia | so it is more of an external feedback mechanism in then | 15:06 |
TheJulia | which also makes sense | 15:06 |
rloo | dtantsur: so 3x +2 is 3 core members regardless of total number of cores. | 15:06 |
dtantsur | for us - yes | 15:06 |
rloo | our existing policy is agreement from all cores. | 15:06 |
dtantsur | right, this is what lazy consensus is about | 15:06 |
dtantsur | once 3x+2 are there, we invoke it (with an announce here/on the ML) | 15:07 |
dtantsur | if no objections are recorded, say, in a week, merge it | 15:07 |
TheJulia | Perhaps we should discuss it during the discussion topic | 15:07 |
dtantsur | yeah, sorry | 15:07 |
openstackgerrit | Mark Goddard proposed openstack/bifrost master: Fix bifrost_inventory.py with node nodes enrolled https://review.opendev.org/734115 | 15:07 |
iurygregory | if we will need to wait for all cores we will have problems =) | 15:07 |
TheJulia | no worries! | 15:07 |
TheJulia | iurygregory: yeah... That is what I'm seeking to avoid | 15:07 |
openstackgerrit | Mark Goddard proposed openstack/bifrost master: Fix bifrost_inventory.py with no nodes enrolled https://review.opendev.org/734115 | 15:07 |
iurygregory | agree | 15:07 |
TheJulia | So crazy thought, do we even need to visit subteam status reports this week? | 15:08 |
TheJulia | since the priority document is up in review? | 15:08 |
dtantsur | I've put some stuff about deploy steps | 15:08 |
dtantsur | no need to spend 10 mins on it though :) | 15:08 |
TheJulia | Then lets jump directly to priorities for the coming week. The review list is huge | 15:08 |
dtantsur | to put it mildly | 15:09 |
TheJulia | yeah.... | 15:09 |
TheJulia | We also haven't held a meeting in a little while | 15:09 |
TheJulia | #topic Deciding on priorities for the coming week | 15:10 |
*** openstack changes topic to "Deciding on priorities for the coming week (Meeting topic: ironic)" | 15:10 | |
TheJulia | #link https://etherpad.opendev.org/p/IronicWhiteBoard | 15:10 |
*** ociuhandu has joined #openstack-ironic | 15:10 | |
TheJulia | Starting at line 130, we have a number of items that merged... and we have a huge number of items that could use reviews and or feedback. | 15:10 |
TheJulia | So any reviews this week would be super appreciated by contributors | 15:10 |
* TheJulia removes the merged items from the list | 15:10 | |
dtantsur | remove deployment API for now, it's not finished | 15:10 |
dtantsur | * let's | 15:10 |
TheJulia | ack | 15:11 |
TheJulia | wow, that was a lot | 15:11 |
TheJulia | Now, new items for this week starting at line 183. If there any objections, now would be good to raise them | 15:12 |
dtantsur | we need to move "for the next meeting" to the bottom, right? | 15:13 |
TheJulia | basically, but I'm cleaning it up now since there may be duplicates | 15:13 |
TheJulia | A lot of the items on that list are fairly small to medium patches | 15:15 |
TheJulia | so some time reviewing would be good this week. | 15:15 |
TheJulia | Also a number of those items are documentation improvements | 15:16 |
TheJulia | Questions/objections? | 15:16 |
TheJulia | Things to add? | 15:16 |
*** gyee has joined #openstack-ironic | 15:17 | |
dtantsur | I hope we can manage it :) | 15:17 |
dtantsur | I'll be out Thu-Fri | 15:17 |
TheJulia | ack | 15:17 |
dtantsur | (the whole RH is out on Fri IIUC) | 15:17 |
rpittau | it looks ok, probably at the limit | 15:17 |
TheJulia | looks like we got some duplicates in there too | 15:17 |
* TheJulia tires to clean some of that up | 15:18 | |
openstackgerrit | Mark Goddard proposed openstack/bifrost master: Fix bifrost_inventory.py with no nodes enrolled https://review.opendev.org/734115 | 15:18 |
TheJulia | I concur, it makes up for last week :) | 15:20 |
TheJulia | Well, I'll take this as lazy consensus and we can move on! | 15:20 |
dtantsur | ++ | 15:20 |
TheJulia | #topic Discussion | 15:20 |
*** openstack changes topic to "Discussion (Meeting topic: ironic)" | 15:20 | |
TheJulia | I added one topic, "What did everyone think of the virtual PTG?" | 15:20 |
TheJulia | Comments/suggestions/ideas? | 15:21 |
dtantsur | + Productive, easy to attend (for me) | 15:21 |
dtantsur | - Missing evenings together | 15:21 |
TheJulia | I really like we had a brainstorming session | 15:21 |
dtantsur | + Inclusive (towards people who cannot travel) | 15:21 |
iurygregory | it was awesome, but i have the feeling it requires more energy =) | 15:21 |
TheJulia | Indeed. Some technical problems though | 15:21 |
dtantsur | it was the first time | 15:22 |
TheJulia | Our two hour time boxing might have been a little too little at least for the first day or two | 15:22 |
rpittau | it felt a bit "compressed" | 15:22 |
TheJulia | That is a good way to put it | 15:22 |
mgoddard | sad that I missed it. Clashed with kolla :( | 15:22 |
*** priteau has quit IRC | 15:22 | |
dtantsur | + Having normal work time in-between allowed follow-uping | 15:22 |
TheJulia | There was no topic evolution, no "oh, we should talk about xyz too! Schedule it!" | 15:22 |
TheJulia | Yeah, normal work was also a source of interrupts | 15:23 |
TheJulia | I felt super drained every day, but a lot is going on right now | 15:23 |
TheJulia | Well, if there is nothing else! Time to move on to the BareMetal SIG | 15:23 |
rloo | - hallway chatter with ironic and non-ironic folks so felt more like 'only ironic' | 15:23 |
TheJulia | yeah, there was pretty much no hallway chatter | 15:23 |
rpittau | yeah, less interaction with other groups | 15:23 |
TheJulia | cross-project communication was basically non-existant | 15:24 |
rloo | i guess 'missing community interaction' ? | 15:24 |
TheJulia | except in the pre-defined verticals | 15:24 |
rloo | but maybe pros outweigh cons. | 15:24 |
TheJulia | I concur, and I think everything said so far echos what my feedback to the foundation contained | 15:24 |
dtantsur | well, we don't know when large-scale travel is enabled again... | 15:25 |
rpittau | - no maple syrup :/ | 15:25 |
dtantsur | - and no mountain views from the venue | 15:25 |
* rloo gets some from fridge and passes to rpittau | 15:25 | |
TheJulia | and sea planes, and the Tap & Barrel | 15:25 |
rpittau | rloo: thanks for the thought :) | 15:26 |
TheJulia | #LongingForTheVancouverConventionCenter | 15:26 |
dtantsur | I'd really, really love the Summit in Berlin to happen, but.. | 15:26 |
TheJulia | ditto | 15:26 |
* iurygregory never went to Vancouver =( | 15:26 | |
TheJulia | Anyway, onward to the SIG | 15:26 |
TheJulia | #topic BareMetal SIG | 15:26 |
*** openstack changes topic to "BareMetal SIG (Meeting topic: ironic)" | 15:26 | |
TheJulia | Looks like arne_wiebalck is pushing things forward (Thanks Arne!) | 15:26 |
TheJulia | A doodle has been posted for a final review of the whitepaper | 15:27 |
TheJulia | #link https://doodle.com/poll/afwgy9zs8fi55wqe | 15:27 |
* arne_wiebalck wakes up | 15:27 | |
TheJulia | arne_wiebalck: is there anything else? | 15:27 |
dtantsur | good morning arne_wiebalck :) | 15:27 |
arne_wiebalck | I don't think so, I hope we can get things into a final state on Wed. | 15:27 |
* iurygregory saw an email from arne_wiebalck to the ml | 15:27 | |
arne_wiebalck | (assuming we meet on Wed) | 15:28 |
*** k_mouza has quit IRC | 15:28 | |
dtantsur | those of us from RH have a conflicting meeting on Tue IIRC | 15:28 |
*** k_mouza has joined #openstack-ironic | 15:28 | |
arne_wiebalck | ok, Wed seems fine so far ... TheJulia ? | 15:28 |
*** iurygregory has quit IRC | 15:29 | |
TheJulia | opening other calendar | 15:29 |
*** iurygregory has joined #openstack-ironic | 15:29 | |
*** k_mouza has quit IRC | 15:29 | |
*** k_mouza has joined #openstack-ironic | 15:29 | |
rpittau | I believe wed is the only option for us | 15:29 |
TheJulia | Wednesday works nicely | 15:30 |
arne_wiebalck | ok, I will close the doodle and sent out an invite | 15:30 |
arne_wiebalck | *send | 15:30 |
TheJulia | ok | 15:30 |
TheJulia | Then onward to RFE review? | 15:31 |
dtantsur | ++ | 15:31 |
TheJulia | #topic RFE Review | 15:31 |
*** openstack changes topic to "RFE Review (Meeting topic: ironic)" | 15:31 | |
TheJulia | dtantsur has two topics for us to discuss in terms of RFEs | 15:31 |
* TheJulia gives dtantsur the stage | 15:31 | |
dtantsur | thx :) | 15:31 |
dtantsur | #link https://storyboard.openstack.org/#!/story/2007771 [dtantsur] Limited deployment capabilities without BMC credentials | 15:32 |
dtantsur | so, this stems from the never-ending (at least on our side) requests to have at least $something working without the power control | 15:32 |
dtantsur | I've noticed that we pretty much have all the bits in place and with a few fixes we can define a flow that will work | 15:32 |
TheJulia | I read this first one and it seemed logical, the very last part seemed a little vague, but it seems reasonable to me | 15:32 |
dtantsur | this RFE is about trying it, fixing lose ends and documenting | 15:33 |
rpittau | I had a quick look and it seems ok to me | 15:33 |
TheJulia | seems really reasonable, since yeah, the bits are all there really | 15:33 |
*** ociuhandu has quit IRC | 15:34 | |
* dtantsur hears nice June crickets | 15:35 | |
TheJulia | heh | 15:36 |
TheJulia | next rfe? | 15:36 |
kaifeng | after initial deployment, someone would need to change the boot order and boot into disk? | 15:36 |
dtantsur | #link https://storyboard.openstack.org/#!/story/2007610 [dtantsur] Networking boot fallback for ironic | 15:36 |
rloo | i'm still reading/groking the rfe | 15:36 |
dtantsur | kaifeng: the 2nd link is actually about it | 15:36 |
rloo | but can give feedback later. | 15:36 |
rloo | seems like a bit of a hack to me, but still thinking about it | 15:36 |
dtantsur | rloo: the 2 RFEs are related, go ahead | 15:36 |
dtantsur | it is a bit of a hack, yes, but a hack that a lot of people will likely appreciate | 15:36 |
rloo | dtantsur: oh, i will / should read both first then. sorry, i can't give good feedback on it in this meeting. (cuz i'm slow) | 15:37 |
TheJulia | turns out a lot of people want slightly different workflows | 15:37 |
dtantsur | no worries, the goal is to highlight the RFEs | 15:37 |
dtantsur | not to make all possible decisions here and now | 15:37 |
dtantsur | the 2nd RFE is about the boot order indeed | 15:37 |
dtantsur | in addition to RFE #1, there are two more cases where we cannot assert the boot order the way we want: SNMP and Redfish | 15:38 |
*** eliaswimmer has joined #openstack-ironic | 15:38 | |
dtantsur | the latter is affected by the lack of (a supported) way to configure persistent boot on some hardware | 15:38 |
dtantsur | the idea is simple (and I've learned today foreman does it too): | 15:38 |
dtantsur | generate PXE environment for active nodes with local boot, so that if they boot from network, we tell them to boot from disk | 15:38 |
TheJulia | I feel a deploy interface cfeature compatability matrix coming soon to a review queue near you | 15:39 |
kaifeng | the 2nd one seems reasonable to me for I just hit a machine failed to perform a uefi local boot today :) | 15:39 |
dtantsur | heh | 15:39 |
TheJulia | s/cfeature/feature/ | 15:39 |
TheJulia | kaifeng: as in the efibootmgr failed to set things up? | 15:39 |
iurygregory | " if they boot from network, we tell them to boot from disk" O.o | 15:39 |
dtantsur | essentially, an iPXE file with 'sanboot' | 15:40 |
dtantsur | we do it for whole disk images with netboot already | 15:40 |
kaifeng | TheJulia: I think it's a bmc firmware issue, it just can't even manually change settings in the bios | 15:40 |
TheJulia | kaifeng: fun! :( | 15:40 |
TheJulia | Well, if there is nothing to discuss further on the RFEs, time for open discussion | 15:41 |
dtantsur | yep | 15:42 |
TheJulia | #topic Open Discussion | 15:42 |
*** openstack changes topic to "Open Discussion (Meeting topic: ironic)" | 15:42 | |
kaifeng | I am hitting this issue https://bugzilla.redhat.com/show_bug.cgi?id=1033739 hard :( | 15:42 |
openstack | bugzilla.redhat.com bug 1033739 in libvirt "Dnsmasq need prefix in dhcp-range for dhcpv6" [Unspecified,Closed: upstream] - Assigned to libvirt-maint | 15:42 |
kaifeng | oh no wrong link, sorry | 15:43 |
rpittau | ok :) | 15:43 |
kaifeng | https://bugzilla.redhat.com/show_bug.cgi?id=1575026 | 15:43 |
openstack | bugzilla.redhat.com bug 1575026 in dnsmasq "Can't PXE/iPXE boot with dnsmasq and DHCPv6" [Medium,Closed: wontfix] - Assigned to pemensik | 15:43 |
kaifeng | I am trying to make ironic inspector supports ipv6 discovery | 15:44 |
dtantsur | you're using RHEL 7? | 15:44 |
kaifeng | I am using centos7, this is the closest thing I googled | 15:44 |
dtantsur | is CentOS 8 an option for you? | 15:44 |
dtantsur | It has/will be fixed there | 15:45 |
dtantsur | hjensas and bfournie have had success with IPv6 and ironic-inspector, but on RHEL 8 | 15:45 |
kaifeng | the weird thing is we never hit this issue in non-ironic related area, i can't tell the difference | 15:46 |
dtantsur | IIRC the problem is with several times we do DHCP during iPXE boot | 15:47 |
TheJulia | kaifeng: There are a couple different issues, but some in part is the way dnsmasq works and how the ipxe client re-does dhcp with a different client identifier | 15:47 |
TheJulia | also, part of this is how dhcpv6 was designed | 15:47 |
* dtantsur is of opinion that IPv6 was designed to show the masses that computers are cursed | 15:48 | |
TheJulia | v4 I guess was the "just work" version | 15:48 |
TheJulia | and v6 was the "lets over-engineer!" | 15:48 |
kaifeng | we are not using ipxe, just pxe | 15:48 |
TheJulia | kaifeng: instances failing to dhcp? | 15:48 |
Qianbiao | hi rpittau do u remember the pxe could not boot issue i post days ago? | 15:49 |
Qianbiao | It seems a bit similiar to the kaifeng post issue. | 15:50 |
TheJulia | I guess we're done with today's meeting. Thanks everyone. I'm sure we'll continue to discuss IPv6 for a little while | 15:50 |
kaifeng | no, i am still trying the introspection poart | 15:50 |
kaifeng | part | 15:50 |
TheJulia | kaifeng: ramdisk failing to get an address? | 15:50 |
kaifeng | the interesting thing is first time works, and if we wait for a while, it also works, but i can't estimate how long | 15:51 |
TheJulia | hjensas: ^^^ | 15:51 |
dtantsur | this does sound like an issue with leases | 15:51 |
kaifeng | if we do two consecutive introspection, dnsmasq would complain no addresses on the second one | 15:51 |
* dtantsur suggests we wrap it up indeed and continue off-meeting | 15:51 | |
TheJulia | yeah, sounds slightly different :\ | 15:51 |
TheJulia | ++ | 15:51 |
TheJulia | Thanks everyone! | 15:51 |
TheJulia | Have a wonderful week! | 15:52 |
hjensas | introspection using neutron, or using dnsmasq standalone with a dynamic pool? | 15:52 |
TheJulia | #endmeeting | 15:52 |
*** openstack changes topic to "Bare Metal Provisioning | Status: http://bit.ly/ironic-whiteboard | Docs: http://docs.openstack.org/ironic/ | Bugs: https://storyboard.openstack.org/#!/project_group/75 | Contributors are generally present between 6 AM and 12 AM UTC, If we do not answer, please feel free to pose questions to openstack-discuss mailing list." | 15:52 | |
openstack | Meeting ended Mon Jun 8 15:52:03 2020 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 15:52 |
openstack | Minutes: http://eavesdrop.openstack.org/meetings/ironic/2020/ironic.2020-06-08-15.00.html | 15:52 |
openstack | Minutes (text): http://eavesdrop.openstack.org/meetings/ironic/2020/ironic.2020-06-08-15.00.txt | 15:52 |
openstack | Log: http://eavesdrop.openstack.org/meetings/ironic/2020/ironic.2020-06-08-15.00.log.html | 15:52 |
dtantsur | well, client ID changes on re-introspection, so until the lease expires, the same address is not given again? | 15:52 |
dtantsur | dunno, shooting in the darkness | 15:52 |
kaifeng | hjensas: just with openstack inspector dnsmasq service | 15:52 |
kaifeng | i tried slaac but i guess the uefi firmware doesn't support it | 15:53 |
hjensas | kaifeng: ok, and you have a dynamic range configured? Not static MAC-to-IP mappings in you dnsmasq config? | 15:53 |
kaifeng | a dynamic pool | 15:53 |
kaifeng | yeah, we don't know the mac address before introspection | 15:53 |
*** gyee has quit IRC | 15:53 | |
TheJulia | hmm | 15:54 |
TheJulia | sounds like one of the dnsmasq issues we've unconvered | 15:54 |
rloo | as an aside, we didn't discuss/agree on what lazy consensus meant wrt the priorities. | 15:54 |
hjensas | kaifeng: ok, the whole client-id issue with IPv6 should'nt be a problem in this case. As long as you don't run out of addresses in the pool ... | 15:54 |
rloo | i don't think we can 'impose' anything, dtantsur or any other suggestion, without some sort of agreement :-( | 15:55 |
kaifeng | i wonder if the dib release lease or not, seems like a lease issue, but my config on dnsmasq seems taking no effect | 15:55 |
dtantsur | rloo: we cannot impose anything at all, to be realistic | 15:55 |
dtantsur | I can +2 the priorities today and get reassigned tomorrow | 15:55 |
hjensas | kaifeng: do you have ndisc package available in your DIB? | 15:55 |
dtantsur | Nobody is going to expel anyone for not working on priorities | 15:56 |
TheJulia | also, we can't realistically wait 3-4 weeks to maybe reach 90% of core reviewers | 15:56 |
rloo | dtantsur: as a team/community. is what i meant. otherwise, why even bring up the topic at the ptg? the current understanding is that we get all cores to +2. | 15:56 |
kaifeng | hjensas: the ramdisk has rdisc6 command | 15:56 |
dtantsur | rloo: if we can make all cores vote within a week (and appoint a person to vote if they're absent).. | 15:57 |
rloo | dtantsur: we can suggest that and if folks agree, then fine. isn't that how we've operated in the past? | 15:57 |
kaifeng | i checked the dhcp all interfaces, it contains the ipv6 support you have added. | 15:57 |
dtantsur | with 2 months release cycles we cannot wait a month to define priorities | 15:57 |
openstackgerrit | Riccardo Pittau proposed openstack/ironic master: Enforce autospec in test_rpc_api and test_steps https://review.opendev.org/734132 | 15:57 |
dtantsur | rloo: sorry, I don't quite get what exactly you're proposing. just wait for reviews until we get 100% cores on board? | 15:57 |
rloo | dtantsur: to be clear, i am not opposed to most suggestions. what i'm opposed to is lack of process/implied something. | 15:58 |
rloo | dtantsur: i'm not proposing ANY CHANGE. You wanted some change, as did others. I don't know what that change is, or if folks agreed to it. | 15:58 |
dtantsur | Note that the priorities document is just a guidance. It's not enforced nor enforceable | 15:59 |
dtantsur | More of "hey, these are the cool ideas we can probably work on the next few months" | 15:59 |
dtantsur | If we want to reach 100% (as before), we also need some accountability for cores that hold the process | 16:00 |
rloo | dtantsur: i know what the priorities document is. Again, the agreement was that before it merged, it needed +2 from all cores. | 16:00 |
hjensas | kaifeng: ok, it should work then. When does it fail? At UEFI PXE, or does it fail to get an address after booting into the agent? | 16:00 |
*** stendulker has quit IRC | 16:00 | |
rloo | dtantsur: so i guess we'll continue with that unless there is some other proposal (to getting priorities PR merged) that is agreed upon. | 16:00 |
dtantsur | we're trying to agree on something right now :) | 16:01 |
*** lucasagomes has quit IRC | 16:01 | |
rloo | dtantsur: and i brought it up in today's meeting cuz it seemed like a simple thing to get a proposal/agreement. | 16:01 |
TheJulia | rloo: I think the disconnect is we've strived to get everyone to agree in the past. That realistically is not working anymore nor has worked for the last two cycles, so using the model of lazy consensus kind of laid out in the openstack community seems reasonable, to me at least. | 16:01 |
dtantsur | okay, so the problem: it takes too long to get all cores on board. with the virtual PTG even longer. | 16:01 |
rloo | TheJulia: dtantsur. Look, I don't disagree with lazy consensus. | 16:01 |
rloo | TheJulia: dtantsur. BUT I don't recall that the details were spelled out at the ptg. | 16:01 |
dtantsur | if we seriously want to release every 2 months (and release often has been a goal for a long time), we must ensure we get consensus quickly each time | 16:02 |
rloo | dtantsur: and again i don't disagree with that. | 16:02 |
kaifeng | hjensas: it appears that after a successful introspection, the second one would stuck at uefi pxe, which is the iaid issue | 16:02 |
dtantsur | rloo: okay, so should we ask TheJulia to write a formal proposal defining how we define priorities? | 16:02 |
dtantsur | and then make 100% cores sign off on that? | 16:02 |
*** k_mouza has quit IRC | 16:02 | |
dtantsur | note that we've historically applied lazy consensus to such a critical thing as adding new cores | 16:03 |
TheJulia | so the openstack lazy consensus model is basically meet the requirements to merge and wait for objections. | 16:03 |
rloo | dtantsur: i think we're discussing a proposal with how we merge the priorities. I don't care if it is TheJulia or someone else. I am not even sure if we need 100% cores. I suspect majority/quorum is sufficient. | 16:03 |
*** k_mouza has joined #openstack-ironic | 16:03 | |
kaifeng | hjensas: remove the lease->iaid check do helps, but we are seeking an alternative | 16:03 |
TheJulia | We seemed to reach consensus that this was viable in the PTG notes | 16:03 |
rloo | TheJulia: I don't recall what the tie period was for 'wait for objections' ? | 16:04 |
rloo | s/tie/time/ | 16:04 |
TheJulia | Generally it is a week | 16:04 |
dtantsur | so 6 cores to +2? | 16:05 |
hjensas | kaifeng: stuck at uefi pxe when re-introspecting the same node after it successfully introspected? | 16:05 |
rloo | TheJulia: so 'wait a maximum of one week for any objections after ... what? the PR is proposed, we get 2x +2, ??' | 16:05 |
dtantsur | in API SIG we used 2 weeks after the 1st +2 | 16:06 |
dtantsur | or 1 week? heck, we haven't had guidelines for a while.. | 16:06 |
kaifeng | hjensas: right, this is what i can see | 16:06 |
rloo | 2 weeks seems too long. | 16:06 |
TheJulia | TC I believe is a week at the requirement to merge | 16:06 |
rloo | 1 week at most. | 16:06 |
rpittau | good night! o/ | 16:08 |
*** rpittau is now known as rpittau|afk | 16:08 | |
kaifeng | hjensas: i am not sure why uefi/grub have different iaid but dnsmasq doens't complain, i am gussing the dhcp in the ramdisk has some side effect on the dnsmasq. | 16:08 |
kaifeng | *guessing | 16:09 |
* dtantsur has to go as well | 16:10 | |
hjensas | kaifeng: the iaid can be generated in a number of different ways, so UEFI and grub do it differently. It's all withing spec, so we can't blame them. | 16:10 |
*** dtantsur is now known as dtantsur|afk | 16:10 | |
dtantsur|afk | o/ | 16:10 |
hjensas | kaifeng: but the differeing iaid's should be fine with dnsmasq configured with the dynamic range. | 16:11 |
kaifeng | so this means something wrong with the dnsmasq configuration? | 16:11 |
openstackgerrit | Merged openstack/ironic stable/ussuri: change devstack ipa-builder default to BRANCHLESS_TARGET_BRANCH https://review.opendev.org/730887 | 16:12 |
kaifeng | i am using something like this: dhcp-range=2020:6:9::,2020:6:9::ffff, 64, 30m | 16:13 |
kaifeng | and with enable-ra | 16:13 |
*** k_mouza has quit IRC | 16:14 | |
openstackgerrit | Vladyslav Drok proposed openstack/ironic master: Release greenthread when computing checksum https://review.opendev.org/734136 | 16:14 |
*** k_mouza has joined #openstack-ironic | 16:14 | |
*** gyee has joined #openstack-ironic | 16:15 | |
hjensas | kaifeng: I did'nt use the RA functionality in dnsmasq. But there where some fixes related to that recently. Can you try compiling dnsmasq from source? | 16:17 |
TheJulia | vdrok: have you asked the oslo folks? | 16:18 |
hjensas | kaifeng: http://thekelleys.org.uk/gitweb/?p=dnsmasq.git;a=commit;h=515ba97595e60c762c448657f3c0e545c1e365f9, http://thekelleys.org.uk/gitweb/?p=dnsmasq.git;a=commit;h=cd672933c9b39c4275f2e8c26cef5107ad6f2c7c, http://thekelleys.org.uk/gitweb/?p=dnsmasq.git;a=commit;h=18a6bdd5414131699090d978b104e7f56ca0562a | 16:18 |
TheJulia | bnemec: It might be handy if you could take a super quick look at https://review.opendev.org/734136 | 16:20 |
patchbot | patch 734136 - ironic - Release greenthread when computing checksum - 1 patch set | 16:20 |
*** alexmcleod has quit IRC | 16:20 | |
kaifeng | hjensas: i turned this on because for a few initial runs it seems the ramdisk can't get ip addresses, i guess for a dynamic range it's not required | 16:21 |
*** alexmcleod has joined #openstack-ironic | 16:21 | |
bnemec | TheJulia: Replied on the review | 16:23 |
TheJulia | bnemec: thanks | 16:25 |
openstackgerrit | Merged openstack/ironic-inspector master: Enable Basic HTTP authentication middleware. https://review.opendev.org/729463 | 16:27 |
vdrok | TheJulia: bnemec thanks, will propose a fix to oslo instead - https://bugs.launchpad.net/oslo.utils/+bug/1882569 | 16:27 |
openstack | Launchpad bug 1882569 in oslo.utils "fileutils.compute_file_checksum does not allow greenthread switching" [Undecided,New] | 16:27 |
*** Qianbiao is now known as Qianbiao|afk | 16:27 | |
bnemec | Cool, thanks | 16:27 |
TheJulia | vdrok: awesome, thanks! | 16:29 |
*** sshnaidm is now known as sshnaidm|afk | 16:34 | |
*** k_mouza has quit IRC | 16:37 | |
TheJulia | dtantsur|afk: thinking about the rfe for boot failures, I think you might have to touch some of the code around adopt just so we treat those machines the same. We run takeover on them, but don't do any config writing to disk and ideally we should | 16:41 |
*** k_mouza has joined #openstack-ironic | 16:41 | |
*** k_mouza has quit IRC | 16:52 | |
arne_wiebalck | bye everyone o/ | 16:53 |
*** gyee has quit IRC | 16:54 | |
*** k_mouza has joined #openstack-ironic | 16:56 | |
*** dking has joined #openstack-ironic | 16:56 | |
*** gyee has joined #openstack-ironic | 16:58 | |
*** gyee has quit IRC | 17:00 | |
*** derekh has quit IRC | 17:01 | |
*** jawad_axd has joined #openstack-ironic | 17:05 | |
openstackgerrit | Merged openstack/ironic-lib stable/ussuri: Add netboot to all partition jobs https://review.opendev.org/734102 | 17:12 |
*** dougsz has quit IRC | 17:15 | |
*** k_mouza has quit IRC | 17:28 | |
openstackgerrit | Merged openstack/ironic-python-agent-builder master: Fix python-dev package mapping https://review.opendev.org/733462 | 17:37 |
openstackgerrit | Merged openstack/sushy-tools master: Simplify the CI jobs and add a vmedia+UEFI job https://review.opendev.org/733495 | 17:40 |
JayF | So maybe I'm late to the party, but I found a trick -- if you're outputting virtual consoles (such as those you'd get with devstack/libvirt machines) and are trying to read intermediate failure output, `less -R` parses the escape characters in a way that ignores erasing the lines, so you can see the progression instead of the "latest message" being overridden like when viewing ipxe results in tail | 18:01 |
JayF | or cat. | 18:01 |
*** jawad_axd has quit IRC | 18:02 | |
TheJulia | ohhh! | 18:10 |
TheJulia | there is a command for actually replaying text files with escape characters, I just ddon't remember what it is anymore | 18:11 |
TheJulia | I used it some with our logs | 18:11 |
TheJulia | came in handy when I wanted to replay at like 10 characters per second | 18:11 |
*** dustinc has joined #openstack-ironic | 18:16 | |
*** k_mouza has joined #openstack-ironic | 18:29 | |
*** kaiokmo has quit IRC | 18:30 | |
*** k_mouza has quit IRC | 18:33 | |
*** k_mouza has joined #openstack-ironic | 18:38 | |
iurygregory | for some reason I had to set VENV to /opt/stack/bifrost, even if this is our default venv O.O | 18:44 |
openstackgerrit | Verification of a change to openstack/ironic failed: devstack: increase concurrency and worker timeout for sushy-tools https://review.opendev.org/733957 | 18:46 |
*** k_mouza has quit IRC | 18:51 | |
openstackgerrit | Merged openstack/ironic master: [doc] Harmonize the prompt for bash commands https://review.opendev.org/733503 | 18:59 |
*** kaifeng has quit IRC | 19:02 | |
*** dougsz has joined #openstack-ironic | 19:08 | |
*** uzumaki has quit IRC | 19:08 | |
JayF | TheJulia: if you remember that, please tell me. less -R is borderline-miraculous, I am amazed that somehow over the years of working with Ironic I didn't discover it until now | 19:12 |
*** jawad_axd has joined #openstack-ironic | 19:19 | |
*** Lucas_Gray has joined #openstack-ironic | 19:21 | |
*** jawad_axd has quit IRC | 19:27 | |
*** Lucas_Gray has quit IRC | 19:28 | |
JayF | Has anyone tried to configure IPA to log to the console before? I'm pretty sure we should be able to get it to work with some edits to either the servicefile or the journal.conf, but I didn't want to reinvent the wheel if someone else already figured it out. | 19:59 |
*** jawad_axd has joined #openstack-ironic | 20:01 | |
TheJulia | JayF: I unfortunately don't remember what we do with the upstream ramdisks to make that happen :( | 20:07 |
JayF | if upstream ramdisks do that, it's not documented in the admin/troubleshooting guide | 20:07 |
openstackgerrit | Hervé Beraud proposed openstack/bifrost master: drop mock from lower-constraints https://review.opendev.org/734221 | 20:23 |
NobodyCam | Good Morning Ironic folks! happy Monday ;p | 20:29 |
*** jawad_axd has quit IRC | 20:31 | |
*** k_mouza has joined #openstack-ironic | 20:52 | |
*** k_mouza has quit IRC | 20:56 | |
*** dougsz has quit IRC | 21:09 | |
*** jtomasek has quit IRC | 21:18 | |
JayF | You know, https://docs.openstack.org/diskimage-builder/latest/elements/journal-to-console/README.html exists. Going to use this (or at least try it). | 21:45 |
*** gyee has joined #openstack-ironic | 22:07 | |
JayF | Can confirm that works as advertised, and outputting *all* logs to console is more useful, anyway. | 22:07 |
JayF | You know what, I should add this to the troubleshooting guide. I wonder if I have git-review setup locally... | 22:10 |
*** threestrands has joined #openstack-ironic | 22:15 | |
openstackgerrit | Jay Faulkner proposed openstack/ironic-python-agent master: Document how to output journal to console https://review.opendev.org/734339 | 22:16 |
*** threestrands has quit IRC | 22:16 | |
*** threestrands has joined #openstack-ironic | 22:17 | |
*** rcernin has joined #openstack-ironic | 22:17 | |
*** threestrands has quit IRC | 22:18 | |
*** threestrands has joined #openstack-ironic | 22:18 | |
*** threestrands has quit IRC | 22:19 | |
*** threestrands has joined #openstack-ironic | 22:20 | |
*** threestrands has quit IRC | 22:21 | |
*** threestrands has joined #openstack-ironic | 22:21 | |
*** threestrands has quit IRC | 22:22 | |
*** threestrands has joined #openstack-ironic | 22:23 | |
*** threestrands has quit IRC | 22:24 | |
*** threestrands has joined #openstack-ironic | 22:24 | |
*** threestrands has quit IRC | 22:25 | |
*** threestrands has joined #openstack-ironic | 22:26 | |
*** threestrands has quit IRC | 22:27 | |
*** threestrands has joined #openstack-ironic | 22:27 | |
*** threestrands has quit IRC | 22:28 | |
*** threestrands has joined #openstack-ironic | 22:29 | |
*** threestrands has quit IRC | 22:30 | |
*** threestrands has joined #openstack-ironic | 22:30 | |
*** tkajinam has joined #openstack-ironic | 22:49 | |
*** dmellado_ has joined #openstack-ironic | 23:10 | |
*** dmellado has quit IRC | 23:11 | |
*** dmellado_ is now known as dmellado | 23:11 | |
openstackgerrit | Hervé Beraud proposed openstack/ironic master: Use unittest.mock instead of mock https://review.opendev.org/734350 | 23:14 |
openstackgerrit | Hervé Beraud proposed openstack/ironic-inspector master: Use unittest.mock instead of mock https://review.opendev.org/734351 | 23:15 |
openstackgerrit | Hervé Beraud proposed openstack/ironic-lib master: Use unittest.mock instead of mock https://review.opendev.org/734352 | 23:15 |
openstackgerrit | Hervé Beraud proposed openstack/ironic-prometheus-exporter master: drop mock from lower-constraints https://review.opendev.org/734353 | 23:15 |
openstackgerrit | Hervé Beraud proposed openstack/ironic-python-agent master: drop mock from lower-constraints https://review.opendev.org/734354 | 23:16 |
openstackgerrit | Hervé Beraud proposed openstack/ironic-ui master: drop mock from lower-constraints and requirements https://review.opendev.org/734355 | 23:16 |
openstackgerrit | Hervé Beraud proposed openstack/networking-baremetal master: drop mock from lower-constraints https://review.opendev.org/734388 | 23:28 |
*** jdandrea has quit IRC | 23:32 | |
TheJulia | dtantsur|afk: Worth discussion tomorrow: How could we we detect/fail on a hung tcp socket at https://github.com/openstack/ironic-python-agent/blob/3fbcc7c38e858fea2279d376c60367a844ac3a04/ironic_python_agent/extensions/standby.py#L343 | 23:46 |
*** rloo has quit IRC | 23:59 |
Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!