| opendevreview | Brendan Shephard proposed openstack/python-tripleoclient master: WIP - Replace non-inclusive language https://review.opendev.org/c/openstack/python-tripleoclient/+/820078 | 00:03 |
|---|---|---|
| opendevreview | Steve Baker proposed openstack/tripleo-common master: Auto-detect partition types for mounting more images https://review.opendev.org/c/openstack/tripleo-common/+/828206 | 01:36 |
| *** rlandy|ruck is now known as rlandy|out | 02:03 | |
| opendevreview | Harald Jensås proposed openstack/tripleo-quickstart-extras master: Add undercloud_parameter_defaults var undercloud-deploy https://review.opendev.org/c/openstack/tripleo-quickstart-extras/+/828197 | 02:04 |
| opendevreview | Brendan Shephard proposed openstack/python-tripleoclient master: Move baremetal Introspect commands from Ansible https://review.opendev.org/c/openstack/python-tripleoclient/+/828207 | 02:09 |
| opendevreview | Brendan Shephard proposed openstack/python-tripleoclient master: Move baremetal Introspect commands from Ansible https://review.opendev.org/c/openstack/python-tripleoclient/+/828207 | 02:14 |
| *** pojadhav|afk is now known as pojadhav | 02:36 | |
| opendevreview | Merged openstack/tripleo-ansible stable/wallaby: [Only Wallaby] BnR added ramdisk_kernel_arguments to the restoration process https://review.opendev.org/c/openstack/tripleo-ansible/+/827151 | 03:07 |
| opendevreview | Merged openstack/tripleo-heat-templates stable/wallaby: Start the neutron metadata agent with cgroupns host https://review.opendev.org/c/openstack/tripleo-heat-templates/+/827969 | 03:10 |
| opendevreview | Merged openstack/tripleo-heat-templates master: Horizon: Manage policy files https://review.opendev.org/c/openstack/tripleo-heat-templates/+/823891 | 03:25 |
| opendevreview | Pooja Jadhav proposed openstack/tripleo-validations master: Move zuul jobs layout to centos9 only for master branch https://review.opendev.org/c/openstack/tripleo-validations/+/828211 | 03:26 |
| opendevreview | Merged openstack/tripleo-heat-templates master: Nova: Use cross_az_attach from nova::cinder https://review.opendev.org/c/openstack/tripleo-heat-templates/+/827944 | 03:34 |
| opendevreview | Merged openstack/tripleo-heat-templates stable/wallaby: Nova: Use internal endpoint to access Keystone https://review.opendev.org/c/openstack/tripleo-heat-templates/+/827408 | 03:34 |
| opendevreview | Rabi Mishra proposed openstack/tripleo-ansible master: Remove old per step startup configs https://review.opendev.org/c/openstack/tripleo-ansible/+/828212 | 03:36 |
| *** ysandeep|out is now known as ysandeep | 03:57 | |
| opendevreview | Merged openstack/puppet-tripleo master: Heat: include an independent class to set up [trustee] https://review.opendev.org/c/openstack/puppet-tripleo/+/824827 | 04:14 |
| opendevreview | Merged openstack/tripleo-heat-templates stable/ussuri: Use only internal endpoints in Octavia controller services https://review.opendev.org/c/openstack/tripleo-heat-templates/+/824988 | 04:14 |
| opendevreview | Takashi Kajinami proposed openstack/tripleo-heat-templates master: Deprecate redundant DockerNovaMigrationSshdPort https://review.opendev.org/c/openstack/tripleo-heat-templates/+/828214 | 04:19 |
| opendevreview | Brendan Shephard proposed openstack/tripleo-heat-templates master: WIP - Address some of the non-inclusive language https://review.opendev.org/c/openstack/tripleo-heat-templates/+/819889 | 04:24 |
| opendevreview | Takashi Kajinami proposed openstack/tripleo-heat-templates master: Deprecate redundant DockerNovaMigrationSshdPort https://review.opendev.org/c/openstack/tripleo-heat-templates/+/828214 | 04:24 |
| opendevreview | Merged openstack/tripleo-heat-templates stable/train: Use only internal endpoints in Octavia controller services https://review.opendev.org/c/openstack/tripleo-heat-templates/+/824992 | 06:08 |
| opendevreview | Sandeep Yadav proposed openstack/tripleo-quickstart-extras master: Include cert dir for container build https://review.opendev.org/c/openstack/tripleo-quickstart-extras/+/828223 | 06:52 |
| opendevreview | Merged openstack/tripleo-heat-templates master: Add heat::trustee parameters https://review.opendev.org/c/openstack/tripleo-heat-templates/+/813740 | 06:55 |
| *** ysandeep is now known as ysandeep|afk | 07:11 | |
| *** amoralej|off is now known as amoralej | 07:33 | |
| *** jpena|off is now known as jpena | 08:03 | |
| fpantano | hey marios: what about adding this https://review.opendev.org/q/topic:split_off_ceph to your breakfast reviews (for the next days) ? :D | 08:12 |
| fpantano | (morning ) | 08:12 |
| marios | o/ fpantano sure adding | 08:14 |
| fpantano | marios ++ thanks :D | 08:14 |
| opendevreview | Sergii Golovatiuk proposed openstack/python-tripleoclient stable/train: Revert "[TRAIN-ONLY] Fix inventory path when upgrading" https://review.opendev.org/c/openstack/python-tripleoclient/+/828234 | 08:42 |
| *** ysandeep|afk is now known as ysandeep | 08:45 | |
| bogdando | ysandeep: hi, do you know perchance if there is an elastic-recheck for rdo CI? | 09:06 |
| bogdando | 3rd partry, SF... rather that | 09:07 |
| ysandeep | bogdando: yes we have something, dpawlik3 and frenzy_friday can answer that question better than me. | 09:07 |
| bogdando | particularly elastic-recheck-query to connect | 09:07 |
| bogdando | thanks | 09:07 |
| bogdando | for zuul upstream there is https://opendev.org/opendev/elastic-recheck.git, so looking for such an analogue | 09:08 |
| opendevreview | Jiri Podivin proposed openstack/python-tripleoclient stable/train: Use Validations libs Client for main validation entry point https://review.opendev.org/c/openstack/python-tripleoclient/+/823644 | 09:09 |
| opendevreview | Jiri Podivin proposed openstack/python-tripleoclient stable/train: Overriding the log path argument for tripleoclient cli https://review.opendev.org/c/openstack/python-tripleoclient/+/823645 | 09:09 |
| opendevreview | Jiri Podivin proposed openstack/python-tripleoclient stable/train: Fix the validations_log_basedir overriding https://review.opendev.org/c/openstack/python-tripleoclient/+/823646 | 09:09 |
| frenzy_friday | hey bogdando, are you looking for http://ci-health.tripleo.org/ ? | 09:09 |
| bogdando | frenzy_friday: a sort of... | 09:10 |
| frenzy_friday | https://hackmd.io/HQ5hyGAOSuG44Le2x6YzUw | 09:10 |
| bogdando | wondering if there is elastic-recheck-query to work with that data | 09:10 |
| bogdando | thanks | 09:10 |
| frenzy_friday | bogdando, https://opendev.org/openstack/tripleo-ci-health-queries is the repo. https://opendev.org/openstack/tripleo-ci-health-queries/src/branch/master/src/data/queries.yml are the queries user adds | 09:11 |
| frenzy_friday | https://opendev.org/openstack/tripleo-ci-health-queries/src/branch/master/output/elastic-recheck are generated by scripts when you run tox | 09:12 |
| frenzy_friday | Sample patch to add queries: https://review.opendev.org/c/openstack/tripleo-ci-health-queries/+/823110 | 09:13 |
| ysandeep | frenzy_friday: thanks! | 09:15 |
| bogdando | what is UI to run a custom query? | 09:16 |
| bogdando | since it seems there is no CLI for that, like elastic-recheck-query provides | 09:16 |
| ysandeep | frenzy_friday, ^^ | 09:18 |
| * frenzy_friday checks | 09:20 | |
| opendevreview | Marios Andreou proposed openstack/tripleo-ci master: Moves master undercloud-upgrade and minor update deprecated add c9 https://review.opendev.org/c/openstack/tripleo-ci/+/828110 | 09:21 |
| frenzy_friday | bogdando, elastic-recheck-query <any_yaml_from_output/elastic-recheck.yaml> should work I think | 09:21 |
| frenzy_friday | same syntax as the generic elastic rechecks | 09:22 |
| bogdando | right, but do you know how to reconnect it to the needed data source? | 09:22 |
| bogdando | which data_source in elasticRecheck.conf to put? | 09:24 |
| bogdando | frenzy_friday: what is processing engine there for the added queries which builds http://ci-health.tripleo.org/ ? I think I could get the data source from there... | 09:29 |
| frenzy_friday | bogdando, maybe this explains a bit more https://hackmd.io/HQ5hyGAOSuG44Le2x6YzUw This is the main elastic recheck repo https://opendev.org/opendev/elastic-recheck/src/branch/rdo/elastic_recheck | 09:32 |
| bogdando | I'm looking exactly there, but not certain what data sources to pick | 09:33 |
| bogdando | to switch away off the hardcoded defaults pointing http://logstash.openstack.org | 09:33 |
| bogdando | ah, branch RDO! | 09:34 |
| frenzy_friday | oohh, you mean the logstash source, it is here | 09:34 |
| * frenzy_friday looking for the confg | 09:34 | |
| frenzy_friday | https://opendev.org/opendev/elastic-recheck/src/branch/rdo/elastic_recheck/config.py#L22 | 09:34 |
| opendevreview | Shnaidman Sagi (Sergey) proposed openstack/tripleo-ci master: Add tripleo-ci-centos-9-standalone-build job https://review.opendev.org/c/openstack/tripleo-ci/+/828257 | 09:35 |
| frenzy_friday | yeah, so the upstream logstash was 1.x.x (which retired) The RDO logstash is 3.x.x A lot of the api endpoints changed bbetween the versions, so we decided to make an rdo branch instead of trying to make ER compatible with both logstashes | 09:36 |
| frenzy_friday | ^ and decoupled the queries | 09:37 |
| bogdando | ack | 09:37 |
| opendevreview | Marios Andreou proposed openstack/tripleo-heat-templates stable/wallaby: NO MERGE Test openstack/tripleo-ci/+/826317 https://review.opendev.org/c/openstack/tripleo-heat-templates/+/828258 | 09:38 |
| opendevreview | Marios Andreou proposed openstack/tripleo-heat-templates stable/victoria: NO MERGE Test openstack/tripleo-ci/+/826317 https://review.opendev.org/c/openstack/tripleo-heat-templates/+/828259 | 09:41 |
| opendevreview | Merged openstack/tripleo-heat-templates stable/ussuri: Add service ordering to cleanup service to avoid conflicts with agent startup https://review.opendev.org/c/openstack/tripleo-heat-templates/+/785216 | 09:42 |
| opendevreview | Merged openstack/tripleo-ansible stable/wallaby: historical variable evaluated as bool https://review.opendev.org/c/openstack/tripleo-ansible/+/828130 | 09:42 |
| opendevreview | Marios Andreou proposed openstack/tripleo-heat-templates stable/ussuri: NO MERGE Test openstack/tripleo-ci/+/826317 https://review.opendev.org/c/openstack/tripleo-heat-templates/+/828260 | 09:42 |
| opendevreview | Jose Luis Franco proposed openstack/tripleo-ansible master: WIP: Clean up old step directories in container config files. https://review.opendev.org/c/openstack/tripleo-ansible/+/828261 | 09:43 |
| opendevreview | Marios Andreou proposed openstack/tripleo-heat-templates stable/train: NO MERGE Test openstack/tripleo-ci/+/826317 https://review.opendev.org/c/openstack/tripleo-heat-templates/+/828262 | 09:43 |
| opendevreview | Merged openstack/tripleo-docs master: Document --ceph-vip option https://review.opendev.org/c/openstack/tripleo-docs/+/827703 | 09:44 |
| bogdando | frenzy_friday: https://paste.opendev.org/show/bwu7lrwU34bDKnpN3MS0/ :( | 09:45 |
| bogdando | perhaps I could try that query via some UI? | 09:48 |
| frenzy_friday | looks like the indices are still messed up. This was happening last week as well. ER cannot find recent indices | 09:50 |
| frenzy_friday | Lemme check in some time | 09:50 |
| bogdando | thanks a lot | 09:51 |
| bogdando | this thing helps a lot in filtering noise out of root cause analysis for bugs :) | 09:52 |
| frenzy_friday | huh.. looks like https://review.rdoproject.org:443/elasticsearch/logstash-2022.02.08/_stats is indeed throwing permission denied. Checking in some time | 09:54 |
| opendevreview | Shnaidman Sagi (Sergey) proposed openstack/tripleo-ci master: DNM test c9 build job https://review.opendev.org/c/openstack/tripleo-ci/+/828265 | 09:58 |
| opendevreview | Riccardo Pittau proposed openstack/diskimage-builder master: [WIP] Fallback to persistent netifs names with systemd https://review.opendev.org/c/openstack/diskimage-builder/+/828266 | 09:59 |
| opendevreview | Merged openstack/tripleo-heat-templates master: Fix ironic boot interface for undercloud https://review.opendev.org/c/openstack/tripleo-heat-templates/+/827929 | 10:01 |
| opendevreview | Jose Luis Franco proposed openstack/tripleo-heat-templates stable/wallaby: DNM: Test undercloud ffu wallaby upgrade. https://review.opendev.org/c/openstack/tripleo-heat-templates/+/823966 | 10:02 |
| *** dviroel|out is now known as dviroel|ruck | 10:05 | |
| opendevreview | Damien Ciabrini proposed openstack/tripleo-common master: Mariadb: auth_ed25519 with FIPS crypto https://review.opendev.org/c/openstack/tripleo-common/+/822190 | 10:41 |
| opendevreview | Riccardo Pittau proposed openstack/diskimage-builder master: [WIP] Fallback to persistent netifs names with systemd https://review.opendev.org/c/openstack/diskimage-builder/+/828266 | 11:08 |
| *** rlandy|out is now known as rlandy|ruck | 11:11 | |
| sshnaidm | rlandy|ruck, fyi https://bugs.launchpad.net/tripleo/+bug/1960308 | 11:19 |
| sshnaidm | bogdando, did you see such error maybe while building containers? RPM: error: cannot open Packages index using bdb_ro - Operation not permitted (1) | 11:20 |
| opendevreview | Damien Ciabrini proposed openstack/tripleo-heat-templates master: Bootstrap DB with auth_ed25519 when it is enabled https://review.opendev.org/c/openstack/tripleo-heat-templates/+/822188 | 11:22 |
| opendevreview | Rabi Mishra proposed openstack/tripleo-common stable/train: [Train-Only] Aligin inventory comparision with master https://review.opendev.org/c/openstack/tripleo-common/+/828282 | 11:27 |
| rlandy|ruck | sshnaidm: will look in a bit - master c9 promotion is very old | 11:28 |
| rlandy|ruck | we need to deal with that atm | 11:29 |
| bogdando | sshnaidm: nope | 11:30 |
| rlandy|ruck | akaha|rover: can you look into https://bugs.launchpad.net/tripleo/+bug/1960308 | 11:35 |
| opendevreview | Harald Jensås proposed openstack/tripleo-quickstart-extras master: Add undercloud_parameter_defaults var undercloud-deploy https://review.opendev.org/c/openstack/tripleo-quickstart-extras/+/828197 | 11:48 |
| opendevreview | Juan Larriba proposed openstack/tripleo-ansible stable/victoria: historical variable evaluated as bool https://review.opendev.org/c/openstack/tripleo-ansible/+/828131 | 11:50 |
| opendevreview | Juan Larriba proposed openstack/tripleo-ansible stable/ussuri: historical variable evaluated as bool https://review.opendev.org/c/openstack/tripleo-ansible/+/828132 | 11:51 |
| opendevreview | Douglas Viroel proposed openstack/openstack-tempest-skiplist master: Skip compute testes that are failing to detach volume https://review.opendev.org/c/openstack/openstack-tempest-skiplist/+/828285 | 11:51 |
| opendevreview | Douglas Viroel proposed openstack/openstack-tempest-skiplist master: Skip compute testes that are failing to detach volume https://review.opendev.org/c/openstack/openstack-tempest-skiplist/+/828285 | 11:52 |
| opendevreview | Juan Larriba proposed openstack/tripleo-ansible master: Only execute task when tripleo_backup_and_restore_historical is true https://review.opendev.org/c/openstack/tripleo-ansible/+/828286 | 11:54 |
| *** ysandeep is now known as ysandeep|mtg | 12:01 | |
| opendevreview | Takashi Kajinami proposed openstack/tripleo-heat-templates master: Deprecate redundant DockerNovaMigrationSshdPort https://review.opendev.org/c/openstack/tripleo-heat-templates/+/828214 | 12:19 |
| *** pojadhav is now known as pojadhav|afk | 12:33 | |
| opendevreview | Riccardo Pittau proposed openstack/diskimage-builder master: [WIP] Fallback to persistent netifs names with systemd https://review.opendev.org/c/openstack/diskimage-builder/+/828266 | 12:37 |
| opendevreview | Bhagyashri Shewale proposed openstack/tripleo-operator-ansible master: Move zuul jobs layout to centos9 only for master branch https://review.opendev.org/c/openstack/tripleo-operator-ansible/+/828296 | 12:42 |
| opendevreview | Merged openstack/openstack-tempest-skiplist master: Skip compute testes that are failing to detach volume https://review.opendev.org/c/openstack/openstack-tempest-skiplist/+/828285 | 12:50 |
| opendevreview | Bhagyashri Shewale proposed openstack/ansible-role-tripleo-modify-image master: Move zuul jobs layout to centos9 only for master branch https://review.opendev.org/c/openstack/ansible-role-tripleo-modify-image/+/828299 | 12:52 |
| *** ysandeep|mtg is now known as ysandeep | 12:55 | |
| opendevreview | Rabi Mishra proposed openstack/tripleo-ansible master: Remove old per step startup configs https://review.opendev.org/c/openstack/tripleo-ansible/+/828212 | 12:59 |
| *** ysandeep is now known as ysandeep|coffee | 13:04 | |
| *** blarnath is now known as d34dh0r53 | 13:05 | |
| opendevreview | Jose Luis Franco proposed openstack/tripleo-ansible master: WIP: Clean up old step directories in container config files. https://review.opendev.org/c/openstack/tripleo-ansible/+/828261 | 13:13 |
| *** amoralej is now known as amoralej|lunch | 13:18 | |
| opendevreview | Jose Luis Franco proposed openstack/tripleo-ansible master: WIP: Clean up old step directories in container config files. https://review.opendev.org/c/openstack/tripleo-ansible/+/828261 | 13:23 |
| opendevreview | Marios Andreou proposed openstack/openstack-tempest-skiplist master: Revert "Adds failing victoria test" https://review.opendev.org/c/openstack/openstack-tempest-skiplist/+/828243 | 13:24 |
| *** pojadhav|afk is now known as pojadhav | 13:31 | |
| *** ysandeep|coffee is now known as ysandeep | 13:39 | |
| *** dasm|off is now known as dasm | 13:55 | |
| opendevreview | Harald Jensås proposed openstack/tripleo-quickstart master: Don't use ipxe-snponly on CentOS 7 https://review.opendev.org/c/openstack/tripleo-quickstart/+/828089 | 14:07 |
| opendevreview | Marios Andreou proposed openstack/openstack-tempest-skiplist master: Revert "Adds failing victoria test" https://review.opendev.org/c/openstack/openstack-tempest-skiplist/+/828243 | 14:10 |
| *** amoralej|lunch is now known as amoralej | 14:22 | |
| opendevreview | John Fulton proposed openstack/tripleo-ansible master: Add playbooks to config overcloud hosts and container regsitry https://review.opendev.org/c/openstack/tripleo-ansible/+/827690 | 14:27 |
| opendevreview | John Fulton proposed openstack/python-tripleoclient master: Support Deployed Ceph container push_destination https://review.opendev.org/c/openstack/python-tripleoclient/+/825416 | 14:27 |
| opendevreview | Takashi Kajinami proposed openstack/python-tripleoclient master: Add Python 3.8 to supported runtime https://review.opendev.org/c/openstack/python-tripleoclient/+/828352 | 14:33 |
| *** Guest2040 is now known as spotz | 14:37 | |
| opendevreview | Bogdan Dobrelya proposed openstack/tripleo-quickstart-extras master: Allow dlrn builds from both deploy release and upgrade release https://review.opendev.org/c/openstack/tripleo-quickstart-extras/+/826284 | 14:46 |
| *** akaha|rover is now known as akahat|dinner | 14:55 | |
| *** dviroel|ruck is now known as dviroel|ruck|lunch | 15:11 | |
| *** ysandeep is now known as ysandeep|dinner | 15:14 | |
| *** pojadhav is now known as pojadhav|dinner | 15:24 | |
| opendevreview | Merged openstack/tripleo-quickstart-extras master: Include cert dir for container build https://review.opendev.org/c/openstack/tripleo-quickstart-extras/+/828223 | 15:39 |
| *** ysandeep|dinner is now known as ysandeep | 15:44 | |
| opendevreview | Damien Ciabrini proposed openstack/tripleo-heat-templates master: Sync updated DB root password in running container https://review.opendev.org/c/openstack/tripleo-heat-templates/+/828370 | 15:53 |
| *** pojadhav|dinner is now known as pojadhav | 15:58 | |
| *** dviroel|ruck|lunch is now known as dviroel|ruck | 16:06 | |
| *** dkehn_ is now known as dkehn | 16:41 | |
| *** jpena is now known as jpena|off | 17:04 | |
| *** akahat|dinner is now known as akahat|rover | 17:05 | |
| *** sshnaidm is now known as sshnaidm|afk | 17:18 | |
| *** ysandeep is now known as ysandeep|out | 17:21 | |
| opendevreview | Harald Jensås proposed openstack/tripleo-quickstart master: Don't use ipxe-snponly on CentOS 7 https://review.opendev.org/c/openstack/tripleo-quickstart/+/828089 | 17:32 |
| *** amoralej is now known as amoralej|off | 17:39 | |
| opendevreview | Merged openstack/tripleo-heat-templates stable/wallaby: Drop Nova and NovaJoin services from UC role data https://review.opendev.org/c/openstack/tripleo-heat-templates/+/827968 | 18:22 |
| opendevreview | Harald Jensås proposed openstack/tripleo-ansible master: Replace wait_for with wait_for_connection https://review.opendev.org/c/openstack/tripleo-ansible/+/828389 | 18:25 |
| opendevreview | Harald Jensås proposed openstack/tripleo-quickstart master: Don't use ipxe-snponly on CentOS 7 https://review.opendev.org/c/openstack/tripleo-quickstart/+/828089 | 18:30 |
| opendevreview | Merged openstack/tripleo-heat-templates master: Fix OS_CLOUD for multistack and nova az tasks https://review.opendev.org/c/openstack/tripleo-heat-templates/+/827527 | 18:54 |
| dviroel|ruck | fmount[m]: hi o/ | 19:01 |
| opendevreview | Ronelle Landy proposed openstack/tripleo-quickstart master: [WIP] Add C9 release file for jenkins jobs https://review.opendev.org/c/openstack/tripleo-quickstart/+/828171 | 19:05 |
| dviroel|ruck | fultonj: hey, you around? | 19:08 |
| fultonj | hi dviroel|ruck | 19:08 |
| fultonj | what's up dviroel? | 19:09 |
| dviroel|ruck | fultonj: hey - so was trying to reach fmount[m], I think that one of his patch added a bug on overcloud-delete | 19:10 |
| dviroel|ruck | fultonj: https://opendev.org/openstack/tripleo-ansible/commit/2636e84c41f8be28c9f72e8ce126beb72b2cb32c | 19:10 |
| dviroel|ruck | fultonj: https://logserver.rdoproject.org/openstack-periodic-integration-main/opendev.org/openstack/tripleo-ci/f052b61dbbd229e1776d3f34c2252ad636b7f78a/periodic-tripleo-ci-centos-9-ovb-1ctlr_1comp-featureset002-master/b519aa2/logs/undercloud/home/zuul/overcloud_delete.log.txt.gz | 19:11 |
| dviroel|ruck | error ^^ | 19:11 |
| dviroel|ruck | now it is failing if we try to delete vip only using service_name | 19:12 |
| opendevreview | Steve Baker proposed openstack/tripleo-common master: Support multiple mounts without explicit nbd devices https://review.opendev.org/c/openstack/tripleo-common/+/828412 | 19:14 |
| fultonj | dviroel|ruck: i'm not certain the two are related | 19:15 |
| fultonj | dviroel|ruck: but do you want to open a bug, propose a revert and run a job which depends-on the revert to see? | 19:15 |
| * fultonj looking closer at the patch to see if we overlooked something | 19:16 | |
| dviroel|ruck | fultonj: yep, we should proceed in that path - just want to bring to you guys | 19:16 |
| fultonj | he was adding a feature which isn't yet a part of the deployment jobs | 19:16 |
| dviroel|ruck | fultonj: starting at line 320 | 19:17 |
| dviroel|ruck | fultonj: it is blocking us to delete vip only providing "service_name" | 19:17 |
| fultonj | i see | 19:17 |
| dviroel|ruck | fultonj: new validation at line 217 | 19:17 |
| dviroel|ruck | 327 | 19:18 |
| dviroel|ruck | raises the exception | 19:18 |
| dviroel|ruck | fultonj: ok, np, we can move forward - we sync again tomorrow with fmount | 19:18 |
| fultonj | i see how the module is now throwing an exception | 19:20 |
| fultonj | thanks for bringing it up dviroel|ruck | 19:20 |
| dviroel|ruck | fultonj: sure, np | 19:22 |
| opendevreview | Merged openstack/tripleo-heat-templates stable/train: Disable libvirtd and stop it https://review.opendev.org/c/openstack/tripleo-heat-templates/+/827019 | 19:23 |
| fultonj | dviroel|ruck: are you putting up a review which tests the cloud delete job with the revert? | 19:24 |
| fultonj | i'm thinking about how https://review.opendev.org/c/openstack/tripleo-ansible/+/825502 was green when we merged and how we might have run that job with the chnage | 19:24 |
| dviroel|ruck | fultonj: will do in a moment | 19:26 |
| *** fpantano is now known as fmount | 19:29 | |
| fmount | dviroel|ruck: fultonj o/ | 19:30 |
| fmount | let me look at the log | 19:30 |
| fmount | dviroel|ruck: not sure reverting the patch is the right way, let me try to fix it | 19:30 |
| fultonj | fmount: +1 | 19:32 |
| opendevreview | Chris Sibbitt proposed openstack/tripleo-ansible stable/victoria: Enable fips test for metrics_qdr https://review.opendev.org/c/openstack/tripleo-ansible/+/828415 | 19:32 |
| fultonj | i was thinking once we had a way to test the delete job (since proposing a tripleo-ansible patch alone doesn't seem to trigger that job) you could try a fix following the revert pattern | 19:33 |
| fultonj | I agree we'd rather merge a fix than revert | 19:33 |
| dviroel|ruck | fmount: sure - the revert was just to confirm the blame :) | 19:34 |
| fmount | dviroel|ruck: it's that patch because I see it raises the exception I added | 19:35 |
| fmount | but let me try a quick fix for it ^ | 19:35 |
| dviroel|ruck | fmount: yeah, that string is unique in codesearch | 19:35 |
| dviroel|ruck | fmount: here https://opendev.org/openstack/tripleo-ansible/src/branch/master/tripleo_ansible/playbooks/cli-overcloud-delete.yaml#L42 | 19:35 |
| fultonj | if out is None and playbook_dir is None: | 19:36 |
| fultonj | pass | 19:36 |
| fultonj | ;) | 19:36 |
| dviroel|ruck | fmount: but since it is late for you, you may want to start the fix tomorrow | 19:37 |
| dviroel|ruck | fmount: i'll just log the bug | 19:38 |
| opendevreview | Francesco Pantano proposed openstack/tripleo-ansible master: Fix tripleo_service_vip exception https://review.opendev.org/c/openstack/tripleo-ansible/+/828417 | 19:39 |
| fmount | dviroel|ruck: can you run the job which is failing depending on ^^^^^^^^ | 19:40 |
| dviroel|ruck | fmount: sure | 19:40 |
| fultonj | thanks fmount ! | 19:42 |
| opendevreview | Brent Eagles proposed openstack/tripleo-heat-templates stable/wallaby: Designate: miniDNS and bind9 instances on the proper networks https://review.opendev.org/c/openstack/tripleo-heat-templates/+/828420 | 19:42 |
| opendevreview | Chris Sibbitt proposed openstack/tripleo-ansible stable/wallaby: Enable fips test for metrics_qdr https://review.opendev.org/c/openstack/tripleo-ansible/+/828421 | 19:50 |
| opendevreview | Chris Sibbitt proposed openstack/tripleo-ansible stable/wallaby: Enable fips test for metrics_qdr https://review.opendev.org/c/openstack/tripleo-ansible/+/828423 | 19:56 |
| opendevreview | Chris Sibbitt proposed openstack/tripleo-ansible stable/wallaby: Enable fips test for collectd https://review.opendev.org/c/openstack/tripleo-ansible/+/828424 | 19:58 |
| opendevreview | Francesco Pantano proposed openstack/tripleo-ansible master: Fix tripleo_service_vip exception https://review.opendev.org/c/openstack/tripleo-ansible/+/828417 | 20:25 |
| opendevreview | Douglas Viroel proposed openstack/openstack-tempest-skiplist master: Skip keystone OAUTH1TokensTest tests https://review.opendev.org/c/openstack/openstack-tempest-skiplist/+/828428 | 21:00 |
| opendevreview | Ronelle Landy proposed openstack/tripleo-ci master: DNM - Test https://review.opendev.org/c/openstack/tripleo-ci/+/820266 | 21:10 |
| opendevreview | Ronelle Landy proposed openstack/tripleo-ci master: DNM - Test https://review.opendev.org/c/openstack/tripleo-ci/+/820266 | 21:10 |
| opendevreview | Merged openstack/diskimage-builder master: Add new container element - Rocky Linux https://review.opendev.org/c/openstack/diskimage-builder/+/825957 | 21:24 |
| fultonj | dviroel|ruck: do you think https://review.opendev.org/c/openstack/tripleo-ansible/+/827484/3 will get merged this week? | 21:34 |
| * fultonj asking since i have a patch blocked | 21:34 | |
| dviroel|ruck | fultonj: oh, failing for you? it was not priority since was not blocking us - but I can take a look - there is only one missing only | 21:36 |
| fultonj | dviroel|ruck: thanks | 21:37 |
| fultonj | i regbased my patch on your patch so it's green now for my molecule test https://review.opendev.org/c/openstack/tripleo-ansible/+/827733 | 21:37 |
| fultonj | if yours can land sooner that would be great. thanks | 21:38 |
| dviroel|ruck | fultonj: oh, i see - ok, i will take a look afterwards | 21:39 |
| fultonj | thanks dviroel|ruck | 21:39 |
| *** dviroel|ruck is now known as dviroel|ruck|afk | 22:02 | |
| opendevreview | Merged openstack/openstack-tempest-skiplist master: Skip keystone OAUTH1TokensTest tests https://review.opendev.org/c/openstack/openstack-tempest-skiplist/+/828428 | 22:13 |
| *** dasm is now known as dasm|off | 23:02 | |
| *** mschuppert6 is now known as mschuppert | 23:02 | |
| *** rlandy|ruck is now known as rlandy|ruck|bbl | 23:24 | |
| rlandy|ruck|bbl | all: content provider jobs are broken again due to opstools change again ... waiting for merge of https://review.rdoproject.org/r/c/rdo-infra/ansible-role-dlrn/+/38804 | 23:25 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!