Friday, 2024-05-24

opendevreviewMichal Arbet proposed openstack/kolla-ansible master: Bash replacement  https://review.opendev.org/c/openstack/kolla-ansible/+/92036802:12
opendevreviewMichal Arbet proposed openstack/kolla-ansible master: Bash replacement  https://review.opendev.org/c/openstack/kolla-ansible/+/92036802:45
fricklerfyi the failures are due to a bug in zuul. there is a fix for that already and it will hopefully be rolled out by tomorrow06:44
mnasiadkamorning07:04
mnasiadkafrickler: the codespell ones here? https://zuul.opendev.org/t/openstack/build/707d770fdbd6484db7d41c6d1f9a675f07:05
fricklermnasiadka: no, the failures from https://review.opendev.org/c/openstack/kolla-ansible/+/920294 that were mentioned yesterday evening07:33
mnasiadkaok, wonder what happened that we get codespell errors in master now :)07:34
mnasiadkaah, codespell 2.3.0 released yesterday07:34
mnasiadkalet me pin07:34
fricklercodespell got less woke it seems :-/07:36
fricklerthough the error it complains about is a valid one I'd say07:36
opendevreviewMichal Nasiadka proposed openstack/kolla-ansible master: CI: Pin codespell to <2.3.0  https://review.opendev.org/c/openstack/kolla-ansible/+/92037307:37
mnasiadkayeah, but I'd prefer to pin now, and then update in a later change07:37
fricklerif there were like 50 changes to make, I could agree, but just for a single typo?07:39
mnasiadkawell, the same problem will be in stable branches07:40
mnasiadkaand updating renos is probably not something we should do again07:40
fricklerok, we should pin codespell for stable branches probably like other linters, agreed07:40
sylvrHello ! kayobe can gather information using PXEboot (with custom RAMdisk) AND by connecting to the node's oob controller is that right ?07:46
sylvris "inspection" for using their oob controller and "introspection" for using the IPA RAMdisk ?07:47
opendevreviewMichal Nasiadka proposed openstack/kolla-ansible master: CI: Pin requests to <2.32 for docker sdk  https://review.opendev.org/c/openstack/kolla-ansible/+/92016907:48
mnasiadkasylvr: by default kayobe (actually bifrost deployed by kolla-ansible by kayobe) uses IPA ramdisk for inspection - but if your hardware is redfish or something similar and there's a inspection driver that allows out of band inspection - then you can use that, but that will require customising bifrost.yml07:52
sylvrmnasiadka: I put "idrac" in "bifrost enabled hardware types" but running `kayobe overcloud inventory discover` does nothing (no inventory) on the bifrost container I can see my nodes, but their oob isn't managed by the seed07:56
mnasiadkayou mean you can see the nodes in "openstack baremetal node list" in the bifrost container?07:57
sylvrI think I might have misconfigured my networks... but I'm having trouble figuring out why07:57
sylvrmnasiadka: yes!07:57
mnasiadkathen overcloud inventory discover should add then to inventory07:59
sylvrthey're all in the "enroll" provisioning state08:00
sylvrthey're unnamed, no UUID and no power state08:00
sylvrrunning "baremetal node show" I don't see any mention of idrac, but only ipmi08:01
opendevreviewMatt Crees proposed openstack/kayobe stable/2023.1: Fix permissions on dev/rabbitmq-migrate-queues.sh  https://review.opendev.org/c/openstack/kayobe/+/92037508:02
mnasiadkasylvr: they should be in available, no enroll08:02
mnasiadkaso you booted them via network?08:03
mnasiadkaor added manually?08:03
sylvrwell, I've turned the machines on and they should try PXEboot on the mgnt network (which the seed has access to)08:04
sylvrI can see them using baremetal node list, and I get information about their PXE interface08:05
mnasiadkaSo, usually we use name from port description to name the nodes (from lldp data) see https://github.com/openstack/kayobe/blob/27ece2622b934eb6f4d9712d1879ac9bab8d236e/ansible/inventory/group_vars/all/inspector#L214 (inspector_rule_var_lldp_switch_port_interface variable)08:06
mnasiadkato change the default enroll node driver - you can set this variable in kolla/config/bifrost/bifrost.yml - https://github.com/openstack/bifrost/blob/e0a5a7216d74821aee8cc32a8767c790ed731483/playbooks/roles/bifrost-ironic-install/templates/ironic-inspector.conf.j2#L9508:07
mnasiadkafrickler: https://review.opendev.org/c/openstack/kolla-ansible/+/920373 - passed CI08:07
sylvrmnasiadka: okay so in kayobe-config/.../inspector.yml ?08:09
sylvr"inspector_discovery_enroll_node_driver"08:10
mnasiadkakayobe-config/etc/kayobe/kolla/config/bifrost/bifrost.yml if I remember correct08:11
mnasiadkakayobe uses kolla-ansible to deploy bifrost, that's why kolla/config08:11
mnasiadkahttps://github.com/openstack/kayobe/blob/27ece2622b934eb6f4d9712d1879ac9bab8d236e/doc/source/configuration/reference/bifrost.rst#L3208:12
fricklermnasiadka: oh, that went fast, +2, do you want to fast-approve, too?08:14
opendevreviewVictor Chembaev proposed openstack/kolla-ansible master: Fix octavia-interface timeout  https://review.opendev.org/c/openstack/kolla-ansible/+/92037708:15
mnasiadkafrickler: added +w, that's only running pep8 and linters mainly, so gate should be fast as well08:16
sylvrmnasiadka: okay, so now my file kayobe-config/etc/kayobe/kolla/config/bifrost/bifrost.yml contain the two lines following : enable_inspector_discovery: "true" inspector_default_node_driver: "idrac"08:16
sylvrand running kayobe seed service upgrade/deploy should do the trick ?08:17
mnasiadkadeploy, not upgrade08:17
mnasiadkaupgrade is only between releases08:17
sylvryes okay08:17
sylvrI'll try this out right now, I'll let you know ! thank you very much08:17
mnasiadkafrickler: It's a bit irritating that ansible-galaxy connects to internet for installing a git based collection - I think I'll try to get rid of galaxy.ansible.com dependency in CI08:19
mnasiadkaah, for ansible-core we're not going to skip ansible-galaxy08:35
mnasiadkafrickler: do you think we could pass that traffic through some caching proxy in opendev?08:35
fricklermnasiadka: iirc this didn't work with the weird setup galaxy is using08:42
opendevreviewMerged openstack/kolla-ansible master: CI: Pin codespell to <2.3.0  https://review.opendev.org/c/openstack/kolla-ansible/+/92037308:44
sylvrmnasiadka: still no inventory... baremetal node show still doesn't contain any mention of idrac... I might have misconfigured a lot of things08:47
opendevreviewMichal Nasiadka proposed openstack/kolla-ansible master: cli: add ansible-galaxy retries  https://review.opendev.org/c/openstack/kolla-ansible/+/92037808:49
mnasiadkafrickler: so let's try retries08:49
fricklermnasiadka: ack, that would match what we do for toolbox build. your code looks incomplete, though08:53
SvenKieskehere is a python implementation which seems to be able to build a ansible-galaxy cache: https://github.com/jctanner/galaxy-mirror08:55
fricklerSvenKieske: I saw that, but it seems incomplete in terms of implementing a TTL for cached items. also not sure how we could integrate that with our AFS setup08:57
SvenKieskeimplement a TTL via cron? :D08:57
SvenKieskeI don't know anything about the AFS though08:58
SvenKieskethere was also an attempt in triple-o, but it was abandoned together with it, it seems: https://review.opendev.org/c/openstack/tripleo-ansible/+/86603108:59
fricklerthe actual mirror servers only have access to a read-only copy of the mirror fs. so a cache-on-demand setup would be not as effective as a real mirror which would create some kind of data that could then be replicated09:00
SvenKieskeokay, was enhanced security the thought about that design decision (read only copy) or what else played into it?09:03
fricklerI think the main reason is that coordinating multiple writers to avoid conflicting updates would be difficult, so there is a single mirror-update node that does the update jobs09:12
SvenKieskeseems reasonable09:21
SvenKieskewe have quite some jobs with POST_FAILURE currently in kolla: https://zuul.opendev.org/t/openstack/builds?project=openstack/kolla09:22
sylvrmodified kayobe-config/etc/kayobe/inspector.yml to reflect the change in the other file (kolla/bifrost.yml) still, baremetal node show doesn't list idrac as driver...09:22
SvenKieskejust wanted to figure out if I can recheck https://review.opendev.org/c/openstack/kolla/+/918974 but I _think_ we may need https://review.opendev.org/c/openstack/kolla/+/920279 first? looking into the builds now..09:22
SvenKieskeor is that some quay.io issue? mhm09:23
fricklerthat looks like the publishing jobs are also affected by the docker-py issue09:23
SvenKieskeah no, it's not INFO:kolla.common.utils:Exception caught: Error while fetching server API version: Not supported URL scheme http+docker09:23
SvenKieskeso it seems we need https://review.opendev.org/c/openstack/kolla/+/920279 (or some variant of it)09:25
opendevreviewWill Szumski proposed openstack/kolla master: Add packaging as an explicit dependency  https://review.opendev.org/c/openstack/kolla/+/92038109:30
opendevreviewWill Szumski proposed openstack/kayobe master: Bump up Ansible supported versions to 8.x/9.x  https://review.opendev.org/c/openstack/kayobe/+/91051309:32
jovial[m]slyvr: You are going to need to enable the idrac hardware type and all the other interfaces like in this: https://github.com/openstack/bifrost/commit/d8751563e74750ce2c7bd0e7b076604a7641d74a09:36
jovial[m]I wouldn't recommend using the idrac driver09:37
jovial[m]and then rediscover the node09:39
sylvrjovial: I did, I only want to use the idrac driver, but it doesn't seems to work... my oob network contain all my idracs configured to do DHCP09:56
sylvrMy plan was to have a switch with the oob network with all my machines idrac with DHCP to be controlled by the seed/bifrost for inspection/introspection09:57
sylvrI'm guessing provisioning can be done with PXEboot or through the idrac, and it is a requirement that my nodes can be baremetal, so PXEboot might not be an option in this case09:58
opendevreviewMichal Arbet proposed openstack/kolla-ansible master: Bash replacement  https://review.opendev.org/c/openstack/kolla-ansible/+/92036811:45
opendevreviewMichal Arbet proposed openstack/kolla-ansible master: Bash replacement  https://review.opendev.org/c/openstack/kolla-ansible/+/92036811:47
opendevreviewMichal Arbet proposed openstack/kolla-ansible master: Bash replacement  https://review.opendev.org/c/openstack/kolla-ansible/+/92036811:49
opendevreviewMerged openstack/kayobe stable/2023.1: Fix permissions on dev/rabbitmq-migrate-queues.sh  https://review.opendev.org/c/openstack/kayobe/+/92037512:03
opendevreviewMichal Arbet proposed openstack/kolla-ansible master: Bash replacement  https://review.opendev.org/c/openstack/kolla-ansible/+/92036812:20
opendevreviewMichal Nasiadka proposed openstack/kolla-ansible master: cli: add ansible-galaxy retries  https://review.opendev.org/c/openstack/kolla-ansible/+/92037812:28
opendevreviewMichal Nasiadka proposed openstack/kolla-ansible master: cli: add ansible-galaxy retries  https://review.opendev.org/c/openstack/kolla-ansible/+/92037812:29
opendevreviewMichal Arbet proposed openstack/kolla-ansible master: Bash replacement  https://review.opendev.org/c/openstack/kolla-ansible/+/92036812:45
opendevreviewMichal Nasiadka proposed openstack/kolla-ansible stable/2024.1: CI: Pin codespell to <2.3.0  https://review.opendev.org/c/openstack/kolla-ansible/+/92035213:02
opendevreviewMichal Nasiadka proposed openstack/kolla-ansible stable/2023.2: CI: Pin codespell to <2.3.0  https://review.opendev.org/c/openstack/kolla-ansible/+/92035414:26
opendevreviewMichal Nasiadka proposed openstack/kolla-ansible stable/2023.1: CI: Pin codespell to <2.3.0  https://review.opendev.org/c/openstack/kolla-ansible/+/92035514:26
opendevreviewMerged openstack/kolla-ansible master: CI: Pin requests to <2.32 for docker sdk  https://review.opendev.org/c/openstack/kolla-ansible/+/92016914:34
opendevreviewMariusz Karpiarz proposed openstack/kolla-ansible master: Allow to run post-deploy without privilege escalation  https://review.opendev.org/c/openstack/kolla-ansible/+/92029414:43
opendevreviewMichal Nasiadka proposed openstack/kolla-ansible master: cli: add ansible-galaxy retries  https://review.opendev.org/c/openstack/kolla-ansible/+/92037815:32
opendevreviewMichal Arbet proposed openstack/kolla-ansible master: Bash replacement  https://review.opendev.org/c/openstack/kolla-ansible/+/92036816:24
opendevreviewDoug Szumski proposed openstack/kolla master: WIP: Restore Zookeeper  https://review.opendev.org/c/openstack/kolla/+/92040516:40

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