Thursday, 2018-11-01

*** markvoelker has quit IRC00:01
*** DanyC has quit IRC00:11
*** mmercer has quit IRC00:41
*** cshen has joined #openstack-ansible00:50
*** aredinger has joined #openstack-ansible00:54
*** cshen has quit IRC00:54
*** aredinger has quit IRC00:59
*** ThiagoCMC has quit IRC01:04
*** zul has quit IRC01:04
*** aredinger has joined #openstack-ansible01:54
*** aredinger has quit IRC01:59
*** markvoelker has joined #openstack-ansible02:03
*** DanyC has joined #openstack-ansible02:12
*** DanyC has quit IRC02:16
*** markvoelker has quit IRC02:35
*** Leo_m has quit IRC02:45
*** Leo_m has joined #openstack-ansible02:50
*** cshen has joined #openstack-ansible02:51
*** aredinger has joined #openstack-ansible02:54
*** cshen has quit IRC02:55
*** aredinger has quit IRC02:59
*** markvoelker has joined #openstack-ansible03:32
*** udesale has joined #openstack-ansible03:50
*** aredinger has joined #openstack-ansible03:54
*** aredinger has quit IRC03:59
*** markvoelker has quit IRC04:06
*** cshen has joined #openstack-ansible04:51
*** aredinger has joined #openstack-ansible04:54
*** cshen has quit IRC04:56
*** aredinger has quit IRC04:59
*** markvoelker has joined #openstack-ansible05:02
*** markvoelker has quit IRC05:36
*** aredinger has joined #openstack-ansible05:44
*** aredinger has quit IRC05:48
*** aredinger has joined #openstack-ansible05:59
*** aredinger has quit IRC06:04
*** faizy98 has joined #openstack-ansible06:13
*** cshen has joined #openstack-ansible06:15
*** cshen has quit IRC06:21
*** chkumar|off is now known as chandankumar06:21
*** markvoelker has joined #openstack-ansible06:33
*** cshen has joined #openstack-ansible06:43
*** aredinger has joined #openstack-ansible06:54
*** gkadam has joined #openstack-ansible06:58
*** aredinger has quit IRC06:58
*** markvoelker has quit IRC07:05
*** DanyC has joined #openstack-ansible07:08
*** cshen has quit IRC07:08
*** cshen has joined #openstack-ansible07:14
*** cshen has quit IRC07:19
*** DanyC has quit IRC07:33
*** cshen has joined #openstack-ansible07:34
openstackgerritMerged openstack/ansible-role-python_venv_build master: Add default distro packages for wheel builds  https://review.openstack.org/61358507:34
*** cshen has quit IRC07:39
*** pcaruana|elisa| has joined #openstack-ansible07:40
*** hamzaachi has joined #openstack-ansible07:53
*** aredinger has joined #openstack-ansible07:54
*** aredinger has quit IRC07:59
*** pcaruana|elisa| has quit IRC07:59
*** udesale has quit IRC08:02
*** markvoelker has joined #openstack-ansible08:03
*** pcaruana has joined #openstack-ansible08:05
*** cshen has joined #openstack-ansible08:06
*** shardy has joined #openstack-ansible08:08
*** shardy_ has joined #openstack-ansible08:08
*** shardy_ has quit IRC08:10
*** shardy_ has joined #openstack-ansible08:11
*** markvoelker has quit IRC08:36
*** gokhani has joined #openstack-ansible08:36
*** cshen has quit IRC08:38
*** maddtux has joined #openstack-ansible08:47
*** cshen has joined #openstack-ansible08:48
*** fghaas has joined #openstack-ansible09:14
*** udesale has joined #openstack-ansible09:16
*** DanyC has joined #openstack-ansible09:19
*** DanyC has quit IRC09:23
*** DanyC has joined #openstack-ansible09:23
*** fghaas has quit IRC09:23
openstackgerritMichael Vollman proposed openstack/openstack-ansible-ops master: Fix ansible path for elk_metrics_6x tests  https://review.openstack.org/61421009:25
*** electrofelix has joined #openstack-ansible09:32
*** markvoelker has joined #openstack-ansible09:33
*** spatel has joined #openstack-ansible09:56
*** spatel has quit IRC10:00
*** markvoelker has quit IRC10:07
*** chandankumar has quit IRC10:09
jrosserhwoarang: could you take a look at https://review.openstack.org/#/c/613375/ and https://review.openstack.org/#/c/613428/ ?10:11
hwoaranghmm10:13
*** hamzaachi has quit IRC10:14
hwoaranglooks like both roles need a bit of clean up10:14
hwoarangbut i can do it after this patch10:14
hwoaranglots of -dev packages are being pulled in by default :/10:14
jrosserdon't worry about that yet10:15
jrosserthose patches are in the context of this https://review.openstack.org/#/c/613078/10:15
hwoarangok10:15
openstackgerritMerged openstack/openstack-ansible stable/rocky: Update rocky index with release date  https://review.openstack.org/61448310:16
*** evrardjp has joined #openstack-ansible10:17
*** chandankumar has joined #openstack-ansible10:31
jrosserhwoarang: fyi theres a cleanup of -dev packages here https://review.openstack.org/#/q/topic:reduce-unnecessary-package-installs+(status:open+OR+status:merged)10:37
hwoarangyeah i think there is a bit more than that. the $service_distro_packages should not contain any devel stuff10:38
*** ansmith has joined #openstack-ansible10:42
*** dave-mccowan has joined #openstack-ansible10:45
*** evrardjp_ has joined #openstack-ansible10:50
*** ansmith has quit IRC10:51
*** evrardjp has quit IRC10:53
openstackgerritMarkos Chandras (hwoarang) proposed openstack/openstack-ansible-tests master: [DNM] - Testing debug log collection  https://review.openstack.org/58164210:57
*** faizy98 has quit IRC10:57
odyssey4mehwoarang yeah, I started working through the roles where I saw devel packages in the lists, but I've not got through them all yet10:57
hwoarangyeah no problem i can help with that10:58
*** faizy98 has joined #openstack-ansible10:58
odyssey4meawesome, thanks :)10:58
*** hamzaachi has joined #openstack-ansible11:00
*** faizy98 has quit IRC11:02
*** faizy98 has joined #openstack-ansible11:03
odyssey4mecores, can we get https://review.openstack.org/614480 merged asap please11:03
*** markvoelker has joined #openstack-ansible11:04
*** pcaruana has quit IRC11:05
openstackgerritMerged openstack/openstack-ansible-os_cinder master: Add libpython2.7 as a required package.  https://review.openstack.org/61337511:10
hwoarangdone11:17
hwoarangodyssey4me: could you get https://review.openstack.org/#/c/614247/ in so i can make some progress on ceph+leap15? thanks!11:17
openstackgerritMerged openstack/openstack-ansible-os_tempest master: Correct Ubuntu package name  https://review.openstack.org/61451511:18
*** udesale has quit IRC11:24
openstackgerritMerged openstack/openstack-ansible-ceph_client master: SUSE: Add support for openSUSE Leap 15  https://review.openstack.org/61424711:31
openstackgerritMichael Vollman proposed openstack/openstack-ansible-ops master: Fix ansible path for elk_metrics_6x tests  https://review.openstack.org/61421011:32
*** markvoelker has quit IRC11:36
openstackgerritJesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-tests stable/rocky: rdo: use rocky mirrors instead of master  https://review.openstack.org/61378711:39
odyssey4memnaser ^ I dunno if that's what you really want - but that seems to exist as a path. It also exists for master, so perhaps that's a more suitable one to use generally?11:43
*** gkadam has quit IRC11:46
*** gkadam has joined #openstack-ansible11:50
*** cshen has quit IRC11:50
*** cshen has joined #openstack-ansible11:52
*** pcaruana has joined #openstack-ansible11:52
*** hamzaachi has quit IRC11:54
*** hamzaachi has joined #openstack-ansible11:55
*** gkadam_ has joined #openstack-ansible12:04
openstackgerritMerged openstack/openstack-ansible master: Restore bionic/ceph voting jobs  https://review.openstack.org/60996512:05
openstackgerritMerged openstack/openstack-ansible master: Update master index with current series status  https://review.openstack.org/61448012:05
*** gkadam has quit IRC12:07
*** vollman has joined #openstack-ansible12:11
openstackgerritMichael Vollman proposed openstack/openstack-ansible-ops master: Fix ansible path for elk_metrics_6x tests  https://review.openstack.org/61421012:11
openstackgerritMarkos Chandras (hwoarang) proposed openstack/openstack-ansible-tests master: test-log-collect.sh: Verify journal consinstency before gathering logs  https://review.openstack.org/61474812:14
openstackgerritMerged openstack/openstack-ansible-os_keystone master: Remove unnecessary package install duplication  https://review.openstack.org/61433912:21
mnaserodyssey4me: yes, sorry, that is the one to go for12:22
odyssey4memnaser alright, should we use that one for master too - that way we won't have to remember to change it for every stable branch creation... I can do a patch to make it do the branch name thing easily enough12:23
mnaserodyssey4me: we will have to do it for each branch, i actually bumped it in openstack_hosts but this is another openstack-ansible-tests biting us in the ass thing12:23
odyssey4memnaser you're talking about these? https://github.com/openstack/openstack-ansible-openstack_hosts/blob/stable/rocky/defaults/main.yml#L139-L14212:25
mnaserodyssey4me: yeah, i fixed those to point to the right path but openstack-ansible-tests was forgotten12:27
odyssey4meit looks like they're both setting the same thing though - why not just remove those overrides from tests?12:28
odyssey4memnaser hmm, not quite the same thing - what would the content of rdo_dlrn be in https://github.com/openstack/openstack-ansible-tests/blob/master/common-tasks/test-set-nodepool-vars.yml#L54 ?12:28
mnaserodyssey4me: isnt the override there to use mirrors in our CI?12:29
mnaseroh i see12:29
mnaseryou mean we just override openstack_hosts_rdo_mirror_url only12:29
mnaserok so the reason is because12:29
mnaserhttps://trunk.rdoproject.org/centos7-rocky/current-passed-ci/delorean.repo12:30
odyssey4memnaser yep it is, but I'm not seeing how it sets an override - it looks like it's using the same values, unless I'm missing something12:30
mnaserwe pull down this file12:30
mnaserand take the baseurl12:30
mnaserrdo_dlrn=`curl --silent ${NODEPOOL_RDO_PROXY}/${DLRN_BASE}/delorean.repo | grep baseurl | cut -d= -f2`12:30
mnaserRDO_MIRROR_HOST=${rdo_dlrn/https:\/\/trunk.rdoproject.org/$NODEPOOL_RDO_PROXY}12:30
mnaseraka replace trunk.rdoproject.org from the baseurl12:30
mnaseri.e. from here https://trunk.rdoproject.org/centos7-rocky/current-passed-ci/delorean.repo12:30
odyssey4meah ok, so we are actually implementing an override12:31
*** markvoelker has joined #openstack-ansible12:31
odyssey4mecould we not just override https://github.com/openstack/openstack-ansible-openstack_hosts/blob/stable/rocky/defaults/main.yml#L140 ?12:31
*** udesale has joined #openstack-ansible12:32
vollmanmnaser: good morning.  I was hoping you could help with manila role next steps.  I have a few changes up.  I was thinking this first?  https://review.openstack.org/#/c/612055/12:34
mnaserodyssey4me: unfortunately not in ci, because we use caching, that url changes all the time12:40
mnaserso you end up with "missing package" stuff crashing the gate12:40
mnaserthats why we have to do the dance to download the .repo file so we get a hashed url (and therefore kinda doing natural cache invalidation)12:40
mnaservollman: morning!  checking12:40
openstackgerritMichael Vollman proposed openstack/openstack-ansible-ops master: Fix ansible path for elk_metrics_6x tests  https://review.openstack.org/61421012:40
mnaservollman: do you have a depends-on patch on that one that shows the role being tested?12:41
vollmanmnaser: The tests change should depend on this? https://review.openstack.org/#/c/611930/12:43
*** ansmith has joined #openstack-ansible12:48
odyssey4memnaser OK, and yet we're expecting users to just use https://trunk.rdoproject.org/centos7-rocky/current-passed-ci/ straight up? I'm trying to understand the purpose of using that ever-changing URL for tests, but not using it in the real world.12:48
mnaserodyssey4me: users can consume current-passed-ci with no problems.  the issue we have in upstream is because: we use the openstack-infra cache, so the xml file for the repo might be cached, therefore if we run CI jobs during a time where RDO is promoting packages12:50
mnaserwe end up in a state where yum is like "i cant find these packages in my mirror"12:50
mnaserbecause12:50
mnaser" https://trunk.rdoproject.org/centos7-rocky/current-passed-ci/" => path doesnt change, infra mirror is not smart enough to know that the repo changed12:50
mnaserusers however will not have this because they will always get fresh content (unless they're jrosser behind an http proxy :P)12:51
mnaseri'm sorry if i'm not doing a good job at explaining myself :X12:51
odyssey4memnaser aha, I see - ok, makes more sense now - thanks!12:51
jrossernot sure how a proxy means non fresh content.....12:52
jrosserproxy != mirror12:52
mnaserinfra doesnt mirror things for rdo12:52
mnaserthey cache them12:52
odyssey4meWell, the proxies in infra are reverse proxies and I think they have retention turned way up.12:52
mnaseryeah12:52
mnaserprobably incorrectly too12:52
openstackgerritArx Cruz proposed openstack/openstack-ansible-os_tempest master: WIP - Enable python-tempestconf support  https://review.openstack.org/61296812:52
jrosserand also reverse proxy != forward proxy, theres overloaded terms there12:53
*** hamzaachi has quit IRC12:53
d34dh0r53shameless plug: https://review.openstack.org/#/c/614626/12:53
odyssey4med34dh0r53 I'm a bit confused about two change ID's there - what's up with that?12:54
d34dh0r53hmm, yeah I'm not sure, that is really odd12:55
d34dh0r53I squashed it into one commit, I wonder if there was a marker in there that confused git review12:55
*** udesale has quit IRC12:56
d34dh0r53I can amend it, but I'm not sure which change-id to use12:56
*** strattao has joined #openstack-ansible12:56
odyssey4med34dh0r53 if you amend it and git review again it may create a duplicate, but that doesn't matter12:57
odyssey4mebut do it from your client, not through gerrit12:57
d34dh0r53the first change id doesn't link to anything to I'll try removing that one12:57
*** gkadam_ has quit IRC12:57
openstackgerritDavid Wilde proposed openstack/openstack-ansible-ops master: MNAIO: flat networking fixes  https://review.openstack.org/61462612:58
odyssey4med34dh0r53 so the flat and lbaas networks weren't being created at all inside the VM's?12:58
odyssey4meI was wondering why this wasn't affecting xenial - then I remembered that the implementation there is not using networkd in the VM's.12:59
d34dh0r53odyssey4me: the br-flat and br-lbaas networks were being created, but both need veth pairs created to do things with that's what this does12:59
openstackgerritMichael Vollman proposed openstack/openstack-ansible-ops master: Fix ansible path for elk_metrics_6x tests  https://review.openstack.org/61421012:59
d34dh0r53odyssey4me: yeah, these veth pairs are being created correctly on xenial13:00
odyssey4meLGTM, and given johnsom reviewed too +w13:00
d34dh0r53odyssey4me: tys!13:01
*** udesale has joined #openstack-ansible13:01
*** cshen has quit IRC13:01
*** rgogunskiy has quit IRC13:07
openstackgerritMerged openstack/openstack-ansible-tests stable/rocky: rdo: use rocky mirrors instead of master  https://review.openstack.org/61378713:07
*** rgogunskiy has joined #openstack-ansible13:16
*** cshen has joined #openstack-ansible13:24
*** cshen has quit IRC13:29
*** maddtux has quit IRC13:32
*** ThiagoCMC has joined #openstack-ansible13:34
openstackgerritMerged openstack/openstack-ansible-ops master: MNAIO: flat networking fixes  https://review.openstack.org/61462613:38
*** aredinger has joined #openstack-ansible13:38
*** cshen has joined #openstack-ansible13:40
*** Leo_m has quit IRC13:44
*** cshen has quit IRC13:48
eumel8o/13:55
eumel8it seems our tox patch needs still some advertising here: https://review.openstack.org/#/c/612510/ jrosser, mnaser, cloudnull, evrardjp, spotz - THX!13:58
noonedeadpunkI just get masakari working, so I'll try to re-write my patch for masakari monitors within several days14:00
odyssey4mejrosser is https://review.openstack.org/#/c/611521/ going to be ported back at all?14:06
jrosseryeah, we should do that14:07
jrosseri wonder how far back is appropriate?14:07
jrosserhmm it's not going to apply directly to queens14:09
odyssey4meI'd say rocky at least, propose it for whichever branches matter to you - anyone else who wants it further back can propose the backport themselves.14:09
openstackgerritJonathan Rosser proposed openstack/openstack-ansible stable/rocky: Ensure ceph server apt pinning is well defined  https://review.openstack.org/61477714:09
jrosserodyssey4me: i'm also suspicious that our var that sets the ceph version actually does that in some situations14:13
*** gkadam has joined #openstack-ansible14:13
*** aludwar has quit IRC14:15
noonedeadpunkcores, can you kindly check https://review.openstack.org/#/c/613977/ and https://review.openstack.org/#/c/614341/ ?14:15
odyssey4mejrosser hwoarang ok so based on https://review.openstack.org/#/c/614342/1/vars/suse.yml should I remove liberasurecode1 ?14:15
odyssey4meshould that package be installed on the target hosts at run time?14:16
jrosserif it is required as a runtime .so then yes14:16
odyssey4meie liberasurecode-devel for swift_devel_distro_packages, and liberasurecode1 in swift_distro_packages14:17
jrosserthat would be the lowest risk thing for now, if we are going to take a second pass across all the roles weeding out any unneeded run time libs14:18
*** aludwar has joined #openstack-ansible14:22
jungleboyjodyssey4me: Can I pester you for a minute?14:23
odyssey4mejungleboyj sure, what's up?14:24
openstackgerritJesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-os_swift master: Remove unnecessary package install duplication  https://review.openstack.org/61434214:24
jungleboyjodyssey4me:  Ended up having to rebuild my env after I got some larger disks.14:24
jungleboyjTrying to deploy openstack-ansible again.14:24
jungleboyjKeeps failing here:14:25
jungleboyjchroot /var/lib/machines/ubuntu-xenial-amd64 /opt/cache-prep-commands.sh > /var/log/lxc-cache-prep-commands.log14:25
odyssey4mejungleboyj and what does it show in /var/log/lxc-cache-prep-commands.log14:26
jungleboyjThere is no log file.  I can run the command by hand it is successful but prompts me about whether it should replace the existing packages.14:26
jrossertheres a patch for this14:28
jrosseri think it is this one https://review.openstack.org/#/c/613407/14:29
openstackgerritJesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-os_swift master: Remove unnecessary package install duplication  https://review.openstack.org/61434214:29
odyssey4mejungleboyj yeah, you'll need to use stable/rocky and not the last tag14:30
odyssey4meassuming you're using rocky14:30
jungleboyjodyssey4me:  I went with the 18.0.0 tag I thought.14:30
*** Leo_m has joined #openstack-ansible14:31
odyssey4mejungleboyj 18.0.0 has some bugs we've got fixed up at the head of stable/rocky - that's one of them14:31
*** Leo_m has quit IRC14:31
*** Leo_m has joined #openstack-ansible14:32
jungleboyjOk.  Cool.  Just swtiched and trying again.14:32
jungleboyjThank you!14:33
odyssey4mejungleboyj don't forget to run the ansible bootstrap after switching your branch/SHA.14:33
jungleboyjOh, I need to rerun that?14:33
*** cshen has joined #openstack-ansible14:33
odyssey4meyep, every time you change it up you'll need to do that so that all the roles get updated14:33
openstackgerritMerged openstack/openstack-ansible-os_heat stable/rocky: Set container_name as host_var in test inventory  https://review.openstack.org/61292914:34
odyssey4meyou're effectively doing a minor upgrade, so https://docs.openstack.org/openstack-ansible/rocky/admin/upgrades/minor-updates.html applies14:34
jungleboyjOk cool.  That explains some of the other issues I was having earlier too.14:35
openstackgerritMichael Vollman proposed openstack/openstack-ansible-ops master: Fix ansible path for elk_metrics_6x tests  https://review.openstack.org/61421014:35
jungleboyjI will be an OSA expert eventually.   :-)14:35
*** ThiagoCMC has quit IRC14:36
*** aludwar has quit IRC14:37
*** aludwar has joined #openstack-ansible14:37
odyssey4mehwoarang could you review https://review.openstack.org/#/c/614436/ please?14:40
openstackgerritMerged openstack/openstack-ansible-os_barbican stable/rocky: Set container_name as host_var in test inventory  https://review.openstack.org/61291814:42
openstackgerritMerged openstack/openstack-ansible-os_keystone stable/rocky: Set container_name as host_var in test inventory  https://review.openstack.org/61292514:44
spotzOk back from rebooting land if needed14:44
odyssey4mejrosser are you happy with https://review.openstack.org/614194 ?14:45
jrosserin principle, but i know very little of swift API to say that the AUTH url is right/wrong14:46
odyssey4methe auth url does now match what the swift role does14:47
odyssey4meand if doing that and those other changes makes rgw more like swift, then I think that's a good thing14:47
jrossernow, the patch to the docs there shows enabling both s3 and swift14:50
odyssey4meyep, nice work there :)14:51
openstackgerritMerged openstack/openstack-ansible-os_horizon stable/rocky: Set container_name as host_var in test inventory  https://review.openstack.org/61292114:51
jrosserand whilst nothing stops you doing that on one rgw instance, i've never managed to get refstack to pass like that14:51
jrosserbecasue in order to do that, both api need to serve from / or one of the refstack tests fails14:51
jrosserwhich obv you can't do with one rgw process14:52
odyssey4meperhaps that's worth noting in review and perhaps it should be added to the docs as a notice/warning?14:52
jrosseri will get stuartgr to leave a comment there - he dug into all that14:54
openstackgerritJesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-os_keystone stable/rocky: Add memcache flushing handler on db migrations  https://review.openstack.org/61332114:54
openstackgerritArx Cruz proposed openstack/openstack-ansible-os_tempest master: WIP - Enable python-tempestconf support  https://review.openstack.org/61296814:55
openstackgerritMerged openstack/openstack-ansible-os_cinder stable/rocky: Set container_name as host_var in test inventory  https://review.openstack.org/61291514:59
openstackgerritMerged openstack/openstack-ansible-os_designate stable/rocky: Set container_name as host_var in test inventory  https://review.openstack.org/61292015:03
jrosserodyssey4me: do you know if non-rgw swift serves at / or /swift ?15:05
*** rgogunskiy has quit IRC15:05
*** gkadam has quit IRC15:09
*** kukacz has quit IRC15:12
odyssey4mejrosser I have no idea.15:12
jrosserlooking at it this would suggest it's at / https://github.com/openstack/openstack-ansible-os_swift/blob/master/defaults/main.yml#L16415:13
jrosserit's interesting15:13
jrosserwithout those AUTH_.... S3 and swift collided on / when both enabled, so we'd moved swift to /swift, and that broke refstack15:14
jrosserit might now be what with those changes the two protocols can co-exist at the same root, but that needs testing15:14
openstackgerritMerged openstack/openstack-ansible-os_nova master: Add libpython2.7 as a required package  https://review.openstack.org/61342815:19
*** kukacz has joined #openstack-ansible15:19
*** fghaas has joined #openstack-ansible15:20
openstackgerritJonathan Rosser proposed openstack/openstack-ansible-lxc_hosts master: Minimise distro packages installed into the lxc image  https://review.openstack.org/61307815:26
openstackgerritFlorian Haas proposed openstack/openstack-ansible master: Include Swift AUTH_%(tenant_id)s suffix in rgw Keystone endpoint  https://review.openstack.org/61419415:27
*** gyee has joined #openstack-ansible15:33
*** strattao has quit IRC15:53
*** hamzaachi has joined #openstack-ansible15:56
openstackgerritMerged openstack/openstack-ansible-os_nova stable/rocky: Set container_name as host_var in test inventory  https://review.openstack.org/61291316:10
*** chhagarw has joined #openstack-ansible16:12
openstackgerritMerged openstack/openstack-ansible-os_tempest master: Fixed missing ; from if statement in tempest_run  https://review.openstack.org/61452116:19
*** cshen has quit IRC16:25
*** aredinger has quit IRC16:30
openstackgerritMerged openstack/openstack-ansible master: Add new tox doc and publishing job  https://review.openstack.org/61251016:33
openstackgerritMerged openstack/openstack-ansible stable/queens: Update all SHAs for 17.1.4  https://review.openstack.org/61374216:33
*** cshen has joined #openstack-ansible16:41
*** cshen has quit IRC16:47
*** cshen has joined #openstack-ansible16:51
*** strattao has joined #openstack-ansible16:53
*** aredinger has joined #openstack-ansible16:54
*** cshen has quit IRC16:56
*** aredinger has quit IRC16:58
*** hamzaachi has quit IRC17:03
*** udesale has quit IRC17:06
*** aredinger has joined #openstack-ansible17:11
openstackgerritMerged openstack/openstack-ansible-openstack_hosts master: Increase inotify watch limit  https://review.openstack.org/61443617:14
*** aredinger has quit IRC17:16
*** fghaas has quit IRC17:18
*** fghaas has joined #openstack-ansible17:19
*** fghaas has quit IRC17:19
*** hamzaachi has joined #openstack-ansible17:20
*** mmercer has joined #openstack-ansible17:37
*** spatel_ has joined #openstack-ansible17:43
spatel_cloudnull: ^^17:44
spatel_or anyone out there...17:44
-spatel_- ### Run openstack playbooks17:44
-spatel_- openstack-ansible setup-hosts.yml --tags openstack_hosts-config --limit $117:44
-spatel_- openstack-ansible setup-hosts.yml --limit $1 --skip-tags openstack_hosts-config17:44
-spatel_- openstack-ansible setup-openstack.yml --skip-tags nova-key-distribute --limit $117:44
-spatel_- openstack-ansible setup-openstack.yml --tags nova-key --skip-tags ceph --limit compute_hosts17:44
spatel_I am running these playbooks to add 1 compute node but its taking 1 hour to add single compute node17:45
spatel_i have noticed ansible scanning every single compute machine in cluster going through each and everyone...17:45
spatel_i have 100 computes running in production17:45
spatel_is that normal?17:46
spatel_what else i can do to make it faster?17:46
*** cshen has joined #openstack-ansible17:48
*** aredinger has joined #openstack-ansible17:54
openstackgerritMerged openstack/openstack-ansible-plugins master: Allow basic auth in netloc_no_port filtered URLs  https://review.openstack.org/61384217:55
*** DanyC has quit IRC17:56
*** aredinger has quit IRC17:58
*** DanyC has joined #openstack-ansible18:03
*** hamzaachi_ has joined #openstack-ansible18:05
*** DanyC has quit IRC18:07
*** hamzaachi has quit IRC18:08
*** aredinger has joined #openstack-ansible18:17
spatel_look like folks are busy somewhere else..18:20
spatel_i can wait18:20
*** ThiagoCMC has joined #openstack-ansible18:40
mnaserspatel_: have you tried fixing it somehow?18:49
jrosserI would say that in this situation ARA Would be very helpful18:51
*** chhagarw has quit IRC18:51
spatel_mnaser: fixing what?18:52
mnasertrying to identify what is actually causing the slowdown18:52
jungleboyjspotz:  Have you seen any issues with stable/rocky with being able to get the instance console through horizon?18:54
jungleboyjWhen I try it tells me the address may have changed or something like that.18:55
jungleboyjAnd the plugin crashes.18:55
spotzjungleboyj: I haven't tried to be honest. Is this the one where you made an image and moved it?18:55
jungleboyjspotz:  What do you mean made an image and moved it?18:56
spotzjungleboyj: I thought you'd made a snapshot and then put it up again. Maybe you just snapshotted18:56
jungleboyjNo, I rebuilt from scratch.18:57
spotzAhh ok18:58
odyssey4mespotz I wonder if that has something to do with https://github.com/openstack/openstack-ansible-os_horizon/commit/6903802129a8e791ebaed4e60f95458b99201a25 ?18:59
jungleboyjHmmm, that looks a little suspicious.  :-)19:00
spotzodyssey4me: Hrm, it's possible. I don't have a rocky build anywhere:(19:00
spotzDepends on what vnc is looking for an if jungleboyj turned off ssl which I didn't think he did19:01
jungleboyjI did not turn off ssl.  The VNC connection is https.  I think priviously though I wasn't using https because I don't remember getting the security warning.19:02
jungleboyjI gotta run for a bit.  Ping me if there is something you want me to try.19:04
jungleboyjI can give it a shot later.19:04
*** pcaruana has quit IRC19:05
*** spatel_ has quit IRC19:11
*** spatel has joined #openstack-ansible19:11
*** aredinger has quit IRC19:13
spatelmnaser: i am not seeing anything slow but all i can see that playbook going through all my compute nodes and that taking long time19:14
mnaserjungleboyj: which console do you use?19:24
jungleboyjThe default.  Think it was spce.19:25
jungleboyj*spice19:25
*** hamzaachi_ has quit IRC19:26
*** ThiagoCMC has quit IRC19:27
*** electrofelix has quit IRC19:28
*** hamzaachi has joined #openstack-ansible19:28
openstackgerritJimmy McCrory proposed openstack/openstack-ansible-plugins stable/rocky: Allow basic auth in netloc_no_port filtered URLs  https://review.openstack.org/61484019:39
openstackgerritJimmy McCrory proposed openstack/openstack-ansible-plugins stable/queens: Allow basic auth in netloc_no_port filtered URLs  https://review.openstack.org/61484119:41
*** fghaas has joined #openstack-ansible19:46
openstackgerritMerged openstack/openstack-ansible master: Docs: Fix VLAN bridge interface in example setup  https://review.openstack.org/61437919:53
spatelmnaser: how much time it takes in your case?19:54
*** aredinger has joined #openstack-ansible19:54
spatelalso i found block migration is broken on queen release :(19:55
spatelit say successfully migrated block disk but i found it truncated disk on destination and VM failed to boot19:55
*** DanyC has joined #openstack-ansible19:55
*** cshen has quit IRC19:57
*** aredinger has quit IRC19:58
*** DanyC has quit IRC20:00
*** hamzaachi has quit IRC20:04
eumel8gotcha: https://docs.openstack.org/openstack-ansible/latest/de/20:06
jrosserspatel: did you raise a bug for that?20:07
*** cshen has joined #openstack-ansible20:09
odyssey4meeumel8 woot!20:11
*** spatel has quit IRC20:12
*** spatel has joined #openstack-ansible20:14
openstackgerritMerged openstack/openstack-ansible-os_ceilometer master: Remove unnecessary package install duplication  https://review.openstack.org/61433720:16
openstackgerritMerged openstack/openstack-ansible-os_glance stable/rocky: Set container_name as host_var in test inventory  https://review.openstack.org/61292320:22
openstackgerritMerged openstack/openstack-ansible-os_cinder master: Remove unnecessary package install duplication  https://review.openstack.org/61433420:31
*** ansmith has quit IRC20:31
*** dave-mccowan has quit IRC20:32
*** DanyC has joined #openstack-ansible20:36
*** spatel has quit IRC20:40
*** evrardjp_ is now known as evrardjp20:42
mnaserodyssey4me: https://github.com/ansible/ansible/pull/47940 we did an oops20:44
odyssey4memnaser whoops :/20:45
mnaserall good20:45
mnaseralso this is a simple one that'll help our local tonyb https://github.com/ansible/ansible/pull/3739020:45
*** aredinger has joined #openstack-ansible20:54
*** aredinger has quit IRC20:59
odyssey4memnaser I'll have to review that in the morning. My brain is not ready for a real review right now. :)21:02
mnaserodyssey4me: good call21:02
mnaserits late, grab some sleep21:02
mnaserhave a good night21:02
jrosseranyone see any issues with scoping of l3_agent_plugins here?https://github.com/openstack/openstack-ansible-os_neutron/blob/master/templates/l3_agent.ini.j221:03
mnaseryes21:04
mnaseri got bit by that21:04
mnaserreally badly21:04
jrosserspecifically i get the [fwaas] section present but [agent] missing21:04
mnaseri didnt get a chance to fix it jrosser :(21:04
mnaserexactly same thing i ran into21:04
jrosserahha21:04
mnaserjrosser: do you have time to take a look at it21:05
mnaser?21:05
jrosseri am looking now (also late...)21:05
jrosserbut i'm not sure what is up, the set _ = pattern looks reasonable21:06
jrosserunless i'm missing something21:06
jamesdentonmaybe >= 121:11
jamesdentonjrosser ^^21:12
jrosserjamesdenton: omg :)21:13
*** strattao has quit IRC21:13
openstackgerritJimmy McCrory proposed openstack/openstack-ansible-os_nova master: Fix issues with nova distro install on Ubuntu  https://review.openstack.org/61437221:21
mnaserim an idiot21:21
mnaserjamesdenton, jrosser ill take credit for that fail21:21
mnaserhttps://github.com/openstack/openstack-ansible-os_neutron/commit/44150511780d6619e4b11a702d787224fb11ea1d#diff-ed0f41199831cf835e86db553a8853d721:22
mnasercommit of shame21:22
jamesdentonwere there any complaints?21:22
jamesdenton:D21:22
mnaserjamesdenton: you in berlin?21:23
jamesdentonyessir21:23
mnaserthats a beer your way21:23
jamesdentonyessss21:23
* mnaser was stuck for an embarsssingly long time and didnt know why and resorted to neutron_l3_agent_conf_override21:23
mnaserjrosser: you gonna push up a patch or want me to?21:24
*** DanyC has quit IRC21:24
jrosseri'm just testing it21:24
jamesdentonglad i could help. fresh eyes.21:24
*** DanyC has joined #openstack-ansible21:24
jamesdentongotta go pick up the kids. good luck!21:25
jrosserlength > 0 would match conditionals elsewhere21:25
mnaser>= 1 ?21:25
jrosser? it can be 2?21:25
mnasersorry you probably use irc cloud21:26
mnaser> = 221:26
mnasergod damnit21:26
mnaser\>=221:26
mnaserescaping irc text, yay21:26
mnaseri shouldnt talk this late21:26
mnaseri meant21:26
mnaser\>= 121:26
jrossermnaser: i did wonder if there were some other bits missing too21:27
jrosserhttps://docs.openstack.org/neutron/latest/admin/fwaas-v2-scenario.html21:27
mnaserjrosser: i got bit by this bug more for the vpnaas which worked fine but was missing the extensions prrt21:28
*** DanyC has quit IRC21:28
jrosserfor fwaas i think we don't configure [service_providers]21:28
* jrosser renews 12hr access token. day tooo long21:30
mnaserhaha21:32
mnaserrealstack token issue21:32
*** ansmith has joined #openstack-ansible21:34
*** dxiri has joined #openstack-ansible21:41
openstackgerritJonathan Rosser proposed openstack/openstack-ansible-os_neutron master: Correct conditional in l3_agent.ini template  https://review.openstack.org/61487821:45
*** dxiri has quit IRC21:45
*** dxiri has joined #openstack-ansible21:46
*** dxiri has quit IRC21:46
*** aredinger has joined #openstack-ansible21:54
*** aredinger has quit IRC21:58
openstackgerritMerged openstack/openstack-ansible stable/pike: Update all SHAs for 16.0.22  https://review.openstack.org/61374622:06
*** aredinger has joined #openstack-ansible22:12
*** aredinger has quit IRC22:16
*** threestrands has joined #openstack-ansible22:17
*** evrardjp has quit IRC22:19
*** evrardjp has joined #openstack-ansible22:22
*** Leo_m has quit IRC22:36
*** fghaas has quit IRC22:45
*** aredinger has joined #openstack-ansible22:54
*** cshen has quit IRC22:56
*** aredinger has quit IRC22:59
*** spatel has joined #openstack-ansible23:21
*** spatel has quit IRC23:25
openstackgerritJimmy McCrory proposed openstack/openstack-ansible-os_tempest master: Ubuntu distro install uses testrepository  https://review.openstack.org/61488823:27
openstackgerritJimmy McCrory proposed openstack/openstack-ansible-os_nova master: Fix issues with nova distro install on Ubuntu  https://review.openstack.org/61437223:31
*** aredinger has joined #openstack-ansible23:54
*** faizy_ has joined #openstack-ansible23:57
*** gyee has quit IRC23:58
*** aredinger has quit IRC23:59

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