Wednesday, 2021-03-17

openstackgerritMerged openstack/swift master: py36: Fix syslog fallback to UDP  https://review.opendev.org/c/openstack/swift/+/78093900:18
seongsoochoCan I ignore this error message in cleanup job of relinker? "Error cleaning up /srv/node/sdl/objects/261/ba1/105f1a630d8689c4354744a3a46fcba1/1579789059.24190.data: DiskFileNotExist()"00:28
*** timburke_ has joined #openstack-swift00:32
*** ChanServ sets mode: +v timburke_00:32
*** timburke has quit IRC00:34
*** timburke_ has quit IRC00:42
*** timburke has joined #openstack-swift01:03
*** ChanServ sets mode: +v timburke01:03
*** ianychoi has joined #openstack-swift01:24
*** ianychoi_ has quit IRC01:26
*** psachin has joined #openstack-swift02:27
zaitcevGuys02:57
zaitcevSuppose I swiftched to "new" expirers on object nodes.02:57
zaitcevOne of them has dequeue_from_legacy=true. It does all the work.02:58
zaitcevAnd02:59
zaitcevIt's the only expirer that recon shows.02:59
zaitcev-> http://192.168.50.2:6000/recon/expirer/object: {'object_expiration_pass': 0.011844873428344727, 'expired_last_pass': 0}03:00
zaitcev-> http://192.168.50.1:6200/recon/expirer/object: {'object_expiration_pass': 9543.668754816055, 'expired_last_pass': 32321}03:00
zaitcevThe 50.1 is the legacy, 50.2 is the new one.03:00
zaitcevIs it going to start working if I switch the new expirer on?03:00
zaitcevI mean, in the proxy.03:01
*** priteau has quit IRC03:03
zaitcevActually, why am I even asking this. Let's just set expiring_objects_container_divisor and see what happens.03:10
*** rcernin has quit IRC03:22
*** evrardjp has quit IRC03:33
*** evrardjp has joined #openstack-swift03:33
*** rcernin has joined #openstack-swift03:33
*** rcernin has quit IRC03:41
*** rcernin has joined #openstack-swift03:42
*** manuvakery1 has joined #openstack-swift04:38
*** m75abrams has joined #openstack-swift04:59
*** timburke has quit IRC05:28
openstackgerritMatthew Oliver proposed openstack/swift master: swift-manage-shard-ranges: check for internal gaps  https://review.opendev.org/c/openstack/swift/+/78101305:49
mattoliverauseongsoocho: it probably means the object was reclaimed during between the relinking and cleanup. I mean you can go check to see if there is a .ts there or in the linked path to confirm if you want.05:57
*** dsariel has joined #openstack-swift06:09
seongsoochomattoliverau:  thanks . How can I get a linked path of original object ?06:13
*** mathiasb has joined #openstack-swift07:43
*** dsariel has quit IRC08:00
*** dsariel has joined #openstack-swift08:01
*** rcernin has quit IRC08:11
*** rpittau|afk is now known as rpittau08:23
*** rcernin has joined #openstack-swift09:17
*** openstackgerrit has quit IRC09:33
*** rcernin_ has joined #openstack-swift09:48
*** rcernin has quit IRC09:55
*** dsariel has quit IRC09:55
*** mathiasb has quit IRC09:55
acolesseongsoocho: mattoliverau it's a .data file so should not be reclaimed, but there are other conditions that trigger a DiskFileNotExist (corrupt metadata maybe?)10:40
acolesthe error is reported for the new location (i.e. in the 'next_part_power' partition)10:40
acolesso (a) try an ls and swift-object-info on that reported path, see what you actually have there10:41
acoles(b) check the old location which will be /srv/node/sdl/objects/130/ba1/105f1a630d8689c4354744a3a46fcba1 , note objects in old partition N should have been linked new partitions 2N and 2N+1, so object in new partition 261 would have been in old partition 13010:44
acolesseongsoocho: please let us know what you discover, we've been fixing a few bugs in relinker, but also found that sometimes it is reporting false positives10:47
*** priteau has joined #openstack-swift11:26
*** rcernin_ has quit IRC11:32
*** takamatsu has quit IRC11:57
seongsoochoacoles:    I checked new partition 2N, and 2N+1, but there are no object ....11:59
seongsoochofor more information, I used train code of relinker for relink job and use recent code of relinker for cleanup...  Will this be a problem?12:00
acolesthat should be ok12:01
acolesis there anything in the old partition?12:01
seongsoochoyes. the object are in the old partition.12:02
seongsoochohttps://pastebin.com/hWXzzVw0. this is my terminal log12:05
seongsoochoAh...  I checked other server which is have replica, the object is deleted in other server..12:11
acolesseongsoocho: is it possible that the object was deleted since you started relinking, and longer than reclaim_age has passed? It could be that the old partition has a .data, then a DELETE wrote a .ts in the new partition, then when the relinker cleanup reached that object the .ts was reclaimed in the new partition. Is the .ts file on other replica's server  newer than the .data in the old partition?12:58
acolesI *think* timburke recommends increasing reclaim_age during relinking???12:59
seongsoochoacoles:  yes..  the object was deleted since I started relinking..  and I increased relcaim_age after the relink  (before the cleanup)13:01
*** openstackgerrit has joined #openstack-swift14:29
openstackgerritMerged openstack/swift master: Fix systemd notify unit test on macos  https://review.opendev.org/c/openstack/swift/+/78068014:29
*** psachin has quit IRC15:51
*** josephillips has joined #openstack-swift15:52
*** m75abrams has quit IRC15:54
*** rpittau is now known as rpittau|afk17:08
*** timburke has joined #openstack-swift17:15
*** ChanServ sets mode: +v timburke17:15
openstackgerritMerged openstack/swift master: relinker: use abs path index in part power replace  https://review.opendev.org/c/openstack/swift/+/78002617:15
*** dosaboy_ is now known as dosaboy17:16
*** alecuyer has quit IRC17:21
*** alecuyer has joined #openstack-swift17:25
*** gyee has joined #openstack-swift17:29
timburkezaitcev, so... any "new" expirers won't do anything: https://github.com/openstack/swift/blob/2.26.0/swift/obj/expirer.py#L309-L31317:36
timburkebecause we never quite got the momentum to review & merge https://review.opendev.org/c/openstack/swift/+/51738917:36
zaitcevYeah, I added "dequeue_from_legacy=True" to all of them and they seem to work now.17:36
zaitcev[object_expiration_pass] low: 96, high: 99, avg: 97.6, total: 292, Failed: 0.0%, no_result: 0, reported: 317:37
openstackgerritTim Burke proposed openstack/swift master: s3api: Add config option to return 429s on ratelimit  https://review.opendev.org/c/openstack/swift/+/72909318:00
openstackgerritAlistair Coles proposed openstack/swift master: swift-manage-shard-ranges: add repair and analyze commands  https://review.opendev.org/c/openstack/swift/+/76562418:30
openstackgerritAlistair Coles proposed openstack/swift master: manage-shard-ranges: add --dry-run option for compact and repair  https://review.opendev.org/c/openstack/swift/+/78114418:30
openstackgerritAlistair Coles proposed openstack/swift master: Fix shrinking making acceptors prematurely active  https://review.opendev.org/c/openstack/swift/+/77400218:30
openstackgerritAlistair Coles proposed openstack/swift master: sharder: stall cleaving at shard range gaps  https://review.opendev.org/c/openstack/swift/+/77758518:30
openstackgerritTim Burke proposed openstack/python-swiftclient master: Allow tempurl times to have units  https://review.opendev.org/c/openstack/python-swiftclient/+/75850018:40
*** gyee has quit IRC18:46
*** josephillips has quit IRC18:49
*** josephillips has joined #openstack-swift18:52
*** gyee has joined #openstack-swift18:53
*** rcernin_ has joined #openstack-swift19:26
*** rcernin_ has quit IRC19:32
openstackgerritMerged openstack/swift master: Make more use of get_partition_for_hash in unit tests  https://review.opendev.org/c/openstack/swift/+/78109319:35
*** gmann is now known as gmann_afk19:41
openstackgerritMerged openstack/swift master: Add test coverage for audit_location_generator yield_hashes arg  https://review.opendev.org/c/openstack/swift/+/78064819:50
*** manuvakery1 has quit IRC19:56
*** rcernin_ has joined #openstack-swift19:57
*** rcernin_ has quit IRC20:02
*** rcernin_ has joined #openstack-swift20:32
*** rcernin_ has quit IRC20:37
*** rcernin_ has joined #openstack-swift20:50
timburkealmost meeting time!20:54
mattoliveraumorning20:54
kota_good morning20:59
acolesgood morning kota_ mattoliverau21:00
kota_acoles, mattoliverau: o/21:01
*** rcernin_ has quit IRC21:04
*** rcernin_ has joined #openstack-swift21:09
*** rcernin_ has quit IRC21:15
*** rcernin_ has joined #openstack-swift21:41
openstackgerritTim Burke proposed openstack/swift master: Add --headless option for cors functional test runner  https://review.opendev.org/c/openstack/swift/+/78108422:09
openstackgerritTim Burke proposed openstack/swift master: Use headless option for cors functional test runner  https://review.opendev.org/c/openstack/swift/+/78108422:12
*** gmann_afk is now known as gmann22:39
openstackgerritTim Burke proposed openstack/swift master: Use headless option for cors functional test runner  https://review.opendev.org/c/openstack/swift/+/78108422:57
*** timburke_ has joined #openstack-swift23:04
*** ChanServ sets mode: +v timburke_23:04
*** timburke has quit IRC23:06
*** timburke_ has quit IRC23:31
*** timburke_ has joined #openstack-swift23:31
*** ChanServ sets mode: +v timburke_23:31
openstackgerritMatthew Oliver proposed openstack/swift master: relinker: Add /recon/relink endpoint and drop progress stats  https://review.opendev.org/c/openstack/swift/+/77965523:49
openstackgerritMatthew Oliver proposed openstack/swift master: relinker: Add policy to relinker progress output  https://review.opendev.org/c/openstack/swift/+/78044023:49

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