Wednesday, 2020-03-04

*** diurnalist has quit IRC00:01
*** igordc has joined #openstack-ironic00:21
*** k_mouza has joined #openstack-ironic00:28
*** k_mouza has quit IRC00:32
*** k_mouza has joined #openstack-ironic00:34
*** igordc has quit IRC00:40
*** k_mouza has quit IRC01:17
*** k_mouza_ has joined #openstack-ironic01:17
*** k_mouza_ has quit IRC01:20
*** igordc has joined #openstack-ironic01:20
*** igordc has quit IRC01:33
*** rcernin has quit IRC02:02
*** threestrands has joined #openstack-ironic02:06
*** gyee has quit IRC02:09
*** k_mouza has joined #openstack-ironic02:12
*** rloo has quit IRC02:16
*** k_mouza has quit IRC02:16
*** Goneri has quit IRC02:41
*** mkrai has joined #openstack-ironic02:49
*** rcernin has joined #openstack-ironic02:52
*** dsneddon has quit IRC03:06
*** ijw has quit IRC03:29
*** k_mouza has joined #openstack-ironic04:01
*** k_mouza has quit IRC04:06
*** tzumainn has quit IRC04:52
*** subhamrai03 has joined #openstack-ironic04:59
openstackgerritVerification of a change to openstack/ironic failed: Redfish: Add root_prefix to Sushy  https://review.opendev.org/70734205:10
*** jdandrea has quit IRC05:13
*** cp- has quit IRC05:38
*** igordc has joined #openstack-ironic05:38
*** cp- has joined #openstack-ironic05:39
*** TxGirlGeek has quit IRC05:44
*** rachit7 has joined #openstack-ironic05:49
*** subhamrai03 has quit IRC05:51
*** igordc has quit IRC06:11
*** yedongcan has joined #openstack-ironic06:20
*** tkaprol has joined #openstack-ironic06:26
*** threestrands has quit IRC06:34
*** threestrands has joined #openstack-ironic06:35
*** threestrands has quit IRC06:36
*** threestrands has joined #openstack-ironic06:36
*** threestrands has quit IRC06:37
*** threestrands has joined #openstack-ironic06:38
*** khansa has joined #openstack-ironic06:38
*** threestrands has quit IRC06:39
*** hamzy has quit IRC07:04
*** hamzy has joined #openstack-ironic07:05
openstackgerritVerification of a change to openstack/ironic-python-agent failed: Fixes error handling if efibootmgr is not present in ramdisk  https://review.opendev.org/70910707:06
arne_wiebalckGood morning, ironic!07:20
openstackgerritArne Wiebalck proposed openstack/ironic-python-agent master: Software RAID: Add UEFI support  https://review.opendev.org/69615607:35
*** hjensas has quit IRC07:36
* TheJulia looks up from the eastern us time zone and grumbles07:37
TheJulia\o/ insomnia07:37
* arne_wiebalck was about to ask ... sth is not right here07:38
arne_wiebalckTheJulia: dtantsur|afk: I removed the WIP tag on the UEFI RAID patch after testing it on real hardware ^^^, I think this is ready for more reviews now.07:39
TheJuliaarne_wiebalck: awesome, thanks07:40
* TheJulia trie to go back to sleep for a few more hours07:41
TheJuliatries07:41
arne_wiebalckAnswering my question from a couple of weeks ago about a BIOS/UEFI unified image: this is feasible :) We have now an image we can use for VMs + physical nodes, BIOS + UEFI, w/ + w/o s/w RAID ... one image to rule them all.07:41
etingofis not that one known as "hybrid ISO"?07:42
arne_wiebalckTheJulia: Please do! I did not mean you should review this now, just wanted to make sure you're aware later.07:42
arne_wiebalcketingof: "hybrid image" maybe ?07:43
arne_wiebalcketingof: but, yes, it is pretty similar07:44
*** rachit7 has quit IRC07:46
etingofwell, I think it mostly applies to ISO07:46
etingofbecause of the hackerish way how early ISOs were made bootable07:47
etingofit feels like ISOs were not designed to hold bootable code, initially07:47
etingof...ISO implies CD media07:49
*** kozhukalov has quit IRC07:51
TheJuliaarne_wiebalck: no worries, I groked that. Got into responding to emails. About three hours ago. :(07:51
*** diurnalist has joined #openstack-ironic07:53
* TheJulia some days wonders if she should just order the "crazy email lady" shirt go go with the "crazy cat lady" shirt07:53
* TheJulia closes photon sensors07:54
arne_wiebalckgn o/07:54
etingofhave a good sleep!07:55
*** diurnalist has quit IRC07:58
*** tesseract has joined #openstack-ironic08:09
*** mkrai has quit IRC08:09
*** hamzy has quit IRC08:12
*** hamzy has joined #openstack-ironic08:13
*** kozhukalov has joined #openstack-ironic08:13
iurygregorygood morning arne_wiebalck etingof and Ironic o/08:16
*** rpittau|afk is now known as rpittau08:16
rpittaugood morning ironic! o/08:16
iurygregorymorning rpittau o/08:17
rpittauhey iurygregory :)08:18
*** hjensas has joined #openstack-ironic08:21
arne_wiebalckGood morning, iurygregory and rpittau o/08:21
rpittauhey arne_wiebalck :)08:21
*** amoralej|off is now known as amoralej08:22
*** mkrai has joined #openstack-ironic08:22
*** k_mouza has joined #openstack-ironic08:37
*** etingof has quit IRC08:38
*** kozhukalov has quit IRC08:39
*** k_mouza has quit IRC08:50
iurygregoryomg we have code that could be removed after Liberty O.o08:57
rpittauI guess people were just very attached to that piece of code :)09:01
iurygregoryseems lovely the piece of code =)09:04
iurygregoryhttps://opendev.org/openstack/ironic-python-agent/src/branch/master/ironic_python_agent/utils.py#L156-L15809:05
iurygregoryby block of code I assume it's talking about the if condition only...09:06
*** lucasagomes has joined #openstack-ironic09:11
*** dougsz has joined #openstack-ironic09:13
rpittauwell if there's no vmedia device that's the only way to catch it in that function, at least looking at _get_vmedia_device09:15
iurygregoryrpittau, yeah09:15
iurygregoryso it's a bit weird the TODO09:15
iurygregoryI'm not sure we can always assume that will be a vmedia device09:16
openstackgerritMerged openstack/sushy stable/train: Fix incorrect usage of assertRaisesRegex in unit tests  https://review.opendev.org/71097909:21
iurygregoryif any core have some time to review https://review.opendev.org/685888 o/ it has a +2 already =)09:25
patchbotpatch 685888 - ironic-prometheus-exporter - DevStack support and Redfish job - 34 patch sets09:25
*** tkajinam has quit IRC09:30
*** aedc_ has quit IRC09:31
*** hamzy has quit IRC09:34
*** hamzy has joined #openstack-ironic09:34
*** ociuhandu has joined #openstack-ironic09:35
*** derekh has joined #openstack-ironic09:40
openstackgerritRiccardo Pittau proposed openstack/ironic master: Bump minimum supported ansible version to 2.7  https://review.opendev.org/71102109:42
openstackgerritMerged openstack/ironic-tempest-plugin master: Refactor validate_ports function  https://review.opendev.org/71105409:59
*** hamzy has quit IRC10:03
*** khansa has quit IRC10:04
*** ociuhandu has quit IRC10:04
*** hamzy has joined #openstack-ironic10:04
*** hamzy has quit IRC10:06
*** hamzy_ has joined #openstack-ironic10:06
*** dtantsur|afk is now known as dtantsur10:09
dtantsurmorning ironic10:09
iurygregorymorning dtantsur10:11
dtantsurFYI https://bugs.launchpad.net/pyghmi/+bug/186598710:11
openstackLaunchpad bug 1865987 in pyghmi "pyghmi seems to have broken virtualbmc since yesterday" [Undecided,New]10:11
*** khansa has joined #openstack-ironic10:13
rpittauah10:14
dtantsurIt hasn't hit upper-constraints yet, I'm working on an exclusion: https://review.opendev.org/#/c/711193/10:14
patchbotpatch 711193 - requirements - Exclude pyghmi 1.5.11 since it breaks VirtualBMC - 1 patch set10:14
iurygregorynice10:17
*** k_mouza has joined #openstack-ironic10:18
*** ociuhandu has joined #openstack-ironic10:19
rpittauthis week doesn't feel right :/10:19
dtantsurat all10:19
*** k_mouza_ has joined #openstack-ironic10:19
*** hamzy has joined #openstack-ironic10:20
*** hamzy_ has quit IRC10:21
*** k_mouza has quit IRC10:23
*** hamzy has quit IRC10:26
*** hamzy has joined #openstack-ironic10:28
*** hamzy has quit IRC10:33
*** aedc has joined #openstack-ironic10:37
openstackgerritMerged openstack/ironic-python-agent master: Fixes error handling if efibootmgr is not present in ramdisk  https://review.opendev.org/70910710:44
openstackgerritDmitry Tantsur proposed openstack/virtualbmc master: Fix logging in vbmcd  https://review.opendev.org/71120311:01
dtantsurbased on the pyghmi debugging ^^^11:01
*** mkrai has quit IRC11:07
*** ddurst has quit IRC11:08
*** ddurst has joined #openstack-ironic11:08
*** rcernin has quit IRC11:08
*** aedc has quit IRC11:23
*** rpittau is now known as rpittau|bbl11:32
openstackgerritVerification of a change to openstack/ironic failed: Remove grenade jobs from queens branch  https://review.opendev.org/71104311:39
*** openstackstatus has joined #openstack-ironic11:50
*** ChanServ sets mode: +v openstackstatus11:50
*** mkrai has joined #openstack-ironic11:52
openstackgerritDmitry Tantsur proposed openstack/virtualbmc master: DNM test with broken pyghmi  https://review.opendev.org/71120911:59
openstackgerritDmitry Tantsur proposed openstack/virtualbmc master: DNM test with broken pyghmi  https://review.opendev.org/71120912:05
*** aedc has joined #openstack-ironic12:08
*** dsneddon has joined #openstack-ironic12:09
*** aedc has quit IRC12:13
*** mkrai has quit IRC12:16
*** aedc has joined #openstack-ironic12:29
*** johnthetubaguy has joined #openstack-ironic12:32
*** dsneddon has quit IRC12:41
*** mkrai has joined #openstack-ironic12:42
iurygregoryWUT "ERROR: Could not find a version that satisfies the requirement netmiko===2.4.2 (from -c /home/zuul/src/opendev.org/openstack/requirements/upper-constraints.txt (line 7)) (from versions: none)"12:45
*** TheJulia has quit IRC12:54
*** TheJulia has joined #openstack-ironic12:54
*** vdrok has quit IRC12:54
*** vdrok has joined #openstack-ironic12:55
TheJuliaWell that doesn't sound good13:01
*** hjensas has quit IRC13:11
*** Goneri has joined #openstack-ironic13:16
iurygregoryTheJulia, seems link infra problems with the mirrors13:17
*** mkrai has quit IRC13:17
dtantsurmorning TheJulia13:19
dtantsuriurygregory: it happens from time to time..13:19
iurygregorydtantsur, first time I saw =)13:19
*** etingof has joined #openstack-ironic13:21
TheJuliagood morning13:23
*** vdrok has quit IRC13:23
*** amoralej is now known as amoralej|lunch13:24
*** NobodyCam has quit IRC13:24
*** vdrok has joined #openstack-ironic13:25
*** NobodyCam has joined #openstack-ironic13:25
*** hjensas has joined #openstack-ironic13:27
openstackgerritDmitry Tantsur proposed openstack/virtualbmc master: DNM test with pyghmi  https://review.opendev.org/71120913:31
*** jdandrea has joined #openstack-ironic13:35
openstackgerritMerged openstack/ironic master: Set abstract for ironic-base  https://review.opendev.org/71101613:40
openstackgerritMerged openstack/ironic master: Redfish: Add root_prefix to Sushy  https://review.opendev.org/70734213:40
openstackgerritDmitry Tantsur proposed openstack/virtualbmc master: DNM test with pyghmi  https://review.opendev.org/71120913:41
iurygregoryok this is getting funny... Looking in indexes: http://mirror.regionone.limestone.openstack.org/pypi/simple, http://mirror.regionone.limestone.openstack.org/wheel/ubuntu-18.04-x86_6413:42
iurygregoryoh god13:42
*** rpittau|bbl is now known as rpittau13:44
* TheJulia thinks nap is required13:45
dtantsurtotally required13:45
TheJuliaone half hour meeting then I can take a nap13:46
dtantsurI guess the same meeting I have13:46
iurygregory++13:46
TheJuliadtantsur: indeed13:46
dtantsura nap would be good if I could sleep normally..13:46
*** amoralej|lunch is now known as amoralej13:53
openstackgerritMerged openstack/ironic stable/rocky: [stable] consume virtualbmc from pip packages  https://review.opendev.org/71076613:55
dtantsuryay!13:58
TheJuliawoot14:00
*** mkrai has joined #openstack-ironic14:02
*** rloo has joined #openstack-ironic14:03
openstackgerritVerification of a change to openstack/ironic failed: tell reno to ignore the kilo branch  https://review.opendev.org/70678614:09
*** aedc has quit IRC14:16
*** ricolin has quit IRC14:25
*** yedongcan has left #openstack-ironic14:28
*** ociuhandu has quit IRC14:40
*** dtantsur is now known as dtantsur|brb14:46
*** tzumainn has joined #openstack-ironic14:53
*** mkrai has quit IRC14:54
*** mmethot has quit IRC14:54
*** priteau has joined #openstack-ironic14:55
*** rh-jelabarre has joined #openstack-ironic14:59
*** ociuhandu has joined #openstack-ironic15:13
*** Lucas_Gray has joined #openstack-ironic15:15
*** ociuhandu has quit IRC15:18
*** aedc has joined #openstack-ironic15:20
*** Lucas_Gray has quit IRC15:23
*** hjensas has quit IRC15:24
*** Lucas_Gray has joined #openstack-ironic15:25
openstackgerritMerged openstack/ironic stable/queens: Remove grenade jobs from queens branch  https://review.opendev.org/71104315:29
*** Lucas_Gray has quit IRC15:29
*** Lucas_Gray has joined #openstack-ironic15:31
*** aedc has quit IRC15:53
*** ociuhandu has joined #openstack-ironic15:56
*** mkrai has joined #openstack-ironic15:56
*** dtantsur|brb is now known as dtantsur16:00
*** diurnalist has joined #openstack-ironic16:07
*** n4i9 has joined #openstack-ironic16:09
*** ociuhandu has quit IRC16:11
n4i9Hello everyone! I am Nikita from IIT BHU,India.I am an Outreachy aspirant and wish to apply to the project for Redfish CLI listed on the Outreachy website.I would be grateful if I could be assigned a bug to solve as my first contribution to Openstack.Thanks in advance!16:12
TheJuliaGreetings n4i916:13
rpittauwelcome n4i9 :)16:14
TheJuliaetingof: It looks like there is nothing under sushy-cli on storyboard.openstack.org. Is there anything that your aware of where you can point n4i9 in a direction to get started?16:15
*** ociuhandu has joined #openstack-ironic16:16
*** iurygregory has quit IRC16:20
TheJulian4i9: I suspect etingof may have left for the day already, if you remain connected to IRC they will hopefully respond tomorrow their time.16:27
dtantsurhe also tends to be present in (his) evenings16:29
*** ricolin has joined #openstack-ironic16:37
dtantsursigh. the requirements check is actually broken on train. what a lovely week...16:38
*** Lucas_Gray has quit IRC16:38
*** gyee has joined #openstack-ironic16:40
*** ociuhandu has quit IRC16:41
openstackgerritRiccardo Pittau proposed openstack/ironic-python-agent master: Rescan after restarting the md device  https://review.opendev.org/70663216:45
openstackgerritDmitry Tantsur proposed openstack/ironic stable/train: DNM test check-requirements  https://review.opendev.org/71127216:45
khansaHey n4i9! I think It may be these tasks https://storyboard.openstack.org/#!/story/2006610  but still need to confirm with etingof16:50
*** ociuhandu has joined #openstack-ironic16:51
*** bdodd has joined #openstack-ironic16:56
* etingof is still alive, despite life17:02
etingoflet me update storyboard on sushy-cli17:02
TheJuliaawesome17:02
*** lucasagomes has quit IRC17:03
dtantsuretingof now https://pics.me.me/the-light-inside-has-broken-but-i-still-work-20092579.png ?17:03
* dtantsur is like that ~ every day17:03
*** mkrai has quit IRC17:04
etingofhow many engineers does it take to replace dtantsur's lightbulb...17:05
* dtantsur sad smile17:05
dtantsurworking CI on all branches would help a tiny bit17:06
TheJuliaCI working would be forever happiness17:09
*** openstackgerrit has quit IRC17:20
rpittauwell gerrit quit17:20
TheJuliaheh17:26
*** n4i9 has quit IRC17:27
mgoddardrpittau: hi17:29
rpittaumgoddard: hey17:29
mgoddardrpittau: are you still working on https://review.opendev.org/#/c/703954/17:30
patchbotpatch 703954 - bifrost - Make CentOS 8 tinyipa job voting again - 27 patch sets17:30
arne_wiebalckbye, everyone o/17:30
rpittaumgoddard: yes, there's a bug in libvirt so we're stuck on that17:30
rpittaumgoddard: got some info from here: https://review.opendev.org/70716217:31
patchbotpatch 707162 - bifrost - [DNM] getting some libvirt info - 7 patch sets17:31
rpittaumgoddard: this is the bug https://bugzilla.redhat.com/show_bug.cgi?id=180422417:32
openstackbugzilla.redhat.com bug 1804224 in libvirt "libvirtd: error : virCPUx86UpdateLive:3110 : operation failed: guest CPU doesn't match specification: missing features: fxsr_opt" [High,Modified] - Assigned to jdenemar17:32
*** igordc has joined #openstack-ironic17:33
*** jcoufal has joined #openstack-ironic17:33
mgoddardrpittau: weird I think we had ironic deploys working on CentOS 8 in CI with kayobe & tenks17:33
mgoddardrpittau: how would you feel about splitting out the non CI stuff?17:34
rpittaumgoddard: the bug seems confirmed :/17:34
mgoddardI'd like to get the bifrost image building in kolla17:34
mgoddardcould be a different version of libvirt17:34
* TheJulia tries to collalesce words17:35
rpittaumgoddard: sure thing, we can split some stuff if needed17:36
dtantsurif anybody has a minute: https://review.opendev.org/#/c/711203/17:36
patchbotpatch 711203 - virtualbmc - Fix logging in vbmcd - 1 patch set17:36
mgoddardrpittau: cool. there is also the ${VENV-} thing17:36
mgoddardin two other patches already :)17:36
rpittaumgoddard: can you point me to that please ?17:37
mgoddardrpittau: https://review.opendev.org/#/c/706002/117:37
patchbotpatch 706002 - bifrost - Fix "unbound variable" error in scripts - 1 patch set17:37
mgoddardlink to the other in that one17:37
rpittaumgoddard: ok, well, they're not updated since a month :/17:38
mgoddardrpittau: I'll update17:39
rpittauok17:39
rpittauI think we should just abandon one of them17:39
*** dtantsur is now known as dtantsur|afk17:41
dtantsur|afkc u17:41
rpittaumgoddard: I will work on the splitting part tomorrow17:41
rpittaubye all! o/17:44
*** rpittau is now known as rpittau|afk17:44
*** openstackgerrit has joined #openstack-ironic17:45
openstackgerritIlya Etingof proposed openstack/ironic master: Improve `redfish` set-boot-device behaviour  https://review.opendev.org/71111817:45
mgoddardcool thanks rpittau|afk. Let me know if you need anythin17:45
* etingof got a lucky patch number - 71111817:46
*** amoralej is now known as amoralej|off17:51
*** ociuhandu_ has joined #openstack-ironic17:54
*** hjensas has joined #openstack-ironic17:55
*** ociuhandu has quit IRC17:57
*** ociuhandu_ has quit IRC17:58
*** priteau has quit IRC17:59
*** dougsz has quit IRC18:00
*** derekh has quit IRC18:01
*** igordc has quit IRC18:04
*** igordc has joined #openstack-ironic18:05
*** ijw has joined #openstack-ironic18:18
*** ijw has quit IRC18:18
*** ijw has joined #openstack-ironic18:19
*** dsneddon has joined #openstack-ironic18:19
*** ijw_ has joined #openstack-ironic18:20
*** ijw has quit IRC18:23
*** igordc has quit IRC18:27
TheJulianice!18:31
openstackgerritVerification of a change to openstack/ironic failed: tell reno to ignore the kilo branch  https://review.opendev.org/70678618:31
*** igordc has joined #openstack-ironic18:33
*** jcoufal_ has joined #openstack-ironic18:51
*** jcoufal has quit IRC18:52
*** tesseract has quit IRC18:55
*** k_mouza_ has quit IRC18:56
*** khansa has quit IRC19:15
*** dsneddon has quit IRC19:20
*** dmellado has quit IRC19:24
*** dmellado has joined #openstack-ironic19:25
*** mgoddard has quit IRC19:27
*** mgoddard has joined #openstack-ironic19:28
*** aedc has joined #openstack-ironic19:31
*** dsneddon has joined #openstack-ironic19:34
*** iurygregory has joined #openstack-ironic19:46
*** khansa has joined #openstack-ironic19:53
openstackgerritIury Gregory Melo Ferreira proposed openstack/ironic-python-agent stable/train: Fixes error handling if efibootmgr is not present in ramdisk  https://review.opendev.org/71132619:58
*** khansa has quit IRC20:33
openstackgerritMerged openstack/ironic stable/queens: [stable] consume virtualbmc from pip packages  https://review.opendev.org/71100520:38
*** iurygregory has quit IRC20:43
*** shadower has joined #openstack-ironic20:44
stevebakerdtantsur|afk: here is the end of the series which reimplements the baremetal provisioning in ansible https://review.opendev.org/#/c/707044/21:04
patchbotpatch 707044 - tripleo-ansible - Playbooks for overcloud node (un)provision - 19 patch sets21:04
*** ociuhandu has joined #openstack-ironic21:06
*** ociuhandu has quit IRC21:10
openstackgerritJulia Kreger proposed openstack/ironic master: WIP: change boot mode to uefi by default  https://review.opendev.org/71133921:14
TheJuliaumm... boom... maybe?21:15
TheJulia44 tests fail locally21:15
TheJuliabut... maybe integration jobs will work?!?21:15
diurnalisto/ Hey all - has anybody else run into the issue of linux-firmware not being included in the default IPA DIB image (built with the -builder repo)? i have bare metal with NICs that have their firmware installed via this package, just discovered this when trying to build my own IPA21:18
clarkbdiurnalist: when you run the disk-image-create command you can provider -p package1,package2,package3 to install additional packages21:22
diurnalistDoes that work even though the linux-firmware package is explicitly _uninstalled_? as in, it is removed by an element?21:23
clarkbthats probably the best way to apply extra pacakges to the image, then IPA can document this system forpeople that need extra things installed21:23
clarkbhrm, that will depend on the ordering. The -p package list is run between install.d and post-install.d scripts21:23
clarkbdiurnalist: can you point to where the removal happens?21:23
clarkbI think I found it21:24
diurnalisthttps://github.com/openstack/diskimage-builder/blob/master/diskimage_builder/elements/yum-minimal/package-installs.yaml#L10-L1321:24
diurnalistsorry, should've used an opendev link21:24
diurnalistyum-minimal is required by centos-minimal21:25
clarkbthe package-installs element does uninstalls as install.d/99-package-uninstalls21:25
clarkbIf I'm reading when the -p args are processed then yes supplying -p linux-firmware will cause it to be installed after the uninstall21:25
clarkb(not super efficient, but should fix the issue)21:25
openstackgerritMerged openstack/ironic-prometheus-exporter master: DevStack support and Redfish job  https://review.opendev.org/68588821:26
openstackgerritMerged openstack/ironic-prometheus-exporter master: Fix None for instance_uuid label  https://review.opendev.org/70634621:26
diurnalistthanks clarkb, i will try that. I was wondering if it makes sense to put this into IPA, given that this disk image is supposed to target bare metal deployments21:26
clarkbya it may. In which case you'd still need to have it run after install.d/99-package-uninstalls, whihc is doable21:27
dking_desktopCould somebody tell me how I might go about troubleshooting using genericswitch (Networking Generic Switch) for use with baremetal? I tried to follow the documentation, but I suspect that I did things incorrectly.21:27
diurnalistit seems like the choice to remove linux-firmware in yum-minimal was somewhat recent (5 months ago)21:27
clarkbdiurnalist: ianw may also have some thoughts21:27
ianwclarkb/diurnalist: i feel like there is, or has been, patches around linux-firmware21:29
dking_desktopI'm not even certain how the local_link_connection settings associate the port with a particular genericswitch device. It seems that the only variable shared is the MAC address.21:29
ianwahh it's cgoncalves @ https://github.com/openstack/diskimage-builder/commit/f909000e5a63eb27c711de2b5af871978d5c27a8 i remember now21:31
diurnalistdking_desktop: the other main thing the local_link_connection has is "switch_info", which refers to the named switch entity in the NGS config. you also provide the switchport as port_id. this allows NGS to know which switch to update, and which switchport on the switch, for a given node21:42
*** jcoufal_ has quit IRC21:42
diurnaliste.g., {'switch_info': 's6000_tor_1', 'port_id': 'Te 1/4/1'}21:43
diurnalistthen in your config, you'd have your switch config in section like [genericswitch:s6000_tor_1] with the host/username/password it uses to log in21:44
*** KeithMnemonic has joined #openstack-ironic21:44
diurnalistNGS then has all the info it needs. it knows which switchport to configure, and which switch it's on, and how to access that switch21:44
KeithMnemonicis there anyone here who might be able to provide me some details of how direct mode works for the ironic-python-agent?  i.e i know it tries to pull from swift but was looking for some more details.21:45
dking_desktopdiurnalist: Thank you! So, the switch_info can be the name of the entry, such as "test_switch" if my ml2 config has "[genericswitch:test_switch]" ?21:45
diurnalistdking_desktop: re: debugging, if you enable the 'debug' setting on neutron-server, it should print lots of debug information about NGS, including stdout from the remote SSH session on the switch21:45
diurnalistdking_desktop: exactamundo21:45
diurnalistapparently one can also use ngs_mac_address to identify the switch, then it doesn't look in the Ironic port. i'm not sure how this works exactly, and have not used it myself. https://docs.openstack.org/networking-generic-switch/latest/configuration.html21:47
diurnalistbut i think you still need port_id regardless21:47
dking_desktopI wonder if that could be made more clear on https://docs.openstack.org/ironic/train/admin/multitenancy.html. I didn't make that connection here. It says this for "switch_info": "Optional. Used to distinguish different switch models or other vendor-specific identifier. Some ML2 plugins may require this field." And I didn't see that listed anywhere that I saw in the Networking Generic Switch documentation, tho21:47
dking_desktoperlooked it.21:47
diurnalistdking_desktop: switch_info is briefly mentioned on the config page i just linked, but it is not obvious21:48
diurnalistit could definitely be more clear :)21:48
dking_desktopOh, okay. I see that. I did use the MAC. Assuming that this is set and it's still not finishing the server BUILD, where could I look in the logs to try to troubleshoot?21:50
*** jdandrea has quit IRC21:51
dking_desktopThe only thing I saw in the logs was when I saw the "test_switch" device registered by neutron-server. I searched all the logs for a mention of the port name and MAC address, and I didn't get any hits.21:52
diurnalisthave you enabled debug logs on neutron-server? you should see NGS attempting to log in to the switch and do the port plugging21:54
diurnalistby default those are hidden though, because they are very chatty21:55
dking_desktopOh, sorry. I missed that part about enabling the debug setting. I suppose I should do that.21:55
JayF\21:55
diurnalisttry that out, i am pretty sure it is required to get more output to see what it's doing21:55
*** hwoarang has quit IRC22:08
*** hwoarang has joined #openstack-ironic22:10
dking_desktopdiurnalist: It takes a long time for the build to timeout, so I'm not sure when I should see something in the logs, but I'm still not seeing anything that I would expect. I'm deploying with kolla-ansible. Would the neutron_server be the right service to set the debug setting and check for the log entries?22:26
dking_desktopIt's the only place I see that looks relevant, and it did seem that it was the service recommended in one article I had come across, but I'm not sure I'm seeing anything specific yet.22:26
*** rh-jelabarre has quit IRC22:26
TheJuliaKeithMnemonic: downloads from swift or an http(s) url directly by the running agent, and can be either streamed to the local disk or the disk partitioned and then streamed into the partition depending on the image type22:34
KeithMnemonicso here is what i see, it seems to stream to disk correctly, and if i try manuall via curl it d/l the image from swift22:34
KeithMnemonicthe logs show it streamed to /dev/sda correctly and then does a hexdump22:35
KeithMnemonicso all seems good22:35
KeithMnemonicuntil it reboots and it can not boot off of the disk22:35
KeithMnemonicis there any other verification to see if /dev/sda is populated correctly22:35
TheJuliaKeithMnemonic: depends on the version of the agent. Have you taken the image and confirmed there is a partition table or a boot loader block at the beginning of the image? Also what boot mode is the machine in?22:37
TheJuliaKeithMnemonic: just sounds like a mismatch, tbh22:38
*** rcernin has joined #openstack-ironic22:39
*** alexmcleod has quit IRC22:42
KeithMnemonicboot mode is direct/bios. this image works fine on other HPE servers DL360, but not on a DL160 so we think it is something with a driver. is there anyway to do more checks on /dev/sda before i reboot it22:47
openstackgerritMerged openstack/networking-generic-switch stable/train: Fix stable/train CI  https://review.opendev.org/70914622:49
KeithMnemonicfor example i see a "hexdump" in the logs. What is it trying to get from that?22:49
KeithMnemonicthis is after the stream/write is completed22:50
*** tkajinam has joined #openstack-ironic22:59
TheJuliaKeithMnemonic: is this one of the HPE servers that only UEFI boots?23:03
TheJuliahexdump is trying to identify the partition id23:04
KeithMnemonicthis is a server that does both, but we have them in bios23:04
KeithMnemonicshould it be 00000000? the partition_id?23:04
TheJuliathat just signals it is a whole disk image23:04
KeithMnemonicok which it is23:04
TheJuliawhen is the ramdisk from?23:04
TheJuliai.e. is it train, or master branch code?23:05
TheJuliais /dev/sda on the machine the correct device?23:05
* TheJulia wonders if somehow it is thinking it shoudl be booting from a different device23:06
TheJuliathe image working on a dl360 makes me think it is something like that23:07
TheJuliaout of curiosity, what driver are you using to talk to the bmc?23:07
KeithMnemoniclol, rocky, this is ipmi23:11
*** threestrands has joined #openstack-ironic23:28
*** Goneri has quit IRC23:36
TheJuliawell... interesting23:40
TheJuliaKeithMnemonic: interesting because we recently made changes to that. We've run into some situations recently where we found that we actually needed to update uefi nvram23:41
*** Goneri has joined #openstack-ironic23:42
KeithMnemonicTheJulia: what is interesting also is on these same 160s we get a stacktrace/core when using iscsi mode23:45
KeithMnemonicit is some sort of odd kernel thing but again only happens on the 160s23:46
KeithMnemonicand it is more than one firmware version as someone from HPE reported this last summer even before the FW version I am currently using was released23:46
TheJuliaif your in a ramdisk on these machines, what does the device list look like?23:46
TheJuliasweet23:47
TheJuliaif you check the ilo, what does it report as the boot device? Curious if it is the same disk or if there are multiple disks23:47
KeithMnemonicI can get back to a ramdisk and check. I can also compare to a 360. Another test I will do is try direct on the 360. just to make sure it works there. So far the 360s are fine on iscsi so I did not need to try direct.23:48
* TheJulia runs down to the hotel bar to get food() since she has to be up in 9 hours at at the airport in 11 hours23:49
KeithMnemonicThere is only one disk, these use the HPE 240 Smart Array so I have 4 drives configured as one LUN23:49
TheJuliaKeithMnemonic: oh... hmmmm23:49
TheJuliawelll... it shouldn't be /dev/sda then, it should be something like /dev/cciss/c023:50
KeithMnemonicI think the 360s have a different smart array card23:50
TheJuliaahhhh23:50
TheJuliaoh23:50
TheJuliait could be a... ATARAID device23:50
* TheJulia had to think what it was23:50
* TheJulia grabs tablet to continue chatting23:51
KeithMnemonicthe 360s have a 440 smart array23:51
KeithMnemonicso it maybe something with the SLES driver.23:51
KeithMnemonicI need to take a dinner break, Thanks for all of the good tips. e can chat again once I have more data points.23:52
KeithMnemonicand if you think of anything, just post it here23:53
*** diurnalist has quit IRC23:55
TheJuliaKeithMnemonic: nothing really comes to mind, but those controllers can be dramatically different :\. I’m curious if the running OS sees the base devices which is why /dev/sda is being chosen instead of a composite device, but the deploy logs and debugging info should reveal all the details in the files23:59

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