Friday, 2018-06-08

*** tosky has quit IRC00:00
*** chyka has quit IRC00:02
*** vnogin has quit IRC00:06
*** jeferobles has joined #openstack-ansible00:35
*** shardy has quit IRC00:41
*** shardy has joined #openstack-ansible00:41
*** DanyC has joined #openstack-ansible01:01
*** jra has quit IRC01:02
*** jra has joined #openstack-ansible01:03
*** gyee has quit IRC01:05
*** DanyC has quit IRC01:05
*** jeferobles has quit IRC01:10
*** jra has quit IRC01:16
*** jra has joined #openstack-ansible01:17
*** dxiri_ has joined #openstack-ansible01:42
*** dxiri has quit IRC01:44
*** dxiri_ has quit IRC01:45
*** dxiri has joined #openstack-ansible01:46
*** dxiri_ has joined #openstack-ansible01:48
*** dxiri has quit IRC01:51
*** dxiri_ has quit IRC01:54
*** dxiri has joined #openstack-ansible02:00
*** DanyC has joined #openstack-ansible02:02
*** jeferobles has joined #openstack-ansible02:05
*** DanyC has quit IRC02:06
*** dxiri_ has joined #openstack-ansible02:23
*** madorn has quit IRC02:24
*** dxiri__ has joined #openstack-ansible02:24
*** dxiri has quit IRC02:24
*** dxiri_ has quit IRC02:27
*** madorn has joined #openstack-ansible02:28
*** weezS has joined #openstack-ansible02:32
*** weezS has quit IRC02:32
openstackgerritKevin Carter (cloudnull) proposed openstack/openstack-ansible-os_horizon master: Add developer panels to horizon  https://review.openstack.org/57331802:37
cloudnulljrosser ^02:37
*** jeferobles has quit IRC02:40
*** jeferobles has joined #openstack-ansible02:41
openstackgerritKevin Carter (cloudnull) proposed openstack/openstack-ansible-ops master: Typo and bug fixes  https://review.openstack.org/57329502:42
openstackgerritKevin Carter (cloudnull) proposed openstack/openstack-ansible-ops master: Typo and bug fixes  https://review.openstack.org/57329502:44
openstackgerritMerged openstack/openstack-ansible-ops master: Make OSA repo location configurable  https://review.openstack.org/57345502:45
openstackgerritMerged openstack/openstack-ansible-ops master: Bypass pip constraints when installing Elastic Curator  https://review.openstack.org/57235502:45
openstackgerritMerged openstack/openstack-ansible-ops master: Point metricbeat haproxy collector to existing stats endpoint  https://review.openstack.org/57235602:45
openstackgerritMerged openstack/openstack-ansible-ops master: Point metricbeat rabbitmq collector to existing rabbitmq endpoint  https://review.openstack.org/57235702:46
openstackgerritMerged openstack/openstack-ansible-ops master: Enable apache2 status and collect with metricbeat  https://review.openstack.org/57235802:46
openstackgerritMerged openstack/openstack-ansible-ops master: Increase accuracy of detecting apache install  https://review.openstack.org/57235902:47
openstackgerritMerged openstack/openstack-ansible-ops master: Enable nginx metric collection with metricbeat  https://review.openstack.org/57236002:47
openstackgerritMerged openstack/openstack-ansible-ops master: Point metricbeat to the correct address for memcached stats  https://review.openstack.org/57236102:47
openstackgerritKevin Carter (cloudnull) proposed openstack/openstack-ansible-ops master: Add logstash filters  https://review.openstack.org/57329702:48
openstackgerritMerged openstack/openstack-ansible-ops master: Optionally collect uwsgi stats from unix domain sockets  https://review.openstack.org/57237402:48
openstackgerritMerged openstack/openstack-ansible-ops master: Add config override docs for gathering uwsgi stats  https://review.openstack.org/57237702:49
openstackgerritMerged openstack/openstack-ansible-ops master: Use hostname rather than ip when collecting mysql stats  https://review.openstack.org/57239202:49
openstackgerritKevin Carter (cloudnull) proposed openstack/openstack-ansible-rabbitmq_server master: Update rabbitmq to the latest stable release  https://review.openstack.org/57220702:59
openstackgerritMerged openstack/openstack-ansible-ops master: Typo and bug fixes  https://review.openstack.org/57329502:59
*** markvoelker has quit IRC03:00
*** markvoelker has joined #openstack-ansible03:02
*** DanyC has joined #openstack-ansible03:03
*** DanyC has quit IRC03:07
*** markvoelker has quit IRC03:07
*** markvoelker has joined #openstack-ansible03:12
openstackgerritKevin Carter (cloudnull) proposed openstack/openstack-ansible-ops master: Add logstash filters  https://review.openstack.org/57329703:14
openstackgerritMerged openstack/openstack-ansible-ops master: pin diskimage-builder version  https://review.openstack.org/57340903:25
*** Bhujay has joined #openstack-ansible03:28
openstackgerritMerged openstack/openstack-ansible-ops master: Add logstash filters  https://review.openstack.org/57329703:32
*** jeferobles has quit IRC03:39
*** dave-mcc_ has quit IRC03:44
*** udesale has joined #openstack-ansible03:48
openstackgerritKevin Carter (cloudnull) proposed openstack/openstack-ansible-rabbitmq_server master: Update rabbitmq to the latest stable release  https://review.openstack.org/57220703:53
*** hassenius has joined #openstack-ansible03:53
openstackgerritKevin Carter (cloudnull) proposed openstack/openstack-ansible master: Set format options when prepairing filesystems  https://review.openstack.org/57230703:57
*** hassenius has quit IRC03:57
openstackgerritKevin Carter (cloudnull) proposed openstack/openstack-ansible master: Allow integration of externally deployed RadosGW  https://review.openstack.org/56827603:58
*** mmalchuk has joined #openstack-ansible04:02
*** poopcat has quit IRC04:02
openstackgerritKevin Carter (cloudnull) proposed openstack/openstack-ansible master: Begin testing opensuse and centos with nspawn  https://review.openstack.org/56819704:04
*** jeferobles has joined #openstack-ansible04:06
*** mmalchuk has quit IRC04:11
openstackgerritKevin Carter (cloudnull) proposed openstack/openstack-ansible-os_aodh master: Add packages required for osprofiler  https://review.openstack.org/57349604:17
*** jeferobles has quit IRC04:18
*** jeferobles has joined #openstack-ansible04:19
openstackgerritKevin Carter (cloudnull) proposed openstack/openstack-ansible-os_barbican master: Add packages required for osprofiler  https://review.openstack.org/57349804:19
openstackgerritKevin Carter (cloudnull) proposed openstack/openstack-ansible-os_ceilometer master: Add packages required for osprofiler  https://review.openstack.org/57349904:20
openstackgerritKevin Carter (cloudnull) proposed openstack/openstack-ansible-os_cinder master: Add packages required for osprofiler  https://review.openstack.org/57350004:21
*** jeferobles has quit IRC04:22
*** jeferobles has joined #openstack-ansible04:23
openstackgerritKevin Carter (cloudnull) proposed openstack/openstack-ansible-os_congress master: Add packages required for osprofiler  https://review.openstack.org/57350104:24
*** chhagarw has joined #openstack-ansible04:25
openstackgerritKevin Carter (cloudnull) proposed openstack/openstack-ansible-os_designate master: Add packages required for osprofiler  https://review.openstack.org/57350204:25
*** jeferobles has quit IRC04:26
openstackgerritKevin Carter (cloudnull) proposed openstack/openstack-ansible-os_glance master: Add packages required for osprofiler  https://review.openstack.org/57350304:26
*** jeferobles has joined #openstack-ansible04:26
openstackgerritKevin Carter (cloudnull) proposed openstack/openstack-ansible-os_gnocchi master: Add packages required for osprofiler  https://review.openstack.org/57350404:27
openstackgerritKevin Carter (cloudnull) proposed openstack/openstack-ansible-os_heat master: Add packages required for osprofiler  https://review.openstack.org/57350504:28
openstackgerritKevin Carter (cloudnull) proposed openstack/openstack-ansible-os_ironic master: Add packages required for osprofiler  https://review.openstack.org/57350604:29
openstackgerritKevin Carter (cloudnull) proposed openstack/openstack-ansible-os_keystone master: Add packages required for osprofiler  https://review.openstack.org/57350804:30
*** chhagarw has quit IRC04:32
openstackgerritKevin Carter (cloudnull) proposed openstack/openstack-ansible-os_magnum master: Add packages required for osprofiler  https://review.openstack.org/57350904:32
openstackgerritKevin Carter (cloudnull) proposed openstack/openstack-ansible-os_molteniron master: Add packages required for osprofiler  https://review.openstack.org/57351004:33
*** prestonr83 has quit IRC04:34
*** prestonr83 has joined #openstack-ansible04:35
openstackgerritKevin Carter (cloudnull) proposed openstack/openstack-ansible-os_neutron master: Add packages required for osprofiler  https://review.openstack.org/57351104:36
openstackgerritKevin Carter (cloudnull) proposed openstack/openstack-ansible-os_nova master: Add packages required for osprofiler  https://review.openstack.org/57351204:38
openstackgerritKevin Carter (cloudnull) proposed openstack/openstack-ansible-os_octavia master: Add packages required for osprofiler  https://review.openstack.org/57351304:38
openstackgerritKevin Carter (cloudnull) proposed openstack/openstack-ansible-os_sahara master: Add packages required for osprofiler  https://review.openstack.org/57351404:40
openstackgerritKevin Carter (cloudnull) proposed openstack/openstack-ansible-os_swift master: Add packages required for osprofiler  https://review.openstack.org/57351604:41
*** jeferobles has quit IRC04:42
*** jeferobles has joined #openstack-ansible04:43
openstackgerritKevin Carter (cloudnull) proposed openstack/openstack-ansible-os_tacker master: Add packages required for osprofiler  https://review.openstack.org/57351704:44
*** ianychoi has quit IRC04:44
openstackgerritKevin Carter (cloudnull) proposed openstack/openstack-ansible-os_trove master: Add packages required for osprofiler  https://review.openstack.org/57351804:45
openstackgerritKevin Carter (cloudnull) proposed openstack/openstack-ansible-os_zun master: Add packages required for osprofiler  https://review.openstack.org/57351904:51
*** jeferobles has quit IRC04:52
openstackgerritKevin Carter (cloudnull) proposed openstack/openstack-ansible-os_monasca master: Add packages required for osprofiler  https://review.openstack.org/57352004:53
*** jeferobles has joined #openstack-ansible04:53
openstackgerritKevin Carter (cloudnull) proposed openstack/openstack-ansible-os_cloudkitty master: Add packages required for osprofiler  https://review.openstack.org/57352104:54
openstackgerritKevin Carter (cloudnull) proposed openstack/openstack-ansible-os_zaqar master: Add packages required for osprofiler  https://review.openstack.org/57352204:55
openstackgerritKevin Carter (cloudnull) proposed openstack/openstack-ansible-os_watcher master: Add packages required for osprofiler  https://review.openstack.org/57352304:56
*** jeferobles has quit IRC04:56
openstackgerritKevin Carter (cloudnull) proposed openstack/openstack-ansible-os_masakari master: Add packages required for osprofiler  https://review.openstack.org/57352404:59
openstackgerritKevin Carter (cloudnull) proposed openstack/openstack-ansible-os_blazar master: Add packages required for osprofiler  https://review.openstack.org/57352505:01
openstackgerritKevin Carter (cloudnull) proposed openstack/openstack-ansible-os_panko master: Add packages required for osprofiler  https://review.openstack.org/57352605:02
*** DanyC has joined #openstack-ansible05:04
cloudnulljrosser https://review.openstack.org/#/q/topic:osprofiler+(status:open+OR+status:merged)+owner:kevin%2540cloudnull.com+status:open - that should add the needed packages to make osprofiler work everywhere. I'm of the opinion that we can enable it using config_template.05:05
cloudnullif the packages are provided by the role we can do everything else in user_vars.05:06
cloudnullthough I guess it might not be terrible to have a generator, but i'd like to limit the impact on group_vars if at all possible.05:06
*** vnogin has joined #openstack-ansible05:07
*** DanyC has quit IRC05:09
*** mma has quit IRC05:10
*** mma has joined #openstack-ansible05:10
*** vnogin has quit IRC05:11
*** mma has quit IRC05:15
*** Bhujay has quit IRC05:19
*** chyka has joined #openstack-ansible05:26
*** Taseer has quit IRC05:28
*** chyka has quit IRC05:31
*** Taseer has joined #openstack-ansible05:33
*** hassenius has joined #openstack-ansible05:55
*** mmalchuk has joined #openstack-ansible05:57
*** pcaruana has joined #openstack-ansible05:57
*** hassenius has quit IRC05:59
-openstackstatus- NOTICE: Zuul stopped receiving gerrit events around 04:00UTC; any changes submitted between then and now will probably require a "recheck" comment to be requeued. Thanks!06:00
*** DanyC has joined #openstack-ansible06:02
*** mma has joined #openstack-ansible06:04
*** armaan has joined #openstack-ansible06:04
*** mma has quit IRC06:10
*** pcaruana has quit IRC06:14
*** pcaruana has joined #openstack-ansible06:15
*** Bhujay has joined #openstack-ansible06:16
*** armaan has quit IRC06:20
*** armaan has joined #openstack-ansible06:20
*** jbadiapa has joined #openstack-ansible06:22
*** radeks has joined #openstack-ansible06:24
*** DanyC has quit IRC06:24
*** radeks_ has joined #openstack-ansible06:28
*** radeks has quit IRC06:30
*** dxiri__ has quit IRC06:31
*** dxiri has joined #openstack-ansible06:31
*** cshen has joined #openstack-ansible06:34
*** mma has joined #openstack-ansible06:35
*** Bhujay has quit IRC06:43
*** armaan has quit IRC06:44
*** fghaas has joined #openstack-ansible06:44
*** armaan has joined #openstack-ansible06:45
*** smorant has joined #openstack-ansible06:51
*** jeferobles has joined #openstack-ansible06:59
*** radeks_ has quit IRC07:01
*** Bhujay has joined #openstack-ansible07:02
*** jeferobles has quit IRC07:03
evrardjpgood morning07:04
*** dxiri has quit IRC07:05
ebbex_evrardjp: mornin'07:07
*** radeks_ has joined #openstack-ansible07:20
*** cjloader has joined #openstack-ansible07:22
*** DanyC has joined #openstack-ansible07:23
*** cjloader has quit IRC07:26
mmalchukevrardjp morning07:28
*** tosky has joined #openstack-ansible07:30
mmalchukevrardjp could you please review and merge pike SHA changes: https://review.openstack.org/#/q/status:open+project:openstack/openstack-ansible+branch:stable/pike+author:maksim.malchuk%2540gmail.com07:30
*** radeks_ has quit IRC07:31
*** jbadiapa has quit IRC07:32
*** jbadiapa has joined #openstack-ansible07:33
*** cshen has quit IRC07:37
*** dxiri has joined #openstack-ansible07:39
evrardjpmmalchuk: I do this today technically with the usual bump, is there a reason you want to it to merge faster? I am fine with that, but I'd like a reference to a bug in those bumps07:44
evrardjpstating the urgency for not waiting the usual 2 week timeframe07:44
evrardjp2 week soak07:44
evrardjpif we merge those they will get included in the release of today, which is probably what you want, right? :)07:44
evrardjpI just need something to point to the reason of an express merge.07:45
openstackgerritPan Xingyu proposed openstack/openstack-ansible-os_masakari master: Optimize the link address in docs  https://review.openstack.org/57360607:51
hwoaranggood morning07:51
hwoarangcores: an easy review for you please :) https://review.openstack.org/#/c/572293/07:51
odyssey4meo/07:52
odyssey4meapologies for being missing yesterday - new laptop, and unfortunately my ZNC address is blocked due to some security at the office07:52
evrardjphwoarang: reviewed07:53
hwoarangawesome thanks!07:54
* hwoarang is going to look at the neutron bug today provided nobody drags him into internal bugs07:55
*** epalper has joined #openstack-ansible07:55
odyssey4mecores, could we get this reno in before the next release please: https://review.openstack.org/#/q/Ic342de2a8fcb67bc8b1c55a26d7eb910ef55c60807:57
mmalchukevrardjp sorry, can't reffer to the bug, all these patches were made as a fast fixes(07:57
odyssey4mealso this bug fix for keystone federation: https://review.openstack.org/57250207:57
*** Bhujay has quit IRC07:57
hwoarangevrardjp: where is the whitespace?07:58
hwoarangwhat kind of whitespace...07:58
hwoarangthank you odyssey4me for taking care of it07:58
hwoarangodyssey4me: these 3 commits contain the same ID but are they really cherry-picks? (also the cherry picked by ... msg is missing)07:59
hwoarangi wonder how they could end up with same id if they are not cherry picks07:59
*** shardy has quit IRC07:59
odyssey4mehwoarang: because they're each taylored to their own series08:00
odyssey4mesame general message, but different series names and different version numbers08:00
hwoarangyep i understand that. so it's really an "edited" cherry pick?08:00
odyssey4meyep08:00
hwoarangok thanks!08:01
evrardjpwe are now with ansible 2.508:01
odyssey4meyep, saw that merge last night - how exciting08:01
evrardjpit's time for cleaning up things08:01
*** shardy has joined #openstack-ansible08:01
evrardjpyeah I have my profiling to rebase :p08:01
hwoaranghmm but -tests is not with 2.5 right?08:02
*** smorant1 has joined #openstack-ansible08:02
odyssey4meI'll get on to moving to using shade as far as possible, and using the deploy host/utility container to do it - that way we clean up some of the host installs08:02
odyssey4metests merged 2.5 some time ago08:02
hwoarangah it is08:02
evrardjpI must see I love the post failures08:02
evrardjpsay*08:02
evrardjpit's so hidden you have the impression everything worked.08:02
odyssey4me'post failures' ?08:02
evrardjphttps://review.openstack.org/#/c/573398/208:02
evrardjpcheck latest zuul08:03
evrardjpodyssey4me: yes tests merged some time ago,but now the integrated merged too08:03
*** Adri2000 has quit IRC08:03
odyssey4menot sure I understand what you mean by post failures08:04
*** smorant has quit IRC08:04
evrardjpmmalchuk: did you do the same for queens?08:06
*** gkadam has joined #openstack-ansible08:06
evrardjpodyssey4me: you can see it's listed there. that's fine, ignore.08:06
mmalchukpatches- yes, sha pins - no08:06
mmalchukfor pike this need urgent for my deploy08:07
evrardjpmmalchuk: please do so for queens too.08:07
mmalchukokay08:07
evrardjpI understand your position08:07
evrardjpthat's why I voted positively08:07
evrardjpBut let's not do too much differences with the policies, we have them for a reason08:07
mmalchukr you ok if i made one squashed commit for queens? or only three cherry-picks acctepable?08:09
*** cshen has joined #openstack-ansible08:09
evrardjpI'd prefer the equivalent patches08:09
evrardjpeasier to track08:09
mmalchukokay08:09
mmalchukwill do08:10
openstackgerritJesse Pretorius (odyssey4me) proposed openstack/openstack-ansible master: Move database creation into role (sahara)  https://review.openstack.org/57179808:10
evrardjpmmalchuk: technically we do master,and we backport when bugs08:10
evrardjpso here because there is no patch in master, it should have been initiated in queens, and then backported back to P/O08:10
mmalchukunderstand, but this is sha pins - it related only for the release branch)08:11
*** Adri2000 has joined #openstack-ansible08:11
evrardjpI understand: the code, yes, is different, but the reason itself is the same. And the reason is important on why we backport to branch x08:12
evrardjpat least that's my opinion08:12
evrardjpbut other ppl can disagree, and I am fine with that :)08:12
mmalchukso in this case for queens tere would be separate patches with the same topics but contains other SHAs08:12
evrardjpdiversity ! :)08:12
*** shardy has quit IRC08:13
evrardjpso what we do, in general, when we cherry pick in the web interface, it adds a nice branch at the end of the topic08:13
openstackgerritJesse Pretorius (odyssey4me) proposed openstack/openstack-ansible master: Move database creation into role (heat)  https://review.openstack.org/57175108:14
*** hassenius has joined #openstack-ansible08:14
evrardjpyou know it's not that important mmalchuk . As long as we don't forget some on the way. That's my reason to -w for now. I am afraid of not remembering that some branch got the fix already, while others do not.08:15
evrardjpit makes sense to get the fix in chronological order for me08:15
odyssey4memmalchuk: is this those bumps to pull in the os_tempest changes for cmd2 ?08:15
mmalchukyep08:16
mmalchukone of these08:16
odyssey4memmalchuk: are you building tempest from git source? if so, then that may be needed, but we build from pypi and use upper-constraints, so those patches aren't required for the default build08:17
openstackgerritMerged openstack/openstack-ansible stable/queens: Add release note with workaround for spice-html5 repo change  https://review.openstack.org/57324008:17
mmalchukone of these if UC pin for cmd2 too08:17
odyssey4memmalchuk: u-c already had pins, so it was fine for the stable branches08:17
mmalchukboth required08:18
odyssey4meand also fine for master08:18
mmalchukodyssey4me AFAIK yesterday don't had08:18
odyssey4mewhich branch are you worried about?08:18
mmalchukstable/pike08:18
evrardjpodyssey4me: we haven't bumped for a while due to summit.08:19
mmalchukodyssey4me ^^^08:19
evrardjpso the requirements repo u-c wasn't updated on O/P/Q recently08:19
evrardjpmy bad08:19
mmalchuk:(08:19
evrardjpanyone can request a release :p08:20
odyssey4meour last release used this sha for pike, and has this pin: https://github.com/openstack/requirements/blob/b09735069fcef17bbc1aee757436a128be5c2dd4/upper-constraints.txt#L38808:20
evrardjpI haven't checked the pins08:20
odyssey4meso again, there is no need to try and pull any of those in, unless you happen to be building from source08:20
odyssey4mefrom git source08:21
odyssey4meand our last release doesn't build from git source08:21
mmalchukwe use git sources to build and build fails08:21
odyssey4meok, yes - if you need to build from git source then it will fail, but you only need the role sha bump for that08:22
mmalchukalso we don't have internet access so without https://review.openstack.org/#/c/573215/ it fails too08:22
*** prestonr83 has quit IRC08:23
evrardjpso https://review.openstack.org/#/c/573207/1/ansible-role-requirements.yml08:23
mmalchukrole sha bump : https://review.openstack.org/#/c/573207/08:23
odyssey4memmalchuk: ok, I don't see a queens patch of the same to make sure queens has the same functionality08:23
*** hassenius has quit IRC08:23
*** prestonr83 has joined #openstack-ansible08:23
evrardjpodyssey4me: I just said the same, mmalchuk will do a patch.08:23
odyssey4mebackports must be done from latest, backwards08:23
mmalchukokay, will do the same for queens then08:23
evrardjpI am -w until so08:24
evrardjpthen we can release the kraken!08:24
odyssey4memmalchuk: use the same change id please08:24
mmalchuksure08:24
mmalchukalso the same topic08:24
mmalchuk)08:24
odyssey4mealso, the commit message should explain why it's important to bring forward rather than wait for the typical two week bump cycle08:24
evrardjpI said that already :)08:25
evrardjpanywya08:25
odyssey4meat least a reference to the patch it's bringing in08:25
odyssey4meok, sorry - didn't see that08:25
hwoarangevrardjp: you have -1 to https://review.openstack.org/#/c/572293/ but the meta is already convered08:25
*** hassenius has joined #openstack-ansible08:25
hwoarangand i can't find any whitespace issues08:25
* odyssey4me is also using a new IRC client and the interface is weird08:25
* hwoarang grabs a coffee just in case08:25
evrardjpoh "all"08:26
evrardjphaha08:26
evrardjpsmart.08:26
evrardjpIt will break things though08:26
hwoarangwhy?08:26
*** rodolof has joined #openstack-ansible08:26
evrardjpit's mine to fix don't worry08:26
evrardjpbecause I am using that meta information for some scripts08:26
evrardjpwe would have an "all" distro for suse08:27
evrardjpwhich is embarassing.08:27
hwoaranghonestly i am not sure how we use the meta info. i thought it was mostly for galaxy08:27
odyssey4meevrardjp: nice work on https://review.openstack.org/#/c/572152/ :)08:28
openstackgerritMaksim Malchuk proposed openstack/openstack-ansible stable/queens: Update os_tempest role SHA for cmd2-pin fix  https://review.openstack.org/57361508:29
evrardjpI don't know if it's sarcasm or not, so I will take it as compliment08:29
*** chhagarw has joined #openstack-ansible08:30
openstackgerritJesse Pretorius (odyssey4me) proposed openstack/openstack-ansible master: Move database creation into role (ironic)  https://review.openstack.org/57177308:30
odyssey4meevrardjp: no, seriously - thanks for reaching out and getting it fixed!08:30
*** shardy has joined #openstack-ansible08:31
odyssey4meok, are there any specific reviews that need some eyes or is there anything specific I can help out with today08:33
openstackgerritMaksim Malchuk proposed openstack/openstack-ansible stable/queens: Update repo_build role SHA for custom PyPi indexes fix  https://review.openstack.org/57361908:33
odyssey4meoh wow, now I'm seeing a ton of post failures - are these what you were talking about evrardjp? https://review.openstack.org/568276 / https://review.openstack.org/56819708:34
evrardjpyes.08:34
odyssey4meah ok - the review you pointed me at earlier just had normal failures :p08:35
odyssey4mehwoarang: is there any specific reason why we can't somehow use a general opensuse vars file to try to reduce the churn in code?08:35
odyssey4meit looks to me like most of the time the vars files don't change much08:36
hwoarangodyssey4me: tell me more08:36
hwoarangsuse.yml instead of suse-42.yml ?08:36
odyssey4meso in some roles we just use 'ubuntu' or 'debian' - it applies to all versions08:36
odyssey4meyep08:36
hwoarangit should be possible except for the lxc_* roles08:36
odyssey4methen we only make changes when they need to be made08:36
hwoarangi am converting that as I add leap 15 jobs08:36
evrardjpodyssey4me: no it didn't have normal failures in the last zuul run.08:36
evrardjpnot to what I have seen08:37
evrardjpbut it's the morning08:37
odyssey4mewe see it with bionic now where we're converting the ubuntu-16 file to just ubuntu/debian where possible, but in some cases where differences are there we keep seperate files08:37
*** markvoelker has quit IRC08:37
hwoarangyeah so far the two suses differ in the contianer stuff. the rest is the same08:38
odyssey4meif we can do that, it really helps reduce the work that needs to be done for new releases, which for leap happens every 6 months?08:38
*** markvoelker has joined #openstack-ansible08:38
* hwoarang is not sure08:38
hwoarangbut yeah i agree08:39
odyssey4methanks :)08:39
odyssey4meooh, I see that RDO is busy prepping to package uwsgi for us08:40
odyssey4meand I see that mnaser is on a mission to remove our EPEL deps08:40
openstackgerritMaksim Malchuk proposed openstack/openstack-ansible stable/queens: Update requirements SHAs for 16.0.14 with cmd2-pin fix  https://review.openstack.org/57362308:41
*** markvoelker has quit IRC08:42
mmalchukevrardjp: https://review.openstack.org/#/q/status:open+project:openstack/openstack-ansible+branch:stable/queens+author:maksim.malchuk%2540gmail.com08:42
evrardjpodyssey4me: yes, it's globally good and improving :)08:43
mmalchukyuup08:44
*** radeks_ has joined #openstack-ansible08:44
*** pvradu has joined #openstack-ansible08:46
evrardjpmmalchuk: why didn't you include what odyssey4me and I just said? :'(08:46
evrardjpI am a sad panda now08:47
mmalchuksorry what I forgot08:48
mmalchuk?08:48
evrardjpmmalchuk: the sooner it's properly done, the faster you get your votes :) I don't want to be a pain but if you learn it's gonna be faster for next iteration08:49
evrardjpmmalchuk: the explanation of why you need that landing fast.08:49
evrardjpmmalchuk: could you clarify something for me?08:52
evrardjpthe repo build is one thing08:52
hwoarangdoes anyone remember the reasoning behind https://github.com/openstack/openstack-ansible-os_neutron/commit/504c351a8340f8d7ec6084012f6f1da5d275885d#diff-e46f21539d96fcf22c70e47dec57e8ba ?08:52
evrardjpand the cmd2-fix is another right?08:52
hwoaranglbaas was added to tests but we are not installing the lbaas packages and neutron seems to break08:52
mmalchuksure, moment08:52
evrardjphwoarang: I have a patch for that08:52
odyssey4mehwoarang: it was just something to test an extra plugin - that was our first one08:53
hwoarangevrardjp: tell me more08:53
evrardjphttps://review.openstack.org/#/c/572133/08:53
odyssey4meevrardjp: different repo ;)08:53
hwoarangtrue08:53
hwoarangi think i am going to disable it on os_neutron . either that or install lbaas08:53
evrardjpfor neutron itself we can make a scenario08:53
odyssey4meso yeah, that should be removed now08:54
evrardjpyeah rollback08:54
hwoarangok08:54
hwoarangsneaky08:54
evrardjpwell08:54
odyssey4mein neutron we should probably make each plugin its own scenario to test them all - but doing that requires understanding how to test them :p08:54
evrardjpyeah. Here I'd say it makes sense to not test lbaas08:54
evrardjpbecause we have octavia role08:54
evrardjpwe should just move to octavia and be done with it08:55
odyssey4meagreed, the old lbaas stuff is no longer supported08:55
odyssey4meI heard a rumour that there was some work going on to migrate the old lbaas to octavia though.08:55
evrardjpjohnsom: will have a script from migrating from lbaasv2 to octavia soon, IIRC. johnsom could you confirm?08:55
odyssey4melol, same-same08:55
mmalchukevrardjp the first thing these without these pins osa pike can't build from sources due to cmd2 bug, the next one is repo_build fails in my case because of limited internet (we use own pypi indexes)08:55
evrardjpmmalchuk: just mention this in the review :)08:56
odyssey4memmalchuk: add the explanations into the review... even if it's a copy (or close to a copy) from the role patch's commit msg08:56
evrardjpTell for you it's urgent, and shouldn't wait for next release because this is broken as hell for you.08:56
odyssey4mecopy-paste the commit, adjust it and refer to the review being pulled in using depends-on08:56
mmalchukodyssey4me in the comments fine?08:56
evrardjpmmalchuk: we need to make sure it's understood by everyone.08:56
odyssey4methat way it's linked and it's easy to see the connection08:57
evrardjpmmalchuk: no in the commit message :)08:57
mmalchukok08:57
evrardjpyou'll get your votes quick, don't worry :)08:57
evrardjphwoarang: do you deal with the removal of this part of os_neutron?08:57
hwoarangi am08:57
evrardjpIf you don't have time, ask mbuil08:57
evrardjphe should be dealing with those08:57
hwoarangoh wait08:57
hwoarangfor testing?08:57
hwoarangno08:57
hwoarangmanuel is the man08:58
evrardjp?08:58
hwoarangi can remove this part from os_neutron tests. for making tests better you need mbuil08:58
evrardjpok.08:59
evrardjpyeah let's do so.08:59
evrardjpthe urgent part is urgent08:59
mmalchukevrardjp i'm worry only for the CI, it fails too often on git clone :(08:59
mmalchukand if I change commit msg - will spend several hours doing rechecks ;(09:00
mmalchukodyssey4me ^^^09:01
evrardjpmmalchuk: it shouldn't.09:01
mmalchukwill see)09:01
openstackgerritMaksim Malchuk proposed openstack/openstack-ansible stable/pike: Update os_tempest role SHA for cmd2-pin fix  https://review.openstack.org/57320709:02
*** pvradu_ has joined #openstack-ansible09:02
*** pvradu has quit IRC09:02
*** pvradu_ is now known as pvradu09:02
evrardjpwe have two transient issues for now, and you're welcome to help. 1) is the git cloning/repo build failing, most of the time due to a weird message of "no space left" . 2) is the tempest testing being too slow.09:02
openstackgerritMaksim Malchuk proposed openstack/openstack-ansible stable/queens: Update os_tempest role SHA for cmd2-pin fix  https://review.openstack.org/57361509:02
openstackgerritMarkos Chandras (hwoarang) proposed openstack/openstack-ansible-os_neutron master: Revert "Ensure LBaaSv2 deployment is tested"  https://review.openstack.org/57362909:03
*** hassenius has quit IRC09:03
evrardjpodyssey4me: could you review mmalchuk patches?09:03
odyssey4meafk for a few mins, brb09:03
mmalchukwait a while i'll change the msg for all of these09:04
evrardjpok09:04
*** radeks_ has quit IRC09:04
openstackgerritMaksim Malchuk proposed openstack/openstack-ansible stable/pike: Update repo_build role SHA for custom PyPi indexes fix  https://review.openstack.org/57321509:07
openstackgerritMaksim Malchuk proposed openstack/openstack-ansible stable/queens: Update repo_build role SHA for custom PyPi indexes fix  https://review.openstack.org/57361909:07
*** vnogin has joined #openstack-ansible09:08
openstackgerritMaksim Malchuk proposed openstack/openstack-ansible stable/pike: Update requirements SHAs for 16.0.14 with cmd2-pin fix  https://review.openstack.org/57322109:09
openstackgerritMaksim Malchuk proposed openstack/openstack-ansible stable/queens: Update requirements SHAs for 17.0.5 with cmd2-pin fix  https://review.openstack.org/57362309:11
mmalchukdone09:12
*** vnogin has quit IRC09:12
*** smorant has joined #openstack-ansible09:14
*** smorant1 has quit IRC09:15
odyssey4memmalchuk: mind if I edit the review commit msg?09:18
mmalchukokay09:18
*** rodolof has quit IRC09:18
openstackgerritJesse Pretorius (odyssey4me) proposed openstack/openstack-ansible stable/queens: Update os_tempest role SHA pin for cmd2 pin  https://review.openstack.org/57362309:21
odyssey4meoh, misread - hang on09:21
odyssey4memmalchuk: why does that requirement sha need to be updated? the existing sha already has a pin09:23
openstackgerritJesse Pretorius (odyssey4me) proposed openstack/openstack-ansible stable/queens: Update os_tempest role SHA pin for cmd2 pin  https://review.openstack.org/57361509:24
openstackgerritJesse Pretorius (odyssey4me) proposed openstack/openstack-ansible stable/pike: Update os_tempest role SHA pin for cmd2 pin  https://review.openstack.org/57320709:24
*** hassenius has joined #openstack-ansible09:26
openstackgerritJesse Pretorius (odyssey4me) proposed openstack/openstack-ansible stable/queens: Update repo_build role SHA for custom PyPi indexes fix  https://review.openstack.org/57361909:26
openstackgerritJesse Pretorius (odyssey4me) proposed openstack/openstack-ansible stable/queens: Update repo_build role SHA for custom PyPi indexes fix  https://review.openstack.org/57361909:28
openstackgerritJesse Pretorius (odyssey4me) proposed openstack/openstack-ansible stable/pike: Update repo_build role SHA for custom PyPi indexes fix  https://review.openstack.org/57321509:28
openstackgerritJesse Pretorius (odyssey4me) proposed openstack/openstack-ansible stable/pike: Update repo_build role SHA for custom PyPi indexes fix  https://review.openstack.org/57321509:28
*** cshen has quit IRC09:30
*** Taseer has quit IRC09:31
openstackgerritJesse Pretorius (odyssey4me) proposed openstack/openstack-ansible stable/queens: Update requirements SHA for updated cmd2 pin  https://review.openstack.org/57362309:32
odyssey4meevrardjp: the role sha updates look fine to me, that requirements sha isn't needed IMO09:33
odyssey4memmalchuk: FYI, if you're building from a git source, the requirements upper-constraints is not applied... that's why the os_tempest role needs that pin09:35
*** cshen has joined #openstack-ansible09:35
*** Taseer has joined #openstack-ansible09:35
evrardjptempest is a different beast than others09:38
evrardjptempest and rally09:38
evrardjpjust fyi09:38
evrardjpmmalchuk: ^09:38
evrardjpor maybe there was magnum too?  I can't remember now09:39
evrardjpI can check09:39
evrardjpanyway09:39
odyssey4menope, just tempest and rally09:39
evrardjpif you have your own pypi mirror mmalchuk, why the need for that?09:39
evrardjpI am curious09:39
odyssey4meand I'm working to make both build from pypi as a default09:40
evrardjpyeah09:40
openstackgerritMerged openstack/openstack-ansible stable/pike: Add release note with workaround for spice-html5 repo change  https://review.openstack.org/57324209:44
odyssey4mehurray, the ansible docs now show the nova-client as required for the os_server_action module :) https://docs.ansible.com/ansible/latest/modules/os_server_action_module.html09:46
odyssey4meand the inventory example config is fixed: https://docs.ansible.com/ansible/latest/plugins/inventory/openstack.html09:47
ebbex_'os_neutron' role does not have 'neutron_ovs_service_name' defined (referenced in neutron_install.yml) in stable/pike.09:48
ebbex_This https://github.com/openstack/openstack-ansible-os_neutron/commit/7dce6e2b07ea1c309f78d84a9dec82e5727d5797#diff-1a6622f32c2ea17bf316b306e80e15ae is the first commit I see it come into play, but I don't think/know if it should be cherry-picked into pike.09:49
odyssey4meebbex_: I think that's just waiting for a release to bring in a patch recently merged into the role09:51
odyssey4mehttps://github.com/openstack/openstack-ansible-os_neutron/commit/390965a56638bb8c89a9761fc95c6d20696b1fb6 has merged, but not yet released09:51
odyssey4meand the current proposed release is just before that SHA: https://github.com/openstack/openstack-ansible/blob/stable/pike/ansible-role-requirements.yml#L10809:52
evrardjpdarnit my weechat is killing some of my buffers. I will come back09:52
odyssey4meevrardjp: how do you feel about bringing that role sha forward into the next proposed release?09:52
evrardjpfine for me09:53
odyssey4meok, lemme propose that09:53
odyssey4mefghaas and ebbex_ will be pleased :)09:53
*** evrardjp has quit IRC09:53
ebbex_odyssey4me: Aha, cool thanks :) So if I want to go really crazy, I'd just set 'stable/pike' instead of a hash in ansible-role-requirements?09:55
openstackgerritJesse Pretorius (odyssey4me) proposed openstack/openstack-ansible stable/pike: Update os_neutron role SHA for OVS fix  https://review.openstack.org/57365009:56
*** epalper has quit IRC09:56
*** evrardjp has joined #openstack-ansible09:56
odyssey4meebbex_: you could if you like, although updating to the right sha would be better, otherwise you might hit a surprise next time you execute your playbooks09:57
*** ericzolf has joined #openstack-ansible09:57
*** cshen has quit IRC09:57
*** evrardjp_ has joined #openstack-ansible10:00
*** evrardjp has quit IRC10:01
*** evrardjp_ has quit IRC10:05
*** evrardjp has joined #openstack-ansible10:06
mmalchuk evrardjp because our own pypi index differ from upstream (not a full mirror)10:09
openstackgerritMaksim Malchuk proposed openstack/openstack-ansible stable/pike: Update os_tempest role SHA pin for cmd2 pin  https://review.openstack.org/57320710:22
openstackgerritMaksim Malchuk proposed openstack/openstack-ansible stable/queens: Update os_tempest role SHA pin for cmd2 pin  https://review.openstack.org/57361510:22
openstackgerritMaksim Malchuk proposed openstack/openstack-ansible stable/pike: Update repo_build role SHA for custom PyPi indexes fix  https://review.openstack.org/57321510:23
openstackgerritMaksim Malchuk proposed openstack/openstack-ansible stable/queens: Update repo_build role SHA for custom PyPi indexes fix  https://review.openstack.org/57361910:23
*** vnogin has joined #openstack-ansible10:23
*** mma has quit IRC10:25
*** udesale_ has joined #openstack-ansible10:27
*** udesale__ has joined #openstack-ansible10:28
*** udesale has quit IRC10:29
*** udesale_ has quit IRC10:31
blinkizAm having same problem as this bug: https://bugs.launchpad.net/openstack-ansible/+bug/177549410:36
openstackLaunchpad bug 1775494 in openstack-ansible "Queens Insallation fails: github spice-html5 link broken" [Undecided,New]10:36
blinkizI think someone talked about this a couple of days ago in this channel. A patch/workaround. .10:36
openstackgerritMarkos Chandras (hwoarang) proposed openstack/openstack-ansible-os_neutron master: Add support for using distribution packages for OpenStack services  https://review.openstack.org/57366110:37
*** mma has joined #openstack-ansible10:37
hwoarangblinkiz: all branches are fixed but there is no proper release with fixes yet10:38
hwoarangso i suppose you need to deploy from the HEAD for the branch you are interested in10:38
*** markvoelker has joined #openstack-ansible10:38
hwoarangblinkiz: look at this release note for example https://review.openstack.org/#/c/573243/10:39
hwoarangfor a workaround10:39
blinkizhwoarang, thank you. Updating source file myself. :)10:41
blinkizwith info from the patch/release note you provided10:41
*** shardy has quit IRC10:46
*** shardy has joined #openstack-ansible10:48
*** vnogin has quit IRC10:48
*** vnogin_ has joined #openstack-ansible10:48
*** radeks_ has joined #openstack-ansible10:50
*** radeks_ has quit IRC11:00
blinkizAm working on getting openstack-ansible work with IPv6. Currently am having problem with task install PIP. It complains about my IPv6 repository not being trusted. The URL works great, it is just not trusted. I have added squared brackets around the IPv6 address and probably that is not understood in a access list.. or something.. https://paste.ubuntu.com/11:04
blinkizhttps://paste.ubuntu.com/p/PYPNkd7vxr/11:05
blinkizThere is currently no firewall blocking so you guys can even access the repository URL that is failing.11:06
odyssey4meblinkiz: what variables are you setting to try to make it reach the ipv6 address?11:07
odyssey4mecan you share a pastebin of them?11:07
blinkizsure11:07
odyssey4methis is the filter which is used to build the trusted host list, and I don't think it's doing the right things for ipv6? https://github.com/openstack/openstack-ansible-plugins/blob/master/filter/osa-filters.py#L176-L18611:09
odyssey4memaybe it is, you'd need to figure out a way to test11:09
odyssey4mebut basically you can see in https://github.com/openstack/openstack-ansible-pip_install/blob/master/templates/pip.conf.j2#L3 that it's used to build a trusted host list11:10
blinkizodyssey4me, am currently working on getting internal_lv_vip_address to work. https://paste.ubuntu.com/p/xYf3Xp2P3H/11:10
odyssey4megiven that the ipv6 address has ':' in it, that filter will not work as designed11:11
odyssey4meit'll need adjustment to recognise ipv4 vs ipv6 notation11:11
blinkizAlright11:11
odyssey4meif you can PR something up to make it work, and add a test for both to the plugins repo, then that'd be awesome11:11
blinkizAm not a developer so I should probably not touch Python code :)11:12
*** markvoelker has quit IRC11:13
odyssey4meblinkiz: then you'll need to register a bug for it and hope someone picks it up11:14
blinkizodyssey4me, That I can do11:14
*** vnogin_ has quit IRC11:18
*** rodolof has joined #openstack-ansible11:19
*** shardy has quit IRC11:19
*** shardy has joined #openstack-ansible11:20
blinkizodyssey4me, get_netloc_no_port, how does the URL look like when coming into the functon? Is it '1.2.3.4:80' or 'http://1.2.3.4:4687' or 'http://1.2.3.4:8755/urlpath/hello/something.txt'11:22
odyssey4mecould be any one of those11:22
blinkizodyssey4me, Alright, and the result should just be 1.2.3.4?11:23
odyssey4meyes11:23
blinkizodyssey4me, great. Am sending it to my python developer here and my firm. We can together figure something out :)11:23
odyssey4methere are several other netlocal related filters which would also require adjustment11:24
odyssey4me*netloc11:24
tnogistoare there any known problems with building repos on pike right now?11:24
blinkizodyssey4me, okay. Lets see where I end up :)11:24
odyssey4megreat, thanks!11:24
odyssey4mehappy to see someone working through that :)11:25
tnogistoCommand "git checkout -q ff72f0f1d3f4bf0b6147ac1f14ebbc5234789a57" failed with error code 128 in /tmp/openstack-builder/ceilometer11:27
*** udesale__ has quit IRC11:29
*** Bhujay has joined #openstack-ansible11:33
*** rodolof has quit IRC11:35
*** vnogin has joined #openstack-ansible11:38
hwoaranghmm gates are a bit sluggish today11:46
*** vnogin has quit IRC11:49
*** vnogin has joined #openstack-ansible11:52
jrossercloudnull: i left some notes on the horizon role profiler patch, short story is that you need DEBUG=True whenever the developer panels are used or it fails with an exception, plus i had to install osprofiler and elasticsearch pip package manually into the container, not the venv, to get it to go12:00
jrosseridk if thats by design or a mistake that the web app doesnt seem to run in the context of the venv?12:02
openstackgerritMarkos Chandras (hwoarang) proposed openstack/openstack-ansible-os_nova master: Add support for using distribution packages for OpenStack services  https://review.openstack.org/56753012:02
*** vnogin has quit IRC12:03
*** dcdamien has quit IRC12:09
*** Bhujay has quit IRC12:11
*** vnogin has joined #openstack-ansible12:12
*** nicolasbock has joined #openstack-ansible12:13
*** mugsie_ is now known as mugsie12:14
*** markvoelker has joined #openstack-ansible12:16
*** armaan has quit IRC12:25
*** smorant1 has joined #openstack-ansible12:30
*** smorant has quit IRC12:33
mgariepymorning everyone12:39
*** armaan has joined #openstack-ansible12:41
*** armaan has quit IRC12:41
*** armaan has joined #openstack-ansible12:41
*** nsingh has joined #openstack-ansible12:45
*** armaan has quit IRC12:46
*** niraj_singh has quit IRC12:48
*** dcdamien has joined #openstack-ansible12:51
openstackgerritVictor Palma proposed openstack/openstack-ansible-ops master: add osquery  https://review.openstack.org/57368712:58
*** vnogin has quit IRC12:59
*** blinkiz has quit IRC13:05
*** vnogin has joined #openstack-ansible13:07
*** maybebuggy has joined #openstack-ansible13:13
evrardjpOK, so I thought it was a remote plan to publish wheels of things on infra, but no it's real already, like for cinder: http://tarballs.openstack.org/cinder/13:22
evrardjpyou get not only tagged ones13:22
evrardjpbut also branched ones.13:22
evrardjp(HEAD of branch)13:22
evrardjpinterestingly enough.13:22
evrardjpwe should get rid of repo build in ALL the branches :p13:23
evrardjphoping the wheels work with the help of upstream pypi and that's it!13:24
hwoarangok but13:24
hwoarangthere are no wheels for suse yet....13:24
evrardjpIt doesn't matter?13:27
evrardjpdoes it?13:27
evrardjpI think it's universal wheels13:27
evrardjpuniversal is the wrong name, sorry13:28
hwoarangah yeah these wheels are fine13:28
evrardjpI think it would work everywhere on python2.13:28
odyssey4meevrardjp: that's a significant change in what we deliver if we do that13:31
jamesdentonwith stable/queens, have you all seen the haproxy playbook set an http monitor on the galera vip?13:31
odyssey4meour offering allows sha-pinned builds, this would force the use of tarballs13:32
odyssey4methis becomes more flexible once we move the builds into the roles exclusively, but we're not yet there13:32
odyssey4meright now the pain is not really the source builds, it's the fact that it's all over the place and very confusing to debug when things go wrong13:33
evrardjpjamesdenton: yes13:34
evrardjpthat's a feature to test better the mariadb cluster state13:34
evrardjpbecause the haproxy checks were okayish13:34
evrardjpbut that's edfinitely better13:35
evrardjpthe code itself is ugly and could be improved though.13:35
evrardjpbut the check in itself is an improvement13:35
evrardjpit tests cluster state instead of just 3306 connect13:35
evrardjpodyssey4me: yeah but saying if you install from source, you install version x from pip, and if you install from packages you install x from your package repos seems clear enough to me13:36
evrardjpit's simple too13:36
odyssey4mepip install <tarball url> vs pip install <git url> has no added costs13:37
odyssey4meit's the same thing - that's not where the complexity lies13:38
odyssey4me*no difference in costs13:38
jamesdentonthanks evrardjp. For some reason, in my environment, haproxy was in a restart loop with that check enabled. I will need to dig into that13:40
johnsomevrardjp: yes, we the live migration tool is WIP. It will live migrate a load balancer from neutron-lbaas to Octavia. Assuming the driver supports a live migration, which they all should.13:40
*** mmalchuk has quit IRC13:45
-openstackstatus- NOTICE: A misapplied distro security package update caused many jobs to fail with a MERGER_FAILURE error between ~06:30-12:30 UTC; these can be safely rechecked now that the problem has been addressed13:45
mnaserhttps://review.openstack.org/#/c/572945/ please :)13:48
*** throwsb1 has joined #openstack-ansible13:48
*** cjloader has joined #openstack-ansible13:48
openstackgerritMarkos Chandras (hwoarang) proposed openstack/openstack-ansible-galera_client master: vars: Do not install MySQL-python  https://review.openstack.org/57369713:52
openstackgerritMerged openstack/openstack-ansible-os_nova master: Move database creation into role  https://review.openstack.org/57178513:53
openstackgerritMarc GariĆ©py (mgariepy) proposed openstack/openstack-ansible master: Only bind to internal IP.  https://review.openstack.org/56914113:56
openstackgerritMerged openstack/openstack-ansible master: Move database creation into role (neutron)  https://review.openstack.org/57178313:58
openstackgerritKevin Carter (cloudnull) proposed openstack/openstack-ansible-os_horizon master: Add developer panels to horizon  https://review.openstack.org/57331813:58
cloudnullmornings13:59
*** mma has quit IRC14:00
*** rodolof has joined #openstack-ansible14:02
*** mma has joined #openstack-ansible14:02
*** ianychoi has joined #openstack-ansible14:04
*** vnogin has quit IRC14:04
*** vnogin has joined #openstack-ansible14:04
*** mma has quit IRC14:06
*** cjloader has quit IRC14:06
*** cjloader has joined #openstack-ansible14:07
mnaserthanks mgariepy !14:08
mgariepymnaser, you are welcome.14:10
jrossercloudnull: just trying horizon profiler patch again14:16
*** shardy has quit IRC14:18
*** shardy has joined #openstack-ansible14:21
cloudnulljrosser excellent.14:27
jrossercloudnull: just before i got nuts :) do i need to do a repo rebuild to get the new pip packages to be installed?14:27
cloudnulli also submitted reviews to add the requirements throughout the roles.14:27
cloudnullto get them into the venv, yes.14:27
cloudnullyou could however add them, then run the role14:28
cloudnulljust to test the mechanics14:28
jrosserah right, i can just make it not skip the pip install task as it normally would14:29
cloudnullthat too :)14:30
cloudnulldeveloper mode would do that too14:30
cloudnullthough that may pollute the environment in ways you dont want14:30
cloudnullanyone around want to give https://review.openstack.org/#/c/573287/ and https://review.openstack.org/#/c/573284/ a push?14:33
*** udesale has joined #openstack-ansible14:38
*** smorant has joined #openstack-ansible14:38
*** fghaas has quit IRC14:38
jrossercloudnull: so it worked on fresh horizon containers if i hack the role to do the pip package installs14:39
cloudnullsweet!14:40
* jrosser shakes fist at spice -> gitlab breaking my venv build :(14:40
cloudnullthats been a gift ...14:40
*** smorant1 has quit IRC14:42
openstackgerritKevin Carter (cloudnull) proposed openstack/openstack-ansible master: [WIP] Add OpenStack Zun support to OSA  https://review.openstack.org/57140014:47
openstackgerritKevin Carter (cloudnull) proposed openstack/openstack-ansible master: Add OpenStack Zun support to OSA  https://review.openstack.org/57140014:47
*** vnogin has quit IRC14:48
guilhermespmorning guys14:50
guilhermespcloudnull: I saw some updates in elk_6x14:50
guilhermespnoticed something missing14:50
openstackgerritKevin Carter (cloudnull) proposed openstack/openstack-ansible master: Add OpenStack Zun support to OSA  https://review.openstack.org/57140014:50
cloudnullo/  guilhermesp14:50
*** pvradu has quit IRC14:50
guilhermesplet me show14:50
guilhermespjust a sec14:50
guilhermesp..templates/30-elasticsearch-output.conf.j2 were renamed to ..templates/99-elasticsearch-output.conf.j214:52
guilhermespbut not in installLogstash playbook14:52
guilhermesphttps://github.com/openstack/openstack-ansible-ops/blob/master/elk_metrics_6x/installLogstash.yml#L7714:52
guilhermespdo I open a pull request or someone else will do?14:52
*** pvradu has joined #openstack-ansible14:53
cloudnullguilhermesp you can for sure open a PR for that.14:53
cloudnullwe don't use github, its all in gerrit14:53
cloudnullhave you used that ?14:54
cloudnulld34dh0r53 ^14:54
guilhermespI used once to submitt a patch to os_monasca. But I can find help here with a colleague that does that often14:54
guilhermespuow, my nervous typing is attacking again14:55
guilhermespsubmit*14:55
cloudnullIf you want to push up a change itd be greatly appreciated.14:55
cloudnullonce its there, we can get it merged quickly14:56
guilhermespWill be a pleasure, but I will afk for lunch time, when I come back to the office I surelly can do that14:56
guilhermespI tell ya if I'd have any trouble with gerrit14:57
*** vnogin has joined #openstack-ansible14:57
cloudnull++14:58
cloudnullthanks guilhermesp and nice catch there.14:58
*** pvradu has quit IRC14:58
mnaserspotz has some really cool docs to get started with gerrit15:00
d34dh0r53doh15:00
mnaserwhich i should save15:00
*** pvradu has joined #openstack-ansible15:01
guilhermespwould be really helpful for future ref d34dh0r5315:02
*** shardy has quit IRC15:02
d34dh0r53awesome, thanks guilhermesp15:03
d34dh0r53let me know if you have any questions15:03
*** armaan has joined #openstack-ansible15:05
*** mma has joined #openstack-ansible15:07
*** BjoernT has joined #openstack-ansible15:07
*** vnogin has quit IRC15:10
*** mma has quit IRC15:11
spotzLatest version - https://github.com/spotz/vancouver18/blob/master/gitandgerrit.pdf15:16
spotzAnd of course the next live version will be Berlin15:18
jrossercloudnull: this is wierd, horizon partly works with osprofiler and elasticsearch pip packages installed in the venv15:20
cloudnulloh?15:21
jrosserbut there are very well defined parts that dont and blow up if you visit those panels15:21
cloudnullhum...15:21
jrosserhowever, just login to the container and "pip install elasticsearch osprofiler" and bingo the whole lot works ok15:21
* cloudnull rekicking the horizon parts of my cloud now 15:21
jrosserso it's an inside/outside the venv thing somehow15:21
jrosserso i rebuilt my repo venvs, redeployed the horizon containers15:22
cloudnullif you install those packages in the root namespace it works?15:22
jrosserseems so15:22
jrosseri couldn't visit "projects" for example15:23
cloudnullhum .15:24
cloudnullthat's quite odd.15:24
jrosserlet me just destory/recreate to be double sure15:25
*** dave-mccowan has joined #openstack-ansible15:25
*** smorant has quit IRC15:29
*** maybebuggy has quit IRC15:31
*** dave-mccowan has quit IRC15:32
jrossercloudnull: redeploying it all brings it back ok. Its a bit frustrating that this seems so very useful and at the same time strangely fragile15:32
*** mma has joined #openstack-ansible15:32
*** mma has quit IRC15:33
*** mma has joined #openstack-ansible15:33
cloudnullI would bet the developer panels are not well tested.15:33
*** jwitko has joined #openstack-ansible15:33
openstackgerritMarkos Chandras (hwoarang) proposed openstack/openstack-ansible-os_neutron master: Add support for using distribution packages for OpenStack services  https://review.openstack.org/57366115:34
cloudnullthe profile command seems to work ok, but workflow though horizon would be super easy to get into the hands of folks15:34
openstackgerritzhulingjie proposed openstack/openstack-ansible stable/pike: Fix the appropriate kernel modules path  https://review.openstack.org/57375015:34
jrosseri suspect not.... perhaps if you can confirm it comes up OK we're good - a deployer knows what they're getting into with enabling this stuff hopefully15:35
jrosserand this patch would enable someone with more clue on the innards of horizon to fix up the trace readback part too15:35
cloudnull++15:36
*** pvradu has quit IRC15:36
*** BjoernT has quit IRC15:40
* cloudnull was testing the various failure conditions 15:40
cloudnullseems to be working well15:41
jrosseri found it sometimes spuriously creates traces, particularly in the developer panels15:41
jrosserbut tbh that doesnt really matter15:41
*** lbragstad is now known as elbragstad15:44
*** vnogin has joined #openstack-ansible15:45
openstackgerritMarkos Chandras (hwoarang) proposed openstack/openstack-ansible-tests master: test-distro_install-vars: Add new var file for distro installations  https://review.openstack.org/57375415:45
*** jra has quit IRC15:45
openstackgerritMarkos Chandras (hwoarang) proposed openstack/openstack-ansible-tests master: test-distro_install-vars: Add new var file for distro installations  https://review.openstack.org/57375415:45
*** jra has joined #openstack-ansible15:46
openstackgerritMarkos Chandras (hwoarang) proposed openstack/openstack-ansible-os_keystone master: tox.ini: Use common vars file for distibution installations  https://review.openstack.org/57375815:48
*** jeferobles has joined #openstack-ansible16:15
cloudnulljrosser on one of my tests horizon is throwing a trace trying to talk to Request URL:https://10.127.83.101/identity/16:15
cloudnullwhen it should be trying to hit Request URL:https://10.127.83.101:5000/identity/16:16
*** armaan has quit IRC16:16
cloudnullwhich is strange, because that URL is not set anywhere.16:16
*** armaan has joined #openstack-ansible16:16
cloudnulland the service catalog has the port16:16
cloudnull| d6a328675ee745f39814b70dcae5685d | RegionOne | keystone     | identity       | True    | public    | https://10.127.83.101:5000                  |16:16
*** mma has quit IRC16:17
cloudnullI wonder if horizon is stripping the port somewhere...16:17
*** vnogin has quit IRC16:19
*** chyka has joined #openstack-ansible16:20
openstackgerritDavid Wilde proposed openstack/openstack-ansible-os_sahara master: Add trustee section by default  https://review.openstack.org/56988616:24
*** hassenius has quit IRC16:24
cloudnullan nevermind, its pebkac16:25
*** jra has quit IRC16:28
*** jra has joined #openstack-ansible16:29
*** nurdie has joined #openstack-ansible16:29
*** nurdie has quit IRC16:35
*** radeks_ has joined #openstack-ansible16:35
*** vnogin has joined #openstack-ansible16:36
*** vnogin has quit IRC16:36
*** Guest14735 is now known as sdake16:41
*** nurdie has joined #openstack-ansible16:43
*** jeferobles has quit IRC16:45
*** senk has quit IRC16:59
*** senk has joined #openstack-ansible16:59
*** pcaruana has quit IRC17:02
*** udesale has quit IRC17:03
openstackgerritJimmy McCrory proposed openstack/openstack-ansible-openstack_hosts master: Replace stat with file tests  https://review.openstack.org/57378717:08
*** radeks_ has quit IRC17:09
*** ericzolf has quit IRC17:10
*** chhagarw has quit IRC17:10
*** gkadam has quit IRC17:11
*** vnogin has joined #openstack-ansible17:13
*** vnogin has quit IRC17:17
*** jra has quit IRC17:17
*** jra has joined #openstack-ansible17:18
-openstackstatus- NOTICE: The Zuul scheduler was offline briefly to clean up from debugging a nodepool issue, so changes uploaded or approved between 16:50 and 17:15 UTC may need to be rechecked or reapproved (all already queued changes are in the process of being reenqueued now)17:23
*** exodusftw has quit IRC17:23
*** electrofelix has quit IRC17:27
*** exodusftw has joined #openstack-ansible17:27
openstackgerritGuilherme  Steinmuller Pimentel proposed openstack/openstack-ansible-ops master: Rename 30-elasticsearch-output.conf.j2  https://review.openstack.org/57379617:41
openstackgerritKevin Carter (cloudnull) proposed openstack/openstack-ansible-ops master: Add threadpool auto tune to the Elasticsearch cluster  https://review.openstack.org/57379717:43
cloudnulljrosser ^ that fixed my horizon traceback issues17:43
openstackgerritKevin Carter (cloudnull) proposed openstack/openstack-ansible-os_horizon master: Add developer panels to horizon  https://review.openstack.org/57331817:49
guilhermesphttps://github.com/openstack/openstack-ansible-ops/blob/3a2328af001770c9a7a4c791b568583c613fc7ad/elk_metrics_6x/installLogstash.yml#L9617:56
guilhermespcloudnull ^ what is the purpose of extras?17:56
guilhermespit is only in the playbook and nowhere else17:57
cloudnullguilhermesp i think that can be removed too17:58
guilhermespso it is breaking in "Logstash Extra Patterns" task17:58
guilhermespok17:58
openstackgerritMerged openstack/openstack-ansible-tests master: Add upgrade jobs for CentOS 7  https://review.openstack.org/57294518:10
andymccr^ nice!18:12
openstackgerritKevin Carter (cloudnull) proposed openstack/openstack-ansible-ops master: Set threadpool tuning and fstype  https://review.openstack.org/57379718:14
*** fghaas has joined #openstack-ansible18:21
*** NostawRm has quit IRC18:21
*** NostawRm has joined #openstack-ansible18:22
openstackgerritGuilherme  Steinmuller Pimentel proposed openstack/openstack-ansible-ops master: Fix installLogstash.yml  https://review.openstack.org/57379618:25
openstackgerritMerged openstack/openstack-ansible-os_designate master: Update to use oslo.messaging services for RPC and Notify  https://review.openstack.org/57281618:26
*** fghaas has left #openstack-ansible18:27
openstackgerritMerged openstack/openstack-ansible-tests master: Move database creation into role (nova)  https://review.openstack.org/57179018:30
openstackgerritKevin Carter (cloudnull) proposed openstack/openstack-ansible-os_horizon master: Add developer panels to horizon  https://review.openstack.org/57331818:31
openstackgerritNicolas Bock proposed openstack/openstack-ansible-tests stable/queens: Refactor hard-coded branch references  https://review.openstack.org/57381218:45
*** senk has quit IRC18:50
*** senk has joined #openstack-ansible18:51
*** jra has quit IRC18:54
*** jra has joined #openstack-ansible18:55
openstackgerritMerged openstack/openstack-ansible-os_aodh master: Update to use oslo.messaging services for RPC and Notify  https://review.openstack.org/56898918:57
jrossercloudnull: interesting.... what is the default number of threads?18:59
*** radeks has joined #openstack-ansible19:04
* jrosser reads, hmm the elasticsearch threads setup is quite something19:08
*** mmalchuk has joined #openstack-ansible19:16
*** mmalchuk_ has joined #openstack-ansible19:16
openstackgerritMerged openstack/openstack-ansible master: Move database creation into role (gnocchi)  https://review.openstack.org/57173819:20
openstackgerritMerged openstack/openstack-ansible stable/ocata: Add release note with workaround for spice-html5 repo change  https://review.openstack.org/57324319:20
*** mmalchuk has quit IRC19:20
*** mma has joined #openstack-ansible19:30
*** mmalchuk_ has quit IRC19:31
*** mmalchuk has joined #openstack-ansible19:31
*** armaan has quit IRC19:34
*** armaan has joined #openstack-ansible19:34
*** mma has quit IRC19:36
*** mma has joined #openstack-ansible19:37
*** hassenius has joined #openstack-ansible19:43
openstackgerritCam J. Loader proposed openstack/openstack-ansible-os_ironic master: Add glance auth information to ironic.conf for communication  https://review.openstack.org/57382219:46
cjloaderis there an osa os_ironic channel for irc?19:54
*** jra has quit IRC19:55
*** jra has joined #openstack-ansible19:55
mgariepyOSA stuff is all here cjloader19:58
cjloaderokay19:58
cjloaderty19:58
*** niraj_singh has joined #openstack-ansible20:02
*** nsingh has quit IRC20:04
*** mma has quit IRC20:04
*** nsingh has joined #openstack-ansible20:11
*** armaan has quit IRC20:12
*** armaan has joined #openstack-ansible20:13
*** mmalchuk has quit IRC20:15
*** niraj_singh has quit IRC20:15
*** vnogin has joined #openstack-ansible20:16
*** radeks has quit IRC20:21
cloudnulljrosser the thread count is the same as the number of cores .20:28
cloudnulland limits the queue to only 20020:28
cloudnullwhich is specific to bulk operations.20:29
cloudnulland osprofile uses bulk operations to record it's metrics20:29
*** nurdie has quit IRC20:30
*** armaan has quit IRC20:31
*** nurdie has joined #openstack-ansible20:31
*** armaan has joined #openstack-ansible20:31
*** nurdie has quit IRC20:35
*** throwsb1 has quit IRC20:40
*** vnogin has quit IRC20:47
*** jwitko has quit IRC21:02
*** mma has joined #openstack-ansible21:04
*** mma has quit IRC21:08
*** KeithMnemonic has quit IRC21:14
*** jwitko has joined #openstack-ansible21:30
openstackgerritMerged openstack/openstack-ansible stable/queens: Update os_tempest role SHA pin for cmd2 pin  https://review.openstack.org/57361521:30
*** vnogin has joined #openstack-ansible21:34
*** jra has quit IRC21:38
*** armaan has quit IRC21:41
*** jra has joined #openstack-ansible21:46
openstackgerritJake Briggs proposed openstack/openstack-ansible-haproxy_server master: Corrected haproxy template. Heeded to have a space between variable and value Also the variable name was incorrect.  https://review.openstack.org/57384321:50
*** nicolasbock has quit IRC22:06
*** nurdie has joined #openstack-ansible22:08
*** cjloader has quit IRC22:22
*** lbragstad has joined #openstack-ansible22:25
*** elbragstad has quit IRC22:25
openstackgerritMerged openstack/openstack-ansible stable/pike: Update os_tempest role SHA pin for cmd2 pin  https://review.openstack.org/57320722:40
openstackgerritMerged openstack/openstack-ansible stable/pike: Update repo_build role SHA for custom PyPi indexes fix  https://review.openstack.org/57321522:40
*** rodolof has quit IRC22:45
*** chyka has quit IRC23:01
*** jra has quit IRC23:35
*** jra has joined #openstack-ansible23:36
*** jra has quit IRC23:38
*** vnogin has quit IRC23:38
*** jra has joined #openstack-ansible23:38
*** dxiri has quit IRC23:39
*** dxiri has joined #openstack-ansible23:40

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