Wednesday, 2020-07-22

*** gyee has quit IRC00:20
timburkeclayg, do we have a set of recommended/supported REGIONS/ZONES/NODES/DISKS/EC_DISKS/REPLICAS for https://github.com/swiftstack/vagrant-swift-all-in-one/ ? i'm trying some kinda off-the-wall configs and it keeps failing bring-up because the dispersion's crap...00:53
timburkewhich, fair enough -- it is -- but idk that we should abort the rest of building the rings because of it...00:53
openstackgerritArzhna Lee proposed openstack/swift master: Allow lowercase region name for AWS .NET SDK compatibility  https://review.opendev.org/74232200:55
openstackgerritArzhna Lee proposed openstack/swift master: Allow lowercase region name for AWS .NET SDK compatibility  https://review.opendev.org/74232201:01
openstackgerritArzhna Lee proposed openstack/swift master: Allow lowercase region name for AWS .NET SDK compatibility  https://review.opendev.org/74232201:05
openstackgerritArzhna Lee proposed openstack/swift master: Allow lowercase region name for AWS .NET SDK compatibility  https://review.opendev.org/74232201:07
openstackgerritArzhna Lee proposed openstack/swift master: Allow lowercase region name for AWS .NET SDK compatibility  https://review.opendev.org/74232201:08
seongsoochomorning!  ^ above issue occured in my cluster.  The AWS only has a lowercase region name . So some of sdk change the region name to lowercase ...01:31
timburkeman, that makes our old default before https://github.com/openstack/swift/commit/692a034 even more of a pain...01:38
timburkewill review tomorrow01:38
seongsoochotimburke:   oh I see.  As my customers use s3 api more and more, there are many issue report. :-)  .01:47
*** baojg has quit IRC03:10
openstackgerritMerged openstack/swift master: Decouple proxy_server tests  https://review.opendev.org/74228103:23
*** psachin has joined #openstack-swift03:31
openstackgerritTim Burke proposed openstack/swift master: py3: Work with proper native string paths in crypto meta  https://review.opendev.org/74203303:38
openstackgerritTim Burke proposed openstack/swift stable/ussuri: py3: Stop munging RAW_PATH_INFO  https://review.opendev.org/74233803:44
openstackgerritTim Burke proposed openstack/swift stable/train: py3: Stop munging RAW_PATH_INFO  https://review.opendev.org/74233903:44
*** mahatic has joined #openstack-swift03:45
*** ChanServ sets mode: +v mahatic03:45
ormandjtimburke: burning the midnight oil03:58
timburkei shouldn't -- but my gate's finally (mostly) fixed, so i wanted to get some stuff moving03:59
ormandji'll look around tomorrow and see how that other patch is behaving to give some more feedback04:00
openstackgerritMerged openstack/swift master: probe tests: Get rid of `server` arg for device_dir() and storage_dir()  https://review.opendev.org/74154404:12
openstackgerritMerged openstack/swift master: Stop syncing empty suffixes list  https://review.opendev.org/74153804:22
*** camelCaser has quit IRC04:23
*** camelCaser has joined #openstack-swift04:26
openstackgerritMerged openstack/swift stable/ussuri: Use latest pip in gate jobs  https://review.opendev.org/74230404:48
openstackgerritMerged openstack/swift stable/train: Use latest pip in gate jobs  https://review.opendev.org/74230504:48
*** m75abrams has joined #openstack-swift04:53
openstackgerritzhouyunyang proposed openstack/swift master: skip same suffixes sync when update_delete  https://review.opendev.org/74234205:06
openstackgerritMerged openstack/swift master: py3: Stop munging RAW_PATH_INFO  https://review.opendev.org/73785605:07
*** rcernin has quit IRC07:32
*** gtema has joined #openstack-swift07:33
openstackgerritzhouyunyang proposed openstack/swift master: skip same suffixes sync when update_delete  https://review.opendev.org/74236007:43
*** gtema has quit IRC08:20
*** gtema has joined #openstack-swift08:24
*** baojg has joined #openstack-swift08:26
*** cschwede has joined #openstack-swift08:32
*** ChanServ sets mode: +v cschwede08:32
*** gtema has quit IRC08:39
*** gtema has joined #openstack-swift08:41
*** rcernin has joined #openstack-swift09:11
*** rcernin has quit IRC09:35
openstackgerritDavid Sariel proposed openstack/swift master: Let developers/operators add watchers to object audit (simplified)  https://review.opendev.org/70665310:35
*** tkajinam has quit IRC10:59
*** gtema has quit IRC11:34
*** baojg has quit IRC11:49
*** rcernin has joined #openstack-swift11:49
*** baojg has joined #openstack-swift11:50
*** gtema has joined #openstack-swift11:58
*** gtema has quit IRC12:03
*** gtema has joined #openstack-swift12:09
*** baojg has quit IRC12:11
*** baojg has joined #openstack-swift12:12
*** nottrobin_ has joined #openstack-swift12:31
*** nottrobin has quit IRC12:38
*** nottrobin_ is now known as nottrobin12:38
*** mahatic has quit IRC12:41
*** baojg has quit IRC13:23
*** baojg has joined #openstack-swift13:24
*** gtema has quit IRC13:31
*** rcernin has quit IRC13:32
*** baojg has quit IRC13:41
*** baojg has joined #openstack-swift13:43
openstackgerritDavid Sariel proposed openstack/swift master: Let developers/operators add watchers to object audit (simplified)  https://review.opendev.org/70665313:57
*** ormandj has quit IRC14:06
*** ormandj has joined #openstack-swift14:09
openstackgerritDavid Sariel proposed openstack/swift master: Let developers/operators add watchers to object audit (simplified)  https://review.opendev.org/70665314:18
*** baojg has quit IRC14:22
*** baojg has joined #openstack-swift14:24
*** psachin has quit IRC14:24
*** paladox has quit IRC14:32
*** paladox has joined #openstack-swift14:53
*** gyee has joined #openstack-swift15:48
*** baojg has quit IRC15:54
*** baojg has joined #openstack-swift15:56
*** m75abrams has quit IRC16:21
*** jadonn has joined #openstack-swift16:33
*** baojg has quit IRC16:34
*** cschwede has quit IRC16:41
openstackgerritClay Gerrard proposed openstack/swift master: Start to decouple Object GET path  https://review.opendev.org/73391117:20
openstackgerritClay Gerrard proposed openstack/swift master: Add concurrent_gets to EC GET requests  https://review.opendev.org/71134217:20
*** Nisha_Agarwal has joined #openstack-swift17:22
Nisha_AgarwalHello all17:23
Nisha_AgarwalI wanted to ask a query....Does swift container configuration support NFS URL scheme configuration as well?17:24
openstackgerritClay Gerrard proposed openstack/swift master: swift-container-info: Show shard ranges summary  https://review.opendev.org/73705617:29
timburkeNisha_Agarwal, i'm not quite sure what that would mean -- would you want to expose a swift container as a filesystem over nfs? i think there are some other projects that might be able to do that, but we don't support it natively17:34
Nisha_Agarwaltimburke, Thanks. I wanted to ask from tempurl perspective. In ironic we create the media and place it in swift containers and generally swift temp url scheme is http(s)17:35
Nisha_Agarwaltimburke, in this aspect is it possible to configure swift containers in such a way that the tempurls generated at run time has nfs url scheme17:36
Nisha_Agarwalthe hardware has a limitation to use nfs or cifs17:37
Nisha_Agarwalso just wanted to know on that aspect17:37
timburkeall of swift's apis are built on http, so it would make sense to me that tempurls would be http (or https) -- if there's a hard requirement to use nfs, you'll likely need to have some sort of a gateway that speaks nfs on one side and translates it to swift (http) requests on the other17:39
timburkei think that'll be entirely separate from thinking about tempurls -- they only really make sense in the context of http (where you can have query parameters as part of the protocol); i don't think nfs supports anything quite like that17:40
timburkethe auth mechanism will likely be very different17:40
Nisha_Agarwaltimburke, ok ...the temporary url generated has to be used for inserting to virtual cdrom of the server17:40
Nisha_Agarwaltimburke, agreed17:40
Nisha_Agarwaltimburke, i couldnt find any documentation related to nfs hence i pinged here to validate if my findings are correct.17:41
timburkefwiw, there's https://github.com/NVIDIA/vdisc/ which says it's "a tool for creating and mounting virtual CD-ROM images backed by object storage"17:42
timburke(full disclosure: that's put out by my employer, and i haven't actually played around with it yet)17:42
*** baojg has joined #openstack-swift19:07
*** Nisha_Agarwal has quit IRC19:27
*** baojg has quit IRC19:31
timburkehrm. i think bug 1446873 and p 740867 complicate the upgrade story for p 735751 a decent bit... is it worth adding a use_legacy_network option, or should we just include a release note like "When upgrading from earlier versions of Swift, expect replication errors until all nodes are upgraded"?19:32
openstackbug 1446873 in OpenStack Object Storage (swift) "ssync doesn't work with replication_server = true" [Medium,In progress] https://launchpad.net/bugs/144687319:32
patchbothttps://review.opendev.org/#/c/740867/ - swift - Allow replication servers to handle all request me... - 2 patch sets19:32
patchbothttps://review.opendev.org/#/c/735751/ - swift - Allow direct and internal clients to use the repli... - 4 patch sets19:32
*** baojg has joined #openstack-swift19:32
claygouch, upgrades 😞19:48
timburkeif we go with an option for it, it won't be as narrow as i'd originally done, either -- we'd want it for *everything* :-(19:50
claygLAME!19:52
claygso but we already deploy with replication_server = True commnted out?  I mean... what is actually blocking requests until post upgrade?19:53
clayglike we might have replication_server = True turned on the database servers; so maybe UpgradeImpact just reminds us to turn that off - then upgrade - then you can turn it back on (because post upgrade `replication_server = True` is the same as `# replication_server`)19:55
claygI guess that's not ideal (you have to change your configs or get errors during upgrade) ... but I can live with it (if it works)19:55
timburke*for us*? hopefully nothing? i guess the release note could be like "if you have replication_servers=true in any configs, remove that line to ensure a smooth rolling upgrade"... not so bad...19:56
timburkedid you just check object, or a/c as well?19:56
claygyeah we HAD to turn off object - but until now there was no obvious reason to disable it for a/c19:58
claygso yeah; I think this is all workable 💪19:58
claygunless we forgot about something 🤣19:58
openstackgerritClay Gerrard proposed openstack/swift master: container-sharding: Stable overlap order  https://review.opendev.org/74253520:21
zaitcevtimburke: are having a meeting in 40 minutes?20:21
timburkethat's my plan -- sorry, haven't updated the agenda yet20:22
openstackgerritMerged openstack/swift stable/train: Breakup reclaim into batches  https://review.opendev.org/74175520:40
openstackgerritMerged openstack/swift stable/ussuri: Breakup reclaim into batches  https://review.opendev.org/74175320:59
openstackgerritTim Burke proposed openstack/swift master: Allow replication servers to handle all request methods  https://review.opendev.org/74086720:59
openstackgerritTim Burke proposed openstack/swift master: Allow direct and internal clients to use the replication network  https://review.opendev.org/73575120:59
openstackgerritTim Burke proposed openstack/swift master: wip: Allow probe tests to run with separate replication network  https://review.opendev.org/74172320:59
*** baojg has quit IRC21:22
*** baojg has joined #openstack-swift21:23
clayglots to talk about today!22:01
timburke:-( so direct_client (at least) *will* spew tracebacks on 405...22:23
timburkeseems like internal_client must not22:28
openstackgerritTim Burke proposed openstack/swift master: Actually interpolate error message  https://review.opendev.org/74254922:43
*** rcernin has joined #openstack-swift22:52
*** rcernin has quit IRC22:58
*** rcernin has joined #openstack-swift23:01
*** rcernin has quit IRC23:04
*** rcernin has joined #openstack-swift23:05

Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!