| *** pmannidi is now known as pmannidi|AFK | 01:13 | |
| *** rlandy|ruck|bbl is now known as rlandy|ruck | 01:26 | |
| opendevreview | Merged openstack/puppet-tripleo stable/wallaby: haproxy: Use healthcheck middleware to monitor service availability https://review.opendev.org/c/openstack/puppet-tripleo/+/810179 | 02:35 |
|---|---|---|
| *** pmannidi|AFK is now known as pmannidi | 03:06 | |
| *** pojadhav|afk is now known as pojadhav | 04:34 | |
| opendevreview | Pranali Deore proposed openstack/tripleo-heat-templates master: Allow configuring secure RBAC in glance https://review.opendev.org/c/openstack/tripleo-heat-templates/+/783623 | 05:28 |
| opendevreview | Merged openstack/tripleo-validations stable/wallaby: Use collections in validations for Ansible 2.11 https://review.opendev.org/c/openstack/tripleo-validations/+/812565 | 05:43 |
| opendevreview | Merged openstack/tripleo-validations stable/wallaby: [validation_init] add new metadata keys in the playbook template https://review.opendev.org/c/openstack/tripleo-validations/+/807995 | 05:43 |
| opendevreview | Merged openstack/tripleo-validations stable/wallaby: remove using containers-prepare-parameter.yaml https://review.opendev.org/c/openstack/tripleo-validations/+/806896 | 05:45 |
| *** amoralej|off is now known as amoralej | 06:03 | |
| opendevreview | Michele Baldessari proposed openstack/puppet-tripleo master: DNM/DNR WIP rabbitmq tls tests https://review.opendev.org/c/openstack/puppet-tripleo/+/812401 | 06:22 |
| *** ysandeep|out is now known as ysandeep | 06:39 | |
| opendevreview | Takashi Kajinami proposed openstack/tripleo-quickstart master: WIP: Remove logic for old releases https://review.opendev.org/c/openstack/tripleo-quickstart/+/812802 | 06:45 |
| *** jpena|off is now known as jpena | 06:47 | |
| opendevreview | Michele Baldessari proposed openstack/puppet-tripleo master: DNM/DNR WIP rabbitmq tls tests https://review.opendev.org/c/openstack/puppet-tripleo/+/812401 | 07:01 |
| opendevreview | Michele Baldessari proposed openstack/puppet-tripleo master: DNM/DNR WIP rabbitmq tls tests https://review.opendev.org/c/openstack/puppet-tripleo/+/812401 | 07:37 |
| opendevreview | Juan Badia Payno proposed openstack/tripleo-ansible master: bnr Grouped tasks in a loop and added OUTPUT_PREFIX_PXE https://review.opendev.org/c/openstack/tripleo-ansible/+/811953 | 07:55 |
| opendevreview | Shnaidman Sagi (Sergey) proposed openstack/diskimage-builder master: Correct path for CentOS 9 stream base image https://review.opendev.org/c/openstack/diskimage-builder/+/806819 | 08:30 |
| opendevreview | Brendan Shephard proposed openstack/tripleo-quickstart master: DNM: Remove protected_resource arg https://review.opendev.org/c/openstack/tripleo-quickstart/+/812924 | 08:31 |
| opendevreview | Merged openstack/tripleo-heat-templates master: Allow configuring secure RBAC in placement https://review.opendev.org/c/openstack/tripleo-heat-templates/+/804259 | 08:34 |
| *** ykarel is now known as ykarel|lunch | 08:34 | |
| hjensas | Morning Tengu! How is it going today? | 08:50 |
| Tengu | hjensas: hello there! well, still head-banging my desk ;). After some more discussions, it seems the actual network.name should NOT be tempered with - so I proposed this RFE: https://bugzilla.redhat.com/show_bug.cgi?id=2011688 it should already help a bit. | 08:51 |
| hjensas | Tengu: where does that come from? | 08:52 |
| Tengu | hjensas: so I've modified tripleo-lab in order to not temper the network.name, but apparently, I'm hitting other things with "only" the name_lower + service_net_map_replace being custom/set. | 08:52 |
| hjensas | Tengu: With composable networks you should be able to use a complete custom set of networks. As long as the ServiceNetMap is written to match it should work. | 08:53 |
| Tengu | hjensas: from the many, many occurrences of hard-coded «defaults» through t-h-t... and from discussions with ramishra | 08:55 |
| Tengu | (sorry, I'm running 3 things in // -.-') | 08:55 |
| hjensas | Tengu: The doc says "TripleO offers the option of deploying with a user-defined list of networks, where each network can be enabled (or not) for each role (group of servers) in the deployment." | 08:56 |
| hjensas | Tengu: https://docs.openstack.org/project-deploy-guide/tripleo-docs/latest/features/custom_networks.html | 08:57 |
| Tengu | hjensas: so we DO have a big issue with the hard-coded things. | 08:57 |
| Tengu | ah, fun, the link I pasted was from slagle page, while I thought it was that very link you pointed. | 08:57 |
| hjensas | Tengu: There should always be an "else" to get a default. But when everything is Custom I think overriding the ServiceNetMap is required. | 08:59 |
| Tengu | hjensas: isn't the service_net_map_replace aimed at this specific part? | 08:59 |
| hjensas | Tengu: service_net_map_replace does not take care of everything. The resolve_network, qdr_metric_network and also the VipSubnetMap need a manual override. | 09:00 |
| Tengu | hjensas: hence your proposal to add some new param for those specific parts? | 09:00 |
| hjensas | Tengu: It should merge with the defaults, so you can override just those parts. | 09:00 |
| opendevreview | Haresh proposed openstack/tripleo-common master: add kernel-modules-extra in overcloud image needed for HWOL https://review.opendev.org/c/openstack/tripleo-common/+/812683 | 09:01 |
| hjensas | Tengu: yes, adding new params for those in role data would remove the need for the if conditions you removed. Even the "if 'ceph' in role.tags|default([])" could go. | 09:02 |
| Tengu | yup | 09:03 |
| opendevreview | Haresh proposed openstack/tripleo-common master: add kernel-modules-extra in overcloud image needed for HWOL https://review.opendev.org/c/openstack/tripleo-common/+/812683 | 09:04 |
| *** jpena is now known as jpena|brb | 09:05 | |
| hjensas | Tengu: https://access.redhat.com/documentation/en-us/red_hat_openstack_platform/16.2/html/advanced_overcloud_customization/custom-composable-networks#assigning-openstack-services-to-composable-networks | 09:06 |
| Tengu | hjensas: ah, we can just override PARTs of the ServiceNetMap then? | 09:07 |
| hjensas | Tengu: The doc there shows how to move specific services to a different network. We should then be able to move _all_ services to custom networks and remove the default networks. Right? :) | 09:07 |
| hjensas | Tengu: yes, ControllerHostnameResolveNetwork: internal_api_cloud_0 | 09:07 |
| hjensas | ComputeHostnameResolveNetwork: internal_api_cloud_0 | 09:07 |
| hjensas | ControllerMetricsQdrNetwork: internal_api_cloud_0 | 09:07 |
| hjensas | ComputeMetricsQdrNetwork: internal_api_cloud_0 | 09:07 |
| Tengu | hjensas: soo. ok. lemme do some more modifications to tripleo-lab (as said: part of the issues was probably on my env, so no surprise here) | 09:10 |
| Tengu | hjensas: btw - wondering if I'm not seeing some other issues in my env: while poking at the network related files, I see StorageAllocationPools - shouldn't it be StorageCloud0AllocationPools? | 09:12 |
| Tengu | heh. yes. should be: {{network.name}}AllocationPools: | 09:12 |
| hjensas | Tengu: yes. | 09:13 |
| Tengu | ok. yeah. there are many issues there. | 09:13 |
| Tengu | hjensas: care to jump on my env? | 09:15 |
| Tengu | I have a terrible doubt. | 09:15 |
| Tengu | hjensas: IP has changed btw: 10.39.193.71 | 09:16 |
| Tengu | hjensas: I think I have a not-nice mix of things. | 09:18 |
| hjensas | Tengu: I'm in, yes that network-env.yaml has a lot of issues. But most of that is'nt used with network-v2. | 09:18 |
| Tengu | hjensas: https://meet.google.com/adw-ixof-egk?pli=1 | 09:18 |
| opendevreview | Takashi Kajinami proposed openstack/tripleo-heat-templates stable/train: [Train-only] Enable usage of memcache for keystone caching by default https://review.opendev.org/c/openstack/tripleo-heat-templates/+/812940 | 09:25 |
| opendevreview | Merged openstack/tripleo-heat-templates master: Allow configuring secure RBAC in nova https://review.opendev.org/c/openstack/tripleo-heat-templates/+/804254 | 09:25 |
| opendevreview | Merged openstack/tripleo-quickstart master: Provision networks before overcloud deploy https://review.opendev.org/c/openstack/tripleo-quickstart/+/812423 | 09:25 |
| hjensas | https://access.redhat.com/documentation/en-us/red_hat_openstack_platform/16.2/html/advanced_overcloud_customization/custom-composable-networks#renaming-default-networks_portal | 09:35 |
| hjensas | https://opendev.org/openstack/tripleo-ansible/src/branch/master/tripleo_ansible/ansible_plugins/modules/tripleo_composable_network.py#L148 | 09:43 |
| hjensas | Tengu: ^ | 09:44 |
| *** ykarel|lunch is now known as ykarel | 09:52 | |
| *** jpena|brb is now known as jpena | 10:00 | |
| opendevreview | Jose Luis Franco proposed openstack/tripleo-ansible master: Add dnf_stream Ansible module. https://review.opendev.org/c/openstack/tripleo-ansible/+/812948 | 10:27 |
| opendevreview | Michele Baldessari proposed openstack/puppet-tripleo master: DNM/DNR WIP rabbitmq tls tests https://review.opendev.org/c/openstack/puppet-tripleo/+/812401 | 10:39 |
| *** rlandy is now known as rlandy|ruck | 10:41 | |
| rlandy|ruck | ramishra: thanks for your work to get the minimal featureset jobs working again | 10:55 |
| rlandy|ruck | https://jenkins-cloudsig-ci.apps.ocp.ci.centos.org/job/tripleo-quickstart-gate-master-tripleo-ci-delorean-full-featureset052/ looking good for the first time in ages | 10:56 |
| Tengu | hjensas: back - guess it's your time to eat? :) | 11:02 |
| *** jcapitao is now known as jcapitao_lunch | 11:07 | |
| *** jpena is now known as jpena|lunch | 11:23 | |
| opendevreview | Yariv proposed openstack/tripleo-upgrade master: [FFWD upgrade] Adding support of live/cold migration with external workload https://review.opendev.org/c/openstack/tripleo-upgrade/+/802524 | 11:27 |
| ramishra | rlandy|ruck: yw | 11:30 |
| hjensas | Tengu: had, food. But I have a couple of meetings to attend. | 11:52 |
| Tengu | hjensas: I'm free for 30 minutes, then mtg for ~30 minutes, then free most of the afternoon | 11:55 |
| Tengu | hjensas: just ping me whenever you have a moment. I've added some more content to the hackmd | 11:55 |
| *** amoralej is now known as amoralej|lunch | 12:12 | |
| opendevreview | Merged openstack/tripleo-heat-templates stable/train: Enable keystone_authtoken/memcach_use_advanced_pool for Sahara https://review.opendev.org/c/openstack/tripleo-heat-templates/+/812640 | 12:12 |
| opendevreview | Merged openstack/tripleo-quickstart-extras master: Support setting growvols_args in topology_map https://review.opendev.org/c/openstack/tripleo-quickstart-extras/+/812414 | 12:12 |
| opendevreview | Merged openstack/tripleo-quickstart master: Set growvols_args for all Controller roles https://review.opendev.org/c/openstack/tripleo-quickstart/+/812416 | 12:12 |
| opendevreview | Merged openstack/validations-common master: Add new metadata keys to the validation playbooks https://review.opendev.org/c/openstack/validations-common/+/808915 | 12:12 |
| opendevreview | Merged openstack/tripleo-validations stable/wallaby: Add new metadata keys to the validation playbooks https://review.opendev.org/c/openstack/tripleo-validations/+/807996 | 12:12 |
| opendevreview | Marios Andreou proposed openstack/tripleo-ci master: [wip] Add support for CentOS Stream 9 https://review.opendev.org/c/openstack/tripleo-ci/+/808177 | 12:20 |
| *** jpena|lunch is now known as jpena | 12:23 | |
| *** jcapitao_lunch is now known as jcapitao | 12:28 | |
| *** raukadah is now known as chandankumar | 12:28 | |
| opendevreview | David Vallee Delisle proposed openstack/tripleo-heat-templates stable/wallaby: Allow configuring secure RBAC in placement https://review.opendev.org/c/openstack/tripleo-heat-templates/+/812443 | 12:29 |
| gfidente | rlandy|ruck can you help get this back in? https://review.opendev.org/c/openstack/tripleo-ansible/+/812568 :D | 12:43 |
| gfidente | and sorry and thanks for helping :D | 12:43 |
| *** pdeore_ is now known as pdeore|afk | 12:47 | |
| opendevreview | John Fulton proposed openstack/tripleo-ansible master: Apply ceph_conf_overrides for non-deployed_ceph services https://review.opendev.org/c/openstack/tripleo-ansible/+/812230 | 12:54 |
| fultonj | fmount[m]: ^ fyi | 12:54 |
| rlandy|ruck | gfidente: sure - I can vote there | 12:56 |
| opendevreview | John Fulton proposed openstack/tripleo-ansible master: Apply ceph_conf_overrides for non-deployed_ceph services https://review.opendev.org/c/openstack/tripleo-ansible/+/812230 | 12:56 |
| opendevreview | Ananya proposed openstack/tripleo-ci-health-queries master: Bugfix for queries with multiple regx/pattern https://review.opendev.org/c/openstack/tripleo-ci-health-queries/+/812987 | 12:56 |
| rlandy|ruck | you'd want a vote from sshnaidm, mwhahaha on tripleo-ansible changes as well | 12:56 |
| rlandy|ruck | scenario001 and 002 are passing there - which iirc was the issue before | 12:57 |
| opendevreview | John Fulton proposed openstack/tripleo-ansible master: Apply ceph_conf_overrides for non-deployed_ceph services https://review.opendev.org/c/openstack/tripleo-ansible/+/812230 | 12:58 |
| *** amoralej|lunch is now known as amoralej | 13:07 | |
| opendevreview | Cedric Jeanneret proposed openstack/puppet-tripleo stable/wallaby: haproxy: Add missing default_backend option https://review.opendev.org/c/openstack/puppet-tripleo/+/812567 | 13:18 |
| sshnaidm | mwhahaha, hi, do you you openstack/release process for releasing tripleo-operators? | 13:27 |
| sshnaidm | mwhahaha, I'd like to see if auto publishing job works, can we release something? | 13:28 |
| hjensas | Tengu: finally, done with the round of meetings. | 13:36 |
| hjensas | Tengu: I would actually like to go back to an all custom network.name scenario. Since we do document how to add custom networks and re-map services in ServiceNetMap. It should be possible to have all custom networks, and re-map everything in ServiceNetMap. | 13:39 |
| Tengu | hjensas: I'll be available in ~10 | 13:42 |
| mwhahaha | sshnaidm: yea i can. we need a metadata update tho or does the job take care of that? | 13:44 |
| sshnaidm | mwhahaha, galaxy.yml is handled by job | 13:44 |
| mwhahaha | k you can propose a release patch and i can +1 for ptl | 13:45 |
| sshnaidm | sure | 13:45 |
| Tengu | hjensas: I'll trash the current env and boostrap it with full custom names. | 13:47 |
| sshnaidm | mwhahaha, https://review.opendev.org/c/openstack/releases/+/813009 | 13:49 |
| sshnaidm | marios, ^^ testing if it works now | 13:49 |
| Tengu | hjensas: deploy started - the OC will NOT be deployed automatically so that we can have a final look to the environment before running it. | 13:50 |
| mwhahaha | k they might balk at the version number if we've added things. will have to see what the changes list has | 13:50 |
| hjensas | Tengu: cool | 13:50 |
| marios | sshnaidm: ack cool | 13:52 |
| *** ykarel_ is now known as ykarel|away | 13:52 | |
| opendevreview | Merged openstack/tripleo-heat-templates master: Fix swift "Could not enable rsyncd" error https://review.opendev.org/c/openstack/tripleo-heat-templates/+/812307 | 13:53 |
| opendevreview | Michele Baldessari proposed openstack/tripleo-heat-templates stable/wallaby: Fix swift "Could not enable rsyncd" error https://review.opendev.org/c/openstack/tripleo-heat-templates/+/813013 | 14:01 |
| opendevreview | Yariv proposed openstack/tripleo-upgrade stable/train: [FFWD upgrade] Adding support of live/cold migration with external workload https://review.opendev.org/c/openstack/tripleo-upgrade/+/803236 | 14:06 |
| opendevreview | Lance Bragstad proposed openstack/tripleo-heat-templates master: Implement project personas in custom keystone policy file https://review.opendev.org/c/openstack/tripleo-heat-templates/+/810325 | 14:10 |
| Tengu | hjensas: UC is available - not deployed yet, still installing dependencies, then it will build the custom tripleo-heat-templates and go for the deploy prep steps and so on. | 14:10 |
| opendevreview | Yariv proposed openstack/tripleo-upgrade master: [FFWD upgrade] Adding support of live/cold migration with external workload https://review.opendev.org/c/openstack/tripleo-upgrade/+/802524 | 14:11 |
| opendevreview | David Hill proposed openstack/tripleo-ansible master: Append lvm2_allow_list only if it's not empty https://review.opendev.org/c/openstack/tripleo-ansible/+/813020 | 14:12 |
| opendevreview | Lance Bragstad proposed openstack/tripleo-heat-templates master: Implement project personas in custom neutron policy file https://review.opendev.org/c/openstack/tripleo-heat-templates/+/810494 | 14:13 |
| Tengu | hjensas: there's a tmux if you want to join | 14:13 |
| opendevreview | Michele Baldessari proposed openstack/tripleo-heat-templates master: Switch to rabbitmq fqdn hostnames and pass proper CA info https://review.opendev.org/c/openstack/tripleo-heat-templates/+/812390 | 14:18 |
| opendevreview | Juan Badia Payno proposed openstack/tripleo-ansible master: bnr Use replace instead of lineinfile and grouped similar tasks in a loop https://review.opendev.org/c/openstack/tripleo-ansible/+/811953 | 14:20 |
| Tengu | hjensas: UC is deploying. We should be good in ~20 minutes (never actually checked the time it takes...) | 14:22 |
| Tengu | hjensas: but I'll get a meeting in ~40 | 14:22 |
| opendevreview | David Hill proposed openstack/tripleo-ansible master: Append lvm2_allow_list only if it's not empty https://review.opendev.org/c/openstack/tripleo-ansible/+/813020 | 14:25 |
| opendevreview | Marios Andreou proposed openstack/tripleo-ci master: Adds tripleo-repos and tripleo-operator jobs in tripleo-ci https://review.opendev.org/c/openstack/tripleo-ci/+/810261 | 14:29 |
| opendevreview | David Hill proposed openstack/tripleo-ansible master: Append lvm2_allow_list only if it's not empty https://review.opendev.org/c/openstack/tripleo-ansible/+/813020 | 14:30 |
| opendevreview | Marios Andreou proposed openstack/tripleo-ci master: Adds tripleo-repos/-operator-ansible collection push jobs https://review.opendev.org/c/openstack/tripleo-ci/+/810261 | 14:30 |
| Tengu | hjensas: UC deployed. fetching OC images and preparing everything for it - including OC deploy script. and it will halt then. | 14:35 |
| opendevreview | Harald Jensås proposed openstack/tripleo-heat-templates master: WIP - Use network.dns_domain if defined https://review.opendev.org/c/openstack/tripleo-heat-templates/+/813037 | 14:38 |
| Tengu | hjensas: ah, the env doesn't have this change -^^ but you can fetch the files and apply them directly | 14:38 |
| Tengu | hjensas: it's provisioning the nodes right nwo. | 14:39 |
| Tengu | *now | 14:39 |
| hjensas | Tengu: I think don't need that change for now. I just think that might be a good change, to allow custom dns_domain without changing network.name. | 14:39 |
| Tengu | lgtm :) | 14:40 |
| Tengu | hjensas: the env has small changes compared to yesterday's - among them, the added "ServiceNetMap" in the overcloud-0-yml/network-env.yaml file | 14:40 |
| hjensas | Tengu: I wonder where it breaks, we probably expect network.name.lower() somewhere else. | 14:40 |
| Tengu | hjensas: yeah... and there are those hard-coded things as well. | 14:41 |
| Tengu | hjensas: node privisionning still running | 14:41 |
| hjensas | Tengu: oh, we need to add the metrics overrides as well in ServiceNetMap | 14:41 |
| Tengu | hjensas: feel free to edit! | 14:41 |
| Tengu | (and I'll report that in tripleo-lab code) | 14:41 |
| Tengu | hjensas: that QDRNetwork-thing? | 14:42 |
| opendevreview | Bogdan Dobrelya proposed openstack/tripleo-heat-templates master: Improve scn10 workers layouts for blocking calls https://review.opendev.org/c/openstack/tripleo-heat-templates/+/813038 | 14:42 |
| Tengu | MetricsQdrNetwork and HostnameResolveNetwork ? | 14:43 |
| opendevreview | Michele Baldessari proposed openstack/puppet-tripleo master: Modernize rabbitmq's TLS support https://review.opendev.org/c/openstack/puppet-tripleo/+/812401 | 14:43 |
| Tengu | hjensas: ah, thanks for correcting that storage_api thing. sorry! | 14:44 |
| hjensas | Tengu: lucky that I saw it. | 14:44 |
| Tengu | I saw it just before while editing the file on my side ^^' was about to warn you | 14:44 |
| opendevreview | David Hill proposed openstack/tripleo-ansible master: Append lvm2_allow_list only if it's not empty https://review.opendev.org/c/openstack/tripleo-ansible/+/813020 | 14:45 |
| Tengu | oh. | 14:45 |
| Tengu | dang. | 14:45 |
| Tengu | hjensas: network are wrongs in the role-data! | 14:45 |
| hjensas | Tengu: indeed. | 14:45 |
| Tengu | forgot to re-add the ansible "replace" bits. pfff | 14:45 |
| hjensas | do we need to manually edit? Or do you have magic that can fix it? | 14:46 |
| hjensas | : | 14:46 |
| Tengu | unfortunately, running the magic will.... well. gimme a second | 14:46 |
| opendevreview | Alex Schultz proposed openstack/tripleo-common stable/ussuri: Add a retry to the instropection waiting workbook https://review.opendev.org/c/openstack/tripleo-common/+/812960 | 14:48 |
| Tengu | hjensas: lemme try to re-run the ansible. | 14:48 |
| hjensas | Tengu: I think I fixed it? | 14:48 |
| hjensas | yeah, missed External | 14:48 |
| Tengu | hjensas: now it's fixed :) | 14:49 |
| Tengu | but yep, sounds good! | 14:49 |
| opendevreview | Alex Schultz proposed openstack/tripleo-common stable/train: Add a retry to the instropection waiting workbook https://review.opendev.org/c/openstack/tripleo-common/+/809148 | 14:50 |
| hjensas | k, let's try it | 14:50 |
| opendevreview | John Eckersberg proposed openstack/puppet-tripleo master: Modernize rabbitmq's TLS support https://review.opendev.org/c/openstack/puppet-tripleo/+/812401 | 14:50 |
| Tengu | hjensas: fingers crossed :) | 14:50 |
| opendevreview | Michele Baldessari proposed openstack/puppet-tripleo master: Modernize rabbitmq's TLS support https://review.opendev.org/c/openstack/puppet-tripleo/+/812401 | 14:53 |
| hjensas | Tengu: dang, already. | 14:54 |
| Tengu | hjensas: err.... that's a new thing | 14:54 |
| Tengu | hjensas: am about to jump on a 30-minutes call, I'll check back after it. Thursday is... well. ^^' | 14:55 |
| hjensas | Tengu: ack, I'll poke a bit. | 14:55 |
| *** pmannidi is now known as pmannidi|AFK | 14:56 | |
| opendevreview | Lance Bragstad proposed openstack/tripleo-heat-templates master: Implement project personas in custom cinder policy file https://review.opendev.org/c/openstack/tripleo-heat-templates/+/810460 | 14:58 |
| lbragstad | abishop thanks for the reviews on https://review.opendev.org/c/openstack/tripleo-heat-templates/+/810460 | 14:58 |
| lbragstad | i fixed up the admin naming and check string | 14:59 |
| opendevreview | Michele Baldessari proposed openstack/puppet-tripleo master: Modernize rabbitmq's TLS support https://review.opendev.org/c/openstack/puppet-tripleo/+/812401 | 14:59 |
| lbragstad | i have a couple questions on the deprecated policies - but once you have a look i'll spin another revision | 14:59 |
| opendevreview | Michele Baldessari proposed openstack/puppet-tripleo master: Modernize rabbitmq's TLS support https://review.opendev.org/c/openstack/puppet-tripleo/+/812401 | 15:00 |
| abishop | lbragstad: thanks, looking at ps3 now and was wondering about the deprecated policies | 15:02 |
| lbragstad | if cinder deprecated the generic policy and replaced it with the granular ones, then the granular ones are called at enforcement, right? | 15:03 |
| abishop | lbragstad: but I think you have a "system_system_" dup (typo?) at L1972 and L1975 | 15:03 |
| abishop | yes, in xena the granular ones are enforced | 15:03 |
| opendevreview | Lance Bragstad proposed openstack/tripleo-heat-templates master: Implement project personas in custom cinder policy file https://review.opendev.org/c/openstack/tripleo-heat-templates/+/810460 | 15:04 |
| lbragstad | abishop ok - then that makes me think we should only include the overrides for what cinder calls enforcement on | 15:04 |
| abishop | just need to keep our wits about us in a downstream wallaby backport, where the granular policies aren't present but the depcrated ones are | 15:05 |
| lbragstad | abishop ok - so we could keep those deprecated policies as overrides, but they won't do anything... or we can remove them, which i still don't think is going to do anything, but will be more accurate with what cinder uses | 15:12 |
| hjensas | Tengu: fixed it, need parameter_merge_strategy for ServiceNetMap in the env file where we override the subset of it. | 15:12 |
| opendevreview | Merged openstack/python-tripleoclient stable/train: [Train-Only] Run Validations outside of mistral for the deployment https://review.opendev.org/c/openstack/python-tripleoclient/+/807125 | 15:21 |
| Tengu | hjensas: oh, ok. | 15:25 |
| Tengu | hjensas: I'm available now. shall we jump on a meet' | 15:25 |
| Tengu | ? | 15:25 |
| Tengu | hjensas: added that parameter to tripleo-lab. | 15:26 |
| Tengu | wait.... step 3? | 15:27 |
| Tengu | oh, puppet host config. ok. not yet containers. | 15:27 |
| Tengu | hjensas: ah! | 15:30 |
| Tengu | that one is due to haproxy not listening on the right interface, I think. | 15:30 |
| Tengu | bingo | 15:31 |
| Tengu | something's wrong again in there. probably a fallback to ctlplane happening because of the custom names. | 15:32 |
| Tengu | hjensas: overcloud-resource-registry-puppet.yaml: HaproxyNetwork: ctlplane humpf.... shouldn't it be something more like external_cloud_0 ? | 15:36 |
| Tengu | ah.... seeing the comment: nope. humpf. | 15:36 |
| hjensas | Tengu: I have to drop in a couple of minutes. HaproxyNetwork is hardcoded to ctlplane. So it's not that. | 15:39 |
| Tengu | hjensas: np | 15:40 |
| Tengu | still digging. | 15:40 |
| Tengu | hjensas: the listen IP is wrong | 15:42 |
| Tengu | hjensas: haproxy is listening on the ctlplane instead of external | 15:42 |
| Tengu | ah. vip_data.json: "tripleo::haproxy::public_virtual_ip": "192.168.24.20", | 15:43 |
| Tengu | yeah, so there's def a fallback. | 15:43 |
| *** amoralej is now known as amoralej|off | 15:46 | |
| hjensas | Tengu: could be tripleo_heat_templates/tests/tripleo-ansible/tripleo-ansible/roles/tripleo_hieradata/templates/vip_data.j2:{% set _ = vip_data.__setitem__('public_virtual_ip', (net_vip_map[networks['External']['name_lower']])) %} | 15:46 |
| Tengu | hjensas: oh. I was trying to find where that hieradata thing was set. | 15:47 |
| Tengu | soooo tripleo-ansible? | 15:47 |
| *** ysandeep is now known as ysandeep|dinner | 15:48 | |
| hjensas | Tengu: I think so, but there is a weird thing in the hieradata role. | 15:48 |
| Tengu | hjensas: tripleo_ansible/roles/tripleo_hieradata/templates/vip_data.j2:{% set _ = vip_data.__setitem__('tripleo::haproxy::public_virtual_ip', (net_vip_map[networks['External']['name_lower']])) %} | 15:48 |
| Tengu | that's here. | 15:48 |
| Tengu | External | 15:48 |
| Tengu | yeah, you had it | 15:48 |
| Tengu | hjensas: {% if 'External' in enabled_networks %} | 15:49 |
| Tengu | dang........ | 15:49 |
| Tengu | {% if 'InternalApi' in enabled_networks %} ouch | 15:49 |
| Tengu | yeah.... well. great. | 15:49 |
| hjensas | Tengu: it seems it should do something like L4 ? But looking up a different key in service_net_map ? | 15:50 |
| Tengu | hjensas: we really need a mapping exposing "network.service_net_map_replace" to the "role.networks" thing. | 15:50 |
| Tengu | else we'll have no choice but lock the network.name | 15:51 |
| Tengu | (thus validate the availability of such default names in the network-data and so on) | 15:51 |
| hjensas | Tengu: public_network: external_cloud_0 is in ansible inventory's service_net_map. | 15:52 |
| hjensas | Tengu: {% set _ = vip_data.__setitem__('tripleo::haproxy::public_virtual_ip', (net_vip_map[service_net_map['public_network']])) %} | 15:53 |
| Tengu | hmmmm...... still, we. err. we're wanting to see if that network is enabled on this role. using [hardcoded] network.name | 15:53 |
| hjensas | Tengu: if we do ^^ instead of "if 'External'" | 15:53 |
| Tengu | hmmm. that would... what would happen if no External? | 15:54 |
| Tengu | i.e. is that public_network thing available and set to ctlplane? | 15:54 |
| Tengu | hjensas: btw I'be pushed a quick, ugly script allowing to clean up the OC | 15:54 |
| hjensas | Tengu: if no external the service net map entry would be public_network: ctlplane | 15:54 |
| Tengu | hjensas: soooo it sounds like a plan indeed. | 15:55 |
| hjensas | Tengu: cause we do _service_nets.get('', 'Fallback to ctlplane default') | 15:55 |
| Tengu | right | 15:55 |
| Tengu | dang.... I'll need some painkiller for the headache I'm getting with all that XD | 15:55 |
| Tengu | hjensas: sooooo..... I think most of the "if 'hardcoded' in enabled_networks" might be replaced by such a thing? same for InternalApi? | 15:56 |
| Tengu | hjensas: are you working on a tripleo-ansible patch already? Do you want me to clean the failed deploy? | 15:58 |
| hjensas | Tengu: L33-L39 in vip_data.j2 as well. | 15:58 |
| Tengu | there's "just" that InternalAPI thing. | 15:59 |
| hjensas | Tengu: and StorageNFS and StorageDashboard . Arent stuff using those services? Should there not be an entry in ServiceNetMap instead ... | 16:01 |
| hjensas | Tengu: {# the internal_api_virtual_ip is needed for contrail only #} | 16:02 |
| Tengu | what does it mean, "contrail only"? is it a deprecated thing? or.. ? | 16:02 |
| Tengu | for those StorageNFS and StorageDashboard, I'd be more than happy seeing them being pushed in the ServiceNetMap yeah. | 16:03 |
| Tengu | though it's probably a question for ppl like fultonj or gfidente or fmount[m] :) | 16:03 |
| *** marios is now known as marios|out | 16:04 | |
| Tengu | hjensas: what about set _ = vip_data.__setitem__('ganesha_vip', (net_vip_map[service_net_map['public_network']])) %} ? | 16:04 |
| Tengu | ganesha is also something related to Storage iirc | 16:04 |
| Tengu | (I've edited it on my side, but it might be wrong due to the lack of fallback....) | 16:04 |
| hjensas | Tengu: I added another item on the hackmd vip_data.j2 seem to need a lot of love. I'll have to go now. | 16:06 |
| Tengu | hjensas: yep, just saw it. I'm about to leave as well, but the env will stay at your disposal. | 16:06 |
| Tengu | hjensas: there's a cleanup script if you want now: cleanup-oc0.sh it will take care of the freeIPA + node unprovisionning. | 16:07 |
| Tengu | that will make the whole thing starting clean - you'll need to run tripleo_overcloud_node_provision.sh before the deploy script. | 16:08 |
| Tengu | hjensas: I'll check from time to time on IRC - and I'm off tomorrow.. | 16:08 |
| hjensas | L17 and L25 seem redundant if we replace CephDashboardNetwork: ctlplane with CephDashboardNetwork: {{ _service_nets.get('storage_dashboard', 'ctlplane') }} in the ServiceNetMap defaults. | 16:10 |
| hjensas | Tengu: ok, so I can use your env tomorrow? | 16:11 |
| Tengu | hjensas: sure thing! | 16:11 |
| Tengu | you can crash the OC using the script (it SHOULD work), and redeploy as many times as you want. | 16:11 |
| hjensas | Tengu: thanks, and I'm -> out. | 16:12 |
| Tengu | hjensas: and I'll shutdown the thing tomorrow evening. | 16:12 |
| Tengu | hjensas: ++ see you around! | 16:12 |
| *** hjensas is now known as hjensas|afk | 16:12 | |
| opendevreview | Ananya proposed openstack/tripleo-ci-health-queries master: Bugfix for queries with multiple regx/pattern https://review.opendev.org/c/openstack/tripleo-ci-health-queries/+/812987 | 16:12 |
| opendevreview | Ananya proposed openstack/tripleo-ci-health-queries master: Bugfix for queries with multiple regex/pattern https://review.opendev.org/c/openstack/tripleo-ci-health-queries/+/812987 | 16:14 |
| opendevreview | Ananya proposed openstack/tripleo-ci-health-queries master: Bugfix for queries with multiple regex/pattern https://review.opendev.org/c/openstack/tripleo-ci-health-queries/+/812987 | 16:28 |
| opendevreview | Lance Bragstad proposed openstack/tripleo-heat-templates master: Implement project personas in custom cinder policy file https://review.opendev.org/c/openstack/tripleo-heat-templates/+/810460 | 16:34 |
| lbragstad | abishop done ^ | 16:34 |
| abishop | lbragstad: cool, checking it out now! | 16:46 |
| *** jpena is now known as jpena|off | 16:46 | |
| opendevreview | Ananya proposed openstack/tripleo-ci-health-queries master: Bugfix for queries with multiple regex/pattern https://review.opendev.org/c/openstack/tripleo-ci-health-queries/+/812987 | 16:48 |
| opendevreview | Merged openstack/tripleo-heat-templates master: Added support for Neutron loggings service plugin configuration https://review.opendev.org/c/openstack/tripleo-heat-templates/+/804223 | 16:48 |
| opendevreview | Francesco Pantano proposed openstack/tripleo-heat-templates stable/train: [TRAIN-ONLY] Restart ceph-nfs after the Ceph Upgrade stage https://review.opendev.org/c/openstack/tripleo-heat-templates/+/812526 | 16:56 |
| opendevreview | Francesco Pantano proposed openstack/tripleo-heat-templates stable/train: [TRAIN-ONLY] Restart ceph-nfs after the Ceph Upgrade stage https://review.opendev.org/c/openstack/tripleo-heat-templates/+/812526 | 16:58 |
| *** ysandeep|dinner is now known as ysandeep | 17:18 | |
| opendevreview | Merged openstack/tripleo-ci-health-queries master: Bugfix for queries with multiple regex/pattern https://review.opendev.org/c/openstack/tripleo-ci-health-queries/+/812987 | 17:27 |
| *** rlandy|ruck is now known as rlandy|ruck|brb | 17:36 | |
| opendevreview | Sandeep Yadav proposed openstack/tripleo-quickstart-extras master: Don't pass --network-ports in node provision https://review.opendev.org/c/openstack/tripleo-quickstart-extras/+/813061 | 17:51 |
| *** rlandy|ruck|brb is now known as rlandy|ruck | 18:02 | |
| opendevreview | Merged openstack/os-net-config stable/victoria: Update sriov config service to handle nic partitioned PF https://review.opendev.org/c/openstack/os-net-config/+/812359 | 18:15 |
| opendevreview | Merged openstack/tripleo-heat-templates stable/train: [Train-only] Enable usage of memcache for keystone caching by default https://review.opendev.org/c/openstack/tripleo-heat-templates/+/812940 | 18:15 |
| opendevreview | Merged openstack/tripleo-heat-templates master: Implement project personas in custom octavia policy file https://review.opendev.org/c/openstack/tripleo-heat-templates/+/810455 | 18:15 |
| *** ysandeep is now known as ysandeep|out | 18:39 | |
| opendevreview | Merged openstack/tripleo-ci master: Add scenario001/004 coverage to image uploader changes https://review.opendev.org/c/openstack/tripleo-ci/+/812506 | 19:41 |
| opendevreview | Merged openstack/tripleo-common master: Revert "Revert "Allow for OCI manifests"" https://review.opendev.org/c/openstack/tripleo-common/+/812511 | 19:41 |
| opendevreview | Alex Schultz proposed openstack/tripleo-common stable/wallaby: Revert "Revert "Allow for OCI manifests"" https://review.opendev.org/c/openstack/tripleo-common/+/812961 | 20:00 |
| opendevreview | Alex Schultz proposed openstack/tripleo-common stable/victoria: Revert "Revert "Allow for OCI manifests"" https://review.opendev.org/c/openstack/tripleo-common/+/812962 | 20:00 |
| opendevreview | Alex Schultz proposed openstack/tripleo-common stable/ussuri: Revert "Revert "Allow for OCI manifests"" https://review.opendev.org/c/openstack/tripleo-common/+/812963 | 20:01 |
| opendevreview | Alex Schultz proposed openstack/tripleo-common stable/train: Revert "Revert "Allow for OCI manifests"" https://review.opendev.org/c/openstack/tripleo-common/+/812964 | 20:01 |
| opendevreview | Ronelle Landy proposed openstack/tripleo-quickstart master: Add enable_secure_rbac option to fs020 https://review.opendev.org/c/openstack/tripleo-quickstart/+/813083 | 20:29 |
| opendevreview | Alex Schultz proposed openstack/python-tripleoclient stable/train: Make the "success" message more "success"-like https://review.opendev.org/c/openstack/python-tripleoclient/+/813084 | 20:31 |
| opendevreview | Merged openstack/tripleo-heat-templates stable/wallaby: Remove zaqar https://review.opendev.org/c/openstack/tripleo-heat-templates/+/811195 | 20:36 |
| opendevreview | Ronelle Landy proposed openstack/tripleo-quickstart master: Add enable_secure_rbac option to fs020 https://review.opendev.org/c/openstack/tripleo-quickstart/+/813083 | 20:40 |
| opendevreview | Douglas Viroel proposed openstack/tripleo-quickstart-extras master: WIP - Persist network configuration across reboot https://review.opendev.org/c/openstack/tripleo-quickstart-extras/+/813085 | 20:58 |
| *** dviroel is now known as dviroel|out | 21:02 | |
| opendevreview | Ade Lee proposed openstack/puppet-tripleo master: Add option to configure snmpd auth type https://review.opendev.org/c/openstack/puppet-tripleo/+/813087 | 21:14 |
| opendevreview | Merged openstack/puppet-tripleo master: Revert "Pin puppet-redis to 8.0.0" https://review.opendev.org/c/openstack/puppet-tripleo/+/806609 | 21:20 |
| opendevreview | Ade Lee proposed openstack/tripleo-heat-templates master: Add auth type for snmpd readonly user https://review.opendev.org/c/openstack/tripleo-heat-templates/+/813089 | 21:29 |
| *** pmannidi|AFK is now known as pmannidi | 23:14 | |
| opendevreview | Merged openstack/puppet-tripleo stable/wallaby: haproxy: Add missing default_backend option https://review.opendev.org/c/openstack/puppet-tripleo/+/812567 | 23:56 |
Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!