| openstackgerrit | Haïkel Guémar proposed openstack/rpm-packaging: oslo.i18n: use %license macro https://review.openstack.org/339183 | 01:06 |
|---|---|---|
| openstackgerrit | Haïkel Guémar proposed openstack/rpm-packaging: Update oslo.messaging to 5.5.0 https://review.openstack.org/339301 | 01:32 |
| openstackgerrit | Haïkel Guémar proposed openstack/rpm-packaging: Update os-brick to 1.4.0 https://review.openstack.org/339304 | 01:37 |
| openstackgerrit | Haïkel Guémar proposed openstack/rpm-packaging: Update oslo.utils to 3.15.0 https://review.openstack.org/339305 | 01:39 |
| openstackgerrit | Haïkel Guémar proposed openstack/rpm-packaging: Add initial spec for oslo.privsep https://review.openstack.org/332936 | 01:43 |
| openstackgerrit | Haïkel Guémar proposed openstack/rpm-packaging: Templatize openstack-macros https://review.openstack.org/333197 | 01:47 |
| openstackgerrit | Merged openstack/rpm-packaging: Add initial spec file for python-glanceclient https://review.openstack.org/280246 | 01:47 |
| openstackgerrit | Haïkel Guémar proposed openstack/rpm-packaging: Add initial spec for futurist https://review.openstack.org/332418 | 01:50 |
| *** coolsvap has joined #openstack-rpm-packaging | 03:48 | |
| *** kzaitsev_ws has quit IRC | 05:22 | |
| *** kzaitsev_ws has joined #openstack-rpm-packaging | 06:09 | |
| openstackgerrit | Merged openstack/rpm-packaging: Capitalize Summary https://review.openstack.org/339096 | 06:33 |
| openstackgerrit | Merged openstack/rpm-packaging: Add initial spec for oslo.versionedobjects https://review.openstack.org/331548 | 06:33 |
| *** pcaruana has joined #openstack-rpm-packaging | 06:51 | |
| dirk | ah, where is the NOT_REGISTERED coming from how? | 07:21 |
| dirk | now | 07:21 |
| dirk | I mean in https://review.openstack.org/#/c/332418/ | 07:21 |
| dirk | toabctl: any idea? | 07:22 |
| toabctl | oh. | 07:22 |
| toabctl | dirk, we need to check the used ssh key I guess | 07:23 |
| toabctl | or maybe just a gerrit problem | 07:23 |
| toabctl | let's ask in #infra | 07:23 |
| toabctl | or let's just try a "recheck-suse" first | 07:24 |
| *** amoralej|off is now known as amoralej | 07:40 | |
| *** apevec has joined #openstack-rpm-packaging | 08:09 | |
| *** apevec has left #openstack-rpm-packaging | 08:35 | |
| *** mivanov has quit IRC | 08:38 | |
| openstackgerrit | Haïkel Guémar proposed openstack/rpm-packaging: Add initial spec for futurist https://review.openstack.org/332418 | 08:55 |
| openstackgerrit | Dirk Mueller proposed openstack/rpm-packaging: Update python-glanceclient to 2.1.0 https://review.openstack.org/339419 | 09:14 |
| openstackgerrit | Dirk Mueller proposed openstack/rpm-packaging: Update oslo.versionedobjects to 1.12.0 https://review.openstack.org/339420 | 09:15 |
| *** coolsvap has quit IRC | 09:32 | |
| *** coolsvap has joined #openstack-rpm-packaging | 09:33 | |
| *** coolsvap has quit IRC | 09:38 | |
| openstackgerrit | Haïkel Guémar proposed openstack/rpm-packaging: Add initial spec for oslo.vmware https://review.openstack.org/339442 | 09:53 |
| openstackgerrit | Haïkel Guémar proposed openstack/rpm-packaging: oslo.i18n: use %license macro https://review.openstack.org/339183 | 09:57 |
| openstackgerrit | Haïkel Guémar proposed openstack/rpm-packaging: Add initial spec for oslo.privsep https://review.openstack.org/332936 | 10:18 |
| number80 | any news on Fuel/SUSE CI? | 10:19 |
| IgorYozhikov | number80, looking into your commits | 10:19 |
| number80 | ack | 10:20 |
| number80 | I'm currently reworking the futurist one btw | 10:21 |
| number80 | I'm giving up with speccleaner being noisy when you have conditionals in Requires/BuildRequires | 10:22 |
| * number80 will try to fix speccleaner | 10:22 | |
| openstackgerrit | Haïkel Guémar proposed openstack/rpm-packaging: Add initial spec for futurist https://review.openstack.org/332418 | 10:24 |
| openstackgerrit | Haïkel Guémar proposed openstack/rpm-packaging: Add initial spec for taskflow https://review.openstack.org/339458 | 10:40 |
| IgorYozhikov | number80, - https://packaging-ci.fuel-infra.org/job/master-rpm-packaging-build-centos7/142/artifact/artifacts/oslo.privsep-rootlog.txt/*view*/ | 10:46 |
| IgorYozhikov | something went wrong | 10:46 |
| IgorYozhikov | with unittests | 10:46 |
| number80 | IgorYozhikov: known issue, but i was asked to try enable them again at previous iteration | 10:56 |
| openstackgerrit | Haïkel Guémar proposed openstack/rpm-packaging: Add initial spec for oslo.privsep https://review.openstack.org/332936 | 10:57 |
| *** coolsvap has joined #openstack-rpm-packaging | 11:24 | |
| *** aplanas has joined #openstack-rpm-packaging | 11:26 | |
| number80 | dirk, IgorYozhikov: I just made tests non-failing for oslo.privsep, it's broken on both RDO and MOS ^ | 11:56 |
| number80 | I agree w/ general good pratice we should avoid that by default but should not block us | 11:56 |
| *** pradk has joined #openstack-rpm-packaging | 12:20 | |
| IgorYozhikov | number80, might be better to use bcond with_tests instead of || : ? | 12:26 |
| number80 | IgorYozhikov: well, it's still useful to have test suite feedback in logs | 12:28 |
| IgorYozhikov | o i c | 12:28 |
| IgorYozhikov | you've got my +2 | 12:29 |
| openstackgerrit | Alberto Planas proposed openstack/rpm-packaging-tools: Fix git URL in README.rst. https://review.openstack.org/339537 | 12:44 |
| number80 | toabctl: https://review.openstack.org/#/c/338095/ this one needs minor fix but should be good to merge | 12:48 |
| openstackgerrit | Merged openstack/rpm-packaging-tools: Fix git URL in README.rst. https://review.openstack.org/339537 | 12:49 |
| openstackgerrit | Thomas Bechtold proposed openstack/rpm-packaging-tools: ALso allow release files with .yml ending https://review.openstack.org/338095 | 12:51 |
| toabctl | number80, updated. but what do you mean with "I also commented your changeset to releases repo (unrelated minor issue messing up tox)" ? | 12:51 |
| *** amoralej is now known as amoralej|lunch | 12:53 | |
| number80 | toabctl: Dirk fixed osprofiler.yml name in releases repo | 12:53 |
| toabctl | ah.ok | 12:53 |
| number80 | but tox fails because of a missing field | 12:53 |
| number80 | still makes sense to merge your changeset as we should not fail on stuff like that | 12:54 |
| * number80 just +2 the review | 12:55 | |
| openstackgerrit | Merged openstack/rpm-packaging: oslo.i18n: use %license macro https://review.openstack.org/339183 | 12:55 |
| openstackgerrit | Thomas Bechtold proposed openstack/rpm-packaging: Add initial spec for oslo.rootwrap https://review.openstack.org/323158 | 13:03 |
| *** coolsvap has quit IRC | 13:04 | |
| openstackgerrit | Haïkel Guémar proposed openstack/rpm-packaging: Add initial spec for oslo.vmware https://review.openstack.org/339442 | 13:04 |
| number80 | toabctl: concerning pbr/setuptools, in Fedora/CentOS, the former has no dependency on the latter | 13:09 |
| number80 | I | 13:09 |
| toabctl | number80, hm. sounds like a bug, no? | 13:09 |
| number80 | toabctl: nope intentional from RH python maintainers | 13:10 |
| number80 | I'm looking into improving handling "base" dependencies | 13:10 |
| toabctl | number80, for all the other specs we don't depend on setuptools. who to handle that? | 13:10 |
| number80 | like BR: on python-setuptools should just be injected into fedora/centos spec files | 13:10 |
| toabctl | number80, you mean with %?{rdo_version} ? | 13:12 |
| number80 | nope, tooling magic | 13:12 |
| *** coolsvap has joined #openstack-rpm-packaging | 13:12 | |
| toabctl | ok. I guess we need that anyway at some poit in renderspec | 13:12 |
| number80 | BR: setuptools is a MUST for us, so no need to clutter template | 13:12 |
| number80 | for now, I'm focusing on moving forward the review queue and have clients packaged for newton | 13:13 |
| number80 | Like ordering BR/R should be handled by renderspec directly | 13:15 |
| number80 | spec-cleaner does that based on rendered strings which changes between distros | 13:15 |
| number80 | so it will be very complex to manage this manually | 13:15 |
| toabctl | yeah. I also wanted to have a look at bindep which can do things depending on arch/platform... | 13:15 |
| number80 | well, we agree that we have a lot of work :) | 13:16 |
| toabctl | not using it but looking into it how things are solved there | 13:16 |
| toabctl | hehe. yes :-) | 13:16 |
| IgorYozhikov | toabctl, number80 - I'm going to PTO to the end of month. | 13:17 |
| toabctl | IgorYozhikov, PTO? | 13:17 |
| number80 | toabctl: vacations | 13:17 |
| IgorYozhikov | please ping astsmtl | 13:17 |
| toabctl | ah. ok | 13:17 |
| * toabctl is not good in acroynms | 13:18 | |
| number80 | IgorYozhikov: enjoy your vacations and yes, we'll bother astsmtl and mivanov in your stead :) | 13:18 |
| toabctl | IgorYozhikov, yes. have nice vacations!!!! | 13:18 |
| IgorYozhikov | number80, :)) | 13:19 |
| * number80 has no PTO this summer | 13:20 | |
| number80 | I'll be in Poland for a week though for a conference | 13:20 |
| openstackgerrit | Merged openstack/rpm-packaging: Update oslo.messaging to 5.5.0 https://review.openstack.org/339301 | 13:49 |
| openstackgerrit | Merged openstack/rpm-packaging: Update python-glanceclient to 2.1.0 https://review.openstack.org/339419 | 13:52 |
| *** amoralej|lunch is now known as amoralej | 13:59 | |
| openstackgerrit | Merged openstack/rpm-packaging-tools: ALso allow release files with .yml ending https://review.openstack.org/338095 | 14:04 |
| IgorYozhikov | number80, no PTO pity | 14:13 |
| number80 | IgorYozhikov: well, gf brings her kid to his dad, so I was not excited spending vacations lonely | 14:14 |
| number80 | *about | 14:14 |
| astsmtl | PiTyO | 14:31 |
| openstackgerrit | Haïkel Guémar proposed openstack/rpm-packaging: Add initial spec for futurist https://review.openstack.org/332418 | 14:34 |
| number80 | fixed it, forgot to remove now useless conditionals | 14:34 |
| number80 | :) | 14:34 |
| openstackgerrit | Haïkel Guémar proposed openstack/rpm-packaging: Add initial spec for taskflow https://review.openstack.org/339458 | 14:49 |
| number80 | I need to fix my script | 14:51 |
| number80 | toabctl: missing package is suse buildroot https://build.opensuse.org/project/monitor/home:suse-cloud-ci:rpm-packaging-openstack-8a87cd65cdec676401efef340f57f8e665a72605 | 14:55 |
| number80 | suds-jurko and subunut | 14:55 |
| number80 | subunit | 14:55 |
| openstackgerrit | Haïkel Guémar proposed openstack/rpm-packaging: Add initial spec for oslo.vmware https://review.openstack.org/339442 | 14:56 |
| openstackgerrit | Haïkel Guémar proposed openstack/rpm-packaging: Add initial spec for oslo.privsep https://review.openstack.org/332936 | 15:00 |
| toabctl | number80, it's python-subunit (the pypi name) | 15:01 |
| number80 | damn an inconsistent name | 15:01 |
| number80 | fixing that immediately | 15:02 |
| toabctl | number80, added the suds-jurko to the CI | 15:02 |
| toabctl | thx for the hint! | 15:02 |
| number80 | np, I'm getting more comfortable with reading 3rd gating logs | 15:03 |
| openstackgerrit | Haïkel Guémar proposed openstack/rpm-packaging: Add initial spec for oslo.vmware https://review.openstack.org/339442 | 15:04 |
| number80 | I need to write a tool to check if requires name are in pypi or not | 15:04 |
| astsmtl | Why do you need such tool? | 15:10 |
| *** apevec has joined #openstack-rpm-packaging | 15:12 | |
| *** pcaruana has quit IRC | 15:13 | |
| openstackgerrit | Haïkel Guémar proposed openstack/rpm-packaging: Add initial spec for tooz https://review.openstack.org/339625 | 15:35 |
| dirk | number80: it's called py2pack show xxx :-( | 15:54 |
| dirk | :-) even | 15:54 |
| number80 | dirk: well, I'm scripting some checks of the mistakes I usually do | 15:55 |
| number80 | astsmtl: this specific one would qualify for gating btw | 15:56 |
| number80 | like s/suds/suds-jurko/ | 15:56 |
| openstackgerrit | Haïkel Guémar proposed openstack/rpm-packaging: Add initial spec for tooz https://review.openstack.org/339625 | 15:59 |
| openstackgerrit | Merged openstack/rpm-packaging: Update oslo.versionedobjects to 1.12.0 https://review.openstack.org/339420 | 16:00 |
| openstackgerrit | Merged openstack/rpm-packaging: Templatize openstack-macros https://review.openstack.org/333197 | 16:05 |
| openstackgerrit | Merged openstack/rpm-packaging: Update oslo.utils to 3.15.0 https://review.openstack.org/339305 | 16:05 |
| dirk | number80: I hope to have an install check in the suse ci soon ish and that will find those bugs then | 16:09 |
| dirk | Automatically | 16:09 |
| dirk | It's just a few lines of code that needs some testing | 16:10 |
| number80 | yes, that's what I'd like to have in our common set of gating | 16:13 |
| number80 | either it's implemented in openstack infra or third-party gating | 16:14 |
| number80 | because, when we'll start adding services, gating will become critical for us | 16:15 |
| *** pcaruana has joined #openstack-rpm-packaging | 16:21 | |
| *** pcaruana has quit IRC | 16:31 | |
| openstackgerrit | Haïkel Guémar proposed openstack/rpm-packaging: Add initial spec for taskflow https://review.openstack.org/339458 | 17:11 |
| number80 | jruzicka: ^ this is an example of RDO specific changes we need to handle (e.g python-networkx-core which specific split we introduced to minimize deps) | 17:12 |
| openstackgerrit | Haïkel Guémar proposed openstack/rpm-packaging: Add initial spec for pycadf https://review.openstack.org/339737 | 17:32 |
| *** amoralej is now known as amoralej|off | 17:52 | |
| jruzicka | number80, oh man... to achieve what is needed, major overhaul might be required :( | 18:20 |
| jruzicka | BuildRequires: {{ py2pkg('devel') }} doesn't allow dropping that req entirely | 18:21 |
| astsmtl | Btw, we shouldn't use {{ py2pkg('devel') }}, because it may leat to false impression that there is such module in Python. | 18:23 |
| astsmtl | s/leat/lead | 18:23 |
| jruzicka | To allow custom ordering of BRs, they'd need to be supplied as a list | 18:24 |
| jruzicka | astsmtl, true, just a bad example ;) | 18:26 |
| jruzicka | not to mention we need to handle non-python reqs as well | 18:26 |
| number80 | jruzicka: well, you're THE man, you'll find clever tricks to manage that :) | 18:27 |
| number80 | yes, non-python will be tough topic but we can manage I think | 18:27 |
| jruzicka | I'll find the tricks for sure, but I'm affraid it will lead to a lots of changes | 18:27 |
| number80 | ack | 18:27 |
| jruzicka | well, it's not horrible for a first iteration ^^ | 18:28 |
| openstackgerrit | Merged openstack/rpm-packaging: Add initial spec for oslo.rootwrap https://review.openstack.org/323158 | 18:28 |
| number80 | ok, found a bug in my lint script | 18:32 |
| openstackgerrit | Haïkel Guémar proposed openstack/rpm-packaging: Add initial spec for tooz https://review.openstack.org/339625 | 18:36 |
| number80 | dirk: some BR for doc subpackage can be confusing, as they're only needed at build time for generating correctly documentation | 18:37 |
| *** pcaruana has joined #openstack-rpm-packaging | 18:37 | |
| dirk | number80: yeah, but stevedore is usually even a runtime requirement | 18:44 |
| dirk | Anyway, the -1 was due to the typo in the BuildRequires | 18:44 |
| dirk | I don't really care where the BuildRequires are specified, afaik there is no real difference.. | 18:44 |
| dirk | I'll fix the suse ci issues in a few hours, enjoying sunset first.. bbl | 18:45 |
| number80 | dirk: well, it was a bug in my linting script, I have two rules concerning sphinx ... | 18:49 |
| number80 | enjoy your w-e :) | 18:49 |
| jruzicka | number80, I put my first thoughts into https://etherpad.openstack.org/p/renderspec_RFE | 19:03 |
| number80 | ack | 19:05 |
| number80 | jruzicka: so we'd have a base template per distro, if I get you right? | 19:05 |
| number80 | It would actually work for me | 19:06 |
| jruzicka | number80, not really :) | 19:06 |
| jruzicka | just some conditionals and possibly per-distro magic in renderspec | 19:07 |
| jruzicka | or... what do you mean by base template? | 19:07 |
| number80 | well, some stuff are always the same, in distro A, you inject some BR, but not on distro B, etc. | 19:08 |
| number80 | the base deps macros is good as-is | 19:08 |
| jruzicka | oh right... yeah, these could be blocks that would just import a template from renderspec | 19:08 |
| number80 | we can move openstack-macros in that base deps btw | 19:08 |
| jruzicka | alternatively it could be done via {% if ... %} | 19:09 |
| jruzicka | but then you'd need to change across all the specs which doesn't sound too great :) | 19:10 |
| number80 | yes, that's the point | 19:10 |
| number80 | we're lazy people, so let's minimize work :) | 19:10 |
| jruzicka | indeed. | 19:10 |
| openstackgerrit | Haïkel Guémar proposed openstack/renderspec: Add support to CentOS https://review.openstack.org/332265 | 19:12 |
| number80 | .bu19 | 19:12 |
| number80 | damn | 19:12 |
| jruzicka | number80, -1 for you, sir ^ | 19:21 |
| jruzicka | ;) | 19:21 |
| number80 | jruzicka: -1 means "I care" | 19:22 |
| jruzicka | <3 | 19:23 |
| number80 | I guess I have to refresh my standard library fu, I'm pretty sure that we have a cool module doing what I want to do already | 19:23 |
| jruzicka | I wouldn't be so sure ;) | 19:24 |
| jruzicka | I'm curious how will this look like, it's a common patter where you'd like to use 'in' but you can't cause it's not simple == | 19:26 |
| number80 | worst case is use a regular expression because you can solve any problem with a regex | 19:26 |
| jruzicka | local is_fedora function? :) | 19:27 |
| number80 | I'm currently timing in ipython which one is the most efficient | 19:28 |
| jruzicka | hahaha | 19:28 |
| number80 | in vs regex | 19:28 |
| jruzicka | regex is less readable | 19:29 |
| jruzicka | well... | 19:29 |
| jruzicka | depends on the reader :-p | 19:29 |
| jruzicka | I'd just got with a local function | 19:29 |
| jruzicka | for s in ['fedora', 'centos', 'red hat']: | 19:30 |
| jruzicka | if s in distname_lower: return True | 19:30 |
| number80 | this one wins according timeit | 19:33 |
| jruzicka | ofc, 'for' is the bane of python ;) | 19:33 |
| number80 | yup | 19:36 |
| openstackgerrit | Haïkel Guémar proposed openstack/renderspec: Add support to CentOS https://review.openstack.org/332265 | 19:38 |
| number80 | slightly different, to return False explicitly | 19:39 |
| number80 | ah no | 19:39 |
| number80 | does not work | 19:39 |
| number80 | I forgot using any | 19:39 |
| number80 | damn, after that, I'm taking a shower and go read a book | 19:42 |
| openstackgerrit | Haïkel Guémar proposed openstack/renderspec: Add support to CentOS https://review.openstack.org/332265 | 19:45 |
| number80 | this one is the right one | 19:45 |
| openstackgerrit | Haïkel Guémar proposed openstack/renderspec: Add support to CentOS https://review.openstack.org/332265 | 19:52 |
| openstackgerrit | Haïkel Guémar proposed openstack/renderspec: Add support to CentOS https://review.openstack.org/332265 | 21:35 |
| *** pradk has quit IRC | 21:54 | |
| *** apevec has quit IRC | 21:58 | |
| *** pradk has joined #openstack-rpm-packaging | 22:00 | |
| *** pradk has quit IRC | 22:39 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!