*** Fengli has joined #openstack-swift | 01:16 | |
*** osmboy has quit IRC | 01:29 | |
*** Tuatara__ has joined #openstack-swift | 02:46 | |
*** evrardjp has quit IRC | 05:33 | |
*** Tuatara__ has quit IRC | 05:33 | |
*** evrardjp has joined #openstack-swift | 05:33 | |
openstackgerrit | Thiago da Silva proposed openstack/swift master: Account listing for object versioning https://review.opendev.org/700411 | 08:21 |
---|---|---|
*** rdejoux has joined #openstack-swift | 08:27 | |
*** Fengli has quit IRC | 09:08 | |
*** theintern_ has joined #openstack-swift | 13:38 | |
clayg | tdasilva: i'm taking a peak at account listings - it looks like you have it doing the double account listing! 👍 | 14:40 |
clayg | oh, i guess it's still a little WIP - i got a traceback | 14:43 |
clayg | I can't quite work out why the gate probetests are hitting 412 in p 700449 | 15:30 |
patchbot | https://review.opendev.org/#/c/700449/ - swift - Allow reconciler to handle reserved names - 2 patch sets | 15:30 |
clayg | Dec 31 15:44:55 saio swift: removed request headers: {'X-Backend-Allow-Reserved-Names': 'true'} (txn: tx64b55a697ed9453e80a1c-005e0b6cf7) !? | 15:45 |
clayg | where do we add required middlewares? I thought it was someplace in common.wsgi? | 15:53 |
clayg | I thought InternalClient somehow managed to prevent the GateKeeper auto insert 🤔 | 15:53 |
clayg | modify_wsgi_pipeline is in proxy.server | 15:54 |
clayg | allow_modify_pipeline=False in InternalClient when calling loadapp (normally loadapp triggers modify_wsgi_pipeline) | 15:57 |
clayg | so I was only able to duplicate the issue locallying by using my proxy conf that explicitly added the gatekeeper - but maybe that's not what's happening in the gate | 16:02 |
clayg | b/c I don't see the "removed request headers" 😞 | 16:03 |
clayg | oh, hrm.. there's no noauth proxy in the gate probetests and the config explicitly adds gatekeeper - ok this is starting to make sense | 16:04 |
openstackgerrit | Clay Gerrard proposed openstack/swift master: Allow reconciler to handle reserved names https://review.opendev.org/700449 | 16:19 |
clayg | 🤞 | 16:19 |
*** rdejoux has quit IRC | 16:21 | |
*** evrardjp has quit IRC | 17:33 | |
*** evrardjp has joined #openstack-swift | 17:33 | |
clayg | 🥳 | 18:03 |
*** theintern_ has quit IRC | 18:46 | |
clayg | So i've got some unittests that create some apps and I'm trying to track down some unexpected double logging... | 19:21 |
clayg | Has anyone else ever noticed we get "Pipeline was modified" and "Loaded override config for" in logs multiple times everytime we start an app? | 19:24 |
clayg | I'm not sure if it's all related, but I was looking at that modify pipeline code -> https://github.com/openstack/swift/blob/7862ec7b8a7c56fffc32e0bfac3665f8b4f198dc/swift/common/wsgi.py#L401 | 19:24 |
clayg | That paste.deploy interface, where the only way to get to the Application is thorugh the app_factory method is pretty dumb | 19:25 |
clayg | because we have to *create* an instance of the app just to get ahold of our modify_wsgi_pipeline method | 19:25 |
clayg | but then we still have to create the app again from scratch before we can return the wrapped pipeline app 🤔 | 19:26 |
clayg | I wonder if we could monkey patch create to cache it 😬 | 19:26 |
clayg | ok, well part of the reason we're getting multiple log lines is that we have workers and each process creates their own app | 19:28 |
clayg | but at least if we cache the app_context's create value we can only log things *once* per worker? it's *probably* safe 🤣 | 19:29 |
mattoliverau | Hey all, didn't realise the NSW bush fires in Oz where making the news elewhere. But seems it has, thought I'd better let people know I'm ok. Luckily in Melbourne visiting my in-laws for New year's. The fire ravaged my area at home, but think I still have a house to go back too. And my parents are ok, last I heard. Communication is down now. I know some friends lost their house a few streets away, but from what I | 19:35 |
mattoliverau | gather my beach house was closer to the water, and just outside the fire front. Well from what I can tell from here. | 19:35 |
openstackgerrit | Clay Gerrard proposed openstack/swift master: Deprecate per-service auto_create_account_prefix https://review.opendev.org/700818 | 19:51 |
clayg | mattoliverau: OMG!!! I'm so glad you're able to connect and let folks know you're ok - please be safe! 🤗 | 19:58 |
*** early has quit IRC | 21:34 | |
*** early has joined #openstack-swift | 21:48 |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!