Thursday, 2020-10-22

*** gyee has quit IRC00:24
*** seongsoocho has quit IRC00:43
*** seongsoocho has joined #openstack-ironic00:43
*** stevebaker has quit IRC01:03
*** Goneri has quit IRC01:04
*** Qianbiao has joined #openstack-ironic01:22
TheJuliagood morning01:37
TheJuliaerbarr: looks like dell jobs are failing with "/tmp/molteniron_checkin.sh: not found"01:41
erbarrTheJulia, yea, there's an issue with the DNS, I tried switching to another one but seems no good either01:43
TheJuliaerbarr: dns howso?01:43
erbarrinternal changes01:44
TheJuliaahh :(02:05
TheJuliaOkay, hopefully that gets resolved quickly :)02:05
*** Qianbiao has quit IRC02:06
*** Qianbiao has joined #openstack-ironic02:07
openstackgerritQianBiao Ng proposed openstack/ironic stable/train: opt: Enhance old stable branches to use latest python-ibmcclient  https://review.opendev.org/75760602:09
openstackgerritArmstrong Liu proposed openstack/ironic-python-agent master: Add an option about collecting network information  https://review.opendev.org/75895002:12
*** mcarden has quit IRC02:18
*** rh-jelabarre has quit IRC02:20
*** rh-jelabarre has joined #openstack-ironic02:21
openstackgerritMerged openstack/ironic master: Change auth type to none in dev-quickstart  https://review.opendev.org/75847602:24
*** tzumainn has quit IRC02:27
*** larsks has quit IRC02:36
openstackgerritMerged openstack/ironic-python-agent-builder master: Add documentation about the available Elements  https://review.opendev.org/75850802:37
openstackgerritMerged openstack/bifrost master: Force local connection for localhost  https://review.opendev.org/75881402:37
openstackgerritMerged openstack/bifrost master: Set safe version of hacking  https://review.opendev.org/75877102:37
*** larsks has joined #openstack-ironic02:50
*** rcernin has quit IRC02:52
*** rcernin has joined #openstack-ironic02:56
*** larsks has quit IRC02:59
TheJuliaCalling it a night, ironic on the spreadsheet just needs a second set of eyes03:01
*** larsks has joined #openstack-ironic03:02
*** larsks has quit IRC03:05
*** larsks has joined #openstack-ironic03:08
*** larsks has quit IRC03:09
*** larsks has joined #openstack-ironic03:12
QianbiaoTheJulia need an eye for what, I can become the eye :)03:43
TheJuliaQianbiao: audit git changes going back to October 1. A few of the cores have done an awesome job already of digging through the spreadsheet I sent to the mailing list, adding entries and checking if everything looks good03:48
QianbiaoI see, TheJulia.03:51
QianbiaoIt seems only networking-baremetal and networking-generic-switch is not confirmed.03:51
TheJuliaQianbiao: I don't think they had any commits in the window of concern04:12
TheJuliaAnyway, I'm going to get some sleep. I have been getting up at 5 AM local all week. :(04:12
Qianbiaosure, good sleep04:12
*** rh-jelabarre has quit IRC06:05
*** rh-jelabarre has joined #openstack-ironic06:06
*** jtomasek has joined #openstack-ironic06:20
openstackgerritPierre Riteau proposed openstack/bifrost stable/victoria: Force local connection for localhost  https://review.opendev.org/75918806:25
openstackgerritArmstrong Liu proposed openstack/ironic-python-agent master: Add an option about collecting network information  https://review.opendev.org/75895006:45
iurygregorygood morning Ironic!06:48
jandersgood morning iurygregory06:49
*** uzumaki has joined #openstack-ironic06:51
*** rpittau|afk is now known as rpittau06:54
rpittaugood morning ironic! o/06:54
*** jawad_axd has joined #openstack-ironic06:54
rpittauhappy thursday!06:54
jandersgood morning rpittau06:55
rpittauhey janders :)06:55
iurygregorymorning rpittau o/06:58
rpittauhey iurygregory :)06:58
* iurygregory have the feeling that arne_wiebalck is late today...06:58
arne_wiebalckGood morning, ironic!07:02
* arne_wiebalck is late today :-D07:02
jandersgood morning arne_wiebalck07:02
arne_wiebalckHey janders iurygregory rpittau o/07:02
rpittauhey arne_wiebalck :)07:02
* iurygregory used his power and arne_wiebalck appeared07:02
iurygregory*magic*07:02
* arne_wiebalck would rather like if iurygregory would use this power to magically make some holidays appear07:03
iurygregoryI did07:03
iurygregorynext week there is 1 holiday for me07:04
iurygregory=D07:04
rpittau:)07:04
iurygregoryI'm sorry I can't share lol07:04
rpittaumy holiday was eaten by the bad sunday monster :/07:04
iurygregoryouch =(07:04
rpittauluckily there's another 10 days later :D07:05
jandersI wish the miracle of compound interest applied to accumulated leave07:06
* iurygregory has 1 next wed and one on 17th nov...07:11
*** smutugrigg has joined #openstack-ironic07:24
*** DjeufackZane has joined #openstack-ironic07:24
openstackgerritRiccardo Pittau proposed openstack/python-ironic-inspector-client master: Set safe version of hacking  https://review.opendev.org/75876407:26
*** uzumaki has quit IRC07:29
*** uzumaki has joined #openstack-ironic07:33
smutugriggHi ironic o/07:35
uzumakihey there! smutugrigg o/07:36
uzumakiMorning iurygregory arne_wiebalck janders o/07:36
smutugrigghi uzumaki o/07:37
uzumakihow you doing smutugrigg ?07:38
*** ociuhandu has joined #openstack-ironic07:41
*** fgofurov has joined #openstack-ironic07:42
smutugriggtrying to work out some changes I need to make but would really like some guidance as I don't want to make a mistake07:42
*** rcernin has quit IRC07:42
smutugriggsorry for uzumaki07:43
*** tosky has joined #openstack-ironic07:44
smutugrigghi iurygregory, could I please request some assitance if you are there please?07:44
smutugriggor TheJulia?07:45
*** rcernin has joined #openstack-ironic07:48
*** alexmcleod has quit IRC07:50
*** alexmcleod has joined #openstack-ironic07:50
iurygregorysmutugrigg, good morning o/07:52
iurygregorysure!07:52
*** rcernin has quit IRC07:54
*** anuradha1904 has joined #openstack-ironic07:57
openstackgerritEmmanouil Bagakis proposed openstack/ironic-python-agent master: Updated Implementation of string interpolation delay on LOG messages  https://review.opendev.org/75920307:58
smutugrigggood morning iurygregory, thank you!07:59
smutugriggDoes the comment "doc/source/admin/node-deployment.rst08:00
smutugriggLine 106:08:00
smutugrigghuh, this file has a different convention, $node_ident instead of <node> everywhere else. If there is a another revision to this change it would be good to make this consistent" require any action from me?08:00
iurygregorylet me check the patch08:00
smutugriggThank you08:00
*** rcernin has joined #openstack-ironic08:00
iurygregorysmutugrigg, oh Julia already +2 I just saw. We can merge your change and you can push a new patch with the suggestions if you can =)08:02
smutugriggOh? I made all the other changes ;)08:02
iurygregoryyou can submit normally, but instead of "git commit --amend"08:03
smutugriggI would really like it if the original change could be merged and happy to work on a new change to make all the suggested changes08:03
iurygregoryyou can just add the changes and do git commit08:03
iurygregoryso it will generate a new one08:03
smutugriggperfect08:04
smutugriggwill hold the changes for now then?08:04
openstackgerritMerged openstack/ironic-python-agent-builder master: Set safe version of hacking  https://review.opendev.org/75876608:04
iurygregoryit should be fine to send the new patch if you already did the changes =)08:04
smutugriggDo I need to follow up with Kaifeng on their review notes on 756708 before the change is ready to submit?08:05
iurygregorylet me check08:06
smutugriggsure08:09
iurygregorysmutugrigg, you mean to switch places to <node> ?08:09
*** lucasagomes has joined #openstack-ironic08:10
smutugriggiurygregory, I was meaning are they happy with all the changes or are your and TheJulia's review sufficient?08:11
*** rcernin has quit IRC08:12
iurygregorysmutugrigg, it should be fine =) try to address the comments on the last patch in a new one08:12
smutugriggGreat, I will do.  Shall I create a new task in https://storyboard.openstack.org/#!/story/2008194 to track the changes and commit to it?08:13
iurygregoryyou can =)08:14
iurygregoryor you can just re-use the same task number08:15
iurygregoryand explain in the commit message08:15
smutugriggOk happy to do that08:15
smutugriggiurygregory, who needs to complete the verify the check to allow me to submit?08:17
iurygregoryyou can submit normally, people will try to review when they have time =)08:17
smutugrigghttps://review.opendev.org/#/c/756708/5 is still in status "Needs verified Label"08:19
patchbotpatch 756708 - ironic - Update .rst files - 5 patch sets08:19
smutugriggDoes someone need to verify check before I am able to submit?08:19
iurygregoryoh this is done by zuul08:20
iurygregoryafter the patch has Workflow +108:21
smutugriggAh, sorry was not aware of that.08:21
smutugriggSo I just wait until Zuul has completed and will be able to submit?08:21
*** rcernin has joined #openstack-ironic08:21
iurygregoryif you look at https://zuul.openstack.org/status and you add 756708 to the search field on the left of the page you can see =)08:21
smutugriggthank you08:23
*** rcernin has quit IRC08:26
*** sshnaidm|afk is now known as sshnaidm|rover08:29
openstackgerritRiccardo Pittau proposed openstack/python-ironic-inspector-client master: Set safe version of hacking  https://review.opendev.org/75876408:30
*** DjeufackZane has quit IRC08:33
*** derekh has joined #openstack-ironic08:38
openstackgerritMerged openstack/ironic-prometheus-exporter master: Set safe version of hacking  https://review.opendev.org/75878908:46
openstackgerritMerged openstack/ironic-python-agent master: Set safe version of hacking  https://review.opendev.org/75876508:46
openstackgerritMerged openstack/ironic master: Update .rst files  https://review.opendev.org/75670808:46
*** ddurst has quit IRC08:48
*** ddurst has joined #openstack-ironic08:55
*** dtantsur|afk is now known as dtantsur08:56
dtantsurmorning ironic08:56
*** iurygregory has quit IRC08:57
rpittauhey dtantsur :)08:58
*** uzumaki has quit IRC09:10
*** uzumaki has joined #openstack-ironic09:10
*** rcernin has joined #openstack-ironic09:21
*** rcernin has quit IRC09:25
*** iurygregory has joined #openstack-ironic09:28
fgofurovHello folks, good morning/afternoon! I have a question and highly appreciate if someone can help/guide me through this09:36
fgofurovThe problem we are facing is with ironic states. For example when the image server does not answer, then the adoption process just hanging forever. We know that it hangs when ironic tries to validate the image usually, but apparently this is not always and sometimes we don't know at what stage it gets stuck/fails silently without changing the state.09:37
fgofurovWe want to ensure that all operations done for the adoption have a timeout and that adoption would always fail if something hangs for too long, and not get stuck in "adopting" state. We can recover from an "adopt failed" state, but the problem for us is recovering from "adopting" state as it does not fail and it just hangs. Any suggestions/pointers appreciated.09:38
fgofurovAs far as I understood, we can retry only if it is in adoption failed state, but to go to this state "adopting" should fail(normally), but in our case it is not failing, it just stuck/hangs in "adopting"..09:41
*** k_mouza has joined #openstack-ironic09:44
openstackgerritMichelle (Shelly) Mutu-Grigg proposed openstack/ironic master: Update .rst files  https://review.opendev.org/75923609:51
openstackgerritMichelle (Shelly) Mutu-Grigg proposed openstack/ironic master: Update .rst files  https://review.opendev.org/75923609:53
smutugrigghi iurygregory, just to let you know, I have now committed the suggested additional changes under change https://review.opendev.org/#/c/759236/ but still pointing to task 4096409:54
patchbotpatch 759236 - ironic - Update .rst files - 2 patch sets09:54
iurygregorysmutugrigg, cool tks!09:55
smutugriggI will try to make an additional update before the deadline if I have time with class.09:55
smutugriggiurygregory, thanks for all your help.09:55
iurygregorynp09:56
smutugriggbye ironic o/09:56
dtantsurfgofurov: hi. Mmm, we need to make sure that all calls to all services have an HTTP timeout.09:56
dtantsurwe've started down that path a while ago, but I guess never completed it.09:56
*** smutugrigg has quit IRC09:57
fgofurovdtantsur: thanks for reply. Do you mean there is an ongoing work? or should I open a ticket, could you please elaborate more a bit?10:00
rpittaufgofurov: maybe opening a story in storyboard with some details would help10:02
openstackgerritlikui proposed openstack/ironic-python-agent master: update lower-constraints.txt  https://review.opendev.org/75923810:03
fgofurovrpittau: sure, definitely, I will do it. Just wondering if we can complete the work regarding this that was not completed as dtantsur was mentioning and and how can we contribute to this from metal3 side10:07
dtantsurfgofurov: I guess it requires some code on the ironic side10:11
dtantsuralthough, there may be just configuration options to change that; maybe we need better defaults10:11
rpittaufgofurov: not sure there's anything that can be done from pure metal3 side10:11
dtantsurmmm, I see, there should be a configurable timeout here: https://opendev.org/openstack/ironic/src/branch/master/ironic/common/image_service.py#L10110:12
rpittaummm I just noticed a little issue with one of the ipa jobs, we need to approve this https://review.opendev.org/757812 ASAP10:19
patchbotpatch 757812 - ironic-python-agent - Run dib ipa src jobs on ubuntu focal - 5 patch sets10:19
rpittauthe ipa-tempest-bios-ipmi-direct-src doesn't always run DIB10:19
rpittauif it gets rax hosts will run tinyipa10:19
rpittauon bionic10:19
rpittaufailing because the kernel is too old for the chroot10:20
iurygregoryrpittau, +210:20
rpittauthanks10:20
iurygregoryfunny10:20
iurygregory:D10:20
rpittauheh10:20
fgofurovdtantsur: yes, seems either timeout needed for "adopting" or we can have a timeout for retrieval of disk images as you pointed out10:20
iurygregoryold kernel etc10:20
rpittauiurygregory: yeah, it's because we're running tinycore 11 now with kernel 5.x10:21
rpittaubionic has kenerl 4.x10:21
rpittauor kernel even10:21
iurygregoryyeah I'm aware of that =D10:21
rpittauoh and this too please https://review.opendev.org/75781110:26
patchbotpatch 757811 - ironic - Use centos as base element for dib images - 2 patch sets10:26
dtantsurfgofurov: I can look into that if you're not planning to10:30
iurygregorydtantsur, I have the feeling openstacksdk CI hates your patch :X10:31
dtantsurCI often hates my patches10:31
iurygregory=(10:32
openstackgerritDmitry Tantsur proposed openstack/ironic master: DNM try to inject the correct grub location  https://review.opendev.org/75880910:37
*** rcernin has joined #openstack-ironic10:43
fgofurovdtantsur: I am happy with that, please do. I will be around and let me know please for coming fixes.. Thanks a lot:)10:44
fgofurovdtantsur: I will open a story a paste a link here to track it10:45
fgofurov*and ^^10:45
*** rcernin has quit IRC10:48
fgofurovdtantsur, rpittau: the story opened for adoption failing here: https://storyboard.openstack.org/#!/story/200826910:49
rpittauit's probably my paranoia that speaks but don't the two commits https://opendev.org/openstack/python-ironicclient/commit/c09eec0025044b4fe359d39cbcded328f484f56a and https://opendev.org/openstack/python-ironicclient/commit/0e4a23393f8e4b24fa9da078dfe0af24e48088c2 look exactly the same ?10:50
rpittauthanks fgofurov10:51
dtantsurrpittau: ehhhmmmm10:54
rpittauam I being paranoid ?10:55
rpittauhttps://review.opendev.org/#/c/751649/ = https://review.opendev.org/#/c/751649/10:55
patchbotpatch 751649 - python-ironicclient - Remove install unnecessary packages (MERGED) - 1 patch set10:55
patchbotpatch 751649 - python-ironicclient - Remove install unnecessary packages (MERGED) - 1 patch set10:55
rpittauwhoops10:55
dtantsurhttps://review.opendev.org/#/c/753922/10:55
patchbotpatch 753922 - python-ironicclient - Remove install unnecessary packages (MERGED) - 1 patch set10:55
rpittauyeah10:56
dtantsurbut yeah, the reviews also look nearly identical: the same reviewers in the same order10:56
rpittauyeeep10:56
* rpittau paranoia intensifies10:56
dtantsurI mean, it's possible in a normal flow10:56
dtantsurthe changes are identical, the 2nd one will result in an empty merge conflict10:57
rpittaummmm10:57
dtantsurthe reviewers matching looks weird, but also not impossible10:57
dtantsurugh s/merge conflict/merge commit/10:57
rpittauthey are 4 days apart, the merge I mean10:57
dtantsurit's probably worth reporting to the admins for double-checking, but I support it's just a funny coincidence10:58
dtantsur(resulting from a lot of people using the same scripts for auto-generating tons of patches)10:58
iurygregoryzuul wouldn't go crazy?10:58
iurygregoryO.o10:58
iurygregoryomg10:59
dtantsurmerging them will make one normal merge commit, then one empty10:59
dtantsuryou get a conflict only when changes are different10:59
dtantsuryou can try it locally to confirm10:59
rpittaudtantsur: how do we report that ?11:00
dtantsurrpittau: no idea, try #opendev?11:00
rpittauack11:00
iurygregoryI do remember reviewing the at least one of the patches...11:05
iurygregorybut its a bit strange...11:06
rpittauI posted on #opendev let's see what they say11:07
*** jawad_axd has quit IRC11:11
openstackgerritDmitry Tantsur proposed openstack/ironic master: Add timeout to image operations in the direct deploy  https://review.opendev.org/75924511:17
dtantsurfgofurov: ^^11:17
fgofurovdtantsur: thanks a lot for your quick response11:30
openstackgerritEmmanouil Bagakis proposed openstack/ironic-python-agent master: This is a follow up to https://review.opendev.org/#/c/756300/  https://review.opendev.org/75924611:37
*** k_mouza_ has joined #openstack-ironic11:43
*** k_mouza has quit IRC11:43
*** k_mouza has joined #openstack-ironic11:44
*** k_mouza_ has quit IRC11:47
openstackgerritVerification of a change to openstack/ironic failed: Run bifrost integration job on focal  https://review.opendev.org/75819811:48
*** sai438 has quit IRC11:55
*** pmannidi has joined #openstack-ironic11:57
*** k_mouza_ has joined #openstack-ironic12:07
*** Goneri has joined #openstack-ironic12:08
*** k_mouza has quit IRC12:10
*** uzumaki has quit IRC12:12
*** rloo has joined #openstack-ironic12:12
*** uzumaki has joined #openstack-ironic12:12
*** k_mouza has joined #openstack-ironic12:13
*** rcernin has joined #openstack-ironic12:17
*** k_mouza_ has quit IRC12:17
*** jawad_axd has joined #openstack-ironic12:19
openstackgerritManuel Schönlaub proposed openstack/sushy master: Adds basic support for TaskService to retrieve task information.  https://review.opendev.org/75690612:19
*** rcernin has quit IRC12:22
*** k_mouza has quit IRC12:29
*** k_mouza has joined #openstack-ironic12:29
TheJuliagood morning12:36
iurygregorygood morning TheJulia =)12:36
dtantsurTheJulia: morning!12:36
iurygregorycorgi overlord will join the sessions today also? \o/12:37
TheJuliadunno12:38
TheJuliaReminder, in about ?22? minutes, we're going to be revisiting https://summit.openinfra.dev/a/event/24757 because of things kind of cratering on Monday12:38
TheJuliatrandles: janders: https://summit.openinfra.dev/a/event/24539 in 45 minutes12:39
openstackgerritDmitry Tantsur proposed openstack/ironic master: DNM try to inject the correct grub location  https://review.opendev.org/75880912:40
iurygregoryTheJulia, the spaghetti session will be on zoom right?12:51
dtantsuranyone has had success with devstack on centos8? is it worth trying?12:51
TheJuliaiurygregory: yes, but hopefully we will have streaming working12:52
iurygregoryTheJulia, cool12:52
rpittaudtantsur: I did make it work a while ago, if memory serves the changes on devstack itself are not that much but it took some effort to make ironic run12:55
dtantsursigh..12:55
dtantsurI'm starting to think it's easier to make UEFI work on bifrost12:56
rpittauheh :/12:57
dtantsur.. or switch to ubuntu :(12:57
jandersTheJulia trandles fingers crossed the platform stands up to the challange this time :)13:02
openstackgerritEmmanouil Bagakis proposed openstack/ironic-python-agent master: Updated Implementation of string interpolation delay on LOG messages  https://review.opendev.org/75920313:05
*** uzumaki has quit IRC13:06
rpiosoGood morning, ironic13:07
rpiosoA new patch set was uploaded yesterday for https://review.opendev.org/#/c/740721/. ajya and I would appreciate folks reading it before the PTG discussion on Tuesday. Review comments are welcome, too :-)13:09
patchbotpatch 740721 - ironic-specs - Pre-defined system hw configuration in single step - 17 patch sets13:09
rpiosodtantsur, TheJulia: ^^^13:09
*** jawad_axd has quit IRC13:28
*** ayoung has quit IRC13:41
*** rcernin has joined #openstack-ironic13:41
*** tzumainn has joined #openstack-ironic13:42
*** rcernin has quit IRC13:45
janderssee you tomorrow Ironic o/13:48
rpittaubye janders :)13:49
arne_wiebalckThanks for the presentation janders, TheJulia, trandles !13:49
TheJuliaHopefully it was informative and didn't seem silly?!13:50
TheJuliaThat was the 3rd or 4th take on the second day13:50
*** sshnaidm|rover is now known as sshnaidm|afk13:54
arne_wiebalckIt definitely was (informative, not silly ;)13:55
*** k_mouza has quit IRC14:01
*** weshay|ruck has left #openstack-ironic14:03
QianbiaoHello, Ironic team.14:05
QianbiaoI got this when visit docs https://pasteboard.co/JwPCdjQ.png14:05
QianbiaoIt may cause a security issue (display folder)14:05
QianbiaoNot sure, just a remind14:06
trandlesthanks arne_wiebalck14:07
*** k_mouza has joined #openstack-ironic14:07
dtantsurQianbiao: wow14:13
dtantsurQianbiao: could you tell it to #openstack-infra with a link to where you see it?14:13
Qianbiaook14:13
rpittauyeah this is not good -> https://docs.openstack.org/diskimage-builder/latest/elements/14:14
iurygregoryyeah14:15
iurygregoryhttps://docs.openstack.org/diskimage-builder/latest/elements.html is ok14:16
TheJuliaI guess it is because there is no index.rst file in the elements folder pointing to ../elements.rst14:18
QianbiaoTheJulia hmm, i think if there are no elements.rst, it should be 404.14:20
Qianbiaomaybe "index on" option is set in the static file server(like apache or nginx)14:21
Qianbiaois the Anaconda in our talking today the https://www.anaconda.com/?14:21
QianbiaoI can not image how it has connection with deploy interface.14:21
dtantsurQianbiao: no, the installer usually used in Red Hat systems14:21
dtantsurhttps://fedoraproject.org/wiki/Anaconda14:22
Qianbiaoaha ty dtantsur14:22
TheJuliapossibly, the tempalte to content generation behavior is a little weird and ath eserver settings may be a little weird14:22
TheJuliaI agree, indexes it shouldn't be doing14:22
QianbiaoTheJulia yeah14:23
QianbiaoI have no idea with a lot of words during the talking, google will be buzy now.. :)14:23
TheJuliaheh14:24
TheJuliaoddly enough, I felt the revisited topic, while kind of hard to keep us moving forward we seemed to actually have more thoughts/ideas/comments raise to the surface14:27
openstackgerritQianBiao Ng proposed openstack/ironic stable/stein: opt: Enhance old stable branches to use latest python-ibmcclient  https://review.opendev.org/75760714:33
TheJuliabefore I pull it up, how does the spreadsheet look for auditing the reviews/changes that got approved?14:39
rpittauTheJulia: not bad, we're still missing some14:41
*** sshnaidm|afk is now known as sshnaidm|rover14:48
TheJuliaawesome14:53
* TheJulia wonders if rover means a mars rover is in use14:53
*** jawad_axd has joined #openstack-ironic14:55
Qianbiaomaybe rover in dream :)14:57
Qianbiaoneed coffee14:57
openstackgerritDmitry Tantsur proposed openstack/ironic master: DNM try to inject the correct grub location  https://review.opendev.org/75880914:59
TheJuliaI made fresh coffee a little while ago and I don't remember what I did with it15:03
TheJulia#badsign15:03
*** jawad_axd has quit IRC15:05
trandlesTheJulia: I did the exact same thing this morning. For some reason I took it with me into the garage when I was putting stuff in the recycling bin.15:07
trandlesFound it sitting on the top of my wife's car...15:08
Qianbiaoneed to you, have a good night, Ironic and everyone :)15:08
*** Qianbiao is now known as Qianbiao|afk15:09
*** Qianbiao|afk has quit IRC15:09
openstackgerritDmitry Tantsur proposed openstack/bifrost master: Add CI jobs with Redfish and UEFI  https://review.opendev.org/74443415:09
openstackgerritManuel Schönlaub proposed openstack/sushy master: Add oem_vendors from resource links  https://review.opendev.org/75532315:23
openstackgerritManuel Schönlaub proposed openstack/sushy master: Add oem_vendors from resource links  https://review.opendev.org/75532315:24
*** gregwork has joined #openstack-ironic15:25
openstackgerritDmitry Tantsur proposed openstack/bifrost master: Add CI jobs with Redfish and UEFI  https://review.opendev.org/74443415:26
openstackgerritAnuradha proposed openstack/ironic master: Changes to add exception in "default_interface()" function  https://review.opendev.org/75668215:31
openstackgerritDmitry Tantsur proposed openstack/bifrost master: Add CI jobs with Redfish and UEFI  https://review.opendev.org/74443415:36
*** rcernin has joined #openstack-ironic15:41
*** rcernin has quit IRC15:46
*** DjeufackZane has joined #openstack-ironic15:48
rpittauGood night! o/15:57
*** rpittau is now known as rpittau|afk15:57
arne_wiebalckbye everyone o/15:59
*** lucasagomes has quit IRC16:00
*** w14161 has quit IRC16:02
*** w14161 has joined #openstack-ironic16:02
*** k_mouza has quit IRC16:06
*** k_mouza_ has joined #openstack-ironic16:06
*** k_mouza_ has quit IRC16:10
dtantsurtzumainn: hey! you're referring to a Ceph iSCSI patch in your talk, do you have a link for it?16:17
dtantsur(and great talk btw)16:17
tzumainndtantsur, thanks! and yes I do: https://review.opendev.org/#/c/662829/16:17
patchbotpatch 662829 - cinder - Add ceph iscsi volume driver - 39 patch sets16:17
tzumainnwe've tested it a bit, and it seeeeems to work; the author says they're just waiting to set up CI to merge it16:17
dtantsurtzumainn: very nice! could you let us know when it merges (and works)? it's a highly demanded thing for ironic.16:18
dtantsur(dunno if you're using it in the ironic context or not)16:18
tzumainndtantsur, we are using it in an ironic context, and I'll definitely let you know!16:19
dtantsurthx!16:19
dtantsurre simplified CLI, I'd love to have `baremetal port list` list node UUIDs16:19
dtantsurcontributions welcome :)16:19
tzumainndtantsur, there is one odd behavior, and I don't know if it's because I set the networking up weirdly, but - if the ceph iscsi gateway is on the provisioning network, and we start by attaching our node to the provisioning network before deploying16:20
tzumainnthen ironic will detach the provisioning network from the node and delete the neutron port and then fail16:20
tzumainnthat doesn't happen if we use the non-provisioning network16:21
dtantsuryeah, ironic cleans up everything on the provisioning network before deploying16:21
dtantsurto make sure there's no artefacts from previous deployments left16:21
dtantsurhow critical is it for you?16:21
tzumainngotcha - so should we just not use the provisioning network for the ceph iscsi gateway?16:21
tzumainnwell, we worked around it for purposes of testing the patch16:22
tzumainnand I don't think we *have* to use the provisioning network in that way - I'll talk to the moc folks16:22
dtantsurtzumainn: I'd avoid doing that if possible16:23
tzumainndtantsur, okay, I'll pass along the recommendation - it'll be stronger with you guys backing me up :)16:24
tzumainnthanks!16:24
tzumainndtantsur, so regarding the CLI, would you guys also be interested in stuff like a more streamlined way of attaching a network to a node, or is that sort of stuff too much?16:25
dtantsurtzumainn: it may be problematic to cross the service boundary (this is why metalsmith exists)16:27
tzumainndtantsur, yeah, I suspected as much - makes sense!16:28
*** ociuhandu_ has joined #openstack-ironic16:28
dtantsurmaybe extending metalsmith may work..16:28
dtantsurjust food for thought :)16:28
* dtantsur needs actual food now16:28
fgofurovBye everyone o/16:30
*** fgofurov has quit IRC16:30
*** dtantsur is now known as dtantsur|afk16:31
dtantsur|afko/16:31
*** ociuhandu has quit IRC16:32
*** ociuhandu_ has quit IRC16:32
*** valleedelisle has joined #openstack-ironic16:39
*** DjeufackZane has quit IRC16:51
*** derekh has quit IRC17:00
*** jtomasek has quit IRC17:07
*** valleedelisle has quit IRC17:13
TheJuliaAnyone know of any good tripleo examples that cover bare metal as a service use cases?17:16
*** jawad_axd has joined #openstack-ironic17:17
*** valleedelisle has joined #openstack-ironic17:24
*** jawad_axd has quit IRC17:32
*** mrquincy has joined #openstack-ironic17:40
mrquincyre: OpenStack Summit, I was hoping to catch the Ironic presentations after the fact but the platform seems kinda broken. Anyone have working links to the presentations: "Deployment with bare metal system configuration molds in Ironic" and "Plans and Directions for the Bare Metal SIG"?17:43
dkingDoes the ironic-api service respond to any environment varables?17:44
clarkbmrquincy: I've asked people who should know. Will let you know what they say17:44
mrquincyThanks, I appreciate it17:45
*** sshnaidm|rover is now known as sshnaidm|off17:45
dkingFor instance, is there any way to be able to set, for example, the database connection in an environment variable before running /usr/local/bin/ironic-api ?17:48
clarkbmrquincy: https://summit.openinfra.dev/a/event/24761 and https://summit.openinfra.dev/a/event/24754 are links to the two sessions you asked about. I'm told they are working on the video uploads as we speak for forum sessions. Sounds like they hope to have them up by tomorrow17:52
mrquincyoh ok, that's good that the videos exist somewhere. I'll check back later.17:53
mrquincyThanks agian17:53
TheJuliadking: not _really_ the expectation is to use the configuration file. I guess patches to oslo_config may be appreciated in that regard17:54
*** ricolin has quit IRC17:54
openstackgerritAnuradha proposed openstack/ironic master: Rewrite devstack/tools/ironic/scripts/create-node.sh in Python  https://review.opendev.org/75876218:19
*** KurtB has quit IRC18:35
openstackgerritMerged openstack/ironic master: Run bifrost integration job on focal  https://review.opendev.org/75819818:39
*** happy-contrib-43 has joined #openstack-ironic18:41
dkingTheJulia: Thank you. I just wanted to confirm. I'm back to trying to put Ironic in a container. At the moment, I'm doing some hacks with the entrypoint to modify the config with the variables. I'll make a note to check with oslo_config later.18:55
happy-contrib-43I'm looking to extend sushy-tools to support sessions. Is that something you guys would be interested in including in the project?19:04
*** eandersson has quit IRC19:10
*** ccstone has quit IRC19:10
*** ccstone has joined #openstack-ironic19:11
*** eandersson has joined #openstack-ironic19:11
openstackgerritMerged openstack/virtualbmc master: Use wallaby jobs  https://review.opendev.org/75876719:13
*** gyee has joined #openstack-ironic19:26
*** ayoung has joined #openstack-ironic19:35
ayoungI registered 3 nodes with ironic, and IPMI does not power cycle them.  THe difference between a working impitool command and the one generated by ironic is the work chassis.  The machines are dell i610s.  Should I have created the nodes inside a chassis structure in my nodes.json file?19:36
*** stevebaker has joined #openstack-ironic19:36
*** rcernin has joined #openstack-ironic19:43
*** rcernin has quit IRC19:48
ayoungHmmm, nope. Seems like it still does not put the chassis in the command.19:57
ayoungIs that a driver config option?19:59
TheJuliablarg20:08
TheJuliaokay, re: audit, bifrost, metalsmith, pyghmi and ironic-staging-drivers only need a second set of eyes20:09
TheJuliaotherwise we're done and good it looks like20:09
TheJuliahappy-contrib-43: I'm sure we would accept the patches to sushy-tools20:09
TheJuliaayoung: huh, could you share an example of what your seeing?20:10
*** akiselev has joined #openstack-ironic20:10
ayoungIN the conductor log I see the following command20:11
ayoungCommand: ipmitool -I lanplus -H 192.168.123.10 -L ADMINISTRATOR -R 1 -N 1 -f /tmp/tmpxbp36v09 power status20:11
ayoungI run that (with my own password file) and I get20:11
ayoungError: Unable to establish IPMI v2 / RMCP+ session20:12
ayoungbut add the word chassis like this20:12
ayoung ipmitool -I lanplus -H 192.168.123.10 -L ADMINISTRATOR  -R 1  -N 1 -U root -f ~/ipmi.password chassis power status20:12
ayoungand it works20:12
* TheJulia looks for her jaw on the floor20:14
trandleson my testbed it's `chassis power status` otherwise I get an error (regardless of password)20:17
TheJuliaI wonder if it is a version/fork issue20:18
TheJuliabecause on my debian machine, "power status" works just fine20:18
TheJuliaand in the help it says it is just as hortcut to power status command20:19
mrquincyCan I run Metal3 over ironic-standalone?20:22
TheJuliamrquincy: functionally metal3 runs ironic-standalone, there is apparently a way to run it separately20:24
TheJuliatrandles: ayoung: ipmitool version, package version, and distribution please!20:24
*** yolanda has quit IRC20:24
* TheJulia has started the great hunt through code and now the hunt through downstreams20:24
*** yolanda has joined #openstack-ironic20:24
ayoungI'm on a RHEL 8 box...so....20:25
ayoung$ rpmquery -f `which ipmitool `20:26
ayoungipmitool-1.8.18-14.el8.x86_6420:26
ayoung[ayoung@nuzleaf nodes]$ ipmitool -I lanplus -H 192.168.123.10 -L ADMINISTRATOR  -R 1  -N 1 -U root -f ~/ipmi.password chassis power status20:27
ayoungChassis Power is off20:27
ayoung[ayoung@nuzleaf nodes]$ ipmitool -I lanplus -H 192.168.123.10 -L ADMINISTRATOR  -R 1  -N 1 -U root -f ~/ipmi.password power status20:27
ayoungChassis Power is off20:27
ayoungwtf20:27
ayoung[ayoung@nuzleaf nodes]$ ipmitool -I lanplus -H 192.168.123.10 -L ADMINISTRATOR -R 1 -N 1 -f ~/ipmi.password power status20:27
ayoungError: Unable to establish IPMI v2 / RMCP+ session20:27
ayoungaH20:27
ayoungI bet it is not that....sorry, I read that line 30 times and just now noticed the -U option20:28
TheJuliaheh20:28
ayoungNOT the chassis thing20:28
trandlesoh no, I haven't been on my testbed since Monday and they did work in the data center...I can't reach it. As soon as it's back online I'll get you the data TheJulia20:28
TheJuliaayoung: so you should be loggintg the username, so if you could provide the "baremetal node show" output20:29
TheJuliathat would be useful20:29
TheJuliasince usernames/passwords in ipmi are optional20:29
TheJulia*bangs head into desk*20:29
TheJuliatrandles: no worries20:29
* ayoung knows what he did wrong20:30
ayoungI'll confirm but20:30
ayoung(reverse-i-search)`for': for UUID in `openstack baremetal node list -f json | jq -r '.[] | .UUID' ` ; do  openstack baremetal node set $UUID --driver-info ipmi_password=`cat ~/ipmi.password`  --driver-info   ipmi_user=admin   ; done20:30
ayoungyep20:30
ayoungadmin should be root20:30
TheJuliaipmi_username20:30
ayoungtrying that now20:32
ayoung| driver_info            | {'ipmi_address': '192.168.123.10', 'ipmi_password': '******', 'ipmi_user': 'root', 'ipmi_username': 'root'}20:33
ayoungThat worked.  I can hear the fans spinning up.  Thanks20:33
NobodyCamGood Morning Ironic folks20:34
ayoungIt does not show the command in the log if it succeeded, just if it failed, but I bet the -L ADMINISDTAOR was due to me not sending the proper ipmi_username parameter.20:34
akiselevHi All! I just joined your channel following dtantsur's guidance. I have a question on Ironic ramdisk logs filename format. Would it be possible to add an actual node name in addition to the default UUID? That nodename may be optional and not affecting the default. The code for ironic-inspector I am talking about: https://opendev.org/openstack/ironic-inspector/src/branch/master/ironic_inspector/conf/processing.py#L87-L9420:35
ayoungOh yeah...its getting noisy in here.20:35
TheJuliaayoung: \o/20:35
TheJuliaayoung: this is why my supermicro boxes are in the garage20:36
TheJuliao/ NobodyCam20:36
TheJuliaakiselev: actual node name as in what dhcp told the machine?20:36
ayoung\m/ d(>_<)b <\m/20:37
NobodyCam:) o/20:37
ayoungMy garage leaks.  I am on my third iteration of digging up the foundation20:38
akiselevYes, I think it is where we get in initially. And later it can be mapped with: ironic node-show --fields name <node UUID>20:38
anuradha1904Hi TheJulia, sorry to disturb you considering you are highly busy. However, if you have some time today, can you check this https://review.opendev.org/#/c/756565/ patch, it has been already approved by iurygregory and Steve Baker, Can we get it merged? Thank you very much.20:38
patchbotpatch 756565 - ironic-inspector - Replaced openstack baremetal to baremetal in the d... - 2 patch sets20:38
TheJuliaakiselev: so is it that you want discovered node names to be populated?20:39
TheJuliaultimately?20:39
akiselevTheJulia: Correct20:39
TheJuliaI say go for it!20:39
TheJuliacreate an entry against ironic-inspector on storyboard.openstack.org20:40
TheJuliaand send me the link after describing what you want to do and I'll ack that its a good idea20:40
akiselevWhat is your estimate for this story - how big is the change?20:41
TheJuliasimple really, you'd have to extract or record it if it is not already record it, and then change ironic-inspector to post a name as well.20:42
TheJuliaon the node create, it shouldn't be that hard20:43
akiselevIs that still true for ironic as well: https://opendev.org/openstack/ironic/src/branch/master/ironic/drivers/utils.py#L266-L28220:45
TheJuliathat would be like 3 lines of code and maybe 30 lines of testing20:45
TheJuliasince it would already be on the node object at that point if posted20:46
akiselevGreat, thank you!20:47
* TheJulia looks around for spoons20:48
ayoungfor UUID in `openstack baremetal node list -f json | jq -r '.[] | .UUID' ` ; do  openstack baremetal node power off $UUID   ; done20:53
ayoungMuch quieter in here now.20:53
* ayoung happy20:53
TheJuliaheh20:54
trandlesI've just been informed my rack PDU didn't survive the data center work /o\21:03
TheJulia:(21:05
TheJuliahttps://giphy.com/gifs/harry-potter-pottermore-M28rUlcjueKUE21:06
trandlesI should have known. This week's been going pretty well.21:07
TheJulia:(21:07
trandlesThe good news is the electricians have been deployed. Hopefully I'm back online in a day or two.21:08
TheJuliawas that via a module?!?21:17
TheJulia :)21:17
openstackgerritMerged openstack/ironic-inspector master: Replaced openstack baremetal to baremetal in the documentation Changed osc_plugin_cli.html to index.html  https://review.opendev.org/75656521:23
JayFWould folks be OK with a patch to IPA to enable manually setting Ironic API version and disabling negotiation? Use case would be locked down API access from node side, and not wanting to enable / as it makes the API more discoverable.21:31
TheJuliayeah, I think that is fine21:32
JayFack; I'll have a patch for it today or early tomorrow21:32
trandlesw00t! The electrician found a breaker tripped on the PDU that my Ops guy missed. I'm back online!21:35
trandlesTheJulia: I know what it was with ipmitool and power state. The testbed is a old greenblade system. The blade _chassis_ required 'chassis power status' but the individual nodes are fine with just 'power status'21:36
openstackgerritJulia Kreger proposed openstack/ironic master: IPMI: Handle vendor set boot device differences  https://review.opendev.org/75719821:44
*** sai438 has joined #openstack-ironic21:52
*** pmannidi has quit IRC21:53
openstackgerritJulia Kreger proposed openstack/ironic-tempest-plugin master: Tear down raid sets  https://review.opendev.org/75714122:21
*** rloo has quit IRC22:28
openstackgerritJulia Kreger proposed openstack/ironic-python-agent master: Don't run os-prober from grub2-mkconfig  https://review.opendev.org/75230822:28
*** DjeufackZane has joined #openstack-ironic22:28
*** akiselev has quit IRC22:29
openstackgerritJulia Kreger proposed openstack/ironic-python-agent master: Support partition image efi contents  https://review.opendev.org/74804922:29
*** rcernin has joined #openstack-ironic22:30
*** rcernin has quit IRC22:49
*** rcernin has joined #openstack-ironic22:51
*** tosky has quit IRC22:54
*** DjeufackZane has quit IRC22:58
*** rh-jelabarre has quit IRC23:15
*** mrquincy has quit IRC23:17
*** benj_ has quit IRC23:21
*** benj_ has joined #openstack-ironic23:26

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