Friday, 2020-08-21

*** ildikov has joined #openstack-ironic00:01
*** pas-ha has joined #openstack-ironic00:02
*** sri_ has joined #openstack-ironic00:02
*** rpioso has quit IRC00:09
*** rpioso has joined #openstack-ironic00:11
*** sri_ has quit IRC00:16
*** sri_ has joined #openstack-ironic00:18
*** rpioso has quit IRC00:18
*** rpioso has joined #openstack-ironic00:19
*** rloo has quit IRC00:37
*** Goneri has joined #openstack-ironic01:01
*** cdearborn has quit IRC01:03
*** zbitter is now known as zaneb01:06
*** Goneri has quit IRC01:14
*** ijw has quit IRC01:45
*** rh-jelabarre has quit IRC01:54
*** gyee has quit IRC01:55
*** Qianbiao has joined #openstack-ironic02:33
*** rcernin has quit IRC02:37
*** mkrai has joined #openstack-ironic02:54
*** rcernin has joined #openstack-ironic02:58
*** rcernin has quit IRC03:04
*** rcernin has joined #openstack-ironic03:04
*** tzumainn has quit IRC03:18
*** Qianbiao has quit IRC04:11
*** hjensas has joined #openstack-ironic04:39
*** mkrai has quit IRC05:22
*** janders has quit IRC05:30
*** janders has joined #openstack-ironic05:30
*** Qianbiao has joined #openstack-ironic05:32
*** Qianbiao has joined #openstack-ironic05:32
*** mkrai has joined #openstack-ironic05:32
*** yolanda has quit IRC06:38
*** weshay|ruck has quit IRC06:38
*** bdodd has quit IRC06:38
*** akahat|rover has quit IRC06:38
*** ddurst has quit IRC06:38
*** rnoriega_ has quit IRC06:38
*** yolanda has joined #openstack-ironic06:43
*** weshay|ruck has joined #openstack-ironic06:43
*** bdodd has joined #openstack-ironic06:43
*** akahat|rover has joined #openstack-ironic06:43
*** ddurst has joined #openstack-ironic06:43
*** rnoriega_ has joined #openstack-ironic06:43
*** ociuhandu has quit IRC06:56
*** ociuhandu has joined #openstack-ironic06:57
*** belmoreira has joined #openstack-ironic06:57
*** jtomasek has joined #openstack-ironic06:59
*** ricolin has quit IRC06:59
*** ociuhandu has quit IRC07:01
*** mkrai has quit IRC07:03
iurygregorygood morning Ironic07:08
iurygregoryhappy friday07:08
*** rcernin has quit IRC07:22
*** Lucas_Gray has joined #openstack-ironic07:25
*** dtantsur|afk is now known as dtantsur07:31
dtantsurmorning ironic07:31
iurygregorymorning dtantsur07:33
dtantsurCould not find a version that satisfies the requirement bcrypt>=3.1.3 (from ironic-lib>=4.1.0->ironic_python_agent) (from versions: )07:34
dtantsurrpittau: is it something your ipa-builder patches handles ^^?07:34
*** dougsz has joined #openstack-ironic07:36
*** mgoddard has joined #openstack-ironic07:36
dtantsurRAMDISK: gzip image found at block 007:36
dtantsurRAMDISK: EOF while reading compressed data07:37
dtantsurmeh wut07:37
*** priteau has joined #openstack-ironic07:40
iurygregorydtantsur, I was thinking if we don't need some unit test to cover https://review.opendev.org/#/c/747144/07:43
patchbotpatch 747144 - ironic - Fix network_data path for dhcpless deployments - 1 patch set07:43
openstackgerritDmitry Tantsur proposed openstack/ironic-python-agent master: Eventlet should be monkey patched as early as possible  https://review.opendev.org/74677407:44
rpittaudtantsur: re bcrypt yes07:45
dtantsuriurygregory: it changes a constant, dunno. but it's suspicious that no tests had to be updated.07:45
rpittaugood morning ironic! Happy Friday! o/07:45
dtantsurhey-hey rpittau07:45
dtantsurrpittau: and the bcrypt patch is blocked by the ramdisk unpacking problem, sweet07:46
dtantsurany clues on that already?07:46
rpittau\o/07:46
rpittaujust saw it07:46
iurygregorymorning rpittau o/07:46
rpittauback from a painting session :)07:46
iurygregoryI hope you enjoyed =)07:46
rpittaualmost done, tomorrow last day :)07:47
iurygregory\o/07:47
jandersgood morning / evening Ironic o/07:48
rpittauinteresting that everything was working perfectly until 3 days ago07:48
rpittauhey janders :)07:48
iurygregorymorning janders o/07:49
rpittaulet's check some history07:49
*** dougsz has quit IRC07:50
*** Lucas_Gray has quit IRC07:50
*** hjensas has quit IRC07:50
rpittauthe kernel is the same07:50
rpittau"There is insufficient memory available for loading the Initial Ramdisk, preventing Linux from accessing the disk."07:55
rpittaudid the image grow in the last 3 days ?07:55
openstackgerritRiccardo Pittau proposed openstack/ironic-python-agent master: [DNM] Increase memory for tinyipa jobs  https://review.opendev.org/74735507:59
openstackgerritRiccardo Pittau proposed openstack/ironic-python-agent-builder master: Fix finalise tinyipa  https://review.opendev.org/74719808:00
*** mkrai has joined #openstack-ironic08:02
dtantsurrpittau: no idea :(08:02
rpittaulet's see if increasing the size changes something, I don't see changes on tinycore that could've caused that08:03
iurygregoryit's magic08:07
*** rcernin has joined #openstack-ironic08:09
brtknrthanks rpittau08:09
brtknrthanks rpioso*08:09
* rpittau proudly stealing rpioso pings since 201008:09
dtantsur:D08:10
iurygregorylol08:10
*** lucasagomes has joined #openstack-ironic08:10
*** ricolin has joined #openstack-ironic08:12
rpittauanyhow, the image does look a tiny bit bigger than 3 days ago08:12
rpittauoh well, let's wait and see08:13
*** rcernin has quit IRC08:13
dtantsurwell, the image may increase with updated pip packages08:14
dtantsurthat's the only explanation I have08:14
rpittauthat is true, it would be kind of hilarious08:15
iurygregoryanyone with some ideas for https://review.opendev.org/741886 the only jobs failling are the ones with uefi and the multinode one (for multinode I'm thinking in decreassing the tempest concurrency this idea came from the nova folks)08:18
patchbotpatch 741886 - ironic - Add tempest default_boot_option and altflavor - 11 patch sets08:18
*** dougsz has joined #openstack-ironic08:22
*** Lucas_Gray has joined #openstack-ironic08:22
*** hjensas has joined #openstack-ironic08:22
brtknrrpioso: what is the uri for system event logs?08:26
*** k_mouza has joined #openstack-ironic08:37
brtknrrpioso: Oh I found it, its called DCIM_SELRecordLog08:38
brtknrrpioso: No, DCIM_SELLogEntry08:41
*** arne_wiebalck has quit IRC08:44
*** arne_wiebalck has joined #openstack-ironic08:46
janderswrapping up for the week - see you next week Ironic o/08:49
jandershave a great weekend everyone08:49
jandersthanks for all your help this week, much appreciated08:49
rpittaujanders: thanks, you too!08:49
iurygregoryenjoy the weekend janders08:50
dtantsurhave a good weekend janders08:54
*** openstackgerrit has quit IRC08:54
jandersthank you rpittau iurygregory dtantsur, will do!08:56
*** rcernin has joined #openstack-ironic09:07
*** openstackgerrit has joined #openstack-ironic09:26
openstackgerritMerged openstack/bifrost bugfix/8.3: Update .gitreview for bugfix/8.3  https://review.opendev.org/74617809:26
openstackgerritMerged openstack/bifrost bugfix/8.3: Update TOX_CONSTRAINTS_FILE for bugfix/8.3  https://review.opendev.org/74617909:26
*** k_mouza has quit IRC09:27
openstackgerritMerged openstack/bifrost bugfix/8.3: dynamic: always use clouds.yaml when it is available  https://review.opendev.org/74697809:37
dtantsurTheJulia: we may want to release 8.3.1 because of ^^^09:39
*** k_mouza has joined #openstack-ironic09:42
openstackgerritMerged openstack/ironic master: Fix network_data path for dhcpless deployments  https://review.opendev.org/74714409:47
*** ociuhandu has joined #openstack-ironic09:53
*** ociuhandu has quit IRC09:53
*** ociuhandu has joined #openstack-ironic09:54
*** Lucas_Gray has quit IRC09:58
*** Lucas_Gray has joined #openstack-ironic10:03
*** belmoreira has quit IRC10:07
*** rcernin has quit IRC10:13
*** mkrai has quit IRC10:14
*** k_mouza has quit IRC10:31
*** mnasiadka has joined #openstack-ironic10:39
*** k_mouza has joined #openstack-ironic10:42
*** jhesketh has quit IRC10:53
openstackgerritDmitry Tantsur proposed openstack/ironic master: Do not wipe cached deploy/clean steps on fast-track deployment  https://review.opendev.org/74738910:54
dtantsurand this is another candidate for a bugfix release ^^10:54
*** jhesketh has joined #openstack-ironic10:55
*** rcernin has joined #openstack-ironic10:59
mnasiadkahello, just stumbled across an issue using stable/ussuri ironic (bifrost deployed) and ironic-python-agent (stable/ussuri as well), where IPA during deployment has a problem sending heartbeats, due to some agent token issue. I read through https://docs.openstack.org/ironic/latest/admin/agent-token.html - but it lacks a bit of a description what settings affect the behaviour (and how).11:06
mnasiadkaThe error I see in conductor logs is: Heartbeat request has been rejected as the version of ironic-python-agent indicated in the heartbeat operation should support agent token functionality.11:06
mnasiadkaWhen [DEFAULT] require_agent_token is set to False - everything works, but I would rather want to understand how to get it to work :)11:07
*** rcernin has quit IRC11:12
dtantsurmnasiadka: you also need to check the agent logs on the node to see what was happening with the token there (did it receive a token, etc)11:14
*** k_mouza has quit IRC11:34
*** Lucas_Gray has quit IRC11:40
*** rh-jelabarre has joined #openstack-ironic11:59
openstackgerritDmitry Tantsur proposed openstack/ironic master: Ensure in-band deploy steps are present in time for fast-track deployments  https://review.opendev.org/74738912:01
openstackgerritDmitry Tantsur proposed openstack/ironic master: Ensure in-band deploy steps are present in time for fast-track deployments  https://review.opendev.org/74738912:02
bbezakdtantsur: these are the logs from agent. Warning look interesting, the similar was on conductor https://www.irccloud.com/pastebin/nx0MJfuk/ipa%20logs12:02
bbezak2020-08-21 11:42:40.820 26 WARNING ironic.conductor.manager [req-3ff50857-31f3-4813-afff-fdcd20d54983 - - - - -] An agent token generation request is being refused as one is already present for node e3efc74c-11f2-4545-85f5-0d5ae8c94f6112:02
bbezakhello btw :)12:02
dtantsuryep, this is bad12:02
dtantsurand hello :)12:02
dtantsurI wonder if ussuri lacks one of the fixes.. can you confirm you use the latest stable/ussuri?12:03
dtantsur(check /opt/stack/ironic against opendev.org)12:03
bbezakyeah, but I'll double check12:03
bbezakyeah, bifrost has latest ironic stable/ussuri12:04
dtantsurbbezak: what's the exact sequence of actions you attempted?12:05
*** k_mouza has joined #openstack-ironic12:06
openstackgerritDmitry Tantsur proposed openstack/ironic stable/ussuri: Wipe agent token during reboot or power off  https://review.opendev.org/74739412:07
dtantsurbbezak: can you try applying ^^^ and seeing if it helps?12:07
bbezakI was using bifrost playbooks deploy-dynamic.yml via Kayobe to deploy hosts. Ok I will try this fix, thx12:12
*** k_mouza has quit IRC12:21
*** k_mouza has joined #openstack-ironic12:21
*** k_mouza has quit IRC12:26
hjensaswhat's the trick to get DIB rootpwd in pxe_append_params to work? (https://docs.openstack.org/ironic-python-agent/ocata/troubleshooting.html#id8). Do I need to escape '$' chars in the encrypted password?12:27
dtantsurhjensas: definitely12:27
bbezakdtantsur: much better now thank you12:28
hjensasdtantsur: ok, trying with escapes then.12:28
dtantsurbbezak: great. Not sure why I forgot to backport it..12:29
rpittaudtantsur: it was the memory of the image, increasing it worked, I'm going to propose a full patch for that12:31
dtantsurrpittau: bumping to 512?12:31
rpittauyeah, should be enough12:32
dtantsurdon't forget any jobs on other projects that may use tinyipa: ironic, inspector, metalsmith..12:32
rpittauyes, I'm going to check them all, starting with ironic-python-agent that is also broken12:32
dtantsur++ thanks!12:32
rpittaunp12:32
iurygregoryif you change in ironic-base it should just work12:35
dtantsurat least metalsmith does not use ironic-base12:36
rpittauiurygregory: it's for tinyipa only, the ironic-base hase the ram for DIB iamges12:36
iurygregoryoh =)12:36
rpittauwhich is 307212:36
iurygregoryyeah12:36
rpittaubit too much :)12:36
openstackgerritRiccardo Pittau proposed openstack/ironic master: Increase memory of tinyipa vms  https://review.opendev.org/74739712:41
openstackgerritRiccardo Pittau proposed openstack/ironic-python-agent master: Increase memory for tinyipa jobs  https://review.opendev.org/74735512:42
openstackgerritRiccardo Pittau proposed openstack/ironic-python-agent-builder master: Fix finalise tinyipa  https://review.opendev.org/74719812:43
dtantsurrpittau: one small issue in the IPA patch12:44
rpittaudtantsur: in ironic ?12:44
dtantsurrpittau: https://review.opendev.org/#/c/747355/2/zuul.d/ironic-python-agent-jobs.yaml12:44
patchbotpatch 747355 - ironic-python-agent - Increase memory for tinyipa jobs - 2 patch sets12:44
rpittauoh12:45
rpittauyeah12:45
rpittauit's experimental, can we remove it after that patch lands ?12:46
openstackgerritRiccardo Pittau proposed openstack/ironic-python-agent master: Increase memory for tinyipa jobs  https://review.opendev.org/74735512:46
iurygregorythe experimental job I think we should just drop12:49
rpittaumetalsmith is already at 1024 so should be ok12:49
openstackgerritRiccardo Pittau proposed openstack/ironic-python-agent master: Remove experimental tinyipa job  https://review.opendev.org/74740012:51
iurygregorycan't we drop all experimental?12:56
iurygregoryI have the feeling we already test - ipa-tempest-dsvm-partition-ipmi-iscsi-tinyipa ...12:56
rpittauwe peobably should, that is quite old also12:56
iurygregoryyeah12:57
iurygregorythe only thing is the dsvm in the name hehe12:57
rpittauok, it feels old :)12:59
dtantsurI think we can drop all these jobs13:11
rpittauokey dokey13:11
openstackgerritRiccardo Pittau proposed openstack/ironic-python-agent master: Remove experimental jobs  https://review.opendev.org/74740013:13
*** Goneri has joined #openstack-ironic13:16
*** k_mouza has joined #openstack-ironic13:19
openstackgerritRiccardo Pittau proposed openstack/ironic-python-agent master: Increase memory for tinyipa jobs  https://review.opendev.org/74735513:30
TheJuliabraaaains13:30
openstackgerritRiccardo Pittau proposed openstack/ironic-python-agent-builder master: Fix finalise tinyipa  https://review.opendev.org/74719813:31
rpittauon the rocks please13:31
dtantsurmorning TheJulia13:31
openstackgerritJulia Kreger proposed openstack/ironic stable/queens: Fix for failure in cleaning  https://review.opendev.org/74728813:33
TheJuliahttps://www.bulleit.com/whiskeys/bulleit-rye/ <-- no, this on the rocks.13:34
rpittau:)13:34
rpittauthey have a France dedicated site, wow13:35
openstackgerritJulia Kreger proposed openstack/ironic stable/queens: Retries and timeout for IPA command  https://review.opendev.org/74727113:35
TheJuliaI've managed to find Bulleit Rye multiple countries13:35
dtantsurI've found another reason to ditch the iscsi deploy: we neither clean up the iscsi target automatically nor re-read the partition table13:35
dtantsursigh13:36
TheJuliaIncluding .au and .cn13:36
TheJuliadtantsur: kill it with fire!13:36
TheJuliathe reason I'm backporting the agent retry changes is beause I've been digging at logs of cases where basically it was communication to the agent that was breaking after an iscsi deploy was finished. They wanted the other patches we did for client retries, but they wouldn't quite help the situation13:37
openstackgerritDmitry Tantsur proposed openstack/metalsmith master: CI: use the direct deploy by default  https://review.opendev.org/74741313:39
rpittaummm interesting13:39
openstackgerritRiccardo Pittau proposed openstack/metalsmith master: [WIP] Use Centos 8  https://review.opendev.org/73727813:39
dtantsurTheJulia: I guess I'll wait for the meetup before proposing a deprecation release note13:40
*** rloo has joined #openstack-ironic13:40
dtantsurbut I did a quick poll among the operators I know and they were mostly "whatever"13:41
rpittaudtantsur: you can change most of the jobs definitions in metalsmith like that I guess13:41
dtantsurmmmm?13:41
dtantsursorry, didn't quite get it13:42
rpittausorry, you changed IRONIC_DEFAULT_DEPLOY_INTERFACE to direct, I guess you'll have to change the children jobs accordingly13:42
TheJulia2020-08-20 21:27:05.488031 | controller |   Could not find a version that satisfies the requirement bcrypt>=3.1.3 (from ironic-lib>=4.1.0->ironic_python_agent) (from versions: )13:44
TheJulia213:44
TheJulia*sigh*13:44
rpittauTheJulia: fix is in the oven, we also had to change size of tinyipa jobs13:45
TheJuliaany reviews needed?13:45
rpittauhttps://review.opendev.org/747198 https://review.opendev.org/747397 https://review.opendev.org/74735513:45
patchbotpatch 747198 - ironic-python-agent-builder - Fix finalise tinyipa - 4 patch sets13:45
patchbotpatch 747397 - ironic - Increase memory of tinyipa vms - 1 patch set13:45
patchbotpatch 747355 - ironic-python-agent - Increase memory for tinyipa jobs - 4 patch sets13:45
TheJuliadragons will appear to set everything on fire13:46
TheJulianice!13:46
rpittaubased on a true story :)13:46
rpittaubtw bcrypt faults "Shipped abi3 Windows wheels (requires pip >= 20)"13:47
rpittauneed to drop, have an appointment in 10 minutes(!), but I'll check the patches later tonight, and feel free to update them if needed! :)13:53
rpittauhave a great weekend everyone! o/13:53
*** jtomasek has quit IRC13:56
*** tzumainn has joined #openstack-ironic13:57
Qianbiaohello ironic.14:03
Qianbiaogood afternoon, (maybe?)14:03
Qianbiaois there any network architecture picture which desc how ironic + openstack neutron works?14:04
dtantsurI don't think we have a picture for that14:08
Qianbiaook ~14:09
dtantsurTheJulia: I'm pondering non-voting CI jobs that fail if one of our services have warnings (maybe with some exclusions) during a run14:09
dtantsurcontext: I've found a bug by finding a warning while looking at some logs for an unrelated reason14:09
Qianbiaodtantsur should bm management port be in same L2 network with openstack services14:10
dtantsurQianbiao: you mean, BMC port? no, not necessary (and even undesired)14:10
TheJuliadtantsur: maybe such a job could be helpful, even experimental to start would be good14:10
Qianbiaodtantsur once ironic could reach BMC port, everything goes?14:12
TheJuliafyi, that ansible job failure on redfish is before the ramdisk job runs.... funny thing is it would have found the error I'm looking at now14:12
dtantsurQianbiao: yep, only L3 connectivity between ironic-conductor and the BMC should be required14:13
TheJuliaand FWIW, L2 just won't work with some BMCs14:13
TheJuliaspecifically supermicro doesn't let you talk to the bmc unless your the default gateway14:13
dtantsurwow14:13
dtantsurstable cores, please: https://review.opendev.org/#/c/747394/14:13
patchbotpatch 747394 - ironic (stable/ussuri) - Wipe agent token during reboot or power off - 1 patch set14:13
Qianbiao TheJulia dtantsur thanks for the information.14:14
Qianbiaoand is there any ansible playbook or docker images which can simple help setting up openstack + ironic env14:15
openstackgerritAija Jaunteva proposed openstack/ironic-specs master: System configuration within whole clean or deploy step  https://review.opendev.org/74072114:15
TheJuliaokay, this is bizzar14:18
dtantsurQianbiao: depending on what exactly you need. devstack provides a testing environment with VMs.14:18
TheJuliasomehow /dev/sr0 is getting chosen as the root device in the ansible drier14:19
Qianbiaodtantsur ok, i will start from devstack.14:20
dtantsurTheJulia: \o/ why not indeed14:20
dtantsurI think the ansible driver's root device handling is much more primitive than IPA's14:21
TheJuliafar14:21
TheJuliafar far14:21
dtantsurspecifically, I'm not sure it has all our code for excluding CD's, floppies, etc14:21
dtantsurI wonder if we should move them to ironic-lib and use in 2 places14:21
TheJulialooks like it does its processing in a dedicated ansible module intree14:22
* dtantsur is wondering what happens if somebody tries to use software RAID with a non-IPA deploy method14:22
TheJuliahahahaha14:41
TheJuliaI know why this happened14:41
TheJuliathe change to enable the test actually set everything to use virtual media and when the tinycore ramdisk booted via cdrom, the ansible deploy module decided that it should be used!14:42
openstackgerritJulia Kreger proposed openstack/ironic master: Ansible deploy - Ignore invalid devices  https://review.opendev.org/74742814:42
dtantsurTheJulia: sounds plausible14:44
dtantsurTheJulia: release note please ^^^14:44
TheJuliak14:44
TheJuliahttps://github.com/openstack/ironic/blob/master/ironic/drivers/redfish.py#L5614:45
TheJuliaand that is why14:45
TheJuliathe union of the supported and enabled interfaces, everything defaulted to it14:45
dtantsuryep. and I already regret it.14:45
dtantsurI mean, regret putting this first, given how well hardware supports it...14:45
TheJuliawe can move it to the back?14:45
dtantsurTheJulia: potentially breaking, but we can14:46
TheJuliait is not widely used, we tell people to use defaults or to submit default interfaces14:46
dtantsurmaybe we should, at least until the industry makes up its mind14:46
TheJuliaWe may want to do the cycle release as a major release then14:46
TheJuliawhich I'm fine with14:46
dtantsuryup. we can remove something deprecated while we're on it :)14:46
dtantsurwanna do a patch or should I?14:46
TheJuliaup to you, I'm going to go write the reno now for the ansible change14:47
dtantsurTheJulia: okay. and please handle floppy devices too14:47
TheJuliaoooh14:47
TheJuliayes14:47
TheJuliafd14:47
dtantsurI cannot for the live of me remember how they're called14:47
dtantsurright14:47
* dtantsur changes redfish14:47
openstackgerritJulia Kreger proposed openstack/ironic master: Ansible deploy - Ignore invalid devices  https://review.opendev.org/74742814:52
openstackgerritDmitry Tantsur proposed openstack/ironic master: Move redfish-virtual-media to the back of supported_boot_interfaces  https://review.opendev.org/74743014:53
dtantsurTheJulia: something like ^^?14:54
TheJulialooks fairly good, mind if I do a quick edit to the reno?14:55
dtantsurnot at all14:55
dtantsurTheJulia: another thing that justifies a major version.. making agent token required?14:56
openstackgerritJulia Kreger proposed openstack/ironic master: Move redfish-virtual-media to the back of supported_boot_interfaces  https://review.opendev.org/74743014:56
dtantsurnow, no, not - the same word, just different spelling14:56
dtantsur:D14:56
dtantsurJayF: yep, the IPA CI is broken, the patches are in the gate14:57
JayFdtantsur: ack; ty14:57
JayFdtantsur: How do you feel about TLS patch? Obviously it's failing because I accidentally a comma, but in theory is it likely to merge?14:57
dtantsurJayF: I think I fixed and +2'ed it14:58
JayFThat was the eventlet patch IIRC, I'll check again14:58
JayFyeah https://review.opendev.org/#/c/747193/ is failing, syntax error (missing comma)14:59
patchbotpatch 747193 - ironic-python-agent - If listen_ssl is true, enable TLS on wsgi server - 2 patch sets14:59
dtantsurah, sorry14:59
dtantsurJayF: LGTM modulo the syntax error15:00
dtantsurI've set up my vim to run flake8 on save, helps avoid such situations15:00
JayFwork machine is a mac, and things sometimes barely work on it :|15:00
dtantsurI see15:00
JayFSupposedly we're getting a linux option soon, which I think I'll avail myself of15:01
JayFespecially upstream, it can be tough to get code between a test server and the local machine without causing massive gerrit spam15:01
JayFwith a github-workflow, you can just push and pull branches15:01
*** trandles has joined #openstack-ironic15:01
TheJuliadtantsur: I tink we should got on it during the midcycle calls. I'd like to but I have long term upgradability concerns for commercial distributions. :\15:02
openstackgerritJay Faulkner proposed openstack/ironic-python-agent master: If listen_ssl is true, enable TLS on wsgi server  https://review.opendev.org/74719315:02
JayFdtantsur: ^ should fix that syntax error, although I understand if you wanna let CI go before a review. Apparently doesn't matter since the CI is broken anyway :|15:02
dtantsurTheJulia: ah, that people that want upgrades to work across several versions but don't want to put much effort upstream? :( yep, I know them15:02
dtantsurJayF: as an aside, do you know that it's now preferred to use URLs in Depends-On?15:03
JayFdtantsur: like https://review.opendev.org/747193 style URLs?15:04
patchbotpatch 747193 - ironic-python-agent - If listen_ssl is true, enable TLS on wsgi server - 3 patch sets15:04
TheJuliadtantsur: yeah..... I've put it on the topic list15:04
dtantsurJayF: exactly15:04
JayFack; thanks for the heads up15:04
dtantsurTheJulia: we can keep IPA support for older ironic. I don't think old IPA + new ironic is going to work for many other reasons.15:06
TheJuliaI'm actually kind of pondering if those vendors want seamless, that they just need to update IPA15:07
TheJuliain advance and maybe "tough" is the approach to take15:07
dtantsurthis requires us to keep support for no agent token in IPA15:07
dtantsurwhich.. is probably fine.15:07
TheJuliayeah15:08
JayFThat convieniently would allow me to follow latest-IPA for the most part as well...15:08
dtantsurgiven that JayF successfully uses Ussuri IPA with Ocata, it works already :)15:08
JayF*for cleaning only15:08
dtantsurI mean, agent tokens don't prevent that15:08
JayFsome restrictions apply, read the fine print, etc15:08
JayFlol15:08
TheJuliawe have put a lot of effort to keep IPA very compatible15:08
JayFI mean, and frankly, IPA just doesn't have a lot of code churn15:09
TheJuliayup15:09
dtantsurwe're trying, but we don't ensure it in the CI15:09
JayFI pretty much did a full review of all code from ocata -> ussuri as part of deciding it was OK, there's not a lot there15:09
dtantsurJayF: wait for us to remove the iscsi deploy ;)15:09
JayFdtantsur: I consider it a personal victory that is being removed15:09
dtantsur:D15:09
TheJuliaI think there has been one case where there was someone who had a breaking config but it was like... queens ipa with fairly recent master branch15:09
JayFdtantsur: The first ever Ironic mid-cycle we went to, in yahoo at Sunnyvale hilariously enough, and the iscsi deploy was explained to us, we basically spent the next two days WTF-ing at it. Agent wins the day \o/15:10
dtantsurright :) well, it was good for start, I guess. remember the old bash ramdisk?15:10
JayFSome memories are better suppressed :}15:10
dtantsurit's still haunting me15:10
JayFTheJulia: I am ++ to the idea that keeping new-IPA -> old-Ironic is reasonable, I don't think newer ironic supporting older IPA holds any value whatsoever15:11
dtantsurat least you haven't seen the several-hundred-lines-of-code bash ramdisk we inherited downstream from one older product15:11
* TheJulia radios into HQ to indicate that dtantsur is remembering the bash ramdisk15:11
JayFTheJulia: is there a direct use case for "I can't run new IPA in my ramdisk"15:11
TheJuliaage of images+drivers really15:11
dtantsurJayF: N+1 ironic with N ipa is useful for upgrades15:11
JayFWhy? Why can't the process be "update IPA first"15:12
JayFTheJulia: AFAICT, there's no reason the IPA *code* can't be built into an older ramdisk/OS, right?15:12
trandlesI want to confirm I didn't do something stupid (likely) before I claim something is broken (less likely). When debugging my bifrost install it doesn't look like BOOTP was allowed. firewall-cmd doesn't show it and when I stopped firewalld service then requests started reaching dnsmasq. I _assume_ this isn't expected and I missed something again.15:12
JayFI guess py27 vs py3x reasons15:12
TheJuliathat is basically it15:12
dtantsurJayF: it can, but people appreciate flexibility15:12
dtantsurTheJulia, JayF, well, we support centos 7 in ipa-builder again..15:13
JayFI mean, we have a downstream expectation that at some point, for older hardware, it's likely we'll have to have a CentOS7 version of the IPA ramdisk as well.15:15
dtantsurwell, good news: it builds again15:15
JayF\o/15:16
TheJuliahttps://thumbs.gfycat.com/HauntingDirtyDragon-size_restricted.gif15:17
* dtantsur prepares to kill non-venv bifrost15:22
*** tkajinam has quit IRC15:26
*** gyee has joined #openstack-ironic15:34
rpiosobrtknr: You are most welcome. Glad you found the WS-Man class.15:34
rpiosorpittau: LOL15:34
*** cp- has quit IRC15:42
*** cp- has joined #openstack-ironic15:43
openstackgerritDmitry Tantsur proposed openstack/bifrost master: Remove support for non-venv deployments  https://review.opendev.org/74743415:45
dtantsurboom!15:45
TheJulianot much code, but pervasive15:45
dtantsuryeah15:45
TheJulialooks riccardo's chagnes are good, but I'm guessing we'll need to backport them all15:46
dtantsurexcept for ipa-builder, yeah15:47
dtantsurthe reason we need image_download_source=http https://zuul.opendev.org/t/openstack/build/74b24b98bde34c19a26aff67f7f0db43/log/controller/ironic-bm-logs/node-0_no_ansi_2020-08-21-15:23:12.log#3551 :)15:48
openstackgerritDmitry Tantsur proposed openstack/metalsmith master: CI: use the direct deploy by default  https://review.opendev.org/74741315:49
*** mkrai has joined #openstack-ironic15:53
*** ociuhandu_ has joined #openstack-ironic15:53
*** ociuhandu has quit IRC15:56
dtantsurtrandles: it used to work at least, including with firewalld15:57
*** ociuhandu_ has quit IRC15:57
bdoddGood morning, ironic15:57
JayFo/15:58
dtantsurtrandles: http://paste.openstack.org/show/797042/ includes dhcp for me. or are you using it on bare metal?15:59
trandlesbare metal15:59
*** lucasagomes has quit IRC16:00
dtantsurmmm, I wonder if we missed it16:00
trandlesdtantsur, updated http://paste.openstack.org/show/797043/16:01
bdoddI created a redfish devstack env. And I can access the SimpleStorage redfish resource on each System (node) in the emulator. But trying to access the Storage resource results in a 500 error.16:01
dtantsurtrandles: lemme try something16:02
bdoddAnyone have pointers to what I need to do to get the Storage resources enabled?16:02
bdoddajya ^^^^ (in case you may know about this)16:03
openstackgerritDmitry Tantsur proposed openstack/bifrost master: [WIP] Explicitly enable DHCP services on baremetal CentOS/RHEL  https://review.opendev.org/74743716:04
dtantsurtrandles: could you try ^^^16:04
TheJuliabdodd: the emulator?16:04
dtantsurbdodd: check sushy-tools logs, probably a bug16:04
bdodddtantsur Thanks. Will check that.16:05
bdoddTheJulia Yes, the sushy-tools emulator16:06
TheJuliait may be that the emulator only supports the static entry return insteadd of the dynamic vm control16:06
dtantsurTheJulia: this is definitely true16:06
dtantsurmaybe SimpleStorage is real, but most resources are static16:07
*** ociuhandu has joined #openstack-ironic16:15
openstackgerritDmitry Tantsur proposed openstack/bifrost master: Explicitly enable DHCP services on baremetal CentOS/RHEL  https://review.opendev.org/74743716:15
trandlesdtantsur, sorry, testing is taking a minute16:18
*** ociuhandu has quit IRC16:19
dtantsurrpittau: we might have a chicken-and-egg problem with your patches..16:19
dtantsurhttps://review.opendev.org/#/c/747355/ requires the ipa-builder fix, which in turn requires it16:20
patchbotpatch 747355 - ironic-python-agent - Increase memory for tinyipa jobs - 4 patch sets16:20
JayFperhaps temporarily make one of the broken jobs on one repo or the other non-voting long enough for it to merge in?16:24
trandlesdtantsur, that worked...requests are getting through to dnsmasq16:24
TheJuliaWhat JayF said is what I did a couple weeks ago instead of torture myself with any other path16:25
TheJuliaand reverted the ci change right after the patchs to fix things merged16:25
trandlestftpd isn't set up to serve pxelinux.0 so that's the next thing to figure out16:26
TheJuliatrandles: well... hmm16:26
trandlesundionly.kpxe causes it to fail and move on16:27
TheJuliaI think we nuked syslinux chain booting16:27
trandlesoops16:27
* TheJulia rasies an eyebrow16:27
TheJuliawait,  your machines are refusing to boot directly to the ipxe binary and want to go through syslinux first?16:27
trandlesipxe isn't supported from what I can tell16:27
* TheJulia tries to wrap her head around this statement16:29
trandleslet me back up...it doesn't look like chainloading into ipxe is working16:30
dtantsurtrandles: maybe your machines are in UEFI?16:30
dtantsuralthough.. bifrost should work anyway16:30
*** k_mouza has quit IRC16:30
trandlesdefinitely not UEFI16:30
trandlesthat gets disabled as soon as we unbox16:31
*** dougsz has quit IRC16:31
trandlesI'll try to capture the SOL output16:32
dtantsurtrandles: what exactly happens with undionly?16:32
dtantsuryes please16:32
trandlesimmediately drops through to grub on the local drive after downloading undionly16:33
dtantsurOo16:33
dtantsurdo you have x86-64 machines?16:33
trandlesyup16:33
TheJuliayeah, console output16:34
TheJuliaIt _might_ be that you need a different ipxe build16:34
TheJuliaor specific drivers, or... somethign16:34
dtantsuror fallback to plain PXE, which we don't have in bifrost..16:34
trandleshttp://paste.openstack.org/show/797045/16:35
trandlesgot it before the screen refreshed with local grub16:35
dtantsurtrandles: mmm, it doesn't seem to get to iPXE even16:35
dtantsurdo the IP and netmask match expected?16:36
* dtantsur thinks of a conflicting DHCP server16:36
*** Qianbiao has quit IRC16:37
dtantsurmy next suggestion was to try iPXE builds from their website (https://boot.ipxe.org/undionly.kpxe), but this doesn't look like it's getting to iPXE at all16:37
trandlesoh, this time I found in the logs, permission problem with undionly.kpxe16:37
TheJuliaoh... hmm16:37
dtantsurSIGH16:37
trandlesthe previous time I didn't have the permission denied...wtf, am I dropping log messages?16:38
trandles*it didn't16:38
dtantsurtrandles: selinux or regular ACL?16:38
dtantsuron my machine it's readable only by root, but dnsmasq should also be run as root..16:38
trandlesin permissive mode, should be regular16:38
dtantsurbtw bifrost does work in enforcing mode (if it does not - please report)16:39
trandleson CentOS 8 dnsmasq is running as dnsmasq16:39
dtantsurmmmmmmm...16:39
dtantsurand it works in the CI because libvirt has iPXE ROM16:39
dtantsurWork On DevStack (tm)16:39
dtantsurokay, should not be hard to fix. hold on.16:40
trandlesI should quit my current job and just be a QA engineer. This entire week has gone exactly like this. On two different internal projects I've been finding corner cases and writing bug reports.16:42
trandlesBTW - what git remote should I be pointed at to test your stuff dtantsur?16:46
openstackgerritDmitry Tantsur proposed openstack/bifrost master: Make the iPXE and PXE boot files world-readable  https://review.opendev.org/74744216:49
dtantsurtrandles: next patch ^^^16:49
dtantsurI don't have a single branch, just use the cherry-pick feature of the gerrit UI16:49
dtantsurnow I have to go, have a great weekend everyone. feel free to ping me on Monday trandles if you still struggle16:49
*** dtantsur is now known as dtantsur|afk16:50
JayFo/16:50
clarkbgit review -d and git review -x will do it for you too16:51
clarkb(you don't need the web ui)16:51
trandlestesting, thanks dtantsur|afk16:51
openstackgerritMerged openstack/ironic master: Increase memory of tinyipa vms  https://review.opendev.org/74739716:55
*** ijw has joined #openstack-ironic17:01
*** mkrai has quit IRC17:04
*** priteau has quit IRC17:07
trandles\o/ everything seems to be working now :)17:09
trandlesthanks for your patience with me #openstack-ironic17:09
TheJuliatrandles: \o/17:16
JayFcongrats on getting it working!17:16
TheJuliatrandles: thank you for pointing out the issue!17:16
*** k_mouza has joined #openstack-ironic17:17
openstackgerritJulia Kreger proposed openstack/bifrost bugfix/8.3: Make the iPXE and PXE boot files world-readable  https://review.opendev.org/74744617:17
openstackgerritJulia Kreger proposed openstack/bifrost bugfix/8.2: Make the iPXE and PXE boot files world-readable  https://review.opendev.org/74744717:17
openstackgerritJulia Kreger proposed openstack/bifrost stable/ussuri: Make the iPXE and PXE boot files world-readable  https://review.opendev.org/74744817:17
openstackgerritJulia Kreger proposed openstack/bifrost stable/train: Make the iPXE and PXE boot files world-readable  https://review.opendev.org/74744917:17
TheJuliaThat seems like it is enough...17:18
*** chenhaw has joined #openstack-ironic17:22
openstackgerritJulia Kreger proposed openstack/ironic master: Ansible deploy - Ignore invalid devices  https://review.opendev.org/74742817:24
trandlesI really should just bite the bullet and figure out how to actually contribute instead of whining in here :P17:28
TheJuliait is not hard!17:31
* TheJulia wonders if they are goign to do a virtual upstream institute this year17:33
openstackgerritJulia Kreger proposed openstack/ironic stable/queens: Fix for failure in cleaning  https://review.opendev.org/74728817:37
trandlesThe node booted and the SOL console got reset in a way that only 4 lines at the top of my terminal were refreshing and scrolling. I saw some IPA stuff scroll by quickly. 'baremetal node list' and 'baremetal introspection list' and 'baremetal port list' are all empty and there's nothing in the logs on the ironic host. :(17:39
trandlesand I can't ping the host17:39
*** rloo has quit IRC17:40
*** rloo has joined #openstack-ironic17:40
* TheJulia tries to grok this17:44
TheJuliais it still on? sounds like it may have hit discovery?17:45
TheJuliabut the node is not present so I'm wondering if it was before hand?17:45
*** rloo has quit IRC17:45
*** dustinc has quit IRC17:45
trandlesit's on17:45
trandlesI'm wondering if it tried to callback to localhost instead of the actual IP of the ironic server17:45
TheJuliapossibly17:46
TheJuliaI'd reset your sol connection and see what is going on17:46
TheJuliatypically IPA will keep scrolling and running17:46
trandlesI'll reset the terminal and reconnect and let it sit to see if anything shows up17:46
*** ijw has quit IRC17:47
trandlesI'm using iterm2 because it's a lot better than the mac builtin terminal, but I do miss the days of a real VT17:47
*** ijw has joined #openstack-ironic17:48
*** rloo has joined #openstack-ironic17:49
TheJuliayeah, I've had that happen to me some too.17:50
trandleshow would I go about modifying the kernel cmdline in bifrost to add "console=ttyS0,115200" ? My testbed is actually a bunch of greenblade nodes so I have a chassis BMC that I can telnet to for a serial console instead of using IPMI's SOL support.17:52
trandlese.g., can I use imgargs?17:53
trandlesoh, imgargs is deprecated...I need to get with the times17:54
TheJuliayou can set pxe_append_something in ironic.conf and restart the conductor17:55
TheJuliait will append the attributes as longa s the node is being controlled17:55
TheJuliaif not, you can just exit the ipxe boot script17:55
trandlesah, ok17:55
trandlesnetwork fails to come up18:05
openstackgerritJulia Kreger proposed openstack/ironic bugfix/15.2: Increase memory of tinyipa vms  https://review.opendev.org/74745518:05
trandlesit at least is trying to reach the correct IP for the ironic server18:07
*** sradhakrishna has joined #openstack-ironic18:08
openstackgerritJulia Kreger proposed openstack/ironic-python-agent master: Temporary disable multiple jobs to land CI fixes.  https://review.opendev.org/74745718:08
sradhakrishnalooking for help with debugging ironic agent. Is there a procedure to get the inspector image to boot a system outside openstack, so I can test my agent independently, before I integrate it with openstack?18:09
openstackgerritJulia Kreger proposed openstack/ironic bugfix/15.2: Increase memory of tinyipa vms  https://review.opendev.org/74745518:10
openstackgerritJulia Kreger proposed openstack/ironic stable/queens: Retries and timeout for IPA command  https://review.opendev.org/74727118:13
TheJuliano approving/rechecking patches on ipa right now please18:16
TheJuliaor I'll become very sad  :(18:16
TheJuliasradhakrishna: You should be able to boot it regardless, but I'm wondering what is actually driving the desire to test independently because they are tightly coupled components18:17
TheJuliaan explict documented procedure, not really but you can just follow the same steps as if your doing discovery and have a network boot config to the kernel/ramdisk18:18
sradhakrishnaTheJulia: We have a custom ironic agent that discovers a custom set of attributes as part of the inspection. Want to test that out18:19
sradhakrishnaTheJulia: Was looking to boot out the image separately and test the agent from inside the system, once booted. Is there an easier way to test?18:20
TheJuliaI mean you can, but you'd still need someplace to post that data to18:20
TheJuliawhich means you'd need an inspector instance running18:21
TheJuliaif your doing a one-off test, you could always make an iso and attach it directly to a test machine and just let it publish that data to the api endpoint18:21
sradhakrishna@TheJulia: I've tried creating an iso and use it to run inspector on the node from horizon, but it failed - couldn't debug further, hence the thought of this one-off test18:23
TheJuliafrom horizon is not going to work18:24
TheJuliayour going to have to do a custom one off iso image by hand18:24
TheJuliayour off the path any of this gets tested18:24
TheJuliabecause your basically doing development of custom agent code18:24
sradhakrishnaTheJulia: So, I've written a custom inspector plugin - a python class deriving from base.ProcessingHook (ironic_inspector.plugins)18:26
sradhakrishnaAnd I've created the iso image.18:27
TheJuliawait, for ironic-inspector?18:27
TheJuliayeah, your going to need to somehow setup an ironic-inspector instance with your patches, and then have an agent boot (somehow) to post to that18:28
TheJuliatypically upstream we would use devstack or bifrost18:28
TheJuliasince all the tooling is there and we would just attach it to a network interface or use test virtual machines.18:28
openstackgerritRiccardo Pittau proposed openstack/ironic-python-agent master: Increase memory for tinyipa jobs  https://review.opendev.org/74735518:34
TheJulia*sigh*18:34
TheJuliarpittau: should I kill my ci change?18:34
trandlesconfirmed IPA isn't bringing up all of my network interfaces...most importantly the one connected to my management network18:35
TheJuliaoh, I see you rebased it18:35
TheJuliatrandles: doh!18:35
TheJuliais it drivers?18:35
trandlesshouldn't be, it's just an igb card18:35
TheJuliawhich ipa image are you using?18:35
trandleswhatever bifrost downloaded or setup18:36
trandlesI'm on the master branch18:36
TheJuliaokay18:36
TheJuliaso.... it likely downloaded tinyipa18:36
openstackgerritRiccardo Pittau proposed openstack/metalsmith master: [WIP] Use Centos 8  https://review.opendev.org/73727818:37
trandlesif there's an easy way to check by looking at the images in /httpboot/ I'll let you know18:37
TheJuliawhat is the size of the files in /httpboot?18:40
TheJuliathe kernel and initramfs18:40
TheJuliawell really the initramfs18:40
TheJuliaif it is like 40-50mb, it is tinyipa18:41
trandles56MB18:41
TheJuliaI'd replace those files with the centos8 equivelent files then18:41
*** ijw has quit IRC18:41
TheJuliatrandles: let me get you a linik!18:41
trandlesgotcha18:41
trandlesoh, even better, I was gonna go searching ;)18:41
TheJuliahttps://tarballs.opendev.org/openstack/ironic-python-agent-builder/dib/files/18:41
TheJuliaenjoy!18:42
TheJuliafwiw18:42
TheJuliaby chance are you using LACP at all?18:42
trandlesI am not18:42
TheJuliaokay18:42
trandlesat least not yet18:42
trandlesbaby steps ;)18:42
TheJuliawell, using lacp on ramdisks is super problematic18:42
TheJuliabut that ramdisk should have a patch to make it somewhat happier18:43
trandlesah ok18:43
TheJuliayou'll still want to set an initial startup delay into ipa, when the time comes we can point you to that config18:43
trandlesmy testbed does not have enough links for LACP18:43
TheJuliababysteps18:43
trandlesproduction will be a different story18:43
TheJuliaok, if it is passive lacp then you have no need to worry18:43
TheJuliaif your active/default on... then network booting is a world of pain18:43
* TheJulia suspects that could be a t-shirt, but that she could never wear it out of the house18:44
trandlesI've hit this before in a totally different context18:44
trandlesreally it boils down to one of the few universal truths: blame the network18:44
TheJuliawell, it is not _just_ the network... but yeah18:45
TheJuliaWell, the universal truths are more complex than that18:46
TheJuliaThere may.... or may not be a spoon18:46
TheJuliaand we may all be holograms in a simulation18:46
trandlesby the by, how does pxelinux.cfg/default get regenerated? restarting ironic-conductor after adding some things to pxe_append_params didn't do anything18:47
trandlesI edited by hand18:47
TheJuliaoh, that file doesn't get regenerated18:47
trandlesit's turtles all the way down18:47
TheJuliayou can edit it at will but if you re-install18:47
*** ijw has joined #openstack-ironic18:48
openstackgerritMerged openstack/ironic bugfix/15.2: Increase memory of tinyipa vms  https://review.opendev.org/74745518:51
openstackgerritJulia Kreger proposed openstack/ironic master: Guard conductor from consuming all of the ram  https://review.opendev.org/72648318:53
*** bnemec has quit IRC18:53
*** AlanMimms has joined #openstack-ironic18:56
AlanMimmsCan someone point me to a list of Redfish functionality/properties/resources requirements for Ironic Stein? I want to be sure our Redfish implementation can work properly and I'm planning the tasks to get it there.18:57
TheJuliaAlanMimms: Greetings!18:58
AlanMimmsHi18:58
TheJuliaGreat question18:58
TheJuliaslightly unfortunate we didn't start on an interop profile ages ago18:58
AlanMimmsHas someone now created one? <hoping>18:59
TheJuliaSo the person to ask for an update on that from is rpioso18:59
TheJuliaI think he is on staycation this week18:59
TheJuliaso stein.. hmmm18:59
trandlesWoot woot, got a node enrolled via inspector19:00
TheJuliatrandles: congrats19:00
trandlesmerci19:00
TheJulialargely it is power control under system and the boot mode overrides19:00
AlanMimmsMurphy's Law of questions. OK. Would it be best to contact @rpioso next week?19:00
TheJuliaAlanMimms: he should be around on Monday, we hold an IRC meeting here at 8 AM US Pacific on Monday morning19:01
TheJuliamornings19:01
AlanMimmsYes, I had guessed as much. I was thinking of looking at the CI tests to see what they do. Maybe that's going to test each Resource/Property?19:01
TheJuliahmmmmm19:01
*** bnemec has joined #openstack-ironic19:01
AlanMimmsI will make a note to contact him on Monday then. Thank you VERY much for the pointer.19:01
TheJuliathat might work19:01
TheJuliawe use an intermediate library called sushy for everything though19:01
TheJuliaso it hides a lot of the end resources from us on a daily basis19:02
TheJuliaBut sushy also has a larger emulator so you can't say "well if I can pretend to be sushy! It must work!"19:02
AlanMimmsYes. Sushy looks very good. But it's general - not specific. So while it can be used to talk to many resources/properties it doesn't say which is actually used.19:02
TheJuliaI'm trying to remember the property uesd to change boot mode, and that seems to be the more questionably supported field by vendors. BootModeOverride* is what we use to set boot to disk or boot to network. We don't use the redfish 1.9 uefi specific fields though19:03
AlanMimmsYeah. 1.9 isn't widely avail yet19:04
TheJuliayeah.... Some vendors have also leaked some of it into patches on earlier versions19:04
AlanMimmsYah. And some of these earlier patches may be the genesis from which the DMTF finally adopted the standardized solution.19:05
TheJuliaA change in 1.8 bit us too19:05
TheJuliaA change in wording that multiple vendors took as "lets break the behavior"19:05
TheJulia:(19:05
TheJuliaThat was for continious overrides with uefi mode....19:06
*** bnemec has quit IRC19:08
AlanMimmsI'm REALLY hoping for full interoperability when our WIP generation ships. Hence my investigation. Thanks for the pointers. Maybe I should subscribe to the mail list.19:08
rpiosoAlanMimms: We have a goal this Victoria release cycle to develop a Redfish Interoperability Profile for the vendor-independent redfish driver/hardware type. It is targeted at master, not the stable branches. However, it could be used as a basis for those.19:08
TheJuliaI guess the question also becomes what part of the feature set matters, and if the focus is just on "getting a deployment to work" or "more"19:09
AlanMimmsHi @rpioso! Thanks for that. I'm really looking for high level requirements to start out here. Details and rigorous testing are for 6-9 months from now19:09
TheJuliarpioso: hey, aren't you on staycation?!?19:09
rpiosoTheJulia: Yep19:09
TheJuliaOr are you downloading MSFT Flight Sim 2020 and bored :)19:09
rpiosoTheJulia: Bored, yes :-)19:09
AlanMimmsYes, exactly, TheJulia. It's "build a VW bug" or "build a Tesla".19:10
rpiosoAlanMimms: To be sure we're on the same page, are you referring to the profile described by this DMTF specification: https://www.dmtf.org/sites/default/files/standards/documents/DSP0272_1.3.0.pdf?19:10
* TheJulia would prefer to just make her FoST and FiST go faster up the mountain instead of "build a tesla"19:11
AlanMimmsYes that's the one. But I'm looking for high level interop basics at this point. Details are for later.19:11
rpiosoAlanMimms: The devil is in the details ;-)19:12
AlanMimmsWe're going to have to shoot for basic interop and control at first.19:12
TheJuliaThat should be a really short list then19:12
AlanMimmsYes, but the devil I'm fighting here is estimation of effort and getting a product manager to believe in how much work it will or won't be.19:13
AlanMimmsI think it probably is a short list.19:13
rpiosoAlanMimms: The DMTF Interoperability Validator, https://github.com/DMTF/Redfish-Interop-Validator, can validate a profile against a Redfish service.19:14
AlanMimmsYep.19:14
AlanMimmsThat's definitely planned as a testing requirement.19:14
rpiosoAlanMimms: Cool. Sounds like you're familiar with the DMTF toolbox: github.com/dmtf/ :-)19:15
AlanMimmsI have some familiarity :) - I have been lurking on the Redfish world for a couple of years getting ready for this project. There's another guy we just hired who is the real expert, but I have learned a few things...19:17
AlanMimmsAt least in theory :)19:17
rpiosoAlanMimms: Interesting input re: basics. We're implementing the profile source module-by-module, which translates to ironic hardware interface Redfish implementation-by-implementation -- .../ironic/drivers/modules/redfish/*.py. When that's done we've been planning to combine them into a single profile for the entire driver/hardware type.19:19
AlanMimmsSweet. I'm looking forward to that release. But I think since I'm in (some) control of our Redfish implementation and I need interop with older Ironic releases prominently used in the field I will have to go through, say, Stein or maybe even older and gather requirements to make sure we can function.19:20
rpiosoAlanMimms: It sounds like it may be useful to offer both, so folks could choose which interfaces they're interested in and to facilitate development of profiles for the stable branches.19:20
TheJuliaSo I have this feeling it boils down to: boot BootSourceOverrideEnabled, BootSourceOverrideMode, BootSourceOverrideTarget, system PowerState, the reset action (ResetType) (i think)  Naturally we also need the allowed option fields for all of that19:20
AlanMimmsOK.19:21
TheJuliathat _should_ cover basics19:22
rpiosoAlanMimms, TheJulia: Most fundamental would be what management.py and power.py use.19:22
AlanMimmsI think because I'm a Redfish VENDOR my perspective is different. I know you work for a vendor too, rpioso. But my guess is you're a consumer of some other internal group's Redfish implementation.19:22
AlanMimmsYes, rpioso, I have done a prelim walk through of those two source files. I need to go back and do that again in detail and with more care.19:23
rpiosoAlanMimms: Yes, I am with Dell EMC. May I ask which vendor you're with?19:23
TheJuliaAlanMimms: most of that is calling sushy's sushy/resource/system/system.py19:23
AlanMimmsJabil Cloud and Compute and Storage group19:23
JayFWe had some folks from DMTF foundation (? I don't remember the exact name) come have a discussion at our mid-cycle. The more we all work together, the easier it is for everyone :)19:23
TheJulias/foundation//19:24
AlanMimmsWe're a white box vendor to tier #1 cloud that is looking to expand to tier #2 asap.19:24
TheJuliaDistributed Management Task Force19:24
TheJuliaAlanMimms: awesome!19:24
AlanMimmsHah. Yes, it is occasionally awesome. Thanks.19:24
TheJuliaAlanMimms: well, most of us are here at least on weekdays so if there is anything we can do to help you on your quest, please don't hesitate19:25
AlanMimmswonderbar! I will probably find the occasion to take advantage of that. This community seems to be really strongly interconnected and interactive and that's both useful and refreshing.19:25
TheJuliawe try, we're all only human!19:26
TheJuliaI think...19:26
AlanMimmsThanks very much.19:26
rpiosoJayF: Yep, we've engaged the two DMTF Redfish Forum co-chairs, mraineri, Dell EMC and Jeff Autor, HPE. They are very much involved :-)19:26
TheJuliaI joke there may be an AI or two....19:26
AlanMimmsIs there doubt about some?19:26
* rpioso doubts himself19:26
JayFpretty sure openstackgerrit is not a human19:26
AlanMimmsGreat rpioso. I have talked with Jeff a lot at OCP Summit. He's a good guy to know.19:26
TheJuliaoh yes, the bots are definitely NOT human19:27
JayFat least until they are :-O19:27
rpiosoAlanMimms: They're both very responsive and helpful.19:27
* TheJulia loads up zuul to see if it is time to cry19:27
* TheJulia sighs19:29
TheJuliaI'm going to have to fire up a new vm or something for htese queens patches19:29
JayFTheJulia: hmm. Let me take a real close look at them. I might be able to figure it out.19:30
JayFTheJulia: I have those in ocata working, so it's probably something I already solved...19:30
JayFTheJulia: you wanna link the one that is failing, and how19:30
TheJuliaJayF: my conudnrum is right now I can't run the unit tests on any of my mahcines19:30
TheJuliabecause everything is python 3.7/3.819:30
TheJuliaand trying to go older just explodes19:30
TheJuliaI also need a 2.7 machine19:30
* TheJulia may actually have one sitting around19:31
TheJuliahttps://review.opendev.org/#/c/747288/ in zuul right now19:31
patchbotpatch 747288 - ironic (stable/queens) - Fix for failure in cleaning - 3 patch sets19:31
*** arne_wiebalck_ has joined #openstack-ironic19:31
*** alan_ has joined #openstack-ironic19:31
*** AlanMimms has quit IRC19:32
*** arne_wiebalck has quit IRC19:32
*** arne_wiebalck_ is now known as arne_wiebalck19:32
TheJuliawell, 2.7 failed due to the race on mysql19:32
alan_* was AlanMimms - not sure how to reclaim my nick after reconnect19:32
*** gyee has quit IRC19:32
TheJuliaalan_: /nick AlanMimms19:32
*** alan_ is now known as AlanMimms19:32
TheJuliahttps://storage.bhs.cloud.ovh.net/v1/AUTH_dcaab5e32b234d56b626f72581e3644c/zuul_opendev_logs_155/747288/3/check/openstack-tox-py27/155ecd7/testr_results.html :(19:32
AlanMimmsOff to lunch - thanks TheJulia and rpioso for the encouragement! [cheers]19:33
JayFAlanMimms: if you /msg NickServ, you can register your nickname with a password to save it for yourself. Then you can tell it to kick out imposters (or old leftover connections) to get it back19:33
*** rh-jlabarre has joined #openstack-ironic19:33
rpiosoAlanMimms: Enjoy. Great meeting you and hope to "see" you on Monday.19:33
TheJuliaoh, i see what the issue is19:33
* TheJulia goes to fix the patch... again19:34
JayFTheJulia: ah, I removed the call downstream because we don't use the drac driver, it's just another instance of deploy_utils.notify_conductor... that needs to instead come from agent_base_vendor.py19:34
TheJuliayup19:34
JayFif it kabooms again, and I see it before you, mind if I try and fix?19:34
JayFlike I said, been down this road so I can probably ID the failures by memory, I did this for ocata literally 2 weeks ago19:34
TheJuliasure, fixing how so hopefully ti shoudl be good to go in about 2 minutes19:36
JayFis "q" as far back as we stable maint for now?19:36
*** gyee has joined #openstack-ironic19:36
openstackgerritJulia Kreger proposed openstack/ironic stable/queens: Fix for failure in cleaning  https://review.opendev.org/74728819:37
TheJuliaI'm mainly going to Q for rhosp 1319:37
TheJuliasince upstream we have far more intensive and extensive testing of it before it goes to our downstream folks so basically if it is a patch to be backported, we try to get it in upstream on the same branch for good measure19:38
*** jhesketh_ has joined #openstack-ironic19:38
JayFspeaking of, any chance my eventlet patch would qualify for an IPA ussuri backport?19:39
JayFMy hunch is no, just because it's probably a touch more invasive than I'd be comfy with for a stable backport19:39
TheJuliaJayF: got a link handy?19:40
clarkbTheJulia: https://github.com/pyenv/pyenv is popular for dealing with the python versions problem19:40
JayFhttps://review.opendev.org/#/c/746774/19:40
patchbotpatch 746774 - ironic-python-agent - Eventlet should be monkey patched as early as poss... - 5 patch sets19:40
TheJuliaclarkb: I'm at the point where I just need fully differnet headers to link against to do module reinstalls from source19:40
JayFwaiting to merge on the tinyipa fixes :|19:40
*** jhesketh has quit IRC19:42
*** rh-jelabarre has quit IRC19:42
*** tzumainn has quit IRC19:42
*** rloo has quit IRC19:42
*** cp- has quit IRC19:42
TheJuliaJayF: I'd be really cautious but if it passes ci on ussuri and the bugfix brancehs I don't see why not19:42
JayFOh, that's right, we have more than just stable branches now19:43
JayFI'll try to backport it when it merges19:43
JayFsame idea -- I'd rather it be tested upstream than carried downstream19:43
*** rloo has joined #openstack-ironic19:44
JayFthe follow-on after that, to enable the TLS support from oslo.service, will probably need to be downstream as it adds a feature + config opt, unless you all are much more flexible about stable backports than previously19:44
JayF( https://review.opendev.org/#/c/747193/ )19:45
patchbotpatch 747193 - ironic-python-agent - If listen_ssl is true, enable TLS on wsgi server - 3 patch sets19:45
*** tzumainn has joined #openstack-ironic19:46
TheJuliaJayF: we're flexible when it comes to making life easier/better for the operator, but features like that would be really hard to justify upstream19:46
TheJuliaif there is a legitimate case that can be made that it is a bug... that changes the story19:46
JayFyeah; that's my feeling as well with an upstream hat on19:46
JayFbut it never hurts to verify19:46
TheJuliayeah19:47
JayFI do think the eventlet patch will have a net-positive effect on removing undefined behavior19:47
TheJuliaI'm going to go lay down for a littl ebit. I keep context switching today and feel like that is all I've done for the last two hours19:47
TheJulia++19:47
JayFo/ go collect some spoons19:47
openstackgerritJay Faulkner proposed openstack/ironic-python-agent master: If listen_ssl is true, enable TLS on wsgi server  https://review.opendev.org/74719319:47
*** cp- has joined #openstack-ironic19:49
trandlesTheJulia, I don't see anything upcoming for the upstream institute. Should I email Kendall and ask if one's happening pre-summit?20:03
TheJuliatrandles: it is still being organized20:14
TheJuliathere was a post seeking volunteers on the mailing list recently20:14
trandlesah, thx20:14
trandlesI subscribe to the discuss mailing list...and my mailbox shows 4638 unread :(20:14
TheJuliaimpressive20:15
TheJuliasuperhero like even!20:15
trandleswhen the mailing lists were consolidated to discuss I stopped trying to keep up...and my outlook/exchange filters got totally hosed during an exchange upgrade20:15
*** ijw has quit IRC20:16
*** Lucas_Gray has joined #openstack-ironic20:24
*** tzumainn has quit IRC20:36
openstackgerritJulia Kreger proposed openstack/ironic master: Detail iPXE + LACP troubleshooting information  https://review.opendev.org/74378721:19
*** rloo has quit IRC21:33
openstackgerritMerged openstack/ironic-python-agent master: Temporary disable multiple jobs to land CI fixes.  https://review.opendev.org/74745721:49
openstackgerritJulia Kreger proposed openstack/ironic-python-agent master: Revert "Temporary disable multiple jobs to land CI fixes."  https://review.opendev.org/74749021:51
openstackgerritMerged openstack/ironic-python-agent master: Increase memory for tinyipa jobs  https://review.opendev.org/74735521:59
*** trandles has quit IRC22:03
*** k_mouza has quit IRC22:11
*** ociuhandu has joined #openstack-ironic22:16
*** ociuhandu has quit IRC22:21
*** hjensas has quit IRC22:26
*** jamesdenton has joined #openstack-ironic22:31
*** Lucas_Gray has quit IRC22:36
openstackgerritJulia Kreger proposed openstack/ironic-python-agent bugfix/6.3: Increase memory for tinyipa jobs  https://review.opendev.org/74749222:39
openstackgerritJulia Kreger proposed openstack/ironic-python-agent bugfix/6.2: Increase memory for tinyipa jobs  https://review.opendev.org/74749322:39
TheJuliaokay, ipa builder patch rechecked, and revert is in ci now22:41
*** AlanMimms has quit IRC22:52
*** sradhakrishna has quit IRC23:17

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