*** Adri2000 has joined #openstack-ansible | 00:15 | |
*** aicilamd has joined #openstack-ansible | 00:27 | |
*** schwicht has joined #openstack-ansible | 02:47 | |
*** shyamb has joined #openstack-ansible | 03:04 | |
*** shyamb has quit IRC | 03:43 | |
*** aicilamd has quit IRC | 04:19 | |
*** aicilamd has joined #openstack-ansible | 06:19 | |
*** aicilamd has quit IRC | 06:23 | |
*** hamzy has quit IRC | 06:29 | |
*** hamzy has joined #openstack-ansible | 06:29 | |
*** hamzaachi has joined #openstack-ansible | 07:41 | |
*** DanyC has joined #openstack-ansible | 08:14 | |
*** DanyC has quit IRC | 08:18 | |
openstackgerrit | Merged openstack/openstack-ansible-ops master: MNAIO: Add playbook to download saved file-backed VM's https://review.openstack.org/597444 | 09:07 |
---|---|---|
openstackgerrit | Merged openstack/openstack-ansible-ops master: MNAIO: Default vm_use_snapshot is group_vars https://review.openstack.org/597477 | 09:07 |
openstackgerrit | Merged openstack/openstack-ansible-ops master: MNAIO: Inject the host ssh public key into the image https://review.openstack.org/597478 | 09:07 |
openstackgerrit | Merged openstack/openstack-ansible-ops master: MNAIO: Make post-install storage provisioning idempotent https://review.openstack.org/597575 | 09:08 |
openstackgerrit | Merged openstack/openstack-ansible-ops master: MNAIO: Apply workaround for bug in libguestfs appliance https://review.openstack.org/598343 | 09:08 |
openstackgerrit | Merged openstack/openstack-ansible-ops master: MNAIO: Make galera startup cleanly when using images https://review.openstack.org/598344 | 09:09 |
*** DanyC has joined #openstack-ansible | 09:15 | |
*** DanyC has quit IRC | 09:19 | |
*** tosky has joined #openstack-ansible | 09:55 | |
*** dave-mccowan has joined #openstack-ansible | 10:43 | |
*** olivierb has quit IRC | 10:46 | |
*** olivierb_ has joined #openstack-ansible | 10:46 | |
*** luksky has joined #openstack-ansible | 11:00 | |
*** hamzaachi has quit IRC | 11:42 | |
*** fghaas has joined #openstack-ansible | 12:24 | |
*** hamzaachi has joined #openstack-ansible | 12:29 | |
*** hamzaachi has quit IRC | 13:02 | |
*** fghaas has quit IRC | 13:33 | |
openstackgerrit | Merged openstack/ansible-role-python_venv_build master: Simplify builds/installs by eliminating venv packaging https://review.openstack.org/598957 | 13:41 |
openstackgerrit | Jesse Pretorius (odyssey4me) proposed openstack/ansible-role-python_venv_build master: Consider pre-release wheels for installation https://review.openstack.org/599224 | 13:55 |
*** hamzaachi has joined #openstack-ansible | 14:20 | |
openstackgerrit | Merged openstack/ansible-role-python_venv_build master: Consider pre-release wheels for installation https://review.openstack.org/599224 | 14:33 |
openstackgerrit | Jesse Pretorius (odyssey4me) proposed openstack/ansible-role-python_venv_build master: Add retries to wheel build task https://review.openstack.org/599225 | 14:39 |
openstackgerrit | Jesse Pretorius (odyssey4me) proposed openstack/ansible-role-python_venv_build master: Use updated syntax for task changed result https://review.openstack.org/599226 | 14:41 |
*** nwonknu has quit IRC | 14:44 | |
*** hamzaachi has quit IRC | 14:46 | |
*** admin0 has quit IRC | 14:48 | |
*** nwonknu has joined #openstack-ansible | 14:52 | |
openstackgerrit | caoyuan proposed openstack/openstack-ansible-os_gnocchi master: Fix usage of "|" for tests https://review.openstack.org/598427 | 14:59 |
openstackgerrit | caoyuan proposed openstack/openstack-ansible-os_aodh master: use include_tasks instead of include https://review.openstack.org/585271 | 15:01 |
openstackgerrit | Jesse Pretorius (odyssey4me) proposed openstack/ansible-role-python_venv_build master: Use the same wheel path as the repo_build https://review.openstack.org/599228 | 15:02 |
openstackgerrit | zhulingjie proposed openstack/openstack-ansible-os_aodh master: Fix usage of https://review.openstack.org/598445 | 15:05 |
openstackgerrit | zhulingjie proposed openstack/openstack-ansible-os_aodh master: Fix usage of "|" for tests https://review.openstack.org/598445 | 15:05 |
openstackgerrit | zhulingjie proposed openstack/openstack-ansible-os_magnum master: Fix usage of "|" for tests https://review.openstack.org/598442 | 15:06 |
openstackgerrit | zhulingjie proposed openstack/openstack-ansible-os_tempest master: Fix usage of "|" for tests https://review.openstack.org/598451 | 15:08 |
openstackgerrit | zhulingjie proposed openstack/openstack-ansible-os_neutron master: Fix usage of "|" for tests https://review.openstack.org/598449 | 15:08 |
openstackgerrit | zhulingjie proposed openstack/openstack-ansible-os_designate master: Fix usage of "|" for tests https://review.openstack.org/598444 | 15:09 |
openstackgerrit | zhulingjie proposed openstack/openstack-ansible-os_trove master: Fix usage of "|" for tests https://review.openstack.org/598443 | 15:09 |
openstackgerrit | zhulingjie proposed openstack/openstack-ansible-os_glance master: Fix usage of "|" for tests https://review.openstack.org/598439 | 15:09 |
*** hamzaachi has joined #openstack-ansible | 15:13 | |
openstackgerrit | Merged openstack/ansible-role-python_venv_build master: Add retries to wheel build task https://review.openstack.org/599225 | 15:21 |
openstackgerrit | Merged openstack/ansible-role-python_venv_build master: Use updated syntax for task changed result https://review.openstack.org/599226 | 15:22 |
mnaser | cloudnull: have you tested the elk metrics stuff lately? I am unable to deploy because for some reason the include in the templates seems to want to resolve ALL the variables | 15:28 |
mnaser | Which results in things like nova_venv_tag not showing up? | 15:28 |
mnaser | Maybe I’m doing something wrong | 15:28 |
*** olivierb_ has quit IRC | 15:34 | |
*** schwicht has quit IRC | 15:38 | |
jrosser | mnaser: i had similar trouble - i added a note in the elk readme about symlinking across the osa group_vars which unstuck things for me.... was a few weeks ago though but all stemmed from stuff in user_variables.yml referencing things in group_vars and it all went down a wormhole from there | 15:38 |
mnaser | jrosser: that’s exactly what was causing this to | 15:40 |
mnaser | happen afaik | 15:40 |
mnaser | Did you run the playbooks inside the elk metrics folder? | 15:40 |
jrosser | yes i did | 15:41 |
jrosser | ln -s /opt/openstack-ansible/inventory/group_vars /opt/openstack-ansible-ops/elk_metrics_6x/group_vars | 15:41 |
jrosser | ^ that dug me out of the hole | 15:41 |
mnaser | With rocky I had to add the extra vars for the source_install file too | 15:41 |
mnaser | Because that wasn’t being included | 15:41 |
mnaser | But it still makes me curious why it’s trying to resolve all variables on an include | 15:41 |
mnaser | That sounds like a bug to me.. | 15:42 |
jrosser | this was queens and embedded ansible though in my case so i expect theres different traps with rocky | 15:42 |
mnaser | It’s the same thing. When I comment out all the include it works fine | 15:42 |
mnaser | But the include in the template seems to want to resolve everything | 15:42 |
jrosser | do you run it with openstack-ansible? | 15:42 |
mnaser | yes | 15:43 |
mnaser | if I comment it out, it runs fine but obviously doesn’t work | 15:43 |
*** schwicht has joined #openstack-ansible | 15:44 | |
jrosser | that will pull in all sorts from /etc/openstack-deploy, you probably have the option of using straightforward ansible-playbook and being a bit more surgical about what you '-e' | 15:44 |
openstackgerrit | Merged openstack/ansible-role-python_venv_build master: Use the same wheel path as the repo_build https://review.openstack.org/599228 | 15:47 |
mnaser | jrosser: and use dynamic inventory for the inventory? | 15:50 |
mnaser | but ill have to pull in extra stuff anyways that are needed like user secrets etc | 15:50 |
mnaser | let me see if symlink works | 15:50 |
odyssey4me | o/ jrosser mnaser | 15:53 |
odyssey4me | happy sunday :p | 15:53 |
mnaser | hey odyssey4me | 15:53 |
mnaser | haha, yep | 15:53 |
mnaser | long flight in less than a week odyssey4me ? | 15:54 |
jrosser | mnaser: i was using the embedded ansible setup script in the elk directory, which wires up the inventory and all that jazz through env vars | 15:54 |
jrosser | odyssey4me: o/ | 15:54 |
odyssey4me | still confirming that this doesn't break the integrated build, but how do you like this for tech debt reduction? :) https://review.openstack.org/#/c/567692/ | 15:54 |
*** schwicht has quit IRC | 15:54 | |
odyssey4me | mnaser: yep, not too long - although I hop through iceland which isn't great... but meh, it'll be good to catch up with everyone again! | 15:55 |
mnaser | odyssey4me: i see more - than + and i am happy | 15:55 |
mnaser | :p | 15:55 |
mnaser | yeah well for berlin you won't have to go through the same fun, but i will instead | 15:55 |
*** hamzaachi has quit IRC | 15:59 | |
odyssey4me | yeah, not sure whether I'll be there just yet - we'll see ;) | 15:59 |
mnaser | jrosser: bleh, still didn't work with ln | 16:00 |
* mnaser is done hacking on this | 16:00 | |
*** fghaas has joined #openstack-ansible | 16:35 | |
openstackgerrit | Jesse Pretorius (odyssey4me) proposed openstack/openstack-ansible master: Use a common python build/install role https://review.openstack.org/579021 | 16:46 |
odyssey4me | mnaser jrosser if you guys have a few mins, it'd be good to get ansible updated https://review.openstack.org/592315 & https://review.openstack.org/592317 | 16:47 |
openstackgerrit | Jesse Pretorius (odyssey4me) proposed openstack/openstack-ansible master: Remove molteniron playbook mistakenly left behind https://review.openstack.org/599237 | 16:51 |
openstackgerrit | Jesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-os_neutron master: Use a common python build/install role https://review.openstack.org/599238 | 17:01 |
odyssey4me | thanks jrosser | 17:01 |
jrosser | novav role tests look hosed on centos distro installs, seems to be messaging related http://logs.openstack.org/73/588273/1/check/openstack-ansible-functional-distro_install-centos-7/340526c/logs/host/neutron/neutron-linuxbridge-agent.log.txt.gz | 17:12 |
openstackgerrit | Jesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-os_nova master: Use a common python build/install role https://review.openstack.org/599240 | 17:13 |
openstackgerrit | Jesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-os_neutron master: Use a common python build/install role https://review.openstack.org/599238 | 17:14 |
odyssey4me | jrosser: hmm, access refused | 17:16 |
odyssey4me | interestingly, the rmq vhost/user tasks worked | 17:19 |
mnaser | odyssey4me, jrosser failure is more about the timeouts | 17:20 |
mnaser | the access refused is because it looks like the service goes up the first time after installation *before* configuration | 17:20 |
odyssey4me | mnaser: so you think that something else may be borked, so it's waiting for a message that never arrives? | 17:22 |
mnaser | odyssey4me: i believe so, because you see that the service was restarted afterwards | 17:23 |
mnaser | and then there was no more access refused | 17:23 |
odyssey4me | yeah, that makes sense | 17:23 |
odyssey4me | does the package auto-start the service? | 17:23 |
mnaser | http://logs.openstack.org/73/588273/1/check/openstack-ansible-functional-distro_install-centos-7/340526c/logs/openstack/openstack1/neutron-server.service.journal.log.txt.gz | 17:23 |
mnaser | that'll do it | 17:23 |
mnaser | yes i think it does | 17:23 |
mnaser | http://logs.openstack.org/73/588273/1/check/openstack-ansible-functional-distro_install-centos-7/340526c/logs/openstack/openstack1/neutron/neutron-server.log.txt.gz#_2018-09-01_21_41_53_057 | 17:24 |
odyssey4me | aha, I wonder if we shouldn't prevent that - otherwise there will be much confusion | 17:24 |
mnaser | i think there probably is a way similar to how update-rc.d or something | 17:24 |
mnaser | but that is certainly the cause | 17:24 |
mnaser | did that change touch the api paste stuff? | 17:24 |
odyssey4me | aha, so neutron-server didn't start | 17:24 |
odyssey4me | so either https://github.com/openstack/openstack-ansible-os_neutron/commit/c0b17e9d89d86d0f9543c401316727cfcd6045f2 didn't work, or it's not enough | 17:25 |
odyssey4me | mnaser: can I leave it to you and your team to work out? It is blocking a bunch of patches, unfortunately - so please do consider it priority in as much as you're able. | 17:26 |
mnaser | didn't that change pass ci or was it still failing | 17:26 |
mnaser | and yes i think we can figure it out | 17:27 |
mnaser | cc guilhermesp when you're around tomorrow ^ | 17:27 |
odyssey4me | that change was in the neutron repo, and it passed - I guess our tests there are either not good enough, or there's some difference in the tests between os_nova and os_neutron which may be causing it to fail in one but not the other | 17:27 |
odyssey4me | thanks mnaser guilhermesp | 17:28 |
mnaser | the good o'l passes in a role and fails in another | 17:28 |
odyssey4me | and thanks jrosser for digging into it | 17:28 |
mnaser | i'll get a chance to finalize our ptg sched | 17:28 |
mnaser | and we can talk about integrated repo for role testing | 17:28 |
mnaser | and make that whole thing consistent for once and for all | 17:28 |
odyssey4me | yep, part of the reason for my patch series is to make using the integrated build possible - because it'll make installs faster | 17:28 |
odyssey4me | ... possible to use for role tests. | 17:29 |
odyssey4me | the inhibiter thus far has been that the integrated build takes way too long | 17:29 |
odyssey4me | I also have that other patch to rebase which switches all the tests to be uniform, all using compute_kit - then we can make roles extend it with extra services | 17:30 |
jrosser | the zuul setup time for the integrated build is surprising | 17:31 |
odyssey4me | jrosser: oh? | 17:32 |
odyssey4me | something we should perhaps consider too is to have the major stakeholders implement external CI for testing - that way we can test more scenarios, but they're not merge blockers | 17:32 |
jrosser | odyssey4me: its ~7mins before bootstrap-host gets going | 17:34 |
jrosser | which for all the local mirrors/git repos still seems an awful lot of setup | 17:34 |
odyssey4me | then perhaps we can have the infra tests just verify the 'default' build (or a set of them), but have external CI test special configs that change defaults | 17:35 |
odyssey4me | jrosser: is that integrated build, or roles? | 17:35 |
odyssey4me | oh, bootstrap-host is integrated build | 17:35 |
jrosser | integrated, yes | 17:35 |
odyssey4me | hmm, and given we don't ask zuul to do the git clones - that does sound like a long time | 17:36 |
odyssey4me | our own git clones would be faster if we did ask it to, and more reliable - but our sha pinning interferes there... once this work of mine is done we'll be able to switch that up | 17:37 |
odyssey4me | w00t - https://review.openstack.org/557043 & https://review.openstack.org/557059 & https://review.openstack.org/567692 just passed the integrated build test :) | 17:38 |
odyssey4me | so, it looks like we'll finally be able to ditch repo build in stein ... although those patches are only an early step, they're a big one | 17:39 |
jrosser | mnaser: bionic still isnt quite in rocky - theres 4 reviews of mine in here to finish that if you've got a minute to look https://review.openstack.org/#/q/status:open+project:openstack/openstack-ansible+branch:stable/rocky | 17:44 |
odyssey4me | my hope is to either make the repo server entirely optional, or to kill it, this cycle | 17:55 |
odyssey4me | it would only remain optional if someone wants to speed up a local deployment and doesn't have their own mirrors | 17:55 |
odyssey4me | I think that we could just move it to a user story instead - advising people how to setup local mirrors. | 17:55 |
odyssey4me | With some stuff in the ops repo to help make that happen easily, of course. | 17:56 |
*** johnny101 has joined #openstack-ansible | 17:57 | |
openstackgerrit | Jesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-os_octavia master: Use a common python build/install role https://review.openstack.org/599244 | 18:01 |
openstackgerrit | Jesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-os_panko master: Use a common python build/install role https://review.openstack.org/599245 | 18:09 |
openstackgerrit | Jesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-os_neutron master: Use a common python build/install role https://review.openstack.org/599238 | 18:13 |
openstackgerrit | Jesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-os_nova master: Use a common python build/install role https://review.openstack.org/599240 | 18:16 |
*** fghaas has quit IRC | 18:19 | |
openstackgerrit | Jesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-os_rally master: Use a common python build/install role https://review.openstack.org/599246 | 18:21 |
openstackgerrit | Jesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-os_rally master: Use a common python build/install role https://review.openstack.org/599246 | 18:24 |
openstackgerrit | Jesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-os_sahara master: Use a common python build/install role https://review.openstack.org/599247 | 18:27 |
openstackgerrit | Jesse Pretorius (odyssey4me) proposed openstack/ansible-role-python_venv_build stable/rocky: Simplify builds/installs by eliminating venv packaging https://review.openstack.org/599248 | 18:48 |
openstackgerrit | Jesse Pretorius (odyssey4me) proposed openstack/ansible-role-python_venv_build stable/rocky: Consider pre-release wheels for installation https://review.openstack.org/599249 | 18:48 |
openstackgerrit | Jesse Pretorius (odyssey4me) proposed openstack/ansible-role-python_venv_build stable/rocky: Add retries to wheel build task https://review.openstack.org/599250 | 18:48 |
openstackgerrit | Jesse Pretorius (odyssey4me) proposed openstack/ansible-role-python_venv_build stable/rocky: Use updated syntax for task changed result https://review.openstack.org/599251 | 18:48 |
openstackgerrit | Jesse Pretorius (odyssey4me) proposed openstack/ansible-role-python_venv_build stable/rocky: Use the same wheel path as the repo_build https://review.openstack.org/599252 | 18:48 |
openstackgerrit | Merged openstack/openstack-ansible master: Update ansible to 2.5.8 https://review.openstack.org/592315 | 18:52 |
openstackgerrit | Jesse Pretorius (odyssey4me) proposed openstack/openstack-ansible stable/rocky: Update ansible to 2.5.8 https://review.openstack.org/599253 | 18:52 |
openstackgerrit | Jesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-os_rally master: Use a common python build/install role https://review.openstack.org/599246 | 19:02 |
openstackgerrit | Jesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-os_searchlight master: Use a common python build/install role https://review.openstack.org/599255 | 19:12 |
openstackgerrit | Jesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-os_swift master: Use a common python build/install role https://review.openstack.org/599256 | 19:16 |
odyssey4me | ok folks, I'm out for the night - cheerio! | 19:17 |
openstackgerrit | Merged openstack/ansible-role-python_venv_build stable/rocky: Simplify builds/installs by eliminating venv packaging https://review.openstack.org/599248 | 19:28 |
openstackgerrit | Merged openstack/ansible-role-python_venv_build stable/rocky: Consider pre-release wheels for installation https://review.openstack.org/599249 | 19:28 |
openstackgerrit | Merged openstack/ansible-role-python_venv_build stable/rocky: Add retries to wheel build task https://review.openstack.org/599250 | 19:28 |
openstackgerrit | Merged openstack/ansible-role-python_venv_build stable/rocky: Use updated syntax for task changed result https://review.openstack.org/599251 | 19:29 |
openstackgerrit | Merged openstack/ansible-role-python_venv_build stable/rocky: Use the same wheel path as the repo_build https://review.openstack.org/599252 | 19:32 |
openstackgerrit | Merged openstack/openstack-ansible-tests master: Update ansible to 2.5.8 https://review.openstack.org/592317 | 19:32 |
openstackgerrit | Mohammed Naser proposed openstack/openstack-ansible-ops master: Refactor templates to use a single macro template file https://review.openstack.org/599257 | 19:58 |
openstackgerrit | Mohammed Naser proposed openstack/openstack-ansible-ops master: Refactor templates to use a single macro template file https://review.openstack.org/599257 | 20:04 |
mnaser | jrosser: that should fix it :) | 20:10 |
jrosser | mnaser: hmmm maybe :) the current trajectory of that code is to separate it out into the separate roles..... | 20:13 |
mnaser | jrosser: well this fixes it in the state that it is now | 20:13 |
mnaser | *for now* | 20:13 |
mnaser | :p | 20:13 |
*** hamzaachi has joined #openstack-ansible | 20:26 | |
*** hamzaachi has quit IRC | 20:48 | |
*** hamzaachi has joined #openstack-ansible | 20:48 | |
*** hamzaachi has quit IRC | 20:53 | |
*** hamzaachi has joined #openstack-ansible | 20:54 | |
openstackgerrit | Mohammed Naser proposed openstack/openstack-ansible-ops master: Allow using custom publish host https://review.openstack.org/599263 | 21:16 |
*** dhellmann has quit IRC | 21:48 | |
*** luksky has quit IRC | 21:52 | |
*** vnogin has joined #openstack-ansible | 22:09 | |
*** dhellmann has joined #openstack-ansible | 22:10 | |
*** hamzaachi has quit IRC | 22:10 | |
*** hamzaachi has joined #openstack-ansible | 22:11 | |
*** hamzaachi has quit IRC | 22:24 | |
openstackgerrit | Jesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-tests stable/rocky: Update ansible to 2.5.8 https://review.openstack.org/599266 | 22:57 |
*** threestrands has joined #openstack-ansible | 23:19 | |
*** tosky has quit IRC | 23:23 | |
*** vnogin has quit IRC | 23:31 | |
*** flaviosr has quit IRC | 23:50 | |
*** flaviosr has joined #openstack-ansible | 23:53 | |
*** speedmann has quit IRC | 23:54 | |
*** flaviosr has quit IRC | 23:54 | |
*** flaviosr has joined #openstack-ansible | 23:54 | |
*** speedmann has joined #openstack-ansible | 23:54 |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!