Wednesday, 2019-09-04

* TheJulia has lots to review tomorrow morning00:10
*** gyee has quit IRC00:21
*** altlogbot_1 has quit IRC00:23
*** irclogbot_0 has quit IRC00:23
*** threestrands has quit IRC00:32
*** Goneri has quit IRC01:02
*** rloo has quit IRC01:05
*** igordc has quit IRC01:16
*** irclogbot_1 has joined #openstack-ironic01:34
*** irclogbot_1 has quit IRC01:39
*** irclogbot_3 has joined #openstack-ironic01:40
*** irclogbot_3 has quit IRC01:43
*** irclogbot_2 has joined #openstack-ironic02:54
*** irclogbot_2 has quit IRC02:59
*** rh-jelabarre has quit IRC03:04
*** ianychoi has quit IRC03:12
*** ianychoi has joined #openstack-ironic03:13
*** irclogbot_0 has joined #openstack-ironic03:34
*** irclogbot_0 has quit IRC03:39
*** mkrai has joined #openstack-ironic03:49
*** gkadam has joined #openstack-ironic03:59
*** gkadam_ has joined #openstack-ironic04:01
*** gkadam_ has quit IRC04:01
*** irclogbot_2 has joined #openstack-ironic04:02
*** gkadam has quit IRC04:03
*** irclogbot_2 has quit IRC04:07
*** ricolin_ has joined #openstack-ironic04:08
*** ricolin_ has quit IRC04:08
*** ricolin has joined #openstack-ironic04:10
openstackgerritShivanand Tendulker proposed openstack/ironic master: iLO driver doc update  https://review.opendev.org/67970204:15
*** irclogbot_0 has joined #openstack-ironic04:22
openstackgerritMerged openstack/ironic-lib master: Deprecate [disk_utils]iscsi_verify_attempts  https://review.opendev.org/67755804:31
openstackgerritMerged openstack/ironic master: Add a release note for iscsi_verify_attempts deprecation  https://review.opendev.org/67979204:31
*** irclogbot_0 has quit IRC04:33
*** rachit7 has joined #openstack-ironic05:17
*** gregoryo has joined #openstack-ironic05:20
*** irclogbot_3 has joined #openstack-ironic05:20
*** irclogbot_3 has quit IRC05:25
*** rcernin_ has joined #openstack-ironic05:31
*** rcernin has quit IRC05:33
openstackgerritMaksim Malchuk proposed openstack/networking-baremetal master: Add the option to choose the Ironic endpoint type  https://review.opendev.org/67081205:47
*** michchap has quit IRC05:58
*** hjensas has quit IRC06:01
*** mbeierl has quit IRC06:03
*** jtomasek has joined #openstack-ironic06:06
*** ricolin_ has joined #openstack-ironic06:15
*** irclogbot_2 has joined #openstack-ironic06:16
*** ricolin has quit IRC06:17
*** irclogbot_2 has quit IRC06:21
*** irclogbot_2 has joined #openstack-ironic06:24
*** irclogbot_2 has quit IRC06:29
arne_wiebalckGood morning, ironic!06:32
*** diga has joined #openstack-ironic06:54
*** rcernin_ has quit IRC07:01
rachit7Good morning arne_wiebalck07:04
*** stendulker has joined #openstack-ironic07:05
arne_wiebalckHey rachit7 o/07:09
openstackgerritAnkit Kumar proposed openstack/ironic master: Add iso less virtual media deploy  https://review.opendev.org/67799607:11
*** toabctl has quit IRC07:11
*** hjensas has joined #openstack-ironic07:15
*** hjensas has quit IRC07:17
*** toabctl has joined #openstack-ironic07:19
*** irclogbot_2 has joined #openstack-ironic07:30
*** irclogbot_2 has quit IRC07:35
openstackgerritparesh sao proposed openstack/ironic master: Follow-up patch for out-of-band `erase_devices` clean step for iLO  https://review.opendev.org/67992707:36
*** pcaruana has joined #openstack-ironic07:42
*** gregoryo has quit IRC07:43
*** lucasagomes has joined #openstack-ironic07:47
*** irclogbot_1 has joined #openstack-ironic07:50
*** irclogbot_1 has quit IRC07:53
*** hjensas has joined #openstack-ironic07:54
*** ociuhandu has joined #openstack-ironic07:57
*** e0ne has joined #openstack-ironic07:59
*** ociuhandu has quit IRC07:59
*** ociuhandu has joined #openstack-ironic08:00
*** hwoarang has quit IRC08:05
*** hwoarang has joined #openstack-ironic08:07
*** e0ne has quit IRC08:07
*** rachit7 has quit IRC08:21
*** irclogbot_2 has joined #openstack-ironic08:22
*** dougsz has joined #openstack-ironic08:23
*** ricolin_ is now known as ricolin08:25
*** irclogbot_2 has quit IRC08:25
*** e0ne has joined #openstack-ironic08:35
*** derekh has joined #openstack-ironic08:36
*** mkrai has quit IRC08:39
*** brtknr has joined #openstack-ironic08:41
brtknrhey all, anyone out there booting up #fedora-coreos on baremetal?08:41
brtknri am strugging to inject ignition metadata to the baremetal instance... which works fine on VM08:42
*** alexmcleod has joined #openstack-ironic08:44
*** priteau has joined #openstack-ironic08:45
*** ociuhandu has quit IRC08:55
*** ftarasenko has joined #openstack-ironic08:55
*** ociuhandu has joined #openstack-ironic08:56
ftarasenkoDear team, can you point me to documentation/code regarding port groups in Ironic used with Nova? Currently I get empty network_data.json in config drive.08:57
ftarasenkoShould the config drive be created manually or I miss something in my configuration?08:57
*** ociuhandu has quit IRC09:00
*** trident has quit IRC09:01
*** irclogbot_2 has joined #openstack-ironic09:08
*** trident has joined #openstack-ironic09:10
*** irclogbot_2 has quit IRC09:11
*** stendulker has quit IRC09:16
*** dsneddon_ has joined #openstack-ironic09:17
*** dsneddon has quit IRC09:18
*** dtantsur|afk is now known as dtantsur09:22
dtantsurftarasenko: I think nova is supposed to create network_data.json, but I don't know this code really well. hjensas, mgoddard?09:23
dtantsurmorning ironic09:23
xXraphXxdtantsur, ftarasenko: hi, yep nova does it but there are some cases when it cannot fill the data, becasue they are not available at the time the configdrive is built (with neutron segments activated for example, or in any case when the port ip allocation is deferred)09:25
dtantsurindeed09:26
dtantsurmorning (?) xXraphXx09:26
ftarasenkohaha, I definetely have segments in my configuration. What should be done in this case - manual built of config drive or we need to patch nova?09:26
ftarasenkoxXraphXx have you seen any working solution? I think, for tripleo manual built of config drives is used.09:29
dtantsurtripleo uses os-net-config for network configuration09:30
xXraphXxftarasenko. No, not seen any upstream solution for now, unless I'm wrong :/. Not sure what the upstream solution should be: may be write a neutron custom ipam ? on my side, i've made sure the data are statically stored on the ironic node driver info. And made a nova driver to override the network metadata. But this is so ugly I did not push it upstream09:31
dtantsurwell, there is a thing in progress for ironic-side mini-IPAM09:33
dtantsurmaybe we could integrate it with nova as well09:33
dtantsurhttp://specs.openstack.org/openstack/ironic-specs/specs/not-implemented/L3-based-deployment.html (etingof is updating it)09:33
* etingof is interested in real-world use-cases09:34
*** ricolin has quit IRC09:34
*** ricolin has joined #openstack-ironic09:35
xXraphXxftarasenko, thinking about some solution that might be a solution for you (or not, depending on your use case): if you can make sure that you specify a fixed ip in all your nova boot, then you will bypass the ip allocation deferred and thus bypass the problem09:36
hjensassounds like nova would need to build the config-drive after the instance has been scheduled to a host, the port is bound and neutron can allocate an IP?09:37
xXraphXxhjensas, yep, once an ironic node is bound to the instance port (after vif_attach call to ironic, if not mistaken)09:38
dtantsurbut we're binding a port quite late in the deployment09:41
dtantsurwe may need a new API endpoint for nova to use09:41
xXraphXxyep in driver.spawn09:41
xXraphXx(was answering to the quite late assertion)09:42
*** ftarasenko has quit IRC09:42
dtantsurit would be great if somebody who has a clue what is going on could drive this work further09:43
xXraphXxmy bad it's a bit before driver.spawn it's in plug_vifs09:44
dtantsurdoes port binding actually happen there?09:44
*** ftarasenko has joined #openstack-ironic09:44
dtantsurI think we attach VIFs, but I don't remember if we actually call into neutron to bind ports09:44
dtantsurI suspect it may only be done in ironic's boot interface09:45
xXraphXxchecking in ironic09:45
openstackgerritDmitry Tantsur proposed openstack/ironic master: Correctly set OS_TEST_TIMEOUT in the multinode CI job  https://review.opendev.org/67994909:48
xXraphXxdtantsur, you're right09:48
dtantsurxXraphXx: we could probably make ironic rebuild network_data.json09:49
dtantsurAfter all, we're already in the business of building config drives09:49
* dtantsur needs more opinions on ^^^09:49
*** rachit7 has joined #openstack-ironic09:49
xXraphXxbinding is, depending on the driver, in the deployment end (def configure_tenant_networks). rebuild network_data could be a good solution, if it's planned for ironic to tamper with the configdrive09:51
dtantsurwe can open a discussion on it. we have a valid use case now.09:53
*** diga has quit IRC09:54
ftarasenkoxXraphXx dtantsur thank you for explanation, try to create workaround that will work in my setup. Not so good in python/openstack to discuss how it should be integrated with upstream, but the solution of rebuilding config drive looks good to me.09:55
hjensasdtantsur: won't we need to write the config-driver prior to calling configure_tenant_networks? There will be a vif attach to the provisioning network, which may trigger ml2 configuring switch etc. Once configure_tenant_networks happens the ml2 will reconfigure. We would have to be done generating and writing the config-drive at that point.09:55
dtantsurouch09:55
dtantsuryes, you're right09:56
dtantsursounds like someone should start an ML thread to collect more opinions09:56
* hjensas wonder if we could use placement+neutron info to update the port with a fixed-ip prior to doing the bind.09:57
hjensasML ++09:57
*** hjensas is now known as hjensas|lunch09:57
dtantsurwell, when configdrive is written, we're still attached to a provisioning network with a different IP09:58
* dtantsur wonders why the port doesn't end up being bound before nova calls us09:58
*** irclogbot_1 has joined #openstack-ironic10:28
*** irclogbot_1 has quit IRC10:33
*** irclogbot_1 has joined #openstack-ironic10:42
*** irclogbot_1 has quit IRC10:45
*** irclogbot_3 has joined #openstack-ironic10:58
*** ociuhandu has joined #openstack-ironic11:01
*** irclogbot_3 has quit IRC11:02
etingofif rpioso o/ got a link to the recommended way of flipping boot mode for virtual media boot - that would be helpful11:02
*** trident has quit IRC11:04
*** dougsz has quit IRC11:07
*** hjensas|lunch is now known as hjensas11:09
*** trident has joined #openstack-ironic11:13
*** alexmcleod has quit IRC11:13
*** alexmcleod has joined #openstack-ironic11:14
openstackgerritDmitry Tantsur proposed openstack/ironic master: devstack: save iPXE httpd logs  https://review.opendev.org/67996811:16
*** mkrai has joined #openstack-ironic11:17
*** alexmcleod has quit IRC11:21
*** lucasagomes has quit IRC11:25
*** lucasagomes has joined #openstack-ironic11:25
*** bfournie has quit IRC11:27
openstackgerritDmitry Tantsur proposed openstack/ironic master: WIP ipv6 jobs  https://review.opendev.org/61316911:28
*** rh-jelabarre has joined #openstack-ironic11:29
*** ftarasenko has quit IRC11:39
*** irclogbot_2 has joined #openstack-ironic11:42
*** rachit7 has quit IRC11:44
*** irclogbot_2 has quit IRC11:48
*** alexmcleod has joined #openstack-ironic12:03
*** tesseract has joined #openstack-ironic12:03
*** bfournie has joined #openstack-ironic12:18
dtantsurmgoddard: are there outstanding n-g-s patches that we need to land in train?12:24
openstackgerritDmitry Tantsur proposed openstack/ironic-ui master: Update various links in the documentation  https://review.opendev.org/61780312:24
dtantsurupdated, please somebody review ^^^12:24
*** dougsz has joined #openstack-ironic12:26
dtantsurhjensas: similar question re networking-baremetal: do we need anything in train?12:31
hjensasdtantsur: https://review.opendev.org/673972 ? I have a couple of other changes I should pick up when I have time, but nothing we need in train afik.12:36
patchbotpatch 673972 - networking-baremetal - Blacklist sphinx 2.1.0 (autodoc bug) - 2 patch sets12:36
*** iurygregory has joined #openstack-ironic12:36
dtantsurok, thanks12:36
iurygregorygood morning all12:36
dtantsurhi iurygregory12:37
dtantsuriurygregory: anything we need in the exporter for train (except for the descriptions patch)?12:37
iurygregorydtantsur, I think we may want to add the initial support for RedFish metrics right etingof12:37
iurygregory?12:38
etingofiurygregory, yes, I plan to work on this hopefully by the end of this week12:38
etingofdtantsur, ^12:40
dtantsuretingof: it really has to happen by the end of this week or it'll miss train12:40
openstackgerritDmitry Tantsur proposed openstack/bifrost master: DNM checking CI health  https://review.opendev.org/67998112:41
etingofdtantsur, yeah, that's tight12:42
dtantsuretingof: yep. do you think it's doable and you will have cycles for it?12:42
etingofdtantsur, is it Friday or Monday?12:42
etingofdtantsur, well, the plan is to add some rudimentary parser for redfish metrics12:42
etingofdtantsur, then improve that by bugfixes12:42
dtantsurI mean.. given that exporter is not used anywhere, I'm fine with not applying feature freeze to it12:43
dtantsurbut if you need to change ironic, the patches have to be up, say, by Monday meeting12:43
dtantsurand if you need to change sushy, by EOD today12:43
etingofdtantsur, I think ironic and sushy parts are all in place for the metrics for now12:48
dtantsurokay, then the schedule is less tight. but let's have this work wrapped up by end of next week please.12:50
*** ociuhandu has quit IRC12:50
dtantsuretingof: what's the status of sushy-tools now?12:54
*** ociuhandu has joined #openstack-ironic12:55
openstackgerritMerged openstack/networking-baremetal master: Blacklist sphinx 2.1.0 (autodoc bug)  https://review.opendev.org/67397212:55
*** mkrai has quit IRC12:56
*** mbeierl has joined #openstack-ironic13:11
openstackgerritDmitry Tantsur proposed openstack/ironic master: Release notes cleanup for 13.0.0  https://review.opendev.org/67998513:15
etingofdtantsur, I just +2 sushy-tools patches that I think are good to go13:15
etingofdtantsur, plus this patch of mine I'd suggest to review -- https://review.opendev.org/#/c/679456/13:16
patchbotpatch 679456 - sushy-tools - Fix non-functional driver configuration on Py2 - 1 patch set13:16
dtantsurk, thx13:16
*** mbeierl has quit IRC13:17
etingofdtantsur, I'd love to merge this patch as well, though there are some nits to address -- https://review.opendev.org/#/c/677119/13:17
patchbotpatch 677119 - sushy-tools - Add Volume resource support - 5 patch sets13:17
*** rachit7 has joined #openstack-ironic13:19
*** mkrai has joined #openstack-ironic13:22
openstackgerritDmitry Tantsur proposed openstack/ironic-python-agent master: Release notes cleanup for the next release  https://review.opendev.org/67998913:25
brtknranyone out there booting up fedora coreos baremetal in openstack?13:26
brtknrhow do you pass user-data?13:26
*** hjensas has quit IRC13:29
dtantsurbrtknr: I'm aware that it's possible. If you use Nova, you may need to look at its documentation. You also need a coreos image that understands configdrives (I think it should be an openstack image).13:30
brtknri have been able to boot up fedora coreos as a VM but not as a baremetal instance...13:31
brtknrI have tried enabling config drive but no luck so far13:32
brtknrin the VM, it appears to get the ignition config from `http://169.254.169.254/openstack/latest/user_data`13:32
dtantsurright, the metadata service13:32
*** mbeierl has joined #openstack-ironic13:32
dtantsurbrtknr: you may need to tell nova to generate configdrive13:32
*** irclogbot_2 has joined #openstack-ironic13:38
*** irclogbot_2 has quit IRC13:42
openstackgerritMerged openstack/sushy master: Add conditional field matching  https://review.opendev.org/67507313:43
*** rloo has joined #openstack-ironic13:45
openstackgerritDmitry Tantsur proposed openstack/bifrost master: Make the opensuse-15 jobs non-voting  https://review.opendev.org/67998113:51
dtantsurfolks, this seems needed for bifrost CI ^^13:51
TheJuliaugh13:51
dtantsurTheJulia: morning13:51
dtantsurI'm not sure how long it will take to recover, but it's not the first day13:52
TheJuliait happens sadly13:52
TheJuliawe've had a few weeks once where debian was having repo problems too13:52
dtantsurI will write something to the ML when it merges13:53
TheJuliabrtknr: That image for fedora coreos is _supposed_ to also check the disks for a config drive partition upon boot, but the build team is very focused on single path images as opposed to checking for all possible metadata services/configuration drive formats. So as dtantsur said, enabling config drive should solve it for you13:55
*** irclogbot_3 has joined #openstack-ironic13:59
brtknrTheJulia: dtantsur: I have already tried enabling config drive but didn't get anywhere... the instance boots up okay... but I cant login because my key is not added14:00
TheJuliabrtknr: do you have a link to the image your using?14:01
brtknrhttps://builds.coreos.fedoraproject.org/prod/streams/testing/builds/30.20190801.0/x86_64/fedora-coreos-30.20190801.0-openstack.qcow2.xz14:01
brtknrthis one14:01
*** irclogbot_3 has quit IRC14:02
TheJuliabrtknr: to you have the general download page your going from?14:02
*** mkrai has quit IRC14:02
*** mkrai_ has joined #openstack-ironic14:03
brtknrhttps://getfedora.org/en/coreos/download/14:03
brtknrthis one14:03
openstackgerritMerged openstack/ironic-ui master: Update various links in the documentation  https://review.opendev.org/61780314:03
brtknrTheJulia: ^14:03
brtknrI'm using the OpenStack iamge14:04
brtknrNot sure how it differs from the general Qemu one14:04
*** irclogbot_3 has joined #openstack-ironic14:05
*** mkrai_ has quit IRC14:05
TheJuliabrtknr: they have specific logic on what to do to get configuration14:05
TheJuliabaked into the image14:06
TheJuliaSomeone should tell the fedora folks that OpenStack can be baremetal as well :(14:06
openstackgerritMerged openstack/sushy-tools master: Add Simple Storage resource support  https://review.opendev.org/66732814:06
brtknrTheJulia: so ive been able to boot up the image on a vm instance but sadly not as baremetal14:07
TheJuliaThank you OOMKiller14:07
*** irclogbot_3 has quit IRC14:08
* TheJulia extracts the image to take apart the qcow214:08
*** mkrai has joined #openstack-ironic14:08
*** irclogbot_1 has joined #openstack-ironic14:10
brtknrTheJulia: what do you use to extract qcow2?14:12
TheJuliaGenerally If I have the disk space, I extract it to raw and mount it with losetup14:13
TheJuliaas a loop device, and go from there14:13
*** irclogbot_1 has quit IRC14:14
dtantsuryou can mount a qcow2 using guestfish14:15
dtantsur(you can even modify it this way)14:15
dtantsurbrtknr: e.g. https://github.com/openstack/metalsmith/blob/master/playbooks/integration/centos-image.yaml#L5514:16
dtantsur(this file a few fun hacks)14:16
brtknri've just mounted /dev/sda1 from the qcow2 image but nothing too exciting in there14:17
brtknrusing guestfish14:17
openstackgerritIury Gregory Melo Ferreira proposed openstack/ironic-prometheus-exporter master: Support for metrics description  https://review.opendev.org/67534514:17
TheJuliabrtknr: the secrets are in the 4th partition14:25
rpiosoetingof: ^^^ I don't understand your request. Please elaborate.14:25
*** edleafe has joined #openstack-ironic14:27
*** edleafe has left #openstack-ironic14:28
brtknrTheJulia: I can mount sda1 and sda2 but not sda414:29
brtknrusing guestfish14:30
*** tesseract has quit IRC14:30
brtknrusing guestmount*14:30
brtknrlibguestfs: error: mount_options: mount exited with status 32: mount: wrong fs type, bad option, bad superblock on /dev/sda4,14:30
TheJuliarutro... ostree/deploy/fedora-coreos/deploy/a9c8d66d3628d1b9b4c4690777e8b730d08329b4359410cb410a2003296af1ca.0/usr/lib/os-release:PLATFORM_ID="platform:f30"14:31
TheJuliathey changed it14:32
etingofrpioso, is there some element in imported configuration XML to specify boot mode? -- https://github.com/etingof/sushy-oem-dellemc/blob/master/sushy_oem_dellemc/resources/manager/manager.py#L4514:32
TheJuliabrtknr: mount it read only xfs14:32
brtknri did: guestmount --ro ./fcos/ -m /dev/sda4 -a fedora-coreos-30.20190801.0-openstack.qcow214:34
rpiosoetingof: There almost certainly is, although I don't know if it works with that referenced code.14:34
rpiosoetingof: Why must boot mode be set when the boot source is set to one-time virtual CD? Isn't the boot mode already set?14:35
etingofrpioso, the code is taken literally from https://github.com/dell/iDRAC-Redfish-Scripting/blob/master/Redfish%20Python/SetNextOneTimeBootVirtualMediaDeviceOemREDFISH.py14:35
TheJuliabrtknr: I didn't use guestfish, I did raw mount14:35
TheJuliawith a loopback14:36
etingofrpioso, can virtual CD image contain legacy or uefi boot loader?14:36
TheJuliaso, if it is doing the openstack metadata, then it should also be looking for the configuration drive14:36
*** irclogbot_2 has joined #openstack-ironic14:37
TheJuliabrtknr: in your nova.conf, is force_config_drive set to true or false?14:38
rpiosoetingof: I see! I expect the boot loader of the inserted media must match the boot mode of the server. Isn't it analogous to PXE booting?14:38
etingofrpioso, exactly! we know our CD image, so we need to set boot mode to the server accordingly. the question is - how?14:39
*** tesseract has joined #openstack-ironic14:39
etingofrpioso, I imagine we can do that in a standard way i.e. by patching redfish system tree. however that seemingly would require pushing a bios configuration job followed by a reboot14:40
*** irclogbot_2 has quit IRC14:40
openstackgerritMerged openstack/ironic stable/stein: Do not wait for console being started on timeout  https://review.opendev.org/67902514:40
etingofrpioso, we do reboot while setting boot device anyway, so I am looking to set both things (boot device and boot mode) in one shot14:40
rpiosoetingof: The standard way sounds workable. Sounds like you're asking about an optimization.14:41
etingofrpioso, well... each server reboot involving bios reconfiguration takes 5 minutes for me14:42
rpiosoetingof: Sounds about right.14:42
rpiosoetingof: It's a one-time thing, right?14:43
etingofrpioso, boot mode? well, every time ironic wants to change it we would have to go through this process14:43
*** tesseract has quit IRC14:44
rpiosoetingof: Soon it'll only be UEFI :-)14:44
rpiosoetingof: I don't see ironic wanting to change it very often (never?), once it's been set.14:45
etingofrpioso, however... can we submit two jobs in a row? because first we push a job to change boot device, then we push a job to change boot mode. do we need to reboot in-between?14:45
brtknrTheJulia:   force_config_drive is not set14:46
brtknrin the conductor config14:46
rpiosoetingof: Perhaps exporting the "Server Configuration Profile" (SCP) would provide info on what to add to the referenced code.14:46
rpiosoetingof: No, there can only be one pending/scheduled BIOS config job.14:47
brtknrTheJulia: however, we've used config drive with centos images and it works fine... by enabling it at the time of instance creation14:47
rpiosoetingof: If memory serves, exporting can be done via the iDRAC web GUI.14:48
TheJuliabrtknr: if you look in ironic logs for the instance, do you see if ironic is acting upon a config drive for the instance deployment? Something seems super fishy14:48
etingofrpioso, does this mean that we can't really set one-time boot from virtual media? otherwise we lose that option upon first reboot (after setting boot device) by the time we reach second reboot (after setting boot mode)?14:49
brtknrTheJulia: would this be ironic-conductor?14:50
rpiosoetingof: Does the "standard way" you mentioned above work?14:50
*** irclogbot_2 has joined #openstack-ironic14:51
TheJuliabrtknr: yes14:51
TheJuliabrtknr: do you see "neither config drive nor metadata service were available in time. Continuing without a config..." on the screen from fcos booting?14:51
etingofrpioso, I need to code that... I've found an ansible playbook doing exactly that so I suppose it should work14:51
rpiosoetingof: I suggest confirming that works and then attempt to optimize it.14:52
brtknrTheJulia: there is no mention of config drive in the logs14:52
brtknrhttps://seashells.io/p/eVhrvxB314:53
brtknrTheJulia: oh wait, it was capital C - Configdrive14:53
TheJuliayup, I was just going to say... :)14:53
*** irclogbot_2 has quit IRC14:54
TheJuliaSo the drive is getting written, something is just broken upon reading it or getting the user-data in place. And if you deploy a VM it just works14:54
etingofrpioso, should we push set_boot_mode thing to ironic/drivers/modules/drac/management.py?14:54
brtknr`Configdrive for node 720d9d7d-e7d9-46e7-8346-7b0dae61dc36 successfully copied onto partition`14:55
openstackgerritVarsha Verma proposed openstack/sushy-tools master: Add Storage and Storage Controllers resource support  https://review.opendev.org/67433914:56
TheJuliabrtknr: hmmmm14:57
brtknrTheJulia: Hmm indeed... as far as I can see, I dont see any attempt to read config drive in the serial console14:58
TheJuliaits logic is all hidden in a go binary...14:59
TheJulia:\14:59
TheJuliathe string config-2 is in there though14:59
openstackgerritVarsha Verma proposed openstack/sushy-tools master: Add Storage and Storage Controllers resource support  https://review.opendev.org/67433915:05
openstackgerritVarsha Verma proposed openstack/sushy-tools master: Add Drive resource support  https://review.opendev.org/67508515:05
brtknrTheJulia: Here's the output of my serial console if you can spot anything useful: https://seashells.io/p/5JWKWs9r15:06
openstackgerritJulia Kreger proposed openstack/ironic-python-agent master: Attempt tgtd session detach  https://review.opendev.org/67933215:07
TheJuliait doesn't even look like ignition fires up15:09
brtknrTheJulia: thats what I noticed too15:10
*** Goneri has joined #openstack-ironic15:11
openstackgerritMerged openstack/bifrost master: Make the opensuse-15 jobs non-voting  https://review.opendev.org/67998115:12
TheJuliabrtknr: what deploy_interface are you using? and what versio of ironic?15:14
*** tesseract has joined #openstack-ironic15:14
brtknrWe are using Ironic, is iPXE a deploy interface?15:16
brtknrRocky ironic*15:17
*** trandles has joined #openstack-ironic15:17
rpiosoetingof: No, we shouldn't. See https://github.com/rpioso/ironic/tree/story/200459215:17
*** mkrai has quit IRC15:17
TheJuliabrtknr: direct or iscsi, pxe is a boot interface.15:18
TheJuliabrtknr: openstack baremetal node show $node_uuid15:18
TheJulia*_interface fields15:18
rpiosoetingof: It's the implementation of https://storyboard.openstack.org/#!/story/200459215:18
TheJuliawe don't have any logic to handle os-trees... specifically15:18
TheJuliaI'm kind of stumped. I feel like I'd need to take apart the coreos system while running15:19
*** irclogbot_0 has joined #openstack-ironic15:19
rpiosoetingof: I'm pushing to submit it for review and inclusion in Train.15:19
*** ddurst has quit IRC15:19
etingofrpioso, afaik, you should get it merged by the end of the week15:20
brtknrTheJulia: iscsi15:20
openstackgerritVarsha Verma proposed openstack/sushy-tools master: Add Volume resource support  https://review.opendev.org/67711915:20
rpiosoetingof: Understood15:20
*** irclogbot_0 has quit IRC15:22
etingofrpioso, however, does that code support setting desired boot mode?15:22
openstackgerritJulia Kreger proposed openstack/ironic-python-agent master: Don't try to put a bootloader in place when bootable  https://review.opendev.org/67933315:25
iurygregorydtantsur, about bifrost, its already switched to zuulv3, the only thing we should do is try to move to use bionic I would say15:28
dtantsuriurygregory: it has stuff in playbooks/legacy using zuul-cloner. we need to change that to new-style jobs.15:29
dtantsurand bionic, yes (OMG)15:29
*** e0ne has quit IRC15:29
*** e0ne_ has joined #openstack-ironic15:29
iurygregoryyeah15:29
rpiosoetingof: It declares support for a minimal derivations of the redfish h/w type's management and power interface implementations, e.g., https://github.com/rpioso/ironic/blob/7a738da3d9b4f73b00e8d855d1eb03afdc23e1dd/ironic/drivers/modules/drac/management.py#L286-L28715:32
*** e0ne_ is now known as e0ne15:33
*** irclogbot_1 has joined #openstack-ironic15:33
etingofrpioso, I can see that set_boot_mode() is not implemented there. that means that we still need to add it.15:34
*** ociuhandu has quit IRC15:34
rpiosoetingof: ?15:34
*** rpioso is now known as rpioso|afk15:35
* rpioso|afk will be back15:35
etingofrpioso|afk, this method is missing -- https://github.com/openstack/ironic/blob/master/ironic/drivers/base.py#L90615:35
*** irclogbot_1 has quit IRC15:36
TheJuliabrtknr: I'm moderately stumped. I'm wondering if boot setup is breaking it, but I can't be sure. I'm also in no position to really test at the moment :\15:36
etingofrpioso|afk, w/o it ironic can't change node boot mode15:36
*** irclogbot_0 has joined #openstack-ironic15:37
*** altlogbot_3 has joined #openstack-ironic15:39
tzumainnTheJulia, hi! larsks and I had a question about the spec we're writing regarding multi-tenancy with respect to an owner and a lessee - would this be the right place to ask?15:43
*** S4ren has joined #openstack-ironic15:48
dtantsurtzumainn: yes15:48
*** lucasagomes has quit IRC15:48
dtantsur(if you mean ironic, of course :)15:48
tzumainnhahaha15:49
TheJuliadtantsur: thanks :)15:49
tzumainnokay, so I think there may be a fairly simple solution, which involves oslo policy15:49
brtknrTheJulia: thanks for looking, I will park this for now15:49
TheJulia(The goal being to take over the world of course)15:49
TheJuliatzumainn: I was kind of expecting that from when we originally talked about it.15:50
dtantsurtzumainn: involving oslo.policy is probably the right thing to do, although I'm not sure if it'll help with filtering node listing or with allocations15:50
tzumainnbasically, when ironic does policy.authorize - for example, https://github.com/openstack/ironic/blob/master/ironic/api/controllers/v1/node.py#L225 - the first cdict argument is actually a 'target' and can be used in a policy rule15:50
etingofrpioso|afk, it seems importing bios configuration into iDRAC fails without any explanation if configuration XML has insignificant whitespaces15:51
tzumainnokay, so my question was regarding the spec - TheJulia, you suggested just writing one spec; however, it seems that there could be an initial spec that just covers an owner, since that would require no db changes15:51
tzumainnand then a follow-up spec that covers the lessee, since that does involve a database change, and there could be complexities depending on how flexible we wanted that side of things to be15:52
tzumainndo you think it might be worth splitting the spec up like that?15:52
dtantsurI'd definitely split the owner and the lessee work15:52
TheJuliawell, owner is already a field15:52
dtantsurthe former is much simpler, the latter may end up attracting opposite opinions15:52
tzumainndtantsur, yeah, that was my fear15:52
dtantsurTheJulia: I guess that's what tzumainn is saying: we have a field, we just need to wire it in15:52
TheJuliathis is true,  but I'm worried about the big picture not being percievable without them being viewed together15:53
dtantsurTheJulia: isn't working ownership quite useful on its own?15:53
tzumainnso this way we separate the policy discussion from how we might model the lessee15:53
TheJuliadtantsur: yeah, and determine how we handle defaults and all15:53
TheJuliamaybe that does mean a separate spec migh tbe best15:53
TheJulialessee or "allocated to" kind of field15:54
*** sthussey has joined #openstack-ironic15:54
tzumainnokay! I'll start with the owner/policy spec then and get that submitted for discussion first15:54
tzumainnthanks!15:54
TheJuliatzumainn: okay, thanks!15:55
*** priteau has quit IRC15:55
dtantsuralso, nice to see you again, tzumainn, that's been a while :)15:55
* dtantsur has to run now15:55
tzumainndtantsur, hahaha, yeah, it's been a bit : )15:55
*** dtantsur is now known as dtantsur|afk15:55
dtantsur|afkhave a good rest of the day folks15:55
*** alexmcleod has quit IRC15:58
etingofrpioso|afk, https://github.com/etingof/sushy-oem-dellemc/blob/master/sushy_oem_dellemc/resources/manager/manager.py#L4515:58
TheJuliagoodnight dtantsur|afk15:59
*** gyee has joined #openstack-ironic15:59
*** ociuhandu has joined #openstack-ironic16:06
*** rachit7 has quit IRC16:08
*** ociuhandu has quit IRC16:11
arne_wiebalckbye everyone o/ (tmrw is a holiday here, so back on Friday)16:15
iurygregorybye arne_wiebalck o/16:16
*** e0ne has quit IRC16:31
*** S4ren has quit IRC16:31
-openstackstatus- NOTICE: Zuul job logs stored in OVH may fail. We have updated the base job to remove OVH from our storage location. If you have POST_FAILURES a recheck should fix them at this point.16:52
*** dougsz has quit IRC16:57
*** derekh has quit IRC17:00
*** ociuhandu has joined #openstack-ironic17:22
*** ociuhandu has quit IRC17:26
*** jtwill98 has joined #openstack-ironic17:28
jtwill98Do you know if someone has recently updated:  https://docs.openstack.org/sushy-tools/latest/user/dynamic-emulator.html#uefi-boot17:30
jtwill98I don't recall it having the SimpleStorage section.17:30
*** rpioso|afk is now known as rpioso17:33
rpiosojtwill98: https://opendev.org/openstack/sushy-tools/commits/branch/master17:33
*** ociuhandu has joined #openstack-ironic17:38
*** ociuhandu has quit IRC17:42
rpiosoetingof:17:44
*** bdodd has joined #openstack-ironic17:50
rpiosoetingof: Doesn't the idrac h/w type's new DracRedfishManagement (entrypoint idrac-redfish) management interface implementation picks up that method via derivation from the standard Redfish's implementation, DracRedfishManagement.17:51
*** ociuhandu has joined #openstack-ironic17:54
*** ociuhandu has quit IRC17:58
*** tesseract has quit IRC18:01
*** iurygregory has quit IRC18:08
*** iurygregory has joined #openstack-ironic18:09
etingofrpioso, what would happen to set_boot_mode() call if a configuration job has been submitted by some other module (e.g. set_boot_device())?18:22
rpiosoetingof: Please answer ^^18:33
etingofrpioso, would intermediate reboot upon submitting a configuration job help? I think I am doing that in sushy-oem-dell...18:36
*** hjensas has joined #openstack-ironic18:43
*** ricolin has quit IRC18:50
*** irclogbot_0 has quit IRC18:54
*** irclogbot_0 has joined #openstack-ironic18:55
*** trandles has quit IRC18:57
*** e0ne has joined #openstack-ironic18:59
*** trandles_ has joined #openstack-ironic18:59
*** irclogbot_0 has quit IRC19:02
*** irclogbot_0 has joined #openstack-ironic19:03
rpiosoetingof: Doesn't the idrac h/w type's new DracRedfishManagement (entrypoint idrac-redfish) management interface implementation pick up that method via derivation from the standard Redfish implementation, DracRedfishManagement?19:12
etingofrpioso, I do not see if DracRedfishManagement is any different from RedfishManagement, which does not do any OEM stuff like intermediate reboots. Let me know if I am missing something.19:41
etingofrpioso, my understanding is that, with current iDRAC, set-boot-device-to-vmedia and set-boot-mode calls must be separated by reboots or the second one will surely fail19:42
etingofrpioso, to avoid running into pending configuration job, may be we should add some job queue checking prior to submitting new jobs or just handle specific OEM error that indicates "queue full" condition and retry. in sushy-oem-dell I am trying out the second approach19:46
*** ociuhandu has joined #openstack-ironic19:47
etingofrpioso, conceptually, I think it would be much better to hide all these implementation details behind BMC's Redfish agent. I hope we will get there, eventually19:51
*** jtomasek has quit IRC20:08
*** ociuhandu has quit IRC20:09
*** ociuhandu has joined #openstack-ironic20:10
*** trandles_ has quit IRC20:16
*** trandles has joined #openstack-ironic20:20
rpiosoetingof: Correct, DracRedfishManagement is presently functionally equivalent to RedfishManagement. Therefore, it has https://github.com/openstack/ironic/blob/master/ironic/drivers/base.py#L90620:22
*** trandles has quit IRC20:24
*** trandles has joined #openstack-ironic20:27
*** e0ne has quit IRC20:37
*** ociuhandu has quit IRC20:38
*** e0ne has joined #openstack-ironic20:48
openstackgerritOpenStack Release Bot proposed openstack/metalsmith stable/train: Update .gitreview for stable/train  https://review.opendev.org/68021820:54
openstackgerritOpenStack Release Bot proposed openstack/metalsmith stable/train: Update TOX/UPPER_CONSTRAINTS_FILE for stable/train  https://review.opendev.org/68021920:54
openstackgerritOpenStack Release Bot proposed openstack/metalsmith master: Update master for stable/train  https://review.opendev.org/68022020:54
*** e0ne has quit IRC20:59
*** bfournie has quit IRC21:20
*** ddurst has joined #openstack-ironic21:39
*** dsneddon_ is now known as dsneddon21:52
*** iurygregory has quit IRC21:58
*** trident has quit IRC22:05
*** trident has joined #openstack-ironic22:14
*** jhesketh has quit IRC22:27
*** jhesketh has joined #openstack-ironic22:33
*** bfournie has joined #openstack-ironic22:39
*** rcernin has joined #openstack-ironic23:04
*** bdodd has quit IRC23:14
openstackgerritMichael Davies proposed openstack/ironic-python-agent-builder master: Allow extra python pakgs into tinyipa  https://review.opendev.org/67973523:51
*** trident has quit IRC23:52

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