| *** chem has quit IRC | 00:51 | |
| *** rlandy is now known as rlandy|bbl | 00:55 | |
| *** rfolco has quit IRC | 01:41 | |
| *** dsneddon has quit IRC | 01:48 | |
| *** dsneddon has joined #oooq | 02:24 | |
| *** dsneddon has quit IRC | 02:49 | |
| *** apetrich has quit IRC | 03:10 | |
| *** dsneddon has joined #oooq | 03:16 | |
| *** dsneddon has quit IRC | 03:22 | |
| *** rlandy|bbl has quit IRC | 03:39 | |
| *** dsneddon has joined #oooq | 03:50 | |
| *** dsneddon has quit IRC | 03:59 | |
| *** epoojad1 has joined #oooq | 04:26 | |
| *** dsneddon has joined #oooq | 04:26 | |
| *** dsneddon has quit IRC | 04:31 | |
| *** udesale has joined #oooq | 04:44 | |
| *** udesale has quit IRC | 04:44 | |
| *** udesale has joined #oooq | 04:44 | |
| *** saneax has quit IRC | 04:55 | |
| *** dsneddon has joined #oooq | 05:04 | |
| *** dsneddon has quit IRC | 05:08 | |
| *** soniya29 has joined #oooq | 05:11 | |
| *** ykarel|away is now known as ykarel | 05:23 | |
| *** skramaja has joined #oooq | 05:25 | |
| *** udesale_ has joined #oooq | 05:34 | |
| *** udesale has quit IRC | 05:37 | |
| *** epoojad1 has quit IRC | 05:40 | |
| *** dsneddon has joined #oooq | 05:41 | |
| *** raukadah is now known as chandankumar | 05:45 | |
| *** dsneddon has quit IRC | 05:46 | |
| *** saneax has joined #oooq | 06:01 | |
| *** surpatil has joined #oooq | 06:03 | |
| *** SurajPatil has joined #oooq | 06:04 | |
| *** yolanda has quit IRC | 06:04 | |
| *** saneax has quit IRC | 06:06 | |
| *** epoojad1 has joined #oooq | 06:08 | |
| *** pojadhav has joined #oooq | 06:13 | |
| *** pojadhav has left #oooq | 06:17 | |
| *** epoojad1 has quit IRC | 06:18 | |
| *** dsneddon has joined #oooq | 06:20 | |
| *** dsneddon has quit IRC | 06:25 | |
| *** pojadhav has joined #oooq | 06:28 | |
| *** jfrancoa has joined #oooq | 06:50 | |
| *** saneax has joined #oooq | 06:53 | |
| *** dsneddon has joined #oooq | 06:57 | |
| *** dsneddon has quit IRC | 07:02 | |
| *** ksambor has joined #oooq | 07:02 | |
| *** soniya29 has quit IRC | 07:12 | |
| *** soniya29 has joined #oooq | 07:17 | |
| *** yolanda has joined #oooq | 07:25 | |
| *** jtomasek has joined #oooq | 07:25 | |
| *** marios has joined #oooq | 07:27 | |
| *** dsneddon has joined #oooq | 07:33 | |
| *** yolanda has quit IRC | 07:33 | |
| *** yolanda has joined #oooq | 07:34 | |
| *** ykarel is now known as ykarel|lunch | 07:36 | |
| *** dsneddon has quit IRC | 07:38 | |
| *** yolanda has quit IRC | 07:48 | |
| *** dsneddon has joined #oooq | 08:13 | |
| *** dsneddon has quit IRC | 08:19 | |
| *** amoralej|off is now known as amoralej | 08:20 | |
| *** tesseract has joined #oooq | 08:20 | |
| *** yolanda has joined #oooq | 08:21 | |
| *** ykarel|lunch is now known as ykarel | 08:38 | |
| *** jpena|off is now known as jpena | 08:52 | |
| *** holser has joined #oooq | 08:54 | |
| *** dtantsur|afk is now known as dtantsur | 08:57 | |
| *** soniya29 has quit IRC | 09:04 | |
| *** tosky has joined #oooq | 09:07 | |
| *** bogdando has joined #oooq | 09:07 | |
| *** apetrich has joined #oooq | 09:18 | |
| *** jaosorior has joined #oooq | 09:19 | |
| *** dsneddon has joined #oooq | 09:25 | |
| *** yolanda has quit IRC | 09:27 | |
| *** udesale_ has quit IRC | 09:28 | |
| *** SurajPatil has quit IRC | 09:28 | |
| *** udesale_ has joined #oooq | 09:28 | |
| *** derekh has joined #oooq | 09:28 | |
| *** SurajPatil has joined #oooq | 09:28 | |
| *** SurajPatil has quit IRC | 09:29 | |
| *** surpatil has quit IRC | 09:29 | |
| *** surpatil has joined #oooq | 09:30 | |
| *** dsneddon has quit IRC | 09:30 | |
| *** yolanda has joined #oooq | 09:33 | |
| *** soniya29 has joined #oooq | 09:36 | |
| *** yolanda has quit IRC | 09:49 | |
| *** jaosorior has quit IRC | 09:54 | |
| *** chem has joined #oooq | 10:02 | |
| *** dsneddon has joined #oooq | 10:04 | |
| arxcruz | zbr|drover: chandankumar please review and if possible merge https://review.opendev.org/#/c/703570/ | 10:06 |
|---|---|---|
| *** dsneddon has quit IRC | 10:09 | |
| *** ykarel is now known as ykarel|afk | 10:11 | |
| *** dsneddon has joined #oooq | 10:42 | |
| *** ykarel|afk is now known as ykarel | 10:43 | |
| *** dsneddon has quit IRC | 10:47 | |
| pojadhav | Hi all, While running overcloud deployment command for tripleo quickstart and I am facing error : AnsibleUndefinedVariable: 'ansible.vars.hostvars.HostVarsVars object' has no attribute 'inventory_ip' | 10:50 |
| *** udesale_ has quit IRC | 10:51 | |
| pojadhav | could anyone help me to resolve this issue. | 10:53 |
| sshnaidm | pojadhav, look for case you use 'inventory_ip' somewhere, I'm not familiar with this variable tbh | 11:06 |
| sshnaidm | pojadhav, grep -r 'inventory_ip' tripleo-* | 11:06 |
| arxcruz | is inventory ip that one that needs to be 127.0.0.2 in quickstart ? | 11:19 |
| *** surpatil has quit IRC | 11:26 | |
| chandankumar | marios, Hello | 11:27 |
| chandankumar | marios, https://6c9c71359db665d0f3a4-5067ad0e5cf1f9e6b3e5798c03f452f6.ssl.cf1.rackcdn.com/703587/1/check/openstack-tox-docs/9906bac/docs/ci/third_party_dependencies_ci.html here hyperlinks are not working please have a look | 11:28 |
| chandankumar | arxcruz, thanks for the review, I will update that soon | 11:29 |
| marios | chandankumar: o/ | 11:31 |
| marios | chandankumar: sure will hav a look (was confusd until i realised you are asking for review @ https://review.opendev.org/#/c/703587 ) | 11:32 |
| zbr|drover | sshnaidm: does https://review.opendev.org/#/c/703586/ look small enough? | 11:33 |
| marios | chandankumar: commented .. not :: | 11:36 |
| *** soniya29 has quit IRC | 11:38 | |
| *** soniya29 has joined #oooq | 11:39 | |
| sshnaidm | zbr|drover, "warn: false" because of using shell? | 11:42 |
| zbr|drover | yeah, and that is not a linting error, is an ansible runtime warning. | 11:43 |
| sshnaidm | zbr|drover, ok | 11:45 |
| sshnaidm | zbr|drover, commented | 11:47 |
| weshay|ruck | marios, had to revert that get_hash path update | 11:55 |
| marios | weshay|ruck: yeah saw posted https://review.rdoproject.org/r/24617 | 11:55 |
| marios | weshay|ruck: i think it was the missing required-projects but we probably need it in all the base jobs there | 11:56 |
| marios | weshay|ruck: thanks for handling the revert | 11:56 |
| weshay|ruck | marios, k.. I posted a log path in the revert | 11:56 |
| weshay|ruck | np | 11:56 |
| weshay|ruck | we can try again today | 11:56 |
| marios | weshay|ruck: see https://review.rdoproject.org/r/#/c/24617/2/zuul.d/tripleo-rdo-base.yaml comments | 11:57 |
| *** dsneddon has joined #oooq | 12:00 | |
| *** rfolco has joined #oooq | 12:00 | |
| sshnaidm | weshay|ruck, mtg? | 12:01 |
| weshay|ruck | sshnaidm, will catch you later today | 12:01 |
| *** dtantsur is now known as dtantsur|bbl | 12:01 | |
| sshnaidm | weshay|ruck, ack | 12:01 |
| *** dsneddon has quit IRC | 12:05 | |
| *** yolanda has joined #oooq | 12:10 | |
| *** yolanda has quit IRC | 12:10 | |
| *** yolanda has joined #oooq | 12:11 | |
| *** amoralej is now known as amoralej|lunch | 12:15 | |
| *** jpena is now known as jpena|lunch | 12:21 | |
| zbr|drover | sshnaidm: i removed the non-esentials from patch, to make it easy to review/merge, see https://review.opendev.org/#/c/703597/1/defaults/main.yml | 12:28 |
| zbr|drover | i did the same with the gz one https://review.opendev.org/#/c/702862/10 -- which apparently worked fine (rdo failure unrelated, and collection did run) | 12:30 |
| sshnaidm | zbr|drover, what's the problem with HOME? | 12:30 |
| zbr|drover | the only thing making harder to review is the sove -> default scenario move. | 12:30 |
| zbr|drover | tox hides HOME by default. I could passenv it, but there are other cases where HOME may be lost | 12:31 |
| zbr|drover | is better to have a valid fallback value instead of empty string, which makes it attempt to write to /.quickstart | 12:31 |
| sshnaidm | zbr|drover, how is tox related here?? | 12:31 |
| zbr|drover | molecule runs from tox | 12:32 |
| sshnaidm | zbr|drover, runs where? | 12:32 |
| zbr|drover | openstack-tox-molecule | 12:32 |
| sshnaidm | zbr|drover, I mean on which platform | 12:32 |
| sshnaidm | zbr|drover, do you run log role in molecule on windows? | 12:33 |
| zbr|drover | nope, is does not matter which platform. | 12:34 |
| zbr|drover | it seems that you are not aware of the need to add passenv = HOME to make HOME available. | 12:34 |
| sshnaidm | zbr|drover, I think there is always default HOME in platform, no matter tox or not tox | 12:35 |
| zbr|drover | sshnaidm: NO, read https://github.com/tox-dev/tox/blob/e639a54eab5641c8dde353a29a516e4bf5487e37/src/tox/config/__init__.py#L670-L680 | 12:36 |
| zbr|drover | usually we add it to passenv in tox.ini, but is even better to fix the role. | 12:37 |
| sshnaidm | zbr|drover, do you want to say tox unset HOME var and if you do "echo $HOME" inside a molecule you'll get undefined? | 12:37 |
| zbr|drover | yeah | 12:38 |
| sshnaidm | zbr|drover, show me this please | 12:38 |
| marios | anyone know what we're using that file for? https://github.com/rdo-infra/review.rdoproject.org-config/blob/master/playbooks/tripleo-ci-periodic-base/post-role.yaml don't see much history on it | 12:41 |
| weshay|ruck | rfolco, you get em all built? | 12:42 |
| rfolco | weshay|ruck, 120 | 12:45 |
| rfolco | weshay|ruck, the epel workaround is that if you don't have epel repo the build will fail. It tried to disable at some point in the build | 12:46 |
| rfolco | weshay|ruck, maybe its under the rhel8 condition, we may need to change that for centos8 | 12:47 |
| *** ykarel is now known as ykarel|afk | 12:47 | |
| sshnaidm | marios, seems like for ovb only, to report to dlrn | 12:47 |
| sshnaidm | hosts: primary:tripleo-ovb-centos-7 | 12:48 |
| zbr|drover | sshnaidm: check https://github.com/ssbarnea/please-sagi/blob/master/tox.ini and run tox | 12:49 |
| marios | sshnaidm: hmm thanks. might need to update any job definition that uses it so it can consume the changes https://review.rdoproject.org/r/#/c/24617/3/roles/dlrn-report/tasks/dlrn-vars-setup.yml | 12:50 |
| chandankumar | weshay|ruck, zbr|drover you might be hitting this in some jobs https://github.com/pypa/pip/issues/7217 | 12:50 |
| chandankumar | pip==20.0.1 will fix it | 12:51 |
| marios | sshnaidm: i couldnt find any job definition under config repo which uses that post-role.yaml or the pre-role.yaml in same directory | 12:51 |
| weshay|ruck | chandankumar, https://bugs.launchpad.net/tripleo/+bug/1860430 | 12:51 |
| openstack | Launchpad bug 1860430 in tripleo "cannot import name SourceDistribution" [Critical,Triaged] | 12:51 |
| zbr|drover | chandankumar: good luck with that, only seeing the URL gave me headaches, there no other project I have more than pip | 12:52 |
| chandankumar | zbr|drover, http://logs.rdoproject.org/70/703570/1/openstack-check/tripleo-ci-centos-7-ovb-1ctlr_1comp-featureset001/3e7d4c5/job-output.txt | 12:53 |
| chandankumar | seen here | 12:53 |
| zbr|drover | yeah pip v20 released today.... https://pypi.org/project/pip/#history | 12:53 |
| zbr|drover | time to take the day off ;) | 12:54 |
| zbr|drover | or to watch talks on infra, probably they will pin it down | 12:54 |
| sshnaidm | zbr|drover, show me please when it's broken with tox molecule job, I'd like to see lookup("HOME") not working in molecule | 12:56 |
| sshnaidm | zbr|drover, that would justify this change | 12:56 |
| sshnaidm | marios, possibly it's not used | 12:58 |
| *** udesale has joined #oooq | 12:59 | |
| marios | sshnaidm: ack thx for checking yeah that's what i was asking earlier... i can't see it used was wondering if someone remembers why/where ... git blame says rlandy will ask when around thanks | 13:01 |
| *** dsneddon has joined #oooq | 13:07 | |
| *** rlandy has joined #oooq | 13:07 | |
| *** jbadiapa has joined #oooq | 13:09 | |
| *** dsneddon has quit IRC | 13:17 | |
| weshay|ruck | marios, rfolco | 13:18 |
| weshay|ruck | [zuul@wes workspace]$ cat containers-successfully-built.log | wc -l | 13:18 |
| weshay|ruck | 112 | 13:18 |
| weshay|ruck | [zuul@wes workspace]$ cat containers-failed-to-build.log | wc -l | 13:18 |
| weshay|ruck | 2 | 13:18 |
| weshay|ruck | is my latest status | 13:18 |
| weshay|ruck | you both have access to this box now | 13:18 |
| *** zbr|drover has quit IRC | 13:18 | |
| *** zbr has joined #oooq | 13:19 | |
| weshay|ruck | [zuul@wes workspace]$ cat containers-failed-to-build.log | 13:19 |
| weshay|ruck | qdrouterd | 13:19 |
| weshay|ruck | rabbitmq | 13:19 |
| rfolco | weshay|ruck, epel!!!! | 13:19 |
| weshay|ruck | rfolco, can't use epel :) | 13:19 |
| rfolco | weshay|ruck, you did not get it yet | 13:19 |
| weshay|ruck | I guess | 13:19 |
| rfolco | weshay|ruck, create a fake epel.repo | 13:19 |
| rfolco | it tries to disable a repo called epel | 13:20 |
| rfolco | it does not use it | 13:20 |
| weshay|ruck | oh | 13:20 |
| rfolco | and rabbit | 13:20 |
| rfolco | is the kolla patch pinning it | 13:20 |
| weshay|ruck | I have epel repos.. they are just disabled | 13:20 |
| rfolco | go to docker/rabbitmq/Dockerfile.j2 and remove 3.7.* pinning | 13:20 |
| weshay|ruck | rfolco, let's get it in a review and pull the review in the role | 13:21 |
| *** pojadhav is now known as pojadhav|away | 13:21 | |
| rfolco | weshay|ruck, it is in Mark's patch, its pinning 3.7.* I don't know why | 13:22 |
| rfolco | https://review.opendev.org/#/c/699414/9/docker/rabbitmq/Dockerfile.j2 | 13:22 |
| *** jpena|lunch is now known as jpena | 13:23 | |
| weshay|ruck | rfolco, let's take this review... resubmit w/ a new commit-id and pull it in our ci | 13:23 |
| rfolco | ok | 13:24 |
| rfolco | cat containers-expected-to-build.log | wc -l | 13:24 |
| rfolco | 122 | 13:24 |
| rfolco | I've got 120 | 13:24 |
| rfolco | excluded skydive, trying to build it again w/ skydive | 13:24 |
| weshay|ruck | 99% sure we don't need those.. but we'll ask in #tripleo mtg | 13:25 |
| weshay|ruck | rfolco, what about line 26 | 13:25 |
| weshay|ruck | {% set rabbitmq_packages = rabbitmq_packages + [ | 13:25 |
| weshay|ruck | 'rabbitmq-server-3.7.10' | 13:25 |
| rfolco | weshay|ruck, thats what I said, I don't understand why the pinning, it says remove the pinning when the deps are resolved. And I tried without the pinning, it works | 13:26 |
| weshay|ruck | rfolco, it's pinned on two lines though | 13:26 |
| rfolco | weshay|ruck, it falls on dnf case btw | 13:26 |
| * weshay|ruck asking about the second | 13:26 | |
| rfolco | weshay|ruck, the second will fall on centos7 I believe | 13:26 |
| chandankumar | sshnaidm, weshay|ruck https://review.opendev.org/#/c/703555/ | 13:27 |
| weshay|ruck | soniya29, can you test https://review.rdoproject.org/r/#/c/24616/ | 13:27 |
| chandankumar | os_tempest dir linls | 13:27 |
| soniya29 | weshay|ruck, sure | 13:28 |
| rfolco | mjturek, did you add your topic to the hackmd agenda ? https://hackmd.io/IhMCTNMBSF6xtqiEd9Z0Kw?both | 13:29 |
| weshay|ruck | rfolco, ping me w/ your dup patch of https://review.opendev.org/#/c/699414/ | 13:29 |
| rfolco | community ci call starts in 1 min at https://meet.google.com/oiv-geho-mai | 13:29 |
| rfolco | agenda https://hackmd.io/IhMCTNMBSF6xtqiEd9Z0Kw?both | 13:29 |
| weshay|ruck | panda, please make sure you attend ^ | 13:30 |
| rfolco | marios, weshay|ruck arxcruz zbr sshnaidm rlandy panda mjturek baha ^ | 13:30 |
| *** marios is now known as marios|call | 13:30 | |
| rfolco | sorry | 13:30 |
| rfolco | https://meet.google.com/bqx-xwht-wky | 13:30 |
| rfolco | correct gmeet ^ | 13:30 |
| arxcruz | rfolco: the calendar is pointing to a different code | 13:30 |
| mjturek | just did rfolco sorry | 13:31 |
| *** skramaja has quit IRC | 13:31 | |
| arxcruz | ah | 13:31 |
| arxcruz | ok | 13:31 |
| *** saneax has quit IRC | 13:44 | |
| *** dsneddon has joined #oooq | 13:48 | |
| *** dsneddon has quit IRC | 13:53 | |
| *** amoralej|lunch is now known as amoralej | 13:57 | |
| *** dtantsur|bbl is now known as dtantsur | 14:01 | |
| zbr | gz dropping: https://review.opendev.org/#/c/702862/ and its parent, look for previous runs, or https://e6792871b9906e88b5d6-ba5467d2de73878d3dd4296de3d69c75.ssl.cf1.rackcdn.com/702862/10/check/openstack-tox-molecule/142504e/ | 14:07 |
| *** marios|call is now known as marios | 14:09 | |
| *** bhagyashris_ has joined #oooq | 14:09 | |
| *** bhagyashris_ has quit IRC | 14:10 | |
| weshay|ruck | sshnaidm, available now if you want | 14:23 |
| sshnaidm | weshay|ruck, k | 14:23 |
| weshay|ruck | https://meet.google.com/opj-yscd-ktw?authuser=1 | 14:23 |
| weshay|ruck | rlandy, please ping me later re: the osp container issue | 14:24 |
| rlandy | weshay|ruck: k - working on it - retesting now | 14:25 |
| *** dsneddon has joined #oooq | 14:26 | |
| *** dsneddon has quit IRC | 14:31 | |
| rlandy | marios: where is the question in your review? may be used downstream - will check | 14:31 |
| marios | rlandy: ack will comment there sec but 14:41 < marios> anyone know what we're using that file for? https://github.com/rdo-infra/review.rdoproject.org-config/blob/master/playbooks/tripleo-ci-periodic-base/post-role.yaml don't see much history on it | 14:35 |
| rlandy | marios: so it's like this ... | 14:42 |
| rlandy | https://github.com/rdo-infra/review.rdoproject.org-config/tree/master/playbooks/tripleo-ci-periodic-base | 14:42 |
| rlandy | has playbooks that do dlrn reporting | 14:42 |
| rlandy | downstream we can't import playbooks | 14:42 |
| rlandy | but we can reuse roles | 14:43 |
| rlandy | https://github.com/rdo-infra/review.rdoproject.org-config/blob/master/roles/dlrn-report/tasks/main.yml | 14:43 |
| rlandy | is what gets used downstream | 14:43 |
| rlandy | http://git.app.eng.bos.redhat.com/git/openstack/tripleo-ci-internal-config.git/tree/playbooks/dlrn/post-dlrn.yaml | 14:44 |
| rlandy | so now we will have to fix internal/downstream as well for your get-hash change | 14:45 |
| rlandy | it will break those jobs | 14:45 |
| *** soniya29 has quit IRC | 14:45 | |
| rlandy | marios: iirc, we added those playbooks to consume the same roles upstream | 14:47 |
| rlandy | marios: I can put in associated patches so that the get-hash move will not break downstream | 14:48 |
| rlandy | it will atm | 14:48 |
| rlandy | weshay|ruck: ^^ fyi | 14:48 |
| marios | rlandy: ack thx, makes sense. yeah we need to change that since it consumes the file i'm changing. we now have a requirement for rdo-infra/ci-config wherever we are using get-hash or anythign that calls get-hash | 14:49 |
| rlandy | marios: ok - give me a few - I'll find the equivalent downstream requirements | 14:50 |
| marios | rlandy: ok no rush thanks | 14:50 |
| marios | for clarity it's the pre-yaml we're most interested in | 14:53 |
| marios | rlandy: https://github.com/rdo-infra/review.rdoproject.org-config/blob/master/playbooks/tripleo-ci-periodic-base/pre-role.yaml#L7 since that calls https://review.rdoproject.org/r/#/c/24617/3/roles/dlrn-report/tasks/dlrn-vars-setup.yml | 14:53 |
| marios | rlandy: but i couldn't see where it was used in a job definition to update required-projects... hound didn't help me | 14:54 |
| rlandy | hound won;t search downstream | 14:54 |
| rlandy | the problem is inheritance | 14:54 |
| rlandy | and reuse | 14:54 |
| rlandy | sec - commenting on the patch | 14:54 |
| marios | rlandy: mine does (hav bash alias that does all 3 hound) | 14:54 |
| rlandy | marios: we never used the playbook I gues | 14:55 |
| rlandy | the role is the important one | 14:55 |
| marios | rlandy: like that (hound) fwiw http://paste.openstack.org/raw/788643/ | 14:55 |
| marios | rlandy: so i couldn't find it in the d/stream hound either | 14:55 |
| rlandy | you'll find include_role: | 14:56 |
| rlandy | name: dlrn-report | 14:56 |
| marios | rlandy: yeah i saw that bit | 14:56 |
| marios | rlandy: but i was looking for a job pre that has that pre-role.yaml | 14:56 |
| rlandy | marios: k - responded on patch | 14:59 |
| rlandy | marios: actually, this would be a good time to make use of the dlrn_report role | 15:00 |
| weshay|ruck | rfolco, patch? | 15:00 |
| rlandy | marios: because then we would have a single place to deal with this | 15:01 |
| rfolco | weshay|ruck, ok I was looking at the diff | 15:01 |
| rlandy | side point :) | 15:01 |
| rlandy | marios: sum total, please don't delete that playbook | 15:01 |
| rfolco | weshay|ruck, these are built for centos7, not built for centos8 (and doesn't fail/complain) : | 15:01 |
| rfolco | ec2-api | 15:01 |
| rfolco | nova-serialproxy | 15:01 |
| rfolco | panko-base | 15:01 |
| rfolco | panko-api | 15:01 |
| rfolco | sensu-base | 15:01 |
| rfolco | weshay|ruck, will create the patch now | 15:02 |
| marios | rlandy: no i am not proposing to delete it. my question was, is it being consumed somewhere, because if so we need to ensure the job definition that uses it include rdo-infra/ci-config in required-projects: | 15:02 |
| marios | rlandy: sounds like 'not yet' so thanks for the sanity check in that case i don't need to update something more or block https://review.rdoproject.org/r/#/c/24617/3/playbooks/tripleo-ci-periodic-base/pre.yaml | 15:02 |
| rlandy | marios: no - juts put a temp -1 there while I ma putting up equivalent downstream patch | 15:03 |
| rlandy | will be up for review shortly | 15:03 |
| marios | rlandy: ack please go ahead and do that ^^ | 15:03 |
| marios | rlandy: and you can remove it when you're ready | 15:03 |
| marios | rlandy: i will revisit tomoorw | 15:03 |
| marios | tomorrow | 15:03 |
| rlandy | ack | 15:03 |
| marios | rlandy: thank you | 15:03 |
| weshay|ruck | zbr, comments on https://review.opendev.org/#/c/702862/10 | 15:04 |
| weshay|ruck | zbr, for size.. fyi http://logs.rdoproject.org/62/702862/10/openstack-check/tripleo-ci-centos-7-ovb-1ctlr_1comp-featureset001/428f9e6/logs/quickstart_files/log-size.txt | 15:05 |
| zbr | weshay|ruck: clark is not avl, but got confirmation from fungi that is ok. they do the compression. | 15:07 |
| weshay|ruck | nice | 15:07 |
| zbr | there are some low concerns reg temp disk on worker, but we are not asked to do anything. if they find an issue, besure I will be pinged first. | 15:07 |
| zbr | 0.5G is not an issue. | 15:08 |
| chandankumar | weshay|ruck, zbr sorry you mean we are going to upload 0.5 g temp dir on log server? | 15:12 |
| weshay|ruck | per job | 15:12 |
| * weshay|ruck checks what we have now w/ compression | 15:12 | |
| chandankumar | weshay|ruck, I think we need to discuss this with jpena and team | 15:12 |
| zbr | chandankumar: not log server, zuul-worker. | 15:13 |
| weshay|ruck | http://logs.rdoproject.org/12/558412/14/openstack-check/tripleo-ci-centos-7-ovb-3ctlr_1comp-featureset001-stein-branch/72bf1ed/logs/quickstart_files/log-size.txt | 15:13 |
| weshay|ruck | 147M | 15:13 |
| chandankumar | zbr, does zuul-worker will not go OOM? | 15:14 |
| chandankumar | due to 0.5 g | 15:14 |
| *** TrevorV has joined #oooq | 15:14 | |
| weshay|ruck | zbr, [08:16:39] <pabelanger> weshay|ruck: set zuul_log_compress: true in your base-minimal jobs and will fix | 15:17 |
| weshay|ruck | https://review.opendev.org/#/c/567696/3 | 15:17 |
| jpena | weshay|ruck: ^^ at least that change will only compress console output, not every single file | 15:20 |
| rfolco | weshay|ruck, https://review.opendev.org/703640 Remove rabbitmq pinning for CentOS8 | 15:22 |
| weshay|ruck | jpena, :( | 15:25 |
| weshay|ruck | rfolco, k | 15:25 |
| migi | weshay|ruck: owalsh thinks this patch broke the featureset021 https://review.opendev.org/#/c/699502/ | 15:32 |
| weshay|ruck | migi, hrm.. k.. I will test that | 15:32 |
| weshay|ruck | migi++ | 15:33 |
| weshay|ruck | thanks for looking into it man :) | 15:33 |
| migi | weshay|ruck: it's owalsh who found it :) | 15:33 |
| migi | weshay|ruck: but it's just a guess so don't take it as resolution | 15:35 |
| *** dsneddon has joined #oooq | 15:37 | |
| zbr | weshay|ruck: so what to do? | 15:38 |
| zbr | enable the change now and look to improve the compression logic on upload-logs after? https://opendev.org/zuul/zuul-jobs/src/commit/7755ef1b8d4d7803c9ad330a1bd9599715e06c60/roles/upload-logs/tasks/main.yaml | 15:40 |
| weshay|ruck | migi, I can test w/ a test revert | 15:40 |
| migi | weshay|ruck: look at this file, all IPs are same https://sf.hosted.upshift.rdu2.redhat.com/logs/36/189436/1/check/periodic-tripleo-ci-centos-7-bm_envD-1ctlr_2comp-featureset021-master/9150aee/logs/overcloud-controller-0/etc/ssh/ssh_known_hosts.gz | 15:40 |
| zbr | it will take at least two hours to merge, so enough time to make it upload-logs big more agressive | 15:40 |
| migi | weshay|ruck: same for other overcloud-* | 15:40 |
| weshay|ruck | zbr, we need logs to work.. so continue.. as we work out compression | 15:41 |
| zbr | i would write a bash script that archives any .txt/.log file above 10mb | 15:41 |
| weshay|ruck | zbr, we probably will need an option to compress for rdo | 15:41 |
| weshay|ruck | jpena, ^ | 15:41 |
| zbr | jpena: i was considering making this task more generic: https://opendev.org/zuul/zuul-jobs/src/commit/7755ef1b8d4d7803c9ad330a1bd9599715e06c60/roles/upload-logs/tasks/main.yaml#L35-L43 | 15:42 |
| *** dsneddon has quit IRC | 15:42 | |
| *** owalsh has joined #oooq | 15:42 | |
| zbr | obviously that there is no existing scenario for upload-logs | 15:43 |
| weshay|ruck | rfolco, I think you have a conflict http://logs.rdoproject.org/40/703640/1/openstack-check/tripleo-build-containers-rhel-8-master/805846d/job-output.txt | 15:43 |
| zbr | in fact i cannot update this role as is upsteam, they don't care about that archival. | 15:44 |
| weshay|ruck | rfolco, although that applied I think for me | 15:45 |
| weshay|ruck | marios, ping me if you have a sec | 15:45 |
| zbr | do we have an existing role that could be hacked to also do the compresion or I just add a new one? | 15:46 |
| chandankumar | see ya | 15:48 |
| *** chandankumar is now known as raukadah | 15:48 | |
| marios | weshay|ruck: sure in 10 mins ok? | 15:50 |
| weshay|ruck | marios, np | 15:50 |
| weshay|ruck | kopecmartin, still cool for tomorrow's demo / show and tell? | 15:50 |
| *** bogdando has quit IRC | 15:52 | |
| kopecmartin | weshay|ruck: it depends on what exactly is the expected outcome of the dome | 15:53 |
| *** jtomasek has quit IRC | 15:58 | |
| *** udesale has quit IRC | 15:59 | |
| *** Trevor_V has joined #oooq | 16:00 | |
| marios | weshay|ruck: o/ | 16:03 |
| weshay|ruck | kopecmartin, meh.. just a reintro to the topic, patches.. nothing new | 16:03 |
| weshay|ruck | because we haven't moved on it yet | 16:03 |
| weshay|ruck | marios, k | 16:03 |
| *** TrevorV has quit IRC | 16:04 | |
| weshay|ruck | marios, rfolco you too if you have time https://meet.google.com/zzt-usnv-brc?authuser=1 | 16:04 |
| rfolco | weshay|ruck, @tc call | 16:04 |
| migi | weshay|ruck: owalsh prepared review https://review.opendev.org/#/c/703649/2 | 16:06 |
| weshay|ruck | migi, thanks | 16:09 |
| weshay|ruck | rfolco, down to | 16:20 |
| weshay|ruck | [zuul@wes workspace]$ cat containers-failed-to-build.log | 16:20 |
| weshay|ruck | qdrouterd | 16:20 |
| weshay|ruck | did you get that to build? | 16:20 |
| *** ykarel|afk is now known as ykarel | 16:20 | |
| rfolco | yes weshay|ruck | 16:21 |
| rfolco | trunk.registry.rdoproject.org/tripleomaster/centos-binary-qdrouterd 65570a311e1303d8084d12ee82868e6d0f411e8a_41250740 5ea8db777fb2 20 hours ago 539 MB | 16:21 |
| jpena | zbr: sorry, I was on a call. About the upload-logs role, it might be possible to iterate through the "{{ zuul.executor.log_root }}/" directory and compress all text files (or something similar). It would also be possible to honor the zuul_log_compress variable, so it works for all logs and not just for the console | 16:21 |
| jpena | however, it looks like an expensive operation to do on the executor | 16:21 |
| zbr | i already started writing that code | 16:22 |
| rfolco | weshay|ruck, If I remember right, that was the one that failed first time because epel didn't exist. It tried to disable it with yum-manager-config --disable epel | 16:22 |
| weshay|ruck | jpena, well.. we were doing something like that.. so now we just need to do it for rdo | 16:22 |
| weshay|ruck | but that will be a second step | 16:22 |
| weshay|ruck | rfolco, oh.. epel on the container itself.. in disabled | 16:22 |
| * weshay|ruck looks at log | 16:22 | |
| *** jtomasek has joined #oooq | 16:25 | |
| rfolco | weshay|ruck, or change this | 16:27 |
| rfolco | docker/qdrouterd/Dockerfile.j2:RUN {% if distro_package_manager == 'dnf' %}dnf config-manager{% else %}yum-config-manager{% endif %} --disable epel centos-release-opstools extras | 16:27 |
| rfolco | which is more appropriate I guess | 16:27 |
| rfolco | let me add this to the kolla patch | 16:28 |
| rfolco | so we can ignore epel hacks | 16:28 |
| weshay|ruck | rfolco++ | 16:28 |
| weshay|ruck | ya.. let's add to the patch so it's repeatable | 16:29 |
| weshay|ruck | thank you | 16:29 |
| *** holser__ has joined #oooq | 16:31 | |
| rfolco | weshay|ruck, what about opstools ? should we assume there will be a opstools repo ? | 16:31 |
| *** holser has quit IRC | 16:31 | |
| weshay|ruck | zbr, how do you want to handle .sh files? https://review.opendev.org/#/c/702862/ | 16:31 |
| weshay|ruck | let's get that fixed.. .then merge this :) | 16:31 |
| weshay|ruck | and this.. https://review.opendev.org/#/c/703586/ when molecule passes | 16:31 |
| *** tosky has quit IRC | 16:33 | |
| zbr | weshay|ruck: how sh files ar handles is not my fault. | 16:34 |
| zbr | i am not doing anything about them. | 16:34 |
| zbr | not sure if is a bug or a feature | 16:34 |
| zbr | interesting is that .bash is loaded as text | 16:35 |
| *** pojadhav|away is now known as pojadhav | 16:36 | |
| sshnaidm | zbr, maybe we can rename their extensions | 16:36 |
| sshnaidm | zbr, like we did with txt.gz | 16:36 |
| sshnaidm | zbr, if .bash is loaded as text, then from script.sh to script.sh.bash | 16:37 |
| zbr | sshnaidm: file bug first, announce upstream. | 16:37 |
| zbr | i don't want more workarounds unless we have no other option | 16:37 |
| sshnaidm | zbr, bug? | 16:37 |
| weshay|ruck | zbr, yes.. I know.. re: it's not your fault | 16:37 |
| weshay|ruck | but let's rename it to something that will render for now | 16:38 |
| sshnaidm | zbr, it's not workarounds, we already do it in log role | 16:38 |
| zbr | if sh is served as binary and .bash is not, is clearly an upload bug. | 16:38 |
| sshnaidm | zbr, lemme show you.. | 16:38 |
| zbr | sshnaidm: i know how to rename files, no need to tell me. | 16:38 |
| zbr | i said, that is a log server bug | 16:39 |
| weshay|ruck | zbr, aye.. | 16:39 |
| weshay|ruck | zbr, and it's also inconsistent | 16:39 |
| weshay|ruck | because they rendered yesterday | 16:39 |
| sshnaidm | zbr, https://github.com/openstack/ansible-role-collect-logs/blob/7277ae921f66874458303061b49625aff2b0fc6c/tasks/publish.yml#L123 | 16:39 |
| weshay|ruck | so it could be only on some cdn's | 16:40 |
| sshnaidm | zbr, I show you where we do it in the role, not how to rename :) | 16:40 |
| weshay|ruck | I'm fine w/ .sh -> .sh.log or .sh.bash | 16:40 |
| weshay|ruck | what ever | 16:40 |
| sshnaidm | zbr, as you see we have a few extensions there, not only sh | 16:41 |
| zbr | one does not write workarounds without mentioning bugs for which these are needed, I will -2 it. | 16:41 |
| weshay|ruck | sshnaidm, ya.. but those other extensions I think are working w/ his patch | 16:41 |
| sshnaidm | weshay|ruck, do all extensions work? | 16:42 |
| zbr | asked upstream, not desired, and fixable. | 16:42 |
| weshay|ruck | sshnaidm, everything except .sh | 16:42 |
| weshay|ruck | afaict | 16:42 |
| sshnaidm | weshay|ruck, that's great | 16:43 |
| weshay|ruck | zbr, ok. so if we're fixing in infra.. let's merge ur patch | 16:44 |
| sshnaidm | zbr, browser considers .sh as application/x-sh format | 16:44 |
| sshnaidm | seems like anything except .sh would work | 16:44 |
| zbr | probably hackable at client side but for convenience, better to fix upstream. | 16:45 |
| weshay|ruck | zbr, sshnaidm thinking I'm going to hit workflow https://review.opendev.org/#/c/702862/10 | 16:46 |
| weshay|ruck | 5. | 16:46 |
| weshay|ruck | 4. | 16:46 |
| weshay|ruck | 3. | 16:46 |
| weshay|ruck | 2. | 16:46 |
| weshay|ruck | objections? | 16:46 |
| *** jpena is now known as jpena|brb | 16:46 | |
| sshnaidm | weshay|ruck, it requires 2 +2s | 16:46 |
| weshay|ruck | sshnaidm, is volunteering to ruck/rove w/o browsable logs in | 16:46 |
| weshay|ruck | 5. | 16:46 |
| weshay|ruck | 4. | 16:46 |
| weshay|ruck | 3. | 16:47 |
| weshay|ruck | 2. | 16:47 |
| weshay|ruck | :) | 16:47 |
| sshnaidm | weshay|ruck, you have 2 +2s | 16:47 |
| weshay|ruck | :) | 16:47 |
| weshay|ruck | zbr++ | 16:47 |
| weshay|ruck | zbr, so next up other than molecule is to figure out compression for rdo | 16:48 |
| sshnaidm | zbr, but still wonder why do you add || true to rm | 16:48 |
| zbr | already did the role locally, but I am going to propose it upstream | 16:48 |
| zbr | sshnaidm: because rm fails when with glob. | 16:48 |
| zbr | alternative was to use failed_when: false | 16:49 |
| *** marios is now known as marios|out | 16:49 | |
| zbr | but better not to delay merging it with another set of 2-3h. | 16:49 |
| zbr | it also has tests, something missing in before | 16:50 |
| sshnaidm | zbr, $ rm -rf /tmp/nonexisting/* ; echo $? | 16:50 |
| sshnaidm | 0 | 16:50 |
| sshnaidm | zbr, what do I do wrong? | 16:50 |
| zbr | you can try to run with quotes, or as non-root, anyway even rm can fail. | 16:52 |
| sshnaidm | Cannot Merge - need to rebase, "Fix set -o pipefail" was just merged | 16:53 |
| sshnaidm | zbr, no, it can't | 16:53 |
| *** dsneddon has joined #oooq | 16:53 | |
| sshnaidm | at least with --force | 16:54 |
| sshnaidm | -f, --force | 16:54 |
| sshnaidm | ignore nonexistent files and arguments, never prompt | 16:54 |
| weshay|ruck | rlandy, fyi.. no action.. rekicking env D w/ a patch | 16:55 |
| rlandy | weshay|ruck: what do mean by no action? | 16:55 |
| rlandy | patch pls | 16:55 |
| weshay|ruck | https://code.engineering.redhat.com/gerrit/#/c/189436/ | 16:56 |
| rlandy | periodic-tripleo-ci-centos-7-bm_envD-1ctlr_2comp-featureset021-master queued | 16:56 |
| *** marios|out has quit IRC | 16:56 | |
| rlandy | patience | 16:56 |
| rlandy | weshay|ruck: wrt osp failures - we have a change in podman versions | 16:57 |
| weshay|ruck | ah | 16:58 |
| *** dsneddon has quit IRC | 16:58 | |
| rlandy | weshay|ruck: hmm we have a node issue | 16:58 |
| rlandy | https://sf.hosted.upshift.rdu2.redhat.com/logs/periodic-hourly/code.engineering.redhat.com/openstack/tripleo-ci-internal-jobs/master/periodic-tripleo-ci-centos-7-bm_envC-1ctlr_1comp-featureset001-train/57a3303/job-output.txt | 16:58 |
| rlandy | which is why your job won't start | 16:58 |
| sshnaidm | zbr, anyways you need to rebase | 16:59 |
| rlandy | 2020-01-21 12:07:11.882705 | TASK [fetch-zuul-cloner-fork : Install zuul-cloner shim dependencies] | 16:59 |
| raukadah | weshay|ruck, rlandy zbr https://review.rdoproject.org/r/#/c/24617/ going to merge it | 16:59 |
| weshay|ruck | rlandy, we probably need to update required projects | 16:59 |
| weshay|ruck | rlandy, to include ci-scripts? | 16:59 |
| weshay|ruck | raukadah, ok.. dokie.. will watch for fall out | 17:00 |
| zbr | sshnaidm: if you rebasse it an not merge https://review.opendev.org/#/c/703597/ it will fail. | 17:00 |
| zbr | molecule will not run without fixing the HOME part. | 17:00 |
| rlandy | yesterday we ran pretty clean | 17:00 |
| rlandy | today something kicked | 17:00 |
| raukadah | rlandy, thank you for checking on downstream stuff | 17:00 |
| rlandy | weshay|ruck: wel your job started | 17:01 |
| sshnaidm | zbr, what is the problem with HOME part? | 17:01 |
| rlandy | raukadah: lol - you're thanking me?? | 17:01 |
| weshay|ruck | rlandy, job failed on 2020-01-21 12:07:11.882705 | TASK [fetch-zuul-cloner-fork : Install zuul-cloner shim dependencies] | 17:01 |
| rlandy | yeah | 17:01 |
| weshay|ruck | but all also failed to report | 17:01 |
| rlandy | that's new | 17:01 |
| raukadah | rlandy, yes, as marios is sleeping so | 17:01 |
| weshay|ruck | rlandy, but ya.. the changes that marios made to upstream | 17:01 |
| weshay|ruck | probably also need to made to internal | 17:01 |
| weshay|ruck | w/ regards to required projects | 17:01 |
| *** tesseract has quit IRC | 17:01 | |
| zbr | pip was fixed, it was broken for ~3h | 17:01 |
| rlandy | it won't get anywhere if it fails that early | 17:01 |
| raukadah | zbr, please update the commit message on ~ home patch and other case when user home is root | 17:02 |
| rlandy | weshay|ruck: ha - your job passed that | 17:02 |
| rlandy | 2020-01-21 16:59:40.356870 | TASK [fetch-zuul-cloner-fork : Install zuul-cloner shim] | 17:03 |
| rlandy | 2020-01-21 16:59:40.893934 | primary | changed | 17:03 |
| rlandy | possible temp change | 17:03 |
| zbr | i am going to include the HOME patch in gz, one to avoid the chain. | 17:03 |
| raukadah | zbr, also please also use the same branch topic for gz related patches so that everyone has eyes on others patches what is going on | 17:04 |
| rlandy | raukadah: how familar are you with latest podman changes? | 17:04 |
| rlandy | podman-1.4.2-6.module+el8.1.0+4830+f49150d7.x86_64 vs podman-1.6.4-2.module+el8.1.1+5363+bf8ff1af.x86_64 | 17:05 |
| raukadah | rlandy, need to take a look based on the issues | 17:05 |
| raukadah | too much diff | 17:05 |
| rlandy | the issue I spoke about at the meeting | 17:05 |
| rlandy | rror initializing image from source docker://192.168.24.1:8787/rh-osbs/rhosp16-openstack-cinder-volume:20200110.1-updated-20200120192136: unsupported docker v2s2 media type: \"\"" | 17:06 |
| raukadah | not seen that one | 17:06 |
| raukadah | rlandy, may be zbr can help | 17:06 |
| raukadah | rlandy, in which job it is coming? | 17:06 |
| sshnaidm | zbr, what the problem are you trying to solve with HOME patch? | 17:07 |
| zbr | sshnaidm: you know what, please download the patch, remove the HOME part from it and run theat molecule scenario. | 17:07 |
| zbr | using "tox -e molecule" | 17:08 |
| raukadah | rlandy, from random google search got this https://github.com/containers/libpod/issues/2013 | 17:08 |
| rlandy | yeah I read that ... opened this issue on Dec 14, 2018 | 17:08 |
| sshnaidm | zbr, no, I don't know, please show me where it fails in job | 17:08 |
| rlandy | version is three days old | 17:08 |
| rlandy | I need to try lock the podman version | 17:09 |
| raukadah | rlandy, which job it is? | 17:09 |
| zbr | sorry but I already wasted too much effort today explaining the issue, evening giving links to tox source code, and also creating repository for you at https://github.com/ssbarnea/please-sagi/blob/master/tox.ini | 17:09 |
| rlandy | tripleo-ci-rhel-8-standalone-rhos-16 | 17:09 |
| raukadah | rlandy, ah | 17:09 |
| rlandy | 2020-01-20 19:31:16 | podman x86_641.6.4-2.module+el8.1.1+5363+bf8ff1af rhosp-rhel-8.1-appstream 12 M | 17:09 |
| rlandy | 2020-01-20 19:31:16 | podman-manpages noarch1.6.4-2.module+el8.1.1+5363+bf8ff1af rhosp-rhel-8.1-appstream 176 k | 17:09 |
| zbr | run tox there and look at what it prints | 17:09 |
| rlandy | ^^ getting podman version | 17:09 |
| sshnaidm | zbr, I'd like to see where it fails in job, I'm not a tox tester | 17:10 |
| weshay|ruck | rlandy, interesting | 17:10 |
| sshnaidm | zbr, if you want to test tox, it's your right, but it's not related to CI and this role | 17:10 |
| raukadah | rlandy, rlandy not encountering in rhel8 podman 1.7 standlone jonb | 17:10 |
| rlandy | diff registry | 17:10 |
| raukadah | weshay|ruck, rlandy https://review.rdoproject.org/r/#/c/24605/ anfd https://review.rdoproject.org/r/#/c/24609/ | 17:10 |
| sshnaidm | zbr, I don't ask for explanations, if you see job failure please provide logs, that's all I'm asking | 17:11 |
| raukadah | may be this misses something in ^^ above review | 17:11 |
| rlandy | raukadah: nah - would have to set the same job up in downstream | 17:11 |
| rlandy | we work with the internal registry | 17:11 |
| raukadah | rlandy, will get the patches up tomorrow | 17:11 |
| raukadah | for downstream job | 17:11 |
| rlandy | raukadah: is there a way to lock the podman version? | 17:12 |
| rlandy | basic test | 17:12 |
| raukadah | rlandy, may be undercloud_rpm_dependencies: >- podman == <version > might work, need to check | 17:14 |
| raukadah | in release file? | 17:14 |
| *** jtomasek has quit IRC | 17:14 | |
| rlandy | hmmm | 17:15 |
| rlandy | let me try that | 17:15 |
| rlandy | idk | 17:16 |
| rlandy | coming from rhosp-rhel-8.1-appstream | 17:16 |
| raukadah | let's see what zuul says | 17:16 |
| *** rfolco is now known as rfolco|brb | 17:16 | |
| rlandy | rpm install | 17:16 |
| raukadah | it is going to be a simple yum install comand i think | 17:16 |
| rlandy | raukadah: no worries - late for you | 17:16 |
| rlandy | will try work it out | 17:16 |
| raukadah | rlandy, https://opendev.org/openstack/tripleo-quickstart-extras/src/branch/master/roles/undercloud-setup/templates/install_packages.sh.j2#L11 | 17:17 |
| raukadah | rlandy, It will work | 17:17 |
| raukadah | unless inject repo part updates that | 17:18 |
| raukadah | i mean from repo setup scripts | 17:18 |
| rlandy | idk | 17:24 |
| rlandy | need to finish ipa install anyways | 17:24 |
| rlandy | will try this later | 17:24 |
| *** chem has quit IRC | 17:24 | |
| *** jpena|brb is now known as jpena | 17:26 | |
| *** chem has joined #oooq | 17:26 | |
| *** jtomasek has joined #oooq | 17:26 | |
| *** amoralej is now known as amoralej|off | 17:46 | |
| *** ykarel is now known as ykarel|away | 17:47 | |
| *** yolanda has quit IRC | 17:52 | |
| *** yolanda has joined #oooq | 17:54 | |
| weshay|ruck | rlandy, https://review.rdoproject.org/r/#/c/24623/ | 17:59 |
| weshay|ruck | sshnaidm, https://review.rdoproject.org/r/#/c/24623/ | 18:00 |
| rlandy | weshay|ruck: fallout from https://review.rdoproject.org/r/#/c/24617/? | 18:01 |
| weshay|ruck | rlandy, aye | 18:03 |
| rlandy | charming | 18:03 |
| weshay|ruck | rfolco|brb, fyi https://review.opendev.org/#/c/700877 https://review.opendev.org/#/c/694799 based on comments in https://review.opendev.org/#/c/703640/2 | 18:05 |
| *** dsneddon has joined #oooq | 18:09 | |
| *** dtantsur is now known as dtantsur|afk | 18:11 | |
| *** jtomasek has quit IRC | 18:13 | |
| *** dsneddon has quit IRC | 18:14 | |
| *** jpena is now known as jpena|off | 18:34 | |
| sshnaidm | weshay|ruck, zbr OVB logs w/o gzip take 512 MB, with gzip 158 MB | 18:38 |
| sshnaidm | it's most space consuming job :/ | 18:38 |
| zbr | sshnaidm: tomorrow I will have a CR to improve upload role to archive big files, i already have the code locally | 18:40 |
| zbr | you only set the min size to compress, and will compress on those, easy to tune. | 18:40 |
| sshnaidm | zbr, that's if problem is with big files, and not with a lot of small ones | 18:41 |
| sshnaidm | var/log is very expensive | 18:42 |
| zbr | truncate? | 18:42 |
| sshnaidm | 301 M and 55 M | 18:42 |
| *** dsneddon has joined #oooq | 18:45 | |
| *** dsneddon has quit IRC | 18:50 | |
| weshay|ruck | sshnaidm, zbr right.. so we need an option for 3rd party to compress | 18:51 |
| weshay|ruck | that is take 2 | 18:51 |
| zbr | weshay|ruck: can we detect from inside role if we are on 3rd party? | 18:52 |
| weshay|ruck | zbr, it's a setting son, and each env will need a different one :) | 18:52 |
| weshay|ruck | zbr, let me show you | 18:52 |
| weshay|ruck | zbr, ensure compression is off here https://opendev.org/openstack/tripleo-ci/src/branch/master/toci-quickstart/config/collect-logs.yml | 18:53 |
| weshay|ruck | zbr, ensure compression is set to true https://opendev.org/openstack/tripleo-ci/src/branch/master/toci-quickstart/config/testenv/multinode-rdocloud.yml https://opendev.org/openstack/tripleo-ci/src/branch/master/toci-quickstart/config/testenv/ovb-rdocloud.yml | 18:54 |
| weshay|ruck | zbr, make sense? | 18:54 |
| zbr | sure, put back the option that I now I ignore (artcl_gzip_only) | 18:54 |
| zbr | now I want you and sagi, to test https://review.opendev.org/#/c/702862/ locally via "tox -e molecule", so you get a better hands-on experience. | 18:55 |
| zbr | for exampl I was forced to include fix for bug missed by arx, https://review.opendev.org/#/c/702862/13/tasks/collect/container.yml | 18:56 |
| zbr | because just running the code uncovers bugs. | 18:56 |
| weshay|ruck | zbr, point me at what I need to +2 :) | 18:56 |
| * weshay|ruck looks | 18:56 | |
| weshay|ruck | zbr, eek.. please update the commit msg when ur ready | 18:57 |
| weshay|ruck | https://review.opendev.org/#/c/702862/ | 18:57 |
| zbr | i am waiting for zuul to reply to remove the skipped jobs and correct the msg | 18:57 |
| weshay|ruck | k | 18:58 |
| weshay|ruck | ah.. tis slow aint it | 18:58 |
| zbr | running molecule on zuul is not really identical with local, surprises do appear from time to time | 18:58 |
| weshay|ruck | just kicked | 18:58 |
| zbr | ~8min locally, could be less than 4 if we sort the pyyaml | 18:58 |
| zbr | in fact i am confident that I can bring this role under 2 mins | 18:59 |
| zbr | we have lots of shell scripts we do run sync, when they could be async. | 19:00 |
| zbr | also we could store info in a yaml file instead of so many very small files. | 19:00 |
| weshay|ruck | [zuul@wes workspace]$ cat containers-successfully-built.log | wc -l | 19:08 |
| weshay|ruck | 114 | 19:08 |
| weshay|ruck | [zuul@wes workspace]$ cat containers-build-errors.log | wc -l | 19:08 |
| weshay|ruck | 0 | 19:08 |
| weshay|ruck | rfolco|brb, ^ | 19:08 |
| zbr | i wonder how to reply to https://bugzilla.redhat.com/show_bug.cgi?id=1790174 | 19:10 |
| openstack | bugzilla.redhat.com bug 1790174 in python-virtualenv "virtualenv activate chokes when run from bash scripts in strict mode due to missing PS1" [Medium,New] - Assigned to python-maint | 19:10 |
| zbr | mainly I asked to backport a bug that was fixed more than an year ago in virtualenv.... | 19:10 |
| *** holser__ has quit IRC | 19:12 | |
| zbr | i wonder what they will say when I mention outdated pip, or tox which is fully missing. | 19:13 |
| weshay|ruck | zbr, so from a ssh connection you source a virt_env and connection drops? | 19:13 |
| zbr | that is a side effect because "source activate" fails | 19:14 |
| weshay|ruck | ah this bug | 19:14 |
| zbr | just because PS1 is not defined | 19:14 |
| zbr | the old undefined variable issue | 19:14 |
| zbr | weshay|ruck: interesting pattern. another issue related to undefined variables ;) | 19:15 |
| *** dsneddon has joined #oooq | 19:15 | |
| weshay|ruck | zbr, ask me when.. I say now!!! | 19:16 |
| weshay|ruck | zbr, :) | 19:16 |
| zbr | more of a "now?".... regex | 19:16 |
| weshay|ruck | zbr, fix your commit msg | 19:16 |
| weshay|ruck | :) | 19:17 |
| weshay|ruck | ur going to forget | 19:17 |
| zbr | nope, still running again locally, found another bug. | 19:17 |
| zbr | without func tests this fix would have taken one print to get right. | 19:18 |
| *** tosky has joined #oooq | 19:25 | |
| zbr | weshay|ruck: i decided to leave sova scenario alone as it was keep causing problems, for example is not safe to run locally. | 19:31 |
| zbr | as it would require root, which I am not going to give. | 19:31 |
| weshay|ruck | zbr, you have to protect the children | 19:33 |
| zbr | https://88ecc26cc7b7b4427f70-0dd3b493df614cc7e41a46754664ea83.ssl.cf2.rackcdn.com/702862/16/check/openstack-tox-molecule/a8589bb/tox/reports.html | 20:06 |
| zbr | that line makes me kinda happy: "msg": "Collected 230 files" | 20:07 |
| zbr | i also activated the timings, maybe this will motivate others to pay more attention on where time is wasted | 20:08 |
| *** jtomasek has joined #oooq | 20:10 | |
| weshay|ruck | zbr++ | 20:12 |
| zbr | i still think that https://review.opendev.org/#/c/703597/ worths being done. | 20:13 |
| zbr | we just avoided it, and I do not see any risks in it. | 20:14 |
| weshay|ruck | zbr, we'd have to change a lot of other defaults to match that.. in the end .. it's a tox thang | 20:14 |
| weshay|ruck | I have spoken | 20:15 |
| weshay|ruck | :) | 20:15 |
| weshay|ruck | zbr, fwiw https://review.opendev.org/#/c/703533/ | 20:15 |
| *** holser has joined #oooq | 20:16 | |
| zbr | weshay|ruck: that is the magic index? | 20:16 |
| weshay|ruck | aye | 20:16 |
| zbr | why not including it in collect logs? | 20:16 |
| weshay|ruck | it is | 20:16 |
| zbr | is see it part of tripleo-ci docs | 20:17 |
| zbr | i mean part of the role | 20:17 |
| weshay|ruck | because it's part of tripleo-ci .. not a collect-logs role :) | 20:17 |
| weshay|ruck | say if openstack-ansible wanted to drop in their own doc | 20:17 |
| weshay|ruck | or infrared wanted to | 20:17 |
| weshay|ruck | and it was originally the footer | 20:17 |
| weshay|ruck | until infra killed that | 20:18 |
| weshay|ruck | I digress | 20:18 |
| zbr | personally I would have being scripted it, not hardcoded like here. | 20:18 |
| zbr | i think that what we do not fix is the missing index on rdo, because upstream we do have it | 20:20 |
| zbr | and we can add stuff to it using zuul_return module | 20:21 |
| zbr | weshay|ruck: fyi, look at https://review.opendev.org/#/c/692356/ -- only the DATE when it originated. | 20:24 |
| weshay|ruck | zbr, I'm familiar | 20:24 |
| weshay|ruck | zbr, as core.. you need to boost reviews ;) | 20:29 |
| weshay|ruck | rlandy, https://review.opendev.org/#/c/703533/ as you have time.. rfolco|brb as well | 20:29 |
| *** yolanda has quit IRC | 20:29 | |
| zbr | as in finding ways to bribe my mates :D | 20:29 |
| weshay|ruck | zbr, pretty much | 20:30 |
| *** yolanda has joined #oooq | 20:30 | |
| rlandy | in abit | 20:31 |
| *** yolanda has quit IRC | 20:46 | |
| *** yolanda has joined #oooq | 20:50 | |
| weshay|ruck | zbr, test report looks nice.. thanks | 20:52 |
| weshay|ruck | rfolco|brb, we need a planning mtg | 21:15 |
| weshay|ruck | before ths | 21:15 |
| weshay|ruck | thrs | 21:15 |
| *** Trevor_V has quit IRC | 21:33 | |
| *** jtomasek has quit IRC | 21:38 | |
| *** jfrancoa has quit IRC | 22:38 | |
| *** holser has quit IRC | 22:54 | |
| *** holser has joined #oooq | 23:36 | |
| *** dsneddon has quit IRC | 23:39 | |
| *** fmount has quit IRC | 23:39 | |
| *** rfolco|brb is now known as rfolco | 23:41 | |
| rfolco | weshay|ruck, me you and raukadah ? | 23:41 |
| rfolco | pre-planning you mean | 23:42 |
| *** tosky has quit IRC | 23:47 | |
| *** holser has quit IRC | 23:58 | |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!