openstackgerrit | Emilien Macchi proposed openstack/puppet-nova: CI test - will fail https://review.openstack.org/297231 | 00:09 |
---|---|---|
openstackgerrit | Emilien Macchi proposed openstack/puppet-openstack_spec_helper: Test puppet-nova Gemfile change https://review.openstack.org/297233 | 00:09 |
openstackgerrit | Emilien Macchi proposed openstack/puppet-openstack_spec_helper: Test puppet-nova Gemfile change https://review.openstack.org/297233 | 00:16 |
openstackgerrit | Emilien Macchi proposed openstack/puppet-tripleo: Add Glance profiles https://review.openstack.org/296076 | 00:20 |
openstackgerrit | Emilien Macchi proposed openstack/puppet-openstack-integration: revert temporary workaround on centos https://review.openstack.org/292460 | 00:21 |
openstackgerrit | Emilien Macchi proposed openstack/puppet-keystone: CI test - dnm https://review.openstack.org/298067 | 00:23 |
*** cwolferh has quit IRC | 00:24 | |
openstackgerrit | Emilien Macchi proposed openstack/puppet-tripleo: IPv6: dual-stack support for Keystone https://review.openstack.org/286344 | 00:25 |
*** saneax is now known as saneax_AFK | 00:39 | |
openstackgerrit | Merged openstack/puppet-tempest: Updates for stable/mitaka https://review.openstack.org/297417 | 00:59 |
*** ducttape_ has joined #puppet-openstack | 01:07 | |
*** ducttape_ has quit IRC | 01:16 | |
*** ilbot3 has quit IRC | 01:45 | |
*** ilbot3 has joined #puppet-openstack | 01:48 | |
*** moravec has joined #puppet-openstack | 02:10 | |
*** rmallah has joined #puppet-openstack | 02:23 | |
openstackgerrit | guessi proposed openstack/puppet-openstack-integration: Fix package version on Fedora https://review.openstack.org/288619 | 02:24 |
rmallah | in a new keystone deployment using puppet after initial db and keystone setup , keystone_domain { 'domain_one': ensure => present } | 02:25 |
rmallah | is not working | 02:25 |
rmallah | Error: Could not prefetch keystone_domain provider 'openstack': Execution of '/usr/bin/openstack domain list --quiet --format csv' returned 1: ERROR: openstack Authorization failed: The resource could not be found. (HTTP 404) | 02:26 |
rmallah | how to create initial projects , domain , user , roles ? | 02:26 |
mfisch | EmilienM: do you have permissions to close https://bugs.launchpad.net/puppet-openstack/+bug/1290385 | 03:54 |
openstack | Launchpad bug 1290385 in puppet-openstack "$openstack::compute::neutron_auth_url does not propagate to nova.conf" [Undecided,New] | 03:54 |
mfisch | crinkle: can you add me to this team? https://launchpad.net/~puppet-openstack | 03:55 |
*** saneax_AFK is now known as saneax | 03:56 | |
crinkle | mfisch: done | 03:58 |
mfisch | ty | 03:58 |
*** dgurtner has joined #puppet-openstack | 04:10 | |
*** dgurtner has quit IRC | 04:10 | |
*** dgurtner has joined #puppet-openstack | 04:10 | |
*** dgurtner has quit IRC | 04:19 | |
*** cdelatte has joined #puppet-openstack | 04:26 | |
*** aimon has quit IRC | 04:28 | |
*** delattec has joined #puppet-openstack | 04:28 | |
*** cdelatte has quit IRC | 04:31 | |
*** rmallah has quit IRC | 04:49 | |
*** rmallah has joined #puppet-openstack | 04:53 | |
*** aimon has joined #puppet-openstack | 05:14 | |
*** aimon has quit IRC | 05:19 | |
*** delatte has joined #puppet-openstack | 05:32 | |
*** delattec has quit IRC | 05:36 | |
*** fedexo has joined #puppet-openstack | 05:41 | |
*** rmallah has quit IRC | 05:47 | |
*** rmallah has joined #puppet-openstack | 05:52 | |
*** aimon has joined #puppet-openstack | 06:08 | |
*** aimon has quit IRC | 06:13 | |
*** fedexo has quit IRC | 06:41 | |
rmallah | can anyone please help me with : Error: Could not prefetch keystone_tenant provider 'openstack': Execution of '/usr/bin/openstack project list --quiet --format csv --long' returned 1: The resource could not be found. (HTTP 404) | 07:04 |
rmallah | i get this when i run puppet agent -t -v on controller node | 07:05 |
openstackgerrit | Liao Penghui proposed openstack/puppet-keystone: Fix bugs for puppet fail when missing endpoint https://review.openstack.org/294506 | 07:18 |
Mosibi_ | rmallah: look in de keystone logfile for errors.. | 07:54 |
*** Mosibi_ is now known as Mosibi | 07:54 | |
*** aimon has joined #puppet-openstack | 07:57 | |
*** aimon has quit IRC | 08:01 | |
rmallah | there are 404s for requests for domains. | 08:14 |
rmallah | eg:GET /v2.0/domains?name=Default HTTP/1.1" 404 318 0.001160 | 08:15 |
rmallah | does openstack/puppet-* manage apt sources to get compatible openstack versions ? | 08:21 |
rmallah | on the node in which puppet agent is run | 08:21 |
openstackgerrit | Michael Polenchuk proposed openstack/puppet-nova: Autorequire nova security group https://review.openstack.org/296548 | 08:49 |
*** Tesrte has joined #puppet-openstack | 09:12 | |
skolekonov|away | rmallah, hi! Can you try to set auth urls without versions? Keystone should discover them | 09:19 |
*** skolekonov|away is now known as skolekonov | 09:19 | |
*** Tesrte has quit IRC | 09:30 | |
*** rmallah has quit IRC | 09:36 | |
*** igajsin1 has joined #puppet-openstack | 09:44 | |
*** aimon has joined #puppet-openstack | 09:45 | |
*** igajsin1 has left #puppet-openstack | 09:45 | |
*** aimon has quit IRC | 09:49 | |
*** rmallah has joined #puppet-openstack | 09:51 | |
*** rmallah has quit IRC | 09:52 | |
openstackgerrit | Oleksiy Molchanov proposed openstack/puppet-keystone: Fix for pki_setup https://review.openstack.org/298183 | 09:58 |
*** trown|outtypewww is now known as trown|PTO | 11:22 | |
*** aimon has joined #puppet-openstack | 11:33 | |
*** aimon has quit IRC | 11:38 | |
*** foxhunt_away is now known as foxhunt72 | 12:03 | |
*** ducttape_ has joined #puppet-openstack | 12:10 | |
*** jayg|g0n3 is now known as jayg | 12:19 | |
openstackgerrit | Oleksiy Molchanov proposed openstack/puppet-keystone: Fix for pki_setup https://review.openstack.org/298183 | 12:22 |
*** dprince has joined #puppet-openstack | 12:24 | |
*** ducttape_ has quit IRC | 12:24 | |
openstackgerrit | Merged openstack/puppet-openstack-integration: Fix bug and non-idiom in bash scripts. https://review.openstack.org/297114 | 12:24 |
*** _degorenko is now known as degorenko | 12:32 | |
degorenko | zigo, ping | 12:33 |
*** johnmilton has joined #puppet-openstack | 13:12 | |
openstackgerrit | Denis Egorenko proposed openstack/puppet-glance: Add posibility to configure Glance backend settings for Glare service https://review.openstack.org/297795 | 13:17 |
*** aimon has joined #puppet-openstack | 13:21 | |
*** ayoung has joined #puppet-openstack | 13:22 | |
*** aimon has quit IRC | 13:26 | |
*** vinsh has quit IRC | 13:28 | |
*** vinsh has joined #puppet-openstack | 13:29 | |
*** vinsh_ has joined #puppet-openstack | 13:30 | |
*** vinsh has quit IRC | 13:30 | |
*** openstackgerrit has quit IRC | 13:48 | |
*** openstackgerrit has joined #puppet-openstack | 13:48 | |
*** richm has joined #puppet-openstack | 13:50 | |
*** ducttape_ has joined #puppet-openstack | 13:52 | |
*** dfisher has joined #puppet-openstack | 14:05 | |
*** andreww has quit IRC | 14:10 | |
*** aimon has joined #puppet-openstack | 14:15 | |
openstackgerrit | David Moreau Simard proposed openstack/puppet-neutron: Fix LinuxBridge Agent configuration file name https://review.openstack.org/298254 | 14:25 |
openstackgerrit | David Moreau Simard proposed openstack/puppet-neutron: Fix LinuxBridge Agent configuration file name https://review.openstack.org/298254 | 14:29 |
openstackgerrit | David Moreau Simard proposed openstack/puppet-neutron: Set default branch to stable/liberty https://review.openstack.org/298260 | 14:31 |
*** andreww has joined #puppet-openstack | 14:37 | |
*** markvoelker has joined #puppet-openstack | 14:38 | |
alee | degorenko, morning | 14:45 |
*** dhollinger has joined #puppet-openstack | 14:45 | |
alee | degorenko, which parameters are you talking about here? https://review.openstack.org/#/c/296159/2/spec/classes/barbican_api_spec.rb | 14:45 |
degorenko | alee, good day :) | 14:46 |
alee | degorenko, just going through your comments .. hopefully we can get all these ironed out today .. | 14:46 |
degorenko | alee, for passed for example rabbit_host | 14:47 |
degorenko | or whatever else | 14:47 |
degorenko | you have a bunch of parameters, which can be passed to puppet manifest | 14:47 |
degorenko | now your test cover only default values | 14:47 |
alee | degorenko, yeah ok | 14:48 |
alee | degorenko, in https://review.openstack.org/#/c/296162/2/manifests/api.pp , I'll go ahead and implement the new parameters | 14:49 |
openstackgerrit | Dmitry Burmistrov proposed openstack/puppet-oslo: Rework oslo-cache class https://review.openstack.org/298273 | 14:50 |
alee | degorenko, to do so though - I need to know how to implement multicfg options .. | 14:50 |
openstackgerrit | Merged openstack/puppet-openstacklib: acceptance: do not enable EPEL https://review.openstack.org/297978 | 14:50 |
alee | degorenko, do you happen to have a link or example of how I can do that? I know they were recently added .. | 14:50 |
alee | EmilienM, ^^ | 14:51 |
degorenko | alee, it's bad practice when you're trying to implement some feature that's not implemented in code yet | 14:51 |
degorenko | alee, these parameters already added? | 14:51 |
alee | degorenko, sure -- the plan was to add a patch right after this set of patches | 14:51 |
alee | degorenko, these parameters are already in the underlying barbican patches | 14:52 |
alee | I mean barbican package | 14:52 |
*** foxhunt72 is now known as foxhunt_away | 14:52 | |
alee | I just did not - at the time - have the ability to enable them in the pupet modules | 14:52 |
alee | becvause they are oslo multicfg options | 14:52 |
alee | ie. I can define the same option multiple times in the config file | 14:53 |
degorenko | alee, can you please provide me example? like, part of config file with these parameters, i didn't get it now , sorry | 14:53 |
alee | so for intance .. | 14:54 |
alee | if I have the parameter .. enabled_plugins = ['foo', 'bar'] | 14:54 |
alee | this will actually show up in the config file as .. | 14:54 |
alee | enabled_plugin = foo | 14:55 |
degorenko | alee no :) | 14:55 |
alee | enabled_plugin = bar | 14:55 |
degorenko | if you need to have enabled_plugin=foo,bar | 14:55 |
degorenko | it is can be easily done by having array for parameter enabled_plugin - as in your example | 14:55 |
alee | ie. the same directive shows up twice .. with different values | 14:55 |
degorenko | and | 14:55 |
degorenko | enabled_plugin = join($enabled_plugin, ',') | 14:56 |
degorenko | it will be written to config as enabled_plugin=foo,bar | 14:56 |
alee | yeah - thats not how it needs to show up | 14:56 |
degorenko | ah | 14:56 |
degorenko | i see | 14:57 |
degorenko | you need 2 different options | 14:57 |
degorenko | on by one right? | 14:57 |
degorenko | one* | 14:57 |
alee | right - it shows up as | 14:58 |
degorenko | i got it now | 14:58 |
degorenko | well, it will be a little bit tricky then | 14:58 |
alee | so for instance -- option = foo, bar will be two lines : option = foo and option == bar | 14:58 |
degorenko | yep | 14:58 |
alee | its something called an oslo Multicfg I think -- looking for link | 14:59 |
alee | degorenko, EmilienM mentioned that it was being implemented in puppet .. | 14:59 |
degorenko | so, for barbican_config provider the problem is that it can manage only one property for each section, we can't do this like you want through standard config provider | 14:59 |
degorenko | hm | 14:59 |
degorenko | really? | 14:59 |
degorenko | let me check | 14:59 |
alee | degorenko, anyways thats the reason the options are not implemented in puppet yet -- they are in the barbican package, but I did not know how to implement in puppet. | 15:01 |
openstackgerrit | Kyrylo Galanov proposed openstack/puppet-swift: Add rmq credentials to each host in proxy https://review.openstack.org/297618 | 15:02 |
alee | degorenko, http://docs.openstack.org/developer/oslo.config/opts.html?highlight=multi#oslo_config.cfg.MultiOpt | 15:02 |
degorenko | alee, i don't see any patches for implementing multiple same parameters in config files | 15:02 |
*** darkxploit has joined #puppet-openstack | 15:03 | |
degorenko | alee, no, i'm wrong, we have some patch with mentioned by you doc | 15:04 |
degorenko | https://github.com/openstack/puppet-openstacklib/commit/029c6a74cce2dceff5ae78c138a050d9eb7a6f9b | 15:04 |
alee | degorenko, ah good | 15:04 |
degorenko | alee, but, i don't know how it works, i didn't see that before | 15:04 |
alee | degorenko, cool - I'll try to tease out the needed syntax from the spec tests .. | 15:06 |
degorenko | alee, i don't see unit tests for this.... :( | 15:07 |
degorenko | oh, now | 15:07 |
degorenko | https://github.com/openstack/puppet-openstacklib/blob/master/spec/unit/puppet/util/openstackconfig_spec.rb | 15:07 |
degorenko | looks like we need that ^ | 15:07 |
alee | degorenko, right | 15:07 |
degorenko | alee, go ahead then :) | 15:09 |
alee | degorenko, yup - fun fun! I'm going to try to get that in today. | 15:09 |
alee | (assuming I dont get pulled off on something else) | 15:10 |
openstackgerrit | Merged openstack/puppet-trove: trivial comment paste-o fix https://review.openstack.org/297340 | 15:11 |
*** saneax is now known as saneax_AFK | 15:15 | |
*** michchap_ has quit IRC | 15:18 | |
Anticimex | hm | 15:19 |
Anticimex | puppet-ceph claims support for the ceph infernalis release, but there is no systemd code in it (ceph infernalis moved start scripts to systemd) | 15:19 |
-openstackstatus- NOTICE: Gerrit is restarting on review.openstack.org in an attempt to address an issue reading an object from the ec2-api repository | 15:22 | |
EmilienM | hello | 15:27 |
degorenko | hey o/ | 15:28 |
*** michchap has joined #puppet-openstack | 15:28 | |
*** EmilienM sets mode: +o degorenko | 15:28 | |
EmilienM | degorenko: CI was in a bad shape this week end | 15:30 |
EmilienM | for different reasons | 15:30 |
EmilienM | only one is remaining AFIK: sahara looks broken on Ubuntu | 15:30 |
degorenko | EmilienM, also i found a new problem with sahara package :( | 15:30 |
EmilienM | yeah, we should ping zigo maybe | 15:30 |
degorenko | i tried to ping him today | 15:31 |
degorenko | but no responce | 15:31 |
EmilienM | yeah it's PTO for many of us (me included) | 15:31 |
degorenko | oh, good to know | 15:31 |
*** mdorman has joined #puppet-openstack | 15:39 | |
*** aimon has quit IRC | 15:43 | |
*** aimon has joined #puppet-openstack | 15:43 | |
EmilienM | degorenko: do you know if service notify would restart a service if the resource is set to ensure => stopped? | 15:50 |
EmilienM | we might have found a bug but I'm not sure | 15:50 |
degorenko | well, we have a new state, i think yes | 15:50 |
degorenko | so, you have one service which was stopped and it notified another service? | 15:51 |
EmilienM | degorenko: https://bugs.launchpad.net/puppet-neutron/+bug/1562922 | 15:51 |
openstack | Launchpad bug 1562922 in puppet-neutron "neutron-server service get notify even when we don't manage service" [Undecided,New] | 15:51 |
* degorenko looking | 15:52 | |
degorenko | hmm | 15:53 |
degorenko | i'm not sure too | 15:55 |
degorenko | but it should not | 15:55 |
EmilienM | degorenko: it should not restart? | 15:57 |
EmilienM | because it does for sure | 15:57 |
EmilienM | we have the bug in tripleo, when upgrading, jayg can confirm | 15:57 |
jayg | yes, the tripleo folks are definitely see this behavior even with manager_service and enabled set to false | 15:58 |
skolekonov | systemd doesn't start services automatically. right? | 15:59 |
EmilienM | no, but I think the ~> runs a restart | 15:59 |
degorenko | ~> also means hard dependency | 16:00 |
degorenko | so in one catalog we will have config changes at all | 16:00 |
degorenko | and only then we will manage service | 16:00 |
EmilienM | I haven't read puppet core code yet but we might need to check | 16:01 |
skolekonov | I'll check | 16:01 |
skolekonov | managed should be set to true also as I remember | 16:01 |
skolekonov | and enabled => false | 16:01 |
EmilienM | right, the systemd service should be disabled | 16:02 |
*** cwolferh has joined #puppet-openstack | 16:04 | |
skolekonov | I checked on Ubuntu. The service (neutron-server) was disabled before Puppet run, then I applied a manifest and it was not restarted. My example http://paste.openstack.org/show/492099/ | 16:07 |
*** andreww is now known as xarses | 16:08 | |
EmilienM | skolekonov: right | 16:08 |
EmilienM | skolekonov: can you run puppet again to make sure it's idempotent? | 16:09 |
jayg | yes, but the service _is_ running in this case, just not managed by puppet | 16:09 |
EmilienM | jayg: the service is managed by pacemaker | 16:09 |
jayg | yes, that is what I am meaning | 16:09 |
EmilienM | jayg: but in your case, the service should be systemd-disabled | 16:10 |
jayg | so it probably gets the status correctly as running and tries to restart | 16:10 |
jayg | you don't think it would report it as running? I seem to remember it doing so | 16:10 |
jayg | even though pacemaker is the one controlling it | 16:10 |
jayg | maybe I am remembering wrong, it was a while ago for me | 16:11 |
skolekonov | EmilienM, as it's an exec it's executed every time, but service is stopped | 16:12 |
EmilienM | skolekonov: right. I wonder if pacemaker enables it in systemd | 16:13 |
skolekonov | You mean it can be connected with pacemaker provider? | 16:14 |
EmilienM | skolekonov: we don't have pacemaker provider | 16:14 |
EmilienM | that manage the service | 16:14 |
EmilienM | we're working on it | 16:14 |
openstackgerrit | Merged openstack/puppet-keystone: Fix for pki_setup https://review.openstack.org/298183 | 16:14 |
EmilienM | jayg: I'm looking at this code now https://github.com/openstack/puppet-pacemaker/blob/master/manifests/resource/service.pp | 16:15 |
* jayg looks | 16:16 | |
EmilienM | it actually create a systemd resource in pacemaker with https://github.com/openstack/puppet-pacemaker/blob/master/manifests/resource/systemd.pp | 16:16 |
EmilienM | so my guess is that it re-creates what puppet disabled | 16:17 |
EmilienM | I'm not familiar enough with pacemaker though | 16:17 |
jayg | yes, as I recall, pacemaker uses the systemd unit files, but it supposed to pass in some kind of extra flag to tell systemd to not manage it | 16:17 |
jayg | this is part of the resource-agents magic iirc | 16:18 |
EmilienM | jayg: where is code of that? | 16:18 |
jayg | let me see if I have a checkout, I am not sure if I ever got the code other than via rpm when we were doing foreman installs | 16:18 |
jayg | EmilienM: https://github.com/ClusterLabs/resource-agents looks like it, though I am not seeing a systemd one right off, looking more now.. | 16:20 |
EmilienM | jayg: looks like https://github.com/ClusterLabs/resource-agents/blob/01352b5a28c3da04436fd19d886dacdce0a8c1a8/ldirectord/systemd/ldirectord.service.in | 16:22 |
EmilienM | or not | 16:22 |
EmilienM | anyway, it does not sounds like something in puppet-neutron | 16:22 |
EmilienM | skolekonov, degorenko: thx guys | 16:22 |
skolekonov | np | 16:23 |
degorenko | yw :) | 16:23 |
*** darkxploit has quit IRC | 16:26 | |
*** fedexo has joined #puppet-openstack | 16:27 | |
*** darkxploit has joined #puppet-openstack | 16:38 | |
*** dfisher has quit IRC | 16:42 | |
*** dfisher has joined #puppet-openstack | 16:42 | |
*** fedexo has quit IRC | 16:49 | |
*** skolekonov is now known as skolekonov|away | 16:53 | |
xarses | dmsimard, please review https://review.openstack.org/#/c/297393/ | 16:58 |
dmsimard | xarses: EmilienM told me about it last week | 16:58 |
dmsimard | xarses: I've actually only ever installed Ceph on top of Ubuntu | 16:59 |
dmsimard | I don't know why EPEL would be necessary at all or why it would be installed by default | 16:59 |
dmsimard | dgurtner or loic might know | 16:59 |
xarses | it used to be necessary, as all of the deps where not present | 16:59 |
dmsimard | the patch itself is fine and backwards compatible, I'm just questioning whether the entire block should not be removed | 17:00 |
xarses | alot has changed since RH acquired Inktank | 17:00 |
dmsimard | xarses: the dependencies might not have been self contained in the ceph repo, you mean ? | 17:00 |
xarses | correct, they where not in either mainline or ceph repo | 17:01 |
dmsimard | xarses: eh, I can submit a dummy patch to see what happens if I tear that out | 17:01 |
openstackgerrit | David Moreau Simard proposed openstack/puppet-ceph: Do not merge: Test without EPEL https://review.openstack.org/298332 | 17:04 |
dmsimard | xarses: ^ | 17:04 |
*** darkxploit has quit IRC | 17:09 | |
*** foxhunt_away is now known as foxhunt72 | 17:22 | |
EmilienM | degorenko: https://review.openstack.org/#/c/292460/2 seems ok now, it's passing on https://review.openstack.org/#/c/298067/ | 17:27 |
degorenko | nice | 17:28 |
EmilienM | degorenko: I replied on https://review.openstack.org/#/c/298047/ | 17:29 |
EmilienM | degorenko: I copied pasted the file from other projects | 17:30 |
degorenko | EmilienM, ack | 17:30 |
degorenko | do we need to update README.rst with proper command? | 17:31 |
degorenko | or just base commit only | 17:31 |
degorenko | ? | 17:31 |
EmilienM | degorenko: commend is good no? | 17:31 |
degorenko | is it just tox? | 17:32 |
degorenko | without any build_sphinx? | 17:32 |
degorenko | i'm not familiar with approach of building docs for repos | 17:33 |
degorenko | just some experience from package building, i may wrong | 17:33 |
degorenko | may be* | 17:33 |
EmilienM | degorenko: try it | 17:35 |
EmilienM | degorenko: run tox, and it builds everything | 17:35 |
degorenko | it builds env | 17:35 |
degorenko | ok, i trust you :) | 17:36 |
EmilienM | yeah | 17:36 |
EmilienM | and run "firefox doc/build/html/index.html" | 17:36 |
EmilienM | you can see the output | 17:36 |
degorenko | i will try, give me a moment :) | 17:36 |
degorenko | EmilienM, cool, works :) | 17:39 |
EmilienM | cool | 17:39 |
EmilienM | dmsimard: it fails without epel I think | 17:39 |
EmilienM | 2016-03-28 17:13:40.086 | Error: Execution of '/usr/bin/yum -d 0 -e 0 -y install ceph' returned 1: Error: Package: 1:ceph-0.94.6-0.el7.x86_64 (ext-ceph-hammer) | 17:39 |
EmilienM | 2016-03-28 17:13:40.087 | Requires: libleveldb.so.1()(64bit) | 17:39 |
EmilienM | 2016-03-28 17:13:40.087 | You could try using --skip-broken to work around the problem | 17:39 |
EmilienM | https://jenkins02.openstack.org/job/gate-puppet-ceph-puppet-beaker-rspec-devstack-centos7/1/consoleFull | 17:39 |
*** dtantsur is now known as dtantsur|afk | 17:39 | |
dmsimard | ... ew | 17:43 |
bkero | wat | 17:48 |
dmsimard | I'm asking about it to the Ceph guys | 17:49 |
xarses | EmilienM: if it's still broken, why do you want to remove the repo then? | 17:54 |
EmilienM | xarses: I want to be allowed to remove the repo | 17:54 |
EmilienM | which is why our modules are flexible | 17:54 |
EmilienM | the fact it does not work without EPEL is a separated issue | 17:54 |
xarses | that's what I figured | 17:54 |
EmilienM | my patch is backward compatible, since it enables it by default like before. | 17:55 |
xarses | yep, I'm still fine with it thanks | 17:57 |
degorenko | EmilienM, mwhahaha can you guys, please, take a look on this: https://review.openstack.org/#/c/297795/ :) | 17:57 |
EmilienM | crinkle: hi, I don't have strong opinion on https://review.openstack.org/#/c/297382/2/reference/projects.yaml but even if we don't release it, it has stable branches every 6 months, so I think it applies to the tags, isn't? | 17:58 |
EmilienM | degorenko: I'll look later | 17:59 |
degorenko | ыгку | 17:59 |
degorenko | oops | 17:59 |
degorenko | sure :D | 17:59 |
*** ericksonsantos has quit IRC | 18:07 | |
*** degorenko is now known as _degorenko|afk | 18:15 | |
crinkle | EmilienM: it has stable branches? | 18:23 |
EmilienM | crinkle: yes | 18:23 |
EmilienM | crinkle: https://github.com/openstack/puppet-openstack_spec_helper/branches | 18:24 |
EmilienM | because the lib needs to know what repos to checkout | 18:24 |
crinkle | EmilienM: okay i see why | 18:25 |
crinkle | EmilienM: +1 | 18:25 |
EmilienM | crinkle: thx! | 18:26 |
openstackgerrit | Merged openstack/puppet-openstack-guide: First commit for Puppet OpenStack Guide https://review.openstack.org/298047 | 18:28 |
dmsimard | EmilienM, xarses: re: EPEL requirement | 18:30 |
* dmsimard is tired from all the digging | 18:30 | |
dmsimard | This is indeed an issue only with the official upstream repositories (which is kind of sad, actually) | 18:30 |
dmsimard | CentOS is working towards providing repositories for Ceph through it's storage SIG initiative (https://wiki.centos.org/SpecialInterestGroup/Storage/) and this repository has the leveldb package baked in (ex: http://cbs.centos.org/repos/storage7-ceph-hammer-candidate/x86_64/os/Packages/) | 18:31 |
openstackgerrit | Cody Herriges proposed openstack/puppet-openstack-integration: Add explicit dependencies to pass puppet4 https://review.openstack.org/296557 | 18:33 |
EmilienM | http://docs.openstack.org/developer/puppet-openstack-guide/ is alive ! | 18:34 |
openstackgerrit | Cody Herriges proposed openstack/puppet-openstack-integration: Add explicit dependencies to pass puppet4 https://review.openstack.org/296557 | 18:34 |
*** foxhunt72 is now known as foxhunt_away | 18:36 | |
dmsimard | xarses: I documented the issue here https://bugs.launchpad.net/puppet-ceph/+bug/1562998 | 18:38 |
openstack | Launchpad bug 1562998 in puppet-ceph "EPEL requirement on CentOS is only because of leveldb" [Undecided,New] | 18:38 |
openstackgerrit | Cody Herriges proposed openstack/puppet-keystone: Ensure endpoints created before admin_token_auth https://review.openstack.org/298373 | 18:50 |
openstackgerrit | Cody Herriges proposed openstack/puppet-openstack-integration: Add explicit dependencies to pass puppet4 https://review.openstack.org/296557 | 18:51 |
openstackgerrit | Cody Herriges proposed openstack/puppet-openstack-integration: Add explicit dependencies to pass puppet4 https://review.openstack.org/296557 | 18:51 |
openstackgerrit | Cody Herriges proposed openstack/puppet-keystone: Ensure endpoints created before admin_token_auth https://review.openstack.org/298373 | 18:56 |
*** brandon has joined #puppet-openstack | 19:02 | |
*** aimon has quit IRC | 19:18 | |
*** aimon has joined #puppet-openstack | 19:19 | |
*** jtsigma has joined #puppet-openstack | 19:22 | |
*** angdraug has joined #puppet-openstack | 19:22 | |
jtsigma | can someone explain to me where in the puppet-ceilometer code (as an example), does it call in the mysql manifest?: https://github.com/openstack/puppet-ceilometer/blob/master/manifests/db/mysql.pp | 19:23 |
jtsigma | i tried reading through the code, and don't see where that particular manifest gets executed | 19:23 |
EmilienM | jtsigma: it is not, you have to declare it in your composition layer | 19:24 |
jtsigma | oooh, ok that makes sense | 19:24 |
jtsigma | thanks EmilienM | 19:24 |
EmilienM | np! | 19:25 |
*** foxhunt_away is now known as foxhunt72 | 19:30 | |
openstackgerrit | Cody Herriges proposed openstack/puppet-cinder: Establish cinder_type after keystone_endpoint https://review.openstack.org/298397 | 19:32 |
openstackgerrit | Cody Herriges proposed openstack/puppet-openstack-integration: Add explicit dependencies to pass puppet4 https://review.openstack.org/296557 | 19:34 |
*** tlbr has quit IRC | 19:53 | |
openstackgerrit | Cody Herriges proposed openstack/puppet-cinder: Establish cinder_type after keystone_endpoint https://review.openstack.org/298397 | 19:59 |
*** foxhunt72 is now known as foxhunt_away | 20:00 | |
openstackgerrit | Merged openstack/puppet-ceph: repo: allow to disable EPEL repository https://review.openstack.org/297393 | 20:04 |
*** johnmilton has quit IRC | 20:09 | |
*** dprince has quit IRC | 20:17 | |
*** jayg is now known as jayg|g0n3 | 20:28 | |
openstackgerrit | Merged openstack/puppet-oslo: Fix values for rabbit_hosts https://review.openstack.org/293964 | 20:30 |
_ody | Is it just me or is CI really slow today? | 20:36 |
xarses | seems slow | 20:36 |
EmilienM | _ody: what is slow? | 20:38 |
EmilienM | getting a node? | 20:38 |
_ody | EmilienM: Running the actual tests. | 20:39 |
_ody | I keep walking around the office thinking it is my wifi. | 20:39 |
EmilienM | _ody: I haven't noticed long runs | 20:39 |
EmilienM | _ody: do you have logs handy to see it's long? | 20:39 |
EmilienM | where do you see it's long? | 20:39 |
_ody | EmilienM: I am just surprised my last change to puppet-cinder has been running tests for over 40 minutes. | 20:41 |
EmilienM | _ody: https://review.openstack.org/#/c/298397/ ? | 20:41 |
EmilienM | it's almost finished: https://jenkins07.openstack.org/job/gate-puppet-openstack-integration-3-scenario002-tempest-devstack-centos7/42/console | 20:42 |
EmilienM | 35 min | 20:42 |
EmilienM | it's average | 20:42 |
EmilienM | it usually takes ~33 min | 20:42 |
_ody | EmilienM:How did you figure out which jenkins that was running on? | 20:43 |
EmilienM | http://status.openstack.org/zuul/ | 20:43 |
_ody | o.O | 20:43 |
_ody | I knew that existed just never noticed how to get to jenkins from it. | 20:44 |
EmilienM | _ody: "TIL" :-) | 20:45 |
_ody | oh. I just didn't click it the right place it would seem. | 20:46 |
mwhahaha | http://cdn.meme.am/instances/66957857.jpg | 20:50 |
EmilienM | mwhahaha: I use this picture when my friends (working on windows) complain about their bugs | 20:52 |
mwhahaha | :D | 20:52 |
EmilienM | mwhahaha: also this one: https://goo.gl/Gt7F2l | 20:53 |
*** ericksonsantos has joined #puppet-openstack | 21:02 | |
openstackgerrit | Emilien Macchi proposed openstack/puppet-openstack-integration: scenario003: deploy Neutron with ML2 linuxbridge https://review.openstack.org/296102 | 21:02 |
_ody | maybe I'll just need to finish up the puppet4 stuff tomorrow. It's taking me 15 minutes to clone a single repo from git.openstack.org. | 21:07 |
openstackgerrit | Merged openstack/puppet-glance: Add posibility to configure Glance backend settings for Glare service https://review.openstack.org/297795 | 21:07 |
EmilienM | _ody: have you talked to #openstack-infra folks? | 21:07 |
* _ody is hung on running "git review" now | 21:07 | |
_ody | EmilienM: Was reading through scroll and and asking folks. Haven't broadcasted the issue yet. | 21:08 |
mwhahaha | do we have an example of the glance_image provider usage anywhere | 21:12 |
*** johnmilton has joined #puppet-openstack | 21:13 | |
mwhahaha | nm it's in the acceptance scripts | 21:14 |
EmilienM | mwhahaha: https://github.com/openstack/puppet-openstack-integration/blob/master/manifests/provision.pp#L67-L87 | 21:14 |
mwhahaha | gracias | 21:15 |
openstackgerrit | Merged openstack/puppet-tripleo: Redirect to https for horizon https://review.openstack.org/294456 | 21:16 |
*** johnmilton has quit IRC | 21:16 | |
*** johnmilton has joined #puppet-openstack | 21:17 | |
*** tlbr has joined #puppet-openstack | 21:18 | |
*** johnmilton has quit IRC | 21:21 | |
*** johnmilton has joined #puppet-openstack | 21:23 | |
*** aimon has quit IRC | 21:29 | |
*** aimon has joined #puppet-openstack | 21:29 | |
openstackgerrit | Merged openstack/puppet-openstack-integration: revert temporary workaround on centos https://review.openstack.org/292460 | 21:33 |
*** johnmilton has quit IRC | 21:42 | |
openstackgerrit | Merged openstack/puppet-swift: Add rmq credentials to each host in proxy https://review.openstack.org/297618 | 21:47 |
*** johnmilton has joined #puppet-openstack | 21:55 | |
EmilienM | crinkle: when you have some time, please look https://review.openstack.org/#/c/296102/ - I remember you deployed linuxbridge for infra cloud, thanks! | 21:55 |
openstackgerrit | Merged openstack/puppet-cinder: Add osapi_max_limit parameter https://review.openstack.org/296931 | 21:57 |
crinkle | EmilienM: +2 | 21:59 |
openstackgerrit | Alex Krzos proposed openstack/puppet-keystone: Tune number of apache processes/threads for Keystone in apache. https://review.openstack.org/297342 | 21:59 |
openstackgerrit | Alex Krzos proposed openstack/puppet-keystone: Tune number of apache processes/threads for Keystone in apache. https://review.openstack.org/297342 | 22:01 |
*** johnmilton has quit IRC | 22:07 | |
openstackgerrit | Cody Herriges proposed openstack/puppet-cinder: Establish cinder_type after keystone_endpoint https://review.openstack.org/298397 | 22:11 |
*** dhollinger has quit IRC | 22:13 | |
*** ducttape_ has quit IRC | 22:19 | |
*** mdorman has quit IRC | 22:20 | |
EmilienM | crinkle: thx | 22:20 |
*** markvoelker has quit IRC | 22:29 | |
openstackgerrit | Cody Herriges proposed openstack/puppet-cinder: Establish cinder_type after keystone_endpoint https://review.openstack.org/298397 | 22:31 |
openstackgerrit | Cody Herriges proposed openstack/puppet-cinder: Establish cinder_type after keystone_endpoint https://review.openstack.org/298397 | 22:43 |
openstackgerrit | Cody Herriges proposed openstack/puppet-cinder: Establish cinder_type after keystone_endpoint https://review.openstack.org/298397 | 23:04 |
EmilienM | crinkle: do you know of Gemfile groups are ignored on the dependencies? example: if the group is in puppet-openstack_spec_helper/Gemfile, does puppet-nova will install the right deps? | 23:05 |
EmilienM | last week you mentionned we still could use a gemfile but I tested it and it does not work, maybe I did wrong | 23:05 |
crinkle | EmilienM: i don't know :/ | 23:05 |
* crinkle will poke it | 23:06 | |
EmilienM | ok np | 23:06 |
* EmilienM reading the fm :) | 23:06 | |
colonwq | EmilienM, When you get a chance, can you look at 296002 and 289027? I think I have polished this mud balls. They deploy correctly in my local environment. | 23:11 |
crinkle | EmilienM: you're right, adding it to the gemfile totally doesn't work | 23:11 |
crinkle | EmilienM: but adding it to the gemspec does | 23:11 |
crinkle | EmilienM: so i'm not sure how to get groups to work in that case | 23:12 |
EmilienM | there are groups in gemspec, but not all | 23:12 |
EmilienM | http://guides.rubygems.org/specification-reference/#add_development_dependency | 23:12 |
EmilienM | development and runtime deps only | 23:13 |
crinkle | right...that's not really what we want | 23:13 |
EmilienM | otherwise, we stop creating groups :( | 23:13 |
EmilienM | colonwq: I'll review it, sure | 23:13 |
colonwq | EmilienM, Thanks. | 23:14 |
EmilienM | colonwq: 296002 has 61 | 23:14 |
EmilienM | -1* | 23:14 |
openstackgerrit | Merged openstack/puppet-keystone: Ensure endpoints created before admin_token_auth https://review.openstack.org/298373 | 23:15 |
openstackgerrit | Merged openstack/puppet-nova: Autorequire nova security group https://review.openstack.org/296548 | 23:15 |
EmilienM | crinkle: what is your opinion on whether or not using gem groups? | 23:16 |
colonwq | EmilienM, dprince thinks I have my depends-on backwards with the two patches. | 23:17 |
*** saneax_AFK is now known as saneax | 23:17 | |
crinkle | EmilienM: so the reasons for having groups were 1) beaker stopped supporting ruby 1.8.7 and separating the groups allowed us to still run unit tests on 1.8.7 (i think this affected the puppetlabs modules more than us) and 2) faster installs since not all the jobs needed all the gems | 23:17 |
*** ducttape_ has joined #puppet-openstack | 23:17 | |
crinkle | EmilienM: I think 1) isn't an issue any more and we can live with 2) | 23:18 |
crinkle | er, we can live with slower installs | 23:18 |
EmilienM | I agree | 23:18 |
colonwq | EmilienM, I reversed the Depends-On direction of the patches. | 23:25 |
EmilienM | colonwq: ok I'll look thx | 23:25 |
openstackgerrit | Emilien Macchi proposed openstack/puppet-openstack_spec_helper: gemspec: add runtime dependencies for modules https://review.openstack.org/298465 | 23:29 |
openstackgerrit | Emilien Macchi proposed openstack/puppet-nova: Gemfile: rely on puppet-openstack_spec_helper for dependencies https://review.openstack.org/298467 | 23:30 |
openstackgerrit | Emilien Macchi proposed openstack/puppet-openstack-cookiecutter: metadata.json: bump to Mitaka https://review.openstack.org/296005 | 23:31 |
*** ducttape_ has quit IRC | 23:43 | |
*** ducttape_ has joined #puppet-openstack | 23:46 | |
*** xarses has quit IRC | 23:49 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!