Wednesday, 2019-10-23

« Tuesday, 2019-10-22 Index Thursday, 2019-10-24 »
*** rcernin has joined #openstack-ironic00:05
*** cdearborn has quit IRC00:10
*** gyee has quit IRC00:18
*** hamzy__ has joined #openstack-ironic00:51
*** hamzy_ has quit IRC00:53
*** mkrai has joined #openstack-ironic00:59
*** TxGirlGeek has joined #openstack-ironic01:05
*** tzumainn has quit IRC01:34
*** mbeierl has quit IRC01:46
*** ociuhandu has joined #openstack-ironic02:06
*** ociuhandu has quit IRC02:12
*** goldyfruit has joined #openstack-ironic02:25
*** rloo has quit IRC02:31
*** rh-jelabarre has quit IRC02:46
*** ricolin has joined #openstack-ironic03:16
*** mkrai has quit IRC03:59
*** mkrai_ has joined #openstack-ironic03:59
*** ociuhandu has joined #openstack-ironic04:31
*** ociuhandu has quit IRC04:35
*** mkrai_ has quit IRC04:38
*** mkrai has joined #openstack-ironic04:42
*** Sayali_Kutwal has joined #openstack-ironic04:43
*** jtomasek has joined #openstack-ironic04:51
*** dpaterson has quit IRC04:55
openstackgerritIlya Etingof proposed openstack/ironic master: Add `instance_info/kernel_append_params` to `redfish`  https://review.opendev.org/68709205:03
openstackgerritIlya Etingof proposed openstack/ironic master: Burn configdrive into virtual media boot ISO  https://review.opendev.org/68725005:03
openstackgerritIlya Etingof proposed openstack/ironic master: Add `network_data` field to ports and portgroups  https://review.opendev.org/68791005:03
openstackgerritIlya Etingof proposed openstack/ironic master: Add `get_node_network_data` method to NetworkInterface  https://review.opendev.org/68818405:04
openstackgerritIlya Etingof proposed openstack/ironic master: Add `get_node_network_data` to non-Neutron NetworkInterface  https://review.opendev.org/68826605:04
openstackgerritIlya Etingof proposed openstack/ironic master: Make redfish boot interface using network interface metadata  https://review.opendev.org/68827205:04
openstackgerritIlya Etingof proposed openstack/ironic master: Add `get_node_network_data` to Neutron NetworkInterface  https://review.opendev.org/68889605:04
openstackgerritIlya Etingof proposed openstack/ironic master: Add L3 boot section to the docs  https://review.opendev.org/68984405:05
*** TxGirlGeek has quit IRC05:17
*** Sayali_Kutwal has quit IRC05:20
*** ociuhandu has joined #openstack-ironic05:35
*** andyzon has joined #openstack-ironic05:35
*** ociuhandu has quit IRC05:39
*** jtomasek has quit IRC05:52
*** igordc has joined #openstack-ironic05:58
*** igordc has quit IRC06:03
*** andyzon has quit IRC06:16
*** belmoreira has joined #openstack-ironic06:27
*** rcernin has quit IRC06:37
*** andyzon has joined #openstack-ironic06:43
*** andyzon has quit IRC06:48
*** andyzon_ has joined #openstack-ironic06:48
*** andyzon has joined #openstack-ironic06:49
*** andyzon_ has quit IRC06:53
*** trident has quit IRC06:55
*** iurygregory has joined #openstack-ironic06:57
iurygregorygood morning Ironic!06:57
*** trident has joined #openstack-ironic07:00
arne_wiebalckgood morning iurygregory and ironic!07:05
iurygregoryarne_wiebalck, o/07:05
*** bobmel has joined #openstack-ironic07:09
kaifengmorning iurygregory arne_wiebalck and ironic o/07:56
iurygregorykaifeng, o/07:56
*** e0ne has joined #openstack-ironic07:57
*** belmoreira has quit IRC07:58
*** yolanda has joined #openstack-ironic07:58
*** mgoddard has joined #openstack-ironic08:05
*** rpittau|afk is now known as rpittau08:07
rpittaugood morning ironic! happy hump day! o/08:07
*** dougsz has joined #openstack-ironic08:13
*** lucasagomes has joined #openstack-ironic08:15
*** e0ne has quit IRC08:22
*** alexmcleod has joined #openstack-ironic08:25
arne_wiebalckhey kaifeng rpittau o/08:32
rpittauhey arne_wiebalck :)08:33
*** jtomasek has joined #openstack-ironic08:36
*** dtantsur|afk is now known as dtantsur08:38
dtantsurmorning ironic08:38
dtantsurarne_wiebalck, I'm totally interested in helping with the bare metal SIG08:39
arne_wiebalckdtantsur: awesome, thanks!08:41
arne_wiebalckdtantsur: how about having two co-chairs then?08:42
dtantsur++08:42
arne_wiebalckdtantsur: to share responsibility, tasks ... and glory :-D08:43
dtantsur:D08:43
openstackgerritMerged openstack/sushy-tools master: Switch to Ussuri job  https://review.opendev.org/68980208:49
dtantsurTheJulia: it may be the right time to transfer the release liaison position to mgoddard (strictly speaking, there can be several liaisons, so you don't *have* to remove me if you want a backup)08:49
mgoddarddtantsur: training complete?08:49
dtantsuryep, congrats :)08:50
*** pcaruana has joined #openstack-ironic09:02
openstackgerritIlya Etingof proposed openstack/ironic-specs master: Update L3 based deployment spec  https://review.opendev.org/67278009:08
openstackgerritDmitry Tantsur proposed openstack/ironic stable/stein: Change MTU logic to allow for lower MTUs automatically  https://review.opendev.org/69052009:09
dtantsurrpittau: this ^^ seems to improve the situation with the tempest plugin09:09
dtantsur(at least it seems to have helped with train)09:10
openstackgerritDmitry Tantsur proposed openstack/ironic stable/rocky: Change MTU logic to allow for lower MTUs automatically  https://review.opendev.org/69052109:12
openstackgerritDmitry Tantsur proposed openstack/ironic stable/queens: Change MTU logic to allow for lower MTUs automatically  https://review.opendev.org/69052209:13
iurygregorymorning rpittau dtantsur o/09:13
rpittauhey iurygregory :)09:13
rpittaudtantsur: checking09:13
arne_wiebalckdiablo_rojo: https://review.opendev.org/#/c/690519/09:14
patchbotpatch 690519 - governance-sigs - Add new chairs for the Baremetal SIG - 1 patch set09:14
*** e0ne has joined #openstack-ironic09:16
*** rcernin has joined #openstack-ironic09:23
*** derekh has joined #openstack-ironic09:31
openstackgerritDmitry Tantsur proposed openstack/ironic master: [WIP] Correct handling of ramdisk_params in (i)PXE boot  https://review.opendev.org/69052809:36
*** yolanda__ has joined #openstack-ironic09:41
*** yolanda has quit IRC09:43
openstackgerritMerged openstack/ironic stable/train: Do not ignore 'fields' query parameter when building next url  https://review.opendev.org/69009109:48
*** hjensas is now known as hjensas|afk09:55
*** yolanda__ is now known as yolanda10:03
*** tssurya has joined #openstack-ironic10:03
openstackgerritRiccardo Pittau proposed openstack/ironic master: [WIP] Using loop instead of with_X  https://review.opendev.org/69053710:07
* iurygregory likes ^ it may help the failure for py3 I think10:09
rpittauiurygregory: that's exactly why I did that :D10:09
* iurygregory try to change the requirements in my wip patch to test10:10
*** seba has quit IRC10:13
rpittauansible 2.5 is also EOL, might be worth to look at 2.610:15
dtantsuryeah. xXraphXx, which version of ansible are you using? re ^^^10:16
xXraphXxchecking10:17
rpittauI mean, we could go to 2.8, but maybe too cuttinge edge ?10:17
xXraphXx2.510:18
*** jtomasek has quit IRC10:30
dtantsurokay, then let's keep 2.5 for now10:35
openstackgerritIury Gregory Melo Ferreira proposed openstack/ironic master: [DNM] ironic-standalone Py3 ansible2.5 dictsort  https://review.opendev.org/68972110:36
* iurygregory prays for devstack10:36
*** pcaruana has quit IRC10:41
*** khansa has joined #openstack-ironic10:42
*** yaawang_ has quit IRC10:51
*** yaawang_ has joined #openstack-ironic10:52
*** belmoreira has joined #openstack-ironic10:55
*** dougsz has quit IRC11:00
*** mkrai has quit IRC11:01
*** dtantsur is now known as dtantsur|brb11:01
openstackgerritIury Gregory Melo Ferreira proposed openstack/ironic master: Update release with information about zuul job  https://review.opendev.org/69002211:06
openstackgerritDmitry Tantsur proposed openstack/ironic master: Correct handling of ramdisk_params in (i)PXE boot  https://review.opendev.org/69052811:10
openstackgerritDmitry Tantsur proposed openstack/ironic master: [WIP] Wire in in-band inspection for PXE boot and neutron-based networking  https://review.opendev.org/68872011:10
openstackgerritDmitry Tantsur proposed openstack/ironic-inspector master: [WIP] Create a job with boot and networking managed by ironic  https://review.opendev.org/68892011:11
*** jtomasek has joined #openstack-ironic11:19
openstackgerritMerged openstack/ironic-inspector stable/train: Silence amqp DEBUG messages in logs  https://review.opendev.org/69014011:40
openstackgerritMerged openstack/python-ironicclient stable/rocky: fix session cert arguments  https://review.opendev.org/68966311:40
*** rh-jelabarre has joined #openstack-ironic11:49
iurygregoryopenstack tox jobs are broken FYI11:50
iurygregoryERROR: Could not find a version that satisfies the requirement keystoneauth1===3.18.0, some mirror issues =)11:51
*** seba has joined #openstack-ironic11:52
*** yaawang_ has quit IRC11:58
*** dtantsur|brb is now known as dtantsur12:09
*** belmoreira has quit IRC12:09
dtantsuriurygregory: probably a temporary post-release condition12:09
dtantsurI hope12:09
iurygregorydtantsur, yeah, infra is working on it12:09
dtantsurmgoddard: we should probably release ironic stein and train because of the merged security fix12:10
rpittauas we used to say: Internet is broken12:10
dtantsurit is12:10
dtantsurand it's not only tox jobs, all jobs are subject to this problem12:11
iurygregoryyeah12:13
* rpittau flips table12:13
*** goldyfruit has quit IRC12:14
* iurygregory doesn't flip the table because there is 2 more people with computers on the same table12:15
rpittauthat would be truly choreographic12:16
dtantsur:D12:16
iurygregoryhehehehehe12:18
openstackgerritkhansa proposed openstack/sushy master:  Story: 2006610  Task: 36779  https://review.opendev.org/69055312:21
mgoddarddtantsur: the JSON RPC one?12:23
dtantsurmgoddard: yep12:23
mgoddardok12:23
etingofiurygregory, should make other people using devstack so you could flip tables altogether12:26
iurygregoryhahahaha12:26
iurygregorythey told to kill devstack XD12:26
openstackgerritMerged openstack/ironic stable/stein: Do not ignore 'fields' query parameter when building next url  https://review.opendev.org/69009212:29
*** hamzy_ has joined #openstack-ironic12:30
openstackgerritRiccardo Pittau proposed openstack/ironic master: [WIP] Using loop instead of with_X  https://review.opendev.org/69053712:31
*** hamzy__ has quit IRC12:33
*** hamzy has joined #openstack-ironic12:35
*** hamzy_ has quit IRC12:37
*** dougsz has joined #openstack-ironic12:40
*** belmoreira has joined #openstack-ironic12:50
*** pcaruana has joined #openstack-ironic12:53
openstackgerritMerged openstack/ironic stable/stein: Change MTU logic to allow for lower MTUs automatically  https://review.opendev.org/69052013:00
*** bobmel has quit IRC13:01
*** hamzy has quit IRC13:03
*** mbeierl has joined #openstack-ironic13:06
*** macme has joined #openstack-ironic13:14
*** tzumainn has joined #openstack-ironic13:15
*** rcernin has quit IRC13:24
*** goldyfruit has joined #openstack-ironic13:25
openstackgerritDmitry Tantsur proposed openstack/ironic-inspector master: [WIP] Create a job with boot and networking managed by ironic  https://review.opendev.org/68892013:25
*** belmoreira has quit IRC13:25
*** whoami-rajat has quit IRC13:27
TheJuliagood morning13:30
khansagood morning :)13:34
iurygregorymorning TheJulia13:36
dtantsurmorning TheJulia13:38
openstackgerritMerged openstack/ironic stable/queens: Change MTU logic to allow for lower MTUs automatically  https://review.opendev.org/69052213:40
TheJuliadtantsur: I thought I... already had...13:45
*** jcoufal has joined #openstack-ironic13:46
TheJuliahmm13:46
TheJuliaSuddenly feel like I'm loosing my mind, but devstack did finally stack properly and pass tests locally yesterday \o/13:47
dtantsur\o/13:47
iurygregoryTheJulia, ironic_standalone with python3?? *-*13:50
TheJuliaiurygregory: general basic baremetal ops with py313:52
iurygregorycool! =)13:52
*** hamzy has joined #openstack-ironic13:53
TheJuliamgoddard: dtantsur: so they moved from tracking liaisons from the wiki to git... so me even putting Mgoddard down got lost in the shuffle. I'll fix that shortly13:53
TheJuliaafter more coffee13:53
TheJuliaGood morning hamzy13:53
dtantsurTheJulia: do I get it right that https://github.com/openstack/bifrost/tree/master/playbooks/roles/bifrost-create-dib-image is for creating the final user image, not the ramdisk?13:53
dtantsurI mean, there is build_ramdisk, but it uses ramdisk-create, so it doesn't work with IPA?13:53
TheJuliadtantsur: either, it just gets re-used13:53
TheJuliacalled from the upper level playbook if memory serves13:54
TheJuliadtantsur: I believe there is conditional logic being used13:54
dtantsuryeah, but since nobody's noticed that build_ramdisk is broken for.. years.. I guess it's fine to just kill/rewrite it?13:54
TheJuliadtantsur: fine with me13:54
TheJuliadib side of it hsould be fine13:54
dtantsurspecifically, this bit https://github.com/openstack/bifrost/blob/master/playbooks/roles/bifrost-create-dib-image/tasks/main.yml#L141-L14413:54
dtantsurTheJulia: the conditional logic is what surprises me. Why do we build either the user image or the ramdisk? won't people want both?13:55
dtantsurand they certainly won't work with the same arguments..13:55
TheJuliadtantsur: it gets called twice13:56
dtantsuroh13:56
TheJuliaif you want a ramdisk here you go, if you want a dib image here you go as well13:56
openstackgerritMerged openstack/ironic stable/rocky: Change MTU logic to allow for lower MTUs automatically  https://review.opendev.org/69052113:56
openstackgerritRiccardo Pittau proposed openstack/ironic master: [WIP] Using loop instead of with_X  https://review.opendev.org/69053714:03
*** yolanda has quit IRC14:07
*** andyzon has quit IRC14:10
openstackgerritDmitry Tantsur proposed openstack/ironic-python-agent-builder master: CLI: add --verbose and --extra-args  https://review.opendev.org/69058014:10
*** belmoreira has joined #openstack-ironic14:11
*** andyzon has joined #openstack-ironic14:11
*** andyzon has quit IRC14:11
*** andyzon has joined #openstack-ironic14:11
*** yolanda has joined #openstack-ironic14:14
*** andyzon has quit IRC14:16
TheJuliahttps://etherpad.openstack.org/p/PVG-Ironic-Planning <-- Since we are likely to have people who have not found the etherpad join us, we will be having an onboarding session and I'm leaving the topics open for addition. If anyone has anything they would like to see discussed, please add it to the list sooner rather than later.14:20
*** diablo_rojo has joined #openstack-ironic14:21
*** priteau has joined #openstack-ironic14:21
*** andyzon has joined #openstack-ironic14:23
*** cxc99 has quit IRC14:24
openstackgerritPranjali Srivastava proposed openstack/ironic master: Improve iDrac Documentation  https://review.opendev.org/66898714:30
openstackgerritDmitry Tantsur proposed openstack/ironic-python-agent master: WIP prevent rescue on unexpected provision state  https://review.opendev.org/69058414:30
dtantsurTheJulia: wdyt ^^^14:30
TheJuliadtantsur: for rescue, but I was thinking in in IPA's API14:31
TheJuliaso if it doesn't know about itself at all, to raise RESTError14:31
dtantsuryou mean if node is None?14:31
TheJuliait has no way... afaik, to be told it's node, so it has to look it up, or eventually learn how to be told OOB14:32
TheJuliadtantsur: if you do agent.get_node_something, it will rasie an exeption or return the current node14:32
dtantsurTheJulia: the cached node is retrieved on lookup14:32
TheJuliaI know it is retreived on lookup, if lookup is failing, it prevents someone from doing something mean like telling the system to clean14:33
TheJuliaor worse, addresses suddenly changing on a subnet with many nodes and bad things happening14:33
dtantsurso, you mean, prevent any operations until lookup succeeds?14:33
TheJuliabasically yes14:33
dtantsurI wonder if the service actually starts until lookup is done14:33
dtantsuror at least we can prevent that14:33
TheJuliaand for flipping comm flow, we can just provide that means of the get_node_infowhateveritiscalled in agent.py to also look in the locally loaded data from a floppy or configuration14:34
dtantsurI think my fix is useful as well. it prevents any way to enter rescue if it's not enabled in ironic (e.g. by talking to IPA on a flat network)14:34
dtantsurTheJulia: it seems that we start API after lookup succeeds14:35
openstackgerritMerged openstack/ironic-inspector master: Switch to Ussri job  https://review.opendev.org/68978814:35
TheJuliadtantsur: I believe heartbeater independently tries to perform lookup and then the agent is started on the main thread14:35
TheJuliaI may be wrong on that though14:35
dtantsurI don't think heartbeater is related to lookup14:35
TheJuliait is, heartbeater can't work without lookup succeeding14:36
dtantsurTheJulia: https://github.com/openstack/ironic-python-agent/blob/master/ironic_python_agent/agent.py#L398-L42914:36
dtantsurso yeah, everything starts after lookup. if lookup fails, nothing will work IIUC.14:36
*** cxc99 has joined #openstack-ironic14:36
TheJuliaoh, i see what it is14:37
TheJuliaif it doesn't know the url, it still starts the API14:37
dtantsuryep, although not sure why14:38
TheJuliawhich means if the lookup fails, node should always be known14:38
TheJuliaerr14:39
TheJuliaif it does not fail, it should always be known14:39
dtantsuryep14:39
TheJuliain the typical code path14:39
TheJuliaI thinking somehting about if heartbeating was working. Maybe it was a session cookie idea14:40
* TheJulia wonders what the use case is for the API just sitting idle without knowing an API endpoint14:40
* TheJulia wonders if this was logic from rackspace's pre-started machine pool14:41
dtantsurTheJulia: the only thing I can come up with is potential inspection extensions14:41
*** mkrai has joined #openstack-ironic14:43
TheJuliaso a general API guard would still be good then, and if we do that as a helper we could add additional validations down the road14:43
TheJulia"I don't know about you and your command, Try again later!"14:43
dtantsurwell, unless we want to make such extensions possible14:43
dtantsur(in this case we can just avoid starting the API)14:43
*** Garyx_ has quit IRC14:44
TheJuliaI think it is still reasonable for the node to know about itself in such cases....14:44
*** Garyx_ has joined #openstack-ironic14:44
dtantsurthere is no way to ask ironic-inspector though (yet?)14:44
dtantsurbut this is pure theory, I don't think anybody has such extensions14:44
TheJuliaAvoiding starting is actually more code and there is the risk the API may not be up before the first callback occurs14:44
dtantsurit seems that it's just one conditional in the code I linked above14:45
TheJuliahence why I was thinking something very similar to what you wrote14:45
TheJuliano, its actually more because you have to loop and wait, the start code is what presently loops14:45
TheJuliaand it hangs on processing one request14:45
dtantsurokay, lemme try something14:46
openstackgerritDmitry Tantsur proposed openstack/ironic-python-agent master: WIP enforce a valid cached node before we accept commands  https://review.opendev.org/69058414:53
dtantsurTheJulia: smth like ^^^?14:53
*** andyzon has quit IRC14:54
TheJuliaarne_wiebalck: proposed a block of whitepaper text, please take a look it was off the top of my head14:55
TheJuliadtantsur: I don't think the node is aware of it's own state14:55
TheJuliaIt is not in the fields provided14:55
TheJuliaat least from lookup14:56
dtantsurTheJulia: you mean, we don't include it in lookup?14:56
TheJuliano we don't14:56
dtantsuryeah, damn14:56
TheJuliaand we shouldn't include it in lookup because that provides insight into the state14:56
dtantsurcould be so handy14:56
TheJuliai know14:56
TheJulia\o/ for hard problems14:56
dtantsurwell, insight into the state is exactly what we could use14:56
TheJuliaWhy do you say that?14:57
dtantsurto be able to do things like this fix14:57
*** pcaruana has quit IRC14:57
TheJuliabut I'm not sure we need to be precisely state aware to fix this14:58
dtantsurwell, it would be very handy and barely gives away any useful information to a potential attacker14:58
TheJuliaI could see this code last night when I went to bed, I just need to sit down and type it out14:58
openstackgerritDmitry Tantsur proposed openstack/ironic-python-agent master: WIP enforce a valid cached node before we accept commands  https://review.opendev.org/69058414:59
dtantsurokay, trimmed down version ^14:59
dtantsurwe can invent something similar, but if we had a provision state, it would be backportable14:59
dtantsurwhile a new flag probably won't14:59
arne_wiebalckTheJulia: thanks, I will!15:00
TheJuliaflag on the base class?15:00
dtantsurTheJulia: some lookup variable or anything to shut off rescue15:00
TheJuliayeah, thats not very very backportable :\15:00
TheJulias/very very/very/15:01
* TheJulia goes and jumps in the shower15:01
*** TxGirlGeek has joined #openstack-ironic15:03
*** mkrai has quit IRC15:04
arne_wiebalckFTR: we needed to move the Baremetal white WIP paper to a new location: https://docs.google.com/document/d/1BmB2JL_oG3lWXId_NXT9KWcBJjqgtnbmixIcNsfGooA/edit?usp=sharing15:04
iurygregorymagic =D {0} ironic_tempest_plugin.tests.scenario.ironic_standalone.test_basic_ops.BaremetalIpmiAnsibleWholedisk.test_ip_access_to_server [415.487521s] ... ok15:05
iurygregoryrpittau++15:06
iurygregoryTheJulia, dtantsur the test worked locally with rpittau patch =D15:06
* rpittau bow with his (red) wizard hat15:06
arne_wiebalckVolunteers to help with the white paper are of course still very welcome :)15:07
iurygregoryhttps://review.opendev.org/#/c/690537/15:07
patchbotpatch 690537 - ironic - [WIP] Using loop instead of with_X - 3 patch sets15:07
openstackgerritRiccardo Pittau proposed openstack/ironic master: Using loop instead of with_X  https://review.opendev.org/69053715:10
*** tssurya has quit IRC15:10
openstackgerritDmitry Tantsur proposed openstack/ironic-inspector master: [WIP] Create a job with boot and networking managed by ironic  https://review.opendev.org/68892015:10
dtantsurthis ^^ is painfully close :)15:12
iurygregoryyou can do it dtantsur o/15:12
openstackgerritkhansa proposed openstack/sushy master:  Story: 2006610  Task: 36779  https://review.opendev.org/69060415:13
openstackgerritRiccardo Pittau proposed openstack/ironic master: Using loop instead of with_X  https://review.opendev.org/69053715:13
openstackgerritIury Gregory Melo Ferreira proposed openstack/ironic master: Change integration jobs to run under Python3  https://review.opendev.org/68752115:19
iurygregorybye everyone o/15:20
openstackgerritDmitry Tantsur proposed openstack/ironic stable/train: CI: limit rescue testing to only two jobs  https://review.opendev.org/69060515:22
*** belmoreira has quit IRC15:24
*** mkrai has joined #openstack-ironic15:25
*** iurygregory has quit IRC15:26
*** gyee has joined #openstack-ironic15:26
*** andyzon has joined #openstack-ironic15:30
openstackgerritkhansa proposed openstack/sushy master: Story: 2006610 Task: 36779 Change-Id: Icf22b9d38e307a07f59112a6c462d48a8ec6c3d6  https://review.opendev.org/68983515:37
TheJuliakhansa: I left some feedback on your change set. When you have some time, please go ahead and revise it. We also have some guidelines and examples someplace in the new contributor documentation for OpenStack that talks about commit messages and general things to do and avoid doing. If you've not run across it, I can take a look for you15:41
*** e0ne has quit IRC15:44
*** goldyfruit has quit IRC15:46
dtantsurfolks, I'm out tomorrow and on Friday, so see you on Monday15:48
*** dtantsur is now known as dtantsur|afk15:48
rpittaudtantsur: enjoy! o/15:48
dtantsur|afkI'm unlikely to enjoy ordering a kitchen for the new flat :D15:48
rpittaudtantsur|afk: well think about all the great meals you'll cook in there :D15:49
dtantsur|afkindeed15:49
*** igordc has joined #openstack-ironic15:51
TheJuliaeek... enjoy!15:51
TheJuliawell, try to!15:51
khansaTheJulia yes sorry, thanks a lot for your feedback. that patch was by mistake. i already abandon it15:56
TheJuliakhansa: you don't have to abandon, you can revise15:56
TheJuliagit review -d #### ; git commit --amend ; git review16:01
khansayes! i will do it in the corresponding patch. thanks16:01
khansayes yes that's what i did :D16:01
TheJulia\o/16:02
*** lucasagomes has quit IRC16:02
*** bobmel has joined #openstack-ironic16:09
*** andyzon has quit IRC16:10
*** bobmel has quit IRC16:11
*** mkrai has quit IRC16:12
*** rloo has joined #openstack-ironic16:16
openstackgerritDmitry Tantsur proposed openstack/ironic master: CI: add ironic-python-agent-builder to the multinode job  https://review.opendev.org/69063416:19
openstackgerritDmitry Tantsur proposed openstack/ironic-tempest-plugin master: Make the multinode job non-voting  https://review.opendev.org/69063516:21
*** andyzon has joined #openstack-ironic16:28
*** jtomasek has quit IRC16:28
*** andyzon has quit IRC16:32
openstackgerritkhansa proposed openstack/sushy master: Story: 2006610 Task: 36779 Change-Id: Icf22b9d38e307a07f59112a6c462d48a8ec6c3d6 Documentation about using OEM Dell EMC extension with sushy it contains a simple script example  https://review.opendev.org/68983516:34
TheJuliakhansa: try making the first line the subject, the second line an empty line, then the tags16:40
*** bobmel has joined #openstack-ironic16:43
khansaTheJulia thanks for feedback16:46
*** andyzon has joined #openstack-ironic16:46
*** igordc has quit IRC16:50
rpittaubye, good night! o/16:50
*** rpittau is now known as rpittau|afk16:50
*** bobmel has quit IRC16:50
botelloTheJulia: I was not able to see if you wrote something about the inspector issue that I'm facing, because the new messages deleted the older ones.16:51
*** andyzon has quit IRC16:54
*** dougsz has quit IRC16:56
openstackgerritkhansa proposed openstack/sushy master: Using OEM extension Documentation  https://review.opendev.org/68983516:58
*** hjensas|afk is now known as hjensas17:00
openstackgerritMerged openstack/ironic master: CI: limit rescue testing to only two jobs  https://review.opendev.org/69010417:06
TheJuliabotello:  hmm, you sent me more messages after I had to step away last night, reading those17:20
TheJuliabotello: okay, your using grub. That was not expected17:21
TheJuliabotello: sorry, got interrupted by contractor at my door, i'll be a little bit17:34
botelloTheJulia: thank you!17:43
NobodyCamGood Morning Ironic'ers17:49
openstackgerritkhansa proposed openstack/sushy master: Using OEM extension Documentation  https://review.opendev.org/68983517:51
*** whoami-rajat has joined #openstack-ironic17:52
*** ricolin has quit IRC17:55
*** bobmel has joined #openstack-ironic17:57
TheJuliabotello: so nodes don't know when to really go to ironic-inspector in that case. The conundrum is with your grub configuration in that it is attempting to find a specific grub configuration and not finding it. Ironic will write that configuration for deployments. Unlike ironic, ironic-inspector has no idea where the client is coming from, so your base template for inspection simply fails inspection then18:00
TheJuliabecause the template needs to just tell the ramdisk to boot and be pointed to ironic-inspector.18:00
TheJuliabotello: I think the confusion is because your using grub18:00
TheJuliabotello: if you look at https://docs.openstack.org/ironic-inspector/latest/install/index.html#configuring-pxe which only covers PXE use pxelinux and ipxe (likely, the ipxe example ism ore applicable), you can see that we define a kernel and ramdisk in the default config file and instruct it to boot18:03
*** ricolin has joined #openstack-ironic18:03
TheJuliabotello: what you can _also_ do, is add an ipa-api-url parameter to the kernel command line which will allow agents that fall into the inspection dhcp response to find the ironic-api endpoint as well to check-in with.18:04
TheJuliao/ ricolin18:04
ricolino/18:04
botelloTheJulia: Yes, thing is, I am trying to run inspector using UEFI systems, that is the reason why I used grub.18:06
botelloTheJulia: I have two different grubs, one for ironic and one for ironic-inspector18:07
TheJuliathere is an ipxe.efi binary, it is just not signed, although MSFT has publically stated they will sign ipxe binaries for secure boot purposes on request18:07
botello@the18:09
TheJuliathat is neither here nor there though, it seems like your inspector grub configuration needs to not be based upon ironic's default grub.conf template18:09
*** bobmel has quit IRC18:09
*** bobmel has joined #openstack-ironic18:10
botelloTheJulia: /tftpboot/EFI/centos/grub.cfg for ironic and /inspector/EFI/centos/grub.cfg for inspector18:10
TheJuliaand what do you have in your inspector grub.cfg?18:10
botelloTheJulia: linuxefi deploy-fedora.kernel ipa-inspection-callback-url=http://10.219.128.64:5050/v1/continue systemd.journald.forward_to_console=yes console=ttyS018:11
TheJuliaand ultimately filtering is, or is not working using dnsmasq for inspection? Seems like it is not based upon your comments from yesterday18:12
botelloTheJulia: As far as I understand, it is working fine, if the node is sent to introspection, the IP is offered by dnsmasq. And if the node is sent to provide, IP is offered by Neutron.18:15
botelloHowever, the grub of inspection is the one that is always sent for both situations, (introspection and provide).18:16
TheJuliathen that sounds perfect18:16
TheJuliaoh18:16
*** hjensas has quit IRC18:16
TheJuliabotello: ironic.conf pxe section, what does that define for the template to use?18:17
*** haleyb has joined #openstack-ironic18:17
*** haleyb has left #openstack-ironic18:17
*** tzumainn has quit IRC18:17
botelloTheJulia: uefi_pxe_config_template = $pybasedir/drivers/modules/pxe_grub_config.template18:19
*** khansa has quit IRC18:19
botelloTheJulia: tftp_root = /tftpboot18:19
TheJuliaare you using different tftp servers or the same IP address?18:20
botelloHmmm, I think I am using the same, cause ironic and inspector services are running on the same machine18:21
TheJuliaokay18:21
*** igordc has joined #openstack-ironic18:24
*** ricolin has quit IRC18:25
TheJuliabotello: so /tftpboot/inspector is a unique folder right?18:27
TheJuliabotello: and /tftpboot/EFI/centos/grub.conf is intended for ironic correct?18:27
TheJuliaso the conundrum that kind of comes to mind is I think grub does tftp://server_ip/EFI/centos/grub.conf based on build parameters by default18:28
*** igordc has quit IRC18:32
*** pcaruana has joined #openstack-ironic18:38
*** bobmel has quit IRC18:40
*** TxGirlGeek has quit IRC18:44
*** pcaruana has quit IRC18:44
openstackgerritIlya Etingof proposed openstack/sushy-tools master: Fix reading/setting libvirt domain boot device  https://review.opendev.org/68618318:48
etingoflibvirt has a dementing capability18:49
*** TxGirlGeek has joined #openstack-ironic18:58
*** jcoufal_ has joined #openstack-ironic18:59
*** jcoufal has quit IRC19:01
*** TxGirlGeek has quit IRC19:03
TheJuliaetingof: like making humans demented?19:19
*** ociuhandu has joined #openstack-ironic19:30
*** ociuhandu has quit IRC19:35
*** priteau has quit IRC19:40
*** ociuhandu has joined #openstack-ironic19:45
*** igordc has joined #openstack-ironic19:47
*** ociuhandu has quit IRC19:50
botelloTheJulia: /tftpboot/EFI/centos/ is intended for ironic, and /inspector/EFI/centos/ is intended for inspector19:50
etingofnot sure about humans, but programmers19:54
etingofapparently, libvirt boot device change is not reflected in the libvirt config while the domain is running19:55
etingofhowever, it will flip to the right boot device upon reboot19:56
etingofthat un-RESTs redfish emulator19:56
etingofthis is something they have in common with idrac19:58
TheJuliaetingof: gah :(20:03
*** e0ne has joined #openstack-ironic20:12
*** alexmcleod has quit IRC20:13
*** botello has quit IRC20:17
*** gonzalez has joined #openstack-ironic20:18
*** botello has joined #openstack-ironic20:19
*** hjensas has joined #openstack-ironic20:20
gonzalezHi! I'm working with ironic, and I'm having a weird behavior after the first deployment, if I delete the instance from the node, the next boot is being from the local drive instead of PXE therefore the clean step fails. I have to manually change the boot order again. Do you know if there is an option in ironic to do it automatically ?20:23
TheJuliabotello: I think the logs you pasted indicated a rewrite was occuring, but seems to be going in the wrong direction. Overall if Ironic is pointing to /tftpboot, and inspector to /inspector, it seems a little weird because inspector should be chrooting into /tftpboot in general. I wonder if there is anyway to structure it all under one main folder, /tftpboot/inspector/etc and /tftpboot/ for standard ironic20:32
TheJuliaboot path20:32
TheJuliagonzalez: ipmi hardware type?20:33
*** e0ne has quit IRC20:43
*** e0ne has joined #openstack-ironic20:45
*** gonzalez has quit IRC20:49
*** gonzalez has joined #openstack-ironic20:49
*** jcoufal_ has quit IRC20:55
*** gonzalez has quit IRC20:55
*** e0ne has quit IRC21:08
*** botello has quit IRC21:09
*** henriqueof has quit IRC21:15
*** gonzalez has joined #openstack-ironic21:23
*** gonzalez has quit IRC21:32
*** gonzalez has joined #openstack-ironic21:32
gonzalezTheJulia : yes, I'm using IPMI21:33
TheJuliathen we have a setting you should try!21:33
*** botello has joined #openstack-ironic21:34
TheJuliagonzalez: try ipmi_force_boot_device=True for the driver_info21:34
botelloTheJulia: I already tried that, to point ironic to /tftpboot and pointing inspector to /tftpboot/inspector, unfortunately the behavior is the same :(21:37
*** goldyfruit has joined #openstack-ironic21:38
*** rh-jelabarre has quit IRC21:42
openstackgerritkhansa proposed openstack/sushy master: Using OEM extension Documentation  https://review.opendev.org/68983521:42
*** khansa has joined #openstack-ironic21:44
*** khansa has quit IRC21:45
TheJuliabotello: it doesn't make any sense...21:52
*** whoami-rajat has quit IRC22:01
*** zaneb has quit IRC22:02
*** goldyfruit has quit IRC22:04
*** zaneb has joined #openstack-ironic22:17
*** gonzalez has quit IRC22:28
*** rcernin has joined #openstack-ironic22:33
openstackgerritMerged openstack/ironic-python-agent-builder master: Update upper-constraints handling  https://review.opendev.org/68891122:36
*** diablo_rojo has quit IRC22:44
*** igordc has quit IRC23:23
openstackgerritMerged openstack/ironic stable/rocky: Ensure pagination marker is always set  https://review.opendev.org/69011723:46
openstackgerritMerged openstack/ironic stable/rocky: Do not ignore 'fields' query parameter when building next url  https://review.opendev.org/69011823:57
*** bfournie has quit IRC23:57
« Tuesday, 2019-10-22 Index Thursday, 2019-10-24 »

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