Friday, 2016-12-23

*** chandankumar|off is now known as chandankumar07:20
openstackgerritThomas Bechtold proposed openstack/rpm-packaging: Initial template for keystone  https://review.openstack.org/39626207:20
*** chandankumar has quit IRC07:22
*** chandankumar has joined #openstack-rpm-packaging07:23
toabctlIgorYozhikov, I updated the keystone template.07:47
*** jpena|off is now known as jpena07:54
*** pcaruana has quit IRC07:54
*** pcaruana has joined #openstack-rpm-packaging07:55
openstackgerritDirk Mueller proposed openstack/rpm-packaging: Clean up mistral  https://review.openstack.org/41447108:02
openstackgerritDirk Mueller proposed openstack/rpm-packaging: Add initial spec for tripleo-common  https://review.openstack.org/35854208:03
*** jpena is now known as jpena|off08:05
openstackgerritDirk Mueller proposed openstack/rpm-packaging: Add initial spec for tripleo-common  https://review.openstack.org/35854208:05
openstackgerritDirk Mueller proposed openstack/rpm-packaging: Clean up mistral  https://review.openstack.org/41447108:07
openstackgerritDirk Mueller proposed openstack/rpm-packaging: Clean up mistral  https://review.openstack.org/41447108:18
*** amoralej|off is now known as amoralej08:20
*** jpena|off is now known as jpena08:27
*** itxaka has joined #openstack-rpm-packaging08:50
toabctljpena, how to handle the pyldap ?09:04
toabctlare you going to package that for rdo?09:04
jpenatoabctl: we need to add that package to our dependency repos, will talk to number80 once he is available09:04
toabctljpena, should we wait before merging keystone09:05
toabctl?09:05
jpenalet me check, if it's going to take a while to package pyldap we shouldn't wait09:07
*** apevec has joined #openstack-rpm-packaging09:21
number80pyldap?09:24
number80Isn't that the same as http://cbs.centos.org/koji/buildinfo?buildID=788709:24
number80ah nope09:25
number80it's in fedora, I'll add it to my rebuild list09:26
jpenanumber80: it seems to be a fork of python-ldap (and conflicts with it)09:29
number80yeah, looking at spec file09:29
number80it has bcond_with python209:30
number80ok, I think we can try it in -pending first though09:30
* number80 will hack package a bit09:30
apevecwat? Multiple ldap modules!09:58
apevechow did that pass openstack/requirements review?09:58
apevecok, found it https://review.openstack.org/31579310:09
apevecso pyldap should python-ldap10:09
apevec*should replace10:12
toabctlapevec, could you have a look at https://review.openstack.org/#/c/396262/ please?10:33
toabctlespecially at the config structure bits10:34
toabctlit is explained in the README there10:34
apevectoabctl, iirc Puppet folks wanted to make sure main .conf wins, not sure they'll like this10:40
toabctlapevec, main-conf == /etc/$project/$project.conf ?10:40
apevecyeah10:41
toabctlapevec, that's not how oslo.config works10:41
toabctlapevec, and imo it's the wrong approach10:41
apevecalso for conf.d to work, we need to ensure min oslo.config version ?10:41
apevectoabctl, puppet wants to have full control :)10:41
toabctlapevec, I know. I had this ML discussion but it turned out that they thought they have control but they didn't10:42
apevecthat was about default values10:42
apevecjpena, number80 - re. pyldap - see inline comment10:43
toabctlapevec, we have the patch in stable/newton for oslo.config . and everything > Newton has it automatically10:43
toabctlapevec, also they are not deleting /Ect/$project.conf or /etc/$project/$program.conf10:43
toabctlapevec, and they didn't know about RDO's dist.conf thingy10:44
apevecdist.conf is before, so /etc overrides it10:44
apevecit was just their expectation that upstream defaults apply/never change10:44
toabctlapevec, yes.10:44
apevecbut good point about other default conf locations...10:45
toabctlso puppet-folks could a) delete all conf.d/ folders or b) use a systemd override and explicitly set --config-file and --config-dir . then the defaults are not working10:46
toabctland afaik it is not possible with the puppet code for openstack to include other config files (eg 3rd party networking stuff)10:47
toabctlor it is possible but in a hacky way (adding new sections to other config files).10:47
openstackgerritJavier Peña proposed openstack/rpm-packaging: Add initial spec for python-tripleoclient  https://review.openstack.org/35736111:36
openstackgerritJavier Peña proposed openstack/rpm-packaging: Clean up mistral  https://review.openstack.org/41447111:41
openstackgerritJavier Peña proposed openstack/rpm-packaging: Add initial spec for tripleo-common  https://review.openstack.org/35854211:41
openstackgerritJavier Peña proposed openstack/rpm-packaging: Add initial spec for python-tripleoclient  https://review.openstack.org/35736111:43
openstackgerritJavier Peña proposed openstack/rpm-packaging: Add initial spec for tripleo-common  https://review.openstack.org/35854211:46
jpenadirk: I see you added some changes to mistral in https://review.openstack.org/358542. Is that needed?11:51
dirkjpena: that's why i rebased the changes.for tripleo-common etc on top of that change, yes11:58
dirkjpena: see commit log on why those changes are needed11:58
jpenadirk: yep, but even when rebased there's still a small change, see current patchset11:59
openstackgerritJavier Peña proposed openstack/rpm-packaging: Add initial spec for tripleo-common  https://review.openstack.org/35854212:15
openstackgerritJavier Peña proposed openstack/rpm-packaging: Add initial spec for tripleo-common  https://review.openstack.org/35854212:15
IgorYozhikovjpena, https://review.openstack.org/#/c/358542/ we need openstack-mistral package12:18
openstackgerritJavier Peña proposed openstack/rpm-packaging: Clean up mistral  https://review.openstack.org/41447112:18
jpenaIgorYozhikov: yes, working on it12:19
openstackgerritJavier Peña proposed openstack/rpm-packaging: Add initial spec for tripleo-common  https://review.openstack.org/35854212:21
openstackgerritJavier Peña proposed openstack/rpm-packaging: Add initial spec for tripleo-common  https://review.openstack.org/35854212:22
openstackgerritJavier Peña proposed openstack/rpm-packaging: Clean up mistral  https://review.openstack.org/41447112:23
openstackgerritJavier Peña proposed openstack/rpm-packaging: Add initial spec for tripleo-common  https://review.openstack.org/35854212:23
jpenaIgorYozhikov, for tripleo-common to work, we need to merge https://review.openstack.org/414471 first12:31
IgorYozhikovjpena, got it, let's wait for CIS12:31
*** jpena is now known as jpena|lunch12:48
*** pradk has quit IRC13:11
number80ack13:35
*** amoralej is now known as amoralej|lunch13:40
*** pradk has joined #openstack-rpm-packaging13:44
*** pradk has quit IRC13:46
*** jpena|lunch is now known as jpena13:49
*** witek has quit IRC13:53
*** amoralej|lunch is now known as amoralej14:07
*** txengr266873468 has joined #openstack-rpm-packaging14:52
openstackgerritMerged openstack/rpm-packaging: Clean up mistral  https://review.openstack.org/41447115:55
openstackgerritJavier Peña proposed openstack/rpm-packaging: Add initial spec for tripleo-common  https://review.openstack.org/35854216:45
*** jpena is now known as jpena|off17:09
*** pcaruana has quit IRC17:14
*** amoralej is now known as amoralej|pto17:31
*** coolsvap has quit IRC19:19
*** Bluewireslight has joined #openstack-rpm-packaging20:58
Bluewireslighthi20:58
*** Bluewireslight has left #openstack-rpm-packaging20:59
*** apevec has quit IRC21:51
*** itxaka has quit IRC23:15

Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!