| *** gyee has quit IRC | 00:04 | |
| openstackgerrit | Merged openstack/openstack-ansible-os_horizon master: Do not log passwords https://review.openstack.org/562582 | 00:07 |
|---|---|---|
| openstackgerrit | Merged openstack/openstack-ansible-os_horizon master: Rename pull_catalog args from version to branch https://review.openstack.org/559290 | 00:14 |
| *** Leo_m has quit IRC | 00:14 | |
| *** tomtom001 has quit IRC | 00:20 | |
| *** tomtom001 has joined #openstack-ansible | 00:21 | |
| *** ethfci_ has joined #openstack-ansible | 00:24 | |
| *** ethfci has quit IRC | 00:24 | |
| *** chyka has joined #openstack-ansible | 00:25 | |
| *** chyka has joined #openstack-ansible | 00:26 | |
| *** ethfci_ has quit IRC | 00:29 | |
| *** ethfci has joined #openstack-ansible | 00:30 | |
| *** chyka has quit IRC | 00:40 | |
| *** chyka has joined #openstack-ansible | 00:40 | |
| *** coreywright has quit IRC | 00:41 | |
| *** chyka has quit IRC | 00:45 | |
| *** coreywright has joined #openstack-ansible | 00:53 | |
| *** dave-mccowan has quit IRC | 01:01 | |
| *** vnogin has joined #openstack-ansible | 01:15 | |
| *** vnogin has quit IRC | 01:20 | |
| *** dave-mccowan has joined #openstack-ansible | 01:26 | |
| *** dxiri has quit IRC | 01:26 | |
| *** TobbeCN has joined #openstack-ansible | 01:37 | |
| *** TobbeCN has quit IRC | 01:41 | |
| *** dave-mccowan has quit IRC | 01:47 | |
| openstackgerrit | Merged openstack/openstack-ansible master: Remove Octavia from translations scenario https://review.openstack.org/562578 | 01:52 |
| *** armaan has quit IRC | 01:54 | |
| *** armaan has joined #openstack-ansible | 01:54 | |
| *** mwarad has joined #openstack-ansible | 02:48 | |
| *** mwarad has quit IRC | 02:55 | |
| *** chyka has joined #openstack-ansible | 02:55 | |
| *** mpjetta has quit IRC | 02:58 | |
| *** threestrands has joined #openstack-ansible | 02:59 | |
| *** nicolasbock has quit IRC | 03:00 | |
| *** mpjetta has joined #openstack-ansible | 03:05 | |
| *** chyka has quit IRC | 03:24 | |
| *** dxiri has joined #openstack-ansible | 03:32 | |
| *** tideline has joined #openstack-ansible | 03:43 | |
| *** dxiri has quit IRC | 03:44 | |
| *** poopcat has quit IRC | 04:06 | |
| *** tideline has quit IRC | 04:06 | |
| *** udesale has joined #openstack-ansible | 04:12 | |
| openstackgerrit | Olivier Bourdon proposed openstack/openstack-ansible-openstack_hosts stable/pike: Fix error when /etc/hosts does not exist https://review.openstack.org/562927 | 04:21 |
| openstackgerrit | Olivier Bourdon proposed openstack/openstack-ansible-openstack_hosts stable/queens: Fix error when /etc/hosts does not exist https://review.openstack.org/562928 | 04:22 |
| *** dxiri has joined #openstack-ansible | 04:38 | |
| *** deng_ has joined #openstack-ansible | 04:46 | |
| *** hamza21 has joined #openstack-ansible | 04:48 | |
| *** deng_ has quit IRC | 04:56 | |
| *** DanyC has joined #openstack-ansible | 04:59 | |
| *** bharat1 has joined #openstack-ansible | 05:01 | |
| *** bharat1 has left #openstack-ansible | 05:04 | |
| *** dxiri has quit IRC | 05:12 | |
| openstackgerrit | Taseer Ahmed proposed openstack/openstack-ansible master: Improve Congress scenario testing https://review.openstack.org/560839 | 05:18 |
| *** udesale_ has joined #openstack-ansible | 05:32 | |
| *** DanyC has quit IRC | 05:33 | |
| *** udesale_ has quit IRC | 05:34 | |
| *** udesale has quit IRC | 05:34 | |
| *** udesale_ has joined #openstack-ansible | 05:34 | |
| *** udesale_ has quit IRC | 05:34 | |
| *** udesale has joined #openstack-ansible | 05:36 | |
| *** tomtom001 has quit IRC | 05:36 | |
| *** tomtom001 has joined #openstack-ansible | 05:36 | |
| *** udesale has quit IRC | 05:41 | |
| *** udesale has joined #openstack-ansible | 05:43 | |
| *** hamza21 has quit IRC | 05:52 | |
| *** armaan has quit IRC | 06:02 | |
| *** armaan has joined #openstack-ansible | 06:02 | |
| *** TobbeCN has joined #openstack-ansible | 06:04 | |
| *** mma has joined #openstack-ansible | 06:08 | |
| *** sar has joined #openstack-ansible | 06:15 | |
| *** shardy has joined #openstack-ansible | 06:29 | |
| *** deng_ has joined #openstack-ansible | 06:31 | |
| evrardjp | odyssey4me hwoarang cloudnull mgariepy logan- d34dh0r53 as a reminder, next week is rocky milestone 1. The community goals are ACKed, and the version is already bumped, so we are good on that level. But we have gates to fix. It would be nice to see nspawn working, and upgrades. | 06:32 |
| *** eumel8 has joined #openstack-ansible | 06:38 | |
| *** pcaruana has joined #openstack-ansible | 06:43 | |
| hwoarang | i am hoping to get the RFC for distro packages done today and then i can jump in and fix other stuff | 06:44 |
| DimGR | hi | 06:44 |
| evrardjp | hi DimGR | 06:56 |
| hwoarang | odyssey4me: evrardjp re https://review.openstack.org/#/c/562606/ i am not sure how we can switch the python interp mid way thourgh. the goal was to use the packages + python from venv for all the os_* modules but that requires setting ansible_python_interp from the beginning | 07:00 |
| *** threestrands has quit IRC | 07:00 | |
| hwoarang | so it's prob best to simply convert tempest to distro/source like keystone otherwise it's getting a bit messy | 07:01 |
| *** sar has quit IRC | 07:01 | |
| openstackgerrit | Jean-Philippe Evrard (evrardjp) proposed openstack/openstack-ansible master: Add Congress scenario as check job https://review.openstack.org/560839 | 07:01 |
| evrardjp | hwoarang: ok | 07:02 |
| evrardjp | hwoarang: it shouldn't be changed during a play, but for a whole play that's doable and easy with group vars if you like | 07:03 |
| *** tomtom001 has quit IRC | 07:06 | |
| evrardjp | hwoarang: I am not 100% sure what you meant though | 07:07 |
| evrardjp | I thought you wanted to flip glance or keystone to 100% using packages, and therefore had to do the same with tempest to avoid cryptography incompatibilities | 07:07 |
| *** eumel8_ has joined #openstack-ansible | 07:07 | |
| *** eumel8 has quit IRC | 07:07 | |
| evrardjp | because we are doing something wrong with the host. So first step was to remove what we did on the host, which seems to be part of your commit | 07:08 |
| *** eumel8_ is now known as eumel8 | 07:08 | |
| evrardjp | but I don't see where python interpreter comes in | 07:08 |
| evrardjp | probably using a set fact instead of a task var would help too | 07:09 |
| *** shardy is now known as shardy_afk | 07:09 | |
| evrardjp | because adding this as a group var seem risky | 07:09 |
| openstackgerrit | Merged openstack/openstack-ansible-plugins stable/queens: Remove python3 job https://review.openstack.org/562855 | 07:10 |
| hwoarang | i will see... | 07:11 |
| evrardjp | I was just asking for understanding/helping, sorry to be intrusive :D | 07:12 |
| *** tomtom001 has joined #openstack-ansible | 07:13 | |
| *** sar has joined #openstack-ansible | 07:17 | |
| *** tosky has joined #openstack-ansible | 07:17 | |
| *** epalper has joined #openstack-ansible | 07:17 | |
| hwoarang | evrardjp: so what odyssey4me suggested yesterday was to stop installing keystoneclient, glanceclient on the host and on the venv on tempest role | 07:21 |
| hwoarang | and instead use the packages from venv | 07:21 |
| hwoarang | the reason we install them on the host is because of all the os_* modules that we use | 07:21 |
| hwoarang | so in order for ansible to make use of the venv packages, we need to switch the interpreter for ansible to use the one from venv | 07:22 |
| hwoarang | but that has to happen after the venv is ready | 07:22 |
| hwoarang | but because everything is a single playbook we can't do that | 07:22 |
| hwoarang | so in the end, it's easier and cleaner to simply do what i did to keystone to make tempest distro + source aware | 07:23 |
| *** radeks has joined #openstack-ansible | 07:23 | |
| evrardjp | that's exactly what I understood | 07:25 |
| evrardjp | so that's why I was proposing a set fact | 07:26 |
| evrardjp | the set_fact is stored across two plays of the same playbook for example | 07:26 |
| evrardjp | we could definitely use that at the beggining of a play: if you have something like <standard ansible venv location>, use it | 07:26 |
| evrardjp | that's good for all the stories, not only distro packages, it could be python3 switch too | 07:27 |
| *** cshen has joined #openstack-ansible | 07:27 | |
| *** yifei has quit IRC | 07:28 | |
| *** yifei has joined #openstack-ansible | 07:28 | |
| *** hamza21 has joined #openstack-ansible | 07:32 | |
| *** sar has quit IRC | 07:36 | |
| *** sar has joined #openstack-ansible | 07:37 | |
| *** niraj_singh has quit IRC | 07:37 | |
| odyssey4me | hwoarang you can switch the python interpreter per task if you want to | 07:40 |
| evrardjp | odyssey4me: that's what he is trying but it doesn't seem to work fine | 07:40 |
| evrardjp | or becomes ugly | 07:40 |
| evrardjp | odyssey4me: hwoarang we definitely need to think about how we're gonna do tempest and rally in the future | 07:41 |
| evrardjp | these are the intertwined bits of the packages/python build/python3 refactor. | 07:41 |
| evrardjp | add on top of that the plugins | 07:41 |
| odyssey4me | ah ok | 07:42 |
| odyssey4me | I think we are going to have to accept that this will create two code paths for the most part | 07:42 |
| evrardjp | I think we should discuss now what we want to do for those, and go with the plan | 07:42 |
| evrardjp | that sounds sensible for package vs source | 07:42 |
| odyssey4me | we may be able to bring some of it back together eventually, but for now it'll be two paths | 07:42 |
| evrardjp | that's fine, but, for example, I don't know what the packages are shipping | 07:42 |
| evrardjp | do they ship plugins? | 07:43 |
| evrardjp | because we could be silently hiding reduced test coverage or unconsistent test coverage between source and packages. | 07:43 |
| openstackgerrit | Jesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-tests stable/queens: Do not log passwords https://review.openstack.org/562853 | 07:43 |
| evrardjp | why the change ^ ? | 07:44 |
| evrardjp | confused at retesting things :p | 07:45 |
| odyssey4me | evrardjp force of habit, I di the backport - only to see that ou had done it... as you see it's no change | 07:45 |
| evrardjp | haha | 07:45 |
| evrardjp | I see :) | 07:45 |
| evrardjp | I didn't it was smart and merged that into one | 07:45 |
| evrardjp | didn't know* | 07:46 |
| *** Adri2000 has quit IRC | 07:46 | |
| evrardjp | to continue the tempest discussion | 07:46 |
| *** Adri2000 has joined #openstack-ansible | 07:47 | |
| evrardjp | either for source or for packages, we need to properly know which python interpreter will be used I guess | 07:47 |
| evrardjp | except if everything is in site packages. | 07:47 |
| evrardjp | but then what mess would that be with new plugins? | 07:47 |
| openstackgerrit | Jesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-os_aodh stable/queens: Replace virtualenv-tools by a script https://review.openstack.org/562971 | 07:49 |
| evrardjp | odyssey4me: I have a question for you unrelated. shouldn't this https://github.com/openstack/openstack-ansible-galera_server/commit/6f1d10063aac57f66dd7549028e090aed00dca7a be applied on galera_client too? | 07:50 |
| openstackgerrit | Jesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-os_aodh stable/queens: Only replace python when re-initializing the venv https://review.openstack.org/562972 | 07:50 |
| evrardjp | I got a gate issue appeared on repo_build role | 07:50 |
| evrardjp | saying there was a conflict in packages | 07:50 |
| *** DanyC has joined #openstack-ansible | 07:51 | |
| odyssey4me | wtf, why does galera_client have those packages | 07:51 |
| odyssey4me | weird - yeah, I'll do that | 07:51 |
| evrardjp | yeah. | 07:51 |
| evrardjp | well the weirdest is not that | 07:51 |
| evrardjp | I built that on my machine, and it worked just fine. Same code. | 07:51 |
| evrardjp | That was my wtf moment | 07:51 |
| odyssey4me | yeah, I suspect that there's a newer version in ubuntu repositories - and our mariadb pin is now old | 07:52 |
| *** TobbeCN has quit IRC | 07:52 | |
| odyssey4me | we've never had this before because we always used new mariadb - but now we're pinning | 07:52 |
| odyssey4me | we we might have to set priorities or something | 07:52 |
| *** DanyC has quit IRC | 07:53 | |
| *** DanyC has joined #openstack-ansible | 07:53 | |
| evrardjp | we set those | 07:53 |
| evrardjp | everything from mariadb has a prio | 07:54 |
| odyssey4me | why do we need build-essentials and python-dev in the mariadb client role? | 07:54 |
| odyssey4me | this is silly | 07:54 |
| evrardjp | https://github.com/openstack/openstack-ansible-galera_server/blob/1075fe411aa7b37e6c536e1c1ea5d4bfd509d2b0/defaults/main.yml#L143-L156 | 07:55 |
| evrardjp | build-essentials was required by python-dev to have some python bits like cryptography | 07:55 |
| evrardjp | let me check which python bit requires it | 07:55 |
| odyssey4me | but galera_client has no python bits surely? | 07:55 |
| odyssey4me | only mysql-python | 07:56 |
| evrardjp | MySQL-python | 07:56 |
| evrardjp | I guess for ssl connection on mysql | 07:56 |
| odyssey4me | and I don't think that's even needed, because our database tasks are delegated to the galera_server hosts anyway | 07:56 |
| evrardjp | let me check if we can change that | 07:56 |
| evrardjp | what about the nova etc? | 07:57 |
| evrardjp | Well | 07:57 |
| evrardjp | we can put that there. | 07:57 |
| evrardjp | in the venv | 07:57 |
| evrardjp | yeah remove it and see what breaks | 07:57 |
| evrardjp | we'll fix that :) | 07:57 |
| evrardjp | let me dobule check | 07:57 |
| odyssey4me | let me do two patches - one for the perl lib, then one for other things | 07:57 |
| evrardjp | omg | 07:58 |
| evrardjp | https://pypi.org/project/MySQL-python/ | 07:58 |
| evrardjp | click on homepage | 07:58 |
| evrardjp | we are hitting sourceforge there. | 07:58 |
| evrardjp | status "unknown" | 07:58 |
| openstackgerrit | Jesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-galera_client master: Remove perl library from Ubuntu installs https://review.openstack.org/562975 | 07:58 |
| evrardjp | do we know where we need this package? | 07:58 |
| odyssey4me | ansible requires that package to do the mysql database tasks | 07:59 |
| odyssey4me | that is all | 07:59 |
| odyssey4me | so we only need it on the server | 07:59 |
| odyssey4me | for openstack we use pymysql | 07:59 |
| evrardjp | wow it seems it's used everywhere | 07:59 |
| odyssey4me | yeah, mysql-python used to be the primary driver used for mysql access before pymysql | 08:00 |
| odyssey4me | I guess it hasn't been cleaned up | 08:00 |
| evrardjp | yup | 08:00 |
| evrardjp | we should get rid of it | 08:00 |
| evrardjp | want me to patch it? | 08:00 |
| evrardjp | so we have two different patches? | 08:00 |
| odyssey4me | https://review.openstack.org/#/c/562276/1 | 08:00 |
| odyssey4me | with that gone, I thk we may be able to clean up a bunch of those distro packages too | 08:01 |
| evrardjp | omg hwoarang is so cool, don't tell him! | 08:01 |
| evrardjp | let me do the same in other roles | 08:02 |
| hwoarang | i think infra roles are done | 08:02 |
| evrardjp | I am not sure we need it in server | 08:02 |
| evrardjp | I can't remember any utility using it | 08:02 |
| hwoarang | galera and lxc_hosts | 08:02 |
| hwoarang | evrardjp: it's needed for mysql_user i think | 08:03 |
| hwoarang | the module | 08:03 |
| evrardjp | oh | 08:03 |
| *** yifei1 has joined #openstack-ansible | 08:03 | |
| evrardjp | let me check real quick | 08:03 |
| evrardjp | dammit | 08:03 |
| evrardjp | you're right | 08:03 |
| odyssey4me | I did say that just a moment ago. | 08:04 |
| hwoarang | yes but my opinion matters more! | 08:04 |
| hwoarang | ;D | 08:04 |
| odyssey4me | hahaha | 08:04 |
| evrardjp | haha | 08:04 |
| evrardjp | I just missed it. | 08:05 |
| hwoarang | mysql-python is not python3 compatible and it's dead upstream | 08:05 |
| hwoarang | there is a bug in ansible to use pymsql | 08:05 |
| evrardjp | magnum and trove are using it too | 08:05 |
| hwoarang | so i expect we wil have issues with ubuntu 18 and Leap 15 | 08:05 |
| hwoarang | in the near futre | 08:05 |
| evrardjp | trove I guess I know why :p | 08:05 |
| odyssey4me | all our database tasks run against the galera_server hosts, so that's the only place we need it | 08:05 |
| odyssey4me | oh, but if magnum/trove are using it and have no ansible tasks doing things, then they can bring it in via requirements - we shouldn't have to specify the package | 08:06 |
| evrardjp | trove and magnum install them in their venv | 08:06 |
| evrardjp | so it's fine anyway | 08:06 |
| evrardjp | odyssey4me: yup that's true too. | 08:07 |
| *** tomtom001 has quit IRC | 08:07 | |
| evrardjp | hwoarang: I'd still be happy to know your opinion about the plugins, see above conversation | 08:08 |
| hwoarang | yeah i need to examine the package :/ | 08:08 |
| *** tomtom001 has joined #openstack-ansible | 08:08 | |
| hwoarang | i can give you the contents :p | 08:08 |
| evrardjp | hwoarang: http://logs.openstack.org/67/562667/3/check/openstack-ansible-functional-opensuse-423/73b5a3a/job-output.txt.gz#_2018-04-20_07_36_30_458616 | 08:09 |
| evrardjp | it's in this patch, so no change in suse: https://review.openstack.org/#/c/562667/3 | 08:09 |
| evrardjp | I thought you might want to know it. | 08:09 |
| evrardjp | :D | 08:09 |
| hwoarang | seems like a network issue | 08:10 |
| evrardjp | rechecked | 08:10 |
| evrardjp | cool | 08:10 |
| evrardjp | thanks | 08:10 |
| *** mbuil has joined #openstack-ansible | 08:11 | |
| andymccr | evrardjp: all the no_log: True patches, is that to avoid leaking of passwords when runs happen in verbose mode? or something similar? | 08:12 |
| odyssey4me | evrardjp hwoarang it'd be helpful to get https://review.openstack.org/#/c/562975/ merged asap | 08:12 |
| odyssey4me | andymccr yep | 08:12 |
| andymccr | sweet | 08:12 |
| andymccr | thats a nice change | 08:12 |
| odyssey4me | no callbacks or verbose mode will allow the password | 08:12 |
| odyssey4me | well, nothing will be logged from that task, actually | 08:12 |
| odyssey4me | just the title | 08:12 |
| odyssey4me | and result | 08:12 |
| *** sar has quit IRC | 08:13 | |
| *** sar has joined #openstack-ansible | 08:13 | |
| openstackgerrit | Merged openstack/openstack-ansible-galera_client master: Drop pointless PIP packages https://review.openstack.org/562276 | 08:17 |
| evrardjp | odyssey4me: so, for tempest install from sources, I think we should provide the defaults repos available. Either we add them into defaults/... to have the packages built into the repo build. | 08:17 |
| evrardjp | Or we include them in the roles | 08:18 |
| evrardjp | or we include them in os_tempest | 08:18 |
| openstackgerrit | Jesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-os_cinder stable/queens: Do not log passwords https://review.openstack.org/562980 | 08:18 |
| evrardjp | for now I would say the easiest is to make them in os_tempest | 08:18 |
| evrardjp | and we can have a split later | 08:18 |
| openstackgerrit | Jesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-os_cinder stable/pike: Do not log passwords https://review.openstack.org/562981 | 08:18 |
| odyssey4me | if you're talking about the plugins, then I say do it in the roles themselves - not os_tempest | 08:19 |
| odyssey4me | if you do it in os_tempest, it'll never leave that role and the role is just getting worse and worse | 08:20 |
| odyssey4me | it's building technical debt, not reducing it | 08:20 |
| evrardjp | I don't see how this changes things. I know you prefer it in the role | 08:20 |
| evrardjp | roles* | 08:20 |
| evrardjp | I just don't see the positive improvement | 08:20 |
| evrardjp | it looks more complex | 08:20 |
| odyssey4me | everything for the role is then in the role - easy to find, easy to adjust on a per service basis | 08:21 |
| odyssey4me | tempest itself is just a framework, which is why it's removing all the plugins itself - our role for tempest should move in the same direction | 08:21 |
| odyssey4me | our tempest role should only setup tempest, not setup things for other services | 08:22 |
| evrardjp | I am fine for the variable let's say | 08:22 |
| evrardjp | to be in the os_keystone role | 08:22 |
| evrardjp | but what about the os_keystone role main function then? | 08:22 |
| evrardjp | Does it becomes bloated with other extra functions like setting up tempest? That doesn't make sense | 08:23 |
| evrardjp | will I have new variables in all of the roles $role_tempest_setup: no, and overrides in our gating? | 08:24 |
| evrardjp | these tasks would also have to be delegated to whatever node tempest runs on | 08:24 |
| odyssey4me | maybe it does, maybe it doesn't - I dunno... the point is that it should spread to the roles I think, not gather in the tempest role | 08:24 |
| evrardjp | maybe we are scaling wrong. | 08:25 |
| evrardjp | maybe what we are trying with setting things into each of the role defaults isn't nice | 08:25 |
| evrardjp | maybe each role should include a standard file location instead. | 08:25 |
| odyssey4me | a playbook could do include_role, tasks_from and delegate it - and it doesn't have to be in the main for keystone | 08:25 |
| evrardjp | that file can have all the locations we need | 08:26 |
| odyssey4me | yes, that's another way to do it - make it pluggable somehow | 08:26 |
| evrardjp | like the defaults/ | 08:26 |
| odyssey4me | although that makes it less obvious | 08:26 |
| evrardjp | so easy to override that file | 08:26 |
| evrardjp | and adds an external dependency to manage | 08:26 |
| evrardjp | or sync | 08:26 |
| odyssey4me | maybe all the keystone role does is place a file in a well known location on a designated host | 08:26 |
| odyssey4me | the tempest role then assembles all the placed config files into a consolidated one that it uses | 08:27 |
| evrardjp | coming back to my idea of setting things into a source of truth :) | 08:27 |
| evrardjp | that's what ansible lacks | 08:27 |
| evrardjp | it seriously needs a salt-mine | 08:27 |
| evrardjp | or whatever is named now | 08:28 |
| evrardjp | let me think a little about tempest then | 08:28 |
| evrardjp | it's the same case as haproxy btw | 08:29 |
| *** cshen has quit IRC | 08:29 | |
| openstackgerrit | Jesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-os_swift stable/pike: Do not log passwords https://review.openstack.org/562984 | 08:29 |
| *** gkadam has joined #openstack-ansible | 08:30 | |
| openstackgerrit | Jesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-os_tempest stable/pike: Do not log passwords https://review.openstack.org/562986 | 08:30 |
| odyssey4me | andymccr mind giving this a nudge? https://review.openstack.org/562975 | 08:32 |
| *** deng_ has quit IRC | 08:32 | |
| odyssey4me | evrardjp yes, and perhaps we should use some sort of fact folder on the deploy host to do that :p | 08:32 |
| evrardjp | mmm | 08:33 |
| evrardjp | that would be an okay start indeed. | 08:33 |
| evrardjp | or not a fact folder but having each detail into its own directory | 08:33 |
| *** electrofelix has joined #openstack-ansible | 08:33 | |
| evrardjp | /etc/openstack_deploy/haproxy/services*, /etc/openstack_deploy/tempest/plugins* | 08:34 |
| *** armaan has quit IRC | 08:34 | |
| *** armaan has joined #openstack-ansible | 08:34 | |
| evrardjp | or maybe name them enabled_services and enabled_plugins | 08:34 |
| odyssey4me | sure | 08:35 |
| evrardjp | so it's easy when you read the content of /etc/openstack_deploy/ to understand what is doing what | 08:35 |
| odyssey4me | it's low tech, and simple to understand | 08:35 |
| openstackgerrit | Markos Chandras (hwoarang) proposed openstack/openstack-ansible-os_keystone master: [RFC] Add support for using distribution packages for OpenStack services https://review.openstack.org/560308 | 08:35 |
| evrardjp | so for the plugin case, we basically line in file on those files in each role, when ${role}_tempest_plugin_install | bool | 08:36 |
| evrardjp | and when tempest installs, it fetches that and uses it | 08:36 |
| evrardjp | or even better, we make that a perfectly valid yaml, so that it gets passed into the role | 08:38 |
| evrardjp | into tempest_plugins | 08:38 |
| evrardjp | and then in the future we move that to group_vars and the cycle is done -- we did that in the past! | 08:39 |
| evrardjp | :p | 08:40 |
| odyssey4me | yeah, although that folder would have to be in user space - we can't make it modify things in /etc on the deploy node | 08:40 |
| *** niraj_singh has joined #openstack-ansible | 08:41 | |
| evrardjp | it's a terrible idea then | 08:41 |
| evrardjp | the source of truth for me should be in user space | 08:41 |
| evrardjp | to be clear | 08:41 |
| evrardjp | of what we do | 08:41 |
| evrardjp | if you think about renaming /etc/openstack_deploy to ~/.config/openstack-ansible, that tempest/enabled_plugins since make sense | 08:42 |
| evrardjp | still* | 08:42 |
| evrardjp | or we re-use the lookup | 08:44 |
| evrardjp | the lookup can fetch data from the roles | 08:45 |
| evrardjp | (we should make it only do the tempest plugin discovery though) | 08:45 |
| evrardjp | we can ship a module_util that's like the current lookup of py_pkg, and we make sub lookups, easier to read, to do specific purposes things | 08:46 |
| evrardjp | one could be finding all the tempest plugins locations | 08:46 |
| evrardjp | but it seems very complex, and very repo_build like | 08:46 |
| evrardjp | I think I looped through ideas, and the only one I like is defining plugins in the os_tempest role, and wire some of its usage in group_vars | 08:47 |
| evrardjp | it's simple | 08:47 |
| evrardjp | all the plugins are defined there by default | 08:48 |
| openstackgerrit | Merged openstack/openstack-ansible-galera_client master: Remove perl library from Ubuntu installs https://review.openstack.org/562975 | 08:48 |
| evrardjp | you can override them | 08:48 |
| evrardjp | at the same time we remove all the tempest_service_available_aodh | 08:49 |
| evrardjp | at the same time we remove all the tempest_service_available_* | 08:49 |
| openstackgerrit | Jesse Pretorius (odyssey4me) proposed openstack/openstack-ansible stable/ocata: Move Upgrade branch change above initial library source https://review.openstack.org/562990 | 08:53 |
| Taseer | evrardjp: Good morning. | 08:54 |
| odyssey4me | evrardjp ^ should resolve the newton->ocata upgrade failure of the unbound variable | 08:54 |
| Taseer | should I remove the depends on from my patch ? | 08:54 |
| evrardjp | odyssey4me: we said we were not touching the order of the git checkout | 08:55 |
| odyssey4me | evrardjp we did? why? | 08:55 |
| evrardjp | to avoid re-introducing a bug | 08:55 |
| odyssey4me | that order is required do source the right vars | 08:55 |
| odyssey4me | *to | 08:55 |
| evrardjp | IMO yes, and it's been a problem we've workarounded so many times. | 08:56 |
| evrardjp | I am fine with that, (because I prefer that order), but we need to be very sure we won't change that in the future again. | 08:56 |
| *** udesale_ has joined #openstack-ansible | 08:58 | |
| evrardjp | and we should have it consistently done accross branches | 08:58 |
| odyssey4me | looking at pike, this is the only missing patch - there are no other bugs introduced which were fixed as far as I can see | 08:58 |
| *** udesale has quit IRC | 08:59 | |
| odyssey4me | well, I checked the newton->pike test, it was failing due to the missing env var which is set in the functions | 09:00 |
| odyssey4me | this is not switching back and forth - this was never done for ocata | 09:00 |
| evrardjp | yeah I am mistaken, N and O aren't consistent | 09:01 |
| evrardjp | and we do || true in some functions if they aren't defined | 09:01 |
| odyssey4me | yes, this is due to https://github.com/openstack/openstack-ansible/commit/37ce304eb4ff546e0be283951b2e36f4e007abe7#diff-30c3701b5527d757251851f9253bbb3c now being in ocata | 09:02 |
| odyssey4me | but not in newton, and we can't do it in newton any more because the roles are EOL | 09:02 |
| odyssey4me | and it would require a change to the pip_install role | 09:02 |
| evrardjp | yeah newton deserves dying. | 09:02 |
| evrardjp | it had a long run | 09:02 |
| *** hamza21 has quit IRC | 09:02 | |
| *** hamza21 has joined #openstack-ansible | 09:03 | |
| *** vnogin has joined #openstack-ansible | 09:03 | |
| openstackgerrit | Jesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-galera_client stable/queens: Remove perl library from Ubuntu installs https://review.openstack.org/562993 | 09:03 |
| evrardjp | Taseer: you can try it, now that congress was updated. Please adapt the commit message to remove the references to that. | 09:04 |
| hwoarang | evrardjp: odyssey4me let me open another can of worms | 09:08 |
| evrardjp | hwoarang: woot | 09:08 |
| evrardjp | do you close them sometimes? | 09:08 |
| hwoarang | not really | 09:08 |
| evrardjp | haha | 09:08 |
| evrardjp | beetlejuice! | 09:09 |
| evrardjp | x3 | 09:09 |
| hwoarang | so pip_install is supposed to install a newer pip on the host, so you can use --contrains later on. but if we start using distro packages, then pip_install shouldn't be needed anymore for distro installs. would it make sense to adapt that role so python-virtual env from distro is used and then pip_install to simply update the pip within the virtualenvs? | 09:09 |
| hwoarang | ie, pip_install role makes no sense if you dont install pip packages on the host | 09:10 |
| hwoarang | smells like another spec.. | 09:10 |
| evrardjp | pip_install_destination: venv | site? | 09:12 |
| evrardjp | that seems weird | 09:12 |
| evrardjp | I'd rather reference what version we have now | 09:13 |
| evrardjp | let me check that | 09:13 |
| evrardjp | maybe we don't need to update it nowadays | 09:13 |
| *** mbuil has quit IRC | 09:13 | |
| evrardjp | hwoarang: what's the version of virtualenv shipped with 42.3? | 09:14 |
| hwoarang | well centos comes with 1.4.1 | 09:14 |
| evrardjp | and what is the pip inside ? | 09:14 |
| hwoarang | and that brings a rather old pip with it | 09:14 |
| evrardjp | I will double check on that, because I know there is some source to add to get newest python | 09:14 |
| hwoarang | can't remember about suse but either way centos is going to be a problem by itself | 09:14 |
| evrardjp | spinning up a machine now :) | 09:14 |
| hwoarang | most of the roles pull virtualenv from pip just so they can use a newer pip withit by default. which would be equal to using the distro virtualenv and upgrade pip inside the virtualenv before anything else | 09:15 |
| hwoarang | or something like that | 09:15 |
| evrardjp | (not using pyenv) | 09:15 |
| evrardjp | hwoarang: yes indeed | 09:16 |
| evrardjp | that's exactly what we do | 09:16 |
| evrardjp | but htat's probably avoidable | 09:16 |
| evrardjp | because of centos | 09:16 |
| evrardjp | :) | 09:16 |
| hwoarang | ok so we are on the same page | 09:19 |
| openstackgerrit | Taseer Ahmed proposed openstack/openstack-ansible master: Improve Congress scenario testing https://review.openstack.org/560839 | 09:22 |
| odyssey4me | this is our virtualenv version requirement, and why: https://github.com/openstack/openstack-ansible/blob/master/scripts/bootstrap-ansible.sh#L120-L123 | 09:24 |
| odyssey4me | hwoarang evrardjp ^ | 09:24 |
| odyssey4me | we could possibly get away with not requiring that, but history has told us that it's better to implement pip, setuptools, wheel from fresh in the virtualenv - because sometimes pip and setuptools break each other and do not allow upgrades/downgrades | 09:25 |
| evrardjp | odyssey4me: but that could be solved by distro packagers | 09:25 |
| evrardjp | we shouldn't fix the world because they didn't do it right | 09:26 |
| evrardjp | (exagerating for the sake of the exercise) | 09:26 |
| odyssey4me | however, we may end up only requiring virtualenv on the build host - because on the target hosts for install, we pull down the virtualenv and just make sure the paths are right | 09:26 |
| hwoarang | evrardjp: fix it like how? you need a newer virtualenv from them | 09:27 |
| odyssey4me | centos 7.5 should be out soon, which may resolve all these issues | 09:27 |
| evrardjp | odyssey4me: I just double checked, it doesn't | 09:27 |
| hwoarang | odyssey4me: thank you for the link btw | 09:27 |
| evrardjp | I thought it did | 09:27 |
| odyssey4me | hwoarang on the host, that's actually all we need - check virtualenv version, if it's too low, then check pip version, if it's too low, then get-pip.py | 09:28 |
| odyssey4me | if pip version is fine, then install the right virtualenv version | 09:28 |
| hwoarang | yeah but that's only needed if you want to use virtualenv on the host right? | 09:29 |
| odyssey4me | yes | 09:29 |
| odyssey4me | I expect that over time we'll reduce the footprint of pip_install | 09:29 |
| hwoarang | ok so in the end, the pip_install role becomes conditional to th einstallation method | 09:29 |
| odyssey4me | so for now, I'd say make two code paths | 09:29 |
| odyssey4me | later on we can consolidate them | 09:29 |
| hwoarang | ok | 09:29 |
| odyssey4me | the reality is that pip doesn't have to be on the host for distro installs | 09:30 |
| hwoarang | ok | 09:30 |
| odyssey4me | so much more ideal, is just skip the role entirely if it's not required - but I guess you're also trying to cater for a mixed install | 09:30 |
| odyssey4me | so for now I guess two code paths will have to do | 09:31 |
| evrardjp | if it's possible to skip, but yes, I expect we'd have two code path, and this whole bootstrapping doesn't need to exist | 09:32 |
| evrardjp | for packages | 09:32 |
| *** vnogin has quit IRC | 09:34 | |
| *** yifei1 has quit IRC | 09:35 | |
| odyssey4me | hwoarang does the rdo/uca/obs repo not provide a newer version of virtualenv perhaps? | 09:39 |
| odyssey4me | given virtualenv is in u-c, I would expect that to be the case | 09:39 |
| odyssey4me | perhaps not, but perhaps that could be made to be the case? | 09:40 |
| evrardjp | I just checked it's not everything is built for integration into site packages | 09:41 |
| *** sar has quit IRC | 09:41 | |
| evrardjp | but maybe there is a trick package that builds it | 09:41 |
| *** viks has joined #openstack-ansible | 09:41 | |
| evrardjp | else pyenv | 09:42 |
| evrardjp | as an intermediate layer | 09:42 |
| odyssey4me | no, I'm not keen on yet another dep and yet another layer | 09:42 |
| odyssey4me | we have too many layers already | 09:42 |
| hwoarang | suse does | 09:42 |
| hwoarang | i mean suse provides a newer virtualenv in the cloud repos | 09:43 |
| hwoarang | i will have to check with the rest | 09:43 |
| evrardjp | or using tox :p | 09:43 |
| odyssey4me | shall we get this one through as well to match the galera_server role removal for queens? https://review.openstack.org/562993 | 09:43 |
| odyssey4me | hwoarang or andymccr | 09:44 |
| hwoarang | ok | 09:44 |
| *** vnogin has joined #openstack-ansible | 09:44 | |
| odyssey4me | oh, and this one's urgent too https://review.openstack.org/562759 | 09:45 |
| openstackgerrit | Markos Chandras (hwoarang) proposed openstack/openstack-ansible-os_keystone master: Avoid mixing pip and distribution packages https://review.openstack.org/562607 | 09:46 |
| openstackgerrit | Markos Chandras (hwoarang) proposed openstack/openstack-ansible-os_keystone master: [RFC] Add support for using distribution packages for OpenStack services https://review.openstack.org/560308 | 09:46 |
| odyssey4me | wow, and this one's not through yet - just as urgent: https://review.openstack.org/562346 | 09:46 |
| *** vnogin has quit IRC | 09:51 | |
| *** vnogin has joined #openstack-ansible | 09:52 | |
| *** mwarad has joined #openstack-ansible | 09:54 | |
| *** sar has joined #openstack-ansible | 09:54 | |
| hwoarang | hmm galera download failed again | 09:54 |
| hwoarang | what is going on... | 09:54 |
| odyssey4me | hmm, interesting - looks like the metal build doesn't use the repo server properly, either that or somehow a bunch of log entries are missing: http://logs.openstack.org/12/549012/3/check/openstack-ansible-deploy-aio_metal-ubuntu-xenial/b4680f6/logs/host/nginx/openstack-slushee.access.log | 09:55 |
| openstackgerrit | Merged openstack/openstack-ansible-galera_client stable/queens: Remove perl library from Ubuntu installs https://review.openstack.org/562993 | 09:57 |
| evrardjp | maybe that's why it's fast :) | 09:57 |
| *** cshen has joined #openstack-ansible | 09:58 | |
| odyssey4me | evrardjp without the repo server, you aren't going to be able to install the optional packages like neutron_lbaas | 09:58 |
| *** sar has quit IRC | 09:59 | |
| evrardjp | I will stop with the jokes. | 09:59 |
| evrardjp | it doesn't seem to make you laugh. | 09:59 |
| odyssey4me | um, repo-use appears to somehow be skipped altogether | 10:01 |
| odyssey4me | hmm, maybe that's haproxy-install going on there | 10:02 |
| odyssey4me | odd, haproxy appears to have galera_client installed on it | 10:02 |
| odyssey4me | hmm, the ara report is a bit confusing for tracking a sequence with some context | 10:03 |
| openstackgerrit | Tovin Seven proposed openstack/openstack-ansible master: Trivial: Update pypi url to new url https://review.openstack.org/563031 | 10:03 |
| odyssey4me | aha: http://logs.openstack.org/12/549012/3/check/openstack-ansible-deploy-aio_metal-ubuntu-xenial/b4680f6/job-output.txt.gz#_2018-04-20_08_13_55_278184 | 10:03 |
| evrardjp | haproxy should not be installed | 10:04 |
| evrardjp | there is no point of doing so | 10:04 |
| odyssey4me | this doesn't result in the hosts being targeted: https://github.com/openstack/openstack-ansible/blob/master/playbooks/repo-use.yml#L21 | 10:04 |
| odyssey4me | ah of course, because the host is in repo_all | 10:05 |
| evrardjp | yes | 10:05 |
| odyssey4me | hmm, ok - this is going to be interesting to overcome | 10:05 |
| odyssey4me | let me build one and see what we can do here | 10:05 |
| evrardjp | I am not sure we need to change anything | 10:06 |
| odyssey4me | evrardjp we do - the build fails because pip can't find the dev packages we built on the repo | 10:06 |
| evrardjp | if we say the repo still has access to upstream pip it must stay the same | 10:06 |
| evrardjp | can't we just add the local files into the index? | 10:06 |
| odyssey4me | just trust me, something must change - I'll work it out | 10:06 |
| odyssey4me | they are - but this play is the thing that tells pip to use the local index now | 10:07 |
| evrardjp | https://ih1.redbubble.net/image.441459994.4619/flat,800x800,075,f.u3.jpg | 10:07 |
| odyssey4me | but you did the !repo_all because you wanted the repo server to use the cache... hmm, but that means it should be using the cache already | 10:07 |
| evrardjp | I've seen that before. | 10:07 |
| odyssey4me | I'll build a host and see what's up here - something does need changing, I just need to see where it's broken properly first. | 10:07 |
| openstackgerrit | Merged openstack/openstack-ansible-repo_server stable/queens: Set the pypiserver root to the correct directory https://review.openstack.org/562759 | 10:09 |
| odyssey4me | it looks to me like the host is *never* configured to use the repo's pypi cache, which is likely the cause - so we have a sequence problem. | 10:09 |
| openstackgerrit | Merged openstack/openstack-ansible-tests master: Add python3 functional voting and non-voting jobs https://review.openstack.org/562667 | 10:09 |
| evrardjp | we can replace the repo use to all | 10:11 |
| evrardjp | but we need to make sure it's still okay for going out if necessary | 10:11 |
| *** udesale__ has joined #openstack-ansible | 10:12 | |
| odyssey4me | evrardjp yes, I'll work it out - originally the repo didn't use the local pypi cache at all, for good reason... but now it uses the local cache, which I think will be an issue for upgrades... so this is something I wanted to figure out anyway | 10:13 |
| odyssey4me | once all these virtualenv changes are merged into queens, are we going to bump role SHA's to include it all in the next release? | 10:14 |
| *** udesale_ has quit IRC | 10:14 | |
| odyssey4me | or are you thinking we leave it in a broken state for this release, and bring it all in for the next? | 10:15 |
| odyssey4me | it does appear that *some* queens builds are working - so I'm not sure it's *critical* to bring those in | 10:16 |
| *** vnogin has quit IRC | 10:18 | |
| *** sar has joined #openstack-ansible | 10:22 | |
| *** sar has quit IRC | 10:23 | |
| *** udesale__ has quit IRC | 10:27 | |
| *** markvoelker has quit IRC | 10:30 | |
| *** markvoelker has joined #openstack-ansible | 10:30 | |
| *** pcaruana has quit IRC | 10:31 | |
| *** armaan has quit IRC | 10:31 | |
| *** armaan has joined #openstack-ansible | 10:32 | |
| *** nicolasbock has joined #openstack-ansible | 10:33 | |
| *** markvoelker has quit IRC | 10:35 | |
| *** nattanon has joined #openstack-ansible | 10:39 | |
| nattanon | Hello guys !!! | 10:40 |
| nattanon | I have got a problem with this error " Unable to connect to Neutron: 'frozenset' object has no attribute '__getitem__' " | 10:40 |
| nattanon | It made some of user cant see their instances on panel project -> instance via horizon | 10:41 |
| nattanon | Do someone ever face this problems ?? | 10:42 |
| nattanon | When press panel (project -> instance) horizon error report " Unable to connect to neutron " | 10:44 |
| odyssey4me | nattanon never heard of that one, personally | 10:45 |
| *** cshen has quit IRC | 10:46 | |
| olivierb- | hello everyone, just a basic question to be sure that my understanding of OSA conf is correct. openstack_user_config.yml contains all infos for all services that I initially decided to include whereas conf.d/xx.yml contains all service that I add later ? right ? | 10:47 |
| odyssey4me | olivierb- you can put them in either place - both get merged together in implementation | 10:48 |
| *** armaan has quit IRC | 10:49 | |
| odyssey4me | some people like to put everything in openstack_user_config, and some liek to keep them in seperate files | 10:49 |
| *** armaan has joined #openstack-ansible | 10:49 | |
| odyssey4me | we use seperate files for gating because it's easier to instrument the modularity for different scenario's when building the config - nothing more | 10:49 |
| *** armaan has quit IRC | 10:50 | |
| *** armaan has joined #openstack-ansible | 10:50 | |
| olivierb- | odyssey4me thanks a lot for this info which leads to another question that just occured to my mind, what occurs then during the merge if some infos is duplicated in bot places ? | 10:51 |
| odyssey4me | olivierb- hmm, I'm not sure, but I think the conf.d entries may merge over the openstack_user_config - but you'd have to look at the dynamic inventory code to be sure | 10:52 |
| odyssey4me | best is not to duplicate | 10:52 |
| olivierb- | yep definitely agree but you never know what the final user/customer will do unfortunately | 10:53 |
| *** armaan has quit IRC | 11:04 | |
| *** armaan has joined #openstack-ansible | 11:05 | |
| *** radeks has quit IRC | 11:06 | |
| *** vnogin has joined #openstack-ansible | 11:12 | |
| *** vnogin has quit IRC | 11:17 | |
| *** cshen has joined #openstack-ansible | 11:20 | |
| *** vnogin has joined #openstack-ansible | 11:29 | |
| *** udesale has joined #openstack-ansible | 11:36 | |
| odyssey4me | hmm, evrardjp hwoarang repo_server lays down https://github.com/openstack/openstack-ansible-repo_server/blob/master/templates/nginx.conf.j2 and keystone appends to it https://github.com/openstack/openstack-ansible-os_keystone/blob/master/tasks/keystone_nginx.yml#L30 | 11:44 |
| odyssey4me | so on a bare metal node (no containers), when re-running repo-server it wipes out the append and fails to restart nginx | 11:44 |
| odyssey4me | I wonder, can we do the custom log format in another file? | 11:45 |
| *** pcaruana has joined #openstack-ansible | 11:46 | |
| odyssey4me | yep, looks like it - alright, will push a patch up | 11:48 |
| *** ansmith has quit IRC | 11:54 | |
| openstackgerrit | Jesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-pip_install master: Disable the pip version check https://review.openstack.org/563093 | 12:00 |
| *** markvoelker has joined #openstack-ansible | 12:02 | |
| openstackgerrit | Jesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-pip_install master: Disable the pip version check https://review.openstack.org/563093 | 12:03 |
| openstackgerrit | Jesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-pip_install master: Ensure that pip.conf templates variables properly https://review.openstack.org/563095 | 12:05 |
| openstackgerrit | Taseer Ahmed proposed openstack/openstack-ansible master: Improve Congress scenario testing https://review.openstack.org/560839 | 12:09 |
| mgariepy | morning everyone | 12:39 |
| evrardjp | morning | 12:40 |
| *** armaan has quit IRC | 12:40 | |
| evrardjp | odyssey4me: logging to journal directly would be great | 12:41 |
| evrardjp | I mean binary format | 12:41 |
| evrardjp | today, on a scale of grumpiness that goes from 0 to 9, I'm at... | 12:42 |
| evrardjp | over 9000. | 12:42 |
| openstackgerrit | Jesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-repo_build master: Make sure that the wheel build task fails if the wheel build fails https://review.openstack.org/563100 | 12:43 |
| odyssey4me | evrardjp ^ finally found why the wheel build fails wasn't resulting in the task failing | 12:43 |
| odyssey4me | evrardjp logging to journal? you mean for the services we implement, or in general? | 12:44 |
| evrardjp | so "good" morning doesn't apply | 12:44 |
| evrardjp | odyssey4me: shipping everything to journal! | 12:44 |
| evrardjp | nginx? same thing! | 12:44 |
| odyssey4me | I'd like to log to journal for everything and get rid of rsyslog as a requirement. People can then always implement rsyslog on their common journal host. | 12:44 |
| evrardjp | not going that far to centralize journals, but at least logging everything to journal | 12:45 |
| odyssey4me | evrardjp ah, sorry - I moved well on from that one! | 12:45 |
| evrardjp | modern things | 12:45 |
| odyssey4me | centralizing journals is better than centralising rsyslog - more compact data shipping | 12:45 |
| odyssey4me | but sure, there are other ways too | 12:45 |
| evrardjp | yeah but I don't care, graylog is doing the right thing :p | 12:46 |
| evrardjp | going to each of the host logging is a "nice to have" | 12:46 |
| evrardjp | for me | 12:46 |
| evrardjp | but apache logging in a different format sucks | 12:46 |
| odyssey4me | clealy I'm on a bug smash week :p | 12:46 |
| *** vnogin has quit IRC | 12:47 | |
| evrardjp | oh yes the classical no failing with multiple tasks | 12:47 |
| evrardjp | maybe we should do a lint test for that | 12:47 |
| evrardjp | if more than two lines of shell script and no set as first line, it smells bad | 12:47 |
| evrardjp | or maybe those should be modules :) | 12:48 |
| odyssey4me | meh, the only reason it's piping is to add a timestamp - this is not a usual thing | 12:48 |
| *** vnogin has joined #openstack-ansible | 12:50 | |
| openstackgerrit | Markos Chandras (hwoarang) proposed openstack/openstack-ansible-pip_install master: tests: Do not always fetch the get-pip.py script during tests https://review.openstack.org/562243 | 12:50 |
| evrardjp | ? | 12:51 |
| hwoarang | odyssey4me: better^? | 12:51 |
| openstackgerrit | Jesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-os_keystone master: Use a nginx conf.d file for the custom logging https://review.openstack.org/563101 | 12:51 |
| odyssey4me | ^ one of those things you would only pick up in an minor upgrade or playbook re-run | 12:51 |
| odyssey4me | hwoarang yeah, that looks reasonable | 12:52 |
| hwoarang | ok | 12:52 |
| odyssey4me | that should actually open the door to idepotence testing on most roles now | 12:54 |
| hwoarang | i am still not sure how exactly pip_install is defined as dependency to roles | 12:57 |
| hwoarang | i can't see it being on meta/main.yml so i suspect that we explicitly call it in the tests repo somewhere | 12:57 |
| odyssey4me | wtf, why is ubuntu using sites-available and the others using conf.d? https://github.com/openstack/openstack-ansible-os_keystone/search?utf8=%E2%9C%93&q=keystone_nginx_conf_path&type= | 12:57 |
| odyssey4me | hwoarang it's implemented in the repo-use.yml playbook, and in the openstack_hosts role | 12:58 |
| hwoarang | oh sneaky | 12:59 |
| hwoarang | openstack_hosts does an include_role on pip_install | 12:59 |
| hwoarang | i wonder why this is not a proper dependency in meta/main.yml then | 12:59 |
| odyssey4me | hmf https://github.com/openstack/openstack-ansible-os_keystone/commit/15733bbd2feece3680ed27a04346750ae380f863 | 12:59 |
| odyssey4me | hwoarang we used to do that - that's why pike takes so long to deploy ;) | 13:00 |
| odyssey4me | it was done this way to radically reduce the timelines for deploying by only implementing it twice - once before the repo is built, and once after | 13:00 |
| odyssey4me | it will eventually reduce again to only a single time | 13:01 |
| hwoarang | i am not sure i understand. include_role and meta/main.yml have the same effect in the end right? | 13:01 |
| evrardjp | hwoarang: yes but ... we smartly run it. | 13:02 |
| evrardjp | let's say that instead of an include_role in ALL the roles | 13:03 |
| evrardjp | (which are applied in a serial way in the playbooks) | 13:03 |
| evrardjp | you just do it once across the board at the beginning | 13:03 |
| hwoarang | i agree | 13:03 |
| openstackgerrit | Jesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-os_keystone master: Use a nginx conf.d file for the custom logging https://review.openstack.org/563101 | 13:03 |
| hwoarang | but what I want to say, what would happen if you had pip_install as dependency in meta/main.yml just in the openstack_hosts role | 13:04 |
| evrardjp | odyssey4me: that's to not disturb apache peasants | 13:04 |
| evrardjp | hwoarang: that would run at the beginning | 13:04 |
| evrardjp | not at the end | 13:04 |
| odyssey4me | evrardjp wut? | 13:04 |
| evrardjp | nginx using sites_enabled | 13:04 |
| hwoarang | evrardjp: would that be bad? do you need some sort of preparation on the host before you run pip_install? | 13:04 |
| evrardjp | that's because ubuntu/debian have these "convenience scripts" to do symlinking. | 13:05 |
| evrardjp | in apache | 13:05 |
| evrardjp | a2ensite etc. | 13:05 |
| evrardjp | so the packaging of nginx took the same idea of sites-available | 13:05 |
| evrardjp | didn't provide the script though, I think | 13:05 |
| evrardjp | but they used that instead of having a conf.d | 13:05 |
| evrardjp | IIRC | 13:05 |
| evrardjp | hwoarang: that's a good question. | 13:06 |
| *** ansmith has joined #openstack-ansible | 13:06 | |
| evrardjp | hwoarang: But I don't think pip_install should be a "Dependency" of the openstack_hosts | 13:06 |
| evrardjp | it should just be a task of the preparation of hosts, like other tasks | 13:06 |
| evrardjp | I don't see the positive improvement of adding it to meta | 13:07 |
| *** dave-mccowan has joined #openstack-ansible | 13:07 | |
| hwoarang | ok | 13:07 |
| evrardjp | I am okau if you explain that to me | 13:07 |
| hwoarang | maybe openstack_hosts should 'absorve' pip_install in the end | 13:07 |
| hwoarang | i dont have anything to explain | 13:07 |
| evrardjp | I just need convincing | 13:07 |
| hwoarang | i was just curious :) | 13:07 |
| evrardjp | hwoarang: probably yes | 13:07 |
| evrardjp | absorb until 0 task remain | 13:08 |
| hwoarang | *absorb! | 13:08 |
| hwoarang | yeah | 13:08 |
| openstackgerrit | Jesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-pip_install master: Disable the pip version check https://review.openstack.org/563093 | 13:08 |
| openstackgerrit | Jesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-pip_install master: Disable the pip version check https://review.openstack.org/563093 | 13:09 |
| openstackgerrit | Merged openstack/openstack-ansible-pip_install master: tests: Do not always fetch the get-pip.py script during tests https://review.openstack.org/562243 | 13:13 |
| openstackgerrit | Jesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-repo_server master: Allow pypiserver to be upgraded before repo build https://review.openstack.org/563106 | 13:15 |
| odyssey4me | yet another gap ^ | 13:15 |
| openstackgerrit | Jesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-repo_server master: Allow pypiserver to be upgraded before repo build https://review.openstack.org/563106 | 13:16 |
| *** udesale has quit IRC | 13:20 | |
| *** udesale has joined #openstack-ansible | 13:20 | |
| *** udesale has quit IRC | 13:23 | |
| *** udesale has joined #openstack-ansible | 13:23 | |
| *** armaan has joined #openstack-ansible | 13:24 | |
| openstackgerrit | Jesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-repo_build master: Override the default index for the wheel build https://review.openstack.org/563111 | 13:26 |
| openstackgerrit | Jesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-repo_build master: Override the pip index used for the wheel build https://review.openstack.org/563111 | 13:27 |
| *** masber has quit IRC | 13:27 | |
| *** admin0 has joined #openstack-ansible | 13:29 | |
| niraj_singh | evrardjp: getting error while repo-build playbook run; fatal: [aio1_repo_container-74a38efe]: UNREACHABLE! => {"changed": false, "msg": "SSH Error: data could not be sent to remote host \"172.29.236.100\". Make sure this host can be reached over ssh", "unreachable": true} | 13:29 |
| openstackgerrit | Jesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-pip_install master: Disable the pip version check https://review.openstack.org/563093 | 13:32 |
| *** masuberu has joined #openstack-ansible | 13:32 | |
| openstackgerrit | Jesse Pretorius (odyssey4me) proposed openstack/openstack-ansible master: Ensure that the repo servers also use the pypi cache https://review.openstack.org/563114 | 13:37 |
| openstackgerrit | Jesse Pretorius (odyssey4me) proposed openstack/openstack-ansible master: Remove default pip_links value https://review.openstack.org/549012 | 13:38 |
| evrardjp | niraj_singh: looks like a connectivity issue, either your plugins are misconfigured, or your network configuration doesn't work | 13:39 |
| openstackgerrit | Jesse Pretorius (odyssey4me) proposed openstack/openstack-ansible master: Ensure that the repo servers also use the pypi cache https://review.openstack.org/563114 | 13:40 |
| evrardjp | niraj_singh: aren't you simply running ./run_tests.sh? | 13:40 |
| openstackgerrit | Jesse Pretorius (odyssey4me) proposed openstack/openstack-ansible master: Remove default pip_links value https://review.openstack.org/549012 | 13:40 |
| niraj_singh | evrardjp: ok | 13:40 |
| evrardjp | you should not have to care about the repo build in that cas | 13:40 |
| evrardjp | case | 13:40 |
| evrardjp | as the roles are independant beings! | 13:40 |
| odyssey4me | if the repo-build is being run, then you're using the integrated build for tests which will take awfully long and cause all sorts of unrelated issues if the role isn't ready | 13:41 |
| odyssey4me | you should be using role tests as evrardjp has suggested | 13:41 |
| niraj_singh | evrardjp: my openstack-ansible VM was currupted that why i again install the openstack-ansible from scratch | 13:42 |
| openstackgerrit | Merged openstack/openstack-ansible-openstack_hosts stable/queens: Fix error when /etc/hosts does not exist https://review.openstack.org/562928 | 13:43 |
| openstackgerrit | Merged openstack/openstack-ansible-openstack_hosts stable/pike: Fix error when /etc/hosts does not exist https://review.openstack.org/562927 | 13:43 |
| niraj_singh | and added masakari configuration. after that i ran repo-build and got this error | 13:43 |
| niraj_singh | odyssey4me: ok | 13:44 |
| openstackgerrit | Jean-Philippe Evrard proposed openstack/openstack-ansible-tests stable/queens: Add python3 functional voting and non-voting jobs https://review.openstack.org/563122 | 13:46 |
| openstackgerrit | Jean-Philippe Evrard proposed openstack/ansible-config_template master: Set py3 job to be voting https://review.openstack.org/561900 | 13:48 |
| openstackgerrit | Jean-Philippe Evrard proposed openstack/openstack-ansible-os_horizon stable/queens: Do not log passwords https://review.openstack.org/563124 | 13:49 |
| openstackgerrit | Jean-Philippe Evrard proposed openstack/openstack-ansible-os_horizon stable/pike: Do not log passwords https://review.openstack.org/563125 | 13:49 |
| odyssey4me | evrardjp :( https://github.com/openstack/networking-odl/blob/master/requirements.txt#L21 | 13:50 |
| evrardjp | oh great. | 13:50 |
| evrardjp | networking ppl... bgpvpn now networking odl | 13:50 |
| evrardjp | hey guess what... let's install ceilometer master branch for all our things! | 13:51 |
| evrardjp | that's proper packaging izit? | 13:51 |
| openstackgerrit | Merged openstack/openstack-ansible-os_monasca-agent stable/queens: Replace virtualenv-tools with a script https://review.openstack.org/562677 | 13:51 |
| evrardjp | I will stop complaining | 13:51 |
| odyssey4me | yeah, this is totally silly - why does the ODL implementation need ceilometer | 13:51 |
| odyssey4me | grrr | 13:51 |
| evrardjp | extra packages are too fancy | 13:51 |
| evrardjp | I will work on bgpaaS later in the cycle | 13:52 |
| evrardjp | bgpvpn | 13:52 |
| evrardjp | sorry | 13:52 |
| evrardjp | I just think there are more urgent things | 13:52 |
| evrardjp | is that blocking you? | 13:52 |
| openstackgerrit | Merged openstack/ansible-hardening master: Switch from debian-jessie to debian-stable (stretch) https://review.openstack.org/562842 | 13:52 |
| odyssey4me | nah, I was just testing minor upgrades to ensure my series of patches works properly | 13:54 |
| odyssey4me | reset all the SHA's to master and hit that | 13:54 |
| odyssey4me | I'll push up a patch to freeze it in our integrated build so that we don't get surprised later | 13:54 |
| odyssey4me | there may be more - busy trying another build | 13:55 |
| hwoarang | i know it's friday and people dont want to look at ugly stuff right before the weekend but... | 13:55 |
| hwoarang | i could use another pair of eyes for https://review.openstack.org/#/c/560308/ now that at least ubuntu disto job is passing | 13:55 |
| *** cjloader_ has joined #openstack-ansible | 13:55 | |
| *** cjloader_ has quit IRC | 13:55 | |
| odyssey4me | wow, I'm impressed that it's passing - well done! | 13:56 |
| *** cjloader has joined #openstack-ansible | 13:56 | |
| openstackgerrit | Merged openstack/openstack-ansible-os_rally stable/queens: Only replace python when re-initializing the venv https://review.openstack.org/562767 | 13:57 |
| *** kstev has joined #openstack-ansible | 13:58 | |
| openstackgerrit | Merged openstack/openstack-ansible-os_tacker stable/queens: Only replace python when re-initializing the venv https://review.openstack.org/562769 | 13:58 |
| openstackgerrit | Jesse Pretorius (odyssey4me) proposed openstack/openstack-ansible master: Freeze the networking-odl repo from SHA bumps https://review.openstack.org/563130 | 14:00 |
| *** mnaser has quit IRC | 14:00 | |
| niraj_singh | i am gettng this error while running ./run_tests.sh http://paste.openstack.org/show/719630/ | 14:00 |
| *** mnaser has joined #openstack-ansible | 14:01 | |
| odyssey4me | niraj_singh is that run tests from the role repo? | 14:01 |
| *** vnogin has quit IRC | 14:01 | |
| odyssey4me | it looks like you have a restricted pip configuration, which only happens on the integrated build - I suggest you start with a fresh host and use the role tests | 14:01 |
| odyssey4me | the integrated build's run_tests.sh only does the lint testing | 14:02 |
| niraj_singh | Sorry for misunderstanding. i haven't configure my role yet. i jsut added the role in /ansible roles dir and added configuration to install role . after that i ran repo-build. | 14:04 |
| *** hamza21 has quit IRC | 14:05 | |
| *** vnogin has joined #openstack-ansible | 14:06 | |
| hwoarang | i would suggest to use role tests and not work on the integrated build directly | 14:06 |
| hwoarang | OMG next PTG is at Denver again | 14:10 |
| hwoarang | THE TRAIN! | 14:10 |
| hwoarang | mhayden: ^ please come! | 14:10 |
| *** esberglu has joined #openstack-ansible | 14:10 | |
| openstackgerrit | Jesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-os_keystone master: Use a nginx conf.d file for the custom logging https://review.openstack.org/563101 | 14:10 |
| odyssey4me | niraj_singh ah, the better path is most definitely to build the role first and the tests for the role - once it's working and more complete, then work on integrating it | 14:12 |
| odyssey4me | otherwise you'll have to work through a ton of other issues | 14:12 |
| odyssey4me | please see the contributor guide - it advises exactly that | 14:12 |
| *** guilhermesp has quit IRC | 14:13 | |
| evrardjp | hwoarang: haha yes | 14:13 |
| evrardjp | hwoarang: great job | 14:14 |
| odyssey4me | oh awesome :) I now have a friend that lives there, so I'll have to stay an extra day or two this time | 14:14 |
| hwoarang | hmm ticket prices are almost double compared to last year | 14:15 |
| hwoarang | iirc it costed about $100 now it's $199 early bird | 14:15 |
| hwoarang | interesting... | 14:15 |
| openstackgerrit | Merged openstack/openstack-ansible-os_designate stable/queens: Only replace python when re-initializing the venv https://review.openstack.org/562766 | 14:15 |
| openstackgerrit | Merged openstack/openstack-ansible-tests stable/queens: Do not log passwords https://review.openstack.org/562853 | 14:15 |
| openstackgerrit | Merged openstack/openstack-ansible-os_heat stable/queens: Only replace python when re-initializing the venv https://review.openstack.org/562794 | 14:16 |
| openstackgerrit | Merged openstack/openstack-ansible-os_gnocchi stable/queens: Only replace python when re-initializing the venv https://review.openstack.org/562791 | 14:16 |
| *** vnogin has quit IRC | 14:16 | |
| openstackgerrit | Merged openstack/openstack-ansible-os_glance stable/queens: Only replace python when re-initializing the venv https://review.openstack.org/562792 | 14:17 |
| *** yifei has quit IRC | 14:17 | |
| *** yifei has joined #openstack-ansible | 14:18 | |
| openstackgerrit | Jesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-os_watcher master: Do not log passwords https://review.openstack.org/563139 | 14:19 |
| openstackgerrit | Jesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-os_watcher master: Only replace python when re-initializing the venv https://review.openstack.org/562202 | 14:20 |
| evrardjp | hwoarang: reviewed | 14:22 |
| hwoarang | thx | 14:22 |
| evrardjp | hwoarang: I see you like my idea of include_vars :) | 14:23 |
| evrardjp | flexibilitay | 14:24 |
| evrardjp | if we can make it voting that would be great. | 14:24 |
| evrardjp | but only after a check experimental if possible :D | 14:24 |
| hwoarang | maybe we can make ubuntu voting. opensuse needs tempest fixes. and no clue why centos is failing | 14:25 |
| hwoarang | anyway | 14:25 |
| evrardjp | that's cool. Good job | 14:27 |
| evrardjp | and precedent is set. | 14:27 |
| evrardjp | merging this before milestone 1 would be great! | 14:28 |
| evrardjp | or milestone 3 | 14:28 |
| evrardjp | :) | 14:28 |
| evrardjp | milestone 1 is next week so... | 14:28 |
| spotz | evrardjp: get min in!!! https://review.openstack.org/#/c/551316/ | 14:28 |
| hwoarang | there is time :) | 14:29 |
| openstackgerrit | Merged openstack/openstack-ansible-os_sahara stable/queens: Only replace python when re-initializing the venv https://review.openstack.org/562796 | 14:29 |
| openstackgerrit | Merged openstack/openstack-ansible-tests stable/pike: Do not log passwords https://review.openstack.org/562854 | 14:29 |
| openstackgerrit | Merged openstack/openstack-ansible-os_neutron stable/queens: Only replace python when re-initializing the venv https://review.openstack.org/562799 | 14:29 |
| openstackgerrit | Merged openstack/openstack-ansible-os_swift stable/queens: Do not log passwords https://review.openstack.org/562688 | 14:30 |
| openstackgerrit | Merged openstack/openstack-ansible-os_cinder stable/queens: Do not log passwords https://review.openstack.org/562980 | 14:30 |
| openstackgerrit | Merged openstack/openstack-ansible-os_cinder stable/queens: Only replace python when re-initializing the venv https://review.openstack.org/562795 | 14:30 |
| openstackgerrit | Gaudenz Steinlin proposed openstack/openstack-ansible stable/ocata: Added monitoring tag to monitoring user creation task. https://review.openstack.org/563143 | 14:31 |
| *** vnogin has joined #openstack-ansible | 14:31 | |
| evrardjp | yup there is time :) I will speak to tony | 14:33 |
| *** epalper has quit IRC | 14:35 | |
| openstackgerrit | Merged openstack/openstack-ansible-os_barbican stable/queens: Only replace python when re-initializing the venv https://review.openstack.org/562797 | 14:38 |
| openstackgerrit | Merged openstack/openstack-ansible-os_magnum stable/pike: Do not log passwords https://review.openstack.org/562684 | 14:38 |
| *** dxiri has joined #openstack-ansible | 14:38 | |
| openstackgerrit | Merged openstack/openstack-ansible-os_tempest stable/queens: Do not log passwords https://review.openstack.org/562680 | 14:39 |
| openstackgerrit | Merged openstack/openstack-ansible-os_magnum stable/queens: Do not log passwords https://review.openstack.org/562683 | 14:39 |
| openstackgerrit | Jesse Pretorius (odyssey4me) proposed openstack/openstack-ansible master: bootstrap-host: Set repo build pip default mirror when in openstack-ci https://review.openstack.org/563145 | 14:43 |
| mnaser | anyone seen this with the dashboard? :( | 14:45 |
| mnaser | Recoverable error: [('system library', 'fopen', 'Too many open files'), ('BIO routines', 'BIO_new_file', 'system lib'), ('x509 certificate routines', 'X509_load_cert_crl_file', 'system lib')] | 14:45 |
| openstackgerrit | Merged openstack/openstack-ansible-os_magnum stable/queens: Only replace python when re-initializing the venv https://review.openstack.org/562673 | 14:46 |
| evrardjp | mnaser: that's recoverable, everything is fine! | 14:47 |
| evrardjp | :p | 14:47 |
| mgariepy | haha | 14:47 |
| evrardjp | sadly I can't say I have | 14:47 |
| evrardjp | can't you change the ulimit with unit files? | 14:47 |
| mnaser | not to the user, it will say "failed to get X from Y" | 14:47 |
| mnaser | well i was just wondering if anyone had to bump it up before me | 14:47 |
| evrardjp | nope which is quite worrying | 14:48 |
| evrardjp | maybe worth asking logan- | 14:48 |
| evrardjp | I am not sure rpc is having an override for that | 14:48 |
| openstackgerrit | Merged openstack/openstack-ansible-os_tempest master: Only replace python when re-initializing the venv https://review.openstack.org/562200 | 14:49 |
| mgariepy | mnaser, on centos ? and on master? | 14:51 |
| *** ansmith has quit IRC | 14:51 | |
| mnaser | mgariepy: centos+pike | 14:51 |
| *** v1k0d3n has quit IRC | 14:51 | |
| *** v1k0d3n has joined #openstack-ansible | 14:51 | |
| odyssey4me | evrardjp witht hat series of patches I submitted earlier, we get a minor upgrade success for the bare metal implementation, FYI | 14:53 |
| evrardjp | cool, we should probably have that tested somewhere :) | 14:53 |
| odyssey4me | sure, perhaps the minor upgrade for it could actually be tested at some point - perhaps even per commit | 14:54 |
| evrardjp | using the same code as the upgrade? doing checkout ? | 14:54 |
| evrardjp | same kind of code at least | 14:55 |
| * mhayden wishes everyone a happy fridayyyy | 14:55 | |
| mhayden | just don't remind cjloader that it's friday, please | 14:55 |
| openstackgerrit | Merged openstack/openstack-ansible-os_nova stable/queens: Only replace python when re-initializing the venv https://review.openstack.org/562709 | 14:55 |
| mgariepy | mnaser, poking my colleague if they seens this issue. | 14:56 |
| openstackgerrit | Jesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-os_tempest stable/queens: Only replace python when re-initializing the venv https://review.openstack.org/563148 | 14:57 |
| mgariepy | mnaser, they havent seens that. | 14:57 |
| odyssey4me | evrardjp nah, I'm thinking more along the lines of figuring out the last tag, then doing a minor upgrade from there - although that's only really useful for the stable branches | 15:01 |
| evrardjp | odyssey4me: yup. | 15:01 |
| evrardjp | but that's a good idea. | 15:01 |
| mgariepy | mnaser, which tag ? my colleagues are running 16.0.10 | 15:01 |
| odyssey4me | evrardjp we might actually have to make it static var, because soemtimes the source for a minor upgrade will be undeployable | 15:01 |
| mnaser | mgariepy: i dont have that info close to me but i used to run into that with other apps too | 15:01 |
| odyssey4me | think of the situation where we missed something that wasn't caught in gate | 15:02 |
| evrardjp | yup but by default it could be a static version | 15:02 |
| evrardjp | let's say first deployable version of a branch | 15:02 |
| evrardjp | if it can do that, it can do what's in between :) | 15:03 |
| evrardjp | (TM) | 15:03 |
| evrardjp | I will make a lot of money with those kinds of reasoning. | 15:03 |
| evrardjp | Future big corporate seller out there! | 15:03 |
| openstackgerrit | Jesse Pretorius (odyssey4me) proposed openstack/openstack-ansible master: Remove default pip_links value https://review.openstack.org/549012 | 15:04 |
| odyssey4me | evrardjp yeah, I do like the idea of making sure that the first release for a series is something we test being upgradable from | 15:05 |
| odyssey4me | so that always verifies that we're able to upgrade from x.0.0 to x.y.z, and therefore we should be upgradable from x.0.1 onwards to x.y.z | 15:05 |
| *** DanyC has quit IRC | 15:06 | |
| odyssey4me | if something goes very wrong with that version, we update it -but set a release note out saying how one has to fix any version from before then to be upgradable... but that's only for situations like the pbr release where things broke quite spectacularly | 15:06 |
| *** tomtom001 has quit IRC | 15:07 | |
| *** eumel8 has quit IRC | 15:08 | |
| openstackgerrit | Merged openstack/openstack-ansible-os_ceilometer stable/queens: Do not log passwords https://review.openstack.org/562705 | 15:08 |
| evrardjp | yup | 15:08 |
| evrardjp | it will be harder for sha bumps though | 15:09 |
| evrardjp | imagine master | 15:09 |
| evrardjp | to define this test, we'd have to do the first version as a m1 | 15:10 |
| evrardjp | that can then be updated | 15:10 |
| openstackgerrit | Merged openstack/openstack-ansible-os_ceilometer stable/pike: Do not log passwords https://review.openstack.org/562706 | 15:11 |
| *** redondo-mk has quit IRC | 15:11 | |
| *** redondo-mk has joined #openstack-ansible | 15:12 | |
| *** wlmbasson has quit IRC | 15:12 | |
| *** wlmbasson has joined #openstack-ansible | 15:12 | |
| odyssey4me | yeah, I think b1/b2/b3 are fair enough sources for that source of test on master | 15:13 |
| odyssey4me | anyway - it's a beautiful day, so I'm out - going to a cheese festival | 15:14 |
| evrardjp | that's only for you! | 15:14 |
| evrardjp | But I will head out too | 15:14 |
| evrardjp | in a bit | 15:14 |
| evrardjp | so FYI | 15:14 |
| evrardjp | I've pushed a version of the ansible-inventory-manage lib | 15:14 |
| evrardjp | we can now use it as toolkit for building group/host/vars management tool | 15:16 |
| evrardjp | interesting milestone | 15:16 |
| evrardjp | definitely needs more testing | 15:16 |
| evrardjp | and help from others | 15:16 |
| evrardjp | jmccrory: I'd love if we can discuss this next week | 15:16 |
| odyssey4me | well, if you move it into the openstack namespace you'll get more eyes on it, and perhaps more contributors? | 15:17 |
| odyssey4me | as long as it's functional, it'll be a start and we can work on integrating it into tests and start modifying docs | 15:17 |
| odyssey4me | we can also publish to pypi, which I assume is the plan? | 15:17 |
| *** cshen has quit IRC | 15:17 | |
| openstackgerrit | Merged openstack/openstack-ansible-os_glance stable/queens: Do not log passwords https://review.openstack.org/562699 | 15:19 |
| *** vnogin has quit IRC | 15:19 | |
| openstackgerrit | Merged openstack/openstack-ansible-os_panko stable/queens: Do not log passwords https://review.openstack.org/562691 | 15:23 |
| *** dave-mccowan has quit IRC | 15:24 | |
| openstackgerrit | Merged openstack/openstack-ansible-os_heat stable/pike: Do not log passwords https://review.openstack.org/562704 | 15:26 |
| openstackgerrit | Merged openstack/openstack-ansible-os_heat stable/queens: Do not log passwords https://review.openstack.org/562703 | 15:26 |
| openstackgerrit | Merged openstack/openstack-ansible-os_trove master: Do not log passwords https://review.openstack.org/562240 | 15:27 |
| openstackgerrit | Merged openstack/openstack-ansible-galera_server stable/pike: Do not log passwords https://review.openstack.org/561897 | 15:30 |
| evrardjp | it's just a lib... It's nothing much. I planned to push it on pypi yes | 15:31 |
| openstackgerrit | Jesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-os_trove stable/queens: Do not log passwords https://review.openstack.org/563156 | 15:31 |
| openstackgerrit | Jesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-os_trove stable/pike: Do not log passwords https://review.openstack.org/563157 | 15:31 |
| evrardjp | enjoy the summer odyssey4me :) | 15:32 |
| openstackgerrit | Merged openstack/openstack-ansible-os_designate stable/pike: Do not log passwords https://review.openstack.org/562697 | 15:33 |
| evrardjp | have a good weekend everyone | 15:33 |
| *** electrofelix has quit IRC | 15:33 | |
| openstackgerrit | Merged openstack/openstack-ansible-os_searchlight stable/queens: Do not log passwords https://review.openstack.org/562707 | 15:33 |
| openstackgerrit | Merged openstack/openstack-ansible-os_ironic stable/pike: Do not log passwords https://review.openstack.org/562679 | 15:34 |
| openstackgerrit | Merged openstack/openstack-ansible-os_aodh stable/pike: Do not log passwords https://review.openstack.org/562690 | 15:35 |
| openstackgerrit | Merged openstack/openstack-ansible-os_aodh stable/queens: Do not log passwords https://review.openstack.org/562689 | 15:35 |
| *** vnogin has joined #openstack-ansible | 15:35 | |
| *** cjloader_ has joined #openstack-ansible | 15:37 | |
| *** cjloader has quit IRC | 15:37 | |
| *** cjloader_ has quit IRC | 15:38 | |
| *** cjloader has joined #openstack-ansible | 15:38 | |
| openstackgerrit | Merged openstack/openstack-ansible-os_gnocchi stable/pike: Do not log passwords https://review.openstack.org/562702 | 15:41 |
| openstackgerrit | Merged openstack/openstack-ansible-os_gnocchi stable/queens: Do not log passwords https://review.openstack.org/562701 | 15:41 |
| openstackgerrit | Merged openstack/openstack-ansible-os_ironic stable/queens: Do not log passwords https://review.openstack.org/562678 | 15:43 |
| openstackgerrit | Merged openstack/openstack-ansible-os_barbican stable/pike: Do not log passwords https://review.openstack.org/562695 | 15:44 |
| openstackgerrit | Merged openstack/openstack-ansible-os_barbican stable/queens: Do not log passwords https://review.openstack.org/562694 | 15:44 |
| *** shardy_afk has quit IRC | 15:46 | |
| openstackgerrit | Merged openstack/openstack-ansible-os_panko stable/pike: Do not log passwords https://review.openstack.org/562693 | 15:46 |
| *** shardy has joined #openstack-ansible | 15:48 | |
| *** vnogin has quit IRC | 15:49 | |
| openstackgerrit | Merged openstack/openstack-ansible-os_neutron stable/pike: Do not log passwords https://review.openstack.org/562686 | 15:50 |
| openstackgerrit | Merged openstack/openstack-ansible-os_neutron stable/queens: Do not log passwords https://review.openstack.org/562685 | 15:50 |
| *** chyka has joined #openstack-ansible | 15:55 | |
| openstackgerrit | Merged openstack/openstack-ansible-os_sahara stable/pike: Do not log passwords https://review.openstack.org/562682 | 15:55 |
| openstackgerrit | Merged openstack/openstack-ansible-os_sahara stable/queens: Do not log passwords https://review.openstack.org/562681 | 15:55 |
| *** cmart has joined #openstack-ansible | 15:55 | |
| openstackgerrit | Merged openstack/openstack-ansible-os_designate stable/queens: Do not log passwords https://review.openstack.org/562696 | 15:57 |
| *** cjloader has quit IRC | 15:58 | |
| *** cjloader has joined #openstack-ansible | 15:58 | |
| openstackgerrit | Merged openstack/openstack-ansible-os_glance stable/pike: Do not log passwords https://review.openstack.org/562700 | 16:03 |
| *** gyee has joined #openstack-ansible | 16:06 | |
| *** cjloader has quit IRC | 16:06 | |
| openstackgerrit | Kevin Carter (cloudnull) proposed openstack/openstack-ansible master: Convert rsyslog to an include_task with group_vars https://review.openstack.org/556002 | 16:06 |
| openstackgerrit | Merged openstack/openstack-ansible-os_octavia stable/queens: Switch to using run_once instead of matching the first node only https://review.openstack.org/561670 | 16:11 |
| openstackgerrit | Merged openstack/openstack-ansible-os_octavia stable/pike: Switch to using run_once instead of matching the first node only https://review.openstack.org/561671 | 16:11 |
| *** vnogin has joined #openstack-ansible | 16:15 | |
| *** vnogin has quit IRC | 16:20 | |
| *** mma has quit IRC | 16:28 | |
| *** shardy has quit IRC | 16:29 | |
| openstackgerrit | Merged openstack/openstack-ansible-os_octavia master: Update auth_uri option to www_authenticate_uri https://review.openstack.org/561728 | 16:31 |
| openstackgerrit | Merged openstack/openstack-ansible-os_octavia stable/queens: Only replace python when re-initializing the venv https://review.openstack.org/562710 | 16:31 |
| *** DanyC has joined #openstack-ansible | 16:38 | |
| *** DanyC_ has joined #openstack-ansible | 16:40 | |
| *** gkadam has quit IRC | 16:42 | |
| *** cjloader has joined #openstack-ansible | 16:43 | |
| *** DanyC has quit IRC | 16:43 | |
| cjloader | mhayden: guess what day it is! | 16:45 |
| mhayden | hah | 16:45 |
| cjloader | mhayden: "Friday, Friday...." | 16:46 |
| mhayden | one must really get down on friday | 16:46 |
| *** ansmith has joined #openstack-ansible | 16:47 | |
| cjloader | at which step does the group_vars get copied over in OSA? | 16:48 |
| cjloader | in queens | 16:48 |
| cjloader | nvm | 16:49 |
| *** armaan has quit IRC | 16:53 | |
| *** armaan has joined #openstack-ansible | 16:54 | |
| *** rmart04 has joined #openstack-ansible | 16:57 | |
| *** rmart04 has quit IRC | 16:58 | |
| *** cmart has quit IRC | 17:00 | |
| *** hamza21 has joined #openstack-ansible | 17:01 | |
| *** mma has joined #openstack-ansible | 17:04 | |
| *** mma has quit IRC | 17:09 | |
| openstackgerrit | Merged openstack/openstack-ansible master: Remove tempest_git_* overrides https://review.openstack.org/559161 | 17:12 |
| *** cmart has joined #openstack-ansible | 17:13 | |
| *** mma has joined #openstack-ansible | 17:15 | |
| *** mma has quit IRC | 17:19 | |
| *** pcaruana has quit IRC | 17:22 | |
| *** klamath has joined #openstack-ansible | 17:26 | |
| *** pcaruana has joined #openstack-ansible | 17:33 | |
| *** dxiri_ has joined #openstack-ansible | 17:36 | |
| *** dxiri has quit IRC | 17:38 | |
| openstackgerrit | Matt Thompson proposed openstack/openstack-ansible stable/ocata: Remove tempest_git_* overrides https://review.openstack.org/563203 | 17:48 |
| openstackgerrit | Matt Thompson proposed openstack/openstack-ansible stable/pike: Remove tempest_git_* overrides https://review.openstack.org/560025 | 17:52 |
| *** mma has joined #openstack-ansible | 17:56 | |
| *** mma has quit IRC | 18:00 | |
| *** mma has joined #openstack-ansible | 18:05 | |
| *** dave-mccowan has joined #openstack-ansible | 18:06 | |
| openstackgerrit | Merged openstack/openstack-ansible-os_octavia master: Downloads amphora images from artifact storage https://review.openstack.org/555084 | 18:09 |
| *** mma has quit IRC | 18:09 | |
| *** mma has joined #openstack-ansible | 18:18 | |
| *** DanyC_ has quit IRC | 18:21 | |
| *** vnogin has joined #openstack-ansible | 18:25 | |
| *** mma has quit IRC | 18:27 | |
| *** pcaruana has quit IRC | 18:29 | |
| *** vnogin has quit IRC | 18:31 | |
| *** mma has joined #openstack-ansible | 18:32 | |
| *** udesale has quit IRC | 18:34 | |
| *** mma has quit IRC | 18:36 | |
| *** nicolasbock has quit IRC | 18:36 | |
| *** vnogin has joined #openstack-ansible | 18:44 | |
| cloudnull | o/ all | 18:53 |
| d34dh0r53 | o/ | 18:58 |
| openstackgerrit | Cam J. Loader proposed openstack/openstack-ansible master: Make some service vars more global https://review.openstack.org/563243 | 19:11 |
| cjloader | d34dh0r53 cloudnull ^ | 19:12 |
| openstackgerrit | Cam J. Loader proposed openstack/openstack-ansible master: Make some service vars more global https://review.openstack.org/563243 | 19:26 |
| *** speedmann has quit IRC | 19:32 | |
| *** nicolasbock has joined #openstack-ansible | 19:33 | |
| openstackgerrit | German Eichberger proposed openstack/openstack-ansible-os_octavia master: Adds certificate generation https://review.openstack.org/553630 | 19:36 |
| *** speedmann has joined #openstack-ansible | 19:42 | |
| *** adcpdk has joined #openstack-ansible | 19:43 | |
| adcpdk | antonym Hi! I noticed you had some ideas in regards to use xen-hypervisor for the compute nodes, are you still available for a quick chat? | 19:45 |
| adcpdk | https://github.com/openstack/openstack-ansible-specs/blob/master/specs/newton/xen-virt-driver.rst | 19:45 |
| openstackgerrit | Merged openstack/openstack-ansible-os_molteniron stable/queens: Only replace python when re-initializing the venv https://review.openstack.org/562717 | 19:53 |
| *** throwsb1 has joined #openstack-ansible | 19:54 | |
| throwsb1 | Hi, is there any good examples on configuring LVM storage? I am deploying a multi-node clustered environment and all I am seeing is examples for a single LVM. | 19:56 |
| throwsb1 | I am going to have 2 storage nodes with LVM and not sure if the vars listed in the example are hardcoded with OSA or they be modified. | 19:57 |
| DimGR | admin0 how is kube | 19:58 |
| openstackgerrit | Matt Thompson proposed openstack/openstack-ansible stable/queens: Remove tempest_git_* overrides https://review.openstack.org/563263 | 20:02 |
| openstackgerrit | Merged openstack/openstack-ansible master: Remove double warning from run-upgrade https://review.openstack.org/559791 | 20:10 |
| *** cjloader_ has joined #openstack-ansible | 20:13 | |
| *** cjloader has quit IRC | 20:13 | |
| openstackgerrit | Merged openstack/openstack-ansible-os_ceilometer stable/queens: Only replace python when re-initializing the venv https://review.openstack.org/562675 | 20:16 |
| *** cjloader_ has quit IRC | 20:18 | |
| openstackgerrit | Merged openstack/openstack-ansible-repo_build master: Make sure that the wheel build task fails if the wheel build fails https://review.openstack.org/563100 | 20:20 |
| *** adcpdk has quit IRC | 20:20 | |
| openstackgerrit | Merged openstack/openstack-ansible-os_panko stable/queens: Replace virtualenv-tools with a script https://review.openstack.org/562720 | 20:28 |
| openstackgerrit | Merged openstack/openstack-ansible-os_trove master: Only replace python when re-initializing the venv https://review.openstack.org/562201 | 20:32 |
| openstackgerrit | Merged openstack/openstack-ansible-os_trove master: Remove tests-repo-clone.sh https://review.openstack.org/557251 | 20:32 |
| *** dave-mccowan has quit IRC | 20:32 | |
| openstackgerrit | Merged openstack/openstack-ansible-os_horizon stable/pike: Do not log passwords https://review.openstack.org/563125 | 20:32 |
| *** vnogin has quit IRC | 20:33 | |
| openstackgerrit | Merged openstack/openstack-ansible-os_horizon master: Only replace python when re-initializing the venv https://review.openstack.org/562176 | 20:34 |
| openstackgerrit | Merged openstack/openstack-ansible-os_horizon stable/queens: Do not log passwords https://review.openstack.org/563124 | 20:34 |
| openstackgerrit | Merged openstack/openstack-ansible-os_ironic stable/queens: Only replace python when re-initializing the venv https://review.openstack.org/562674 | 20:36 |
| *** vnogin has joined #openstack-ansible | 20:39 | |
| openstackgerrit | Merged openstack/openstack-ansible-os_keystone stable/queens: Only replace python when re-initializing the venv https://review.openstack.org/562711 | 20:46 |
| openstackgerrit | Merged openstack/openstack-ansible-os_swift stable/queens: Only replace python when re-initializing the venv https://review.openstack.org/562798 | 20:50 |
| openstackgerrit | Jesse Pretorius (odyssey4me) proposed openstack/openstack-ansible stable/queens: Remove double warning from run-upgrade https://review.openstack.org/563300 | 20:53 |
| openstackgerrit | Jesse Pretorius (odyssey4me) proposed openstack/openstack-ansible stable/pike: Remove double warning from run-upgrade https://review.openstack.org/563301 | 20:53 |
| openstackgerrit | Jesse Pretorius (odyssey4me) proposed openstack/openstack-ansible stable/ocata: Remove double warning from run-upgrade https://review.openstack.org/563302 | 20:54 |
| *** kstev has quit IRC | 20:54 | |
| *** throwsb1 has quit IRC | 20:54 | |
| openstackgerrit | German Eichberger proposed openstack/openstack-ansible-os_octavia master: Adds certificate generation https://review.openstack.org/553630 | 20:54 |
| openstackgerrit | Jesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-repo_build stable/queens: Make sure that the wheel build task fails if the wheel build fails https://review.openstack.org/563303 | 20:55 |
| openstackgerrit | Jesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-os_trove stable/queens: Only replace python when re-initializing the venv https://review.openstack.org/563306 | 20:58 |
| openstackgerrit | Jesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-os_horizon stable/queens: Only replace python when re-initializing the venv https://review.openstack.org/563307 | 21:00 |
| openstackgerrit | German Eichberger proposed openstack/openstack-ansible-os_octavia stable/queens: Downloads amphora images from artifact storage https://review.openstack.org/563309 | 21:01 |
| openstackgerrit | German Eichberger proposed openstack/openstack-ansible-os_octavia stable/queens: Downloads amphora images from artifact storage https://review.openstack.org/563309 | 21:03 |
| *** SimAloo has quit IRC | 21:05 | |
| *** esberglu has quit IRC | 21:15 | |
| *** lbragstad has quit IRC | 21:17 | |
| *** lbragstad has joined #openstack-ansible | 21:19 | |
| *** vnogin has quit IRC | 21:21 | |
| *** TxGirlGeek has joined #openstack-ansible | 21:37 | |
| *** TxGirlGeek has quit IRC | 21:43 | |
| *** hamza21 has quit IRC | 22:03 | |
| antonym | adcpdk: yeah, whats up? | 22:03 |
| *** dxiri_ has quit IRC | 22:06 | |
| *** gyee has quit IRC | 22:06 | |
| *** dxiri has joined #openstack-ansible | 22:07 | |
| *** cmart has quit IRC | 22:11 | |
| *** TxGirlGeek has joined #openstack-ansible | 22:12 | |
| *** TxGirlGeek has quit IRC | 22:33 | |
| *** nicolasbock has quit IRC | 23:11 | |
| *** tosky has quit IRC | 23:15 | |
| *** admin0 has quit IRC | 23:29 | |
| *** chyka has quit IRC | 23:30 | |
| *** vnogin has joined #openstack-ansible | 23:33 | |
| *** vnogin has quit IRC | 23:59 | |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!
