| *** ysandeep|out is now known as ysandeep|ruck | 05:19 | |
| *** ysandeep|ruck is now known as ysandeep|lunch | 08:57 | |
| opendevreview | Dmitriy Rabotyagov proposed openstack/openstack-ansible master: Continue Zed development https://review.opendev.org/c/openstack/openstack-ansible/+/847272 | 09:18 |
|---|---|---|
| opendevreview | Dmitriy Rabotyagov proposed openstack/openstack-ansible-os_horizon master: Release pyscss constraints https://review.opendev.org/c/openstack/openstack-ansible-os_horizon/+/848589 | 09:29 |
| opendevreview | Dmitriy Rabotyagov proposed openstack/openstack-ansible master: Continue Zed development https://review.opendev.org/c/openstack/openstack-ansible/+/847272 | 09:29 |
| *** ysandeep|lunch is now known as ysandeep | 10:14 | |
| jrosser_ | morning | 10:39 |
| damiandabrowski[m] | hi! | 10:39 |
| jrosser_ | i've updated the grafana dashboards and our POST_FAILURE problems are now visible right at the bottom of here https://grafana.opendev.org/d/8cbc9d7032/openstack-ansible?orgId=1&from=now-90d&to=now | 10:40 |
| *** ysandeep is now known as ysandeep|ruck | 10:44 | |
| noonedeadpunk | nice! | 11:05 |
| noonedeadpunk | how you got access to it?:D | 11:06 |
| jrosser_ | noonedeadpunk: https://review.opendev.org/c/openstack/project-config/+/847988 | 11:07 |
| noonedeadpunk | oh, I see | 11:07 |
| jrosser_ | i think ianw has done some stuff so you can mess with it locally in docker now as well to adjust the dashboards | 11:07 |
| jrosser_ | before you had to write the yaml and cross your fingers | 11:08 |
| jrosser_ | but this is super cool in the CI http://storage.bhs.cloud.ovh.net/v1/AUTH_dcaab5e32b234d56b626f72581e3644c/zuul_opendev_logs_534/847988/1/check/project-config-grafana/5345751/screenshots/ | 11:08 |
| noonedeadpunk | yeah, I can imagine you need just to export dashboard then and push the PR | 11:09 |
| noonedeadpunk | not sure what's tested though... | 11:09 |
| noonedeadpunk | oh, ok, that's solves it:) | 11:09 |
| noonedeadpunk | that's smart | 11:10 |
| *** dviroel_ is now known as dviroel | 11:36 | |
| *** ysandeep|ruck is now known as ysandeep|break | 11:58 | |
| jrosser_ | looks like we never finished deploying horizon with metal jobs | 12:40 |
| *** ysandeep|break is now known as ysandeep | 13:10 | |
| *** ysandeep is now known as ysandeep|ruck | 13:11 | |
| jrosser_ | what is happening with rally | 13:12 |
| jrosser_ | seems broken on master and xena at least | 13:12 |
| noonedeadpunk | well rally quite explainable though | 13:21 |
| noonedeadpunk | `rally-openstack 2.1.0 depends on gnocchiclient!=7.0.7\n The user requested (constraint) gnocchiclient===7.0.7` | 13:21 |
| noonedeadpunk | but I thought we're using rally-provided u-c for it... | 13:23 |
| noonedeadpunk | oh, wow, why we use that old rally | 13:24 |
| opendevreview | Damian Dąbrowski proposed openstack/openstack-ansible-os_tempest master: Make _tempest_plugins a pure, nested dictionary https://review.opendev.org/c/openstack/openstack-ansible-os_tempest/+/848642 | 14:07 |
| opendevreview | Damian Dąbrowski proposed openstack/openstack-ansible-os_tempest master: Allow to define extra tempest plugins https://review.opendev.org/c/openstack/openstack-ansible-os_tempest/+/848643 | 14:07 |
| damiandabrowski[m] | hi! 848642 will require changes in few other repos, but before doing that can I ask You to share Your thoughts about this change? ;) | 14:09 |
| *** ysandeep|ruck is now known as ysandeep|dinner | 14:25 | |
| *** ysandeep|dinner is now known as ysandeep | 14:48 | |
| *** dviroel is now known as dviroel|lunch | 14:58 | |
| noonedeadpunk | damiandabrowski[m]: why not to convert it to simple list then? | 15:37 |
| noonedeadpunk | isn't even easier then dict? | 15:38 |
| noonedeadpunk | do we use dict key anywhere? | 15:38 |
| noonedeadpunk | As eventually `tempest_plugins` is a list at the end? | 15:39 |
| noonedeadpunk | And we can change _tempest_plugins is any way we want even without release note I believe | 15:39 |
| damiandabrowski[m] | hmm, i think tempest_plugins can't be a simple list as it contains key-value pairs? :D | 15:40 |
| noonedeadpunk | ok, list of dicts | 15:40 |
| noonedeadpunk | so why not to make _tempest_plugins to be in the same format then? | 15:40 |
| noonedeadpunk | as it would shorten https://review.opendev.org/c/openstack/openstack-ansible-os_tempest/+/848642/1/defaults/main.yml change to just `tempest_plugins: "{{ _tempest_plugins | selectattr('install', 'equalto', true) }}` ? | 15:42 |
| damiandabrowski[m] | ok, i can try to make it a list of dicts tomorrow | 15:43 |
| *** ysandeep is now known as ysandeep|out | 16:06 | |
| *** dviroel|lunch is now known as dviroel | 16:16 | |
| opendevreview | Jonathan Rosser proposed openstack/openstack-ansible-haproxy_server master: Allow customisation of stick-tables for each service. https://review.opendev.org/c/openstack/openstack-ansible-haproxy_server/+/848657 | 17:50 |
| opendevreview | Jonathan Rosser proposed openstack/openstack-ansible master: Add default rate-limits for API endpoints and Horizon authentication https://review.opendev.org/c/openstack/openstack-ansible/+/848659 | 18:02 |
| jrosser_ | noonedeadpunk: do you think I should use `haproxy_allowlist_networks` in that patch ^ | 18:03 |
| * noonedeadpunk will look tomorrow on them | 18:59 | |
| noonedeadpunk | so regarding rally that commit was 3 days ago https://github.com/openstack/rally-openstack/commit/3984f97c1c9fd0695bab123da81323e83abac3a3 | 19:03 |
| noonedeadpunk | which basically solves our thing | 19:03 |
| noonedeadpunk | but I kind of lost if we bump somehow that | 19:03 |
| noonedeadpunk | seems we just install latest tag from pypi | 19:04 |
| jrosser_ | not sure we version the requirements file from rally? | 19:05 |
| noonedeadpunk | rally itself is in u-c. But not rally-openstack, which we likely can have in requirements | 19:06 |
| noonedeadpunk | https://opendev.org/openstack/openstack-ansible-os_rally/src/commit/c3d105fd9edb3bbb2dbd781c88b9af7f23f267eb/defaults/main.yml#L53 | 19:06 |
| opendevreview | Dmitriy Rabotyagov proposed openstack/openstack-ansible-os_rally master: Controll rally-openstack istalled version https://review.opendev.org/c/openstack/openstack-ansible-os_rally/+/848666 | 19:29 |
| opendevreview | Dmitriy Rabotyagov proposed openstack/openstack-ansible master: Continue Zed development https://review.opendev.org/c/openstack/openstack-ansible/+/847272 | 19:32 |
| opendevreview | Dmitriy Rabotyagov proposed openstack/openstack-ansible-os_rally master: Control rally-openstack installed version https://review.opendev.org/c/openstack/openstack-ansible-os_rally/+/848666 | 19:33 |
| opendevreview | Dmitriy Rabotyagov proposed openstack/openstack-ansible-os_rally master: Control rally-openstack installed version https://review.opendev.org/c/openstack/openstack-ansible-os_rally/+/848666 | 19:34 |
| *** dviroel is now known as dviroel|biab | 19:37 | |
| ianw | jrosser_/noonedeadpunk: https://review.opendev.org/c/openstack/project-config/+/833214 will get you a local grafana and load the files into it | 22:37 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!