| openstackgerrit | Tim Burke proposed openstack/swift master: Add debugging info to SignatureDoesNotMatch responses https://review.openstack.org/575808 | 00:27 |
|---|---|---|
| *** gyee has quit IRC | 00:39 | |
| openstackgerrit | Tim Burke proposed openstack/swift master: s3api: Stop mangling Authorization header for v4 signatures https://review.openstack.org/591862 | 01:07 |
| clayg | whoa! mattoliverau has some new fancy OpenStack gig!? http://lists.openstack.org/pipermail/openstack-dev/2018-August/133309.html | 01:48 |
| clayg | mattoliverau: WTG! | 01:48 |
| mattoliverau | clayg: lol, ta, not really just trying to make the world a better place, and hopefully get more devs involved (cause we (swift) could also do with it) :P also makes sure SUSE lets me play upstream more, which hopefully means more Swift time.. so attempting 2 birds with one stone ;) | 01:52 |
| clayg | you're an electric personality - your energy is absolutely a force of positive change - openstack is lucky to have you applying yourself in new ways - suse is smart to leverage your talents | 01:55 |
| clayg | thanks for everything you do mattoliverau | 01:56 |
| mattoliverau | thanks man :) | 01:56 |
| *** openstack has joined #openstack-swift | 02:35 | |
| *** ChanServ sets mode: +o openstack | 02:35 | |
| *** paladox is now known as PuppyKun_ | 02:43 | |
| *** PuppyKun_ is now known as paladox | 02:44 | |
| *** pcaruana has joined #openstack-swift | 05:12 | |
| *** hseipp has joined #openstack-swift | 07:09 | |
| *** hseipp has quit IRC | 07:11 | |
| *** hseipp has joined #openstack-swift | 07:11 | |
| *** mikecmpbll has joined #openstack-swift | 08:02 | |
| *** mikecmpbll has quit IRC | 08:14 | |
| *** cbartz has joined #openstack-swift | 08:15 | |
| *** mikecmpbll has joined #openstack-swift | 08:31 | |
| *** tdasilva has quit IRC | 08:33 | |
| *** samueldmq_ has joined #openstack-swift | 09:16 | |
| *** samueldmq has quit IRC | 09:18 | |
| *** samueldmq_ is now known as samueldmq | 09:18 | |
| *** gkadam has joined #openstack-swift | 09:30 | |
| *** quicker_io has joined #openstack-swift | 09:57 | |
| *** mikecmpbll has quit IRC | 10:15 | |
| *** mikecmpbll has joined #openstack-swift | 10:30 | |
| *** hseipp has quit IRC | 12:03 | |
| *** tdasilva has joined #openstack-swift | 12:38 | |
| *** ChanServ sets mode: +v tdasilva | 12:38 | |
| openstackgerrit | Thiago da Silva proposed openstack/swift master: s3_acl: Require swift_owner authz to create buckets https://review.openstack.org/577941 | 12:56 |
| *** mvkr has quit IRC | 13:10 | |
| *** edausq has quit IRC | 13:10 | |
| *** edausq has joined #openstack-swift | 13:11 | |
| *** mvkr has joined #openstack-swift | 13:40 | |
| openstackgerrit | Thiago da Silva proposed openstack/swift master: Add debugging info to SignatureDoesNotMatch responses https://review.openstack.org/575808 | 13:52 |
| *** quicker_io has quit IRC | 14:07 | |
| *** d0ugal has joined #openstack-swift | 14:58 | |
| *** gyee has joined #openstack-swift | 15:32 | |
| *** mvkr has quit IRC | 15:46 | |
| *** itlinux has joined #openstack-swift | 15:52 | |
| *** FracKen has joined #openstack-swift | 15:56 | |
| *** cbartz has quit IRC | 16:12 | |
| *** zaitcev has joined #openstack-swift | 16:29 | |
| *** ChanServ sets mode: +v zaitcev | 16:29 | |
| clayg | tdasilva: we need to fix swift-recon-cron to use a with lock: symantic instead of the insane thing it does now | 16:39 |
| clayg | if the process dies hard it leaves the file around and the whole thing wedges - using a flock would ensure if the process holding the lock ever goes away the next run can grab the lock | 16:39 |
| clayg | i don't really understand how it's been left broken for so long considering how cheap it is to fix... | 16:40 |
| *** mikecmpbll has quit IRC | 16:46 | |
| timburke | tdasilva: thanks for cleaning up my end-of-the-day patches! sorry, i mainly wanted to get the work off my machine | 16:47 |
| tdasilva | clayg: hey, yeah i was looking at that probe test failure caught my attention | 16:52 |
| tdasilva | timburke: yw | 16:53 |
| *** openstackstatus has joined #openstack-swift | 16:58 | |
| *** ChanServ sets mode: +v openstackstatus | 16:58 | |
| *** gkadam has quit IRC | 17:05 | |
| *** gkadam has joined #openstack-swift | 17:20 | |
| *** mikecmpbll has joined #openstack-swift | 17:45 | |
| *** gyee has quit IRC | 17:54 | |
| *** mvkr has joined #openstack-swift | 17:56 | |
| *** gkadam has quit IRC | 18:19 | |
| *** openstackgerrit has quit IRC | 19:19 | |
| *** openstackgerrit has joined #openstack-swift | 19:20 | |
| openstackgerrit | Tim Burke proposed openstack/swift master: s3api: Stop mangling Authorization header for v4 signatures https://review.openstack.org/591862 | 19:20 |
| *** onovy has quit IRC | 19:37 | |
| *** onovy has joined #openstack-swift | 19:45 | |
| openstackgerrit | Tim Burke proposed openstack/swift master: s3api: Stop mangling Authorization header for v4 signatures https://review.openstack.org/591862 | 19:51 |
| timburke | wait, what? http://logs.openstack.org/41/577941/3/check/openstack-tox-lower-constraints/23bfa90/job-output.txt.gz#_2018-08-15_19_36_27_051571 | 20:10 |
| timburke | ugh, and i just noticed our shiny new lower-constraints job skips 1418 tests... | 20:11 |
| timburke | 'cause xattrs | 20:12 |
| DHE | oh dear | 20:13 |
| tdasilva | I left a +2 on p 575808, but held off on +A since clayg and zaitcev had left comments there in case you guys wanted to take a second look, otherwise i'd be ok to +A | 20:47 |
| patchbot | https://review.openstack.org/#/c/575808/ - swift - Add debugging info to SignatureDoesNotMatch responses - 5h 23m 59s spent in CI | 20:47 |
| tdasilva | reminder: no community meeting today | 20:50 |
| zaitcev | tdasilva: Tim explained my concerns, I just need to have a quick look again. | 20:53 |
| clayg | oh, umm... notmyname is out today right? | 21:02 |
| timburke | yup | 21:02 |
| clayg | sweet! I say we cancel the meeting? | 21:02 |
| timburke | i suppose i wouldn't mind getting more people's feedback on http://eavesdrop.openstack.org/irclogs/%23openstack-swift/%23openstack-swift.2018-08-13.log.html#t2018-08-13T17:35:38 but that can happen async, or wait until next week, or even until denver | 21:06 |
| timburke | hey, if i wait long enough, the question becomes moot because all of the stable branches will *also* be v3-only! | 21:07 |
| openstackgerrit | Thiago da Silva proposed openstack/swift master: Fix locking in swift-recon-cron https://review.openstack.org/592200 | 21:08 |
| tdasilva | timburke: IMO it would be ok to remove the v3-only job, especially since it is enabled in legacy-swift-dsvm-functional in queens | 21:12 |
| tdasilva | timburke: were you able to confirm that when moving legacy-swift-dsvm-functional in-repo, other projects will be able to use in-repo job too? | 21:14 |
| timburke | tdasilva: yup; see https://review.openstack.org/#/c/591528/ for example. haven't gotten around to proposing anything for devstack or devstack-gate yet, though | 21:15 |
| patchbot | patch 591528 - python-swiftclient - Use Swift's in-tree DSVM test - 1h 22m 19s spent in CI | 21:15 |
| timburke | (if we even still *need* the job for devstack-gate... i *think* that was maybe only there because the legacy jobs were using devstack-gate for setup??) | 21:17 |
| timburke | and *then* there's the need to think about stable branches for devstack... | 21:17 |
| timburke | there's a reason i wanted to just deal with the v3 job first | 21:18 |
| timburke | side-note: it's a little distressing how https://review.openstack.org/#/c/591530/ has four passing jobs... docs and pep8 i get, but the other two... :-( | 21:20 |
| patchbot | patch 591530 - python-swiftclient - DNM: horribly break auth - 1h 24m 40s spent in CI | 21:20 |
| tdasilva | pretty neat the way you can refer to another job defined in-repo in some other project | 21:20 |
| timburke | makes me wonder what happens if names collide, though | 21:20 |
| tdasilva | yeah and it doesn't really make it easy to find, plus is this only for repos under the openstack namespace?? | 21:22 |
| timburke | too much magic for my taste, but that seems to generally be the case for me & ansible... | 21:22 |
| *** rcernin has joined #openstack-swift | 21:29 | |
| timburke | tdasilva: looking at https://review.openstack.org/#/c/592200 -- what do you think about dropping the try/except and letting the exception bubble up? i'm not sure how much the logging buys us vs letting python do it -- and there'd be a side-benefit of exiting non-zero on error | 21:30 |
| patchbot | patch 592200 - swift - Fix locking in swift-recon-cron | 21:30 |
| timburke | idk... the current behavior just seemed kinda weird to me the more i thought about it | 21:30 |
| openstackgerrit | Tim Burke proposed openstack/swift master: Fix locking in swift-recon-cron https://review.openstack.org/592200 | 21:32 |
| tdasilva | timburke: I see your point about exiting non-zero on error, but not sure I like the stacktrace | 21:37 |
| timburke | ...aren't we going to get that anyway, logging at exception? | 21:38 |
| tdasilva | maybe we can print a message and sys.exit(1) ?? | 21:38 |
| tdasilva | we would get the exception in the logs | 21:40 |
| tdasilva | current way | 21:40 |
| timburke | *shrug* i mean, it's definitely better than it was... idk. let's merge it. if it bothers us enough, we can always change it later | 21:40 |
| timburke | oh! except! we might want the `with` inside the `try` | 21:41 |
| tdasilva | good point, it might raise a timeout | 21:42 |
| timburke | ...in which case we almost certainly *do* want the exit(1), to match the current couldn't-get-the-lock behavior | 21:42 |
| openstackgerrit | Tim Burke proposed openstack/swift master: Fix locking in swift-recon-cron https://review.openstack.org/592200 | 21:45 |
| openstackgerrit | Thiago da Silva proposed openstack/swift master: Fix locking in swift-recon-cron https://review.openstack.org/592200 | 21:48 |
| tdasilva | added just a print there too | 21:48 |
| openstackgerrit | Tim Burke proposed openstack/swift master: Fix locking in swift-recon-cron https://review.openstack.org/592200 | 21:51 |
| timburke | tabbies! | 21:51 |
| timburke | maybe we should just switch from gerrit to etherpad? :P | 21:51 |
| openstackgerrit | Merged openstack/swift master: Fix the deletion of non-existent keys https://review.openstack.org/575842 | 21:55 |
| tdasilva | lol | 21:55 |
| tdasilva | thanks! | 21:56 |
| timburke | p 575860 is reminding me that i'd really like to get some cross-api func tests going ... currently we don't really validate that objects written via the S3 api are readable via Swift or vice-versa ... | 21:59 |
| patchbot | https://review.openstack.org/#/c/575860/ - swift - Include '-' in multipart ETags - 10h 2m 43s spent in CI | 21:59 |
| *** FracKen has left #openstack-swift | 22:18 | |
| *** itlinux has quit IRC | 22:22 | |
| *** d0ugal has quit IRC | 22:33 | |
| openstackgerrit | Tim Burke proposed openstack/swift master: Include '-' in multipart ETags https://review.openstack.org/575860 | 22:34 |
| *** d0ugal has joined #openstack-swift | 22:44 | |
| *** mikecmpbll has quit IRC | 22:50 | |
| *** d0ugal has quit IRC | 23:01 | |
| *** d0ugal has joined #openstack-swift | 23:02 | |
| mattoliverau | morning | 23:14 |
| *** rcernin has quit IRC | 23:18 | |
| *** rcernin has joined #openstack-swift | 23:19 | |
| *** d0ugal has quit IRC | 23:42 | |
| *** d0ugal has joined #openstack-swift | 23:55 | |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!