Perl 6 - the future is here, just unevenly distributed

IRC log for #puppet-openstack, 2017-10-10

| Channels | #puppet-openstack index | Today | | Search | Google Search | Plain-Text | summary

All times shown according to UTC.

Time Nick Message
00:45 openstackgerrit Merged openstack/puppet-keystone master: Add use_journal option for logging configuration  https://review.openstack.org/508500
00:45 openstackgerrit Merged openstack/puppet-openstack-integration master: Use --os-identity-api-version 3 explicitly in osc  https://review.openstack.org/509978
01:10 openstackgerrit Merged openstack/puppet-cinder master: Remove deprecated classes for setting up backends  https://review.openstack.org/510409
01:10 openstackgerrit Merged openstack/puppet-cinder master: Remove deprecated option from emc_vnx-backend  https://review.openstack.org/510406
01:11 openstackgerrit Merged openstack/puppet-cinder stable/ocata: Read os_region_name from cinder.conf when doing requests with the Openstack client  https://review.openstack.org/510447
01:11 openstackgerrit Merged openstack/puppet-cinder stable/newton: Read os_region_name from cinder.conf when doing requests with the Openstack client  https://review.openstack.org/510448
01:11 openstackgerrit Merged openstack/puppet-nova master: Remove some useless method  https://review.openstack.org/510279
01:17 Kvisle joined #puppet-openstack
01:31 xarses joined #puppet-openstack
01:55 ilbot3 joined #puppet-openstack
01:55 Topic for #puppet-openstack is now CI Status: YELLOWish - scenarios 1/4 failing (Most jobs are fixed for Zuulv3, but we could still have edge cases) | Puppet OpenStack modules https://docs.openstack.org/puppet-openstack-guide/latest/
03:10 iurygregory joined #puppet-openstack
03:40 iurygregory joined #puppet-openstack
03:53 openstackgerrit Merged openstack/puppet-nova master: Expose log_outputs parameter for libvirt  https://review.openstack.org/510544
04:00 jaosorior joined #puppet-openstack
04:00 ykarel joined #puppet-openstack
04:08 etingof joined #puppet-openstack
04:12 xarses joined #puppet-openstack
04:43 openstackgerrit yatin proposed openstack/puppet-glance master: Use Keystone v3 credentials  https://review.openstack.org/510735
04:46 ratailor joined #puppet-openstack
04:57 ratailor_ joined #puppet-openstack
04:57 udesale joined #puppet-openstack
04:58 openstackgerrit yatin proposed openstack/puppet-sahara master: Use Keystone v3 credentials  https://review.openstack.org/510739
05:00 rwsu joined #puppet-openstack
05:02 ratailor__ joined #puppet-openstack
05:04 openstackgerrit Juan Antonio Osorio Robles proposed openstack/puppet-aodh master: Accept empty strings for log_dir  https://review.openstack.org/510744
05:04 jtomasek joined #puppet-openstack
05:07 openstackgerrit Juan Antonio Osorio Robles proposed openstack/puppet-aodh master: Allow setting log files for apache vhost  https://review.openstack.org/510745
05:15 skramaja joined #puppet-openstack
05:26 mwarad joined #puppet-openstack
05:35 salmankhan joined #puppet-openstack
05:55 udesale__ joined #puppet-openstack
05:57 udesale__ joined #puppet-openstack
05:59 nmathew joined #puppet-openstack
06:03 jaosorior joined #puppet-openstack
06:04 yprokule joined #puppet-openstack
06:09 etingof joined #puppet-openstack
06:22 spectr joined #puppet-openstack
06:54 tdb joined #puppet-openstack
06:58 rcernin joined #puppet-openstack
07:09 nmathew joined #puppet-openstack
07:12 ykarel joined #puppet-openstack
07:27 jaganathan joined #puppet-openstack
07:27 tesseract joined #puppet-openstack
07:28 openstackgerrit Merged openstack/puppet-cinder stable/pike: Always set keymgr_api_class  https://review.openstack.org/510665
07:28 openstackgerrit Merged openstack/puppet-cinder stable/ocata: Always set keymgr_api_class  https://review.openstack.org/510666
07:28 openstackgerrit Merged openstack/puppet-cinder stable/newton: Always set keymgr_api_class  https://review.openstack.org/510667
07:29 openstackgerrit Merged openstack/puppet-zaqar master: Allow setting log files for apache vhost  https://review.openstack.org/510073
07:33 jaosorior joined #puppet-openstack
07:35 shardy joined #puppet-openstack
07:40 chem joined #puppet-openstack
07:43 openstackgerrit Merged openstack/puppet-swift stable/newton: memcache_servers support in object-expirer and container-reconciler  https://review.openstack.org/503060
07:45 openstackgerrit OpenStack Proposal Bot proposed openstack/puppet-openstack-integration master: Updated from Puppet OpenStack modules constraints  https://review.openstack.org/510778
07:47 gfidente joined #puppet-openstack
07:47 gfidente joined #puppet-openstack
08:18 karthiks joined #puppet-openstack
08:36 derekh joined #puppet-openstack
08:46 openstackgerrit Merged openstack/puppet-heat master: Allow setting log files for apache vhost  https://review.openstack.org/510068
08:52 ratailor_ joined #puppet-openstack
08:55 salmankhan joined #puppet-openstack
09:06 karts joined #puppet-openstack
09:29 spectr joined #puppet-openstack
09:34 salmankhan joined #puppet-openstack
09:34 spectr joined #puppet-openstack
09:41 nmathew- joined #puppet-openstack
09:42 rmart04 joined #puppet-openstack
10:15 openstackgerrit Alfredo Moralejo proposed openstack/puppet-openstack-integration master: Disable trove until integration with keystone v3 is fixed upstream  https://review.openstack.org/510821
10:52 ratailor__ joined #puppet-openstack
11:06 ykarel_ joined #puppet-openstack
11:08 dtantsur joined #puppet-openstack
11:09 dtantsur joined #puppet-openstack
11:13 dtantsur joined #puppet-openstack
11:13 salmankhan joined #puppet-openstack
11:37 ratailor_ joined #puppet-openstack
11:51 morazi joined #puppet-openstack
11:59 ratailor__ joined #puppet-openstack
12:02 pradk joined #puppet-openstack
12:03 vnogin joined #puppet-openstack
12:21 ratailor_ joined #puppet-openstack
12:28 dprince joined #puppet-openstack
12:35 ansmith joined #puppet-openstack
12:40 makowals joined #puppet-openstack
12:55 spectr joined #puppet-openstack
13:00 rmart04_ joined #puppet-openstack
13:02 ykarel joined #puppet-openstack
13:07 openstackgerrit Matthew J Black proposed openstack/puppet-horizon stable/ocata: Add parameter to configure instance defaults  https://review.openstack.org/510874
13:11 shardy joined #puppet-openstack
13:11 ykarel_ joined #puppet-openstack
13:15 indistylo joined #puppet-openstack
13:21 agurenko joined #puppet-openstack
13:26 rmart04 joined #puppet-openstack
13:36 rmart04_ joined #puppet-openstack
13:37 mnaser amoralej left a review about the trove patch :P
13:38 shardy joined #puppet-openstack
13:39 openstackgerrit Alfredo Moralejo proposed openstack/puppet-openstack-integration master: Disable trove until integration with keystone v3 is fixed upstream  https://review.openstack.org/510821
13:39 amoralej thanks mnaser :(
13:39 mnaser haha :P
13:40 amoralej maybe i did it before taking my first coffee
13:40 mnaser i was just gonna say :p
13:41 rmart04 joined #puppet-openstack
13:55 rmart04_ joined #puppet-openstack
13:56 rmart04_ joined #puppet-openstack
14:03 archit joined #puppet-openstack
14:10 rmart04 joined #puppet-openstack
14:11 atoth joined #puppet-openstack
14:14 openstackgerrit Daniel Pawlik proposed openstack/puppet-openstack_extras master: Added OS_AUTH_TYPE parameter to auth file  https://review.openstack.org/510899
14:15 alee joined #puppet-openstack
14:18 openstackgerrit Daniel Pawlik proposed openstack/puppet-openstack_extras master: Added OS_AUTH_TYPE parameter to auth file  https://review.openstack.org/510899
14:32 openstackgerrit Daniel Pawlik proposed openstack/puppet-gnocchi master: Add support for Debian system  https://review.openstack.org/508892
14:56 xarses joined #puppet-openstack
14:59 EmilienM mwhahaha: if we ask for longer newton eol in tripleo, should we also ask for puppetopenstack?
15:00 mwhahaha EmilienM: ask mnaser. do we think we'll be backporting puppet stuff?
15:00 EmilienM well
15:00 EmilienM we always have backports
15:02 mwhahaha I mean really the biggest reason to keep it open is the long life upgrade stuff
15:02 mnaser EmilienM mwhahaha if tripleo gets a newton eol extension i'm okay with puppet openstack maintaining stable/newton around a bit longer
15:02 mwhahaha So i guess it would be good to ask those guys if they foresee any puppet items that the'll need to back port
15:03 mnaser because i can imagine it being a giant pain if you cannot backport puppet changes
15:03 mwhahaha EmilienM: I guess that falls under what tonyb wants to figure out. I don't see it being too big of an issue to keep the puppet bits around a bit longer
15:03 mnaser but once tripleo gets newton eol i'd eol it right away
15:04 * mnaser cant change topic but it should be changed
15:04 mnaser :>
15:05 Topic for #puppet-openstack is now CI Status: GREENish (Most jobs are fixed for Zuulv3, but we could still have edge cases) | Puppet OpenStack modules https://docs.openstack.org/puppet-openstack-guide/latest/
15:05 mwhahaha mnaser: so we merged the tripleo ci fixes for v3, are we still seeing failures on those?
15:05 * mwhahaha hasn't looked yet
15:06 mnaser mwhahaha havent seen zuul report in a while, i think its queue is huuugely backed up
15:06 mnaser we'll find out on wednesday i guess
15:06 mnaser :p
15:06 mwhahaha k
15:16 amoralej https://review.openstack.org/#/c/504443 has broken ceilometer tempest
15:17 ccamacho joined #puppet-openstack
15:17 mwhahaha damn it
15:18 mwhahaha why didn't afazekas look at the puppet failure
15:19 spectr joined #puppet-openstack
15:21 * mwhahaha wants to go back to bed
15:27 EmilienM mwhahaha, amoralej : let's just pin tempest
15:28 mwhahaha or disable ceilometer tempest
15:28 EmilienM hum no
15:28 EmilienM I would rather pin tempest
15:28 amoralej the problem is that the pin will block promotion when a fix in ceilometer is pushed
15:28 amoralej unless the fix in ceilometer is backwards compatible
15:28 mwhahaha which it probably wont' be
15:28 EmilienM lol
15:28 amoralej yeah, that's my guess
15:28 vnogin joined #puppet-openstack
15:29 mnaser if we disable ceilometer tempest won't that get rid of the integration autoscaling check?
15:30 mwhahaha not sure which test is impacted
15:30 mwhahaha pradk: you aware of the ceilometer tempest issue -^
15:30 mwhahaha maybe we can prioritize the ceilometer fix
15:30 pradk mwhahaha, hmm lemme see
15:31 udesale__ joined #puppet-openstack
15:32 shardy_ joined #puppet-openstack
15:33 pradk amoralej, mwhahaha, hmm if we can temporarily pin tempest, we can look into fixing this in ceilo and then we can revert the ping and merge the ceilo fix?
15:33 mwhahaha sounds reasonable
15:33 pradk i'm fine either way
15:33 mwhahaha pradk: how is this not something that the qa folks would find themselves in their own testing?
15:33 * mwhahaha really hates it when this happens
15:34 amoralej the only problem is that, since we unpin tempest until we get a promotion, upstream gates will be blocked
15:34 amoralej iiuc
15:34 pradk yea not sure
15:34 pradk hmm if its a simple fix may be we just do that and fix ceilo
15:35 mwhahaha looks like they jsut moved the class
15:35 mwhahaha so it might be a trivial fix
15:37 pradk k i think i see it, lemme try a patch
15:37 vnogin joined #puppet-openstack
15:40 salmankhan joined #puppet-openstack
15:49 rmart04 joined #puppet-openstack
15:50 ntpttr_laptop joined #puppet-openstack
16:04 pradk amoralej, mwhahaha, https://review.openstack.org/510919 this should do it i think
16:05 mwhahaha thanks
16:05 amoralej pradk, is this issue blocking ceilometer gate also?
16:05 pradk yes
16:08 indistylo joined #puppet-openstack
16:26 Topic for #puppet-openstack is now CI Status: YELLOWish (Tempest change broke scenaro001) | Puppet OpenStack modules https://docs.openstack.org/puppet-openstack-guide/latest/
16:31 openstackgerrit Trygve Vea proposed openstack/puppet-neutron master: Deprecate parameters deprecated in Ocata  https://review.openstack.org/510675
16:34 mgagne joined #puppet-openstack
16:34 salmankhan joined #puppet-openstack
16:52 EmilienM do we know how to remove fuel jobs?
16:54 mnaser EmilienM which ones?
16:55 mnaser i asked for openstack-infra to remove them a while back and they did, so they shouldnt be voting
16:55 mnaser or even reporting
16:55 mnaser at least for puppet modules
16:56 amoralej mwhahaha, i just realized, although we get the ceilometer patch fixed upstream, we'll be unable to promote it until we get https://review.openstack.org/#/c/510821/ merged
16:56 EmilienM i still see them
16:56 amoralej so it's chicken-and-egg
16:56 EmilienM I think?
16:56 mwhahaha stupid chicken/egg
16:56 EmilienM mnaser: or maybe I saw an old patch
16:56 EmilienM mnaser: nevermind
16:56 mwhahaha amoralej: we can temp disable ceilometer to get that trove patch in
16:57 amoralej or we can pin tempest in the same patch
16:57 EmilienM yeah I guess we have no choice
16:57 mwhahaha amoralej: so if you line up p-o-i changes we can just merge them all at once
16:58 mnaser id disable ceilometer and trove in the same patch and then submit a patch enabling ceilometer back again
16:58 mnaser even though it'll get a -1, it'll be good for an open review to be infront of us
16:58 EmilienM yep
16:58 EmilienM ++
16:58 EmilienM and at the same time block patches in puppet modules who touch telemetry stuff
16:58 mwhahaha well ceilometer is going ot be fixed before trove
16:58 EmilienM since we don't test it
16:58 mwhahaha so i thought if we split those out
16:58 mwhahaha it would be better
16:59 mwhahaha patch disable ceilometer <- patch disable trove <- patch to renable ceilometer <- patch to renable trove
16:59 pradk https://review.openstack.org/510919 is about to merge
16:59 pradk should fix ceilo
16:59 mwhahaha pradk: yea but we can't get it packaged until we disable trove
16:59 mwhahaha and we can't merge disable trove until ceilometer fix is packaged :D
17:00 pradk got it
17:00 amoralej what if i just disable ceilometer in tempest, will it work?
17:01 EmilienM I'm still wondering why tempest isn't part of the promotion pipeline
17:01 EmilienM and why we deploy from master
17:01 amoralej https://github.com/openstack/puppet-openstack-integration/blob/master/fixtures/scenario001.pp#L87
17:01 mwhahaha amoralej: yea that's what i meant
17:01 amoralej EmilienM, because you didn't want to install tempest from rpm :)
17:01 amoralej we run tempest from rpm in all jobs in rdo-ci but in puppet-promotion pipeline
17:02 EmilienM amoralej: well, I changed my mind I guess
17:02 EmilienM we could change I guess
17:02 amoralej that'd fix it also
17:02 amoralej there is a parameter for that in run_tests.sh, iirc
17:02 amoralej i think i added it
17:04 amoralej EmilienM, mwhahaha, mnaser do you think it's better to move to use tempest from rpm in master?
17:04 amoralej i'm +1 with it
17:04 EmilienM I do
17:04 amoralej i can propose the patch instead of disable ceilo
17:04 mnaser there is TEMPEST_FROM_SOURCE
17:04 amoralej exactly
17:04 amoralej that's the var we change in RDO-CI
17:04 amoralej and it has been working since ocata times
17:04 mnaser honestly, if rdo runs tempest from rpm and promotes packages that are validated with rpm, we should run those
17:05 mnaser it would eliminate an issue like todays
17:05 mnaser we'll just blame amoralej if something goes wrong :D
17:05 amoralej :D
17:05 mnaser oh
17:05 mnaser one thing
17:06 mnaser we need to make sure that tempest jobs run with master
17:06 mnaser and p-o-i uses the rpms
17:06 mwhahaha so it moves the tempest changes to promotion though
17:06 mnaser because then when we run p-o-i in tempest gates.. it'll report a false result
17:07 mwhahaha is that better to move that problem to the promotion process or leave it as is
17:07 mnaser well.. if tempest isn't paying attention to our failures...
17:07 mnaser is there a benefit of doing this
17:07 * mnaser thinks as long as our stuff is non-voting for tempest, it'll likely be ignored again in the future
17:08 mwhahaha your turn to fight that fight
17:08 * mwhahaha sighs and reminisces about this exact argument
17:09 openstackgerrit Alfredo Moralejo proposed openstack/puppet-openstack-integration master: Switch tempest to be installed from RPM instead of source  https://review.openstack.org/510950
17:10 mwhahaha since they just broke us and ceilometer in one fell swoop, it's a larger issue about gating in CI for tempest
17:12 etingof joined #puppet-openstack
17:12 mnaser lets get packaged tempest in the gate
17:12 mnaser and what ill do post-zuulv3 cutover is
17:12 mnaser we'll run TEMPEST_FROM_SOURCE for tempest gates and hope they look out for these issues
17:12 mnaser but continue to install from RPM entirely in our gates
17:12 mwhahaha i think we already had that bit setup
17:13 mwhahaha so just need to verify we don't break it when we switch to rpm
17:14 mnaser mwhahaha https://github.com/openstack/puppet-openstack-integration/blob/master/run_tests.sh#L101-L129 only when TEMPEST_FROM_SOURCE was set to true
17:14 mnaser which it always was
17:14 mwhahaha right i meant the support for rpm installs
17:14 mwhahaha so it's just the logic around which repo that needs to be added
17:16 amoralej we can make that if if [[ "${ZUUL_PROJECT}" != "openstack/tempest" ]] then TEMPEST_FROM_SOURCE=true
17:16 mwhahaha right
17:16 mnaser amoralej thats what i was thinking
17:19 openstackgerrit Alfredo Moralejo proposed openstack/puppet-openstack-integration master: Switch tempest to be installed from RPM instead of source  https://review.openstack.org/510950
17:27 rwsu joined #puppet-openstack
17:52 openstackgerrit Alfredo Moralejo proposed openstack/puppet-openstack-integration master: Disable trove until integration with keystone v3 is fixed upstream  https://review.openstack.org/510821
18:00 mwhahaha pradk: btw the qa folks say that client file can probably get refactored since tempest provides a lot of that for you. jfyi if you want to create a tech-debt bug to take a look at it later
18:00 pradk cool
18:12 tdb joined #puppet-openstack
18:27 guimaluf joined #puppet-openstack
18:59 openstackgerrit Emilien Macchi proposed openstack/puppet-nova master: Configure nova/ironic with domain parameters  https://review.openstack.org/510992
19:00 rwsu joined #puppet-openstack
19:04 mnaser if someone has a moment to look over amoralej|off patch at unblocking gates
19:04 mnaser it failed in tempest
19:04 mnaser http://logs.openstack.org/50/510950/2/check/gate-puppet-openstack-integration-4-scenario004-tempest-centos-7/6a4b9c4/console.html#_2017-10-10_19_01_02_789832
19:04 mnaser we probably need to address permissions or use a different path if possible, i have to run now but i'd appreciat eif someone can take a moment to look a tit
19:04 mnaser if not ill do it later
19:16 rcernin joined #puppet-openstack
19:25 mwhahaha that's interesting
19:46 openstackgerrit John Fulton proposed openstack/puppet-ceph master: Update Mon profile to not declare Ceph keys if already defined  https://review.openstack.org/511001
19:48 vnogin joined #puppet-openstack
20:16 ntpttr_laptop joined #puppet-openstack
20:29 EmilienM mnaser: looking at it
20:30 EmilienM I think I have a patch
20:37 openstackgerrit joined #puppet-openstack
20:37 openstackgerrit Emilien Macchi proposed openstack/puppet-openstack-integration master: run_tests/rpm: manage /tmp/openstack/tempest  https://review.openstack.org/511024
20:38 EmilienM maybe that would be enough ^ otherwise i'll look why not
20:38 openstackgerrit Emilien Macchi proposed openstack/puppet-openstack-integration master: Disable trove until integration with keystone v3 is fixed upstream  https://review.openstack.org/510821
21:09 ansmith joined #puppet-openstack
21:39 openstackgerrit Lokesh Jain proposed openstack/puppet-horizon master: Added customization-module option to Horizon config  https://review.openstack.org/511054
22:20 vnogin joined #puppet-openstack
22:23 alee joined #puppet-openstack
22:41 mnaser EmilienM i think you'll have to squash your change with amoralej|off
22:41 mnaser because it just tried to install from source in your change in CI
22:41 mnaser http://logs.openstack.org/24/511024/1/check/gate-puppet-openstack-integration-4-scenario001-tempest-centos-7/1404eda/console.html
23:14 ansmith joined #puppet-openstack
23:30 salmankhan joined #puppet-openstack
23:37 openstackgerrit Merged openstack/puppet-cinder master: Refactor Beaker jobs to use integration job classes  https://review.openstack.org/508170

| Channels | #puppet-openstack index | Today | | Search | Google Search | Plain-Text | summary