| openstackgerrit | Thomas Goirand proposed openstack/deb-murano-dashboard: Fixed openstack-auth -> django-openstack-auth https://review.openstack.org/379969 | 07:28 |
|---|---|---|
| openstackgerrit | Thomas Goirand proposed openstack/deb-openstack-pkg-tools: Fixed db port when altering db for setup https://review.openstack.org/380019 | 08:48 |
| zigo | onovy: tlbr: I could log into Horizon using Experimental packages (with a small fix in openstack-pkg-tools which in CR just above...). | 08:56 |
| zigo | :) | 08:56 |
| openstackgerrit | Thomas Goirand proposed openstack/deb-ironic-ui: Merge tag '2.1.0' into debian/newton https://review.openstack.org/380049 | 09:23 |
| onovy | zigo: cool | 09:41 |
| openstackgerrit | Thomas Goirand proposed openstack/deb-ironic-ui: Merge tag '2.1.0' into debian/newton https://review.openstack.org/380049 | 10:16 |
| openstackgerrit | Merged openstack/deb-openstack-pkg-tools: Fixed db port when altering db for setup https://review.openstack.org/380019 | 10:36 |
| openstackgerrit | Merged openstack/deb-auto-backports: Removed liberasurecode because we are packing it https://review.openstack.org/379408 | 10:42 |
| openstackgerrit | Thomas Goirand proposed openstack/deb-ironic-ui: Merge tag '2.1.0' into debian/newton https://review.openstack.org/380049 | 10:55 |
| *** tlbr has quit IRC | 11:43 | |
| *** tlbr has joined #openstack-pkg | 11:50 | |
| openstackgerrit | Thomas Goirand proposed openstack/deb-ironic-ui: Merge tag '2.1.0' into debian/newton https://review.openstack.org/380049 | 11:51 |
| openstackgerrit | Thomas Goirand proposed openstack/deb-ironic-ui: Merge tag '2.1.0' into debian/newton https://review.openstack.org/380049 | 11:51 |
| openstackgerrit | Thomas Goirand proposed openstack/deb-openstack-pkg-tools: Releasing to unstable. https://review.openstack.org/380202 | 12:03 |
| openstackgerrit | Thomas Goirand proposed openstack/deb-ironic-ui: Merge tag '2.1.0' into debian/newton https://review.openstack.org/380049 | 12:20 |
| openstackgerrit | Merged openstack/deb-openstack-pkg-tools: Releasing to unstable. https://review.openstack.org/380202 | 12:21 |
| openstackgerrit | Merged openstack/deb-murano-dashboard: Fixed openstack-auth -> django-openstack-auth https://review.openstack.org/379969 | 12:25 |
| openstackgerrit | Merged openstack/deb-networking-mlnx: Service name fixed to networking-mlnx-eswitchd https://review.openstack.org/379368 | 12:28 |
| zigo | lennyb: I have approved your patch here: https://review.openstack.org/#/c/379368/ | 12:31 |
| zigo | lennyb: But I think it's a mistake. | 12:31 |
| zigo | lennyb: The file really is /usr/bin/eswitchd | 12:32 |
| zigo | lennyb: Not it wont work anymore ... :/ | 12:32 |
| zigo | I'm reverting it. | 12:32 |
| openstackgerrit | Thomas Goirand proposed openstack/deb-networking-mlnx: Revert "Service name fixed to networking-mlnx-eswitchd" https://review.openstack.org/380221 | 12:33 |
| openstackgerrit | Merged openstack/deb-ironic-ui: Merge tag '2.1.0' into debian/newton https://review.openstack.org/380049 | 12:41 |
| openstackgerrit | Thomas Goirand proposed openstack/deb-openstack-pkg-tools: Correct debian/newton branch in gbp.conf https://review.openstack.org/380260 | 12:45 |
| tlbr | zigo, hi, could you please review https://anonscm.debian.org/cgit/openstack/glare.git/ | 12:45 |
| zigo | tlbr: Sure ! | 12:45 |
| tlbr | thanks ) | 12:45 |
| zigo | tlbr: It's not lintian clean, by the first look. | 12:46 |
| zigo | tlbr: Please build the package and fix that. | 12:46 |
| zigo | tlbr: For example, python-pbr is there twice (in build-depends and build-depends-indep) | 12:46 |
| zigo | You would have seen it if running Lintian. | 12:46 |
| tlbr | zigo, http://paste.openstack.org/show/nho9YafGARcTV9NarQgz/ | 12:47 |
| zigo | tlbr: You don't need to specify version for testrepository... | 12:47 |
| zigo | tlbr: Yup, you should fix all of that. | 12:47 |
| zigo | tlbr: You forgot the # before the bug number in debian/changelog! :) | 12:48 |
| tlbr | i've already added ITP BTW | 12:48 |
| tlbr | sec | 12:48 |
| tlbr | done | 12:49 |
| zigo | tlbr: grep -r -i copyright * <--- Shows a lot more copyright holders than what you've put in debian/copyright. This will be rejected by FTP masters FOR SURE ! :) | 12:49 |
| tlbr | building | 12:49 |
| tlbr | Michael told me, that he is the author | 12:50 |
| tlbr | and overall licence is ASL 2.0 | 12:50 |
| zigo | tlbr: Author != copyright holders. | 12:51 |
| zigo | tlbr: Do the grep as I did, and copy what you see there in debian/copyright. | 12:51 |
| zigo | That's what the FTP masters will do anyway. | 12:51 |
| zigo | For example: | 12:51 |
| zigo | glare/objects/meta/validators.py:# Copyright 2016 OpenStack Foundation | 12:51 |
| zigo | glare/objects/meta/attribute.py:# Copyright (c) 2016 Mirantis, Inc. | 12:51 |
| tlbr | hm, ok | 12:52 |
| tlbr | thanks | 12:52 |
| zigo | grep -r -i copyright * | grep "OpenStack Foundation" <--- Shows that you should put (c) 2010-2016, OpenStack Foundation. | 12:52 |
| tlbr | sec | 12:52 |
| zigo | Then you'd do: grep -r -i copyright * | grep -v "OpenStack Foundation" and then move to search for the next copyright holder. | 12:52 |
| zigo | That's what I do... | 12:52 |
| tlbr | ok, thanks | 12:53 |
| zigo | debian/glare-common.install <--- You should put /usr/bin/* in the python-glare (so that it is easy to switch to python3) | 12:53 |
| zigo | Yes, we didn't do that before, though we should start doing it... | 12:53 |
| tlbr | ok | 12:53 |
| zigo | There's no stuff with debconf to configure the package ... | 12:54 |
| tlbr | it shouldn't be there for now | 12:54 |
| tlbr | guyz from Poznan need it in Debian ASAP | 12:54 |
| zigo | Well, I do want it to be there for consistency ... | 12:55 |
| zigo | I'll write it if you want (later today...). | 12:55 |
| zigo | debian/python-glare.install <--- Should include usr/lib/python2* not just all python. | 12:59 |
| zigo | The package should generate the config file using etc/oslo-config-generator/glare.conf | 13:00 |
| zigo | It currently doesn't. | 13:00 |
| tlbr | zigo, it does | 13:10 |
| tlbr | $ grep oslo-config-generator debian/rules | 13:10 |
| tlbr | PYTHONPATH=$(CURDIR)/debian/tmp/usr/lib/python2.7/dist-packages oslo-config-generator \ | 13:10 |
| tlbr | --config-file=etc/oslo-config-generator/glare.conf | 13:10 |
| tlbr | zigo, so instead of usr/lib/python*/dist-packages/* I should put usr/lib/python* ? | 13:11 |
| tlbr | to debian/python-glare.install | 13:11 |
| tlbr | looks like i also need to put glare user | 13:12 |
| tlbr | ah, got it! I've replaced usr/lib/python*/dist-packages/* with usr/lib/python2*/dist-packages/* to debian/python-glare.install | 13:34 |
| tlbr | *in | 13:34 |
| tlbr | and added d/glare-common.postinst.in | 13:35 |
| zigo | I'm fixing some install stuff in debian/rules | 15:03 |
| zigo | There's some issues. | 15:05 |
| zigo | I wont have enough batteries. | 15:05 |
| zigo | I'll come back later tonight... | 15:05 |
| zigo | TTYL | 15:05 |
| tlbr | ok ok | 15:17 |
| tlbr | i've noticed that it FTBS too | 15:17 |
| openstackgerrit | Merged openstack/deb-openstack-pkg-tools: Correct debian/newton branch in gbp.conf https://review.openstack.org/380260 | 17:21 |
| openstackgerrit | Thomas Goirand proposed openstack/deb-aodh: Build-Depends on openstack-pkg-tools >= 53~. https://review.openstack.org/380486 | 18:38 |
| openstackgerrit | Thomas Goirand proposed openstack/deb-ironic: Also package Ironic's install-guide. https://review.openstack.org/380504 | 19:07 |
| zigo | tlbr: Can you ask guys doing python-glareclient to tag a release? | 19:11 |
| zigo | Even if it's 0.0.1 ... | 19:11 |
| zigo | So that we know what's packaged. | 19:11 |
| zigo | I rate packaging stuff like 0.0.0~20160930.git ... | 19:11 |
| zigo | tlbr: I also need the keystone endpoint type, and I don't think we should upload before that... | 19:13 |
| zigo | tlbr: It otherwise is already a working package ! :P | 19:18 |
| *** openstackgerrit has quit IRC | 19:18 | |
| *** openstackgerrit has joined #openstack-pkg | 19:18 | |
| zigo | tlbr: If found it, it's simply "artifact" | 19:23 |
| zigo | tlbr: I uploaded glare 0.1.0-1 | 19:35 |
| zigo | Let's hope it makes it in Experimental soon. | 19:36 |
| zigo | tlbr: Given the fact that there's about 80 items in the queue, it may take a few days. | 19:38 |
| zigo | tlbr: Notice that *EVERY TIME*, the NEW queue gets low before debconf ... https://ftp-master.debian.org/stat/new-year.png | 19:39 |
| zigo | :) | 19:39 |
| openstackgerrit | Thomas Goirand proposed openstack/deb-ironic: Also package Ironic's install-guide. https://review.openstack.org/380504 | 21:03 |
| openstackgerrit | Merged openstack/deb-aodh: Build-Depends on openstack-pkg-tools >= 53~. https://review.openstack.org/380486 | 21:08 |
| openstackgerrit | Merged openstack/deb-networking-mlnx: Revert "Service name fixed to networking-mlnx-eswitchd" https://review.openstack.org/380221 | 21:11 |
| openstackgerrit | Thomas Goirand proposed openstack/deb-glance: Merge tag '13.0.0_rc2' into debian/newton https://review.openstack.org/380659 | 21:39 |
| openstackgerrit | Thomas Goirand proposed openstack/deb-ironic: Also package Ironic's install-guide. https://review.openstack.org/380504 | 22:05 |
| tlbr | zigo, ok, thanks for the info! | 23:22 |
| tlbr | guys haven't uploaded glareclient to pypi, so there is no need to package it yet | 23:23 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!
