*** ducttape_ has joined #puppet-openstack | 00:07 | |
*** saneax is now known as saneax-_-|AFK | 00:14 | |
*** xarses has joined #puppet-openstack | 00:33 | |
*** ducttape_ has quit IRC | 00:46 | |
*** hjensas has joined #puppet-openstack | 00:48 | |
openstackgerrit | Merged openstack/puppet-gnocchi: Use stable/newton spec helper https://review.openstack.org/378810 | 00:56 |
---|---|---|
*** markvoelker has joined #puppet-openstack | 01:13 | |
openstackgerrit | Adam Vinsh proposed openstack/puppet-swift: Replace all proxy template code with ini provider https://review.openstack.org/378950 | 01:42 |
*** Vinsh has quit IRC | 01:44 | |
*** ilbot3 has quit IRC | 01:46 | |
*** ducttape_ has joined #puppet-openstack | 01:46 | |
*** ilbot3 has joined #puppet-openstack | 01:48 | |
*** ducttape_ has quit IRC | 01:52 | |
*** jayg|g0n3 has quit IRC | 02:00 | |
*** jayg|g0n3 has joined #puppet-openstack | 02:00 | |
*** radez has quit IRC | 02:00 | |
*** radez has joined #puppet-openstack | 02:00 | |
*** ducttape_ has joined #puppet-openstack | 02:16 | |
*** ducttape_ has quit IRC | 02:42 | |
*** Vinsh has joined #puppet-openstack | 03:03 | |
*** Vinsh has quit IRC | 03:12 | |
openstackgerrit | Merged openstack/puppet-glance: Use stable/newton spec helper https://review.openstack.org/378809 | 03:19 |
openstackgerrit | zhongshengping proposed openstack/puppet-cinder: Deprecate backup_topic option https://review.openstack.org/376207 | 03:35 |
*** mdorman_ has quit IRC | 03:37 | |
*** beagles has quit IRC | 03:47 | |
*** dtantsur has quit IRC | 03:47 | |
openstackgerrit | Merged openstack/puppet-vswitch: Use stable/newton spec helper https://review.openstack.org/378836 | 03:50 |
*** mdorman has joined #puppet-openstack | 03:51 | |
*** petems has quit IRC | 03:52 | |
*** petems has joined #puppet-openstack | 03:53 | |
*** delattec has joined #puppet-openstack | 03:53 | |
*** dtantsur has joined #puppet-openstack | 03:54 | |
*** b3nt_pin has joined #puppet-openstack | 03:54 | |
*** b3nt_pin is now known as Guest17228 | 03:54 | |
*** cdelatte has quit IRC | 03:56 | |
openstackgerrit | David Moreau Simard proposed openstack/puppet-openstack-integration: Update SSL certificates with SubjectAltNames https://review.openstack.org/379008 | 04:20 |
openstackgerrit | David Moreau Simard proposed openstack/puppet-openstack-integration: Update SSL certificates with SubjectAltNames https://review.openstack.org/379009 | 04:22 |
openstackgerrit | Sharat Sharma proposed openstack/puppet-mistral: Added retries for db_sync https://review.openstack.org/379148 | 05:32 |
*** richm has quit IRC | 05:40 | |
openstackgerrit | Sharat Sharma proposed openstack/puppet-mistral: Added retries for db_sync https://review.openstack.org/379148 | 05:58 |
openstackgerrit | Sharat Sharma proposed openstack/puppet-aodh: Added retries for db_sync https://review.openstack.org/379166 | 06:08 |
openstackgerrit | Sharat Sharma proposed openstack/puppet-barbican: Added retries for db_sync https://review.openstack.org/379169 | 06:11 |
*** hjensas has joined #puppet-openstack | 06:13 | |
*** hjensas has quit IRC | 06:13 | |
*** hjensas has joined #puppet-openstack | 06:13 | |
openstackgerrit | Sharat Sharma proposed openstack/puppet-ceilometer: Added retries for db_sync https://review.openstack.org/379170 | 06:15 |
openstackgerrit | Sharat Sharma proposed openstack/puppet-cinder: Added retries for db_sync https://review.openstack.org/379171 | 06:19 |
*** rcernin has joined #puppet-openstack | 06:22 | |
openstackgerrit | Sharat Sharma proposed openstack/puppet-designate: Added retries for db_sync https://review.openstack.org/379172 | 06:23 |
openstackgerrit | Sharat Sharma proposed openstack/puppet-ec2api: Added retries for db_sync https://review.openstack.org/379176 | 06:28 |
*** saneax-_-|AFK is now known as saneax | 06:33 | |
openstackgerrit | Sharat Sharma proposed openstack/puppet-glance: Added retries for db_sync https://review.openstack.org/379183 | 06:43 |
*** leanderthal|afk is now known as leanderthal | 06:43 | |
openstackgerrit | Sharat Sharma proposed openstack/puppet-ec2api: Added retries for db_sync https://review.openstack.org/379176 | 06:47 |
openstackgerrit | Sharat Sharma proposed openstack/puppet-mistral: Added retries for db_sync https://review.openstack.org/379148 | 06:47 |
openstackgerrit | Sharat Sharma proposed openstack/puppet-designate: Added retries for db_sync https://review.openstack.org/379172 | 06:51 |
*** rwsu has joined #puppet-openstack | 06:52 | |
openstackgerrit | Sharat Sharma proposed openstack/puppet-gnocchi: Added retries for db_sync https://review.openstack.org/379196 | 06:53 |
openstackgerrit | Sharat Sharma proposed openstack/puppet-aodh: Added retries for db_sync https://review.openstack.org/379166 | 06:57 |
openstackgerrit | Sharat Sharma proposed openstack/puppet-heat: Added retries for db_sync https://review.openstack.org/379202 | 06:57 |
*** jaosorior has joined #puppet-openstack | 06:58 | |
openstackgerrit | Sharat Sharma proposed openstack/puppet-ironic: Added retries for db_sync https://review.openstack.org/379206 | 07:00 |
openstackgerrit | Sharat Sharma proposed openstack/puppet-ceilometer: Added retries for db_sync https://review.openstack.org/379170 | 07:00 |
openstackgerrit | Sharat Sharma proposed openstack/puppet-keystone: Added retries for db_sync https://review.openstack.org/379209 | 07:03 |
openstackgerrit | Sharat Sharma proposed openstack/puppet-magnum: Added retries for db_sync https://review.openstack.org/379210 | 07:06 |
*** amoralej|off is now known as amoralej | 07:06 | |
openstackgerrit | Sharat Sharma proposed openstack/puppet-cinder: Added retries for db_sync https://review.openstack.org/379171 | 07:07 |
openstackgerrit | Sharat Sharma proposed openstack/puppet-manila: Added retries for db_sync https://review.openstack.org/379211 | 07:08 |
openstackgerrit | Sharat Sharma proposed openstack/puppet-murano: Added retries for db_sync https://review.openstack.org/379212 | 07:10 |
openstackgerrit | Sharat Sharma proposed openstack/puppet-neutron: Added retries for db_sync https://review.openstack.org/379214 | 07:13 |
openstackgerrit | Sharat Sharma proposed openstack/puppet-nova: Added retries for db_sync https://review.openstack.org/379217 | 07:16 |
openstackgerrit | Sharat Sharma proposed openstack/puppet-glance: Added retries for db_sync https://review.openstack.org/379183 | 07:17 |
openstackgerrit | Sharat Sharma proposed openstack/puppet-octavia: Added retries for db_sync https://review.openstack.org/379223 | 07:19 |
openstackgerrit | Sharat Sharma proposed openstack/puppet-panko: Added retries for db_sync https://review.openstack.org/379225 | 07:22 |
openstackgerrit | Carlos Camacho proposed openstack/puppet-gnocchi: Remove unnecesary spaces in log messages. https://review.openstack.org/376254 | 07:22 |
openstackgerrit | Sharat Sharma proposed openstack/puppet-rally: Added retries for db_sync https://review.openstack.org/379226 | 07:24 |
openstackgerrit | Sharat Sharma proposed openstack/puppet-sahara: Added retries for db_sync https://review.openstack.org/379227 | 07:25 |
openstackgerrit | Sharat Sharma proposed openstack/puppet-trove: Added retries for db_sync https://review.openstack.org/379228 | 07:27 |
openstackgerrit | Sharat Sharma proposed openstack/puppet-vitrage: Added retries for db_sync https://review.openstack.org/379231 | 07:30 |
openstackgerrit | Sharat Sharma proposed openstack/puppet-zaqar: Added retries for db_sync https://review.openstack.org/379232 | 07:33 |
*** gfidente has joined #puppet-openstack | 07:34 | |
*** jpich has joined #puppet-openstack | 07:48 | |
*** paramite has joined #puppet-openstack | 07:49 | |
*** b00tcat` has joined #puppet-openstack | 08:11 | |
openstackgerrit | OpenStack Proposal Bot proposed openstack/puppet-openstack-integration: Promote RDO repository to latest consistent URL https://review.openstack.org/379264 | 08:23 |
*** dgurtner has joined #puppet-openstack | 08:25 | |
*** dgurtner has quit IRC | 08:25 | |
*** dgurtner has joined #puppet-openstack | 08:25 | |
*** derekh has joined #puppet-openstack | 08:38 | |
*** guimaluf has quit IRC | 08:50 | |
*** jpena|off is now known as jpena | 08:57 | |
openstackgerrit | Sharat Sharma proposed openstack/puppet-congress: Added retries for db_sync https://review.openstack.org/379291 | 09:03 |
*** jaosorior is now known as jaosorior_lunch | 09:05 | |
openstackgerrit | Sharat Sharma proposed openstack/puppet-glare: Added retries for db_sync https://review.openstack.org/379297 | 09:10 |
openstackgerrit | Sharat Sharma proposed openstack/puppet-keystone: Added retries for db_sync https://review.openstack.org/379209 | 09:12 |
openstackgerrit | Sharat Sharma proposed openstack/puppet-vitrage: Added retries for db_sync https://review.openstack.org/379231 | 09:13 |
*** hjensas has quit IRC | 09:39 | |
*** hjensas has joined #puppet-openstack | 09:51 | |
*** hjensas has joined #puppet-openstack | 09:51 | |
*** rmart04 has joined #puppet-openstack | 09:54 | |
*** richm has joined #puppet-openstack | 10:10 | |
*** hjensas has quit IRC | 10:15 | |
*** ayoung has quit IRC | 10:27 | |
*** ayoung has joined #puppet-openstack | 10:40 | |
*** morazi has joined #puppet-openstack | 10:58 | |
openstackgerrit | Merged openstack/puppet-openstack-integration: Promote RDO repository to latest consistent URL https://review.openstack.org/379264 | 11:01 |
*** _degorenko is now known as degorenko | 11:02 | |
*** jaosorior_lunch is now known as jaosorior | 11:02 | |
*** hjensas has joined #puppet-openstack | 11:05 | |
*** dprince has joined #puppet-openstack | 11:06 | |
*** hjensas has quit IRC | 11:16 | |
*** shardy has joined #puppet-openstack | 11:18 | |
*** ccamacho is now known as ccamacho|lunch | 11:18 | |
*** openstackgerrit has quit IRC | 11:19 | |
*** openstackgerrit has joined #puppet-openstack | 11:19 | |
*** hjensas has joined #puppet-openstack | 11:19 | |
*** dtantsur has quit IRC | 11:29 | |
*** dgurtner has quit IRC | 11:30 | |
*** dgurtner has joined #puppet-openstack | 11:31 | |
*** dgurtner has quit IRC | 11:31 | |
*** dgurtner has joined #puppet-openstack | 11:31 | |
colonwq | EmilienM or anyone awake, Can you help with this issue: http://logs.openstack.org/06/339106/49/experimental/gate-puppet-openstack-integration-4-scenario004-tempest-centos-7-nv/44602a5/console.html#_2016-09-28_23_47_00_358547 | 11:32 |
*** lucas-afk is now known as lucasagomes | 11:33 | |
*** johnmilton has quit IRC | 11:35 | |
*** jpena is now known as jpena|lunch | 11:43 | |
*** furlongm has quit IRC | 11:46 | |
*** furlongm has joined #puppet-openstack | 11:46 | |
*** furlongm has quit IRC | 11:50 | |
*** furlongm has joined #puppet-openstack | 11:50 | |
*** Hatsjoe has quit IRC | 11:54 | |
*** Hatsjoe has joined #puppet-openstack | 11:54 | |
*** Hatsjoe has joined #puppet-openstack | 11:54 | |
*** amoralej is now known as amoralej|lunch | 11:59 | |
*** ducttape_ has joined #puppet-openstack | 12:03 | |
*** ducttape_ has quit IRC | 12:04 | |
*** ducttape_ has joined #puppet-openstack | 12:04 | |
*** johnmilton has joined #puppet-openstack | 12:06 | |
*** johnmilton has quit IRC | 12:06 | |
*** johnmilton has joined #puppet-openstack | 12:07 | |
*** jayg|g0n3 is now known as jayg | 12:09 | |
*** guimaluf has joined #puppet-openstack | 12:11 | |
EmilienM | morning | 12:12 |
colonwq | EmilienM, thanks for the feedback on 377158. I thought I had flipped the right bits to run the tempest test. | 12:15 |
*** ccamacho|lunch is now known as ccamacho | 12:15 | |
EmilienM | colonwq: did you checked in logs? | 12:15 |
EmilienM | colonwq: always look at logs, that's why we have them | 12:15 |
colonwq | EmilienM, I am down in the tempest section now. | 12:16 |
openstackgerrit | Maciej Relewicz proposed openstack/puppet-glare: Update main glare class https://review.openstack.org/373472 | 12:18 |
*** trown|outtypewww is now known as trown | 12:18 | |
colonwq | ok. let me go learn more about tempest. I missed something. | 12:19 |
*** fultonj has joined #puppet-openstack | 12:23 | |
*** Guest17228 is now known as beagles | 12:27 | |
openstackgerrit | Merged openstack/puppet-openstack-integration: Update SSL certificates with SubjectAltNames https://review.openstack.org/379008 | 12:32 |
openstackgerrit | Merged openstack/puppet-openstack-integration: Update SSL certificates with SubjectAltNames https://review.openstack.org/379009 | 12:32 |
*** ducttape_ has quit IRC | 12:35 | |
openstackgerrit | Juan Antonio Osorio Robles proposed openstack/puppet-tripleo: Enable TLS in the internal network for gnocchi https://review.openstack.org/377645 | 12:37 |
openstackgerrit | Juan Antonio Osorio Robles proposed openstack/puppet-tripleo: Enable TLS in the internal network for ceilometer https://review.openstack.org/377644 | 12:37 |
openstackgerrit | Juan Antonio Osorio Robles proposed openstack/puppet-tripleo: Enable TLS in the internal network for aodh https://review.openstack.org/365457 | 12:37 |
openstackgerrit | Juan Antonio Osorio Robles proposed openstack/puppet-tripleo: Enable TLS in the internal network for keystone https://review.openstack.org/327029 | 12:37 |
openstackgerrit | Juan Antonio Osorio Robles proposed openstack/puppet-tripleo: Enable internal TLS for MysSQL https://review.openstack.org/378471 | 12:37 |
*** dtantsur has joined #puppet-openstack | 12:38 | |
EmilienM | dmsimard: can you backport it to newton too? (ssl) | 12:38 |
EmilienM | degorenko: can you look https://review.openstack.org/#/c/375968/ please? | 12:43 |
openstackgerrit | David Moreau Simard proposed openstack/puppet-openstack-integration: Update SSL certificates with SubjectAltNames https://review.openstack.org/379434 | 12:44 |
openstackgerrit | Emilien Macchi proposed openstack/puppet-openstack-integration: Gemfile: checkout spec_helper to stable/mitaka https://review.openstack.org/378580 | 12:45 |
degorenko | EmilienM, alex was first while i reviewed this patch :D but anyway +2 | 12:46 |
degorenko | mwhahaha, are we going to add os_workers everywhere? | 12:47 |
mwhahaha | Yea | 12:47 |
iurygregory | morning | 12:47 |
*** shardy has quit IRC | 12:48 | |
mwhahaha | Hadn't planned for Newton but definitely for ocata | 12:48 |
EmilienM | cool ^ | 12:48 |
EmilienM | can I have reviews on https://review.openstack.org/#/c/378986/ and https://review.openstack.org/#/c/378989/ please? | 12:48 |
EmilienM | I'll have to backport https://review.openstack.org/#/c/378986/ to stable/newton | 12:49 |
mwhahaha | I got a spreadsheet to track some work for ocata like the os_workers thing | 12:49 |
dmsimard | EmilienM: yeah was just waiting on master merge | 12:56 |
EmilienM | mwhahaha: can you share the link please? | 12:57 |
*** ducttape_ has joined #puppet-openstack | 12:57 | |
*** jpena|lunch is now known as jpena | 12:59 | |
*** ducttape_ has quit IRC | 13:05 | |
openstackgerrit | David Moreau Simard proposed openstack/puppet-openstack-integration: [mitaka] bump tempest to 12.2.0 https://review.openstack.org/378920 | 13:10 |
*** jtriley has joined #puppet-openstack | 13:13 | |
*** jperry has joined #puppet-openstack | 13:13 | |
*** jaosorior has quit IRC | 13:21 | |
*** jaosorior has joined #puppet-openstack | 13:22 | |
*** amoralej|lunch is now known as amoralej | 13:22 | |
EmilienM | dmsimard: it doesn't work | 13:23 |
EmilienM | dmsimard: well maybe with your patch? | 13:23 |
openstackgerrit | Mykyta Karpin proposed openstack/puppet-nova: [WIP] Add class for migration options configuration in qemu.conf https://review.openstack.org/378754 | 13:23 |
dmsimard | EmilienM: yeah, I rebased because the tempest failures are SSL related http://logs.openstack.org/20/378920/1/check/gate-puppet-openstack-integration-3-scenario002-tempest-centos-7/3ca4af0/console.html#_2016-09-28_20_07_37_086011 | 13:23 |
EmilienM | you rocks | 13:24 |
dmsimard | I didn't blindly rebase, you know :p | 13:24 |
dmsimard | FWIW, Ubuntu has the same errors so SSL is enabled on Mitaka I guess | 13:24 |
dmsimard | do you remember why it was disabled for newton ? | 13:24 |
dmsimard | let's try and re-enable it and see what breaks | 13:25 |
EmilienM | yeah, some self signed related stuff | 13:25 |
EmilienM | I didn't investigate much | 13:25 |
dmsimard | oh so maybe I fixed that too then | 13:25 |
dmsimard | EmilienM: it also looks like Ubuntu has gnocchi for newton after all | 13:27 |
dmsimard | so I'll try and re-enable that too. | 13:27 |
EmilienM | dmsimard: in separated patches | 13:28 |
EmilienM | for ssl I mean | 13:28 |
EmilienM | dmsimard: if you don't enable gnocchi on ubuntu I won't be unhappy :) (they don't test much their stuffs tbh) | 13:29 |
openstackgerrit | David Moreau Simard proposed openstack/puppet-openstack-integration: Re-enable SSL testing on Ubuntu https://review.openstack.org/379463 | 13:29 |
dmsimard | EmilienM: eh let's see | 13:30 |
openstackgerrit | David Moreau Simard proposed openstack/puppet-openstack-integration: Re-enable Gnocchi testing on Ubuntu https://review.openstack.org/379466 | 13:31 |
mwhahaha | https://docs.google.com/a/next-development.com/spreadsheets/d/1GV4v-T8E7Sq6GDE12pcTk_i0hFaVrDGnIDDVikXrQ34/edit?usp=sharing spreadsheet for some of the ocata cleanup work | 13:34 |
openstackgerrit | David Moreau Simard proposed openstack/puppet-openstack-integration: Re-enable Mistral testing https://review.openstack.org/339813 | 13:34 |
dmsimard | EmilienM: that mistral is a rebase of your patch ^ | 13:34 |
openstackgerrit | Merged openstack/puppet-gnocchi: Default metricd workers to a safer value https://review.openstack.org/375968 | 13:34 |
openstackgerrit | Merged openstack/puppet-openstack_extras: repos: disable EPEL by default https://review.openstack.org/378989 | 13:34 |
EmilienM | oh no mistral | 13:34 |
EmilienM | mistral tempest tests are really unstable | 13:34 |
dmsimard | Eh, I rebased it anyway, let's see | 13:34 |
EmilienM | their gate isn't even voting on their own tests | 13:35 |
dmsimard | nice | 13:35 |
dmsimard | did you scream at them ? :P | 13:35 |
EmilienM | of course dude | 13:35 |
EmilienM | I did something better | 13:35 |
EmilienM | I added tripleo multinode job (voting) in their gate | 13:35 |
EmilienM | they realized it after a week lol | 13:36 |
dmsimard | you did that without asking? lol | 13:36 |
dmsimard | EmilienM: see, tempest bump worked https://review.openstack.org/#/c/378920/ | 13:36 |
EmilienM | nice | 13:37 |
EmilienM | I'll let core people to review | 13:37 |
dmsimard | too late | 13:37 |
dmsimard | already added the army | 13:37 |
dmsimard | do we have a tag for newton yet ? | 13:38 |
dmsimard | just 12.2.0 too I guess ? | 13:38 |
EmilienM | for now, let's keep on master | 13:39 |
EmilienM | mtreinish: do we plan to have a tempest tag for newton? | 13:39 |
dmsimard | sounds like we need a 13 since I guess 12 is for mitaka ? /me shrugs | 13:39 |
openstackgerrit | David Moreau Simard proposed openstack/puppet-openstack-integration: Re-enable SSL testing on Ubuntu https://review.openstack.org/379463 | 13:40 |
dmsimard | EmilienM: hey I think that linuxbridge issue was fixed in Ubuntu, right ? | 13:41 |
mtreinish | EmilienM: yes tempest-13 will be tagged at the relase | 13:41 |
dmsimard | mtreinish: cool thanks | 13:41 |
mtreinish | dmsimard: http://docs.openstack.org/developer/tempest/overview.html#release-versioning | 13:41 |
EmilienM | awesome | 13:41 |
EmilienM | mtreinish++ | 13:42 |
openstackgerrit | David Moreau Simard proposed openstack/puppet-openstack-integration: Re-enable linuxbridge testing on Ubuntu https://review.openstack.org/379472 | 13:42 |
amoralej | hi puppeteers, | 13:42 |
amoralej | i have a question regarding new lbaasv2 support | 13:43 |
*** ducttape_ has joined #puppet-openstack | 13:43 | |
EmilienM | amoralej: we have a specialist! mjblack | 13:43 |
amoralej | in order to enable it in horizon, we have to install a new package | 13:43 |
openstackgerrit | David Moreau Simard proposed openstack/puppet-openstack-integration: Re-enable trove testing on Ubuntu https://review.openstack.org/379474 | 13:43 |
mjblack | amoralej: lol in newton? | 13:43 |
mjblack | mitaka the lbaas v2 dashboard wasnt "ready" | 13:44 |
amoralej | i think since mitaka | 13:44 |
amoralej | https://github.com/openstack/neutron-lbaas-dashboard | 13:44 |
amoralej | so, will not release a new version in newton? | 13:45 |
mjblack | I dont think UCA has packages for it | 13:46 |
mjblack | and I havent seen if it is ready | 13:46 |
mjblack | but | 13:46 |
mjblack | that raises an interesting issue... | 13:46 |
mjblack | EmilienM: we probably should add to the puppet-horizon module a way to install custom dashboards | 13:47 |
openstackgerrit | David Moreau Simard proposed openstack/puppet-openstacklib: Add a refreshonly parameter to service_validation https://review.openstack.org/379476 | 13:47 |
amoralej | that was my question, if manage it through puppet-horizon | 13:47 |
amoralej | or in puppet-neutron (currently puppet-murano can install murano-dashboard, i.e.) | 13:47 |
EmilienM | mwhahaha, degorenko: wdyt https://review.openstack.org/379166 ? | 13:48 |
furlongm | hi, could i get a review on https://review.openstack.org/#/c/377304/ | 13:48 |
mjblack | I would make it a resource in puppet-horizon that is called by the other modules to standardize the installation | 13:48 |
openstackgerrit | David Moreau Simard proposed openstack/puppet-openstack-integration: Add Designate test coverage to scenario003 https://review.openstack.org/378995 | 13:48 |
mjblack | similar to oslo::db, keystone::resource::authtoken, etc | 13:48 |
EmilienM | mjblack: I would prefer using puppet horizon | 13:49 |
EmilienM | but what is it, just a parameter? | 13:49 |
EmilienM | or a package? | 13:49 |
mjblack | "package" | 13:49 |
mjblack | in the loose sense | 13:49 |
mwhahaha | EmilienM: I created that bug, so we should be doing retries due to the nature of the refresh only | 13:49 |
EmilienM | mjblack, amoralej: please sync with degorenko, he did it for murano | 13:49 |
*** mesamoo has joined #puppet-openstack | 13:49 | |
EmilienM | mwhahaha: ok so +2 | 13:49 |
mjblack | I dont know about RDO but UCA I dont think is packaging the dashboard for v2 | 13:49 |
EmilienM | mwhahaha: can you make the doc public please? | 13:50 |
degorenko | EmilienM, yeah, we stucked with this problem on our ci. Such change will be useful | 13:51 |
amoralej | not in UCA, in RDO there is already a package | 13:51 |
EmilienM | degorenko: ok | 13:51 |
EmilienM | degorenko: thank you | 13:51 |
degorenko | sure | 13:51 |
degorenko | is there some other issue? | 13:52 |
mwhahaha | EmilienM: thought i did, should be public now | 13:52 |
degorenko | which i can help :) | 13:52 |
mjblack | amoralej: so the question I have is, is it possible that UCA just bundled it with horizon? | 13:52 |
EmilienM | mwhahaha: it is now! thanks :) | 13:52 |
mwhahaha | https://docs.google.com/spreadsheets/d/1GV4v-T8E7Sq6GDE12pcTk_i0hFaVrDGnIDDVikXrQ34/edit?usp=sharing | 13:52 |
EmilienM | degorenko: lbaas dashboard. You did the similar thing for murano | 13:52 |
EmilienM | degorenko: maybe you can help mjblack and amoralej | 13:52 |
amoralej | i have no idea mjblack | 13:52 |
EmilienM | mwhahaha: oh wow nice doc | 13:53 |
degorenko | yeah, sure, i have meeting right now, will be free in ~10 minutes. mjblack amoralej please ask | 13:53 |
EmilienM | mwhahaha: I would suggest to put the gerrit topic for each topic | 13:53 |
EmilienM | somewhere in the doc | 13:53 |
mwhahaha | k | 13:53 |
mjblack | btw | 13:53 |
mjblack | mwhahaha, EmilienM, degorenko: could use a look over on... https://review.openstack.org/#/c/376556/ | 13:54 |
iurygregory | puppet-qdr? O.o i didn't know about that one lol | 13:58 |
EmilienM | we have 39 Puppet modules :) wow | 13:59 |
iurygregory | gogo 40 lol | 13:59 |
*** hjensas has quit IRC | 14:01 | |
openstackgerrit | David Moreau Simard proposed openstack/puppet-openstack-integration: Enable Volume encryption testing again https://review.openstack.org/363142 | 14:02 |
dmsimard | http://i.imgur.com/l97K0Mn.jpg https://review.openstack.org/#/q/topic:enable-all-the-things | 14:06 |
iurygregory | EmilienM, https://review.openstack.org/#/c/378986/ merge conflict =( | 14:06 |
EmilienM | oh noes | 14:07 |
iurygregory | yes =( | 14:07 |
*** agrebennikov has joined #puppet-openstack | 14:07 | |
*** skramaja_ has joined #puppet-openstack | 14:09 | |
*** skramaja has quit IRC | 14:09 | |
openstackgerrit | Emilien Macchi proposed openstack/puppet-openstack_extras: repos: configure RDO Virtualization repository https://review.openstack.org/378986 | 14:10 |
*** leanderthal is now known as leanderthal|afk | 14:10 | |
*** skramaja_ has quit IRC | 14:13 | |
*** karthiks has quit IRC | 14:15 | |
openstackgerrit | Sharat Sharma proposed openstack/puppet-zaqar: Added retries for db_sync https://review.openstack.org/379232 | 14:15 |
dmsimard | EmilienM: gnocchi working on newton https://review.openstack.org/#/c/379466/ | 14:15 |
openstackgerrit | Sharat Sharma proposed openstack/puppet-octavia: Added retries for db_sync https://review.openstack.org/379223 | 14:15 |
openstackgerrit | Sharat Sharma proposed openstack/puppet-trove: Added retries for db_sync https://review.openstack.org/379228 | 14:16 |
*** saneax is now known as saneax-_-|AFK | 14:16 | |
openstackgerrit | Sharat Sharma proposed openstack/puppet-ec2api: Added retries for db_sync https://review.openstack.org/379176 | 14:17 |
degorenko | mjblack, commented one nit on 376556 | 14:19 |
EmilienM | dmsimard: where? http://logs.openstack.org/66/379466/1/check/gate-puppet-openstack-integration-4-scenario001-tempest-ubuntu-xenial/a7c6b5a/console.html#_2016-09-29_13_58_21_118169 | 14:19 |
dmsimard | EmilienM: well, I mean, it doesn't fail ? :p | 14:19 |
dmsimard | EmilienM: are you going to not install gnocchi because UCA doesn't ship the tempest plugin ? | 14:19 |
mjblack | degorenko: lol yeah it needs to be done but my change was originally just to get ubuntu working, it seems to have morphed... | 14:20 |
degorenko | mjblack, whats issue with some neutron dashboard or what? | 14:20 |
mjblack | degorenko: I'm thinking that we might need a resource in puppet-horizon to install dashboards in a consistent way and then have classes in respected modules that call it, similar to the authtoken | 14:21 |
dmsimard | EmilienM: best we can do is at least install it and let UCA know the tempest plugin needs to be packaged I think | 14:21 |
mjblack | and it looks like RDO has a lbaas v2 dashboard package that needs to be installed but I'm unsure if UCA packaged the dashboard as part of horizon package | 14:22 |
EmilienM | dmsimard: yes, we need to let them know | 14:24 |
EmilienM | dmsimard: gnocchi/uca is really low prio in my current backlog | 14:24 |
EmilienM | if you want to look go ahead or i'll look next week | 14:24 |
*** saneax-_-|AFK is now known as saneax | 14:25 | |
*** dfisher has joined #puppet-openstack | 14:26 | |
degorenko | mjblack, well, for example, murano has dashboard class inside module. Not sure, can we have some common class. There are a lot of project related options, which are placed with template inside dashboard class. So, not sure totally.But we can try to use for example concat with fragment | 14:27 |
mjblack | degorenko: murano might be an exception to the rule but most horizon dashboards are installed via pip/package and then enabled | 14:29 |
dmsimard | EmilienM: yeah no worries, just re-enabling everything as everything /should/ work as we are nearing release, if something's not working it's a concern for me | 14:29 |
degorenko | mjblack, so, you think all other dashboards will not have some custom settings and so on? | 14:29 |
mjblack | not sure | 14:30 |
degorenko | yeah, so that's why i'm not sure :) | 14:30 |
mjblack | havent played with many dashboards | 14:30 |
mjblack | only the dnsaas and lbaasv2 | 14:30 |
amoralej | sorry, i was in a call | 14:30 |
*** karthiks has joined #puppet-openstack | 14:30 | |
amoralej | for the lbaasv2 case, it's only install the package | 14:30 |
amoralej | in RDO at least | 14:30 |
*** ducttape_ has quit IRC | 14:31 | |
degorenko | then, why just not install this package on orchestration layer? | 14:31 |
amoralej | but i'm not sure for other dashboards | 14:31 |
amoralej | yeap, i can do it in packstack, i.e. | 14:31 |
amoralej | but i usually get advised to reduce orchestration and let puppet-* do things | 14:32 |
degorenko | mjblack, amoralej, well, in this case, why not add some class dashboard? like it done for client? | 14:33 |
amoralej | you mean in neutron::dashboard::lbaas ? | 14:33 |
degorenko | amoralej, no, just neutron::dashboard[s] | 14:35 |
amoralej | yeap, it'd be ok | 14:36 |
*** karthiks has quit IRC | 14:38 | |
*** morazi has quit IRC | 14:46 | |
*** ducttape_ has joined #puppet-openstack | 14:52 | |
*** agrebennikov has quit IRC | 15:00 | |
openstackgerrit | Keith Schincke proposed openstack/puppet-openstack-integration: Add cinder::backup test to scenario0001 https://review.openstack.org/377158 | 15:00 |
openstackgerrit | Alex Schultz proposed openstack/puppet-keystone: Install python memcache package https://review.openstack.org/362270 | 15:00 |
openstackgerrit | Alex Schultz proposed openstack/puppet-openstack-cookiecutter: Add retries to db-sync https://review.openstack.org/378724 | 15:03 |
openstackgerrit | Sharat Sharma proposed openstack/puppet-barbican: Added retries for db_sync https://review.openstack.org/379169 | 15:07 |
openstackgerrit | Merged openstack/puppet-glare: Update main glare class https://review.openstack.org/373472 | 15:07 |
*** hjensas has joined #puppet-openstack | 15:09 | |
EmilienM | iurygregory: could you use https://docs.google.com/spreadsheets/d/1GV4v-T8E7Sq6GDE12pcTk_i0hFaVrDGnIDDVikXrQ34/edit#gid=0 for verbose and authtoken work please? | 15:10 |
openstackgerrit | Merged openstack/puppet-congress: Added retries for db_sync https://review.openstack.org/379291 | 15:10 |
iurygregory | EmilienM, ack :D | 15:10 |
*** Vinsh has joined #puppet-openstack | 15:11 | |
*** lucasagomes is now known as lucas-hungry | 15:12 | |
openstackgerrit | Merged openstack/puppet-rally: Added retries for db_sync https://review.openstack.org/379226 | 15:12 |
EmilienM | mwhahaha, iurygregory: https://review.openstack.org/379466 | 15:13 |
EmilienM | I'm not sure why we enable gnocchi here | 15:13 |
EmilienM | while it's not tested by tempest | 15:13 |
EmilienM | well ¯\_(ツ)_/¯ | 15:13 |
mwhahaha | so we had it enabeld for redhat and not testing in tempest? | 15:14 |
iurygregory | but we test for RDO o.o | 15:14 |
EmilienM | it is tested on centos | 15:14 |
EmilienM | because tempest plugin is packaged | 15:14 |
openstackgerrit | Merged openstack/puppet-glare: Added retries for db_sync https://review.openstack.org/379297 | 15:14 |
EmilienM | but the plugin is not packaged in UCA | 15:14 |
EmilienM | you're allow to sigh now | 15:14 |
iurygregory | dmsimard, ^ | 15:15 |
EmilienM | he knows | 15:15 |
EmilienM | we discussed about that this morning here | 15:15 |
mwhahaha | so why are we testing with packaged plugins | 15:15 |
mwhahaha | because they are matched with the packages? | 15:15 |
EmilienM | it's packaged in RDO not in UCA | 15:15 |
openstackgerrit | Merged openstack/puppet-manila: Added retries for db_sync https://review.openstack.org/379211 | 15:15 |
EmilienM | I think they don't include the tests in the package | 15:15 |
mwhahaha | no why are we not pulling the plugins from source like we do with tempest | 15:16 |
mwhahaha | i guess it's a disconnect with how we do our tempest testing | 15:16 |
EmilienM | right | 15:16 |
openstackgerrit | Merged openstack/puppet-murano: Added retries for db_sync https://review.openstack.org/379212 | 15:16 |
iurygregory | so maybe we should enable the tests for gnocchi to run no? | 15:16 |
mwhahaha | well i'd still like to know if it installed correctly | 15:16 |
EmilienM | mwhahaha: we deploy tempest from source | 15:16 |
EmilienM | and the rest from packaging | 15:17 |
openstackgerrit | Merged openstack/puppet-vitrage: Added retries for db_sync https://review.openstack.org/379231 | 15:17 |
openstackgerrit | Merged openstack/puppet-magnum: Added retries for db_sync https://review.openstack.org/379210 | 15:17 |
dmsimard | EmilienM, mwhahaha, iurygregory: I think installing it and not testing it is better than not installing it at all | 15:17 |
dmsimard | Shows that it sort of works | 15:17 |
mwhahaha | we should add a basic service validation check | 15:17 |
mwhahaha | like is it even listening | 15:18 |
iurygregory | +1 | 15:20 |
dmsimard | well, actually | 15:20 |
openstackgerrit | Merged openstack/puppet-panko: Added retries for db_sync https://review.openstack.org/379225 | 15:21 |
dmsimard | isn't gnocchi indirectly used as part of other tempest tests ? | 15:21 |
dmsimard | like storing metrics or what have you | 15:21 |
openstackgerrit | Peter Zhurba proposed openstack/puppet-glare: Add posibility install glare from pip https://review.openstack.org/373805 | 15:21 |
* dmsimard shrugs | 15:21 | |
mwhahaha | probably not if it wasn't installed and it was working :D | 15:21 |
openstackgerrit | Emilien Macchi proposed openstack/puppet-openstack_spec_helper: pin beaker to 2.x versions https://review.openstack.org/379538 | 15:22 |
EmilienM | backport to liberty ^ | 15:22 |
*** mesamoo has quit IRC | 15:23 | |
dmsimard | mwhahaha: good point | 15:24 |
*** mesamoo has joined #puppet-openstack | 15:24 | |
dmsimard | So, I mean, what do we do with projects like those ? UCA tends to lag behind RDO a lot. For example, we just recently added designate but UCA didn't have the tempest plugin packaged (at least iirc) | 15:25 |
*** xarses has quit IRC | 15:25 | |
dmsimard | We could add a basic serverspec layer or something | 15:25 |
dmsimard | like we do in acceptance | 15:26 |
mwhahaha | so i would use the service_validation resource and do some basic stuff that we can check universally | 15:26 |
mwhahaha | at the end of the install | 15:26 |
mwhahaha | then fireup tempest | 15:26 |
EmilienM | mhh, it will make jobs slower | 15:27 |
EmilienM | with low benefit for centos jobs | 15:27 |
EmilienM | since we run tempest | 15:27 |
mwhahaha | not necessarily because it may prevent failures from running tempest | 15:27 |
EmilienM | I would rather try to talk with UCA folks and add tempest tests in the package, or install the plugin with pip on ubuntu systems | 15:27 |
mwhahaha | speeding up failure reports | 15:27 |
* iurygregory lunch time brb | 15:27 | |
* mwhahaha shrugs | 15:27 | |
dmsimard | mwhahaha: so a post-provision manifest with a service validation per service ? | 15:28 |
EmilienM | to me, it sounds overkill | 15:28 |
EmilienM | it adds a layer of testing and things to maintain | 15:28 |
mwhahaha | yea the basics like is the service responding | 15:28 |
EmilienM | puppet already check that | 15:28 |
EmilienM | with Service resource, no? | 15:28 |
mwhahaha | it's a basic smoke test prior to executing indepth functional testing | 15:28 |
mwhahaha | no | 15:28 |
mwhahaha | it doesn't ensure it started | 15:28 |
mwhahaha | it just starts it | 15:28 |
mwhahaha | so if it dies puppet is non the wiser | 15:28 |
mwhahaha | if it errors then yea puppet knows | 15:29 |
EmilienM | well, in the second Puppet run it check idempotency | 15:29 |
EmilienM | so if process dies, we see it in second puppet run | 15:29 |
EmilienM | and job fails | 15:29 |
mwhahaha | ah true | 15:29 |
mwhahaha | nm then | 15:29 |
EmilienM | but, we don't see it if it dies after a second puppe trun | 15:29 |
EmilienM | not sure it happens though | 15:29 |
mwhahaha | forgot about the 2nd puppet run | 15:29 |
EmilienM | I'm not really in favor of adding checks in poi, we rely on tempest | 15:30 |
EmilienM | and for cases like gnocchi on ubuntu: shrugs | 15:30 |
openstackgerrit | Merged openstack/puppet-openstack-cookiecutter: Add retries to db-sync https://review.openstack.org/378724 | 15:30 |
dmsimard | EmilienM: we're probably going to run into that a lot | 15:30 |
dmsimard | EmilienM: as tempest plugins are being split out of tree | 15:30 |
EmilienM | dmsimard: logs thing is fixed? | 15:30 |
dmsimard | EmilienM: UCA isn't able to keep up | 15:30 |
dmsimard | EmilienM: yeah | 15:31 |
EmilienM | dmsimard: define "a lot" | 15:31 |
dmsimard | EmilienM: well we know there's at /least/ two right now, designate and gnocchi .. and whatever project will split their plugin out of tree next | 15:31 |
EmilienM | not sure "a lot" is true, we'll see it for some services that are not popular in OpenStack | 15:31 |
EmilienM | but Designate and Gnocchi are far from being popular | 15:31 |
EmilienM | I think we were the first to actually test Gnocchi outside devstack. | 15:32 |
dmsimard | we are the first to test a lot of things | 15:32 |
EmilienM | before, i would rather try to report the feedback to UCA | 15:33 |
EmilienM | like we did before | 15:33 |
dmsimard | EmilienM: not sure the logs issue is fixed.. still seeing a lot of failures | 15:34 |
*** amoralej is now known as amoralej|off | 15:37 | |
*** hjensas has quit IRC | 15:41 | |
*** xarses has joined #puppet-openstack | 15:43 | |
openstackgerrit | Mykyta Karpin proposed openstack/puppet-nova: [WIP] Add class for migration options configuration in qemu.conf https://review.openstack.org/378754 | 15:45 |
*** rmart04 has quit IRC | 15:45 | |
*** openstackgerrit has quit IRC | 15:49 | |
*** openstackgerrit has joined #puppet-openstack | 15:50 | |
*** karthiks has joined #puppet-openstack | 15:52 | |
*** paramite has quit IRC | 15:54 | |
openstackgerrit | Merged openstack/puppet-openstack_spec_helper: pin beaker to 2.x versions https://review.openstack.org/379538 | 16:00 |
openstackgerrit | Juan Antonio Osorio Robles proposed openstack/puppet-tripleo: Deploy nova over Apache httpd https://review.openstack.org/376267 | 16:05 |
*** rcernin has quit IRC | 16:05 | |
openstackgerrit | Merged openstack/puppet-openstack-integration: Re-enable trove testing on Ubuntu https://review.openstack.org/379474 | 16:10 |
EmilienM | I'm always scared to enable things on ubuntu :P | 16:11 |
iurygregory | don't be scared | 16:11 |
dmsimard | EmilienM: stable release man, it's supposed to work :) | 16:12 |
dmsimard | if it doesn't, it needs fixing | 16:12 |
EmilienM | we don't have the same definition of stable :) | 16:13 |
mwhahaha | stable-ish | 16:14 |
EmilienM | I prefer ^ | 16:14 |
EmilienM | RDO packages are stable because they are tested by something everyone can contribute and the matrix is heavy | 16:15 |
EmilienM | UCA packages are not stable because they are tested by something not transparent where only UCA folks can contribute and the matrix is light | 16:15 |
*** Vinsh has quit IRC | 16:15 | |
EmilienM | that's the main difference. | 16:15 |
*** lucas-hungry is now known as lucasagomes | 16:15 | |
*** dgurtner has quit IRC | 16:16 | |
EmilienM | but I trust zigo to produce gated packages for ubuntu | 16:16 |
EmilienM | and one day, we'll get rid of UCA :-) | 16:16 |
openstackgerrit | Merged openstack/puppet-aodh: Added retries for db_sync https://review.openstack.org/379166 | 16:17 |
*** Vinsh has joined #puppet-openstack | 16:17 | |
*** Vinsh has quit IRC | 16:17 | |
*** Vinsh has joined #puppet-openstack | 16:17 | |
mjblack | degorenko: so with your comment on my change | 16:23 |
mjblack | I'm wondering we need to specify some of those file resources since they should be managed by the existing types and providers | 16:24 |
mjblack | sorry types/providers | 16:25 |
*** jperry has quit IRC | 16:29 | |
*** jperry has joined #puppet-openstack | 16:30 | |
dmsimard | EmilienM: ok. | 16:30 |
*** derekh has quit IRC | 16:34 | |
openstackgerrit | Michele Baldessari proposed openstack/puppet-tripleo: Change rabbitmq queues HA mode from ha-all to ha-exactly https://review.openstack.org/379586 | 16:36 |
degorenko | mjblack, don't get it. You just tagging your resources with some specific tag and use it then with collector. Btw, it's not rule or requirement :) you can go as is | 16:40 |
mjblack | right | 16:41 |
mjblack | but I dont think we need to do file types for files put out by the packages and managed by the module types | 16:41 |
degorenko | we dont, yes | 16:42 |
*** priteau has joined #puppet-openstack | 16:44 | |
*** trown is now known as trown|lunch | 16:53 | |
mwhahaha | looks like the designate tempest test might be flaky, http://logs.openstack.org/61/378061/1/check/gate-puppet-openstack-integration-4-scenario003-tempest-centos-7/bdaacad/console.html#_2016-09-29_16_54_45_170427 | 16:57 |
openstackgerrit | Merged openstack/puppet-ec2api: Added retries for db_sync https://review.openstack.org/379176 | 16:58 |
dmsimard | mwhahaha: no servers configured means the pool update didn't go through | 16:58 |
dmsimard | mwhahaha: remember the execution order thing? | 16:58 |
mwhahaha | well it works sometimes | 16:58 |
dmsimard | mwhahaha: it worked sometimes before too, that's why I added an explicit service_validation to try and make sure that no longer happens | 16:59 |
dmsimard | it no longer seemed to flap but it looks like there should be something else I should be doing then .. | 16:59 |
openstackgerrit | Davanum Srinivas (dims) proposed openstack/puppet-tripleo: Update .gitreview for stable/newton https://review.openstack.org/379599 | 16:59 |
dmsimard | mwhahaha: see: http://logs.openstack.org/61/378061/1/check/gate-puppet-openstack-integration-4-scenario003-tempest-centos-7/bdaacad/logs/designate/designate-manage.txt.gz#_2016-09-29_16_36_49_842 | 17:00 |
dmsimard | mwhahaha: that's why I added the service_validation :( | 17:01 |
mwhahaha | hmm did we not merge that yet? | 17:03 |
dmsimard | mwhahaha: yeah, it was part of what merged with the initial designate support | 17:03 |
mwhahaha | well looks like it didn't work | 17:03 |
dmsimard | https://github.com/openstack/puppet-openstack-integration/blob/master/manifests/designate.pp#L91-L115 | 17:03 |
mwhahaha | cause my change should have that | 17:03 |
openstackgerrit | Keith Schincke proposed openstack/puppet-openstack-integration: Add cinder::backup test to scenario0001 https://review.openstack.org/377158 | 17:04 |
openstackgerrit | Merged openstack/puppet-gnocchi: Added retries for db_sync https://review.openstack.org/379196 | 17:06 |
openstackgerrit | Merged openstack/puppet-octavia: Added retries for db_sync https://review.openstack.org/379223 | 17:06 |
mnaser | hmm | 17:16 |
mnaser | https://github.com/openstack/puppet-ceph/blob/master/manifests/repo.pp#L128-L134 | 17:16 |
mnaser | is there a reason in specific it is not pointing to http://mirror.centos.org/centos/7/storage/x86_64/ceph-jewel/ ? | 17:17 |
mnaser | i mean this works well for me because i want 10.2.3 which is out in buildlogs but not released to mirrors .. but i feel like it's probably not the most stable thing to deliver? | 17:17 |
xarses | mnaser: when we switched to jewel, the packages where not in mirrors | 17:17 |
mnaser | gotcha | 17:17 |
mnaser | i think centos-ceph-jewel-release is out (let me check) | 17:18 |
xarses | if you think we should switch let us know // submit a CR for it | 17:18 |
mnaser | centos-release-ceph-jewel is released so it can be installed instead of manual yumrepo management.. but i'm unsure on the effects of doing this (will you end up with 2 repos if you used the former method?) | 17:19 |
mnaser | right now i prefer the buildlogs because we're testing 10.2.3 which isnt out in release but that's probably not ideal for prod setups, you'd end up with 10.2.3 that has not been officially 'released' by the sig | 17:20 |
*** jaosorior has quit IRC | 17:20 | |
mnaser | its up for discussion :> | 17:20 |
*** jpena is now known as jpena|off | 17:23 | |
*** degorenko is now known as _degorenko | 17:25 | |
*** gfidente is now known as gfidente|afk | 17:31 | |
dmsimard | colonwq: added a comment | 17:36 |
colonwq | dmsimard, thanks. I guess it is not as simple as 'volume_feature_enabled/backup' to true. | 17:39 |
dmsimard | colonwq: we explicitely tag which tests we want to enable | 17:39 |
openstackgerrit | Adam Vinsh proposed openstack/puppet-swift: Replace all proxy template code with ini provider https://review.openstack.org/378950 | 17:39 |
dmsimard | colonwq: https://github.com/openstack/puppet-openstack-integration/blob/master/run_tests.sh#L186-L205 | 17:40 |
dmsimard | colonwq: you still need the tempest config, yes -- but also tag the tests to whitelist them (and run them) | 17:40 |
colonwq | dmsimard, Ah. Thanks for pointing me in the correct direction. I'll locally test and update the patch. | 17:41 |
*** trown|lunch is now known as trown | 17:56 | |
openstackgerrit | Michele Baldessari proposed openstack/puppet-tripleo: Change rabbitmq queues HA mode from ha-all to ha-exactly https://review.openstack.org/379586 | 18:06 |
*** jpich has quit IRC | 18:07 | |
*** saneax is now known as saneax-_-|AFK | 18:11 | |
*** karthiks has quit IRC | 18:12 | |
*** xarses has quit IRC | 18:18 | |
*** ducttape_ has quit IRC | 18:19 | |
*** xarses has joined #puppet-openstack | 18:20 | |
*** ducttape_ has joined #puppet-openstack | 18:22 | |
mnaser | EmilienM, im working on a patch to have puppet-ceph install centos-release-ceph-jewel .. looking at stable/hammer, that rpm was installed with an exec https://github.com/openstack/puppet-ceph/blob/stable/hammer/manifests/repo.pp#L117-L123 .. was there a reason it was done that way instead of package { '..': } ? | 18:25 |
*** karthiks has joined #puppet-openstack | 18:25 | |
mnaser | (asking you because i saw you made that commit :>) | 18:25 |
mwhahaha | mnaser: probably due to the retries | 18:35 |
mwhahaha | since we would get failures trying to download that rpm, the normal package provider does not have a retry concept | 18:35 |
*** xarses has quit IRC | 18:36 | |
mnaser | aah okay | 18:37 |
mnaser | mwhahaha, so i guess i'll reimplement it similar to how it was originally done | 18:37 |
*** xarses has joined #puppet-openstack | 18:38 | |
*** xarses has quit IRC | 18:40 | |
*** xarses has joined #puppet-openstack | 18:42 | |
openstackgerrit | Merged openstack/puppet-cinder: Added retries for db_sync https://review.openstack.org/379171 | 18:42 |
openstackgerrit | Mohammed Naser proposed openstack/puppet-ceph: Switch to centos-release-ceph-jewel https://review.openstack.org/379695 | 18:43 |
openstackgerrit | Merged openstack/puppet-ceilometer: Added retries for db_sync https://review.openstack.org/379170 | 18:43 |
openstackgerrit | Merged openstack/puppet-designate: Added retries for db_sync https://review.openstack.org/379172 | 18:43 |
openstackgerrit | Merged openstack/puppet-sahara: Added retries for db_sync https://review.openstack.org/379227 | 18:43 |
openstackgerrit | Merged openstack/puppet-nova: Added retries for db_sync https://review.openstack.org/379217 | 18:43 |
openstackgerrit | Merged openstack/puppet-ironic: Remove unnecesary spaces in log messages. https://review.openstack.org/376249 | 18:43 |
openstackgerrit | Merged openstack/puppet-neutron: Added retries for db_sync https://review.openstack.org/379214 | 18:43 |
openstackgerrit | Merged openstack/puppet-barbican: Added retries for db_sync https://review.openstack.org/379169 | 18:52 |
openstackgerrit | Merged openstack/puppet-openstack-integration: [mitaka] bump tempest to 12.2.0 https://review.openstack.org/378920 | 19:02 |
openstackgerrit | Merged openstack/puppet-openstack-integration: Update SSL certificates with SubjectAltNames https://review.openstack.org/379434 | 19:16 |
*** mesamoo has quit IRC | 19:17 | |
openstackgerrit | Michele Baldessari proposed openstack/puppet-tripleo: Change rabbitmq queues HA mode from ha-all to ha-exactly https://review.openstack.org/379586 | 19:27 |
openstackgerrit | Merged openstack/puppet-zaqar: Added retries for db_sync https://review.openstack.org/379232 | 19:32 |
*** bfg9000_ has joined #puppet-openstack | 19:38 | |
*** johnmilton has quit IRC | 19:40 | |
*** holser_ has quit IRC | 19:42 | |
*** bfg9000 has quit IRC | 19:43 | |
*** bfg9000_ is now known as bfg9000 | 19:43 | |
*** ken_barber has quit IRC | 19:43 | |
*** khappone_ has quit IRC | 19:43 | |
*** khappone has joined #puppet-openstack | 19:43 | |
iurygregory | EmilienM, you need to update https://review.openstack.org/#/c/378986 =) | 19:44 |
*** holser_ has joined #puppet-openstack | 19:47 | |
*** fc___ has joined #puppet-openstack | 19:47 | |
*** fc__ has quit IRC | 19:50 | |
*** fc___ is now known as fc__ | 19:50 | |
*** ken_barber has joined #puppet-openstack | 19:51 | |
*** timrc_ is now known as timrc | 19:52 | |
*** lucasagomes is now known as lucas-afk | 19:54 | |
*** johnmilton has joined #puppet-openstack | 19:55 | |
*** morazi has joined #puppet-openstack | 19:58 | |
*** ayoung has quit IRC | 20:01 | |
openstackgerrit | Alex Schultz proposed openstack/puppet-swift: Fixed swift::proxy::cache requirement on memcached https://review.openstack.org/379796 | 20:10 |
*** jayg is now known as jayg|g0n3 | 20:11 | |
*** ayoung has joined #puppet-openstack | 20:27 | |
dmsimard | It will be sort of funny to have Ansible stuff in-tree in Puppet module repositories when Zuul v3 lands :) | 20:30 |
*** morazi has quit IRC | 20:32 | |
*** morazi has joined #puppet-openstack | 20:32 | |
dmsimard | mwhahaha: I was trying to look at that designate failure you sent me. You know what's frustrating ? The refresh did not return an error and yet it obviously did not work. | 20:32 |
mwhahaha | ha | 20:33 |
dmsimard | Success: http://logs.openstack.org/61/378061/1/check/gate-puppet-openstack-integration-4-scenario003-tempest-centos-7/bdaacad/console.html#_2016-09-29_16_36_49_904897 | 20:33 |
dmsimard | Failure: http://logs.openstack.org/61/378061/1/check/gate-puppet-openstack-integration-4-scenario003-tempest-centos-7/bdaacad/logs/designate/designate-manage.txt.gz#_2016-09-29_16_36_49_842 | 20:33 |
dmsimard | Timestamp is consistent | 20:33 |
dmsimard | So what's up with that -- upstream bug and designate-manage doesn't return 1 on failure ? | 20:34 |
mwhahaha | sounds like it | 20:34 |
dmsimard | The refresh would've definitely failed if it had returned 1, yes ? | 20:34 |
dmsimard | I'll check it out... | 20:34 |
mwhahaha | i would hope that it would have returned 1 with that critical error | 20:35 |
mwhahaha | but is it one of those things where it posts async? | 20:35 |
*** jtriley has quit IRC | 20:37 | |
mwhahaha | dmsimard: also what ansible stuff in the repo? we already have python stuff for reno :D | 20:38 |
*** ayoung has quit IRC | 20:39 | |
*** hjensas has joined #puppet-openstack | 20:43 | |
*** hjensas has joined #puppet-openstack | 20:43 | |
*** trown is now known as trown|outtypewww | 20:47 | |
*** hjensas has quit IRC | 20:51 | |
*** amoralej|off has quit IRC | 20:53 | |
*** jesusaur has quit IRC | 20:53 | |
*** jesusaur has joined #puppet-openstack | 20:54 | |
*** amoralej has joined #puppet-openstack | 20:55 | |
openstackgerrit | Merged openstack/puppet-keystone: Install python memcache package https://review.openstack.org/362270 | 20:57 |
*** jpena|off has quit IRC | 21:00 | |
openstackgerrit | Merged openstack/puppet-openstacklib: Add a refreshonly parameter to service_validation https://review.openstack.org/379476 | 21:00 |
openstackgerrit | Merged openstack/puppet-ironic: Added retries for db_sync https://review.openstack.org/379206 | 21:00 |
openstackgerrit | Merged openstack/puppet-designate: Update define tests to use rspec-puppet-facts https://review.openstack.org/378061 | 21:00 |
*** jpena|off has joined #puppet-openstack | 21:02 | |
*** fultonj has quit IRC | 21:04 | |
*** dtantsur has quit IRC | 21:48 | |
openstackgerrit | Adam Vinsh proposed openstack/puppet-swift: Replace proxy concat/template with ini provider https://review.openstack.org/378950 | 21:54 |
Vinsh | ^^ Ready for action. | 21:56 |
mwhahaha | Vinsh: you might want to rebase that on top of https://review.openstack.org/#/c/379796/ | 21:57 |
mwhahaha | unless you want to deal with the merge conflict later :D | 21:58 |
Vinsh | (┛◉Д◉)┛彡┻━┻ | 21:58 |
mwhahaha | cause i need to backport that change | 21:58 |
mwhahaha | so it needs to land first | 21:58 |
Vinsh | Got it. Will do that now | 21:59 |
*** jperry has quit IRC | 22:00 | |
openstackgerrit | Adam Vinsh proposed openstack/puppet-swift: Replace proxy concat/template with ini provider https://review.openstack.org/378950 | 22:11 |
Vinsh | mwhahaha: There we go, rebased to match that. | 22:11 |
Vinsh | but mo'betta :) | 22:11 |
Vinsh | Next week, will tackle account/container/object templates. | 22:12 |
Vinsh | That proxy work was tedious! | 22:12 |
mwhahaha | I bet thanks | 22:15 |
openstackgerrit | Emilien Macchi proposed openstack/puppet-openstack_extras: repos: configure RDO Virtualization repository https://review.openstack.org/378986 | 22:18 |
EmilienM | iurygregory: thanks | 22:19 |
*** ducttape_ has quit IRC | 22:20 | |
*** b00tcat` has quit IRC | 22:22 | |
*** ducttape_ has joined #puppet-openstack | 22:26 | |
openstackgerrit | Adam Vinsh proposed openstack/puppet-swift: Replace proxy concat/template with ini provider https://review.openstack.org/378950 | 22:28 |
*** ayoung has joined #puppet-openstack | 22:28 | |
*** ntpttr has quit IRC | 22:31 | |
*** ntpttr has joined #puppet-openstack | 22:31 | |
openstackgerrit | Merged openstack/puppet-trove: Added retries for db_sync https://review.openstack.org/379228 | 22:32 |
*** ducttape_ has quit IRC | 22:33 | |
openstackgerrit | Merged openstack/puppet-glance: Added retries for db_sync https://review.openstack.org/379183 | 22:34 |
*** moravec has joined #puppet-openstack | 22:35 | |
*** iurygregory_ has joined #puppet-openstack | 22:47 | |
*** dencaval has quit IRC | 22:48 | |
openstackgerrit | Merged openstack/puppet-keystone: Added retries for db_sync https://review.openstack.org/379209 | 22:57 |
openstackgerrit | Merged openstack/puppet-mistral: Added retries for db_sync https://review.openstack.org/379148 | 22:57 |
*** xarses has quit IRC | 22:59 | |
*** dprince has quit IRC | 23:05 | |
*** ayoung has quit IRC | 23:06 | |
*** priteau has quit IRC | 23:16 | |
*** saneax-_-|AFK is now known as saneax | 23:20 | |
*** dfisher has quit IRC | 23:20 | |
openstackgerrit | Merged openstack/puppet-heat: Added retries for db_sync https://review.openstack.org/379202 | 23:21 |
openstackgerrit | Adam Vinsh proposed openstack/puppet-swift: Replace proxy concat/template with ini provider https://review.openstack.org/378950 | 23:32 |
*** markvoelker has quit IRC | 23:36 | |
*** ducttape_ has joined #puppet-openstack | 23:41 | |
*** morazi has quit IRC | 23:45 | |
*** ayoung has joined #puppet-openstack | 23:51 | |
*** ducttape_ has quit IRC | 23:54 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!