| *** zzzeek has quit IRC | 00:13 | |
| *** zzzeek has joined #openstack-ironic | 00:15 | |
| *** zzzeek has quit IRC | 00:20 | |
| *** zzzeek has joined #openstack-ironic | 00:21 | |
| *** zzzeek has quit IRC | 00:33 | |
| *** zzzeek has joined #openstack-ironic | 00:35 | |
| *** ianychoi__ is now known as ianychoi | 00:44 | |
| *** JamesBenson has joined #openstack-ironic | 01:18 | |
| *** JamesBenson has quit IRC | 01:18 | |
| *** zzzeek has quit IRC | 01:22 | |
| *** zzzeek has joined #openstack-ironic | 01:24 | |
| *** gyee has quit IRC | 01:25 | |
| *** zzzeek has quit IRC | 01:45 | |
| *** zzzeek has joined #openstack-ironic | 01:47 | |
| *** JamesBenson has joined #openstack-ironic | 01:52 | |
| *** JamesBenson has quit IRC | 01:57 | |
| *** rloo has quit IRC | 02:01 | |
| *** k_mouza has joined #openstack-ironic | 02:25 | |
| *** k_mouza has quit IRC | 02:30 | |
| *** rcernin has quit IRC | 02:46 | |
| *** Qianbiao has joined #openstack-ironic | 02:49 | |
| *** rcernin has joined #openstack-ironic | 02:56 | |
| *** stevebaker has quit IRC | 03:16 | |
| *** VisBits_ has joined #openstack-ironic | 03:18 | |
| *** VisBits has quit IRC | 03:20 | |
| *** k_mouza has joined #openstack-ironic | 03:24 | |
| *** k_mouza has quit IRC | 03:28 | |
| *** k_mouza has joined #openstack-ironic | 03:32 | |
| *** k_mouza has quit IRC | 03:36 | |
| *** stevebaker has joined #openstack-ironic | 03:42 | |
| *** mkrai has joined #openstack-ironic | 03:44 | |
| openstackgerrit | Verification of a change to openstack/ironic failed: Remove install unnecessary packages https://review.opendev.org/751636 | 03:44 |
|---|---|---|
| *** cdearborn has quit IRC | 03:49 | |
| *** JamesBenson has joined #openstack-ironic | 03:53 | |
| *** JamesBenson has quit IRC | 03:58 | |
| *** uzumaki has joined #openstack-ironic | 04:12 | |
| openstackgerrit | Verification of a change to openstack/ironic failed: Remove install unnecessary packages https://review.opendev.org/751636 | 04:37 |
| *** mkrai has quit IRC | 04:39 | |
| *** Qianbiao has quit IRC | 04:43 | |
| *** stevebaker has quit IRC | 04:54 | |
| *** mkrai has joined #openstack-ironic | 04:54 | |
| *** k_mouza has joined #openstack-ironic | 05:18 | |
| *** rcernin has quit IRC | 05:19 | |
| *** k_mouza has quit IRC | 05:23 | |
| *** rcernin has joined #openstack-ironic | 05:27 | |
| *** rcernin has quit IRC | 05:28 | |
| *** rcernin has joined #openstack-ironic | 05:28 | |
| *** abdysn has joined #openstack-ironic | 06:31 | |
| iurygregory | good morning Ironic | 06:33 |
| janders | good morning iurygregory | 06:34 |
| iurygregory | hey janders o/ | 06:34 |
| *** Qianbiao has joined #openstack-ironic | 06:38 | |
| arne_wiebalck | Good morning iurygregory janders and ironic! | 06:45 |
| janders | hey arne_wiebalck o/ | 06:47 |
| iurygregory | hey arne_wiebalck o/ | 06:48 |
| *** Abdallahyas has joined #openstack-ironic | 07:02 | |
| Qianbiao | Morning arne_wiebalck janders iurygregory | 07:02 |
| iurygregory | hey Qianbiao o/ | 07:02 |
| arne_wiebalck | Hey Qianbiao o/ | 07:02 |
| Qianbiao | Hey folks o/ | 07:03 |
| Qianbiao | Hi iurygregory, is there a spec or sequence diagram for nova ironic driver. | 07:04 |
| *** abdysn has quit IRC | 07:05 | |
| iurygregory | no idea =) | 07:05 |
| Qianbiao | ~ ok. is nova-ironic-driver a seperated project? | 07:05 |
| iurygregory | https://opendev.org/openstack/nova/src/branch/master/nova/virt/ironic/ | 07:05 |
| Qianbiao | nice, will look at the souce code. | 07:06 |
| openstackgerrit | QianBiao Ng proposed openstack/ironic-python-agent master: Fix: make Intel CNA hardware manager none generic https://review.opendev.org/752024 | 07:11 |
| openstackgerrit | QianBiao Ng proposed openstack/ironic stable/ussuri: opt: Enhance old stable branches to use latest python-ibmcclient https://review.opendev.org/752006 | 07:17 |
| openstackgerrit | Iury Gregory Melo Ferreira proposed openstack/bifrost master: Add ipe installation to bifrost https://review.opendev.org/752042 | 07:18 |
| openstackgerrit | zhufl proposed openstack/ironic master: Fix invalid assert_has_calls https://review.opendev.org/750910 | 07:18 |
| *** afasano has joined #openstack-ironic | 07:24 | |
| Qianbiao | half an hour passes... Receiving objects: 13% (78240/583653), 36.86 MiB | 6.00 KiB/s | 07:34 |
| arne_wiebalck | You're cloning what? | 07:34 |
| Qianbiao | nova project.. | 07:34 |
| arne_wiebalck | took 38 secs here | 07:37 |
| Qianbiao | ..... | 07:37 |
| Qianbiao | You are driving me crazy now. | 07:38 |
| arne_wiebalck | sorry :) | 07:38 |
| arne_wiebalck | Just wanted to check if there is an issue at the source, it is not super snappy but not super slow either. | 07:39 |
| *** k_mouza has joined #openstack-ironic | 07:40 | |
| Qianbiao | arne_wiebalck I can never clone from opendev.com, Only from github mirror. | 07:40 |
| arne_wiebalck | Qianbiao: I used github as well. | 07:41 |
| Qianbiao | :( Should consider migrate now. | 07:42 |
| *** tosky has joined #openstack-ironic | 07:43 | |
| *** k_mouza has quit IRC | 07:44 | |
| *** mkrai has quit IRC | 07:47 | |
| iurygregory | I've always cloned from opendev.com .-. | 07:50 |
| Qianbiao | Is there an option that can add only up status ports when inspecting. | 07:54 |
| Qianbiao | current options "add_ports=active" requires a ip | 07:54 |
| Qianbiao | but i hope if a port is physical ready, then i should add it. | 07:55 |
| openstackgerrit | vinay kumar muddu proposed openstack/ironic master: Adds ilo-uefi-https boot interface to ilo5 https://review.opendev.org/745501 | 08:01 |
| *** JamesBenson has joined #openstack-ironic | 08:03 | |
| *** rcernin has quit IRC | 08:06 | |
| openstackgerrit | Iury Gregory Melo Ferreira proposed openstack/bifrost master: Add ipe installation to bifrost https://review.opendev.org/752042 | 08:08 |
| *** JamesBenson has quit IRC | 08:08 | |
| *** lucasagomes has joined #openstack-ironic | 08:09 | |
| *** Nisha_Agarwal has joined #openstack-ironic | 08:13 | |
| *** Abdallahyas has quit IRC | 08:13 | |
| Nisha_Agarwal | Hi ironic!!! | 08:13 |
| *** Abdallahyas has joined #openstack-ironic | 08:16 | |
| openstackgerrit | Merged openstack/ironic master: Remove install unnecessary packages https://review.opendev.org/751636 | 08:17 |
| *** Abdallahyas has quit IRC | 08:19 | |
| *** abdysn has joined #openstack-ironic | 08:19 | |
| *** Nisha_Agarwal has quit IRC | 08:21 | |
| *** yolanda has quit IRC | 08:22 | |
| *** k_mouza has joined #openstack-ironic | 08:22 | |
| *** yolanda has joined #openstack-ironic | 08:22 | |
| *** Nisha_Agarwal has joined #openstack-ironic | 08:25 | |
| *** k_mouza has quit IRC | 08:26 | |
| *** mkrai has joined #openstack-ironic | 08:30 | |
| *** k_mouza has joined #openstack-ironic | 08:31 | |
| *** Abdallahyas has joined #openstack-ironic | 08:32 | |
| *** dtantsur|afk is now known as dtantsur | 08:33 | |
| dtantsur | morning ironic | 08:33 |
| janders | good morning dtantsur | 08:33 |
| *** k_mouza has quit IRC | 08:35 | |
| *** abdysn has quit IRC | 08:36 | |
| iurygregory | good morning dtantsur | 08:37 |
| *** abdysn has joined #openstack-ironic | 08:37 | |
| iurygregory | quick question about the bifrost patch to add ipe, do I need to add the options that the user can override to the group_vars? | 08:37 |
| *** jtomasek_ has joined #openstack-ironic | 08:37 | |
| *** Abdallahyas has quit IRC | 08:38 | |
| *** jtomasek has quit IRC | 08:38 | |
| *** k_mouza has joined #openstack-ironic | 08:44 | |
| *** Nisha_Agarwal has quit IRC | 08:46 | |
| *** derekh has joined #openstack-ironic | 08:53 | |
| Qianbiao | Morning dtantsur | 09:07 |
| dtantsur | iurygregory: I don't think so.. we haven't been doing it consistently, and honestly I think we should leave only critical options there. | 09:08 |
| iurygregory | dtantsur, cool =) | 09:08 |
| Qianbiao | dtantsur do u know a method to accomplishment this story: https://storyboard.openstack.org/#!/story/2008149 | 09:10 |
| dtantsur | Qianbiao: why doesn't all work for you? | 09:12 |
| Qianbiao | my server got 6 eth port, 2 ib port. | 09:12 |
| Qianbiao | then it means it will create 8 port in neutron for single server | 09:12 |
| Qianbiao | I hope it only create physical link ready ports. | 09:13 |
| Qianbiao | those port link not ready has no meaning indeed. | 09:13 |
| dtantsur | do we have a reliable way to determine if a port is ready? | 09:14 |
| dtantsur | I think that was a conundrum the last time | 09:14 |
| *** ociuhandu has joined #openstack-ironic | 09:15 | |
| Qianbiao | dtantsur maybe link status? | 09:15 |
| dtantsur | I don't think it is quite reliable | 09:16 |
| openstackgerrit | Merged openstack/bifrost master: Force updating repos with copy_from_local_path https://review.opendev.org/750617 | 09:17 |
| openstackgerrit | Merged openstack/bifrost master: Re-download checkums on upgrades and updates https://review.opendev.org/750665 | 09:19 |
| janders | Qianbiao dtantsur link status might be even more "interesting" with IB | 09:20 |
| janders | as there is the _INIT_ state when the physical layer is up but the node is syncing with the subnet manager | 09:21 |
| janders | not sure how that would be interpreted and the INIT state can hang around for some time (up to tens of seconds) | 09:21 |
| Qianbiao | janders yes, i notice that too. | 09:22 |
| Qianbiao | janders do you manage the IB through neutron driver, i got some design question about combine IB partition with neutron ml2 driver. | 09:24 |
| openstackgerrit | zhufl proposed openstack/ironic master: Fix invalid assert_has_calls https://review.opendev.org/750910 | 09:29 |
| *** Lucas_Gray has joined #openstack-ironic | 09:31 | |
| openstackgerrit | Jacob Anders proposed openstack/ironic master: Generic way to configure clean step priorites https://review.opendev.org/744117 | 09:33 |
| *** abdysn has quit IRC | 09:35 | |
| *** abdysn has joined #openstack-ironic | 09:38 | |
| *** ociuhandu_ has joined #openstack-ironic | 09:52 | |
| *** Abdallahyas has joined #openstack-ironic | 09:52 | |
| *** abdysn has quit IRC | 09:55 | |
| *** ociuhandu has quit IRC | 09:55 | |
| *** Abdallahyas has quit IRC | 10:00 | |
| *** JamesBenson has joined #openstack-ironic | 10:04 | |
| *** mkrai has quit IRC | 10:04 | |
| *** jawad_axd has joined #openstack-ironic | 10:07 | |
| *** JamesBenson has quit IRC | 10:09 | |
| *** gryf has quit IRC | 10:12 | |
| *** gryf has joined #openstack-ironic | 10:14 | |
| openstackgerrit | Iury Gregory Melo Ferreira proposed openstack/ironic master: Native zuulv3 grenade multinode multitenant https://review.opendev.org/705030 | 10:17 |
| *** Lucas_Gray has quit IRC | 10:19 | |
| *** Nisha_Agarwal has joined #openstack-ironic | 10:20 | |
| *** Lucas_Gray has joined #openstack-ironic | 10:21 | |
| tosky | \o/ | 10:21 |
| openstackgerrit | Merged openstack/ironic master: Add release note for dhcp-less deploy https://review.opendev.org/751139 | 10:22 |
| iurygregory | tosky, we will merge this week =) | 10:24 |
| *** ociuhandu_ has quit IRC | 10:45 | |
| *** ociuhandu has joined #openstack-ironic | 10:46 | |
| janders | does Ironic provide something like a "default" config (e.g. ironic.conf with most/entirety of it commented out, to give examples of allowed options)? | 10:55 |
| janders | if so, how do I add an entry to it while adding a new config option? | 10:55 |
| dtantsur | janders: `tox -egenconfig` | 11:00 |
| dtantsur | will be somewhere in etc/ironic/ironic.conf.sample | 11:00 |
| *** Nisha_Agarwal has quit IRC | 11:09 | |
| *** JamesBenson has joined #openstack-ironic | 11:15 | |
| *** JamesBenson has quit IRC | 11:17 | |
| *** JamesBenson has joined #openstack-ironic | 11:17 | |
| *** zzzeek has quit IRC | 11:27 | |
| *** zzzeek has joined #openstack-ironic | 11:30 | |
| janders | thank you dtantsur. This is what generates the config, correct? How can modify what goes inside? | 11:51 |
| dtantsur | janders: see the files in ironic/conf/ | 11:51 |
| janders | dtantsur thanks - now I got it. So if I understand correctly anything in ironic/conf/*.py that has default values set will end up in the tox-generated config file, right? | 11:53 |
| *** ociuhandu_ has joined #openstack-ironic | 12:01 | |
| *** derekh has quit IRC | 12:01 | |
| *** ociuhandu has quit IRC | 12:04 | |
| *** mkrai has joined #openstack-ironic | 12:05 | |
| *** Goneri has joined #openstack-ironic | 12:13 | |
| openstackgerrit | Yogesh Ramachandra proposed openstack/ironic master: modified error message Fix idrac-wsman bios factory-reset setting https://review.opendev.org/748696 | 12:13 |
| openstackgerrit | Yogesh Ramachandra proposed openstack/ironic master: Fix idrac-wsman bios factory-reset setting https://review.opendev.org/748696 | 12:15 |
| dtantsur | janders: well, everything. the default values will be reflected | 12:21 |
| janders | dtantsur thanks for the clarification! | 12:22 |
| openstackgerrit | vinay kumar muddu proposed openstack/ironic master: Adds ilo-uefi-https boot interface to ilo5 https://review.opendev.org/745501 | 12:23 |
| janders | see you tomorrow Ironic o/ | 12:23 |
| tosky | iurygregory: yep, it's ready to go | 12:26 |
| *** mkrai has quit IRC | 12:29 | |
| iurygregory | tosky, \o/ | 12:30 |
| *** Qianbiao has quit IRC | 12:33 | |
| *** Qianbiao has joined #openstack-ironic | 12:34 | |
| *** uzumaki has quit IRC | 12:41 | |
| openstackgerrit | Verification of a change to openstack/ironic failed: Also wipe agent token on manual power off or reboot https://review.opendev.org/751813 | 12:43 |
| openstackgerrit | Ankit Kumar proposed openstack/ironic master: Adding changes for iso less vmedia support https://review.opendev.org/752001 | 12:54 |
| *** rloo has joined #openstack-ironic | 13:18 | |
| *** ricolin has quit IRC | 13:20 | |
| *** derekh has joined #openstack-ironic | 13:20 | |
| TheJulia | yay looks like CI is happier \o/ | 13:23 |
| *** tzumainn has joined #openstack-ironic | 13:28 | |
| dtantsur | morning TheJulia, yes, it is | 13:28 |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic master: Allow configuring IPMI cipher suite https://review.opendev.org/752237 | 13:30 |
| dtantsur | I'm surprised we don't have ^^ already. cc arne_wiebalck | 13:30 |
| *** sshnaidm is now known as sshnaidm|afk | 13:32 | |
| iurygregory | thank you zuul | 13:32 |
| iurygregory | we will give you a cookie | 13:32 |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic master: Allow configuring IPMI cipher suite https://review.opendev.org/752237 | 13:33 |
| *** benj_ has quit IRC | 13:38 | |
| openstackgerrit | Yushiro FURUKAWA proposed openstack/ironic master: Fix handling OctetString for pysnmp https://review.opendev.org/750401 | 13:38 |
| *** Qianbiao has quit IRC | 13:40 | |
| *** Qianbiao has joined #openstack-ironic | 13:40 | |
| *** ricolin has joined #openstack-ironic | 13:45 | |
| TheJulia | dtantsur: comment on the cipher suite patch | 13:47 |
| TheJulia | just a thought | 13:47 |
| dtantsur | mmm, lemme find the bugzilla | 13:48 |
| TheJulia | it is stalled/hanging ipmitool commands | 13:48 |
| *** k_mouza has quit IRC | 13:49 | |
| dtantsur | no, it's another issue (although it may help with that one too) | 13:49 |
| TheJulia | fun! | 13:49 |
| TheJulia | I saw the bugzilla ?yesterday? or the day before but don't remember the exact error | 13:50 |
| *** benj_ has joined #openstack-ironic | 13:51 | |
| dtantsur | https://bugzilla.redhat.com/show_bug.cgi?id=1873614 | 13:51 |
| openstack | bugzilla.redhat.com bug 1873614 in ipmitool "[OSP16][RHEL8.2] some nodes do not support Cipher Suite 17 in lanplus mode" [Unspecified,New] - Assigned to vdolezal | 13:51 |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic master: Allow configuring IPMI cipher suite https://review.opendev.org/752237 | 13:52 |
| dtantsur | TheJulia: ^^ | 13:52 |
| dtantsur | good point re adding error messages, makes it much easier to google problems | 13:52 |
| * arne_wiebalck is in a meeting rn, will check later | 13:53 | |
| TheJulia | dtantsur: thanks! | 13:53 |
| *** k_mouza has joined #openstack-ironic | 13:56 | |
| openstackgerrit | Verification of a change to openstack/ironic failed: Do not assume that prepare_image is the last command to run https://review.opendev.org/749913 | 14:02 |
| iurygregory | funny 3 or 17 LOL | 14:02 |
| TheJulia | Qianbiao: FYI, I left you a message on 752024 | 14:05 |
| Qianbiao | TheJulia ok will checking a bit later | 14:06 |
| TheJulia | Slightly concerned by the behavior change and would like to see a test verifying it is working as noted | 14:06 |
| TheJulia | Qianbiao: no worries, have a wonderful evening | 14:06 |
| dtantsur | iurygregory: I proudly present you cipher 0: http://fish2.com/ipmi/cipherzero.html | 14:12 |
| iurygregory | dtantsur, ty! | 14:12 |
| dtantsur | and yes, you can find the whole listing in the spec, but only 3 and 17 use sane combinations (AES+HMAC+SHA*) | 14:12 |
| iurygregory | gotcha! | 14:12 |
| iurygregory | the code looks goot to me | 14:13 |
| Qianbiao | <TheJulia> sorry, being later, in a meeting. | 14:19 |
| Qianbiao | i see your comment. i am not sure whether there is a testcase again this situation in common dispatch_manager | 14:20 |
| Qianbiao | function. I will try to add one for cna a bit later. | 14:20 |
| Qianbiao | Indeed the code has been test in our test lab env. | 14:21 |
| TheJulia | Qianbiao: awesome | 14:23 |
| Qianbiao | I got a simple patch for ibmcclient lib upgrade, may cores has a minute review it. https://review.opendev.org/#/c/752006/ | 14:23 |
| patchbot | patch 752006 - ironic (stable/ussuri) - opt: Enhance old stable branches to use latest pyt... - 4 patch sets | 14:23 |
| TheJulia | if there is some sort of similar case that you find, just let me know. | 14:23 |
| Qianbiao | TheJulia sure, will do it now. | 14:23 |
| TheJulia | Qianbiao: that is quite a version jump in requirements :( | 14:24 |
| Qianbiao | <TheJulia> yes, do u remember you modified that version :) | 14:25 |
| Qianbiao | check master, the version is just the same as master branch | 14:25 |
| Qianbiao | new version | 14:25 |
| *** priteau has joined #openstack-ironic | 14:30 | |
| Qianbiao | Hi, TheJulia, check this: https://github.com/openstack/ironic-python-agent/blob/master/ironic_python_agent/tests/unit/test_multi_hardware.py#L147-L154 | 14:32 |
| dtantsur | Qianbiao: you shouldn't raise the lower limit | 14:32 |
| dtantsur | changing from >=0.1.0,<0.2.0 to >=0.1.0,<0.3.0 is fine (but you need to make sure 0.1.0, 0.2.0 and 0.3.0 all work) | 14:32 |
| Qianbiao | <dtantsur> the sad thing is 0.2.1 of ibmclient will break things.. | 14:32 |
| TheJulia | I seem to remember it was 0.2.0 had six and 0.2.1 was broken. 0.2.1 could just be excluded. | 14:33 |
| dtantsur | you can exclude it by doing >=0.1.0,!=0.2.1,<0.3.0 | 14:33 |
| Qianbiao | it's 0.2.1 cause we fix the version to >=0.1.0,<0.2.0 | 14:33 |
| Qianbiao | ok | 14:33 |
| TheJulia | Yeah, I'd be cood with what dmitry proposed | 14:33 |
| TheJulia | s/cood/good/ | 14:33 |
| Qianbiao | ok, will update it now. | 14:33 |
| TheJulia | ok | 14:33 |
| Qianbiao | TheJulia, testcase is here: https://github.com/openstack/ironic-python-agent/blob/master/ironic_python_agent/tests/unit/test_multi_hardware.py#L147-L154 | 14:34 |
| TheJulia | Qianbiao: perfect | 14:35 |
| TheJulia | thanks | 14:35 |
| Qianbiao | yw | 14:35 |
| *** maelk has quit IRC | 14:36 | |
| *** fdegir has quit IRC | 14:36 | |
| *** ociuhandu_ has quit IRC | 14:36 | |
| *** Lucas_Gray has quit IRC | 14:36 | |
| *** ociuhandu has joined #openstack-ironic | 14:37 | |
| *** k_mouza has quit IRC | 14:55 | |
| *** ociuhandu_ has joined #openstack-ironic | 14:57 | |
| *** ociuhandu has quit IRC | 14:58 | |
| *** k_mouza has joined #openstack-ironic | 15:01 | |
| *** gyee has joined #openstack-ironic | 15:05 | |
| TheJulia | a relatively quick change that needs another reviewer https://review.opendev.org/#/c/749758/ | 15:10 |
| patchbot | patch 749758 - metalsmith - Rework jobs so failed devstack does not retry the job - 8 patch sets | 15:10 |
| openstackgerrit | Merged openstack/ironic master: Accept and use a TLS certificate from the agent https://review.opendev.org/747136 | 15:14 |
| dtantsur | \o/ | 15:15 |
| TheJulia | congrats! | 15:15 |
| arne_wiebalck | dtantsur: yes, we have run into the cipher issue as well | 15:15 |
| dtantsur | today I tried to understand if the way I generate the TLS certificate is compatible with FIPS 140-2. I *think* it is. But I could not understand for sure. | 15:16 |
| arne_wiebalck | dtantsur: there are multiple issues, actually, the one from the bug is the latest we found | 15:16 |
| arne_wiebalck | dtantsur: rather than setting this per node (which we considered) is to have a proxy tool which handles this | 15:16 |
| arne_wiebalck | dtantsur: so, Ironic calls our proxy ipmitool and it handles all the cases | 15:17 |
| dtantsur | wow | 15:17 |
| arne_wiebalck | dtantsur: I think the worst we found are BMCs advertising one version, but then failing when the client uses it | 15:18 |
| dtantsur | arne_wiebalck: yep, that's the bug we see | 15:18 |
| dtantsur | hence this configuration | 15:18 |
| arne_wiebalck | dtantsur: the proxy avoids to have per node configs | 15:18 |
| arne_wiebalck | dtantsur: but I think it is good to have it | 15:18 |
| dtantsur | ++ | 15:18 |
| arne_wiebalck | dtantsur: we would not have done a proxy tool if we had this at the time, I would think | 15:19 |
| arne_wiebalck | dtantsur: it also means we carry now a downstream patch ... | 15:20 |
| arne_wiebalck | dtantsur: but it avoids to try and set the config after we ran into an error | 15:20 |
| arne_wiebalck | dtantsur: because you won't know the config you need upfront | 15:21 |
| openstackgerrit | Verification of a change to openstack/ironic-python-agent failed: Generate a TLS certificate and send it to ironic https://review.opendev.org/749930 | 15:23 |
| *** Nisha_Agarwal has joined #openstack-ironic | 15:25 | |
| Nisha_Agarwal | dtantsur, ping | 15:27 |
| Nisha_Agarwal | dtantsur, i raised a defect on bifrost | 15:27 |
| Nisha_Agarwal | https://storyboard.openstack.org/#!/story/2008151 | 15:27 |
| dtantsur | Nisha_Agarwal: mmm, makes me think, we should remove the --release argument | 15:27 |
| Nisha_Agarwal | dtantsur, This defect applies to stable/train and stable/ussuri ... | 15:28 |
| dtantsur | folks should rather use stable/ussuri bifrost to install stable/ussuri | 15:28 |
| dtantsur | TheJulia: thoughts ^^? | 15:28 |
| Nisha_Agarwal | on master the ironic-api and conductor comes up successfully | 15:28 |
| TheJulia | dtantsur: yeah, I've not looked at the story but I'm going to guess that we didn't update the tag to pull from? | 15:28 |
| Nisha_Agarwal | dtantsur, but when i used stable branch bifrost, it cloned ironic of master branch | 15:29 |
| TheJulia | well, it should always clone it | 15:29 |
| TheJulia | it needs to checkout the branch | 15:29 |
| dtantsur | TheJulia: it's worse, we never care to ensure that master bifrost can install older ironics | 15:29 |
| dtantsur | (and we probably shouldn't) | 15:29 |
| TheJulia | oh, yeah, no we shouldn't support that case | 15:29 |
| TheJulia | moving targets in both directions are a bad idea | 15:30 |
| dtantsur | Nisha_Agarwal: yep, before bifrost-cli bifrost could not self-detect the branch it's running from | 15:30 |
| Nisha_Agarwal | TheJulia, but the documentation clearly says that master branch bifrost can be used to install stable branches | 15:30 |
| Nisha_Agarwal | dtantsur, yes | 15:30 |
| dtantsur | Nisha_Agarwal: could you point me at that? | 15:30 |
| Nisha_Agarwal | dtantsur, at what? | 15:30 |
| dtantsur | Nisha_Agarwal: the piece of the documentation that I need to update :) | 15:30 |
| dtantsur | because we've never really supported that in practice | 15:30 |
| TheJulia | it may have worked in cases, but realistically things change | 15:31 |
| Nisha_Agarwal | dtantsur, ok that i can give...but if we have to recommend the customer for isntalling standalone ironic of stable branch, what shud be the documentation | 15:31 |
| TheJulia | I know it sure worked in the early days.... | 15:31 |
| dtantsur | I guess we had a smaller rate of changes back than | 15:32 |
| Nisha_Agarwal | dtantsur, the fix is this http://paste.openstack.org/show/797936/ | 15:32 |
| dtantsur | well, I'd prefer not to do that, it was a useful change | 15:33 |
| TheJulia | dtantsur: yeah, the pattern wasn't really changing and adding in things like json-rpc really change the overall pattern | 15:33 |
| Nisha_Agarwal | dtantsur, but looks like ironic of previous releases doesnt come up in standalone mode | 15:33 |
| Nisha_Agarwal | the same config file works for master branch of ironic | 15:34 |
| Nisha_Agarwal | dtantsur, the other day i had asked for standalone ironic documentation which we should give to the customer....we were working on that | 15:34 |
| Nisha_Agarwal | dtantsur, and as part of that we just tried documentaion of stable releases and master releases...but when it didnt work, i used the bifrost-cli --help (and even code) to see how it takes the stable release code for ironci installation | 15:36 |
| Nisha_Agarwal | and used the command line as given | 15:36 |
| Nisha_Agarwal | dtantsur, we cannot recommend user to use the master branch ironic right now...it has to be stable release | 15:36 |
| Nisha_Agarwal | TheJulia, ^^^ | 15:36 |
| TheJulia | Nisha_Agarwal: we're literally a week or two away from that point. | 15:37 |
| Nisha_Agarwal | TheJulia, agreed but it is committed for this week :( | 15:37 |
| *** Qianbiao has quit IRC | 15:38 | |
| TheJulia | What is committed for this week? | 15:38 |
| openstackgerrit | Dmitry Tantsur proposed openstack/bifrost master: Discourage using non-matching versions of Bifrost and Ironic https://review.opendev.org/752264 | 15:38 |
| iurygregory | it should work to install releases that have the bifrost-cli available (aka master and probably the bugfix branches...) | 15:38 |
| dtantsur | documentation update ^^ | 15:38 |
| *** Qianbiao has joined #openstack-ironic | 15:38 | |
| Nisha_Agarwal | TheJulia, the documentation | 15:38 |
| *** k_mouza has quit IRC | 15:39 | |
| dtantsur | TheJulia: my patch contains explanation how to use versions of bifrost before bifrost-cli (i.e. ussuri and older) | 15:39 |
| dtantsur | ehmmm, wanted to ping Nisha_Agarwal ^^ | 15:39 |
| Nisha_Agarwal | dtantsur, then there is no way we could be using bifrost for standalone? | 15:39 |
| dtantsur | that's a broad statement, you just need to use the right parameters on stable/ussuri | 15:40 |
| TheJulia | I'm not sure a tarball would work at all unless it had the git components... | 15:40 |
| *** k_mouza has joined #openstack-ironic | 15:40 | |
| dtantsur | tarball created with 'setup.py sdist' should in theory work. never tried it. | 15:41 |
| TheJulia | bottom line, bifrost was always intended as a developer/operator tool in the realm of executing the needful tasks/actions to get the job done. | 15:42 |
| dtantsur | okay, I have a funny idea, hold on | 15:43 |
| *** stendulker_ has joined #openstack-ironic | 15:44 | |
| *** uzumaki has joined #openstack-ironic | 15:44 | |
| Qianbiao | Nisha_Agarwal TL;DR if you want a quick tool to setup standalone env for ironic. | 15:45 |
| Qianbiao | you may try this https://github.com/jriguera/ansible-ironic-standalone | 15:45 |
| Qianbiao | we use this playbook and update some minor part to support train, ussuri | 15:46 |
| Qianbiao | I did not know there is bifrost at that time(2018), only find this one. | 15:46 |
| *** jtomasek_ has quit IRC | 15:49 | |
| openstackgerrit | QianBiao Ng proposed openstack/ironic stable/ussuri: opt: Enhance old stable branches to use latest python-ibmcclient https://review.opendev.org/752006 | 15:51 |
| openstackgerrit | Dmitry Tantsur proposed openstack/bifrost master: Recover support for installing stable/ussuri https://review.opendev.org/752268 | 15:51 |
| dtantsur | maybe it's silly, dunno ^^^ | 15:51 |
| openstackgerrit | Dmitry Tantsur proposed openstack/bifrost master: Recover support for installing stable/ussuri https://review.opendev.org/752268 | 15:52 |
| Qianbiao | dtantsur TheJulia updated https://review.opendev.org/752006 | 15:56 |
| patchbot | patch 752006 - ironic (stable/ussuri) - opt: Enhance old stable branches to use latest pyt... - 5 patch sets | 15:56 |
| Qianbiao | And this: https://review.opendev.org/#/c/752024/ | 15:56 |
| patchbot | patch 752024 - ironic-python-agent - Fix: make Intel CNA hardware manager none generic - 5 patch sets | 15:56 |
| *** lucasagomes has quit IRC | 16:01 | |
| *** uzumaki has quit IRC | 16:02 | |
| Qianbiao | need to go, see you all, have a good night. | 16:06 |
| *** Qianbiao is now known as Qianbiao|afk | 16:06 | |
| *** Qianbiao|afk has quit IRC | 16:06 | |
| *** stendulker_ has quit IRC | 16:10 | |
| *** tosky has quit IRC | 16:11 | |
| *** jawad_axd has quit IRC | 16:24 | |
| *** ociuhandu has joined #openstack-ironic | 16:30 | |
| *** ociuhandu_ has quit IRC | 16:32 | |
| *** ociuhandu has quit IRC | 16:35 | |
| *** jawad_axd has joined #openstack-ironic | 16:37 | |
| *** tonyb has quit IRC | 16:41 | |
| *** tonyb has joined #openstack-ironic | 16:42 | |
| *** k_mouza has quit IRC | 16:48 | |
| *** Nisha_Agarwal has quit IRC | 16:54 | |
| *** fdegir has joined #openstack-ironic | 17:01 | |
| *** tonyb has quit IRC | 17:01 | |
| *** uzumaki has joined #openstack-ironic | 17:02 | |
| *** maelk has joined #openstack-ironic | 17:02 | |
| *** derekh has quit IRC | 17:04 | |
| *** uzumaki has quit IRC | 17:07 | |
| openstackgerrit | Verification of a change to openstack/ironic failed: Increase memory of tinyipa vms https://review.opendev.org/747633 | 17:08 |
| *** afasano has quit IRC | 17:10 | |
| *** dtantsur is now known as dtantsur|afk | 17:13 | |
| *** jawad_axd has quit IRC | 17:15 | |
| openstackgerrit | Verification of a change to openstack/ironic-inspector failed: Power off before inspection https://review.opendev.org/749845 | 17:56 |
| *** jawad_axd has joined #openstack-ironic | 18:00 | |
| *** jawad_axd has quit IRC | 18:15 | |
| openstackgerrit | Jay Faulkner proposed openstack/ironic-python-agent-builder master: Fix typo in spelling of "finalise", breaking build https://review.opendev.org/752293 | 18:35 |
| JayF | ^ something lost in translation when I moved my downstream changes upstream; I discovered it today when rebasing downstream code. Apologies, it's another embarassing typo. | 18:35 |
| iurygregory | JayF, can happen to everyone =) | 18:38 |
| *** tosky has joined #openstack-ironic | 18:51 | |
| openstackgerrit | Merged openstack/ironic master: Add redfish options to the generated docs https://review.opendev.org/751839 | 19:09 |
| openstackgerrit | Merged openstack/ironic-python-agent master: [doc] Clarify custom hardware manager inheritance https://review.opendev.org/752002 | 19:10 |
| *** priteau has quit IRC | 19:12 | |
| *** jawad_axd has joined #openstack-ironic | 19:19 | |
| openstackgerrit | Verification of a change to openstack/ironic failed: Also wipe agent token on manual power off or reboot https://review.opendev.org/751813 | 19:36 |
| *** stevebaker has joined #openstack-ironic | 19:52 | |
| *** jawad_axd has quit IRC | 20:06 | |
| openstackgerrit | Julia Kreger proposed openstack/ironic-python-agent master: WIP: don't update the grubenv https://review.opendev.org/752308 | 20:12 |
| openstackgerrit | Julia Kreger proposed openstack/metalsmith master: Replace the Centos7 image job with a Centos8 UEFI based job https://review.opendev.org/748315 | 20:13 |
| TheJulia | among the things I am conflicted over is ^^^ | 20:13 |
| * TheJulia hopes CI will quickly return results. | 20:13 | |
| *** cdearborn has joined #openstack-ironic | 20:18 | |
| openstackgerrit | Jay Faulkner proposed openstack/ironic-python-agent master: Simplify heartbeating by removing use of select() https://review.opendev.org/746774 | 20:22 |
| *** Lucas_Gray has joined #openstack-ironic | 20:25 | |
| *** zzzeek has quit IRC | 21:05 | |
| *** zzzeek has joined #openstack-ironic | 21:07 | |
| *** Lucas_Gray has quit IRC | 21:17 | |
| *** zzzeek has quit IRC | 21:22 | |
| *** zzzeek has joined #openstack-ironic | 21:25 | |
| *** rcernin has joined #openstack-ironic | 21:31 | |
| openstackgerrit | Iury Gregory Melo Ferreira proposed openstack/ironic master: Native zuulv3 grenade multinode multitenant https://review.opendev.org/705030 | 21:34 |
| *** rcernin has quit IRC | 21:36 | |
| openstackgerrit | Merged openstack/ironic-python-agent-builder master: Fix typo in spelling of "finalise", breaking build https://review.opendev.org/752293 | 21:55 |
| *** tosky_ has joined #openstack-ironic | 21:56 | |
| *** tosky has quit IRC | 21:59 | |
| *** tosky_ is now known as tosky | 22:00 | |
| *** rcernin has joined #openstack-ironic | 22:09 | |
| *** rcernin has quit IRC | 22:18 | |
| janders | good morning Ironic o/ | 22:26 |
| *** JamesBenson has quit IRC | 22:29 | |
| *** rcernin has joined #openstack-ironic | 22:32 | |
| *** rcernin has quit IRC | 22:33 | |
| *** rcernin has joined #openstack-ironic | 22:33 | |
| TheJulia | :) | 22:35 |
| stevebaker | :D | 22:37 |
| *** zzzeek has quit IRC | 22:38 | |
| *** zzzeek has joined #openstack-ironic | 22:40 | |
| openstackgerrit | Christopher Dearborn proposed openstack/ironic master: Redfish driver firmware update https://review.opendev.org/749619 | 22:50 |
| *** zzzeek has quit IRC | 22:58 | |
| *** tosky has quit IRC | 22:59 | |
| *** zzzeek has joined #openstack-ironic | 23:00 | |
| *** zzzeek has quit IRC | 23:17 | |
| *** zzzeek has joined #openstack-ironic | 23:19 | |
| *** Qianbiao has joined #openstack-ironic | 23:23 | |
| openstackgerrit | QianBiao Ng proposed openstack/ironic stable/ussuri: opt: Enhance old stable branches to use latest python-ibmcclient https://review.opendev.org/752006 | 23:30 |
| openstackgerrit | Christopher Dearborn proposed openstack/ironic master: Add support for firmware update in idrac hw type https://review.opendev.org/752344 | 23:31 |
| *** zzzeek has quit IRC | 23:43 | |
| *** zzzeek has joined #openstack-ironic | 23:44 | |
| *** zzzeek has quit IRC | 23:49 | |
| *** zzzeek has joined #openstack-ironic | 23:52 | |
| openstackgerrit | Verification of a change to openstack/ironic failed: Also wipe agent token on manual power off or reboot https://review.opendev.org/751813 | 23:54 |
| *** zzzeek has quit IRC | 23:57 | |
| *** zzzeek has joined #openstack-ironic | 23:58 | |
Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!