Wednesday, 2019-08-28

*** tetsuro has joined #openstack-placement01:12
*** tetsuro has quit IRC01:13
*** cdent has joined #openstack-placement08:05
openstackgerritChris Dent proposed openstack/placement master: Merge request log and request id middlewares  https://review.opendev.org/67698209:00
openstackgerritChris Dent proposed openstack/placement master: Clarify the NOTE associated with ordering of middleware  https://review.opendev.org/67333509:00
*** cdent has quit IRC11:05
*** cdent has joined #openstack-placement12:15
cdentefried, gibi, tetsuro, mriedem: your thoughts desired on this rather niche bug: https://storyboard.openstack.org/#!/story/200646712:33
*** ttsiouts has joined #openstack-placement12:43
*** mriedem has joined #openstack-placement13:09
*** ttsiouts has quit IRC13:48
*** ttsiouts has joined #openstack-placement13:48
*** ttsiouts has quit IRC13:53
cdentefried: as someone who has wanted to do some wacky things with gabbi, there's a pull request in progress implementing a feature that needs some feedback; casting environment variables to other types: https://github.com/cdent/gabbi/pull/27514:10
*** cdent has quit IRC14:16
*** cdent has joined #openstack-placement14:16
*** cdent has quit IRC14:22
*** openstackgerrit has quit IRC14:22
efriedcdent: the gabbi feature seems reasonable. I don't know how much use I personally would have for it.14:43
*** cdent has joined #openstack-placement15:46
*** efried is now known as efried_rollin16:16
*** N3l1x has joined #openstack-placement16:32
*** mriedem is now known as mriedem_away16:49
*** artom has quit IRC16:56
* cdent surrenders on the orjson front17:37
*** cdent has quit IRC17:42
*** cdent has joined #openstack-placement17:46
*** openstackgerrit has joined #openstack-placement18:05
openstackgerritChris Dent proposed openstack/placement master: Add allocation_conflict_retry_count conf setting  https://review.opendev.org/67910818:05
openstackgerritChris Dent proposed openstack/placement master: Introduce jitter during allocations replace_all retry  https://review.opendev.org/67910918:05
*** cdent has quit IRC18:14
*** mriedem_away is now known as mriedem18:21
*** efried_rollin is now known as efried19:22
openstackgerritmelanie witt proposed openstack/osc-placement master: Add --aggregate option to 'resource provider inventory set'  https://review.opendev.org/64089819:50
openstackgerritmelanie witt proposed openstack/osc-placement master: Add --amend option to 'resource provider inventory set'  https://review.opendev.org/67552019:50
openstackgerritmelanie witt proposed openstack/osc-placement master: Add --dry-run option to 'resource provider inventory set'  https://review.opendev.org/67552119:50
openstackgerritmelanie witt proposed openstack/osc-placement master: Add --aggregate option to 'resource provider inventory set'  https://review.opendev.org/64089820:12
openstackgerritmelanie witt proposed openstack/osc-placement master: Add --amend option to 'resource provider inventory set'  https://review.opendev.org/67552020:12
openstackgerritmelanie witt proposed openstack/osc-placement master: Add --dry-run option to 'resource provider inventory set'  https://review.opendev.org/67552120:12
*** s10 has joined #openstack-placement20:12
melwittmriedem: updated ^ but I don't understand why the bottom patch fails the requirements-check due to addition of oslo.utils20:21
mriedemyou mean the sphinx thing?20:23
mriedemERROR: Could not find a global requirements entry to match package sphinx. If the package is already included in the global list, the name or platform markers there may not match the local settings.20:23
melwittyeah20:23
mriedemlikely means https://github.com/openstack/osc-placement/blob/master/doc/requirements.txt#L4 is busted somehow, unrelated to your change, but the requirements-check just happened now b/c you changed requirements somewhere20:24
melwittohhh crap20:24
mriedemyeah: Validating requirements.txt20:24
mriedemValidating doc/requirements.txt20:24
mriedemyou just triggered the job,20:24
mriedembut something is already broken20:24
melwittyeah I see20:25
mriedemthe sphinx thing is a known thing that people have been fixing in train, just need to find the pattern i think20:25
mriedemhttps://github.com/openstack/nova/commit/b1ef5aa72f85752a7d7e1219100bd7262b42267a#diff-46507bcc13aee955f6fe861c30ea544420:25
mriedemsomething like that probably20:25
melwittthank you20:25
mriedemso you'd have to push a fix for that at the bottom of your series20:25
melwittgot it20:26
openstackgerritmelanie witt proposed openstack/osc-placement master: Add --aggregate option to 'resource provider inventory set'  https://review.opendev.org/64089820:35
openstackgerritmelanie witt proposed openstack/osc-placement master: Add --amend option to 'resource provider inventory set'  https://review.opendev.org/67552020:35
openstackgerritmelanie witt proposed openstack/osc-placement master: Add --dry-run option to 'resource provider inventory set'  https://review.opendev.org/67552120:35
openstackgerritmelanie witt proposed openstack/osc-placement master: Sync sphinx constraints with global-requirements  https://review.opendev.org/67913720:35
melwitt"ERROR: Requirement for package 'sphinx' has no lower bound" yet from what I read here, lower-constraints are optional... https://docs.openstack.org/project-team-guide/dependency-management.html#solution20:44
melwitthttps://storage.bhs1.cloud.ovh.net/v1/AUTH_dcaab5e32b234d56b626f72581e3644c/logs_98/640898/14/check/requirements-check/93f532c/job-output.txt20:45
melwittdagnabbit20:45
*** belmoreira has joined #openstack-placement20:59
melwittI think I see what I did wrong21:01
melwittugh, I used a different change as an example not realizing it's a different thing entirely21:04
mriedemyeah i guess you need >=1.6.221:04
openstackgerritmelanie witt proposed openstack/osc-placement master: Cap sphinx for py2 to match global requirements  https://review.opendev.org/67913721:05
openstackgerritmelanie witt proposed openstack/osc-placement master: Add --aggregate option to 'resource provider inventory set'  https://review.opendev.org/64089821:05
openstackgerritmelanie witt proposed openstack/osc-placement master: Add --amend option to 'resource provider inventory set'  https://review.opendev.org/67552021:05
openstackgerritmelanie witt proposed openstack/osc-placement master: Add --dry-run option to 'resource provider inventory set'  https://review.opendev.org/67552121:05
*** belmoreira has quit IRC21:16
*** artom has joined #openstack-placement21:20
*** belmoreira has joined #openstack-placement21:25
*** s10 has quit IRC21:42
*** mriedem has quit IRC21:54
*** belmoreira has quit IRC22:11
*** N3l1x has quit IRC22:18
*** tetsuro has joined #openstack-placement23:44

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