| opendevreview | Takashi Kajinami proposed openstack/tripleo-heat-templates stable/wallaby: Role specific container support https://review.opendev.org/c/openstack/tripleo-heat-templates/+/855301 | 00:30 |
|---|---|---|
| opendevreview | Takashi Kajinami proposed openstack/tripleo-heat-templates master: OVN: Fix wrong image used in external update tasks https://review.opendev.org/c/openstack/tripleo-heat-templates/+/855302 | 00:33 |
| opendevreview | Takashi Kajinami proposed openstack/tripleo-heat-templates stable/wallaby: Set /run/podman ACL before starting collectd https://review.opendev.org/c/openstack/tripleo-heat-templates/+/854991 | 00:44 |
| opendevreview | Merged openstack/tripleo-quickstart master: Delete existing dstat files during teardown https://review.opendev.org/c/openstack/tripleo-quickstart/+/855115 | 02:34 |
| opendevreview | Merged openstack/puppet-tripleo master: tls_proxy: Make sure port is an integer https://review.opendev.org/c/openstack/puppet-tripleo/+/855155 | 03:15 |
| opendevreview | Takashi Kajinami proposed openstack/tripleo-image-elements master: Simplify openvswitch, remove os-svc-install https://review.opendev.org/c/openstack/tripleo-image-elements/+/838636 | 03:57 |
| opendevreview | Takashi Kajinami proposed openstack/tripleo-image-elements master: Remove the remaining os-*-config elements https://review.opendev.org/c/openstack/tripleo-image-elements/+/838637 | 03:58 |
| *** dasm|off is now known as Guest1645 | 04:00 | |
| opendevreview | David Sedgmen proposed openstack/puppet-tripleo stable/train: Resues fix from "Always update the local certmonger ca cert" https://review.opendev.org/c/openstack/tripleo-heat-templates/+/785020 https://review.opendev.org/c/openstack/puppet-tripleo/+/855310 | 05:38 |
| opendevreview | Luis Tomas Bolivar proposed openstack/tripleo-heat-templates stable/wallaby: Remove unneeded parameter tripleo_frr_ovn_bgp_agent_bridge_mappings https://review.opendev.org/c/openstack/tripleo-heat-templates/+/855330 | 06:32 |
| opendevreview | Luis Tomas Bolivar proposed openstack/tripleo-ansible stable/wallaby: Remove unneeded parameter tripleo_frr_ovn_bgp_agent_bridge_mappings https://review.opendev.org/c/openstack/tripleo-ansible/+/855331 | 06:35 |
| opendevreview | Steve Baker proposed openstack/tripleo-image-elements master: Switch to MiB for all partition/volume sizes https://review.opendev.org/c/openstack/tripleo-image-elements/+/847865 | 06:35 |
| opendevreview | Steve Baker proposed openstack/tripleo-image-elements master: Switch to LVM thin provisioning https://review.opendev.org/c/openstack/tripleo-image-elements/+/847867 | 06:35 |
| opendevreview | Luis Tomas Bolivar proposed openstack/tripleo-ansible stable/wallaby: Remove unneeded parameter tripleo_frr_ovn_bgp_agent_bridge_mappings https://review.opendev.org/c/openstack/tripleo-ansible/+/855331 | 06:35 |
| opendevreview | Merged openstack/tripleo-heat-templates master: Allow for jinja2 templating of puppet_config https://review.opendev.org/c/openstack/tripleo-heat-templates/+/853897 | 07:03 |
| opendevreview | Cedric Jeanneret proposed openstack/tripleo-ansible master: Ensure we properly set the right IP version https://review.opendev.org/c/openstack/tripleo-ansible/+/855314 | 07:17 |
| Tengu | ysandeep|out: -^^ should help | 07:19 |
| Tengu | ysandeep|out: updated the Depends-On on the testproject. Let's see. | 07:20 |
| opendevreview | daniel.pawlik proposed openstack/diskimage-builder master: Change conditions for checking if DIB_RELEASE is greater than 7 https://review.opendev.org/c/openstack/diskimage-builder/+/855315 | 07:21 |
| opendevreview | daniel.pawlik proposed openstack/diskimage-builder master: Change conditions for checking if DIB_RELEASE is greater than 7 https://review.opendev.org/c/openstack/diskimage-builder/+/855315 | 07:32 |
| *** jpena|off is now known as jpena | 07:33 | |
| opendevreview | chandan kumar proposed openstack/tripleo-quickstart master: Use featureset053 for standalone roles testing https://review.opendev.org/c/openstack/tripleo-quickstart/+/854337 | 08:26 |
| opendevreview | chandan kumar proposed openstack/tripleo-quickstart-extras master: Initial playbook to compute node using standalone ansible playbook https://review.opendev.org/c/openstack/tripleo-quickstart-extras/+/843835 | 08:26 |
| opendevreview | chandan kumar proposed openstack/tripleo-ci master: Add tripleo-external-compute-deployment-pipeline job template https://review.opendev.org/c/openstack/tripleo-ci/+/843836 | 08:31 |
| opendevreview | Rodolfo Alonso proposed openstack/tripleo-heat-templates master: Add parameter OVNEncapTos https://review.opendev.org/c/openstack/tripleo-heat-templates/+/854127 | 08:54 |
| opendevreview | Cedric Jeanneret proposed openstack/tripleo-ansible master: [WIP] replace puppetlabs-apache https://review.opendev.org/c/openstack/tripleo-ansible/+/853481 | 09:37 |
| opendevreview | Cedric Jeanneret proposed openstack/tripleo-heat-templates master: [WIP] Convert apache configurations to new role https://review.opendev.org/c/openstack/tripleo-heat-templates/+/854568 | 09:37 |
| chandankumar | bogdando: hello | 11:17 |
| chandankumar | bogdando: can we break the relationship chain https://review.opendev.org/c/openstack/tripleo-ansible/+/854374/10 so that we can easily merge it | 11:18 |
| *** dviroel|out is now known as dviroel | 11:23 | |
| bogdando | chandankumar: will do | 11:26 |
| opendevreview | Bogdan Dobrelya proposed openstack/tripleo-ansible master: Control molecule tags with test molecule script https://review.opendev.org/c/openstack/tripleo-ansible/+/854374 | 11:28 |
| opendevreview | Bogdan Dobrelya proposed openstack/tripleo-ansible master: tripleo_nova_libvirt: add for standalone roles https://review.opendev.org/c/openstack/tripleo-ansible/+/842152 | 11:28 |
| opendevreview | Bogdan Dobrelya proposed openstack/tripleo-ansible master: tripleo_nova_compute: add for standalone roles https://review.opendev.org/c/openstack/tripleo-ansible/+/843659 | 11:28 |
| opendevreview | Bogdan Dobrelya proposed openstack/tripleo-ansible master: nova-libvirt: adjust molecule and tasks layout https://review.opendev.org/c/openstack/tripleo-ansible/+/853810 | 11:28 |
| opendevreview | Bogdan Dobrelya proposed openstack/tripleo-ansible master: Add openssh test_deps as container manager needs it https://review.opendev.org/c/openstack/tripleo-ansible/+/855358 | 11:28 |
| chandankumar | bogdando: for openssh case, we can also do something like this https://opendev.org/openstack/tripleo-ansible/src/branch/master/tripleo_ansible/roles/tripleo_collectd/molecule/common/prepare.yml#L26 | 11:32 |
| chandankumar | it will be limited to role testing only | 11:32 |
| bogdando_ | chandankumar: but test_dep role is already testing only | 11:35 |
| bogdando_ | just that openssh is common dep for all standalone roles, as they gonna test with podman | 11:35 |
| bogdando_ | but on a role level, we can add a flag test_deps_setup_tripleo: true to make that content installed, | 11:36 |
| bogdando_ | makes sense? | 11:36 |
| opendevreview | Cedric Jeanneret proposed openstack/tripleo-ansible master: Ensure we properly set the right IP version https://review.opendev.org/c/openstack/tripleo-ansible/+/855314 | 11:37 |
| chandankumar | bogdando_: ok make sense! | 11:40 |
| bogdando_ | chandankumar: commented https://review.opendev.org/c/openstack/tripleo-ansible/+/855358 | 11:40 |
| *** bogdando_ is now known as bogdando | 11:42 | |
| chandankumar | thank you, will take a look. | 11:42 |
| chandankumar | Tengu: https://review.opendev.org/c/openstack/tripleo-ansible/+/854374 good to go | 11:46 |
| Tengu | bogdando++ | 11:48 |
| Tengu | nudged. | 11:48 |
| bogdando_ | chandankumar: mhm, I have that now https://paste.opendev.org/show/btxveeuKjwlMRdwrctMP/ | 11:52 |
| bogdando_ | shall I add a flag to not install centos release :D | 11:52 |
| bogdando_ | that's when trying it in podman driver | 11:52 |
| bogdando_ | I think /etc/pki/rpm-gpg is RO mount | 11:54 |
| *** bogdando_ is now known as bogdando | 11:54 | |
| bogdando | hm, no it's not, but what's a problem then? touch: cannot touch '/etc/pki/rpm-gpg/test': Permission denied | 11:55 |
| bogdando | ah, that's rootless podman | 11:55 |
| bogdando | ... | 11:55 |
| chandankumar | bogdando: https://opendev.org/openstack/tripleo-ansible/src/branch/master/tripleo_ansible/roles/test_deps/tasks/main.yml#L107 | 12:01 |
| bogdando | yes, and that's fails in rootless podman driver | 12:01 |
| chandankumar | bogdando: if we change the selinux context here https://opendev.org/openstack/tripleo-ansible/src/branch/master/.config/molecule/config_podman.yml#L18 then it will work? | 12:05 |
| chandankumar | na it is not going to work | 12:06 |
| opendevreview | James Slagle proposed openstack/ansible-role-chrony master: Add tasks for run and sync to chrony role https://review.opendev.org/c/openstack/ansible-role-chrony/+/842223 | 12:07 |
| bogdando | I'm trying to override platform settings to use overlay mount | 12:11 |
| bogdando | for gpg diur | 12:11 |
| bogdando | worked like a charm | 12:12 |
| bogdando | we could think on changing default to /etc/pki/rpm-gpg/test:/etc/pki/rpm-gpg/test:O everythere | 12:12 |
| bogdando | btw, this thing (platform options) is still repeated for ~47 molecules | 12:12 |
| bogdando | 80 results in 40 files :) | 12:13 |
| bogdando | for now, I will only care about libvirt role in development | 12:13 |
| bogdando | Tengu: ^^ | 12:13 |
| opendevreview | Merged openstack/tripleo-heat-templates master: Ironic: Require number for IronicIPXEPort https://review.opendev.org/c/openstack/tripleo-heat-templates/+/855129 | 12:16 |
| opendevreview | Merged openstack/tripleo-ansible master: Control molecule tags with test molecule script https://review.opendev.org/c/openstack/tripleo-ansible/+/854374 | 12:16 |
| Tengu | bogdando: what repo? | 12:17 |
| opendevreview | mbu proposed openstack/validations-libs master: Define and loop on config file section https://review.opendev.org/c/openstack/validations-libs/+/855365 | 12:47 |
| opendevreview | Takashi Kajinami proposed openstack/tripleo-heat-templates master: Add parameter OVNEncapTos https://review.opendev.org/c/openstack/tripleo-heat-templates/+/854127 | 13:38 |
| opendevreview | Cedric Jeanneret proposed openstack/tripleo-ansible master: [WIP] replace puppetlabs-apache https://review.opendev.org/c/openstack/tripleo-ansible/+/853481 | 13:40 |
| chandankumar | bogdando: ack | 13:47 |
| *** Guest1645 is now known as dasm | 13:50 | |
| opendevreview | Luis Tomas Bolivar proposed openstack/tripleo-heat-templates master: Add role specific settings for ovn_bgp_agent parameters https://review.opendev.org/c/openstack/tripleo-heat-templates/+/854574 | 13:51 |
| opendevreview | Bogdan Dobrelya proposed openstack/tripleo-heat-templates stable/wallaby: Fix permissions for nova/libvirt VNC certs/keys https://review.opendev.org/c/openstack/tripleo-heat-templates/+/855187 | 13:52 |
| opendevreview | Juan Pablo Martí proposed openstack/tripleo-validations master: Avoid failure when non DPDK instances present https://review.opendev.org/c/openstack/tripleo-validations/+/855271 | 13:53 |
| bogdando | chandankumar: nailed it, overlay mounts in podman config for the win | 14:08 |
| bogdando | a patch is incoming | 14:08 |
| bogdando_ | my irc is faily today, sorry | 14:09 |
| opendevreview | Bogdan Dobrelya proposed openstack/tripleo-ansible master: Add openssh test_deps as container manager needs it https://review.opendev.org/c/openstack/tripleo-ansible/+/855358 | 14:16 |
| opendevreview | Bogdan Dobrelya proposed openstack/tripleo-ansible master: tripleo_nova_libvirt: add for standalone roles https://review.opendev.org/c/openstack/tripleo-ansible/+/842152 | 14:16 |
| opendevreview | Bogdan Dobrelya proposed openstack/tripleo-ansible master: tripleo_nova_compute: add for standalone roles https://review.opendev.org/c/openstack/tripleo-ansible/+/843659 | 14:16 |
| opendevreview | Bogdan Dobrelya proposed openstack/tripleo-ansible master: nova-libvirt: adjust molecule and tasks layout https://review.opendev.org/c/openstack/tripleo-ansible/+/853810 | 14:16 |
| opendevreview | Bogdan Dobrelya proposed openstack/tripleo-ansible master: molecule: podman overlay mounts for dnf in config https://review.opendev.org/c/openstack/tripleo-ansible/+/855374 | 14:16 |
| *** bogdando_ is now known as bogdando | 14:16 | |
| bogdando | chandankumar: https://review.opendev.org/c/openstack/tripleo-ansible/+/855374 | 14:17 |
| bogdando | thanks for merging https://review.opendev.org/c/openstack/tripleo-ansible/+/854374, Tengu | 14:18 |
| bogdando | there is one more aside of the above https://review.opendev.org/c/openstack/tripleo-ansible/+/855358 | 14:18 |
| bogdando | we'll also need to decide how to manage the /etc/ssh/known_hosts requirement of containers manager - globally, or repeating it for each standalone role (which is fine for now) | 14:19 |
| bogdando | slagle: ^^ | 14:19 |
| slagle | bogdando: that's currently managed on the host isn't it? | 14:21 |
| slagle | are you saying multiple roles might need to add entries there? | 14:22 |
| bogdando | right. but not when standalone roles do... standalone | 14:22 |
| slagle | i don't see that being an issue | 14:22 |
| bogdando | 1) known_hosts is absent in container, also openssh doesn't provide a default, so we need ansible to call it out | 14:22 |
| bogdando | 2) /etc/ssh is ensured by openssh, and we need test_deps to cover it | 14:22 |
| bogdando | but that wasn't working for podman driver | 14:23 |
| bogdando_ | slagle: so we can either create those for each standalone role, risking to lose changes in permissions or context etc. for some roles, when changing it for others... or we can manage those via meta dependency role, as I proposed earlier in keystone standalone WIP patches | 14:28 |
| bogdando_ | I hope that makes sense. Please take a look commit msgs as well | 14:29 |
| bogdando_ | also see https://review.opendev.org/c/openstack/tripleo-ansible/+/853810/16..19/tripleo_ansible/roles/tripleo_nova_libvirt/tasks/install.yml#41 | 14:30 |
| slagle | it doesn't make sense completely. i don't understand why any of those things would happen | 14:30 |
| bogdando_ | which things? missing paths? | 14:30 |
| slagle | we can have a role to manage it. if a standalone role neds to add entries, it can use that role to do so | 14:30 |
| slagle | or, we make a role to do it | 14:30 |
| slagle | if the current role doesn't work | 14:31 |
| *** bogdando_ is now known as bogdando | 14:31 | |
| bogdando | maybe on freshly provisioned nodes, when someone wants to deploy a standalone role there accessing it frist time via IPMI concole instead of ssh? ¯\_(ツ)_/¯ | 14:31 |
| bogdando | that's the only case I can see when known_ssh is missing, outside of podman molecule driver | 14:31 |
| bogdando | right, slagle, just saying we'll need that role, or changes in all roles like I linked | 14:32 |
| bogdando | and those molecule-related changes I proposed for test_deps to work for podman driver scenarios | 14:33 |
| bogdando | slagle: we may want to look into using roles/tripleo_ssh_known_hosts/tasks/main.yml | 14:44 |
| bogdando | as I commented, not sure with data to wire in for it though | 14:45 |
| bogdando | with which | 14:45 |
| bogdando | yes, nova standalone roles will likely need this role colled during install | 14:46 |
| bogdando | called | 14:47 |
| *** dviroel is now known as dviroel|lunch | 15:25 | |
| opendevreview | David Hill proposed openstack/ansible-role-redhat-subscription master: Fix satellite discovery due to tripleo_free bug https://review.opendev.org/c/openstack/ansible-role-redhat-subscription/+/855385 | 16:03 |
| *** dviroel|lunch is now known as dviroel | 16:25 | |
| *** jpena is now known as jpena|off | 16:43 | |
| opendevreview | Merged openstack/tripleo-ansible stable/train: bnr - backup of the control plane in composable roles deployment https://review.opendev.org/c/openstack/tripleo-ansible/+/850927 | 18:39 |
| *** dviroel is now known as dviroel|afk | 19:47 | |
| opendevreview | Steve Baker proposed openstack/tripleo-image-elements master: Switch to MiB for all partition/volume sizes https://review.opendev.org/c/openstack/tripleo-image-elements/+/847865 | 20:34 |
| opendevreview | Steve Baker proposed openstack/tripleo-image-elements master: Switch to LVM thin provisioning https://review.opendev.org/c/openstack/tripleo-image-elements/+/847867 | 20:34 |
| *** dasm is now known as dasm|off | 21:40 | |
| *** rlandy is now known as rlandy|bbl | 22:25 | |
| *** dviroel|afk is now known as dviroel | 23:12 | |
| opendevreview | Merged openstack/tripleo-heat-templates stable/wallaby: Fix merge of multiple firewall rules https://review.opendev.org/c/openstack/tripleo-heat-templates/+/855135 | 23:32 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!