Wednesday, 2017-08-23

dtrainorAnyone have an example of using driver_info in a node definition file?00:01
dtrainori want to use pxe_snmp but i understand that I need to create some properties of that driver under driver_info00:02
rloodtrainor: this might help https://docs.openstack.org/ironic/latest/admin/drivers/snmp.html00:04
rloodtrainor: i gotta run. sorry.00:04
dtrainoryea, that's where i saw the need for driver_info :)00:05
dtrainorthanks00:05
dtrainori get:  ValueError: Expecting , delimiter: line 7 column 18 (char 134)00:05
*** zshi_laptop has joined #openstack-ironic00:12
*** efried has quit IRC00:15
*** efried has joined #openstack-ironic00:25
*** makowals has quit IRC00:30
*** charcol has quit IRC00:34
*** charcol has joined #openstack-ironic00:35
*** makowals has joined #openstack-ironic00:42
*** zhangfei has joined #openstack-ironic00:43
*** zhangfei has quit IRC00:47
*** makowals has quit IRC00:53
*** rbudden has quit IRC00:54
*** zhangfei has joined #openstack-ironic01:00
*** rloo has quit IRC01:01
*** priteau has joined #openstack-ironic01:03
*** efried has quit IRC01:03
*** makowals has joined #openstack-ironic01:07
*** priteau has quit IRC01:07
openstackgerritMerged openstack/ironic master: Documentation for 'snmp' hardware type  https://review.openstack.org/49346801:07
*** zshi_laptop has quit IRC01:11
*** efried has joined #openstack-ironic01:13
*** wong has joined #openstack-ironic01:14
wongmorning01:14
*** yangyapeng has joined #openstack-ironic01:15
*** dyasny has joined #openstack-ironic01:19
*** ijw has joined #openstack-ironic01:23
*** ijw has quit IRC01:23
*** litao__ has joined #openstack-ironic01:29
*** makowals has quit IRC01:30
*** shweta has quit IRC01:34
*** makowals has joined #openstack-ironic01:54
*** makowals has quit IRC02:03
*** yangyapeng has quit IRC02:10
*** yangyapeng has joined #openstack-ironic02:10
*** cdearborn_ has quit IRC02:19
*** makowals has joined #openstack-ironic02:28
openstackgerritDao Cong Tien proposed openstack/ironic master: Documentation for irmc hardware type  https://review.openstack.org/49048302:33
*** weshay is now known as weshay_PTO_AM02:37
*** mtanino has quit IRC02:40
TheJuliadtrainor: it might help if you could provide an direct example of what your sending the api to get such an error02:49
TheJuliawong: good morning, and goodnight :)02:49
wong:)02:50
*** priteau has joined #openstack-ironic03:03
*** priteau has quit IRC03:08
jobcespedesTheJulia: Thanks for your reply, it confirms the options available. I'd rather have automatation, but some colaborators, with a better secutiry backgroud, have those concerns.03:12
*** aparnav has joined #openstack-ironic03:15
TheJuliajobcespedes: understood, and they are natural security concerns to have, which ultimately centralizes on the risk/reward trade-off of automation, coupled with security posture, and who is actually interacting with the baremeral hardware fleet03:17
TheJuliaI'm suddenly surprised we've not pondered better controls around boot security, in terms of executed bytecode, but really that would be a novaish discussion or a discussion of controls and processes if directly managing a fleet without nova.03:18
TheJuliajobcespedes: you could do uefi with the system configured to allow particular signers.03:20
TheJuliaThat might be acceptable, from a wholistic standpoint03:21
*** yangyape_ has joined #openstack-ironic03:23
*** yangyapeng has quit IRC03:23
jobcespedesTheJulia: Sorry, your recomendation goes beyond my expertise/knowledge :P I'm going to share it with those collaborators I mentioned, they are more prepare than me in the infrastructure and security scopes . The problem was, as I understood and as they described it,  access to boot config means access to everything (if credentials were compromised)03:35
TheJuliajobcespedes: I'm not too familiar with the internals of ilo permissions, but it might be worth also discussing with some of the ilo devs. Nisha_Agarwal is a good person to ask when she is around.03:39
jobcespedesTheJulia: I'm going to try. Thank you :)03:41
TheJuliajobcespedes: No problem!  Let us know if we can be of any further assistance. :)03:43
*** udesale has joined #openstack-ironic03:43
TheJuliaAnd with that, I'm going to go get some sleep03:43
TheJuliaHave a wonderful day!03:43
wongbye,good night03:54
*** Nisha_Agarwal has joined #openstack-ironic03:54
jobcespedesGood night!03:56
*** jobcespedes has quit IRC03:57
*** links has joined #openstack-ironic03:57
*** ijw has joined #openstack-ironic04:00
*** ijw has quit IRC04:05
*** vmud213 has joined #openstack-ironic04:11
openstackgerritShivanand Tendulker proposed openstack/ironic master: Update documentation for ``ilo`` hardware type  https://review.openstack.org/49648004:23
openstackgerritzenghui.shi proposed openstack/ironic-specs master: New driver interface for BIOS configuration  https://review.openstack.org/49648104:41
*** makowals has quit IRC04:42
*** MattMan_1 has quit IRC04:46
*** MattMan_1 has joined #openstack-ironic04:46
*** makowals has joined #openstack-ironic04:47
*** makowals has quit IRC04:57
*** jtomasek_ has joined #openstack-ironic05:01
*** aignatov has quit IRC05:02
*** jtomasek has quit IRC05:02
*** namnh has quit IRC05:02
*** zhenguo has quit IRC05:02
*** dims has quit IRC05:02
*** sdake has quit IRC05:02
*** fpxie has quit IRC05:02
*** mordred has quit IRC05:02
*** afazekas has quit IRC05:02
*** sergek_ has quit IRC05:02
*** kiseok7 has quit IRC05:02
*** andyjones2001 has quit IRC05:02
*** eglute has quit IRC05:02
*** jtanner has quit IRC05:02
*** hogepodge has quit IRC05:02
*** ildikov has quit IRC05:02
*** pas-ha has quit IRC05:02
*** kencjohnston_ has quit IRC05:02
*** zul has quit IRC05:02
*** namnh has joined #openstack-ironic05:02
*** kiseok7 has joined #openstack-ironic05:02
*** zhenguo has joined #openstack-ironic05:02
*** hogepodge has joined #openstack-ironic05:02
*** afazekas has joined #openstack-ironic05:02
*** ildikov has joined #openstack-ironic05:02
*** makowals has joined #openstack-ironic05:03
*** sdake has joined #openstack-ironic05:03
*** sdake has quit IRC05:03
*** sdake has joined #openstack-ironic05:03
*** sergek_ has joined #openstack-ironic05:03
*** pas-ha has joined #openstack-ironic05:03
*** jcook has quit IRC05:04
*** mordred has joined #openstack-ironic05:04
*** eglute has joined #openstack-ironic05:04
*** priteau has joined #openstack-ironic05:04
*** kencjohnston has joined #openstack-ironic05:04
*** dims has joined #openstack-ironic05:04
*** aignatov has joined #openstack-ironic05:05
*** radez has quit IRC05:06
*** trown|outtypewww has quit IRC05:06
*** rook has quit IRC05:06
*** trown has joined #openstack-ironic05:06
*** jcook has joined #openstack-ironic05:07
*** priteau has quit IRC05:09
*** sasha2 has quit IRC05:09
*** kzaitsev_pi has joined #openstack-ironic05:11
*** radez has joined #openstack-ironic05:11
*** rook has joined #openstack-ironic05:12
*** rook is now known as Guest9146405:12
*** priteau has joined #openstack-ironic05:21
*** zshi_laptop has joined #openstack-ironic05:27
*** jtanner has joined #openstack-ironic05:31
*** mjura has joined #openstack-ironic05:32
*** masber has quit IRC05:34
*** amotoki has joined #openstack-ironic05:34
*** baoli has joined #openstack-ironic05:36
*** oomichi has quit IRC05:37
*** oomichi has joined #openstack-ironic05:39
*** baoli_ has joined #openstack-ironic05:49
*** ChubYann has quit IRC05:51
*** baoli has quit IRC05:52
*** hoonetorg has quit IRC05:54
*** baoli has joined #openstack-ironic05:55
*** rbartal has joined #openstack-ironic05:56
*** charcol has quit IRC05:57
*** baoli_ has quit IRC05:58
*** priteau has quit IRC05:59
*** baoli has quit IRC06:02
*** baoli has joined #openstack-ironic06:03
*** hoonetorg has joined #openstack-ironic06:08
*** vishwana_ has joined #openstack-ironic06:15
*** wong_ has joined #openstack-ironic06:15
*** wong has quit IRC06:16
*** rcernin has joined #openstack-ironic06:16
*** vishwanathj has quit IRC06:18
*** jlvillal has quit IRC06:21
*** jlvillal has joined #openstack-ironic06:21
*** kevinbenton has quit IRC06:22
*** kevinbenton has joined #openstack-ironic06:24
*** baoli has quit IRC06:28
*** masber has joined #openstack-ironic06:32
*** aparnav has quit IRC06:39
*** aparnav has joined #openstack-ironic06:39
Tenguhello ironic :)06:43
Tengusmall question: how may I ensure ironic activated the GPT thing for volumes >2To ? I'm not 100% it's the case, apparently 200G are missing from my 2.2To volume…06:43
*** JayF has quit IRC06:44
Tengui.e. is there any flag in the "capabilities" as shown with "ironic node show <uid>"?06:44
*** baoli has joined #openstack-ironic06:45
*** yangyape_ has quit IRC06:47
*** yangyapeng has joined #openstack-ironic06:48
*** baoli has quit IRC06:49
*** baoli has joined #openstack-ironic06:51
*** baoli has quit IRC06:55
*** fpxie has joined #openstack-ironic06:59
*** andyjones2001 has joined #openstack-ironic07:00
*** zul has joined #openstack-ironic07:00
*** charcol has joined #openstack-ironic07:07
*** dtrainor has quit IRC07:09
*** pcaruana has joined #openstack-ironic07:11
*** tesseract-RH has joined #openstack-ironic07:17
*** charcol has quit IRC07:27
*** baoli has joined #openstack-ironic07:28
*** hjensas has joined #openstack-ironic07:31
*** hjensas has quit IRC07:31
*** hjensas has joined #openstack-ironic07:31
*** baoli has quit IRC07:32
*** dtrainor has joined #openstack-ironic07:32
*** racedo has joined #openstack-ironic07:38
*** hjensas has quit IRC07:39
wong_hi, anybody saw this before?http://paste.openstack.org/show/619133/07:43
wong_I type clean-step to config raid,after that,the node status was in manageable,then I made status to provide,the error appeared07:45
*** hjensas has joined #openstack-ironic07:46
*** hjensas has quit IRC07:46
*** hjensas has joined #openstack-ironic07:46
*** athomas has joined #openstack-ironic07:49
*** vmud213 has quit IRC07:49
openstackgerritVasyl Saienko proposed openstack/ironic master: Set device_owner/vnic_type when update mac_address  https://review.openstack.org/49624907:53
*** etingof has quit IRC07:55
*** livelace has joined #openstack-ironic07:59
*** openstackgerrit has quit IRC08:03
*** Nisha_Agarwal has quit IRC08:06
*** derekh has joined #openstack-ironic08:07
*** amarao has joined #openstack-ironic08:09
*** hjensas has quit IRC08:11
*** etingof has joined #openstack-ironic08:14
*** hjensas has joined #openstack-ironic08:14
*** hjensas has quit IRC08:14
*** hjensas has joined #openstack-ironic08:14
*** openstackgerrit has joined #openstack-ironic08:15
openstackgerritMerged openstack/bifrost master: Updated from global requirements  https://review.openstack.org/49517508:15
*** karimb has joined #openstack-ironic08:16
*** lucas-afk is now known as lucasagomes08:18
*** amarao has quit IRC08:19
*** kristaps_ has quit IRC08:25
*** amarao has joined #openstack-ironic08:34
pas-hamorning ironic :)08:35
*** mgoddard has joined #openstack-ironic08:39
*** aparnav has quit IRC08:44
vdrokmorning all!08:45
*** aparnav has joined #openstack-ironic08:45
*** hjensas has quit IRC08:47
* Tengu checks his watch08:50
Tenguhmm. timezone magic ;).08:50
*** litao__ has quit IRC08:51
*** fdegir has quit IRC08:51
TenguAnyone can tell me how to ensure ironic activated the GPT thing? I'm not really sure it's the case on my computes, because there's missing 200G on the 2.2To.08:51
*** litao__ has joined #openstack-ironic08:51
*** rwsu has joined #openstack-ironic08:51
*** sergek_ has quit IRC08:52
*** zhenguo has quit IRC08:52
*** fdegir has joined #openstack-ironic08:52
*** sergek_ has joined #openstack-ironic08:52
*** zhenguo has joined #openstack-ironic08:53
*** hjensas has joined #openstack-ironic09:00
*** hjensas has joined #openstack-ironic09:00
makowalsTengu: How about this one ? https://docs.openstack.org/project-install-guide/baremetal/draft/advanced.html#when-used-with-compute-service09:01
makowalsTengu: How about just checking "fdisk -l device" from inside the box to check what happened ?09:01
*** hjensas has quit IRC09:01
*** karimb has quit IRC09:02
*** tuanluong has joined #openstack-ironic09:04
Tengumakowals: we see the 200G at the end of the partition09:05
Tenguwhat's weird: I think it should have added the gpt flag while introspecting, shouldn't it?09:06
makowalsTengu: How about posting output of fdisk ?09:06
Tengu2s09:06
Tengumakowals: http://paste.openstack.org/show/619146/09:07
*** e0ne has joined #openstack-ironic09:07
*** etingof has quit IRC09:08
Tenguaha, I have to add the capability to the flavor as well. That's most probably the error I have.09:08
makowalsTengu: Okay, so it's obvious it's missing this gpt flag and created dos partition label; good question, should inspector even detect it ?09:08
Tengumakowals: I think so, yes. but according the doc you pointed, the flavor must have that flag as well.09:09
Tenguso detecting while introspecting would partially solve the issue09:09
TenguI know now what to do, thanks :)09:09
*** karimb has joined #openstack-ironic09:10
makowalsNo problem, cheers !09:10
*** sdake has quit IRC09:15
*** fdegir has quit IRC09:16
*** sdake has joined #openstack-ironic09:16
*** sdake is now known as Guest2894309:16
*** fdegir has joined #openstack-ironic09:17
*** karimb has quit IRC09:20
*** sambetts|afk is now known as sambetts09:20
sambettsMorning all09:21
*** etingof has joined #openstack-ironic09:21
lucasagomessambetts, morning09:23
*** karimb has joined #openstack-ironic09:24
openstackgerritliumengke proposed openstack/ironic master: fix a typo in agent.py: s/doman/domain/  https://review.openstack.org/49657409:25
sambettso/ lucasagomes09:25
*** Nisha_Agarwal has joined #openstack-ironic09:27
*** amarao1 has joined #openstack-ironic09:30
*** vmud213 has joined #openstack-ironic09:32
*** zshi has joined #openstack-ironic09:33
*** Nisha_Agarwal has quit IRC09:34
*** mtanino has joined #openstack-ironic09:41
*** tuanluong has quit IRC09:48
*** vmud213 has quit IRC10:13
*** yangyapeng has quit IRC10:18
*** hjensas has joined #openstack-ironic10:19
*** hjensas has quit IRC10:19
*** hjensas has joined #openstack-ironic10:19
*** amarao1 has quit IRC10:23
*** milan has joined #openstack-ironic10:25
milanmorning Ironic! :)10:26
milan#pixiesay -mdead alive?10:27
milanhmmmmm10:27
*** dtantsur|afk is now known as dtantsur10:30
dtantsurlate morning folks!10:31
sambettso/ dtantsur10:31
dtantsuroh, so my Pixie container is not so HA, I see :D10:31
*** udesale has quit IRC10:31
openstackgerritPavlo Shchelokovskyy proposed openstack/ironic master: Finalize migration to keystoneauth adapters  https://review.openstack.org/47882510:31
openstackgerritPavlo Shchelokovskyy proposed openstack/ironic master: Use adapters for cinderclient  https://review.openstack.org/47617110:31
openstackgerritPavlo Shchelokovskyy proposed openstack/ironic master: Use adapters for neutronclient  https://review.openstack.org/47617010:31
openstackgerritPavlo Shchelokovskyy proposed openstack/ironic master: Rework keystone auth for glance  https://review.openstack.org/46772810:31
openstackgerritPavlo Shchelokovskyy proposed openstack/ironic master: Fix swiftclient creation  https://review.openstack.org/47617310:31
openstackgerritPavlo Shchelokovskyy proposed openstack/ironic master: Use adapters for inspectorclient  https://review.openstack.org/47617210:31
openstackgerritPavlo Shchelokovskyy proposed openstack/ironic master: Introduce keystoneauth adapters for clients  https://review.openstack.org/46962710:31
*** mtanino has quit IRC10:31
milanmorning dtantsur pas-ha sambetts! :)10:32
pas-hamorning milan10:32
*** PixieBoots has joined #openstack-ironic10:34
dtantsur#pixiesay -mmeh Containers are overrated10:35
PixieBoots¯\_(ツ)_/¯: Containers are overrated10:35
*** yuriyz has joined #openstack-ironic10:37
*** karimb has quit IRC10:38
*** Guest91464 is now known as rook10:39
*** jkilpatr has quit IRC10:42
*** hoonetorg has quit IRC10:43
*** hoonetorg has joined #openstack-ironic10:43
openstackgerritMerged openstack/ironic master: Set explicit default to enabled driver interfaces  https://review.openstack.org/49287710:50
*** karimb has joined #openstack-ironic10:52
vdrokmorning milan dtantsur and sambetts lucasagomes !10:52
openstackgerritOpenStack Proposal Bot proposed openstack/ironic-ui stable/pike: Imported Translations from Zanata  https://review.openstack.org/49620010:54
*** vmud213 has joined #openstack-ironic10:56
*** amarao1 has joined #openstack-ironic11:00
lucasagomeso/11:02
nicodemosmorning11:11
*** fellypefca has joined #openstack-ironic11:14
dtantsurlucasagomes: Pixie died in the container, just imagine :) I had to resurrect it with my dark PTL powers :)11:16
lucasagomesdtantsur, oh noes11:18
lucasagomesdtantsur, network failure or ?11:18
dtantsurlucasagomes: I could not figure out, and could not afford spending too much time on it11:18
lucasagomesfair enough, yeah the code is very PoC-ish11:19
lucasagomesnot sure it would handle reconnecting for example11:19
*** jkilpatr has joined #openstack-ironic11:20
dtantsurI suspect it did not reconnect indeed11:22
dtantsurour lab does go offline from time to time11:22
dtantsuranyway, docker restart works quite well :)11:22
*** aparnav has quit IRC11:22
lucasagomescool :-)11:22
*** baoli has joined #openstack-ironic11:29
*** bfournie has quit IRC11:30
*** yangyapeng has joined #openstack-ironic11:31
sambettsdtantsur: I have a concern that zeroing out the flavor properties might break https://github.com/openstack/ironic/blob/master/ironic/drivers/modules/iscsi_deploy.py#L8811:34
*** baoli has quit IRC11:34
*** yangyapeng has quit IRC11:35
dtantsursambetts: we don't zero flavor properties, we use special resource markers for that11:37
dtantsurzeroing flavor properties won't work with nova either11:37
*** mat128 has joined #openstack-ironic11:37
sambettsdtantsur: so you have to specify the CPU/DISK/RAM for a flavor anyway? ... I don't think we specify anywhere what those should be in a resource class world, we specfically talk about setting them to Zero using the custom resource classes11:39
sambettsdtantsur: which in "new flavor" terms is setting them to zero11:39
openstackgerritMerged openstack/ironic master: Correct URLs in install docs  https://review.openstack.org/49641111:39
sambettsdtantsur: we should say something like, you need to set the flavor disk_gb to be the lowest amount offered by the hardware pool assigned the resource class you've given to the falvor11:40
* sambetts hates resource classes so much... and wished he'd been involved earlier to prevent this... and go down the sensible path of just setting min and max amounts on the resource classes in the resource provider 11:46
sambettsliterally we could have solved all of ironic's scheduling pains by just having the ironic virt driver report vcpus = 10 min = 10 max = 10 and the same for disk and ram to the placement API for each node and taken advantage of existing features of placement instead of inventing the worlds most hacky solution11:52
*** led_ has quit IRC11:54
*** lets has quit IRC11:54
*** litao__ has quit IRC11:57
*** yangyapeng has joined #openstack-ironic11:57
*** lucasagomes is now known as lucas-hungry12:01
*** bfournie has joined #openstack-ironic12:02
jrolltripleo doesn't do exact matching, fwiw12:04
*** zhangfei has quit IRC12:04
*** dprince has joined #openstack-ironic12:04
jrollthis is the best we could do that people wouldn't rage about and block12:04
jrollsorry ¯\_(ツ)_/¯12:04
* Tengu enforce profile directly when importing nodes in openstack…12:05
Tengueasier, faster, probably cleaner. I should also automatically add the right kernel stuff so that introspection would be useless.12:05
sambettsjroll: solution for that is that you put bounds on what ironic thinks a node has in-terms of resources, e.g. only tell ironic a node has 500gb of disk12:10
sambettswhen it really has 1tb12:11
jrollfrom what I remember, that wouldn't work for the people not doing exact match12:11
jrollbut I'm not really here to debate solutions12:11
jrolljust to tell you we couldn't find any good alternatives in two years of debate12:12
*** kencjohnston has quit IRC12:12
*** ildikov has quit IRC12:12
sambetts jroll: yeah :/ its too late now... I managed to convince jaypipes at the summit that it was probably a neter solution just to use the mechanisums that exist today in the placement API but it was too late at that point12:13
jrollyeah12:13
jrollthe hope was to work back towards that12:13
* sambetts hadn't been involved enough to realise what was going on, or what the capabilites of the placement API was until jay gave an in-depth run down at the summit 12:13
jrollI also think those capabilities grew since we started down this path12:14
sambetts+10000 to working back towards using real world values that make sense to the people selecting the flavor12:14
jrolli.e. what we had when we started this didn't have enough to do the right thing12:14
sambettsyeah12:14
*** kencjohnston has joined #openstack-ironic12:14
sambettsgot to love that bleeding edge12:14
*** ildikov has joined #openstack-ironic12:14
jrollbleeding edge? in openstack? :P12:15
sambetts;)12:15
*** karimb has quit IRC12:16
*** zhangfei has joined #openstack-ironic12:16
*** mat128 has quit IRC12:19
*** jcoufal has joined #openstack-ironic12:19
*** francoblanco has quit IRC12:22
*** francoblanco has joined #openstack-ironic12:22
*** jaypipes has joined #openstack-ironic12:23
*** pas-ha has quit IRC12:23
*** dutsmoc has quit IRC12:23
*** pas-ha has joined #openstack-ironic12:24
*** comstud has joined #openstack-ironic12:28
dtantsurexact matching does not really work for installer case like tripleo..12:36
dtantsuranyway, to your question sambetts: the CPU/DISK/RAM set in flavor is not going anywhere, until flavors themselves are going (which I think is the plan)12:36
sambettsdtantsur: we need to make it obvious in the docs that even though we're setting in all caps those fields to zero, that they actually do still play a part12:37
sambettsand should be set accordingly12:37
dtantsuralso the DISK is a really severe problem, because bare metals have many disks12:37
sambetts+1 to the many disks thing12:37
dtantsursambetts: ++ to make it clear in the docs (feel free to comment on my patch)12:38
sambettsdtantsur: already done :-P12:38
milanmorning vdrok! :)12:38
dtantsurnice! /me was having coffee12:38
sambettsdtantsur: it was while I was reviewing that patch that caused me to think oh dear whats are we supposed to do now12:39
dtantsurno options are great, when you try to fit all varieties of hardware into just a handful of properties to schedule on12:39
dtantsurthis is why we invented introspection rules initially (at least that's why I needed them): to be able to set capabilities automatically and schedule on them12:39
sambettsoh totally, thats why I love the placement API and resource class thing, I just don't think that having a single resource class for baremetal makes sense,12:40
sambetts12:40
dtantsurI can see why it's a problem.. an alternative would be to make scheduler aware of things like ironic that are always all-or-nothing12:41
sambettsdtantsur: it has it12:42
dtantsurreally? I think nova folks told me otherwise, but I may have misunderstood12:42
sambettsdtantsur: in the placement API it has the concept of minimum_divisible_amounts and maximium_divisible_amount for every resource class12:42
sambettsdtantsur: so for a VM hypervisor you can have 128gb of ram and say mimomum_divisible_amount: 1gb12:43
dtantsuroh, sweet. then I wonder why we don't use them indeed12:43
sambettsdtantsur: which means that flavors asking for less than 1gb can't be put on the hypervisor12:43
*** hw_wutianwei has quit IRC12:43
sambettsand so for baremetal we just set minimum to the same as total12:43
*** jlabarre has joined #openstack-ironic12:44
dtantsurwell, we don't need requests to be rejected, we need them rounded up12:44
dtantsurso e.g. I can ask for 512 MiB, but I'll still get all 128GiB12:44
*** karimb has joined #openstack-ironic12:44
dtantsurrefusing to schedule in this case is essentially forcing exact filters, which are just as an ugly hack12:45
sambettsthat could be an operator choice though12:45
sambettswe could have the minimum/max fed into placement for each node12:45
sambettsso for some nodes you might only want them matched by exact falvors12:45
dtantsurI'm not sure how it would look.. would it be something new on flavors to say that? on nodes?12:46
sambettsdtantsur: flavors would stay the same as whereever they are going12:46
sambettsthis is all to do with populating the information in the placement API to make it aware of what a node is capable of12:47
dtantsurso how would you populate it? if you report min=max=properties.memory_mb, how will it work with smaller flavors?12:47
*** udesale has joined #openstack-ironic12:48
*** rbudden has joined #openstack-ironic12:48
dtantsuralso keep in mind that reporting any local_gb is meaningless is many real cases12:48
sambettsdtantsur: initally I would have a flag, which is "allow_smaller_flavors_on_bigger_nodes=True/False" which enables or disable setting minimum to total12:49
dtantsurand if minimum is not set to total, will it be divisible then?12:49
dtantsurso e.g. if I take 4 GiB out of 16, will placement think that 12 are still left?12:50
sambettsah yeah... that wouldn't workl12:50
sambettsbut tbh my thoughts on reporting to placement is that if you want nodes to be treated the same, you just tell Ironic that they have the same amount of resources12:50
sambettsand then use exact matching all the time12:51
dtantsurthis is not going to fly, really..12:51
dtantsure.g. it throws away inspection12:51
sambettsnope it doesn'#t12:52
sambettsI already have a bunch of introspection rules that equal out all my nodes12:52
sambettsbecause they are all very slightly different12:52
dtantsurwell, this is a gross hack :)12:52
dtantsurand keep in mind OOB inspection too12:52
sambettsso I have rules that say if disk_gb > 500gb then disk_gb == 500gb12:52
dtantsurwe already give it some hard time with resource classes12:52
dtantsurwith your rule, the nodes is no longer deployable by flavor requiring 600gb12:53
sambetts I don't create a flavor with 600gb12:53
dtantsurwell, you don't, but people may :)12:53
sambettsthe introspection rules that I create are designed to match my flavors12:53
sambettsso if I wanted a flavor that had 600gb then I would create another rule which is if disk_gb > 600gb then disk_gb == 600gb12:54
sambettsso then nodes with between 600 and 500 are equaled out to 500gb, and I can have a flavor that asks for 50012:54
sambettsand then nodes greater than 600 and equaled out to 600 and match the flavor that asks for 60012:55
dtantsurnow, why is it better than having 2 resource classes? :) you're essentially reinventing them12:55
sambetts-1 resource classes have zero ties to physical capabilites, I'm creating bounds to group my nodes based on real values that they are capable of12:56
dtantsuryep, you're creating resource classes implicitly based on disk size (of which disk, by the way?)12:56
sambettsyou might have some nodes that have 600gb of disk but also no ram12:56
sambettsalmost*12:57
sambettsit all evens itself out and there are no hacks needed from a operators's perspective with custom flavors and setting arbitrty values on nodes12:58
sambettsI like having flavors that can map to a VM or Baremetal too12:58
dtantsuryeah, this is a big downside that we cannot do it12:58
dtantsurI'm not sure how many people require such flavors, to be honest, but it's sad it's no longer possible12:59
*** cdearborn has joined #openstack-ironic12:59
*** sasha2 has joined #openstack-ironic13:01
*** rloo has joined #openstack-ironic13:02
*** lucas-hungry is now known as lucasagomes13:03
TheJuliaGood morning everyone13:03
nicodemosmorning TheJulia13:04
dtantsurTheJulia: morning! https://review.openstack.org/#/c/495316/ is the biggest blocker for the release, mind checking it?13:05
patchbotpatch 495316 - ironic (stable/pike) - [reno] Add prelude for Pike (9.1.0) release13:05
TheJuliaWow, lots of discussion today13:05
sambettso/ TheJulia13:05
sambettsmostly me ranking13:05
sambettsranting*13:05
dtantsurresource classes flame war, not the first one and certainly not the last...13:05
sambetts;)13:05
* dtantsur is not looking forward to more people figuring it out13:05
sambettsjust got it in me to pick holes in everything today... /me feels slightly sorry for the patches I'm reviewing13:06
*** rpioso has joined #openstack-ironic13:06
rloohi and good morning sambetts, dtantsur, TheJulia. It is worth reading about sambetts's ranting?13:07
rpiosoGood morning!13:08
rloosambetts: fwiw, it is good to pick holes, if that helps things get better!'13:08
rloomorning rpioso13:08
rpiosorloo: :)13:08
sambettsrloo: perhaps if you aren't aware of some of the capabilites of the placement API13:08
dtantsurmorning rloo13:08
TheJuliadtantsur: +2+A?13:08
TheJuliaerr13:09
TheJuliano ?, I just +A'ed it13:09
dtantsurgood ;)13:09
* TheJulia needs coffee13:09
dtantsurrloo: would you like to wait for all the docs patches to land in stable/pike? I'd like to cut the release once the reno merges13:09
* TheJulia resumes brain on BFV thoughts while making some coffee13:09
dtantsurthe docs are always backportable, we may just get a point release in the beginning of september13:09
milanmorning nicodemos rloo rpioso TheJulia! :)13:09
*** karimb has quit IRC13:10
milansambetts, https://review.openstack.org/#/c/490008/5/api-ref/source/baremetal-api-v1-drivers.inc I wanted to check out how it renders13:10
patchbotpatch 490008 - ironic - Switch API ref to use versionadded syntax throughout13:10
*** fellypefca has quit IRC13:10
*** fellypefca has joined #openstack-ironic13:10
milansambetts,  but the links led to the current/latest docs13:10
* milan might not know what he's doing though13:10
sambettsmilan: you need to click on the api-ref job13:11
sambettsmilan: they are rendered separately from the main docs13:11
milanaaaah13:11
milansambetts, thx13:11
*** karimb has joined #openstack-ironic13:11
* milan wonders why the green text13:12
rpiosomilan: Morning :)13:13
rloodtantsur: i'd like the patches that fix urls, to land if possible.13:13
rloodtantsur: what is the status of those? let me see where we're at, just catching up ...13:13
dtantsurmm, okay. this probably pushes our release to tomorrow13:13
fellypefcaGood morning, everyone.13:13
rloodtantsur: oh? what's the 'deadline' for today?13:14
rloodtantsur: i mean, time. or do you think cuz of how long it'll take to approve etc?13:14
dtantsurrloo: I just cannot imagine merging all this  today :)13:14
dtantsuryeah, and there are only a couple of work hours left for me13:14
rloodtantsur: doc patches should be fast...13:14
rloodtantsur: so i'd like the configuration doc patch too but that might take more time, i see you had comments but i haven't looked into it13:15
*** vmud213 has quit IRC13:15
rloodtantsur: could you or TheJulia +A this: https://review.openstack.org/#/c/496354/13:16
patchbotpatch 496354 - ironic (stable/pike) - Fix incorrect documentation urls13:16
dtantsurrloo: out of curiosity, why do you want them in pike right now? because of people packaging docs?13:16
rloodtantsur: cuz i have spent over a day dedicated to doc changes, which is what i said i'd do before. i have other things i'd like to do so don't plan on continuing after this.13:16
rloodtantsur: and cuz, if we are going to do it, we might as well do it now13:17
TheJuliadtantsur: do people even package the docs?13:17
rloodtantsur: cuz i am familiar with the patches. and i'm not going to be later.13:17
dtantsurTheJulia: RDO does13:17
rloodtantsur: that's all.13:17
*** osdev has joined #openstack-ironic13:17
TheJuliadtantsur: might be it as far as I'm aware.13:17
dtantsurI don't suggest to stop working on docs, I only suggest to cut the release at some point today13:18
dtantsurbecause it's a non-standard release, and I don't want to propose it hours before the final and utter deadline13:18
rloodtantsur: right, and I'm only answering your questions :)13:18
openstackgerritRuby Loo proposed openstack/ironic stable/pike: Correct URLs in install docs  https://review.openstack.org/49670513:19
dtantsuryeah, but my question was mostly about wanting them in pike *right now, before 9.1.0*, not about working on them at all or whether to backport them at all13:19
rloodtantsur: ok, so let me answer it this way, there isn't anything urgent/blocking landing 9.1.0 as long as the prelude is in.13:20
rloodtantsur: the rest is to make it better.13:20
rloodtantsur: and i don't plan on spending much more time on the doc patches. so personally, i just want to get them out of the way.13:21
dtantsurrloo: got it! let me request the release today, and let's finish the doc patches today-tomorrow13:21
rloodtantsur: and if we say 'we can still review and update docs', are we really going to do it soon as opposed to ... whenever...13:21
*** mjturek has joined #openstack-ironic13:22
dtantsurat least let's finish what we already have proposed (link updates mostly)13:22
dtantsurI don't suggest to keep your patches up for months ;)13:22
rloodtantsur: no one suggests that but we do have patches that have been around for months :-(.13:22
rloodtantsur: i actually started to look at patches, saw some more than 2 years old. sigh.13:23
*** makowals_ has joined #openstack-ironic13:23
dtantsuryeah, this happens13:23
rloodtantsur: anyway, that's another story :)13:23
* dtantsur remember his nova patch merging after more than a year of inactivity13:23
*** yangyapeng has quit IRC13:24
rlooso these two doc (on master) patches have a +2 already: https://review.openstack.org/#/c/496320/ & https://review.openstack.org/#/c/496361/13:24
patchbotpatch 496320 - ironic - correct URLs in admin docs13:24
patchbotpatch 496361 - ironic - correct URLs in contributor docs & main index13:24
rloothey should be easy to review/land and backport to pike13:24
*** makowals has quit IRC13:25
rloothen almost all of the links will be fixed. (the remainder are in the other doc patches that will land later I hope)13:25
dtantsurokay, let's finish there two as well13:25
rloothe doc migration broke more links than people may realize. I am concerned about that for docs out there (people's notes, whatever).13:26
*** bnemec has joined #openstack-ironic13:26
openstackgerrityolanda.robla proposed openstack/bifrost master: Change perms for the VM logs, fix log publish  https://review.openstack.org/49454913:27
dtantsurrloo: yeah, I"m still fixing tripleo-docs13:29
*** igordc has joined #openstack-ironic13:29
rloodtantsur: :-(13:29
dtantsurone approved, 2nd to go13:29
TheJuliarloo: you could have left the // ;)13:32
rlooTheJulia: Nope, that is wrong!13:33
rlooTheJulia: i can't help it, I like things to be juuuuust right :)13:33
dtantsurfolks, I completely forgot about a meetup on the PTG.. does anyone know the Denver area enough to find some place for a dinner/beers?13:35
TheJuliasadly no13:35
TheJuliaI will, however, be there a week before the PTG13:35
TheJuliaso I can also go do some exploring13:35
TheJuliabut seriously, if some knows of anything, it my gas tank would appreciate not driving and searching around downtown13:37
dtantsurTheJulia: yes please! it's somewhat harder to figure out things from over the ocean :)13:37
dtantsurrloo: you can already propose backports for your patches, just mind the cherry-pick line13:38
*** links has quit IRC13:38
TheJuliaIs the linux-iscsi.org web page loading for people ?13:39
rloodtantsur: right. and cherry-pick line is working now (with gerrit 'cherry-pick' button). i noticed it working yesterday.13:39
dtantsurTheJulia: "waiting.." for me13:39
sambettsTheJulia, dtantsur: if we managed to organise anything can we try for it to be a bit closer to the PTG than the meet up in Atlanta?13:39
openstackgerritIlya Etingof proposed openstack/ironic master: Configuration documentation migrated  https://review.openstack.org/49186113:39
rloodtantsur: wrt meetup, we should come up with an evening too13:39
dtantsursambetts: you mean, to the venue?13:40
dtantsurrloo: you mean, the time slot? I was about to create a doodle13:40
rloodtantsur: yeah, that, and thx!13:40
sambettsdtantsur: yeah, in Atlanta the meet up was like 20min drive away or more if I remember correctly13:40
dtantsurI'll throw it on the ML, maybe somebody will tell us a good place13:40
rloosambetts: you mean you didn't like walking or taking a cab to the place? :)13:40
rlooTheJulia: not working for me. that link i mean.13:40
rlooTheJulia: still spinning...13:40
*** sauloaislan has joined #openstack-ironic13:41
sambettsI don't mind walking, but in Atlanta it was a little too far to walk when I looked it up on the map13:41
TheJuliasambetts: I'll try for walking distance. It will be good for me to sample many burgers and beers13:41
sauloaislanMorning!13:41
sambettsTheJulia: thanks13:41
sambetts:)13:41
*** jaypipes has quit IRC13:42
dtantsurthe poll is https://doodle.com/poll/nvavg9ab9ebq2e4v13:42
sauloaislandtantsur: Hi. Can you take a look at this backport?  https://review.openstack.org/#/c/489598/13:43
patchbotpatch 489598 - ironic (stable/ocata) - Removes agent mixin from oneview drivers13:43
dtantsurwill do, not sure about today (release time!)13:43
openstackgerritRuby Loo proposed openstack/ironic stable/pike: Documentation for 'snmp' hardware type  https://review.openstack.org/49671013:44
dtantsuralso FYI team photos https://docs.google.com/spreadsheets/d/18DmI9ydq6dKr4hAcoAN83vYXWn8TV1NEvy09Ll02_Y0/edit#gid=013:47
rloodtantsur: is the schedule out for PTG, we should do the team photo during one of our breaks, like last time13:48
rloodtantsur: oh, it is just tues and wed. so wed for sure13:49
sauloaislandtantsur: Ok thank you !13:49
dtantsurrloo: the last time we just had a break when we needed13:50
dtantsurit's like 5 mins13:50
rloodtantsur: i was trying to remember how it worked. i think we start at around 9am?13:51
openstackgerritMerged openstack/ironic master: fix a typo in agent.py: s/doman/domain/  https://review.openstack.org/49657413:51
dtantsuryep. we had a slot booked, so we just paused, went out for photo, and then went back to discussing stuff13:51
openstackgerritMerged openstack/ironic stable/pike: [reno] Add prelude for Pike (9.1.0) release  https://review.openstack.org/49531613:51
openstackgerritMerged openstack/ironic stable/pike: Fix incorrect documentation urls  https://review.openstack.org/49635413:51
openstackgerritMerged openstack/ironic master: correct URLs in admin docs  https://review.openstack.org/49632013:51
openstackgerritMerged openstack/ironic master: correct URLs in contributor docs & main index  https://review.openstack.org/49636113:52
rloodtantsur: ok, i think any time is good then. even 10 would be a good time for a break. i just wouldn't pick 10 though cuz if they are late setting up or there are issues, it might take longer if we're the first team.13:52
*** mat128 has joined #openstack-ironic13:52
dtantsurI'd pick 11 or something like that13:52
openstackgerritRuby Loo proposed openstack/ironic stable/pike: correct URLs in admin docs  https://review.openstack.org/49671213:53
openstackgerritRuby Loo proposed openstack/ironic stable/pike: correct URLs in contributor docs & main index  https://review.openstack.org/49671313:53
*** mat128 has quit IRC13:53
*** zhangfei has quit IRC13:54
rloodtantsur: that would work, thx.13:55
*** ijw has joined #openstack-ironic13:55
dtantsuranybody else has an opinion on photo time? I'd go ahead and ask for 11 otherwise13:56
*** Goneri has joined #openstack-ironic13:57
openstackgerritMerged openstack/ironic master: Documentation for irmc hardware type  https://review.openstack.org/49048313:57
*** mat128 has joined #openstack-ironic13:57
openstackgerritRuby Loo proposed openstack/ironic stable/pike: Documentation for irmc hardware type  https://review.openstack.org/49671613:59
milansambetts, sry, the green text is killing my eyes :P14:01
rloodtantsur: all the doc patches that i wanted for pike, have been approved in master branch. except yours: https://review.openstack.org/#/c/496309/14:01
patchbotpatch 496309 - ironic - Minor improvements to the resource classes documen...14:01
*** mjura has quit IRC14:01
*** mtanino has joined #openstack-ironic14:02
rloodtantsur: i didn't look to see the comments, is that something you want to try to get in?14:02
dtantsurrloo: it's not THAT urgent IMO. mostly clarifications.14:03
openstackgerritMerged openstack/ironic stable/pike: Correct URLs in install docs  https://review.openstack.org/49670514:03
rloodtantsur: ok then. i'll ping you & TheJulia when the stable/pike patches are ready to approve.14:04
dtantsurrloo: do we have backports proposed from everythin we want in Pike right now?14:04
dtantsuraha, good :)14:04
rloodtantsur: yup14:04
rloodtantsur, TheJulia: after those pike patches land, TheJulia will need to update https://review.openstack.org/#/c/495315/14:04
patchbotpatch 495315 - ironic (stable/pike) - Change doc links for pike branch14:04
rloodtantsur, TheJulia: this is ready to land in pike: https://review.openstack.org/#/c/496712/14:05
patchbotpatch 496712 - ironic (stable/pike) - correct URLs in admin docs14:05
openstackgerritSam Betts proposed openstack/ironic master: Add documentation describing each Ironic state  https://review.openstack.org/49671914:05
sambettsrloo: ^14:05
rloosambetts: what? trying to get something in under the wire?14:05
*** hamzy has quit IRC14:05
sambettsrloo: had a "get it done" moment when I saw you discussion with dtantsur about the link to the spec14:06
dtantsurMOAR DOCS MOAAARRRR14:06
dtantsurthis is a good thing, but not blocking the release14:06
sambettsdtantsur: nope, I don't expect it to be backported14:06
*** zhangfei has joined #openstack-ironic14:07
rloosambetts: thx! but yeah. I'll look at it today though :)14:07
sambettsjust wanted to get it up there and done so that we don't forget about it again14:07
masbergood afternoon, which interface is Ironic using for PXE boot?14:07
sambettsmilan: thats part of the openstack doc's theme unfortunately14:07
rloodtantsur, TheJulia: this is also ready to approve in pike: https://review.openstack.org/#/c/496713/14:07
patchbotpatch 496713 - ironic (stable/pike) - correct URLs in contributor docs & main index14:07
*** udesale has quit IRC14:08
dtantsurmasber: you mean, which NIC? whatever is configured in BIOS, we don't change it14:08
sambettsmasber: it'll attempt to use all NICs you've told Ironic about that don't have pxe_enabled=False14:08
milansambetts, maybe the "note" shouldn't bee that long at least, or use the optional second arg <a brief description>14:08
milansambetts, or wdyt about making it a paragraph?14:09
dtantsurrloo: done. now only TheJulia's patch left?14:09
* TheJulia has too many windows open :(14:09
rloodtantsur: yup. although i wanted to ask you, did you want to also get this one in: https://review.openstack.org/#/c/491861/14:09
patchbotpatch 491861 - ironic - Configuration documentation migrated14:09
TheJuliaand a kitten driving me insane14:10
sambettsmilan: yeah thats probably a good idea...14:10
rlooTheJulia: don't worry about it, things are in control. I'll ping you when it is time to update your patch. Things are working their way to being merged.14:10
*** yangyapeng has joined #openstack-ironic14:10
openstackgerritMerged openstack/ironic master: Configuration documentation migrated  https://review.openstack.org/49186114:11
sambettsmilan: tbh that one might actually be worth changing to a .. versionchanged::14:11
* TheJulia goes back to hunting something down in linux-iscsi docs14:11
milansambetts, yup14:11
openstackgerritRuby Loo proposed openstack/ironic stable/pike: Configuration documentation migrated  https://review.openstack.org/49672014:12
*** zshi_laptop has quit IRC14:12
milansambetts, hopefully that doesn't render violet ;)14:12
sambettsmilan: orange I think14:12
*** shoaib has joined #openstack-ironic14:12
milanomg14:12
milan:D14:12
openstackgerritMerged openstack/ironic stable/pike: correct URLs in admin docs  https://review.openstack.org/49671214:13
rloodtantsur: sorry, i forgot, another one ready in pike: https://review.openstack.org/#/c/496716/14:14
patchbotpatch 496716 - ironic (stable/pike) - Documentation for irmc hardware type14:14
masberdtantsur, sorry I didn't explain myself properly, I meant which Openstack interface? external, management or guest? or any of them?14:14
sambettsneutron port?14:14
masberyes14:15
*** udesale has joined #openstack-ironic14:15
dtantsurrloo: do you want https://review.openstack.org/#/c/491861/ there too?14:16
patchbotpatch 491861 - ironic - Configuration documentation migrated (MERGED)14:16
openstackgerritMerged openstack/ironic stable/pike: correct URLs in contributor docs & main index  https://review.openstack.org/49671314:16
dtantsurthis is the last chance to get something in :)14:16
sambettsthat depends how you've deployed Ironic, if you are using ironic with dynamic networking then it'll pxe boot for deployment in the provisioning network defined in your ironic.conf, if you are using ironic with flat networking, then it'll try to use whatever the network interface that you pass into nova14:17
rloodtantsur: already cherry-picked, waiting for jenkins14:17
rloodtantsur: https://review.openstack.org/#/c/496720/ :)14:17
patchbotpatch 496720 - ironic (stable/pike) - Configuration documentation migrated14:17
shoaibHi guys, I am very new to ironic, and desperately need some assistance. I installed openstack using packstack all-in-one RDO on a Hardware node. Enrolled an Ironic node, but wwhen the node comes up, It doesn't get any DHCP response from the Controller node running Ironic conductor, Neutron, Nova-compute etc   https://thepasteb.in/p/j2hlQ3m5nxOcP14:17
openstackgerrityolanda.robla proposed openstack/bifrost master: DNM: Change perms for the VM logs, fix log publish  https://review.openstack.org/49454914:18
dtantsursambetts: maybe you know ^^^ I was not able to quickly figure out yesterday14:18
shoaibIronic port-create with MAC ID, does not get any ""vif_port_id" automatically assigned to it, and that port is also not in the Neutron port-list... Any help would be greatly appreciated :)  https://thepasteb.in/p/j2hlQ3m5nxOcP14:19
rloodtantsur: ugh, i just saw this, haven't looked at it yet: https://review.openstack.org/#/c/496480/14:19
patchbotpatch 496480 - ironic - Update documentation for ``ilo`` hardware type14:19
rloodtantsur: forget it, that patch updates a release note and i think it is too late for that right? cuz we cut Pike? (We have to document this, i keep forgetting...)14:20
*** rbartal has quit IRC14:21
sambettsshoaib: are you using nova to deploy the node?14:21
dtantsurrloo: yeah, we probably should not update release notes, they don't replace documentation after all14:22
dtantsurI mean, after the release14:22
rloodtantsur: if i recall, if you update after the branch is cut, it will put that reno in the master/current releases instead.14:22
shoaibsambetts: I tried using Nova boot but that failed saying No valid host found, Doesn't deployment comes after a Node PXE boots ?14:22
shoaibsambetts: Does ironic-node create also fill that port in the neutron list ?14:23
openstackgerritMerged openstack/ironic stable/pike: Documentation for irmc hardware type  https://review.openstack.org/49671614:23
sambettsshoaib: no, when you issue nova boot --nic net-id=<neutron network>, nova creates a neutron port for you, then schedules the instance onto the ironic node, then attaching the neutron port to the ironic node, then triggers Ironic to deploy the node14:24
dtantsurrloo: yeah, likely14:25
sambettsshoaib: when using ironic with nova, you shouldn't intract with ironic directly to deploy nodes14:25
sambettsshoaib: if you are getting No Valid Host Found that indicates to me that there are bigger issues with your environment than neutron ports not being created and attached14:25
sambettsshoaib: you should check your node.properties are set up correctly and your nova flavors are created correctly to match your node's properties14:26
*** makowals_ has quit IRC14:26
sambettsshoaib: and make sure your ironic node is in avaiable state and not in maintenance14:26
*** Nisha_Agarwal has joined #openstack-ironic14:27
sambettsif all those things are correct, you should see your resources appear in nova hypervisors-stats14:27
sambettsand when you do nova boot it should work14:27
shoaibsambetts: My node shows Available and Power on,  https://thepasteb.in/p/P1hvEoX1Py4il14:28
shoaibsambetts: Nova should be used when its not a Standalone Bare Metal node right ? I'm trying to get the concept, because openstack Wiki shows several ways of deploying it14:29
shoaibsambetts: Also I am able to power on/off/ reboot my Node using IPMI with ironic commands14:31
*** udesale has quit IRC14:31
*** udesale__ has joined #openstack-ironic14:31
*** baoli has joined #openstack-ironic14:32
*** JayF has joined #openstack-ironic14:32
*** JayF has joined #openstack-ironic14:32
*** baoli has quit IRC14:33
openstackgerrityolanda.robla proposed openstack/bifrost master: DNM: Change perms for the VM logs, fix log publish  https://review.openstack.org/49454914:34
* jlvillal hopes to get his development workstation back up and running today after an SSD drive failure yesterday14:35
sambettsshoaib: yes, nova should be used when Ironic is in non-standalone mode14:36
jlvillalsambetts, shoaib: Or possibly using Mogan. https://wiki.openstack.org/wiki/Mogan   Though that is not used very much, but it is another option out there.14:38
openstackgerritMerged openstack/ironic stable/pike: Configuration documentation migrated  https://review.openstack.org/49672014:39
*** trown is now known as trown|brb14:41
rloodtantsur: all the stablel/pike patches have merged, except: https://review.openstack.org/#/c/496710/.14:41
patchbotpatch 496710 - ironic (stable/pike) - Documentation for 'snmp' hardware type14:41
rloodtantsur: i just looked and it makes a change to drivers/modules/snmp.py so it won't be fast14:41
rloodtantsur: so i think we shouldn't wait for it14:42
rloodtantsur: unless you want to?14:42
shoaibsambetts: Btw I'm not using ironic-inspector, thats not necessary right ? And I also am now so confused about the sequence in which to boot up the system .. I did it in this order, configured neutron first, then created ironic node, updated it with deploy images, followed by ironic port creation, and then Nova boot ?14:42
openstackgerritSam Betts proposed openstack/ironic master: Switch API ref to use versionadded syntax throughout  https://review.openstack.org/49000814:42
dtantsurrloo: let's release without it, I guess14:43
rlooTheJulia: we're ready to get an updated https://review.openstack.org/#/c/495315/14:43
patchbotpatch 495315 - ironic (stable/pike) - Change doc links for pike branch14:43
*** hamzy has joined #openstack-ironic14:43
shoaibShould a Node be Booted with Nova before or after Ironic node-create ? Thanks..I've been using these instructions here https://docs.openstack.org/ironic/latest/install/index.html , but they are not so easy to follow14:44
sambettsafter14:44
sambettsshoaib: inspector is optional, when you created the ironic node did you specify all its properties, cpu, local_gb and memory_mb ?14:44
*** Nisha_Agarwal has quit IRC14:44
sambettsshoaib: also which version of OpenStack are you using?14:44
rloodtantsur: yup. we can always backport later :)14:45
TheJuliarloo: okay, finishing thoughts into an ether pad, and I'll go update it14:45
rlooTheJulia: thx. I suspect most of your changes in that patch will be removed cuz I changed a lot of the links to be relative/internal ones.14:45
* dtantsur wastes a lot of corporate paper and toner on his relocation paperwork in the meantime :)14:49
*** trown|brb is now known as trown14:50
masbersambets, I am using kolla-ansible for deployment, does it means that the neutron port is neutron_external_interface?14:50
sambettsneutron_external_interface is the neutron public API IP interface14:51
shoaibsambetts: I'm using Ocata14:51
sambettsshoaib: ok, did you set the properties, as specified in enrollment?14:52
*** lihi has quit IRC14:52
*** oanson has quit IRC14:53
shoaibsambetts: Yes i specified the properties, here is the output of my ironic node-show https://thepasteb.in/p/k5hYzrBJpJJfE14:53
*** leyal has quit IRC14:53
*** lihi has joined #openstack-ironic14:54
*** oanson has joined #openstack-ironic14:54
shoaibsambetts: Maybe im using thr wrong Nova boot command, I used this "nova boot --flavor my-baremetal-flavor --image my-image --key-name admin metal"14:54
sambettsshoaib: does your baremetal flavor exactly match your node's properties??14:54
*** Nisha_Agarwal has joined #openstack-ironic14:54
*** leyal has joined #openstack-ironic14:55
TheJuliawheeee lots of merge conflicts14:56
shoaibsambetts: No actually, my flavor has "memory_mb=1024, Disk=100, " different from node properties14:56
shoaibsambetts: My flavor VCPU=2, whereas for the node, its set to 48, Do all these 3 parameters need to match ?14:57
sambettsshoaib: your flavor needs to exactly match your nodes properties to schedulae correctly14:58
*** livelace has quit IRC14:58
rlooTheJulia: sorry. I was wondering if it might be easier if you started from scratch, using your script again14:59
TheJuliarloo: I'd have to write it again since I did it as a one-liner14:59
* TheJulia just fixes14:59
dtantsurfind doc/source -name \*.rst -exec sed -i s/latest/pike/g \{\} \;   :)15:00
* dtantsur did not check it15:00
TheJuliahehe15:01
TheJuliaCatches some other links though15:01
*** crushil has joined #openstack-ironic15:01
TheJuliaso the search/replace needs to be much more specific.15:01
shoaibsambetts: Ok, I'll modify the node/flavor properties ... Also with "nova boot --nic net-id=<neutron network>", Is the neutron network the output of my "neutron net-list" or "neutron subnet-list " ?15:01
sambettsnet-list15:02
*** Nisha_Agarwal has quit IRC15:05
openstackgerritJulia Kreger proposed openstack/ironic stable/pike: Change doc links for pike branch  https://review.openstack.org/49531515:05
*** zhangfei has quit IRC15:05
openstackgerrityolanda.robla proposed openstack/bifrost master: DNM: Change perms for the VM logs, fix log publish  https://review.openstack.org/49454915:09
openstackgerrityolanda.robla proposed openstack/bifrost master: Change perms for the VM logs  https://review.openstack.org/49454915:09
*** makowals has joined #openstack-ironic15:12
*** srobert has joined #openstack-ironic15:13
dtantsurTheJulia, rloo is right, DIB does not branch15:14
TheJuliaYeah, they don't15:14
TheJuliadid I not fix that?15:14
rlooTheJulia: do you think it is possible to get whoever/infra? to put a link from https://docs.openstack.org/diskimage-builder/pike/ to ..latest15:14
rlooTheJulia: you did, but I think I missed some from previous patch set. not sure.15:15
TheJuliafunky....15:15
* dtantsur is on a meeting15:15
TheJuliarloo: I'll just edit the text since they don't branch15:15
*** udesale__ has quit IRC15:15
rlooTheJulia: my other thought is for a link from https://docs.openstack.org/diskimage-builder to go to ../latest. then we can delete 'latest' from the urls.15:16
rlooTheJulia: yeah, edit the patch works for me!15:16
openstackgerritJulia Kreger proposed openstack/ironic stable/pike: Change doc links for pike branch  https://review.openstack.org/49531515:19
TheJuliathey might branch one day, who knows15:20
*** makowals has quit IRC15:20
NobodyCamGood Morning Ironic'ers15:21
shoaibsambetts: I have another question, how does Ironic or Nova gets to know about the tftp Boot .. Is it through the TFTP boot server port 69, which is opened by the server ? Thanks15:23
rlooTheJulia: true...15:24
dtantsurmorning NobodyCam15:24
NobodyCamhey hey Morning dtantsur :)15:24
rloomorning NobodyCam!15:24
NobodyCammorning rloo :)15:24
shoaibWhen using Ironic with Nova, do i need to set "dhcp_provider = neutron" in the ironic.conf file ? Thanks15:36
*** yufei has joined #openstack-ironic15:38
dtantsurTheJulia, rloo, do we want to fix https://review.openstack.org/#/c/495315/4/doc/source/admin/boot-from-volume.rst@19 ? this should be an internal link...15:40
patchbotpatch 495315 - ironic (stable/pike) - Change doc links for pike branch15:40
rloodtantsur: i don't think it is urgent for pike. i will fix on master though.15:40
dtantsurokie15:40
rloodtantsur: the link works, just suboptimal :)15:40
dtantsurokay, now it travels through the gate, thanks rloo and TheJulia15:42
rloodtantsur: yay!15:43
*** yufei has quit IRC15:43
openstackgerritRushil Chugh proposed openstack/ironic-specs master: Add spec for XClarity Driver  https://review.openstack.org/47692215:44
openstackgerrityolanda.robla proposed openstack/bifrost master: DNM: Change perms for the VM logs  https://review.openstack.org/49454915:45
* dtantsur likes how https://docs.openstack.org/ironic/latest/ looks now!15:46
*** bnemec has quit IRC15:46
rloodtantsur: are we ready to bike shed? I think user guide should be sooner.15:46
*** bnemec has joined #openstack-ironic15:47
rloodtantsur: but maybe we shouldn't discuss now. i really would like some openstack guidelines/consistency wrt that landing page.15:47
dtantsuryeah.. I'm fine as long as it can be found.15:47
shoaibHey guys, can anyone kindly answer this for me ? When using Ironic with Nova, do i need to set "dhcp_provider = neutron" in the ironic.conf file ? Thank You very much15:47
dtantsurshoaib: isn't it the default? I think this is the correct value, yes15:47
rloodtantsur: guidelines AND change the highlighting from red to something not so bad.15:48
dtantsurrloo: have you raised this color issue to someone?15:48
rloodtantsur: no, i should, i know. do you know who? or should i just do mail list thing.15:49
dtantsurrloo: this feels like something to get an ML thread15:49
dtantsur.. about15:49
rloodtantsur: ok, will do that today...15:49
dtantsurthanks!15:50
dtantsur-2 POST_FAILURE \o/15:50
shoaibdntantsur: This isn't the default, This is disabled by default.  I used RDO packstack to install openstack, and by default its set to None, and I see no DHCP response coming back to the Bare metal node, maybe this is the cause15:51
*** yufei has joined #openstack-ironic15:53
openstackgerritRuby Loo proposed openstack/ironic master: [doc] change absolute to relative URL  https://review.openstack.org/49680415:54
rloough, what happened here? https://review.openstack.org/#/c/495315/15:54
patchbotpatch 495315 - ironic (stable/pike) - Change doc links for pike branch15:54
rloooh, i see dtantsur has already rechecked. very odd...15:55
dtantsurPOST_FAILURE is an infra issue15:55
dtantsurusually when a worker cannot upload the logs back15:55
dtantsurrloo: ^^^15:55
rloodtantsur: ahh. thx for explaining.15:56
*** rama_y has joined #openstack-ironic15:56
dtantsurnp15:56
shoaibWhen using nova boot to boot ironic baremetal instance, Do i have to provide the UUID of the Ironic node that was created ? I'm trying to figure out the command.. Can anyone kindly assist or give me the syntax of the full command15:59
*** Guest28943 is now known as sdake16:00
*** sdake has joined #openstack-ironic16:00
shoaibnova  boot --image BAREMETAL_USER_IMAGE --flavor BAREMETAL_FLAVOR --nic net-id=IRONIC_NETWORK_ID --key default <IRONIC-NODE_UUID> ? Does this look right ?16:00
shoaibsambetts: nova  boot --image BAREMETAL_USER_IMAGE --flavor BAREMETAL_FLAVOR --nic net-id=IRONIC_NETWORK_ID --key default <IRONIC-NODE_UUID> ? Does this command look right ?16:01
dtantsurshoaib: nova picks a node itself, based on the flavor you request. you can't provide it16:01
shoaibdtantsur: So the node nova boots, would have a different uuid from the one that ironic created ? Thanks16:02
dtrainorHey there TheJulia, you're right, I could have provided some more information about my instackenv.json formatting and using pxe_snmp :) I think it's just a formatting error with driver_info, python-openstackclient never makes an API call.  Here's what I'm working with http://paste.openstack.org/show/619198/16:02
dtrainorPer https://docs.openstack.org/ironic/latest/admin/drivers/snmp.html , and that's why I was looking for an example using driver_info to make sure I wasn't doing something not smart16:02
dtantsurshoaib: nova instance UUID is not related to ironic node UUID. when nova picks a node, it will put the instance UUID into node's instance_uuid field16:03
dtantsurdtrainor: I wonder if tripleo even supports the snmp driver..16:03
TheJuliadtrainor: also, driver_info is not a list16:04
dtantsurdtrainor: tripleo does not support snmp out of box. you have to use native ironic commands instead of instackenv.json and tripleo commands..16:04
dtantsurdtrainor: actually, your instackenv.json is a correct file (minus driver_info, which is a dict) to be loaded into ironic directly. this is not an instackenv.json as understood by tripleo16:05
rloodtantsur: jenkins just gave its nod to this. i still think it may not be worth waiting for but thought i'd let you know: https://review.openstack.org/#/c/496710/16:05
patchbotpatch 496710 - ironic (stable/pike) - Documentation for 'snmp' hardware type16:05
* dtantsur saves some good rant16:05
dtrainorhaha16:06
dtantsurrloo: I'm going soon, so I'd prefer to request the release already16:06
rloodtantsur: ++16:06
TheJuliadtrainor: doooo it!16:06
*** bnemec has quit IRC16:06
* dtantsur watches https://review.openstack.org/#/c/495315/16:07
patchbotpatch 495315 - ironic (stable/pike) - Change doc links for pike branch16:07
dtantsurI wonder why we even run the coverage test on docs-only patches..16:07
dtrainorhrm ok then, i'll look in to a) using driver_info as a dict b) look in to using native ironic commands instead of instackenv.json and tripleo commands16:07
dtrainorthank you16:07
*** jkilpatr has quit IRC16:08
* milan relocate16:08
*** milan has quit IRC16:08
dtantsurTheJulia: tomorrow I think of releasing ironic-ui again to include the translations16:10
TheJuliaLikely a good idea, I think it was korean that was added16:10
dtantsurI think I've approved German as well16:10
dtantsuryeah, and the proposed on is korean16:11
openstackgerritMerged openstack/ironic stable/pike: Change doc links for pike branch  https://review.openstack.org/49531516:11
dtantsurw0000000t16:11
TheJuliaI'll double check later today16:11
dtantsurand https://review.openstack.org/#/c/496820/ is our release request16:13
patchbotpatch 496820 - releases - Release ironic 9.1.0 (final) for Pike16:13
*** karimb has quit IRC16:20
*** racedo has quit IRC16:20
openstackgerrityolanda.robla proposed openstack/bifrost master: DNM: Change perms for the VM logs  https://review.openstack.org/49454916:20
*** weshay_PTO_AM is now known as weshay16:25
dtantsurokay, our release request has got 1x +2, so I think I'm good to call it a day :)16:27
dtantsursee you all tomorrow16:27
sambetts\o/16:27
sambettsthanks dtantsur, have a good evening16:27
*** dtantsur is now known as dtantsur|afk16:29
*** rcernin has quit IRC16:29
*** yufei has quit IRC16:29
*** ijw has quit IRC16:30
rloothx and bye dtantsur|afk!16:31
*** srobert has quit IRC16:31
shoaibsambetts: Hi Sam, sorry to bug you again, but i fixed the flavor and Ironic node properties, but when i do Nova compute, i get this error "VirtualInterfacePlugException: Cannot attach VIF 1904bf62-a5cc-4d18-919a-933aa98ad297 to the node 786303b7-4d25-4dac-ad6a-ef5cf05ee99e due to error: UUID or name of cleaning network is not set in configuration (16:32
shoaibHTTP 400)16:32
shoaibsambetts: I have the properties of flavor and instance matching, but now with nova boot i get this error "error: UUID or name of cleaning network is not set in configuration (HTTP 40016:32
*** e0ne has quit IRC16:32
shoaibWhat am i missing now. Whats a cleaning network ?16:33
sambettsthat is a configuration issue in your ironic.conf, in the [neutron] section cleaning_network needs to be configured to either the name or the uuid of the network in neutron that should be used as the cleaning network16:33
*** mat128 has quit IRC16:35
rama_yHi sambetts: when I boot an instance on the baremetal node, cannot ping/ssh into the instance; neutron-port shows status DOWN. Any suggestions?16:35
*** yufei has joined #openstack-ironic16:38
sambettsrama_y: the neutron port will say DOWN because of a bug in ironic + neutron integration but it should actually be working, the ability to ping/ssh into the instance might be two things, 1. is your network accessible from the outside world, if not then you need to use a neutron router and floating ip to provide external access to the node, and 2. have you configured your neutron security groups16:38
*** yufei has left #openstack-ironic16:38
sambettsto allow ping/ssh to the instance16:38
shoaibsambetts: To resolve the cleaning network issue, I must add the flat_network name in this paramter "cleaning_network = <None>" in ironic.conf ?16:39
sambettsshoaib: yes16:39
* sambetts --> home 16:42
*** sambetts is now known as sambetts|afk16:42
sambetts|afkNight all!16:42
jlvillalsambetts|afk: Ciao!16:42
rloogood night sambetts|afk16:43
rama_ysambetts|afk, thanks & good night16:43
*** jkilpatr has joined #openstack-ironic16:53
*** yangyapeng has quit IRC16:53
*** mgoddard has quit IRC16:55
*** jcoufal has quit IRC16:56
*** trown is now known as trown|lunch16:56
*** amarao1 has quit IRC17:00
*** amarao has quit IRC17:00
*** mtanino_ has joined #openstack-ironic17:03
*** mtanino_ has quit IRC17:03
*** karimb has joined #openstack-ironic17:03
*** mtanino has quit IRC17:04
shoaibsambetts: I still run into "No Valid Hosts Found" when i do nova-boot according to the nova-conductor.log... The CPU/Memory settings are set to much less then what my HW node can handle17:05
shoaibAnybody still here to help out ? I'm facing issues trying to Boot a bare metal instance with Nova . Keep on getting "No Valid Hosts found. there are not enough hosts available" error.. Thanks17:07
*** lucasagomes is now known as lucas-afk17:07
*** pcaruana has quit IRC17:08
*** ijw has joined #openstack-ironic17:08
*** derekh has quit IRC17:10
*** fellypefca has quit IRC17:10
shoaibTo configure Baremetal instances with Nova, Do i need to set "baremetal_scheduler_default_filters" or any specific filters in nova.conf ? Thanks17:12
*** tesseract-RH has quit IRC17:12
TheJuliashoaib: the requested instance needs to match the hardware17:14
TheJuliaerr, requested flavor17:14
TheJuliathe filters, if memory serve, with baremetal result in needing to do exact matching17:15
TheJuliasince you can have only one instance per "hypervisor" with the ironic nova virt driver17:15
shoaibTheJulia: When you say, match the hardware, Does it mean, that I have to use all resources available on HW, or just a subset of Available memory , CPUs etc on Hardware ?17:18
TheJuliaExact matching, so if the hardware is 4 cpus, then the nova flavor needs to be 4vcpus17:19
TheJuliaif it has 8192MB of ram, then the flavor needs to have that much ram17:19
shoaibTheJulia: Maybe this is a silly question, but if my Hardware node is available state, does ironic has the capability to determine how much HW CPUs, specs are available ?17:23
shoaibTheJulia : or can ironic retrieve that data through IPMI using some CLI command ? Thanks17:23
TheJuliashoaib: That is what we have inspector for and the inspect provision state verb17:24
TheJuliahowever, that is a separate service and configuration17:24
*** racedo has joined #openstack-ironic17:24
TheJuliaIPMI differs so much that there is no real use to trying to get that data out, since I think it would be vendor/chipset specific too.17:25
TheJuliashoaib: inspector will also create ports for the node17:25
TheJuliawhich is important for PXE/iPXE17:25
shoaibTheJulia: so when i created ironic port create , I also dont see that port in the "neutron port-list" , Does this ironic port gets populated in neutron after Nova boots up an instance ?17:27
TheJuliaso the neutron port is virtual, the ironic port is physical. Ironic only creates ports in neutron to do things like attaching to cleaning networks.  Nova creates a port for an instance, and tells ironic about that, and then ironic tells neutron to attach that when required.17:28
*** makowals has joined #openstack-ironic17:28
TheJuliaso yes, as a part of boot/deploy17:29
*** mat128 has joined #openstack-ironic17:31
shoaibTheJulia: Can you kindly point me to a good set of instructions to start, use ironic-inspector... Cause I don't know about the specs of the HW node (customer specs), I guess using Ironic inspector maybe the quickest way for me17:34
*** mgoddard has joined #openstack-ironic17:34
*** mat128 has quit IRC17:34
*** rcernin has joined #openstack-ironic17:34
shoaibTheJulia: Can the ironic inspector be used as a standalone tool, to use a few CLi commands and use it to retrieve HW specs (CPUD, Memory etc) ? Thanks17:34
TheJuliahttps://docs.openstack.org/ironic-inspector should have links to installation and use documentation17:35
TheJuliait can't really be used as a stand-alone tool, in terms of retrieving data, it is intended to really search/publish back into ironic17:35
*** alex_xu has quit IRC17:37
*** amarao has joined #openstack-ironic17:39
*** alex_xu has joined #openstack-ironic17:42
dtrainorhmm, 'm getting farther, i can get ironic to recognize the node defined with node-create/node-update etc, but when I try to inspect, I get:  EBUG wsme.api [req-59f676bc-f1f8-4dc7-9edc-f6f26b5471c6 a99d2bccad954320ac7fb744f0d8afb1 a7bfd3a5e4ef42e398ebd44f6a96d563 - - -] Client-side error: Driver pxe_snmp does not support management (disabled or not implemented).17:43
TheJuliadtrainor: is pxe_snmp on the enabled_drivers list in ironic.conf17:44
openstackgerritRuby Loo proposed openstack/ironic master: [admin docs] ironic -> openstack baremetal CLI  https://review.openstack.org/49433517:44
dtrainorit is, and openstack-ironic-* has been restarted17:44
TheJuliaoh17:45
TheJuliaoh17:45
dtrainoroh dear17:45
* TheJulia looks at somthing17:45
dtrainorbased on what i'm learning, i would wager that some functions of pxe_snmp are simply not developed in to ironic yet to support all the capabilities17:46
*** jcoufal has joined #openstack-ironic17:46
openstackgerritRuby Loo proposed openstack/ironic master: [admin docs] ironic -> openstack baremetal CLI  https://review.openstack.org/49433517:46
*** trown|lunch is now known as trown17:46
TheJuliadtrainor: https://github.com/openstack/ironic/blob/master/ironic/drivers/pxe.py#L10117:46
dtrainoroh.17:47
dtrainorok, that makes sense17:47
TheJuliamanagement is used to say "Hey, PXE boot new time you power on" or "boot from disk now"17:47
dtrainorriiight17:47
TheJuliaand I guess, if you were up for some low level programming and wanted to define an OID...17:48
TheJuliait could be a thing....17:48
shoaibHi, I have a question about the sequence in which Baremetal node boot ... when i do ironic-node create, and provide it with the deploy_image, and power is up, is it at this stage that the node is supposed to bootup using tftp and PXe ?17:48
dtrainori am using  properties/capabilities=boot_option:local, wonder if that's making a sad17:48
dtrainorgood call, great clue, thank you17:48
shoaibOr do i need to issue a "nova boot command with a user_image" to kickstart the deploy_boot (kernel/ramdisk)17:48
TheJuliadtrainor: likely part of the problem17:48
dtrainori'll run with that.  i appreciate it.17:49
TheJuliashoaib: so adding a node and moving it from manageable to available engages cleaning which does use the deployment image/ramdisk17:49
TheJuliawell, deployment kernel/ramdisk17:49
TheJuliashoaib: When deploying, nova boot triggers the state change to deploy the node, which boots the deployment kernel/ramdisk, writes the user image to disk, and reboots the node17:50
shoaibTheJulia: See when i created a node using ironic-create, I didnt manually change the state to available, it came up as available"17:50
shoaibTheJulia: and then i powered my node, at which stage it sends a DHCP request, but doesnt get any response back17:50
TheJuliashoaib: ahh, so that was changed in api version 1.917:51
*** livelace has joined #openstack-ironic17:51
shoaibTheJulia: upto this stage, I hadn't issued "nova boot--" command, and thats where I'm confused17:51
TheJuliashoaib: try using the latest api version, 'latest' should cause it to be autonegotated17:51
dtrainorhrm, TheJulia when i verify the node, I see:  | inspect    | None   | not supported |17:52
TheJuliayeah, that kind of makes sense, although it should be supportable17:52
TheJuliahmmmmmmm17:52
TheJuliaWhere is a milan when we have a question17:53
dtrainorhehe17:53
TheJuliaso, basically, if there is an inspect interface, I think you would likely get further17:53
dtrainoras in, there's no inspect interface in pxe_snmp ?17:54
*** yangyapeng has joined #openstack-ironic17:54
shoaibTheJulia: So at what stage does the DHCP response comes in play, Is it the "nova boot" that triggers a DHCP response to be sent back to the BareMetal node that was enrolled >?17:54
TheJuliadtrainor: maybe something like https://github.com/openstack/ironic/blob/master/ironic/drivers/ipmi.py#L7217:54
TheJuliadtrainor: no, there is not from what I saw17:55
dtrainoroh i see what you did there17:55
TheJuliadtrainor: https://github.com/openstack/ironic/blob/master/ironic/drivers/pxe.py#L101 and look 21 lines below :)17:55
dtrainoryep i saw that, cool17:56
TheJuliashoaib: there are a few steps in between....17:56
dtrainori'll hack on this for a minute, thanks again17:56
TheJuliadhcp response is set by ironic upon attaching the port17:56
*** racedo has quit IRC17:56
TheJuliaso when ironic powers the node up, the correct reply is sent by neutron17:56
TheJuliawell, whatever neutron's dhcp server is, typically dnsmasq17:57
*** yufei has joined #openstack-ironic17:58
*** livelace has quit IRC17:58
shoaibTheJulia: See thats my problem, I did do an ironic port-create, after creating an ironic node, this is all before nova-boot, and I dont see this ironic port attaching to neutron, which you are saying is very very important to get a DHCP response17:58
*** yangyapeng has quit IRC17:58
openstackgerritRuby Loo proposed openstack/ironic master: [admin docs] ironic -> openstack baremetal CLI  https://review.openstack.org/49433517:58
TheJuliareplies are going to be delayed, I have to jump on a call17:59
shoaibTheJulia: So Nova boot is never going to work for me, if my Node doesn't even get a DHCP response, right ? As neutron doesn't have the port created with the MAC Address of the baremetal node in its list ?17:59
*** e0ne has joined #openstack-ironic18:00
*** yufei has left #openstack-ironic18:00
*** amarao has quit IRC18:04
*** e0ne has quit IRC18:13
*** e0ne has joined #openstack-ironic18:22
shoaibTheJulia: Should nova-boot be triggered after the Node is up, available, and has a DHCP response received ? Thanks18:31
*** makowals has quit IRC18:31
openstackgerritRuby Loo proposed openstack/ironic master: remove REST API examples from RAID doc  https://review.openstack.org/49684018:31
openstackgerritMichael Johnson proposed openstack/python-ironicclient master: Add auto-generated CLI reference  https://review.openstack.org/49684118:33
*** e0ne has quit IRC18:33
openstackgerritMichael Johnson proposed openstack/python-ironicclient master: Add auto-generated CLI reference  https://review.openstack.org/49684118:35
*** jcoufal has quit IRC18:36
*** livelace has joined #openstack-ironic18:37
*** jcoufal has joined #openstack-ironic18:38
*** e0ne has joined #openstack-ironic18:40
*** livelace has quit IRC18:41
*** ChubYann has joined #openstack-ironic18:43
*** etingof has quit IRC18:44
*** rcernin has quit IRC18:47
*** johnsom has joined #openstack-ironic18:50
*** jtomasek_ has quit IRC18:52
TheJuliashoaib: okay, back18:53
TheJuliashoaib: so, all attachments take place after the nova boot18:53
TheJuliayou can set default dhcp replies for things like inspection in your dnsmasq config18:54
TheJuliaironic takes care of that neutron interaction for port creation, so if you have created a port in neutron manually for the node,  you need to delete it18:54
TheJulianova boot is what starts the process, so the node should be off, and booting the node should result in the power being turned on18:55
*** yangyapeng has joined #openstack-ironic18:55
shoaibTheJulia: I didn't create any port in neutron manually, I just created the ironic port..18:56
TheJuliashoaib: so what exactly is your current state, and what is your expected state?18:57
TheJuliabecause I feel like there is a disconnect there18:57
*** mgoddard has quit IRC18:57
shoaibTheJulia: At this stage, I have an ironic node status as "available " { which came up by itself } , and power is on , but Nova boot keeps failing with this error : https://thepasteb.in/p/pghQ3v2r9RnSR18:58
shoaibThis is my output from "ironic node-show " https://thepasteb.in/p/LghNnmW6p9jfZ18:58
*** yangyapeng has quit IRC18:59
TheJuliashoaib: so that looks like a pure scheduling issue. I'd verify that your nova-compute process is running with the ironic virt driver, and check that the scheduler process is seeing inventory before trying to boot a node18:59
*** etingof has joined #openstack-ironic19:00
shoaibTheJulia: I'm not sure about the expected state, frankly i expected my BareMetal Node to start booting with the Deployment ramdisk/kernel .. Instead the bare-metal node shows error saying that "No DHCP response received"19:00
TheJuliaOkay, so  you can power down the baremetal node, no sense in having it powered on right now19:01
TheJuliayour issue, presently is entirely before ironic is told to deploy19:01
shoaibTheJulia: Allright, Can you also kindly clarify what you mean by  "ironic takes care of that neutron interaction for port creation" ... Somebody here earlier mentioned that when you do ironic port-create, that port should have a ""vif_port_id" too so that it can be populated in the neutron-lists19:04
TheJuliayou will only have a vif_port_id once nova has tried to initiate a deployment19:05
shoaibTheJulia: This is the output from my ironic port  https://thepasteb.in/p/RghnqRYvwpJtz19:05
TheJuliabut your not even getting that far because the scheduler is failing19:05
shoaibTheJulia: Oh ok, I get it .. So basically Nova is the root of the problem, alhtough i see nova-compute and the nova-schedular running19:06
TheJuliathey may be running, but I suspect there will be errors in the logs19:07
TheJuliaor it will just not be seeing the baremetal hardware as available resources19:07
shoaibTheJulia: also in the latest release, you mentioned that an ironic node created with status "enroll", and then a user has to manually set it to "deploy ", and "available " state, Is that correct ? Or is that only going to happen in Pike release ?19:09
TheJuliaso, kind of19:11
TheJulianot deploy, that is what nova does19:11
TheJuliahttps://docs.openstack.org/ironic/latest/contributor/states.html might help show how it works19:12
*** ijw has quit IRC19:13
TheJuliashoaib: and it is not in the latest release, it has been in the api for... four releases I believe19:17
shoaibTheJulia: Ok, so basically as far as I understand this state machine, the state will go to "Deploy" after the "Available" state, and its nova that kicks it into that mode19:17
TheJuliacorrect19:18
TheJuliaA human _can_ do it manually without nova, but the vast majority of users use nova19:18
*** srobert has joined #openstack-ironic19:19
*** mat128 has joined #openstack-ironic19:20
*** ijw has joined #openstack-ironic19:21
shoaibTheJulia: Where is the nova-compute supposed to run, on the bare-metal node, or on the Controller node ?19:23
rloomariojv: if you're still interested, look what someone just submitted: https://review.openstack.org/#/c/496841/19:23
patchbotpatch 496841 - python-ironicclient - Add auto-generated CLI reference19:23
TheJuliashoaib: controller node19:25
*** ijw has quit IRC19:28
*** ijw has joined #openstack-ironic19:29
*** baoli has joined #openstack-ironic19:34
*** baoli has quit IRC19:38
shoaibTheJulia: Yeah in my case, looks like nova cant even see the baremetal node. In my hypervisor-list, all i see is the Ironic node, Shouldn't a Compute-Host name also be under the Hypervisor-list ?19:42
*** mgoddard has joined #openstack-ironic19:44
TheJuliaSounds kind of like your nova-compute node is misconfigured19:44
openstackgerritRamamani Yeleswarapu proposed openstack/ironic-ui master: Unit tests for set target raid configuration  https://review.openstack.org/49685719:45
shoaibTheJulia: I'm looking in the nova.conf file, and I also see "baremetal_enabled_filters" disabled, and "use_baremetal_filter=False" ... Would this impact nova from finding a baremetal node host ?19:46
TheJuliaMaybe, we're reaching the edge of my Nova knoweldge. I would recommend checking out install docs19:47
osdevTheJulia: In the non-baremetal case, nova-compute runs on the host which is hosting the virtual machines; it seems in the baremetal/Ironic case we expect nova-compute to also run on the "controller" -("controller" here meaning where nova-scheduler, ironic-conductor) is currently running?  (Or did I misinterpret?)19:53
*** yangyapeng has joined #openstack-ironic19:56
TheJuliaWe don't really care where it runs, as long as it is considered a running controller service as it is what queries and returns data from ironic to nova20:00
*** yangyapeng has quit IRC20:01
TheJuliaNaturally, it can't run on an undeployed baremetal node since it is actually what is executing the virt driver20:01
johnsomrloo Hi, thanks for the bug.  I do have a question about one of your comments however.  The index you were requesting, would that be on the osc_plugin_cli.rst page?  After you click into osc/v1/index.html the index is in the left nav20:06
rloohi johnsom. give me a few secs to find it...20:07
rloojohnsom: so you're asking me about this page? https://docs.openstack.org/python-octaviaclient/latest/cli/index.html20:08
rloojohnsom: where i'd like to see that equivalent in ironic?20:08
johnsomrloo Yeah, would you like that tree in the http://docs-draft.openstack.org/41/496841/2/check/gate-python-ironicclient-docs-ubuntu-xenial/1fbc257//doc/build/html/cli/osc_plugin_cli.html#command-reference section?20:08
rloojohnsom: ideally no. but i think for now, it might be easiest. let me think.20:09
johnsomrloo On the new way, the list moves to the left nav, like here: http://docs-draft.openstack.org/27/496827/1/check/gate-python-octaviaclient-docs-ubuntu-xenial/9b45153//doc/build/html/cli/index.html20:09
rloojohnsom: right, but it is ugly.20:10
rloojohnsom: i mean, i guess when we get used to the stuff on the left nav20:10
osdevTheJulia: thanks, I suppose even if we had many baremetal nodes, the single instance of nova-compute (which could be running on a controller) would handle the returning of data from ironic to nova.  hope I understood that corrrectly.20:10
johnsomrloo I don't disagree20:10
rloojohnsom: right now, i think that stuff is ugly so i'm not used to it20:10
rloojohnsom: the extra neurons i have to use to actually distinguish the two left panes or whatever :-(20:11
johnsomrloo I wish we had more control over the left nav behavior20:11
TheJuliaosdev: correct, although you can run multiple nova-compute processes with ironic, they will distribute the baremetal nodes they are responsible for and acting upon across the running nova-compute services20:12
rloojohnsom: i'm just focussed on changing the RED for now...20:12
rloojohnsom: but that doesn't answer your question :)20:12
rloojohnsom: is the plan to get rid of https://docs.openstack.org/python-octaviaclient/latest/cli/index.html?20:12
johnsomrloo Well, that is what I did, but your comments are making me thing about it.  The challenge is with that intermediate page the left nav gets odd20:13
rloojohnsom: cuz I find when I look at http://docs-draft.openstack.org/41/496841/2/check/gate-python-ironicclient-docs-ubuntu-xenial/1fbc257//doc/build/html/cli/osc_plugin_cli.html#command-reference I only see 5 things and there are way more commands than that. i know it isn't listing the commands but it almost seems that way.20:13
johnsomI can bump that up to go another level deep and see what happens20:14
rloojohnsom: ok. if it is too hard, i am happy to get what we get, which is better than what we had. but if I can ask, I'll ask :)20:14
rloojohnsom: am i correct in thinking that you are the only one adding these pages, so you are setting the precedent?20:15
*** Goneri has quit IRC20:15
johnsomrloo lol, ummm, I don't know if I would go that far...  Just wanted good docs for octavia and got asked to do the same for ironic.20:16
rloojohnsom: ha ha!20:16
johnsomrloo So, it works fine with adding the layer down.20:16
rloojohnsom: sweet.20:16
johnsomrloo At some point that OSC page could be reworked, but that is a style thing IMO.20:17
rloojohnsom: we're all so busy that it is best to get it as 'right as we can' when we do it...20:18
johnsomrloo Yeah, immediately after this I need to go start beating on the octavia-dashboard, so I understand busy.20:19
rloojohnsom: I appreciate you offering to help out ironic!20:20
openstackgerritMichael Johnson proposed openstack/python-ironicclient master: Add auto-generated CLI reference  https://review.openstack.org/49684120:20
rloojohnsom: if what I'm asking for is too much work, don't do it :)20:20
johnsomrloo Take a look at that and let me know what you think.20:20
rloojohnsom: will do, thx!20:20
*** jkilpatr has quit IRC20:22
*** dprince has quit IRC20:26
*** rcernin has joined #openstack-ironic20:35
*** mgoddard has quit IRC20:43
*** jcoufal has quit IRC20:43
rloojohnsom: thx, +2!20:43
johnsomrloo Cool, happy to help20:44
rloojohnsom: :) Now let's hope for more agreement!20:44
*** shoaib has quit IRC20:48
*** e0ne has quit IRC20:52
*** yangyapeng has joined #openstack-ironic20:56
*** masber has quit IRC20:57
openstackgerritRamamani Yeleswarapu proposed openstack/ironic master: [TEST] [DO NOT MERGE] Testing tls-proxy enabling  https://review.openstack.org/49266421:00
*** trown is now known as trown|outtypewww21:01
*** yangyapeng has quit IRC21:01
*** srobert has quit IRC21:10
*** racedo has joined #openstack-ironic21:10
*** crushil has quit IRC21:15
*** racedo has quit IRC21:20
*** mat128 has quit IRC21:23
*** bfournie has quit IRC21:25
*** cdearborn has quit IRC21:28
*** jkilpatr has joined #openstack-ironic21:29
*** crushil has joined #openstack-ironic21:32
*** ijw has quit IRC21:53
*** jlabarre has quit IRC21:53
*** ijw has joined #openstack-ironic21:56
*** yangyapeng has joined #openstack-ironic21:57
mariojvthanks rloo and johnsom, +1ed it22:00
rloomariojv: thx! :)22:00
*** yangyapeng has quit IRC22:02
*** ijw has quit IRC22:07
*** ijw has joined #openstack-ironic22:20
*** crushil has quit IRC22:29
*** rpioso has quit IRC22:43
*** yangyapeng has joined #openstack-ironic22:58
*** mjturek has quit IRC23:00
*** yangyapeng has quit IRC23:02
*** athomas has quit IRC23:11
*** srobert has joined #openstack-ironic23:24
*** srobert has quit IRC23:26
*** karimb has quit IRC23:30
*** lennyb has quit IRC23:31
*** rloo has quit IRC23:33
*** baoli has joined #openstack-ironic23:34
*** baoli has quit IRC23:39
*** lennyb has joined #openstack-ironic23:41
*** charcol has joined #openstack-ironic23:44
*** ijw has quit IRC23:47
*** bnemec has joined #openstack-ironic23:54
*** ijw has joined #openstack-ironic23:58
*** yangyapeng has joined #openstack-ironic23:59

Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!