| *** xarses has quit IRC | 00:21 | |
| 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 | 00:45 |
| 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:10 |
| 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:11 |
| *** Kvisle has quit IRC | 01:16 | |
| *** Kvisle has joined #puppet-openstack | 01:17 | |
| *** xarses has joined #puppet-openstack | 01:31 | |
| *** xarses has quit IRC | 01:42 | |
| *** ilbot3 has quit IRC | 01:46 | |
| *** ilbot3 has joined #puppet-openstack | 01:55 | |
| *** rwsu has quit IRC | 02:40 | |
| *** iurygregory has quit IRC | 03:07 | |
| *** iurygregory has joined #puppet-openstack | 03:10 | |
| *** iurygregory has quit IRC | 03:34 | |
| *** iurygregory has joined #puppet-openstack | 03:40 | |
| openstackgerrit | Merged openstack/puppet-nova master: Expose log_outputs parameter for libvirt https://review.openstack.org/510544 | 03:53 |
| *** jaosorior has joined #puppet-openstack | 04:00 | |
| *** ykarel has joined #puppet-openstack | 04:00 | |
| *** etingof has quit IRC | 04:00 | |
| *** etingof has joined #puppet-openstack | 04:08 | |
| *** xarses has joined #puppet-openstack | 04:12 | |
| openstackgerrit | yatin proposed openstack/puppet-glance master: Use Keystone v3 credentials https://review.openstack.org/510735 | 04:43 |
| *** ratailor has joined #puppet-openstack | 04:46 | |
| *** ratailor_ has joined #puppet-openstack | 04:57 | |
| *** udesale has joined #puppet-openstack | 04:57 | |
| openstackgerrit | yatin proposed openstack/puppet-sahara master: Use Keystone v3 credentials https://review.openstack.org/510739 | 04:58 |
| *** ratailor has quit IRC | 05:00 | |
| *** rwsu has joined #puppet-openstack | 05:00 | |
| *** ratailor__ has joined #puppet-openstack | 05:02 | |
| openstackgerrit | Juan Antonio Osorio Robles proposed openstack/puppet-aodh master: Accept empty strings for log_dir https://review.openstack.org/510744 | 05:04 |
| *** jtomasek has joined #puppet-openstack | 05:04 | |
| *** ratailor_ has quit IRC | 05:05 | |
| openstackgerrit | Juan Antonio Osorio Robles proposed openstack/puppet-aodh master: Allow setting log files for apache vhost https://review.openstack.org/510745 | 05:07 |
| *** ykarel is now known as ykarel|afk | 05:11 | |
| *** skramaja has joined #puppet-openstack | 05:15 | |
| *** mwarad has joined #puppet-openstack | 05:26 | |
| *** jaosorior has quit IRC | 05:27 | |
| *** salmankhan has joined #puppet-openstack | 05:35 | |
| *** salmankhan has quit IRC | 05:40 | |
| *** etingof has quit IRC | 05:43 | |
| *** udesale__ has joined #puppet-openstack | 05:55 | |
| *** udesale__ has quit IRC | 05:57 | |
| *** udesale__ has joined #puppet-openstack | 05:57 | |
| *** udesale has quit IRC | 05:57 | |
| *** nmathew has joined #puppet-openstack | 05:59 | |
| *** jaosorior has joined #puppet-openstack | 06:03 | |
| *** yprokule has joined #puppet-openstack | 06:04 | |
| *** etingof has joined #puppet-openstack | 06:09 | |
| *** spectr has joined #puppet-openstack | 06:22 | |
| *** ykarel|afk is now known as ykarel | 06:38 | |
| *** tdb has quit IRC | 06:47 | |
| *** xarses has quit IRC | 06:51 | |
| *** tdb has joined #puppet-openstack | 06:54 | |
| *** rcernin has joined #puppet-openstack | 06:58 | |
| *** jaosorior has quit IRC | 07:04 | |
| *** ykarel has quit IRC | 07:05 | |
| *** nmathew has quit IRC | 07:07 | |
| *** nmathew has joined #puppet-openstack | 07:09 | |
| *** ykarel has joined #puppet-openstack | 07:12 | |
| *** jaganathan has joined #puppet-openstack | 07:27 | |
| *** tesseract has joined #puppet-openstack | 07:27 | |
| 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:28 |
| openstackgerrit | Merged openstack/puppet-zaqar master: Allow setting log files for apache vhost https://review.openstack.org/510073 | 07:29 |
| *** jaosorior has joined #puppet-openstack | 07:33 | |
| *** shardy has joined #puppet-openstack | 07:35 | |
| *** chem has joined #puppet-openstack | 07:40 | |
| *** jpena|off is now known as jpena | 07:41 | |
| openstackgerrit | Merged openstack/puppet-swift stable/newton: memcache_servers support in object-expirer and container-reconciler https://review.openstack.org/503060 | 07:43 |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/puppet-openstack-integration master: Updated from Puppet OpenStack modules constraints https://review.openstack.org/510778 | 07:45 |
| *** gfidente has joined #puppet-openstack | 07:47 | |
| *** gfidente has quit IRC | 07:47 | |
| *** gfidente has joined #puppet-openstack | 07:47 | |
| *** amoralej|off is now known as amoralej | 07:49 | |
| *** karthiks has quit IRC | 08:01 | |
| *** karthiks has joined #puppet-openstack | 08:18 | |
| *** derekh has joined #puppet-openstack | 08:36 | |
| openstackgerrit | Merged openstack/puppet-heat master: Allow setting log files for apache vhost https://review.openstack.org/510068 | 08:46 |
| *** ykarel is now known as ykarel|lunch | 08:48 | |
| *** ratailor_ has joined #puppet-openstack | 08:52 | |
| *** ratailor__ has quit IRC | 08:55 | |
| *** salmankhan has joined #puppet-openstack | 08:55 | |
| *** salmankhan has quit IRC | 09:00 | |
| *** karts has joined #puppet-openstack | 09:06 | |
| *** karthiks has quit IRC | 09:06 | |
| *** spectr has quit IRC | 09:29 | |
| *** ntpttr has quit IRC | 09:29 | |
| *** spectr has joined #puppet-openstack | 09:29 | |
| *** spectr has quit IRC | 09:32 | |
| *** salmankhan has joined #puppet-openstack | 09:34 | |
| *** spectr has joined #puppet-openstack | 09:34 | |
| *** nmathew- has joined #puppet-openstack | 09:41 | |
| *** rmart04 has joined #puppet-openstack | 09:42 | |
| *** nmathew has quit IRC | 09:44 | |
| *** mwarad has quit IRC | 09:48 | |
| *** dtantsur|afk is now known as dtantsur | 09:53 | |
| *** nmathew- has quit IRC | 10:01 | |
| *** ykarel|lunch is now known as ykarel | 10:02 | |
| 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:15 |
| *** morazi has quit IRC | 10:37 | |
| *** ratailor__ has joined #puppet-openstack | 10:52 | |
| *** ratailor_ has quit IRC | 10:55 | |
| *** ansmith has quit IRC | 11:01 | |
| *** dtantsur has quit IRC | 11:04 | |
| *** ykarel_ has joined #puppet-openstack | 11:06 | |
| *** ykarel has quit IRC | 11:08 | |
| *** dtantsur has joined #puppet-openstack | 11:08 | |
| *** dtantsur has quit IRC | 11:08 | |
| *** salmankhan has quit IRC | 11:08 | |
| *** dtantsur has joined #puppet-openstack | 11:09 | |
| *** dtantsur has quit IRC | 11:09 | |
| *** dtantsur has joined #puppet-openstack | 11:13 | |
| *** salmankhan has joined #puppet-openstack | 11:13 | |
| *** jpena is now known as jpena|lunch | 11:23 | |
| *** ykarel_ is now known as ykarel | 11:24 | |
| *** ratailor_ has joined #puppet-openstack | 11:37 | |
| *** ratailor__ has quit IRC | 11:41 | |
| *** morazi has joined #puppet-openstack | 11:51 | |
| *** trown|outtypewww is now known as trown | 11:55 | |
| *** ratailor__ has joined #puppet-openstack | 11:59 | |
| *** pradk has joined #puppet-openstack | 12:02 | |
| *** ratailor_ has quit IRC | 12:02 | |
| *** vnogin has joined #puppet-openstack | 12:03 | |
| *** karts is now known as karthiks | 12:15 | |
| *** ratailor_ has joined #puppet-openstack | 12:21 | |
| *** ratailor__ has quit IRC | 12:24 | |
| *** dprince has joined #puppet-openstack | 12:28 | |
| *** alee has quit IRC | 12:34 | |
| *** ratailor_ has quit IRC | 12:35 | |
| *** ansmith has joined #puppet-openstack | 12:35 | |
| *** makowals has joined #puppet-openstack | 12:40 | |
| *** jpena|lunch is now known as jpena | 12:40 | |
| *** iberezovskiy has quit IRC | 12:41 | |
| *** udesale__ has quit IRC | 12:44 | |
| *** spectr has quit IRC | 12:53 | |
| *** spectr has joined #puppet-openstack | 12:55 | |
| *** rmart04_ has joined #puppet-openstack | 13:00 | |
| *** shardy has quit IRC | 13:00 | |
| *** ykarel has quit IRC | 13:01 | |
| *** ykarel has joined #puppet-openstack | 13:02 | |
| *** rmart04 has quit IRC | 13:03 | |
| *** rmart04_ is now known as rmart04 | 13:03 | |
| openstackgerrit | Matthew J Black proposed openstack/puppet-horizon stable/ocata: Add parameter to configure instance defaults https://review.openstack.org/510874 | 13:07 |
| *** skramaja has quit IRC | 13:10 | |
| *** jistr is now known as jistr|mtg | 13:11 | |
| *** shardy has joined #puppet-openstack | 13:11 | |
| *** ykarel_ has joined #puppet-openstack | 13:11 | |
| *** ykarel has quit IRC | 13:12 | |
| *** ykarel_ is now known as ykarel | 13:12 | |
| *** indistylo has joined #puppet-openstack | 13:15 | |
| *** jistr|mtg is now known as jistr | 13:19 | |
| *** agurenko has joined #puppet-openstack | 13:21 | |
| *** rmart04 has quit IRC | 13:25 | |
| *** rmart04 has joined #puppet-openstack | 13:26 | |
| *** shardy has quit IRC | 13:29 | |
| *** ccamacho has quit IRC | 13:30 | |
| *** jaganathan has quit IRC | 13:33 | |
| *** rmart04_ has joined #puppet-openstack | 13:36 | |
| mnaser | amoralej left a review about the trove patch :P | 13:37 |
| *** shardy has joined #puppet-openstack | 13:38 | |
| *** rmart04 has quit IRC | 13:38 | |
| 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:39 |
| amoralej | maybe i did it before taking my first coffee | 13:40 |
| mnaser | i was just gonna say :p | 13:40 |
| *** rmart04 has joined #puppet-openstack | 13:41 | |
| *** rmart04_ has quit IRC | 13:42 | |
| *** rmart04_ has joined #puppet-openstack | 13:55 | |
| *** rmart04_ has quit IRC | 13:56 | |
| *** rmart04_ has joined #puppet-openstack | 13:56 | |
| *** rmart04 has quit IRC | 13:57 | |
| *** rmart04_ is now known as rmart04 | 13:57 | |
| *** archit has joined #puppet-openstack | 14:03 | |
| *** rmart04 has quit IRC | 14:05 | |
| *** rmart04 has joined #puppet-openstack | 14:10 | |
| *** atoth has joined #puppet-openstack | 14:11 | |
| openstackgerrit | Daniel Pawlik proposed openstack/puppet-openstack_extras master: Added OS_AUTH_TYPE parameter to auth file https://review.openstack.org/510899 | 14:14 |
| *** alee has joined #puppet-openstack | 14:15 | |
| openstackgerrit | Daniel Pawlik proposed openstack/puppet-openstack_extras master: Added OS_AUTH_TYPE parameter to auth file https://review.openstack.org/510899 | 14:18 |
| *** indistylo has quit IRC | 14:23 | |
| openstackgerrit | Daniel Pawlik proposed openstack/puppet-gnocchi master: Add support for Debian system https://review.openstack.org/508892 | 14:32 |
| *** rcernin has quit IRC | 14:49 | |
| *** xarses has joined #puppet-openstack | 14:56 | |
| *** ykarel is now known as ykarel|afk | 14:57 | |
| EmilienM | mwhahaha: if we ask for longer newton eol in tripleo, should we also ask for puppetopenstack? | 14:59 |
| 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:00 |
| mwhahaha | I mean really the biggest reason to keep it open is the long life upgrade stuff | 15:02 |
| *** rmart04 has quit IRC | 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:02 |
| 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:03 |
| * mnaser cant change topic but it should be changed | 15:04 | |
| mnaser | :> | 15:04 |
| *** mwhahaha sets mode: +o mnaser | 15:04 | |
| *** mwhahaha changes topic to "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:05 | |
| 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:06 |
| *** spectr has quit IRC | 15:07 | |
| *** vnogin has quit IRC | 15:10 | |
| *** yprokule has quit IRC | 15:10 | |
| amoralej | https://review.openstack.org/#/c/504443 has broken ceilometer tempest | 15:16 |
| *** ccamacho has joined #puppet-openstack | 15:17 | |
| mwhahaha | damn it | 15:17 |
| mwhahaha | why didn't afazekas look at the puppet failure | 15:18 |
| *** spectr has joined #puppet-openstack | 15:19 | |
| * mwhahaha wants to go back to bed | 15:21 | |
| EmilienM | mwhahaha, amoralej : let's just pin tempest | 15:27 |
| 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 has joined #puppet-openstack | 15:28 | |
| mnaser | if we disable ceilometer tempest won't that get rid of the integration autoscaling check? | 15:29 |
| 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:30 |
| *** udesale__ has joined #puppet-openstack | 15:31 | |
| *** shardy_ has joined #puppet-openstack | 15:32 | |
| 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 |
| *** udesale__ has quit IRC | 15:33 | |
| mwhahaha | sounds reasonable | 15:33 |
| *** vnogin has quit IRC | 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:33 | |
| 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:34 |
| mwhahaha | looks like they jsut moved the class | 15:35 |
| mwhahaha | so it might be a trivial fix | 15:35 |
| *** ccamacho has quit IRC | 15:37 | |
| pradk | k i think i see it, lemme try a patch | 15:37 |
| *** vnogin has joined #puppet-openstack | 15:37 | |
| *** salmankhan has quit IRC | 15:40 | |
| *** shardy_ has quit IRC | 15:40 | |
| *** salmankhan has joined #puppet-openstack | 15:40 | |
| *** rmart04 has joined #puppet-openstack | 15:49 | |
| *** ntpttr_laptop has joined #puppet-openstack | 15:50 | |
| *** ykarel|afk has quit IRC | 15:58 | |
| *** rmart04 has quit IRC | 15:58 | |
| pradk | amoralej, mwhahaha, https://review.openstack.org/510919 this should do it i think | 16:04 |
| mwhahaha | thanks | 16:05 |
| amoralej | pradk, is this issue blocking ceilometer gate also? | 16:05 |
| pradk | yes | 16:05 |
| *** indistylo has joined #puppet-openstack | 16:08 | |
| *** mnaser changes topic to "CI Status: YELLOWish (Tempest change broke scenaro001) | Puppet OpenStack modules https://docs.openstack.org/puppet-openstack-guide/latest/" | 16:26 | |
| openstackgerrit | Trygve Vea proposed openstack/puppet-neutron master: Deprecate parameters deprecated in Ocata https://review.openstack.org/510675 | 16:31 |
| *** salmankhan has quit IRC | 16:32 | |
| *** Guest66098 is now known as mgagne | 16:34 | |
| *** mgagne has quit IRC | 16:34 | |
| *** mgagne has joined #puppet-openstack | 16:34 | |
| *** salmankhan has joined #puppet-openstack | 16:34 | |
| *** salmankhan has quit IRC | 16:39 | |
| *** amorin has quit IRC | 16:40 | |
| *** hjensas has quit IRC | 16:43 | |
| *** shardy has quit IRC | 16:46 | |
| EmilienM | do we know how to remove fuel jobs? | 16:52 |
| *** tesseract has quit IRC | 16:53 | |
| *** derekh has quit IRC | 16:53 | |
| mnaser | EmilienM which ones? | 16:54 |
| 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:55 |
| 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:56 |
| 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:57 |
| 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:58 |
| 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 | 16:59 |
| pradk | got it | 17:00 |
| amoralej | what if i just disable ceilometer in tempest, will it work? | 17:00 |
| 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:01 |
| 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:02 |
| 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 |
| *** etingof has quit IRC | 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:04 |
| 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 |
| *** trown is now known as trown|lunch | 17:05 | |
| *** jpena is now known as jpena|off | 17:05 | |
| mnaser | oh | 17:05 |
| mnaser | one thing | 17:05 |
| 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:06 |
| 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:07 | |
| mwhahaha | your turn to fight that fight | 17:08 |
| * mwhahaha sighs and reminisces about this exact argument | 17:08 | |
| 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:09 |
| mwhahaha | since they just broke us and ceilometer in one fell swoop, it's a larger issue about gating in CI for tempest | 17:10 |
| *** etingof has 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:12 |
| mwhahaha | so just need to verify we don't break it when we switch to rpm | 17:13 |
| 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:14 |
| amoralej | we can make that if if [[ "${ZUUL_PROJECT}" != "openstack/tempest" ]] then TEMPEST_FROM_SOURCE=true | 17:16 |
| mwhahaha | right | 17:16 |
| *** rwsu has quit IRC | 17:16 | |
| mnaser | amoralej thats what i was thinking | 17:16 |
| 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:19 |
| *** rwsu has joined #puppet-openstack | 17:27 | |
| *** rwsu has quit IRC | 17:32 | |
| *** agurenko has quit IRC | 17:34 | |
| *** dtantsur is now known as dtantsur|afk | 17:38 | |
| *** vnogin has quit IRC | 17:42 | |
| *** ntpttr_laptop has quit IRC | 17:43 | |
| openstackgerrit | Alfredo Moralejo proposed openstack/puppet-openstack-integration master: Disable trove until integration with keystone v3 is fixed upstream https://review.openstack.org/510821 | 17:52 |
| 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:00 |
| *** trown|lunch is now known as trown | 18:05 | |
| *** jtomasek has quit IRC | 18:09 | |
| *** tdb has quit IRC | 18:12 | |
| *** tdb has joined #puppet-openstack | 18:12 | |
| *** amoralej is now known as amoralej|off | 18:13 | |
| *** guimaluf has joined #puppet-openstack | 18:27 | |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-nova master: Configure nova/ironic with domain parameters https://review.openstack.org/510992 | 18:59 |
| *** gfidente is now known as gfidente|afk | 18:59 | |
| *** rwsu has joined #puppet-openstack | 19:00 | |
| 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:04 |
| *** rcernin has joined #puppet-openstack | 19:16 | |
| mwhahaha | that's interesting | 19:25 |
| *** gfidente|afk has quit IRC | 19:39 | |
| 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:46 |
| *** archit has quit IRC | 19:47 | |
| *** vnogin has joined #puppet-openstack | 19:48 | |
| *** indistylo has quit IRC | 19:58 | |
| *** ntpttr_laptop has joined #puppet-openstack | 20:16 | |
| EmilienM | mnaser: looking at it | 20:29 |
| EmilienM | I think I have a patch | 20:30 |
| *** openstackgerrit has quit IRC | 20:33 | |
| *** openstackgerrit has 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:37 |
| 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 | 20:38 |
| *** dprince has quit IRC | 20:42 | |
| *** trown is now known as trown|outtypewww | 20:48 | |
| *** ansmith has quit IRC | 20:53 | |
| *** ansmith has joined #puppet-openstack | 21:09 | |
| *** morazi has quit IRC | 21:13 | |
| *** alee has quit IRC | 21:16 | |
| *** rcernin has quit IRC | 21:19 | |
| openstackgerrit | Lokesh Jain proposed openstack/puppet-horizon master: Added customization-module option to Horizon config https://review.openstack.org/511054 | 21:39 |
| *** ansmith has quit IRC | 22:03 | |
| *** vnogin has quit IRC | 22:20 | |
| *** vnogin has joined #puppet-openstack | 22:20 | |
| *** vnogin has quit IRC | 22:22 | |
| *** alee has joined #puppet-openstack | 22:23 | |
| 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 | 22:41 |
| *** xarses has quit IRC | 23:09 | |
| *** ansmith has joined #puppet-openstack | 23:14 | |
| *** ntpttr_laptop has quit IRC | 23:15 | |
| *** salmankhan has joined #puppet-openstack | 23:30 | |
| *** salmankhan has quit IRC | 23:35 | |
| openstackgerrit | Merged openstack/puppet-cinder master: Refactor Beaker jobs to use integration job classes https://review.openstack.org/508170 | 23:37 |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!