Friday, 2020-05-22

* NobodyCam Waves hi to all the Ironic'ers out there00:04
*** cp- has quit IRC00:25
*** cp- has joined #openstack-ironic00:26
*** Goneri has quit IRC00:42
*** michchap has quit IRC00:43
openstackgerritVerification of a change to openstack/ironic-python-agent failed: Convert jobs to dib  https://review.opendev.org/71862701:14
*** k_mouza has joined #openstack-ironic01:17
*** k_mouza has quit IRC01:21
*** ociuhandu has joined #openstack-ironic01:30
*** ociuhandu has quit IRC01:35
openstackgerritSteve Baker proposed openstack/ironic-lib master: Implement Basic HTTP authentication middleware  https://review.opendev.org/72907001:55
*** uzumaki has joined #openstack-ironic02:04
*** Qianbiao has joined #openstack-ironic02:06
*** uzumaki has quit IRC02:55
*** uzumaki has joined #openstack-ironic02:56
openstackgerritzhangboye proposed openstack/ironic-inspector master: Fix hacking min version to 3.0.1  https://review.opendev.org/73017703:13
*** uzumaki has quit IRC04:04
*** sshnaidm is now known as sshnaidm|off04:33
*** mkrai has joined #openstack-ironic05:04
*** k_mouza has joined #openstack-ironic05:15
*** k_mouza has quit IRC05:19
*** mkrai has quit IRC05:26
*** uzumaki has joined #openstack-ironic05:54
*** Qianbiao has quit IRC05:56
*** ociuhandu has joined #openstack-ironic06:01
*** ociuhandu has quit IRC06:05
maelkHi there! I'm working on the Metal3 project, and following our upgrade to master (current-tripleo), we tried to re-enable fast-track. We are still seeing the error `failed deploy step {'step': 'deploy', 'priority': 100, 'argsinfo': None, 'interface': 'deploy'}. Error: Agent driver requires agent_url in driver_internal_info: ironic_lib.exception.Iro06:15
maelknicException: Agent driver requires agent_url in driver_internal_info` . We did not try to use fast track until now because we needed to upgrade Ironic. However, this is now seen with the packages from master. Could it be that the bug is still there somehow ?06:15
*** uzumaki has quit IRC06:16
*** ociuhandu has joined #openstack-ironic06:16
arne_wiebalckGood morning, ironic!06:26
*** Qianbiao has joined #openstack-ironic06:51
iurygregorygood morning arne_wiebalck and Ironic o/07:05
arne_wiebalckhey iurygregory o/07:05
Qianbiaohello ironic07:13
Qianbiaohello iurygregory arne_wiebalck o/07:13
iurygregoryQianbiao, good morning o/07:15
iurygregoryarne_wiebalck, https://review.opendev.org/729746 can you review pls? *-*07:15
patchbotpatch 729746 - ironic-lib - Fix DIB jobs - 3 patch sets07:15
iurygregoryyou will make our CI happy07:16
Qianbiaohi iurygregory, need help from you. ;)07:16
Qianbiaoabout the inspector issue i post yestoday. I think it's not caused by enroll_driver (also, i have update the item).07:16
iurygregoryQianbiao, gotcha, let me just grab more coffee o/07:17
QianbiaoCheck the log line 8 and 13 http://paste.openstack.org/show/793869/07:17
Qianbiaosure07:17
iurygregorysure07:17
*** ociuhandu has quit IRC07:19
*** jawad_axd has joined #openstack-ironic07:28
iurygregoryQianbiao, well the you are missing the info for the node07:30
iurygregoryMissing the following iBMC properties in node 9dbda2df-53b2-4f87-aed2-d02450964224 driver_info: ['ibmc_address', 'ibmc_username', 'ibmc_password']07:30
iurygregorywithout this it won't work07:30
Qianbiaoiurygregory that is not the root cause.07:31
Qianbiaothe reason is : do not know why inspector could not the original node.07:31
Qianbiaoline 8, it use mac to find the node.07:32
Qianbiaobut line 13, it use an empty attributes.07:32
iurygregoryoh I see07:32
QianbiaoThen, the final result is: inspector auto create a new node for me. but not update the original node i try to inspect07:33
QianbiaoThe log from IPA is here: http://paste.openstack.org/show/793871/07:34
iurygregoryQianbiao, I assume you passed the credentials when you created the node you wanted to introspect right?07:40
Qianbiaoyes07:41
iurygregorystrange .-.07:41
iurygregorybecause if the credentials are there inspector should find the node without problems07:42
QianbiaoI do not know what happen between line8 and line1307:42
iurygregoryyou have 2 ports in that node that can do pxe?07:43
Qianbiaoi think the why attributes disappear is key point.07:43
Qianbiaoonly one port can pxe.07:43
iurygregoryok cool07:43
Qianbiaothe other one is not used.07:43
iurygregoryI'm running out of ideas D:07:44
Qianbiao:( i have been caught by this for whole day.07:45
openstackgerritMerged openstack/tenks master: Switch to newer openstackdocstheme and reno versions  https://review.opendev.org/73009708:03
*** tkajinam_ has quit IRC08:05
Qianbiaooh iurygregory08:17
Qianbiaoi think i fingure out why.08:17
iurygregory\o/08:17
Qianbiaois there an IPA and inspector compatible list?08:18
iurygregorywell in theory you should use the ipa from the version of ironic08:18
*** ociuhandu has joined #openstack-ironic08:22
openstackgerritRiccardo Pittau proposed openstack/ironic-inspector master: Update lower-constraints.txt  https://review.opendev.org/72723708:22
iurygregorya wild rpittau|afk appeared08:23
*** ociuhandu has quit IRC08:28
*** ociuhandu has joined #openstack-ironic08:28
Qianbiaohi iurygregory, i find the code is never changed.08:29
Qianbiaohttps://opendev.org/openstack/ironic-inspector/src/branch/master/ironic_inspector/process.py#L8508:29
*** ociuhandu has quit IRC08:29
Qianbiaoit try to get mac from "all_interface"08:30
*** ociuhandu has joined #openstack-ironic08:30
Qianbiaobut in fact it should be interfaces.08:30
iurygregorymaybe there is a config that you can change not sure08:34
iurygregoryneed to run for a meeting =(08:34
*** ociuhandu has quit IRC08:35
*** ociuhandu has joined #openstack-ironic08:37
rpittau|afkiurygregory: it's supper effective!08:38
*** derekh has joined #openstack-ironic08:39
openstackgerritMerged openstack/bifrost master: Install python3-pip system package  https://review.opendev.org/72924908:44
openstackgerritMerged openstack/bifrost master: Update docs references for CentOS 8  https://review.opendev.org/72927308:44
openstackgerritMerged openstack/bifrost stable/ussuri: Switch git_branch to stable/ussuri  https://review.opendev.org/72930208:44
*** yaawang has quit IRC08:57
*** dougsz has joined #openstack-ironic08:57
*** yaawang has joined #openstack-ironic08:57
openstackgerritMerged openstack/ironic-ui master: Switch to newer openstackdocstheme and reno versions  https://review.opendev.org/72952609:01
*** hjensas|afk is now known as hjensas09:13
openstackgerritVerification of a change to openstack/virtualbmc failed: Update lower-constraints.txt  https://review.opendev.org/72724909:32
*** k_mouza has joined #openstack-ironic09:35
openstackgerritVerification of a change to openstack/python-ironicclient failed: Update lower-constraints.txt  https://review.opendev.org/72724309:47
openstackgerritMerged openstack/python-ironic-inspector-client master: Update lower-constraints.txt  https://review.opendev.org/72724509:52
*** derekh has quit IRC10:09
*** derekh has joined #openstack-ironic10:09
*** aedc has joined #openstack-ironic10:23
*** derekh has quit IRC10:33
*** derekh has joined #openstack-ironic10:34
*** derekh has quit IRC10:35
*** derekh has joined #openstack-ironic10:35
openstackgerritMerged openstack/networking-baremetal master: Convert networking-baremetal job to dib  https://review.opendev.org/71869710:43
*** k_mouza has quit IRC10:44
openstackgerritMerged openstack/ironic-lib master: Fix DIB jobs  https://review.opendev.org/72974611:02
*** k_mouza has joined #openstack-ironic11:18
*** yaawang has quit IRC11:20
*** jawad_axd has quit IRC11:21
*** dougsz has quit IRC11:27
*** jawad_axd has joined #openstack-ironic11:37
*** Qianbiao has quit IRC11:39
*** jawad_axd has quit IRC11:42
maelkHello! I have a question regarding raw image streaming. I have set the `instance_info/image_disk_format` to raw, however, during the host validation, ironic still complains that the target host does not have enough RAM for the raw image. Is there anything that needs to be configured to allow raw image streaming (it is enabled in the configuration).11:57
maelk Thank you for the help11:57
*** jawad_axd has joined #openstack-ironic12:00
*** ociuhandu has quit IRC12:05
*** ociuhandu has joined #openstack-ironic12:06
*** ociuhandu has quit IRC12:11
*** michchap has joined #openstack-ironic12:18
maelkI have investigated a bit this raw image streaming. Basically if not using Glance, there is no way to stream the image. However, in Metal3 we can set the image_disk_format to raw, so we could base the decision on this for streaming rather than refusing it because it is an http(s) download. Would that be ok ?12:19
openstackgerritMerged openstack/ironic master: Collect tftpd info from journald  https://review.opendev.org/71774512:21
*** jawad_axd has quit IRC12:24
*** derekh has quit IRC12:24
*** priteau has joined #openstack-ironic12:33
iurygregorymaelk, if you are using ansible driver you can do that stream_url https://github.com/openstack/ironic/blob/bbb9bbb305066ce56c4635ecab3cbef50af8ce9a/doc/source/admin/drivers/ansible.rst#included-custom-ansible-modules12:37
*** rh-jelabarre has joined #openstack-ironic12:39
*** rcernin has quit IRC12:39
maelkiurygregory I don't think we use ansible. we are hitting this : https://opendev.org/openstack/ironic/src/branch/master/ironic/drivers/modules/agent.py#L10012:39
maelkI would like to modify https://opendev.org/openstack/ironic/src/branch/master/ironic/drivers/modules/agent.py#L91 to put an or where we check if the image_disk_format parameter is 'raw' because once we pass this check, the image format is taken into consideration : https://opendev.org/openstack/ironic/src/branch/master/ironic/drivers/modules/agent.12:42
maelkpy#L215 . It's just the check that is blocking us as far as I understood12:42
iurygregoryI would say that using or can cause problems since we can only stream when it's raw12:54
iurygregorymaybe adding a config option that would allow to skip in case of deployments that is not using glance12:54
maelkThe condition would be that disk format is raw12:54
maelkI am putting what I mean in patchset12:55
iurygregorycool =)12:55
iurygregoryplease also create a story if you haven't =)12:55
iurygregoryhttps://storyboard.openstack.org/#!/project/openstack/ironic12:55
*** Qianbiao has joined #openstack-ironic12:56
*** jdandrea has joined #openstack-ironic12:56
openstackgerritMaĆ«l Kimmerlin proposed openstack/ironic master: Do not fail when raw image is larger than memory for http  https://review.opendev.org/73031712:57
iurygregoryoh I see12:58
iurygregoryI thought you want to change the and to or lol12:58
maelkdoes that seem acceptable for you ?12:59
iurygregorymaybe you don't even need to add a new parameter to the check_image_size13:01
iurygregoryif we can figure out that glance is not in use and you are ussing http for the image13:01
iurygregoryyou can add a condition to skip check_image_size in validate13:02
iurygregorythis would be another approach13:02
iurygregoryboth seems good to me13:02
iurygregorylet's see what the community thinks about it =)13:02
maelkit might be more complicated, since the glance check is part of the check_memory_size  function. We would need to do it earlier13:05
*** ociuhandu has joined #openstack-ironic13:08
*** tzumainn has joined #openstack-ironic13:11
iurygregorymaelk, can you create a story for this?13:11
maelkdone : https://storyboard.openstack.org/#!/story/200770613:16
iurygregorycool13:16
maelkI am a bit unsure how to test the changes in the validate function though13:17
iurygregoryno worries13:17
iurygregorythis probably needs an ironic-triaged tag13:17
iurygregoryso on monday at the upstream meeting I will bring that up if you are not here13:18
maelkthat would be great, thank you!13:18
openstackgerritMerged openstack/metalsmith master: Switch to newer openstackdocstheme and reno versions  https://review.opendev.org/73009013:18
iurygregoryyw13:18
*** rloo has joined #openstack-ironic13:23
*** cdearborn has joined #openstack-ironic13:24
*** Goneri has joined #openstack-ironic13:39
*** Qianbiao has quit IRC13:41
*** Qianbiao has joined #openstack-ironic13:53
*** jawad_axd has joined #openstack-ironic13:57
*** priteau has quit IRC13:59
*** jawad_axd has quit IRC14:02
*** jawad_axd has joined #openstack-ironic14:19
*** jawad_axd has quit IRC14:23
Qianbiaohello mealk, i am ibmc maintainer :)14:32
QianbiaoYou can directly @ me here also.14:32
Qianbiaos/mealk/maelk/14:32
Qianbiaomaelk14:32
*** jawad_axd has joined #openstack-ironic14:39
*** jawad_axd has quit IRC14:44
NobodyCamGood Morning Ironic'ers14:47
iurygregorygood morning NobodyCam o/14:50
iurygregoryhappy Friday14:50
TheJuliaGood morning NobodyCam14:58
TheJuliayour... up... early :)14:58
*** jawad_axd has joined #openstack-ironic15:00
*** jawad_axd has quit IRC15:05
*** uzumaki has joined #openstack-ironic15:12
NobodyCamGood Morning iurygregory and TheJulia :)15:15
NobodyCamyea15:15
iurygregorygood morning TheJulia o/15:15
iurygregoryit's Friday no one should wake up early D:15:15
NobodyCamooh but it a long weekend15:16
iurygregoryhehehe15:16
*** gyee has joined #openstack-ironic15:17
TheJuliaiurygregory: ++15:29
TheJuliaNobodyCam: ++15:29
TheJuliaI've got estate stuff to do... Been on hold for 45 minutes so far15:30
iurygregoryouch =(15:30
TheJuliaon a plus side, it involves closing out the estate... but... blarg15:32
iurygregorynews about the car? =)15:33
TheJuliawhich one?15:35
TheJuliaI did put a cold air intake on my fiesta last night15:36
TheJuliathat was FUN15:36
JayFis that sarcasm?15:36
iurygregoryI was going to ask about the fiesta =)15:37
iurygregoryJayF, I think it was only a pun =)15:38
*** ociuhandu has quit IRC15:39
*** ociuhandu has joined #openstack-ironic15:41
NobodyCamhehehehe15:41
NobodyCamFiesta's are FUN!15:42
iurygregoryit's a nice car15:42
NobodyCamTheJulia: Was the mishimoto carb approved?15:43
* iurygregory totally miss driving.15:43
NobodyCamiurygregory: miss Driving?15:44
iurygregoryNobodyCam, I don't have a car anymore =)15:44
iurygregoryI had before I moved to CZ15:45
NobodyCam:'(15:45
JayFI got a new truck about a month before quarantine started. It's the most useless thing ever, to have a nice new truck and nowhere to go :|15:45
iurygregoryand ofc I can't use my driver license from Brazil in Europe =(15:45
NobodyCamuggh,15:45
iurygregoryJayF, ouch D:15:46
*** ociuhandu has quit IRC15:46
openstackgerritTzu-Mainn Chen proposed openstack/ironic master: Allow node lessee to see node's ports  https://review.opendev.org/73036615:47
*** k_mouza has quit IRC16:03
*** k_mouza has joined #openstack-ironic16:04
*** Qianbiao has quit IRC16:31
iurygregoryhumm not bad 63 tests to change in ironic_lib to use privsep16:44
*** ociuhandu has joined #openstack-ironic16:53
*** ociuhandu has quit IRC16:57
iurygregoryIII SPUC17:00
*** ociuhandu has joined #openstack-ironic17:21
*** uzumaki has quit IRC17:42
*** ociuhandu has quit IRC17:55
*** ociuhandu has joined #openstack-ironic17:56
*** ociuhandu has quit IRC18:01
TheJuliaNobodyCam: yes!18:21
*** ociuhandu has joined #openstack-ironic18:47
iurygregoryJayF, the commercial is very funny XD18:50
*** Lucas_Gray has joined #openstack-ironic18:56
JayF:D19:01
*** ociuhandu has quit IRC19:18
*** ociuhandu has joined #openstack-ironic19:19
*** ociuhandu has quit IRC19:24
*** dougsz has joined #openstack-ironic19:41
*** k_mouza has quit IRC19:45
*** Lucas_Gray has quit IRC19:47
*** Lucas_Gray has joined #openstack-ironic19:49
*** Lucas_Gray has quit IRC19:56
*** k_mouza has joined #openstack-ironic20:02
*** ociuhandu has joined #openstack-ironic20:03
*** ociuhandu has quit IRC20:15
*** stbenjam has quit IRC20:23
*** k_mouza has quit IRC20:26
*** dougsz has quit IRC20:30
*** rh-jelabarre has quit IRC20:33
*** bnemec is now known as beekneemech21:13
*** dking has quit IRC21:18
*** uzumaki has joined #openstack-ironic21:24
*** uzumaki has quit IRC21:28
*** rloo has quit IRC22:08
*** ccstone9 has joined #openstack-ironic22:23
*** ccstone has quit IRC22:24
*** ccstone9 is now known as ccstone22:24
*** Lucas_Gray has joined #openstack-ironic22:25
*** k_mouza has joined #openstack-ironic22:27
*** k_mouza has quit IRC22:32
*** ociuhandu has joined #openstack-ironic23:04
*** ociuhandu has quit IRC23:08

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