| *** pojadhav|out is now known as pojadhav | 01:42 | |
| opendevreview | Merged openstack/tripleo-common master: Remove now useless sudo call for port check https://review.opendev.org/c/openstack/tripleo-common/+/804838 | 01:45 |
|---|---|---|
| opendevreview | Merged openstack/tripleo-quickstart master: Install tripleo.repos collection https://review.opendev.org/c/openstack/tripleo-quickstart/+/801776 | 01:55 |
| opendevreview | Merged openstack/tripleo-quickstart-extras master: Fix GH PRs build with Ansible install https://review.opendev.org/c/openstack/tripleo-quickstart-extras/+/804801 | 01:55 |
| opendevreview | Merged openstack/tripleo-ansible master: Remove deny list check from cli-enable-ssh-admin.yaml https://review.opendev.org/c/openstack/tripleo-ansible/+/803611 | 01:55 |
| opendevreview | Takashi Kajinami proposed openstack/puppet-tripleo master: haproxy: enable forwardfor for all http endpoints https://review.opendev.org/c/openstack/puppet-tripleo/+/793783 | 02:03 |
| opendevreview | Takashi Kajinami proposed openstack/tripleo-heat-templates master: Log source ips instead of controller ips in apache access log https://review.opendev.org/c/openstack/tripleo-heat-templates/+/793787 | 02:05 |
| opendevreview | Takashi Kajinami proposed openstack/tripleo-heat-templates master: Log source ips instead of controller ips in apache access log https://review.opendev.org/c/openstack/tripleo-heat-templates/+/793787 | 02:06 |
| opendevreview | wes hayutin proposed openstack/python-tripleoclient master: add py39 tox to python-tripleoclient https://review.opendev.org/c/openstack/python-tripleoclient/+/804897 | 02:12 |
| opendevreview | wes hayutin proposed openstack/python-tripleoclient master: add py39 tox to python-tripleoclient https://review.opendev.org/c/openstack/python-tripleoclient/+/804897 | 02:13 |
| weshay|ruck | chandankumar, help push on https://review.opendev.org/c/openstack/python-tripleoclient/+/804897 please :) | 02:13 |
| opendevreview | Steve Baker proposed openstack/tripleo-image-elements master: Add small /srv volume for optional swift data https://review.opendev.org/c/openstack/tripleo-image-elements/+/804945 | 04:23 |
| *** ysandeep|out is now known as ysandeep | 04:30 | |
| opendevreview | Merged openstack/tripleo-quickstart-extras master: Bump Ceph container daemons to v6.0.4 https://review.opendev.org/c/openstack/tripleo-quickstart-extras/+/803493 | 04:55 |
| *** ykarel|away is now known as ykarel | 05:02 | |
| opendevreview | Rabi Mishra proposed openstack/python-tripleoclient stable/train: [train-only] Honour DeploymentServerBlacklist https://review.opendev.org/c/openstack/python-tripleoclient/+/804946 | 05:02 |
| Tengu | chandankumar: hello there! quick question: is there already a known issue about pip unable to install packages in tox-py36 jobs on wallaby? Such as this: https://zuul.opendev.org/t/openstack/build/f822b7467e4e440db4cb6dd773130a5b | 06:15 |
| Tengu | chandankumar: iirc there was a whole lot of things a couple of weeks ago, but I lost track. iirc it was due to a pip change, and in the end the correction was to set the actual locales to something other than C... ? | 06:15 |
| chandankumar | Tengu: checking | 06:16 |
| chandankumar | Tengu: https://bugs.launchpad.net/tripleo/+bug/1940313 | 06:17 |
| Tengu | "yay". | 06:17 |
| Tengu | ah, ramishra already reported that wallaby is broken | 06:17 |
| Tengu | have to find what was done for the other version | 06:22 |
| Tengu | https://review.opendev.org/c/openstack/tripleo-quickstart/+/802431 | 06:23 |
| Tengu | that was the correction apparently. | 06:23 |
| chandankumar | Tengu: I am taking a look at this https://review.opendev.org/c/openstack/python-tripleoclient/+/804887 why depends-on is not working | 06:33 |
| chandankumar | tox environment chnages are there in zuul inventory | 06:34 |
| Tengu | meh... can't say :/ | 06:35 |
| *** iurygregory_ is now known as iurygregory | 06:42 | |
| opendevreview | Cedric Jeanneret proposed openstack/tripleo-operator-ansible master: Add tripleo_overcloud_deploy_skip_nodes_and_networks https://review.opendev.org/c/openstack/tripleo-operator-ansible/+/804954 | 06:47 |
| opendevreview | Merged openstack/tripleo-docs master: Document ``--skip-nodes-and-networks`` https://review.opendev.org/c/openstack/tripleo-docs/+/804893 | 06:56 |
| opendevreview | chandan kumar proposed openstack/tripleo-quickstart master: Add back collections_paths in ansible.cfg https://review.opendev.org/c/openstack/tripleo-quickstart/+/804955 | 07:03 |
| opendevreview | Michele Baldessari proposed openstack/tripleo-heat-templates master: Remove deprecated ipv6 corosync parameters https://review.opendev.org/c/openstack/tripleo-heat-templates/+/739936 | 07:19 |
| *** rpittau|afk is now known as rpittau | 07:22 | |
| *** jpena|off is now known as jpena | 07:34 | |
| opendevreview | Jiri Podivin proposed openstack/validations-libs master: DNM https://review.opendev.org/c/openstack/validations-libs/+/804957 | 07:54 |
| jpodivin_ | chandankumar: I've just tested a patch potentially fixing the encoding issue https://review.opendev.org/c/openstack/validations-libs/+/804957 | 08:09 |
| Tengu | but only for one project.. | 08:09 |
| jpodivin_ | if we could set that var in zuul, we might be able to fix this. | 08:09 |
| *** jpodivin_ is now known as jpodivin | 08:10 | |
| Tengu | jpodivin: that's what https://review.opendev.org/c/openstack/openstack-zuul-jobs/+/804890/2/zuul.d/jobs.yaml does in fact | 08:10 |
| Tengu | there's apparently an issue with some depends-on though | 08:10 |
| Tengu | UNLESS.....we have to set something in the tox.ini in order to "forward" that env var. | 08:10 |
| jpodivin | passenv | 08:11 |
| Tengu | still - if tox.ini are overriding the env var like it was done in the one you modified for your test, we're screwed. | 08:11 |
| jpodivin | but that would defeat the purpose of generalizing in the zuul. | 08:11 |
| jpodivin | Tengu: I wouldn't be that harsh. | 08:11 |
| jpodivin | :) | 08:11 |
| jpodivin | It's just extra work we didn't plan for. | 08:11 |
| Tengu | one might put it like that, yeah...... | 08:12 |
| Tengu | well. the right question: why set it to C ? | 08:12 |
| Tengu | -.- | 08:12 |
| jpodivin | Tengu: I think we were just trying to be too clever, that's all. | 08:13 |
| jpodivin | and it did work for a long time. | 08:13 |
| jpodivin | still, I'm going to try and pass the env in the zuul-jobs, if it works we are clear. | 08:14 |
| jpodivin | if it doesn't, we will have interesting couple of days :D | 08:14 |
| opendevreview | Jiri Podivin proposed openstack/validations-libs master: DNM Depends-On: https://review.opendev.org/c/openstack/openstack-zuul-jobs/+/804962 https://review.opendev.org/c/openstack/validations-libs/+/804963 | 08:18 |
| opendevreview | Jiri Podivin proposed openstack/validations-libs master: DNM https://review.opendev.org/c/openstack/validations-libs/+/804963 | 08:19 |
| jpodivin | there, now we just need to wait and see what happens | 08:19 |
| opendevreview | chandan kumar proposed openstack/python-tripleoclient master: WIP: Enforce UTF8 with python3.6 https://review.opendev.org/c/openstack/python-tripleoclient/+/804887 | 08:21 |
| chandankumar | jpodivin: thank you for looking into that, if that works we can ask infra to merge this, it will unblock our jobs | 08:22 |
| opendevreview | Michele Baldessari proposed openstack/puppet-tripleo master: WIP Enforce firewall rules creation ordering for pcsd https://review.opendev.org/c/openstack/puppet-tripleo/+/706763 | 08:24 |
| *** ysandeep is now known as ysandeep|lunch | 08:26 | |
| jpodivin | hm, nope, same problem | 08:28 |
| opendevreview | Merged openstack/tripleo-ansible master: Allow watch_frr options to be set and support frr-8.0 https://review.opendev.org/c/openstack/tripleo-ansible/+/804111 | 08:33 |
| jpodivin | I wonder, don't changes to zuul jobs require promotion to become active? | 08:33 |
| jpodivin | chandankumar: I mean, I'm not certain, but that would explain why we the Depends-On doesn't help. | 08:34 |
| chandankumar | jpodivin: depends-on is getting picked up there as per zuul inventory | 08:35 |
| chandankumar | env is not getting applied I am not sure | 08:36 |
| jpodivin | chandankumar: yes, but there are cases when you need promotion of the code. I recall one time we were changing jobs for the validations, and the new stuff didn't come up until next promotion. | 08:36 |
| opendevreview | Michele Baldessari proposed openstack/tripleo-heat-templates master: DNM test notificationdriver with multiple backends in CI https://review.opendev.org/c/openstack/tripleo-heat-templates/+/694016 | 08:36 |
| jpodivin | as for the env. It would be good to see what exactly are the environment variables on the machine while it's running the tox. | 08:37 |
| chandankumar | jpodivin: can I edit your patch? Want to set this also LANG en_US.UTF-8 | 08:38 |
| jpodivin | chandankumar: sure, why not. | 08:38 |
| jpodivin | chandankumar: for the record, when changing the vars in tox, the working combination was LANG=en_US.UTF-8 | 08:39 |
| jpodivin | LANGUAGE=en_US:en | 08:39 |
| jpodivin | LC_ALL=en_US.UTF-8 | 08:39 |
| jpodivin | Like so https://review.opendev.org/c/openstack/validations-libs/+/804957/1/tox.ini | 08:40 |
| opendevreview | Cedric Jeanneret proposed openstack/tripleo-common stable/wallaby: Remove now useless sudo call for port check https://review.opendev.org/c/openstack/tripleo-common/+/804863 | 08:53 |
| opendevreview | Cedric Jeanneret proposed openstack/tripleo-common stable/victoria: Remove now useless sudo call for port check https://review.opendev.org/c/openstack/tripleo-common/+/804864 | 08:53 |
| opendevreview | Cedric Jeanneret proposed openstack/tripleo-common stable/ussuri: Remove now useless sudo call for port check https://review.opendev.org/c/openstack/tripleo-common/+/804865 | 08:53 |
| opendevreview | Cedric Jeanneret proposed openstack/tripleo-common stable/train: Remove now useless sudo call for port check https://review.opendev.org/c/openstack/tripleo-common/+/804866 | 08:54 |
| opendevreview | Michele Baldessari proposed openstack/tripleo-ansible master: Introduce throttle mechanism for tripleo_container_manage https://review.opendev.org/c/openstack/tripleo-ansible/+/787723 | 08:59 |
| opendevreview | Michele Baldessari proposed openstack/tripleo-heat-templates master: Use throttle mechanism for frr container start https://review.opendev.org/c/openstack/tripleo-heat-templates/+/787724 | 09:01 |
| opendevreview | Pooja Jadhav proposed openstack/tripleo-quickstart-extras master: Fix multinode-ipa job from building-test-packages twice https://review.opendev.org/c/openstack/tripleo-quickstart-extras/+/804971 | 09:04 |
| *** ykarel is now known as ykarel|lunch | 09:04 | |
| opendevreview | Juan Badia Payno proposed openstack/tripleo-ansible master: BnR: managed ceph properly to take a backup of the node https://review.opendev.org/c/openstack/tripleo-ansible/+/802688 | 09:07 |
| *** chem_ is now known as chem | 09:12 | |
| *** chem is now known as Guest4716 | 09:13 | |
| jpodivin | hm, I'm starting to think we are missing something in our understanding of how zuul manages env vars. | 09:21 |
| opendevreview | Sorin Sbârnea proposed openstack/python-tripleoclient master: WIP: Enforce UTF8 with python3.6 https://review.opendev.org/c/openstack/python-tripleoclient/+/804887 | 09:22 |
| opendevreview | chandan kumar proposed openstack/python-tripleoclient master: WIP: Enforce UTF8 with python3.6 https://review.opendev.org/c/openstack/python-tripleoclient/+/804887 | 09:26 |
| opendevreview | Pooja Jadhav proposed openstack/tripleo-quickstart-extras master: Fix multinode-ipa job from building-test-packages twice https://review.opendev.org/c/openstack/tripleo-quickstart-extras/+/804971 | 09:28 |
| opendevreview | Shnaidman Sagi (Sergey) proposed openstack/tripleo-ansible stable/wallaby: Convert all OS module imports to collections imports https://review.opendev.org/c/openstack/tripleo-ansible/+/804869 | 09:36 |
| opendevreview | Michele Baldessari proposed openstack/puppet-tripleo master: Enforce firewall rules creation ordering for pcsd https://review.opendev.org/c/openstack/puppet-tripleo/+/706763 | 09:38 |
| *** ysandeep|lunch is now known as ysandeep | 09:47 | |
| chandankumar | jpodivin: Tengu https://review.opendev.org/c/openstack/openstack-zuul-jobs/+/804890 will fix it | 10:10 |
| chandankumar | jpodivin: green run here: https://review.opendev.org/c/openstack/python-tripleoclient/+/804887 | 10:10 |
| chandankumar | zbr++ | 10:11 |
| jpodivin | so it was the LC setting . | 10:11 |
| chandankumar | yes | 10:11 |
| jpodivin | odd it didn't work here https://review.opendev.org/c/openstack/openstack-zuul-jobs/+/804962/ | 10:12 |
| chandankumar | LC_ALL: en_US.UTF-8 was only needed | 10:13 |
| jpodivin | what's confusing, is that inclusion of the other variables shouldn't cause it to fail, but it did. | 10:14 |
| chandankumar | https://github.com/pypa/pip/issues/10219 has more info | 10:14 |
| jpodivin | See. https://review.opendev.org/c/openstack/validations-libs/+/804963 This one sets both lang and LC_ALL | 10:15 |
| jpodivin | but it still fails. | 10:15 |
| jpodivin | The question is why. one shouldn't affect the other. And even if it does, you have set them to the same value so. | 10:16 |
| opendevreview | Shnaidman Sagi (Sergey) proposed openstack/tripleo-ansible stable/wallaby: Convert all OS module imports to collections imports https://review.opendev.org/c/openstack/tripleo-ansible/+/804869 | 10:19 |
| opendevreview | Jiri Podivin proposed openstack/validations-libs master: DNM https://review.opendev.org/c/openstack/validations-libs/+/804963 | 10:20 |
| chandankumar | jpodivin: so lang will set the default local and LC_ALL will override all the local setting to en_US.UTF-8 irrespective of the local so that;s why it is working | 10:21 |
| chandankumar | that is my guess | 10:21 |
| jpodivin | chandankumar: Yes, but LC_ALL was set in your patch as well. Yet it didn't work. | 10:22 |
| opendevreview | Merged openstack/python-tripleoclient stable/train: TRAIN-ONLY Fix overcloud support report for T https://review.opendev.org/c/openstack/python-tripleoclient/+/804182 | 10:22 |
| chandankumar | both combination Lang and LC_ALL is not working | 10:23 |
| jpodivin | Basically, the patches differ in one line. And that line (setting LANG) shouldn't impact things, because as you said the LC_ALL overrides the local settings. It's just very strange | 10:23 |
| jpodivin | chandankumar: isn't it? but that's what zbr patch does and it passes. | 10:23 |
| jpodivin | nah, that's the other way around. | 10:24 |
| jpodivin | but the first PS I've I tried is actually just using the LC_ALL, https://review.opendev.org/c/openstack/openstack-zuul-jobs/+/804962/1/zuul.d/jobs.yaml and yet it didn't work. | 10:25 |
| jpodivin | The only different thing is order, but vars is a dict so that shouldn't matter ... I don't like this, not knowing why stuff works. | 10:27 |
| opendevreview | Juan Badia Payno proposed openstack/tripleo-ansible master: BnR: managed ceph properly to take a backup of the node https://review.opendev.org/c/openstack/tripleo-ansible/+/802688 | 10:32 |
| chandankumar | jpodivin: yes, that is weired, in first patchset it does not worked | 10:32 |
| chandankumar | https://storage.bhs.cloud.ovh.net/v1/AUTH_dcaab5e32b234d56b626f72581e3644c/zuul_opendev_logs_b2a/804887/3/check/openstack-tox-py36/b2ae78a/zuul-info/inventory.yaml | 10:32 |
| opendevreview | Shnaidman Sagi (Sergey) proposed openstack/tripleo-ansible stable/wallaby: Adjust molecule jobs for Ansible 2.11 https://review.opendev.org/c/openstack/tripleo-ansible/+/804990 | 10:35 |
| opendevreview | Christian Schwede proposed openstack/puppet-tripleo master: Add new parameter enforce_new_defaults for Swift https://review.opendev.org/c/openstack/puppet-tripleo/+/804983 | 10:35 |
| opendevreview | Christian Schwede proposed openstack/tripleo-heat-templates master: Add new parameter SwiftEnforceSecureRbac for Swift https://review.opendev.org/c/openstack/tripleo-heat-templates/+/804984 | 10:36 |
| *** odyssey4me is now known as Guest4722 | 10:37 | |
| opendevreview | Shnaidman Sagi (Sergey) proposed openstack/tripleo-ansible stable/wallaby: Adjust molecule jobs for Ansible 2.11 https://review.opendev.org/c/openstack/tripleo-ansible/+/804990 | 10:37 |
| zbr | jpodivin: chandankumar yes, LANG does not have any impact. the one we need is LC_ALL. | 10:40 |
| zbr | still, i do not think that hardcoding LC_ALL inside tox is the best approach, but given time constraints.... | 10:41 |
| jpodivin | zbr: as far as I know we don't have viable alternative, safe giving up on the 36 entirely | 10:42 |
| chandankumar | zbr: only question is https://review.opendev.org/c/openstack/openstack-zuul-jobs/+/804962/1/zuul.d/jobs.yaml#188 tested against https://review.opendev.org/c/openstack/python-tripleoclient/+/804887/3/ does not work | 10:42 |
| jpodivin | ^ yep, I'm kind of wondering about that https://zuul.opendev.org/t/openstack/status#804963 | 10:44 |
| zbr | jpodivin: look at lastest patch, v4 that one works. | 10:44 |
| opendevreview | Shnaidman Sagi (Sergey) proposed openstack/tripleo-heat-templates stable/wallaby: Change tests to run on ansible-core 2.11 https://review.opendev.org/c/openstack/tripleo-heat-templates/+/804991 | 10:44 |
| jpodivin | zbr: https://review.opendev.org/c/openstack/openstack-zuul-jobs/+/804890/ ? | 10:44 |
| zbr | now the patch does also clean the tox.ini file, important bit is that previous hack was wrong. | 10:45 |
| jpodivin | Which patch do you mean? https://review.opendev.org/c/openstack/openstack-zuul-jobs/+/804890/ has just 3 PS | 10:45 |
| zbr | https://review.opendev.org/c/openstack/python-tripleoclient/+/804887 | 10:45 |
| zbr | that one has a depends on on the modification of openstack-tox-py36 job | 10:46 |
| chandankumar | oh I missed the tox.ini changes | 10:46 |
| jpodivin | ah, so it comes down to modifying the tox.ini. :( | 10:47 |
| zbr | as you can see some repos may also need tox.ini patched if they missed to pass LC_ALL | 10:47 |
| chandankumar | https://review.opendev.org/c/openstack/python-tripleoclient/+/804887/5/tox.ini | 10:47 |
| jpodivin | I suppose we could just set the LC_ALL in tox ini then? | 10:47 |
| zbr | depends on the project | 10:47 |
| jpodivin | https://review.opendev.org/c/openstack/validations-libs/+/804957/1/tox.ini | 10:47 |
| zbr | yes we can but it would not be ideal, we may miss to spot some platform specific bug related to it | 10:47 |
| jpodivin | But if we merge the two patches proposed ... | 10:47 |
| zbr | One of the reasons why I proposed https://github.com/tox-dev/tox/pull/2162 | 10:48 |
| jpodivin | we will not be able to test those encodings anyway. | 10:48 |
| jpodivin | zbr: I know, but they didn't seem eager to look at it. | 10:48 |
| jpodivin | which is slightly ... dissappointing. | 10:48 |
| zbr | it is less than an hour old | 10:49 |
| jpodivin | zbr: I'm not saying they won't do it at all, just that they didn't exactly seem eager to do it, in the issue report discussion I saw. | 10:49 |
| zbr | "peer pressure" works for things like this, it would worth adding a comment. | 10:50 |
| jpodivin | Not that it matters when they get to it. The important thing is that we can't really leave the gate blocked. | 10:50 |
| jpodivin | zbr: will do. | 10:50 |
| opendevreview | Shnaidman Sagi (Sergey) proposed openstack/python-tripleoclient stable/wallaby: Use 'all' for ANSIBLE_VARS_PLUGIN_STAGE https://review.opendev.org/c/openstack/python-tripleoclient/+/804992 | 10:50 |
| jpodivin | but I think we should hard set the LC_ALL in our projects in the meantime | 10:50 |
| jpodivin | with a todo, explaining the issue and linking | 10:51 |
| jpodivin | It's one patch, per project/branch so it's going to be a bit painful. | 10:52 |
| jpodivin | but better than the clogged pipelines | 10:52 |
| zbr | jpodivin: yes, but be aware that the number of places is huge. that is why is good to first start the process for having it fixed at source asap, followed by applying workaround locally | 10:52 |
| zbr | sometimes the upstream issues is addressed before we manage to implement hack in all places | 10:53 |
| jpodivin | zbr: yeah, sometimes, anyway you might want to post the info on trello. | 10:53 |
| jpodivin | zbr: last time I checked, the solution proposed on the CIX was to just turn off voting for the 36 | 10:53 |
| opendevreview | Shnaidman Sagi (Sergey) proposed openstack/tripleo-heat-templates stable/wallaby: Revert change to use community pacemaker_cluster module https://review.opendev.org/c/openstack/tripleo-heat-templates/+/804993 | 10:53 |
| zbr | or maybe we enjoy the Whac-A-Mole game with LC_ALL | 10:53 |
| jpodivin | well, unless your patch get's merged in tox we might have no option. | 10:55 |
| zbr | Lets do this, which is is better imho: setenv: `LC_ALL={env:LC_ALL:en_US.UTF-8}` | 10:57 |
| zbr | it sets it unless is already set by system | 10:57 |
| *** ykarel|lunch is now known as ykarel | 10:58 | |
| jpodivin | zbr: you mean like the https://review.opendev.org/c/openstack/validations-libs/+/804957 ? | 10:58 |
| zbr | works like a passenv with fallback default for when it is not set | 10:58 |
| jpodivin | but with the deref to the set var ... got it. | 10:58 |
| zbr | that one disregards system setting | 10:58 |
| zbr | setenv instead of passenv | 10:58 |
| jpodivin | zbr: hm, isn't that what we want? Set the locale to something the python can always work with? | 10:59 |
| zbr | and what if that allows you to write some code that will make python choke on systems where locale is not set? | 11:00 |
| zbr | you will fail to find that bug with your tox jobs | 11:01 |
| zbr | it is bit nuanced between making it pass now while introducing some risks for the future | 11:01 |
| jpodivin | that's a good question, but I think ascii is the smallest viable subset of unicode, and it's actually mapping very well in all locales | 11:01 |
| zbr | when we alter encoding we also alter it for python itself, directly affecting the testing | 11:01 |
| jpodivin | as UTF-8 | 11:01 |
| jpodivin | zbr: yes, that being said. The basic charset is mapped to the same codes, as far as I understand it, UTF-8 while still providing local variants, has the ASCII chars in the same place everywhere doesn't it ? | 11:03 |
| zbr | yeah | 11:04 |
| zbr | the only risk added is that you may do "print(":emoji:") in your code, pass testing of that but when you run in production on py36/centos-7 you may discover an exception instead of of an emoji. | 11:06 |
| opendevreview | chandan kumar proposed openstack/puppet-tripleo master: Revert "Set memcached server list from memcached_node_names" https://review.opendev.org/c/openstack/puppet-tripleo/+/804343 | 11:07 |
| Tengu | hjensas: heya! just to confirm: passing the --skip-nodes-and-networks to the deploy is working on my lab! I've pushed a change on tripleo-operator-ansible in order to expose that new parameter, for now there isn't any way to pass it :/ | 11:07 |
| jpodivin | zbr: Is that scenario likely? But besides that, are we even performing systematic testing for different locales? | 11:11 |
| jpodivin | I don't think we do. | 11:11 |
| hjensas | Tengu: yes, I +2 the operator patch. I aggree with your comment on review that we don't need to change the code, the doc and adding the support in operator should be enough. | 11:11 |
| opendevreview | Sorin Sbârnea proposed openstack/python-tripleoclient master: Workaround py36 encoding pip bug https://review.opendev.org/c/openstack/python-tripleoclient/+/804887 | 11:12 |
| *** dviroel|ruck|out is now known as dviroel|ruck | 11:12 | |
| opendevreview | Cedric Jeanneret proposed openstack/tripleo-specs master: Healthchecks: a new hope https://review.opendev.org/c/openstack/tripleo-specs/+/803152 | 11:13 |
| Tengu | hjensas: yup. since it's working like that, we're good. Thanks again for that pointer! | 11:13 |
| Tengu | fultonj: heya! since you're using tripleo-lab: it's for now "broken" for master deploy, it needs https://review.opendev.org/c/openstack/tripleo-operator-ansible/+/804954 (and an subsequent update in tripleo-lab itself) | 11:14 |
| opendevreview | Sorin Sbârnea proposed openstack/python-tripleoclient master: Workaround py36 encoding pip bug https://review.opendev.org/c/openstack/python-tripleoclient/+/804887 | 11:15 |
| opendevreview | Michele Baldessari proposed openstack/puppet-tripleo master: Add /dev/log to ovn-dbs-bundle https://review.opendev.org/c/openstack/puppet-tripleo/+/730878 | 11:16 |
| *** beagles is now known as eagles | 11:28 | |
| *** jpena is now known as jpena|lunch | 11:34 | |
| *** rlandy is now known as rlandyrover | 11:36 | |
| *** rlandyrover is now known as rlandy|rover | 11:36 | |
| rlandy|rover | bogdando: jpodivin: hi ... re: https://review.opendev.org/c/openstack/python-tripleoclient/+/804897 | 11:50 |
| rlandy|rover | we have a gate blocker here | 11:50 |
| bogdando | ok, let's merge then | 11:50 |
| rlandy|rover | do we want to merge this patch? | 11:50 |
| rlandy|rover | ok | 11:50 |
| rlandy|rover | please w+ then | 11:50 |
| rlandy|rover | it takes us time to get patches through zuul-jobs | 11:50 |
| jpodivin | rlandy|rover: yeah, I know, it's just that simply setting the env var would be enough | 11:51 |
| jpodivin | this is ... well, throwing out the child with batwater. | 11:51 |
| jpodivin | *bathwater | 11:51 |
| jpodivin | That's why I personally would prefer the https://review.opendev.org/c/openstack/python-tripleoclient/+/804887 | 11:52 |
| mwhahaha | oh i didn't realize the py39 dropped 3.6 | 11:53 |
| mwhahaha | we don't want that | 11:53 |
| mwhahaha | we should take the other patch | 11:53 |
| mwhahaha | https://review.opendev.org/c/openstack/python-tripleoclient/+/804887 should be the fix | 11:53 |
| jpodivin | mwhahaha: exactly | 11:54 |
| bogdando | okay | 11:54 |
| rlandy|rover | ok - thanks all | 11:55 |
| opendevreview | Juan Badia Payno proposed openstack/tripleo-ansible master: BnR: managed ceph properly to take a backup of the node https://review.opendev.org/c/openstack/tripleo-ansible/+/802688 | 11:55 |
| cloudnull | mornings | 12:01 |
| opendevreview | Harald Jensås proposed openstack/tripleo-operator-ansible master: Add '--vip-file' to overcloud deploy https://review.opendev.org/c/openstack/tripleo-operator-ansible/+/805011 | 12:02 |
| opendevreview | alexey-mr proposed openstack/os-net-config master: allow vhost0 to pass list of members https://review.opendev.org/c/openstack/os-net-config/+/804994 | 12:05 |
| weshay|ruck | mwhahaha, we didn't drop py36 https://review.opendev.org/c/openstack/python-tripleoclient/+/804897/10/zuul.d/layout.yaml | 12:05 |
| weshay|ruck | it's just non-voting | 12:05 |
| jpodivin | weshay|ruck: effectively though, it would no longer be a deciding factor for the pipeline correct? If we can avoid that. | 12:07 |
| weshay|ruck | ? | 12:07 |
| jpodivin | weshay|ruck: with zbr patch we can have both. | 12:07 |
| jpodivin | weshay|ruck: Functional, voting py36 job | 12:08 |
| weshay|ruck | his patch didn't fix it yesterday.. is it working now? | 12:08 |
| weshay|ruck | https://review.opendev.org/c/openstack/python-tripleoclient/+/804887 | 12:08 |
| weshay|ruck | yup.. cool | 12:08 |
| jpodivin | weshay|ruck: oh yes, turns out it's a different var. | 12:08 |
| jpodivin | I've been testing it this morning. | 12:09 |
| jpodivin | well European morning :) | 12:09 |
| jpodivin | It does require change to the tox.ini, but that was on the table anyway. | 12:09 |
| weshay|ruck | zbr++ | 12:09 |
| opendevreview | Harald Jensås proposed openstack/python-tripleoclient stable/victoria: Fix network environment validation https://review.opendev.org/c/openstack/python-tripleoclient/+/804995 | 12:15 |
| opendevreview | Harald Jensås proposed openstack/python-tripleoclient stable/ussuri: Fix network environment validation https://review.opendev.org/c/openstack/python-tripleoclient/+/804996 | 12:15 |
| opendevreview | Harald Jensås proposed openstack/python-tripleoclient stable/train: Fix network environment validation https://review.opendev.org/c/openstack/python-tripleoclient/+/804997 | 12:15 |
| mwhahaha | weshay|ruck: the other patch is a better solution | 12:16 |
| weshay|ruck | aye.. agree | 12:17 |
| weshay|ruck | wasn't working yesterday.. but zbr got it | 12:18 |
| opendevreview | Shnaidman Sagi (Sergey) proposed openstack/tripleo-heat-templates stable/wallaby: Change tests to run on ansible-core 2.11 https://review.opendev.org/c/openstack/tripleo-heat-templates/+/804991 | 12:18 |
| opendevreview | Shnaidman Sagi (Sergey) proposed openstack/python-tripleoclient stable/wallaby: Workaround py36 encoding pip bug https://review.opendev.org/c/openstack/python-tripleoclient/+/804998 | 12:22 |
| opendevreview | Shnaidman Sagi (Sergey) proposed openstack/python-tripleoclient stable/wallaby: Workaround py36 encoding pip bug https://review.opendev.org/c/openstack/python-tripleoclient/+/804998 | 12:23 |
| *** sshnaidm|pto is now known as sshnaidm | 12:25 | |
| sshnaidm | weshay|ruck, mwhahaha ported to wallaby ^ | 12:26 |
| rlandy|rover | fultonj: https://bugs.launchpad.net/tripleo/+bug/1940329 - the CIX is moved to done - can we close this LP bug out? | 12:26 |
| *** jpena|lunch is now known as jpena | 12:32 | |
| weshay|ruck | rock | 12:36 |
| opendevreview | Jiri Podivin proposed openstack/validations-libs master: Setting language for the python3.6 tox environment https://review.opendev.org/c/openstack/validations-libs/+/805013 | 12:50 |
| opendevreview | Jiri Podivin proposed openstack/validations-common master: Setting language for the python3.6 tox environment https://review.opendev.org/c/openstack/validations-common/+/805014 | 12:53 |
| opendevreview | Jiri Podivin proposed openstack/tripleo-validations master: Setting language for the python3.6 tox environment https://review.opendev.org/c/openstack/tripleo-validations/+/805015 | 12:55 |
| opendevreview | Jiri Podivin proposed openstack/validations-libs master: DNM https://review.opendev.org/c/openstack/validations-libs/+/804963 | 12:56 |
| weshay|ruck | sshnaidm++ | 13:08 |
| weshay|ruck | fmount++ fultonj++ | 13:17 |
| fultonj | rlandy|rover: sounds good to me. thank you. | 13:18 |
| opendevreview | Shnaidman Sagi (Sergey) proposed openstack/tripleo-ansible stable/wallaby: Convert all OS module imports to collections imports https://review.opendev.org/c/openstack/tripleo-ansible/+/804869 | 13:26 |
| fultonj | Tengu: thanks for letting me know about https://review.opendev.org/c/openstack/tripleo-operator-ansible/+/804954 | 13:29 |
| opendevreview | Merged openstack/tripleo-heat-templates master: Change tests to run on ansible-core 2.11 https://review.opendev.org/c/openstack/tripleo-heat-templates/+/803625 | 13:35 |
| opendevreview | Merged openstack/tripleo-operator-ansible master: Add tripleo_overcloud_deploy_skip_nodes_and_networks https://review.opendev.org/c/openstack/tripleo-operator-ansible/+/804954 | 13:35 |
| Tengu | fultonj: np - tripleo-lab has an update, but as written in the commit, it has to get the new param merged. It's in gate afaik. | 13:36 |
| dviroel|ruck | fmount: hi, around? | 13:47 |
| Tengu | ah, it actually just merged :D | 13:51 |
| Tengu | perfect | 13:51 |
| fmount | dviroel|ruck: o/ | 13:54 |
| dviroel|ruck | fmount: hey, need some help with this bug https://bugs.launchpad.net/tripleo/+bug/1940434 | 13:55 |
| dviroel|ruck | fmount: something related to 'ceph_lv_wal' | 13:56 |
| dviroel|ruck | according with https://logserver.rdoproject.org/openstack-periodic-integration-stable1/opendev.org/openstack/tripleo-ci/master/periodic-tripleo-ci-centos-8-scenario010-standalone-wallaby/b65afff/logs/undercloud/var/log/ceph/ceph-volume.log.txt.gz | 13:56 |
| fmount | dviroel|ruck: /me looking | 13:56 |
| fmount | dviroel|ruck: let me check the logs and I'll be back in a bit | 13:59 |
| opendevreview | Giulio Fidente proposed openstack/tripleo-heat-templates master: Consider GlanceMultistoreConfig when setting glance_pool https://review.opendev.org/c/openstack/tripleo-heat-templates/+/805029 | 14:00 |
| mwhahaha | bandini: if you hit problems with frr stuff and really really need it we can https://review.opendev.org/c/openstack/tripleo-ansible/+/787723 but i'd really like it to not be the default | 14:01 |
| bandini | ack, we can rehash it once we hit issues | 14:03 |
| bandini | s/once/if/ | 14:06 |
| opendevreview | John Fulton proposed openstack/python-tripleoclient master: Introduce "openstack overcloud ceph deploy" https://review.opendev.org/c/openstack/python-tripleoclient/+/800341 | 14:14 |
| mwhahaha | fultonj: gfidente: puppet-ceph is still a requirement on tripleoclient, we dropped all usages of that right? | 14:25 |
| mwhahaha | fultonj: gfidente: er i mean puppet-tripleo | 14:25 |
| opendevreview | Takashi Kajinami proposed openstack/tripleo-heat-templates master: Manila: Drop tenant_id templating from v2 endpoint https://review.opendev.org/c/openstack/tripleo-heat-templates/+/802998 | 14:25 |
| gfidente | fmount ^^ yes there shouldn't be any dep on it at this point | 14:25 |
| opendevreview | Takashi Kajinami proposed openstack/tripleo-heat-templates master: DNM: Testing the parent change https://review.opendev.org/c/openstack/tripleo-heat-templates/+/803033 | 14:26 |
| opendevreview | Francesco Pantano proposed openstack/tripleo-heat-templates stable/wallaby: Move the remaining wallaby jobs to cephadm https://review.opendev.org/c/openstack/tripleo-heat-templates/+/805034 | 14:27 |
| fmount | gfidente: ack thanks | 14:29 |
| mwhahaha | actually i think it was already cleaned up, i just didn't have it local | 14:35 |
| mwhahaha | thanks | 14:35 |
| opendevreview | Merged openstack/tripleo-ansible master: tripleo_push_artifacts: ensure destination exists https://review.opendev.org/c/openstack/tripleo-ansible/+/804883 | 14:37 |
| *** ykarel is now known as ykarel|away | 14:42 | |
| weshay|ruck | fmount++ | 14:44 |
| opendevreview | Takashi Kajinami proposed openstack/tripleo-heat-templates master: Make default of NovaSyncPowerStateInterval consistent with nova https://review.opendev.org/c/openstack/tripleo-heat-templates/+/776825 | 14:44 |
| opendevreview | Francesco Pantano proposed openstack/tripleo-heat-templates master: Enable CephDashboard for scenario001 https://review.opendev.org/c/openstack/tripleo-heat-templates/+/804896 | 15:09 |
| opendevreview | Merged openstack/puppet-tripleo stable/wallaby: Replace deprecated is_integer https://review.opendev.org/c/openstack/puppet-tripleo/+/804729 | 15:24 |
| opendevreview | Merged openstack/python-tripleoclient master: Workaround py36 encoding pip bug https://review.opendev.org/c/openstack/python-tripleoclient/+/804887 | 15:24 |
| opendevreview | Merged openstack/python-tripleoclient stable/wallaby: Workaround py36 encoding pip bug https://review.opendev.org/c/openstack/python-tripleoclient/+/804998 | 15:24 |
| *** ysandeep is now known as ysandeep|away | 15:38 | |
| *** jpena is now known as jpena|off | 15:42 | |
| *** rpittau is now known as rpittau|afk | 16:06 | |
| opendevreview | wes hayutin proposed openstack/tripleo-ci master: add tripleo-common checkout to content-provider https://review.opendev.org/c/openstack/tripleo-ci/+/805061 | 16:54 |
| opendevreview | Bogdan Dobrelya proposed openstack/puppet-tripleo master: Add /dev/log to ovn-dbs-bundle https://review.opendev.org/c/openstack/puppet-tripleo/+/730878 | 16:55 |
| opendevreview | Merged openstack/tripleo-heat-templates master: cinder_api needs etcd certs inside the container https://review.opendev.org/c/openstack/tripleo-heat-templates/+/804788 | 18:13 |
| opendevreview | Francesco Pantano proposed openstack/tripleo-heat-templates master: Enable CephDashboard for scenario001 https://review.opendev.org/c/openstack/tripleo-heat-templates/+/804896 | 18:20 |
| *** fmount is now known as fmount|afk | 18:26 | |
| opendevreview | Douglas Viroel proposed openstack/tripleo-common stable/wallaby: Bump Ceph container daemons to v6.0.4 https://review.opendev.org/c/openstack/tripleo-common/+/805007 | 18:42 |
| opendevreview | Merged openstack/tripleo-heat-templates stable/wallaby: Revert change to use community pacemaker_cluster module https://review.opendev.org/c/openstack/tripleo-heat-templates/+/804993 | 18:56 |
| opendevreview | Merged openstack/tripleo-common stable/wallaby: Remove now useless sudo call for port check https://review.opendev.org/c/openstack/tripleo-common/+/804863 | 18:56 |
| *** dviroel|ruck is now known as dviroel|ruck|out | 19:07 | |
| opendevreview | James Parker proposed openstack/tripleo-upgrade master: Add different migration actions based on guest https://review.opendev.org/c/openstack/tripleo-upgrade/+/798373 | 20:26 |
| opendevreview | James Parker proposed openstack/tripleo-upgrade master: Add different migration actions based on guest https://review.opendev.org/c/openstack/tripleo-upgrade/+/798373 | 20:29 |
| opendevreview | Steve Baker proposed openstack/tripleo-docs master: Document overriding growvols arguments https://review.opendev.org/c/openstack/tripleo-docs/+/805081 | 20:30 |
| opendevreview | Steve Baker proposed openstack/tripleo-docs master: Remove mention of deprecated IronicIPXEEnabled https://review.opendev.org/c/openstack/tripleo-docs/+/805082 | 20:30 |
| opendevreview | James Parker proposed openstack/tripleo-upgrade master: Add different migration actions based on guest https://review.opendev.org/c/openstack/tripleo-upgrade/+/798373 | 20:30 |
| opendevreview | John Fulton proposed openstack/tripleo-docs master: Document "deployed ceph" https://review.opendev.org/c/openstack/tripleo-docs/+/804245 | 22:11 |
| bshephar | fultonj: Are we removing ceph-ansible as part of that ^ Or we plan to support both methods in Wallaby | 23:17 |
Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!