*** gyee has quit IRC | 00:31 | |
*** k_mouza has joined #openstack-ironic | 00:32 | |
*** k_mouza has quit IRC | 00:36 | |
*** anuradha1904 has quit IRC | 00:43 | |
*** ianychoi__ is now known as ianychoi | 00:44 | |
*** rcernin has quit IRC | 00:48 | |
*** rcernin has joined #openstack-ironic | 00:55 | |
*** outbrito has quit IRC | 01:00 | |
*** Goneri has quit IRC | 01:06 | |
*** rcernin has quit IRC | 01:08 | |
*** k_mouza has joined #openstack-ironic | 01:17 | |
*** JamesBenson has joined #openstack-ironic | 01:20 | |
*** k_mouza has quit IRC | 01:22 | |
*** rcernin has joined #openstack-ironic | 01:25 | |
*** JamesBenson has quit IRC | 01:25 | |
*** rcernin has quit IRC | 01:39 | |
*** k_mouza has joined #openstack-ironic | 01:39 | |
*** k_mouza has quit IRC | 01:43 | |
*** ricolin_ has joined #openstack-ironic | 01:45 | |
*** k_mouza has joined #openstack-ironic | 01:47 | |
*** k_mouza has quit IRC | 01:51 | |
*** tkajinam has quit IRC | 01:52 | |
*** tkajinam has joined #openstack-ironic | 01:53 | |
*** eandersson has quit IRC | 01:56 | |
*** eandersson has joined #openstack-ironic | 01:57 | |
*** JamesBenson has joined #openstack-ironic | 02:01 | |
*** k_mouza has joined #openstack-ironic | 02:04 | |
*** JamesBenson has quit IRC | 02:06 | |
*** k_mouza has quit IRC | 02:09 | |
*** rcernin has joined #openstack-ironic | 02:13 | |
*** JamesBenson has joined #openstack-ironic | 02:14 | |
*** k_mouza has joined #openstack-ironic | 02:14 | |
*** k_mouza has quit IRC | 02:18 | |
*** JamesBenson has quit IRC | 02:18 | |
*** mkrai has joined #openstack-ironic | 02:22 | |
*** Qianbiao has joined #openstack-ironic | 02:29 | |
*** JamesBenson has joined #openstack-ironic | 02:39 | |
*** JamesBenson has quit IRC | 02:44 | |
*** k_mouza has joined #openstack-ironic | 02:48 | |
*** k_mouza has quit IRC | 02:53 | |
*** JamesBenson has joined #openstack-ironic | 03:05 | |
*** JamesBen_ has joined #openstack-ironic | 03:09 | |
*** JamesBenson has quit IRC | 03:09 | |
*** mkrai has quit IRC | 03:24 | |
*** mkrai_ has joined #openstack-ironic | 03:24 | |
*** mkrai_ has quit IRC | 03:28 | |
*** JamesBen_ has quit IRC | 04:02 | |
*** tzumainn has quit IRC | 04:02 | |
*** k_mouza has joined #openstack-ironic | 04:05 | |
*** JamesBenson has joined #openstack-ironic | 04:07 | |
*** ociuhandu has joined #openstack-ironic | 04:08 | |
*** k_mouza has quit IRC | 04:09 | |
*** JamesBenson has quit IRC | 04:10 | |
*** mkrai has joined #openstack-ironic | 04:12 | |
*** ociuhandu has quit IRC | 04:12 | |
*** cdearborn has quit IRC | 04:14 | |
*** JamesBenson has joined #openstack-ironic | 04:16 | |
*** JamesBenson has quit IRC | 04:27 | |
*** smutugrigg has joined #openstack-ironic | 04:42 | |
*** smutugrigg35 has joined #openstack-ironic | 04:42 | |
*** smutugrigg35 has quit IRC | 04:42 | |
*** smutugrigg has quit IRC | 04:42 | |
*** smutugrigg has joined #openstack-ironic | 04:43 | |
*** anuradha1904 has joined #openstack-ironic | 04:45 | |
*** JamesBenson has joined #openstack-ironic | 04:50 | |
*** JamesBenson has quit IRC | 04:55 | |
*** mkrai has quit IRC | 05:03 | |
*** mkrai has joined #openstack-ironic | 05:06 | |
*** abdysn has joined #openstack-ironic | 05:44 | |
*** kafilat has joined #openstack-ironic | 06:01 | |
*** smutugrigg has quit IRC | 06:06 | |
*** SuiongNg|ITRI_TW has joined #openstack-ironic | 06:06 | |
*** smutugrigg has joined #openstack-ironic | 06:11 | |
*** uzumaki has joined #openstack-ironic | 06:45 | |
arne_wiebalck | Good morning, ironic o/ | 06:58 |
---|---|---|
janders | good morning arne_wiebalck o/ | 07:02 |
arne_wiebalck | Hey janders o/ | 07:02 |
*** ociuhandu has joined #openstack-ironic | 07:02 | |
iurygregory | good morning arne_wiebalck janders and Ironic | 07:04 |
janders | good morning iurygregory o/ | 07:06 |
iurygregory | janders, o/ | 07:07 |
*** mkrai has quit IRC | 07:08 | |
*** mkrai_ has joined #openstack-ironic | 07:08 | |
arne_wiebalck | hey iurygregory o/ | 07:12 |
*** mkrai_ has quit IRC | 07:22 | |
*** rpittau|afk is now known as rpittau | 07:27 | |
rpittau | good morning ironic! o/ | 07:27 |
janders | good morning rpittau! o/ | 07:30 |
rpittau | hey janders :) | 07:30 |
iurygregory | arne_wiebalck, o/ | 07:32 |
*** uzumaki has quit IRC | 07:33 | |
*** jamesdenton has quit IRC | 07:33 | |
*** rajinir has quit IRC | 07:33 | |
iurygregory | morning rpittau o/ | 07:33 |
rpittau | hey iurygregory :) | 07:33 |
*** uzumaki has joined #openstack-ironic | 07:33 | |
* iurygregory is using the CERN mug today | 07:33 | |
* arne_wiebalck guesses iurygregory's coffee tastes particle-ly good today? | 07:34 | |
rpittau | what happen if you bump the CERN mug with another one from opposite direction and same speed? | 07:35 |
iurygregory | you will need to go to CERN to buy a new one | 07:36 |
iurygregory | =P | 07:36 |
*** mgoddard has quit IRC | 07:36 | |
janders | I wonder: if one spills coffee from a CERN mug, do the coffee drops in particle collision like patterns? | 07:37 |
janders | also, is there a such thing as a CERN espresso cup? | 07:37 |
*** uzumaki has quit IRC | 07:38 | |
*** jamesdenton has joined #openstack-ironic | 07:38 | |
*** rajinir has joined #openstack-ironic | 07:38 | |
*** mgoddard has joined #openstack-ironic | 07:39 | |
iurygregory | janders, https://twitter.com/_Skolt_Iury_/status/1232698121857241088 | 07:42 |
*** jtomasek_ has joined #openstack-ironic | 07:45 | |
*** jtomasek has quit IRC | 07:45 | |
janders | nice :) | 07:45 |
*** tosky has joined #openstack-ironic | 07:54 | |
*** jtomasek_ has quit IRC | 07:56 | |
*** rcernin has quit IRC | 07:57 | |
*** openstackgerrit has joined #openstack-ironic | 08:05 | |
openstackgerrit | Kafilat Adeleke proposed openstack/ironic master: Changes links so we do not have "here" as the link https://review.opendev.org/756574 | 08:05 |
*** lucasagomes has joined #openstack-ironic | 08:12 | |
*** Abdallahyas has joined #openstack-ironic | 08:16 | |
*** Abdallahyas has quit IRC | 08:16 | |
*** Abdallahyas has joined #openstack-ironic | 08:16 | |
*** mkrai_ has joined #openstack-ironic | 08:19 | |
*** abdysn has quit IRC | 08:20 | |
smutugrigg | hi, I was wondering if I could ask for some guidance please? I am an Outreachy intern applicant and am a bit lost. | 08:27 |
openstackgerrit | Riccardo Pittau proposed openstack/ironic-python-agent-builder master: Use tinycore 11 to build tinyipa https://review.opendev.org/711647 | 08:28 |
smutugrigg | I have requested to work on task 40964 under story/2008194 | 08:28 |
smutugrigg | But would like some guidance | 08:28 |
iurygregory | smutugrigg, hello o/ | 08:29 |
smutugrigg | I apologise in advance, this is all a bit new to me | 08:30 |
smutugrigg | hi iurygregory o/ | 08:30 |
*** abdysn has joined #openstack-ironic | 08:30 | |
iurygregory | smutugrigg, no worries =) Have you looked at the OpenStack new contributor documentation already and set up the gerrit account ? | 08:30 |
smutugrigg | iurygregory, yes I have and have cloned the ironic project from Git | 08:31 |
iurygregory | smutugrigg, cool! | 08:31 |
smutugrigg | iurygregory, yah! | 08:32 |
iurygregory | smutugrigg, so tehre are a few initial tasks that you can choose the one you want | 08:32 |
iurygregory | please 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 you | 08:33 |
smutugrigg | iurygregory, I have assigned task 40964 to myself for action | 08:33 |
*** Abdallahyas has quit IRC | 08:33 | |
smutugrigg | Could you please help me with what I need to do please? | 08:33 |
* iurygregory refreshs the browser | 08:33 | |
iurygregory | sure | 08:33 |
smutugrigg | thanks | 08:34 |
iurygregory | give me a minute let me check the code | 08:34 |
smutugrigg | cool | 08:34 |
openstackgerrit | Riccardo Pittau proposed openstack/sushy master: [DNM] test ci with ubuntu focal https://review.opendev.org/756660 | 08:34 |
*** derekh has joined #openstack-ironic | 08:35 | |
iurygregory | smutugrigg, 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 remove | 08:39 |
smutugrigg | iurygregory, to check if I understand correctly the following text: | 08:41 |
smutugrigg | IRONIC_CMD="openstack baremetal" | 08:41 |
smutugrigg | if [[ -n "$IRONIC_DEFAULT_API_VERSION" ]]; then | 08:41 |
smutugrigg | IRONIC_CMD="$IRONIC_CMD --os-baremetal-api-version $IRONIC_DEFAULT_API_VERSION" | 08:41 |
smutugrigg | fi | 08:41 |
smutugrigg | will become | 08:41 |
iurygregory | smutugrigg, this is in a shell sciprt file | 08:42 |
smutugrigg | Oh sorry | 08:42 |
iurygregory | you need to only update on files that are .rst | 08:42 |
iurygregory | =) | 08:42 |
smutugrigg | sorry | 08:42 |
iurygregory | for example | 08:42 |
iurygregory | https://github.com/openstack/ironic/blob/cef7ee08d0c7fd0406fcfe42d080431476372b19/doc/source/admin/portgroups.rst | 08:43 |
openstackgerrit | Riccardo Pittau proposed openstack/ironic-inspector master: [DNM] test ci with ubuntu focal https://review.opendev.org/756663 | 08:44 |
iurygregory | I 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 |
smutugrigg | iurygregory, 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 |
smutugrigg | would be come openstack --os-baremetal-api-version baremetal port set \ | 08:45 |
smutugrigg | ? | 08:45 |
iurygregory | would become openstack baremetal port set | 08:45 |
iurygregory | you need to also remove --os-baremetal-api-version | 08:46 |
smutugrigg | Ah ha that makes sense | 08:46 |
smutugrigg | Thank 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 |
smutugrigg | Then push the changes to the project back to https://review.opendev.org | 08:48 |
smutugrigg | ? | 08:48 |
iurygregory | yeah =) | 08:48 |
smutugrigg | Oh phew, I got there in the end. Thank you | 08:49 |
iurygregory | if you have any problems when trying to submit the changes let me know | 08:49 |
smutugrigg | Will do, thank you | 08:49 |
iurygregory | good luck | 08:49 |
*** mkrai_ has quit IRC | 08:51 | |
*** mkrai_ has joined #openstack-ironic | 08:51 | |
*** abdysn has quit IRC | 08:56 | |
smutugrigg | iurygregory, hello o/ | 08:58 |
iurygregory | smutugrigg, hi | 08:58 |
smutugrigg | I notice that there is a large number of the '--os-baremetal-api-version 1.31' instructions | 08:59 |
smutugrigg | after just 'openstack' | 08:59 |
smutugrigg | should these be updated as well? | 08:59 |
iurygregory | yeah | 08:59 |
iurygregory | if they are on rst files =) | 08:59 |
smutugrigg | Ok will update those as well.....only if in .rst files ;) | 08:59 |
iurygregory | =) | 09:00 |
smutugrigg | iurygregory, hello o/ | 09:04 |
smutugrigg | hopefully last question ion this task I found one last instance "(The alternative is to | 09:05 |
smutugrigg | add ``--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 dtantsur | 09:05 | |
smutugrigg | should this remain or be removed? | 09:05 |
* iurygregory checks | 09:05 | |
dtantsur | morning ironic | 09:05 |
iurygregory | good morning dtantsur | 09:05 |
rpittau | hey dtantsur :) | 09:06 |
iurygregory | smutugrigg, this one you can keep it I would say it only says how the user can set | 09:06 |
smutugrigg | thanks iurygregory, will go ahead with trying to push now | 09:07 |
iurygregory | thank you | 09:07 |
*** abdysn has joined #openstack-ironic | 09:19 | |
*** abdysn has quit IRC | 09:19 | |
*** Abdallahyas has joined #openstack-ironic | 09:19 | |
*** abdysn has joined #openstack-ironic | 09:19 | |
*** mkrai_ has quit IRC | 09:22 | |
*** johnsom has quit IRC | 09:25 | |
*** johnsom has joined #openstack-ironic | 09:25 | |
*** mkrai has joined #openstack-ironic | 09:36 | |
*** k_mouza has joined #openstack-ironic | 09:38 | |
*** zzzeek has quit IRC | 09:40 | |
*** rnoriega_ has quit IRC | 09:43 | |
*** zzzeek has joined #openstack-ironic | 09:44 | |
*** uzumaki has joined #openstack-ironic | 09:50 | |
mgoddard | hi dtantsur. Any thoughts on https://review.opendev.org/#/c/749014/ ? Move to CentOS only support for Stein & earlier? | 09:50 |
patchbot | patch 749014 - bifrost (stable/stein) - [Train and older] Fix several CI problems - 5 patch sets | 09:50 |
dtantsur | ugh | 09:50 |
dtantsur | mgoddard: that sounds like we don't really support it any more.. | 09:51 |
mgoddard | dtantsur: well, that is the alternative being suggested | 09:51 |
*** rnoriega_ has joined #openstack-ironic | 09:51 | |
mgoddard | I'm proposing we continue to support what still works | 09:51 |
dtantsur | mgoddard: at the very least we need a release note, I guess? | 09:52 |
mgoddard | dtantsur: +1 | 09:54 |
mgoddard | it still seems odd to me that inspector works in devstack on ubuntu but not bifrost | 09:55 |
*** rcernin has joined #openstack-ironic | 09:56 | |
iurygregory | it's called magic | 09:56 |
mgoddard | maybe due to apache vs eventlet? | 09:57 |
*** rcernin has quit IRC | 10:00 | |
dtantsur | mgoddard: I don't think we use wsgi in bifrost | 10:02 |
dtantsur | it may be because of mixing system and pip packages.. | 10:02 |
*** bdodd has quit IRC | 10:09 | |
*** dsneddon has quit IRC | 10:09 | |
openstackgerrit | Mark Goddard proposed openstack/bifrost stable/stein: [Train and older] Fix several CI problems https://review.opendev.org/749014 | 10:09 |
*** dsneddon has joined #openstack-ironic | 10:10 | |
*** smutugrigg has quit IRC | 10:13 | |
rpittau | ehmmm "Default interface set for deploy interface is not enabled" | 10:27 |
rpittau | did we remove any config from inspector grenade job ? | 10:28 |
*** rcernin has joined #openstack-ironic | 10:28 | |
iurygregory | rpittau, we need to set to direct... | 10:28 |
iurygregory | I thought we took care of that O.o | 10:28 |
rpittau | ah | 10:28 |
* iurygregory finishing lunch | 10:28 | |
openstackgerrit | Verification of a change to openstack/ironic failed: Wiping agent tokens on reboot via API - take 2 https://review.opendev.org/756479 | 10:29 |
rpittau | ok ,let's check that | 10:29 |
*** rcernin has quit IRC | 10:31 | |
rpittau | odd that until october 2 was working though | 10:33 |
iurygregory | O.o | 10:33 |
rpittau | so we removed iscsi from enabled interfaces afterwards ? | 10:34 |
rpittau | I mean, could be | 10:34 |
openstackgerrit | Riccardo Pittau proposed openstack/ironic-inspector master: [DNM] test ci with ubuntu focal https://review.opendev.org/756663 | 10:35 |
rpittau | changed to direct here to test ^ | 10:35 |
rpittau | ah I see, forgot we deprecated iscsi, so that got removed from devstack conf but not changed in inspector, ok | 10:39 |
rpittau | if the test works I'll add a proper patch | 10:39 |
openstackgerrit | Manuel Schönlaub proposed openstack/sushy master: Add oem_vendors from manager links https://review.opendev.org/755323 | 10:39 |
*** abdysn has quit IRC | 10:47 | |
*** rh-jlabarre has quit IRC | 10:49 | |
openstackgerrit | Manuel Schönlaub proposed openstack/sushy master: Add oem_vendors from manager links https://review.opendev.org/755323 | 10:49 |
openstackgerrit | Dmitry 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/756676 | 10:51 |
openstackgerrit | Dmitry 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/756677 | 10:51 |
openstackgerrit | Manuel Schönlaub proposed openstack/sushy master: Add oem_vendors from manager links https://review.opendev.org/755323 | 10:53 |
openstackgerrit | Mark Goddard proposed openstack/bifrost stable/stein: [Train and older] Fix several CI problems https://review.opendev.org/749014 | 10:59 |
*** zzzeek has quit IRC | 11:01 | |
*** vmud213 has joined #openstack-ironic | 11:03 | |
*** zzzeek has joined #openstack-ironic | 11:03 | |
*** JamesBenson has joined #openstack-ironic | 11:13 | |
*** mkrai has quit IRC | 11:29 | |
*** Goneri has joined #openstack-ironic | 11:37 | |
*** mkrai has joined #openstack-ironic | 11:47 | |
*** outbrito has joined #openstack-ironic | 11:50 | |
*** vmud213 has quit IRC | 11:52 | |
openstackgerrit | Anuradha proposed openstack/ironic master: Adding the catch block for exception IncompatibleInterface in default_interface https://review.opendev.org/756682 | 11:58 |
*** jtomasek has joined #openstack-ironic | 11:58 | |
*** jtomasek_ has joined #openstack-ironic | 12:06 | |
*** jtomasek has quit IRC | 12:07 | |
*** bdodd has joined #openstack-ironic | 12:19 | |
*** mkrai has quit IRC | 12:19 | |
janders | see you tomorrow Ironic o/ | 12:43 |
*** rloo has joined #openstack-ironic | 12:53 | |
*** abdysn has joined #openstack-ironic | 13:02 | |
*** lmcgann has joined #openstack-ironic | 13:06 | |
*** k_mouza has quit IRC | 13:08 | |
TheJulia | good morning | 13:12 |
*** k_mouza has joined #openstack-ironic | 13:13 | |
rpittau | good morning TheJulia :) | 13:14 |
dtantsur | morning TheJulia | 13:15 |
TheJulia | apparently I am using the wrong coffee mug this morning | 13:16 |
openstackgerrit | Riccardo Pittau proposed openstack/ironic-inspector master: Use direct as IRONIC_DEFAULT_DEPLOY_INTERFACE https://review.opendev.org/756705 | 13:22 |
openstackgerrit | Riccardo Pittau proposed openstack/ironic-inspector master: Use direct as IRONIC_DEFAULT_DEPLOY_INTERFACE for grenade https://review.opendev.org/756705 | 13:22 |
*** mkrai has joined #openstack-ironic | 13:26 | |
openstackgerrit | Michelle (Shelly) Mutu-Grigg proposed openstack/ironic master: Update .rst files to remove --os-baremetal-api-version https://review.opendev.org/756708 | 13:27 |
rpittau | "We now know the new Python will arrive every October" not sure how I feel about this | 13:27 |
*** smutugrigg has joined #openstack-ironic | 13:31 | |
smutugrigg | iurygregory, hello o/ | 13:33 |
iurygregory | smutugrigg, hi | 13:37 |
*** uzumaki has quit IRC | 13:38 | |
smutugrigg | managed to push the changes up, was wondering if would be possible if you could let me know if I followed best practices? | 13:38 |
iurygregory | smutugrigg, I'll add your patch to my list and I will look at it o/ | 13:39 |
smutugrigg | thank you | 13:40 |
smutugrigg | Is it alright if I try a medium task as well? | 13:40 |
*** tzumainn has joined #openstack-ironic | 13:42 | |
iurygregory | smutugrigg, yes it's | 13:43 |
smutugrigg | thank you iurygregory, much appreciated | 13:43 |
iurygregory | https://storyboard.openstack.org/#!/story/list?status=active&tags=outreachy would provide some | 13:43 |
smutugrigg | thank you | 13:43 |
iurygregory | feel free to ask here in the channel what we think about the task or any questions you may have | 13:43 |
smutugrigg | that is great and really helpful, thanks. | 13:45 |
*** larsks has quit IRC | 13:58 | |
*** Qianbiao has quit IRC | 14:01 | |
*** Qianbiao has joined #openstack-ironic | 14:02 | |
*** smutugrigg has quit IRC | 14:02 | |
*** Qianbiao has quit IRC | 14:03 | |
*** Qianbiao has joined #openstack-ironic | 14:03 | |
*** k_mouza has quit IRC | 14:06 | |
*** mkrai has quit IRC | 14:09 | |
*** mkrai has joined #openstack-ironic | 14:09 | |
*** Qianbiao has quit IRC | 14:10 | |
*** Goneri has quit IRC | 14:15 | |
*** k_mouza has joined #openstack-ironic | 14:16 | |
*** k_mouza has quit IRC | 14:18 | |
*** k_mouza has joined #openstack-ironic | 14:18 | |
openstackgerrit | Leo McGann proposed openstack/ironic-specs master: Add security interface spec https://review.opendev.org/576718 | 14:22 |
*** Goneri has joined #openstack-ironic | 14:24 | |
*** mkrai has quit IRC | 14:28 | |
mgoddard | dtantsur: looks like ironic-staging drivers has no stable/victoria branch | 14:29 |
dtantsur | ouch | 14:29 |
dtantsur | mgoddard: I'll create it after the current meeting, ok? | 14:29 |
mgoddard | dtantsur: sure | 14:29 |
*** k_mouza has quit IRC | 14:29 | |
mgoddard | surprised it wasn't picked up by bifrost - kolla bifrost image fails to build on stable/victoria | 14:30 |
rpioso | Good morning, ironic | 14:32 |
*** k_mouza has joined #openstack-ironic | 14:35 | |
openstackgerrit | Mark Goddard proposed openstack/bifrost stable/stein: [Train and older] Fix several CI problems https://review.opendev.org/749014 | 14:36 |
*** k_mouza has quit IRC | 14:37 | |
*** k_mouza has joined #openstack-ironic | 14:37 | |
openstackgerrit | Mark Goddard proposed openstack/bifrost master: Use upper constraints when installing Bifrost requirements https://review.opendev.org/756754 | 14:39 |
rpioso | TheJulia: 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 |
patchbot | patch 750605 - ironic (stable/train) - DRAC: Fix RAID create_config clean step - 3 patch sets | 14:46 |
patchbot | patch 755895 - ironic (stable/train) - DRAC: Fix a failure to create virtual disk bug - 2 patch sets | 14:46 |
rpioso | TheJulia: 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-ironic | 14:56 | |
openstackgerrit | Anuradha proposed openstack/ironic master: Adding requested changes https://review.opendev.org/756682 | 14:56 |
openstackgerrit | Merged openstack/ironic-python-agent stable/victoria: Don't try to call GRUB when root UUID is not provided https://review.opendev.org/756533 | 14:58 |
*** priteau has joined #openstack-ironic | 14:59 | |
TheJulia | rpioso: ack, thanks | 15:01 |
*** larsks has joined #openstack-ironic | 15:15 | |
*** mkrai has quit IRC | 15:18 | |
anuradha1904 | Thank you for the review iurygregory , rpittau I am adding the changes as suggested both of you. | 15:21 |
*** k_mouza has quit IRC | 15:21 | |
*** sdanni_ has joined #openstack-ironic | 15:23 | |
openstackgerrit | Merged openstack/ironic-python-agent stable/victoria: Log a warning of target_boot_mode does not match current boot mode https://review.opendev.org/756676 | 15:24 |
dtantsur | mgoddard: created a staging-drivers release, will create a branch once it goes through | 15:26 |
mgoddard | great, 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 IRC | 15:30 | |
*** larsks has joined #openstack-ironic | 15:31 | |
*** k_mouza has joined #openstack-ironic | 15:33 | |
*** dking has joined #openstack-ironic | 15:34 | |
dking | For 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_wiebalck | sdanni_: 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_wiebalck | dking: we build from within a container with IPAB | 15:38 |
*** abdysn has quit IRC | 15:38 | |
arne_wiebalck | dking: well, we added some scripts and build with these :) | 15:39 |
*** mkrai has joined #openstack-ironic | 15:40 | |
sdanni_ | arne_wiebalck, thanks! | 15:40 |
dking | arne_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 |
TheJulia | dking: Interestingly enough, there is a discussion of this on the mailing list now | 15:40 |
dking | I think it was kpartx that was giving me trouble. | 15:41 |
dking | TheJulia: Oh, neat. I hadn't checked lately. | 15:41 |
TheJulia | the conundrum at the moment is we're kind of loosing the ability to build dib images on focal for centos | 15:42 |
TheJulia | it used to work, so the thought seems to be maybe container based image builds | 15:42 |
TheJulia | I _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 IPA | 15:43 |
dking | Yeah, I see the thread. It's different than my current situation, as everything we're doing at the moment is CentOS 8. | 15:43 |
dking | But 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 |
TheJulia | I'm *not* sure it has ever been explicitly tested by the ironic community though | 15:44 |
TheJulia | and we build ramdisks generally as your trying to do, so milage may vary | 15:45 |
dking | IPA ramdisks are pretty much essential, so I imagine that there has to be some official CI/CD process that builds them? | 15:45 |
TheJulia | yes, build them from zuul and post them to tarballs.o.o and those jobs are executed in a vm with root privilges | 15:46 |
TheJulia | because dib is basically trying to use loopbacks and you need root privs to do that | 15:47 |
TheJulia | and mount/unmount/etc | 15:47 |
dking | Okay. 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 |
TheJulia | in essence that is exactly what zuul is doing | 15:50 |
arne_wiebalck | dking: 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 |
TheJulia | it just uses the hashes that passed CI | 15:50 |
arne_wiebalck | *comprehensive list | 15:50 |
TheJulia | arne_wiebalck: that would be quite helpful I think | 15:50 |
dking | arne_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 |
openstackgerrit | Anuradha proposed openstack/ironic master: Changes to add exception in "default_interface()" function https://review.opendev.org/756682 | 15:51 |
arne_wiebalck | dking: we have a builder host where I spin up the container inside of which I build the IPA images | 15:53 |
arne_wiebalck | dking: the full recipe would probably include the setup of the host, the setup of the container plus the delta to upstream IPA | 15:54 |
arne_wiebalck | IPAB | 15:54 |
dking | arne_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_wiebalck | dking: the colleague who did this is off today, I will check with him tomorrow when he's back | 15:56 |
*** Saf has joined #openstack-ironic | 15:57 | |
*** gyee has joined #openstack-ironic | 16:00 | |
dking | arne_wiebalck: I very much appreciate it. | 16:00 |
rpittau | good night! o/ | 16:01 |
*** lucasagomes has quit IRC | 16:01 | |
*** rpittau is now known as rpittau|afk | 16:01 | |
arne_wiebalck | TheJulia: 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 |
patchbot | patch 756300 - ironic-python-agent - Software RAID: Re-add missing devices - 1 patch set | 16:04 |
openstackgerrit | Dmitry Tantsur proposed openstack/ironic master: PoC call set_boot_device earlier https://review.opendev.org/756881 | 16:09 |
*** Saf has quit IRC | 16:11 | |
TheJulia | arne_wiebalck: that is great news | 16:12 |
dtantsur | \o/ | 16:13 |
TheJulia | I'm going to go to the store, I'll be back a little later | 16:15 |
*** yoctozepto has quit IRC | 16:16 | |
anuradha1904 | TheJulia : 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.py | 16:16 |
*** yoctozepto has joined #openstack-ironic | 16:16 | |
patchbot | patch 756682 - ironic - Changes to add exception in "default_interface()" ... - 3 patch sets | 16:16 |
TheJulia | anuradha1904: documentation is a good question, unfortunately there is also not a straight forward answer! that being said, Let me get you a link | 16:18 |
TheJulia | anuradha1904: 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 DriverLoadTestCase | 16:20 |
TheJulia | In that, you can see where we set the conditions and test the results, some of those cases we test for exceptions | 16:21 |
TheJulia | So 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 |
anuradha1904 | Sure, got it, will look through it! Thank you for your help. | 16:22 |
TheJulia | you 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 run | 16:22 |
TheJulia | and just run those or that specific test | 16:22 |
anuradha1904 | I do use Tox before putting out a patch, and when everything is successful, I add the code for review. | 16:23 |
openstackgerrit | Dmitry Tantsur proposed openstack/ironic master: Call management.set_boot_device before IPA's install_bootloader https://review.opendev.org/756881 | 16:24 |
dtantsur | TheJulia, bfournie, needs testing somehow ^^ | 16:24 |
TheJulia | anuradha1904: excellent | 16:24 |
TheJulia | anuradha1904: even then, we still like to have tests to ensure that the expected exception is raised | 16:25 |
TheJulia | dtantsur: hmm yeah :\ | 16:25 |
anuradha1904 | Sure, Got it now, will add the test, thank you very much for the help. | 16:26 |
*** ricolin_ has quit IRC | 16:29 | |
dtantsur | mgoddard: stable/victoria created for staging-drivers | 16:34 |
mgoddard | thanks dtantsur! | 16:39 |
*** k_mouza has quit IRC | 16:40 | |
*** dtantsur is now known as dtantsur|afk | 16:46 | |
dtantsur|afk | on this positive note I'm wishing you a good rest of the day | 16:46 |
mgoddard | and you dtantsur|afk | 16:49 |
openstackgerrit | Merged openstack/ironic master: migrate testing to ubuntu focal https://review.opendev.org/739448 | 16:59 |
openstackgerrit | Merged openstack/ironic master: Updates ironic documentation https://review.opendev.org/756560 | 17:00 |
*** ociuhandu_ has joined #openstack-ironic | 17:03 | |
*** ociuhandu has quit IRC | 17:06 | |
*** nandita has joined #openstack-ironic | 17:07 | |
*** ociuhandu_ has quit IRC | 17:07 | |
*** shreya11 has joined #openstack-ironic | 17:08 | |
shreya11 | Hello 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 |
nandita | Hello 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 <3 | 17:10 |
*** derekh has quit IRC | 17:16 | |
dking | nandita: 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.html | 17:16 |
dking | nandita: 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 |
nandita | thank 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-ironic | 17:23 | |
*** uzumaki has joined #openstack-ironic | 17:30 | |
*** k_mouza has joined #openstack-ironic | 17:31 | |
*** k_mouza has quit IRC | 17:35 | |
*** mkrai has quit IRC | 17:40 | |
openstackgerrit | Manuel Schönlaub proposed openstack/sushy master: Adds basic support for TaskService to retrieve task information. https://review.opendev.org/756906 | 17:43 |
openstackgerrit | Manuel Schönlaub proposed openstack/sushy master: Adds basic support for TaskService to retrieve task information. https://review.opendev.org/756906 | 17:45 |
*** ociuhandu has joined #openstack-ironic | 17:47 | |
*** penick has quit IRC | 17:49 | |
*** ociuhandu has quit IRC | 17:52 | |
TheJulia | Hello shreya11 and nandita! Welcome! | 17:55 |
TheJulia | shreya11 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 a | 18:02 |
TheJulia | specific 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 IRC | 18:06 | |
*** uzumaki has joined #openstack-ironic | 18:07 | |
openstackgerrit | Manuel Schönlaub proposed openstack/sushy master: Adds basic support for TaskService to retrieve task information. https://review.opendev.org/756906 | 18:29 |
shreya11 | Thanks @TheJulia. Will surely do as told. :) | 18:31 |
TheJulia | Any ironic-cores around? | 18:36 |
iurygregory | TheJulia, I'm just got back from dinner | 18:42 |
iurygregory | something urgent? | 18:42 |
TheJulia | just my change for ci on stable/ussuri | 18:42 |
TheJulia | not super urgent but nobody seems to love it | 18:42 |
iurygregory | shreya11, nandita Welcome! =) feel free to ask questions o/ | 18:42 |
iurygregory | TheJulia, https://review.opendev.org/#/c/755849/ ? | 18:43 |
patchbot | patch 755849 - ironic (stable/ussuri) - CI: Making some of the older jobs more specific - 1 patch set | 18:43 |
iurygregory | approved | 18:44 |
iurygregory | I thought I looked at the ones for CI... I missed this one sorry | 18:44 |
TheJulia | much appreciated | 18:45 |
TheJulia | hmm, I guess I should look at maybe tuning back the standalone jobs or something | 18:45 |
nandita | thank 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 |
anuradha1904 | Hey 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 the | 18:52 |
anuradha1904 | test then? THank you for the help. | 18:52 |
anuradha1904 | 18:52 | |
iurygregory | nandita, I assume it's morning for you, sometimes I'm online during this time =) | 18:56 |
nandita | it's actually 12am (midnight) here. But thank you for letting me know! | 18:58 |
*** jtomasek_ has quit IRC | 19:03 | |
rpioso | TheJulia: Thank you for the reviews :-) | 19:05 |
TheJulia | I'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 |
TheJulia | anuradha1904: 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 node | 19:08 |
nandita | That makes sense @TheJulia. Thank you! | 19:10 |
iurygregory | yeah I'm UTC +2 | 19:10 |
TheJulia | iurygregory: I can never remember because flying there always just destroys me. | 19:11 |
iurygregory | TheJulia, yeah =( | 19:12 |
TheJulia | I 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 |
iurygregory | if the flight is more than 5hours I need to sleep XD | 19:14 |
iurygregory | but my neck will complain ... | 19:15 |
TheJulia | I never sleep heading to Europe. Heading to Asia it is actually more likely | 19:15 |
iurygregory | wow | 19:16 |
anuradha1904 | Hey 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 IRC | 19:19 | |
*** larsks has quit IRC | 19:22 | |
*** uzumaki has joined #openstack-ironic | 19:24 | |
*** priteau has quit IRC | 19:31 | |
*** larsks has joined #openstack-ironic | 19:37 | |
*** larsks has quit IRC | 19:44 | |
TheJulia | you 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-ironic | 19:45 | |
*** hoonetorg has quit IRC | 19:47 | |
*** hoonetorg has joined #openstack-ironic | 20:00 | |
*** iurygregory has quit IRC | 20:00 | |
*** iurygregory has joined #openstack-ironic | 20:02 | |
openstackgerrit | Julia Kreger proposed openstack/metalsmith master: Add a Centos8 UEFI based job https://review.opendev.org/748315 | 20:05 |
openstackgerrit | Merged openstack/ironic stable/victoria: Wiping agent tokens on reboot via API - take 2 https://review.opendev.org/756479 | 20:07 |
anuradha1904 | TheJulia: Thank you very much for the help, will start implementing it. | 20:11 |
*** zzzeek has quit IRC | 20:25 | |
*** zzzeek has joined #openstack-ironic | 20:27 | |
openstackgerrit | Merged openstack/ironic-python-agent stable/ussuri: Log a warning of target_boot_mode does not match current boot mode https://review.opendev.org/756677 | 20:38 |
openstackgerrit | Julia Kreger proposed openstack/ironic-python-agent master: Don't run os-prober from grub2-mkconfig https://review.opendev.org/752308 | 20:49 |
openstackgerrit | Julia Kreger proposed openstack/ironic-python-agent master: Don't run os-prober from grub2-mkconfig https://review.opendev.org/752308 | 20:50 |
openstackgerrit | Verification of a change to openstack/ironic failed: Increase memory of tinyipa vms https://review.opendev.org/747633 | 20:54 |
openstackgerrit | Merged openstack/metalsmith master: metalsmith_deployment role switch to metalsmith_instances https://review.opendev.org/737888 | 21:09 |
openstackgerrit | Julia Kreger proposed openstack/ironic-python-agent master: Support partition image efi contents https://review.opendev.org/748049 | 21:16 |
openstackgerrit | Julia Kreger proposed openstack/ironic-python-agent master: Add fstab pointer to EFI partition https://review.opendev.org/748260 | 21:22 |
janders | good morning Ironic o/ | 21:22 |
openstackgerrit | Julia Kreger proposed openstack/ironic-python-agent master: Don't run os-prober from grub2-mkconfig https://review.opendev.org/752308 | 21:24 |
openstackgerrit | Merged openstack/metalsmith master: CI: Extend job timeout slightly for metalsmith jobs https://review.opendev.org/755798 | 21:24 |
openstackgerrit | Julia Kreger proposed openstack/ironic-python-agent master: Support partition image efi contents https://review.opendev.org/748049 | 21:25 |
openstackgerrit | Julia Kreger proposed openstack/ironic-python-agent master: Add fstab pointer to EFI partition https://review.opendev.org/748260 | 21:25 |
TheJulia | good morning janders | 21:25 |
*** lmcgann has quit IRC | 21:26 | |
openstackgerrit | Verification of a change to openstack/ironic failed: CI: Making some of the older jobs more specific https://review.opendev.org/755849 | 21:40 |
*** kafilat has quit IRC | 22:10 | |
*** rloo has quit IRC | 22:23 | |
*** k_mouza has joined #openstack-ironic | 22:30 | |
*** larsks has quit IRC | 22:31 | |
*** k_mouza has quit IRC | 22:35 | |
*** larsks has joined #openstack-ironic | 22:37 | |
*** k_mouza has joined #openstack-ironic | 22:43 | |
*** rcernin has joined #openstack-ironic | 22:47 | |
*** k_mouza has quit IRC | 22:48 | |
openstackgerrit | Julia Kreger proposed openstack/ironic-inspector master: WIP: Try2... power off the node before inspection https://review.opendev.org/756930 | 22:50 |
TheJulia | erbarr: 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 url | 22:54 |
TheJulia | wrong that is being posted in the results? | 22:54 |
patchbot | patch 756881 - ironic - Call management.set_boot_device before IPA's insta... - 2 patch sets | 22:54 |
*** outbrito has quit IRC | 22:55 | |
* TheJulia lets CI run on patches | 22:57 | |
openstackgerrit | Julia Kreger proposed openstack/ironic-python-agent master: Don't run os-prober from grub2-mkconfig https://review.opendev.org/752308 | 22:59 |
openstackgerrit | Julia Kreger proposed openstack/ironic-python-agent master: Support partition image efi contents https://review.opendev.org/748049 | 22:59 |
openstackgerrit | Julia Kreger proposed openstack/ironic-python-agent master: Add fstab pointer to EFI partition https://review.opendev.org/748260 | 22:59 |
TheJulia | now CI should run :\ | 22:59 |
*** tosky has quit IRC | 22:59 | |
openstackgerrit | Merged openstack/ironic-python-agent master: Reduce the duration of retries in the inspector tests https://review.opendev.org/756488 | 23:04 |
openstackgerrit | Merged openstack/ironic-inspector master: [DOC] Add note regarding ``[pxe_filter]deny_unknown_macs`` https://review.opendev.org/756583 | 23:04 |
*** pmannidi has joined #openstack-ironic | 23:23 | |
*** anuradha1904 has quit IRC | 23:35 | |
*** JamesBenson has quit IRC | 23:48 | |
*** JamesBenson has joined #openstack-ironic | 23:48 | |
*** gyee has quit IRC | 23:50 | |
*** k_mouza has joined #openstack-ironic | 23:59 |
Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!