| *** TxGirlGeek has joined #openstack-ansible | 00:01 | |
| openstackgerrit | Kevin Carter (cloudnull) proposed openstack/openstack-ansible-plugins master: Correct severe performance problem https://review.openstack.org/581893 | 00:04 |
|---|---|---|
| openstackgerrit | Kevin Carter (cloudnull) proposed openstack/openstack-ansible-plugins master: Correct severe performance problem https://review.openstack.org/581893 | 00:06 |
| *** poul has joined #openstack-ansible | 00:10 | |
| antonym | didn't seem to pick up anything new | 00:15 |
| *** poul has quit IRC | 00:15 | |
| *** poul has joined #openstack-ansible | 00:28 | |
| *** poul has quit IRC | 00:32 | |
| *** sew has joined #openstack-ansible | 00:34 | |
| *** poul has joined #openstack-ansible | 00:46 | |
| *** poul has quit IRC | 00:51 | |
| *** chyka has joined #openstack-ansible | 00:51 | |
| *** TxGirlGeek has quit IRC | 00:54 | |
| *** komish has quit IRC | 00:54 | |
| *** chyka has quit IRC | 00:56 | |
| *** sew has quit IRC | 00:57 | |
| *** poul has joined #openstack-ansible | 01:04 | |
| *** poul has quit IRC | 01:08 | |
| *** poul has joined #openstack-ansible | 01:13 | |
| *** poul has quit IRC | 01:17 | |
| *** tux_ has joined #openstack-ansible | 01:22 | |
| openstackgerrit | Merged openstack/openstack-ansible-galera_server master: fix tox python3 overrides https://review.openstack.org/580012 | 01:49 |
| *** poul has joined #openstack-ansible | 02:07 | |
| *** poul has quit IRC | 02:11 | |
| *** poul has joined #openstack-ansible | 02:24 | |
| *** poul has quit IRC | 02:29 | |
| openstackgerrit | Merged openstack/openstack-ansible master: Be consistent with haproxy group name https://review.openstack.org/580359 | 02:37 |
| *** poul has joined #openstack-ansible | 02:42 | |
| openstackgerrit | Merged openstack/openstack-ansible-os_neutron master: fix tox python3 overrides https://review.openstack.org/580058 | 02:45 |
| *** sew has joined #openstack-ansible | 02:46 | |
| *** poul has quit IRC | 02:46 | |
| *** sew has quit IRC | 02:48 | |
| *** sew has joined #openstack-ansible | 02:49 | |
| *** nurdie has joined #openstack-ansible | 02:53 | |
| *** cjloader_ has quit IRC | 02:57 | |
| *** cjloader has joined #openstack-ansible | 02:57 | |
| *** nurdie has quit IRC | 02:58 | |
| *** poul has joined #openstack-ansible | 03:00 | |
| *** poul has quit IRC | 03:04 | |
| *** sew has quit IRC | 03:09 | |
| *** cjloader has quit IRC | 03:09 | |
| *** cjloader has joined #openstack-ansible | 03:12 | |
| *** FrankZhang has quit IRC | 03:13 | |
| *** cjloader has quit IRC | 03:17 | |
| cloudnull | antonym hard to say. maybe tomorrow we can hack on it some | 03:17 |
| *** spsurya_ has joined #openstack-ansible | 03:32 | |
| openstackgerrit | Kevin Carter (cloudnull) proposed openstack/openstack-ansible master: Restructure playbook serialization https://review.openstack.org/581942 | 03:45 |
| *** udesale has joined #openstack-ansible | 03:51 | |
| *** cjloader has joined #openstack-ansible | 03:52 | |
| *** poul has joined #openstack-ansible | 03:53 | |
| *** udesale has quit IRC | 03:55 | |
| *** udesale has joined #openstack-ansible | 03:55 | |
| *** nurdie has joined #openstack-ansible | 03:55 | |
| *** poul has quit IRC | 03:57 | |
| *** cjloader has quit IRC | 03:57 | |
| *** nurdie has quit IRC | 04:00 | |
| cloudnull | logan- mind giving https://review.openstack.org/#/c/581893/ a look aagain | 04:04 |
| cloudnull | passed the experimental tests | 04:05 |
| logan- | yeah, I'm trying to hammer on it locally but debugging other ansible 2.5 connection plugin issues to get it working :( i feel like i need to do some local tests on it before i can vote | 04:06 |
| *** poopcat has quit IRC | 04:07 | |
| cloudnull | ++ please do | 04:09 |
| *** poul has joined #openstack-ansible | 04:11 | |
| *** cjloader has joined #openstack-ansible | 04:12 | |
| *** poul has quit IRC | 04:15 | |
| *** cjloader has quit IRC | 04:17 | |
| *** chyka has joined #openstack-ansible | 04:18 | |
| *** chyka has quit IRC | 04:23 | |
| *** agrebennikov has joined #openstack-ansible | 04:26 | |
| *** poul has joined #openstack-ansible | 04:28 | |
| *** mattinator has quit IRC | 04:30 | |
| tux_ | cloudnull: https://bugs.launchpad.net/openstack-ansible/+bug/1781321 | 04:32 |
| openstack | Launchpad bug 1781321 in openstack-ansible "ceph-ansible error No first item, sequence was empty" [Undecided,New] | 04:32 |
| tux_ | i want to set mon_host in ceph-ansible | 04:33 |
| tux_ | look like its hardcoded to br-mgmt | 04:34 |
| *** poul has quit IRC | 04:34 | |
| *** nurdie has joined #openstack-ansible | 04:36 | |
| logan- | your ceph mons list is formatted wrong. use a space between - and the list item. ie '-172.16.200.211' should be '- 172.16.200.211' | 04:37 |
| *** nurdie has quit IRC | 04:41 | |
| tux_ | logan-: it was copy paste its, it is corrected already "- 172.16.200.211" | 04:43 |
| logan- | ah ok | 04:43 |
| tux_ | logan-: last 24 hour i am banging my head on this issue :( | 04:44 |
| tux_ | logan-: how other folks using this scenario in production? Even documentation is wrong in OSAD https://docs.openstack.org/openstack-ansible/latest/user/ceph/full-deploy.html | 04:45 |
| logan- | monitor_address_block: "172.16.200.0/24" doesnt seem to match the block where your mons are | 04:45 |
| logan- | mon host = 192.168.100.232,192.168.100.203,192.168.100.188 | 04:45 |
| tux_ | yes, i can see that so i did "ceph_mons" override but that didn't have any effect | 04:46 |
| tux_ | its ignoring "ceph_mons", look like its hardcoded somewhere so only taking br-mgmt ips | 04:47 |
| *** poul has joined #openstack-ansible | 04:47 | |
| *** cjloader has joined #openstack-ansible | 04:48 | |
| tux_ | logan-: are you using openstack-ansible with ceph deployment? what you have configure there? | 04:48 |
| logan- | can you pastebin the 'ip addr' output from one of your mon containers? | 04:49 |
| logan- | do the mons have an interface on 172.16.200.0/24? | 04:49 |
| tux_ | logan-: http://paste.openstack.org/show/725631/ | 04:49 |
| tux_ | logan-: mon has br-storage(172.16.200.0/24), br-mgmt network | 04:50 |
| tux_ | logan-: look at here http://paste.openstack.org/show/725632/ | 04:51 |
| *** poul has quit IRC | 04:52 | |
| logan- | 'lxc-attach -n ostack-controller-02_ceph-mon_container-b67c137c -- ip addr' | 04:52 |
| logan- | please pastebin | 04:52 |
| *** cjloader has quit IRC | 04:52 | |
| tux_ | logan-: http://paste.openstack.org/show/725633/ | 04:53 |
| tux_ | i can ping all mon 172.16.200.x IPs also OSD too | 04:54 |
| logan- | id suggest removing ceph_mons from your vars file, remove monitor_address_block, and add monitor_interface: eth2 | 04:54 |
| tux_ | i need to add monitor_address too, what should i set there? | 04:55 |
| logan- | no, you should not need it | 04:55 |
| tux_ | Ok running playbook lets see | 04:56 |
| logan- | "# You must define either monitor_interface, monitor_address or monitor_address_block." -- https://github.com/ceph/ceph-ansible/blob/9f54b3b4a7c1a7015def3a7987e3e9e426385251/group_vars/all.yml.sample#L346 | 04:57 |
| logan- | ceph-ansible supports several different options for determining the mon addresses, so it is a little confusing | 04:57 |
| logan- | i think the way you had monitor_address_block defined should have worked | 04:57 |
| logan- | i wonder if you have cached facts though | 04:57 |
| tux_ | i will delete all cached facts and will try that too | 04:58 |
| tux_ | logan-: FAILED! => {"failed": true, "msg": "'dict object' has no attribute u'ansible_eth2'"} | 04:58 |
| tux_ | got this error | 04:59 |
| logan- | rm -rf /etc/openstack_deploy/ansible_facts | 04:59 |
| logan- | then run again | 04:59 |
| tux_ | ok | 04:59 |
| *** mattinator has joined #openstack-ansible | 04:59 | |
| tux_ | I am just running " openstack-ansible ceph-install.yml" playbook | 05:00 |
| tux_ | FYI | 05:00 |
| *** chhagarw has joined #openstack-ansible | 05:01 | |
| logan- | 👍 | 05:01 |
| tux_ | logan-: hmmm look like it passed "generate ceph configuration file: ceph.conf]" | 05:04 |
| *** poul has joined #openstack-ansible | 05:04 | |
| tux_ | lets see... what next | 05:04 |
| tux_ | damn!!! mon host = 172.16.200.195,172.16.200.76,172.16.200.211 | 05:05 |
| tux_ | i can see change in file now :) | 05:05 |
| tux_ | logan-: damn it now i got this error http://paste.openstack.org/show/725634/ | 05:06 |
| tux_ | ceph-create-keys:ceph-mon is not in quorum: u'probing' | 05:07 |
| *** poul has quit IRC | 05:08 | |
| tux_ | should i stop container and restart them ? | 05:10 |
| logan- | I have to go to sleep, but not reaching quorum usually means your mons can't reach each other over the network. you'll have to do some network debugging.. check the mon logs, netstat and tcpdump if necessary to figure out why they aren't reaching quorum | 05:14 |
| tux_ | Ok i will spend some time... thanks for help... | 05:14 |
| logan- | if they were partially bootstrapped with the wrong ips before, you may need to delete the mon containers and re create them | 05:14 |
| tux_ | logan-: i may bug you again :) | 05:14 |
| tux_ | i will try to do that too... | 05:14 |
| tux_ | its 1:15AM night so i will do that tomorrow :) | 05:15 |
| tux_ | logan-: thank you man!!! it was great help though | 05:15 |
| logan- | np.. good luck tomrorow! | 05:15 |
| *** cjloader has joined #openstack-ansible | 05:19 | |
| *** poul has joined #openstack-ansible | 05:22 | |
| *** cjloader has quit IRC | 05:23 | |
| *** poul has quit IRC | 05:26 | |
| *** pmannidi has joined #openstack-ansible | 05:30 | |
| *** nurdie has joined #openstack-ansible | 05:31 | |
| *** poul has joined #openstack-ansible | 05:40 | |
| *** poul has quit IRC | 05:44 | |
| *** tux_ has quit IRC | 05:45 | |
| *** chhagarw has quit IRC | 05:54 | |
| *** chhagarw has joined #openstack-ansible | 05:54 | |
| *** tux_ has joined #openstack-ansible | 06:04 | |
| *** tux_ has quit IRC | 06:07 | |
| Taseer | evrardjp: Good Morning. All of my jobs are failing at the preliminary step: "clearing IP tables". Is this happening to other jobs ? | 06:12 |
| *** biffi has joined #openstack-ansible | 06:14 | |
| *** armaan has joined #openstack-ansible | 06:18 | |
| *** cjloader has joined #openstack-ansible | 06:20 | |
| *** cjloader has quit IRC | 06:24 | |
| *** smorant has joined #openstack-ansible | 06:25 | |
| openstackgerrit | Jesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-galera_server stable/queens: Retry yum/zypper package installs https://review.openstack.org/581961 | 06:34 |
| *** sanjayu_ has joined #openstack-ansible | 06:36 | |
| *** sanjayu_ is now known as saneax | 07:00 | |
| *** mattinator has quit IRC | 07:04 | |
| evrardjp | Taseer: I don't know | 07:04 |
| evrardjp | maybe a new issue | 07:04 |
| evrardjp | since we changed something in testing | 07:05 |
| *** epalper has joined #openstack-ansible | 07:05 | |
| *** poul has joined #openstack-ansible | 07:06 | |
| *** poul has joined #openstack-ansible | 07:07 | |
| Taseer | evrardjp: okay. so I'll wait maybe | 07:07 |
| evrardjp | Taseer: better is to find the root cause yourself and fix it :) | 07:10 |
| *** poul has quit IRC | 07:10 | |
| Taseer | evrardjp: sure :) | 07:10 |
| evrardjp | no I mean it :) | 07:10 |
| evrardjp | That's how you learn | 07:10 |
| odyssey4me | o/ | 07:12 |
| *** cjloader has joined #openstack-ansible | 07:12 | |
| *** gkadam has joined #openstack-ansible | 07:14 | |
| *** poul has joined #openstack-ansible | 07:15 | |
| Taseer | evrardjp: let me look into the matter | 07:16 |
| *** cjloader has quit IRC | 07:17 | |
| odyssey4me | evrardjp: worth a look if you have some time to figure it out: https://review.openstack.org/581893 | 07:19 |
| evrardjp | yeah our strategy and connection plugins are inefficient | 07:22 |
| evrardjp | I will have a look | 07:22 |
| *** radeks has joined #openstack-ansible | 07:22 | |
| evrardjp | I had a try with running mitogen | 07:22 |
| evrardjp | to be smarter at that | 07:22 |
| evrardjp | but I can check that too | 07:22 |
| odyssey4me | Shouldn't we try to upstream the remote ssh plugin structure? | 07:23 |
| odyssey4me | That way we can ditch our custom strategy and connection plugins. | 07:23 |
| jrosser | that issue feels pretty much the same as we had with 2.3(?) and use of environment: | 07:28 |
| jrosser | or rather, what it makes a sinle do looks very similar | 07:29 |
| jrosser | *ansible | 07:29 |
| evrardjp | odyssey4me: we should | 07:30 |
| evrardjp | the patch is great | 07:30 |
| evrardjp | I will review it when my laptop is back up | 07:30 |
| evrardjp | it is kinda "wet" for now :( | 07:30 |
| evrardjp | but the code is great. | 07:31 |
| evrardjp | I will just ask to replace the verbosity but except that, it's good | 07:31 |
| evrardjp | cloudnull: good find, very nice : ) | 07:33 |
| Dmitriy | guys, I'm about to write masakari-monitors installation for openstack-ansible-os_masakari role. Has someone already started this work? And did you plan to add masakari-monitors installation to this role? As it seems to be designed without monitors installation support... | 07:48 |
| *** Dmitriy is now known as noonedeadpunk | 07:50 | |
| *** noonedeadpunk is now known as Dmitriy | 07:50 | |
| *** Dmitriy has left #openstack-ansible | 07:50 | |
| *** Dmitriy has joined #openstack-ansible | 07:51 | |
| *** cjloader has joined #openstack-ansible | 07:52 | |
| *** chyka has joined #openstack-ansible | 07:54 | |
| *** cjloader has quit IRC | 07:57 | |
| *** chyka has quit IRC | 07:59 | |
| odyssey4me | looks like the nova-status check merge is breaking the integrated build, sometimes: http://logs.openstack.org/25/579625/2/gate/openstack-ansible-deploy-ceph-ubuntu-xenial/519d097/job-output.txt.gz#_2018-07-11_21_54_12_219403 | 07:59 |
| odyssey4me | it failed on 1 of 4 jobs | 08:00 |
| *** DanyC has joined #openstack-ansible | 08:00 | |
| odyssey4me | lemme ad in a retry/delay thing | 08:01 |
| *** suggestable has joined #openstack-ansible | 08:03 | |
| *** tosky has joined #openstack-ansible | 08:03 | |
| ebbex_ | odyssey4me: now that the ml2.ovs.dvr change is in master, how do I proceed with https://review.openstack.org/#/c/581041/ | 08:10 |
| odyssey4me | ebbex_: typically now it would be cherry-picked (with -x to show the master commit sha) | 08:12 |
| odyssey4me | this can be done from inside gerrit using the 'cherry pick' button, or via the git cli | 08:12 |
| *** cjloader has joined #openstack-ansible | 08:13 | |
| odyssey4me | the reason why it should merge in master first is that sometimes the implementation changes before it merges due to review feedback, and if the back port is done at the same time as the initial patch (not after the pick), then reviewers have to take a lot more care when looking to see if all the same feedback has been applied | 08:13 |
| odyssey4me | it's just far easier for everyone to get the first one merged, then do the backports | 08:14 |
| openstackgerrit | Jesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-os_nova master: Retry nova-status check and only run it once https://review.openstack.org/582087 | 08:15 |
| odyssey4me | more info here: https://docs.openstack.org/openstack-ansible/latest/contributor/contribute.html#backporting | 08:15 |
| openstackgerrit | Jesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-os_nova master: Retry nova-status check and only run it once https://review.openstack.org/582087 | 08:16 |
| odyssey4me | evrardjp hwoarang andymccr ^ that should hopefully solve the inconsistent fails we're seeing when running that check | 08:17 |
| *** cjloader has quit IRC | 08:18 | |
| *** nurdie has quit IRC | 08:27 | |
| *** saneax has quit IRC | 08:27 | |
| *** nurdie has joined #openstack-ansible | 08:28 | |
| ebbex_ | odyssey4me: sure that makes sense :) So in this case I sorta got lucky as there was no change to the commit. I had to do a small change to master because of an indentation fix. | 08:29 |
| ebbex_ | But unless that gets requested backported, the change can stay the same? | 08:29 |
| *** nurdie has quit IRC | 08:34 | |
| evrardjp | odyssey4me: do we store the config files of each service in the gate logs? We should IMO | 08:35 |
| odyssey4me | evrardjp: I think we do for role tests, but not the integrated build | 08:38 |
| odyssey4me | and yes, we should | 08:38 |
| evrardjp | I think https://github.com/openstack/openstack-ansible/commit/d3a0bfb728a75db4c3cc79d4604c8555a472dc6f broke memcached | 08:38 |
| openstackgerrit | Tuan Do Anh proposed openstack/openstack-ansible master: Update pypi url to new url https://review.openstack.org/582096 | 08:38 |
| evrardjp | building an AIO right now | 08:39 |
| odyssey4me | ebbex_: reviewers will be looking for the 'cherry-picked from ...' portion in the commit msg, or something in the commit message to explain why that's not there | 08:40 |
| odyssey4me | sometimes it's better to also port back anything the master patch depended on, but it's not always necessary | 08:40 |
| *** electrofelix has joined #openstack-ansible | 08:46 | |
| *** cjloader has joined #openstack-ansible | 08:46 | |
| ebbex_ | odyssey4me: So how would you reccomend proceeding in this case? Update the commit message with a 'cherry-picked from commit <hash>'? | 08:48 |
| ebbex_ | or abandon the changes and do new cherry-picks? | 08:49 |
| *** cjloader has quit IRC | 08:51 | |
| ebbex_ | This is the other commit btw, https://review.openstack.org/#/c/581351/ which I think is fine for master, but not really a bug-fix needed in stable branches. | 08:52 |
| *** harryhuang_ has joined #openstack-ansible | 08:56 | |
| *** cshen has joined #openstack-ansible | 08:57 | |
| openstackgerrit | Jean-Philippe Evrard proposed openstack/openstack-ansible master: Fix memcached_servers content https://review.openstack.org/582105 | 08:57 |
| odyssey4me | ebbex_: just update the commit message in this case I think | 08:58 |
| evrardjp | odyssey4me: and any core available: can we merge this ^ ? | 08:58 |
| *** tux_ has joined #openstack-ansible | 08:58 | |
| evrardjp | please don't make that wait | 09:00 |
| odyssey4me | evrardjp: I think to get that merged you're going to need https://review.openstack.org/582087 merged | 09:00 |
| evrardjp | ok I will merge then | 09:01 |
| odyssey4me | otherwise you'll get transient failures | 09:01 |
| evrardjp | because here the thing is plain broken | 09:01 |
| *** agrebennikov has quit IRC | 09:01 | |
| evrardjp | so we are not using memcached | 09:01 |
| evrardjp | which might lead to other issues. | 09:01 |
| evrardjp | ok merging it | 09:02 |
| openstackgerrit | Erik Berg proposed openstack/openstack-ansible-os_neutron stable/pike: Install openvswitch packages on 'ml2.ovs.dvr' configuration https://review.openstack.org/581041 | 09:02 |
| odyssey4me | evrardjp: asked a question in your review | 09:03 |
| openstackgerrit | Erik Berg proposed openstack/openstack-ansible-os_neutron stable/queens: Install openvswitch packages on 'ml2.ovs.dvr' configuration https://review.openstack.org/581040 | 09:04 |
| evrardjp | Answered. | 09:05 |
| ebbex_ | odyssey4me: Thanks for the info/tips :) | 09:06 |
| evrardjp | if you don't believe me, build an AIO. | 09:06 |
| evrardjp | or I can show you one. | 09:06 |
| *** suggestable_ has joined #openstack-ansible | 09:06 | |
| evrardjp | odyssey4me: answered more clearly | 09:09 |
| *** cshen has quit IRC | 09:10 | |
| *** suggestable has quit IRC | 09:10 | |
| odyssey4me | oh I see | 09:11 |
| odyssey4me | I do wish gertty would put the reply in-line below mine, instead of in the comment body | 09:11 |
| odyssey4me | it makes following replies to questions quite hard to follow | 09:11 |
| *** cjloader has joined #openstack-ansible | 09:13 | |
| evrardjp | that's because I am clicking on the "reply" button of gerrit. I should instead reply on your quote. | 09:14 |
| evrardjp | but laziness and fast answers. | 09:14 |
| evrardjp | to be honest that's one more reason I don't like using >- for readability reasons. | 09:15 |
| evrardjp | This has bit me long ago, and it still does it. | 09:15 |
| evrardjp | I hate those > | things | 09:16 |
| *** cjloader has quit IRC | 09:18 | |
| openstackgerrit | Jesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-os_cinder master: Use rsync instead of synchronize module for remote copy https://review.openstack.org/582123 | 09:24 |
| odyssey4me | evrardjp: ^ you did warn us... ;) cc mnaser | 09:25 |
| openstackgerrit | Jesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-os_cinder master: Use rsync instead of synchronize module for remote copy https://review.openstack.org/582123 | 09:26 |
| odyssey4me | oh crap, wrong way around | 09:26 |
| openstackgerrit | Jesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-os_cinder master: Use rsync instead of synchronize module for remote copy https://review.openstack.org/582123 | 09:27 |
| openstackgerrit | Jesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-os_cinder master: Enable CentOS 7 distro jobs https://review.openstack.org/579945 | 09:27 |
| evrardjp | odyssey4me: hahah | 09:29 |
| odyssey4me | tyvm ebbex_ ! | 09:29 |
| evrardjp | I'd be happy if at some point I'd be listened to instead of just ignored :p "No everything will be fine" :) | 09:29 |
| evrardjp | haha it doesn't matter, I am glad you fixed things :) | 09:30 |
| odyssey4me | evrardjp: you approved it, but warned that it might cause trouble :p | 09:31 |
| odyssey4me | no-one ignored you, your note was duly noted :p | 09:31 |
| *** pmannidi has quit IRC | 09:36 | |
| evrardjp | well ... If I haven't approved it, it would probably still be pending review? : p | 09:37 |
| evrardjp | interestingly I think we are carrying useless code for at least 3 years | 09:38 |
| evrardjp | (not related) | 09:38 |
| evrardjp | odyssey4me: can you think of an use case where the line number 1 here is different than line number 2? | 09:39 |
| evrardjp | https://github.com/openstack/openstack-ansible-rabbitmq_server/blob/master/defaults/main.yml#L37-L38 | 09:39 |
| *** smorant has quit IRC | 09:39 | |
| *** cjloader has joined #openstack-ansible | 09:48 | |
| odyssey4me | evrardjp: there was a stage where ansible_hostname gave us a fqdn, which is why that's there - I think it was in the transition to mitaka when we did all those ssl-related changes | 09:51 |
| *** cjloader has quit IRC | 09:53 | |
| evrardjp | odyssey4me: okay because that's not supposed to happen anymore | 10:02 |
| evrardjp | ansible_hostname is a reserved word that should be a hostname | 10:02 |
| evrardjp | fqdns are in ansible_fqdn | 10:02 |
| evrardjp | so we've been populating these fiels with twice the same value | 10:03 |
| evrardjp | (ofc except if our inventory messes up with that) | 10:03 |
| *** tux_ has quit IRC | 10:04 | |
| odyssey4me | evrardjp: perhaps that should be using ansible_fqdn and ansible_hostname instead then | 10:05 |
| *** rodolof has joined #openstack-ansible | 10:09 | |
| *** Ankita has joined #openstack-ansible | 10:11 | |
| *** cjloader has joined #openstack-ansible | 10:12 | |
| evrardjp | odyssey4me: yeah I am fixing it | 10:15 |
| *** cjloader has quit IRC | 10:18 | |
| *** Ankita has quit IRC | 10:26 | |
| *** kstev has joined #openstack-ansible | 10:31 | |
| *** udesale has quit IRC | 10:35 | |
| *** cjloader has joined #openstack-ansible | 10:53 | |
| openstackgerrit | Merged openstack/openstack-ansible-galera_server master: Do not remove MariaDB-server https://review.openstack.org/578844 | 10:55 |
| *** cjloader has quit IRC | 10:58 | |
| *** cshen has joined #openstack-ansible | 11:11 | |
| *** chhagarw has quit IRC | 11:12 | |
| *** cjloader has joined #openstack-ansible | 11:12 | |
| *** cshen has quit IRC | 11:15 | |
| *** cjloader has quit IRC | 11:17 | |
| openstackgerrit | Jean-Philippe Evrard proposed openstack/openstack-ansible-rabbitmq_server master: Cleanup generation of rabbitmq server data https://review.openstack.org/582150 | 11:20 |
| Tahvok | cloudnull: can you please explain why this logic is done in groups_vars/all and not inside the glance role? https://github.com/openstack/openstack-ansible/commit/54874018ec48acbd459fab5e09b77994b0c177fc#diff-457a8ea254f42d8a83bdf0a328d95e46R49 | 11:29 |
| *** chyka has joined #openstack-ansible | 11:30 | |
| *** nurdie has joined #openstack-ansible | 11:30 | |
| *** ansmith has quit IRC | 11:32 | |
| *** mardim has quit IRC | 11:34 | |
| odyssey4me | Tahvok: O think mnaseris already working on a patch to move it into the glance role | 11:35 |
| odyssey4me | https://review.openstack.org/579224 | 11:35 |
| *** nurdie has quit IRC | 11:36 | |
| *** chyka has quit IRC | 11:36 | |
| Tahvok | nice | 11:40 |
| Tahvok | Thanks odyssey4me ! | 11:40 |
| *** kstev has quit IRC | 11:41 | |
| Tahvok | I'm just working on a patch to clean glance vars, and was wondering about it | 11:41 |
| *** eumel8 has joined #openstack-ansible | 11:42 | |
| Tahvok | There was a site to search cross-repo. Can someone give me the link please? | 11:44 |
| eumel8 | maybe here: http://codesearch.openstack.org/ | 11:46 |
| Tahvok | eumel8: that's it. Thanks! | 11:46 |
| Tahvok | Can someone also please plus this two? https://review.openstack.org/#/c/549291/ https://review.openstack.org/#/c/549294/ | 11:47 |
| *** armaan has quit IRC | 11:48 | |
| *** armaan has joined #openstack-ansible | 11:49 | |
| openstackgerrit | Frank Kloeker proposed openstack/openstack-ansible master: [doc] Clarify attendance of Core Reviewers at PTG & Summit https://review.openstack.org/582161 | 11:49 |
| eumel8 | ^^ also this. And now we have a German version of OSA docs :) | 11:50 |
| *** cjloader has joined #openstack-ansible | 11:50 | |
| odyssey4me | eumel8: oh really? | 11:51 |
| odyssey4me | evrardjp: Tahvok needs another reviewer other than me ;) | 11:52 |
| eumel8 | https://translate.openstack.org/webtrans/translate?project=openstack-ansible&iteration=master&localeId=de&locale=en-US | 11:53 |
| eumel8 | There is still an issue with the sync job so I think there will be nothing to merge. I will check the situation Friday morning when the periodic job is running again | 11:54 |
| *** cjloader has quit IRC | 11:54 | |
| *** udesale has joined #openstack-ansible | 11:57 | |
| eumel8 | and just to mention: OSA doc is really valuable. Very good job, guys! | 11:57 |
| odyssey4me | awesome, thanks eumel8! | 12:01 |
| *** tux_ has joined #openstack-ansible | 12:08 | |
| evrardjp | nice patches Tahvok | 12:12 |
| *** cjloader has joined #openstack-ansible | 12:12 | |
| evrardjp | wow thanks eumel8 | 12:14 |
| eumel8 | evrardjp: we're not on the worldcup anymore - so we have time ;) | 12:15 |
| *** tux_ has quit IRC | 12:16 | |
| guilhermesp | eumel8: hahaha | 12:17 |
| *** cjloader has quit IRC | 12:18 | |
| evrardjp | we still have one match to play, but nobody cares I guess :p | 12:21 |
| evrardjp | "we" -> Belgium | 12:21 |
| eumel8 | evrardjp: It's Saturday and it's glorious enough | 12:27 |
| *** cmart has joined #openstack-ansible | 12:31 | |
| evrardjp | :) | 12:31 |
| *** hachi has joined #openstack-ansible | 12:33 | |
| openstackgerrit | Jean-Philippe Evrard proposed openstack/openstack-ansible-os_nova master: Ensure python3 compatibility https://review.openstack.org/581767 | 12:34 |
| openstackgerrit | Jean-Philippe Evrard proposed openstack/openstack-ansible master: Update all SHAs for milestone 3 https://review.openstack.org/574006 | 12:36 |
| openstackgerrit | Merged openstack/openstack-ansible-os_nova master: Retry nova-status check and only run it once https://review.openstack.org/582087 | 12:37 |
| *** armaan has quit IRC | 12:41 | |
| *** armaan has joined #openstack-ansible | 12:41 | |
| *** vakuznet has joined #openstack-ansible | 12:42 | |
| openstackgerrit | Jesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-os_cinder master: Use rsync instead of synchronize module for remote copy https://review.openstack.org/582123 | 12:44 |
| openstackgerrit | Jesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-os_cinder master: Enable CentOS 7 distro jobs https://review.openstack.org/579945 | 12:44 |
| *** cjloader has joined #openstack-ansible | 12:48 | |
| *** cjloader has quit IRC | 12:54 | |
| *** sew has joined #openstack-ansible | 13:01 | |
| *** ansmith has joined #openstack-ansible | 13:02 | |
| *** cjloader has joined #openstack-ansible | 13:12 | |
| openstackgerrit | Markos Chandras (hwoarang) proposed openstack/openstack-ansible master: Add support for using distribution packages for OpenStack services https://review.openstack.org/579770 | 13:12 |
| *** throwsb1 has joined #openstack-ansible | 13:12 | |
| *** KeithMnemonic has joined #openstack-ansible | 13:14 | |
| *** udesale has quit IRC | 13:15 | |
| *** cjloader has quit IRC | 13:17 | |
| *** chhagarw has joined #openstack-ansible | 13:26 | |
| *** tux_ has joined #openstack-ansible | 13:31 | |
| *** nurdie has joined #openstack-ansible | 13:31 | |
| guilhermesp | to give a taste, here's one of our monasca dashboards https://imgur.com/a/mJvzc8e :D | 13:32 |
| *** tux__ has joined #openstack-ansible | 13:32 | |
| odyssey4me | guilhermesp: nice! | 13:34 |
| guilhermesp | odyssey4me: o/ | 13:35 |
| *** tux_ has quit IRC | 13:36 | |
| mgariepy | that's really cool ^^ | 13:36 |
| devx | does the command openstack-ansible lxc-containers-create.yml -e 'container_group=my-service" also create the entries in /etc/hosts? | 13:45 |
| *** jwitko has joined #openstack-ansible | 13:46 | |
| *** jwitko has quit IRC | 13:47 | |
| *** jwitko has joined #openstack-ansible | 13:48 | |
| openstackgerrit | Andy Smith proposed openstack/openstack-ansible-os_ceilometer master: Update to use oslo.messaging service for RPC and Notify https://review.openstack.org/579909 | 13:56 |
| vakuznet | I need guidance what to do when zuul failure is not releated to change https://review.openstack.org/#/c/581419 | 14:00 |
| *** armaan_ has joined #openstack-ansible | 14:01 | |
| *** armaan has quit IRC | 14:02 | |
| *** cshen has joined #openstack-ansible | 14:02 | |
| *** rodolof has quit IRC | 14:04 | |
| tomtom002 | Hello, I've installed Octavia in Queens and created the endpoints, I'm trying to configure the Horizon Load Balancers dashboard to work with Octavia, is that supposed to be working at this time? It gives me the following error: http://paste.openstack.org/show/725707/ when I go to use the dashboard. I have created the endpoints and was just wondering if there was some other configuration | 14:05 |
| tomtom002 | necessary for the Load Balancer dashboard to work with Octavia? | 14:05 |
| *** kstev has joined #openstack-ansible | 14:05 | |
| *** nurdie has quit IRC | 14:06 | |
| *** nurdie has joined #openstack-ansible | 14:07 | |
| johnsom | tomtom002 It looks like the endpoint configured might be wrong. I see "/api/lbaas/loadbalancers/" in your pastebin, but the normal/default path is "/v2.0/lbaas/loadbalancers" | 14:07 |
| *** kstev1 has joined #openstack-ansible | 14:07 | |
| johnsom | https://developer.openstack.org/api-ref/load-balancer/v2/index.html | 14:08 |
| openstackgerrit | Merged openstack/openstack-ansible master: Be consistent with memcached group name https://review.openstack.org/580394 | 14:08 |
| openstackgerrit | Merged openstack/openstack-ansible master: Remove not needed glance variables https://review.openstack.org/549294 | 14:09 |
| *** biffi has quit IRC | 14:09 | |
| tomtom002 | johnsom - ok thanks! I will definitely check it out. | 14:09 |
| *** parasitid has quit IRC | 14:10 | |
| cloudnull | mornings https://review.openstack.org/#/c/581893/ | 14:10 |
| *** kstev has quit IRC | 14:10 | |
| cloudnull | anyone want to give that a review ? | 14:11 |
| cloudnull | i'm keen to get that back into stable soon-ish | 14:11 |
| *** nurdie has quit IRC | 14:11 | |
| cloudnull | evrardjp anything i can help with on that memcached issue you mentioned? | 14:12 |
| *** tux__ has quit IRC | 14:12 | |
| *** hwoarang has quit IRC | 14:12 | |
| *** cjloader has joined #openstack-ansible | 14:12 | |
| *** jwitko has quit IRC | 14:13 | |
| *** jwitko has joined #openstack-ansible | 14:13 | |
| mgariepy | cloudnull, done | 14:14 |
| cloudnull | :) | 14:14 |
| *** cjloader_ has joined #openstack-ansible | 14:15 | |
| *** cjloader has quit IRC | 14:15 | |
| *** FrankZhang has joined #openstack-ansible | 14:18 | |
| evrardjp | cloudnull: which memcached issue? | 14:19 |
| cloudnull | evrardjp "I think https://github.com/openstack/openstack-ansible/commit/d3a0bfb728a75db4c3cc79d4604c8555a472dc6f broke memcached" ? | 14:19 |
| evrardjp | cloudnull: I have a patch already | 14:20 |
| cloudnull | ah cool! | 14:20 |
| evrardjp | I am surprised I haven't voted on https://review.openstack.org/#/c/581893/ | 14:20 |
| cloudnull | was just reading back scrollback | 14:20 |
| evrardjp | let's backport this | 14:20 |
| evrardjp | great patch btw | 14:20 |
| openstackgerrit | Marc Gariépy (mgariepy) proposed openstack/openstack-ansible-tests master: Use 'become' for destroy container playbook. https://review.openstack.org/582195 | 14:21 |
| evrardjp | I am tired of these stupid changes like | to become "is" . I will fix those warnings. | 14:21 |
| mgariepy | vakuznet, can you depend on ^^ to test it fixes your issue ? | 14:21 |
| *** parasitid has joined #openstack-ansible | 14:22 | |
| *** parasitid has quit IRC | 14:26 | |
| odyssey4me | evrardjp: oh dear, https://review.openstack.org/#/c/582105/ needs a rebase | 14:27 |
| odyssey4me | if you can get that done real quick, I'm sure cloudnull and I will revote :) | 14:27 |
| cloudnull | ++ | 14:28 |
| odyssey4me | mnaser: FYI the synchronize module sucks, this works better: https://review.openstack.org/582123 | 14:28 |
| *** radeks has quit IRC | 14:28 | |
| openstackgerrit | Jean-Philippe Evrard proposed openstack/openstack-ansible master: Fix memcached_servers content https://review.openstack.org/582105 | 14:28 |
| evrardjp | oh got this interface is stupid | 14:29 |
| FrankZhang | evrardjp: the commit message was updated, hope it looks better. https://review.openstack.org/#/c/579284/ | 14:29 |
| openstackgerrit | Marc Gariépy (mgariepy) proposed openstack/openstack-ansible-os_neutron stable/pike: move haproxy to neutron-metadata-agent https://review.openstack.org/581762 | 14:33 |
| openstackgerrit | Jean-Philippe Evrard proposed openstack/openstack-ansible master: Fix memcached_servers content https://review.openstack.org/582105 | 14:33 |
| evrardjp | odyssey4me: ^ | 14:33 |
| evrardjp | cloudnull: ^ | 14:34 |
| openstackgerrit | Marc Gariépy (mgariepy) proposed openstack/openstack-ansible-os_neutron stable/queens: move haproxy to neutron-metadata-agent https://review.openstack.org/581761 | 14:34 |
| openstackgerrit | Kevin Carter (cloudnull) proposed openstack/openstack-ansible-plugins stable/queens: Correct severe performance problem https://review.openstack.org/582199 | 14:35 |
| *** mugsie has quit IRC | 14:36 | |
| *** mugsie has joined #openstack-ansible | 14:36 | |
| *** mugsie has quit IRC | 14:36 | |
| *** mugsie has joined #openstack-ansible | 14:36 | |
| openstackgerrit | Kevin Carter (cloudnull) proposed openstack/openstack-ansible-plugins stable/pike: Correct severe performance problem https://review.openstack.org/582201 | 14:37 |
| guilhermesp | odyssey4me: good point, synchronize would be use if this goes on https://bugs.launchpad.net/openstack-ansible/+bug/1778098 | 14:38 |
| openstack | Launchpad bug 1778098 in openstack-ansible "os_horizon role fails if horizon_custom_themes is specified" [Low,Confirmed] - Assigned to Amy Marrich (amy-marrich) | 14:38 |
| cloudnull | evrardjp done | 14:38 |
| *** tux_ has joined #openstack-ansible | 14:40 | |
| openstackgerrit | Merged openstack/openstack-ansible-plugins master: Correct severe performance problem https://review.openstack.org/581893 | 14:41 |
| vakuznet | mgariepy, do you mean i should change is to |? | 14:41 |
| *** kstev1 has quit IRC | 14:42 | |
| jrosser | cloudnull: have you ever had to increase the max connections for memcached? | 14:43 |
| mgariepy | vakuznet, add : Depends-On I999aa55269348497154a8eb073c2ff663242af4e | 14:43 |
| mgariepy | in the commit msg of your patch | 14:43 |
| odyssey4me | cloudnull: any specific reason why https://review.openstack.org/#/q/Ic61364b304bd4cc614d3b945f06fc7297a65e440 is using a different change id to https://review.openstack.org/#/c/581893/ ? | 14:43 |
| *** parasitid has joined #openstack-ansible | 14:44 | |
| *** nurdie has joined #openstack-ansible | 14:46 | |
| openstackgerrit | Vadim Kuznetsov proposed openstack/openstack-ansible-os_barbican master: Add default roles used by Barbican https://review.openstack.org/581419 | 14:46 |
| cloudnull | jrosser i've never had to specifically touch that setting | 14:48 |
| *** TxGirlGeek has joined #openstack-ansible | 14:48 | |
| cloudnull | however mnaser was saying the other day that it should be driven a lot higher | 14:48 |
| cloudnull | odyssey4me I had to port the change back to queens , it wouldn't cherry-pick cleanly | 14:49 |
| cloudnull | let me get that sorted | 14:49 |
| jrosser | cloudnull: we're not doing anything spectacular and 1024 seems to be too small | 14:50 |
| cloudnull | ++ that's what mnaser was saying too | 14:50 |
| jrosser | but at least 50% of the connections are from the infra keystone containers | 14:50 |
| jrosser | which smells wrong | 14:50 |
| openstackgerrit | Kevin Carter (cloudnull) proposed openstack/openstack-ansible-plugins stable/queens: Correct severe performance problem https://review.openstack.org/582199 | 14:51 |
| openstackgerrit | Kevin Carter (cloudnull) proposed openstack/openstack-ansible-plugins stable/pike: Correct severe performance problem https://review.openstack.org/582201 | 14:51 |
| openstackgerrit | Kevin Carter (cloudnull) proposed openstack/openstack-ansible-plugins stable/ocata: Correct severe performance problem https://review.openstack.org/582204 | 14:53 |
| evrardjp | sorry for the spam | 14:54 |
| openstackgerrit | Jean-Philippe Evrard proposed openstack/openstack-ansible-os_designate master: Fix usage of "|" for tests https://review.openstack.org/582205 | 14:54 |
| openstackgerrit | Jean-Philippe Evrard proposed openstack/openstack-ansible-os_watcher master: Fix usage of "|" for tests https://review.openstack.org/582206 | 14:54 |
| openstackgerrit | Jean-Philippe Evrard proposed openstack/openstack-ansible-os_cloudkitty master: Fix usage of "|" for tests https://review.openstack.org/582207 | 14:54 |
| openstackgerrit | Jean-Philippe Evrard proposed openstack/openstack-ansible-os_zun master: Fix usage of "|" for tests https://review.openstack.org/582208 | 14:54 |
| openstackgerrit | Jean-Philippe Evrard proposed openstack/openstack-ansible-os_masakari master: Fix usage of "|" for tests https://review.openstack.org/582209 | 14:55 |
| openstackgerrit | Jean-Philippe Evrard proposed openstack/openstack-ansible-os_horizon master: Fix usage of "|" for tests https://review.openstack.org/582211 | 14:55 |
| openstackgerrit | Jean-Philippe Evrard proposed openstack/openstack-ansible-tests master: Fix usage of "|" for tests https://review.openstack.org/582212 | 14:55 |
| openstackgerrit | Jean-Philippe Evrard proposed openstack/openstack-ansible-galera_client master: Fix usage of "|" for tests https://review.openstack.org/582213 | 14:55 |
| openstackgerrit | Jean-Philippe Evrard proposed openstack/openstack-ansible-os_neutron master: Fix usage of "|" for tests https://review.openstack.org/582214 | 14:55 |
| openstackgerrit | Jean-Philippe Evrard proposed openstack/openstack-ansible-os_blazar master: Fix usage of "|" for tests https://review.openstack.org/582215 | 14:56 |
| openstackgerrit | Jean-Philippe Evrard proposed openstack/openstack-ansible-plugins master: Fix usage of "|" for tests https://review.openstack.org/582216 | 14:56 |
| openstackgerrit | Merged openstack/openstack-ansible stable/queens: Add example host confd file for barbican https://review.openstack.org/579280 | 14:56 |
| openstackgerrit | Jean-Philippe Evrard proposed openstack/openstack-ansible-os_glance master: Fix usage of "|" for tests https://review.openstack.org/582217 | 14:56 |
| openstackgerrit | Jean-Philippe Evrard proposed openstack/openstack-ansible-os_zaqar master: Fix usage of "|" for tests https://review.openstack.org/582219 | 14:56 |
| openstackgerrit | Jean-Philippe Evrard proposed openstack/ansible-role-python_venv_build master: Fix usage of "|" for tests https://review.openstack.org/582220 | 14:56 |
| openstackgerrit | Jean-Philippe Evrard proposed openstack/openstack-ansible-os_panko master: Fix usage of "|" for tests https://review.openstack.org/582221 | 14:57 |
| openstackgerrit | Jean-Philippe Evrard proposed openstack/openstack-ansible-rabbitmq_server master: Fix usage of "|" for tests https://review.openstack.org/582222 | 14:57 |
| openstackgerrit | Jean-Philippe Evrard proposed openstack/openstack-ansible-os_molteniron master: Fix usage of "|" for tests https://review.openstack.org/582223 | 14:57 |
| openstackgerrit | Jean-Philippe Evrard proposed openstack/openstack-ansible-os_tacker master: Fix usage of "|" for tests https://review.openstack.org/582224 | 14:57 |
| openstackgerrit | Jean-Philippe Evrard proposed openstack/openstack-ansible-os_sahara master: Fix usage of "|" for tests https://review.openstack.org/582225 | 14:57 |
| openstackgerrit | Jean-Philippe Evrard proposed openstack/openstack-ansible-ceph_client master: Fix usage of "|" for tests https://review.openstack.org/582226 | 14:58 |
| openstackgerrit | Jean-Philippe Evrard proposed openstack/openstack-ansible-os_monasca master: Fix usage of "|" for tests https://review.openstack.org/582227 | 14:58 |
| openstackgerrit | Jean-Philippe Evrard proposed openstack/ansible-hardening master: Fix usage of "|" for tests https://review.openstack.org/582228 | 14:58 |
| openstackgerrit | Jean-Philippe Evrard proposed openstack/openstack-ansible-os_monasca-agent master: Fix usage of "|" for tests https://review.openstack.org/582229 | 14:58 |
| *** kstev has joined #openstack-ansible | 14:58 | |
| openstackgerrit | Jean-Philippe Evrard proposed openstack/openstack-ansible-os_ironic master: Fix usage of "|" for tests https://review.openstack.org/582230 | 14:58 |
| openstackgerrit | Jean-Philippe Evrard proposed openstack/openstack-ansible-os_heat master: Fix usage of "|" for tests https://review.openstack.org/582231 | 14:58 |
| evrardjp | jrosser: what are you talking about? | 14:59 |
| openstackgerrit | Jean-Philippe Evrard proposed openstack/openstack-ansible-os_barbican master: Fix usage of "|" for tests https://review.openstack.org/582232 | 14:59 |
| openstackgerrit | Jean-Philippe Evrard proposed openstack/openstack-ansible-os_almanach master: Fix usage of "|" for tests https://review.openstack.org/582234 | 14:59 |
| jrosser | evrardjp: we are running out of connections to memcached | 14:59 |
| openstackgerrit | Jean-Philippe Evrard proposed openstack/openstack-ansible-os_aodh master: Fix usage of "|" for tests https://review.openstack.org/582235 | 14:59 |
| FrankZhang | evrardjp:is here right place to add a new scenario? Thanks https://github.com/openstack/openstack-ansible/blob/master/tests/vars/bootstrap-aio-vars.yml | 14:59 |
| tosky | evrardjp: oh, about ^^ , will the old syntax be removed? | 14:59 |
| openstackgerrit | Jean-Philippe Evrard proposed openstack/openstack-ansible-memcached_server master: Fix usage of "|" for tests https://review.openstack.org/582237 | 14:59 |
| evrardjp | tosky: yes. | 14:59 |
| odyssey4me | tosky: eventually, yes - but warnings suck | 15:00 |
| openstackgerrit | Jean-Philippe Evrard proposed openstack/ansible-config_template master: Fix usage of "|" for tests https://review.openstack.org/582238 | 15:00 |
| evrardjp | jrosser: interesting -- master or stable? | 15:00 |
| jrosser | queens | 15:00 |
| openstackgerrit | Jean-Philippe Evrard proposed openstack/openstack-ansible-os_gnocchi master: Fix usage of "|" for tests https://review.openstack.org/582239 | 15:00 |
| jrosser | the connections mostly look sensible | 15:00 |
| evrardjp | jrosser: ok. version? | 15:00 |
| openstackgerrit | Jean-Philippe Evrard proposed openstack/openstack-ansible-os_ceilometer master: Fix usage of "|" for tests https://review.openstack.org/582240 | 15:00 |
| openstackgerrit | Jean-Philippe Evrard proposed openstack/openstack-ansible-repo_build master: Fix usage of "|" for tests https://review.openstack.org/582241 | 15:00 |
| evrardjp | there was a change during queens cycle for memcached connections | 15:00 |
| jrosser | but the keystone containers stand out as they have ~170 connections to each memcached, per keystone container | 15:00 |
| openstackgerrit | Jean-Philippe Evrard proposed openstack/openstack-ansible-lxc_container_create master: Fix usage of "|" for tests https://review.openstack.org/582242 | 15:00 |
| jrosser | and that seems to be growing over time | 15:01 |
| openstackgerrit | Jean-Philippe Evrard proposed openstack/openstack-ansible-galera_server master: Fix usage of "|" for tests https://review.openstack.org/582243 | 15:01 |
| evrardjp | jrosser: We are not using the pooled backend. | 15:01 |
| evrardjp | that's worth discussing | 15:01 |
| openstackgerrit | Jean-Philippe Evrard proposed openstack/ansible-role-systemd_mount master: Fix usage of "|" for tests https://review.openstack.org/582244 | 15:01 |
| *** armaan_ has quit IRC | 15:01 | |
| openstackgerrit | Jean-Philippe Evrard proposed openstack/openstack-ansible-os_cinder master: Fix usage of "|" for tests https://review.openstack.org/582246 | 15:01 |
| evrardjp | FrankZhang: I think there are examples somewhere | 15:01 |
| evrardjp | let me find that for you | 15:01 |
| openstackgerrit | Jean-Philippe Evrard proposed openstack/openstack-ansible-os_nova master: Fix usage of "|" for tests https://review.openstack.org/582247 | 15:01 |
| FrankZhang | thanks | 15:01 |
| openstackgerrit | Jean-Philippe Evrard proposed openstack/openstack-ansible-repo_server master: Fix usage of "|" for tests https://review.openstack.org/582248 | 15:01 |
| jrosser | evrardjp: it's 17.0.4 | 15:02 |
| openstackgerrit | Jean-Philippe Evrard proposed openstack/openstack-ansible-os_searchlight master: Fix usage of "|" for tests https://review.openstack.org/582249 | 15:02 |
| openstackgerrit | Jean-Philippe Evrard proposed openstack/openstack-ansible-os_magnum master: Fix usage of "|" for tests https://review.openstack.org/582250 | 15:02 |
| openstackgerrit | Jean-Philippe Evrard proposed openstack/openstack-ansible-os_trove master: Fix usage of "|" for tests https://review.openstack.org/582251 | 15:02 |
| *** joseg has joined #openstack-ansible | 15:02 | |
| openstackgerrit | Jean-Philippe Evrard proposed openstack/openstack-ansible-pip_install master: Fix usage of "|" for tests https://review.openstack.org/582252 | 15:02 |
| openstackgerrit | Jean-Philippe Evrard proposed openstack/openstack-ansible-os_tempest master: Fix usage of "|" for tests https://review.openstack.org/582253 | 15:02 |
| openstackgerrit | Jean-Philippe Evrard proposed openstack/openstack-ansible-haproxy_server master: Fix usage of "|" for tests https://review.openstack.org/582254 | 15:03 |
| openstackgerrit | Jean-Philippe Evrard proposed openstack/openstack-ansible-lxc_hosts master: Fix usage of "|" for tests https://review.openstack.org/582255 | 15:03 |
| evrardjp | FrankZhang: https://review.openstack.org/#/c/503971/44 | 15:03 |
| evrardjp | this is just an example | 15:03 |
| openstackgerrit | Jean-Philippe Evrard proposed openstack/ansible-role-systemd_networkd master: Fix usage of "|" for tests https://review.openstack.org/582256 | 15:03 |
| openstackgerrit | Jean-Philippe Evrard proposed openstack/openstack-ansible-openstack_hosts master: Fix usage of "|" for tests https://review.openstack.org/582258 | 15:03 |
| evrardjp | jrosser: should be okay | 15:03 |
| openstackgerrit | Jean-Philippe Evrard proposed openstack/openstack-ansible-os_octavia master: Fix usage of "|" for tests https://review.openstack.org/582259 | 15:03 |
| FrankZhang | cool, let me take a look | 15:03 |
| evrardjp | FrankZhang: you have to adapt to your needs | 15:04 |
| openstackgerrit | Jean-Philippe Evrard proposed openstack/openstack-ansible-os_congress master: Fix usage of "|" for tests https://review.openstack.org/582260 | 15:04 |
| evrardjp | but yeah the vars you've pointed is a critical point : ) | 15:04 |
| openstackgerrit | Jean-Philippe Evrard proposed openstack/openstack-ansible master: Fix usage of "|" for tests https://review.openstack.org/582261 | 15:04 |
| *** rmart04 has joined #openstack-ansible | 15:04 | |
| openstackgerrit | Jean-Philippe Evrard proposed openstack/openstack-ansible-rsyslog_client master: Fix usage of "|" for tests https://review.openstack.org/582262 | 15:04 |
| jrosser | evrardjp: we're going to try to understand why keystone seems to be making an ever increasing number of connections | 15:04 |
| openstackgerrit | Jean-Philippe Evrard proposed openstack/openstack-ansible-os_rally master: Fix usage of "|" for tests https://review.openstack.org/582263 | 15:04 |
| openstackgerrit | Jean-Philippe Evrard proposed openstack/openstack-ansible-ops master: Fix usage of "|" for tests https://review.openstack.org/582264 | 15:05 |
| openstackgerrit | Jean-Philippe Evrard proposed openstack/openstack-ansible-os_swift master: Fix usage of "|" for tests https://review.openstack.org/582265 | 15:05 |
| openstackgerrit | Jean-Philippe Evrard proposed openstack/openstack-ansible-rsyslog_server master: Fix usage of "|" for tests https://review.openstack.org/582266 | 15:05 |
| mgariepy | evrardjp, is it gerrit or the is really 2 space before the is ? | 15:05 |
| openstackgerrit | Jean-Philippe Evrard proposed openstack/openstack-ansible-os_keystone master: Fix usage of "|" for tests https://review.openstack.org/582267 | 15:05 |
| openstackgerrit | Jean-Philippe Evrard proposed openstack/openstack-ansible-nspawn_hosts master: Fix usage of "|" for tests https://review.openstack.org/582268 | 15:05 |
| *** tailDASHeF has quit IRC | 15:06 | |
| *** tailDASHeF has joined #openstack-ansible | 15:07 | |
| cloudnull | it looks like some of them have two spaces | 15:12 |
| openstackgerrit | Logan V proposed openstack/openstack-ansible master: Move get-ansible-role-requirements to scripts https://review.openstack.org/582271 | 15:12 |
| openstackgerrit | Logan V proposed openstack/openstack-ansible master: Use an absolute path for the role requirements file https://review.openstack.org/582272 | 15:12 |
| openstackgerrit | Logan V proposed openstack/openstack-ansible master: Use ZUUL_SRC_PATH to pass the path to Zuul git sources https://review.openstack.org/582273 | 15:13 |
| *** tux_ has quit IRC | 15:15 | |
| *** rmart04_ has joined #openstack-ansible | 15:15 | |
| evrardjp | mgariepy: yeah some have two some have one | 15:15 |
| evrardjp | I didn't bother making the difference in my script | 15:15 |
| evrardjp | if we want we can eval them, case by case | 15:16 |
| evrardjp | (we should anyway! | 15:16 |
| *** rmart04_ has left #openstack-ansible | 15:16 | |
| evrardjp | ) | 15:16 |
| evrardjp | I don't really care enough tbh I just wanted those anoying things to go away | 15:16 |
| *** rmart04 has quit IRC | 15:17 | |
| openstackgerrit | Kevin Carter (cloudnull) proposed openstack/openstack-ansible master: Add networkd prefix when creating networks https://review.openstack.org/571898 | 15:17 |
| evrardjp | I don't want to change the is bool though, that's so unclear to me | 15:18 |
| openstackgerrit | Jean-Philippe Evrard proposed openstack/openstack-ansible-os_blazar master: Fix usage of "|" for tests https://review.openstack.org/582215 | 15:18 |
| evrardjp | mgariepy: if that's what you meant with "the is really 2 space before the is" | 15:19 |
| openstackgerrit | Jean-Philippe Evrard proposed openstack/openstack-ansible-nspawn_hosts master: Fix usage of "|" for tests https://review.openstack.org/582268 | 15:21 |
| *** chhagarw has quit IRC | 15:21 | |
| openstackgerrit | Logan V proposed openstack/openstack-ansible master: Use an absolute path for the role requirements file https://review.openstack.org/582272 | 15:22 |
| mgariepy | evrardjp, yep i was really meaning the double space in the line. but it's not a really big deal i guess. | 15:24 |
| evrardjp | mgariepy: I hope that ppl will feel encouraged to edit those | 15:27 |
| evrardjp | :) | 15:27 |
| evrardjp | I did my part | 15:27 |
| evrardjp | :p | 15:27 |
| *** gillesMo has joined #openstack-ansible | 15:29 | |
| *** gyee has joined #openstack-ansible | 15:30 | |
| *** gyee has quit IRC | 15:34 | |
| openstackgerrit | Merged openstack/openstack-ansible-tests master: Do not use glance servers variable https://review.openstack.org/549291 | 15:43 |
| *** TxGirlGeek has quit IRC | 15:47 | |
| *** gkadam has quit IRC | 15:47 | |
| *** agrebennikov has joined #openstack-ansible | 15:48 | |
| *** exodusftw has quit IRC | 15:56 | |
| *** exodusftw has joined #openstack-ansible | 15:57 | |
| openstackgerrit | Andy McCrae proposed openstack/openstack-ansible master: Use --version-sort instead of -n for role version bump https://review.openstack.org/582291 | 16:01 |
| *** chhagarw has joined #openstack-ansible | 16:02 | |
| *** hamzy_ has quit IRC | 16:04 | |
| *** hamzy_ has joined #openstack-ansible | 16:04 | |
| *** poul has quit IRC | 16:06 | |
| *** gyee has joined #openstack-ansible | 16:08 | |
| devx | how can I force the creation/update of mysql user? | 16:12 |
| openstackgerrit | Frank Zhang proposed openstack/openstack-ansible master: Add aio host conf for barbican role https://review.openstack.org/579284 | 16:16 |
| *** hwoarang has joined #openstack-ansible | 16:17 | |
| *** hwoarang has quit IRC | 16:17 | |
| *** hwoarang has joined #openstack-ansible | 16:17 | |
| *** hwoarang has quit IRC | 16:18 | |
| cloudnull | devx which mysql user? | 16:18 |
| devx | let's say a custom user? | 16:19 |
| devx | cloudnull: is there a difference? | 16:19 |
| cloudnull | we create independent mysql users for various databases. | 16:20 |
| cloudnull | like nova has several DBs with different users/pws. | 16:20 |
| devx | right | 16:20 |
| devx | I'm creatign a user but in my testing I droped the user and was hoping that re-running the playbook it would recreate the user | 16:21 |
| *** armaan has joined #openstack-ansible | 16:22 | |
| devx | but i guess this `{"changed": false, "skip_reason": "Conditional result was False"}` would prevent it from recreating it | 16:22 |
| devx | i also update the user thinking it would detect the difference and no joy | 16:22 |
| devx | I'm sure it's a Layer 8 problem | 16:23 |
| *** tux_ has joined #openstack-ansible | 16:23 | |
| *** shardy has quit IRC | 16:25 | |
| *** spsurya_ has quit IRC | 16:26 | |
| cloudnull | is there a "when" in the task creating that user? | 16:26 |
| *** suggestable_ has quit IRC | 16:27 | |
| devx | yes but is set to always.. | 16:28 |
| devx | cloudnull: https://github.com/devx/openstack-ansible-ops/blob/osquery-install/osquery/installKolideFleet.yml | 16:28 |
| cloudnull | ah change that to tags: | 16:30 |
| cloudnull | when is like a task limiter | 16:31 |
| devx | ok, let me try that | 16:32 |
| openstackgerrit | Jesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-os_magnum master: Execute service setup against a delegated host using Ansible built-in modules https://review.openstack.org/582296 | 16:32 |
| *** TxGirlGeek has joined #openstack-ansible | 16:33 | |
| openstackgerrit | Logan V proposed openstack/openstack-ansible master: Use ZUUL_SRC_PATH to pass the path to Zuul git sources https://review.openstack.org/582273 | 16:34 |
| *** TxGirlGeek has quit IRC | 16:35 | |
| openstackgerrit | Jesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-os_masakari master: Execute service setup against a delegated host using Ansible built-in modules https://review.openstack.org/582300 | 16:42 |
| *** hachi has quit IRC | 16:44 | |
| logan- | jmccrory: you got a sec? I have a question about this line https://github.com/openstack/openstack-ansible-plugins/commit/4a262efbb4445c252c667b513c24d1e87ed03c35#diff-3043ae1dfa14b68868a8b96510af02a3R402 | 16:52 |
| logan- | in my inventory, self.container_name == self.host | 16:52 |
| logan- | so it breaks container detection | 16:52 |
| logan- | so trying to understand why the line is added there and also why the tests pass since https://github.com/openstack/openstack-ansible-plugins/blob/master/tests/group_vars/all_containers.yml#L16 | 16:53 |
| *** suggestable_ has joined #openstack-ansible | 16:55 | |
| jmccrory | hmm I think that was for delegating to a container where the container_name isn't in the inventory, https://github.com/openstack/openstack-ansible-plugins/blob/master/tests/test-connection-plugin.yml#L123-L132 | 16:56 |
| jmccrory | that line might not be the right way to handle it though | 16:56 |
| *** cshen has quit IRC | 16:59 | |
| *** sew has quit IRC | 16:59 | |
| openstackgerrit | Kevin Carter (cloudnull) proposed openstack/openstack-ansible master: Remove inventory path for group/host_vars https://review.openstack.org/581561 | 17:00 |
| openstackgerrit | Kevin Carter (cloudnull) proposed openstack/openstack-ansible-ops master: Add dynamic retention policies to curator https://review.openstack.org/581118 | 17:01 |
| *** armaan has quit IRC | 17:03 | |
| openstackgerrit | Jesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-os_monasca master: Execute service setup against a delegated host using Ansible built-in modules https://review.openstack.org/582304 | 17:03 |
| openstackgerrit | Vadim Kuznetsov proposed openstack/openstack-ansible-os_barbican master: Add default roles used by Barbican https://review.openstack.org/581419 | 17:03 |
| openstackgerrit | Kevin Carter (cloudnull) proposed openstack/openstack-ansible-ops master: Add dynamic retention policies to curator https://review.openstack.org/581118 | 17:04 |
| jrosser | cloudnull: memcached connection numbers after restarting one https://imagebin.ca/v/48RwnYRn92vM | 17:04 |
| *** TxGirlGeek has joined #openstack-ansible | 17:09 | |
| *** DanyC has quit IRC | 17:12 | |
| openstackgerrit | Jesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-os_neutron master: Execute service setup against a delegated host using Ansible built-in modules https://review.openstack.org/582307 | 17:15 |
| cloudnull | jrosser looks like we need to significantly raise the ceiling their | 17:17 |
| *** mmercer has joined #openstack-ansible | 17:17 | |
| jrosser | cloudnull: i think that would be wise, its also fails in a basically silent way | 17:18 |
| cloudnull | https://github.com/openstack/openstack-ansible-memcached_server/blob/master/defaults/main.yml#L51 | 17:20 |
| cloudnull | looks like it's set to 4096 in master. | 17:20 |
| jrosser | everything still works but very slow as it waits for the connection to memcached to timeout, then takes whatever the slower route would have been anyway | 17:20 |
| jrosser | generating not meaningful log message along the way | 17:20 |
| cloudnull | is there a way you could get the average memory use per connection ? | 17:20 |
| cloudnull | maybe we could just have it computed based on available memory instead of setting an arbitrary number of connections? | 17:21 |
| cloudnull | or do we think 4096 is "good enough" for now ? | 17:21 |
| jrosser | could be, im on 1024 currently so certainly need to bump that up | 17:21 |
| jrosser | we are looking into why it is so much in the first place though | 17:22 |
| openstackgerrit | Kevin Carter (cloudnull) proposed openstack/openstack-ansible-memcached_server stable/queens: Increase default connection count to 4096 https://review.openstack.org/582308 | 17:22 |
| openstackgerrit | Kevin Carter (cloudnull) proposed openstack/openstack-ansible-memcached_server stable/pike: Increase default connection count to 4096 https://review.openstack.org/582309 | 17:22 |
| openstackgerrit | Logan V proposed openstack/openstack-ansible-plugins master: [DNM] Debug the gate https://review.openstack.org/582310 | 17:23 |
| openstackgerrit | Kevin Carter (cloudnull) proposed openstack/openstack-ansible-memcached_server stable/ocata: Increase default connection count to 4096 https://review.openstack.org/582311 | 17:23 |
| jrosser | i was thinking of doing a logstash filter to add in the ratio of used vs max connections for each process | 17:23 |
| cloudnull | that cherry-picks the default change | 17:23 |
| jrosser | plotting ratios is hard in kibana and that feels like a preprocessing step anyway | 17:24 |
| jrosser | wrong to put it in the query | 17:24 |
| cloudnull | ++ totally agree. | 17:24 |
| cloudnull | a grok might be a great way to go. | 17:25 |
| jrosser | obv. the max connections from /proc might not be the same as that configured in an app | 17:25 |
| *** epalper has quit IRC | 17:38 | |
| guilhermesp | ppl if anyone could make it clearer for me.. with the deprecation of repo-build in rocky, how would the operator choose a specific sha or stable relase from source. I'm still trying to understand how the source installation will be done in face of distro installation | 17:41 |
| odyssey4me | guilhermesp: repo-build is still in rocky for now, but if you do a distro based build it won't be used | 17:42 |
| *** electrofelix has quit IRC | 17:42 | |
| odyssey4me | guilhermesp: if you use the distro build, you will get whatever the distro package provides and don't get to choose a SHA | 17:42 |
| odyssey4me | if you use a source-based build, it will work as usual | 17:42 |
| *** poul has joined #openstack-ansible | 17:43 | |
| odyssey4me | repo build will eventually go away, but then all the source building will be in the role itself for each service... which will be why repo-build will no longer be necessary | 17:43 |
| guilhermesp | so are we going to have 2 options of deployment for now, right? | 17:44 |
| guilhermesp | ah yes, got it odyssey4me | 17:44 |
| *** Dmitriy is now known as noonedeadpunk | 17:44 | |
| odyssey4me | yep - two options, either source or packages... never both on a per service basis | 17:44 |
| guilhermesp | Clearer now. Appreciated | 17:45 |
| *** poul has quit IRC | 17:47 | |
| *** tux_ has quit IRC | 17:48 | |
| *** gillesMo has quit IRC | 17:49 | |
| openstackgerrit | Jesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-os_nova master: Execute service setup against a delegated host using Ansible built-in modules https://review.openstack.org/582315 | 17:50 |
| tomtom002 | Hello, does anyone have the current link for the best way to build the amphora image for octavia? I see the test one that was built with the deployment of the octavia containers, but I remember before having to build it, but it had to be on a bare metal system, is that still true? | 17:57 |
| *** sew has joined #openstack-ansible | 17:58 | |
| johnsom | tomtom002 https://github.com/openstack/octavia/blob/master/diskimage-create/README.rst | 17:58 |
| tomtom002 | I've used the following link before to build: https://github.com/openstack/octavia/tree/master/diskimage-create Just wondering if there's an update that I should folow? | 17:58 |
| tomtom002 | johnsom, thank you! just checking to make sure. | 17:58 |
| *** poopcat has joined #openstack-ansible | 17:59 | |
| *** TxGirlGeek has quit IRC | 18:00 | |
| *** chhagarw has quit IRC | 18:00 | |
| *** tux_ has joined #openstack-ansible | 18:03 | |
| *** mmercer has quit IRC | 18:06 | |
| *** poul has joined #openstack-ansible | 18:09 | |
| *** marc1 has joined #openstack-ansible | 18:10 | |
| *** marc1 has quit IRC | 18:10 | |
| *** mmercer has joined #openstack-ansible | 18:11 | |
| openstackgerrit | Jesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-os_octavia master: Execute service setup against a delegated host using Ansible built-in modules https://review.openstack.org/582324 | 18:11 |
| *** poul has quit IRC | 18:14 | |
| *** hamzy_ has quit IRC | 18:18 | |
| openstackgerrit | Jesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-os_panko master: Execute service setup against a delegated host using Ansible built-in modules https://review.openstack.org/582328 | 18:18 |
| *** tux_ has quit IRC | 18:20 | |
| *** hamzy has joined #openstack-ansible | 18:20 | |
| *** armaan has joined #openstack-ansible | 18:24 | |
| openstackgerrit | Logan V proposed openstack/openstack-ansible-plugins master: tests: Do not use ansible_host to pass container IP https://review.openstack.org/582329 | 18:26 |
| openstackgerrit | Logan V proposed openstack/openstack-ansible-plugins master: [WIP] Allow container detection when host matches container_name https://review.openstack.org/582330 | 18:26 |
| openstackgerrit | Jesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-os_sahara master: Execute service setup against a delegated host using Ansible built-in modules https://review.openstack.org/582331 | 18:27 |
| *** epalper has joined #openstack-ansible | 18:27 | |
| *** poul has joined #openstack-ansible | 18:28 | |
| *** epalper has quit IRC | 18:28 | |
| *** armaan has quit IRC | 18:28 | |
| *** armaan has joined #openstack-ansible | 18:29 | |
| *** TxGirlGeek has joined #openstack-ansible | 18:30 | |
| *** poul has quit IRC | 18:32 | |
| *** armaan has quit IRC | 18:33 | |
| *** hachi has joined #openstack-ansible | 18:35 | |
| *** tux_ has joined #openstack-ansible | 18:35 | |
| openstackgerrit | Jesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-os_searchlight master: Execute service setup against a delegated host using Ansible built-in modules https://review.openstack.org/582333 | 18:37 |
| *** poul has joined #openstack-ansible | 18:37 | |
| *** poul has quit IRC | 18:42 | |
| openstackgerrit | Andy Smith proposed openstack/openstack-ansible master: [WIP] Add qdrouterd role for messaging backend deployment https://review.openstack.org/570261 | 18:54 |
| openstackgerrit | Merged openstack/openstack-ansible master: Fix memcached_servers content https://review.openstack.org/582105 | 18:54 |
| *** poul has joined #openstack-ansible | 18:55 | |
| *** poul has quit IRC | 18:58 | |
| *** hwoarang has joined #openstack-ansible | 18:58 | |
| *** hwoarang has quit IRC | 18:58 | |
| *** hwoarang has joined #openstack-ansible | 18:58 | |
| *** poul has joined #openstack-ansible | 18:58 | |
| *** hachi has quit IRC | 19:00 | |
| guilhermesp | odyssey4me: as I'm doing some cleaning up in the os_monasca role, should I consider doing myself the role cleanup simplification like was done in os_glance ( i.e: https://github.com/openstack/openstack-ansible-os_glance/commit/ae5fd94c5fd2cec1dd4ea4a9386ff09621d10aab ) | 19:02 |
| guilhermesp | or is this scheduled for you? | 19:02 |
| guilhermesp | for exemplo, os_monasca already have a sha to move database creation | 19:02 |
| guilhermesp | **exemple | 19:03 |
| guilhermesp | https://github.com/openstack/openstack-ansible-os_monasca/commit/356317735b463f6902813a142138b63021b917ab | 19:03 |
| *** poul has quit IRC | 19:03 | |
| *** poul has joined #openstack-ansible | 19:05 | |
| *** cmart has quit IRC | 19:07 | |
| *** poul has quit IRC | 19:10 | |
| *** sew has quit IRC | 19:12 | |
| odyssey4me | guilhermesp: you're absolutely welcome to do that proactively if you wish | 19:14 |
| odyssey4me | please use the same topic (branch name) and commit message as https://review.openstack.org/568517 to make it easy for me to track which have been done, and which not | 19:15 |
| odyssey4me | I see I made a mistake in https://github.com/openstack/openstack-ansible-os_monasca/commit/356317735b463f6902813a142138b63021b917ab ('magnum_db_setup_host' instead of monasca...) | 19:16 |
| odyssey4me | sorry for that | 19:16 |
| guilhermesp | no problem odyssey4me I already fixed | 19:16 |
| guilhermesp | https://review.openstack.org/#/c/578721/15 | 19:16 |
| guilhermesp | I'm not sure if we can merge this and after do the cleanup | 19:16 |
| guilhermesp | what you think? | 19:17 |
| *** tux_ has quit IRC | 19:17 | |
| guilhermesp | or is it better to do first some basic cleanup and after use the topics related to each simplification? | 19:17 |
| *** cshen has joined #openstack-ansible | 19:18 | |
| guilhermesp | at the moment, I'm just adapting grafana as we started to use cloudalchemy's role | 19:18 |
| *** poul has joined #openstack-ansible | 19:23 | |
| odyssey4me | guilhermesp: given that it's now passing a functional deployment, finalise that patch - then do patches after to tidy up anything that needs it, including these extra changes | 19:23 |
| odyssey4me | it's better to do small patches which fix a single issue - it makes them easier to review and easier to revert if there's a problem | 19:24 |
| odyssey4me | also easier to backport when that's needed | 19:24 |
| guilhermesp | k then. I'm going to port ansible-ansible. The functional tests also needs to be changed from non-voting to voting ( https://github.com/openstack/openstack-ansible-os_monasca/commit/330c0afa791349080dfdc1afa0589cc693c1bc1c ). I think these are the last things that needs to be done for now | 19:27 |
| *** poul has quit IRC | 19:27 | |
| *** tux_ has joined #openstack-ansible | 19:40 | |
| *** poul has joined #openstack-ansible | 19:40 | |
| openstackgerrit | Jesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-os_swift master: Execute service setup against a delegated host using Ansible built-in modules https://review.openstack.org/582354 | 19:41 |
| * odyssey4me is starting to regret life choices :p | 19:43 | |
| odyssey4me | almost there though... just 4 repositories to go | 19:43 |
| logan- | lol | 19:45 |
| *** poul has quit IRC | 19:45 | |
| mgariepy | you are doing a great job odyssey4me, don't give up ! | 19:45 |
| guilhermesp | hahahha odyssey4me I will try to do my best to help in these kind of jobs, at least in os_monasca, so you won't regret anymore | 19:47 |
| openstackgerrit | Jesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-os_tacker master: Execute service setup against a delegated host using Ansible built-in modules https://review.openstack.org/582356 | 19:49 |
| openstackgerrit | Logan V proposed openstack/openstack-ansible-plugins master: [WIP] Fix physical_host loading from group/host vars https://review.openstack.org/582358 | 19:49 |
| odyssey4me | guilhermesp: :) that would be most helpful - keep a look out to what's going on in other roles, and if you see something that might be good to adopt, do the same in os_monasca | 19:49 |
| guilhermesp | yep :) I'm taking os_glance as a reference | 19:51 |
| *** TxGirlGeek has quit IRC | 19:52 | |
| *** hachi has joined #openstack-ansible | 19:53 | |
| *** agrebennikov_ has joined #openstack-ansible | 19:54 | |
| openstackgerrit | Logan V proposed openstack/openstack-ansible-plugins master: [WIP] Fix physical_host loading from group/host vars https://review.openstack.org/582358 | 19:58 |
| *** TxGirlGeek has joined #openstack-ansible | 19:58 | |
| *** poul has joined #openstack-ansible | 19:58 | |
| openstackgerrit | Jesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-tests master: Use local connection and become for host preparation https://review.openstack.org/582359 | 20:02 |
| *** poul has quit IRC | 20:03 | |
| odyssey4me | someone asked about that failure earlier today - maybe vakuznet? | 20:03 |
| openstackgerrit | Jesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-os_magnum master: Execute service setup against a delegated host using Ansible built-in modules https://review.openstack.org/582296 | 20:05 |
| vakuznet | I guess i did | 20:06 |
| vakuznet | https://review.openstack.org/#/c/581419/ review please | 20:06 |
| *** poul has joined #openstack-ansible | 20:08 | |
| *** ansmith has quit IRC | 20:09 | |
| odyssey4me | vakuznet: lgtm! | 20:09 |
| odyssey4me | thanks! | 20:09 |
| odyssey4me | and apologies for breaking things - sometimes that happens with big pattern changes | 20:10 |
| mgariepy | done vakuznet | 20:10 |
| odyssey4me | we just need another core to review https://review.openstack.org/#/c/582195/ and that'll go through | 20:11 |
| mgariepy | have a nice evening everyone | 20:11 |
| vakuznet | thanks! | 20:12 |
| *** poul has quit IRC | 20:12 | |
| odyssey4me | thanks logan- :) | 20:12 |
| logan- | :) | 20:14 |
| *** cmart has joined #openstack-ansible | 20:21 | |
| *** sew has joined #openstack-ansible | 20:23 | |
| *** poul has joined #openstack-ansible | 20:25 | |
| openstackgerrit | Jesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-os_trove master: Execute service setup against a delegated host using Ansible built-in modules https://review.openstack.org/582365 | 20:27 |
| *** poul has quit IRC | 20:30 | |
| openstackgerrit | Guilherme Steinmuller Pimentel proposed openstack/openstack-ansible-os_monasca master: Fix monasca https://review.openstack.org/578721 | 20:34 |
| *** rodolof has joined #openstack-ansible | 20:37 | |
| *** rodolof has quit IRC | 20:38 | |
| *** cshen has quit IRC | 20:43 | |
| *** poul has joined #openstack-ansible | 20:43 | |
| odyssey4me | nice work there guilhermesp - thanks to taking up the challenge! let's see if that passes :) | 20:43 |
| *** poul has quit IRC | 20:47 | |
| openstackgerrit | Jesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-os_watcher master: Execute service setup against a delegated host using Ansible built-in modules https://review.openstack.org/582377 | 20:48 |
| *** fdsa has joined #openstack-ansible | 20:50 | |
| guilhermesp | thanks odyssey4me ! It's been quite fun for me too to contribute :) | 20:50 |
| guilhermesp | If passes, I would appreciate your review. Dunno if evrardjp wants to review too, but as he commented earlier, he just suggested remove [WIP] tag and change it back to voting | 20:51 |
| fdsa | Hi everybody. How do I correct the error: "The conditional check '(cluster_name.spl | 20:52 |
| fdsa | it()[-1] | default(false)) != galera_cluster_name' failed. | 20:52 |
| fdsa | In the "Fail if galera_cluster_name doesnt match provided value" task? | 20:52 |
| fdsa | Sorry about the multiple lines. | 20:52 |
| *** throwsb1 has quit IRC | 20:55 | |
| openstackgerrit | Jesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-os_zaqar master: Execute service setup against a delegated host using Ansible built-in modules https://review.openstack.org/582381 | 20:56 |
| odyssey4me | guilhermesp: he can review, but you'll need another core given he's a co-author | 20:56 |
| odyssey4me | and that, is finally that for this round - now to see what falls out of those, and to fix them | 20:56 |
| odyssey4me | then on to moving the mq tasks into the roles | 20:57 |
| guilhermesp | yep odyssey4me I also decided to port grafana with another patch. There are too much fragmented information in this patch | 20:58 |
| *** tux_ has quit IRC | 20:58 | |
| odyssey4me | guilhermesp: yes, that would be best | 21:00 |
| *** poul has joined #openstack-ansible | 21:00 | |
| odyssey4me | good decision! | 21:00 |
| *** TxGirlGeek has quit IRC | 21:01 | |
| openstackgerrit | Merged openstack/openstack-ansible-tests master: Use 'become' for destroy container playbook. https://review.openstack.org/582195 | 21:01 |
| *** TxGirlGeek has joined #openstack-ansible | 21:01 | |
| *** FrankZhang has quit IRC | 21:04 | |
| *** poul has quit IRC | 21:05 | |
| openstackgerrit | Jesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-os_panko master: Use tests repo for common role test requirements https://review.openstack.org/579597 | 21:05 |
| *** poul has joined #openstack-ansible | 21:10 | |
| openstackgerrit | Jesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-os_panko master: Follow the new PTI for document build https://review.openstack.org/582390 | 21:10 |
| *** cshen has joined #openstack-ansible | 21:12 | |
| openstackgerrit | Jesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-os_panko master: Use tests repo for common role test requirements https://review.openstack.org/579597 | 21:12 |
| openstackgerrit | Jesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-os_panko master: Execute service setup against a delegated host using Ansible built-in modules https://review.openstack.org/582328 | 21:12 |
| *** cjloader_ has quit IRC | 21:14 | |
| *** poul has quit IRC | 21:14 | |
| *** cshen has quit IRC | 21:16 | |
| *** jwitko has quit IRC | 21:26 | |
| *** cmart has quit IRC | 21:27 | |
| *** poul has joined #openstack-ansible | 21:27 | |
| *** rmart04 has joined #openstack-ansible | 21:28 | |
| *** ansmith has joined #openstack-ansible | 21:29 | |
| *** poul has quit IRC | 21:32 | |
| openstackgerrit | Merged openstack/openstack-ansible-os_masakari master: Execute service setup against a delegated host using Ansible built-in modules https://review.openstack.org/582300 | 21:33 |
| openstackgerrit | Merged openstack/openstack-ansible-os_monasca master: Execute service setup against a delegated host using Ansible built-in modules https://review.openstack.org/582304 | 21:35 |
| *** cmart has joined #openstack-ansible | 21:39 | |
| *** TxGirlGeek has quit IRC | 21:43 | |
| *** poul has joined #openstack-ansible | 21:44 | |
| guilhermesp | odyssey4me: going to debbug later, but Is failing in one single tempest test ( wft ). | 21:44 |
| guilhermesp | **wtf | 21:45 |
| odyssey4me | might be transient, try again and see | 21:47 |
| *** poul has quit IRC | 21:50 | |
| *** TxGirlGeek has joined #openstack-ansible | 21:51 | |
| *** poul has joined #openstack-ansible | 21:54 | |
| *** poul has quit IRC | 21:58 | |
| *** tux_ has joined #openstack-ansible | 21:59 | |
| *** sew has quit IRC | 22:02 | |
| *** poul has joined #openstack-ansible | 22:04 | |
| *** eumel8 has quit IRC | 22:05 | |
| openstackgerrit | Merged openstack/openstack-ansible-os_barbican master: Add default roles used by Barbican https://review.openstack.org/581419 | 22:06 |
| cloudnull | logan- on https://review.openstack.org/#/c/582358/2/strategy/linear.py should we just test the host var and if none, use the task var? | 22:07 |
| *** poul has quit IRC | 22:08 | |
| odyssey4me | ok folks, I'm out for the night - cheerio! | 22:12 |
| openstackgerrit | Merged openstack/openstack-ansible-os_blazar master: Fix usage of "|" for tests https://review.openstack.org/582215 | 22:18 |
| cloudnull | take care odyssey4me | 22:18 |
| *** poul has joined #openstack-ansible | 22:21 | |
| openstackgerrit | Merged openstack/openstack-ansible-lxc_hosts master: Fix usage of "|" for tests https://review.openstack.org/582255 | 22:22 |
| *** kstev has quit IRC | 22:22 | |
| *** TxGirlGeek has quit IRC | 22:23 | |
| *** poul has quit IRC | 22:25 | |
| *** hachi has quit IRC | 22:28 | |
| *** jwitko has joined #openstack-ansible | 22:30 | |
| *** TxGirlGeek has joined #openstack-ansible | 22:32 | |
| openstackgerrit | Merged openstack/openstack-ansible-plugins master: Fix usage of "|" for tests https://review.openstack.org/582216 | 22:36 |
| openstackgerrit | Merged openstack/openstack-ansible-os_neutron master: Execute service setup against a delegated host using Ansible built-in modules https://review.openstack.org/582307 | 22:36 |
| openstackgerrit | Merged openstack/openstack-ansible-os_congress master: Fix usage of "|" for tests https://review.openstack.org/582260 | 22:36 |
| *** poul has joined #openstack-ansible | 22:38 | |
| *** poul has quit IRC | 22:42 | |
| *** nurdie has quit IRC | 22:44 | |
| *** tux_ has quit IRC | 22:44 | |
| openstackgerrit | Merged openstack/openstack-ansible-os_gnocchi master: Fix usage of "|" for tests https://review.openstack.org/582239 | 22:45 |
| openstackgerrit | Merged openstack/openstack-ansible-os_rally master: Fix usage of "|" for tests https://review.openstack.org/582263 | 22:48 |
| *** KeithMnemonic has quit IRC | 22:48 | |
| *** nurdie has joined #openstack-ansible | 22:50 | |
| evrardjp | we should probably discuss PTG planning soon | 22:51 |
| openstackgerrit | Merged openstack/openstack-ansible-os_trove master: Fix usage of "|" for tests https://review.openstack.org/582251 | 22:53 |
| *** nurdie has quit IRC | 22:54 | |
| *** tux_ has joined #openstack-ansible | 22:55 | |
| *** poul has joined #openstack-ansible | 22:56 | |
| openstackgerrit | Kevin Carter (cloudnull) proposed openstack/openstack-ansible master: Update all SHAs for milestone 3 https://review.openstack.org/574006 | 22:59 |
| openstackgerrit | Merged openstack/openstack-ansible-os_keystone master: Fix usage of "|" for tests https://review.openstack.org/582267 | 22:59 |
| *** tux_ has quit IRC | 22:59 | |
| openstackgerrit | Merged openstack/openstack-ansible-os_ceilometer master: Fix usage of "|" for tests https://review.openstack.org/582240 | 23:00 |
| openstackgerrit | Merged openstack/openstack-ansible-os_aodh master: Fix usage of "|" for tests https://review.openstack.org/582235 | 23:00 |
| *** poul has quit IRC | 23:00 | |
| openstackgerrit | Merged openstack/openstack-ansible-os_glance master: Fix usage of "|" for tests https://review.openstack.org/582217 | 23:01 |
| *** tosky has quit IRC | 23:01 | |
| *** TxGirlGeek has quit IRC | 23:04 | |
| *** cshen has joined #openstack-ansible | 23:12 | |
| *** cjloader has joined #openstack-ansible | 23:12 | |
| openstackgerrit | Logan V proposed openstack/openstack-ansible-plugins master: Fix physical_host loading from group/host vars https://review.openstack.org/582358 | 23:13 |
| logan- | cloudnull: task_vars exposes host.vars I think | 23:14 |
| cloudnull | ah then we can just use that | 23:16 |
| cloudnull | makes sense. | 23:17 |
| cloudnull | the host vars would be present in the task | 23:17 |
| logan- | running 582358 and https://review.openstack.org/#/c/582330/ thru my CI with ansible 2.5 now hopefully it all works | 23:18 |
| openstackgerrit | Logan V proposed openstack/openstack-ansible-plugins master: tests: Do not use ansible_host to pass container IP https://review.openstack.org/582329 | 23:19 |
| openstackgerrit | Logan V proposed openstack/openstack-ansible-plugins master: [WIP] Allow container detection when host matches container_name https://review.openstack.org/582330 | 23:19 |
| logan- | still need to see what delegation issues arise from https://review.openstack.org/#/c/582330/ as jmccrory mentioned earlier | 23:19 |
| cloudnull | I missed that one ? | 23:23 |
| cloudnull | what is happening with it ? | 23:23 |
| logan- | what im seeing is if container_name == inventory_hostname the container detection is disabled | 23:24 |
| logan- | the plugins test inventory accidentally avoided it | 23:24 |
| logan- | https://review.openstack.org/#/c/582329/ fixes that | 23:24 |
| logan- | err sorry I meant if container_name == ansible_host | 23:25 |
| openstackgerrit | Merged openstack/openstack-ansible-os_nova master: Fix usage of "|" for tests https://review.openstack.org/582247 | 23:25 |
| openstackgerrit | Logan V proposed openstack/openstack-ansible-plugins master: [DNM] Debug the gate https://review.openstack.org/582310 | 23:31 |
| cloudnull | logan- i see | 23:31 |
| cloudnull | so we need those two prs to correct that connection issue you were seeing ? | 23:32 |
| logan- | hopefully | 23:32 |
| logan- | waiting on my gate to find out :) | 23:33 |
| * cloudnull gotta go clean things having folks over for dinner but i'll check back later this evening | 23:33 | |
| cloudnull | ping me if there's something i can review to help make that go | 23:34 |
| logan- | enjoy! | 23:34 |
| logan- | thanks | 23:34 |
| openstackgerrit | Merged openstack/openstack-ansible master: Correct E408 pep8 lint errors https://review.openstack.org/579625 | 23:35 |
| cloudnull | im keen to get all that nailed down. I've a huge deployment getting prep'd and anything that I can do to improve performance / reliability I'm +9000 | 23:36 |
| cloudnull | hence why I was looking at all that in the first place :) | 23:37 |
| cloudnull | 23:37 | |
| cloudnull | anyway, gotta run | 23:37 |
| cloudnull | cheers all | 23:37 |
| logan- | :) yea great to see the physical hosts list stuff fixed up | 23:41 |
| *** sew has joined #openstack-ansible | 23:42 | |
| *** cshen has quit IRC | 23:46 | |
| *** cjloader has quit IRC | 23:55 | |
| *** markvoelker has quit IRC | 23:59 | |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!