| *** rlandy is now known as rlandy|out | 00:07 | |
| opendevreview | Takashi Kajinami proposed openstack/tripleo-heat-templates master: WIP: Offload cron job to host crond https://review.opendev.org/c/openstack/tripleo-heat-templates/+/858929 | 00:14 |
|---|---|---|
| opendevreview | James Slagle proposed openstack/tripleo-ansible master: Add initial standalone playbooks and inventory for a compute node https://review.opendev.org/c/openstack/tripleo-ansible/+/840509 | 00:16 |
| opendevreview | James Slagle proposed openstack/tripleo-ansible master: tripleo_iscsid: update for standalone roles https://review.opendev.org/c/openstack/tripleo-ansible/+/842437 | 00:17 |
| opendevreview | James Slagle proposed openstack/tripleo-ansible master: tripleo_logrotate_crond: add for standalone roles https://review.opendev.org/c/openstack/tripleo-ansible/+/841997 | 00:18 |
| opendevreview | James Slagle proposed openstack/tripleo-ansible master: tripleo_nova_compute: add for standalone roles https://review.opendev.org/c/openstack/tripleo-ansible/+/843659 | 00:19 |
| opendevreview | James Slagle proposed openstack/tripleo-ansible master: tripleo_nova_libvirt: make pid ns configurable https://review.opendev.org/c/openstack/tripleo-ansible/+/859739 | 00:22 |
| opendevreview | Takashi Kajinami proposed openstack/tripleo-heat-templates master: Drop legacy networking resources https://review.opendev.org/c/openstack/tripleo-heat-templates/+/859520 | 00:41 |
| opendevreview | Brendan Shephard proposed openstack/tripleo-heat-templates stable/wallaby: Fix tls-e CA cert declaration for OVN https://review.opendev.org/c/openstack/tripleo-heat-templates/+/858039 | 00:46 |
| opendevreview | Takashi Kajinami proposed openstack/tripleo-heat-templates master: Remove unused ManagedEndpoints from stack outputs https://review.opendev.org/c/openstack/tripleo-heat-templates/+/859740 | 00:50 |
| opendevreview | Takashi Kajinami proposed openstack/tripleo-heat-templates master: Remove unused ManagedEndpoints from stack outputs https://review.opendev.org/c/openstack/tripleo-heat-templates/+/859740 | 00:51 |
| opendevreview | Merged openstack/tripleo-ansible stable/train: [TRAIN-ONLY] - Fix ceph-ansible permissions when workdir is created https://review.opendev.org/c/openstack/tripleo-ansible/+/859443 | 01:57 |
| opendevreview | Takashi Kajinami proposed openstack/python-tripleoclient master: overcloud-deploy: Only support pre-provisioned node https://review.opendev.org/c/openstack/python-tripleoclient/+/839464 | 02:20 |
| opendevreview | Takashi Kajinami proposed openstack/tripleo-heat-templates master: Drop environment for the removed upgrade converge step https://review.opendev.org/c/openstack/tripleo-heat-templates/+/859743 | 02:25 |
| opendevreview | Takashi Kajinami proposed openstack/python-tripleoclient master: Remove logic to detect legacy network resources https://review.opendev.org/c/openstack/python-tripleoclient/+/859745 | 02:49 |
| *** yadnesh|away is now known as yadnesh | 03:36 | |
| *** ysandeep|sick is now known as ysandeep | 03:48 | |
| opendevreview | Bhagyashri Shewale proposed openstack/tripleo-quickstart master: Set NovaHWMachineType: x86_64=q35 for fs001,020 and 035 https://review.opendev.org/c/openstack/tripleo-quickstart/+/859581 | 03:51 |
| opendevreview | Rabi Mishra proposed openstack/tripleo-ansible master: Drop role_name option from tripleo_ovn_mac_addresses https://review.opendev.org/c/openstack/tripleo-ansible/+/859747 | 04:26 |
| *** ysandeep is now known as ysandeep|afk | 04:45 | |
| opendevreview | Luca Miccini proposed openstack/tripleo-heat-templates master: Allow overriding corosync token_timeout https://review.opendev.org/c/openstack/tripleo-heat-templates/+/859568 | 05:06 |
| ykarel | ramishra, tkajinam when you get chance please review https://review.opendev.org/c/openstack/tripleo-heat-templates/+/859288 | 05:13 |
| *** amoralej|off is now known as amoralej | 06:07 | |
| *** ysandeep|afk is now known as ysandeep | 06:19 | |
| opendevreview | Takashi Kajinami proposed openstack/tripleo-ci master: Collect configuration files for host crond service https://review.opendev.org/c/openstack/tripleo-ci/+/859771 | 06:22 |
| opendevreview | Takashi Kajinami proposed openstack/tripleo-heat-templates master: WIP: Offload cron job to host crond https://review.opendev.org/c/openstack/tripleo-heat-templates/+/858929 | 06:22 |
| opendevreview | Takashi Kajinami proposed openstack/tripleo-heat-templates master: WIP: Offload cron job to host crond https://review.opendev.org/c/openstack/tripleo-heat-templates/+/858929 | 06:24 |
| marios | bshephar: o/ just workflowed that https://review.opendev.org/c/openstack/python-tripleoclient/+/852306 in case you were waiting for something ? | 06:40 |
| bshephar | marios: o/ hey man. Thanks. It wasn't urgent, but appreciate that. | 06:43 |
| bshephar | I'll be back later this evening, if you had some time around the multi-rhel call we could have a quick sync on the upstream CI job? | 06:44 |
| marios | bshephar: yeah i was going to say something similar sounds good, though it will be crazy hours for you then... maybe tomorrow morning first thing my time may work better for you? anyway we can decide later o/ | 06:45 |
| *** jpena|off is now known as jpena | 06:51 | |
| *** lbragstad8 is now known as lbragstad | 06:56 | |
| opendevreview | Mridula Joshi proposed openstack/tripleo-heat-templates stable/train: Add glance_api_cron container https://review.opendev.org/c/openstack/tripleo-heat-templates/+/858150 | 06:58 |
| opendevreview | Mridula Joshi proposed openstack/tripleo-heat-templates stable/train: Add glance_api_cron container https://review.opendev.org/c/openstack/tripleo-heat-templates/+/858150 | 06:59 |
| opendevreview | Karthik S proposed openstack/os-net-config master: WIP: Add nmstate provider support https://review.opendev.org/c/openstack/os-net-config/+/859552 | 07:02 |
| mrjoshi | hey, looking for reviews on these patces https://review.opendev.org/c/openstack/puppet-tripleo/+/858141, https://review.opendev.org/c/openstack/tripleo-heat-templates/+/858150 | 07:21 |
| opendevreview | Bhagyashri Shewale proposed openstack/tripleo-heat-templates master: Create separate ovb-ha file for downstream https://review.opendev.org/c/openstack/tripleo-heat-templates/+/859783 | 07:34 |
| opendevreview | Bhagyashri Shewale proposed openstack/tripleo-quickstart master: Set NovaHWMachineType: x86_64=q35 for fs001,020 and 035 https://review.opendev.org/c/openstack/tripleo-quickstart/+/859581 | 07:45 |
| opendevreview | Francesco Pantano proposed openstack/tripleo-heat-templates stable/train: [TRAIN-ONLY] - Fix ceph-ansible group_vars/ permissions https://review.opendev.org/c/openstack/tripleo-heat-templates/+/859444 | 07:51 |
| *** ysandeep is now known as ysandeep|lunch | 08:05 | |
| opendevreview | Merged openstack/tripleo-heat-templates master: Enable dns_domain_ports extension by default https://review.opendev.org/c/openstack/tripleo-heat-templates/+/859288 | 08:54 |
| opendevreview | Merged openstack/tripleo-heat-templates master: Move releasenote to correct directory https://review.opendev.org/c/openstack/tripleo-heat-templates/+/859363 | 08:54 |
| opendevreview | Cedric Jeanneret proposed openstack/tripleo-ansible master: Re-add AUDIT_WRITE capability to containers https://review.opendev.org/c/openstack/tripleo-ansible/+/859289 | 09:05 |
| Tengu | chkumar|rover: heya! do you know anything about this tempest failure on stable/train? https://storage.bhs.cloud.ovh.net/v1/AUTH_dcaab5e32b234d56b626f72581e3644c/zuul_opendev_logs_462/859402/1/check/tripleo-ci-centos-8-scenario010-ovn-provider-standalone-train/462444c/logs/undercloud/var/log/tempest/stestr_results.html | 09:17 |
| Tengu | linked to octavia wanting to insert some header apparently... | 09:18 |
| Tengu | smells bad. | 09:18 |
| chkumar|rover | Tengu: nope | 09:18 |
| chkumar|rover | frenzyfriday|ruck: ^^ can you open a bug and skip this test | 09:19 |
| chkumar|rover | till we investigate | 09:19 |
| Tengu | thanks :) | 09:19 |
| frenzyfriday|ruck | checking | 09:19 |
| fpantano | chkumar|rover: ramishra_ tkajinam can you check -> https://review.opendev.org/c/openstack/tripleo-heat-templates/+/859444 | 09:21 |
| fpantano | it should be ready to go now ^ | 09:21 |
| opendevreview | Jiri Podivin proposed openstack/tripleo-heat-templates master: DNM Ansible configuration of gnocchi httpd https://review.opendev.org/c/openstack/tripleo-heat-templates/+/858882 | 09:22 |
| frenzyfriday|ruck | checking | 09:31 |
| opendevreview | Francesco Pantano proposed openstack/tripleo-heat-templates stable/train: [TRAIN-ONLY] - Fix ceph-ansible group_vars/ permissions https://review.opendev.org/c/openstack/tripleo-heat-templates/+/859444 | 09:39 |
| *** ysandeep|lunch is now known as ysandeep | 09:44 | |
| opendevreview | Cedric Jeanneret proposed openstack/tripleo-ansible master: Re-add AUDIT_WRITE capability to containers https://review.opendev.org/c/openstack/tripleo-ansible/+/859289 | 09:44 |
| opendevreview | Ananya proposed openstack/openstack-tempest-skiplist master: Skips octavia_tempest_plugin.tests.scenario.v2.test_listener.ListenerScenarioTest.test_listener_CRUD for scenario010-ovn-provider-standalone-train https://review.opendev.org/c/openstack/openstack-tempest-skiplist/+/859825 | 09:49 |
| frenzyfriday|ruck | chkumar|rover, Tengu - https://review.opendev.org/c/openstack/openstack-tempest-skiplist/+/859825 to skip the octavia test. Looks like there was a similar bug https://bugs.launchpad.net/tripleo/+bug/1895705. I will take a look. | 09:51 |
| bshephar | marios: Tonight is fine man. Either before or after the multi-rhel call works for me. | 09:51 |
| Tengu | frenzyfriday|ruck: thanks | 09:52 |
| opendevreview | Luca Miccini proposed openstack/puppet-tripleo master: Allow passing aditional arguments to wsrep_provider_options https://review.opendev.org/c/openstack/puppet-tripleo/+/859553 | 09:53 |
| opendevreview | Luca Miccini proposed openstack/tripleo-heat-templates master: DNM: Testing patches for LP #1987092 https://review.opendev.org/c/openstack/tripleo-heat-templates/+/859712 | 09:55 |
| marios | ack will have to be after bshephar have scrum before | 10:04 |
| opendevreview | Jiri Podivin proposed openstack/validations-libs master: VF container security and stability adjustment https://review.opendev.org/c/openstack/validations-libs/+/854782 | 10:07 |
| yadnesh | hey chkumar|rover could you please take a look at this failure https://73ff8251b57577f31d40-ab9ce58930e4dc9293c41b1c971f724e.ssl.cf2.rackcdn.com/857865/1/check/tripleo-ci-centos-8-undercloud-ffu-wallaby/9b712f9/logs/undercloud/home/zuul/install_packages.sh.log | 10:09 |
| yadnesh | a couple of patches are failing on same job | 10:10 |
| chkumar|rover | yadnesh: will take a look, thanks ! | 10:15 |
| opendevreview | Merged openstack/python-tripleoclient master: Add timeout arg to node delete commands https://review.opendev.org/c/openstack/python-tripleoclient/+/852306 | 10:15 |
| chkumar|rover | yadnesh: weired 2022-09-29 08:12:23 | Warning: failed loading '/etc/yum.repos.d/delorean.repo', skipping. | 10:18 |
| chkumar|rover | https://73ff8251b57577f31d40-ab9ce58930e4dc9293c41b1c971f724e.ssl.cf2.rackcdn.com/857865/1/check/tripleo-ci-centos-8-undercloud-ffu-wallaby/9b712f9/logs/undercloud/etc/yum.repos.d/delorean.repo | 10:19 |
| yadnesh | that doesn't look good | 10:19 |
| opendevreview | Jiri Podivin proposed openstack/validations-libs master: VF container security and stability adjustment https://review.opendev.org/c/openstack/validations-libs/+/854782 | 10:20 |
| frenzyfriday|ruck | hey Arx, good morning. Do you remember anything about this bug: https://bugs.launchpad.net/tripleo/+bug/1893192 (stuff from looong time back) This is marked duplicate of https://bugs.launchpad.net/tripleo/+bug/1896277 but we are hitting this again. | 10:21 |
| opendevreview | Merged openstack/openstack-tempest-skiplist master: Skips octavia_tempest_plugin.tests.scenario.v2.test_listener.ListenerScenarioTest.test_listener_CRUD for scenario010-ovn-provider-standalone-train https://review.opendev.org/c/openstack/openstack-tempest-skiplist/+/859825 | 10:22 |
| frenzyfriday|ruck | According to the original bug https://bugs.launchpad.net/tripleo/+bug/1896277 - tempest was raising tempest.lib.exceptions.NotImplemented. So ignoring failures due to NotImplemented worked. But in this case tempest is raising tempest.lib.exceptions.BadRequest | 10:22 |
| chkumar|rover | yadnesh: can you open a bug for the same | 10:22 |
| yadnesh | chkumar|rover, sure | 10:23 |
| frenzyfriday|ruck | sorry, ^ arxcruz | 10:23 |
| mrjoshi | #tripleo Hi, requesting for reviews on these patches https://review.opendev.org/c/openstack/puppet-tripleo/+/858141, https://review.opendev.org/c/openstack/tripleo-heat-templates/+/858150 | 10:23 |
| chkumar|rover | yadnesh: you can propose similar fix for cs8 release file https://opendev.org/openstack/tripleo-quickstart/commit/f50d529ccafd173bec580d806b171ffffacd564a | 10:24 |
| chkumar|rover | yadnesh: update the same in wallaby train release file https://opendev.org/openstack/tripleo-quickstart/src/branch/master/config/release/tripleo-ci/CentOS-8 | 10:24 |
| arxcruz | frenzyfriday|ruck chkumar|rover remember, tests to be skipped on train need cix approval | 10:24 |
| chkumar|rover | it will fix the issue | 10:24 |
| chkumar|rover | arxcruz: yes good point | 10:24 |
| frenzyfriday|ruck | oh | 10:25 |
| chkumar|rover | arxcruz: test is already skipped on periodic-tripleo-ci-centos-8-scenario010-ovn-provider-standalone-train | 10:25 |
| chkumar|rover | so i think we are good there | 10:25 |
| chkumar|rover | https://review.opendev.org/c/openstack/openstack-tempest-skiplist/+/859825/1/roles/validate-tempest/vars/tempest_skip.yml#697 | 10:26 |
| arxcruz | chkumar|rover frenzyfriday|ruck does https://review.rdoproject.org/r/c/rdo-jobs/+/29619/1/zuul.d/standalone-jobs.yaml still up? | 10:26 |
| frenzyfriday|ruck | arxcruz, yep | 10:26 |
| arxcruz | maybe there were some change in the job definition, let me check | 10:26 |
| yadnesh | chkumar|rover, sure | 10:26 |
| chkumar|rover | arxcruz: yes, thank you for helping on this :-) | 10:27 |
| frenzyfriday|ruck | arxcruz, https://github.com/openstack/tripleo-ci/blob/master/zuul.d/standalone-jobs.yaml#L683 we have this one now | 10:27 |
| arxcruz | yeah, i see | 10:27 |
| arxcruz | frenzyfriday|ruck do you have logs? | 10:27 |
| frenzyfriday|ruck | In the original bug tempest was raising not_implemented_is_error, but in this case it raises badRequest. maybe thats why loadbalancer-feature-enabled.not_implemented_is_error: false doesnt work? | 10:27 |
| frenzyfriday|ruck | logs: https://storage.bhs.cloud.ovh.net/v1/AUTH_dcaab5e32b234d56b626f72581e3644c/zuul_opendev_logs_462/859402/1/check/tripleo-ci-centos-8-scenario010-ovn-provider-standalone-train/462444c/logs/undercloud/var/log/tempest/stestr_results.html | 10:27 |
| *** rlandy|out is now known as rlandy | 10:28 | |
| arxcruz | frenzyfriday|ruck https://github.com/openstack/tripleo-ci/blob/master/zuul.d/standalone-jobs.yaml#L681 | 10:37 |
| arxcruz | need to either remove this, or change it to HTTP | 10:37 |
| arxcruz | and check if it will continue to fail | 10:37 |
| frenzyfriday|ruck | arxcruz, thank you! Lemme change and try | 10:38 |
| arxcruz | the octavia plugin is a little bit old, it's failing because this is set to TCP, and failing with insert-headers is not a valid option for a TCP protocol | 10:38 |
| arxcruz | frenzyfriday|ruck https://github.com/openstack/octavia-tempest-plugin/blob/1.2.0/octavia_tempest_plugin/tests/scenario/v2/test_listener.py#L55-L57 | 10:39 |
| arxcruz | and https://github.com/openstack/octavia-tempest-plugin/blob/1.2.0/octavia_tempest_plugin/tests/scenario/v2/test_listener.py#L120 | 10:39 |
| opendevreview | Yadnesh Kulkarni proposed openstack/tripleo-quickstart master: Use nodepool_rdo_proxy url instead of rdo trunk url https://review.opendev.org/c/openstack/tripleo-quickstart/+/859829 | 10:39 |
| arxcruz | so, there is no if the protocol is tcp or http, it will try to insert the header | 10:40 |
| opendevreview | Ananya proposed openstack/tripleo-ci master: WIP: Removing loadbalancer-feature-enabled.l4_protocol: 'TCP' https://review.opendev.org/c/openstack/tripleo-ci/+/859830 | 10:43 |
| opendevreview | Ananya proposed openstack/openstack-tempest-skiplist master: Revert "Skips octavia_tempest_plugin.tests.scenario.v2.test_listener.ListenerScenarioTest.test_listener_CRUD for scenario010-ovn-provider-standalone-train" https://review.opendev.org/c/openstack/openstack-tempest-skiplist/+/859796 | 10:44 |
| opendevreview | Yadnesh Kulkarni proposed openstack/tripleo-quickstart master: Use nodepool_rdo_proxy url instead of rdo trunk url https://review.opendev.org/c/openstack/tripleo-quickstart/+/859829 | 10:50 |
| lecris[m] | `ERROR: Internal Error: heatclient.exc.HTTPBadRequest: ERROR: Internal Error` where can I check the log for this? | 10:51 |
| lecris[m] | Nvm it was my heat template syntax issue | 10:54 |
| *** dviroel|out is now known as dviroel | 11:17 | |
| opendevreview | yatin proposed openstack/tripleo-heat-templates stable/wallaby: Enable dns_domain_ports extension by default https://review.opendev.org/c/openstack/tripleo-heat-templates/+/859834 | 11:19 |
| opendevreview | Jiri Podivin proposed openstack/validations-libs master: VF container security and stability adjustment https://review.opendev.org/c/openstack/validations-libs/+/854782 | 11:20 |
| opendevreview | Yadnesh Kulkarni proposed openstack/tripleo-quickstart master: Use nodepool_rdo_proxy url instead of rdo trunk url https://review.opendev.org/c/openstack/tripleo-quickstart/+/859829 | 11:43 |
| opendevreview | Arx Cruz proposed openstack/tripleo-quickstart-extras master: Remove pingtest from tripleo-quickstart-extras https://review.opendev.org/c/openstack/tripleo-quickstart-extras/+/859835 | 11:53 |
| *** amoralej is now known as amoralej|lunch | 12:10 | |
| *** ysandeep is now known as ysandeep|afk | 12:12 | |
| *** dviroel is now known as dviroel|doc_appt | 12:27 | |
| opendevreview | Bhagyashri Shewale proposed openstack/tripleo-heat-templates master: Create separate ovb-ha file for downstream https://review.opendev.org/c/openstack/tripleo-heat-templates/+/859783 | 12:31 |
| opendevreview | Bhagyashri Shewale proposed openstack/tripleo-quickstart master: Set NovaHWMachineType: x86_64=q35 for fs001,020 and 035 https://review.opendev.org/c/openstack/tripleo-quickstart/+/859581 | 12:32 |
| opendevreview | Jiri Podivin proposed openstack/validations-libs master: VF container security and stability adjustment https://review.opendev.org/c/openstack/validations-libs/+/854782 | 12:41 |
| opendevreview | Bhagyashri Shewale proposed openstack/tripleo-quickstart master: Set NovaHWMachineType: x86_64=q35 for fs001,020 and 035 https://review.opendev.org/c/openstack/tripleo-quickstart/+/859581 | 12:46 |
| *** dasm|off is now known as dasm | 12:48 | |
| *** ysandeep|afk is now known as ysandeep | 12:49 | |
| fpantano | chkumar|rover: o/ is https://storage.gra.cloud.ovh.net/v1/AUTH_dcaab5e32b234d56b626f72581e3644c/zuul_opendev_logs_34f/859444/4/check/tripleo-ci-centos-8-scenario010-ovn-provider-standalone-train/34f9cb9/logs/undercloud/var/log/tempest/tempest_run.log a known issue? | 12:49 |
| fpantano | I see this consistent failure in the upstream (train) | 12:50 |
| chkumar|rover | fpantano: we have a bug opened for that | 12:52 |
| chkumar|rover | we just skipped the failing test | 12:53 |
| chkumar|rover | fpantano: https://bugs.launchpad.net/tripleo/+bug/1991232 | 12:53 |
| chkumar|rover | fpantano: you can recheck the patch | 12:53 |
| chkumar|rover | it will pass | 12:53 |
| fpantano | chkumar|rover: ack thanks I'll do | 12:54 |
| fpantano | I see (on the same patch) | 12:54 |
| fpantano | https://zuul.opendev.org/t/openstack/build/b86004aa1d7a45cd9845e488a87e073b <- also failing | 12:55 |
| fpantano | but not sure you already tracked it | 12:55 |
| chkumar|rover | looking at this one | 12:55 |
| chkumar|rover | fpantano: we fixed tripleo-ci-centos-8-scenario000-multinode-oooq-container-updates-train also yesterday | 12:56 |
| chkumar|rover | https://review.opendev.org/c/openstack/tripleo-quickstart/+/859516 | 12:56 |
| chkumar|rover | fpantano: thanks for checking in with us, recheck will make the job green | 12:57 |
| fpantano | chkumar|rover: nice thanks, I was pretty sure you already tracked the two issues :D | 12:58 |
| fpantano | chkumar|rover: thanks for the help | 12:58 |
| fpantano | fultonj: FYI ^ | 12:58 |
| chkumar|rover | fpantano: skip list is alreadt merfed https://review.opendev.org/c/openstack/openstack-tempest-skiplist/+/859825 | 12:58 |
| fultonj | thank you chkumar|rover | 12:59 |
| fpantano | ah, so it's just matter of rechecking the patch | 12:59 |
| chkumar|rover | yup correct! | 12:59 |
| fpantano | ++ cool thanks! | 12:59 |
| *** amoralej|lunch is now known as amoralej | 13:06 | |
| opendevreview | Arx Cruz proposed openstack/openstack-tempest-skiplist master: Update releases in tempest allow list https://review.opendev.org/c/openstack/openstack-tempest-skiplist/+/859846 | 13:16 |
| opendevreview | Yadnesh Kulkarni proposed openstack/tripleo-heat-templates master: Add new paramater "CeilometerEnableTenantDiscovery" https://review.opendev.org/c/openstack/tripleo-heat-templates/+/856840 | 13:16 |
| opendevreview | mbu proposed openstack/python-tripleoclient master: Execute undercloud preflight validations via VF binary https://review.opendev.org/c/openstack/python-tripleoclient/+/859608 | 13:55 |
| opendevreview | James Slagle proposed openstack/tripleo-ansible master: Add initial standalone playbooks and inventory for a compute node https://review.opendev.org/c/openstack/tripleo-ansible/+/840509 | 13:57 |
| opendevreview | James Slagle proposed openstack/tripleo-ansible master: tripleo_compute_node: add for standalone roles https://review.opendev.org/c/openstack/tripleo-ansible/+/859858 | 13:57 |
| *** dkehn_ is now known as dkehn | 14:04 | |
| opendevreview | Ronelle Landy proposed openstack/tripleo-ci master: Revert "Victoria tear down : emit release file" https://review.opendev.org/c/openstack/tripleo-ci/+/859800 | 14:15 |
| *** marios is now known as marios|call | 14:18 | |
| opendevreview | Jiri Podivin proposed openstack/validations-libs master: VF container security and stability adjustment https://review.opendev.org/c/openstack/validations-libs/+/854782 | 14:24 |
| *** ysandeep is now known as ysandeep|dinner | 14:28 | |
| *** marios|call is now known as marios | 14:45 | |
| *** dviroel|doc_appt is now known as dviroel|doc | 14:46 | |
| *** dviroel|doc is now known as dviroel | 14:46 | |
| lecris[m] | hjensas: Tengu Can you check https://bugs.launchpad.net/tripleo/+bug/1988550. Ipa fails to deploy with custom network names | 14:58 |
| Tengu | errrr it's supposed to be working... | 14:59 |
| Tengu | I mean, we extensively tested that feature back then | 14:59 |
| lecris[m] | The missmatch is that `fqdn_$NETWORK` gives something like `controller-0.internalapi.openstack.lab` | 15:02 |
| Tengu | I think this is expected. | 15:02 |
| lecris[m] | And if a second stack is created this could differ as `controller-0.internalapi.openstack2.lab` | 15:03 |
| Tengu | my lab is using only custom names for instance... and we could deploy the TLS-e usinf IPA with it. | 15:03 |
| lecris[m] | So we have to different `name_lower` networks. `internalapi_stacka` and `internalapi_stackb` | 15:03 |
| Tengu | (in a call, then EOD - but it's weird) | 15:04 |
| hjensas | lecris[m]: I think you need to set CloudName, CloudNameInternal, CloudNameStorageManagement and CloudName{{network.name}}. | 15:04 |
| lecris[m] | But in krb-service-principles jinja, it basically creates `controller-0.internalapistacka.openstack2.lab` | 15:04 |
| lecris[m] | Those are set | 15:05 |
| lecris[m] | But those set for `overcloud....`, not for `controller-0....` | 15:05 |
| lecris[m] | Tengu: It very much depends on what custom names are used apparently | 15:06 |
| opendevreview | Jiri Podivin proposed openstack/validations-libs master: VF container security and stability adjustment https://review.opendev.org/c/openstack/validations-libs/+/854782 | 15:12 |
| lecris[m] | Equivalently, [this part could be fixed to use the actual fqdn | 15:12 |
| lecris[m] | * [this part](https://opendev.org/x/tripleo-ipa/src/commit/e3afdf14b6569d04c02481c4c5aeb11e854b5043/tripleo_ipa/ansible_plugins/filter/service_metadata.py#L65-L69) could | 15:13 |
| lecris[m] | Or rather this section: https://opendev.org/x/tripleo-ipa/src/commit/e3afdf14b6569d04c02481c4c5aeb11e854b5043/tripleo_ipa/ansible_plugins/filter/service_metadata.py#L70-L75 | 15:14 |
| hjensas | lecris[m]: I see the CI uses different 'name' for the network as well. see: https://opendev.org/openstack/tripleo-heat-templates/src/branch/master/ci/custom_names_network_data.yaml | 15:15 |
| lecris[m] | But the name is consistent with the `name_lower` | 15:17 |
| lecris[m] | We should test by changing `dns_domain` to `tenantcloud.ooo1.test` for example instead of `tenantcloud1.ooo.test` | 15:17 |
| hjensas | yes, name is CamelCase and name_lower use under_score i.e ExternalCloudA -> external_cloud_a | 15:18 |
| lecris[m] | But that is hard-coding a standard that otherwise is not picked up by `fqdn_$NETWORK` | 15:19 |
| lecris[m] | The CI does not pick up on these inconsistencies | 15:20 |
| hjensas | lecris[m]: fqdn_$NETWORK becomes fqdn_external_cloud_a - It's getting $NETWORK from the ServiceNetMap | 15:20 |
| hjensas | ServiceNetMap contains name_lower. | 15:21 |
| opendevreview | Ronelle Landy proposed openstack/tripleo-ci master: Revert "Victoria tear down : emit release file" https://review.opendev.org/c/openstack/tripleo-ci/+/859800 | 15:21 |
| lecris[m] | I mean it can happen that `fqdn_external_cloud_a=tenantcloud.ooo2.test`, but this [1] hard-codes to something else | 15:22 |
| lecris[m] | [1] https://opendev.org/x/tripleo-ipa/src/commit/e3afdf14b6569d04c02481c4c5aeb11e854b5043/tripleo_ipa/ansible_plugins/filter/service_metadata.py#L70-L75 | 15:22 |
| lecris[m] | `hostname.interface.domain` isntead of `hostname.fqdn_$NETWORK` value | 15:23 |
| hjensas | lecris[m]: it is assuming that name_lower: external_cloud_a means dns_name exteranlclouda.example.net is aligned. | 15:24 |
| hjensas | lecris[m]: It's not very nice. | 15:24 |
| opendevreview | Ronelle Landy proposed openstack/tripleo-heat-templates stable/wallaby: DNM - Testing LP #1991240 https://review.opendev.org/c/openstack/tripleo-heat-templates/+/859867 | 15:24 |
| rlandy | marios: pojadhav: frenzyfriday|ruck: ^^ fyi | 15:24 |
| lecris[m] | hjensas: Yes that should be fixed. Issue is that `compact_service_...` stripped away the underscores_ which could have been used to construct back `fqdn_$NETWORK` | 15:26 |
| marios | ack rlandy noting | 15:27 |
| rlandy | rcastillo" ^^ you'll need that for tomorrow | 15:27 |
| rlandy | rr | 15:27 |
| lecris[m] | hjensas: Maybe we can convert them all to `managed_service_....`? | 15:28 |
| hjensas | lecris[m]: I'm having issues getting full context again. | 15:35 |
| hjensas | lecris[m]: should https://opendev.org/openstack/tripleo-heat-templates/src/commit/a6576e1c5980b4ddcf292599037eb089975aa76f/extraconfig/nova_metadata/krb-service-principals/role.role.j2.yaml#L97 | 15:35 |
| hjensas | lecris[m]: have the "fqdn" and instead of doing concat('compact_service_', $.service), $.network.replace('_', '')) it could do oncat('compact_service_', $.service), $fqdns.get($.network)) | 15:36 |
| *** marios is now known as marios|out | 15:37 | |
| lecris[m] | Hmm, that would mess up with the implementation in https://opendev.org/x/tripleo-ipa/src/commit/e3afdf14b6569d04c02481c4c5aeb11e854b5043/tripleo_ipa/ansible_plugins/filter/service_metadata.py#L70-L75 | 15:38 |
| lecris[m] | Instead of a `compact_service_` it should be similar to `managed_service_` above it | 15:38 |
| lecris[m] | The thing is that `managed_service_` is unique per networks, while `compact_service` generates for each node | 15:39 |
| lecris[m] | So instead of `fqdn`, it should be `fqdn` without the hostname part | 15:40 |
| lecris[m] | hjensas: Or maybe that works if it can pick up from `host_vars` (`fqdn_$NETWORK`) | 15:44 |
| lecris[m] | I can try and see what your snippet does | 15:45 |
| hjensas | lecris[m]: I may be completely way off. :) I think I am actually. | 15:46 |
| opendevreview | Arx Cruz proposed openstack/tripleo-ci master: WIP: Removing loadbalancer-feature-enabled.l4_protocol: 'TCP' https://review.opendev.org/c/openstack/tripleo-ci/+/859830 | 15:55 |
| *** amoralej is now known as amoralej|off | 15:55 | |
| *** ysandeep|dinner is now known as ysandeep|out | 15:56 | |
| *** jpena is now known as jpena|off | 16:05 | |
| *** yadnesh is now known as yadnesh|away | 16:18 | |
| opendevreview | Yadnesh Kulkarni proposed openstack/tripleo-heat-templates master: Add new paramater "CeilometerEnableTenantDiscovery" https://review.opendev.org/c/openstack/tripleo-heat-templates/+/856840 | 16:19 |
| *** chkumar|rover is now known as raukadah | 17:15 | |
| opendevreview | Arx Cruz proposed openstack/tripleo-ci master: Removing loadbalancer-feature-enabled.l4_protocol: 'TCP' https://review.opendev.org/c/openstack/tripleo-ci/+/859830 | 17:31 |
| opendevreview | Arx Cruz proposed openstack/tripleo-ci master: Removing loadbalancer-feature-enabled.l4_protocol: 'TCP' https://review.opendev.org/c/openstack/tripleo-ci/+/859830 | 17:31 |
| opendevreview | John Fulton proposed openstack/tripleo-ansible master: Update tripleo_nova_libvirt to configure nova_libvirt_init_secret https://review.opendev.org/c/openstack/tripleo-ansible/+/858585 | 18:37 |
| opendevreview | John Fulton proposed openstack/tripleo-ansible master: Update tripleo_nova_libvirt to configure nova_libvirt_init_secret https://review.opendev.org/c/openstack/tripleo-ansible/+/858585 | 18:40 |
| *** dviroel is now known as dviroel|afk | 19:42 | |
| *** dviroel|afk is now known as dviroel | 20:52 | |
| *** tosky_ is now known as tosky | 21:24 | |
| *** dasm is now known as dasm|off | 21:24 | |
| *** rlandy is now known as rlandy|out | 22:10 | |
| opendevreview | Merged openstack/puppet-tripleo stable/train: Glance: Include the required classes for image cache https://review.opendev.org/c/openstack/puppet-tripleo/+/858141 | 23:54 |
| *** ysandeep|out is now known as ysandeep | 23:58 | |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!