*** markvoelker has joined #puppet-openstack | 01:05 | |
*** markvoelker has quit IRC | 01:09 | |
openstackgerrit | zhongshengping proposed openstack/puppet-keystone: Deprecate signing_dir option https://review.openstack.org/416105 | 01:27 |
---|---|---|
*** michapma_alt has joined #puppet-openstack | 01:34 | |
*** michchap_alt has joined #puppet-openstack | 01:34 | |
openstackgerrit | zhongshengping proposed openstack/puppet-ironic: Deprecate signing_dir option https://review.openstack.org/415350 | 01:40 |
openstackgerrit | zhongshengping proposed openstack/puppet-mistral: Add the acceptance test for Debian system https://review.openstack.org/416108 | 01:55 |
*** markvoelker has joined #puppet-openstack | 02:05 | |
*** clayton has quit IRC | 02:06 | |
*** markvoelker has quit IRC | 02:10 | |
*** clayton has joined #puppet-openstack | 02:25 | |
*** snarwade has joined #puppet-openstack | 02:41 | |
*** ilbot3 has quit IRC | 02:46 | |
*** ilbot3 has joined #puppet-openstack | 02:48 | |
*** markvoelker has joined #puppet-openstack | 03:06 | |
*** markvoelker has quit IRC | 03:10 | |
*** clayton has quit IRC | 03:34 | |
*** ilbot3 has quit IRC | 03:34 | |
*** snarwade has quit IRC | 03:34 | |
*** michapma_alt has quit IRC | 03:34 | |
*** clayton has joined #puppet-openstack | 03:34 | |
*** ilbot3 has joined #puppet-openstack | 03:35 | |
*** snarwade has joined #puppet-openstack | 03:35 | |
*** michapma_alt has joined #puppet-openstack | 03:35 | |
*** clayton has left #puppet-openstack | 03:35 | |
*** clayton has joined #puppet-openstack | 03:35 | |
*** skramaja has joined #puppet-openstack | 05:22 | |
openstackgerrit | zhongshengping proposed openstack/puppet-trove: Fix database_idle_timeout option https://review.openstack.org/416145 | 06:50 |
openstackgerrit | zhongshengping proposed openstack/puppet-murano: Unknown variable: '::murano::params::default_external_network' https://review.openstack.org/416153 | 06:58 |
*** ccamacho has quit IRC | 07:15 | |
*** tesseract has joined #puppet-openstack | 07:30 | |
*** snarwade is now known as snarwade| | 07:35 | |
*** snarwade| is now known as snarwade-lunch | 07:35 | |
*** shardy has joined #puppet-openstack | 07:58 | |
*** rcernin has joined #puppet-openstack | 08:03 | |
*** saneax-_-|AFK is now known as saneax | 08:12 | |
*** acabot has joined #puppet-openstack | 08:28 | |
*** ccamacho has joined #puppet-openstack | 08:40 | |
*** jpich has joined #puppet-openstack | 08:47 | |
*** toMeloos has joined #puppet-openstack | 08:56 | |
*** danpawlik has quit IRC | 09:08 | |
*** danpawlik has joined #puppet-openstack | 09:09 | |
*** snarwade-lunch is now known as snarwade | 09:11 | |
*** jtomasek has quit IRC | 09:17 | |
*** derekh has joined #puppet-openstack | 09:30 | |
*** paramite has joined #puppet-openstack | 09:30 | |
*** markvoelker has joined #puppet-openstack | 10:10 | |
*** gfidente has joined #puppet-openstack | 10:12 | |
*** gfidente has quit IRC | 10:12 | |
*** gfidente has joined #puppet-openstack | 10:12 | |
*** markvoelker has quit IRC | 10:15 | |
*** rmart04 has joined #puppet-openstack | 10:17 | |
*** gfidente^2nd has joined #puppet-openstack | 10:30 | |
*** gfidente has quit IRC | 10:32 | |
*** giulivo^3rd has joined #puppet-openstack | 10:33 | |
*** gfidente^2nd has quit IRC | 10:35 | |
*** dtantsur|afk is now known as dtantsur | 10:38 | |
*** chem has quit IRC | 10:39 | |
*** giulivo^3rd has quit IRC | 10:43 | |
*** giulivo^3rd has joined #puppet-openstack | 10:46 | |
*** jtomasek has joined #puppet-openstack | 11:11 | |
*** markvoelker has joined #puppet-openstack | 11:11 | |
*** markvoelker has quit IRC | 11:16 | |
openstackgerrit | Tobias Urdin proposed openstack/puppet-neutron: Add enable_new_agents config option to neutron::server https://review.openstack.org/416214 | 11:17 |
*** chem has joined #puppet-openstack | 11:24 | |
*** ntpttr has quit IRC | 11:30 | |
*** ntpttr has joined #puppet-openstack | 11:39 | |
*** hjensas has quit IRC | 11:54 | |
openstackgerrit | Giulio Fidente proposed openstack/puppet-ceph: Improves support for ceph-mds https://review.openstack.org/402385 | 12:00 |
*** snarwade has quit IRC | 12:00 | |
*** hjensas has joined #puppet-openstack | 12:05 | |
*** markvoelker has joined #puppet-openstack | 12:12 | |
*** markvoelker has quit IRC | 12:16 | |
*** NachoDuck has quit IRC | 12:19 | |
*** Hunner has quit IRC | 12:19 | |
*** bmjen has quit IRC | 12:19 | |
*** bmjen has joined #puppet-openstack | 12:19 | |
*** Hunner has joined #puppet-openstack | 12:19 | |
*** Hunner has joined #puppet-openstack | 12:19 | |
EmilienM | hello | 12:21 |
*** NachoDuck has joined #puppet-openstack | 12:23 | |
*** lucasagomes is now known as lucas-hungry | 12:25 | |
*** hjensas_ has joined #puppet-openstack | 12:27 | |
*** hjensas_ has quit IRC | 12:27 | |
*** hjensas has quit IRC | 12:27 | |
*** hjensas has joined #puppet-openstack | 12:27 | |
*** giulivo^3rd is now known as gfidente | 12:31 | |
*** gfidente has quit IRC | 12:31 | |
*** gfidente has joined #puppet-openstack | 12:31 | |
*** gfidente has quit IRC | 12:43 | |
*** gfidente has joined #puppet-openstack | 12:45 | |
*** toMeloos has quit IRC | 12:47 | |
*** saneax is now known as saneax-_-|AFK | 12:49 | |
*** pradk has joined #puppet-openstack | 13:00 | |
*** fultonj has joined #puppet-openstack | 13:11 | |
tobasco | EmilienM: hello, all well? :) could you give some feedback on my open reviews https://review.openstack.org/#/q/owner:tobias%2540busybox.se+status:open | 13:11 |
tobasco | thanks | 13:11 |
EmilienM | tobasco: sure thing | 13:12 |
*** markvoelker has joined #puppet-openstack | 13:12 | |
*** ccamacho is now known as ccamacho|lunch | 13:15 | |
*** fultonj has quit IRC | 13:16 | |
*** markvoelker has quit IRC | 13:17 | |
*** fultonj has joined #puppet-openstack | 13:17 | |
*** fultonj has quit IRC | 13:18 | |
*** dprince has joined #puppet-openstack | 13:18 | |
*** ansmith has joined #puppet-openstack | 13:18 | |
*** toMeloos has joined #puppet-openstack | 13:19 | |
*** trown has joined #puppet-openstack | 13:20 | |
*** lucas-hungry is now known as lucasagomes | 13:31 | |
danpawlik | EmilienM, mwhahaha: Hello, could you check https://review.openstack.org/#/c/404957/ ? Plz :> | 13:33 |
openstackgerrit | Merged openstack/puppet-nova: Add nova::compute::libvirt::qemu class to configure limits https://review.openstack.org/411983 | 13:39 |
*** gfidente has quit IRC | 13:45 | |
*** fultonj has joined #puppet-openstack | 13:46 | |
*** yankcrime has quit IRC | 13:47 | |
*** _nick has joined #puppet-openstack | 13:47 | |
*** cgfbee has quit IRC | 13:47 | |
*** gfidente has joined #puppet-openstack | 13:48 | |
*** skramaja has quit IRC | 13:48 | |
*** skramaja has joined #puppet-openstack | 13:48 | |
*** furlongm_ has quit IRC | 13:48 | |
*** furlongm has joined #puppet-openstack | 13:48 | |
*** cgfbee has joined #puppet-openstack | 13:53 | |
*** lhinds has quit IRC | 13:54 | |
*** jmiu has quit IRC | 13:54 | |
*** jmiu has joined #puppet-openstack | 13:55 | |
*** lhinds has joined #puppet-openstack | 13:55 | |
*** ccamacho|lunch is now known as ccamacho | 14:01 | |
*** trown|brb has joined #puppet-openstack | 14:04 | |
*** trown has quit IRC | 14:05 | |
*** trown|brb is now known as trown | 14:05 | |
hjensas | EmilienM: degorenko: Any feedback on: https://review.openstack.org/#/c/412086/ ? | 14:08 |
openstackgerrit | Merged openstack/puppet-watcher: Add Ubuntu functional tests https://review.openstack.org/404957 | 14:13 |
*** markvoelker has joined #puppet-openstack | 14:13 | |
EmilienM | hjensas: recheck | 14:14 |
hjensas | EmilienM: ok, thanks. One of the beaker-rspec tests seem to fail every time, but not always the same one. AFAICT from logs it fails because of other issues in CI. Hope it all passes now. :) | 14:16 |
*** markvoelker has quit IRC | 14:18 | |
*** cdelatte has joined #puppet-openstack | 14:28 | |
openstackgerrit | Emilien Macchi proposed openstack/puppet-openstack-integration: scenario001: test Telemetry with Auto-Scaling scenario https://review.openstack.org/410865 | 14:29 |
*** morazi has joined #puppet-openstack | 14:29 | |
*** jperry has joined #puppet-openstack | 14:36 | |
*** markvoelker has joined #puppet-openstack | 14:38 | |
*** alee has joined #puppet-openstack | 14:41 | |
*** ken_barber has quit IRC | 14:46 | |
*** toMeloos has quit IRC | 15:03 | |
*** rmart04_ has joined #puppet-openstack | 15:06 | |
*** rmart04 has quit IRC | 15:06 | |
*** rmart04_ is now known as rmart04 | 15:06 | |
*** rmart04 has quit IRC | 15:12 | |
*** rmart04 has joined #puppet-openstack | 15:17 | |
*** med_` is now known as medberry | 15:28 | |
*** medberry is now known as med_ | 15:28 | |
*** med_ is now known as Guest46232 | 15:29 | |
*** Guest46232 is now known as medberry | 15:29 | |
*** medberry has joined #puppet-openstack | 15:29 | |
openstackgerrit | Merged openstack/puppet-cloudkitty: Remove signing_dir option https://review.openstack.org/415086 | 15:38 |
openstackgerrit | Merged openstack/puppet-glare: Remove signing_dir option https://review.openstack.org/415345 | 15:41 |
*** rmart04 has quit IRC | 15:49 | |
*** rmart04 has joined #puppet-openstack | 15:53 | |
*** rmart04 has quit IRC | 15:54 | |
*** rmart04_ has joined #puppet-openstack | 15:59 | |
*** richm has joined #puppet-openstack | 16:01 | |
openstackgerrit | Merged openstack/puppet-openstack-guide: Delete unnecessary utf-8 coding https://review.openstack.org/415054 | 16:02 |
openstackgerrit | Merged openstack/puppet-openstack-specs: Delete unnecessary utf-8 coding https://review.openstack.org/414822 | 16:03 |
openstackgerrit | Merged openstack/puppet-murano: Deprecate signing_dir option https://review.openstack.org/415355 | 16:08 |
openstackgerrit | Merged openstack/puppet-magnum: Deprecate signing_dir option https://review.openstack.org/415352 | 16:08 |
openstackgerrit | Emilien Macchi proposed openstack/puppet-openstack-integration: aodh: don't hardcode keystone endpoint version https://review.openstack.org/416281 | 16:09 |
openstackgerrit | Merged openstack/puppet-manila: Deprecate signing_dir option https://review.openstack.org/415354 | 16:11 |
openstackgerrit | Emilien Macchi proposed openstack/puppet-openstack-integration: aodh: don't hardcode keystone endpoint version https://review.openstack.org/416281 | 16:11 |
openstackgerrit | Pradeep Kilambi proposed openstack/puppet-nova: Add enabled_perf_events option for libvirt https://review.openstack.org/416282 | 16:14 |
openstackgerrit | Tobias Urdin proposed openstack/puppet-neutron: Add enable_new_agents config option to neutron::server https://review.openstack.org/416287 | 16:19 |
openstackgerrit | Merged openstack/puppet-barbican: Deprecate signing_dir option https://review.openstack.org/415082 | 16:21 |
openstackgerrit | Merged openstack/puppet-ec2api: Remove keystone_url option https://review.openstack.org/413390 | 16:21 |
*** michchap_alt has quit IRC | 16:21 | |
*** michapma_alt has quit IRC | 16:22 | |
*** rmart04_ has quit IRC | 16:23 | |
*** _nick is now known as yankcrime | 16:24 | |
*** rcernin has quit IRC | 16:25 | |
*** hjensas has quit IRC | 16:25 | |
*** ccamacho has quit IRC | 16:26 | |
*** fultonj has quit IRC | 16:29 | |
*** tesseract has quit IRC | 16:31 | |
*** fultonj has joined #puppet-openstack | 16:37 | |
EmilienM | mwhahaha: when you have time, can you look https://review.openstack.org/#/c/402385/ and give thoughts? cc gfidente | 16:45 |
mwhahaha | i am | 16:46 |
gfidente | we'll want to have a param for the package state at least I think | 16:46 |
gfidente | I was just excited it was finally green as it unblocks some tripleo submissions :P | 16:47 |
mwhahaha | i agree on the package params thing, the other not so much | 16:47 |
gfidente | so out of curiosity | 16:49 |
gfidente | is there a reason why the SERVICE DEFAULT behaviour isn't implemented in any other shared lib? | 16:51 |
mwhahaha | it's a concept we came up with | 16:51 |
gfidente | to me sounds like it would be nice to have that even in stdlib maybe | 16:52 |
openstackgerrit | Pradeep Kilambi proposed openstack/puppet-ceilometer: Add deprecation warning to ceilo api https://review.openstack.org/416296 | 16:52 |
mwhahaha | well there are other ways to do it in puppet with resource purging but those get dangerous | 16:52 |
mwhahaha | we have it because we wanted a way to specific that we don't want a value in the config which would have the effect of defaulting to whatever the app had | 16:53 |
gfidente | yeah I was trying to say that it sounds sane to me to have it widely available in stdlib indeed | 16:54 |
openstackgerrit | Michael Bayer proposed openstack/puppet-nova: Allow db sync timeouts to be configurable https://review.openstack.org/414613 | 16:55 |
mwhahaha | we also have a bunch of code in our providers to handle it which isn't not necessarily the case with all inifile stuff | 16:55 |
mwhahaha | i mean it makes sense but it's just not universally used. we just wanted to cleanup our code around removing values | 16:56 |
*** medberry is now known as med_ | 17:14 | |
*** trown is now known as trown|lunch | 17:15 | |
*** vinsh has joined #puppet-openstack | 17:20 | |
*** dtantsur is now known as dtantsur|afk | 17:23 | |
alee | EmilienM, mwhahaha -- can you guys take a look at https://review.openstack.org/396789 ? | 17:23 |
*** jaosorior has joined #puppet-openstack | 17:26 | |
*** jpich has quit IRC | 17:28 | |
*** rwsu has joined #puppet-openstack | 17:30 | |
*** dfisher has joined #puppet-openstack | 17:34 | |
*** lucasagomes is now known as lucas-afk | 17:41 | |
*** jaosorior has quit IRC | 17:43 | |
alee | EmilienM, thanks for review -- fixing. Also https://review.openstack.org/#/c/412444/ when you get a chance please. | 17:43 |
*** shardy has quit IRC | 17:46 | |
*** derekh has quit IRC | 17:47 | |
openstackgerrit | Merged openstack/puppet-monasca: Update agent init file for better shutdown https://review.openstack.org/410943 | 17:58 |
*** fultonj has quit IRC | 18:19 | |
*** fultonj has joined #puppet-openstack | 18:20 | |
*** ducttape_ has joined #puppet-openstack | 18:25 | |
openstackgerrit | Merged openstack/puppet-gnocchi: Deprecate signing_dir option https://review.openstack.org/415347 | 18:33 |
openstackgerrit | Merged openstack/puppet-heat: Deprecate signing_dir option https://review.openstack.org/415348 | 18:33 |
openstackgerrit | Merged openstack/puppet-neutron: Deprecate signing_dir option https://review.openstack.org/415357 | 18:33 |
openstackgerrit | Merged openstack/puppet-nova: Deprecate signing_dir option https://review.openstack.org/415360 | 18:34 |
openstackgerrit | Merged openstack/puppet-openstack_extras: Removes unnecessary utf-8 coding https://review.openstack.org/414793 | 18:34 |
*** toMeloos has joined #puppet-openstack | 18:36 | |
*** mc___ has joined #puppet-openstack | 18:38 | |
*** mc___ has quit IRC | 18:38 | |
*** trown|lunch is now known as trown | 18:39 | |
openstackgerrit | Merged openstack/puppet-panko: Remove signing_dir option https://review.openstack.org/415362 | 18:41 |
openstackgerrit | Merged openstack/puppet-tempest: Delete unnecessary utf-8 coding https://review.openstack.org/414839 | 18:41 |
openstackgerrit | Merged openstack/puppet-nova: Add resize_confirm_window option to nova::compute https://review.openstack.org/415162 | 18:41 |
openstackgerrit | Merged openstack/puppet-nova: Add the resume_guests_state_on_host_boot option to nova::compute https://review.openstack.org/415163 | 18:41 |
*** chem has quit IRC | 19:03 | |
*** chem has joined #puppet-openstack | 19:03 | |
*** NachoDuck has quit IRC | 19:35 | |
*** jayg|g0n3 has quit IRC | 19:35 | |
*** sbadia has quit IRC | 19:35 | |
*** Lokesh_Jain_ has quit IRC | 19:35 | |
*** dtantsur|afk has quit IRC | 19:35 | |
*** dtantsur has joined #puppet-openstack | 19:35 | |
*** jayg|g0n3 has joined #puppet-openstack | 19:36 | |
*** sbadia has joined #puppet-openstack | 19:38 | |
*** davidmichaelkarr has quit IRC | 19:39 | |
*** rmart04 has joined #puppet-openstack | 19:39 | |
openstackgerrit | Merged openstack/puppet-ec2api: Remove signing_dir option https://review.openstack.org/415311 | 19:41 |
*** Lokesh_Jain_ has joined #puppet-openstack | 19:41 | |
openstackgerrit | Pradeep Kilambi proposed openstack/puppet-ceilometer: Add deprecation warning to ceilo api https://review.openstack.org/416296 | 19:43 |
*** rmart04 has quit IRC | 19:50 | |
*** davidmichaelkarr has joined #puppet-openstack | 19:51 | |
openstackgerrit | Merged openstack/puppet-vitrage: Remove signing_dir option https://review.openstack.org/415365 | 19:52 |
*** NachoDuck has joined #puppet-openstack | 19:54 | |
openstackgerrit | Giulio Fidente proposed openstack/puppet-ceph: Improves support for ceph-mds https://review.openstack.org/402385 | 19:59 |
*** openstackgerrit has quit IRC | 20:03 | |
*** openstackgerrit has joined #puppet-openstack | 20:07 | |
openstackgerrit | Merged openstack/puppet-watcher: Add the port check for breaker https://review.openstack.org/414371 | 20:07 |
*** paramite has quit IRC | 20:28 | |
alee | mwhahaha, I commented on your comment in https://review.openstack.org/#/c/396789 -- does that make sense? | 20:28 |
mwhahaha | alee: yea i don't think you understand what happens when you run it again | 20:29 |
mwhahaha | alee: so by defining it that way in puppet, it's not idempotent because the second run will copy that file in place again | 20:29 |
mwhahaha | and then all the nova_join_config items will be rerun | 20:29 |
mwhahaha | alee: so that copy should have existed in packaging and we should not be doing that in puppet | 20:30 |
alee | mwhahaha, yes - but at the end of the second run, the file will look exactly the same as before. | 20:30 |
mwhahaha | but it will have performed changing actions | 20:31 |
mwhahaha | thus not idempotent | 20:31 |
mwhahaha | just because the file will be the same every time doesn't mean it's idempotent | 20:31 |
mwhahaha | the 2nd run should not do anything | 20:31 |
alee | mwhahaha, if you like I can add a "replace"= "no" in there. will that work? | 20:32 |
mwhahaha | i'd rather not do it at all | 20:32 |
mwhahaha | it's not something we should be doing in puppet if there's nothing gained from it | 20:32 |
alee | mwhahaha, ok - let me talk with rcrit .. | 20:33 |
mwhahaha | if it's to get a documented config file it doesn't really matter because the providers aren't going to put the config with the docs | 20:33 |
mwhahaha | and if there's some default config that needs to happen that file, then it needs to be put in place by the packaging | 20:34 |
alee | mwhahaha, let me show you whats in that file .. | 20:36 |
alee | mwhahaha, http://paste.openstack.org/show/593811/ | 20:36 |
alee | mwhahaha, thats what is delivered as the template file. | 20:37 |
alee | mwhahaha, and we set the parameters that are in CAPS | 20:37 |
mwhahaha | then that needs to be done as part of packaging | 20:37 |
mwhahaha | it's a config file with default values, so it needs to be provided when installed | 20:38 |
alee | mwhahaha, so you're saying that we need to deliver this file to /etc/nova/join.conf ? | 20:39 |
mwhahaha | yes | 20:39 |
alee | mwhahaha, what about the templated values? | 20:39 |
mwhahaha | they should be some default | 20:40 |
mwhahaha | like localhost or whatever | 20:40 |
mwhahaha | this is just like any other service, you get a config file with things commented out | 20:40 |
mwhahaha | also honestly all of these should be configurable | 20:41 |
alee | mwhahaha, ok - and as its a config file, it wont get automatically changed on rpm update if it has been customied .. | 20:41 |
mwhahaha | so what openstack services traditionally have is default values for all this stuff so if they aren't defined in the config it uses some default | 20:42 |
mwhahaha | the service should mirror that practice such that we only have to specify what we want from the defaults and then it wouldn't matter if you install this file or not | 20:42 |
mwhahaha | The use of a template or having to have these values defined in the configuration file is a bit concerning for me | 20:42 |
alee | mwhahaha, ok - I'll talk with rcrit when he returns and see if we can change the service to mirror that practice. | 20:44 |
alee | shouldn't be too hard' | 20:44 |
alee | (most of them already use defaults_ | 20:45 |
mwhahaha | yea i'd also recommend exposing all these options to be configurable. | 20:45 |
mwhahaha | that way you can also use the $::os_service_default pattern as well | 20:46 |
alee | mwhahaha, ok | 20:46 |
gfidente | mwhahaha, the ceph-mds package name | 20:54 |
gfidente | if I define it in the class the user can't change it though | 20:54 |
gfidente | what is the benefit compared to making it a param? | 20:54 |
mwhahaha | it's a param in the mds class | 20:54 |
mwhahaha | they can change it | 20:54 |
mwhahaha | you don't need it as a param in both | 20:54 |
mwhahaha | params.pp should be OS specific defaults | 20:54 |
gfidente | which in this case just happens to be the same for all | 20:55 |
mwhahaha | mds.pp uses the ceph::params::pkg_mds as the default | 20:55 |
gfidente | okay | 20:55 |
gfidente | but then the list of "base" packages is customizable | 20:55 |
mwhahaha | what | 20:56 |
gfidente | line 38 in params.pp | 20:56 |
gfidente | though it's not a "base" package | 20:56 |
gfidente | I am just not sure if it is more appropriate to remove it as class param from params.pp or from mds.pp | 20:57 |
mwhahaha | remove from params.pp | 20:57 |
gfidente | I think you are for keeping the param in mds.pp and remove from params.pp ? | 20:57 |
mwhahaha | leave in mds.pp | 20:57 |
gfidente | ok | 20:57 |
mwhahaha | define $pkg_mds in the params class | 20:57 |
gfidente | yep | 20:57 |
mwhahaha | https://github.com/openstack/puppet-cinder/blob/master/manifests/params.pp#L8 as an example | 20:58 |
openstackgerrit | Giulio Fidente proposed openstack/puppet-ceph: Improves support for ceph-mds https://review.openstack.org/402385 | 21:11 |
gfidente | tty :) | 21:12 |
*** gfidente has quit IRC | 21:12 | |
openstackgerrit | Merged openstack/puppet-neutron: Add enable_new_agents config option to neutron::server https://review.openstack.org/416214 | 21:25 |
*** morazi has quit IRC | 21:36 | |
openstackgerrit | Merged openstack/puppet-glance: Deprecate signing_dir option https://review.openstack.org/415313 | 21:43 |
*** toMeloos has quit IRC | 21:47 | |
*** ansmith has quit IRC | 21:48 | |
*** cdelatte has quit IRC | 21:48 | |
openstackgerrit | Pradeep Kilambi proposed openstack/puppet-nova: Add enabled_perf_events option for libvirt https://review.openstack.org/416282 | 21:57 |
*** agrebennikov has joined #puppet-openstack | 21:58 | |
*** dprince has quit IRC | 21:59 | |
*** spredzy has quit IRC | 22:10 | |
*** spredzy has joined #puppet-openstack | 22:12 | |
*** fultonj has quit IRC | 22:36 | |
EmilienM | dmsimard: can you please review https://review.openstack.org/#/c/416281/ ? | 22:39 |
EmilienM | mwhahaha: i'm still investigating why puppet-ceph hammer is broken - I'm also trying to use our spec_helper in stable/hammer https://review.openstack.org/#/c/414631/ | 22:40 |
EmilienM | Error: Could not start Service[radosgw-radosgw.gateway]: Execution of 'service radosgw start' returned 1: start: Unknown parameter: id | 22:40 |
EmilienM | also Error: Invalid parameter tenant on Keystone_user[testuser] | 22:41 |
EmilienM | it sounds like beaker tests are in bad shape | 22:41 |
EmilienM | (on mitaka/hammer) | 22:41 |
EmilienM | i'm very attempted to remove ceph rgw tests | 22:43 |
EmilienM | on this branch | 22:43 |
EmilienM | because they deploy juno, etc... Such a mess to refactor and I'm not sure it's worth it | 22:43 |
EmilienM | we have seen that rgw was tested & "stable" after hammer | 22:43 |
openstackgerrit | Emilien Macchi proposed openstack/puppet-ceph: Align stars to enable stable/hammer CI again https://review.openstack.org/414631 | 22:46 |
openstackgerrit | Pradeep Kilambi proposed openstack/puppet-ceilometer: Add deprecation warning to ceilo api https://review.openstack.org/416296 | 22:48 |
mwhahaha | EmilienM: I thought i saw that error previously | 22:51 |
mwhahaha | and it was a systemd thing or something | 22:51 |
EmilienM | mwhahaha: about https://review.openstack.org/#/c/411362/12/lib/puppet/provider/glance_image/openstack.rb - wdyt about denis's comment? | 22:52 |
mwhahaha | yea we should probably switch to the new param | 22:53 |
EmilienM | ok | 22:53 |
mwhahaha | also isn't visibility the one that they recently changed to be more than true/false | 22:53 |
mwhahaha | like shared or something as well | 22:54 |
dmsimard | EmilienM: why was it versioned at all in the initial implementation ? | 23:03 |
openstackgerrit | Merged openstack/puppet-cinder: Support for ScaleIO Cinder backend https://review.openstack.org/412086 | 23:03 |
dmsimard | i.e, is this something new some ocata ? | 23:03 |
EmilienM | dmsimard: I think I did that, and I don't know | 23:03 |
dmsimard | ok just trying to understand, was anything broken due to that ? | 23:04 |
*** colonwq has quit IRC | 23:05 | |
EmilienM | dmsimard: yes, we're trying to make autoscaling working in our gate | 23:05 |
EmilienM | dmsimard: and it works with keystone v3 | 23:05 |
dmsimard | Ok. | 23:05 |
dmsimard | sounds like an easy backport to newton https://github.com/openstack/puppet-openstack-integration/blob/stable/newton/manifests/aodh.pp#L78 | 23:06 |
EmilienM | not sure I'll backport it | 23:07 |
EmilienM | I want to start removing ceilometer api in our gate | 23:07 |
EmilienM | it's deprecated in ocata | 23:07 |
openstackgerrit | Emilien Macchi proposed openstack/puppet-oslo: Update olso.log parameters https://review.openstack.org/410467 | 23:08 |
openstackgerrit | Emilien Macchi proposed openstack/puppet-ceph: repos: allow to configure Debian mirror https://review.openstack.org/410455 | 23:08 |
*** _ducttape_ has joined #puppet-openstack | 23:09 | |
openstackgerrit | Emilien Macchi proposed openstack/puppet-openstack-integration: Remove testing for Glance Registry https://review.openstack.org/410448 | 23:09 |
EmilienM | I'm also investigating this error: http://logs.openstack.org/01/406301/10/check/gate-puppet-openstack-integration-4-scenario003-tempest-centos-7/527aac2/console.html#_2017-01-03_02_49_05_598167 | 23:11 |
EmilienM | on scenario003 with https://review.openstack.org/#/c/406301 | 23:11 |
*** jperry has quit IRC | 23:11 | |
EmilienM | oh it sounds like it overlaps with horizon | 23:11 |
*** ducttape_ has quit IRC | 23:12 | |
*** openstack has joined #puppet-openstack | 23:15 | |
*** vinsh has quit IRC | 23:28 | |
*** saneax-_-|AFK is now known as saneax | 23:29 | |
*** michapma_alt has joined #puppet-openstack | 23:40 | |
*** michchap_alt has joined #puppet-openstack | 23:40 | |
openstackgerrit | Merged openstack/puppet-openstack-integration: aodh: don't hardcode keystone endpoint version https://review.openstack.org/416281 | 23:47 |
*** ducttape_ has joined #puppet-openstack | 23:57 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!