openstackgerrit | Matthew Thode proposed openstack/openstack-ansible-os_ironic master: Overhaul ironic role https://review.openstack.org/578219 | 00:36 |
---|---|---|
*** vnogin has joined #openstack-ansible | 00:39 | |
*** vnogin has quit IRC | 00:44 | |
*** tux_ has joined #openstack-ansible | 01:24 | |
openstackgerrit | Leopard Ma proposed openstack/ansible-config_template master: Add source code repository and project home notes link to README https://review.openstack.org/579415 | 01:30 |
*** vnogin has joined #openstack-ansible | 01:40 | |
openstackgerrit | Leopard Ma proposed openstack/openstack-ansible-os_glance master: Add source code repository notes link to README https://review.openstack.org/579416 | 01:41 |
*** vnogin has quit IRC | 01:45 | |
*** openstackgerrit has quit IRC | 01:49 | |
*** dxiri has joined #openstack-ansible | 02:12 | |
*** tux_ has quit IRC | 02:12 | |
*** dxiri has quit IRC | 02:17 | |
*** openstackgerrit has joined #openstack-ansible | 02:22 | |
openstackgerrit | Leopard Ma proposed openstack/openstack-ansible-os_gnocchi master: Add source code repository notes link to README https://review.openstack.org/579417 | 02:22 |
*** dxiri has joined #openstack-ansible | 02:34 | |
*** dxiri has quit IRC | 02:39 | |
*** vnogin has joined #openstack-ansible | 02:41 | |
*** idlemind has joined #openstack-ansible | 02:44 | |
*** vnogin has quit IRC | 02:45 | |
openstackgerrit | Leopard Ma proposed openstack/openstack-ansible-os_heat master: Add source code repository notes link to README https://review.openstack.org/579419 | 02:58 |
openstackgerrit | Leopard Ma proposed openstack/openstack-ansible-os_horizon master: Add source code repository notes link to README https://review.openstack.org/579421 | 03:05 |
*** dxiri has joined #openstack-ansible | 03:37 | |
*** vnogin has joined #openstack-ansible | 03:42 | |
*** dxiri has quit IRC | 03:42 | |
*** vnogin has quit IRC | 03:46 | |
*** udesale has joined #openstack-ansible | 03:57 | |
*** Bhujay has joined #openstack-ansible | 04:00 | |
*** jonher_ has joined #openstack-ansible | 04:04 | |
*** jonher has quit IRC | 04:06 | |
*** Bhujay has quit IRC | 04:13 | |
*** dxiri has joined #openstack-ansible | 04:20 | |
*** dxiri has quit IRC | 04:25 | |
*** ethfci has quit IRC | 04:26 | |
*** vnogin has joined #openstack-ansible | 04:42 | |
*** vnogin has quit IRC | 04:47 | |
*** jmnk has joined #openstack-ansible | 05:09 | |
*** armaan has joined #openstack-ansible | 05:59 | |
*** maybebuggy has joined #openstack-ansible | 06:15 | |
*** udesale_ has joined #openstack-ansible | 06:16 | |
*** udesale has quit IRC | 06:19 | |
*** jcosmao has quit IRC | 06:26 | |
openstackgerrit | Jesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-tests master: Use tests repo for common role test requirements https://review.openstack.org/579208 | 06:28 |
odyssey4me | o/ | 06:28 |
*** threestrands has joined #openstack-ansible | 06:33 | |
*** threestrands has quit IRC | 06:33 | |
*** threestrands has joined #openstack-ansible | 06:33 | |
openstackgerrit | Jesse Pretorius (odyssey4me) proposed openstack/openstack-ansible master: Ensure python-keystoneclient has a lower bound https://review.openstack.org/579463 | 06:37 |
*** threestrands has quit IRC | 06:37 | |
*** nicolasbock has joined #openstack-ansible | 06:39 | |
*** fghaas has joined #openstack-ansible | 06:41 | |
openstackgerrit | Leopard Ma proposed openstack/openstack-ansible-ceph_client master: Add source code repository notes link to README https://review.openstack.org/579464 | 06:42 |
*** udesale__ has joined #openstack-ansible | 06:42 | |
*** vnogin has joined #openstack-ansible | 06:43 | |
*** udesale_ has quit IRC | 06:45 | |
openstackgerrit | Jesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-tests master: Use tests repo for common role test requirements https://review.openstack.org/579208 | 06:47 |
*** vnogin has quit IRC | 06:48 | |
*** udesale_ has joined #openstack-ansible | 06:58 | |
openstackgerrit | Jesse Pretorius (odyssey4me) proposed openstack/ansible-role-python_venv_build master: Add EPEL repo for CentOS tests https://review.openstack.org/579468 | 07:00 |
*** erolg has joined #openstack-ansible | 07:01 | |
*** udesale__ has quit IRC | 07:01 | |
*** armaan has quit IRC | 07:07 | |
openstackgerrit | Merged openstack/openstack-ansible-os_ironic stable/queens: Add glance auth information to ironic.conf for communication https://review.openstack.org/579279 | 07:07 |
openstackgerrit | Jesse Pretorius (odyssey4me) proposed openstack/ansible-role-systemd_mount master: Follow the new PTI for document build https://review.openstack.org/579469 | 07:08 |
*** armaan has joined #openstack-ansible | 07:08 | |
*** admin0 has joined #openstack-ansible | 07:08 | |
openstackgerrit | Jesse Pretorius (odyssey4me) proposed openstack/ansible-role-systemd_mount master: Follow the new PTI for document build https://review.openstack.org/579469 | 07:08 |
*** epalper has joined #openstack-ansible | 07:10 | |
openstackgerrit | Jesse Pretorius (odyssey4me) proposed openstack/ansible-role-systemd_networkd master: Follow the new PTI for document build https://review.openstack.org/579470 | 07:12 |
*** lvdombrkr has joined #openstack-ansible | 07:12 | |
openstackgerrit | Jesse Pretorius (odyssey4me) proposed openstack/ansible-role-systemd_service master: Follow the new PTI for document build https://review.openstack.org/579471 | 07:13 |
*** tux_ has joined #openstack-ansible | 07:14 | |
openstackgerrit | Jesse Pretorius (odyssey4me) proposed openstack/ansible-role-systemd_service master: Follow the new PTI for document build https://review.openstack.org/579471 | 07:17 |
*** tux_ has quit IRC | 07:18 | |
evrardjp | g00d morning ppl | 07:18 |
openstackgerrit | Jesse Pretorius (odyssey4me) proposed openstack/ansible-role-systemd_service master: Follow the new PTI for document build https://review.openstack.org/579471 | 07:18 |
openstackgerrit | Jesse Pretorius (odyssey4me) proposed openstack/ansible-role-systemd_service master: Follow the new PTI for document build https://review.openstack.org/579471 | 07:19 |
evrardjp | I got banned from #ansible-devel, with many others. | 07:20 |
evrardjp | it's fixed now, but you should reconnect, should you have been banned in the week-end | 07:21 |
openstackgerrit | Jesse Pretorius (odyssey4me) proposed openstack/ansible-role-systemd_networkd master: Follow the new PTI for document build https://review.openstack.org/579470 | 07:21 |
openstackgerrit | Jesse Pretorius (odyssey4me) proposed openstack/ansible-role-systemd_mount master: Follow the new PTI for document build https://review.openstack.org/579469 | 07:22 |
odyssey4me | evrardjp: I guess you had a good resting weekend then. :p | 07:23 |
evrardjp | odyssey4me: OMG not resting, and not good, but I had a weekend NOT linked to anything IT. | 07:25 |
*** armaan has quit IRC | 07:26 | |
*** armaan has joined #openstack-ansible | 07:26 | |
openstackgerrit | Jesse Pretorius (odyssey4me) proposed openstack/ansible-role-systemd_mount master: Follow the new PTI for document build https://review.openstack.org/579469 | 07:28 |
openstackgerrit | Jesse Pretorius (odyssey4me) proposed openstack/ansible-role-systemd_networkd master: Follow the new PTI for document build https://review.openstack.org/579470 | 07:30 |
openstackgerrit | Jesse Pretorius (odyssey4me) proposed openstack/ansible-role-systemd_service master: Follow the new PTI for document build https://review.openstack.org/579471 | 07:31 |
openstackgerrit | Nguyen Hung Phuong proposed openstack/ansible-hardening master: fix tox python3 overrides https://review.openstack.org/579476 | 07:32 |
openstackgerrit | Jesse Pretorius (odyssey4me) proposed openstack/ansible-role-python_venv_build master: Add EPEL repo for CentOS tests https://review.openstack.org/579468 | 07:33 |
odyssey4me | evrardjp hwoarang I've been doing some tidying up and have also tried doing some simplifying - I have some reviews for you when you're ready. :) | 07:35 |
*** rpittau has joined #openstack-ansible | 07:35 | |
*** gillesMo has joined #openstack-ansible | 07:38 | |
openstackgerrit | Jesse Pretorius (odyssey4me) proposed openstack/ansible-role-systemd_service master: Follow the new PTI for document build https://review.openstack.org/579471 | 07:38 |
openstackgerrit | Jesse Pretorius (odyssey4me) proposed openstack/ansible-role-systemd_networkd master: Follow the new PTI for document build https://review.openstack.org/579470 | 07:39 |
openstackgerrit | Jesse Pretorius (odyssey4me) proposed openstack/ansible-role-systemd_mount master: Follow the new PTI for document build https://review.openstack.org/579469 | 07:39 |
evrardjp | odyssey4me: that sounds great | 07:41 |
evrardjp | I am still catching up the weekend | 07:41 |
*** vnogin has joined #openstack-ansible | 07:44 | |
evrardjp | so, folks, I am currently working on doing what we said we're gonna formalize in the past | 07:44 |
evrardjp | a tedious role by role review of the mandatory vars | 07:44 |
evrardjp | I think I will do it the current way (have a variable listing the required variables), and then I will move on to the future approach. | 07:45 |
odyssey4me | oh man, that is tedious | 07:45 |
evrardjp | yeah. | 07:46 |
evrardjp | well | 07:46 |
evrardjp | I got not much choice. | 07:46 |
evrardjp | I am building a converter from HP/SUSE Ardana Config processor to our inventory | 07:47 |
evrardjp | or to a native inventory, but with direct input of what's required | 07:47 |
evrardjp | I need to know what we need. | 07:47 |
odyssey4me | ok, wow - fair enough | 07:48 |
*** vnogin has quit IRC | 07:49 | |
odyssey4me | I always thought that if we could somehow built a lint test which checks for variables used, but not provided, then it would be much more effective at ensuring that we keep the defaults and required lists current | 07:49 |
evrardjp | yeah. | 07:50 |
evrardjp | But guess what | 07:50 |
evrardjp | we've wired so many things with the tests repo... | 07:50 |
evrardjp | it doesn't make our life easier | 07:50 |
evrardjp | example | 07:50 |
evrardjp | well let's not go deep into that | 07:50 |
evrardjp | we have so many things to do | 07:51 |
odyssey4me | yep, if we can converge to using the integrated repo for everything then it can be made to be better as we won't be straddling two test systems | 07:53 |
*** vnogin has joined #openstack-ansible | 07:53 | |
*** vnogin has quit IRC | 07:58 | |
openstackgerrit | Jesse Pretorius (odyssey4me) proposed openstack/ansible-role-python_venv_build master: Add EPEL repo for nginx packages on CentOS tests https://review.openstack.org/579468 | 08:05 |
*** shardy has joined #openstack-ansible | 08:05 | |
*** armaan has quit IRC | 08:06 | |
*** armaan has joined #openstack-ansible | 08:07 | |
evrardjp | odyssey4me: could you answer my question here: https://review.openstack.org/#/c/579208/7 ? | 08:07 |
*** armaan has quit IRC | 08:12 | |
*** armaan has joined #openstack-ansible | 08:13 | |
openstackgerrit | Jean-Philippe Evrard proposed openstack/openstack-ansible master: Do not build cryptography https://review.openstack.org/574181 | 08:21 |
evrardjp | odyssey4me: you have a grafana install experience with cloudalchemy, right? | 08:23 |
odyssey4me | evrardjp: yep | 08:23 |
evrardjp | do you have a play/role I can see for comparison? | 08:23 |
evrardjp | I am looking for the mandatory vars | 08:24 |
evrardjp | and I am lazy | 08:24 |
odyssey4me | evrardjp: sure: https://github.com/rcbops/rpc-gating/tree/master/grafana | 08:24 |
odyssey4me | the mandatory vars are all in the preflight part | 08:24 |
odyssey4me | https://github.com/cloudalchemy/ansible-grafana/blob/master/tasks/preflight.yml | 08:25 |
evrardjp | http://logs.openstack.org/21/578721/8/check/openstack-ansible-functional-ubuntu-xenial/d9ce89b/job-output.txt.gz#_2018-06-30_13_48_01_701954 | 08:25 |
evrardjp | that doesn't seem to be preflight | 08:25 |
odyssey4me | what's the review where that came from? | 08:26 |
evrardjp | but I am just quickly browsing | 08:26 |
evrardjp | fixing monasca | 08:26 |
evrardjp | https://review.openstack.org/#/c/578721/8 | 08:26 |
openstackgerrit | Merged openstack/openstack-ansible-os_aodh master: Fix the ansible role aim to Aodh https://review.openstack.org/577433 | 08:26 |
openstackgerrit | Merged openstack/openstack-ansible-os_keystone master: Delete the unused port 35357 https://review.openstack.org/577445 | 08:27 |
*** chigang has quit IRC | 08:28 | |
evrardjp | odyssey4me: maybe it doesn't load the defaults properly | 08:29 |
odyssey4me | yeah, because http://logs.openstack.org/21/578721/8/check/openstack-ansible-functional-ubuntu-xenial/d9ce89b/job-output.txt.gz#_2018-06-30_13_47_49_154516 should be setting that var | 08:29 |
*** cshen has joined #openstack-ansible | 08:30 | |
odyssey4me | https://github.com/cloudalchemy/ansible-grafana/blob/c9d97ca84abf5daca3d591c497a03e24d52f2674/vars/debian.yml#L2 | 08:30 |
evrardjp | oh yeah | 08:30 |
evrardjp | classic | 08:30 |
evrardjp | nice wrong file loading | 08:30 |
evrardjp | this is why I hate the dwim thing | 08:30 |
odyssey4me | oh of course - it's picked up a file from our role rather than from the right role | 08:30 |
evrardjp | yeah | 08:30 |
evrardjp | include_role. .. | 08:31 |
openstackgerrit | Merged openstack/openstack-ansible-os_panko master: Fix the ansible role aim to Panko https://review.openstack.org/577440 | 08:32 |
openstackgerrit | Jean-Philippe Evrard proposed openstack/openstack-ansible-os_monasca master: [WIP] Fix monasca https://review.openstack.org/578721 | 08:32 |
openstackgerrit | Jesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-tests master: Use tests repo for common role test requirements https://review.openstack.org/579208 | 08:33 |
admin0 | hi all .. looking for a sample config with ceph | 08:33 |
evrardjp | it would be very cool if we work on the upgrades now | 08:38 |
evrardjp | a few roles are broken in terms of upgrades. | 08:38 |
openstackgerrit | Jesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-tests master: Use the TESTING_BRANCH env var for constraints https://review.openstack.org/579371 | 08:39 |
evrardjp | cloudnull: could you stop +w your own patches when there is no 2x +2 and no urgency, please? | 08:40 |
odyssey4me | evrardjp: they all are - but we can't fix them without re-doing how the upgrade test is done | 08:40 |
*** hachi has joined #openstack-ansible | 08:41 | |
*** gaudenz has quit IRC | 08:41 | |
*** vnogin has joined #openstack-ansible | 08:45 | |
openstackgerrit | Jesse Pretorius (odyssey4me) proposed openstack/openstack-ansible stable/queens: Fix loop variable name for nested loop https://review.openstack.org/579489 | 08:46 |
openstackgerrit | Jesse Pretorius (odyssey4me) proposed openstack/openstack-ansible stable/pike: Fix loop variable name for nested loop https://review.openstack.org/579490 | 08:46 |
openstackgerrit | Jesse Pretorius (odyssey4me) proposed openstack/openstack-ansible stable/ocata: Fix loop variable name for nested loop https://review.openstack.org/579491 | 08:47 |
*** gaudenz has joined #openstack-ansible | 08:49 | |
*** armaan has quit IRC | 08:50 | |
*** armaan has joined #openstack-ansible | 08:50 | |
evrardjp | odyssey4me: could you have a look at this: https://review.openstack.org/#/c/568517/9 ? | 08:52 |
evrardjp | ok I have to spend time on something else now | 08:53 |
evrardjp | ttyl everyone | 08:53 |
odyssey4me | evrardjp: we can use the sync mechanism without doing that I think | 08:53 |
odyssey4me | we just have to tell the sync what to do to modify the file for each role | 08:54 |
evrardjp | odyssey4me: well no, as the variables would be specific | 08:54 |
odyssey4me | the variables and file name are easy to search & replace | 08:54 |
evrardjp | yeah but why doing search and replace, why you can just have it mapped :) | 08:54 |
evrardjp | while* | 08:54 |
evrardjp | fine for me in the search and replace | 08:55 |
evrardjp | but it seems more error prone | 08:55 |
odyssey4me | that's an evolution which can be worked on later, I feel that moving the stuff into the role is more urgent as it's inhibiting the venv build work | 08:55 |
evrardjp | and new features | 08:55 |
evrardjp | yeah so this is easily done | 08:55 |
odyssey4me | this is why a sync mechanism is not necessary right now, we just need the thing in and working | 08:55 |
evrardjp | wouldn't you prefer setting a good precedent? | 08:55 |
evrardjp | it's a few vars to map I can do it if you like | 08:56 |
odyssey4me | as a patch on top, go for it - but I think you and I both have enough to work on already | 08:56 |
openstackgerrit | Merged openstack/openstack-ansible-os_ironic master: Overhaul ironic role https://review.openstack.org/578219 | 08:57 |
odyssey4me | working on that will eat up another day or two of time which I don't have | 08:57 |
*** electrofelix has joined #openstack-ansible | 08:57 | |
odyssey4me | I'd suggest registering a bug for it and leaving that as-is for now. We can tackle a sync mechanism for it next cycle or after M3 perhaps. | 08:58 |
evrardjp | I think I am not clear in english again. Let me show you with code | 08:59 |
*** armaan has quit IRC | 08:59 | |
*** armaan has joined #openstack-ansible | 08:59 | |
odyssey4me | I'd really like to put https://review.openstack.org/#/q/topic:move-db-setup-into-roles+status:open behind me, so reviews of those would be appreciated. | 09:01 |
odyssey4me | of course something's broken in the octavia and trove roles :/ | 09:02 |
openstackgerrit | Merged openstack/openstack-ansible-tests master: Remove zuul v2 shim https://review.openstack.org/579195 | 09:02 |
odyssey4me | trove is broken on centos only - looks like some sort of setuptools issue | 09:03 |
*** gkadam has joined #openstack-ansible | 09:06 | |
openstackgerrit | Merged openstack/openstack-ansible-repo_build master: Add option for default pip package install https://review.openstack.org/574546 | 09:07 |
*** hachi has quit IRC | 09:08 | |
*** hachi has joined #openstack-ansible | 09:10 | |
*** hachi_ has joined #openstack-ansible | 09:13 | |
openstackgerrit | Jean-Philippe Evrard proposed openstack/openstack-ansible-os_glance master: Allow an easy sync of MQ files https://review.openstack.org/579498 | 09:13 |
openstackgerrit | Merged openstack/openstack-ansible-rabbitmq_server master: Restart after plugins or policies have changed https://review.openstack.org/579299 | 09:13 |
evrardjp | odyssey4me: yeah I reviewed some of those | 09:13 |
odyssey4me | thanks evrardjp - appreciate it | 09:13 |
evrardjp | I understand if we do this and rabbitmq we'll be in a easier place | 09:13 |
odyssey4me | yep, my next set is to get https://review.openstack.org/568142 done across all roles too | 09:14 |
evrardjp | odyssey4me: yeah, and rabbitmq I guess? | 09:14 |
odyssey4me | it should remove all the host packages we install | 09:14 |
odyssey4me | yes, this is after the rabbitmq bits | 09:15 |
openstackgerrit | Jesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-tests master: Document the purpose of the test-requirements.txt file https://review.openstack.org/579231 | 09:16 |
evrardjp | have you seen my patch? Because I think that if we do the pattern in my patch for DB too later, we could have a cookie cutter role in tasks that syncs those two (db, mq) files | 09:16 |
openstackgerrit | Jesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-tests master: Add shade & python-keystoneclient to the tox venv https://review.openstack.org/579233 | 09:16 |
evrardjp | maybe eveb more | 09:16 |
*** hachi has quit IRC | 09:16 | |
openstackgerrit | Jesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-tests master: Make run_tests_common idempotent https://review.openstack.org/579372 | 09:16 |
evrardjp | (the service setup for example, could also be like it) | 09:17 |
evrardjp | reduce maintenance is always good : ) | 09:17 |
odyssey4me | evrardjp: yeah, but that'll be done in this series https://review.openstack.org/568142 | 09:17 |
evrardjp | I think these are orthogonal, and can be done one after the other, or together | 09:18 |
evrardjp | but anyway, we are on the right path | 09:18 |
*** seba has quit IRC | 09:20 | |
*** seba has joined #openstack-ansible | 09:22 | |
*** flaper87 has quit IRC | 09:24 | |
*** flaper87 has joined #openstack-ansible | 09:24 | |
openstackgerrit | Merged openstack/openstack-ansible-os_zun master: Move database creation into role https://review.openstack.org/578874 | 09:26 |
openstackgerrit | Merged openstack/openstack-ansible-os_masakari master: Move database creation into role https://review.openstack.org/578815 | 09:26 |
*** cshen has quit IRC | 09:29 | |
openstackgerrit | Nguyen Hung Phuong proposed openstack/ansible-hardening master: fix tox python3 overrides https://review.openstack.org/579476 | 09:32 |
*** cshen has joined #openstack-ansible | 09:33 | |
*** rmart04 has joined #openstack-ansible | 09:38 | |
*** spsurya_ has joined #openstack-ansible | 09:40 | |
*** hachi_ has quit IRC | 09:40 | |
openstackgerrit | Taseer Ahmed proposed openstack/openstack-ansible-os_blazar master: This patch ensures that the functional tests for Blazar pass. https://review.openstack.org/571388 | 09:44 |
*** armaan has quit IRC | 09:47 | |
*** armaan has joined #openstack-ansible | 09:47 | |
openstackgerrit | Ilya Popov proposed openstack/openstack-ansible master: IP addresses in examples were changed https://review.openstack.org/572935 | 09:49 |
openstackgerrit | Nguyen Hung Phuong proposed openstack/ansible-hardening master: fix tox python3 overrides https://review.openstack.org/579476 | 09:57 |
openstackgerrit | Merged openstack/openstack-ansible-os_barbican master: Move database creation into role https://review.openstack.org/574847 | 10:03 |
openstackgerrit | Merged openstack/openstack-ansible-os_barbican master: Move test-vars into overrides file https://review.openstack.org/574848 | 10:03 |
openstackgerrit | Jesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-os_zun master: Execute service setup against a delegated host using Ansible built-in modules https://review.openstack.org/579506 | 10:03 |
openstackgerrit | Jesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-os_zun master: Execute service setup against a delegated host using Ansible built-in modules https://review.openstack.org/579506 | 10:06 |
*** epalper has quit IRC | 10:14 | |
*** epalper has joined #openstack-ansible | 10:14 | |
*** gillesMo has quit IRC | 10:16 | |
*** cshen has quit IRC | 10:16 | |
*** cshen has joined #openstack-ansible | 10:17 | |
openstackgerrit | Merged openstack/openstack-ansible-os_swift master: Update to use oslo.messaging service for Notify https://review.openstack.org/574359 | 10:19 |
openstackgerrit | Jesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-os_ceilometer master: Execute service setup against a delegated host using Ansible built-in modules https://review.openstack.org/579509 | 10:23 |
openstackgerrit | Jesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-os_ceilometer master: Execute service setup against a delegated host using Ansible built-in modules https://review.openstack.org/579509 | 10:27 |
openstackgerrit | Jesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-os_ceilometer master: Execute service setup against a delegated host using Ansible built-in modules https://review.openstack.org/579509 | 10:30 |
openstackgerrit | Jesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-os_glance master: Execute service setup against a delegated host using Ansible built-in modules https://review.openstack.org/568142 | 10:32 |
openstackgerrit | Jesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-os_glance master: Execute service setup against a delegated host using Ansible built-in modules https://review.openstack.org/568142 | 10:48 |
*** tux_ has joined #openstack-ansible | 10:51 | |
openstackgerrit | Jesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-os_glance master: Execute service setup against a delegated host using Ansible built-in modules https://review.openstack.org/568142 | 10:55 |
*** tux_ has quit IRC | 10:55 | |
*** epalper has quit IRC | 10:55 | |
openstackgerrit | Jesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-os_ironic master: Execute service setup against a delegated host using Ansible built-in modules https://review.openstack.org/579514 | 10:55 |
*** epalper has joined #openstack-ansible | 10:55 | |
openstackgerrit | Merged openstack/openstack-ansible-os_neutron master: templates: dnsmasq-neutron.conf: Fix user for dnsmasq process https://review.openstack.org/579127 | 11:00 |
openstackgerrit | Merged openstack/openstack-ansible-os_neutron master: SUSE: Add missing Neutron distribution packages https://review.openstack.org/579151 | 11:00 |
odyssey4me | evrardjp: I missed your comment about the cookie cutter role earlier - what patch are you referncing? | 11:02 |
evrardjp | something like this: https://review.openstack.org/#/c/579498/ | 11:02 |
evrardjp | (to be expended) | 11:02 |
evrardjp | (and partially) | 11:02 |
evrardjp | https://review.openstack.org/#/c/458562/ doesn't work | 11:03 |
evrardjp | well | 11:03 |
evrardjp | it works | 11:03 |
evrardjp | but it won't scale :) | 11:03 |
odyssey4me | with regards to https://review.openstack.org/#/c/579498/1/tasks/main.yml - I thought that was what you were referring to... we'd have to restructure it for roles which don't do rpc and others that don't do notifications | 11:05 |
odyssey4me | if you're really keen to use that approach then I can revise, but I'll need help with the part which does the proposal bot sync | 11:06 |
odyssey4me | I'm happy to push up the patches to the roles, but I'm not entirely sure how the proposal bot thing works. I've always let hwoarang sort that out. :p | 11:06 |
evrardjp | it's just a shell script, but I can help. But indeed we might need to simplify this into a single mq file with just two tasks, and include it twice, for some roles, once for some others | 11:07 |
evrardjp | because the tasks are exactly the same | 11:08 |
odyssey4me | yep, I can revise to sort that out | 11:08 |
evrardjp | well we can do it later, as you said | 11:08 |
evrardjp | I thought it was just more convenient | 11:09 |
openstackgerrit | Merged openstack/openstack-ansible master: Move database creation into role (heat) https://review.openstack.org/571751 | 11:13 |
openstackgerrit | Jean-Philippe Evrard proposed openstack/openstack-ansible-os_glance master: Allow an easy sync of MQ files https://review.openstack.org/579498 | 11:16 |
openstackgerrit | Jesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-os_glance master: Move MQ vhost/user creation into role https://review.openstack.org/568517 | 11:20 |
odyssey4me | evrardjp: oh, I didn't see your update - I had a go at it too | 11:21 |
openstackgerrit | Jesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-os_glance master: Move MQ vhost/user creation into role https://review.openstack.org/568517 | 11:22 |
evrardjp | odyssey4me: if you go with with_items it's probably good to do loop: with a loop var instead | 11:23 |
evrardjp | nah it doesn't matter | 11:23 |
evrardjp | as long as it works :p | 11:23 |
odyssey4me | seems fine to me, and using with_items looks cleaner IMO | 11:24 |
evrardjp | well I am not sure it reads as easily | 11:25 |
evrardjp | I mean in ARA, or logs | 11:25 |
*** udesale_ has quit IRC | 11:25 | |
odyssey4me | yeah, due to the stuff handed to it I have to do no_log, which sucks a bit | 11:25 |
odyssey4me | but I think your method would require the same, no? | 11:26 |
evrardjp | yeah | 11:26 |
evrardjp | you can rename the key :p | 11:26 |
evrardjp | but I don't like this idea | 11:26 |
evrardjp | because you still pass the items | 11:26 |
odyssey4me | I considered that, and started doing it - then realised Iwasn't sure what to set it to | 11:26 |
evrardjp | well I realized that this would bypass the current linting | 11:27 |
evrardjp | so we might need to improve our lint test there | 11:27 |
odyssey4me | doesn't matter - we still don't want the password exposed | 11:27 |
evrardjp | yeah that's what I mean | 11:27 |
evrardjp | currently it's possible | 11:27 |
evrardjp | if you name the thing: | 11:27 |
evrardjp | oslomsg_blabla instead of oslomsg_password, you'd go in clear | 11:28 |
evrardjp | but I don't know how to fix that | 11:28 |
evrardjp | that sounds very hacky hacky to do oslomsg_blabla instead of naming it correctly oslomsg_password | 11:28 |
evrardjp | so it's not really the fault of the lint test | 11:28 |
odyssey4me | yep | 11:28 |
odyssey4me | far better to improve the modules so that they don't log the password, and you therefore don't need no_log | 11:29 |
odyssey4me | ok, where would we keep this file in the tests repo to sync with? | 11:29 |
evrardjp | well | 11:29 |
evrardjp | odyssey4me: I suggest we build an empty role with files to sync | 11:30 |
evrardjp | in tests | 11:30 |
odyssey4me | maybe 'sync/mq_setup.yml' ? | 11:30 |
evrardjp | yeah | 11:30 |
odyssey4me | we'd have to figure out how to make sure it's linted | 11:31 |
odyssey4me | for now I just want to make sure I add a notice to the file to say it's maintained there | 11:31 |
evrardjp | We can do that later probably but okay | 11:31 |
evrardjp | I think this: https://github.com/openstack/openstack-ansible-tests/blob/master/ensure-oslomsg.yml can be replaced in the future too : ) | 11:32 |
odyssey4me | yes, that'll be removed once all these are done | 11:32 |
openstackgerrit | Jesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-os_glance master: Move MQ vhost/user creation into role https://review.openstack.org/568517 | 11:35 |
odyssey4me | evrardjp: could you prep a patch to add that to the tests repo and do whatever needs doing to have it synchronised if any changes to it merge? | 11:36 |
*** jroll has quit IRC | 11:36 | |
evrardjp | yeah | 11:37 |
*** jroll has joined #openstack-ansible | 11:40 | |
*** kaiokmo has joined #openstack-ansible | 11:46 | |
*** ansmith has quit IRC | 11:49 | |
evrardjp | my bash foo is low | 11:51 |
openstackgerrit | Jean-Philippe Evrard proposed openstack/openstack-ansible-tests master: [WIP] Sync task files to roles https://review.openstack.org/579527 | 11:51 |
evrardjp | this is not the full thing | 11:52 |
evrardjp | hwoarang: could you help on the bash-isms? | 11:52 |
evrardjp | making sure it's even cleaner than that? I am generally bad with arrays | 11:52 |
evrardjp | wow that's a bad patch. | 11:53 |
openstackgerrit | Jean-Philippe Evrard proposed openstack/openstack-ansible-tests master: [WIP] Sync task files to roles https://review.openstack.org/579527 | 11:55 |
kaiokmo | evrardjp: are you there? :) | 12:00 |
evrardjp | I am | 12:00 |
kaiokmo | any thoughts why monasca is catching http://logs.openstack.org/21/578721/9/check/openstack-ansible-functional-ubuntu-xenial/c8a9e55/job-output.txt.gz#_2018-07-02_09_15_06_313089, instead of https://github.com/cloudalchemy/ansible-grafana/blob/master/vars/debian.yml? | 12:02 |
kaiokmo | it is supposed to be https://travis-ci.org/cloudalchemy/ansible-grafana/jobs/398753847#L1264 | 12:02 |
openstackgerrit | Jean-Philippe Evrard proposed openstack/openstack-ansible-tests master: [WIP] Sync task files to roles https://review.openstack.org/579527 | 12:03 |
evrardjp | kaiokmo: yes | 12:03 |
evrardjp | kaiokmo: it loads the worng file | 12:03 |
*** harryhuang_ has quit IRC | 12:04 | |
evrardjp | does it still do it? | 12:04 |
kaiokmo | yes. at least on the last patch https://review.openstack.org/#/c/578721/ | 12:04 |
evrardjp | oh yeah interesting | 12:04 |
evrardjp | include role really sucks | 12:05 |
*** cjloader has joined #openstack-ansible | 12:05 | |
evrardjp | kaiokmo: I don't have the time to debug this sadly | 12:13 |
evrardjp | but it looks like it's yet another ansible dwim_relative issue | 12:13 |
evrardjp | probably worth adding a folder to the lookup | 12:14 |
evrardjp | see also: https://docs.ansible.com/ansible/2.4/playbooks_loops.html#finding-first-matched-files | 12:14 |
openstackgerrit | Jean-Philippe Evrard proposed openstack/openstack-ansible-tests master: [WIP] Sync task files to roles https://review.openstack.org/579527 | 12:16 |
kaiokmo | ah, ok. I'll take a further look | 12:16 |
kaiokmo | thank you :) | 12:16 |
cjloader | gerrit down? | 12:23 |
cjloader | geritt** | 12:23 |
openstackgerrit | Heba Naser proposed openstack/openstack-ansible master: Use tests repo for common role test requirements https://review.openstack.org/579397 | 12:24 |
*** cjloader has quit IRC | 12:26 | |
evrardjp | sorry I can't help more right now kaiokmo | 12:28 |
evrardjp | classic move. I think it's time we implement path: though | 12:28 |
evrardjp | and everyone should do the same | 12:29 |
evrardjp | else it will become a nightmare real quick | 12:29 |
mnaser | morning | 12:30 |
mnaser | evrardjp: im abandoning my cryptography change | 12:31 |
mnaser | evrardjp: can you maybe change commit msg to change this from dnm https://review.openstack.org/#/c/574182/2 ? | 12:33 |
openstackgerrit | Jean-Philippe Evrard proposed openstack/openstack-ansible master: Allow flexible tag for Rabbit monitoring user https://review.openstack.org/579536 | 12:33 |
evrardjp | mnaser: ok | 12:33 |
openstackgerrit | Jean-Philippe Evrard proposed openstack/openstack-ansible-tests master: Do not build cryptography for tests https://review.openstack.org/574182 | 12:34 |
evrardjp | mnaser: it seems to work for suse now, so i suppose we can go ahead, and +w this one: https://review.openstack.org/#/c/574181/ | 12:34 |
mnaser | yay | 12:35 |
evrardjp | odyssey4me: your comment is referring to https://review.openstack.org/#/c/574182/ | 12:35 |
evrardjp | so you can vote on it :) | 12:35 |
evrardjp | mnaser: I removed the [dnm] | 12:36 |
mnaser | and i added my cr+2 :) | 12:36 |
evrardjp | I saw that :) | 12:41 |
evrardjp | thanks | 12:41 |
mnaser | odyssey4me: that work on delegating to localhost for openstack actions is *sweet* | 12:41 |
mnaser | maybe/hopefully we can switch to all openstack upstream modules | 12:41 |
mnaser | we could probably make a role that does the whole: add service, add service user, give admin role, add endpoints | 12:42 |
* jrosser worries about proxies with all that | 12:42 | |
evrardjp | jrosser: you would be able to delegate to another node | 12:43 |
evrardjp | liek the utility | 12:43 |
evrardjp | but we'd have to build the utility node before doing all the openstack builds | 12:43 |
evrardjp | it's thinkable | 12:43 |
jrosser | evrardjp: indeed - i was just doing some thinking :) | 12:43 |
openstackgerrit | Heba Naser proposed openstack/openstack-ansible-lxc_container_create master: Use tests repo for common role test requirements https://review.openstack.org/579539 | 12:44 |
jrosser | i already have a couple of ugly cases where deploy node proxy != target node proxy | 12:44 |
odyssey4me | the utility node is already built very early | 12:45 |
jrosser | but to be fair if this is all via the internal vip then no_proxy covers that and everything is good | 12:45 |
*** gkadam has quit IRC | 12:47 | |
*** cmart has joined #openstack-ansible | 12:48 | |
odyssey4me | jrosser: yep, it's all done to the admin endpoint - so it's internal | 12:49 |
jrosser | odyssey4me: no problem - i think the things that have unstuck me are where delegate_to has wanted to use the deploy node, but deployment_environment_variables points to a proxy that is only valid on the target nodes | 12:50 |
jrosser | hatop download being one that comes to mind | 12:50 |
odyssey4me | jrosser: well, with all of these the delegation is configurable with a single var - so if that's an issue, it's easy to work around | 12:50 |
odyssey4me | and I'm not using delegation with omission this time ;) | 12:51 |
jrosser | sure, yes i've done that as they've come up | 12:51 |
jrosser | ha yes, the error message with that is superbly obtuse | 12:51 |
openstackgerrit | Jesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-os_glance master: Move MQ vhost/user creation into role https://review.openstack.org/568517 | 12:54 |
*** jbadiapa_ has quit IRC | 12:55 | |
*** ansmith has joined #openstack-ansible | 12:55 | |
evrardjp | jrosser: I am thinking to build hatop package for rh/suse/ubuntu eventually | 12:56 |
evrardjp | so it can be easily installed | 12:56 |
jrosser | that would be nice, as it's very handy | 12:57 |
*** fghaas has quit IRC | 12:57 | |
evrardjp | yeah | 12:57 |
jrosser | evrardjp: we also did a really trivial web page view that made all the "down" things float to the top | 12:57 |
evrardjp | well I don't use it as often as I was 4 years ago, but still. | 12:57 |
evrardjp | jrosser: what do you mean? | 12:57 |
jrosser | there's also the haproxy stats http stuff you can enable | 12:58 |
evrardjp | yeah, and? | 12:58 |
*** gkadam has joined #openstack-ansible | 12:58 | |
evrardjp | I almost never use that web interface, as I use the admin socks stats | 12:58 |
jrosser | but unless i'm missing something there didnt seem to be a way of prioritising the things that were down | 12:58 |
jrosser | it's pages and pages long | 12:59 |
evrardjp | if you integrate this into your monitoring it becomes very clear : ) | 12:59 |
*** vnogin has quit IRC | 13:01 | |
*** fghaas has joined #openstack-ansible | 13:03 | |
*** vnogin has joined #openstack-ansible | 13:05 | |
*** ansmith has quit IRC | 13:06 | |
*** chhagarw has joined #openstack-ansible | 13:07 | |
openstackgerrit | Jesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-os_ironic master: Execute service setup against a delegated host using Ansible built-in modules https://review.openstack.org/579514 | 13:09 |
*** ansmith has joined #openstack-ansible | 13:09 | |
*** vnogin has quit IRC | 13:09 | |
*** vnogin has joined #openstack-ansible | 13:10 | |
*** chhavi__ has joined #openstack-ansible | 13:10 | |
openstackgerrit | Heba Naser proposed openstack/openstack-ansible-lxc_hosts master: Use tests repo for common role test requirements https://review.openstack.org/579548 | 13:12 |
*** chhagarw has quit IRC | 13:12 | |
odyssey4me | cores - I could do with another review for https://review.openstack.org/579208 and https://review.openstack.org/579231 to move things along | 13:15 |
*** jbadiapa has joined #openstack-ansible | 13:18 | |
odyssey4me | also https://review.openstack.org/579463 and https://review.openstack.org/574841 | 13:24 |
odyssey4me | and of course https://review.openstack.org/#/q/topic:move-db-setup-into-roles+status:open | 13:26 |
openstackgerrit | Heba Naser proposed openstack/openstack-ansible-memcached_server master: Use tests repo for common role test requirements https://review.openstack.org/579554 | 13:26 |
*** tux_ has joined #openstack-ansible | 13:30 | |
*** hachi has joined #openstack-ansible | 13:32 | |
openstackgerrit | Heba Naser proposed openstack/openstack-ansible-nspawn_container_create master: Use tests repo for common role test requirements https://review.openstack.org/579558 | 13:34 |
*** throwsb1 has joined #openstack-ansible | 13:36 | |
openstackgerrit | Heba Naser proposed openstack/openstack-ansible master: Use tests repo for common role test requirements https://review.openstack.org/579397 | 13:38 |
openstackgerrit | Jesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-tests master: Install hacking first and let it pull in right linter versions https://review.openstack.org/579562 | 13:40 |
openstackgerrit | Jesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-tests master: Install hacking first and let it pull in right linter versions https://review.openstack.org/579562 | 13:42 |
*** jwitko has joined #openstack-ansible | 13:45 | |
openstackgerrit | Heba Naser proposed openstack/openstack-ansible-nspawn_hosts master: Use tests repo for common role test requirements https://review.openstack.org/579564 | 13:55 |
*** sxc731 has joined #openstack-ansible | 13:57 | |
*** sxc731 has left #openstack-ansible | 13:57 | |
*** gkadam_ has joined #openstack-ansible | 13:57 | |
*** gkadam has quit IRC | 13:58 | |
openstackgerrit | Heba Naser proposed openstack/openstack-ansible-openstack_hosts master: Use tests repo for common role test requirements https://review.openstack.org/579565 | 14:01 |
*** dxiri has joined #openstack-ansible | 14:01 | |
tux_ | cloudnull: I have sent you email but let me ask here too, is it ok to run Ceph mon on Infra* nodes? | 14:02 |
tux_ | what is the risk here? | 14:02 |
tux_ | Anyone here who does that? | 14:03 |
evrardjp | ceph clusters separate are the best. | 14:03 |
evrardjp | IMO | 14:03 |
tux_ | I have only 5 server for storage and if i turn 3 node for mon then run little short :( | 14:03 |
tux_ | evrardjp: totally agreed with you idea to run seperate | 14:04 |
logan- | tux_: i do that. the mons are not resource heavy and it works fine in my experience. | 14:05 |
tux_ | logan-: how big your cluster is? | 14:06 |
logan- | ~100 disks | 14:06 |
tux_ | TB? | 14:06 |
tux_ | I heard mon keep logs so it may fill your disk if you don't have enough space.. i have 400GB disk on infra nodes | 14:07 |
logan- | around 250tb | 14:08 |
tux_ | logan-: holy smoke... i have only 5TB storage requirement | 14:08 |
logan- | /var/lib/ceph on the mon nodes is less than 100mb | 14:09 |
logan- | until your cluster grows quite a bit i think it is overkill to dedicated hardware to mons | 14:10 |
openstackgerrit | Heba Naser proposed openstack/openstack-ansible-openstack_openrc master: Use tests repo for common role test requirements https://review.openstack.org/579566 | 14:10 |
*** maybebuggy has quit IRC | 14:13 | |
openstackgerrit | Heba Naser proposed openstack/openstack-ansible-os_aodh master: Use tests repo for common role test requirements https://review.openstack.org/579567 | 14:16 |
openstackgerrit | Merged openstack/openstack-ansible master: Do not build cryptography https://review.openstack.org/574181 | 14:17 |
openstackgerrit | Heba Naser proposed openstack/openstack-ansible-os_barbican master: Use tests repo for common role test requirements https://review.openstack.org/579569 | 14:20 |
*** armaan has quit IRC | 14:21 | |
*** armaan has joined #openstack-ansible | 14:22 | |
openstackgerrit | Heba Naser proposed openstack/openstack-ansible-os_ceilometer master: Use tests repo for common role test requirements https://review.openstack.org/579572 | 14:25 |
openstackgerrit | Merged openstack/openstack-ansible-os_nova master: Restore openstack-ansible-upgrade-ubuntu-xenial https://review.openstack.org/575153 | 14:27 |
openstackgerrit | Heba Naser proposed openstack/openstack-ansible-os_cinder master: Use tests repo for common role test requirements https://review.openstack.org/579574 | 14:31 |
openstackgerrit | Heba Naser proposed openstack/openstack-ansible-os_designate master: Use tests repo for common role test requirements https://review.openstack.org/579575 | 14:36 |
odyssey4me | tux_: the thing about storage that you can easily scale sideways is that it always grows ;) | 14:36 |
*** jwitko has quit IRC | 14:38 | |
*** jwitko has joined #openstack-ansible | 14:39 | |
*** p-to-q-upgrade has joined #openstack-ansible | 14:40 | |
openstackgerrit | Heba Naser proposed openstack/openstack-ansible-os_gnocchi master: Use tests repo for common role test requirements https://review.openstack.org/579578 | 14:42 |
*** cmart has quit IRC | 14:42 | |
*** elbragstad is now known as lbragstad | 14:43 | |
p-to-q-upgrade | Hi, wanted to report issues with Pike to Queens upgrade I encountered. In short: cinder broke while doing "Clean up unnecessary containers". Details here: https://pastebin.com/raw/r6cV37kr | 14:45 |
p-to-q-upgrade | Remains to be understood how to remove old containers in a way that does not brake cinder services. | 14:45 |
odyssey4me | p-to-q-upgrade: please register a bug instead | 14:47 |
*** armaan has quit IRC | 14:47 | |
odyssey4me | it seems like perhaps the new services weren't running properly? can you add to the bug any diagnostics you performed on the new services which are all running in the same api container? | 14:47 |
*** admin0 has quit IRC | 14:48 | |
*** ansmith has quit IRC | 14:48 | |
openstackgerrit | Heba Naser proposed openstack/openstack-ansible-os_heat master: Use tests repo for common role test requirements https://review.openstack.org/579584 | 14:49 |
*** ansmith has joined #openstack-ansible | 14:50 | |
p-to-q-upgrade | odyssey4me I just enabled debug=True, restarted everything and watched for logs. Ok, will hind right place to register the bug. | 14:50 |
odyssey4me | p-to-q-upgrade: see the channel topic, bugs registered in launchpad | 14:50 |
p-to-q-upgrade | right :) | 14:51 |
openstackgerrit | Cam J. Loader proposed openstack/openstack-ansible stable/queens: Update ironic role SHA for conf fixes https://review.openstack.org/579585 | 14:51 |
*** gkadam_ has quit IRC | 14:52 | |
odyssey4me | you may also need to specify anything special you have for your cinder-volume service in the bug report - what driver, is the service in a container or outside the container - did you verify that the new services were up and registered their agents | 14:52 |
odyssey4me | all that sort of useful stuff | 14:52 |
*** gkadam_ has joined #openstack-ansible | 14:52 | |
openstackgerrit | Heba Naser proposed openstack/openstack-ansible-os_horizon master: Use tests repo for common role test requirements https://review.openstack.org/579586 | 14:53 |
cloudnull | mornings | 14:53 |
*** admin0 has joined #openstack-ansible | 14:55 | |
mnaser | morning cloudnull ! :) | 14:57 |
odyssey4me | o/ cloudnull mnaser | 14:58 |
mnaser | side node: what do the ssl jobs stand for | 14:58 |
*** ansmith has quit IRC | 14:58 | |
odyssey4me | mnaser: they were implemented by a guy who was doing a ssl galera connectivity implementation... but he disappeared part way through the work | 14:59 |
mnaser | cause i got the galera ssl jobs to be passing and voting | 14:59 |
mnaser | but i guess the rest of the services would have to be updated and figured out | 15:00 |
odyssey4me | yep | 15:00 |
mnaser | cause if no one is picking up the work | 15:00 |
mnaser | we're running an extra job for no reason | 15:00 |
odyssey4me | yeah, I'd say kill them - if someone wants to pick up the work they can do it - but the jobs are just wasting resources at this time | 15:00 |
odyssey4me | if it's not enough of a priority to the contributors we have right now, then none of them will work on it | 15:01 |
tux_ | odyssey4me: first starting i am planning to use infra* node for ceph mon and in future if we scale then i will build add more mon node and shift load from infra to their own separate monitor node, do you think that is possible? | 15:01 |
odyssey4me | Once all the python build simplification is finally done, I might pick up some security related work for fun. | 15:01 |
odyssey4me | I'm thinking being able to deploy using a non-root user would be the first thing I'd like to tackle, then maybe galera/ssl later. | 15:02 |
*** cshen has quit IRC | 15:02 | |
openstackgerrit | Heba Naser proposed openstack/openstack-ansible-os_magnum master: Use tests repo for common role test requirements https://review.openstack.org/579588 | 15:02 |
*** admin0 has quit IRC | 15:04 | |
*** admin0 has joined #openstack-ansible | 15:04 | |
openstackgerrit | Heba Naser proposed openstack/openstack-ansible-os_molteniron master: Use tests repo for common role test requirements https://review.openstack.org/579589 | 15:09 |
*** ansmith has joined #openstack-ansible | 15:11 | |
openstackgerrit | Heba Naser proposed openstack/openstack-ansible-os_neutron master: Use tests repo for common role test requirements https://review.openstack.org/579595 | 15:16 |
openstackgerrit | Merged openstack/openstack-ansible-tests master: Do not build cryptography for tests https://review.openstack.org/574182 | 15:20 |
openstackgerrit | Merged openstack/openstack-ansible-tests master: Use tests repo for common role test requirements https://review.openstack.org/579208 | 15:20 |
openstackgerrit | Merged openstack/openstack-ansible-os_swift master: Replace swiftoperator role with swift_operator_role variable https://review.openstack.org/577332 | 15:22 |
openstackgerrit | Heba Naser proposed openstack/openstack-ansible-os_panko master: Use tests repo for common role test requirements https://review.openstack.org/579597 | 15:22 |
prometheanfire | odyssey4me: evrardjp do you think a backport of the ironic review is warranted, it may be useful to not have to do a hard changeover for rocky | 15:25 |
openstackgerrit | Heba Naser proposed openstack/openstack-ansible-os_rally master: Use tests repo for common role test requirements https://review.openstack.org/579598 | 15:26 |
odyssey4me | prometheanfire: I wouldn't object to it - I think it makes it easier to maintain, and Queens is still reasonably young | 15:30 |
evrardjp | prometheanfire: please propose and let the review play | 15:30 |
openstackgerrit | Merged openstack/openstack-ansible-tests master: Document the purpose of the test-requirements.txt file https://review.openstack.org/579231 | 15:30 |
openstackgerrit | Merged openstack/openstack-ansible-tests master: Add shade & python-keystoneclient to the tox venv https://review.openstack.org/579233 | 15:30 |
openstackgerrit | Heba Naser proposed openstack/openstack-ansible-os_sahara master: Use tests repo for common role test requirements https://review.openstack.org/579602 | 15:31 |
*** sxc731 has joined #openstack-ansible | 15:33 | |
*** sxc731 has left #openstack-ansible | 15:33 | |
*** tux__ has joined #openstack-ansible | 15:33 | |
openstackgerrit | Heba Naser proposed openstack/openstack-ansible-os_swift master: Use tests repo for common role test requirements https://review.openstack.org/579605 | 15:35 |
prometheanfire | k | 15:35 |
*** tux_ has quit IRC | 15:36 | |
openstackgerrit | Heba Naser proposed openstack/openstack-ansible-os_tacker master: Use tests repo for common role test requirements https://review.openstack.org/579607 | 15:41 |
openstackgerrit | Merged openstack/openstack-ansible-apt_package_pinning master: Use tests repo for common role test requirements https://review.openstack.org/579398 | 15:42 |
openstackgerrit | Merged openstack/openstack-ansible-galera_client master: Use tests repo for common role test requirements https://review.openstack.org/579401 | 15:43 |
openstackgerrit | Merged openstack/ansible-config_template master: Use tests repo for common role test requirements https://review.openstack.org/579387 | 15:44 |
openstackgerrit | Heba Naser proposed openstack/openstack-ansible-os_trove master: Use tests repo for common role test requirements https://review.openstack.org/579610 | 15:45 |
openstackgerrit | Merged openstack/openstack-ansible-ceph_client master: Use tests repo for common role test requirements https://review.openstack.org/579400 | 15:46 |
openstackgerrit | Merged openstack/openstack-ansible-haproxy_server master: Use tests repo for common role test requirements https://review.openstack.org/579404 | 15:46 |
openstackgerrit | Merged openstack/openstack-ansible-memcached_server master: Use tests repo for common role test requirements https://review.openstack.org/579554 | 15:47 |
openstackgerrit | Merged openstack/openstack-ansible-lxc_hosts master: Use tests repo for common role test requirements https://review.openstack.org/579548 | 15:48 |
openstackgerrit | Merged openstack/openstack-ansible-os_cloudkitty master: Move database creation into role https://review.openstack.org/574867 | 15:48 |
openstackgerrit | Matthew Thode proposed openstack/openstack-ansible-os_ironic stable/queens: Overhaul ironic role https://review.openstack.org/579613 | 15:48 |
openstackgerrit | Heba Naser proposed openstack/openstack-ansible-pip_install master: Use tests repo for common role test requirements https://review.openstack.org/579614 | 15:49 |
openstackgerrit | Merged openstack/openstack-ansible-openstack_openrc master: Use tests repo for common role test requirements https://review.openstack.org/579566 | 15:51 |
openstackgerrit | Jesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-os_zun master: Use tests repo for common role test requirements https://review.openstack.org/579380 | 15:51 |
openstackgerrit | Merged openstack/openstack-ansible-nspawn_hosts master: Use tests repo for common role test requirements https://review.openstack.org/579564 | 15:52 |
openstackgerrit | Merged openstack/openstack-ansible-openstack_hosts master: Use tests repo for common role test requirements https://review.openstack.org/579565 | 15:55 |
*** vnogin has quit IRC | 15:55 | |
*** ansmith has quit IRC | 15:55 | |
openstackgerrit | Merged openstack/openstack-ansible-lxc_container_create master: Use tests repo for common role test requirements https://review.openstack.org/579539 | 15:56 |
openstackgerrit | Heba Naser proposed openstack/openstack-ansible-plugins master: Use tests repo for common role test requirements https://review.openstack.org/579617 | 15:56 |
tomtom001 | Hello, I have successfully installed queens and am trying to upload a cirros image to glance. I have verified that swift is working correctly and glance seems to be but when I upload the cirros image the following traceback appears: http://paste.openstack.org/show/724764/ has anyone experienced the same issue? Or does anyone know what the problem is? | 15:58 |
tomtom001 | I have verified via telnet that the address shown is reachable on port 9292 but it seems to break during the upload. | 15:58 |
*** gyee has joined #openstack-ansible | 15:58 | |
*** cshen has joined #openstack-ansible | 15:58 | |
*** cjloader has joined #openstack-ansible | 16:00 | |
openstackgerrit | Heba Naser proposed openstack/openstack-ansible-rabbitmq_server master: Use tests repo for common role test requirements https://review.openstack.org/579619 | 16:02 |
openstackgerrit | Merged openstack/ansible-hardening master: Use tests repo for common role test requirements https://review.openstack.org/579396 | 16:05 |
*** ansmith has joined #openstack-ansible | 16:05 | |
openstackgerrit | Heba Naser proposed openstack/openstack-ansible-repo_build master: Use tests repo for common role test requirements https://review.openstack.org/579621 | 16:06 |
openstackgerrit | Heba Naser proposed openstack/openstack-ansible-repo_server master: Use tests repo for common role test requirements https://review.openstack.org/579622 | 16:10 |
openstackgerrit | Merged openstack/openstack-ansible-os_barbican master: Use tests repo for common role test requirements https://review.openstack.org/579569 | 16:12 |
*** armaan has joined #openstack-ansible | 16:12 | |
openstackgerrit | Jesse Pretorius (odyssey4me) proposed openstack/openstack-ansible master: Use tests repo for common role test requirements https://review.openstack.org/579397 | 16:12 |
openstackgerrit | Merged openstack/openstack-ansible-os_designate master: Use tests repo for common role test requirements https://review.openstack.org/579575 | 16:13 |
openstackgerrit | Heba Naser proposed openstack/openstack-ansible-rsyslog_client master: Use tests repo for common role test requirements https://review.openstack.org/579623 | 16:14 |
*** cshen has quit IRC | 16:14 | |
openstackgerrit | Merged openstack/openstack-ansible-os_keystone master: Use tests repo for common role test requirements https://review.openstack.org/579213 | 16:14 |
*** exodusftw has quit IRC | 16:14 | |
openstackgerrit | Merged openstack/openstack-ansible-os_ceilometer master: Use tests repo for common role test requirements https://review.openstack.org/579572 | 16:16 |
openstackgerrit | Heba Naser proposed openstack/openstack-ansible-rsyslog_server master: Use tests repo for common role test requirements https://review.openstack.org/579624 | 16:17 |
*** stuartgr1 has quit IRC | 16:20 | |
openstackgerrit | Merged openstack/openstack-ansible-os_glance master: Use tests repo for common role test requirements https://review.openstack.org/579373 | 16:21 |
openstackgerrit | Jesse Pretorius (odyssey4me) proposed openstack/openstack-ansible master: Correct E408 pep8 lint errors https://review.openstack.org/579625 | 16:23 |
openstackgerrit | Jesse Pretorius (odyssey4me) proposed openstack/openstack-ansible master: Use tests repo for common role test requirements https://review.openstack.org/579397 | 16:24 |
openstackgerrit | Merged openstack/openstack-ansible-os_ceilometer master: Use tests repo for common role test requirements https://review.openstack.org/579374 | 16:24 |
*** chyka has joined #openstack-ansible | 16:24 | |
*** chyka has joined #openstack-ansible | 16:25 | |
openstackgerrit | Heba Naser proposed openstack/openstack-ansible-ops master: Use tests repo for common role test requirements https://review.openstack.org/579628 | 16:26 |
openstackgerrit | Merged openstack/openstack-ansible-os_molteniron master: Use tests repo for common role test requirements https://review.openstack.org/579589 | 16:27 |
cjloader | can I get reviews on https://review.openstack.org/#/c/579585/? trying to bump os-ironic sha?? | 16:28 |
odyssey4me | cjloader: looking | 16:29 |
odyssey4me | cjloader: looks good, +2+w | 16:29 |
openstackgerrit | Heba Naser proposed openstack/openstack-ansible-os_almanach master: Use tests repo for common role test requirements https://review.openstack.org/579629 | 16:29 |
cjloader | odyssey4me: ty | 16:30 |
openstackgerrit | Merged openstack/openstack-ansible-os_cinder master: Use tests repo for common role test requirements https://review.openstack.org/579574 | 16:30 |
*** erolg has quit IRC | 16:31 | |
openstackgerrit | Heba Naser proposed openstack/openstack-ansible-os_blazar master: Use tests repo for common role test requirements https://review.openstack.org/579632 | 16:33 |
openstackgerrit | Andy Smith proposed openstack/openstack-ansible-os_barbican master: Update to use oslo.messaging service for RPCand Notify https://review.openstack.org/574864 | 16:33 |
*** dxiri has quit IRC | 16:34 | |
*** epalper has quit IRC | 16:34 | |
openstackgerrit | Merged openstack/openstack-ansible-os_ironic master: Use tests repo for common role test requirements https://review.openstack.org/579375 | 16:34 |
*** dxiri has joined #openstack-ansible | 16:35 | |
*** armaan has quit IRC | 16:36 | |
*** armaan has joined #openstack-ansible | 16:36 | |
openstackgerrit | Heba Naser proposed openstack/openstack-ansible-os_cloudkitty master: Use tests repo for common role test requirements https://review.openstack.org/579633 | 16:38 |
*** lvdombrkr has quit IRC | 16:40 | |
openstackgerrit | Heba Naser proposed openstack/openstack-ansible-os_congress master: Use tests repo for common role test requirements https://review.openstack.org/579636 | 16:42 |
openstackgerrit | Merged openstack/openstack-ansible-os_nova master: Use tests repo for common role test requirements https://review.openstack.org/579377 | 16:44 |
openstackgerrit | Heba Naser proposed openstack/openstack-ansible-os_masakari master: Use tests repo for common role test requirements https://review.openstack.org/579637 | 16:48 |
openstackgerrit | Heba Naser proposed openstack/openstack-ansible-os_monasca master: Use tests repo for common role test requirements https://review.openstack.org/579638 | 16:51 |
openstackgerrit | Merged openstack/openstack-ansible-os_heat master: Use tests repo for common role test requirements https://review.openstack.org/579584 | 16:53 |
openstackgerrit | Merged openstack/openstack-ansible-os_magnum master: Use tests repo for common role test requirements https://review.openstack.org/579376 | 16:53 |
openstackgerrit | Merged openstack/openstack-ansible-os_octavia master: Use tests repo for common role test requirements https://review.openstack.org/579378 | 16:53 |
*** cshen has joined #openstack-ansible | 16:54 | |
openstackgerrit | Heba Naser proposed openstack/openstack-ansible-os_monasca-agent master: Use tests repo for common role test requirements https://review.openstack.org/579639 | 16:55 |
odyssey4me | mnaser: if you're about, evrardjp was looking for a second opinion on https://review.openstack.org/579468 | 16:57 |
openstackgerrit | Merged openstack/openstack-ansible-os_gnocchi master: Use tests repo for common role test requirements https://review.openstack.org/579578 | 16:57 |
openstackgerrit | Heba Naser proposed openstack/openstack-ansible-os_monasca-ui master: Use tests repo for common role test requirements https://review.openstack.org/579640 | 16:58 |
odyssey4me | also, any other cores about - could do with reviews for https://review.openstack.org/579470 / https://review.openstack.org/579469 / https://review.openstack.org/579471 | 17:00 |
openstackgerrit | Heba Naser proposed openstack/openstack-ansible-os_searchlight master: Use tests repo for common role test requirements https://review.openstack.org/579641 | 17:00 |
openstackgerrit | Heba Naser proposed openstack/openstack-ansible-os_watcher master: Use tests repo for common role test requirements https://review.openstack.org/579642 | 17:03 |
openstackgerrit | Merged openstack/openstack-ansible-os_tempest master: Use tests repo for common role test requirements https://review.openstack.org/579379 | 17:04 |
openstackgerrit | Heba Naser proposed openstack/openstack-ansible-os_zaqar master: Use tests repo for common role test requirements https://review.openstack.org/579643 | 17:06 |
openstackgerrit | Merged openstack/openstack-ansible-os_aodh master: Use tests repo for common role test requirements https://review.openstack.org/579567 | 17:06 |
openstackgerrit | Jesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-os_ironic master: Execute service setup against a delegated host using Ansible built-in modules https://review.openstack.org/579514 | 17:07 |
*** electrofelix has quit IRC | 17:08 | |
*** rmart04 has quit IRC | 17:09 | |
openstackgerrit | Jesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-os_ironic master: Execute service setup against a delegated host using Ansible built-in modules https://review.openstack.org/579514 | 17:09 |
openstackgerrit | Jesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-os_ceilometer master: Execute service setup against a delegated host using Ansible built-in modules https://review.openstack.org/579509 | 17:11 |
*** armaan has quit IRC | 17:12 | |
*** armaan has joined #openstack-ansible | 17:12 | |
openstackgerrit | Andy Smith proposed openstack/openstack-ansible-os_magnum master: Update to use oslo.messaging service for RPC and Notify https://review.openstack.org/579645 | 17:12 |
*** FrankZhang has joined #openstack-ansible | 17:14 | |
*** chyka has quit IRC | 17:16 | |
*** tux__ has quit IRC | 17:16 | |
*** chyka has joined #openstack-ansible | 17:18 | |
*** chyka has quit IRC | 17:18 | |
*** chhavi__ has quit IRC | 17:34 | |
*** jwitko has quit IRC | 17:35 | |
openstackgerrit | Jesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-os_barbican master: Execute service setup against a delegated host using Ansible built-in modules https://review.openstack.org/579648 | 17:36 |
openstackgerrit | Taseer Ahmed proposed openstack/openstack-ansible-os_blazar master: This patch ensures that the functional tests for Blazar pass. https://review.openstack.org/571388 | 17:38 |
*** cshen has quit IRC | 17:41 | |
*** sxc731 has joined #openstack-ansible | 17:41 | |
*** cshen has joined #openstack-ansible | 17:42 | |
tomtom001 | I had an issue with glance not uploading to swift, by default it was set to v2, by changing to v3 the broken pipe issue no longer occurred. I posted the changes I made here: https://bugs.launchpad.net/glance/+bug/1772651 | 17:42 |
openstack | Launchpad bug 1772651 in Glance "Glance image creation fails in openstack Queens " [Undecided,Incomplete] | 17:42 |
*** sxc731 has left #openstack-ansible | 17:43 | |
openstackgerrit | Merged openstack/openstack-ansible-os_zun master: Use tests repo for common role test requirements https://review.openstack.org/579380 | 17:44 |
openstackgerrit | Jesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-os_cinder master: Execute service setup against a delegated host using Ansible built-in modules https://review.openstack.org/579651 | 17:45 |
*** tux_ has joined #openstack-ansible | 17:50 | |
*** hachi has quit IRC | 17:55 | |
openstackgerrit | Jesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-os_designate master: Execute service setup against a delegated host using Ansible built-in modules https://review.openstack.org/579655 | 17:55 |
openstackgerrit | Jesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-os_designate master: Execute service setup against a delegated host using Ansible built-in modules https://review.openstack.org/579655 | 17:56 |
mnaser | odyssey4me: regarding https://review.openstack.org/#/c/579468/ for the repo_server role, i actually fell back to using upstream nginx repos directly | 18:08 |
mnaser | do we want to follow the same pattern? given that we mostly use upstream repos for most other projects anyways? | 18:08 |
*** cmart has joined #openstack-ansible | 18:09 | |
*** tux_ has quit IRC | 18:12 | |
*** rmart04 has joined #openstack-ansible | 18:14 | |
*** FrankZhang_ has joined #openstack-ansible | 18:15 | |
*** cshen has quit IRC | 18:15 | |
*** FrankZhang has quit IRC | 18:19 | |
openstackgerrit | Merged openstack/ansible-role-systemd_service master: Follow the new PTI for document build https://review.openstack.org/579471 | 18:20 |
openstackgerrit | Matthew Thode proposed openstack/openstack-ansible-os_ironic stable/queens: Overhaul ironic role https://review.openstack.org/579613 | 18:20 |
openstackgerrit | Merged openstack/ansible-role-systemd_mount master: Follow the new PTI for document build https://review.openstack.org/579469 | 18:21 |
openstackgerrit | Merged openstack/openstack-ansible-repo_server master: Use tests repo for common role test requirements https://review.openstack.org/579622 | 18:24 |
mnaser | cloudnull: i found the issue with systemd_networkd failing of certain os | 18:25 |
mnaser | https://github.com/ansible/ansible/issues/42004 | 18:25 |
mnaser | https://github.com/openstack/ansible-role-systemd_networkd/blob/305db5348ecf10c432d5820d21c46ea3d2f2eac4/tests/test.yml#L24-L28 | 18:25 |
*** rmart04 has quit IRC | 18:25 | |
mnaser | cloudnull: so either install pause somewhere, or add ncurses | 18:26 |
mnaser | err | 18:26 |
mnaser | install curses or drop pause | 18:26 |
cloudnull | ORLY!? | 18:26 |
mnaser | http://logs.openstack.org/91/576991/2/check/openstack-ansible-functional-opensuse-423/d9e1b88/job-output.txt.gz#_2018-06-29_07_26_30_062807 | 18:27 |
mnaser | or maybe we install python-ncurses but only in the test suite | 18:27 |
mnaser | or whatever the suse equiv is | 18:27 |
openstackgerrit | Merged openstack/openstack-ansible-rsyslog_client master: Use tests repo for common role test requirements https://review.openstack.org/579623 | 18:29 |
openstackgerrit | Kevin Carter (cloudnull) proposed openstack/ansible-role-systemd_networkd master: remove pause to fix upstream bug https://review.openstack.org/579662 | 18:31 |
cloudnull | mnaser ^ I just removed it | 18:31 |
mnaser | man | 18:31 |
mnaser | the realpath cleanups | 18:31 |
mnaser | make my eyes happy | 18:31 |
openstackgerrit | Kevin Carter (cloudnull) proposed openstack/ansible-role-systemd_networkd master: Remove pause to fix upstream bug https://review.openstack.org/579662 | 18:32 |
openstackgerrit | Merged openstack/openstack-ansible-pip_install master: Use tests repo for common role test requirements https://review.openstack.org/579614 | 18:32 |
openstackgerrit | Merged openstack/openstack-ansible-rsyslog_server master: Use tests repo for common role test requirements https://review.openstack.org/579624 | 18:35 |
cloudnull | odyssey4me https://review.openstack.org/#/q/topic:bp/python-build-install-simplification+(status:open) these changes look great! | 18:35 |
openstackgerrit | Jesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-os_heat master: Execute service setup against a delegated host using Ansible built-in modules https://review.openstack.org/579663 | 18:35 |
openstackgerrit | Merged openstack/openstack-ansible-os_rally master: Move database creation into role https://review.openstack.org/578852 | 18:36 |
openstackgerrit | Merged openstack/ansible-role-systemd_networkd master: Add release note link in README https://review.openstack.org/579073 | 18:36 |
openstackgerrit | Jesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-os_heat master: Execute service setup against a delegated host using Ansible built-in modules https://review.openstack.org/579663 | 18:39 |
*** FrankZhang_ has quit IRC | 18:39 | |
*** FrankZhang has joined #openstack-ansible | 18:40 | |
openstackgerrit | Merged openstack/openstack-ansible-repo_build master: Use tests repo for common role test requirements https://review.openstack.org/579621 | 18:41 |
*** tux_ has joined #openstack-ansible | 18:41 | |
openstackgerrit | Merged openstack/openstack-ansible-os_octavia master: Move database creation into role https://review.openstack.org/578849 | 18:41 |
openstackgerrit | Jesse Pretorius (odyssey4me) proposed openstack/ansible-role-systemd_networkd master: Switch to using project-templates https://review.openstack.org/576991 | 18:41 |
openstackgerrit | Jesse Pretorius (odyssey4me) proposed openstack/ansible-role-systemd_networkd master: Follow the new PTI for document build https://review.openstack.org/579470 | 18:42 |
openstackgerrit | Merged openstack/openstack-ansible-plugins master: Use tests repo for common role test requirements https://review.openstack.org/579617 | 18:42 |
odyssey4me | cloudnull: thanks - that'll really help us to clean up the host installs somewhat | 18:42 |
openstackgerrit | Merged openstack/openstack-ansible-os_monasca-agent master: Use tests repo for common role test requirements https://review.openstack.org/579639 | 18:43 |
odyssey4me | and hopefully by the end of it we can also kill a few of our custom modules | 18:43 |
cloudnull | ++ | 18:43 |
openstackgerrit | Merged openstack/openstack-ansible-os_monasca-ui master: Use tests repo for common role test requirements https://review.openstack.org/579640 | 18:46 |
openstackgerrit | Merged openstack/openstack-ansible-os_monasca master: Use tests repo for common role test requirements https://review.openstack.org/579638 | 18:47 |
openstackgerrit | Merged openstack/ansible-role-systemd_service master: Add release note link in README https://review.openstack.org/579076 | 18:48 |
openstackgerrit | Merged openstack/ansible-role-python_venv_build master: Add EPEL repo for nginx packages on CentOS tests https://review.openstack.org/579468 | 18:50 |
openstackgerrit | Merged openstack/openstack-ansible-rabbitmq_server master: Use tests repo for common role test requirements https://review.openstack.org/579619 | 18:51 |
prometheanfire | evrardjp: the sd_bus thing, do we know what's causing it? | 18:52 |
openstackgerrit | Merged openstack/openstack-ansible-galera_server master: Use tests repo for common role test requirements https://review.openstack.org/579403 | 18:53 |
mnaser | cloudnull, odyssey4me: https://review.openstack.org/#/c/578887/ do we want to approve this to move the swift gate through? it looks like there probably won't be anyone stepping up | 18:55 |
*** jwitko has joined #openstack-ansible | 18:59 | |
openstackgerrit | Merged openstack/openstack-ansible-os_almanach master: Use tests repo for common role test requirements https://review.openstack.org/579629 | 19:04 |
openstackgerrit | Merged openstack/openstack-ansible-os_congress master: Use tests repo for common role test requirements https://review.openstack.org/579636 | 19:05 |
openstackgerrit | Merged openstack/openstack-ansible-os_magnum master: Move database creation into role https://review.openstack.org/578812 | 19:16 |
*** cshen has joined #openstack-ansible | 19:20 | |
openstackgerrit | Merged openstack/openstack-ansible-os_watcher master: Use tests repo for common role test requirements https://review.openstack.org/579642 | 19:24 |
openstackgerrit | Merged openstack/openstack-ansible-os_cloudkitty master: Use tests repo for common role test requirements https://review.openstack.org/579633 | 19:24 |
*** cjloader has quit IRC | 19:24 | |
*** cmart has quit IRC | 19:25 | |
*** vnogin has joined #openstack-ansible | 19:26 | |
*** cmart has joined #openstack-ansible | 19:28 | |
*** shardy has quit IRC | 19:32 | |
*** cshen has quit IRC | 19:38 | |
openstackgerrit | Merged openstack/openstack-ansible-os_swift master: Use tests repo for common role test requirements https://review.openstack.org/579605 | 19:45 |
openstackgerrit | Merged openstack/ansible-role-systemd_networkd master: Remove pause to fix upstream bug https://review.openstack.org/579662 | 19:50 |
openstackgerrit | Merged openstack/ansible-role-systemd_networkd master: Switch to using project-templates https://review.openstack.org/576991 | 19:55 |
*** cshen has joined #openstack-ansible | 19:57 | |
openstackgerrit | Jesse Pretorius (odyssey4me) proposed openstack/ansible-role-systemd_networkd master: Follow the new PTI for document build https://review.openstack.org/579470 | 19:59 |
*** exodusftw has joined #openstack-ansible | 20:00 | |
*** cjloader has joined #openstack-ansible | 20:03 | |
*** cjloader has quit IRC | 20:07 | |
*** exodusftw has quit IRC | 20:36 | |
*** cjloader has joined #openstack-ansible | 20:36 | |
openstackgerrit | Merged openstack/openstack-ansible master: Move database creation into role (designate) https://review.openstack.org/578809 | 20:37 |
openstackgerrit | Merged openstack/openstack-ansible-os_ironic master: Add packages required for osprofiler https://review.openstack.org/573506 | 20:39 |
*** exodusftw has joined #openstack-ansible | 20:41 | |
*** cjloader has quit IRC | 20:41 | |
*** exodusftw has quit IRC | 20:46 | |
*** exodusftw has joined #openstack-ansible | 20:46 | |
*** tux_ has quit IRC | 20:49 | |
*** cmart has quit IRC | 20:49 | |
*** ansmith has quit IRC | 20:50 | |
guilhermesp | evrardjp: indeed includ_role sucks https://github.com/ansible/ansible/issues/21890 Someone is suggesting a PR that fixes it, but I assume that won't be tangible in our case, isn't it? | 21:10 |
*** jwitko_ has joined #openstack-ansible | 21:10 | |
guilhermesp | using only the "role" statement in the play would be an option too? | 21:11 |
*** jwitko has quit IRC | 21:15 | |
*** jwitko_ has quit IRC | 21:16 | |
openstackgerrit | Merged openstack/ansible-role-systemd_networkd master: Follow the new PTI for document build https://review.openstack.org/579470 | 21:19 |
*** throwsb1 has quit IRC | 21:20 | |
*** nicolasbock has quit IRC | 21:24 | |
cloudnull | wow, guilhermesp - that's a bummer. | 21:24 |
cloudnull | with include_role it would seem that the documented functionality of "private" doesn't actually work | 21:24 |
cloudnull | odyssey4me ^ | 21:24 |
guilhermesp | indeed cloudnull Trying to figure out what to do | 21:25 |
cloudnull | mnaser +2 on the swift se linux change . | 21:25 |
cloudnull | guilhermesp what was the issue you are seein g?v | 21:25 |
cloudnull | just trying to access vars from an included role? | 21:28 |
openstackgerrit | Matthew Thode proposed openstack/openstack-ansible-tests master: Use requirements from ara to install ara https://review.openstack.org/579694 | 21:31 |
*** erolg has joined #openstack-ansible | 21:34 | |
guilhermesp | cloudnull: yes, evrardjp was discussing this earlier with odyssey4me and kaiokmo . Insetead of getting the vars from ansible-grafana, include_role is getting vars from our monasca role | 21:34 |
prometheanfire | cloudnull: evrardjp odyssey4me ^ may fix the ara/sd_bus thing :P | 21:34 |
guilhermesp | which is not the expected behavior | 21:34 |
*** tonytan4ever has joined #openstack-ansible | 21:35 | |
guilhermesp | here: http://logs.openstack.org/21/578721/9/check/openstack-ansible-functional-ubuntu-xenial/c8a9e55/job-output.txt.gz#_2018-07-02_09_15_06_313089 | 21:35 |
tonytan4ever | Here: https://review.openstack.org/#/c/579694/1 someone here care to review this for us? | 21:36 |
guilhermesp | and here is the expected: https://github.com/cloudalchemy/ansible-grafana/blob/c9d97ca84abf5daca3d591c497a03e24d52f2674/vars/debian.yml#L2 | 21:36 |
tonytan4ever | It breaks our release so we really need it. | 21:36 |
guilhermesp | i tried to force the private parameter of include_role to be false, but there wasn't any effect | 21:37 |
guilhermesp | and it seems that I have provoked more discussions about it hahahahhaha ( https://github.com/ansible/ansible/issues/21890 ) | 21:40 |
guilhermesp | afk for a moment | 21:43 |
cloudnull | tonytan4ever what is breaking there? | 21:46 |
prometheanfire | cloudnull: actually gate-check-commit needs updating too | 21:46 |
prometheanfire | ../openstack-ansible/scripts/gate-check-commit.sh: ${PIP_COMMAND} install ${PIP_OPTS} ara==0.14.0 "${ANSIBLE_PACKAGE:-ansible}" | 21:46 |
prometheanfire | that has the same error described in my commit message https://review.openstack.org/579694 | 21:46 |
tonytan4ever | cloudnull: maas xenial gates | 21:46 |
prometheanfire | tonytan4ever: osa itself needs a change, will get it in a sec | 21:48 |
tonytan4ever | k | 21:48 |
cloudnull | tonytan4ever prometheanfire, i'm confused, is `pip install ara` not working? | 21:50 |
cloudnull | if you didn't have the exact path on your system ara would be install from pip, is that broken ? | 21:50 |
prometheanfire | cloudnull: nope | 21:50 |
cloudnull | ** from pypi | 21:50 |
prometheanfire | it seems like it's not respecting requirents.txt | 21:51 |
openstackgerrit | Matthew Thode proposed openstack/openstack-ansible master: Use requirements from ara to install ara https://review.openstack.org/579699 | 21:52 |
prometheanfire | tonytan4ever: ^ | 21:52 |
cloudnull | dmsimard ^ | 21:52 |
cloudnull | dmsimard is something you've seen recently ? | 21:52 |
tonytan4ever | +1ed | 21:53 |
prometheanfire | cloudnull: here's a log https://gist.githubusercontent.com/prometheanfire/b668ad87eb4d9bddc911e48d0395ab4f/raw/fed3bb5bb12b29542a4a124306e2fdb36ec8514d/gistfile1.txt | 21:54 |
prometheanfire | cloudnull: see what version of ara is being installed | 21:54 |
cloudnull | looks like that's coming from https://github.com/openstack/requirements/blob/master/upper-constraints.txt#L512 | 21:55 |
prometheanfire | hmm, ya | 21:56 |
prometheanfire | I thought osa didn't use requirements for itself | 21:56 |
cloudnull | we've used requirements since they've been there | 21:57 |
cloudnull | we can override them , but we generally just take them as-is | 21:57 |
prometheanfire | ya, but I thought OSA itself didn't use it | 21:57 |
*** ansmith has joined #openstack-ansible | 21:57 | |
prometheanfire | so, either ARA needs fixing, or reqs need fixing | 21:58 |
prometheanfire | preferably ara | 21:58 |
cloudnull | we did just have a mess with flask recently. let me go find that pr | 21:58 |
evrardjp[m] | guilhermesp: prepare for the next match! ;) | 21:58 |
prometheanfire | :D | 21:58 |
cloudnull | https://review.openstack.org/574414 | 21:59 |
cloudnull | https://review.openstack.org/578770 | 21:59 |
cloudnull | stemming from https://review.openstack.org/#/c/574006/4 | 22:00 |
evrardjp[m] | prometheanfire: if you need further reviews, don't hesitate to ping. same for others. | 22:00 |
*** sgrasley has quit IRC | 22:00 | |
*** sgrasley has joined #openstack-ansible | 22:00 | |
evrardjp[m] | what's the issue with req and ara ? | 22:00 |
cloudnull | looks like ara needs newer flask | 22:01 |
cloudnull | similar to keystone and designate | 22:01 |
prometheanfire | evrardjp[m]: maybe my fix won't even work | 22:01 |
prometheanfire | constraints override requirements, and constraints say 1.0.2 right now | 22:01 |
*** tux_ has joined #openstack-ansible | 22:01 | |
cloudnull | tonytan4ever can you not use an earlier release of tests / global requirements ? | 22:01 |
cloudnull | pin to some SHA where things were generally working ? | 22:02 |
cloudnull | maybe just use stable/queens? | 22:02 |
tonytan4ever | Okay, I can try that. | 22:02 |
evrardjp[m] | yeah I'd ping dmsimard to see if he can bump ara's reqs ? | 22:03 |
prometheanfire | ya, I'm making a storyboard for that now | 22:03 |
evrardjp[m] | adapt* | 22:03 |
evrardjp[m] | ok cool | 22:03 |
evrardjp[m] | I'm out for today. | 22:03 |
*** tonytan4ever has quit IRC | 22:03 | |
prometheanfire | nn | 22:03 |
*** tonytan4ever_brb has joined #openstack-ansible | 22:03 | |
*** flaviosr has quit IRC | 22:05 | |
*** sgrasley has quit IRC | 22:05 | |
*** sgrasley has joined #openstack-ansible | 22:05 | |
prometheanfire | evrardjp[m]: cloudnull tonytan4ever_brb https://storyboard.openstack.org/#!/story/2002800 | 22:07 |
prometheanfire | cloudnull: think I should just close my reviews (since they won't work) | 22:07 |
prometheanfire | might be useful if only so we bump the version of ara | 22:07 |
*** cjloader has joined #openstack-ansible | 22:14 | |
*** cshen has quit IRC | 22:22 | |
*** flaviosr has joined #openstack-ansible | 22:23 | |
*** cshen has joined #openstack-ansible | 22:34 | |
*** sgrasley_ has joined #openstack-ansible | 22:34 | |
*** sgrasley has quit IRC | 22:38 | |
*** sgrasley_ is now known as sgrasley | 22:38 | |
*** cshen has quit IRC | 22:38 | |
openstackgerrit | Merged openstack/openstack-ansible-os_swift master: Drop SELinux support from os_swift https://review.openstack.org/578887 | 22:40 |
openstackgerrit | Merged openstack/openstack-ansible master: Move database creation into role (congress) https://review.openstack.org/578805 | 22:40 |
openstackgerrit | Jesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-os_keystone master: Add Ubuntu Bionic 18.04 support https://review.openstack.org/575954 | 22:42 |
dmsimard | evrardjp[m], cloudnull: please ping tomorrow, >40°C here and stuff | 22:45 |
openstackgerrit | Merged openstack/openstack-ansible master: Move database creation into role (rally) https://review.openstack.org/578853 | 22:45 |
openstackgerrit | Merged openstack/openstack-ansible master: Move database creation into role (barbican) https://review.openstack.org/574850 | 22:45 |
cjloader | is there an issue with cinder role | 22:48 |
*** fghaas has quit IRC | 22:48 | |
cjloader | fatal: [aio1_cinder_api_container-240c51a3]: FAILED! => {"censored": "the output has been hidden due to the fact that 'no_log: true' was specified for this result", "changed": false, "failed": true} | 22:48 |
openstackgerrit | Merged openstack/openstack-ansible master: Remove rabbitmq vars from aodh playbook https://review.openstack.org/574841 | 22:48 |
openstackgerrit | Leopard Ma proposed openstack/openstack-ansible-galera_client master: Add source code repository notes link to README https://review.openstack.org/579711 | 23:04 |
*** tux_ has quit IRC | 23:05 | |
openstackgerrit | Leopard Ma proposed openstack/openstack-ansible-galera_server master: Add source code repository notes link to README https://review.openstack.org/579712 | 23:07 |
*** erolg has quit IRC | 23:11 | |
*** pmannidi has joined #openstack-ansible | 23:15 | |
openstackgerrit | Merged openstack/openstack-ansible-os_glance master: Cleanup directory creation tasks https://review.openstack.org/568131 | 23:15 |
mnaser | small easy +W | 23:20 |
openstackgerrit | Mohammed Naser proposed openstack/openstack-ansible-os_trove master: Use tests repo for common role test requirements https://review.openstack.org/579610 | 23:20 |
openstackgerrit | Mohammed Naser proposed openstack/openstack-ansible-os_trove master: correct typo in service setup https://review.openstack.org/578584 | 23:20 |
openstackgerrit | Mohammed Naser proposed openstack/openstack-ansible-os_trove master: Add systemd tags to include role https://review.openstack.org/578608 | 23:21 |
openstackgerrit | Mohammed Naser proposed openstack/openstack-ansible-os_trove master: Switch to using project-templates https://review.openstack.org/577529 | 23:21 |
openstackgerrit | Mohammed Naser proposed openstack/openstack-ansible-os_trove master: Add packages required for osprofiler https://review.openstack.org/573518 | 23:21 |
openstackgerrit | Mohammed Naser proposed openstack/openstack-ansible-os_trove master: Update to use oslo.messaging service for RPC and Notify https://review.openstack.org/574789 | 23:21 |
openstackgerrit | Mohammed Naser proposed openstack/openstack-ansible-os_trove master: fix tox python3 overrides https://review.openstack.org/576062 | 23:22 |
openstackgerrit | Mohammed Naser proposed openstack/openstack-ansible-os_sahara master: Use tests repo for common role test requirements https://review.openstack.org/579602 | 23:25 |
openstackgerrit | Mohammed Naser proposed openstack/openstack-ansible-os_sahara master: Switch to using project-templates https://review.openstack.org/577523 | 23:25 |
openstackgerrit | Leopard Ma proposed openstack/openstack-ansible-memcached_server master: Add source code repository notes link to README https://review.openstack.org/579715 | 23:25 |
guilhermesp | evrardjp[m]: friday the mother russia will be as hooot as was today! :D till that, my focus will be our monasca hahaha | 23:28 |
*** erolg has joined #openstack-ansible | 23:29 | |
openstackgerrit | Merged openstack/openstack-ansible-os_searchlight master: Use tests repo for common role test requirements https://review.openstack.org/579641 | 23:35 |
*** threestrands has joined #openstack-ansible | 23:43 | |
*** gyee has quit IRC | 23:45 | |
*** erolg has quit IRC | 23:45 | |
openstackgerrit | Merged openstack/openstack-ansible-os_watcher master: Update server to service https://review.openstack.org/578732 | 23:46 |
openstackgerrit | Merged openstack/openstack-ansible-os_ceilometer master: Update server to service https://review.openstack.org/578729 | 23:47 |
*** dxiri has quit IRC | 23:47 | |
openstackgerrit | Merged openstack/openstack-ansible-os_trove master: Update server to service https://review.openstack.org/578731 | 23:47 |
*** aludwar has joined #openstack-ansible | 23:52 | |
openstackgerrit | Merged openstack/openstack-ansible-os_cloudkitty master: Update server to service https://review.openstack.org/578730 | 23:56 |
openstackgerrit | Merged openstack/ansible-role-python_venv_build master: Use tests repo for common role test requirements https://review.openstack.org/579389 | 23:56 |
openstackgerrit | Merged openstack/ansible-role-python_venv_build master: Switch to using project-templates https://review.openstack.org/576984 | 23:56 |
openstackgerrit | Merged openstack/openstack-ansible-os_zaqar master: Use tests repo for common role test requirements https://review.openstack.org/579643 | 23:56 |
openstackgerrit | Merged openstack/openstack-ansible-os_zaqar master: Update server to service https://review.openstack.org/578733 | 23:56 |
*** vnogin has quit IRC | 23:56 |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!