| TheJulia | stevebaker: in theory supported if someone were to make a conforming template, but in practice I don't think I've heard of anyone doing that in... ages. | 00:26 |
|---|---|---|
| TheJulia | stevebaker: there *is* a syslinux version for uefi... just not in centos8 | 00:27 |
| stevebaker | TheJulia: its not really in active development, deprecating now could allow quite some cleanup in the future https://git.kernel.org/pub/scm/boot/syslinux/syslinux.git/log/ | 00:32 |
| stevebaker | TheJulia: although, what is the Power boot chain? | 00:33 |
| TheJulia | stevebaker: lets deprecate it then1 | 00:59 |
| TheJulia | stevebaker: firmware -> kernel, basically | 00:59 |
| TheJulia | but... I think it does load the syslinux templates, and ?maybe? even grub | 01:00 |
| * TheJulia really wants to split the pxe interface further | 01:00 | |
| opendevreview | Steve Baker proposed openstack/ironic master: Use packaged grub efi for network boot https://review.opendev.org/c/openstack/ironic/+/806808 | 01:39 |
| opendevreview | Steve Baker proposed openstack/ironic master: Deprecate [pxe]pxe_config_subdir, write to root dir https://review.opendev.org/c/openstack/ironic/+/807804 | 01:39 |
| opendevreview | Merged openstack/ironic master: Fix to unblock oslodb 11.0.0 https://review.opendev.org/c/openstack/ironic/+/807278 | 04:31 |
| iurygregory | good morning Ironic o/ | 06:24 |
| opendevreview | Tadeas Kot proposed openstack/ironic-inspector master: Add support for state selector in the list introspection https://review.opendev.org/c/openstack/ironic-inspector/+/807578 | 08:34 |
| dtantsur | morning ironic | 08:53 |
| iurygregory | good morning dtantsur | 09:05 |
| opendevreview | Nisha Agarwal proposed openstack/ironic master: Clean step to remove CA certificates from iLO https://review.opendev.org/c/openstack/ironic/+/799562 | 09:16 |
| *** asgr is now known as as-gr | 09:19 | |
| as-gr | Hi all. Has anyone successfully used the Ironic Tempest plugin with an Ironic deployment with conductor groups? Tempest is using a default group, and I can't see a way of making the tests work (at least in v2.0.0). Does anyone have any pointers please? | 09:21 |
| iurygregory | what errors are you getting when trying to run the ironic tempest plugin? | 09:26 |
| as-gr | iurygregory: No valid host was found. Reason: No conductor service registered which supports driver fake-hardware for conductor group "" | 09:34 |
| iurygregory | as-gr, you need to update the configuration with the hardware you are using =) | 09:35 |
| as-gr | In the deployment I'm working on, the conductor groups all have names, and they are configured to support the fake-hardware driver as tested via the CLI, but I cannot seem to 'use' a non-default conductor group in tempest | 09:35 |
| iurygregory | hummm | 09:35 |
| iurygregory | you probably need to specify the conductor group when creating the node | 09:39 |
| iurygregory | https://docs.openstack.org/ironic/latest/admin/conductor-groups.html | 09:39 |
| dtantsur | iurygregory: the migration tests fix, we probably need to backport it to at least a couple of recent versions | 11:05 |
| dtantsur | especially bugfix branches which use master constraints | 11:06 |
| iurygregory | dtantsur, yeah makes sense I will do after lunch | 11:07 |
| opendevreview | Michal Nasiadka proposed openstack/networking-generic-switch master: Add ngs_manage_vlans configuration https://review.opendev.org/c/openstack/networking-generic-switch/+/763794 | 11:53 |
| dtantsur | does anyone remember why we have parallel_image_downloads default to False? looks like a scale killer | 11:54 |
| dtantsur | TheJulia or JayF ^^ | 11:54 |
| as-gr | iurygregory: I'm using tests which create a fake-hardware node as part of the test itself, I think I may need to modify the test to allow for conductor groups in this case | 11:54 |
| iurygregory | as-gr, a possible workaround is having conductor_group set to the default value with fake-hardware available | 12:19 |
| as-gr | Not for my use case unfortunately, I don't have the default CG available. I will see what I can do in terms of a modification | 12:21 |
| opendevreview | Dmitry Tantsur proposed openstack/ironic master: Always update cache for HTTP images if Last Modified is unknown https://review.opendev.org/c/openstack/ironic/+/807863 | 12:23 |
| iurygregory | as-gr, you can try to set the field in each node also baremetal node set --conductor-group "OperatorDefinedString" <uuid> | 12:28 |
| iurygregory | or add support to define a value for conductor_group in the tempest.conf in the [baremetal] section and this value will be used when the nodes are created also | 12:29 |
| as-gr | iurygregory, I'll try and see if I can add support, if I can I will push it upstream :) | 12:49 |
| iurygregory | as-gr, sure =) | 12:50 |
| opendevreview | Merged openstack/sushy-tools master: Basic support for vmedia TLS certificates from version 1.4.0 https://review.opendev.org/c/openstack/sushy-tools/+/805621 | 12:54 |
| opendevreview | Merged openstack/sushy-tools master: Clean up error handling in the emulator https://review.opendev.org/c/openstack/sushy-tools/+/806429 | 12:58 |
| opendevreview | Dmitry Tantsur proposed openstack/ironic master: Enable parallel downloads and allow tuning concurrency https://review.opendev.org/c/openstack/ironic/+/807870 | 13:14 |
| TheJulia | dtantsur: I wanted to change it a long time ago, but I got push back because it would be a behavior change :( | 13:21 |
| dtantsur | TheJulia: hmm, well, I'm about to change it :) | 13:21 |
| TheJulia | dtantsur: but ultimately, no, I don't remember why the default was false | 13:21 |
| TheJulia | dtantsur: okay | 13:22 |
| iurygregory | good morning TheJulia =) | 13:24 |
| opendevreview | Dmitry Tantsur proposed openstack/ironic master: Enable parallel downloads and allow tuning concurrency https://review.opendev.org/c/openstack/ironic/+/807870 | 13:33 |
| TheJulia | I need lots of coffee this morning | 13:46 |
| TheJulia | stevebaker: I think dib headaches are going to diminish some in the near future. :) | 13:52 |
| dtantsur | are you dropping dib? :D | 14:01 |
| TheJulia | no | 14:02 |
| opendevreview | Merged openstack/sushy-tools master: Move virtual media endpoint handlers to a separate blueprint https://review.opendev.org/c/openstack/sushy-tools/+/806603 | 14:06 |
| TheJulia | dtantsur: I reached out to ian, and made a reasonable argument w/r/t the set core reviewers. Turns out somehow it is now owned by infra, which is a problem, but realistically they also are not ones to review generally applicable things so we're kind of in a bind for ironic when we go to fix things. | 14:17 |
| TheJulia | dtantsur: largely because the membership reminds me of my days at HPE | 14:17 |
| dtantsur | heh, I see | 14:28 |
| dtantsur | well, at least we got IPA-builder | 14:28 |
| TheJulia | Yeah | 14:28 |
| TheJulia | this is more to do with OS images | 14:28 |
| TheJulia | in part, the fun of grub and its... evolving changes in newer version which are being stamped out | 14:29 |
| dtantsur | well, I literally just did $ baremetal node set testvm1 --instance-info image_source=https://cloud-images.ubuntu.com/releases/bionic/release/ubuntu-18.04-server-cloudimg | 14:29 |
| dtantsur | -amd64.img --instance-info image_checksum=https://cloud-images.ubuntu.com/releases/bionic/release/MD5SUMS --instance-info image_download_source=local | 14:29 |
| dtantsur | so, managing quite well without DIB for instance images :D | 14:29 |
| dtantsur | I wonder if TripleO could just use virt-customize | 14:30 |
| dtantsur | e.g. that's how I prepare my testing VMs https://github.com/dtantsur/config/blob/master/virt-install.sh#L39-L57 | 14:30 |
| TheJulia | There is a reason to not use virt-customize, I just don't remember what it is right now | 14:31 |
| TheJulia | and realistically they should be able to, the conundrum is people want all sorts of different things for their sheep-cattle hybrids | 14:32 |
| dtantsur | yeah, of course they do | 14:33 |
| dtantsur | I wish we had something to customize operating systems post-installation | 14:33 |
| * TheJulia raises an eyebrow that github didn't load for a few minutes | 14:33 | |
| dtantsur | just imagine I could write a yaml file describing the desired state? | 14:33 |
| * dtantsur is probably too sarcastic today | 14:34 | |
| TheJulia | dtantsur: and hope.... :( | 14:34 |
| opendevreview | Merged openstack/ironic master: Add release note upgrade version check handling change https://review.opendev.org/c/openstack/ironic/+/807403 | 14:39 |
| TheJulia | dtantsur: sounds like https://review.opendev.org/c/openstack/ironic/+/807187 might/maybe should be backported? | 14:45 |
| dtantsur | TheJulia: I'd like to backport it, yes | 14:45 |
| TheJulia | flagged it then since I'm in agreement | 14:45 |
| dtantsur | going a bit earlier today, see you tomorrow | 14:57 |
| opendevreview | Iury Gregory Melo Ferreira proposed openstack/ironic bugfix/18.1: Fix to unblock oslodb 11.0.0 https://review.opendev.org/c/openstack/ironic/+/807910 | 15:06 |
| *** tosky is now known as Guest6702 | 15:22 | |
| *** tosky_ is now known as tosky | 15:22 | |
| opendevreview | Merged openstack/ironic-tempest-plugin master: remove ebtables fix env var from wallaby stable job https://review.opendev.org/c/openstack/ironic-tempest-plugin/+/799679 | 16:00 |
| TheJulia | dtantsur: tomorrow, lets chat about https://review.opendev.org/c/openstack/ironic/+/807295 I left some feedback on it. I have an idea. Hopefully my concerns make sense when you read them | 16:10 |
| opendevreview | Julia Kreger proposed openstack/ironic master: Always update cache for HTTP images if Last Modified is unknown https://review.opendev.org/c/openstack/ironic/+/807863 | 16:41 |
| TheJulia | dtantsur: If you could, revisit https://review.opendev.org/c/openstack/ironic/+/803128 We've got enough in that folder which makes me think it is best to just include it. | 16:44 |
| opendevreview | Verification of a change to openstack/ironic-python-agent master failed: Trivial: minor fixes in error messages https://review.opendev.org/c/openstack/ironic-python-agent/+/807713 | 19:15 |
| opendevreview | Julia Kreger proposed openstack/ironic master: Facilitate asset copy for bootloader ops https://review.opendev.org/c/openstack/ironic/+/806815 | 19:47 |
| opendevreview | Merged openstack/ironic-lib stable/wallaby: Avoid using blkid in favour of lsblk and parted https://review.opendev.org/c/openstack/ironic-lib/+/807752 | 20:01 |
| opendevreview | Merged openstack/ironic bugfix/18.1: Fix to unblock oslodb 11.0.0 https://review.opendev.org/c/openstack/ironic/+/807910 | 20:01 |
| opendevreview | Verification of a change to openstack/ironic bugfix/18.0 failed: Improve agent_client logging https://review.opendev.org/c/openstack/ironic/+/795151 | 20:01 |
| stevebaker | morning | 20:18 |
| opendevreview | Iury Gregory Melo Ferreira proposed openstack/ironic master: Support HttpHeaders in create_subscription https://review.opendev.org/c/openstack/ironic/+/806859 | 20:33 |
| -opendevstatus- NOTICE: The Gerrit service on review.opendev.org is going offline momentarily for a host migration and zuul upgrade, downtime should be only a few minutes. | 21:05 | |
Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!