| opendevreview | Steve Baker proposed openstack/bifrost master: Use distro conf loader_file_paths to copy ipxe/grub files https://review.opendev.org/c/openstack/bifrost/+/818709 | 01:17 |
|---|---|---|
| opendevreview | Steve Baker proposed openstack/bifrost master: WIP support grub network boot https://review.opendev.org/c/openstack/bifrost/+/807220 | 01:17 |
| opendevreview | Steve Baker proposed openstack/bifrost master: Use distro conf loader_file_paths to copy ipxe/grub files https://review.opendev.org/c/openstack/bifrost/+/818709 | 02:26 |
| opendevreview | Steve Baker proposed openstack/bifrost master: WIP support grub network boot https://review.opendev.org/c/openstack/bifrost/+/807220 | 02:26 |
| rpittau | good morning ironic! Happy Friday! o/ | 07:21 |
| opendevreview | Riccardo Pittau proposed openstack/ironic master: Remove default option from create iso image https://review.opendev.org/c/openstack/ironic/+/818874 | 07:27 |
| opendevreview | Riccardo Pittau proposed openstack/ironic master: [WIP] replace mkisofs with pycdlib https://review.opendev.org/c/openstack/ironic/+/819121 | 07:27 |
| janders | good morning rpittau o/ | 07:40 |
| rpittau | hey janders :) | 07:40 |
| dtantsur | morning ironic, happy Friday! | 09:04 |
| dtantsur | and.. happy new covid, I guess? :( | 09:04 |
| rpittau | :/ | 09:07 |
| dtantsur | looking for some reviews on https://review.opendev.org/c/openstack/ironic/+/819269 and https://review.opendev.org/c/openstack/sushy-tools/+/817559 please | 09:07 |
| dtantsur | the former is blocking a new sushy release. arne_wiebalck if you have a minute for at least the first one ^^ | 09:07 |
| dtantsur | https://review.opendev.org/c/openstack/ironic/+/818553 and https://review.opendev.org/c/openstack/sushy/+/818554 are also very nice to land to avoid regressions | 09:08 |
| dtantsur | all have 1x +2 already | 09:08 |
| janders | Happy Friday dtantsur o/ | 09:28 |
| janders | yeah we'll see how long our "open borders" will last with these new developments | 09:29 |
| dtantsur | yeah.. | 09:29 |
| arne_wiebalck | Good morning, Ironic! | 09:48 |
| arne_wiebalck | dtantsur: checking! | 09:48 |
| iurygregory | Happy Friday Ironic =D | 09:50 |
| dtantsur | hey-hey iurygregory | 09:51 |
| dtantsur | how is your PTO? | 09:52 |
| rpittau | hey iurygregory :) | 09:52 |
| iurygregory | dtantsur, quite good (but well cz now is in some sort of lockdown starting today... I hope I will be able to travel to Brazil on sunday) | 09:52 |
| iurygregory | and today is the first day of snow *-* | 09:53 |
| dtantsur | good luck! it feels like we all will be in a lockdown very soon... | 09:53 |
| dtantsur | snow, nice! we may have some over the weekend | 09:53 |
| rpittau | probably snowing here too starting tonight | 09:53 |
| rpittau | but no mention of lockdown apparently, for the moment | 09:53 |
| iurygregory | it's good to see snow before traveling to Brazil (since I will have to face the summer time...) | 09:54 |
| *** dtantsur_ is now known as dtantsur | 10:22 | |
| opendevreview | Dmitry Tantsur proposed openstack/bifrost master: Force usage of Python 3.9 on RHEL/CentOS 8 https://review.opendev.org/c/openstack/bifrost/+/819422 | 10:28 |
| dtantsur | rpittau: dunno if this will work ^^^ | 10:28 |
| dtantsur | maybe we should forcibly remove /opt/stack/bifrost on upgrades? | 10:29 |
| rpittau | dtantsur: there's no python39 package for centos9, it's just python3, so maybe exclude that in bindep.txt for future compatibility? | 10:41 |
| *** dtantsur_ is now known as dtantsur | 10:51 | |
| dtantsur | rpittau: I can, but this approach is not going to work: there is no python39-firewall >_< | 10:53 |
| dtantsur | and ansible blows up without it | 10:53 |
| dtantsur | (I guess we'll have the same problems with selinux) | 10:53 |
| rpittau | mmmm what about py38? | 10:55 |
| dtantsur | same | 10:55 |
| dtantsur | they're both unsupported from the RHEL perspective | 10:55 |
| opendevreview | Merged openstack/ironic master: Add an option to create inspector-compatible boot.ipxe https://review.opendev.org/c/openstack/ironic/+/817998 | 10:55 |
| rpittau | well I really hope we get those centos-9-stream nodes soon.... | 10:57 |
| dtantsur | rpittau: it does not cancel the fact that we may need to abruptly drop support for 8 | 10:57 |
| dtantsur | OR we can maintain our fork of requirements and otherwise make sure we don't install packages that are not compatible with 3.6 | 10:57 |
| rpittau | yeah, that was unfortunately clear | 10:57 |
| dtantsur | I'm a bit angry about this. There is absolutely no pressure to drop 3.6 support, it's not like 2.7. | 10:58 |
| rpittau | we're totally on the same page on that | 11:01 |
| opendevreview | Merged openstack/sushy-tools master: CertificateService: implement CertificateLocations https://review.opendev.org/c/openstack/sushy-tools/+/817559 | 11:11 |
| dtantsur | another (a bit longer) thing I'd appreciate reviews on: https://review.opendev.org/c/openstack/sushy/+/817792 | 11:12 |
| opendevreview | Merged openstack/ironic master: Do not assume sushy constants are strings - part 2 https://review.opendev.org/c/openstack/ironic/+/819269 | 11:25 |
| opendevreview | Dmitry Tantsur proposed openstack/ironic stable/xena: Do not assume sushy constants are strings - part 2 https://review.opendev.org/c/openstack/ironic/+/819450 | 11:26 |
| opendevreview | Merged openstack/ironic master: Add a unit test job with Sushy from source https://review.opendev.org/c/openstack/ironic/+/818553 | 11:26 |
| opendevreview | Merged openstack/bifrost master: Use boot.ipxe generated by Ironic https://review.opendev.org/c/openstack/bifrost/+/818000 | 11:53 |
| opendevreview | Merged openstack/sushy master: Run Ironic unit tests to avoid regressions https://review.opendev.org/c/openstack/sushy/+/818554 | 12:21 |
| opendevreview | Merged openstack/ironic-python-agent master: Updating yoga tested python versions in classifier https://review.opendev.org/c/openstack/ironic-python-agent/+/819237 | 12:37 |
| opendevreview | Merged openstack/sushy master: Updating yoga tested python versions in classifier https://review.opendev.org/c/openstack/sushy/+/819242 | 13:19 |
| opendevreview | Merged openstack/python-ironic-inspector-client master: Updating python testing as per Yoga testing runtime https://review.opendev.org/c/openstack/python-ironic-inspector-client/+/819239 | 13:22 |
| opendevreview | Merged openstack/sushy-tools master: Test all supported python version https://review.opendev.org/c/openstack/sushy-tools/+/819243 | 13:22 |
| opendevreview | Merged openstack/virtualbmc master: Updating python testing as per Yoga testing runtime https://review.opendev.org/c/openstack/virtualbmc/+/819244 | 13:39 |
| ajya | dtantsur: about virtual media boot for iDRAC and version detecting - though in short-term it's easier to add that to prepare_ramdisk, validation does not really belong there and prepare_ramdisk is already doing a lot of stuff. | 13:49 |
| ajya | Maybe let's start caching BMC version as well? Add method to management interface e.g., detect_bmc_version. Could become useful in future for other scenarios and validations. | 13:49 |
| dtantsur | ajya: in this particular case, doing it in validation may be less and less useful over time | 13:52 |
| dtantsur | since more and more hardware will just have the right version | 13:52 |
| dtantsur | also a new management interface method will not be backportable, so we should probably do both | 13:52 |
| ajya | yes, but maybe there will be more bugs to detect :) | 13:53 |
| ajya | ok, so for now create backportable change and then think about future | 13:53 |
| dtantsur | yeah, that's what I'd suggest | 13:55 |
| dtantsur | I don't disagree with detecting firmware version.. but maybe we should stop adding more methods to ManagementInterface, and, hmm, use OOB inspection somehow? dunno | 13:55 |
| opendevreview | Merged openstack/ironic master: Remove default option from create iso image https://review.opendev.org/c/openstack/ironic/+/818874 | 13:56 |
| ajya | that would require running inspection, at least as a dev I don't do it every time | 14:00 |
| dtantsur | I mean, not the inspection we have now.. some rethinking of inspection | 14:01 |
| ajya | another option could be broader method for these metadata that can populate more properties as need arises (if that will count as backportable) | 14:01 |
| dtantsur | I haven't put a lot of thoughts in it, I must admit | 14:01 |
| ajya | ok, then maybe it could work, if it's different, this could be one of the usecases for updated inspection | 14:01 |
| dtantsur | yeah. let's keep thinking about it. | 14:02 |
| dtantsur | for now I'd prepare something as simple as possible to make the backports easy | 14:02 |
| dtantsur | (think about TheJulia needing to backport it to Train :) | 14:02 |
| ajya | ok, thanks | 14:04 |
| opendevreview | Verification of a change to openstack/ironic stable/xena failed: Do not assume sushy constants are strings - part 2 https://review.opendev.org/c/openstack/ironic/+/819450 | 14:56 |
| rpittau | bye everyone, have a great weekend! o/ | 15:00 |
| TheJulia | Weekend++ | 15:07 |
| opendevreview | Dmitry Tantsur proposed openstack/ironic master: WIP generate inspector.ipxe https://review.opendev.org/c/openstack/ironic/+/819482 | 15:25 |
| opendevreview | Verification of a change to openstack/ironic stable/xena failed: Do not assume sushy constants are strings - part 2 https://review.opendev.org/c/openstack/ironic/+/819450 | 15:43 |
| dtantsur | rpittau: when you're back: this is how we can get centos-9 nodes without a nodeset: https://review.opendev.org/c/openstack/puppet-openstack-integration/+/793462/39/zuul.d/integration.yaml | 16:43 |
| arne_wiebalck | bye everyone, see you on Monday o/ | 17:17 |
| opendevreview | Merged openstack/ironic stable/xena: Do not assume sushy constants are strings - part 2 https://review.opendev.org/c/openstack/ironic/+/819450 | 18:54 |
Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!