*** pmannidi is now known as pmannidi|brb | 00:04 | |
*** pmannidi|brb is now known as pmannidi | 01:06 | |
opendevreview | Brendan Shephard proposed openstack/tripleo-heat-templates master: Fix pacemaker_cluster args and formatting https://review.opendev.org/c/openstack/tripleo-heat-templates/+/803534 | 01:46 |
---|---|---|
opendevreview | Brendan Shephard proposed openstack/tripleo-heat-templates master: Fix pacemaker_cluster args and formatting https://review.opendev.org/c/openstack/tripleo-heat-templates/+/803534 | 01:51 |
opendevreview | Takashi Kajinami proposed openstack/tripleo-heat-templates master: Replace deprecated database_connection of puppet base classes https://review.opendev.org/c/openstack/tripleo-heat-templates/+/803526 | 02:53 |
opendevreview | Takashi Kajinami proposed openstack/tripleo-heat-templates master: Replace deprecated glance::api::database_connection https://review.opendev.org/c/openstack/tripleo-heat-templates/+/803526 | 02:57 |
opendevreview | Takashi Kajinami proposed openstack/tripleo-heat-templates master: Replace deprecated database_connection of puppet base classes https://review.opendev.org/c/openstack/tripleo-heat-templates/+/803538 | 02:57 |
opendevreview | Merged openstack/tripleo-quickstart-extras master: Bump nautilus containers to v4.0.19 https://review.opendev.org/c/openstack/tripleo-quickstart-extras/+/803483 | 03:56 |
opendevreview | Merged openstack/tripleo-heat-templates stable/wallaby: Do not fail if haproxy container UUID changes https://review.opendev.org/c/openstack/tripleo-heat-templates/+/802470 | 03:56 |
*** pojadhav- is now known as pojadhav | 04:07 | |
*** ykarel|away is now known as ykarel | 04:47 | |
tkajinam | bandini, I confirmed the issue is solved in rdo job https://review.opendev.org/c/openstack/tripleo-heat-templates/+/803526 | 06:12 |
tkajinam | hopefully we can merge that once we get results from zuul. one of RDO job failed but I believe it was because of unrelated cause | 06:13 |
bandini | excellent! let me take a look | 06:16 |
bandini | tkajinam: I'll file an LP for the other RDO job that failed | 06:23 |
* bandini still investigating | 06:23 | |
bandini | but it is unrelated it seems | 06:23 |
tkajinam | bandini, I think that's a kind of temporal one | 06:24 |
tkajinam | bandini, it passed on patch set 1. I made some change in patch set 2 and reverted that change in patch set 3 | 06:24 |
bandini | yeah I have seen it yesterday on my env as well, but did not have the time | 06:24 |
tkajinam | so patch set 1 and 3 should be same | 06:24 |
bandini | it needs some more analysis | 06:24 |
bandini | ack | 06:24 |
tkajinam | bandini, ah, yeah. it's good to record it for further investigation | 06:24 |
*** marios is now known as marios|ruck | 06:31 | |
opendevreview | Jiri Podivin proposed openstack/validations-common master: DNM https://review.opendev.org/c/openstack/validations-common/+/803316 | 06:34 |
tkajinam | Tengu ramishra, if you are around, my I ask your +2 on this patch ? https://review.opendev.org/c/openstack/tripleo-heat-templates/+/803526 | 06:52 |
Tengu | lemme check | 06:53 |
tkajinam | context can be found in the associated bug | 06:54 |
Tengu | reading already :) | 06:54 |
tkajinam | Tengu, thanks ! | 06:54 |
Tengu | goto(gate) | 06:54 |
Tengu | brb, breakfast | 06:55 |
tkajinam | Tengu, nice. thanks again and have a good one | 06:55 |
*** jpena|off is now known as jpena | 07:08 | |
*** pojadhav- is now known as pojadhav | 07:09 | |
opendevreview | Jiri Podivin proposed openstack/validations-common master: DNM https://review.opendev.org/c/openstack/validations-common/+/803316 | 07:35 |
*** rpittau|afk is now known as rpittau | 07:47 | |
marios|ruck | slaweq: o/ hey can you please add the related bug the zuul run will be very fast there https://review.opendev.org/c/openstack/openstack-tempest-skiplist/+/803432 | 07:49 |
*** pmannidi is now known as pmannidi|brb | 07:49 | |
opendevreview | Marios Andreou proposed openstack/openstack-tempest-skiplist master: Revert "Add TestNetworkBasicOps.test_subnet_details to skip list" https://review.opendev.org/c/openstack/openstack-tempest-skiplist/+/803432 | 08:13 |
marios|ruck | slaweq: ^^ | 08:13 |
slaweq | marios|ruck: thx, I was going to do it after the meeting :) | 08:14 |
opendevreview | Jiri Podivin proposed openstack/validations-common master: DNM https://review.opendev.org/c/openstack/validations-common/+/803316 | 08:14 |
slaweq | bug as we discussed with weshay|ruck yesterday on the cix call, we should wait few more days with that revert | 08:15 |
slaweq | he will check on weekend and then we will go with it | 08:15 |
marios|ruck | slaweq: np but i think there is a nit there? https://review.opendev.org/c/openstack/openstack-tempest-skiplist/+/803432/3#message-1045ec92fcf4370a0568ed0abd870c0e07e22bd0 | 08:15 |
marios|ruck | slaweq: thanks check after the meeting and you can update if necessary for that ^^ | 08:15 |
marios|ruck | slaweq: thank you for posting it and testing | 08:15 |
slaweq | marios|ruck: that nit is wrong conflict resolve :) | 08:16 |
slaweq | I will fix it in few minutes | 08:16 |
marios|ruck | slaweq: ack thanks take ur time | 08:16 |
opendevreview | Merged openstack/tripleo-upgrade stable/ussuri: Make the gerrit_url of patches configurable https://review.opendev.org/c/openstack/tripleo-upgrade/+/803435 | 08:17 |
opendevreview | Slawek Kaplonski proposed openstack/openstack-tempest-skiplist master: Revert "Add TestNetworkBasicOps.test_subnet_details to skip list" https://review.opendev.org/c/openstack/openstack-tempest-skiplist/+/803432 | 08:18 |
opendevreview | Jiri Podivin proposed openstack/validations-common master: DNM https://review.opendev.org/c/openstack/validations-common/+/803316 | 08:36 |
opendevreview | Arx Cruz proposed openstack/tripleo-quickstart-extras master: Update list_allowed variables https://review.opendev.org/c/openstack/tripleo-quickstart-extras/+/802484 | 08:45 |
Tengu | bandini, tkajinam quick question about https://bugs.launchpad.net/tripleo/+bug/1938932 - it's only on master? or will it need any bacports ? | 08:54 |
opendevreview | Jiri Podivin proposed openstack/validations-libs master: Debbuging guidelines for validations-libs https://review.opendev.org/c/openstack/validations-libs/+/797284 | 08:55 |
marios|ruck | Tengu: bandini: i commented on the cix and bug this is duplicate we are already tracking it in prod chain blocked | 09:00 |
Tengu | marios|ruck: oh. meh. | 09:01 |
marios|ruck | Tengu: it will cause confusion | 09:01 |
Tengu | and I was there, trying to triage a bit the CIX.... pfrrtt | 09:01 |
marios|ruck | Tengu: np i am trying to update the older card with the fix and we should close the new one | 09:01 |
Tengu | marios|ruck: does LP support "closed as duplicate" ? it should automatically update the trellp | 09:02 |
Tengu | *trello | 09:02 |
Tengu | marios|ruck: errr... | 09:02 |
Tengu | it doesn't seem to be the same actually. | 09:02 |
marios|ruck | Tengu: no our script isn't checking state | 09:02 |
Tengu | at least, the linked patches in the "first" LP aren't really pointing to the fact it's using sqlite instead of mariadb... | 09:03 |
Tengu | marios|ruck: I'm not that sure it's really the same. | 09:03 |
Tengu | bandini: care to confirm? -^^ | 09:03 |
marios|ruck | Tengu: i am basing on ykarel comment there https://bugs.launchpad.net/tripleo/+bug/1938932/comments/3 | 09:03 |
marios|ruck | Tengu: the original bug was manifesting in tempestconf failing | 09:03 |
Tengu | yeah.... not sure, really. the mentionned patches aren't on the same thing. Like, not at all. | 09:04 |
marios|ruck | Tengu: yes see the description | 09:04 |
marios|ruck | Tengu: from https://bugs.launchpad.net/tripleo/+bug/1938932 | 09:04 |
marios|ruck | Tengu: In fact RDO was red exactly because of this I think: | 09:04 |
marios|ruck | https://logserver.rdoproject.org/44/786644/11/openstack-check/tripleo-ci-centos-8-ovb-3ctlr_1comp-featureset001/0025d8b/logs/undercloud/var/log/tempest/tempestconf.log.txt.gz | 09:04 |
marios|ruck | Tengu: (quoting bandin ^^)) | 09:04 |
opendevreview | Jiri Podivin proposed openstack/validations-common master: DNM https://review.opendev.org/c/openstack/validations-common/+/803316 | 09:05 |
Tengu | "exception.DBNonExistentTable: (sqlite3.OperationalError) no such table" it really sounds like an sqlite3 vs mariadb thing. | 09:05 |
Tengu | so tkajinam patch seems to be the actual correct one | 09:05 |
marios|ruck | Tengu: hmmm so the connection=mysql+pymysql is correct there https://logserver.rdoproject.org/openstack-component-glance/opendev.org/openstack/tripleo-ci/master/periodic-tripleo-ci-centos-8-scenario001-standalone-glance-master/f778547/logs/undercloud/var/lib/config-data/puppet-generated/glance_api/etc/glance/glance-api.conf.txt.gz | 09:07 |
Tengu | marios|ruck: especially since it's affecting *master* - apparently the patches linked in the other one are for wallaby | 09:07 |
Tengu | marios|ruck: soooo....... NOT the same. More than probably. | 09:07 |
marios|ruck | Tengu: yeah i am not so sure any more... time to start undoing the comments ;) | 09:08 |
Tengu | :)) | 09:08 |
opendevreview | Brendan Shephard proposed openstack/tripleo-heat-templates master: Revert change to use community pacemaker_cluster module https://review.opendev.org/c/openstack/tripleo-heat-templates/+/803534 | 09:09 |
opendevreview | Jiri Podivin proposed openstack/validations-common master: DNM https://review.opendev.org/c/openstack/validations-common/+/803316 | 09:30 |
opendevreview | Jiri Podivin proposed openstack/validations-common master: DNM https://review.opendev.org/c/openstack/validations-common/+/803316 | 09:32 |
opendevreview | Carlos Gonçalves proposed openstack/tripleo-ansible master: tripleo_frr: set default version and profile https://review.opendev.org/c/openstack/tripleo-ansible/+/803546 | 09:34 |
opendevreview | Brendan Shephard proposed openstack/tripleo-heat-templates master: Revert change to use community pacemaker_cluster module https://review.opendev.org/c/openstack/tripleo-heat-templates/+/803534 | 09:42 |
opendevreview | Brendan Shephard proposed openstack/tripleo-heat-templates master: Revert change to use community pacemaker_cluster module https://review.opendev.org/c/openstack/tripleo-heat-templates/+/803534 | 09:43 |
opendevreview | Brendan Shephard proposed openstack/tripleo-heat-templates master: Revert change to use community pacemaker_cluster module https://review.opendev.org/c/openstack/tripleo-heat-templates/+/803534 | 09:54 |
opendevreview | Carlos Gonçalves proposed openstack/tripleo-ansible master: tripleo_frr: set a MD5 password https://review.opendev.org/c/openstack/tripleo-ansible/+/803548 | 09:54 |
opendevreview | Merged openstack/tripleo-heat-templates master: Replace deprecated glance::api::database_connection https://review.opendev.org/c/openstack/tripleo-heat-templates/+/803526 | 09:58 |
opendevreview | Jiri Podivin proposed openstack/validations-common master: DNM https://review.opendev.org/c/openstack/validations-common/+/803316 | 10:11 |
* ykarel reads back | 10:13 | |
ykarel | marios|ruck, both looks different to me | 10:16 |
marios|ruck | ykarel: yeah i updated the bug again to say it was different after more digging thanks for checking | 10:17 |
ykarel | marios|ruck, also looks like blkid error is not fatal as i see it in successful jobs too, may be that too already sorted | 10:20 |
marios|ruck | ykarel: k | 10:20 |
opendevreview | Jiri Podivin proposed openstack/validations-common master: DNM https://review.opendev.org/c/openstack/validations-common/+/803316 | 10:28 |
opendevreview | Shnaidman Sagi (Sergey) proposed openstack/tripleo-ansible master: Adjust molecule jobs for Ansible 2.11 https://review.opendev.org/c/openstack/tripleo-ansible/+/803516 | 10:28 |
shyamb | Hi | 10:36 |
opendevreview | Shnaidman Sagi (Sergey) proposed openstack/tripleo-quickstart-extras master: Run TripleO jobs on Ansible 2.11 https://review.opendev.org/c/openstack/tripleo-quickstart-extras/+/797823 | 10:36 |
opendevreview | mbu proposed openstack/validations-libs master: Add validation config file mechanism https://review.opendev.org/c/openstack/validations-libs/+/792460 | 10:46 |
ykarel | marios|ruck, added comment https://bugs.launchpad.net/tripleo/+bug/1938558/comments/3 on cause and steps to unblock along with some other contexts around that | 11:00 |
opendevreview | Jiri Podivin proposed openstack/validations-common master: DNM https://review.opendev.org/c/openstack/validations-common/+/803316 | 11:01 |
marios|ruck | thank you ykarel will check in a bit | 11:01 |
*** jcapitao is now known as jcapitao_lunch | 11:03 | |
opendevreview | Francesco Pantano proposed openstack/tripleo-heat-templates master: DNM - Test Pending Ceph Pacific daemons in CI https://review.opendev.org/c/openstack/tripleo-heat-templates/+/778915 | 11:05 |
opendevreview | Sorin Sbârnea proposed openstack/ansible-role-collect-logs master: Cleanup galaxy collection metadata https://review.opendev.org/c/openstack/ansible-role-collect-logs/+/803501 | 11:05 |
opendevreview | Shnaidman Sagi (Sergey) proposed openstack/tripleo-quickstart-extras master: Run TripleO jobs on Ansible 2.11 https://review.opendev.org/c/openstack/tripleo-quickstart-extras/+/797823 | 11:14 |
opendevreview | Shnaidman Sagi (Sergey) proposed openstack/tripleo-operator-ansible master: Add auto releasing job for tripleo-operators collection https://review.opendev.org/c/openstack/tripleo-operator-ansible/+/803588 | 11:17 |
*** dviroel|out is now known as dviroel | 11:25 | |
opendevreview | Shnaidman Sagi (Sergey) proposed openstack/tripleo-operator-ansible master: Add auto releasing job for tripleo-operators collection https://review.opendev.org/c/openstack/tripleo-operator-ansible/+/803588 | 11:27 |
opendevreview | Shnaidman Sagi (Sergey) proposed openstack/tripleo-operator-ansible master: Add auto releasing job for tripleo-operators collection https://review.opendev.org/c/openstack/tripleo-operator-ansible/+/803588 | 11:28 |
*** jpena is now known as jpena|lunch | 11:37 | |
opendevreview | Jiri Podivin proposed openstack/validations-common master: DNM https://review.opendev.org/c/openstack/validations-common/+/803316 | 11:38 |
opendevreview | Shnaidman Sagi (Sergey) proposed openstack/tripleo-operator-ansible master: Add auto releasing job for tripleo-operators collection https://review.opendev.org/c/openstack/tripleo-operator-ansible/+/803588 | 11:50 |
opendevreview | Jiri Podivin proposed openstack/validations-common master: DNM https://review.opendev.org/c/openstack/validations-common/+/803316 | 12:03 |
opendevreview | Merged openstack/python-tripleoclient master: Fail the overcloud deploy is ceph-ansible.yaml is included https://review.opendev.org/c/openstack/python-tripleoclient/+/797867 | 12:03 |
opendevreview | chandan kumar proposed openstack/tripleo-quickstart-extras master: Move build-test-packages as a part of undercloud setup https://review.opendev.org/c/openstack/tripleo-quickstart-extras/+/790926 | 12:15 |
opendevreview | Merged openstack/tripleo-heat-templates stable/train: Support cinder image conversion on an NFS share https://review.opendev.org/c/openstack/tripleo-heat-templates/+/802076 | 12:27 |
*** jcapitao_lunch is now known as jcapitao | 12:28 | |
opendevreview | mbu proposed openstack/validations-libs master: Add validation config file mechanism https://review.opendev.org/c/openstack/validations-libs/+/792460 | 12:34 |
*** jpena|lunch is now known as jpena | 12:39 | |
opendevreview | Jiri Podivin proposed openstack/validations-common master: DNM https://review.opendev.org/c/openstack/validations-common/+/803316 | 12:46 |
opendevreview | Martin Schuppert proposed openstack/tripleo-common master: Re-add tripleoclient and tempest container https://review.opendev.org/c/openstack/tripleo-common/+/801987 | 12:48 |
opendevreview | Jiri Podivin proposed openstack/validations-common master: Test validations for false positives https://review.opendev.org/c/openstack/validations-common/+/803002 | 12:54 |
opendevreview | Jiri Podivin proposed openstack/validations-common master: False positive checks for component jobs https://review.opendev.org/c/openstack/validations-common/+/803163 | 12:54 |
*** pojadhav- is now known as pojadhav | 13:05 | |
opendevreview | Dan Prince proposed openstack/tripleo-common master: inventory.py: all_get_neutron_data to handle 404 https://review.opendev.org/c/openstack/tripleo-common/+/803599 | 13:08 |
opendevreview | Jiri Podivin proposed openstack/validations-common master: DNM https://review.opendev.org/c/openstack/validations-common/+/803316 | 13:21 |
opendevreview | Shnaidman Sagi (Sergey) proposed openstack/tripleo-operator-ansible master: Add auto releasing job for tripleo-operators collection https://review.opendev.org/c/openstack/tripleo-operator-ansible/+/803588 | 13:35 |
*** pojadhav- is now known as pojadhav | 13:39 | |
sshnaidm | cloudnull, zbr please review in your time https://review.opendev.org/c/openstack/tripleo-operator-ansible/+/803588 | 13:57 |
cloudnull | sshnaidm looking now | 14:03 |
opendevreview | Jiri Podivin proposed openstack/validations-common master: Test validations for false positives https://review.opendev.org/c/openstack/validations-common/+/803002 | 14:07 |
opendevreview | Shnaidman Sagi (Sergey) proposed openstack/tripleo-ansible master: Adjust molecule jobs for Ansible 2.11 https://review.opendev.org/c/openstack/tripleo-ansible/+/803516 | 14:07 |
opendevreview | Jiri Podivin proposed openstack/validations-common master: False positive checks for component jobs https://review.opendev.org/c/openstack/validations-common/+/803163 | 14:07 |
*** rlandy is now known as rlandy|ruck | 14:07 | |
opendevreview | Takashi Kajinami proposed openstack/puppet-tripleo master: Octavia: Add optional inclusion of driver_agent class https://review.opendev.org/c/openstack/puppet-tripleo/+/798407 | 14:11 |
owalsh | Tengu: hey, I'm trying a simple deploy of master with tripleo-lab and hitting a few issues, do we expect it to work? | 14:13 |
opendevreview | Takashi Kajinami proposed openstack/tripleo-heat-templates master: Include octavia::driver_agent via puppet-tripleo https://review.opendev.org/c/openstack/tripleo-heat-templates/+/798408 | 14:13 |
Tengu | owalsh: hey! soooo it seems there's an issue with some generated files for network-v2 | 14:13 |
opendevreview | Douglas Viroel proposed openstack/tripleo-repos master: Improve ansible module for yum-config https://review.opendev.org/c/openstack/tripleo-repos/+/800922 | 14:14 |
Tengu | owalsh: basically it wants to override now "protected" keys from within t-h-t - I saw it yesterday, but haven't have the time to dig in it. | 14:14 |
Tengu | the issue is NOT in tripleo-lab, but with "a package" providng the tooling - either tripleoclient, or maybe tripleo-common. have to work my way through. | 14:15 |
Tengu | owalsh: now, you're maybe facing another issue - but that's the one I know of, and once the files were edited and the OC deploy restarted, it worked without any issue. | 14:15 |
Tengu | owalsh: sooo.... what are you seeing? :) | 14:15 |
owalsh | Tengu: current issue is provisioning of the controllers/computes failing, ceph vms are ok though | 14:15 |
Tengu | hmmm.... I didn't see this issue. | 14:16 |
Tengu | owalsh: now, with the pile of custom env I'm usually using, maybe some defaults are "deprecated" :/ | 14:17 |
owalsh | Tengu: ack, is that custom env anywhere? | 14:17 |
Tengu | no, it's located in the "local_env" that is ignored by git - I have a load of things in it, as well as rhel related things ^^' | 14:18 |
Tengu | plus some custom settings like my proxy, own registry... | 14:18 |
owalsh | Tengu: ack, thanks, I'll start digging into logs :-) Probably send a few pull requests too for minor issues I hit along the way | 14:19 |
Tengu | owalsh: ok! thanks for contributing :) | 14:19 |
owalsh | Tengu: np, for creating it :-) | 14:20 |
owalsh | thanks ^ | 14:20 |
Tengu | :) | 14:20 |
opendevreview | Merged openstack/tripleo-validations master: Ansible lint custom rule expansion https://review.opendev.org/c/openstack/tripleo-validations/+/798636 | 14:21 |
opendevreview | Merged openstack/validations-common master: Removing validations-libs from test-requirements https://review.opendev.org/c/openstack/validations-common/+/802680 | 14:21 |
*** pojadhav is now known as pojadhav|out | 14:23 | |
opendevreview | Shnaidman Sagi (Sergey) proposed openstack/tripleo-operator-ansible master: Add auto releasing job for tripleo-operators collection https://review.opendev.org/c/openstack/tripleo-operator-ansible/+/803588 | 14:40 |
sshnaidm | cloudnull, replied ^ | 14:40 |
opendevreview | Jiri Podivin proposed openstack/validations-common master: DNM https://review.opendev.org/c/openstack/validations-common/+/803316 | 14:42 |
sshnaidm | cloudnull, also addressed issues in https://review.opendev.org/c/openstack/tripleo-ansible/+/803121 | 14:42 |
TheJulia | is rdo third party ci healthy? | 14:43 |
sshnaidm | TheJulia, should be now | 14:44 |
opendevreview | James Slagle proposed openstack/python-tripleoclient master: Save needed stack outputs in working_dir https://review.opendev.org/c/openstack/python-tripleoclient/+/803609 | 14:46 |
opendevreview | James Slagle proposed openstack/python-tripleoclient master: Refactor --stack-only, --setup-only, --config-download-only https://review.opendev.org/c/openstack/python-tripleoclient/+/803610 | 14:46 |
opendevreview | James Slagle proposed openstack/tripleo-ansible master: Remove deny list check from cli-enable-ssh-admin.yaml https://review.opendev.org/c/openstack/tripleo-ansible/+/803611 | 14:46 |
opendevreview | James Slagle proposed openstack/tripleo-ansible master: Align tasks between config-download playbooks https://review.opendev.org/c/openstack/tripleo-ansible/+/803612 | 14:46 |
opendevreview | James Slagle proposed openstack/tripleo-ansible master: Remove cli-config-download-export.yaml https://review.opendev.org/c/openstack/tripleo-ansible/+/803613 | 14:46 |
cloudnull | TheJulia o/ | 14:48 |
cloudnull | sshnaidm ++ lgtm | 14:48 |
opendevreview | Martin Schuppert proposed openstack/tripleo-ansible master: Disable mlockall if postcopy is enabled https://review.opendev.org/c/openstack/tripleo-ansible/+/803614 | 14:49 |
opendevreview | Martin Schuppert proposed openstack/tripleo-ansible master: Disable mlockall if postcopy is enabled https://review.opendev.org/c/openstack/tripleo-ansible/+/803614 | 14:50 |
opendevreview | Francesco Pantano proposed openstack/tripleo-heat-templates master: DNM - Test Pending Ceph Pacific daemons in CI https://review.opendev.org/c/openstack/tripleo-heat-templates/+/778915 | 14:52 |
TheJulia | sshnaidm: ok, thanks | 14:52 |
*** ykarel is now known as ykarel|away | 14:53 | |
opendevreview | Shnaidman Sagi (Sergey) proposed openstack/tripleo-quickstart-extras master: Run TripleO jobs on Ansible 2.11 https://review.opendev.org/c/openstack/tripleo-quickstart-extras/+/797823 | 14:56 |
opendevreview | Martin Schuppert proposed openstack/tripleo-heat-templates master: Disable postcopy for ovn/ovs-dpdk https://review.opendev.org/c/openstack/tripleo-heat-templates/+/803616 | 14:57 |
*** jpena is now known as jpena|off | 15:15 | |
opendevreview | Jiri Podivin proposed openstack/validations-common master: DNM https://review.opendev.org/c/openstack/validations-common/+/803316 | 15:19 |
opendevreview | Shnaidman Sagi (Sergey) proposed openstack/tripleo-heat-templates master: Change tests to run on ansible-core 2.11 https://review.opendev.org/c/openstack/tripleo-heat-templates/+/803625 | 15:21 |
opendevreview | Kevin Carter proposed openstack/tripleo-specs master: TripleO.Next - Task-Core + Directord https://review.opendev.org/c/openstack/tripleo-specs/+/801630 | 15:23 |
opendevreview | amolkahat proposed openstack/tripleo-quickstart-extras master: [WIP] Migrate overcloud-deploy.sh shell script to ansible roles https://review.opendev.org/c/openstack/tripleo-quickstart-extras/+/803627 | 15:26 |
sshnaidm | odyssey4me, hey, can you please help with 'transfer' job in tripleo-ansible? It fails with 2.11: https://zuul.opendev.org/t/openstack/build/7876b8f086a242af92c6794e09904f3e/log/job-output.txt#1236 | 15:44 |
sshnaidm | odyssey4me, I don't see this var is defined, but in 2.9 it works fine | 15:44 |
sshnaidm | odyssey4me, so something changed between versions I suppose and fails the job | 15:45 |
odyssey4me | sshnaidm looking | 15:48 |
sshnaidm | odyssey4me, the patch: https://review.opendev.org/c/openstack/tripleo-ansible/+/803516 | 15:48 |
odyssey4me | sshnaidm looks like that came in from dciabrin https://opendev.org/openstack/tripleo-ansible/commit/6f75fd172326b010e680fc0bdda8d880e256a58e | 15:51 |
sshnaidm | odyssey4me, hmm.. not from here? https://github.com/openstack/tripleo-ansible/commit/a025bfec8fa9b807a89e89a82a0c6921bf158e40 | 15:54 |
odyssey4me | sshnaidm I see no reference to 'test2_excluded_files' in that commit? | 15:54 |
sshnaidm | odyssey4me, and where are test_dst_files defined? | 15:55 |
sshnaidm | I see the same pattern, but maybe I miss the something.. | 15:57 |
odyssey4me | sshnaidm tripleo_ansible/roles/tripleo_transfer/molecule/default/molecule.yml | 15:57 |
odyssey4me | all vars for molecule are specified in the molecule file which provides the inventory | 15:57 |
sshnaidm | odyssey4me, I see, so need to define there test2_excluded_files as well | 15:58 |
sshnaidm | how did it pass so far with 2.9.. no idea | 15:59 |
odyssey4me | sshnaidm I see the problem, but I have no idea how https://opendev.org/openstack/tripleo-ansible/commit/6f75fd172326b010e680fc0bdda8d880e256a58e could have passed tests | 15:59 |
odyssey4me | sshnaidm test2_excluded_files: "{{ test2_excluded_files | map('regex_replace', '(.*)', test2_dst_dir ~ '/\\1') }}" can only work if test2_excluded_files already exists, but it doesn't | 16:00 |
sshnaidm | yeah, mystery | 16:00 |
odyssey4me | sshnaidm best to try and find out from the author what the intent was there | 16:01 |
sshnaidm | odyssey4me, yeah, thanks, will ping dciabrin about that | 16:02 |
*** marios|ruck is now known as marios|out | 16:08 | |
*** sshnaidm is now known as sshnaidm|afk | 16:29 | |
bandini | sshnaidm|afk, odyssey4me: dciabrin is back on Monday from PTO (fyi) | 16:34 |
*** rpittau is now known as rpittau|afk | 16:41 | |
opendevreview | Merged openstack/tripleo-heat-templates master: Revert change to use community pacemaker_cluster module https://review.opendev.org/c/openstack/tripleo-heat-templates/+/803534 | 18:09 |
opendevreview | amolkahat proposed openstack/tripleo-quickstart-extras master: [WIP] Migrate overcloud-deploy.sh shell script to ansible roles https://review.opendev.org/c/openstack/tripleo-quickstart-extras/+/803627 | 18:20 |
opendevreview | Shnaidman Sagi (Sergey) proposed openstack/tripleo-quickstart-extras master: Run TripleO jobs on Ansible 2.11 https://review.opendev.org/c/openstack/tripleo-quickstart-extras/+/797823 | 18:23 |
opendevreview | Merged openstack/tripleo-heat-templates master: Replace the deprecated nova::scheduler::filter::scheduler_max_attempts https://review.opendev.org/c/openstack/tripleo-heat-templates/+/787962 | 18:31 |
-opendevstatus- NOTICE: The Gerrit service on review.opendev.org is going down for a quick restart to adjust its database connection configuration, and should return to service momentarily | 20:04 | |
opendevreview | Kevin Carter proposed openstack/tripleo-specs master: TripleO.Next - Task-Core + Directord https://review.opendev.org/c/openstack/tripleo-specs/+/801630 | 20:07 |
opendevreview | Francesco Pantano proposed openstack/tripleo-heat-templates master: DNM - Test Pending Ceph Pacific daemons in CI https://review.opendev.org/c/openstack/tripleo-heat-templates/+/778915 | 20:37 |
opendevreview | Shnaidman Sagi (Sergey) proposed openstack/tripleo-quickstart-extras master: Run TripleO jobs on Ansible 2.11 https://review.opendev.org/c/openstack/tripleo-quickstart-extras/+/797823 | 21:51 |
*** rlandy|ruck is now known as rlandy|ruck|bbl | 22:06 | |
opendevreview | Merged openstack/tripleo-ci master: CI: Trigger standalone job for changes related to iscsid https://review.opendev.org/c/openstack/tripleo-ci/+/800805 | 23:44 |
opendevreview | Merged openstack/tripleo-ci master: CI: Test ironic puppet manifests in standalone job https://review.opendev.org/c/openstack/tripleo-ci/+/800807 | 23:46 |
Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!