Friday, 2018-08-24

*** mikecmpbll has quit IRC00:10
openstackgerritTim Burke proposed openstack/swift master: py3: port bufferedhttp (hopefully)  https://review.openstack.org/54872700:16
*** gyee has quit IRC00:44
*** ChanServ has quit IRC01:00
*** ChanServ has joined #openstack-swift01:03
*** barjavel.freenode.net sets mode: +o ChanServ01:03
*** itlinux has quit IRC01:07
*** itlinux has joined #openstack-swift01:07
openstackgerritMerged openstack/swift master: Fix warning while building docs  https://review.openstack.org/59585601:38
*** raginbajin has quit IRC01:46
*** raginbajin has joined #openstack-swift01:49
*** threestrands has quit IRC02:08
*** ianychoi has quit IRC02:28
*** joeljwright has quit IRC02:28
*** jlvillal has quit IRC02:28
*** rledisez has quit IRC02:28
*** zaitcev has quit IRC02:28
*** Miouge has quit IRC02:28
*** aluria has quit IRC02:28
openstackgerritKota Tsuyuzaki proposed openstack/swift master: Configure diskfile per storage policy  https://review.openstack.org/44712902:29
*** ianychoi has joined #openstack-swift02:30
*** joeljwright has joined #openstack-swift02:30
*** jlvillal has joined #openstack-swift02:30
*** rledisez has joined #openstack-swift02:30
*** zaitcev has joined #openstack-swift02:30
*** aluria has joined #openstack-swift02:30
*** Miouge has joined #openstack-swift02:30
*** barjavel.freenode.net sets mode: +vv joeljwright zaitcev02:30
*** openstack has joined #openstack-swift13:21
*** ChanServ sets mode: +o openstack13:21
*** openstackstatus has joined #openstack-swift13:25
*** ChanServ sets mode: +v openstackstatus13:25
*** itlinux-away has quit IRC13:53
*** itlinux has joined #openstack-swift14:35
*** viks__ has quit IRC14:38
*** links has quit IRC14:52
*** jlvillal is now known as jlviva-viva14:52
*** silor has quit IRC14:54
*** silor has joined #openstack-swift14:54
*** itlinux has quit IRC14:55
*** pcaruana has quit IRC14:59
*** itlinux has joined #openstack-swift15:09
*** mikecmpbll has quit IRC15:30
*** silor has quit IRC15:45
*** itlinux has quit IRC16:02
*** itlinux has joined #openstack-swift16:06
*** itlinux is now known as itlinux-away16:16
*** itlinux-away is now known as itlinux16:16
notmynamegood morning16:18
*** gyee has joined #openstack-swift16:19
notmynameclayg: timburke: tdasilva: Thoughts on potential dark data creation...16:27
notmynameIf we have the same drives for ACO, and the drives start to fill up, I think we may have the opportunity for async pendings to get lost and create dark data.16:27
notmynameobject PUT comes in, writes to the drive, fsyncs, then tries to update the container. Updates a different drive, gets a 507, fails, and tries to write an async pending. the current drive is now full and the async pending write fails (possibly).16:27
notmynameI can't see anything in the code that handles an async failing to be written. I think we simply hope that the other replicas manage to write it down. but there may be more opportunity for failure there that we realize, at least when you've got a cluster-full scenario16:27
notmynameam I missing something obvious?16:28
timburkeseems to make sense... i wonder if we should start marking data/meta/ts files after either we either succeed in doing the container update or succeed in fsyncing the async pending... then have the auditor start kicking out async pendings for unmarked guys16:33
timburkehttps://review.openstack.org/#/c/212824/ would be so nice to have :-(16:33
patchbotpatch 212824 - swift - Let developers/operators add watchers to object audit - 8h 24m 15s spent in CI16:33
notmynametimburke: I think we're correct in making the object server response 2xx once the object is written and not after the async is written (ie container update or async doesn't affect the object write response code). however, there is still a window (see above or similar) where the container update is lost16:37
notmynameor rather, doing two writes (object + async) in a single transaction that successfully rolls back if the second one fails sounds *way* harder to solve :-)16:38
*** gkadam has quit IRC16:38
notmynamecschwede: thanks for working on the py3 stuff! :-)16:39
timburkeright -- i'm jus saying that we could switch to writing down "hey, there's no work to be done here" instead of writing down "go do this extra work later" to ensure a disk-full causes us to err on the side of doing work twice rather than not at all16:39
*** gkadam has joined #openstack-swift16:52
*** silor has joined #openstack-swift16:57
*** silor has quit IRC17:20
*** itlinux is now known as itlinux-away17:27
*** itlinux-away is now known as itlinux17:29
*** itlinux is now known as itlinux-away17:29
*** itlinux-away is now known as itlinux17:30
*** itlinux is now known as itlinux-away17:30
*** itlinux-away has quit IRC17:35
notmynametimburke: https://review.openstack.org/#/c/595316/ vs https://review.openstack.org/#/c/596210/ ? I'm not sure which should land18:07
patchbotpatch 595316 - swift - Reference legacy-tempest-dsvm-neutron-full and neu... - 1h 44m 34s spent in CI18:07
patchbotpatch 596210 - swift - import zuul job settings from project-config - 1h 47m 28s spent in CI18:07
* notmyname was looking at the "mechanical changes for py3" that were proposed overnight18:07
timburkesome combination of both, i think18:08
timburkei think not running these jobs on purely requirements or setup.cfg changes is a mistake, but one that's pre-existing. i intentionally *didn't* include those when i wrote my patcch18:12
timburkepulling in the templates and queue and post jobs and all that it likely a good thing -- we don't really want to have anything left in project-config i think18:13
notmynameI'm not as concerned with the differences in excluded files. more about the other stanzas like the templates one. I'm not as sure what that means18:13
timburkethe experimental jobs, i'm not familiar with. i find it strange that we'd have irrelevant-files for those, since they're only run on-demand anyway (by leaving a "check experimental" comment)18:15
notmynamewhat kind of stuff would we define there? like a one-off "test that these two things still work together"?18:16
notmynameor a performance test or soemthing like that?18:16
timburkei guess? *shrug*18:16
notmynameor is it supposed to be a staging area? like we've used non-voting jobs for18:16
*** gkadam has quit IRC18:30
*** silor has joined #openstack-swift18:58
*** mrjk has quit IRC19:13
*** silor has quit IRC19:50
*** mrjk has joined #openstack-swift19:53
zaitcevGuys20:04
zaitcevAnyone wants to share a room with me in Denver?20:04
zaitcevThe best they have is $167.20:05
* zaitcev pokes cschwede20:05
*** itlinux has joined #openstack-swift20:13
*** itlinux has quit IRC20:58
*** onovy has joined #openstack-swift20:59
openstackgerritMerged openstack/swift-bench master: import zuul job settings from project-config  https://review.openstack.org/59621221:03
notmynametimburke: I've learned a few things today...21:46
notmynameeventually http://logs.openstack.org/56/591156/2/check/zuul-build-dashboard/a1d106d/npm/html/jobs.html will be useful for understanding zuul job definitions21:46
notmynameeg http://logs.openstack.org/56/591156/2/check/zuul-build-dashboard/a1d106d/npm/html/job.html?job_name=tempest-full-py3 and keep clicking the parent link21:47
notmynamealso, the jobs in swift that have "neutron" in them is because it's to distinguish against nova-net21:47
notmynamealso, I sortof learned how jobs compose21:47
notmyname'internally in zuul, that is a "project-pipeline job variant" which is a form of job definition.  but it only applies to swift because it's attached to swift's project-pipeline configuration.  but it's probably easier to think of things in programming terms and consider that more like the "invocation" of the job.'21:48
notmynamefrom corvus21:48
notmynameand we can think of stuff like legacy-tempest-dsvm-neutron-full in zuul_jobs and the swift repo as additive21:48
notmynameand eventually many, but not all, jobs are to be defined in-repo. this is close to what we thought, but not exactly21:49
notmynamesome, I assume including legacy-tempest-dsvm-neutron-full and similar ones, will still be defined in zuul_jobs because they're cross project21:49
notmynameie don't define them in each project that runs them21:50
notmynamehttps://zuul-ci.org/docs/zuul/user/config.html#configuration-loading and https://zuul-ci.org/docs/zuul/user/config.html#job have some more info on jobs21:51
notmynamealso, lib-forward-testing-python3 is ultimately defined (with a few layers of indirection) in tempest, and it explicitly disables swift. so let's not add it to our repos21:51
notmynameI'm told that disabling is because swift doesn't support py3 yet, so I assume the plan is to add it later21:52
notmynameand also, http://codesearch.openstack.org/?q=tempest-full-py3&i=nope&files=&repos= exists and is the recommended way to find stuff like zuul job definitions that can live in any repo whatsoever21:53
zaitcevHmm.22:18
zaitcevnotmyname: do you know anyone who wants to share rooms?22:18
*** itlinux has joined #openstack-swift22:19
notmynamezaitcev: I do not know. sorry. perhaps asking on the mailing list or in the -ptg channel could find someone22:20
timburkenotmyname: i'm skeptical of the future utility of a page that (currently; i expect the list to only grow) takes up 30 screens on my Giant Monitor of Doomâ„¢22:22
*** mikecmpbll has joined #openstack-swift22:30
*** gyee has quit IRC22:52
*** itlinux has quit IRC22:57
notmynametimburke: ...ok.... http://logs.openstack.org/60/595260/2/gate/swift-tox-func-s3api/6a0b60f/job-output.txt.gz#_2018-08-24_22_37_22_54351023:08
timburkenotmyname: i've seen that sort of thing on occasion... i think it comes back to timeouts popping while talking to backend servers?23:10
notmynamehmm23:10
timburkeissue a(nother) recheck when it finally reports the failure23:11
notmynameI did23:14
notmynamewell, didn't wait on the failure report, but it is failed in the gate23:15
openstackgerritMerged openstack/swift master: Add irrelevant-files lists for in-tree check/gate jobs  https://review.openstack.org/59530923:51
notmynameI was hoping the swift stuff in the gate would land ...swiftly... but alas, no such luck23:55
notmynamepatch 59530923:55
patchbothttps://review.openstack.org/#/c/595309/ - swift - Add irrelevant-files lists for in-tree check/gate ... (MERGED) - 8h 11m 9s spent in CI23:55
notmynameyikes. 8+ hours23:56
notmynamehave a good weekend, swift-world23:56
*** tristanC has quit IRC23:56

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