| *** weifan has quit IRC | 00:08 | |
| openstackgerrit | melanie witt proposed openstack/openstack-ansible master: Revert "Use non-broken commit for NoVNC" https://review.opendev.org/671642 | 00:19 |
|---|---|---|
| *** gokhani has quit IRC | 00:20 | |
| *** melwitt has joined #openstack-ansible | 00:23 | |
| *** vnogin has joined #openstack-ansible | 00:51 | |
| *** TxGirlGeek has joined #openstack-ansible | 00:56 | |
| *** vnogin has quit IRC | 00:56 | |
| *** BjoernT has joined #openstack-ansible | 00:57 | |
| *** BjoernT has quit IRC | 00:58 | |
| *** gyee has quit IRC | 01:00 | |
| openstackgerrit | Merged openstack/openstack-ansible-os_swift master: Convert dynamic includes to static imports https://review.opendev.org/671329 | 01:03 |
| *** BjoernT has joined #openstack-ansible | 01:17 | |
| *** BjoernT has quit IRC | 01:24 | |
| *** TxGirlGeek has quit IRC | 02:05 | |
| *** joshualyle has joined #openstack-ansible | 02:22 | |
| *** vnogin has joined #openstack-ansible | 03:23 | |
| *** vnogin has quit IRC | 03:28 | |
| *** rpittau|afk is now known as rpittau | 03:44 | |
| *** udesale has joined #openstack-ansible | 04:04 | |
| *** shyamb has joined #openstack-ansible | 04:15 | |
| *** pcaruana has joined #openstack-ansible | 04:37 | |
| *** rpittau is now known as rpittau|Imonapla | 04:38 | |
| *** rpittau|Imonapla is now known as rpittau|flying | 04:38 | |
| *** poopcat has quit IRC | 04:51 | |
| *** poopcat has joined #openstack-ansible | 04:52 | |
| *** poopcat has quit IRC | 04:58 | |
| *** TxGirlGeek has joined #openstack-ansible | 05:22 | |
| *** TxGirlGeek has quit IRC | 05:23 | |
| *** KeithMnemonic has quit IRC | 05:24 | |
| *** raukadah is now known as chandankumar | 05:36 | |
| *** KeithMnemonic has joined #openstack-ansible | 05:40 | |
| *** udesale has quit IRC | 05:42 | |
| *** udesale has joined #openstack-ansible | 05:42 | |
| *** cshen has joined #openstack-ansible | 05:47 | |
| *** joshualyle has quit IRC | 05:49 | |
| *** shyamb has quit IRC | 05:50 | |
| *** KeithMnemonic has quit IRC | 05:51 | |
| *** jbadiapa has quit IRC | 06:03 | |
| *** udesale has quit IRC | 06:12 | |
| *** KeithMnemonic has joined #openstack-ansible | 06:15 | |
| *** udesale has joined #openstack-ansible | 06:17 | |
| *** stuartgr has quit IRC | 06:23 | |
| *** stuartgr has joined #openstack-ansible | 06:24 | |
| *** miloa has joined #openstack-ansible | 06:41 | |
| miloa | Morning | 06:42 |
| miloa | I've got some questions about the ceph_client role. I saw that I can provide a file for ceph.conf, so osa does not get it from mons. But for keyrings i don't see any option to provide the files in order to not get from mons. | 06:47 |
| *** shyamb has joined #openstack-ansible | 06:48 | |
| miloa | There is a variable to set to provide keyrings files "manualy" like ceph.conf file ? | 06:49 |
| jrosser | miloa: remeber that OSA just calls the ceph-ansible roles | 06:54 |
| jrosser | so you set vars in OSA host/group vars and these get picked up by ceph-ansible | 06:55 |
| jrosser | so it's more a question of if ceph-ansible can do what you want | 06:55 |
| jrosser | miloa: i'm assuming you are not able to have the playbooks ssh to the ceph mons to retrieve this data | 06:57 |
| miloa | jrosser: the ceph_client role is part of ceph-ansible role ? because reading the code it seems that ceph_client is part of osa. | 06:59 |
| jrosser | oh right, yes thats correct | 06:59 |
| jrosser | ceph_extra_confs i think | 07:00 |
| jrosser | miloa: there seems to be an example right here in the docs for keyrings coming from the deploy host https://github.com/openstack/openstack-ansible-ceph_client/blob/master/doc/source/configure-ceph.rst | 07:01 |
| miloa | jrosser: yes I saw it, but seems to work only for extra conf files, but for the main/principal ceph.conf it seems it does not apply. | 07:03 |
| jrosser | i'm a bit confused - you asked about keyrings specifically? | 07:04 |
| miloa | yes | 07:04 |
| miloa | In the user_variables I can define my ceph.conf file content, but I can not specify keyrings files. | 07:05 |
| jrosser | and keyring_src/keyring_dest in ceph_extra_confs is not right? that seems to reference keyring sources on the deploy host | 07:07 |
| miloa | the problem is that it applies first ceph-auth (so as i cannot ssh to the mons it fails) then ceph-auth-extra where i can specify keyrings files. | 07:12 |
| *** zbr has quit IRC | 07:15 | |
| *** zbr has joined #openstack-ansible | 07:16 | |
| *** joshualyle has joined #openstack-ansible | 07:22 | |
| *** tosky has joined #openstack-ansible | 07:24 | |
| *** shyamb has quit IRC | 07:34 | |
| *** shyamb has joined #openstack-ansible | 07:35 | |
| *** gokhani has joined #openstack-ansible | 07:38 | |
| *** ivve has joined #openstack-ansible | 07:44 | |
| *** dr_feelgood has quit IRC | 07:46 | |
| *** luksky11 has joined #openstack-ansible | 07:46 | |
| *** jbadiapa has joined #openstack-ansible | 07:49 | |
| *** dr_feelgood has joined #openstack-ansible | 07:50 | |
| *** shyamb has quit IRC | 07:59 | |
| *** shyamb has joined #openstack-ansible | 08:02 | |
| *** dr_feelgood has quit IRC | 08:03 | |
| *** dr_feelgood has joined #openstack-ansible | 08:05 | |
| *** rpittau|flying is now known as rpittau | 08:13 | |
| noonedeadpunk | jrosser (and everyone) - I'd not vote on these sync tasks, as I'm going to drop xenial from vagrantfile | 08:17 |
| noonedeadpunk | I hope that this will just cause update of existing patches | 08:18 |
| jrosser | noonedeadpunk: yes i left them all unmerged because there was something odd about the opensuse additions too | 08:18 |
| jrosser | both had the same box name for 15.0 and 15.1 | 08:18 |
| noonedeadpunk | So you was already making fix for this? | 08:18 |
| jrosser | and it was unclear if that was intentional or a mistake | 08:18 |
| noonedeadpunk | Ah, ok | 08:18 |
| jrosser | noonedeadpunk: this is ready https://review.opendev.org/#/c/671454/ | 08:19 |
| jrosser | and this would be good to merge too as its a bugfix https://review.opendev.org/#/c/671157/ | 08:20 |
| jrosser | miloa: i think that you should look at if its possible to add a new boolen default var to ceph_client which stops it from running ansible tasks against the mon hosts | 08:21 |
| noonedeadpunk | don't we rely on .my.cnf in terms of mysql_db and mysql_user modules? | 08:22 |
| noonedeadpunk | So since my.cnf will define host, there shouldn't be a problem? | 08:22 |
| noonedeadpunk | https://opendev.org/openstack/openstack-ansible-galera_client/src/branch/master/templates/client.my.cnf.j2 | 08:23 |
| noonedeadpunk | or it covers the case of delegating to host, which doesn't have my.cnf ? But in this case it will be missing username and password | 08:26 |
| miloa | jrosser: thanks, i will check that. Perhaps will try to propose some code modification (althrought, i am not a developper :) ) | 08:26 |
| jrosser | miloa: dont worry :) you should be able to find examples in the rest of the code | 08:27 |
| jrosser | adding extra "when: ceph_client_mon_access is true" to the tasks that fail should let you move through it step my step | 08:28 |
| *** shyamb has quit IRC | 08:29 | |
| jrosser | noonedeadpunk: so right now the ansible module tries to contact localhost which is the ansible default for that module https://docs.ansible.com/ansible/2.5/modules/mysql_db_module.html | 08:29 |
| jrosser | i'd have to dig a bit into the ansible code i think but it doesnt say anything about taking settings from the mysql config files | 08:29 |
| jrosser | although i don't have particularly extensive experience with mysql | 08:30 |
| noonedeadpunk | But it has config_file which defaults to ~/.my.cnf | 08:30 |
| noonedeadpunk | however I'm not sure that it takes host from it, since it's a bit unclear.... | 08:31 |
| jrosser | so the other thing is that https://opendev.org/openstack/openstack-ansible-galera_client/src/branch/master/templates/client.my.cnf.j2 could indeed be the source of the connection to 'localhost' | 08:32 |
| jrosser | now that you point to that file :) | 08:32 |
| jrosser | so i could easily try a test patch to that to try connecting to the actual hostname rather than localhost | 08:32 |
| *** shyamb has joined #openstack-ansible | 08:32 | |
| *** gtarnaras has joined #openstack-ansible | 08:33 | |
| *** cshen has quit IRC | 08:33 | |
| noonedeadpunk | Wondering how safe it's to drop https://opendev.org/openstack/openstack-ansible-tests/src/branch/master/zuul.d/jobs.yaml#L131-L150 | 08:38 |
| *** dr_feelgood has quit IRC | 08:46 | |
| *** siqbal has joined #openstack-ansible | 08:46 | |
| *** dr_feelgood has joined #openstack-ansible | 08:49 | |
| *** cshen has joined #openstack-ansible | 08:50 | |
| jrosser | noonedeadpunk: well, for tests on master that use the integrated repo and try to use xenial, they'll fail now anyway with this merged https://github.com/openstack/openstack-ansible/commit/8c0f5c119acec11fcae8284aef1ecdabb125ea39 | 08:50 |
| noonedeadpunk | I'm more worried about ones w/o integrated tests | 08:51 |
| jrosser | just beware theres a bionic job in the middle of the ones you linked | 08:51 |
| noonedeadpunk | yep, already saw it | 08:51 |
| noonedeadpunk | thanks) | 08:51 |
| jrosser | yes, i wonder how easy it is to find those | 08:52 |
| noonedeadpunk | at least found galera_client and nspawn_hosts | 08:52 |
| jrosser | slightly sad that you don't seem to be able to add filters with wildcards here http://zuul.openstack.org/builds?branch=master | 08:54 |
| *** dr_feelgood has quit IRC | 08:54 | |
| noonedeadpunk | yeah:( | 08:55 |
| *** cshen has quit IRC | 08:55 | |
| openstackgerrit | Dmitriy Rabotyagov (noonedeadpunk) proposed openstack/openstack-ansible-galera_client master: Drop support for Ubuntu Xenial https://review.opendev.org/671685 | 08:57 |
| *** shyamb has quit IRC | 08:58 | |
| *** shyamb has joined #openstack-ansible | 08:58 | |
| jrosser | noonedeadpunk: shouldnt we convert the xenial job there to bionic? or is it a useless test now? | 08:59 |
| noonedeadpunk | I thought it's using integrated tests and xenial is just extra from templates | 09:00 |
| noonedeadpunk | so zuul is testing bionic | 09:00 |
| noonedeadpunk | at least queued it:) | 09:01 |
| *** dr_feelgood has joined #openstack-ansible | 09:01 | |
| openstackgerrit | Dmitriy Rabotyagov (noonedeadpunk) proposed openstack/openstack-ansible-nspawn_hosts master: Drop support for Ubuntu Xenial https://review.opendev.org/671687 | 09:05 |
| openstackgerrit | Dmitriy Rabotyagov (noonedeadpunk) proposed openstack/openstack-ansible-tests master: Drop support for Ubuntu Xenial https://review.opendev.org/671689 | 09:10 |
| *** cshen has joined #openstack-ansible | 09:11 | |
| jrosser | noonedeadpunk: yes sure but the removed xenial job was an special case ssl test? | 09:13 |
| *** cshen has quit IRC | 09:16 | |
| noonedeadpunk | hm, maybe I missunderstood smth, but shouldn't here https://opendev.org/openstack/openstack-ansible-galera_client/src/branch/master/tox.ini be smth called "ssl" then? | 09:26 |
| noonedeadpunk | or how it works then? | 09:27 |
| noonedeadpunk | jrosser: ^ | 09:28 |
| *** yolanda has quit IRC | 09:30 | |
| evrardjp | hey, did Taseer show up in this channel? I was in a trip, and I didn't have a chance to say hi | 09:35 |
| *** cshen has joined #openstack-ansible | 09:38 | |
| *** gtarnaras has quit IRC | 09:41 | |
| *** gtarnaras has joined #openstack-ansible | 09:46 | |
| *** vnogin has joined #openstack-ansible | 09:55 | |
| *** vnogin has quit IRC | 10:01 | |
| openstackgerrit | Dmitriy Rabotyagov (noonedeadpunk) proposed openstack/openstack-ansible-os_gnocchi master: service_setup: refactor service setup to a single file https://review.opendev.org/670471 | 10:01 |
| *** udesale has quit IRC | 10:05 | |
| *** udesale has joined #openstack-ansible | 10:06 | |
| *** shyamb has quit IRC | 10:13 | |
| *** shyamb has joined #openstack-ansible | 10:13 | |
| *** luksky11 has quit IRC | 10:28 | |
| *** shyamb has quit IRC | 10:41 | |
| *** udesale has quit IRC | 10:44 | |
| *** udesale has joined #openstack-ansible | 10:45 | |
| *** dr_feelgood has quit IRC | 10:46 | |
| *** udesale has quit IRC | 10:54 | |
| *** udesale has joined #openstack-ansible | 10:55 | |
| *** luksky11 has joined #openstack-ansible | 10:56 | |
| *** udesale has quit IRC | 11:02 | |
| *** kplant has joined #openstack-ansible | 11:15 | |
| *** shyamb has joined #openstack-ansible | 11:40 | |
| *** dr_feelgood has joined #openstack-ansible | 11:48 | |
| *** cshen has quit IRC | 11:48 | |
| *** dr_feelgood has quit IRC | 11:49 | |
| *** cshen has joined #openstack-ansible | 11:49 | |
| *** dr_feelgood has joined #openstack-ansible | 11:49 | |
| *** spatel has joined #openstack-ansible | 11:52 | |
| *** dr_feelgood has quit IRC | 12:01 | |
| *** dr_feelgood has joined #openstack-ansible | 12:02 | |
| *** spatel has quit IRC | 12:08 | |
| *** dr_feelgood has quit IRC | 12:09 | |
| *** dr_feelgood has joined #openstack-ansible | 12:09 | |
| *** dr_feelgood has quit IRC | 12:11 | |
| openstackgerrit | Dmitriy Rabotyagov (noonedeadpunk) proposed openstack/openstack-ansible-os_swift master: Use systemd-journald instead of log files https://review.opendev.org/671711 | 12:11 |
| *** dr_feelgood has joined #openstack-ansible | 12:12 | |
| openstackgerrit | Dmitriy Rabotyagov (noonedeadpunk) proposed openstack/openstack-ansible-os_ceilometer master: service_setup: refactor service setup to a single file https://review.opendev.org/670473 | 12:13 |
| openstackgerrit | Dmitriy Rabotyagov (noonedeadpunk) proposed openstack/openstack-ansible-os_swift master: service_setup: refactor service setup to a single file https://review.opendev.org/671484 | 12:14 |
| openstackgerrit | Dmitriy Rabotyagov (noonedeadpunk) proposed openstack/openstack-ansible-os_swift master: service_setup: refactor service setup to a single file https://review.opendev.org/671484 | 12:16 |
| mnaser | evrardjp: not sure :X | 12:19 |
| mnaser | evrardjp: but i fixed this for you :) https://review.opendev.org/#/c/671488/ | 12:19 |
| *** udesale has joined #openstack-ansible | 12:21 | |
| *** joshualyle has quit IRC | 12:23 | |
| *** gokhani has quit IRC | 12:23 | |
| mnaser | can i get reviews on this to fix our stable branches after backport: https://review.opendev.org/#/c/671569/ | 12:24 |
| *** dr_feelgood has quit IRC | 12:24 | |
| *** vnogin has joined #openstack-ansible | 12:29 | |
| *** shyamb has quit IRC | 12:30 | |
| *** dr_feelgood has joined #openstack-ansible | 12:32 | |
| openstackgerrit | Merged openstack/openstack-ansible-os_manila master: Updated from OpenStack Ansible Tests https://review.opendev.org/671605 | 12:34 |
| openstackgerrit | Merged openstack/openstack-ansible-ops master: Updated from OpenStack Ansible Tests https://review.opendev.org/671589 | 12:35 |
| openstackgerrit | Merged openstack/openstack-ansible-pip_install master: Updated from OpenStack Ansible Tests https://review.opendev.org/671623 | 12:36 |
| *** spatel has joined #openstack-ansible | 12:36 | |
| openstackgerrit | Dmitriy Rabotyagov (noonedeadpunk) proposed openstack/openstack-ansible-os_zun stable/stein: Convert dynamic includes to static imports https://review.opendev.org/671715 | 12:37 |
| *** goldyfruit has quit IRC | 12:37 | |
| openstackgerrit | Dmitriy Rabotyagov (noonedeadpunk) proposed openstack/openstack-ansible-os_swift stable/stein: Convert dynamic includes to static imports https://review.opendev.org/671716 | 12:39 |
| *** shyamb has joined #openstack-ansible | 12:40 | |
| openstackgerrit | Merged openstack/openstack-ansible-lxc_hosts master: Updated from OpenStack Ansible Tests https://review.opendev.org/671583 | 12:40 |
| openstackgerrit | Merged openstack/openstack-ansible-nspawn_hosts master: Updated from OpenStack Ansible Tests https://review.opendev.org/671586 | 12:42 |
| *** spatel has quit IRC | 12:42 | |
| openstackgerrit | Merged openstack/openstack-ansible-nspawn_container_create master: Updated from OpenStack Ansible Tests https://review.opendev.org/671585 | 12:43 |
| *** shyamb has quit IRC | 12:44 | |
| *** shyamb has joined #openstack-ansible | 12:44 | |
| openstackgerrit | Merged openstack/openstack-ansible-lxc_container_create master: Updated from OpenStack Ansible Tests https://review.opendev.org/671582 | 12:45 |
| openstackgerrit | Merged openstack/openstack-ansible-apt_package_pinning master: Updated from OpenStack Ansible Tests https://review.opendev.org/671577 | 12:45 |
| *** dr_feelgood has quit IRC | 12:46 | |
| openstackgerrit | Merged openstack/openstack-ansible-plugins master: Updated from OpenStack Ansible Tests https://review.opendev.org/671624 | 12:46 |
| *** poopcat has joined #openstack-ansible | 12:48 | |
| openstackgerrit | Merged openstack/openstack-ansible-nspawn_hosts master: Drop support for Ubuntu Xenial https://review.opendev.org/671687 | 12:48 |
| *** udesale has quit IRC | 12:49 | |
| *** dr_feelgood has joined #openstack-ansible | 12:51 | |
| openstackgerrit | Merged openstack/openstack-ansible-repo_build master: Updated from OpenStack Ansible Tests https://review.opendev.org/671626 | 12:51 |
| openstackgerrit | Merged openstack/openstack-ansible-haproxy_server master: Updated from OpenStack Ansible Tests https://review.opendev.org/671581 | 12:52 |
| *** dr_feelgood has quit IRC | 12:53 | |
| *** udesale has joined #openstack-ansible | 12:53 | |
| openstackgerrit | Merged openstack/ansible-hardening master: Updated from OpenStack Ansible Tests https://review.opendev.org/671631 | 12:54 |
| *** dr_feelgood has joined #openstack-ansible | 13:00 | |
| *** dr_feelgood has quit IRC | 13:01 | |
| *** dr_feelgood has joined #openstack-ansible | 13:04 | |
| openstackgerrit | Dmitriy Rabotyagov (noonedeadpunk) proposed openstack/openstack-ansible-os_keystone stable/stein: Convert dynamic includes to static imports https://review.opendev.org/671719 | 13:10 |
| openstackgerrit | Dmitriy Rabotyagov (noonedeadpunk) proposed openstack/openstack-ansible-os_cinder stable/stein: Convert dynamic includes to static imports https://review.opendev.org/671722 | 13:14 |
| openstackgerrit | Dmitriy Rabotyagov (noonedeadpunk) proposed openstack/openstack-ansible-os_aodh stable/stein: Convert dynamic includes to static imports https://review.opendev.org/671724 | 13:18 |
| *** shyamb has quit IRC | 13:19 | |
| openstackgerrit | Dmitriy Rabotyagov (noonedeadpunk) proposed openstack/openstack-ansible-os_ironic stable/stein: Convert dynamic includes to static imports https://review.opendev.org/671726 | 13:27 |
| openstackgerrit | Merged openstack/openstack-ansible-tests master: sync: keep doc/requirements.txt syncd across all repos https://review.opendev.org/671488 | 13:29 |
| openstackgerrit | Merged openstack/openstack-ansible-os_zun master: Updated from OpenStack Ansible Tests https://review.opendev.org/671622 | 13:30 |
| noonedeadpunk | mnaser: looks like swift passes journalctl and works http://logs.openstack.org/11/671711/1/check/openstack-ansible-deploy-aio_metal-ubuntu-bionic/33e0e41/logs/host/ | 13:31 |
| openstackgerrit | Merged openstack/openstack-ansible-os_neutron stable/stein: Add placement client for neutron https://review.opendev.org/671157 | 13:32 |
| *** dr_feelgood has quit IRC | 13:33 | |
| mnaser | noonedeadpunk: nice :D | 13:34 |
| *** goldyfruit has joined #openstack-ansible | 13:34 | |
| *** dr_feelgood has joined #openstack-ansible | 13:34 | |
| noonedeadpunk | But gnocchi and service setup has problems with bionic http://logs.openstack.org/71/670471/4/check/openstack-ansible-deploy-aio_distro_metal-ubuntu-bionic/38f6c7c/logs/ara-report/result/f15c55cf-cedc-438f-ad07-97b0c7487d3e/ | 13:35 |
| noonedeadpunk | What makes things more interesting, that debian passed this part before recheck | 13:36 |
| openstackgerrit | Merged openstack/ansible-role-systemd_mount master: Updated from OpenStack Ansible Tests https://review.opendev.org/671633 | 13:36 |
| *** dr_feelgood has quit IRC | 13:37 | |
| *** siqbal has quit IRC | 13:37 | |
| openstackgerrit | Merged openstack/openstack-ansible-memcached_server master: Updated from OpenStack Ansible Tests https://review.opendev.org/671584 | 13:39 |
| openstackgerrit | Merged openstack/openstack-ansible-rsyslog_server master: Updated from OpenStack Ansible Tests https://review.opendev.org/671629 | 13:39 |
| openstackgerrit | Merged openstack/openstack-ansible-os_tempest master: Updated from OpenStack Ansible Tests https://review.opendev.org/671620 | 13:40 |
| openstackgerrit | Merged openstack/ansible-role-systemd_service master: Updated from OpenStack Ansible Tests https://review.opendev.org/671635 | 13:40 |
| openstackgerrit | Merged openstack/ansible-role-python_venv_build master: Updated from OpenStack Ansible Tests https://review.opendev.org/671632 | 13:40 |
| *** siqbal has joined #openstack-ansible | 13:40 | |
| miloa | jrosser: actually testing a patch on our pre-prod infra in order to provide keyrings from files for ceph_client role (ceph_auth.yml) if it works, will propose it to review :) | 13:40 |
| jrosser | miloa: thats fabulous - thanks for taking the time to dig into it | 13:41 |
| openstackgerrit | Merged openstack/openstack-ansible-os_designate master: Updated from OpenStack Ansible Tests https://review.opendev.org/671597 | 13:41 |
| openstackgerrit | Merged openstack/openstack-ansible-os_sahara master: Updated from OpenStack Ansible Tests https://review.opendev.org/671616 | 13:41 |
| openstackgerrit | Merged openstack/openstack-ansible-os_placement master: Updated from OpenStack Ansible Tests https://review.opendev.org/671614 | 13:42 |
| *** strattao has joined #openstack-ansible | 13:43 | |
| *** vnogin has quit IRC | 13:43 | |
| openstackgerrit | Merged openstack/openstack-ansible-galera_server master: Updated from OpenStack Ansible Tests https://review.opendev.org/671580 | 13:43 |
| openstackgerrit | Merged openstack/ansible-config_template master: Updated from OpenStack Ansible Tests https://review.opendev.org/671630 | 13:44 |
| openstackgerrit | Merged openstack/openstack-ansible-os_cinder master: Updated from OpenStack Ansible Tests https://review.opendev.org/671594 | 13:44 |
| openstackgerrit | Merged openstack/openstack-ansible-rsyslog_client master: Updated from OpenStack Ansible Tests https://review.opendev.org/671628 | 13:45 |
| openstackgerrit | Merged openstack/openstack-ansible-openstack_hosts master: Updated from OpenStack Ansible Tests https://review.opendev.org/671587 | 13:46 |
| openstackgerrit | Merged openstack/ansible-role-systemd_networkd master: Updated from OpenStack Ansible Tests https://review.opendev.org/671634 | 13:47 |
| openstackgerrit | Merged openstack/openstack-ansible-repo_server master: Updated from OpenStack Ansible Tests https://review.opendev.org/671627 | 13:47 |
| openstackgerrit | Merged openstack/openstack-ansible-os_neutron master: Updated from OpenStack Ansible Tests https://review.opendev.org/671610 | 13:47 |
| openstackgerrit | Dmitriy Rabotyagov (noonedeadpunk) proposed openstack/openstack-ansible-os_barbican stable/stein: Convert dynamic includes to static imports https://review.opendev.org/671728 | 13:47 |
| openstackgerrit | Merged openstack/openstack-ansible-openstack_openrc master: Updated from OpenStack Ansible Tests https://review.opendev.org/671588 | 13:48 |
| openstackgerrit | Merged openstack/openstack-ansible-os_nova master: Updated from OpenStack Ansible Tests https://review.opendev.org/671611 | 13:48 |
| openstackgerrit | Merged openstack/openstack-ansible-os_magnum master: Updated from OpenStack Ansible Tests https://review.opendev.org/671604 | 13:49 |
| openstackgerrit | Merged openstack/openstack-ansible-os_rally master: Updated from OpenStack Ansible Tests https://review.opendev.org/671615 | 13:50 |
| openstackgerrit | Merged openstack/openstack-ansible-rabbitmq_server master: Updated from OpenStack Ansible Tests https://review.opendev.org/671625 | 13:51 |
| openstackgerrit | Merged openstack/openstack-ansible-os_ironic master: Updated from OpenStack Ansible Tests https://review.opendev.org/671602 | 13:51 |
| *** poopcat1 has joined #openstack-ansible | 13:52 | |
| openstackgerrit | Merged openstack/openstack-ansible-os_mistral master: Updated from OpenStack Ansible Tests https://review.opendev.org/671607 | 13:52 |
| openstackgerrit | Dmitriy Rabotyagov (noonedeadpunk) proposed openstack/openstack-ansible-os_mistral stable/stein: Convert dynamic includes to static imports https://review.opendev.org/671730 | 13:52 |
| openstackgerrit | Dmitriy Rabotyagov (noonedeadpunk) proposed openstack/openstack-ansible-os_mistral stable/stein: Convert dynamic includes to static imports https://review.opendev.org/671730 | 13:54 |
| *** poopcat has quit IRC | 13:54 | |
| openstackgerrit | Merged openstack/openstack-ansible-os_congress master: Updated from OpenStack Ansible Tests https://review.opendev.org/671596 | 13:55 |
| openstackgerrit | Merged openstack/openstack-ansible-os_blazar master: Updated from OpenStack Ansible Tests https://review.opendev.org/671592 | 13:55 |
| openstackgerrit | Merged openstack/openstack-ansible-os_glance master: Updated from OpenStack Ansible Tests https://review.opendev.org/671598 | 13:56 |
| openstackgerrit | Dmitriy Rabotyagov (noonedeadpunk) proposed openstack/openstack-ansible-os_mistral stable/stein: Convert dynamic includes to static imports https://review.opendev.org/671730 | 13:56 |
| *** cshen has quit IRC | 13:56 | |
| openstackgerrit | Dmitriy Rabotyagov (noonedeadpunk) proposed openstack/openstack-ansible-os_magnum stable/stein: Convert dynamic includes to static imports https://review.opendev.org/671734 | 13:58 |
| openstackgerrit | Merged openstack/openstack-ansible-os_tacker master: Updated from OpenStack Ansible Tests https://review.opendev.org/671619 | 13:59 |
| *** goldyfruit has quit IRC | 13:59 | |
| openstackgerrit | Merged openstack/openstack-ansible-os_masakari master: Updated from OpenStack Ansible Tests https://review.opendev.org/671606 | 13:59 |
| *** goldyfruit has joined #openstack-ansible | 14:00 | |
| openstackgerrit | Merged openstack/openstack-ansible-os_trove master: Updated from OpenStack Ansible Tests https://review.opendev.org/671621 | 14:01 |
| openstackgerrit | Merged openstack/openstack-ansible-os_swift master: Updated from OpenStack Ansible Tests https://review.opendev.org/671617 | 14:01 |
| openstackgerrit | Dmitriy Rabotyagov (noonedeadpunk) proposed openstack/openstack-ansible-os_blazar stable/stein: Convert dynamic includes to static imports https://review.opendev.org/671735 | 14:01 |
| openstackgerrit | Merged openstack/openstack-ansible-os_rally master: build: use upper-constraints.txt from rally-openstack https://review.opendev.org/671569 | 14:02 |
| openstackgerrit | Dmitriy Rabotyagov (noonedeadpunk) proposed openstack/openstack-ansible-os_congress stable/stein: Convert dynamic includes to static imports https://review.opendev.org/671737 | 14:05 |
| openstackgerrit | Merged openstack/openstack-ansible-galera_client master: Updated from OpenStack Ansible Tests https://review.opendev.org/671579 | 14:05 |
| openstackgerrit | Merged openstack/openstack-ansible-galera_client master: Drop support for Ubuntu Xenial https://review.opendev.org/671685 | 14:05 |
| *** luksky11 has quit IRC | 14:06 | |
| openstackgerrit | Dmitriy Rabotyagov (noonedeadpunk) proposed openstack/openstack-ansible-os_sahara stable/stein: Convert dynamic includes to static imports https://review.opendev.org/671738 | 14:06 |
| openstackgerrit | Dmitriy Rabotyagov (noonedeadpunk) proposed openstack/openstack-ansible-os_trove stable/stein: Convert dynamic includes to static imports https://review.opendev.org/671741 | 14:08 |
| *** spatel has joined #openstack-ansible | 14:15 | |
| mnaser | jrosser: for upgrade jobs it looks like we're bootstrapping using master for older jobs | 14:20 |
| mnaser | so getting errors like nova_placement_service_password is not defined | 14:20 |
| jrosser | hmm | 14:20 |
| jrosser | well, is this a sort of conceptual problem though | 14:21 |
| jrosser | that we check out the master repo and boostrap | 14:21 |
| jrosser | then checkout N-1 for the upgrade starting point | 14:21 |
| mnaser | right, in upgrade jobs, we should check out source branch, bootstrap that one, fully deploy it, and then run upgrade | 14:21 |
| * jrosser needs to refresh how this works | 14:21 | |
| jrosser | btw saw your comment about git "ref is not a tree" | 14:22 |
| jrosser | wierdly all the reports of that we get here in irc from folks running into that in the wild are for the github repos, not the opendev ones | 14:23 |
| jrosser | like unbound and so on | 14:23 |
| * mnaser shrugs | 14:24 | |
| mnaser | i.. dunno | 14:24 |
| jrosser | i'm sure i found an ansible related bug report, but it related to submodules | 14:26 |
| mnaser | jrosser: i wonder if it easier for us to add that variable and leave it undefined | 14:26 |
| mnaser | and let the upgrade utility stuff generate it | 14:26 |
| mnaser | seems like a hacky workaround tho :P | 14:26 |
| mnaser | jrosser: hmm i was wrong http://logs.openstack.org/05/671105/2/check/openstack-ansible-upgrade-aio_metal-centos-7/15b80c6/job-output.txt.gz#_2019-07-19_12_56_38_216429 | 14:27 |
| mnaser | http://logs.openstack.org/05/671105/2/check/openstack-ansible-upgrade-aio_metal-centos-7/15b80c6/logs/etc/host/openstack_deploy/user_secrets.yml.tar.txt.gz | 14:29 |
| mnaser | i wonder where this is from | 14:29 |
| jrosser | so i see bootstrap-ansible run twice | 14:32 |
| jrosser | looks like bootstrap-aio is done on master? | 14:33 |
| mnaser | ok bootstrap-aio is done on master | 14:33 |
| mnaser | and then after that it checks out | 14:33 |
| mnaser | indeed we are bootstrapping twice jrosser | 14:34 |
| mnaser | well i guess the biggest thing is that we are running the bootstrap-aio in master | 14:34 |
| mnaser | and yeah i think thats what you meant | 14:34 |
| * mnaser looks | 14:34 | |
| jrosser | in fact a super goog improvement would be to make the boostrap-aio script print the branch/sha as the very first thing it does | 14:35 |
| mnaser | ahhh | 14:35 |
| mnaser | also | 14:35 |
| jrosser | becasue http://logs.openstack.org/05/671105/2/check/openstack-ansible-upgrade-aio_metal-centos-7/15b80c6/job-output.txt.gz#_2019-07-19_12_53_42_784742 on its own is really hard to backtrack and figure out exactly what version we are at | 14:35 |
| mnaser | {% if zuul.job is match('^openstack-ansible-deploy-([^-]+)-.*$') %} | 14:35 |
| jrosser | well thats not going to help | 14:36 |
| jrosser | :) | 14:36 |
| *** gtarnaras has quit IRC | 14:39 | |
| *** gtarnaras has joined #openstack-ansible | 14:42 | |
| *** luksky11 has joined #openstack-ansible | 14:46 | |
| openstackgerrit | Merged openstack/openstack-ansible master: Install placement osc plugin on utility machines https://review.opendev.org/671638 | 14:48 |
| *** siqbal has quit IRC | 14:49 | |
| mnaser | logan-: any idea why the bootstrap stuff was moved to pre ? | 14:53 |
| mnaser | just trying to gather if it was a "nice to have gate optimization" or beyond that | 14:54 |
| *** tosky_ has joined #openstack-ansible | 14:59 | |
| openstackgerrit | Merged openstack/openstack-ansible-tests master: Drop support for Ubuntu Xenial https://review.opendev.org/671689 | 14:59 |
| *** tosky has quit IRC | 15:02 | |
| openstackgerrit | Dmitriy Rabotyagov (noonedeadpunk) proposed openstack/openstack-ansible-os_blazar stable/stein: Convert dynamic includes to static imports https://review.opendev.org/671735 | 15:07 |
| *** tosky_ is now known as tosky | 15:10 | |
| *** poopcat1 is now known as poopcat | 15:12 | |
| spatel | jamesdenton: ty | 15:17 |
| spatel | logan-: did you guys figured out Ubuntu bridge flooding issue? | 15:18 |
| logan- | mnaser: yes... many reasons. overall job stability and the ability to implement custom scenarios in roles that consume integrated jobs. | 15:20 |
| logan- | 2 job stability reasons are: 1) jobs that fail in pre-run are retried automatically by zuul. so doing boostrap-ansible in the pre-run is very beneficial for us because we sometimes see roles fail to clone from github, etc etc. we do some internet heavy operations during pre-run now so we get less job failures off the bat due to bad nodes. we also are slightly less likely to experience job timeouts because bootstrap-ansible no longer cuts | 15:22 |
| logan- | into the run phase timeout. | 15:22 |
| logan- | also, jobs that inherit from integrated repo jobs can implement custom configurations in the role (think the nova-lxd job setting up its custom user_variables, or other role scenario jobs we might want to build). the child jobs pre-run always runs after the parent job, so child jobs can inherit integrated jobs and do pre-runs with a fully bootstrapped AIO config already in place. | 15:23 |
| logan- | spatel: no, we've found that building a bridge outside of neutron on ubuntu configures ageing with 300 as default. so far it seems like it is only neutron specific bridges showing this behavior. have not determined why yet. | 15:26 |
| spatel | logan-: i am waiting for jamesdenton to chime in | 15:27 |
| *** zbr is now known as zbr|out | 15:30 | |
| openstackgerrit | Mikael Loaec proposed openstack/openstack-ansible-ceph_client master: Importing keyrings from files rather than from mons https://review.opendev.org/671762 | 15:30 |
| *** luksky11 has quit IRC | 15:30 | |
| miloa | jrosser: here it is https://review.opendev.org/671762 | 15:31 |
| *** strattao has quit IRC | 15:31 | |
| noonedeadpunk | folks, can someone please help me to read the code? Since either it's smth really weired, or I'm missreading things... | 15:37 |
| noonedeadpunk | So here we're iterating over a dict https://opendev.org/openstack/openstack-ansible-os_manila/src/branch/master/tasks/main.yml#L156 In case it's not defined, we're skipping this, right? | 15:37 |
| noonedeadpunk | and we're defining _manila_backends only inside https://opendev.org/openstack/openstack-ansible-os_manila/src/branch/master/tasks/manila_backends.yml which is included AFTER that one | 15:38 |
| noonedeadpunk | moreover, it's going to be redefined since this set_fact is also iterating over groups... | 15:39 |
| melwitt | hi, can anyone point me to where tempest test results are in a job run log? I'm missing it somehow http://logs.openstack.org/42/671642/1/check/openstack-ansible-deploy-aio_lxc-centos-7/8a480fa/logs/ | 15:42 |
| jrosser | melwitt: http://logs.openstack.org/42/671642/1/check/openstack-ansible-deploy-aio_lxc-centos-7/8a480fa/logs/openstack/aio1_utility_container-5210902f/utility/stestr_results.html | 15:43 |
| melwitt | thank you! | 15:43 |
| melwitt | jrosser++ | 15:43 |
| jrosser | np :) | 15:43 |
| spatel | jrosser: i want to give end users to decide which compute node he/she can target just like admin, should i be playing nova_policy_overrides: ? or something else? | 15:45 |
| jrosser | spatel: i'm not sure, sorry. i've made flavors private to particular projects to give things like GPU, but thats not quite the same | 15:48 |
| jrosser | noonedeadpunk: so this accumulates a list of backends? https://opendev.org/openstack/openstack-ansible-os_manila/src/branch/master/tasks/manila_backends.yml#L36 | 15:49 |
| *** pcaruana has quit IRC | 15:49 | |
| *** rpittau is now known as rpittau|afk | 15:50 | |
| noonedeadpunk | jrosser: I gues yes. But with_items confuses me. And that it's collected after NOT running manila_backend_cephfs.yml | 15:50 |
| spatel | jrosser: sure, i am looking something like where end user can park machine to whatever compute host he want, i know we need to change policy.json but trying to push that out via override | 15:51 |
| noonedeadpunk | since it's never going to be defined (unless someone will define this inside user_variable) | 15:51 |
| noonedeadpunk | spatel: you may look at scheduler filters https://docs.openstack.org/nova/rocky/user/filter-scheduler.html | 15:52 |
| melwitt | mnaser: I don't see that any novnc tests run as part of the CI jobs https://review.opendev.org/671642 how did you figure out that novnc consoles got broken in the past? | 15:53 |
| noonedeadpunk | spatel: actually if compute nodes are the same won't this solve your question? https://docs.openstack.org/senlin/stein/scenarios/affinity.html | 15:57 |
| noonedeadpunk | So that users will be able to select whether to place their instances on the same or separate computing nodes | 15:58 |
| jrosser | noonedeadpunk: looks like there was a fairly massive patch to make ceph work at all https://opendev.org/openstack/openstack-ansible-os_manila/commit/b9809ef89dcb963dba43fa3fb91d2555b88e897d | 15:58 |
| jrosser | so i wonder if this has actually had much testing | 15:58 |
| *** gyee has joined #openstack-ansible | 15:58 | |
| noonedeadpunk | yeah... | 15:58 |
| jrosser | i had to pretty much un-brok the octavia role these last few weeks so i wonder how much actual real-life use some of this gets | 15:59 |
| mnaser | melwitt: so we used to have spice as default for the longest time ever but logan- pushed up a patch moving away from it | 15:59 |
| mnaser | let me find it | 15:59 |
| logan- | did it merge? seemed like it was sitting in review forever | 15:59 |
| mnaser | https://review.opendev.org/#/c/662302/ | 15:59 |
| mnaser | it did merge 7 days ago | 15:59 |
| logan- | ahh cool | 16:00 |
| melwitt | mnaser: I mean I don't think I see any console test? http://logs.openstack.org/42/671642/1/check/openstack-ansible-deploy-aio_metal-ubuntu-bionic/63ee856/logs/openstack/aio1-utility/stestr_results.html | 16:00 |
| mnaser | melwitt: i think unfortunately we might not be running any console tests to shorten our run test times.. we should probably add it there | 16:00 |
| mnaser | melwitt: do you know of a good set of console tests we can add to our whitelists to catch this? | 16:01 |
| melwitt | mnaser: ah, ok. was just curious how you discovered it the first time | 16:01 |
| melwitt | mnaser: I think there's only two novnc tests total | 16:01 |
| noonedeadpunk | cloudnull: probably you can give some input regarding manila role and the part with _manila_backends? | 16:02 |
| mnaser | melwitt: i caught it in our own deployment when it broke on me ;p | 16:02 |
| melwitt | mnaser: ah heh | 16:02 |
| mnaser | the mnaser ci that is | 16:02 |
| melwitt | :D | 16:02 |
| noonedeadpunk | human monitoring is great:) | 16:03 |
| melwitt | mnaser: yeah, so test_novnc is the one I know of for making sure console works https://github.com/openstack/tempest/blob/master/tempest/api/compute/servers/test_novnc.py | 16:03 |
| mnaser | yeah we run tempest.scenario.test_server_basic_ops right now | 16:04 |
| mnaser | https://opendev.org/openstack/openstack-ansible-os_tempest/src/branch/master/defaults/main.yml#L93 | 16:05 |
| melwitt | yeah, I see | 16:05 |
| mnaser | let me add that | 16:05 |
| mnaser | unless you wanna push up a patch that adds it :D | 16:05 |
| mnaser | (if you're busy, no worries) | 16:05 |
| melwitt | up to you :) | 16:05 |
| mnaser | melwitt: if you dont mind doing it and then that'll make it easier to land =P | 16:06 |
| mnaser | i can +2 it :P | 16:06 |
| melwitt | oh, right | 16:06 |
| melwitt | ok, can do | 16:06 |
| *** tosky has quit IRC | 16:08 | |
| noonedeadpunk | jrosser: I think that's the answer about it's testing:) https://opendev.org/openstack/openstack-ansible-os_manila/src/branch/master/zuul.d/project.yaml | 16:09 |
| mnaser | logan-: man it would really be nice if our stuff just assumed you have ansible and then did the rest from there | 16:11 |
| mnaser | and zuul ran those playbooks | 16:11 |
| mnaser | (for gate checks and stuff) | 16:12 |
| jrosser | noonedeadpunk: https://review.opendev.org/#/c/647856/ | 16:12 |
| logan- | mnaser: its pivoting towards that.. the gate-check-commit at least has the ability to skip bootstrapping completely now which it did not have before | 16:13 |
| logan- | breaking things up into pre-run and run helped split that up | 16:13 |
| noonedeadpunk | looks like Kevin gave up adding tests | 16:13 |
| * noonedeadpunk feels that manila is not going to come out from incubated state | 16:15 | |
| *** goldyfruit has quit IRC | 16:16 | |
| openstackgerrit | melanie witt proposed openstack/openstack-ansible-os_tempest master: Add test_novnc to test nova vnc consoles https://review.opendev.org/671769 | 16:16 |
| *** gtarnaras has quit IRC | 16:17 | |
| openstackgerrit | melanie witt proposed openstack/openstack-ansible master: Revert "Use non-broken commit for NoVNC" https://review.opendev.org/671642 | 16:17 |
| spatel | noonedeadpunk: thanks for that link.. lookiing | 16:18 |
| *** goldyfruit has joined #openstack-ansible | 16:20 | |
| logan- | mnaser: btw sorry it took a bit to look this up but the commit message on d8a0380 may explain the pre-run change a little better than I did above in case I missed anything | 16:21 |
| mnaser | logan-: yeah its all good, i think i have a good idea of how i can work around it | 16:22 |
| *** goldyfruit has quit IRC | 16:25 | |
| mnaser | logan-: btw i think we might hit a point where we need to just have a branchless openstack-ansible-ci repo | 16:26 |
| mnaser | esp for upgrade jobs | 16:26 |
| openstackgerrit | Mohammed Naser proposed openstack/openstack-ansible master: Add metal upgrade jobs https://review.opendev.org/671105 | 16:27 |
| openstackgerrit | Mohammed Naser proposed openstack/openstack-ansible master: Add metal upgrade jobs https://review.opendev.org/671105 | 16:27 |
| logan- | yeah | 16:30 |
| logan- | maybe we could change our use of the -tests repo to head that way | 16:31 |
| *** goldyfruit has joined #openstack-ansible | 16:31 | |
| *** gtarnaras has joined #openstack-ansible | 16:32 | |
| *** gtarnaras has quit IRC | 16:33 | |
| *** gtarnaras has joined #openstack-ansible | 16:34 | |
| *** goldyfruit has quit IRC | 16:35 | |
| openstackgerrit | Mohammed Naser proposed openstack/openstack-ansible master: Remove unused MAX_RETRIES https://review.opendev.org/671772 | 16:36 |
| *** gtarnaras has quit IRC | 16:38 | |
| *** chandankumar is now known as raukadah | 16:39 | |
| *** nurdie has joined #openstack-ansible | 16:40 | |
| *** goldyfruit has joined #openstack-ansible | 16:47 | |
| *** mwe1 has quit IRC | 16:54 | |
| openstackgerrit | Dmitriy Rabotyagov (noonedeadpunk) proposed openstack/openstack-ansible-os_neutron master: Disable file log for Calico felix https://review.opendev.org/671775 | 16:57 |
| openstackgerrit | Merged openstack/openstack-ansible-os_trove master: Add integrated coverage https://review.opendev.org/660298 | 17:00 |
| tbarron | noonedeadpunk: so what are the blockers for OSA with manila? | 17:02 |
| noonedeadpunk | tbarron: there are no CI, and some parts looks strange | 17:03 |
| *** hamzaachi has joined #openstack-ansible | 17:03 | |
| tbarron | noonedeadpunk: so "just" a matter of no one working on it anymore I guess | 17:04 |
| tbarron | noonedeadpunk: i'm just a manila guy and would like to see deployment tools besides just tripleo supporting it | 17:04 |
| noonedeadpunk | exactly:( | 17:04 |
| jrosser | it doesnt look like the role was ever got to a point that it had a sensible test setup | 17:04 |
| jrosser | once thats in place the maintainance of it should be reasonably OK | 17:05 |
| jrosser | but we never really got it finished it would seem | 17:05 |
| tbarron | noonedeadpunk: jrosser: but i have my hands full with manila itself and my $job is for folks who use tripleo so | 17:05 |
| tbarron | noonedeadpunk: jrosser: am happy to help e.g. on #openstack-manila if you hit issues that we might know something about | 17:06 |
| tbarron | noonedeadpunk: jrosser: but don't have osa cycles personally even though I support the idea :) | 17:06 |
| noonedeadpunk | tbarron: I guess we just need someone who uses it and aware of things. | 17:08 |
| tbarron | noonedeadpunk: so feel free to add me to OSA reviews if you need manila side judgment ... | 17:09 |
| *** poopcat has quit IRC | 17:15 | |
| openstackgerrit | Merged openstack/openstack-ansible stable/stein: Enable gating on distro_metal-ubuntu-bionic job https://review.opendev.org/671571 | 17:20 |
| *** joshualyle has joined #openstack-ansible | 17:21 | |
| mnaser | noonedeadpunk: if you can add manila coverage to gate, that'd be useful for us | 17:24 |
| mnaser | it's something i want us to add and i think its literally a matter of adding coverage, afaik cloudnull had it running | 17:24 |
| mnaser | its just a test thing | 17:24 |
| noonedeadpunk | From what I saw, it's not only about coverage:) But I think it's possible to achieve | 17:25 |
| mnaser | cause i think i remebmer working on it a abit at the time | 17:25 |
| mnaser | it wasn't too far out afaik | 17:25 |
| noonedeadpunk | I'll write this down not to forget | 17:26 |
| mnaser | jrosser: if you get around nspawn .. that'd be nice | 17:27 |
| mnaser | melwitt: dang i think your thing didn't test it because: compute-feature-enabled.vnc_console | 17:29 |
| mnaser | that's set to False | 17:30 |
| mnaser | http://logs.openstack.org/69/671769/1/check/openstack-ansible-deploy-aio_metal-ubuntu-bionic/3aa929a/logs/openstack/aio1-utility/tempest.log.txt.gz#_2019-07-19_17_06_58_043 | 17:30 |
| openstackgerrit | Merged openstack/openstack-ansible-os_horizon master: Updated from OpenStack Ansible Tests https://review.opendev.org/671601 | 17:31 |
| *** gtarnaras has joined #openstack-ansible | 17:32 | |
| melwitt | mnaser: dangit. yeah, that would be a problem | 17:35 |
| noonedeadpunk | mnaser: am I right, that with migration to journalctl we don't need rsyslog_client? https://opendev.org/openstack/openstack-ansible/src/branch/master/playbooks/common-playbooks/neutron.yml#L107-L123 | 17:35 |
| openstackgerrit | Merged openstack/openstack-ansible-os_zun stable/stein: Convert dynamic includes to static imports https://review.opendev.org/671715 | 17:36 |
| *** gtarnaras has quit IRC | 17:37 | |
| mnaser | noonedeadpunk: yes, then we don't longer need it yes | 17:39 |
| openstackgerrit | melanie witt proposed openstack/openstack-ansible-os_tempest master: Add test_novnc to test nova vnc consoles https://review.opendev.org/671769 | 17:40 |
| *** gtarnaras has joined #openstack-ansible | 17:48 | |
| *** gtarnaras has quit IRC | 17:48 | |
| *** udesale has quit IRC | 17:48 | |
| openstackgerrit | Merged openstack/openstack-ansible-os_congress stable/stein: Convert dynamic includes to static imports https://review.opendev.org/671737 | 17:49 |
| *** tosky has joined #openstack-ansible | 17:49 | |
| openstackgerrit | Dmitriy Rabotyagov (noonedeadpunk) proposed openstack/openstack-ansible master: [WIP] Drop creation of lig directories https://review.opendev.org/671783 | 17:50 |
| *** strattao has joined #openstack-ansible | 17:50 | |
| noonedeadpunk | ok, so have everyone great weekends | 17:53 |
| *** poopcat has joined #openstack-ansible | 17:55 | |
| openstackgerrit | Merged openstack/openstack-ansible-os_blazar stable/stein: Convert dynamic includes to static imports https://review.opendev.org/671735 | 17:55 |
| *** hamzaachi has quit IRC | 17:59 | |
| *** poopcat has quit IRC | 17:59 | |
| *** poopcat has joined #openstack-ansible | 18:02 | |
| kaiokmo | noonedeadpunk: you too :) | 18:02 |
| *** poopcat has quit IRC | 18:03 | |
| *** poopcat has joined #openstack-ansible | 18:04 | |
| openstackgerrit | Merged openstack/openstack-ansible-os_swift master: Use systemd-journald instead of log files https://review.opendev.org/671711 | 18:04 |
| *** poopcat has quit IRC | 18:09 | |
| *** poopcat has joined #openstack-ansible | 18:10 | |
| mnaser | noonedeadpunk: later D: | 18:11 |
| openstackgerrit | Merged openstack/openstack-ansible-os_keystone stable/stein: Convert dynamic includes to static imports https://review.opendev.org/671719 | 18:11 |
| mnaser | logan-: you were around for when this was added back then.. whats the point of the varstest jobs? | 18:11 |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/openstack-ansible-apt_package_pinning master: Updated from OpenStack Ansible Tests https://review.opendev.org/671784 | 18:15 |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/openstack-ansible-ceph_client master: Updated from OpenStack Ansible Tests https://review.opendev.org/671578 | 18:16 |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/openstack-ansible-galera_client master: Updated from OpenStack Ansible Tests https://review.opendev.org/671785 | 18:16 |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/openstack-ansible-galera_server master: Updated from OpenStack Ansible Tests https://review.opendev.org/671786 | 18:16 |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/openstack-ansible-haproxy_server master: Updated from OpenStack Ansible Tests https://review.opendev.org/671787 | 18:16 |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/openstack-ansible-lxc_container_create master: Updated from OpenStack Ansible Tests https://review.opendev.org/671788 | 18:17 |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/openstack-ansible-lxc_hosts master: Updated from OpenStack Ansible Tests https://review.opendev.org/671802 | 18:17 |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/openstack-ansible-memcached_server master: Updated from OpenStack Ansible Tests https://review.opendev.org/671804 | 18:17 |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/openstack-ansible-nspawn_container_create master: Updated from OpenStack Ansible Tests https://review.opendev.org/671805 | 18:17 |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/openstack-ansible-nspawn_hosts master: Updated from OpenStack Ansible Tests https://review.opendev.org/671806 | 18:18 |
| openstackgerrit | Merged openstack/openstack-ansible-os_cinder stable/stein: Convert dynamic includes to static imports https://review.opendev.org/671722 | 18:18 |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/openstack-ansible-openstack_hosts master: Updated from OpenStack Ansible Tests https://review.opendev.org/671807 | 18:18 |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/openstack-ansible-openstack_openrc master: Updated from OpenStack Ansible Tests https://review.opendev.org/671808 | 18:18 |
| openstackgerrit | Merged openstack/openstack-ansible-os_sahara stable/stein: Convert dynamic includes to static imports https://review.opendev.org/671738 | 18:18 |
| openstackgerrit | Merged openstack/openstack-ansible-os_magnum stable/stein: Convert dynamic includes to static imports https://review.opendev.org/671734 | 18:18 |
| openstackgerrit | Merged openstack/openstack-ansible-os_mistral stable/stein: Convert dynamic includes to static imports https://review.opendev.org/671730 | 18:18 |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/openstack-ansible-ops master: Updated from OpenStack Ansible Tests https://review.opendev.org/671809 | 18:18 |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/openstack-ansible-os_aodh master: Updated from OpenStack Ansible Tests https://review.opendev.org/671590 | 18:19 |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/openstack-ansible-os_barbican master: Updated from OpenStack Ansible Tests https://review.opendev.org/671591 | 18:19 |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/openstack-ansible-os_blazar master: Updated from OpenStack Ansible Tests https://review.opendev.org/671810 | 18:19 |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/openstack-ansible-os_ceilometer master: Updated from OpenStack Ansible Tests https://review.opendev.org/671593 | 18:20 |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/openstack-ansible-os_cinder master: Updated from OpenStack Ansible Tests https://review.opendev.org/671811 | 18:20 |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/openstack-ansible-os_cloudkitty master: Updated from OpenStack Ansible Tests https://review.opendev.org/671595 | 18:20 |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/openstack-ansible-os_congress master: Updated from OpenStack Ansible Tests https://review.opendev.org/671812 | 18:20 |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/openstack-ansible-os_designate master: Updated from OpenStack Ansible Tests https://review.opendev.org/671813 | 18:20 |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/openstack-ansible-os_glance master: Updated from OpenStack Ansible Tests https://review.opendev.org/671814 | 18:21 |
| openstackgerrit | Merged openstack/openstack-ansible-os_aodh stable/stein: Convert dynamic includes to static imports https://review.opendev.org/671724 | 18:21 |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/openstack-ansible-os_gnocchi master: Updated from OpenStack Ansible Tests https://review.opendev.org/671599 | 18:21 |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/openstack-ansible-os_heat master: Updated from OpenStack Ansible Tests https://review.opendev.org/671600 | 18:21 |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/openstack-ansible-os_horizon master: Updated from OpenStack Ansible Tests https://review.opendev.org/671815 | 18:22 |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/openstack-ansible-os_ironic master: Updated from OpenStack Ansible Tests https://review.opendev.org/671816 | 18:22 |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/openstack-ansible-os_keystone master: Updated from OpenStack Ansible Tests https://review.opendev.org/671603 | 18:22 |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/openstack-ansible-os_magnum master: Updated from OpenStack Ansible Tests https://review.opendev.org/671817 | 18:23 |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/openstack-ansible-os_manila master: Updated from OpenStack Ansible Tests https://review.opendev.org/671818 | 18:23 |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/openstack-ansible-os_masakari master: Updated from OpenStack Ansible Tests https://review.opendev.org/671819 | 18:23 |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/openstack-ansible-os_mistral master: Updated from OpenStack Ansible Tests https://review.opendev.org/671820 | 18:23 |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/openstack-ansible-os_neutron master: Updated from OpenStack Ansible Tests https://review.opendev.org/671821 | 18:24 |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/openstack-ansible-os_nova master: Updated from OpenStack Ansible Tests https://review.opendev.org/671822 | 18:24 |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/openstack-ansible-os_octavia master: Updated from OpenStack Ansible Tests https://review.opendev.org/671612 | 18:25 |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/openstack-ansible-os_panko master: Updated from OpenStack Ansible Tests https://review.opendev.org/671613 | 18:25 |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/openstack-ansible-os_placement master: Updated from OpenStack Ansible Tests https://review.opendev.org/671823 | 18:25 |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/openstack-ansible-os_rally master: Updated from OpenStack Ansible Tests https://review.opendev.org/671824 | 18:25 |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/openstack-ansible-os_sahara master: Updated from OpenStack Ansible Tests https://review.opendev.org/671825 | 18:25 |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/openstack-ansible-os_swift master: Updated from OpenStack Ansible Tests https://review.opendev.org/671826 | 18:26 |
| openstackgerrit | Andreas Jaeger proposed openstack/openstack-ansible-lxc_hosts master: Updated from OpenStack Ansible Tests https://review.opendev.org/671802 | 18:26 |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/openstack-ansible-os_tacker master: Updated from OpenStack Ansible Tests https://review.opendev.org/671827 | 18:26 |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/openstack-ansible-os_tempest master: Updated from OpenStack Ansible Tests https://review.opendev.org/671829 | 18:27 |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/openstack-ansible-os_trove master: Updated from OpenStack Ansible Tests https://review.opendev.org/671830 | 18:27 |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/openstack-ansible-os_zun master: Updated from OpenStack Ansible Tests https://review.opendev.org/671831 | 18:27 |
| *** poopcat has quit IRC | 18:27 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/openstack-ansible-pip_install master: Updated from OpenStack Ansible Tests https://review.opendev.org/671832 | 18:27 |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/openstack-ansible-plugins master: Updated from OpenStack Ansible Tests https://review.opendev.org/671833 | 18:28 |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/openstack-ansible-rabbitmq_server master: Updated from OpenStack Ansible Tests https://review.opendev.org/671834 | 18:28 |
| *** poopcat has joined #openstack-ansible | 18:28 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/openstack-ansible-repo_build master: Updated from OpenStack Ansible Tests https://review.opendev.org/671835 | 18:28 |
| openstackgerrit | Merged openstack/openstack-ansible-os_barbican stable/stein: Convert dynamic includes to static imports https://review.opendev.org/671728 | 18:28 |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/openstack-ansible-repo_server master: Updated from OpenStack Ansible Tests https://review.opendev.org/671836 | 18:29 |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/openstack-ansible-rsyslog_client master: Updated from OpenStack Ansible Tests https://review.opendev.org/671837 | 18:29 |
| *** poopcat has quit IRC | 18:29 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/openstack-ansible-rsyslog_server master: Updated from OpenStack Ansible Tests https://review.opendev.org/671838 | 18:29 |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/ansible-config_template master: Updated from OpenStack Ansible Tests https://review.opendev.org/671839 | 18:30 |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/ansible-hardening master: Updated from OpenStack Ansible Tests https://review.opendev.org/671840 | 18:30 |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/ansible-role-python_venv_build master: Updated from OpenStack Ansible Tests https://review.opendev.org/671841 | 18:30 |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/ansible-role-systemd_mount master: Updated from OpenStack Ansible Tests https://review.opendev.org/671842 | 18:31 |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/ansible-role-systemd_networkd master: Updated from OpenStack Ansible Tests https://review.opendev.org/671843 | 18:31 |
| *** poopcat has joined #openstack-ansible | 18:31 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/ansible-role-systemd_service master: Updated from OpenStack Ansible Tests https://review.opendev.org/671844 | 18:31 |
| *** hamzaachi has joined #openstack-ansible | 18:33 | |
| *** goldyfruit has quit IRC | 18:35 | |
| openstackgerrit | Merged openstack/openstack-ansible-os_trove stable/stein: Convert dynamic includes to static imports https://review.opendev.org/671741 | 18:36 |
| *** luksky11 has joined #openstack-ansible | 18:41 | |
| openstackgerrit | Merged openstack/openstack-ansible-os_swift stable/stein: Convert dynamic includes to static imports https://review.opendev.org/671716 | 18:48 |
| openstackgerrit | Merged openstack/openstack-ansible-os_ironic stable/stein: Convert dynamic includes to static imports https://review.opendev.org/671726 | 18:48 |
| * jrosser facepalm http://logs.openstack.org/16/671816/1/check/openstack-tox-docs/966d3bf/job-output.txt.gz#_2019-07-19_18_40_07_341701 | 18:49 | |
| openstackgerrit | Mohammed Naser proposed openstack/openstack-ansible-tests master: sync: fix documentation config https://review.opendev.org/671845 | 18:52 |
| mnaser | jrosser: yes | 18:53 |
| mnaser | i messed up big time | 18:53 |
| mnaser | :( | 18:53 |
| mnaser | the patch above should fix it | 18:53 |
| mnaser | and kinda self-test | 18:53 |
| jrosser | at least it just pushes another revision rather than a whole new slew of patches | 18:53 |
| mnaser | yeah that's the nice part | 18:53 |
| jrosser | mnaser: btw dmitry left a comment here which is kinda valid https://review.opendev.org/#/c/671454/ | 18:56 |
| jrosser | but the only place we install galera_client is the utility host, and nothing delegates db setup there atm | 18:57 |
| mnaser | are you sure? | 18:57 |
| mnaser | i think galera_client is setup on galera server too | 18:57 |
| mnaser | which is where we delegate | 18:57 |
| mnaser | jrosser: oh crap | 18:58 |
| mnaser | you know why we havent hit this? | 18:58 |
| mnaser | all metal deploys | 18:58 |
| mnaser | lol | 18:58 |
| mnaser | galera_client runs in utility which is the same as galera which just works(tm) | 18:58 |
| jrosser | hah | 18:59 |
| mnaser | imho galera_client should run against galera_server too | 18:59 |
| melwitt | mnaser: bah, need a nova conf setting to enable vnc too | 19:00 |
| jrosser | mnaser: but today all the db setup tasks are delegated to galera[0] for containers or metal installs | 19:01 |
| jrosser | and stuff seems to be working | 19:01 |
| *** goldyfruit has joined #openstack-ansible | 19:02 | |
| jrosser | melwitt: do you need more than this is already enabling? https://github.com/openstack/openstack-ansible-os_nova/blob/51b6474082e929fcd61a410d32b1350dd25b9d60/defaults/main.yml#L280-L283 | 19:06 |
| melwitt | jrosser: I think I need this option enabled, it defaults to True, but the result in the test run implies that it's False https://github.com/openstack/nova/blob/master/nova/conf/vnc.py#L28 | 19:07 |
| melwitt | yeah I see it false here in the failing test run http://logs.openstack.org/42/671642/2/check/openstack-ansible-deploy-aio_metal-debian-stable/5dcbc0d/logs/etc/host/nova/nova.conf.txt.gz | 19:10 |
| melwitt | [vnc]enabled = False | 19:10 |
| jrosser | melwitt: https://opendev.org/openstack/openstack-ansible/src/branch/master/inventory/group_vars/nova_all.yml#L23 | 19:13 |
| *** hamzaachi has quit IRC | 19:14 | |
| jrosser | that means the default console type in the os_nova ansible role got recently patched to be novnc | 19:14 |
| jrosser | but that var is overridden to spice when that role is consumed by openstack-ansible as a whole | 19:15 |
| melwitt | oh, hm | 19:15 |
| jrosser | logan-: did you intend to switch the integrated repo deploys from spice->novnc with this? https://review.opendev.org/#/c/662302/1 | 19:16 |
| openstackgerrit | Jonathan Rosser proposed openstack/openstack-ansible master: Switch nova console type from spice to novnc https://review.opendev.org/671848 | 19:21 |
| openstackgerrit | Jonathan Rosser proposed openstack/openstack-ansible-os_tempest master: Add test_novnc to test nova vnc consoles https://review.opendev.org/671769 | 19:21 |
| jrosser | melwitt: ^ lets see what that does | 19:21 |
| *** kplant has quit IRC | 19:21 | |
| melwitt | jrosser: cool thanks | 19:23 |
| mnaser | jrosser: yeah it works because by default it listens to localhost i assume | 19:27 |
| jrosser | yes thats right | 19:28 |
| *** hamzaachi has joined #openstack-ansible | 19:28 | |
| * jrosser patching to add the client role just now | 19:28 | |
| jrosser | well, no, it listens to 0.0.0.0 | 19:29 |
| jrosser | and the ansible module connects to localhost and that works | 19:29 |
| jrosser | when we bind to mgmt only it breaks | 19:29 |
| openstackgerrit | Jonathan Rosser proposed openstack/openstack-ansible master: Run galera client role against galera hosts https://review.opendev.org/671849 | 19:30 |
| openstackgerrit | Mohammed Naser proposed openstack/openstack-ansible master: Add metal upgrade jobs https://review.opendev.org/671105 | 19:32 |
| openstackgerrit | Jonathan Rosser proposed openstack/openstack-ansible master: [WIP] Bind services to mgmt network addresses https://review.opendev.org/670051 | 19:33 |
| jrosser | mnaser: youve seen those upgrade jobs are making containers? | 19:36 |
| mnaser | jrosser: yeah .. the scenario wasnt being picked up because of the openstack-ansible-deploy hardcode | 19:36 |
| jrosser | ok | 19:36 |
| mnaser | so right now i havent even successfully gotten it to deploy N-1 :P | 19:36 |
| *** joshualyle has quit IRC | 19:41 | |
| *** joshualyle has joined #openstack-ansible | 19:46 | |
| logan- | jrosser: yes | 19:54 |
| logan- | interesting, I didn't know we were overriding it to spice somewhere else | 19:56 |
| logan- | mnaser: the varstest question... i'm not very familiar with that job. I know I glanced at it when I was working on the tests a couple months ago but I barely remember what it does. | 20:00 |
| logan- | I remember looking at it thinking about ripping it out and then it seemed like it might be marginally useful | 20:00 |
| openstackgerrit | Logan V proposed openstack/openstack-ansible master: Stop creating calico log directory https://review.opendev.org/671856 | 20:05 |
| mnaser | logan-: ack ok cool | 20:18 |
| mnaser | jrosser: so far so good... i think i might have at least the n-1 deploy part done successfully.. | 20:19 |
| mnaser | its in os_tempset and it didnt create containers | 20:19 |
| logan- | :) | 20:19 |
| logan- | nice | 20:19 |
| * mnaser is happy at the progress we're making recently | 20:19 | |
| mnaser | a lot of good clean-up an stability improvements | 20:19 |
| mnaser | once we get stein=>master working well with ci, then we can backport to do rocky=>stein | 20:19 |
| mnaser | at least we'll have the proper "infrastructure" to do it | 20:20 |
| jrosser | ++ to that, its really good work recently | 20:20 |
| jrosser | quite a lot of things tripping over service startup / db migrate race | 20:22 |
| mnaser | i think logically we should just always have db setup / mq setup before systemd stuff | 20:22 |
| jrosser | are things any different at upgrade because restarting services in handlers happens last | 20:31 |
| *** strattao has quit IRC | 20:35 | |
| mnaser | jrosser: we have a flush handlers at the end of every role afaik | 20:48 |
| mnaser | ok neat | 20:48 |
| mnaser | we have a clean n-1 deploy | 20:49 |
| mnaser | fatal: [localhost]: FAILED! => {"changed": false, "msg": "There are files in /etc/openstack_deploy/env.d which override the default inventory\nlayout in /home/zuul/src/opendev.org/openstack/openstack-ansible/inventory/env.d. The difference between these files\nshould be carefully reviewed to understand whether the changes are still necessary\nand applicable to the environment. If all the user-space | 20:49 |
| mnaser | env.d files are necessary,\nthen please export SKIP_CUSTOM_ENVD_CHECK=true and re-run the playbook or\nrun-upgrade.sh script.\n"} | 20:49 |
| mnaser | #progress | 20:49 |
| jrosser | mnaser: it looks like adding galera_client to the galera containers is making the lxc jobs look better too | 20:50 |
| jrosser | there are some distro_metal failures related to db connection fail but i think we're just flushing out more things to tidy up | 20:51 |
| *** hamzaachi has quit IRC | 20:59 | |
| mnaser | jrosser: and it looks like my upgrade failures are because the env.d/aio_metal.yml files are not the same | 20:59 |
| mnaser | https://github.com/openstack/openstack-ansible/blob/master/scripts/upgrade-utilities/deploy-config-changes.yml#L43-L65 | 21:00 |
| mnaser | im not sure the best way to go about this | 21:01 |
| jrosser | is this becasue we create a custom env.d specially for metal AIO test | 21:01 |
| jrosser | so it inevitably fails | 21:01 |
| mnaser | yeah | 21:01 |
| mnaser | im wondering how much coverage we lose by putting SKIP_CUSTOM_ENVD_CHECK .. | 21:01 |
| jrosser | we give folks a huge freedom to reassign the functions to hosts as they choose | 21:02 |
| jrosser | and I think that this task is there to say "hey this is your problem at this point" when the environment looks different from default | 21:03 |
| jrosser | as the upgrade scripts can't be expected to deal with anything other than the default layout | 21:03 |
| mnaser | jrosser: but im wondering if we made something like removed a certain container or converged them into something else | 21:03 |
| mnaser | like what if i was runnig all metal, and then had to upgrade and didn't have the placement config and now i have a placement container | 21:03 |
| jrosser | not that i recall, but that has happened in the past, with neutron stuff for example | 21:03 |
| mnaser | yeah and nova too | 21:04 |
| mnaser | and heat afaik | 21:04 |
| mnaser | we converged them into a single container | 21:04 |
| mnaser | so there has been times where we've made adjustments | 21:04 |
| jrosser | so maybe actually having to drop a custom env.d file to enable an all metal AIO is something we shouldnt have to do | 21:05 |
| jrosser | perhaps that should be more native somehow | 21:05 |
| mnaser | well i know that there's another way to do it cloudnull setup a while back | 21:05 |
| jrosser | then this wouldnt get stuck at that env.d check | 21:05 |
| mnaser | wait maybe that never merged | 21:06 |
| mnaser | it was an openstack_user_config leve lthing | 21:06 |
| jrosser | wasnt that a flag per host just to do all metal | 21:06 |
| mnaser | yeah | 21:06 |
| jrosser | i think if we can do something outside messing with env.d that would be a good improvement | 21:07 |
| mnaser | https://review.opendev.org/#/c/635997/ | 21:07 |
| mnaser | no_containers: true | 21:08 |
| jrosser | cool - right i'm done for today, happy upgrades :) | 21:10 |
| mnaser | jrosser: take care! have a good weekend | 21:10 |
| openstackgerrit | Mohammed Naser proposed openstack/openstack-ansible master: Add metal upgrade jobs https://review.opendev.org/671105 | 21:15 |
| openstackgerrit | Mohammed Naser proposed openstack/openstack-ansible master: metal: add docs + start using no_containers https://review.opendev.org/671862 | 21:15 |
| ioni | novnc works on stein fine? | 21:23 |
| mnaser | ioni: yep | 21:23 |
| *** luksky11 has quit IRC | 21:23 | |
| ioni | i know it had some problems with nova | 21:23 |
| ioni | some authorization issues | 21:23 |
| mnaser | ioni: that's weird, it should be resolved in latest stein | 21:24 |
| ioni | ok, i'll update to 19.0.0 | 21:24 |
| ioni | i'm still on rc3 | 21:24 |
| mnaser | let me know if you have issues | 21:24 |
| mnaser | yeah that might contribute | 21:24 |
| ioni | https://review.opendev.org/#/c/671642/ | 21:26 |
| ioni | found that | 21:27 |
| mnaser | othats actually something that resolves the previous bug that was fixed | 21:28 |
| ioni | ok, so actually i still have to wait for stein, modifications are still in pending | 21:31 |
| ioni | https://bugs.launchpad.net/openstack-ansible/+bug/1822676 | 21:31 |
| openstack | Launchpad bug 1822676 in openstack-ansible "novnc no longer sets token inside cookie" [Undecided,In progress] - Assigned to melanie witt (melwitt) | 21:31 |
| nurdie | Rocky 18.1.8 fresh epel-repo_build.repo file from ansible. I'm running CentOS7 and it's failing hard. Any ideas? https://pastebin.com/uNDqgDAr | 21:31 |
| nurdie | I want to scream "UPSTREAM PROBLEM" and turn my head, but I can download those from a browser just fine | 21:32 |
| *** vnogin has joined #openstack-ansible | 21:32 | |
| *** vnogin has quit IRC | 21:32 | |
| nurdie | I can also wget them just fine from the container | 21:33 |
| nurdie | openstack-ansible playbooks/repo-build.yml -e repo_build_git_reclone=yes -e repo_build_wheel_rebuild=yes -e repo_build_venv_rebuild=yes | 21:35 |
| nurdie | I'm currently running that now | 21:35 |
| *** tosky has quit IRC | 21:43 | |
| *** hamzaachi has joined #openstack-ansible | 21:57 | |
| *** spatel has quit IRC | 22:04 | |
| *** KeithMnemonic has quit IRC | 22:04 | |
| *** joshualyle has quit IRC | 22:23 | |
| openstackgerrit | Mikael Loaec proposed openstack/openstack-ansible-ceph_client master: Importing keyrings from files rather than from mons https://review.opendev.org/671762 | 22:25 |
| miloa | jrosser: In this 2nd patch https://review.opendev.org/671762 i corrected some problem with ceph_nova_secret, i didn't notice at first. I deployed on our pre-prod, using keyrings from files seems ok. Waiting to see what will say Zuul :) Have a good weekend. | 22:29 |
| openstackgerrit | Dmitriy Rabotyagov (noonedeadpunk) proposed openstack/openstack-ansible-tests master: sync: make doc/requirements.txt pass linters https://review.opendev.org/671871 | 22:30 |
| openstackgerrit | Merged openstack/openstack-ansible-os_neutron master: Disable file log for Calico felix https://review.opendev.org/671775 | 22:30 |
| *** miloa has quit IRC | 22:31 | |
| *** gyee has quit IRC | 22:34 | |
| *** nurdie_ has joined #openstack-ansible | 22:47 | |
| *** nurdie has quit IRC | 22:50 | |
| *** gyee has joined #openstack-ansible | 22:57 | |
| *** nurdie has joined #openstack-ansible | 23:23 | |
| *** nurdie has quit IRC | 23:31 | |
| *** nurdie has joined #openstack-ansible | 23:56 | |
| *** nurdie_ has joined #openstack-ansible | 23:57 | |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!