*** 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!