*** tetsuro has joined #openstack-placement | 01:12 | |
*** tetsuro has quit IRC | 01:13 | |
*** cdent has joined #openstack-placement | 08:05 | |
openstackgerrit | Chris Dent proposed openstack/placement master: Merge request log and request id middlewares https://review.opendev.org/676982 | 09:00 |
---|---|---|
openstackgerrit | Chris Dent proposed openstack/placement master: Clarify the NOTE associated with ordering of middleware https://review.opendev.org/673335 | 09:00 |
*** cdent has quit IRC | 11:05 | |
*** cdent has joined #openstack-placement | 12:15 | |
cdent | efried, gibi, tetsuro, mriedem: your thoughts desired on this rather niche bug: https://storyboard.openstack.org/#!/story/2006467 | 12:33 |
*** ttsiouts has joined #openstack-placement | 12:43 | |
*** mriedem has joined #openstack-placement | 13:09 | |
*** ttsiouts has quit IRC | 13:48 | |
*** ttsiouts has joined #openstack-placement | 13:48 | |
*** ttsiouts has quit IRC | 13:53 | |
cdent | efried: 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/275 | 14:10 |
*** cdent has quit IRC | 14:16 | |
*** cdent has joined #openstack-placement | 14:16 | |
*** cdent has quit IRC | 14:22 | |
*** openstackgerrit has quit IRC | 14:22 | |
efried | cdent: the gabbi feature seems reasonable. I don't know how much use I personally would have for it. | 14:43 |
*** cdent has joined #openstack-placement | 15:46 | |
*** efried is now known as efried_rollin | 16:16 | |
*** N3l1x has joined #openstack-placement | 16:32 | |
*** mriedem is now known as mriedem_away | 16:49 | |
*** artom has quit IRC | 16:56 | |
* cdent surrenders on the orjson front | 17:37 | |
*** cdent has quit IRC | 17:42 | |
*** cdent has joined #openstack-placement | 17:46 | |
*** openstackgerrit has joined #openstack-placement | 18:05 | |
openstackgerrit | Chris Dent proposed openstack/placement master: Add allocation_conflict_retry_count conf setting https://review.opendev.org/679108 | 18:05 |
openstackgerrit | Chris Dent proposed openstack/placement master: Introduce jitter during allocations replace_all retry https://review.opendev.org/679109 | 18:05 |
*** cdent has quit IRC | 18:14 | |
*** mriedem_away is now known as mriedem | 18:21 | |
*** efried_rollin is now known as efried | 19:22 | |
openstackgerrit | melanie witt proposed openstack/osc-placement master: Add --aggregate option to 'resource provider inventory set' https://review.opendev.org/640898 | 19:50 |
openstackgerrit | melanie witt proposed openstack/osc-placement master: Add --amend option to 'resource provider inventory set' https://review.opendev.org/675520 | 19:50 |
openstackgerrit | melanie witt proposed openstack/osc-placement master: Add --dry-run option to 'resource provider inventory set' https://review.opendev.org/675521 | 19:50 |
openstackgerrit | melanie witt proposed openstack/osc-placement master: Add --aggregate option to 'resource provider inventory set' https://review.opendev.org/640898 | 20:12 |
openstackgerrit | melanie witt proposed openstack/osc-placement master: Add --amend option to 'resource provider inventory set' https://review.opendev.org/675520 | 20:12 |
openstackgerrit | melanie witt proposed openstack/osc-placement master: Add --dry-run option to 'resource provider inventory set' https://review.opendev.org/675521 | 20:12 |
*** s10 has joined #openstack-placement | 20:12 | |
melwitt | mriedem: updated ^ but I don't understand why the bottom patch fails the requirements-check due to addition of oslo.utils | 20:21 |
mriedem | you mean the sphinx thing? | 20:23 |
mriedem | ERROR: 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 |
melwitt | yeah | 20:23 |
mriedem | likely 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 somewhere | 20:24 |
melwitt | ohhh crap | 20:24 |
mriedem | yeah: Validating requirements.txt | 20:24 |
mriedem | Validating doc/requirements.txt | 20:24 |
mriedem | you just triggered the job, | 20:24 |
mriedem | but something is already broken | 20:24 |
melwitt | yeah I see | 20:25 |
mriedem | the sphinx thing is a known thing that people have been fixing in train, just need to find the pattern i think | 20:25 |
mriedem | https://github.com/openstack/nova/commit/b1ef5aa72f85752a7d7e1219100bd7262b42267a#diff-46507bcc13aee955f6fe861c30ea5444 | 20:25 |
mriedem | something like that probably | 20:25 |
melwitt | thank you | 20:25 |
mriedem | so you'd have to push a fix for that at the bottom of your series | 20:25 |
melwitt | got it | 20:26 |
openstackgerrit | melanie witt proposed openstack/osc-placement master: Add --aggregate option to 'resource provider inventory set' https://review.opendev.org/640898 | 20:35 |
openstackgerrit | melanie witt proposed openstack/osc-placement master: Add --amend option to 'resource provider inventory set' https://review.opendev.org/675520 | 20:35 |
openstackgerrit | melanie witt proposed openstack/osc-placement master: Add --dry-run option to 'resource provider inventory set' https://review.opendev.org/675521 | 20:35 |
openstackgerrit | melanie witt proposed openstack/osc-placement master: Sync sphinx constraints with global-requirements https://review.opendev.org/679137 | 20: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#solution | 20:44 |
melwitt | https://storage.bhs1.cloud.ovh.net/v1/AUTH_dcaab5e32b234d56b626f72581e3644c/logs_98/640898/14/check/requirements-check/93f532c/job-output.txt | 20:45 |
melwitt | dagnabbit | 20:45 |
*** belmoreira has joined #openstack-placement | 20:59 | |
melwitt | I think I see what I did wrong | 21:01 |
melwitt | ugh, I used a different change as an example not realizing it's a different thing entirely | 21:04 |
mriedem | yeah i guess you need >=1.6.2 | 21:04 |
openstackgerrit | melanie witt proposed openstack/osc-placement master: Cap sphinx for py2 to match global requirements https://review.opendev.org/679137 | 21:05 |
openstackgerrit | melanie witt proposed openstack/osc-placement master: Add --aggregate option to 'resource provider inventory set' https://review.opendev.org/640898 | 21:05 |
openstackgerrit | melanie witt proposed openstack/osc-placement master: Add --amend option to 'resource provider inventory set' https://review.opendev.org/675520 | 21:05 |
openstackgerrit | melanie witt proposed openstack/osc-placement master: Add --dry-run option to 'resource provider inventory set' https://review.opendev.org/675521 | 21:05 |
*** belmoreira has quit IRC | 21:16 | |
*** artom has joined #openstack-placement | 21:20 | |
*** belmoreira has joined #openstack-placement | 21:25 | |
*** s10 has quit IRC | 21:42 | |
*** mriedem has quit IRC | 21:54 | |
*** belmoreira has quit IRC | 22:11 | |
*** N3l1x has quit IRC | 22:18 | |
*** tetsuro has joined #openstack-placement | 23:44 |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!