Tuesday, 2020-03-24

« Monday, 2020-03-23 Index Wednesday, 2020-03-25 »
*** spatel has joined #openstack-ansible-sig01:39
*** spatel has quit IRC01:44
*** evrardjp has quit IRC05:36
*** evrardjp has joined #openstack-ansible-sig05:36
*** spatel has joined #openstack-ansible-sig05:40
*** spatel has quit IRC05:46
*** gtema has joined #openstack-ansible-sig07:33
*** dtantsur|afk is now known as dtantsur08:47
*** sshnaidm|pto is now known as sshnaidm08:48
*** priteau has joined #openstack-ansible-sig08:56
*** gtema_ has joined #openstack-ansible-sig08:59
*** gtema has quit IRC09:02
*** spatel has joined #openstack-ansible-sig09:42
*** spatel has quit IRC09:47
*** priteau has quit IRC09:48
*** gtema has joined #openstack-ansible-sig10:12
*** gtema_ has quit IRC10:16
*** holser has joined #openstack-ansible-sig10:19
*** gshippey has joined #openstack-ansible-sig10:50
*** gtema has quit IRC10:54
*** gtema has joined #openstack-ansible-sig10:56
*** holser has quit IRC11:17
*** holser has joined #openstack-ansible-sig11:18
*** gtema has quit IRC12:18
*** gtema has joined #openstack-ansible-sig12:49
gundalowAnyone know why the OpenStack zuul job which runs on ansible/ansible PRs is failing on all PRs https://github.com/ansible/ansible/pull/68122#issuecomment-60276172012:54
gundalowI think given that ansible/ansible:devel doesn't have any OpenStack modules in anymore that this Zuul job can be removed from ansible/ansible12:55
*** gtema_ has joined #openstack-ansible-sig12:57
*** gtema_ has quit IRC12:59
*** gtema_ has joined #openstack-ansible-sig12:59
mordredgundalow: I agree. removing now13:00
*** gtema has quit IRC13:00
openstackgerritMark Chappell proposed openstack/ansible-collections-openstack master: New Module: os_keystone_mapping  https://review.opendev.org/71364313:02
openstackgerritMark Chappell proposed openstack/ansible-collections-openstack master: New module: os_keystone_mapping_info  https://review.opendev.org/71368613:02
openstackgerritMark Chappell proposed openstack/ansible-collections-openstack master: New Module: os_keystone_identity_provider  https://review.opendev.org/71463213:02
openstackgerritMark Chappell proposed openstack/ansible-collections-openstack master: New Module: os_keystone_identity_provider_info  https://review.opendev.org/71463313:02
*** gtema_ is now known as gtema13:05
sshnaidmhow did it trigger on these files at all13:45
mordredsshnaidm: I do not know. I want to look in to it with corvus when he's up13:51
mordredsshnaidm: it should not have13:51
sshnaidmmordred, should we remove "devel" branch here? https://opendev.org/openstack/project-config/src/branch/master/zuul.d/projects.yaml#L2913:51
mordredsshnaidm: yeah, probalby. I mean, it's not urgent, both of those jobs now have branch matchers, but it's maybe worth cleaning up13:57
mordredsshnaidm: (I already removed the jobs running on the devel branch this morning fwiw)13:57
*** gtema_ has joined #openstack-ansible-sig13:59
*** gtema has quit IRC14:02
sshnaidmmordred, ack, should be ok then14:12
gundalowI guess the question is, Do you want CI on the `stable-2.9` branch still?14:54
mordredyeah - I've got some patches up for that14:54
mordredgundalow: https://review.opendev.org/#/c/711471/ - but we're going to figure out why that job ran on non-openstack related change first14:55
gundalowSweet, thanks :)14:55
mordredgundalow: because it shouldn't have :)14:55
gundalowFYI the much smaller devel is open for business again14:55
dtantsurhey, is the meeting here and now?15:01
sshnaidm#startmeeting ansible-sig15:04
openstackMeeting started Tue Mar 24 15:04:57 2020 UTC and is due to finish in 60 minutes.  The chair is sshnaidm. Information about MeetBot at http://wiki.debian.org/MeetBot.15:04
openstackUseful Commands: #action #agreed #help #info #idea #link #topic #startvote.15:05
*** openstack changes topic to " (Meeting topic: ansible-sig)"15:05
openstackThe meeting name has been set to 'ansible_sig'15:05
sshnaidmsorry, my bad15:05
sshnaidmok, so who is still available for the meeting? :)15:05
gtema_me for 10%15:05
sshnaidm#link https://etherpad.openstack.org/p/openstack-ansible-modules15:06
sshnaidmagenda today ^15:06
sshnaidmdtantsur, mordred odyssey4me and all15:06
dtantsuro/15:06
sshnaidmfirstly I'd like to ask for reviews where it's possible: https://review.opendev.org/#/q/topic:collection-jobs+(status:open+OR+status:merged)15:08
sshnaidmseems like train and rocky on openstacksdk are broken15:08
sshnaidmpep8 failures and neutron tests15:08
dtantsurthere have been a lot of transient issues today15:09
dtantsurmake sure it's not them15:09
sshnaidmdtantsur, it's from a few days earlier15:09
sshnaidmbut I'll rebase patches, just in case it helps..15:09
sshnaidmwhen this is merged, I think we're good about coverage in SDK and Ansible branches15:10
sshnaidmnext jobs it's testing collections with Ansible 2.8, which will require a little trick, cause you can't build collection in 2.8, only can use it :)15:10
sshnaidmand job to run unittests that currently exist in collection, I think we ignored them so far15:11
sshnaidmhopefully it will encourage people to write them more..15:11
sshnaidmany comments so far?15:12
sshnaidmmoving on15:12
sshnaidm* Keep extracting common code to library and standartization of modules15:12
sshnaidmlike mordred has started with a general class for Openstack Ansible module15:13
odyssey4meo/15:13
sshnaidmodyssey4me, hey15:13
sshnaidmit's a way we need to go imho, to try pulling out everything Ansible specific from modules to library15:14
sshnaidmto make module writing easier for people unfamiliar with Ansible, modules, etc15:14
trembleAnsibleAWSModule makes things much easier for AWS modules.15:15
sshnaidmtremble, right, the same idea with OpenstackAnsible class15:15
dtantsurI agree, it's hugely important15:15
dtantsuresp if we see a growth of contributors with little ansible experience15:15
dtantsur(like me)15:15
sshnaidmdtantsur, and I'm afraid we won't see much if we don't do it :)15:16
dtantsurexactly15:16
sshnaidmI don't have specific steps here, so point to think and to experiment in patches till next meeting15:17
sshnaidmmaybe even a spec, not sure though if we need it in this stage15:17
sshnaidmok, I think we all agree on that, moving on15:18
sshnaidm*    Pushing collection15:19
sshnaidmfor that we need mordred, so skipping for now15:19
sshnaidm* Work on a new job from scratch for modules, trying to reduce the time.15:19
sshnaidmback to CI, I'm not so familiar with devstack jobs, but wonder if it's possible to make it shorter somehow..15:19
sshnaidmor maybe to set up something from scratch15:20
sshnaidmcurrently it's quite long and spamming a lot in logs15:20
sshnaidmmoving on15:20
sshnaidm* Work on a separate jobs for specific modules for more focused and optimal testing15:21
sshnaidmjust an idea to have module-specific jobs15:21
sshnaidmand reduce load, for example if change is in neutron, maybe to skip glance modules check15:21
sshnaidmand sub-question, how to test ironic modules15:22
sshnaidmdtantsur, ^^15:22
sshnaidmwe'll need real bm nodes for functional tests15:23
dtantsurnope15:23
sshnaidmis it possible on third-party only or we can do something in openstack infra15:23
dtantsurironic has a virtual CI15:23
dtantsurwe even have a job on openstacksdk15:23
dtantsurwe can either use our devstack plugin OR bifrost15:23
sshnaidmdtantsur, nodes as virtual bms?15:23
dtantsuryep15:23
sshnaidmdtantsur, so we can run introspection on them?15:24
dtantsuryes, we can15:24
dtantsuryou'll need to add the ironic-inspector devstack plugin to the mix15:24
sshnaidminteresting, that's good news15:24
dtantsurthis is without inspector: https://github.com/openstack/openstacksdk/blob/master/.zuul.yaml#L230-L27715:24
sshnaidmdtantsur, and openstacksdk-functional-devstack-ironic runs actually with virtual nodes?15:25
dtantsuryep, although I don't think we actually use them in the tests15:26
dtantsur(we use the fake driver in SDK tests, I think)15:26
sshnaidmdtantsur, ok, seems like we'll need to take it offline to some specific ironic modules talk15:26
dtantsuryeah15:27
sshnaidmI'm close to finish with BM modules for tripleo, so we'd better to start thinking adapting them for collection15:27
dtantsurindeed15:27
sshnaidmcool15:27
sshnaidm* Adding TripleO jobs and testing15:27
sshnaidmwell, this one is too early to talk about maybe, but also good to plan ahead15:28
sshnaidmas we use in tripleo Keystone modules, but it's taken from Ansible repo15:28
sshnaidmall because mordred doesn't push collection :D15:29
dtantsurthat mordred!15:29
sshnaidmyeah15:29
dtantsuranyway, this should be solved within a few weeks15:29
dtantsurthen we can just add jobs as usual: first non-voting, then consistently green, then maybe voting15:30
* dtantsur has reservations about tripleo jobs being consistently green15:30
sshnaidmdtantsur, yeah, that's the plan15:30
sshnaidmdtantsur, well, we have standalone jobs that should fit our goals here15:30
dtantsurideally, we'll need something with ironic15:31
dtantsurbut that can wait15:31
sshnaidmand 3-party OVB jobs for ironic15:31
dtantsurright15:31
sshnaidmbut they're loooong15:31
dtantsurwe can limit them to patches touching ironic15:31
sshnaidmyeah15:31
sshnaidmand it has pretty good coverage of ironic there15:32
dtantsurand we'll add bifrost jobs15:32
dtantsurthat is, when somebody actually updates bifrost to use the collections15:32
dtantsuri.e. when it's too late and we get broken15:32
sshnaidmI'll need to look more about bifrost15:32
dtantsursshnaidm: that's essentially where tripleo is moving :)15:33
sshnaidmdoes it use current os_ironic modules?15:33
dtantsurironic + ansible (optionally keystone)15:33
dtantsuryes15:33
sshnaidmdtantsur, oh, you mean novaless and metalsmith, etc?15:33
dtantsuron tripleo side - yes15:33
dtantsurbifrost has been like that since inception15:33
sshnaidmok, so I think Steve Baker is actually did it for tripleo15:34
dtantsuryep15:34
sshnaidmin Ussuri we have it by default iirc15:34
dtantsuractually, I think the ironic modules have been developed for bifrost15:34
dtantsurinitially15:34
dtantsuryeah15:34
sshnaidmso you mean in tripleo we use actually os_ironic* modules?15:34
dtantsurnope, in bifrost15:34
sshnaidmah, ok15:35
dtantsurI guess we should move on from the ironic topics :)15:35
sshnaidmdtantsur, yeah, so it seems like important to have these jobs on os_ironic..15:35
sshnaidmdtantsur, ok15:35
sshnaidmI think I finished with today agenda actually15:35
sshnaidmso now free talk if anyone has15:36
dtantsurI stayed till the end, do I receive a treat? :D15:36
sshnaidmdtantsur, of course15:36
dtantsur\o/15:36
sshnaidmfirst ironic ansible module goes to dtantsur !15:37
* dtantsur bites the first ironic ansible module15:37
sshnaidmdtantsur, don't scare him, will run away..15:38
sshnaidmok, so let's continue talks here if there are any15:38
sshnaidm#endmeeting15:38
*** openstack changes topic to "OpenStack Ansible SIG | https://etherpad.openstack.org/p/ansible-sig"15:38
openstackMeeting ended Tue Mar 24 15:38:30 2020 UTC.  Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4)15:38
openstackMinutes:        http://eavesdrop.openstack.org/meetings/ansible_sig/2020/ansible_sig.2020-03-24-15.04.html15:38
openstackMinutes (text): http://eavesdrop.openstack.org/meetings/ansible_sig/2020/ansible_sig.2020-03-24-15.04.txt15:38
openstackLog:            http://eavesdrop.openstack.org/meetings/ansible_sig/2020/ansible_sig.2020-03-24-15.04.log.html15:38
trembledtantsur, Speaking of Ironic...   https://903cda9ebd0e1994db88-a122e892f22a38ddc5f951fb4c0c5ba5.ssl.cf5.rackcdn.com/714120/2/gate/openstack-tox-lower-constraints/9e8f4d6/testr_results.html I don't see how I could have caused an additional failure there, especially since it was working...16:09
trembleOops, wrong channel...16:09
openstackgerritMerged openstack/ansible-collections-openstack master: Don't fetch subunit in jobs  https://review.opendev.org/71434216:16
*** dtantsur is now known as dtantsur|afk16:25
*** sshnaidm is now known as sshnaidm|afk17:13
*** gtema_ has quit IRC17:30
*** evrardjp has quit IRC17:36
*** evrardjp has joined #openstack-ansible-sig17:36
*** openstackstatus has quit IRC17:45
*** tremble has quit IRC17:45
*** dmsimard|off has quit IRC17:45
*** tremble has joined #openstack-ansible-sig17:45
*** openstackstatus has joined #openstack-ansible-sig17:46
*** ChanServ sets mode: +v openstackstatus17:46
*** holser__ has joined #openstack-ansible-sig19:17
*** holser has quit IRC19:18
*** holser has joined #openstack-ansible-sig19:32
*** holser__ has quit IRC19:34
mordredsshnaidm|afk, dtantsur|afk: sorry - I had a conflicting meeting this morning - I have a fun topic for you tomorrow though19:47
*** gshippey has quit IRC19:49
*** gtema has joined #openstack-ansible-sig20:12
*** gtema has quit IRC20:17
sshnaidm|afkintriguing :)20:28
*** holser has quit IRC21:18
*** holser has joined #openstack-ansible-sig21:19
*** holser has quit IRC21:36
*** gtema has joined #openstack-ansible-sig22:14
*** gtema has quit IRC22:18
openstackgerritMonty Taylor proposed openstack/ansible-collections-openstack master: Add ansible-collections-openstack to the required-projects  https://review.opendev.org/71478422:18
openstackgerritMonty Taylor proposed openstack/ansible-collections-openstack master: Update the badge shown in galaxy  https://review.opendev.org/71479523:32
« Monday, 2020-03-23 Index Wednesday, 2020-03-25 »

Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!