| *** rlandy|afk is now known as rlandy|out | 00:51 | |
| opendevreview | chenwei proposed openstack/tripleo-docs master: description about image tag is incorrect in tips_tricks.rst https://review.opendev.org/c/openstack/tripleo-docs/+/849304 | 01:48 |
|---|---|---|
| *** akahat is now known as akahat|ruck | 05:00 | |
| *** ysandeep|out is now known as ysandeep | 05:05 | |
| *** chkumar|rover is now known as chandankumar | 05:05 | |
| *** undefined is now known as Guest5693 | 05:12 | |
| opendevreview | chandan kumar proposed openstack/openstack-tempest-skiplist master: Revert "Skip LoadBalancerScenarioTest for octavia" https://review.opendev.org/c/openstack/openstack-tempest-skiplist/+/850555 | 05:22 |
| opendevreview | chandan kumar proposed openstack/tripleo-ci master: [DNM] standalone sc10 ovn skip testing https://review.opendev.org/c/openstack/tripleo-ci/+/850715 | 05:24 |
| ykarel | tkajinam, hi | 05:28 |
| ykarel | rememeber that hiera vs lookup thing with ovn non ha dbs? | 05:29 |
| ykarel | i need that for switching undercloud to ovn https://review.opendev.org/c/openstack/tripleo-heat-templates/+/850158 | 05:30 |
| ykarel | it works switching back to hiera function but pep8 fails as there is validation for it, i can add exclude for ovn dbs template so the check pass | 05:31 |
| ykarel | wanted to check if you see some other option to handle it | 05:31 |
| ykarel | iirc you mentioned full revert but /me not sure if it's worth just for this case, but you may better know | 05:32 |
| *** amoralej|off is now known as amoralej | 06:17 | |
| opendevreview | Takashi Kajinami proposed openstack/tripleo-heat-templates master: ovn: Replace hiera by lookup https://review.opendev.org/c/openstack/tripleo-heat-templates/+/850717 | 06:39 |
| tkajinam | ^^^ ykarel | 06:40 |
| ykarel | tkajinam, yeap i was also trying puppet lookup locally but i found it was returning too slow compared to hiera cli | 06:40 |
| tkajinam | I was looking for some solutions with heat templating but it requires a few more investigation | 06:41 |
| ykarel | it worked fine for you? | 06:41 |
| ykarel | speed wise | 06:41 |
| ykarel | i didn't tried --facts option though | 06:41 |
| tkajinam | hmm.. let me dig into it further | 06:42 |
| tkajinam | seems like system facts are still loaded even with that --facts | 06:42 |
| tkajinam | we might need to present the fact cache directory | 06:42 |
| ykarel | just checked --facts dummy json is also slow | 06:43 |
| tkajinam | yeah | 06:43 |
| ykarel | it takes approx 20 seconds on my side | 06:43 |
| *** arxcruz is now known as arxcruz|rover | 06:54 | |
| opendevreview | Takashi Kajinami proposed openstack/tripleo-heat-templates master: Fix indirect reference to hieradata https://review.opendev.org/c/openstack/tripleo-heat-templates/+/848305 | 07:08 |
| opendevreview | Takashi Kajinami proposed openstack/tripleo-heat-templates master: ovn: Replace hiera by lookup https://review.opendev.org/c/openstack/tripleo-heat-templates/+/850717 | 07:12 |
| opendevreview | Takashi Kajinami proposed openstack/tripleo-heat-templates master: etcd: Fix indirect reference to hieradata https://review.opendev.org/c/openstack/tripleo-heat-templates/+/850725 | 07:18 |
| ykarel | tkajinam, with ^ it works fine for you speed wise? | 07:25 |
| tkajinam | ykarel, I don't have any test env handy but that's following how we workarounded performance problem caused by fact gathering in the past. | 07:26 |
| tkajinam | these implementations are quite messy. I don't know how we ended up using hieradata although we hare not using any puppet code in it | 07:27 |
| ykarel | tkajinam, ok let's see how that goes but for now Fix indirect reference to hieradata is enough | 07:27 |
| ykarel | i see some issues will comemnt | 07:27 |
| tkajinam | ykarel, yeah but we need to replace the hiera CLI later as it is preventing us from bumping hiera version | 07:27 |
| tkajinam | hiera v3 was deprecated long ago. we need to switch to v5 | 07:28 |
| ykarel | tkajinam, yeap agree, i meant just for unblocking ovn standalone it's enough | 07:28 |
| tkajinam | yeah | 07:28 |
| ykarel | switching can be done in followup | 07:28 |
| tkajinam | that's why I'm submitting two patches separately | 07:28 |
| ykarel | tkajinam, added some comments on inderect reference patch please check | 07:32 |
| ykarel | once it's fixed i will rebase my patch over it | 07:32 |
| opendevreview | Takashi Kajinami proposed openstack/tripleo-heat-templates master: Fix indirect reference to hieradata https://review.opendev.org/c/openstack/tripleo-heat-templates/+/848305 | 07:33 |
| opendevreview | chandan kumar proposed openstack/tripleo-common master: Add an Ansible execution environment to our Build https://review.opendev.org/c/openstack/tripleo-common/+/776674 | 07:34 |
| opendevreview | Takashi Kajinami proposed openstack/tripleo-heat-templates master: Fix indirect reference to hieradata https://review.opendev.org/c/openstack/tripleo-heat-templates/+/848305 | 07:34 |
| opendevreview | chandan kumar proposed openstack/tripleo-common stable/wallaby: Add an Ansible execution environment to our Build https://review.opendev.org/c/openstack/tripleo-common/+/850559 | 07:34 |
| opendevreview | Takashi Kajinami proposed openstack/tripleo-heat-templates master: Fix indirect reference to hieradata https://review.opendev.org/c/openstack/tripleo-heat-templates/+/848305 | 07:39 |
| opendevreview | Takashi Kajinami proposed openstack/tripleo-heat-templates master: ovn: Replace hiera by lookup https://review.opendev.org/c/openstack/tripleo-heat-templates/+/850717 | 07:42 |
| *** ysandeep is now known as ysandeep|lunch | 07:44 | |
| opendevreview | yatin proposed openstack/tripleo-heat-templates master: Switch default ML2 plugin in Undercloud to OVN https://review.opendev.org/c/openstack/tripleo-heat-templates/+/850158 | 07:44 |
| opendevreview | Takashi Kajinami proposed openstack/tripleo-heat-templates master: Fix indirect reference to hieradata https://review.opendev.org/c/openstack/tripleo-heat-templates/+/848305 | 08:30 |
| opendevreview | chandan kumar proposed openstack/tripleo-common master: Add an Ansible execution environment to our Build https://review.opendev.org/c/openstack/tripleo-common/+/776674 | 08:30 |
| opendevreview | chandan kumar proposed openstack/tripleo-common stable/wallaby: Add an Ansible execution environment to our Build https://review.opendev.org/c/openstack/tripleo-common/+/850559 | 08:33 |
| opendevreview | Merged openstack/tripleo-heat-templates stable/train: Reduce frequency of task retries https://review.opendev.org/c/openstack/tripleo-heat-templates/+/849992 | 09:06 |
| opendevreview | chandan kumar proposed openstack/tripleo-ci master: [DNM] tcib_args https://review.opendev.org/c/openstack/tripleo-ci/+/850736 | 09:12 |
| opendevreview | chandan kumar proposed openstack/tripleo-ci master: [DNM] tcib_args https://review.opendev.org/c/openstack/tripleo-ci/+/850736 | 09:28 |
| opendevreview | Cedric Jeanneret proposed openstack/tripleo-ansible master: Better logging management for nftables rules https://review.opendev.org/c/openstack/tripleo-ansible/+/850222 | 09:35 |
| opendevreview | Cedric Jeanneret proposed openstack/tripleo-ansible master: Actually accept all on lo and ipv6-icmp https://review.opendev.org/c/openstack/tripleo-ansible/+/850620 | 09:36 |
| *** ysandeep|lunch is now known as ysandeep | 09:42 | |
| opendevreview | Cedric Jeanneret proposed openstack/tripleo-ansible master: Actually accept all on lo and ipv6-icmp https://review.opendev.org/c/openstack/tripleo-ansible/+/850620 | 09:45 |
| opendevreview | chandan kumar proposed openstack/tripleo-ci master: [DNM] tcib_args https://review.opendev.org/c/openstack/tripleo-ci/+/850736 | 10:09 |
| Tengu | chandankumar: heya! Care to have a review on https://review.opendev.org/c/openstack/ansible-role-collect-logs/+/850331 ? I've added some of the generated output, especially in the last comment. Would be great to get it in imho :) | 10:10 |
| opendevreview | Andre Aranha proposed openstack/tripleo-heat-templates stable/wallaby: Add credentials cache for novajoin user https://review.opendev.org/c/openstack/tripleo-heat-templates/+/850560 | 10:33 |
| *** rlandy|out is now known as rlandy | 10:33 | |
| opendevreview | chandan kumar proposed openstack/tripleo-ci master: [DNM] tcib_args https://review.opendev.org/c/openstack/tripleo-ci/+/850736 | 10:37 |
| chandankumar | Tengu: done! | 10:37 |
| opendevreview | chandan kumar proposed openstack/tripleo-ci master: [DNM] tcib_args https://review.opendev.org/c/openstack/tripleo-ci/+/850736 | 11:17 |
| Tengu | chandankumar: thanks! | 11:21 |
| Tengu | chandankumar: are we waiting for some more votes from CI folks? | 11:21 |
| chandankumar | Tengu: just got +wed | 11:28 |
| Tengu | thanks rlandy :) | 11:29 |
| Tengu | chandankumar: just got the notification :) | 11:29 |
| chandankumar | Tengu: hello | 11:39 |
| chandankumar | Tengu: https://review.rdoproject.org/r/c/testproject/+/44079 | 11:39 |
| chandankumar | here I am trying to build ansible ee container | 11:39 |
| chandankumar | we need to pass this https://review.opendev.org/c/openstack/tripleo-common/+/850559/2/container-images/tcib/tripleo-ansible-ee/tripleo-ansible-ee.yaml#3 | 11:40 |
| chandankumar | I am trying to pass it via this https://review.opendev.org/c/openstack/tripleo-ci/+/850736/5/roles/build-containers/templates/tripleo-build.sh.j2 | 11:40 |
| Tengu | chandankumar: ah, reviving cloudnull topic? | 11:41 |
| chandankumar | but during the run time it is failing 'tcib_release' is undefined | 11:41 |
| chandankumar | https://logserver.rdoproject.org/79/44079/4/check/tripleo-centos-8-tripleo-ansible-ee-container-wallaby/7548506/logs/build.log | 11:41 |
| chandankumar | Not sure what is going on | 11:41 |
| chandankumar | Tengu: yes, we need it for external compute work | 11:41 |
| Tengu | tcib_release is something that should be defined in the build env iirc | 11:41 |
| Tengu | or... isn't it one of those extra params I've added support for back then? | 11:42 |
| Tengu | ysandeep: "tcib_release" - does it ring any bells on your side? | 11:42 |
| chandankumar | Tengu: extra params which you have implemented | 11:42 |
| chandankumar | https://logserver.rdoproject.org/79/44079/4/check/tripleo-centos-8-tripleo-ansible-ee-container-wallaby/7548506/logs/build_containers.sh | 11:42 |
| chandankumar | it is passed already --tcib-extra tcib_release=8 \ | 11:43 |
| Tengu | I see it yeah | 11:43 |
| chandankumar | I think -extra-config /home/zuul/workspace/extra_config.yaml does not play nicelu with tcib_extra | 11:43 |
| chandankumar | need to reproduce locally | 11:44 |
| Tengu | chandankumar: did you see it's failing for tripleoclient, not tripleo-ansible-ee? | 11:44 |
| Tengu | chandankumar: also: 2022-07-22 11:34:55 | build_containers.sh: line 55: --volume: command not found | 11:44 |
| Tengu | I guess there's a missing \ somewhere. | 11:44 |
| Tengu | chandankumar: the blank line here: https://logserver.rdoproject.org/79/44079/4/check/tripleo-centos-8-tripleo-ansible-ee-container-wallaby/7548506/logs/build_containers.sh | 11:44 |
| Tengu | see, before the --volume /etc/yum.repos.d:/etc/distro.repos.d:z \ | 11:44 |
| Tengu | it's broken. | 11:44 |
| chandankumar | you mean extra space? | 11:45 |
| Tengu | chandankumar: commented here: https://review.opendev.org/c/openstack/tripleo-ci/+/850736 | 11:45 |
| opendevreview | chandan kumar proposed openstack/tripleo-ci master: [DNM] tcib_args https://review.opendev.org/c/openstack/tripleo-ci/+/850736 | 11:46 |
| Tengu | it should work better. | 11:46 |
| chandankumar | Tengu: ah, thanks trying now :-) | 11:46 |
| Tengu | np :) | 11:49 |
| * ysandeep was in a mtg, reading back | 11:50 | |
| Tengu | chandankumar: I'll be on PTO next week - but once I'm back, we can collaborate on that AEE thing. | 11:50 |
| Tengu | I wanted to hit that earlier, but with the 17.0 release sprint... ^^' | 11:50 |
| chandankumar | Tengu: sure, We can hash that out | 11:58 |
| dviroel | fultonj: https://review.opendev.org/c/openstack/tripleo-quickstart-extras/+/850349 - ceph-nightly jobs are still broken, I am making some change in standalone role that make things easier, imo. Please check when yuo have some time | 11:58 |
| fultonj | dviroel: +2 thanks for simplifying it | 12:01 |
| fultonj | fyi i'm on pto the rest of the day back monday | 12:01 |
| dviroel | fultonj: thanks, enjoy your pto | 12:03 |
| chandankumar | Tengu: it is working https://logserver.rdoproject.org/79/44079/4/check/tripleo-centos-8-tripleo-ansible-ee-container-wallaby/5faabbd/logs/container-builds/4c29fc6e-2e8f-40b9-92a6-d114b634e228/tripleo-ansible-ee/Dockerfile | 12:16 |
| chandankumar | ARG = TRIPLEO_ANSIBLE_REQ="/usr/share/openstack-tripleo-common-containers/container-images/kolla/tripleo-ansible-ee/requirements.yaml" | 12:17 |
| chandankumar | but somehow equal sign coming from somewhere | 12:17 |
| chandankumar | https://review.opendev.org/c/openstack/tripleo-ci/+/850736/6/roles/build-containers/tasks/main.yaml#209 seems to be correct ? | 12:17 |
| Tengu | gimme a sec | 12:17 |
| chandankumar | ok sure | 12:18 |
| Tengu | chandankumar: I don't remember - is TCIB code in a dedicated repo? or is it in tripleo-common? | 12:20 |
| *** amoralej is now known as amoralej|lunch | 12:20 | |
| Tengu | chandankumar: apparently AEE is the first (and only?) container using that tcib_args | 12:20 |
| opendevreview | yatin proposed openstack/tripleo-heat-templates master: Switch default ML2 plugin in Undercloud to OVN https://review.opendev.org/c/openstack/tripleo-heat-templates/+/850158 | 12:20 |
| Tengu | I have the feeling there's a wrong conversion at some point. | 12:20 |
| chandankumar | Tengu: yes | 12:20 |
| chandankumar | entry point https://opendev.org/openstack/python-tripleoclient/src/branch/master/tripleoclient/v2/tripleo_container_image.py | 12:21 |
| opendevreview | yatin proposed openstack/tripleo-heat-templates master: [DNM] OVN DHCP Test https://review.opendev.org/c/openstack/tripleo-heat-templates/+/850272 | 12:21 |
| Tengu | chandankumar: there's an issue in the way the Dockerfile is generated - lemme find out where. | 12:22 |
| chandankumar | it calls this https://opendev.org/openstack/python-tripleoclient/src/branch/master/tripleoclient/v2/tripleo_container_image.py#L676 | 12:23 |
| fultonj | i saw tripleo-ci-centos-9-content-provider fail twice in a row in https://review.opendev.org/c/openstack/python-tripleoclient/+/850309 | 12:24 |
| Tengu | found out | 12:24 |
| fultonj | AFAICT content provider doesn't use ceph so it shouldn't be affected by my change | 12:25 |
| Tengu | chandankumar: https://opendev.org/openstack/tripleo-ansible/src/branch/master/tripleo_ansible/roles/tripleo_container_image_build/templates/Containerfile.j2#L17 | 12:25 |
| chandankumar | We need to remove = from there | 12:25 |
| Tengu | chandankumar: indeed: https://docs.docker.com/engine/reference/builder/#arg | 12:26 |
| Tengu | want me to push a patch? | 12:26 |
| chandankumar | Tengu: yes, please | 12:26 |
| fultonj | i guess i'll recheck my job that failed on tripleo-ci-centos-9-content-provider | 12:28 |
| opendevreview | Cedric Jeanneret proposed openstack/tripleo-ansible master: Remove wrong equal sign https://review.opendev.org/c/openstack/tripleo-ansible/+/850747 | 12:28 |
| Tengu | chandankumar: -^ | 12:28 |
| fultonj | unless there's a known issue | 12:28 |
| opendevreview | chandan kumar proposed openstack/tripleo-ansible stable/wallaby: Remove wrong equal sign https://review.opendev.org/c/openstack/tripleo-ansible/+/850562 | 12:30 |
| fultonj | same deal w/ tripleo-ci-centos-9-content-provider-wallaby | 12:30 |
| fultonj | in https://review.opendev.org/c/openstack/tripleo-ansible/+/850307 | 12:30 |
| chandankumar | Tengu: I am also thinking about making this image customizable FROM quay.io/ansible/ansible-runner:latest via args | 12:33 |
| chandankumar | so that user can use custom image | 12:33 |
| fultonj | ruck? | 12:34 |
| fultonj | https://zuul.opendev.org/t/openstack/builds?job_name=tripleo-ci-centos-9-content-provider-wallaby&skip=0 | 12:35 |
| opendevreview | Jiri Podivin proposed openstack/python-tripleoclient master: Removing TODO lacking sufficient description https://review.opendev.org/c/openstack/python-tripleoclient/+/850748 | 12:39 |
| Tengu | chandankumar: hmmm, maybe, yeah. Though we should first ensure the default is building and usable within tripleo. | 12:43 |
| Tengu | chandankumar: also, I want to check with ansible-runner, in order to keep the container running instead of having it up'n'down for every single playbook. | 12:44 |
| Tengu | chandankumar: that only makes us lose about 10s PER PLAYBOOK. Which is unbearable. | 12:44 |
| opendevreview | Jiri Podivin proposed openstack/python-tripleoclient master: Removing Popen object return from utils.run_command_and_log https://review.opendev.org/c/openstack/python-tripleoclient/+/850749 | 12:52 |
| *** ysandeep is now known as ysandeep|afk | 13:07 | |
| *** amoralej|lunch is now known as amoralej | 13:07 | |
| *** Guest5693 is now known as rcastillo | 13:09 | |
| tkajinam | May I have 2nd review ? https://review.opendev.org/c/openstack/tripleo-heat-templates/+/848601 | 13:10 |
| tkajinam | wallaby backport was already merged. it has been left there for a while after my +2 | 13:11 |
| chandankumar | Tengu: ok, I will keep you posted | 13:11 |
| *** ysandeep|afk is now known as ysandeep | 13:31 | |
| *** ysandeep is now known as ysandeep|mtg | 13:31 | |
| chandankumar | Tengu: ++ thank you, it worked :-) | 14:01 |
| Tengu | chandankumar: heh, no problem :) | 14:02 |
| Tengu | it was an easy one! | 14:02 |
| Tengu | chandankumar: I'm about to drop - and I'll be back on Aug 2nd. Hope nothing will blow up in the meantime ;) | 14:03 |
| *** dasm|off is now known as dasm | 14:12 | |
| chandankumar | Tengu: enjoy your vacation :-) | 14:17 |
| opendevreview | Merged openstack/puppet-tripleo stable/wallaby: Support mariabackup as a galera SST method https://review.opendev.org/c/openstack/puppet-tripleo/+/850035 | 15:03 |
| opendevreview | Merged openstack/tripleo-common stable/wallaby: New dedicated user for Galera SST synchronization https://review.opendev.org/c/openstack/tripleo-common/+/850292 | 15:03 |
| opendevreview | Merged openstack/ansible-role-collect-logs master: Get a dedicated, well formatted file for dropped packets https://review.opendev.org/c/openstack/ansible-role-collect-logs/+/850331 | 15:03 |
| opendevreview | Merged openstack/validations-libs master: Expanding parser actions to allow for multiple key-value pairs https://review.opendev.org/c/openstack/validations-libs/+/817717 | 15:04 |
| tkajinam | https://review.opendev.org/c/openstack/tripleo-heat-templates/+/848601 | 15:10 |
| tkajinam | Unable to freeze job graph: Job tripleo-ci-centos-8-standalone-on-multinode-ipa does not specify a run playbook | 15:10 |
| opendevreview | Takashi Kajinami proposed openstack/tripleo-ci master: Revert "Create containers and standalone-ipa multinode -options jobs" https://review.opendev.org/c/openstack/tripleo-ci/+/850564 | 15:23 |
| opendevreview | Takashi Kajinami proposed openstack/tripleo-heat-templates stable/train: DNM: Testing 850564 https://review.opendev.org/c/openstack/tripleo-heat-templates/+/850764 | 15:24 |
| *** dviroel is now known as dviroel|lunch | 15:25 | |
| *** undefined is now known as Guest5751 | 15:26 | |
| *** ysandeep|mtg is now known as ysandeep|out | 15:26 | |
| *** Guest5751 is now known as rcastillo_ | 15:26 | |
| *** rpioso_ is now known as rpioso | 15:27 | |
| *** philroche_ is now known as philroche | 15:27 | |
| *** johnsom_ is now known as johnsom | 15:27 | |
| opendevreview | Takashi Kajinami proposed openstack/puppet-tripleo master: Remove reference to unused hieradata keys https://review.opendev.org/c/openstack/puppet-tripleo/+/831779 | 15:37 |
| opendevreview | Takashi Kajinami proposed openstack/puppet-tripleo master: Remove reference to unused hieradata keys https://review.opendev.org/c/openstack/puppet-tripleo/+/831779 | 15:38 |
| *** amoralej is now known as amoralej|off | 15:38 | |
| opendevreview | Rajesh Tailor proposed openstack/puppet-tripleo master: Fix typos in parameter descriptions, comments and tests https://review.opendev.org/c/openstack/puppet-tripleo/+/848988 | 15:52 |
| *** dviroel|lunch is now known as dviroel | 16:42 | |
| dviroel | tkajinam: hey, fixing now the multinode-ipa job issue | 17:13 |
| dviroel | tkajinam: or we would need 2 reverts | 17:13 |
| opendevreview | Bhagyashri Shewale proposed openstack/tripleo-ci master: Resolve freez graph issue https://review.opendev.org/c/openstack/tripleo-ci/+/850770 | 17:15 |
| dviroel | bhagyashris: oh, you were faster | 17:15 |
| bhagyashris | dviroel, hahaha | 17:15 |
| dviroel | bhagyashris: i was in the git-review command line already | 17:16 |
| dviroel | lol | 17:16 |
| dviroel | that should fix the conflicting branches, thanks | 17:16 |
| bhagyashris | tkajinam, hey this will solve the problem https://review.opendev.org/c/openstack/tripleo-ci/+/850770 | 17:16 |
| bhagyashris | yeah | 17:16 |
| opendevreview | Bhagyashri Shewale proposed openstack/tripleo-heat-templates stable/train: [collectd] Remove processes plugin from default list https://review.opendev.org/c/openstack/tripleo-heat-templates/+/848601 | 17:19 |
| opendevreview | Bhagyashri Shewale proposed openstack/tripleo-heat-templates stable/train: DNM: Testing 850564 https://review.opendev.org/c/openstack/tripleo-heat-templates/+/850764 | 17:24 |
| *** dviroel is now known as dviroel|afk | 17:28 | |
| *** rcastillo_ is now known as rcastillo | 17:54 | |
| opendevreview | Merged openstack/tripleo-ansible master: Add raft election timer support https://review.opendev.org/c/openstack/tripleo-ansible/+/848453 | 19:54 |
| *** rlandy is now known as rlandy|biab | 20:42 | |
| *** rlandy|biab is now known as rlandy | 21:17 | |
| opendevreview | Merged openstack/tripleo-ci master: Resolve freez graph issue https://review.opendev.org/c/openstack/tripleo-ci/+/850770 | 22:00 |
| *** dasm is now known as dasm|off | 22:11 | |
| opendevreview | Takashi Kajinami proposed openstack/tripleo-heat-templates stable/train: [collectd] Remove processes plugin from default list https://review.opendev.org/c/openstack/tripleo-heat-templates/+/848601 | 22:29 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!