| *** sshnaidm is now known as sshnaidm|afk | 04:39 | |
| jrosser | morning | 08:04 |
|---|---|---|
| opendevreview | Jonathan Rosser proposed openstack/openstack-ansible-tests master: Fix repo passing CI https://review.opendev.org/c/openstack/openstack-ansible-tests/+/798638 | 08:06 |
| noonedeadpunk | o/ | 08:23 |
| jrosser | ^ i still wonder if this patch will fail on permissions | 08:24 |
| noonedeadpunk | last recheck it even didn't apply all depends-on patches... | 08:27 |
| noonedeadpunk | oh, pki fully passed for rabbit | 08:29 |
| jrosser | buster is broken https://0ba57ff75479255f8769-2b6c314b914fb3f996b5c8a41dca9d5c.ssl.cf2.rackcdn.com/798638/7/check/openstack-ansible-functional-debian-buster/70f51d0/job-output.txt | 08:29 |
| noonedeadpunk | I wonder if we want to set no_log for `Install Server certificates to targets` as that looks not right? http://paste.openstack.org/show/807062/ | 08:30 |
| noonedeadpunk | well, `physical_host` is smth that we always have in dynamic_inventory | 08:32 |
| noonedeadpunk | and smth we haven't used for functional... | 08:32 |
| noonedeadpunk | we can hardcode it to localhost for example | 08:33 |
| noonedeadpunk | somewhere here https://opendev.org/openstack/openstack-ansible-tests/src/branch/master/test-vars.yml | 08:33 |
| opendevreview | Dmitriy Rabotyagov proposed openstack/openstack-ansible-ceph_client stable/wallaby: Update .gitreview for stable/wallaby https://review.opendev.org/c/openstack/openstack-ansible-ceph_client/+/798463 | 08:35 |
| jrosser | they key certainly shouldnt be shown | 08:37 |
| noonedeadpunk | it's hashed, but still... | 08:38 |
| opendevreview | Dmitriy Rabotyagov proposed openstack/openstack-ansible-tests master: Fix repo passing CI https://review.opendev.org/c/openstack/openstack-ansible-tests/+/798638 | 08:51 |
| opendevreview | Dmitriy Rabotyagov proposed openstack/openstack-ansible-tests master: Remove linters from openstack-ansible-role-jobs https://review.opendev.org/c/openstack/openstack-ansible-tests/+/798814 | 09:24 |
| opendevreview | Dmitriy Rabotyagov proposed openstack/openstack-ansible-tests master: Remove linters from openstack-ansible-role-jobs https://review.opendev.org/c/openstack/openstack-ansible-tests/+/798814 | 09:24 |
| opendevreview | Dmitriy Rabotyagov proposed openstack/openstack-ansible-ceph_client master: Add integrated linters test https://review.opendev.org/c/openstack/openstack-ansible-ceph_client/+/798816 | 09:26 |
| opendevreview | Jonathan Herlin proposed openstack/openstack-ansible stable/victoria: Integrate cloudkitty https://review.opendev.org/c/openstack/openstack-ansible/+/797030 | 09:27 |
| noonedeadpunk | awesome, https://review.opendev.org/c/openstack/openstack-ansible-tests/+/798638 passed | 09:36 |
| opendevreview | Dmitriy Rabotyagov proposed openstack/ansible-role-pki master: Use slurp module to collect server certificates for installation https://review.opendev.org/c/openstack/ansible-role-pki/+/798713 | 09:37 |
| noonedeadpunk | mgariepy: if you're around - voting on https://review.opendev.org/c/openstack/openstack-ansible-tests/+/798638 (and all dependants) will be awesome | 10:07 |
| noonedeadpunk | (and dependant relations | 10:07 |
| noonedeadpunk | ebbex: spotz and other cores you're super welcome as well :) ^ | 10:43 |
| *** sshnaidm|afk is now known as sshnaidm | 10:53 | |
| mgariepy | i'll review this in an hour or so. i need to go to the office . | 11:42 |
| opendevreview | Georgina Shippey proposed openstack/openstack-ansible-os_keystone master: Fix typo in keystone-httpd template https://review.opendev.org/c/openstack/openstack-ansible-os_keystone/+/798841 | 11:44 |
| noonedeadpunk | mgariepy: no rush, gshippey already reviewed ;) | 11:46 |
| opendevreview | Merged openstack/ansible-role-pki master: Add pki_setup_host_python_interpreter variable https://review.opendev.org/c/openstack/ansible-role-pki/+/798685 | 12:21 |
| opendevreview | Merged openstack/ansible-role-pki master: Use slurp module to collect CA certificates for installation https://review.opendev.org/c/openstack/ansible-role-pki/+/798712 | 12:45 |
| opendevreview | Merged openstack/ansible-role-pki master: Use slurp module to collect server certificates for installation https://review.opendev.org/c/openstack/ansible-role-pki/+/798713 | 12:45 |
| opendevreview | Merged openstack/ansible-role-pki master: Allow non-root user to read CA certificates https://review.opendev.org/c/openstack/ansible-role-pki/+/798723 | 12:45 |
| opendevreview | Merged openstack/ansible-role-pki master: Allow non root user to read server certificates https://review.opendev.org/c/openstack/ansible-role-pki/+/798725 | 12:45 |
| opendevreview | Dmitriy Rabotyagov proposed openstack/ansible-role-pki stable/wallaby: Add pki_setup_host_python_interpreter variable https://review.opendev.org/c/openstack/ansible-role-pki/+/798873 | 12:45 |
| opendevreview | Dmitriy Rabotyagov proposed openstack/ansible-role-pki stable/wallaby: Use slurp module to collect CA certificates for installation https://review.opendev.org/c/openstack/ansible-role-pki/+/798874 | 12:46 |
| opendevreview | Dmitriy Rabotyagov proposed openstack/ansible-role-pki stable/wallaby: Use slurp module to collect server certificates for installation https://review.opendev.org/c/openstack/ansible-role-pki/+/798875 | 12:46 |
| opendevreview | Dmitriy Rabotyagov proposed openstack/ansible-role-pki stable/wallaby: Use slurp module to collect server certificates for installation https://review.opendev.org/c/openstack/ansible-role-pki/+/798875 | 12:46 |
| opendevreview | Dmitriy Rabotyagov proposed openstack/ansible-role-pki stable/wallaby: Allow non-root user to read CA certificates https://review.opendev.org/c/openstack/ansible-role-pki/+/798876 | 12:47 |
| opendevreview | Dmitriy Rabotyagov proposed openstack/ansible-role-pki stable/wallaby: Allow non root user to read server certificates https://review.opendev.org/c/openstack/ansible-role-pki/+/798877 | 12:47 |
| opendevreview | Dmitriy Rabotyagov proposed openstack/ansible-role-pki stable/wallaby: Allow non root user to read server certificates https://review.opendev.org/c/openstack/ansible-role-pki/+/798877 | 12:48 |
| opendevreview | Dmitriy Rabotyagov proposed openstack/openstack-ansible-tests stable/wallaby: Fix repo passing CI https://review.opendev.org/c/openstack/openstack-ansible-tests/+/798664 | 12:50 |
| opendevreview | Dmitriy Rabotyagov proposed openstack/openstack-ansible-tests stable/wallaby: Fix repo passing CI https://review.opendev.org/c/openstack/openstack-ansible-tests/+/798664 | 12:52 |
| opendevreview | Merged openstack/openstack-ansible-tests master: Fix repo passing CI https://review.opendev.org/c/openstack/openstack-ansible-tests/+/798638 | 13:29 |
| opendevreview | Damian Dąbrowski proposed openstack/openstack-ansible-galera_server master: Improve support for tags https://review.opendev.org/c/openstack/openstack-ansible-galera_server/+/798853 | 13:34 |
| jrosser | oh chicken/egg problem https://opendev.org/openstack/openstack-ansible/raw/stable/wallaby/global-requirement-pins.txt : HTTP Error 404: Not Found"} | 13:35 |
| opendevreview | Merged openstack/ansible-role-pki stable/wallaby: Add pki_setup_host_python_interpreter variable https://review.opendev.org/c/openstack/ansible-role-pki/+/798873 | 13:49 |
| opendevreview | Merged openstack/ansible-role-pki stable/wallaby: Allow non-root user to read CA certificates https://review.opendev.org/c/openstack/ansible-role-pki/+/798876 | 13:49 |
| opendevreview | Merged openstack/ansible-role-pki stable/wallaby: Allow non root user to read server certificates https://review.opendev.org/c/openstack/ansible-role-pki/+/798877 | 13:49 |
| opendevreview | Merged openstack/ansible-role-pki stable/wallaby: Use slurp module to collect CA certificates for installation https://review.opendev.org/c/openstack/ansible-role-pki/+/798874 | 13:49 |
| opendevreview | Merged openstack/ansible-role-pki stable/wallaby: Use slurp module to collect server certificates for installation https://review.opendev.org/c/openstack/ansible-role-pki/+/798875 | 13:49 |
| jrosser | noonedeadpunk: not sure what we do next, as 798638 is going to fail without the integrated repo being branched yet | 13:51 |
| noonedeadpunk | doh | 13:59 |
| noonedeadpunk | ┻━┻︵ \(°□°)/ ︵ ┻━┻ | 14:00 |
| noonedeadpunk | well, then we should branch integrated - dunno... | 14:05 |
| noonedeadpunk | and then fix things, and final release... | 14:05 |
| noonedeadpunk | we have 2 days.... | 14:05 |
| jrosser | or we override the location not to wallaby branch temporarily - it's just for global pins which i guess is the same | 14:08 |
| noonedeadpunk | I think anyway we need rc1 and release though | 14:12 |
| noonedeadpunk | it just could be the same I guess | 14:12 |
| noonedeadpunk | but yes - we can override ofc | 14:12 |
| opendevreview | Dmitriy Rabotyagov proposed openstack/openstack-ansible master: Create stable/wallaby branch https://review.opendev.org/c/openstack/openstack-ansible/+/798901 | 14:15 |
| opendevreview | Damian Dąbrowski proposed openstack/openstack-ansible-galera_server master: Improve support for tags https://review.opendev.org/c/openstack/openstack-ansible-galera_server/+/798903 | 14:21 |
| opendevreview | Damian Dąbrowski proposed openstack/openstack-ansible-galera_server master: Improve support for tags https://review.opendev.org/c/openstack/openstack-ansible-galera_server/+/798853 | 14:25 |
| opendevreview | Dmitriy Rabotyagov proposed openstack/openstack-ansible master: Create stable/wallaby branch https://review.opendev.org/c/openstack/openstack-ansible/+/798901 | 14:27 |
| opendevreview | Dmitriy Rabotyagov proposed openstack/openstack-ansible master: Create stable/wallaby branch https://review.opendev.org/c/openstack/openstack-ansible/+/798901 | 14:31 |
| dmsimard | noonedeadpunk: replied to the issue, it would be a good addition, thanks | 14:39 |
| noonedeadpunk | oh, thanks! | 14:39 |
| noonedeadpunk | I think it's worth showing both original and delegated actually | 14:42 |
| noonedeadpunk | jsut to see whole picture | 14:42 |
| noonedeadpunk | but not sure how better to render that... | 14:42 |
| dmsimard | yup, that's what I suggested | 14:42 |
| noonedeadpunk | eah, indeed )) | 14:45 |
| opendevreview | Hitesh Kumar proposed openstack/openstack-ansible-specs master: Migrate from testr to stestr https://review.opendev.org/c/openstack/openstack-ansible-specs/+/798906 | 14:48 |
| * jrosser raises eyebrow | 14:49 | |
| opendevreview | Georgina Shippey proposed openstack/openstack-ansible-os_keystone master: Improvements to federation packaging https://review.opendev.org/c/openstack/openstack-ansible-os_keystone/+/798911 | 15:00 |
| opendevreview | Dmitriy Rabotyagov proposed openstack/openstack-ansible master: Start Xena cycle https://review.opendev.org/c/openstack/openstack-ansible/+/798920 | 15:37 |
| opendevreview | Dmitriy Rabotyagov proposed openstack/openstack-ansible master: Start Xena cycle https://review.opendev.org/c/openstack/openstack-ansible/+/798920 | 15:50 |
| opendevreview | Dmitriy Rabotyagov proposed openstack/openstack-ansible-openstack_hosts master: Update release codename https://review.opendev.org/c/openstack/openstack-ansible-openstack_hosts/+/798923 | 15:57 |
| noonedeadpunk | wtf with neutron in https://review.opendev.org/c/openstack/openstack-ansible/+/798901 | 16:05 |
| noonedeadpunk | http://paste.openstack.org/show/807079/ | 16:06 |
| jrosser | thats very odd - thats the venv install? | 16:23 |
| jrosser | or the wheel build... ? | 16:23 |
| *** anbanerj|rover is now known as frenzy_friday | 17:07 | |
| noonedeadpunk | sorry went out :( | 18:56 |
| noonedeadpunk | that was venv install | 18:57 |
| noonedeadpunk | xena patch is even more depressing... | 19:09 |
| mgariepy | https://storage.gra.cloud.ovh.net/v1/AUTH_dcaab5e32b234d56b626f72581e3644c/zuul_opendev_logs_54b/798920/2/check/openstack-ansible-deploy-aio_lxc-ubuntu-focal/54bbffb/logs/ara-report/results/1989.html always fun when openstack complete fails. | 19:37 |
| noonedeadpunk | mgariepy: I found fix for that :) https://opendev.org/openstack/osc-placement/commit/281ace20df8bd01f4cb8ae29c39d80e754a9d0a3 | 19:39 |
| mgariepy | cool. | 19:40 |
| noonedeadpunk | well... I wish they bump placement client version... | 19:41 |
| mgariepy | all the patch for W are ready to be approved? | 19:42 |
| noonedeadpunk | we're a bit stuck with https://review.opendev.org/c/openstack/openstack-ansible/+/798901 | 19:43 |
| noonedeadpunk | currently aio is being spawned, but I'm pretty sleepy to understand smth :( | 19:43 |
| noonedeadpunk | and eventually without branching integrated repo we kind of stuck with backporting https://review.opendev.org/c/openstack/openstack-ansible-tests/+/798664 | 19:44 |
| * noonedeadpunk just reproduced | 19:53 | |
| mgariepy | have a nice weekend(4 days long weekend for me!) | 20:02 |
| noonedeadpunk | thanks :( | 20:04 |
| noonedeadpunk | it doesn't seem to be really good | 20:04 |
| noonedeadpunk | WTF https://opendev.org/openstack/requirements/src/branch/stable/wallaby/upper-constraints.txt#L63 | 20:05 |
| jrosser | oh no :( | 20:07 |
| jrosser | well, I guess we have code for filtering things out of u-c | 20:08 |
| noonedeadpunk | https://review.opendev.org/c/openstack/requirements/+/793108 | 20:09 |
| jrosser | ffs | 20:10 |
| jrosser | that’s not really looking like a good reason either | 20:10 |
| noonedeadpunk | yep.. suggested revert | 20:10 |
| noonedeadpunk | as for now I think we need to use older uc to cover that fast | 20:11 |
| noonedeadpunk | prometheanfire: any opinions if around? | 20:11 |
| opendevreview | Dmitriy Rabotyagov proposed openstack/openstack-ansible master: Create stable/wallaby branch https://review.opendev.org/c/openstack/openstack-ansible/+/798901 | 20:15 |
| opendevreview | Dmitriy Rabotyagov proposed openstack/openstack-ansible master: Start Xena cycle https://review.opendev.org/c/openstack/openstack-ansible/+/798920 | 20:24 |
| opendevreview | Dmitriy Rabotyagov proposed openstack/openstack-ansible master: Revert placement client override https://review.opendev.org/c/openstack/openstack-ansible/+/798955 | 20:26 |
| noonedeadpunk | eventualy this neutron bump is kind of the reason why all of our attempts to bump and do stable release were failing for the last month :( | 20:28 |
| noonedeadpunk | *neutron in u-c | 20:36 |
| prometheanfire | noonedeadpunk: we can upgrade nutron in UC, since neutron is used as a lib some places it gets into UC for stable branches (so master doesn't break victoria for instance) | 20:39 |
| prometheanfire | horizon has the same issue | 20:39 |
| prometheanfire | as long as a 18 release is published we can update it | 20:39 |
| opendevreview | Jonathan Rosser proposed openstack/openstack-ansible-os_neutron master: Exclude neutron from venv constraints https://review.opendev.org/c/openstack/openstack-ansible-os_neutron/+/798960 | 21:25 |
| opendevreview | Jonathan Rosser proposed openstack/openstack-ansible-os_neutron stable/wallaby: Exclude neutron from venv constraints https://review.opendev.org/c/openstack/openstack-ansible-os_neutron/+/798881 | 21:25 |
| prometheanfire | might want to do the same with horizon | 21:39 |
| jrosser | it is already done https://github.com/openstack/openstack-ansible-os_horizon/blob/master/tasks/horizon_install_source.yml#L38-L47 | 21:40 |
| jrosser | difference with neutron is that it is a new thing | 21:40 |
| jrosser | and also of note for horizon it is there in u-c master branch | 21:41 |
| jrosser | which just makes the neutron situation all the more wierd imho | 21:41 |
| jrosser | prometheanfire: the behaviour of the new pip resolver makes it even more wrong to put what for most uses are requirements, into u-c | 21:47 |
| jrosser | it didnt really matter with the old resolver, but the new one rejects it totally and fails | 21:47 |
| prometheanfire | jrosser: talking about shas? | 22:01 |
| jrosser | in the old resolver having differing specs for things kind of did something "sensible" so you'd get away with it | 22:03 |
| jrosser | not just just a strightforward failure | 22:03 |
| jrosser | so you could have a specific sha called for at the same time as u-c asking for foo==a.b.c and it would work out ok | 22:04 |
| jrosser | things are much stricter now, so trying to install neutron constrained by a sha whilst also respecting what is in u-c is guaranteed fail | 22:07 |
| prometheanfire | right | 22:08 |
| opendevreview | Georgina Shippey proposed openstack/openstack-ansible-os_keystone master: Refactor out library/keystone_sp and updates to use collections https://review.opendev.org/c/openstack/openstack-ansible-os_keystone/+/798962 | 22:29 |
Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!