| *** markvoelker has joined #openstack-ansible | 00:10 | |
| *** markvoelker has quit IRC | 00:15 | |
| *** gyee has quit IRC | 00:18 | |
| *** threestrands has joined #openstack-ansible | 00:24 | |
| csmart | evrardjp: so... as far as I can see, the AIO doco says it uses one ethernet card (as per the reference diagram) | 01:36 |
|---|---|---|
| csmart | https://docs.openstack.org/openstack-ansible/latest/user/aio/quickstart.html#reference-diagram-for-an-aio-build | 01:36 |
| csmart | but do I need to configure AIO like a non-AIO and set up the networks and custom yaml for where to put the networks? | 01:37 |
| csmart | (or pre-configure the bridges?) | 01:37 |
| *** markvoelker has joined #openstack-ansible | 03:10 | |
| *** markvoelker has quit IRC | 03:15 | |
| *** nicolasbock has quit IRC | 03:25 | |
| *** rohit02 has joined #openstack-ansible | 03:40 | |
| *** gkadam has joined #openstack-ansible | 03:51 | |
| *** gkadam has quit IRC | 03:51 | |
| *** cp is now known as cp- | 04:02 | |
| *** udesale has joined #openstack-ansible | 04:06 | |
| *** markvoelker has joined #openstack-ansible | 04:10 | |
| *** markvoelker has quit IRC | 04:15 | |
| *** raukadah is now known as chkumar|rover | 04:26 | |
| *** dave-mccowan has quit IRC | 04:36 | |
| *** macz has joined #openstack-ansible | 04:58 | |
| *** macz has quit IRC | 05:03 | |
| *** markvoelker has joined #openstack-ansible | 05:10 | |
| *** markvoelker has quit IRC | 05:15 | |
| *** rohit02 has quit IRC | 05:34 | |
| *** rohit02 has joined #openstack-ansible | 05:34 | |
| *** shyamb has joined #openstack-ansible | 05:54 | |
| *** hamzy has quit IRC | 05:59 | |
| *** shyamb has quit IRC | 06:02 | |
| *** shyamb has joined #openstack-ansible | 06:09 | |
| *** macz has joined #openstack-ansible | 06:11 | |
| *** macz has quit IRC | 06:15 | |
| *** prometheanfire has quit IRC | 06:18 | |
| *** prometheanfire has joined #openstack-ansible | 06:22 | |
| *** dr_feelgood has joined #openstack-ansible | 06:34 | |
| *** rohit02 has quit IRC | 06:35 | |
| *** shyam89 has joined #openstack-ansible | 06:36 | |
| *** shyamb has quit IRC | 06:36 | |
| *** macz has joined #openstack-ansible | 06:39 | |
| *** macz has quit IRC | 06:44 | |
| *** shyam89 has quit IRC | 06:52 | |
| *** shyamb has joined #openstack-ansible | 06:52 | |
| *** dr_feelgood has quit IRC | 06:52 | |
| *** shyamb has quit IRC | 06:58 | |
| *** shyam89 has joined #openstack-ansible | 06:58 | |
| *** trident has quit IRC | 07:00 | |
| *** cshen has joined #openstack-ansible | 07:03 | |
| *** markvoelker has joined #openstack-ansible | 07:06 | |
| *** trident has joined #openstack-ansible | 07:10 | |
| *** markvoelker has quit IRC | 07:15 | |
| openstackgerrit | Mikael Loaec proposed openstack/openstack-ansible-ceph_client master: WIP: Importing keyrings from files rather than from mons https://review.opendev.org/671762 | 07:17 |
| *** ivve has joined #openstack-ansible | 07:17 | |
| evrardjp | csmart: maybe this would help https://docs.openstack.org/openstack-ansible/latest/user/test/example.html ? | 07:17 |
| *** macz has joined #openstack-ansible | 07:17 | |
| evrardjp | I have trouble helping because I don't know how your network is designed and what you want to achieve :) | 07:17 |
| openstackgerrit | Mikael Loaec proposed openstack/openstack-ansible-ceph_client master: WIP: Importing keyrings from files rather than from mons https://review.opendev.org/671762 | 07:18 |
| *** macz has quit IRC | 07:22 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/openstack-ansible master: Imported Translations from Zanata https://review.opendev.org/678479 | 07:27 |
| *** threestrands has quit IRC | 07:32 | |
| *** markvoelker has joined #openstack-ansible | 07:35 | |
| *** markvoelker has quit IRC | 07:40 | |
| *** miloa has joined #openstack-ansible | 07:42 | |
| *** shyamb has joined #openstack-ansible | 07:42 | |
| miloa | Morning | 07:42 |
| miloa | jrosser and evrardjp: if you can check https://review.opendev.org/#/c/671762/ to tell me if I am going on the right direction. Thanks :) | 07:42 |
| *** shyam89 has quit IRC | 07:43 | |
| *** shyamb has quit IRC | 07:50 | |
| evrardjp | arxcruz: are you there? | 07:55 |
| evrardjp | I am thinking of reducing tech debt in the os_tempest role by solely relying on python-tempestconf. However that means we need to package it appropriately, and make sure all the features we are looking for are there | 07:56 |
| chkumar|rover | evrardjp: Hello | 07:57 |
| chkumar|rover | evrardjp: sounds like a good idea | 07:57 |
| chkumar|rover | for relying on os_tempest | 07:58 |
| evrardjp | chkumar|rover: could you explain how the resource creation can be customized with tempestconf? Maybe explain how the discover mechanism works too? | 07:58 |
| evrardjp | I suppose this browses endpoints on the internal interface of a cloud, but not sure | 07:58 |
| evrardjp | (because users might not be authorized) | 07:59 |
| chkumar|rover | evrardjp: tempestconf support both using rc file as well as clouds.yaml | 07:59 |
| chkumar|rover | resource creation is only happen when the user is admin or have the permission | 08:00 |
| chkumar|rover | if it is non-admin there is a flag --non-admin which will just discover the resources and sets the value | 08:00 |
| evrardjp | but non-admin might not have access to discovery? | 08:01 |
| evrardjp | How does discovery works? | 08:01 |
| evrardjp | In a cloud I have access here, as a non-admin, I cannot list endpoints directly, but I can still run the cloud just fine. I could theoretically run tempest there, as I don't need admin to create things like networks/volumes/computes | 08:02 |
| chkumar|rover | evrardjp: under the hood, it uses tempest.lib and use the creds from rc/cloud.yaml file to discover the resources | 08:04 |
| evrardjp | In other words, I understand the admin case for getting details of endpoints through an internal interface, but anything else could be a user thing | 08:04 |
| csmart | evrardjp: thanks, yep read through those :-) this is just an AIO... all in one so single VM to test... I thought it used to work with single interface | 08:04 |
| chkumar|rover | in admin case, it first check for resources if not available then try to create it and if creation is denied it will flag warning messages | 08:05 |
| evrardjp | chkumar|rover: mmm, in that case, if it's just wrapping tempest.lib, why do we have this CLI in the first place, vs having those discovery options directly in tempest? | 08:06 |
| evrardjp | sorry to be a pain, but I like to understand things before committing :) | 08:06 |
| evrardjp | csmart: it can work in a single interface | 08:06 |
| evrardjp | sorry i thought you were NOT doing an AIO :) | 08:07 |
| evrardjp | csmart: so what's the issue? | 08:07 |
| csmart | lol no I AM doing an AIO ;-) | 08:07 |
| evrardjp | you followed the AIO guide and it didn't work? | 08:07 |
| csmart | yep | 08:07 |
| evrardjp | did you compare with what we are doing in gates? | 08:07 |
| csmart | basically, ansible task fails to ping a network (details above, I can find it again) | 08:07 |
| csmart | when I connect to the container, it has custom routes that are expecting eth1 | 08:07 |
| csmart | haven't checked gate yet, I thought it would be simple :-D | 08:08 |
| evrardjp | it must be | 08:08 |
| openstackgerrit | Merged openstack/openstack-ansible master: Imported Translations from Zanata https://review.opendev.org/678479 | 08:08 |
| evrardjp | that's why I am confused, I didn't think this could be a problem :) | 08:08 |
| chkumar|rover | tempest is written to work on tempest.conf, and it is totally depend on cloud deployer how they want to test their cloud and based on that they can set the value, conf generation was never part of that | 08:08 |
| evrardjp | csmart: you can try the gate-check-commit script on a different machine, and compare? | 08:08 |
| csmart | ansible task in aio1_utility_container failed: TASK [os_tempest : Ping router ip address] | 08:09 |
| evrardjp | chkumar|rover: I didn't say otherwise -- but it might be a nice feature to add to tempest directly | 08:09 |
| evrardjp | tempest discover | 08:09 |
| csmart | yeah sure, I'll take a look | 08:10 |
| csmart | as reference, this is what I see in the container http://paste.openstack.org/show/765333/ | 08:10 |
| chkumar|rover | evrardjp: https://etherpad.openstack.org/p/TempestConfigDesignSession | 08:10 |
| chkumar|rover | it was a topic from stein ptg | 08:10 |
| chkumar|rover | it tries to answer those questions | 08:11 |
| csmart | this route is strange to me, becasue there is no eth1 device in an AIO: | 08:11 |
| csmart | "172.29.236.0/22 dev eth1 proto kernel scope link src 172.29.236.59" | 08:11 |
| csmart | anyway, home time, I'll pick this up later, thanks!! | 08:11 |
| evrardjp | chkumar|rover: oh good | 08:12 |
| chkumar|rover | tempestconf has support for most of the services also | 08:13 |
| chkumar|rover | if anything missing there, can we added soon | 08:13 |
| chkumar|rover | we can also manipulate (add/remove) the tempest.conf file using the tempestconf tool itself + accounts.yaml generation and it is also integrated with refstack-client to run refstack tests | 08:14 |
| chkumar|rover | https://opendev.org/openstack/refstack-client/src/branch/master/specs/queens/approved/tempest_autoconfig.rst | 08:14 |
| chkumar|rover | more info is here also https://etherpad.openstack.org/p/tempest-conf-plugin-discovery-process | 08:15 |
| evrardjp | I see | 08:16 |
| evrardjp | that makes me sad and happy at the same time :) | 08:17 |
| evrardjp | csmart: try the gate-check-commit and compare, I think you found a bug | 08:17 |
| *** shyamb has joined #openstack-ansible | 08:37 | |
| *** markvoelker has joined #openstack-ansible | 08:40 | |
| chkumar|rover | evrardjp: :-) | 08:40 |
| jrosser | csmart: there is an eth1 inside the container - thats what you are seeing there | 08:40 |
| jrosser | tempest runs inside the utility container, so the "ping the router" task is an early bail out to just abort if there is something wrong with the networking that means it's pointless to even try running the tempest tests | 08:41 |
| *** markvoelker has quit IRC | 08:45 | |
| jrosser | evrardjp: could do with some more insight on this https://review.opendev.org/#/c/673150/ | 08:45 |
| *** macz has joined #openstack-ansible | 08:53 | |
| arxcruz | evrardjp: yes | 08:54 |
| arxcruz | evrardjp: I believe chkumar|rover already cover what you asked :) sorry, my daughter stay at home today, it's a nightmare :D | 08:57 |
| *** macz has quit IRC | 08:58 | |
| arxcruz | evrardjp: btw https://review.opendev.org/#/c/677925/ | 08:59 |
| evrardjp | done | 09:02 |
| evrardjp | for arxcruz, will check jrosser | 09:02 |
| arxcruz | thanks | 09:03 |
| evrardjp | jrosser: you are checking something from 9 weeks ago | 09:06 |
| openstackgerrit | Jean-Philippe Evrard proposed openstack/openstack-ansible-os_neutron stable/rocky: DNM Test jobs https://review.opendev.org/678761 | 09:07 |
| evrardjp | as you can see in this one ^ there is no ovn running (if you check in zuul) | 09:07 |
| evrardjp | oh wait you do depends on in your patch | 09:08 |
| evrardjp | between two branches on the same repo | 09:09 |
| evrardjp | that's not going to cut it | 09:09 |
| *** CeeMac has joined #openstack-ansible | 09:12 | |
| *** zbr has joined #openstack-ansible | 09:13 | |
| *** macz has joined #openstack-ansible | 09:14 | |
| openstackgerrit | Jean-Philippe Evrard proposed openstack/openstack-ansible-os_neutron stable/rocky: Remove incorrect job definitions https://review.opendev.org/678763 | 09:17 |
| evrardjp | jrosser: do you mind if I change your patch? | 09:18 |
| openstackgerrit | Jean-Philippe Evrard proposed openstack/openstack-ansible-os_neutron stable/stein: Swap special CI jobs from xenial to bionic https://review.opendev.org/673150 | 09:19 |
| jrosser | evrardjp: please do :) | 09:19 |
| *** macz has quit IRC | 09:19 | |
| evrardjp | noonedeadpunk: jrosser if you could vote on https://review.opendev.org/678763 that would be awesomne | 09:20 |
| jrosser | I even less understand why the ovn jobs are needed on rocky but it makes zuul sad if they’re missing | 09:20 |
| evrardjp | yhaha | 09:21 |
| evrardjp | so | 09:21 |
| evrardjp | I am no zuul expert, but let me explain how I think this works | 09:21 |
| evrardjp | basically zuuls maps the _whole_ list of jobs in all branches for all projects in a tenant | 09:21 |
| evrardjp | when doing in a branch different than master, it just adds a branch matcher | 09:22 |
| *** shyam89 has joined #openstack-ansible | 09:22 | |
| evrardjp | so when you removed the job definition in stein, but the job was still used in rocky, then you broke the general state of the config | 09:22 |
| *** shyamb has quit IRC | 09:23 | |
| evrardjp | if you remove the incorrect definition in rocky (and merge it), then you could change the jobs, because they won't be used elsewhere | 09:23 |
| evrardjp | The depends-on here wouldn't work, because I think zuul would need to check out both branches of the same repo to figure out state | 09:24 |
| evrardjp | I guess this could be implemented by having a DAG of the dependency, and checking the zuul definition in order, but that's probably more complex than just watching the state of one repo | 09:25 |
| evrardjp | but I am no expert, as I said | 09:25 |
| evrardjp | jrosser: this whole convo is for you ^ :) | 09:25 |
| openstackgerrit | Mikael Loaec proposed openstack/openstack-ansible-ceph_client master: WIP: Importing keyrings from files rather than from mons https://review.opendev.org/671762 | 09:27 |
| *** shyam89 has quit IRC | 09:34 | |
| jrosser | evrardjp: thankyou :) and thanks for he patch! | 09:40 |
| *** _nwonknu has quit IRC | 09:41 | |
| *** nwonknu has joined #openstack-ansible | 09:59 | |
| noonedeadpunk | mornings | 10:12 |
| noonedeadpunk | wow, there was so cool discussion that I've missed | 10:17 |
| *** macz has joined #openstack-ansible | 10:20 | |
| *** macz has quit IRC | 10:24 | |
| *** shyam89 has joined #openstack-ansible | 10:27 | |
| *** macz has joined #openstack-ansible | 10:40 | |
| csmart | evrardjp: gate-check-commit isn't doing anything different, so must be me.... thanks | 10:42 |
| csmart | jrosser: ok thanks, I did think I looked in the container and didn't see eth1 but I must be mistaken, I'll take another look when I rebuild, thanks | 10:43 |
| csmart | I'm just running this in a kvm guest on my laptop, maybe something went wrong with a suspend or something, I'll try again and see | 10:43 |
| csmart | thanks guys | 10:43 |
| *** macz has quit IRC | 10:45 | |
| openstackgerrit | Merged openstack/openstack-ansible-os_tempest master: Add provider_physical_network option to private network https://review.opendev.org/677925 | 10:47 |
| *** macz has joined #openstack-ansible | 11:01 | |
| *** nicolasbock has joined #openstack-ansible | 11:02 | |
| *** udesale has quit IRC | 11:02 | |
| *** fghaas has joined #openstack-ansible | 11:03 | |
| *** macz has quit IRC | 11:06 | |
| csmart | yep there is an eth1 in the container, the routes still look weird but I'll carry on! thanks | 11:11 |
| noonedeadpunk | evrardjp: so, I've got problems regarding ocal linters check since it requires our included roles and plugins to be present for venv, which actually results in run_tests.sh and cloning of openstack-ansible-tests | 11:14 |
| *** cshen has quit IRC | 11:15 | |
| noonedeadpunk | which returns back to the same point | 11:16 |
| *** hamzy has joined #openstack-ansible | 11:18 | |
| miloa | in the file roles/ceph_client/tasks/ceph_get_mon_host.yml it uses wait_for_connection to check if the monitors are up. wait_for_connection uses ansible ping module, which check ssh port. But the default monitor port number is 6789. | 11:21 |
| miloa | Would it be better to check with a wait_for on default monitors port ? | 11:22 |
| *** macz has joined #openstack-ansible | 11:22 | |
| noonedeadpunk | miloa: I'd say it's the default one | 11:22 |
| noonedeadpunk | https://docs.ceph.com/docs/mimic/rados/configuration/network-config-ref/#monitor-ip-tables | 11:23 |
| noonedeadpunk | agh | 11:23 |
| miloa | noonedeadpunk: yes it is the default. | 11:23 |
| * noonedeadpunk should read the questions better | 11:24 | |
| miloa | :D | 11:24 |
| noonedeadpunk | So, since we're going to reach ceph monitor via ssh, I guess we're checking it before trying to enter | 11:25 |
| miloa | because sometimes you do not have ssh access to the monitor from the osa-deployer. | 11:25 |
| noonedeadpunk | I guess that's exactly why we're checking it | 11:25 |
| noonedeadpunk | And you may have even more cases, when deploy host can't have access to the storage network | 11:26 |
| noonedeadpunk | So port 6789 won't be reachable as well | 11:26 |
| *** macz has quit IRC | 11:27 | |
| miloa | i would understand if it is the role use for mon install but it is the ceph_client | 11:27 |
| noonedeadpunk | So if we can't reach SSH than we can't get ceph keys from the mon host | 11:27 |
| noonedeadpunk | I guess you're working on patch to change this | 11:28 |
| miloa | yes i know but i working on it : https://review.opendev.org/#/c/671762/ | 11:28 |
| miloa | because in our usecase we do not have ssh access to ceph cluster from deployer, neither from openstack hosts | 11:29 |
| noonedeadpunk | So I guess that you don't need this task if all data gathered from monitor is already present | 11:29 |
| *** cshen has joined #openstack-ansible | 11:31 | |
| noonedeadpunk | This patch is good even if there is possibility to ssh on ceph host. When I was running OSA for the first time, I was extremely cautious about providing osa ssh access to the ceph cluster, as it was already working and with production resources on it | 11:31 |
| openstackgerrit | Merged openstack/openstack-ansible-os_neutron stable/rocky: Remove incorrect job definitions https://review.opendev.org/678763 | 11:31 |
| noonedeadpunk | So I'd really wish to have smth like this that time | 11:31 |
| evrardjp | noonedeadpunk: yes, but we can implement our own thing using tox -e ansible-lint ... Do you want me to help there? | 11:34 |
| *** dave-mccowan has joined #openstack-ansible | 11:34 | |
| miloa | It's just that when we deploy, we have to wait for timeout of the monitor check used during cinder, nova and glance so it slow down a lot our deployment :) | 11:35 |
| noonedeadpunk | evrardjp: actualy I won't reject some help:) because I get this while trying to launch ansible-lint on the role folder http://paste.openstack.org/show/765493/ | 11:35 |
| noonedeadpunk | and "ansible-playbook --syntax-check" complains about missing python_venv_build (which is basically true) | 11:36 |
| miloa | noonedeadpunk: so if the purpose was only to check if the ceph monitor service is running and accesible, perhaps using the default port would be enough ? | 11:37 |
| noonedeadpunk | miloa: I guess the purpose was to check if we're able to connect to the monitor hosts via ssh to gather data from it. If we're talking about this https://opendev.org/openstack/openstack-ansible-ceph_client/src/branch/master/tasks/ceph_get_mon_host.yml#L18 | 11:39 |
| noonedeadpunk | So checking 6789 is not really good, since it will break bunch of deployments, where deployment host don't have access to storage network | 11:40 |
| noonedeadpunk | evrardjp: so I guess that we need to create ansible-lint test for integrated repo | 11:41 |
| noonedeadpunk | to get all ansible plugins installed and roles present | 11:42 |
| miloa | noonedeadpunk: thanks understood :) so perhaps a sort of conditional check if ceph_overrides and ceph_keyring dir are defined or not. | 11:45 |
| noonedeadpunk | miloa: so in case of providing ceph config and keys in file, you don't actually need running ceph_get_mon_host.yml | 11:45 |
| noonedeadpunk | as well as any task delegated to ceph_mon_host afterwards | 11:47 |
| miloa | noonedeadpunk: ok, if the patch i am working on work, i would look at this part (no monitor check in case of ceph key and config provided by files) after :) many thanks. | 11:49 |
| *** shyam89 has quit IRC | 11:50 | |
| *** shyam89 has joined #openstack-ansible | 11:55 | |
| *** ansmith has quit IRC | 11:57 | |
| *** aedc has joined #openstack-ansible | 12:07 | |
| *** markvoelker has joined #openstack-ansible | 12:10 | |
| *** shyam89 has quit IRC | 12:13 | |
| noonedeadpunk | evrardjp: I think it's worth to update supported suse versions in meta for yours goodbye_42.3 | 12:15 |
| evrardjp | we are not removing support of 42.3 for those branches | 12:19 |
| evrardjp | noonedeadpunk: ^ | 12:20 |
| evrardjp | we are merely not testing it because it's unmaintained. Ppl are still allowed to do crazy stuff | 12:20 |
| noonedeadpunk | ok, that's fair:) | 12:20 |
| evrardjp | I hesitated between the two tbh. Then I realised that this would already take quite a while | 12:22 |
| noonedeadpunk | yep, you're right, that it's probably not worth spended time on it... | 12:23 |
| evrardjp | I am thinking maybe a follow up | 12:23 |
| noonedeadpunk | just meta doesn't mention suse 15, but it's quite another story | 12:23 |
| evrardjp | but there is a gap where we basically don't have anything | 12:24 |
| evrardjp | it's like pseudo working | 12:24 |
| evrardjp | hahah | 12:24 |
| *** ansmith has joined #openstack-ansible | 12:38 | |
| CeeMac | afternoon | 12:40 |
| *** cshen has quit IRC | 12:45 | |
| openstackgerrit | Merged openstack/openstack-ansible-tests stable/rocky: Remove OpenSUSE Leap 42.3 from role jobs templates https://review.opendev.org/677947 | 12:58 |
| *** cshen has joined #openstack-ansible | 13:02 | |
| *** yolanda has quit IRC | 13:06 | |
| *** yolanda__ has joined #openstack-ansible | 13:06 | |
| *** macz has joined #openstack-ansible | 13:10 | |
| *** KeithMnemonic has joined #openstack-ansible | 13:14 | |
| *** macz has quit IRC | 13:15 | |
| openstackgerrit | Merged openstack/openstack-ansible-nspawn_container_create stable/rocky: Remove openSUSE 42.3 job https://review.opendev.org/677985 | 13:22 |
| openstackgerrit | Merged openstack/openstack-ansible-nspawn_hosts stable/rocky: Remove openSUSE 42.3 job https://review.opendev.org/677984 | 13:24 |
| openstackgerrit | Merged openstack/openstack-ansible-galera_client stable/rocky: Remove openSUSE 42.3 job https://review.opendev.org/677989 | 13:27 |
| openstackgerrit | Merged openstack/openstack-ansible-lxc_hosts stable/rocky: Remove openSUSE 42.3 job https://review.opendev.org/677988 | 13:29 |
| openstackgerrit | Merged openstack/openstack-ansible-galera_server stable/rocky: Remove openSUSE 42.3 job https://review.opendev.org/677132 | 13:31 |
| *** udesale has joined #openstack-ansible | 13:32 | |
| openstackgerrit | Merged openstack/openstack-ansible-plugins stable/rocky: Remove openSUSE 42.3 job https://review.opendev.org/677962 | 13:34 |
| openstackgerrit | Dmitriy Rabotyagov (noonedeadpunk) proposed openstack/openstack-ansible master: [docs] Be more specific about pip command https://review.opendev.org/678839 | 13:36 |
| openstackgerrit | Merged openstack/openstack-ansible-os_ceilometer stable/rocky: Remove openSUSE 42.3 job https://review.opendev.org/677983 | 13:47 |
| *** macz has joined #openstack-ansible | 13:48 | |
| *** macz has quit IRC | 13:52 | |
| openstackgerrit | Merged openstack/openstack-ansible-os_horizon stable/rocky: Remove openSUSE 42.3 job https://review.opendev.org/677958 | 13:53 |
| openstackgerrit | Merged openstack/openstack-ansible-os_glance stable/rocky: Remove openSUSE 42.3 job https://review.opendev.org/677978 | 13:55 |
| openstackgerrit | Merged openstack/openstack-ansible-os_aodh stable/pike: Remove openSUSE 42.3 job https://review.opendev.org/677130 | 13:56 |
| openstackgerrit | Merged openstack/openstack-ansible-os_designate stable/rocky: Dropping support of 42.3 https://review.opendev.org/677951 | 13:57 |
| openstackgerrit | Merged openstack/openstack-ansible-os_aodh stable/rocky: Drop openSUSE 42.3 job https://review.opendev.org/677438 | 13:58 |
| openstackgerrit | Merged openstack/openstack-ansible-os_panko stable/pike: Remove openSUSE 42.3 job https://review.opendev.org/677129 | 14:02 |
| openstackgerrit | Merged openstack/openstack-ansible-lxc_container_create stable/rocky: Remove openSUSE 42.3 job https://review.opendev.org/677987 | 14:02 |
| openstackgerrit | Merged openstack/openstack-ansible-os_cinder stable/rocky: Remove openSUSE 42.3 job https://review.opendev.org/677982 | 14:04 |
| openstackgerrit | Merged openstack/openstack-ansible-os_neutron stable/rocky: Remove openSUSE 42.3 job https://review.opendev.org/677973 | 14:05 |
| openstackgerrit | Merged openstack/openstack-ansible-os_swift stable/rocky: Remove openSUSE 42.3 job https://review.opendev.org/677965 | 14:20 |
| *** udesale has quit IRC | 14:32 | |
| *** spsurya has joined #openstack-ansible | 14:35 | |
| openstackgerrit | Merged openstack/openstack-ansible-os_neutron stable/stein: Swap special CI jobs from xenial to bionic https://review.opendev.org/673150 | 14:40 |
| *** aedc has quit IRC | 14:40 | |
| jrosser | Could folks have a look at this https://review.opendev.org/#/c/678308/ | 14:43 |
| jrosser | I want to know if that should be on master first even though there are no lxd jobs there any more | 14:44 |
| noonedeadpunk | jrosser: I'd probably not place that for master... | 14:46 |
| jrosser | ok, if it looks ok we need that to merge to unwedge os_neutron on any branches with lxd jobs | 14:48 |
| openstackgerrit | kourosh vivan proposed openstack/openstack-ansible-os_octavia master: Octavia communicate with barbican using internal https://review.opendev.org/678871 | 14:49 |
| openstackgerrit | Merged openstack/openstack-ansible-os_horizon stable/rocky: Remove openSUSE 42.3 job https://review.opendev.org/677975 | 15:09 |
| *** cshen has quit IRC | 15:11 | |
| openstackgerrit | Mikael Loaec proposed openstack/openstack-ansible-ceph_client master: WIP: Importing keyrings from files rather than from mons https://review.opendev.org/671762 | 15:14 |
| openstackgerrit | Merged openstack/openstack-ansible-os_keystone stable/rocky: Remove openSUSE 42.3 job https://review.opendev.org/677974 | 15:25 |
| jrosser | jmccrory: seems https://review.opendev.org/#/c/673677/ is needed, it made the ansible 2.8.3 patch work but it's own tests are failing..... | 15:26 |
| *** miloa has quit IRC | 15:28 | |
| *** chkumar|rover is now known as raukadah | 15:31 | |
| *** gyee has joined #openstack-ansible | 15:37 | |
| *** macz has joined #openstack-ansible | 15:42 | |
| *** tiffanie has joined #openstack-ansible | 15:53 | |
| *** tiffanie has left #openstack-ansible | 15:55 | |
| *** tiffanie has joined #openstack-ansible | 15:55 | |
| tiffanie | #startmeeting openstack_ansible_meeting | 16:00 |
| openstack | Meeting started Tue Aug 27 16:00:06 2019 UTC and is due to finish in 60 minutes. The chair is tiffanie. Information about MeetBot at http://wiki.debian.org/MeetBot. | 16:00 |
| openstack | Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. | 16:00 |
| *** openstack changes topic to " (Meeting topic: openstack_ansible_meeting)" | 16:00 | |
| openstack | The meeting name has been set to 'openstack_ansible_meeting' | 16:00 |
| tiffanie | #topic office hours | 16:00 |
| *** openstack changes topic to "office hours (Meeting topic: openstack_ansible_meeting)" | 16:00 | |
| noonedeadpunk | o/ | 16:00 |
| mnaser | henlo | 16:01 |
| guilhermesp | o/ | 16:01 |
| * mnaser hasn't been around as much, been busy doing magnum-y things | 16:01 | |
| guilhermesp | we have murano in wip process though https://review.opendev.org/#/c/677830/ | 16:02 |
| noonedeadpunk | So, https://review.opendev.org/#/c/676410/ is basicaly ready and there are bunch of patches that already use that role (like https://review.opendev.org/#/c/678216/) | 16:03 |
| noonedeadpunk | guilhermesp: I guess I've left another bunch of comments there | 16:03 |
| openstackgerrit | Merged openstack/openstack-ansible-os_panko stable/queens: Remove openSUSE 42.3 job https://review.opendev.org/677949 | 16:03 |
| guilhermesp | also I've been seeing discussions about the POST_FAILRES for rocky branch | 16:03 |
| guilhermesp | yeah noonedeadpunk I will take a look in a while | 16:03 |
| evrardjp | o/ | 16:04 |
| noonedeadpunk | yeah, rocky post failures has been fixed thanks to jrosser and infra folks | 16:04 |
| jrosser | Hopefully post failures are sorted out now, team effort there fixed it | 16:05 |
| guilhermesp | that's wonderful | 16:05 |
| jrosser | Ah snap :) | 16:05 |
| noonedeadpunk | but I have some "issue" with linters check for uwsgi role | 16:05 |
| noonedeadpunk | So we used to check linters with script from tests repo https://opendev.org/openstack/openstack-ansible-tests/src/branch/master/test-ansible-lint.sh | 16:06 |
| noonedeadpunk | Since we're migrating to the integrated tests, necessity of usage of tests repo reduces. And this makes possible to drop tests folder in repos, run_tests.sh and more things | 16:07 |
| noonedeadpunk | So if we want to continue doing ansible-lint checks, we should probably move it to the main repo | 16:10 |
| *** jawad_axd has joined #openstack-ansible | 16:10 | |
| noonedeadpunk | What I was thinking about is to add another action for gate-check-commit.sh, like check, to launch all linters with it | 16:10 |
| noonedeadpunk | But right now I don't know how to limit it to the exact role, since scenario is not designed right now for passing role names as is | 16:11 |
| noonedeadpunk | or playbook name, like os-nova-install.yml | 16:12 |
| evrardjp | the alternative would be to implement it directly with ansible-lint from tox? | 16:13 |
| evrardjp | It's a little repeat ourselves, but that doesn't sound like a bad idea, does it? | 16:14 |
| noonedeadpunk | but as I said, we will be missing plugins, ike config_template | 16:14 |
| noonedeadpunk | and this returns us back to usage of openstack-ansible-tests | 16:14 |
| evrardjp | isn't that working with dependencies? | 16:14 |
| evrardjp | yeah let's not head that way | 16:15 |
| noonedeadpunk | so yes, than we should carry dependencies, install them, etc - so run_tests.sh actually does the same... | 16:15 |
| jrosser | If we run an integrated test, all the things we need must be present, what do I miss? | 16:16 |
| noonedeadpunk | jrosser: you're right, and that's the thing I was thinking about. But in this case we should somehow provide what should be tested by linters | 16:17 |
| noonedeadpunk | ofc we may provide setup-everything.yml for the check | 16:17 |
| jrosser | The scenario is converted into a list of enabled services iirc | 16:17 |
| noonedeadpunk | But I'd say that number of found problems would be enormous | 16:18 |
| noonedeadpunk | yeah.. but it's not correlated neither with the name of playbooks nor with the name of the roles. | 16:18 |
| *** dr_feelgood has joined #openstack-ansible | 16:18 | |
| noonedeadpunk | ofc, we can probably generate scenario in other way when action is check | 16:19 |
| *** dr_feelgood has quit IRC | 16:19 | |
| evrardjp | sorry let me clarify: | 16:19 |
| noonedeadpunk | but I'm not sure that I'm leading the right way | 16:20 |
| evrardjp | I meant that the roles, per se, are not reusable because they don't contain dependencies in meta/main.yml. And that's something we can nowadays fix. Maybe by doing that, ansible-lint would be smart enough and work (but I am not sure there) | 16:20 |
| noonedeadpunk | I guess it may work only for dependant roles but not plugins/modules | 16:21 |
| evrardjp | The other way would be to implement another job, that ensures all the right repos are present in zuul, and for which the parent folder is a role folder in an ansible configuration. This would allow to test a role (current patchset folder) with a complete view of other roles | 16:22 |
| *** ivve has quit IRC | 16:22 | |
| evrardjp | dependencies of roles shipping plugins work inside ansible. Maybe not in ansible-lint, but in ansible it totally works. But that's a feature gap that could be fixable | 16:23 |
| evrardjp | I am not saying we should go for that absolutely | 16:23 |
| evrardjp | I am just saying we should avoid rewiring things in tests now that we are getting away with it | 16:24 |
| evrardjp | after that, it's up to the person implementing things. I am happy to help PoC things if you want | 16:24 |
| evrardjp | and help you figure that out | 16:24 |
| mnaser | i'm just wondering what is the path to success to at least get us uwsgi refactored out | 16:25 |
| mnaser | rather than block out all that work to fix a long standing issue with linting | 16:25 |
| evrardjp | mnaser: you're right there. Maybe we can just merge it without linting, and implement linting on top? | 16:26 |
| mnaser | i think indirectly it is being linted through roles that consume it | 16:26 |
| evrardjp | yeah that too | 16:26 |
| evrardjp | so that kinda helps on the story to not care about linting right now | 16:27 |
| noonedeadpunk | So using another job which will prepare things sounds like a moving things from tests repo to each role, which might extend the maintenance costs | 16:27 |
| mnaser | i think long term, we should move towards using collections | 16:27 |
| noonedeadpunk | But if dependencies inside meta really works, that would be fantastic | 16:27 |
| evrardjp | noonedeadpunk: it can be inside one repo, or inside the main repo | 16:27 |
| evrardjp | The idea with that new job, is that it simply applies to all repos without issue. | 16:28 |
| evrardjp | (even outside OSA I mean) | 16:28 |
| evrardjp | for our tripleO folks :) | 16:29 |
| evrardjp | anyway | 16:29 |
| evrardjp | I can check the dependency real quick today | 16:29 |
| noonedeadpunk | it would be great tbh | 16:29 |
| jrosser | For bind-to-mgmt I am stuck on rabbit and galera - maybe we need to look at what other install tools do there, not had time to progress that | 16:33 |
| jrosser | I also have Octavia on my todo list but also lack of time - the stable branches are all in a bad way due to changes in the amphora test image which are now only usable on master | 16:35 |
| jrosser | I think we need some new code to build our own images in test jobs now | 16:36 |
| guilhermesp | oh... I was about to test octavia in stable/stein... good to know I may face bad things in the course of testing it on metal | 16:36 |
| guilhermesp | maybe I could try to help with it | 16:36 |
| jrosser | You can deploy it but it won’t work with the image we lost in defaults/main.yml | 16:37 |
| *** dr_feelgood has joined #openstack-ansible | 16:37 | |
| guilhermesp | oh dear... | 16:37 |
| jrosser | If you can write some tasks to build the image that would be great, I can dig out some notes later | 16:37 |
| noonedeadpunk | btw, I'm worried that neutron role is broken as well since it doesn't pass calico test https://review.opendev.org/#/c/677311/ | 16:37 |
| noonedeadpunk | probably we should ask logan- to check it if he can? | 16:38 |
| johnsom | Ping me in #openstack-lbaas if you have questions about the image builds. | 16:38 |
| guilhermesp | great johnsom thanks! | 16:39 |
| jawad_axd | Please help with this playbook error http://paste.openstack.org/show/765664/. Ceph integration works fine with glance and nova. For cinder, it creates container for cinder_volume on ceph node but that container seems to have no internet access. Any suggestions? | 16:46 |
| jawad_axd | Using Stein. | 16:47 |
| tiffanie | #endmeeting | 17:00 |
| *** openstack changes topic to "Launchpad: https://launchpad.net/openstack-ansible || Weekly Meetings: https://wiki.openstack.org/wiki/Meetings/openstack-ansible || Review Dashboard: http://bit.ly/2xA1eZC" | 17:00 | |
| openstack | Meeting ended Tue Aug 27 17:00:15 2019 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 17:00 |
| openstack | Minutes: http://eavesdrop.openstack.org/meetings/openstack_ansible_meeting/2019/openstack_ansible_meeting.2019-08-27-16.00.html | 17:00 |
| openstack | Minutes (text): http://eavesdrop.openstack.org/meetings/openstack_ansible_meeting/2019/openstack_ansible_meeting.2019-08-27-16.00.txt | 17:00 |
| openstack | Log: http://eavesdrop.openstack.org/meetings/openstack_ansible_meeting/2019/openstack_ansible_meeting.2019-08-27-16.00.log.html | 17:00 |
| *** jawad_axd has quit IRC | 17:03 | |
| evrardjp | noonedeadpunk: I have news for you | 17:08 |
| evrardjp | :) | 17:08 |
| evrardjp | good and bad news | 17:08 |
| noonedeadpunk | that was really fast:) | 17:08 |
| evrardjp | the bad news is that ansible-lint isn't smart enough to work out the dependency | 17:08 |
| evrardjp | however the good news is that if you add that into dependency, and point to the role path, that works | 17:09 |
| noonedeadpunk | can you give an example?:) | 17:09 |
| evrardjp | ofc | 17:09 |
| evrardjp | :) | 17:09 |
| *** dr_feelgood has quit IRC | 17:18 | |
| openstackgerrit | Jean-Philippe Evrard proposed openstack/ansible-config_template master: Use galaxy_tags instead of categories https://review.opendev.org/678914 | 17:23 |
| evrardjp | noonedeadpunk: ^ | 17:23 |
| *** dr_feelgood has joined #openstack-ansible | 17:23 | |
| evrardjp | first part | 17:24 |
| noonedeadpunk | yeah, we need it in EVERY role | 17:24 |
| noonedeadpunk | I'd excluded 701,702,204 for now | 17:25 |
| *** ivve has joined #openstack-ansible | 17:25 | |
| evrardjp | mnaser: do you remember by heart what the env var is for the folder where zuul clones the projects required by a job ? | 17:25 |
| noonedeadpunk | wow, that would be extremelly helpfull | 17:25 |
| *** ivve has quit IRC | 17:26 | |
| *** ivve has joined #openstack-ansible | 17:27 | |
| evrardjp | mmm I think there are just ansible vars, darn | 17:28 |
| evrardjp | noonedeadpunk: in the meantime two things: | 17:31 |
| evrardjp | 1) the role is not in the notify list of this channel, we should change this | 17:31 |
| evrardjp | 2) https://review.opendev.org/678917 | 17:32 |
| noonedeadpunk | evrardjp: https://review.opendev.org/#/c/677738/ | 17:32 |
| evrardjp | awesome! | 17:32 |
| evrardjp | I have to go | 17:33 |
| evrardjp | my computer is swapping like hell, and I have other things to do in my life too :) | 17:33 |
| evrardjp | hahah | 17:33 |
| noonedeadpunk | yeah, sure, thanks for help | 17:33 |
| evrardjp | but it should get you a first idea on how to do this noonedeadpunk | 17:33 |
| *** dr_feelgood has quit IRC | 18:02 | |
| openstackgerrit | Guilherme Steinmuller Pimentel proposed openstack/openstack-ansible-os_murano master: Initial commit to os_murano role https://review.opendev.org/677830 | 18:37 |
| guilhermesp | noonedeadpunk: comments addressed :) ^ | 18:38 |
| *** markvoelker has quit IRC | 19:07 | |
| *** markvoelker has joined #openstack-ansible | 19:08 | |
| *** dr_feelgood has joined #openstack-ansible | 20:03 | |
| *** dr_feelgood has quit IRC | 20:08 | |
| *** prometheanfire has quit IRC | 20:08 | |
| *** prometheanfire has joined #openstack-ansible | 20:09 | |
| *** weshay is now known as weshay_pto | 20:11 | |
| *** dr_feelgood has joined #openstack-ansible | 20:37 | |
| *** ansmith has quit IRC | 20:41 | |
| *** dr_feelgood has quit IRC | 20:43 | |
| jrosser | mnaser: some of your nova notifications patches are blocked on this https://review.opendev.org/#/c/678308/ | 20:59 |
| *** spsurya has quit IRC | 21:07 | |
| *** ansmith has joined #openstack-ansible | 21:15 | |
| *** markvoelker has quit IRC | 21:19 | |
| *** ivve has quit IRC | 21:25 | |
| csmart | jrosser: evrardjp sorry for the noise, I must have broken something, worked this time | 21:27 |
| csmart | thanks | 21:27 |
| *** fghaas has quit IRC | 21:43 | |
| *** trident has quit IRC | 22:05 | |
| *** trident has joined #openstack-ansible | 22:13 | |
| *** markvoelker has joined #openstack-ansible | 22:15 | |
| *** markvoelker has quit IRC | 22:20 | |
| *** markvoelker has joined #openstack-ansible | 23:15 | |
| *** markvoelker has quit IRC | 23:20 | |
| *** prometheanfire has quit IRC | 23:33 | |
| *** prometheanfire has joined #openstack-ansible | 23:33 | |
| *** macz has quit IRC | 23:36 | |
| *** markvoelker has joined #openstack-ansible | 23:55 | |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!