Thursday, 2020-10-08

*** gyee has quit IRC00:31
*** k_mouza has joined #openstack-ironic00:32
*** k_mouza has quit IRC00:36
*** anuradha1904 has quit IRC00:43
*** ianychoi__ is now known as ianychoi00:44
*** rcernin has quit IRC00:48
*** rcernin has joined #openstack-ironic00:55
*** outbrito has quit IRC01:00
*** Goneri has quit IRC01:06
*** rcernin has quit IRC01:08
*** k_mouza has joined #openstack-ironic01:17
*** JamesBenson has joined #openstack-ironic01:20
*** k_mouza has quit IRC01:22
*** rcernin has joined #openstack-ironic01:25
*** JamesBenson has quit IRC01:25
*** rcernin has quit IRC01:39
*** k_mouza has joined #openstack-ironic01:39
*** k_mouza has quit IRC01:43
*** ricolin_ has joined #openstack-ironic01:45
*** k_mouza has joined #openstack-ironic01:47
*** k_mouza has quit IRC01:51
*** tkajinam has quit IRC01:52
*** tkajinam has joined #openstack-ironic01:53
*** eandersson has quit IRC01:56
*** eandersson has joined #openstack-ironic01:57
*** JamesBenson has joined #openstack-ironic02:01
*** k_mouza has joined #openstack-ironic02:04
*** JamesBenson has quit IRC02:06
*** k_mouza has quit IRC02:09
*** rcernin has joined #openstack-ironic02:13
*** JamesBenson has joined #openstack-ironic02:14
*** k_mouza has joined #openstack-ironic02:14
*** k_mouza has quit IRC02:18
*** JamesBenson has quit IRC02:18
*** mkrai has joined #openstack-ironic02:22
*** Qianbiao has joined #openstack-ironic02:29
*** JamesBenson has joined #openstack-ironic02:39
*** JamesBenson has quit IRC02:44
*** k_mouza has joined #openstack-ironic02:48
*** k_mouza has quit IRC02:53
*** JamesBenson has joined #openstack-ironic03:05
*** JamesBen_ has joined #openstack-ironic03:09
*** JamesBenson has quit IRC03:09
*** mkrai has quit IRC03:24
*** mkrai_ has joined #openstack-ironic03:24
*** mkrai_ has quit IRC03:28
*** JamesBen_ has quit IRC04:02
*** tzumainn has quit IRC04:02
*** k_mouza has joined #openstack-ironic04:05
*** JamesBenson has joined #openstack-ironic04:07
*** ociuhandu has joined #openstack-ironic04:08
*** k_mouza has quit IRC04:09
*** JamesBenson has quit IRC04:10
*** mkrai has joined #openstack-ironic04:12
*** ociuhandu has quit IRC04:12
*** cdearborn has quit IRC04:14
*** JamesBenson has joined #openstack-ironic04:16
*** JamesBenson has quit IRC04:27
*** smutugrigg has joined #openstack-ironic04:42
*** smutugrigg35 has joined #openstack-ironic04:42
*** smutugrigg35 has quit IRC04:42
*** smutugrigg has quit IRC04:42
*** smutugrigg has joined #openstack-ironic04:43
*** anuradha1904 has joined #openstack-ironic04:45
*** JamesBenson has joined #openstack-ironic04:50
*** JamesBenson has quit IRC04:55
*** mkrai has quit IRC05:03
*** mkrai has joined #openstack-ironic05:06
*** abdysn has joined #openstack-ironic05:44
*** kafilat has joined #openstack-ironic06:01
*** smutugrigg has quit IRC06:06
*** SuiongNg|ITRI_TW has joined #openstack-ironic06:06
*** smutugrigg has joined #openstack-ironic06:11
*** uzumaki has joined #openstack-ironic06:45
arne_wiebalckGood morning, ironic o/06:58
jandersgood morning arne_wiebalck o/07:02
arne_wiebalckHey janders o/07:02
*** ociuhandu has joined #openstack-ironic07:02
iurygregorygood morning arne_wiebalck janders and Ironic07:04
jandersgood morning iurygregory o/07:06
iurygregoryjanders, o/07:07
*** mkrai has quit IRC07:08
*** mkrai_ has joined #openstack-ironic07:08
arne_wiebalckhey iurygregory o/07:12
*** mkrai_ has quit IRC07:22
*** rpittau|afk is now known as rpittau07:27
rpittaugood morning ironic! o/07:27
jandersgood morning rpittau! o/07:30
rpittauhey janders :)07:30
iurygregoryarne_wiebalck, o/07:32
*** uzumaki has quit IRC07:33
*** jamesdenton has quit IRC07:33
*** rajinir has quit IRC07:33
iurygregorymorning rpittau o/07:33
rpittauhey iurygregory :)07:33
*** uzumaki has joined #openstack-ironic07:33
* iurygregory is using the CERN mug today07:33
* arne_wiebalck guesses iurygregory's coffee tastes particle-ly good today?07:34
rpittauwhat happen if you bump the CERN mug with another one from opposite direction and same speed?07:35
iurygregoryyou will need to go to CERN to buy a new one07:36
iurygregory=P07:36
*** mgoddard has quit IRC07:36
jandersI wonder: if one spills coffee from a CERN mug, do the coffee drops in particle collision like patterns?07:37
jandersalso, is there a such thing as a CERN espresso cup?07:37
*** uzumaki has quit IRC07:38
*** jamesdenton has joined #openstack-ironic07:38
*** rajinir has joined #openstack-ironic07:38
*** mgoddard has joined #openstack-ironic07:39
iurygregoryjanders, https://twitter.com/_Skolt_Iury_/status/123269812185724108807:42
*** jtomasek_ has joined #openstack-ironic07:45
*** jtomasek has quit IRC07:45
jandersnice :)07:45
*** tosky has joined #openstack-ironic07:54
*** jtomasek_ has quit IRC07:56
*** rcernin has quit IRC07:57
*** openstackgerrit has joined #openstack-ironic08:05
openstackgerritKafilat Adeleke proposed openstack/ironic master: Changes links so we do not have "here" as the link  https://review.opendev.org/75657408:05
*** lucasagomes has joined #openstack-ironic08:12
*** Abdallahyas has joined #openstack-ironic08:16
*** Abdallahyas has quit IRC08:16
*** Abdallahyas has joined #openstack-ironic08:16
*** mkrai_ has joined #openstack-ironic08:19
*** abdysn has quit IRC08:20
smutugrigghi, I was wondering if I could ask for some guidance please?  I am an Outreachy intern applicant and am a bit lost.08:27
openstackgerritRiccardo Pittau proposed openstack/ironic-python-agent-builder master: Use tinycore 11 to build tinyipa  https://review.opendev.org/71164708:28
smutugriggI have requested to work on task 40964 under story/200819408:28
smutugriggBut would like some guidance08:28
iurygregorysmutugrigg, hello o/08:29
smutugriggI apologise in advance, this is all a bit new to me08:30
smutugrigghi iurygregory o/08:30
*** abdysn has joined #openstack-ironic08:30
iurygregorysmutugrigg, no worries =) Have you looked at the OpenStack new contributor documentation already and set up the gerrit account ?08:30
smutugriggiurygregory, yes I have and have cloned the ironic project from Git08:31
iurygregorysmutugrigg, cool!08:31
smutugriggiurygregory, yah!08:32
iurygregorysmutugrigg, so tehre are a few initial tasks that you can choose the one you want08:32
iurygregoryplease choose one and work on it, if you have any questions about what needs to be done I'm more than happy to explain to you08:33
smutugriggiurygregory, I have assigned task 40964 to myself for action08:33
*** Abdallahyas has quit IRC08:33
smutugriggCould you please help me with what I need to do please?08:33
* iurygregory refreshs the browser08:33
iurygregorysure08:33
smutugriggthanks08:34
iurygregorygive me a minute let me check the code08:34
smutugriggcool08:34
openstackgerritRiccardo Pittau proposed openstack/sushy master: [DNM] test ci with ubuntu focal  https://review.opendev.org/75666008:34
*** derekh has joined #openstack-ironic08:35
iurygregorysmutugrigg, so basically you need to look at the rst files in the ironic repository, when you find an occurence of the `openstack baremetal` command and it has --os-baremetal-api-version parameter on it you will need to remove08:39
smutugriggiurygregory, to check if I understand correctly the following text:08:41
smutugriggIRONIC_CMD="openstack baremetal"08:41
smutugriggif [[ -n "$IRONIC_DEFAULT_API_VERSION" ]]; then08:41
smutugrigg    IRONIC_CMD="$IRONIC_CMD --os-baremetal-api-version $IRONIC_DEFAULT_API_VERSION"08:41
smutugriggfi08:41
smutugriggwill become08:41
iurygregorysmutugrigg, this is in a shell sciprt file08:42
smutugriggOh sorry08:42
iurygregoryyou need to only update on files that are .rst08:42
iurygregory=)08:42
smutugriggsorry08:42
iurygregoryfor example08:42
iurygregoryhttps://github.com/openstack/ironic/blob/cef7ee08d0c7fd0406fcfe42d080431476372b19/doc/source/admin/portgroups.rst08:43
openstackgerritRiccardo Pittau proposed openstack/ironic-inspector master: [DNM] test ci with ubuntu focal  https://review.opendev.org/75666308:44
iurygregoryI think the reason why we want to do this is just because in some cases the API is already old, so it doesn't make sense to require the version =)08:44
smutugriggiurygregory, thank you for your patience.  To check if I have it correct this time the following text: openstack --os-baremetal-api-version 1.26 baremetal port set \08:45
smutugriggwould be come openstack --os-baremetal-api-version baremetal port set \08:45
smutugrigg?08:45
iurygregorywould become openstack baremetal port set08:45
iurygregoryyou need to also remove --os-baremetal-api-version08:46
smutugriggAh ha that makes sense08:46
smutugriggThank you.  So I search through all the .rst files to find the '--os-baremetal-api-version' after a `openstack baremetal` command and remove it.08:47
smutugriggThen push the changes to the project back to https://review.opendev.org08:48
smutugrigg?08:48
iurygregoryyeah =)08:48
smutugriggOh phew, I got there in the end.  Thank you08:49
iurygregoryif you have any problems when trying to submit the changes let me know08:49
smutugriggWill do, thank you08:49
iurygregorygood luck08:49
*** mkrai_ has quit IRC08:51
*** mkrai_ has joined #openstack-ironic08:51
*** abdysn has quit IRC08:56
smutugriggiurygregory, hello o/08:58
iurygregorysmutugrigg, hi08:58
smutugriggI notice that there is a large number of the '--os-baremetal-api-version 1.31' instructions08:59
smutugriggafter just 'openstack'08:59
smutugriggshould these be updated as well?08:59
iurygregoryyeah08:59
iurygregoryif they are on rst files =)08:59
smutugriggOk will update those as well.....only if in .rst files ;)08:59
iurygregory=)09:00
smutugriggiurygregory, hello o/09:04
smutugrigghopefully last question ion this task I found one last instance "(The alternative is to09:05
smutugriggadd ``--os-baremetal-api-version 1.15`` to the command.)::09:05
smutugrigg    export OS_BAREMETAL_API_VERSION=1.15"09:05
*** dtantsur|afk is now known as dtantsur09:05
smutugriggshould this remain or be removed?09:05
* iurygregory checks09:05
dtantsurmorning ironic09:05
iurygregorygood morning dtantsur09:05
rpittauhey dtantsur :)09:06
iurygregorysmutugrigg, this one you can keep it I would say it only says how the user can set09:06
smutugriggthanks iurygregory, will go ahead with trying to push now09:07
iurygregorythank you09:07
*** abdysn has joined #openstack-ironic09:19
*** abdysn has quit IRC09:19
*** Abdallahyas has joined #openstack-ironic09:19
*** abdysn has joined #openstack-ironic09:19
*** mkrai_ has quit IRC09:22
*** johnsom has quit IRC09:25
*** johnsom has joined #openstack-ironic09:25
*** mkrai has joined #openstack-ironic09:36
*** k_mouza has joined #openstack-ironic09:38
*** zzzeek has quit IRC09:40
*** rnoriega_ has quit IRC09:43
*** zzzeek has joined #openstack-ironic09:44
*** uzumaki has joined #openstack-ironic09:50
mgoddardhi dtantsur. Any thoughts on https://review.opendev.org/#/c/749014/ ? Move to CentOS only support for Stein & earlier?09:50
patchbotpatch 749014 - bifrost (stable/stein) - [Train and older] Fix several CI problems - 5 patch sets09:50
dtantsurugh09:50
dtantsurmgoddard: that sounds like we don't really support it any more..09:51
mgoddarddtantsur: well, that is the alternative being suggested09:51
*** rnoriega_ has joined #openstack-ironic09:51
mgoddardI'm proposing we continue to support what still works09:51
dtantsurmgoddard: at the very least we need a release note, I guess?09:52
mgoddarddtantsur: +109:54
mgoddardit still seems odd to me that inspector works in devstack on ubuntu but not bifrost09:55
*** rcernin has joined #openstack-ironic09:56
iurygregoryit's called magic09:56
mgoddardmaybe due to apache vs eventlet?09:57
*** rcernin has quit IRC10:00
dtantsurmgoddard: I don't think we use wsgi in bifrost10:02
dtantsurit may be because of mixing system and pip packages..10:02
*** bdodd has quit IRC10:09
*** dsneddon has quit IRC10:09
openstackgerritMark Goddard proposed openstack/bifrost stable/stein: [Train and older] Fix several CI problems  https://review.opendev.org/74901410:09
*** dsneddon has joined #openstack-ironic10:10
*** smutugrigg has quit IRC10:13
rpittauehmmm "Default interface set for deploy interface is not enabled"10:27
rpittaudid we remove any config from inspector grenade job ?10:28
*** rcernin has joined #openstack-ironic10:28
iurygregoryrpittau, we need to set to direct...10:28
iurygregoryI thought we took care of that O.o10:28
rpittauah10:28
* iurygregory finishing lunch10:28
openstackgerritVerification of a change to openstack/ironic failed: Wiping agent tokens on reboot via API - take 2  https://review.opendev.org/75647910:29
rpittauok ,let's check that10:29
*** rcernin has quit IRC10:31
rpittauodd that until october 2 was working though10:33
iurygregoryO.o10:33
rpittauso we removed iscsi from enabled interfaces afterwards ?10:34
rpittauI mean, could be10:34
openstackgerritRiccardo Pittau proposed openstack/ironic-inspector master: [DNM] test ci with ubuntu focal  https://review.opendev.org/75666310:35
rpittauchanged to direct here to test ^10:35
rpittauah I see, forgot we deprecated iscsi, so that got removed from devstack conf but not changed in inspector, ok10:39
rpittauif the test works I'll add a proper patch10:39
openstackgerritManuel Schönlaub proposed openstack/sushy master: Add oem_vendors from manager links  https://review.opendev.org/75532310:39
*** abdysn has quit IRC10:47
*** rh-jlabarre has quit IRC10:49
openstackgerritManuel Schönlaub proposed openstack/sushy master: Add oem_vendors from manager links  https://review.opendev.org/75532310:49
openstackgerritDmitry Tantsur proposed openstack/ironic-python-agent stable/victoria: Log a warning of target_boot_mode does not match current boot mode  https://review.opendev.org/75667610:51
openstackgerritDmitry Tantsur proposed openstack/ironic-python-agent stable/ussuri: Log a warning of target_boot_mode does not match current boot mode  https://review.opendev.org/75667710:51
openstackgerritManuel Schönlaub proposed openstack/sushy master: Add oem_vendors from manager links  https://review.opendev.org/75532310:53
openstackgerritMark Goddard proposed openstack/bifrost stable/stein: [Train and older] Fix several CI problems  https://review.opendev.org/74901410:59
*** zzzeek has quit IRC11:01
*** vmud213 has joined #openstack-ironic11:03
*** zzzeek has joined #openstack-ironic11:03
*** JamesBenson has joined #openstack-ironic11:13
*** mkrai has quit IRC11:29
*** Goneri has joined #openstack-ironic11:37
*** mkrai has joined #openstack-ironic11:47
*** outbrito has joined #openstack-ironic11:50
*** vmud213 has quit IRC11:52
openstackgerritAnuradha proposed openstack/ironic master: Adding the catch block for exception IncompatibleInterface in default_interface  https://review.opendev.org/75668211:58
*** jtomasek has joined #openstack-ironic11:58
*** jtomasek_ has joined #openstack-ironic12:06
*** jtomasek has quit IRC12:07
*** bdodd has joined #openstack-ironic12:19
*** mkrai has quit IRC12:19
janderssee you tomorrow Ironic o/12:43
*** rloo has joined #openstack-ironic12:53
*** abdysn has joined #openstack-ironic13:02
*** lmcgann has joined #openstack-ironic13:06
*** k_mouza has quit IRC13:08
TheJuliagood morning13:12
*** k_mouza has joined #openstack-ironic13:13
rpittaugood morning TheJulia :)13:14
dtantsurmorning TheJulia13:15
TheJuliaapparently I am using the wrong coffee mug this morning13:16
openstackgerritRiccardo Pittau proposed openstack/ironic-inspector master: Use direct as IRONIC_DEFAULT_DEPLOY_INTERFACE  https://review.opendev.org/75670513:22
openstackgerritRiccardo Pittau proposed openstack/ironic-inspector master: Use direct as IRONIC_DEFAULT_DEPLOY_INTERFACE for grenade  https://review.opendev.org/75670513:22
*** mkrai has joined #openstack-ironic13:26
openstackgerritMichelle (Shelly) Mutu-Grigg proposed openstack/ironic master: Update .rst files to remove --os-baremetal-api-version  https://review.opendev.org/75670813:27
rpittau"We now know the new Python will arrive every October" not sure how I feel about this13:27
*** smutugrigg has joined #openstack-ironic13:31
smutugriggiurygregory, hello o/13:33
iurygregorysmutugrigg, hi13:37
*** uzumaki has quit IRC13:38
smutugriggmanaged to push the changes up, was wondering if would be possible if you could let me know if I followed best practices?13:38
iurygregorysmutugrigg, I'll add your patch to my list and I will look at it o/13:39
smutugriggthank you13:40
smutugriggIs it alright if I try a medium task as well?13:40
*** tzumainn has joined #openstack-ironic13:42
iurygregorysmutugrigg, yes it's13:43
smutugriggthank you iurygregory, much appreciated13:43
iurygregoryhttps://storyboard.openstack.org/#!/story/list?status=active&tags=outreachy would provide some13:43
smutugriggthank you13:43
iurygregoryfeel free to ask here in the channel what we think about the task or any questions you may have13:43
smutugriggthat is great and really helpful, thanks.13:45
*** larsks has quit IRC13:58
*** Qianbiao has quit IRC14:01
*** Qianbiao has joined #openstack-ironic14:02
*** smutugrigg has quit IRC14:02
*** Qianbiao has quit IRC14:03
*** Qianbiao has joined #openstack-ironic14:03
*** k_mouza has quit IRC14:06
*** mkrai has quit IRC14:09
*** mkrai has joined #openstack-ironic14:09
*** Qianbiao has quit IRC14:10
*** Goneri has quit IRC14:15
*** k_mouza has joined #openstack-ironic14:16
*** k_mouza has quit IRC14:18
*** k_mouza has joined #openstack-ironic14:18
openstackgerritLeo McGann proposed openstack/ironic-specs master: Add security interface spec  https://review.opendev.org/57671814:22
*** Goneri has joined #openstack-ironic14:24
*** mkrai has quit IRC14:28
mgoddarddtantsur: looks like ironic-staging drivers has no stable/victoria branch14:29
dtantsurouch14:29
dtantsurmgoddard: I'll create it after the current meeting, ok?14:29
mgoddarddtantsur: sure14:29
*** k_mouza has quit IRC14:29
mgoddardsurprised it wasn't picked up by bifrost - kolla bifrost image fails to build on stable/victoria14:30
rpiosoGood morning, ironic14:32
*** k_mouza has joined #openstack-ironic14:35
openstackgerritMark Goddard proposed openstack/bifrost stable/stein: [Train and older] Fix several CI problems  https://review.opendev.org/74901414:36
*** k_mouza has quit IRC14:37
*** k_mouza has joined #openstack-ironic14:37
openstackgerritMark Goddard proposed openstack/bifrost master: Use upper constraints when installing Bifrost requirements  https://review.opendev.org/75675414:39
rpiosoTheJulia: Needful Dell EMC reviewers have approved a couple of idrac RAID backports to stable/train, https://review.opendev.org/750605 and https://review.opendev.org/#/c/755895. The latter depends on the former and is its child commit.14:46
patchbotpatch 750605 - ironic (stable/train) - DRAC: Fix RAID create_config clean step - 3 patch sets14:46
patchbotpatch 755895 - ironic (stable/train) - DRAC: Fix a failure to create virtual disk bug - 2 patch sets14:46
rpiosoTheJulia: You previously asked me to let you when they're ready :-) You had WF-1 the first one until then, errr now.14:53
*** mkrai has joined #openstack-ironic14:56
openstackgerritAnuradha proposed openstack/ironic master: Adding requested changes  https://review.opendev.org/75668214:56
openstackgerritMerged openstack/ironic-python-agent stable/victoria: Don't try to call GRUB when root UUID is not provided  https://review.opendev.org/75653314:58
*** priteau has joined #openstack-ironic14:59
TheJuliarpioso: ack, thanks15:01
*** larsks has joined #openstack-ironic15:15
*** mkrai has quit IRC15:18
anuradha1904Thank you for the review iurygregory , rpittau I am adding the changes as suggested both of you.15:21
*** k_mouza has quit IRC15:21
*** sdanni_ has joined #openstack-ironic15:23
openstackgerritMerged openstack/ironic-python-agent stable/victoria: Log a warning of target_boot_mode does not match current boot mode  https://review.opendev.org/75667615:24
dtantsurmgoddard: created a staging-drivers release, will create a branch once it goes through15:26
mgoddardgreat, thanks dtantsur :)15:26
sdanni_Hi all! Is there a way to include a python package in the IPA image that will be installed by the setup script?15:27
*** larsks has quit IRC15:30
*** larsks has joined #openstack-ironic15:31
*** k_mouza has joined #openstack-ironic15:33
*** dking has joined #openstack-ironic15:34
dkingFor those of you building DIB images, what does your CI/CD look like? Are you using shell runners with root access? I've noticed that I had issues building inside of a docker container, and I was wondering what works for other folks.15:37
arne_wiebalcksdanni_: if you build the image with the IPA builder, you can either add your own element with the list of packages you'd like to install (or have sth in post-install.d)15:37
arne_wiebalckdking: we build from within a container with IPAB15:38
*** abdysn has quit IRC15:38
arne_wiebalckdking: well, we added some scripts and build with these :)15:39
*** mkrai has joined #openstack-ironic15:40
sdanni_arne_wiebalck, thanks!15:40
dkingarne_wiebalck: I'm using IPAB. Currently, I'm doing it from the command line, which does work. However, when I tried to build images with DIB in docker previously, I came across some issues, even after running in privileged mode. Would you be able to share any info on what you had to do to get that working?15:40
TheJuliadking: Interestingly enough, there is a discussion of this on the mailing list now15:40
dkingI think it was kpartx that was giving me trouble.15:41
dkingTheJulia: Oh, neat. I hadn't checked lately.15:41
TheJuliathe conundrum at the moment is we're kind of loosing the ability to build dib images on focal for centos15:42
TheJuliait used to work, so the thought seems to be maybe container based image builds15:42
TheJuliaI _suspect_ if dib were to be purged of things like kpartx or those moved to loopback based image builds, then inside a container should work becaues functionally we need a cpio file, not a filesystem for IPA15:43
dkingYeah, I see the thread. It's different than my current situation, as everything we're doing at the moment is CentOS 8.15:43
dkingBut it does help me realize that other people are building inside of containers, which makes me wonder what I'm doing wrong that it's not working.15:44
TheJuliaI'm *not* sure it has ever been explicitly tested by the ironic community though15:44
TheJuliaand we build ramdisks generally as your trying to do, so milage may vary15:45
dkingIPA ramdisks are pretty much essential, so I imagine that there has to be some official CI/CD process that builds them?15:45
TheJuliayes, build them from zuul and post them to tarballs.o.o and those jobs are executed in a vm with root privilges15:46
TheJuliabecause dib is basically trying to use loopbacks and you need root privs to do that15:47
TheJuliaand mount/unmount/etc15:47
dkingOkay. That is what I was coming across, too. We were thinking that we would be creating CI runners that would create VMs for us with root access. I just didn't know if there were a more simple way to build them.15:49
TheJuliain essence that is exactly what zuul is doing15:50
arne_wiebalckdking: I can see if we can compile a comprehensive of relevant changes/additions we needed to do to make it work in containers. I did not make it myself, I am just using the recipe :)15:50
TheJuliait just uses the hashes that passed CI15:50
arne_wiebalck*comprehensive list15:50
TheJuliaarne_wiebalck: that would be quite helpful I think15:50
dkingarne_wiebalck: I would appreciate anything you have on that, especially if it helps us bypass the process we mentioned above, where we have to spin up VMs to do the work.15:50
openstackgerritAnuradha proposed openstack/ironic master: Changes to add exception in "default_interface()" function  https://review.opendev.org/75668215:51
arne_wiebalckdking: we have a builder host where I spin up the container inside of which I build the IPA images15:53
arne_wiebalckdking: the full recipe would probably include the setup of the host, the setup of the container plus the delta to upstream IPA15:54
arne_wiebalckIPAB15:54
dkingarne_wiebalck: Okay. I still think that would be helpful. I appreciate seeing what other folks are doing. When it overlaps with what I'm doing, I feel more sane, and when it doesn't, I learn something.15:55
arne_wiebalckdking: the colleague who did this is off today, I will check with him tomorrow when he's back15:56
*** Saf has joined #openstack-ironic15:57
*** gyee has joined #openstack-ironic16:00
dkingarne_wiebalck: I very much appreciate it.16:00
rpittaugood night! o/16:01
*** lucasagomes has quit IRC16:01
*** rpittau is now known as rpittau|afk16:01
arne_wiebalckTheJulia: dtantsur: https://review.opendev.org/#/c/756300/ seems to do the job: I just installed >100 physical servers and all of them came up with a healthy RAID (and an ESP on all holder disks which was the original problem).16:04
patchbotpatch 756300 - ironic-python-agent - Software RAID: Re-add missing devices - 1 patch set16:04
openstackgerritDmitry Tantsur proposed openstack/ironic master: PoC call set_boot_device earlier  https://review.opendev.org/75688116:09
*** Saf has quit IRC16:11
TheJuliaarne_wiebalck: that is great news16:12
dtantsur\o/16:13
TheJuliaI'm going to go to the store, I'll be back a little later16:15
*** yoctozepto has quit IRC16:16
anuradha1904TheJulia : Hi! Thank you very much for your review. can you please help me with the unit test part? Documentation on how to write the unit test will be helpful. Thank you.https://review.opendev.org/#/c/756682/3/ironic/common/driver_factory.py16:16
*** yoctozepto has joined #openstack-ironic16:16
patchbotpatch 756682 - ironic - Changes to add exception in "default_interface()" ... - 3 patch sets16:16
TheJuliaanuradha1904: documentation is a good question, unfortunately there is also not a straight forward answer! that being said, Let me get you a link16:18
TheJuliaanuradha1904: If you take a look at https://opendev.org/openstack/ironic/src/branch/master/ironic/tests/unit/common/test_driver_factory.py you'll see the DriverLoadTestCase16:20
TheJuliaIn that, you can see where we set the conditions and test the results, some of those cases we test for exceptions16:21
TheJuliaSo you would add a new test into that file, which essentially sets the preconditions where you expect the failure to occur, and test calling the method. If the test got the expected failure, then success!16:21
anuradha1904Sure, got it, will look through it! Thank you for your help.16:22
TheJuliayou can run the unit tests using the ``tox`` command, or a specific python3 unit test say ``tox -epy3 -- test_example_name`` and the test runner will use it as a regex to match tests to run16:22
TheJuliaand just run those or that specific test16:22
anuradha1904I do use Tox before putting out a patch, and when everything is successful, I add the code for review.16:23
openstackgerritDmitry Tantsur proposed openstack/ironic master: Call management.set_boot_device before IPA's install_bootloader  https://review.opendev.org/75688116:24
dtantsurTheJulia, bfournie, needs testing somehow ^^16:24
TheJuliaanuradha1904: excellent16:24
TheJuliaanuradha1904: even then, we still like to have tests to ensure that the expected exception is raised16:25
TheJuliadtantsur: hmm yeah :\16:25
anuradha1904Sure, Got it now, will add the test, thank you very much for the help.16:26
*** ricolin_ has quit IRC16:29
dtantsurmgoddard: stable/victoria created for staging-drivers16:34
mgoddardthanks dtantsur!16:39
*** k_mouza has quit IRC16:40
*** dtantsur is now known as dtantsur|afk16:46
dtantsur|afkon this positive note I'm wishing you a good rest of the day16:46
mgoddardand you dtantsur|afk16:49
openstackgerritMerged openstack/ironic master: migrate testing to ubuntu focal  https://review.opendev.org/73944816:59
openstackgerritMerged openstack/ironic master: Updates ironic documentation  https://review.opendev.org/75656017:00
*** ociuhandu_ has joined #openstack-ironic17:03
*** ociuhandu has quit IRC17:06
*** nandita has joined #openstack-ironic17:07
*** ociuhandu_ has quit IRC17:07
*** shreya11 has joined #openstack-ironic17:08
shreya11Hello everyone! I am Shreya, an outreachy applicant for this cohort. I am looking forward to contribute to 'Improving Ironic API on OpenStack SDK' project. I am well equipped with python and REST API.  Looking forward to amazing time with all of you. :)17:10
nanditaHello everyone. My name is Nandita. A mother of 2, I hail from India. I got selected in the preliminary round of outreachy applications, and I am looking forward to contribute to the Openstack. I have been programming for the past 2 years and I am interested in contributing in the project "Extend Ironic's Ansible modules". Looking forward to amazing contribution period <317:10
*** derekh has quit IRC17:16
dkingnandita: Welcome. I hope that you find the process to be enjoyable and without too many issues. If you haven't already read it, this is a good place to start: https://docs.opendev.org/opendev/infra-manual/latest/developers.html17:16
dkingnandita: If you have questions along the way, folks here are usually very helpful. You'll find more folks here around the European timezones, so usually a couple hours earlier than it is now. About now, folks are starting to drop off, though us US folks are still here.17:18
nanditathank you so much @dking. Will follow the provided link and will get back to if I have any doubts.17:19
*** penick has joined #openstack-ironic17:23
*** uzumaki has joined #openstack-ironic17:30
*** k_mouza has joined #openstack-ironic17:31
*** k_mouza has quit IRC17:35
*** mkrai has quit IRC17:40
openstackgerritManuel Schönlaub proposed openstack/sushy master: Adds basic support for TaskService to retrieve task information.  https://review.opendev.org/75690617:43
openstackgerritManuel Schönlaub proposed openstack/sushy master: Adds basic support for TaskService to retrieve task information.  https://review.opendev.org/75690617:45
*** ociuhandu has joined #openstack-ironic17:47
*** penick has quit IRC17:49
*** ociuhandu has quit IRC17:52
TheJuliaHello shreya11 and nandita! Welcome!17:55
TheJuliashreya11 and nandita: We have a story in our work tracking system, Storyboard, with tasks to familiarize yourself with the basic process of working in openstack. Above and beyond the link dking shared a little earlier. https://storyboard.openstack.org/#!/story/2008194 is the story. Please mark yourself as owning one of the tasks before you attempt to accomplish it. In our work tracking system we also have a18:02
TheJuliaspecific outreachy tag https://storyboard.openstack.org/#!/story/list?tags=outreachy for items that dive a little deeper into things. For example earlier today someone submitted a code change change and we soon responded and provided guidance on unit testing. It is all one step at a time, please don't hestitate to reach out to myself or iurygregory if you have questions.18:02
*** uzumaki has quit IRC18:06
*** uzumaki has joined #openstack-ironic18:07
openstackgerritManuel Schönlaub proposed openstack/sushy master: Adds basic support for TaskService to retrieve task information.  https://review.opendev.org/75690618:29
shreya11Thanks @TheJulia. Will surely do as told. :)18:31
TheJuliaAny ironic-cores around?18:36
iurygregoryTheJulia, I'm just got back from dinner18:42
iurygregorysomething urgent?18:42
TheJuliajust my change for ci on stable/ussuri18:42
TheJulianot super urgent but nobody seems to love it18:42
iurygregoryshreya11, nandita Welcome! =) feel free to ask questions o/18:42
iurygregoryTheJulia, https://review.opendev.org/#/c/755849/ ?18:43
patchbotpatch 755849 - ironic (stable/ussuri) - CI: Making some of the older jobs more specific - 1 patch set18:43
iurygregoryapproved18:44
iurygregoryI thought I looked at the ones for CI... I missed this one sorry18:44
TheJuliamuch appreciated18:45
TheJuliahmm, I guess I should look at maybe tuning back the standalone jobs or something18:45
nanditathank you so much TheJulia and iurygregory. I had a little question. How will establish our communication as I am living in UTC +5:30 ?18:52
anuradha1904Hey TheJulia : I have a question while trying to write the test I realized that a function has already been written to throw error  IncompatibleInterface if hw_type is a hardware type and the requested implementation is not compatible with it. here in this line https://opendev.org/openstack/ironic/src/branch/master/ironic/tests/unit/common/test_driver_factory.py#L453 , and I am bit confused how to add the18:52
anuradha1904test then? THank you for the help.18:52
anuradha190418:52
iurygregorynandita, I assume it's morning for you, sometimes I'm online during this time =)18:56
nanditait's actually 12am (midnight) here. But thank you for letting me know!18:58
*** jtomasek_ has quit IRC19:03
rpiosoTheJulia: Thank you for the reviews :-)19:05
TheJuliaI'm in UTC-7, I think iurygregory is ?UTC+2? so we likely just need to figure out where the best overlap times are to sync up, but we both stay in IRC around the clock (we might just not answer for 24 hours.19:06
TheJuliaanuradha1904: I _guess_ you'd dupliate the test, create slightly different name and then kind of go from there. That test specifically is testing if it set on the node. I think your patch is more focused on what the default of the configuration is, not the node19:08
nanditaThat makes sense @TheJulia. Thank you!19:10
iurygregoryyeah I'm UTC +219:10
TheJuliaiurygregory: I can never remember because flying there always just destroys me.19:11
iurygregoryTheJulia, yeah =(19:12
TheJuliaI guess because I never get any sleep on long haul flight and then wander around CDG like a zombie before my connection... and then yeah. I think the last time I ended up also having to add an entire extra day to get home :\19:12
iurygregoryif the flight is more than 5hours I need to sleep XD19:14
iurygregorybut my neck will complain ...19:15
TheJuliaI never sleep heading to Europe. Heading to Asia it is actually more likely19:15
iurygregorywow19:16
anuradha1904Hey TheJulia : so I have to write fake config i.e wrong hardware and fake interface implementation, and then raise the error  IncompatibleInterface error as the won't be compatible, am I following right?19:18
*** uzumaki has quit IRC19:19
*** larsks has quit IRC19:22
*** uzumaki has joined #openstack-ironic19:24
*** priteau has quit IRC19:31
*** larsks has joined #openstack-ironic19:37
*** larsks has quit IRC19:44
TheJuliayou don't need to create an interface, I think it is about the default value that someone sets in the config, so you can override that very much like the existing test, you'd just have something like self.config(default_$interface_interface='nonexistantvalue') or set it to 'fake' set the enabled_hardware_types to not have 'fake-hardware'19:45
*** larsks has joined #openstack-ironic19:45
*** hoonetorg has quit IRC19:47
*** hoonetorg has joined #openstack-ironic20:00
*** iurygregory has quit IRC20:00
*** iurygregory has joined #openstack-ironic20:02
openstackgerritJulia Kreger proposed openstack/metalsmith master: Add a Centos8 UEFI based job  https://review.opendev.org/74831520:05
openstackgerritMerged openstack/ironic stable/victoria: Wiping agent tokens on reboot via API - take 2  https://review.opendev.org/75647920:07
anuradha1904TheJulia: Thank you very much for the help, will start implementing it.20:11
*** zzzeek has quit IRC20:25
*** zzzeek has joined #openstack-ironic20:27
openstackgerritMerged openstack/ironic-python-agent stable/ussuri: Log a warning of target_boot_mode does not match current boot mode  https://review.opendev.org/75667720:38
openstackgerritJulia Kreger proposed openstack/ironic-python-agent master: Don't run os-prober from grub2-mkconfig  https://review.opendev.org/75230820:49
openstackgerritJulia Kreger proposed openstack/ironic-python-agent master: Don't run os-prober from grub2-mkconfig  https://review.opendev.org/75230820:50
openstackgerritVerification of a change to openstack/ironic failed: Increase memory of tinyipa vms  https://review.opendev.org/74763320:54
openstackgerritMerged openstack/metalsmith master: metalsmith_deployment role switch to metalsmith_instances  https://review.opendev.org/73788821:09
openstackgerritJulia Kreger proposed openstack/ironic-python-agent master: Support partition image efi contents  https://review.opendev.org/74804921:16
openstackgerritJulia Kreger proposed openstack/ironic-python-agent master: Add fstab pointer to EFI partition  https://review.opendev.org/74826021:22
jandersgood morning Ironic o/21:22
openstackgerritJulia Kreger proposed openstack/ironic-python-agent master: Don't run os-prober from grub2-mkconfig  https://review.opendev.org/75230821:24
openstackgerritMerged openstack/metalsmith master: CI: Extend job timeout slightly for metalsmith jobs  https://review.opendev.org/75579821:24
openstackgerritJulia Kreger proposed openstack/ironic-python-agent master: Support partition image efi contents  https://review.opendev.org/74804921:25
openstackgerritJulia Kreger proposed openstack/ironic-python-agent master: Add fstab pointer to EFI partition  https://review.opendev.org/74826021:25
TheJuliagood morning janders21:25
*** lmcgann has quit IRC21:26
openstackgerritVerification of a change to openstack/ironic failed: CI: Making some of the older jobs more specific  https://review.opendev.org/75584921:40
*** kafilat has quit IRC22:10
*** rloo has quit IRC22:23
*** k_mouza has joined #openstack-ironic22:30
*** larsks has quit IRC22:31
*** k_mouza has quit IRC22:35
*** larsks has joined #openstack-ironic22:37
*** k_mouza has joined #openstack-ironic22:43
*** rcernin has joined #openstack-ironic22:47
*** k_mouza has quit IRC22:48
openstackgerritJulia Kreger proposed openstack/ironic-inspector master: WIP: Try2... power off the node before inspection  https://review.opendev.org/75693022:50
TheJuliaerbarr: o/ so we're looking at making a structural change to the UEFI process because we're seeing some weirdness on >1 vendor platform with sending raw commands now. Dmitry has submitted a patch into CI as a WIP just to see if anything explodes in any epic way/shape/form (https://review.opendev.org/#/c/756881/) and I just tried 3 or 4 different posted log urls and I got a 404 back on all of them. is the url22:54
TheJuliawrong that is being posted in the results?22:54
patchbotpatch 756881 - ironic - Call management.set_boot_device before IPA's insta... - 2 patch sets22:54
*** outbrito has quit IRC22:55
* TheJulia lets CI run on patches22:57
openstackgerritJulia Kreger proposed openstack/ironic-python-agent master: Don't run os-prober from grub2-mkconfig  https://review.opendev.org/75230822:59
openstackgerritJulia Kreger proposed openstack/ironic-python-agent master: Support partition image efi contents  https://review.opendev.org/74804922:59
openstackgerritJulia Kreger proposed openstack/ironic-python-agent master: Add fstab pointer to EFI partition  https://review.opendev.org/74826022:59
TheJulianow CI should run :\22:59
*** tosky has quit IRC22:59
openstackgerritMerged openstack/ironic-python-agent master: Reduce the duration of retries in the inspector tests  https://review.opendev.org/75648823:04
openstackgerritMerged openstack/ironic-inspector master: [DOC] Add note regarding ``[pxe_filter]deny_unknown_macs``  https://review.opendev.org/75658323:04
*** pmannidi has joined #openstack-ironic23:23
*** anuradha1904 has quit IRC23:35
*** JamesBenson has quit IRC23:48
*** JamesBenson has joined #openstack-ironic23:48
*** gyee has quit IRC23:50
*** k_mouza has joined #openstack-ironic23:59

Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!