Thursday, 2018-04-19

*** osnaya has joined #openstack-ansible00:21
*** cjloader_ has quit IRC00:30
osnaya@cloudnull @odyssey4me @logan container creation error during setup-hosts.yml log excerpt @http://paste.openstack.org/show/719520/ Can anyone provide pointers?00:31
osnaya<osnaya> @cloudnull @odyssey4me @logan same error occured for infra1_heat_apis_container, heat_engine and gnocchi as well00:38
*** yifei1 has joined #openstack-ansible00:40
yifei1admin0: sorry I was offline yesterday00:41
yifei1admin0: I did not try that. I don't know whether you noticed that in Queens OSA deploys all neutron agents on bare metal nodes for both control and compute nodes.00:43
yifei1admin0: I think in your FAQ you only considered running neutron agents in Linux containers on control nodes.00:44
osnayaHas anyone else seen the issues during containers creation such as @ @http://paste.openstack.org/show/719520/ ??00:47
*** tobberydberg has quit IRC00:52
osnayawill catch up later...00:54
*** osnaya has quit IRC00:54
*** dxiri has quit IRC01:11
*** yifei1 has quit IRC01:23
*** yifei1 has joined #openstack-ansible01:27
*** dave-mccowan has joined #openstack-ansible01:37
*** tobberydberg has joined #openstack-ansible01:37
*** mma has quit IRC01:41
*** tobberydberg has quit IRC01:42
mmercercloudnull: sorry, been at a company outing all day -- so trying to execute the hatop as instructed yields -- insufficient permissions for socket path /var/run/haproxy.sock (which is weird because im executing as root)01:44
*** tobberydberg has joined #openstack-ansible01:44
mmercerwell, this is peculiar...01:52
*** markvoelker_ has joined #openstack-ansible01:52
*** markvoelker has quit IRC01:53
mmercerhaproxy.cfg has the correct stats socket line with proper permissions....  service is running successfully,  but none of the groups is binding the listening ports....  and i cant run hatop because of insufficient permissions  (this is with tag 17.0.2 btw)01:54
*** Leo_m has quit IRC01:54
mmercerok,  helps to use the stats file not the socket =D01:59
mmerceri have a *lot* of downs right now, but im also assuming that is at minimum because the infrastructure step has yet to actually complete to begin with02:06
mmercerok,  so repo_all-front-1 is open,  repo_all-back is up02:14
mmercerbut i cannot resolve any pip downloads against it because the actual port is not responsive on the aio instance02:14
mmerceriptables rules missing ?02:14
mmercerhmm, im not seeing haproxy be dependent on any other services, which doesnt make sense.  given that there are both ebtables and iptables, there has to be rules defined in one of them to allow the port through02:27
mmercerbut even beyond that,i dont see the listening port via ss | grep portnum, which should show the instantiated port02:27
*** masuberu has joined #openstack-ansible02:27
*** deng_ has joined #openstack-ansible02:28
*** yifei1 has quit IRC02:29
*** masber has quit IRC02:31
mmercerdefinitely not seeing it in firewalld,  checked the active zone (currently public), and only ssh and dhcpv6-client are enabled, surprisingly02:31
*** udesale has joined #openstack-ansible03:08
*** esberglu has joined #openstack-ansible03:18
openstackgerritMatt Thompson proposed openstack/openstack-ansible master: Remove tempest_git_* overrides  https://review.openstack.org/55916103:19
*** masuberu has quit IRC03:23
*** masuberu has joined #openstack-ansible03:33
*** nicolasbock has quit IRC03:34
*** masber has joined #openstack-ansible03:35
*** masuberu has quit IRC03:38
mmercerok, came to this page: https://docs.openstack.org/openstack-ansible/latest/admin/maintenance-tasks/firewalls.html  and it sounds like you have to manually configure firewalld on the aio machine manually yourself04:08
*** poopcat has quit IRC04:11
*** dxiri has joined #openstack-ansible04:31
*** Chris__ has joined #openstack-ansible04:41
openstackgerritKevin Carter (cloudnull) proposed openstack/openstack-ansible-os_nova master: Convert role to use a common systemd service role  https://review.openstack.org/46617904:42
Chris__I'm experiencing a bug in openstack ansible when deploying, http://paste.openstack.org/show/719522/, it seems like its looking for a neutron_lbaas version that does not exist within the python package index. I looked online at https://pypi.org, and found that neutron_lbaas is depreciated, and as of right now the latest version available for download dis 12.0.0.0, does anyone know how to bypass this issue?04:44
openstackgerritMerged openstack/openstack-ansible-ops master: Update stack and add heartbeat  https://review.openstack.org/56211604:48
*** openstack_ansibl has quit IRC04:59
*** epalper has joined #openstack-ansible04:59
*** bhujay has joined #openstack-ansible05:05
*** epalper has quit IRC05:05
*** Chris__ has quit IRC05:09
*** udesale_ has joined #openstack-ansible05:19
*** udesale has quit IRC05:19
*** mwarad has joined #openstack-ansible05:20
openstackgerritMatthew Thode proposed openstack/openstack-ansible-os_ironic master: update for depricated driver definition  https://review.openstack.org/56127705:20
*** armaan has joined #openstack-ansible05:20
cloudnullChris__ My bet would be that the package is not part of your index at "http://int-dev.atl.sandvine.cloud:8181/os-releases/17.0.3/ubuntu-16.04-x86_64/"05:26
odyssey4mecloudnull looks like it's using the index at http://int-dev.atl.sandvine.cloud:8181/simple which is nginx, which will query pypiserver first, then pypi - and pypiserver should be serving it - if it's in the absolute constraints file then it did build the wheel05:28
odyssey4methis has come up a few times, but I've not been able to replicate it05:28
prometheanfireodyssey4me: if I could get direction on https://review.openstack.org/561277 that'd help (see the last few comments), using set_fact has ordering issues because of the facts usage in vars05:29
odyssey4meit's almost like pypiserver is either serving up the pypi index instead of its own, or it's not finding the package for some reason05:29
cloudnullits finding the other versions of that package05:29
cloudnull(from versions: 11.0.2, 11.0.3, 12.0.0.0b1, 12.0.0.0b2, 12.0.0.0b3, 12.0.0.0rc1, 12.0.0)05:29
cloudnullbut from where I have no ide a05:30
odyssey4mecloudnull yeah, that looks like pypi's results to me05:30
*** dxiri has quit IRC05:31
cloudnullmaybe pre=true is not enabled in the pip.conf ?05:31
cloudnullso its not finding or allowing a "dev" (pre-release) package ?05:31
odyssey4meit should be, we enable by default05:31
*** dxiri has joined #openstack-ansible05:31
cloudnullid be curious to see what pip search shows and whats in that dir as built packages05:32
cloudnullalso if theres anything interesting in the pypyserver logs05:33
cloudnullmaybe its skipping files for whatever reason05:33
cloudnulladmin0: ^05:33
odyssey4meprometheanfire that makes my brain hurt05:34
prometheanfireodyssey4me: the indirection/ordering issues?05:35
odyssey4meman, that thing is just a mess of vars - isn't there a way to simplify that into a dict05:36
odyssey4mewhat's with all the vars of vars?05:36
*** dxiri has quit IRC05:36
odyssey4meand then set facts of the same vars05:36
prometheanfireodyssey4me: hell if I know, I came in trying to fix https://bugs.launchpad.net/openstack-ansible/+bug/175835105:36
openstackLaunchpad bug 1758351 in openstack-ansible "ironic is using old things" [High,In progress] - Assigned to Matthew Thode (prometheanfire)05:36
cloudnulllooks like ps2 works?05:37
prometheanfireps2 was a good system05:37
odyssey4meworks, but just as bad in terms of all the vars all over the place05:37
odyssey4mealso, note that works just means that it converges - there's no real test in this role afaik05:38
prometheanfireodyssey4me: also true05:38
cloudnullit also doesn't really change how the role functions05:38
cloudnullhowever I agree, that role is a mess of vars.05:38
odyssey4melooks like the set fact is just setting a bunch of stuff to use in the templates, so why not just implement most of what's being done with the vars of vars in the template and use simple conditionals for the templates to adapt based on standalone vs whatever the other thing is05:39
cloudnullsome cleanup would be nice, though idk if it needs to ride in on that pr.05:39
prometheanfireI'm willing to tackle it in another review05:39
odyssey4meI'd suggest cleaning it up first, then doing the new vars thing on top of that05:40
prometheanfirethat'll delay me a couple weeks probably, have to focus on n->q stuff05:40
odyssey4mewell, as it stands it's unreadable05:40
openstackgerritKevin Carter (cloudnull) proposed openstack/openstack-ansible master: Use production like interfaces in the gate  https://review.openstack.org/55363605:41
prometheanfireyep05:41
prometheanfireodyssey4me: patchset 2 is my prefered solution for right now, it's a little cleaner (doesn't use set_fact)05:41
cloudnullok, gotta run.05:42
prometheanfireya, about to go to bed myself05:42
* cloudnull needs sleep, has another whole day of meetings tomorrow05:42
prometheanfirewoo05:42
cloudnullindeed. take care all05:43
odyssey4meright now, there is ironic_openstack_driver_loaded_list which is only used in two places - defaults/main and the set_fact in ironic_post_install, that set fact sets ironic_driver_list which is used in the template - take out the middle man and just build that in the template05:43
prometheanfireodyssey4me: that's how I generally prefer to do things in puppet as well05:49
prometheanfirelike https://blog.mthode.org/posts/2017/Jun/gentoo-portage-templates/05:49
odyssey4meI suggest taking proactive step to improve the role, instead of putting blinkers on and only doing what the bug needs solved.05:49
odyssey4meI'm reviewing what's there and making some suggestions.05:50
prometheanfireodyssey4me: I'd love to, but I can't do it immediately05:50
odyssey4medoing it immediately will get your patch in faster, because it'll be easier to read.05:50
odyssey4megetting reviews is all about making it easy for the reviewer - when it's complicated, no-one wants to take responsibility for it05:51
*** armaan has quit IRC05:51
odyssey4meespecially for a service the majority of core reviewers don't understand05:51
prometheanfireI have someone threatening to revoke our business licence, an old bdfl trying to come back and take over, a very long meeting to run/plan RFPs to send out for acocunting, find a lawyer05:51
prometheanfirethat's just the gentoo shit05:52
prometheanfireso I don't have time right now05:52
*** yolanda has joined #openstack-ansible05:52
prometheanfireI'd love to basically rewrite the ironic stuff (like you said, it kinda sucks), but can't commit to it05:52
odyssey4mewell, I don't know what to say - I've given some advice and am reviewing the patch. The rest is up to you. If this matters to you, make the time. If not, then it'll stagnate.05:53
prometheanfireyep05:53
prometheanfirewe agree there :P05:54
prometheanfirestill working on the gentoo stuff, which is more important to me, devstack is dragging their feet reviewing patches05:55
prometheanfireanyway, nn05:56
*** deng_ has quit IRC05:56
*** TobbeCN has joined #openstack-ansible05:57
* prometheanfire wonders if osa can use libvirt-python-4.2.0 on xenial (broken it seems), anyway, nn for realz yo05:58
*** _mwarad_ has joined #openstack-ansible06:03
*** mwarad has quit IRC06:03
*** bhujay has quit IRC06:04
*** bhujay has joined #openstack-ansible06:04
*** cshen has joined #openstack-ansible06:08
*** cshen has quit IRC06:09
*** viktor_t has joined #openstack-ansible06:13
*** cshen has joined #openstack-ansible06:14
*** armaan has joined #openstack-ansible06:16
*** _mwarad_ has quit IRC06:16
*** sar has joined #openstack-ansible06:16
*** bhujay has quit IRC06:20
*** bhujay has joined #openstack-ansible06:20
*** pcaruana has joined #openstack-ansible06:21
*** Taseer has quit IRC06:26
openstackgerritJesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-os_ironic master: Do not log passwords  https://review.openstack.org/56211106:35
evrardjpprometheanfire: let me check at that06:36
*** Taseer has joined #openstack-ansible06:38
odyssey4meevrardjp if you could work over all the no password reviews I'd be grateful - they're blocking the virtualenv/pip fixes06:39
evrardjpyup I have them open, I got connectivity issues yesterday.06:40
evrardjphard to do proper reviews on a phone.06:40
odyssey4meah, I didn't realise you were afk - no worries06:41
odyssey4meI thought you were just offline to concentrate on other work.06:41
jrosserodyssey4me: upstream pypi repos seem to hace changed cdn arrangement in the last few days, pretty much everything is now a redirect06:43
odyssey4mejrosser hmm, that's a good point - perhaps pypiserver needs an update06:43
jrosserthat broke our internal proxy/mirror in unusual ways, one to watch out for06:43
evrardjpodyssey4me: I was, then I tried to get back online.06:44
evrardjpthat's true I read an article about it!06:45
evrardjpyeah we should do things.06:45
evrardjpodyssey4me: I am very sad, I thought these trivial patches would be merged by other ppl by now.06:46
odyssey4meapparently everyone is busy :/06:47
*** blinkiz has quit IRC06:48
evrardjpodyssey4me: for trove should we make centos nv ?06:49
openstackgerritMerged openstack/openstack-ansible-os_searchlight master: Do not log passwords  https://review.openstack.org/56222606:51
evrardjphttp://paste.openstack.org/show/719524/06:51
evrardjpI guess cryptography version isn't aligned06:52
*** deng_ has joined #openstack-ansible06:58
evrardjpoh I guess chicken and egg06:59
*** osnaya has joined #openstack-ansible07:00
evrardjpodyssey4me: you're gonna kill me, but do we REALLY need the constraints mechanism in the pip_install role?07:02
evrardjp(for a later improvement)07:02
evrardjpI cannot understand why we don't simplify this now in master (keeping the rest the same)07:02
odyssey4mesimplify how?07:03
*** yifei1 has joined #openstack-ansible07:03
osnaya@admin0 @evrardjp container creation error during setup-hosts.yml log excerpt @http://paste.openstack.org/show/719520/ Can anyone provide pointers? exact same error seen  for rabbitmq, nova_api_placement, heat_engine, heat_api and gnocchi....07:04
evrardjpodyssey4me: just use pip from packages and pip install --upgrade pip==x setuptools==y wheels==z virtualenv==t (which versions x,y,z,t in the role defaults as requirements, not constraints)07:04
odyssey4meosnaya I have no freaking idea - perhaps your hosts are dirty from a previous configuration?07:05
osnaya@admin0 @evrardjp @odyssey4me ... with the error during container creation on setup-hosts.yml I am blocked and need help please07:05
odyssey4melooks like the networking setup failed, so it's most likely something to do with that07:05
evrardjposnaya: yeah what odyssey4me is true07:06
odyssey4meevrardjp for what purpose? what does it help?07:06
evrardjpyou have a dangling veth pair for example07:06
osnaya@odyssey4me Is there a way to clean up other than reimage with Ubuntu?07:06
odyssey4meosnaya given that we have no idea what you did, you're the only one who can answer that07:06
evrardjpodyssey4me: I will check if simply installing virtualenv from packages wouldn't be enough for all we need on the hosts, as part of the plan07:07
evrardjpit would be an intermediate step07:07
odyssey4meevrardjp hwoarang has a few patches up to try something like that07:07
evrardjpif the os doesn't provide the proper package, that would install the proper version07:07
*** mbuil has joined #openstack-ansible07:07
evrardjpI will spin up recent centos to see where the packaging is.07:07
evrardjpbecause it was the only one holding us back07:08
*** shardy has joined #openstack-ansible07:10
odyssey4meevrardjp as mentioned before, if we can progress the python build simplification, then pip_install will only have to execute on a much smaller amount of hosts anyway - so at this point I'd rather have you try and figure out how to help me progress that - I seem to be getting stuck and going in circles07:13
osnaya@odyssey4me @evrardjp ... Ok in that case re-imaging the only option for now, right? OK, will re-install tomorrow... Just curious what pointed out that there is dangling veth pair?07:14
odyssey4meosnaya it'll certainly be simpler07:14
osnaya@odyssey4me @evrardjp @admin0 Wanted to check before I re-image.... since someone else reported similar issue on nova_api_.... wanted to understand if there is anything I should look out for...07:16
odyssey4meosnaya then that's between you and admin007:17
openstackgerritMerged openstack/openstack-ansible-os_ceilometer master: Do not log passwords  https://review.openstack.org/56222507:18
evrardjpodyssey4me: how can I help on the circle thing?07:21
openstackgerritMerged openstack/openstack-ansible-os_heat master: Do not log passwords  https://review.openstack.org/56222307:24
hwoarangevrardjp: odyssey4me the patches i have posted for lxc_hosts and galera friends seem to be enough for infra to be pip clean07:24
evrardjphwoarang: that's cool07:25
evrardjphow do you ensure pip clealiness?07:25
evrardjpshould we build a test for that?07:25
odyssey4meawesome :)07:25
odyssey4methat means that we can ditch the pip_install execution from there if we want to07:25
evrardjphwoarang: I have done a check experimental to see if there is no problem in the general build07:26
hwoarangok07:26
*** osnaya has quit IRC07:26
evrardjpodyssey4me: they are already ditched07:26
evrardjpyou mean the pip_install meta dependency?07:26
evrardjpnow everything is in openstack_hosts07:26
evrardjpbut I mean we can do a generic flip in openstack_hosts07:27
openstackgerritMerged openstack/openstack-ansible-os_gnocchi master: Do not log passwords  https://review.openstack.org/56222107:27
evrardjpput it only in containers07:27
odyssey4meyes, that's what I mean - openstack_hosts still installs it everywhere - we may be able to reduce that footprint07:27
evrardjpyes if it becomes something NOT to do across the board, it has no place in openstack_hosts.07:27
openstackgerritMerged openstack/openstack-ansible-os_glance master: Do not log passwords  https://review.openstack.org/56222907:28
*** pmannidi has quit IRC07:29
openstackgerritMerged openstack/openstack-ansible-os_designate master: Do not log passwords  https://review.openstack.org/56222207:32
*** mma has joined #openstack-ansible07:32
openstackgerritMerged openstack/openstack-ansible-os_barbican master: Do not log passwords  https://review.openstack.org/56223007:35
openstackgerritMerged openstack/openstack-ansible-os_panko master: Do not log passwords  https://review.openstack.org/56224607:36
*** electrofelix has joined #openstack-ansible07:40
openstackgerritMerged openstack/openstack-ansible-os_cinder master: Do not log passwords  https://review.openstack.org/56223107:40
*** epalper has joined #openstack-ansible07:41
evrardjpor if it's still to do accross the board EXCEPTED some groups, then I guess we can avoid the run when: '<exception>' in groupnames07:44
*** pcaruana has quit IRC07:45
*** pcaruana has joined #openstack-ansible07:46
openstackgerritMerged openstack/openstack-ansible-os_aodh master: Do not log passwords  https://review.openstack.org/56222707:47
openstackgerritMerged openstack/openstack-ansible-os_swift master: Do not log passwords  https://review.openstack.org/56223607:48
openstackgerritMerged openstack/openstack-ansible-os_neutron master: Do not log passwords  https://review.openstack.org/56223307:50
openstackgerritMerged openstack/openstack-ansible-os_magnum master: Do not log passwords  https://review.openstack.org/56223207:55
openstackgerritMerged openstack/openstack-ansible-os_sahara master: Do not log passwords  https://review.openstack.org/56223407:59
*** DanyC has joined #openstack-ansible08:00
*** tosky has joined #openstack-ansible08:03
*** DanyC has quit IRC08:04
*** admin0 has quit IRC08:05
hwoarangodyssey4me: evrardjp passing '-c http://git.openstack.org/cgit/openstack/requirements/plain/upper-constraints.txt?h=master' to pip it means 'use the packages listed there are upper constrains' right? so if a lower version is installed it shouldn't be upgraded08:09
hwoarangcorrect?08:09
*** bhujay has quit IRC08:10
openstackgerritMerged openstack/openstack-ansible-os_nova stable/pike: Do not log passwords  https://review.openstack.org/56202208:10
openstackgerritMerged openstack/openstack-ansible-os_nova stable/queens: Do not log passwords  https://review.openstack.org/56202108:10
odyssey4mehwoarang nope - whatever's in the constraints will be installed08:12
odyssey4merequirements are ignored08:12
hwoarangso what's the different with the -r parameter/08:12
odyssey4me-r tells it what to install from a package standpoint, constraints tells it what to install from a version standpoint08:13
hwoarangok08:13
odyssey4meyou can have hundreds of packages in constraints, but only the packages in -r will be installed08:13
odyssey4meit's a little counterintuitive, I know08:13
hwoarangi guess there is a good reason to request specific versions?08:13
*** cshen has quit IRC08:14
odyssey4meyep, it's the only way we can pin things to ensure that the versions are always the same for every tag08:14
hwoarangok then i hit a deadend08:14
odyssey4meotherwise your experience today is different to your experience tomorrow08:14
hwoarangbecause the contrains in os_tempest always pulls the latest cryptography08:14
hwoarangoverriding the distro package08:14
hwoarangso the tempest role has to be converted to distro/source as well for tempest to actually work08:14
hwoarangwith distro keystone08:15
odyssey4mecan the role not have entirely seperate code paths for distro vs non distro?08:15
evrardjpyou can install a specific version with the -r08:15
hwoarangyou mean os_tempest?08:15
hwoarangsure08:15
hwoarangi was just hoping to use the existing code in os_tempest to make keystone work08:15
*** gkadam has joined #openstack-ansible08:16
odyssey4meyeah, I think it's going to require some work in the tempest role too08:16
odyssey4mealthough tempest is installed into a venv, so what's the issue with cryptography? is it installed on the host?08:17
hwoarangyeah os_tempest also pulls packages on host08:19
odyssey4meah, we should try to eliminate that08:19
hwoaranghttps://github.com/openstack/openstack-ansible-os_tempest/blob/master/tasks/tempest_install.yml#L3608:19
hwoarangyeah...08:19
hwoarangthat messes things up08:19
*** mwarad has joined #openstack-ansible08:20
odyssey4mehmm, https://github.com/openstack/openstack-ansible-os_tempest/blob/master/defaults/main.yml#L152-L15808:20
openstackgerritMerged openstack/openstack-ansible-os_tempest master: Do not log passwords  https://review.openstack.org/56223908:20
hwoarangyeah the pip packages look rather strange :p08:20
hwoarangno clue why they need to be on the host08:20
*** mwarad has quit IRC08:21
odyssey4meso virtualenv is there for obvious reasons, shade is there and the others are there because we're using the ansible modules to set things up in the cloud08:21
*** mwarad has joined #openstack-ansible08:22
odyssey4mewe could either move those into the venv, and use the venv when executing those tasks with ansible_python_interpreter08:22
odyssey4meor we could move those into the ansible-runtime venv and make those tasks delegate to localhost08:22
odyssey4meif we get rid of the use of our custom modules, we could probably also get rid of several of those packages08:23
hwoarangthe bit i dont understand is why the various openstack clients have to be one the host *and* in the tempest venv08:23
odyssey4methey don't - it's just lazy08:24
hwoarangthey are used by ansible i suppose so it's easier ot have them on the host than using them from venv08:24
hwoarangthat's my understanding of what you said08:24
odyssey4menope, just set the python interpreter - that's it08:24
hwoarangok08:24
odyssey4meperhaps when this was first done that wasn't an option - or it wasn't known08:24
hwoarangok i will have a look then08:25
odyssey4meI'd go for removing those and setting the interpreter on all the tasks.08:25
*** radeks has joined #openstack-ansible08:25
odyssey4mevirtualenv should ideally be the only one to stay - and we can use a distro package for that.08:25
odyssey4mewe should be able to do much the same in most roles08:26
hwoarangwhy do we need virtualenv there? other os_* roles do not explicitly pull virtualenv08:26
hwoarangat least keystone doesn't08:26
hwoarangsince that's in front of me08:27
odyssey4meprobably an oversight - it probably gets installed by something else...08:27
hwoarangmaybe we assume that something pulled that in already at an earlier stage08:27
hwoarangyeah08:27
*** DanyC has joined #openstack-ansible08:29
openstackgerritMerged openstack/openstack-ansible-os_ironic master: Do not log passwords  https://review.openstack.org/56211108:32
*** mma has quit IRC08:33
Taseerevrardjp: can we close congress once and for all ? => https://review.openstack.org/#/c/560839/08:34
Taseeror do I still need to have some functional testing ?08:36
*** jwitko has quit IRC08:53
*** bhujay has joined #openstack-ansible09:00
evrardjpTaseer: it's not killing once and for all, it's just the start :)09:04
evrardjpbut yes, for me it means the work is done09:04
Taseerevrardjp: I see I got your approval. But again, do you still want functional tests in the future ? so I had prepare myself for further work09:06
*** hamza21 has joined #openstack-ansible09:27
*** mwarad has quit IRC09:34
openstackgerritMarkos Chandras (hwoarang) proposed openstack/openstack-ansible-os_keystone master: templates: keystone-uwsgi: Allow uwsgi to autoload required plugins.  https://review.openstack.org/56030709:38
openstackgerritMarkos Chandras (hwoarang) proposed openstack/openstack-ansible-os_keystone master: [RFC] Add support for using distribution packages for OpenStack services  https://review.openstack.org/56030809:38
*** vnogin has joined #openstack-ansible09:39
*** foutatoro has joined #openstack-ansible09:44
*** armaan has quit IRC09:55
*** cshen has joined #openstack-ansible09:55
*** armaan has joined #openstack-ansible09:55
*** vnogin has quit IRC09:58
*** vnogin has joined #openstack-ansible10:02
*** cshen has quit IRC10:02
*** bhujay has quit IRC10:33
*** dxiri has joined #openstack-ansible10:38
*** sar has quit IRC10:40
*** dxiri has quit IRC10:42
ivvehey guys, would it be considered a deployerror to deploy the openstack with defaults (self signed cert) and not have aodh confed not to verify ssl cert?10:44
evrardjpTaseer: no I think those functional tests are enough10:44
evrardjpTaseer: just maintain them as working :)10:44
ivvei mean deployer must be aware of it & use an override10:44
evrardjplet's move them to voting when we had multiple runs okay without timeouts10:44
ivvefeels like it should default to, aodh.conf rest_notifier_ssl_verify = false10:45
ivvein the [DEFAULT] stanza10:45
openstackgerritJesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-repo_server master: Set the pypiserver root to the correct directory  https://review.openstack.org/56254410:45
odyssey4meevrardjp hwoarang https://review.openstack.org/562544 solves an inconsistent bug which has been popping up - it'd be good to get that resolved asap10:46
evrardjpodyssey4me: depends on ^ for https://review.openstack.org/#/c/549012/2 ?10:51
odyssey4meevrardjp then I lose my votes ;)10:51
evrardjpI will apply them.10:51
odyssey4mebut yeah, will do that now10:51
evrardjpExcept if you want to keep mhayden vote.10:51
evrardjpthat's collector now10:51
evrardjp:)10:52
openstackgerritJesse Pretorius (odyssey4me) proposed openstack/openstack-ansible master: Remove default pip_links value  https://review.openstack.org/54901210:52
evrardjpandymccr: could you quickly vote on https://review.openstack.org/#/c/562544/1 please?10:52
andymccrsi si10:53
odyssey4metyvm all10:53
andymccrnice catch. damn10:53
*** nicolasbock has joined #openstack-ansible10:57
*** armaan has quit IRC10:59
*** armaan has joined #openstack-ansible10:59
*** sar has joined #openstack-ansible11:00
*** cshen has joined #openstack-ansible11:00
*** ansmith has quit IRC11:13
*** mwarad has joined #openstack-ansible11:19
*** deng_ has quit IRC11:20
*** armaan has quit IRC11:37
evrardjphwoarang: andymccr cloudnull mgariepy we have plenty of reviews pending, for example Jesse's patches are queuing up: https://review.openstack.org/#/q/owner:jesse.pretorius%2540rackspace.co.uk+status:open++label:%22-Workflow%22+is:mergeable11:39
openstackgerritMerged openstack/openstack-ansible stable/queens: Add IP addresses to infra node br-vxlan in the examples  https://review.openstack.org/56232911:40
openstackgerritMerged openstack/openstack-ansible-os_monasca-agent master: Replace virtualenv-tools with a script  https://review.openstack.org/56220811:42
*** mugsie has quit IRC11:42
*** mugsie has joined #openstack-ansible11:42
*** mugsie has quit IRC11:42
*** mugsie has joined #openstack-ansible11:42
*** vnogin has quit IRC11:43
openstackgerritJean-Philippe Evrard proposed openstack/openstack-ansible-os_monasca stable/ocata: Initial OSA zuul v3 role jobs  https://review.openstack.org/51664611:45
*** vnogin has joined #openstack-ansible11:46
*** vnogin has quit IRC11:47
*** viktor_t has quit IRC11:51
*** sar has quit IRC11:54
*** sar has joined #openstack-ansible11:58
*** armaan has joined #openstack-ansible12:00
openstackgerritMerged openstack/openstack-ansible-galera_server stable/queens: Remove perl library from Ubuntu installs  https://review.openstack.org/56231712:00
*** mwarad has quit IRC12:01
*** vnogin has joined #openstack-ansible12:03
*** vnogin has quit IRC12:07
*** vnogin has joined #openstack-ansible12:07
*** deng_ has joined #openstack-ansible12:10
*** armaan has quit IRC12:11
*** vnogin has quit IRC12:17
*** vnogin has joined #openstack-ansible12:17
*** armaan has joined #openstack-ansible12:17
*** armaan has quit IRC12:22
*** vnogin has quit IRC12:31
openstackgerritMerged openstack/openstack-ansible-tests stable/pike: Extend linters time out  https://review.openstack.org/56202712:32
openstackgerritMerged openstack/openstack-ansible-os_nova stable/queens: Re-order database migrations  https://review.openstack.org/56202312:37
mgariepymorning everyone12:37
* mgariepy reviewing 12:38
andymccrmgariepy: hey!12:43
mgariepywhat's up andymccr12:43
*** sar has quit IRC12:45
*** vnogin has joined #openstack-ansible12:45
*** ansmith has joined #openstack-ansible12:47
*** yifei1 has quit IRC12:47
*** vnogin has quit IRC12:51
openstackgerritMerged openstack/openstack-ansible-os_searchlight master: Only replace python when re-initializing the venv  https://review.openstack.org/56219612:51
*** cmart has joined #openstack-ansible12:55
openstackgerritMerged openstack/openstack-ansible stable/queens: Remove spurious VxLAN bridge IP from ceph osd hosts  https://review.openstack.org/56087813:01
*** armaan has joined #openstack-ansible13:02
*** esberglu has quit IRC13:05
*** markvoelker_ has quit IRC13:16
*** mwarad has joined #openstack-ansible13:18
*** markvoelker has joined #openstack-ansible13:18
openstackgerritMerged openstack/openstack-ansible-os_ceilometer master: Only replace python when re-initializing the venv  https://review.openstack.org/56216313:19
openstackgerritJean-Philippe Evrard proposed openstack/openstack-ansible master: Remove Octavia from translations scenario  https://review.openstack.org/56257813:19
evrardjpcould I have votes on ^13:19
evrardjpthat would unblock translations13:19
evrardjpandymccr: maybe ?13:19
andymccrlooking!13:20
andymccrLGTM13:20
*** esberglu has joined #openstack-ansible13:24
openstackgerritJean-Philippe Evrard proposed openstack/openstack-ansible-os_horizon master: Do not log passwords  https://review.openstack.org/56258213:26
openstackgerritJean-Philippe Evrard proposed openstack/openstack-ansible-os_horizon master: Rename pull_catalog args from version to branch  https://review.openstack.org/55929013:27
openstackgerritMerged openstack/openstack-ansible-repo_build stable/ocata: Fix repo_build_venv_rebuild toggle effectiveness  https://review.openstack.org/56233713:27
evrardjpthat one is preventing success in horizon and for the next sha bump: https://review.openstack.org/#/c/562582/13:28
evrardjpthanks everyone!13:28
mgariepyhwoarang, http://logs.openstack.org/33/562033/1/gate/openstack-ansible-functional-opensuse-423/0bd6742/job-output.txt.gz#_2018-04-19_13_04_13_77563513:28
mgariepyseems like the image got renamed..13:29
mgariepyhttps://github.com/openSUSE/docker-containers-build/commit/81638d69fffe52e00863e83e9949a0c5d2cfdba313:29
evrardjpthat's gonna be painful13:32
evrardjphaters are gonna hate!13:32
odyssey4me:/ and the old image is no longer present? that's not good practise13:32
openstackgerritMerged openstack/openstack-ansible-os_ironic master: Only replace python when re-initializing the venv  https://review.openstack.org/56217713:33
odyssey4methankfully that's master only, but if this continues happening then we're going to have trouble with these oci image sources - we need a stable source13:33
hwoarangmgariepy: i am on it13:34
openstackgerritMerged openstack/openstack-ansible-tests stable/ocata: Extend linters time out  https://review.openstack.org/56202813:34
hwoarangmgariepy: https://github.com/openSUSE/docker-containers-build/issues/2913:34
hwoarangi am working on some ansible code to determine the build number on the fly13:34
evrardjphwoarang: you're the deal13:35
*** TobbeCN has quit IRC13:35
hwoarangodyssey4me: from what i can tell these images are just autogenerated from time to time (for all distros) so it may not be the same image all the time. The rest of the distros do not carry any build info on them as well. they use the same tarball for all releases13:35
*** TobbeCN has joined #openstack-ansible13:36
hwoarangi have to ask cloudnull why we moved to these images instead of those provided by images.linuxcontainers.org13:36
idlemindso the successerator (did i spell that right) that you guys use for retries ... can that be harvested for use in other playbooks? in particular i'm looking for a solution to deal w/intermittent failures of the new "network_cli" connection while connected via VPN the hosts aren't really down, just for some reason I occasionally see failures connecting to some of the devices that way.13:36
odyssey4meidlemind we haven't used the successerator for some time13:37
evrardjphwoarang: I think it's not something to only ask. I think it should be documented.13:37
evrardjpPart of the reference architecture if possible13:37
evrardjpa new section here: https://docs.openstack.org/openstack-ansible/latest/reference/architecture/index.html , like container sources?13:37
*** jwitko has joined #openstack-ansible13:38
odyssey4meor perhaps we should just build our own instead?13:38
hwoarangmaybe yes13:38
evrardjpodyssey4me: what do you mean there?13:38
evrardjpI am scared.13:38
evrardjphwoarang: I am pretty sure it's cloudnull 's crusage to make lxc BE THE SAME as nspawn.13:39
odyssey4mewell, there's a role for diskimage-builder - and the base image does not need very much13:39
idlemindodyssey4me shame, ya i didn't get a response in #ansible-network on the retry ability of network_cli but it doesn't seem to follow the normal ansible_ssh_retries ... i'll have to keep looking or i'll look through the repo for osa to have a glance at the successerator and see if i can do something similar ... i want to switch more of my customer facing real job work to ansible driven but a lot of that is done over vpn13:39
idlemindand it appears that sometimes network_cli fails when it shouldn't (immediately check via ping and manual ssh and the host is there)13:39
odyssey4meevrardjp well, the justification was that we had to remove a bunch of stuff from it and that it was too heavy13:39
evrardjpodyssey4me: for me, we should just say: Use that tar.13:39
evrardjpthat tar is from a website13:40
odyssey4meevrardjp sure, but what tar -we have no stable source for it13:40
*** TobbeCN has quit IRC13:40
evrardjpI agree but it's the same story as everyone else13:40
*** mwarad has quit IRC13:40
evrardjpcan't we suck it up from dockerhub?13:40
odyssey4meidlemind using retry + delay + until | success with a register?13:40
evrardjphttps://hub.docker.com/r/opensuse/amd64/ ?13:42
odyssey4meevrardjp we already have implementations for using diskimage-builder inside trove and octavia, it seems sensible to me to unify those and the base prep13:42
odyssey4meand, at the same time, pull the base prep out of the {lxc,nspawn}_hosts roles13:42
evrardjpodyssey4me: don't get me wrong, I am not against it13:42
evrardjpI'd simply like us to say "Use that image"13:42
evrardjpthat image can even be a group var13:43
odyssey4methat's already an option afaik13:43
hwoarangseems i opened the usual can of worms13:43
evrardjphwoarang: hahahahah13:43
evrardjphwoarang: I love it very much when you do it :)13:43
hwoarangyou go fix one thing and ten more things pop up on the side13:44
evrardjpodyssey4me: I think we all agree13:44
openstackgerritMerged openstack/openstack-ansible-repo_build stable/pike: Fix repo_build_venv_rebuild toggle effectiveness  https://review.openstack.org/56233613:44
evrardjpwe should spin that out of the role, the role should just consume images13:44
odyssey4megiven that I have more familiarity with how dib works, perhaps I should just go ahead and do it - I've had enough of walking in this circle13:44
evrardjpthe images can be built with DIB at the beggining of the process13:44
evrardjpodyssey4me: well13:44
evrardjpit's gonna be a problem with OOMing IIRC.13:44
evrardjpby adding that in the process of the deploy13:45
evrardjpand definitely lengthen the build13:45
evrardjpso we need a way to have a separate repo doing it13:45
evrardjpand the others consuming it13:45
hwoarangfyi we are doing that in OPNFV and dib is not cheap. you are looking at 10-15+ extra time for deployments13:45
evrardjpwhich is the cause of issues IIRC.13:45
odyssey4meI doubt it will cause an OOM, but yes it may lengthen the build a bit.13:45
evrardjpodyssey4me: that's how we got the first OOMing in octavia13:46
evrardjpIIRC13:46
odyssey4mewell, we only need a very small base - no kernel, etc - bear that in mind13:46
*** deng_ has quit IRC13:46
odyssey4methis is not a dib for a whole host base like bifrost needs, or a VM base like octavia builds13:47
odyssey4mewe could, for now, flip the default back to the 'legacy' lxc pull13:49
evrardjpodyssey4me: could you tell me how dib can leverage debootstrap for example?13:49
evrardjpBecause for me the lightest image is debootstrap + dbus13:50
*** dxiri has joined #openstack-ansible13:50
odyssey4meevrardjp dib does that, but also caches and has a ton of optimisation it'd be a waste for us to repeat13:50
evrardjpor pacstrap for arch :)13:50
odyssey4mealso dib uses the best system on each distro for the job13:50
evrardjpthat's cool13:50
evrardjpthat's very nice13:51
evrardjpit proves I really have 0 knowledge of how cool dib is13:51
evrardjpwe have too many things going on13:51
evrardjplet's not do that right now13:51
odyssey4medebootstrap is debian/ubuntu only IIRC13:51
evrardjpodyssey4me: yes.13:51
openstackgerritMerged openstack/openstack-ansible-os_magnum master: Only replace python when re-initializing the venv  https://review.openstack.org/56217913:51
evrardjplet's focus on python simplification it's a big chunk13:52
*** dxiri has quit IRC13:52
evrardjpand hwoarang mnaser focusing on packages13:52
evrardjpwhile I focus on the inventory13:52
odyssey4mesure, but we also need a stable source for these base images - otherwise the rocky release is at risk of being useless13:52
hwoarang-ESOMANYTHINGSTODO13:53
evrardjp:)13:53
evrardjphwoarang: the plan is not complete, there is Tahvok cleaning up stuff too13:53
evrardjpand new features!13:53
evrardjpI am just worried we are stretched thin13:54
evrardjpbut indeed making sure it's stable is coming first13:54
evrardjpbut I think python/packages are postive improvements for stability13:54
openstackgerritMarkos Chandras (hwoarang) proposed openstack/openstack-ansible-lxc_hosts master: SUSE: Determine the build number on the fly  https://review.openstack.org/56259913:54
hwoarangevrardjp: odyssey4me what do you think^13:55
evrardjpmaybe I should focus more on switching images then13:55
evrardjphwoarang: is that the same version number for all arches?13:55
hwoarangit's not the same but you retrieve that on the fly13:56
hwoarangparsing the appropriate Dockerfile13:56
odyssey4meseemse reasonable enough13:57
odyssey4meI guess another option is to use a base docker image from docker hub and get the image details by doing an api query and getting the response? I'm not sure how stable that interface is.13:57
evrardjpodyssey4me: there is probably a module for that13:58
hwoarangno clue... i still want to know what was wrong with images.linuxcontainers.org13:58
hwoarang:)13:58
evrardjphttp://docs.ansible.com/ansible/2.5/modules/docker_image_facts_module.html#docker-image-facts-module13:58
odyssey4methe only trouble with this mechanism is that it reliaes on this two step process, instead of just downloading a tarball like it does now13:58
evrardjphttp://docs.ansible.com/ansible/2.5/modules/docker_login_module.html#docker-login-module13:58
hwoarangoh that module needs a bunch of docker deps13:58
evrardjpfetching it would be http://docs.ansible.com/ansible/2.5/modules/docker_image_module.html#docker-image-module13:59
evrardjpyeah and we've never been using docker13:59
evrardjpso now we have all the API issues we've never seen before13:59
hwoaranguse docker to configure lxc13:59
hwoarangoh the irony13:59
evrardjpthe question is where is the irony... :)13:59
*** niraj_singh has joined #openstack-ansible14:00
jrosserhas anyone deployed fwaas? i follow the steps in the os_neutron role docs and get "neutron.manager [req-1133af1e-a39e-4004-9ee2-026ed69516bc - - - - -] Plugin 'firewall' not found."14:00
*** cmart has quit IRC14:00
hwoarangi think that was recently fixed14:00
hwoarangmbuil: ^ ?14:00
hwoarangand it should already be backported in queens14:01
evrardjpmaybe a bump was not yet done? Not so sure to have seen that in Q14:01
evrardjpor it might be long ago14:01
odyssey4memachinectl no longer does pull-dkr "because of the forever breaking api of docker"14:02
odyssey4meso yeah, no go on that idea14:02
evrardjpyeah I remember that PR :)14:03
mbuilhwoarang, jrosser: it was fixed last week14:05
mbuilfor Queens --> https://review.openstack.org/#/c/560020/14:05
openstackgerritMerged openstack/openstack-ansible-os_tempest stable/queens: Allow public network name to be changed  https://review.openstack.org/56223714:06
*** nicolasbock has quit IRC14:10
jrossermbuil: ah interesting, would you expect fwaas v1 to fail without that too?14:10
openstackgerritMerged openstack/openstack-ansible-os_tempest stable/pike: Allow public network name to be changed  https://review.openstack.org/56223814:11
openstackgerritMarkos Chandras (hwoarang) proposed openstack/openstack-ansible-os_tempest master: Avoid mixing pip and distribution packages.  https://review.openstack.org/56260614:12
evrardjpjrosser: v1 is supposed to be dead IIRC14:12
evrardjpor maybe I am confused?14:12
jrosserme too :)14:12
evrardjpconfused I think14:12
*** Adri2000 has quit IRC14:13
*** cmart has joined #openstack-ansible14:13
jrosseri mean, it's a valid quesstion which i *should* be using14:13
evrardjpI think it's better to find info more upstream14:14
jrossereither way i can't make the install if either work, v2 seems to have a patch ^^^ but v1 i don't know why14:14
* jrosser digs a bit more14:14
mbuilevrardjp, jrosser: correct, fwaas v1 is not supported anymore14:15
evrardjpok14:15
evrardjpthat's what I thought.14:15
evrardjpcool14:15
evrardjpwe should remove all references to v1 then14:15
jrosserah right ok - configure_network_services.rst needs a tweek then14:15
mbuilthe official statement is: "fwaas v1 is deprecated. it is still in the tree, but new usage is highly discouraged."14:16
openstackgerritMerged openstack/openstack-ansible-os_nova stable/pike: Re-order database migrations  https://review.openstack.org/56202414:17
openstackgerritMarkos Chandras (hwoarang) proposed openstack/openstack-ansible-os_keystone master: [RFC] Add support for using distribution packages for OpenStack services  https://review.openstack.org/56030814:17
openstackgerritMarkos Chandras (hwoarang) proposed openstack/openstack-ansible-os_keystone master: Avoid mixing pip and distribution packages  https://review.openstack.org/56260714:17
hwoarangevrardjp: odyssey4me do you have a list of stuff that needs revewing? i am done coding for the day14:17
hwoarangotherwise i can simply go through the osa dashboard14:17
evrardjpyup go through the dashbaord :)14:18
hwoarangok14:18
odyssey4mehwoarang if we can get something merged to unblock suse tests for master then that'd be a good start... much of what merged today (or just got voted on) I need to port back to queens at least14:18
hwoarangodyssey4me: i proposed https://review.openstack.org/#/c/562599/ to unblock the gates14:19
*** epalper has quit IRC14:26
openstackgerritAndy Smith proposed openstack/openstack-ansible-os_cinder master: (WIP) Update to use oslo.messaging service for RPC and Notify  https://review.openstack.org/56057414:28
*** armaan has quit IRC14:28
openstackgerritAndy Smith proposed openstack/openstack-ansible-os_keystone master: (WIP) Update to use oslo.messaging service for RPC and Notify  https://review.openstack.org/56202614:33
*** vnogin has joined #openstack-ansible14:34
*** vnogin has quit IRC14:35
evrardjpansmith: thanks for the continued effort! :)14:37
openstackgerritMerged openstack/openstack-ansible-repo_build stable/queens: Fix repo_build_venv_rebuild toggle effectiveness  https://review.openstack.org/56233514:39
spotzHey don't forget my little patch! https://review.openstack.org/#/c/551316/ :)14:39
ansmithevrardjp: wanted to clean up CI a bit before asking for folks to take a look at the reviews14:40
ansmithevrardjp: probably getting close to doing a sanity check on the approach knowing there is still more to do14:41
*** hamza21 has quit IRC14:44
toskyevrardjp: hi again - we discussed about adding a cross-repo job today in the sahara meeting and it was agreed that it's good14:46
evrardjpgreat!14:46
*** markvoelker has quit IRC14:47
evrardjpodyssey4me and I have worked on the infra testing, so we can help you on the jobs14:47
evrardjpkeystone was a first example14:47
toskyso I will work on this; I guess I would need to start with one job to not overload the infrastructure too much14:47
toskyI think I can manage the job part in general; I suspect I will ask more questions about the specific policies for your jobs14:47
evrardjpyes, just one job is fine: The deploy of sahara with the os_sahara role14:47
toskysure, but there are many platforms :)14:48
evrardjpso latest sahara changes can be properly tested14:48
evrardjpoh sure :)14:48
evrardjpstarting with ubuntu is the easiest, because that's what we've done already14:48
evrardjpbut extending should be easy14:48
*** markvoelker has joined #openstack-ansible14:48
*** SimAloo has joined #openstack-ansible14:49
toskyI've also seen a centos 7 job, if I'm not mistaken14:49
evrardjpbut I'd like we go for another test later: to add a full OSA deploy scenario with sahara14:49
toskybut let's see, changing that is easy14:49
evrardjptosky: indeed, and SUSE14:49
evrardjpyeah, and ubuntu has been our history for a long while, so it's the most stable and the fastest.14:49
toskyoh, you didn't mean a full OSA deploy scenario? I though you mean a job which is equivalent to the devstack deployment14:49
openstackgerritMerged openstack/openstack-ansible stable/pike: Fix osa_toolkit dictutils import  https://review.openstack.org/56000814:50
openstackgerritMerged openstack/openstack-ansible stable/queens: Add nova/ironic/glance cross-service vars to group_vars/all  https://review.openstack.org/56002414:50
evrardjpyou don't need the full things if you're just looking for a devstack equivalent14:50
odyssey4meevrardjp let's get it in to stop the bleeding: https://review.openstack.org/#/c/562599/114:50
evrardjptosky: that's the good thing :)14:50
evrardjpodyssey4me: adding a bug14:50
toskytechnically I'd like to add a full deployment + scenario tests with the fake plugin instead or in addition to the tempest tests14:50
toskylike the sahara-tests-scenario jobs defined in the sahara-tests repository14:51
toskythat's a better measurement that everything is fine14:51
openstackgerritJesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-rabbitmq_server master: Revert "Configure plugins and policies after RabbitMQ"  https://review.openstack.org/56262114:52
evrardjpodyssey4me: https://bugs.launchpad.net/openstack-ansible/+bug/176543214:52
openstackLaunchpad bug 1765432 in openstack-ansible "Decide the source of the base images" [Undecided,New]14:52
evrardjpmnaser: if https://review.openstack.org/#/c/562621/1 merges before you got the chance to look at it, please be aware of the revert.14:53
odyssey4metosky yeah, our ideal is some sort of canary test rather than a full function api test or something like that - we're not wanting to test that the API is conforming to the spec, we're wanting to verify that we have a functioning system14:53
*** vnogin has joined #openstack-ansible14:54
mnaserevrardjp: sorry :(14:55
*** vnogin has quit IRC14:55
mnaserevrardjp: would it be okay to share some of those breakages14:55
mnaserodyssey4me: ^^14:55
toskyodyssey4me: sure, that's why I suggest a scenario test14:55
toskywith the fake plugin14:56
toskythat's our canary test, sadly, due to the interaction between sahara and any possible openstack project under the sun14:56
openstackgerritJean-Philippe Evrard proposed openstack/openstack-ansible-plugins master: Remove python3 job  https://review.openstack.org/56266614:58
odyssey4metosky ah, if you have a fake plugin then you could shorten the test time quite a bit on that role - right now we deploy all the services!14:58
evrardjpandymccr: we should use the same topic14:59
odyssey4memnaser sure, lemme register a bug14:59
evrardjplet's name it osa_python3?14:59
toskyodyssey4me: and we would need all the services14:59
toskyI think I'm not explaining enough how sahara need a full deployment if you want to avoid surprises14:59
andymccrevrardjp: ok cool15:00
*** cjloader_ has joined #openstack-ansible15:01
openstackgerritJean-Philippe Evrard proposed openstack/openstack-ansible-plugins master: Remove python3 job  https://review.openstack.org/56266615:02
openstackgerritAndy McCrae proposed openstack/openstack-ansible-tests master: Add python3 functional voting and non-voting jobs  https://review.openstack.org/56266715:02
evrardjpandymccr: for your patch to pass, we need to remove the job completely first, so my patch should be included first.15:02
andymccrok i can put a depends on15:02
openstackgerritAndy McCrae proposed openstack/openstack-ansible-tests master: Add python3 functional voting and non-voting jobs  https://review.openstack.org/56266715:03
andymccrthat look better?15:03
openstackgerritMerged openstack/openstack-ansible stable/queens: Use a sensible vlan range in the example configs  https://review.openstack.org/56087715:03
evrardjpandymccr: you mind if I edit it?15:04
*** TobbeCN has joined #openstack-ansible15:04
evrardjpnot sure the depends on is a good idea for that15:04
andymccrgo for it15:04
evrardjpbut let's try15:04
andymccri'll check the diff so i know what i didnt do :D15:05
evrardjpyeah depends on doesn't seem to work15:05
openstackgerritJean-Philippe Evrard proposed openstack/openstack-ansible-tests master: Add python3 functional voting and non-voting jobs  https://review.openstack.org/56266715:05
evrardjpyou did good15:05
evrardjpjust simplifying15:05
andymccrahh i see15:05
andymccrcool15:05
evrardjp(doing like the other jobs)15:05
andymccrgood tip15:05
evrardjpso now you can use them in the roles, the way you want15:06
hwoarangevrardjp: odyssey4me sadly the opensuse tarballs are going away https://github.com/openSUSE/docker-containers-build/issues/2915:07
odyssey4memnaser https://bugs.launchpad.net/openstack-ansible/+bug/176543815:07
openstackLaunchpad bug 1765438 in openstack-ansible "RabbitMQ policy application not working for AIO" [Undecided,New]15:07
hwoarangso we must come with an alternative soonish...15:07
hwoarangpossibly go back to images.linuxcontainers.org15:08
*** kstev has joined #openstack-ansible15:08
odyssey4meyeah15:08
*** TobbeCN has quit IRC15:08
odyssey4meor perhaps the docker api has stabilised enough now that we can patch machinectl ;)15:09
evrardjplol15:09
evrardjpodyssey4me: ask hwoarang how easy is it to include a patch into systemd15:09
evrardjp:p15:09
hwoarang:(15:10
evrardjpsorry I had to :(15:10
*** vnogin has joined #openstack-ansible15:10
evrardjpdid you see my bug?15:10
evrardjpCould you add this behavior change into the bug?15:10
*** zerick has joined #openstack-ansible15:10
evrardjphttps://bugs.launchpad.net/openstack-ansible/+bug/176543215:10
openstackLaunchpad bug 1765432 in openstack-ansible "Decide the source of the base images" [Undecided,New]15:10
openstackgerritMerged openstack/openstack-ansible-lxc_hosts master: SUSE: Determine the build number on the fly  https://review.openstack.org/56259915:10
evrardjphwoarang: it would be nice to vote on https://review.openstack.org/#/c/562621/115:11
*** DanyC has quit IRC15:11
*** DanyC has joined #openstack-ansible15:12
evrardjpFYI, I've asked for the inclusion of masakari into the openstack ansible namespace, and I will work with niraj_singh on making this happen. Thanks niraj_singh !15:12
* odyssey4me has no idea what masakari is, but thanks niraj_singh :)15:13
evrardjphwoarang: woot it's not a hard no15:15
evrardjpif we have official light images from openSUSE team, that would be very nice.15:15
*** DanyC has quit IRC15:16
openstackgerritMerged openstack/openstack-ansible-os_panko master: Replace virtualenv-tools with a script  https://review.openstack.org/56219015:17
evrardjpwe got a few patches flowing it's very cool15:17
*** mbuil has quit IRC15:17
odyssey4meyeah, thanks for putting that workaround together so quickly hwoarang15:17
evrardjpodyssey4me: thanks for translating my words, I realize it was not really clear, once again.15:18
evrardjp:D15:19
evrardjpgood teaming!15:19
*** mma has joined #openstack-ansible15:22
evrardjphwoarang: to what I see https://github.com/openSUSE/docker-containers-build/issues/29 , we can still use it for a certain time15:22
openstackgerritJesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-os_magnum stable/queens: Only replace python when re-initializing the venv  https://review.openstack.org/56267315:22
evrardjpbut that's indeed a ticking time bomb15:22
openstackgerritJesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-os_ironic stable/queens: Only replace python when re-initializing the venv  https://review.openstack.org/56267415:23
openstackgerritJesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-os_ceilometer stable/queens: Only replace python when re-initializing the venv  https://review.openstack.org/56267515:23
openstackgerritJesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-os_searchlight stable/queens: Only replace python when re-initializing the venv  https://review.openstack.org/56267615:23
*** vnogin has quit IRC15:24
openstackgerritJesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-os_monasca-agent stable/queens: Replace virtualenv-tools with a script  https://review.openstack.org/56267715:24
xdfilHi, folks.15:26
openstackgerritJesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-os_ironic stable/queens: Do not log passwords  https://review.openstack.org/56267815:27
openstackgerritJesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-os_ironic stable/pike: Do not log passwords  https://review.openstack.org/56267915:27
openstackgerritJesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-os_tempest stable/queens: Do not log passwords  https://review.openstack.org/56268015:27
openstackgerritJesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-os_sahara stable/queens: Do not log passwords  https://review.openstack.org/56268115:27
openstackgerritJesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-os_sahara stable/pike: Do not log passwords  https://review.openstack.org/56268215:28
openstackgerritJesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-os_magnum stable/queens: Do not log passwords  https://review.openstack.org/56268315:28
openstackgerritJesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-os_magnum stable/pike: Do not log passwords  https://review.openstack.org/56268415:28
openstackgerritJesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-os_neutron stable/queens: Do not log passwords  https://review.openstack.org/56268515:28
openstackgerritJesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-os_neutron stable/pike: Do not log passwords  https://review.openstack.org/56268615:28
openstackgerritJesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-os_swift stable/queens: Do not log passwords  https://review.openstack.org/56268815:29
*** vnogin has joined #openstack-ansible15:29
evrardjphey xdfil15:29
xdfilsay i'm using 17.0.3, where the default group_var setting for neutron_agents, is_metal: true15:29
xdfilBut, when the play runs it makes a neutron_agents container still.15:29
openstackgerritJesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-os_aodh stable/queens: Do not log passwords  https://review.openstack.org/56268915:29
openstackgerritJesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-os_aodh stable/pike: Do not log passwords  https://review.openstack.org/56269015:29
evrardjpxdfil: check the inventory maybe?15:29
xdfilI have to copy the env.d to my /etc/openstack_deploy for it to build as metal15:29
openstackgerritJesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-os_panko stable/queens: Do not log passwords  https://review.openstack.org/56269115:29
evrardjpxdfil: and you have an env.d already about that group?15:30
openstackgerritJesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-os_panko stable/pike: Do not log passwords  https://review.openstack.org/56269315:30
openstackgerritJesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-os_barbican stable/queens: Do not log passwords  https://review.openstack.org/56269415:30
openstackgerritJesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-os_barbican stable/pike: Do not log passwords  https://review.openstack.org/56269515:30
openstackgerritJesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-os_designate stable/queens: Do not log passwords  https://review.openstack.org/56269615:30
openstackgerritJesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-os_designate stable/pike: Do not log passwords  https://review.openstack.org/56269715:30
evrardjpenv.d would have precedence to group var as being host var15:30
xdfilso it's "fixed" but I'm seeing a pattern emerging that I am bad at predicting which variables take precedence15:31
openstackgerritJesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-os_glance stable/queens: Do not log passwords  https://review.openstack.org/56269915:31
openstackgerritJesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-os_glance stable/pike: Do not log passwords  https://review.openstack.org/56270015:31
openstackgerritJesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-os_gnocchi stable/queens: Do not log passwords  https://review.openstack.org/56270115:31
openstackgerritJesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-os_gnocchi stable/pike: Do not log passwords  https://review.openstack.org/56270215:31
xdfilno I didn't have a neutron.yml in my /etc15:31
openstackgerritJesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-os_heat stable/queens: Do not log passwords  https://review.openstack.org/56270315:31
openstackgerritJesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-os_heat stable/pike: Do not log passwords  https://review.openstack.org/56270415:31
xdfilI had a cinder.yml  that's all15:32
pabelangermorning, has OSA started testing on ansible 2.5?15:32
openstackgerritJesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-os_ceilometer stable/queens: Do not log passwords  https://review.openstack.org/56270515:32
evrardjppabelanger: yes15:32
openstackgerritJesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-os_ceilometer stable/pike: Do not log passwords  https://review.openstack.org/56270615:32
evrardjppabelanger: we have a few bugs pending15:32
openstackgerritJesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-os_searchlight stable/queens: Do not log passwords  https://review.openstack.org/56270715:32
evrardjpwe want to introduce a job for devel in the future15:32
pabelangerevrardjp: ansible bugs or issues with playbooks?15:32
evrardjpwe have, but we want to make that more interesting15:32
evrardjpright now it's issues with plugins15:32
pabelangerdo you mind sharing the location of your bugs?  We're discussing updating zuul directly to 2.5 and skipping 2.415:34
pabelangercurrently 2.315:34
evrardjpwow15:34
evrardjpthat's a big change15:34
pabelangerright, trying to see if we should step to 2.4 first15:34
pabelangerthen into 2.515:34
evrardjpwell if you can afford the 2.5 move go for it15:34
evrardjp2.5.1 is just out15:35
openstackgerritJesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-os_nova stable/queens: Only replace python when re-initializing the venv  https://review.openstack.org/56270915:35
pabelangerfear is breakage of all jobs in openstack obviously, so would need to see the breaking changes from 2.3 to 2.515:35
openstackgerritJesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-os_octavia stable/queens: Only replace python when re-initializing the venv  https://review.openstack.org/56271015:35
evrardjppabelanger: jmccrory: is working on those 2.5 bump15:35
openstackgerritJesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-os_keystone stable/queens: Only replace python when re-initializing the venv  https://review.openstack.org/56271115:35
openstackgerritJesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-os_molteniron stable/queens: Only replace python when re-initializing the venv  https://review.openstack.org/56271715:36
evrardjpwe have this one: https://review.openstack.org/#/c/550665/15:36
evrardjppabelanger: ^15:36
evrardjpcheck also the check experimental there15:36
evrardjpfor 2.4 we have a few changes too.15:37
evrardjpI can't remember. Let me check if I put all of that into a topic15:37
pabelangerk, thanks15:38
odyssey4meapologies for that patch storm - that's most of the merged changes relating to the password linting and the py3 issues in venvs done15:39
odyssey4methere'll be some more tomorrow once the outstanding ones merge, and I got some merge conflicts when trying to port some back - so I'll have to resolve those manually15:39
odyssey4meassuming those all pass tests, I'd appreciate it if we could get those out of the queue asap15:39
evrardjppabelanger: https://review.openstack.org/#/q/topic:ansible-2.4 for 2.415:39
*** dxiri has joined #openstack-ansible15:40
pabelangerevrardjp: thanks! looking15:40
evrardjpodyssey4me: will review them15:40
evrardjppabelanger: there are probably other patches, so let's just say we'll do better this time, and mark everything as ansible_update.15:41
evrardjpansible-update15:41
pabelangerwfm15:41
openstackgerritJesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-os_panko stable/queens: Replace virtualenv-tools with a script  https://review.openstack.org/56272015:42
evrardjpoh sorry we already had this one merged too: https://review.openstack.org/#/c/543576/15:42
*** armaan has joined #openstack-ansible15:42
evrardjpit  will stay ansible2.5 I guess15:42
evrardjpelse we'll lose track15:42
evrardjpgood luck anyway15:43
evrardjpif you need help, just shoot15:43
evrardjp:)15:43
*** mma has quit IRC15:43
odyssey4mealright everyone - I've had a long day so I'm going to go off early - have a good day/evening!15:44
evrardjpodyssey4me: really thank you for the work!15:45
evrardjptty tomorrow!15:45
*** cshen has quit IRC15:45
*** armaan has quit IRC15:47
spotzseeya odyssey4me15:48
*** armaan has joined #openstack-ansible15:48
xdfilwhat happens if I don't specify a repo_host?15:49
xdfilshould things still work?15:49
*** RandomTech has joined #openstack-ansible15:50
RandomTechHello everyone, does anyone happen to know were osa stores its playbooks run log?15:52
jrosserRandomTech: /openstack/log/ansible-logging/ansible.log on your deploy host15:53
RandomTechthank you15:54
RandomTechseems my setup openstack failed hm15:56
RandomTechill try rerunning it15:56
evrardjpxdfil: good question. It's not planned for now. but I'd be happy if someone could work on that.15:59
xdfilSo then... at this moment if I don't define one I shouldn't expect things to work?15:59
xdfilevrardjp: to your point though that does seem like something I'd work on. the repo container and I have never gotten along that well16:01
*** armaan has quit IRC16:02
mmercerdo any of the devs actually use centos 7 or is everyone running ubuntu for the aio and whatnot ?16:02
*** nicolasbock has joined #openstack-ansible16:02
evrardjpxdfil: odyssey4me is working on the simplification there. But our role tests are basically not using the repo server, so it's possible to do it. It will just be very slow.16:02
*** pcaruana has quit IRC16:03
evrardjp(because you remove the caching layers)16:03
xdfilI hate all the caching hehe16:04
evrardjpmmercer: most of the ppl are using ubuntu16:04
mmercerthats what i figured.16:04
mmercerguess ill do a ubuntu install, maybe ill finally be able to get this darned thing working16:04
evrardjpthen less people, but still some ppl, on suse and centos16:04
evrardjpmmercer: centos has the less love16:04
xdfilcaching  in regards to OSA has wasted far more of my time than it has saved :)16:04
evrardjpand is the slowest.16:04
evrardjpmmercer: darn thing?16:05
evrardjpit should work fine16:05
evrardjpofc master is a moving target16:05
evrardjpstable/ should work16:05
mmercerevrardjp: ive done 5 bare metal installs following the quickstart aio guide,  and it *does not* get past the pip part (tag 17.0.2)16:05
mmercerspecifically on setup-infrastructure.yml16:06
evrardjpI guess the issue you have is with pip10?16:06
mmercermmm, on centos 7, were running pip 7 or 816:06
evrardjpmmercer: so if you start clean, and then post a log of the issue, we can help16:06
mmercerit gets past modern pip in some of the cases, but then after modern pip, it fails to reach the repo container16:06
evrardjpnetwork issue then16:07
evrardjp?16:07
evrardjpplease paste a full log it's easier16:07
evrardjpthis way we can track things16:07
mmercerwill go dig it out in a few minutes16:07
evrardjpcool16:07
evrardjphwoarang: a few last things I should do before I leave for today?16:08
*** electrofelix has quit IRC16:08
*** radeks has quit IRC16:10
odyssey4mexdfil the package caching you can disable, if you have a pre-built repo you can use it too - otherwise you'll need the repo16:11
openstackgerritJean-Philippe Evrard proposed openstack/openstack-ansible-tests master: Do not log passwords  https://review.openstack.org/56273316:11
hwoarangevrardjp: not really16:11
evrardjphwoarang: cool16:11
evrardjpodyssey4me: could you quickly vote on https://review.openstack.org/#/c/562733/ ?16:11
evrardjpit's in the chain of patches for unbreaking translations16:11
evrardjpsame hwoarang  :)16:12
evrardjpthank you!16:12
evrardjpI am glad I broke things so well.16:12
evrardjpwith the do not log passwords.16:12
evrardjpit was elegant :)16:12
odyssey4meevrardjp :) now we try to merge all the backports of those patches before we merge the lint again16:13
odyssey4memaybe best to mark those lint changes -w for now16:14
*** vnogin has quit IRC16:14
evrardjpodyssey4me: fair :)16:14
evrardjpIt proves the usefulness of making it a lint test:D16:14
odyssey4mevery useful, and very good - great way to maintain a standard16:15
odyssey4mewe should do more of that16:15
evrardjpIf we do much more of that, we should refactor the job runner16:15
evrardjpbut that's fine for now16:15
evrardjpbasically we have one runner per test, which is expensive16:15
*** cshen has joined #openstack-ansible16:24
*** yifei has quit IRC16:25
*** yifei has joined #openstack-ansible16:26
*** chyka has joined #openstack-ansible16:27
openstackgerritMarkos Chandras (hwoarang) proposed openstack/openstack-ansible-os_keystone master: Avoid mixing pip and distribution packages  https://review.openstack.org/56260716:30
openstackgerritMarkos Chandras (hwoarang) proposed openstack/openstack-ansible-os_keystone master: [RFC] Add support for using distribution packages for OpenStack services  https://review.openstack.org/56030816:30
openstackgerritMerged openstack/openstack-ansible-plugins master: Remove python3 job  https://review.openstack.org/56266616:45
RandomTechhey when running the setup-openstack playbook i encountered this error: http://paste.openstack.org/show/719574/16:46
RandomTechdo you think i should rerun the first two playbooks?16:47
*** jiaopengju has quit IRC16:47
*** cshen has quit IRC16:47
*** Leo_m has joined #openstack-ansible16:49
RandomTechwait i think something really weird happened with the lxc containers its saying this container is on the wrong host possibly16:52
RandomTechevrardjp: if you have a moment does that error message seem odd to you as well?16:53
*** Leo_m has quit IRC16:54
*** Leo_m has joined #openstack-ansible17:02
openstackgerritMerged openstack/openstack-ansible-repo_server master: Set the pypiserver root to the correct directory  https://review.openstack.org/56254417:02
openstackgerritGerman Eichberger proposed openstack/openstack-ansible master: Adds the certificate client key password to user_secrets  https://review.openstack.org/55824417:09
*** gkadam has quit IRC17:09
openstackgerritJesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-repo_server stable/queens: Set the pypiserver root to the correct directory  https://review.openstack.org/56275917:11
openstackgerritMerged openstack/ansible-role-python_venv_build master: Only replace python when re-initializing the venv  https://review.openstack.org/56203317:13
*** yifei1 has joined #openstack-ansible17:17
*** yifei has quit IRC17:18
*** yifei1 is now known as yifei17:18
*** armaan has joined #openstack-ansible17:24
openstackgerritMerged openstack/openstack-ansible-os_tacker master: Only replace python when re-initializing the venv  https://review.openstack.org/56219917:24
*** udesale_ has quit IRC17:24
*** shardy has quit IRC17:35
openstackgerritMerged openstack/openstack-ansible-os_rally master: Only replace python when re-initializing the venv  https://review.openstack.org/56219117:35
openstackgerritMerged openstack/openstack-ansible-os_designate master: Only replace python when re-initializing the venv  https://review.openstack.org/56216817:42
openstackgerritJesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-os_designate stable/queens: Only replace python when re-initializing the venv  https://review.openstack.org/56276617:42
openstackgerritJesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-os_rally stable/queens: Only replace python when re-initializing the venv  https://review.openstack.org/56276717:43
openstackgerritJesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-os_tacker stable/queens: Only replace python when re-initializing the venv  https://review.openstack.org/56276917:43
*** hamza21 has joined #openstack-ansible17:44
*** chyka_ has joined #openstack-ansible17:47
openstackgerritMerged openstack/openstack-ansible-os_aodh master: Only replace python when re-initializing the venv  https://review.openstack.org/56215817:48
openstackgerritMerged openstack/openstack-ansible-os_gnocchi master: Only replace python when re-initializing the venv  https://review.openstack.org/56217117:49
*** chyka has quit IRC17:50
openstackgerritMerged openstack/openstack-ansible-os_glance master: Only replace python when re-initializing the venv  https://review.openstack.org/56215717:54
openstackgerritMerged openstack/openstack-ansible-os_heat master: Only replace python when re-initializing the venv  https://review.openstack.org/56217317:54
*** armaan has quit IRC17:56
openstackgerritMerged openstack/openstack-ansible-os_cinder master: Only replace python when re-initializing the venv  https://review.openstack.org/56216417:58
openstackgerritMerged openstack/openstack-ansible-os_sahara master: Only replace python when re-initializing the venv  https://review.openstack.org/56219518:01
*** poopcat has joined #openstack-ansible18:06
openstackgerritMerged openstack/openstack-ansible-os_barbican master: Only replace python when re-initializing the venv  https://review.openstack.org/56216118:12
*** pabelanger has quit IRC18:17
*** pabelanger has joined #openstack-ansible18:17
openstackgerritMerged openstack/openstack-ansible-os_swift master: Only replace python when re-initializing the venv  https://review.openstack.org/56219718:23
*** vakuznet has joined #openstack-ansible18:24
openstackgerritMerged openstack/openstack-ansible-os_neutron master: Only replace python when re-initializing the venv  https://review.openstack.org/56218318:36
*** TobbeCN has joined #openstack-ansible18:36
*** TobbeCN has quit IRC18:41
openstackgerritMarkos Chandras (hwoarang) proposed openstack/openstack-ansible-os_keystone master: Avoid mixing pip and distribution packages  https://review.openstack.org/56260718:47
openstackgerritMarkos Chandras (hwoarang) proposed openstack/openstack-ansible-os_keystone master: [RFC] Add support for using distribution packages for OpenStack services  https://review.openstack.org/56030818:47
openstackgerritJesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-os_gnocchi stable/queens: Only replace python when re-initializing the venv  https://review.openstack.org/56279118:50
openstackgerritJesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-os_glance stable/queens: Only replace python when re-initializing the venv  https://review.openstack.org/56279218:50
openstackgerritJesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-os_heat stable/queens: Only replace python when re-initializing the venv  https://review.openstack.org/56279418:51
openstackgerritJesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-os_cinder stable/queens: Only replace python when re-initializing the venv  https://review.openstack.org/56279518:51
openstackgerritJesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-os_sahara stable/queens: Only replace python when re-initializing the venv  https://review.openstack.org/56279618:51
openstackgerritJesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-os_barbican stable/queens: Only replace python when re-initializing the venv  https://review.openstack.org/56279718:51
openstackgerritJesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-os_swift stable/queens: Only replace python when re-initializing the venv  https://review.openstack.org/56279818:52
openstackgerritJesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-os_neutron stable/queens: Only replace python when re-initializing the venv  https://review.openstack.org/56279918:52
openstackgerritMarkos Chandras (hwoarang) proposed openstack/openstack-ansible-os_tempest master: Avoid mixing pip and distribution packages.  https://review.openstack.org/56260619:00
*** DanyC has joined #openstack-ansible19:00
*** DanyC has quit IRC19:05
*** sar has joined #openstack-ansible19:23
*** foutatoro has quit IRC19:33
openstackgerritMerged openstack/openstack-ansible-tests master: Do not log passwords  https://review.openstack.org/56273319:47
*** guilhermesp has joined #openstack-ansible19:51
guilhermesphey there! Does anyone knows the best way to apply a cherry pick in a openstack-ansible production environment? In my case, is just a bugfix in horizon https://review.openstack.org/#/c/510188/619:52
openstackgerritMarkos Chandras (hwoarang) proposed openstack/openstack-ansible-os_tempest master: Avoid mixing pip and distribution packages.  https://review.openstack.org/56260619:57
*** hamza21 has quit IRC20:01
openstackgerritMarkos Chandras (hwoarang) proposed openstack/openstack-ansible-os_keystone master: Avoid mixing pip and distribution packages  https://review.openstack.org/56260720:04
openstackgerritMarkos Chandras (hwoarang) proposed openstack/openstack-ansible-os_keystone master: [RFC] Add support for using distribution packages for OpenStack services  https://review.openstack.org/56030820:04
jrosserguilhermesp: i think you would make your own git repo, cherry pick the patch onto it, and then override these vars to suit https://github.com/openstack/openstack-ansible/blob/master/playbooks/defaults/repo_packages/openstack_services.yml#L95-L9820:04
*** RandomTech has quit IRC20:04
*** Adri2000 has joined #openstack-ansible20:06
guilhermespjrosser: i see.. is it possible to only change the commit in "horizon_git_install_branch" with the commit id (here's the commit id https://git.openstack.org/cgit/openstack/horizon/commit/?h=stable/pike&id=780961ac1c470d3997367e6c7d335b460e162286) and then run the os-horizon-install.yml OR are there  hidden steps that I'm not understanding clearly?20:11
*** tosky has quit IRC20:15
*** tosky has joined #openstack-ansible20:16
pabelangerevrardjp: good news, ansible 2.5.1 just worked with my playbooks / roles. That's the first time I've upgrades without having to change anything20:17
pabelangerofcourse I get some deprecated warnings now, but I can fix them20:17
spotzpabelanger: Always good news20:36
pabelangeryah, a first20:38
*** ansmith has quit IRC20:42
*** DanyC has joined #openstack-ansible20:48
*** sar has quit IRC20:49
*** DanyC_ has joined #openstack-ansible20:50
*** DanyC has quit IRC20:53
*** jwitko has quit IRC21:02
*** armaan has joined #openstack-ansible21:14
*** vnogin has joined #openstack-ansible21:15
*** kstev has quit IRC21:19
*** vnogin has quit IRC21:19
*** vakuznet has quit IRC21:20
*** gyee has joined #openstack-ansible21:22
gyeelbragstad, on the fernet token rotation feature, by looking at the code, looks like the rotation won't happen if the first node is not online for whatever reason. That right?21:24
lbragstadi was under the impression that it would21:24
gyeehow? as the cron job is schedule on the first node only21:25
* lbragstad goes to dig21:25
lbragstadgyee: oh - weird21:27
lbragstadhttps://github.com/openstack/openstack-ansible-os_keystone/blob/master/tasks/keystone_fernet_keys_autorotate.yml#L40-L4721:27
lbragstadmaybe i imagined that21:28
lbragstadgyee: yeah - you're right21:28
gyeethat make sure the cron job is on the first node21:28
gyeein a HA cluster, if the first node goes down, we will miss a rotation or two21:28
lbragstadsure21:29
lbragstaddo you have monitoring on your hosts?21:29
gyeeyes we do, maybe not that big of an issue21:29
gyeeanyway, just making sure I am not reading the code wrong :-)21:30
*** ansmith has joined #openstack-ansible21:30
lbragstadi guess if you can detect the first host goes down you could drop the auto rotation script on another host or resolve it later21:30
*** esberglu has quit IRC21:30
lbragstadgyee: yeah - no, you're right. sorry for the bum direction21:30
gyeeyeah, I have some logic to do these kind of dance for galera21:30
gyeemaybe we can do the same for fernet? it's much easier for fernet as the files are numbers21:31
gyeewe can find the one with the highest name and do the sync21:32
gyeehighest number21:32
lbragstadwell - all the keys should be consistent across the cluster, right?21:32
lbragstadso you could just put the auto rotation script on any host that is !down21:32
lbragstador the first host that passes a health check?21:33
gyeeI am worried about split brain situation21:34
gyeesay node #1 is down, while node #2 and #3 had rotated21:34
lbragstadoh - if the first node comes back up and assume responsibility for the key repository?21:34
gyeeright21:34
gyeeworst case is that the new key got sync'ed to node 2 and 3 again, which may invalid tokens issued by those nodes21:35
gyeesame file name, different key21:36
lbragstadyeah - maybe the process of recovering the node removes the autorotation bit from node 121:36
lbragstadand pull a fresh copy of the keys from 2 or 3 right before tying it back into the load balancer?21:36
gyeeright21:36
gyeelet me play around with it and see21:37
lbragstadcool - let me know what you find out21:37
*** DanyC_ has quit IRC21:50
*** tomtom001 has quit IRC21:52
*** tomtom001 has joined #openstack-ansible21:52
evrardjpguilhermesp: https://docs.openstack.org/openstack-ansible/latest/user/source-overrides/index.html21:53
evrardjpparticularily https://docs.openstack.org/openstack-ansible/latest/user/source-overrides/index.html#overriding-other-upstream-projects-source-code21:53
xdfilpip_configure: false21:53
evrardjpit's in the docs...21:53
evrardjppabelanger: generally following the migration guide is enough :) Except if you have weird fancy things :p21:54
evrardjpglad it was fine21:54
evrardjphas anyone tried a master build recently?21:55
xdfilI can't imagine the total amount of time I have spent chasing my tail through defaults/vars/group_vars trying to stop things from happening with pip21:55
xdfilI just found this lol21:55
xdfilpip_configure: false21:55
xdfilsay it with me now kids!21:55
openstackgerritPaul Belanger proposed openstack/ansible-hardening master: Switch from debian-jessie to debian-stable (stretch)  https://review.openstack.org/56284221:55
openstackgerritPaul Belanger proposed openstack/ansible-hardening master: Switch from debian-jessie to debian-stable (stretch)  https://review.openstack.org/56284222:12
*** cjloader_ has quit IRC22:20
openstackgerritJean-Philippe Evrard (evrardjp) proposed openstack/openstack-ansible-tests stable/queens: Do not log passwords  https://review.openstack.org/56285322:22
openstackgerritJean-Philippe Evrard (evrardjp) proposed openstack/openstack-ansible-tests stable/pike: Do not log passwords  https://review.openstack.org/56285422:22
openstackgerritJean-Philippe Evrard (evrardjp) proposed openstack/openstack-ansible-plugins stable/queens: Remove python3 job  https://review.openstack.org/56285522:23
openstackgerritJean-Philippe Evrard (evrardjp) proposed openstack/ansible-config_template master: Temporarily disable python3 job  https://review.openstack.org/56286022:33
*** chyka has joined #openstack-ansible22:34
*** chyka_ has quit IRC22:37
*** TobbeCN has joined #openstack-ansible22:37
*** TobbeCN has quit IRC22:41
*** armaan has quit IRC22:50
*** armaan has joined #openstack-ansible22:50
*** tosky has quit IRC23:10
*** chyka_ has joined #openstack-ansible23:15
openstackgerritMerged openstack/ansible-config_template master: Temporarily disable python3 job  https://review.openstack.org/56286023:17
*** chyka has quit IRC23:19
*** chyka_ has quit IRC23:20
*** cmart has quit IRC23:21
*** chyka has joined #openstack-ansible23:41
*** chyka has quit IRC23:46

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