Friday, 2019-08-09

timburkeclayg, oh right! of *course* symlink's fine with the override footer -- it's zero bytes and doesn't *use* footers! looks like it broke slo, though: https://logs.opendev.org/51/675451/1/check/swift-tox-func-ec/8401cd9/job-output.txt.gz#_2019-08-08_23_01_33_28285700:00
*** gyee has quit IRC00:43
*** spsurya has joined #openstack-swift01:05
mattoliveraubelated morning :)01:08
mattoliverauclayg, timburke: yeah, symlinks feel close to posix symlinks, but suddendly hardlink's aren't the same thing.. so makes me now really like the name. static link is _much_ better.01:09
*** baojg has joined #openstack-swift01:23
*** altlogbot_0 has quit IRC02:44
*** altlogbot_2 has joined #openstack-swift02:46
*** redrobot has quit IRC03:22
*** zaitcev has quit IRC03:27
*** psachin has joined #openstack-swift03:36
*** zaitcev has joined #openstack-swift03:39
*** ChanServ sets mode: +v zaitcev03:39
*** baojg has quit IRC04:03
*** zaitcev_ has joined #openstack-swift04:38
*** ChanServ sets mode: +v zaitcev_04:38
*** zaitcev has quit IRC04:42
*** ianychoi_ has quit IRC04:44
*** baojg has joined #openstack-swift04:44
*** [diablo]9 has quit IRC05:25
*** zaitcev__ has joined #openstack-swift05:27
*** ChanServ sets mode: +v zaitcev__05:27
*** zaitcev_ has quit IRC05:31
*** e0ne has joined #openstack-swift05:40
*** e0ne has quit IRC05:59
*** zaitcev__ has quit IRC06:47
*** zaitcev__ has joined #openstack-swift07:00
*** ChanServ sets mode: +v zaitcev__07:00
*** tesseract has joined #openstack-swift07:04
*** takamatsu has joined #openstack-swift07:35
*** zaitcev__ has quit IRC07:43
*** rcernin has quit IRC07:46
*** zaitcev__ has joined #openstack-swift07:55
*** ChanServ sets mode: +v zaitcev__07:55
*** psachin has quit IRC08:07
*** mikecmpbll has joined #openstack-swift08:07
*** psachin has joined #openstack-swift08:08
*** zaitcev_ has joined #openstack-swift08:16
*** ChanServ sets mode: +v zaitcev_08:16
openstackgerritMerged openstack/swift master: py3: Add swift-tox-func-encryption-py37 gate job  https://review.opendev.org/67487808:19
*** zaitcev__ has quit IRC08:19
*** jistr is now known as jistr|afk08:24
*** e0ne has joined #openstack-swift09:24
*** tesseract has quit IRC09:28
*** tesseract has joined #openstack-swift09:28
*** tesseract has quit IRC09:29
*** tesseract has joined #openstack-swift09:29
*** zaitcev_ has quit IRC09:33
*** zaitcev_ has joined #openstack-swift09:47
*** ChanServ sets mode: +v zaitcev_09:47
*** hoonetorg has quit IRC09:57
*** jistr|afk is now known as jistr10:18
*** hoonetorg has joined #openstack-swift10:20
viks___hi, i upgraded from queens to stein, but i see object server service  cpu usage is around 40% even if no load is there? Anyone have any idea why this is happening?  Not sure what % of cpu earlier for object server process i.e. in queens. Also my overall cpu usage(user) of storage node is around 80% after upgrade, Earlier  it was around 45%.10:51
*** tdasilva has joined #openstack-swift11:34
*** ChanServ sets mode: +v tdasilva11:34
DHEcan you identify the specific process consuming the CPU?12:36
*** zaitcev_ has quit IRC12:43
viks___object server12:52
*** zaitcev_ has joined #openstack-swift12:58
*** ChanServ sets mode: +v zaitcev_12:58
*** henriqueof has joined #openstack-swift12:58
*** Guest1857 has joined #openstack-swift12:59
*** zaitcev_ has quit IRC13:19
*** BjoernT has joined #openstack-swift13:28
*** jistr is now known as jistr|call13:29
*** zaitcev_ has joined #openstack-swift13:32
*** ChanServ sets mode: +v zaitcev_13:32
*** zaitcev__ has joined #openstack-swift13:33
*** ChanServ sets mode: +v zaitcev__13:33
*** zaitcev_ has quit IRC13:37
*** jistr|call is now known as jistr13:38
*** tdasilva has quit IRC13:44
*** tdasilva has joined #openstack-swift13:45
*** ChanServ sets mode: +v tdasilva13:45
viks___Is it safe to upgrade from one version to another version keeping the conf files intact?13:49
DHEgenerally yes, but do read the changelog for anything you should be aware of13:53
*** spsurya has quit IRC13:54
*** zaitcev__ is now known as zaitcev14:14
*** psachin has quit IRC14:16
*** zaitcev has quit IRC14:21
*** zaitcev has joined #openstack-swift14:35
*** ChanServ sets mode: +v zaitcev14:35
viks___ok14:49
claygviks___: keep an eye on it, it might go down15:01
claygMostly the work done by the object-server is related to client traffic - so you wouldn't expect a resource change unless it's related to a change in your traffic patterns15:02
clayghowever, unless you're running seperate dedicated replication object-server processes - they're also involved in REPLICATE requests15:02
claygand those requests can take up cpu time (although really it's just cpu time waiting on os calls talking to disks)15:03
claygI don't remember the exact time frames but there were some bug fixes and optimizations made to hash recalculation (REPLICATE requests) that might cause a temporary (?) spike in hash recalculation of old hash files... maybe...15:05
claygdepending on the size of your cluster is could go away after a few days or a week - but if keeps up, it'd be interesting to me at least to know more about what's going on15:06
*** zaitcev has quit IRC15:27
*** zaitcev has joined #openstack-swift15:40
*** ChanServ sets mode: +v zaitcev15:40
*** e0ne has quit IRC15:44
*** gyee has joined #openstack-swift15:50
*** mikecmpbll has quit IRC16:03
*** guimaluf has quit IRC16:09
*** henriqueof has quit IRC16:14
openstackgerritClay Gerrard proposed openstack/swift master: Consolidate Container-Update-Override headers  https://review.opendev.org/67545116:14
openstackgerritClay Gerrard proposed openstack/swift master: WIP: Use Sysmeta Container Override in footers  https://review.opendev.org/67565716:14
*** Guest1857 is now known as redrobot16:17
*** tdasilva has quit IRC16:24
claygmattoliverau: timburke: I'm going to try redoing the docs with defining a "Static Symlink"16:29
claygoh this is great, I'm even taking to refering to "dynamic Symlinks" to draw contrast where appropriate16:32
openstackgerritClay Gerrard proposed openstack/swift master: Allow "harder" symlinks  https://review.opendev.org/63309416:43
clayghopefully if Zuul goes for it p 675451 will be non-controversial16:44
patchbothttps://review.opendev.org/#/c/675451/ - swift - Consolidate Container-Update-Override headers - 2 patch sets16:44
claygI think the docs on p 633094 have a much better shot of being understandable with static vs. dynamic symlinks (cc mattoliverau)16:45
patchbothttps://review.opendev.org/#/c/633094/ - swift - Allow "harder" symlinks - 25 patch sets16:45
*** csmart has quit IRC16:45
timburke👍16:45
*** irclogbot_1 has quit IRC16:47
*** irclogbot_1 has joined #openstack-swift16:50
*** zaitcev has quit IRC16:56
*** csmart has joined #openstack-swift16:56
viks___clayg: Thanks..  Not running dedicated replication object-server processes(i think you meant replication_server = false under [app:object-server]).. cluster size is not so big.. it's 2 storage nodes each having 9 object storage disks occupying around 25Gb of disk usage16:58
*** mikecmpbll has joined #openstack-swift16:59
*** tesseract has quit IRC17:00
*** zaitcev has joined #openstack-swift17:07
*** ChanServ sets mode: +v zaitcev17:07
openstackgerritMerged openstack/swift master: slo: Add X-Manifest-Etag to responses  https://review.opendev.org/67546917:21
claygviks___: well all of that sounds fine except for the inrecased cpu load then 🤣17:25
claygviks___: you could stop the object-replicators and see if cpu changes - that might hint at something... or maybe see how many worker processes is being spawned?  I think the default is "auto" but that's probably not a recent change...17:26
viks___clayg: 4 obkect-server workers because of 4 CPU s.. each process has 21 threads including main thread.. I tried changing reducing and increasing no. of  threads but CPU usage is around 45%.. if I increase workers, per CPU usage reduces but overall CPU usage remains around 80%.. earlier overall CPU usage itself was around 45%..17:37
*** mikecmpbll has quit IRC17:58
*** mikecmpb_ has joined #openstack-swift17:59
claygtimburke: I'm thinking about the tempest tests and p 63385718:01
patchbothttps://review.opendev.org/#/c/633857/ - swift - symlink-backed versioned_writes - 10 patch sets18:01
claygI feel like the right answer is to go ahead and make use_symlinks default to False?  But I'd like the versioned_writes test work in at least most of the test enviornments 🤔18:01
claygAt first I thought if it defaults to false, but I enabled it etc/proxy-server.conf (like we'd want all new clusters to do) then the tempest test would pass and the functional environments we configure on our own would work... but I'm not sure how all of our test environments are configured18:02
clayglike... do the dsvm deployments use our example config - or they just pass because we were defaulting to on?18:03
claygi don't think I can very easily make tests work with *either* config (basicaly bifricate all the assertions that the diff updates)18:03
claygworse if we have both branches, we'd probably want to run both configs in the gate, and that seems like a lot of overhead for maintaining tests... seems like a not great situation...18:04
claygnot obvious to me at all which direction I should be going18:04
claygthe threads are the eventlet tpool of 20 and the main thread18:05
claygviks___: ^ the 4 workers sounds right18:05
claygviks___: I guess I'm still guessing it's somehow related to replication somehow (temporarily stopping replicators would test hypothosis)18:06
claygviks___: I don't *remember* merging any changes that said "make object-server half as efficient" 🤣18:06
claygmaybe less tonge in cheek, we monitor that sort of thing in our clusters and haven't seen any other complaints - but we wouldn't have upgraded directly from Q to S so it could be a frog in water effect 🤔18:08
claygviks___: try upgrading to an even NEWER swift!!! new swift is best.18:08
openstackgerritPete Zaitcev proposed openstack/swift master: py3: Fix testUtf8Container (test.functional.tests.TestContainer)  https://review.opendev.org/67524418:20
claygmaybe I could mostly get the old tests passing on either config... I might even be able to minimize overhead with an `_assertContainerCount(n)` that says `if not self.use_symlinks: n -= 1` 🙄18:22
viks___clayg: Ok.. I'll try to investigate by stopping the replicators maybe on Monday, and update here my observations..18:24
claygviks___: thanks!18:28
claygGL18:28
timburkeclayg, it might be reasonable to have some func tests running use_symlinks=true and some use_symlinks=false... we already have a bunch of overlap between, say, swift-tox-func and swift-tox-func-encryption -- we could just pick one and say, "ok, you get symlinks"19:29
timburkei'm kinda amazed that swift-multinode-rolling-upgrade passes on that patch...19:29
claygWell, that runs old code old func new code new func yeah?19:30
claygI think with enough work I can get most of the existing tests passing on both. And the new tests can Skip 🤷‍♂️19:31
claygThen I just default false and try to get some env to run with use_symlinks - does that sound like a reasonable direction?19:31
*** e0ne has joined #openstack-swift19:41
*** BjoernT has quit IRC19:44
timburkeoh! right. it's new code for storage, old code for proxy, old func tests. so proxy doesn't support the new stuff and it's fine19:45
timburkeyeah, sounds reasonable19:45
claygha, found a bug with use_symlinks = false already!  😢19:53
openstackgerritPete Zaitcev proposed openstack/swift master: Allow Connection: value to be case-insensitive  https://review.opendev.org/67569620:07
*** diablo_rojo has joined #openstack-swift20:16
*** e0ne has quit IRC20:18
*** diablo_rojo has quit IRC20:51
claygtimburke: ok, fixed that bug - i'll need to keep working through tests to see what else pops up21:01
zaitcevUgh21:21
*** e0ne has joined #openstack-swift21:21
zaitcevWe have a functest that does this: sends PUT with Transfer-Encoding: gzip,chunked but omits Content-Length. It expects 501.21:22
zaitcevGuess what21:22
zaitcevApache LB turns it around with 500 before it hits Swift proxy.21:23
zaitcevAt this point I stopped Apache and returned to stunnel to terminate SSL.21:23
timburkewhee!21:26
timburkeyeah, personally i much prefer an ssl terminator that *just* terminates21:27
timburkemaybe https://github.com/openstack/swift/blob/2.22.0/test/functional/__init__.py#L822 would be useful?21:30
timburkei know we use it in some places like https://github.com/openstack/swift/blob/2.20.0/test/functional/tests.py#L2527-L2529 (though that skip message seems to be exactly wrong...)21:30
*** e0ne has quit IRC21:32
*** e0ne has joined #openstack-swift21:36
*** e0ne has quit IRC21:45
zaitcevAug  9 18:08:14 rhev-a24c-01 object-server[29481]: STDERR: Exception ignored in:22:09
zaitcevAug  9 18:08:14 rhev-a24c-01 object-server[29481]: STDERR: <function _after_fork at 0x7fb2975e9620>22:09
*** mikecmpb_ has quit IRC22:09
*** mikecmpbll has joined #openstack-swift22:14
zaitcevSeems working fine though.22:15
timburkezaitcev, any more context to that? like what the exception was, or maybe even better, the stack?22:22
zaitcevNo, that's all there is to it. Every server spews it on start, just once.22:23
*** mikecmpbll has quit IRC22:24
zaitcevIf you didn't see it, it's probably Fedora specific22:24
zaitcevFunctests pass22:25
zaitcevExcept  unittest2.loader._FailedTest.test.functional.s3api, which has "ImportError: No module named boto3"22:26
zaitcevI don't know where to get that module.22:26
*** mikecmpbll has joined #openstack-swift22:27
*** mikecmpb_ has joined #openstack-swift22:31
*** mikecmpbll has quit IRC22:32
timburkehttps://sjc.edge.kernel.org/fedora-buffet/fedora/linux/releases/30/Everything/x86_64/os/Packages/p/python3-boto3-1.9.101-1.fc30.noarch.rpm won't do?22:40
zaitcevHmm.22:53
zaitcevI forgot that the _client_ is actually on RHEL 7.5. Sorry that I misled you. It's the py3 server that's on F30.22:55
timburkeah, gotcha22:56
timburkethere's always pip...22:56
zaitcevYeah.22:56
openstackgerritTim Burke proposed openstack/swift master: py3: port test/functional/test_versioned_writes.py  https://review.opendev.org/67571023:00
openstackgerritTim Burke proposed openstack/swift master: py3: port test/functional/test_versioned_writes.py  https://review.opendev.org/67571023:01
openstackgerritTim Burke proposed openstack/swift master: py3: mostly port s3 func tests  https://review.opendev.org/67471623:10
openstackgerritTim Burke proposed openstack/swift master: py3: Finish porting s3 func tests  https://review.opendev.org/67522723:10
*** zaitcev has quit IRC23:24
*** zaitcev has joined #openstack-swift23:36
*** ChanServ sets mode: +v zaitcev23:36

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