| *** zul_ has joined #openstack-swift | 00:07 | |
| *** zul_ has quit IRC | 00:46 | |
| *** zhurong has joined #openstack-swift | 00:46 | |
| *** hoonetorg has quit IRC | 01:02 | |
| *** m_kazuhiro has joined #openstack-swift | 01:17 | |
| kota_ | good morning | 01:17 |
|---|---|---|
| m_kazuhiro | kota_: good morning | 01:17 |
| kota_ | m_kazuhiro: hi | 01:18 |
| *** m_kazuhiro has quit IRC | 01:19 | |
| *** m_kazuhiro has joined #openstack-swift | 01:22 | |
| mattoliverau | kota_, m_kazuhiro morning | 01:30 |
| kota_ | mattoliverau: o/ | 01:31 |
| m_kazuhiro | mattoliverau: morning! | 01:32 |
| openstackgerrit | Emilien Macchi proposed openstack/swift master: Add sem-ver flag so pbr generates correct version https://review.openstack.org/453424 | 01:34 |
| *** vint_bra has joined #openstack-swift | 01:35 | |
| *** zhurong has quit IRC | 01:52 | |
| *** yulijie has quit IRC | 01:56 | |
| *** zhurong has joined #openstack-swift | 01:57 | |
| *** chosafine has joined #openstack-swift | 02:04 | |
| *** jamielennox is now known as jamielennox|away | 02:07 | |
| *** tanee_away is now known as tanee | 02:11 | |
| *** jamielennox|away is now known as jamielennox | 02:21 | |
| *** vint_bra has quit IRC | 02:22 | |
| *** winggundamth has joined #openstack-swift | 02:33 | |
| *** zhurong has quit IRC | 02:38 | |
| *** SkyRocknRoll has joined #openstack-swift | 03:00 | |
| *** zhurong has joined #openstack-swift | 03:13 | |
| *** links has joined #openstack-swift | 03:16 | |
| mahatic | good morning | 03:22 |
| *** klrmn has quit IRC | 03:25 | |
| m_kazuhiro | mahatic: good morning | 03:36 |
| *** gkadam has joined #openstack-swift | 03:45 | |
| *** ljjjustin has joined #openstack-swift | 03:47 | |
| mahatic | m_kazuhiro: o/ | 03:53 |
| *** chosafine has quit IRC | 03:54 | |
| *** rcernin has joined #openstack-swift | 04:06 | |
| *** stradling has joined #openstack-swift | 04:09 | |
| *** zhurong has quit IRC | 04:12 | |
| *** rcernin has quit IRC | 04:13 | |
| *** m_kazuhiro has quit IRC | 04:21 | |
| *** zaitcev has quit IRC | 04:35 | |
| *** zaitcev has joined #openstack-swift | 04:35 | |
| *** ChanServ sets mode: +v zaitcev | 04:35 | |
| *** zhurong has joined #openstack-swift | 04:35 | |
| *** psachin has joined #openstack-swift | 04:40 | |
| *** chosafine has joined #openstack-swift | 04:43 | |
| mattoliverau | mahatic: morning | 04:43 |
| *** SkyRocknRoll has quit IRC | 04:47 | |
| *** chlong has quit IRC | 04:49 | |
| mahatic | mattoliverau: o/ | 04:49 |
| *** chosafine has quit IRC | 04:55 | |
| *** m_kazuhiro has joined #openstack-swift | 04:58 | |
| *** jaosorior_away is now known as jaosorior | 05:11 | |
| *** chosafine has joined #openstack-swift | 05:13 | |
| *** chosafine has quit IRC | 05:13 | |
| *** tonyb_ is now known as tonyb | 05:25 | |
| *** hoonetorg has joined #openstack-swift | 05:33 | |
| openstackgerrit | junboli proposed openstack/swift master: Update doc infos to ocata https://review.openstack.org/453467 | 05:33 |
| *** bkopilov has quit IRC | 05:35 | |
| *** bkopilov has joined #openstack-swift | 05:35 | |
| *** stradling has quit IRC | 05:36 | |
| *** ChubYann has quit IRC | 05:39 | |
| *** cshastri has joined #openstack-swift | 05:57 | |
| *** pcaruana has joined #openstack-swift | 06:08 | |
| *** hoonetorg has quit IRC | 06:29 | |
| *** rcernin has joined #openstack-swift | 06:31 | |
| *** hoonetorg has joined #openstack-swift | 06:42 | |
| *** sputnik13 has quit IRC | 06:44 | |
| *** silor has joined #openstack-swift | 06:45 | |
| *** sputnik13 has joined #openstack-swift | 06:45 | |
| *** tesseract has joined #openstack-swift | 06:52 | |
| *** hseipp has joined #openstack-swift | 06:53 | |
| *** silor1 has joined #openstack-swift | 06:58 | |
| *** dja has quit IRC | 07:00 | |
| *** silor has quit IRC | 07:00 | |
| *** silor1 is now known as silor | 07:00 | |
| acoles | good morning | 07:17 |
| acoles | kota_: clayg: thanks for adding your thought to the etherpad.. good stuff. | 07:26 |
| kota_ | good morning acoles | 07:27 |
| kota_ | acoles: and just putting small acks to the patch 441921 and moving the patch 449310 | 07:28 |
| patchbot | https://review.openstack.org/#/c/441921/ - swift - Add Composite Ring Functionality | 07:28 |
| patchbot | https://review.openstack.org/#/c/449310/ - swift - Add id to RingBuilder to differentiate rings in co... | 07:28 |
| kota_ | acoles: is the patch 449310 ready for review, right? | 07:28 |
| patchbot | https://review.openstack.org/#/c/449310/ - swift - Add id to RingBuilder to differentiate rings in co... | 07:28 |
| acoles | kota_: yes and no :) | 07:28 |
| *** m_kazuhiro_ has joined #openstack-swift | 07:28 | |
| acoles | kota_: yes, it's in good shape wrt test etc IMHO | 07:29 |
| kota_ | acoles: ok | 07:29 |
| acoles | kota_: no, I think clayg is advocating on etherpad that the id is only set on save() - but I am reading etherpad quickly, need to digest it more | 07:30 |
| kota_ | ah, you ware writing the reason for yes/no | 07:31 |
| *** m_kazuhiro has quit IRC | 07:31 | |
| acoles | kota_: I think in my comment on patch 449310 I described that alternative (set id on save, remove the assign_id method) | 07:32 |
| patchbot | https://review.openstack.org/#/c/449310/ - swift - Add id to RingBuilder to differentiate rings in co... | 07:32 |
| acoles | kota_: yep. yes because... no because ... :) | 07:32 |
| acoles | kota_: an interesting thought from etherpad comments - maybe even a newly created builder does not have an id until it has been saved? at first I thought that too demanding, but actually maybe it's a good idea to always save a *new* builder before using it with composite-builder? If you have any more thoughts on that then please comment on patch. | 07:38 |
| kota_ | acoles: i think that from Clay's idea but yes, so the point we can create the new uuid is only at the save method for the builder. and it looks the difference with your patch (I'm at the builder.py on youre patch) | 07:41 |
| *** oshritf has joined #openstack-swift | 07:42 | |
| *** gkadam is now known as gkadam-afk | 07:43 | |
| *** amoralej|off is now known as amoralej | 07:43 | |
| acoles | kota_: I think assign_id would be removed; that code goes to the save method; and the __init__ changes go away | 07:45 |
| kota_ | acoles: yes, I'm writing comments about that | 07:46 |
| acoles | kota_: thanks! | 07:46 |
| *** SkyRocknRoll has joined #openstack-swift | 07:48 | |
| *** geaaru has joined #openstack-swift | 07:50 | |
| *** silor has quit IRC | 07:56 | |
| *** zacksh has quit IRC | 07:58 | |
| *** zacksh has joined #openstack-swift | 08:00 | |
| *** hseipp has quit IRC | 08:01 | |
| *** cbartz has joined #openstack-swift | 08:27 | |
| *** jaosorior is now known as jaosorior_lunch | 08:31 | |
| *** kei_yama has quit IRC | 08:38 | |
| kota_ | acoles: pushed my comments on the patch 449310 | 08:41 |
| patchbot | https://review.openstack.org/#/c/449310/ - swift - Add id to RingBuilder to differentiate rings in co... | 08:41 |
| *** oshritf has quit IRC | 08:50 | |
| dewanee_ | does anyone know what this can be caused by: | 08:51 |
| dewanee_ | swift-object-reconstructor: Unexpected entity in data dir: u'/srv/node/mpath0p1/objects- | 08:51 |
| dewanee_ | 1/auditor_status_ZBF.json' | 08:51 |
| *** oshritf has joined #openstack-swift | 08:51 | |
| *** oshritf has quit IRC | 08:54 | |
| dewanee_ | everything seems to be fine | 08:59 |
| dewanee_ | otherwise | 08:59 |
| *** oshritf has joined #openstack-swift | 09:00 | |
| charz | dewanee_: It just a warning message form reconstructor because it's not a partition folder, you can just ignore that. | 09:05 |
| *** psachin has quit IRC | 09:07 | |
| *** psachin has joined #openstack-swift | 09:11 | |
| dewanee_ | good to know, thanks charz | 09:27 |
| charz | dewanee_: np :) | 09:28 |
| dewanee_ | I have another issue with swift-object-replicator reporting ~50k failures | 09:30 |
| dewanee_ | as far as I understand failures are expected but I can't quantify how many should trigger a "something not working" warning | 09:31 |
| openstackgerrit | Kazuhiro MIYAHARA proposed openstack/swift master: Refactor diskfile check implementation of automated tiering https://review.openstack.org/453548 | 09:50 |
| *** m_kazuhiro_ has quit IRC | 09:56 | |
| *** mvk has quit IRC | 10:00 | |
| *** ljjjustin has quit IRC | 10:12 | |
| *** pcaruana|afk| has joined #openstack-swift | 10:21 | |
| *** pcaruana|afk| has quit IRC | 10:23 | |
| *** pcaruana has quit IRC | 10:24 | |
| *** pcaruana has joined #openstack-swift | 10:25 | |
| *** zhurong has quit IRC | 10:30 | |
| *** jaosorior_lunch is now known as jaosorior | 10:31 | |
| *** mvk has joined #openstack-swift | 10:32 | |
| *** hseipp has joined #openstack-swift | 10:49 | |
| *** vint_bra has joined #openstack-swift | 11:12 | |
| *** dja has joined #openstack-swift | 11:19 | |
| *** dja has quit IRC | 11:24 | |
| *** winggundamth has quit IRC | 11:24 | |
| *** dja has joined #openstack-swift | 11:27 | |
| *** stradling has joined #openstack-swift | 11:30 | |
| *** vint_bra has quit IRC | 12:05 | |
| *** klamath has joined #openstack-swift | 12:07 | |
| *** klamath has quit IRC | 12:07 | |
| *** klamath has joined #openstack-swift | 12:07 | |
| *** SkyRocknRoll has quit IRC | 12:18 | |
| *** larivee has joined #openstack-swift | 12:30 | |
| larivee | good day all, Is there an easier way than to iterate over all object of a tenant to purge that tenant from Swift ? ( from an admin / operator access view ) | 12:31 |
| *** silor has joined #openstack-swift | 12:38 | |
| acoles | larivee: the account reaper can remove deleted account objects https://docs.openstack.org/developer/swift/overview_reaper.html | 12:39 |
| larivee | thx acoles | 12:40 |
| *** silor has quit IRC | 12:47 | |
| *** gkadam-afk has quit IRC | 12:49 | |
| *** tongli has joined #openstack-swift | 12:49 | |
| *** psachin has quit IRC | 12:52 | |
| -openstackstatus- NOTICE: The Gerrit service on http://review.openstack.org is being restarted to address hung remote replication tasks, and should return to an operable state momentarily | 12:52 | |
| *** psachin has joined #openstack-swift | 13:05 | |
| *** links has quit IRC | 13:08 | |
| *** catintheroof has joined #openstack-swift | 13:10 | |
| *** catintheroof has quit IRC | 13:10 | |
| *** catintheroof has joined #openstack-swift | 13:11 | |
| *** amoralej is now known as amoralej|lunch | 13:17 | |
| *** chlong has joined #openstack-swift | 13:17 | |
| *** cshastri_ has joined #openstack-swift | 13:23 | |
| *** cshastri has quit IRC | 13:26 | |
| *** psachin has quit IRC | 13:26 | |
| *** cshastri_ has quit IRC | 13:27 | |
| *** psachin_ has joined #openstack-swift | 13:32 | |
| *** psachin_ has quit IRC | 13:37 | |
| EmilienM | hello here, can you review this patch please? https://review.openstack.org/#/c/453424/ - it would help TripleO CI to test upgrades from Ocata to Pike. Thanks a lot! | 13:40 |
| *** cshastri_ has joined #openstack-swift | 13:40 | |
| tdasilva | EmilienM: can you explain that patch, I don't understand it all | 13:45 |
| tdasilva | EmilienM: I see no files being changed, what am i missing? | 13:45 |
| EmilienM | tdasilva: in the commit message I tried to explain it :D | 13:45 |
| EmilienM | tdasilva: so : | 13:45 |
| *** _JZ_ has joined #openstack-swift | 13:45 | |
| acoles | tdasilva: EmilienM I am just reading the linked pbr doc and infer that there is some significance to the Sem-Ver tag?? | 13:45 |
| EmilienM | the tooling that build packages read commit message | 13:46 |
| *** cshastri__ has joined #openstack-swift | 13:46 | |
| EmilienM | and will bump pbr to the next Y number | 13:46 |
| EmilienM | starting from this commit | 13:46 |
| EmilienM | because you didn't push for a tag yet in Pike | 13:46 |
| tdasilva | "We then walk git history back to the last release. Within each commit we look for a Sem-Ver: pseudo header, and if found parse it looking for keywords." | 13:46 |
| EmilienM | yup | 13:47 |
| EmilienM | we did it in nova, ironic, and all other projects that tripleo use | 13:47 |
| EmilienM | it has zero impact on your project I think, it just help packagers to test upgrades | 13:47 |
| acoles | and Sem-Ver: feature causes a minor version bump | 13:48 |
| *** cshastri_ has quit IRC | 13:49 | |
| *** psachin_ has joined #openstack-swift | 13:49 | |
| EmilienM | acoles: Y | 13:49 |
| EmilienM | X.Y.Z | 13:49 |
| EmilienM | I thought Swift would a Y bump this time | 13:49 |
| EmilienM | but if you prefer X or Z I don't care | 13:49 |
| EmilienM | just update the patch, so we can start upgrading swift in our CI | 13:50 |
| tdasilva | acoles: right, so in X.Y.Z feature will bump Y, but this is all just pseudo versioning. distro packages will start showing up as 2.14.0 correct? | 13:50 |
| EmilienM | yes exact | 13:50 |
| *** cshastri__ has quit IRC | 13:50 | |
| tdasilva | once we release swift again, then distro packages will just continue with 2.14.x | 13:51 |
| tdasilva | weird, but i think I got it | 13:51 |
| EmilienM | we could get rid of this if projects would produce a tag right after openstack releases (right after ocata GA for example) | 13:51 |
| EmilienM | but nobody or a few do it | 13:51 |
| EmilienM | which is a pain for us, because we can't test upgrades from ocata to pike until m1 | 13:51 |
| tdasilva | EmilienM: like a release tag? | 13:51 |
| EmilienM | yes | 13:51 |
| EmilienM | life would be easier with a tag | 13:52 |
| EmilienM | but teams want to wait for m1 to push for the first pike tag | 13:52 |
| EmilienM | and when I say pike, I say for all releases, newton, ocata, etc | 13:52 |
| EmilienM | it makes our life harder because we can't test package upgrades | 13:52 |
| tdasilva | EmilienM: yeah, in swift case is not even about waiting for m1. I think our release policy is more like. "Do we have anything important to release right now, then let's do it" | 13:53 |
| tdasilva | whenenever in the cycle that happens to be | 13:53 |
| EmilienM | tdasilva: right | 13:55 |
| EmilienM | anyway, please take this one in consideration, it would really help. Thanks | 13:55 |
| *** vint_bra has joined #openstack-swift | 13:56 | |
| *** amoralej|lunch is now known as amoralej | 13:57 | |
| *** cshastri__ has joined #openstack-swift | 14:02 | |
| *** zul has quit IRC | 14:05 | |
| *** jistr is now known as jistr|mtg | 14:07 | |
| *** cshastri__ is now known as cshastri | 14:12 | |
| *** zul has joined #openstack-swift | 14:22 | |
| *** caiobrentano has joined #openstack-swift | 14:46 | |
| *** zul_ has joined #openstack-swift | 14:46 | |
| *** zul_ has quit IRC | 14:59 | |
| jlvillal | notmyname, Did the patchbot leave #openstack-ironic? | 15:02 |
| * jlvillal realizes that is a rhetorical question :) | 15:02 | |
| *** oshritf has quit IRC | 15:03 | |
| *** chlong has quit IRC | 15:03 | |
| *** chlong has joined #openstack-swift | 15:04 | |
| *** cbartz has left #openstack-swift | 15:06 | |
| *** cshastri has quit IRC | 15:09 | |
| *** psachin_ has quit IRC | 15:14 | |
| *** tongli has quit IRC | 15:17 | |
| *** chlong has quit IRC | 15:21 | |
| *** m_kazuhiro has joined #openstack-swift | 15:25 | |
| *** vint_bra has quit IRC | 15:30 | |
| *** caiobrentano_ has joined #openstack-swift | 15:31 | |
| *** vint_bra has joined #openstack-swift | 15:32 | |
| *** m_kazuhiro has quit IRC | 15:33 | |
| *** caiobrentano has quit IRC | 15:34 | |
| *** chlong has joined #openstack-swift | 15:34 | |
| *** jistr|mtg is now known as jistr | 15:40 | |
| notmyname | good morning | 16:03 |
| notmyname | jlvillal: yes. sorry about that. I'll ask patchbot to rejoin | 16:04 |
| notmyname | jlvillal: patchbot is new and improved. sortof. less crazy. also very slightly less helpful. | 16:04 |
| jlvillal | notmyname, Thanks! | 16:04 |
| notmyname | EmilienM: if you just need a different version tag, why choose Y before the project has released something that bumps Y? bumping Z seems to more closely match what you're getting at? | 16:07 |
| notmyname | if we bump Y now to get to an effective 2.14.0 does that mean our next tag needs to be 2.14.1? | 16:08 |
| notmyname | I always liked the idea of versions based on VCS tags (instead of strings in text files), but now you're saying that the current version is some combination of VCS tag plus a special magic string in a commit message? | 16:09 |
| notmyname | but wait. the pbr docs say "A missing Sem-Ver line is equivalent to Sem-Ver: bugfix. " Doesn't that mean we've effectively got 2.13.1 now anyway? (ie incremented Z) | 16:12 |
| clarkb | notmyname: yes incremented Z if you don't specify | 16:17 |
| notmyname | ok, so if that's the case, why the need for the post-ocata patch? | 16:18 |
| notmyname | I don't understand why this patch is needed now, or why it hasn't been needed after past cycle releases | 16:25 |
| notmyname | (and the concept of a commit message in the VCS history having semantic meaning on the produced software as opposed to just describing what/how a particular commit works is scary to me | 16:26 |
| notmyname | acoles: tdasilva: you've looked at this patch, so please tell me where I'm overreacting :-) | 16:29 |
| *** JimCheung has joined #openstack-swift | 16:34 | |
| acoles | notmyname: tdasilva made a comment earlier that this was 'pseudo versioning' and next release would still be 2.14.0...and at that point I got confused :/ | 16:37 |
| *** chsc has joined #openstack-swift | 16:42 | |
| EmilienM | notmyname: feel free to change it | 16:43 |
| EmilienM | notmyname: I really don't care about the X Y Z | 16:43 |
| EmilienM | I just need a bump :D | 16:43 |
| *** tesseract has quit IRC | 16:43 | |
| notmyname | EmilienM: how is it not already bumped? | 16:43 |
| notmyname | how or why this particular patch is fixing something is not at all clear to me | 16:44 |
| EmilienM | notmyname: when is your lastest release ? | 16:44 |
| notmyname | for ocata | 16:44 |
| notmyname | we've landed commits since then, so why isn't pbr calling it 2.13.1 already? | 16:45 |
| notmyname | ie something different than the tag that was released most recently | 16:46 |
| EmilienM | notmyname: because no tag has been pushed, it's a packaging thing | 16:47 |
| EmilienM | notmyname: distros need a new semver | 16:47 |
| EmilienM | so we either need a tag now, or my patch | 16:47 |
| *** ChubYann has joined #openstack-swift | 16:48 | |
| notmyname | packagers are using pbr to get a version number? | 16:50 |
| notmyname | which looks at the tag (most recent is 2.13.0) and parses commit messages to see if a bigger one is needed | 16:50 |
| notmyname | right? | 16:50 |
| notmyname | and pbr is currently reporting swift as "swift2.13.1.dev61 pre-release bff9f89" | 16:52 |
| *** hseipp has quit IRC | 16:53 | |
| *** jamielennox has quit IRC | 17:04 | |
| *** jamielennox has joined #openstack-swift | 17:07 | |
| *** jaosorior is now known as jaosorior_away | 17:08 | |
| *** klrmn has joined #openstack-swift | 17:08 | |
| *** neonpastor has joined #openstack-swift | 17:15 | |
| *** mvk has quit IRC | 17:22 | |
| tdasilva | notmyname: maybe i misunderstood, but I don't think that patch forces us to tag our next release as 2.14.1 | 17:28 |
| notmyname | yeah, that's what I was trying to figure out. does it? but if not, why is it needed anyway? just wanting to understand what's needed, what this fixes, and what the implications are | 17:29 |
| tdasilva | notmyname: i'm not 100% about this, so take it with a grain of salt: | 17:39 |
| tdasilva | ocata repo for rdo: https://trunk.rdoproject.org/centos7-ocata/b7/c2/b7c2a9196009ef78560acfb3826b9106f95d7fdf_50de241a/ | 17:39 |
| tdasilva | if you search for swift, you will notice 2.13.1 | 17:40 |
| tdasilva | and this is rdo master - which i assume will become pike | 17:40 |
| tdasilva | https://trunk.rdoproject.org/centos7-master-head/bc/9a/bc9a7075d3accf3160043e8366745a1dd895b08d_0025310a/ | 17:40 |
| tdasilva | also 2.13.1 | 17:40 |
| notmyname | ok | 17:40 |
| tdasilva | so I'm assuming that | 17:41 |
| tdasilva | I'm assuming that EmilienM's patch will push the master version to 2.14.0 | 17:41 |
| tdasilva | and allow them to test upgrades | 17:41 |
| tdasilva | EmilienM: ^^^ is that correct? | 17:41 |
| notmyname | upgrades can't be tested from 2.13.0 to 2.13.1? | 17:42 |
| tdasilva | but ocata is already at 2.13.1 | 17:42 |
| tdasilva | so I assume that's the issue | 17:42 |
| notmyname | oh, interesting. maybe that's the part I didn't catch earlier | 17:43 |
| notmyname | oh! because we've landed backports for ocata | 17:43 |
| notmyname | which bumps the Z number automatically | 17:43 |
| notmyname | but we haven't pushed anything that triggers pbr to bump the Y number, so both master and stable/ocata are listed as 2.13.1 | 17:44 |
| notmyname | right? | 17:44 |
| tdasilva | that's how I understood | 17:45 |
| *** SkyRocknRoll has joined #openstack-swift | 17:46 | |
| *** tonanhngo has joined #openstack-swift | 17:47 | |
| acoles | tdasilva: thanks again for your work on https://review.openstack.org/#/c/451524/ | 17:47 |
| tdasilva | acoles: i'm afraid the hard part now begins with trying to figure out step 2 | 17:48 |
| patchbot | No data found for patch 2 | 17:48 |
| tdasilva | lol | 17:48 |
| acoles | patchbot has finger on the pulse :) | 17:48 |
| notmyname | that's weird | 17:49 |
| *** tonanhngo_ has joined #openstack-swift | 17:50 | |
| tdasilva | step 2 | 17:51 |
| patchbot | No data found for patch 2 | 17:51 |
| tdasilva | p 2 | 17:51 |
| patchbot | No data found for patch 2 | 17:51 |
| *** tonanhngo has quit IRC | 17:51 | |
| *** SkyRocknRoll has quit IRC | 17:52 | |
| *** SkyRocknRoll has joined #openstack-swift | 17:52 | |
| *** tonanhngo_ has quit IRC | 17:54 | |
| *** tonanhngo has joined #openstack-swift | 17:55 | |
| *** silor has joined #openstack-swift | 17:58 | |
| *** mvk has joined #openstack-swift | 18:00 | |
| *** silor has quit IRC | 18:10 | |
| *** silor has joined #openstack-swift | 18:11 | |
| notmyname | EmilienM: is what tdasilva and I were talking about close to the truth about the version numbers? | 18:11 |
| EmilienM | tdasilva: it's correct | 18:12 |
| *** stradling has quit IRC | 18:13 | |
| EmilienM | notmyname: true | 18:16 |
| notmyname | thanks | 18:16 |
| *** silor has quit IRC | 18:16 | |
| notmyname | that makes sense to me | 18:16 |
| notmyname | pbr parsing commit message is a separate issue (and one I still don't like), but I understand the need for the patch now | 18:17 |
| notmyname | EmilienM: and to be explicit, does your patch mean that our next tag has to skip 2.14.0? | 18:17 |
| patchbot | No data found for patch 2 | 18:17 |
| notmyname | patchbot: I will cut you | 18:17 |
| *** silor has joined #openstack-swift | 18:18 | |
| EmilienM | notmyname: I think I never said I liked it | 18:18 |
| EmilienM | I'm an operator who wants to upgrade OpenStack now | 18:18 |
| notmyname | no, that's a separate conversation from the patch and unblocking you | 18:19 |
| EmilienM | notmyname: let me check that information | 18:19 |
| EmilienM | notmyname: yes right | 18:19 |
| notmyname | ok, thanks | 18:19 |
| *** geaaru has quit IRC | 18:23 | |
| tdasilva | is right we have to skip 2.14.0? | 18:24 |
| patchbot | No data found for patch 2 | 18:24 |
| tdasilva | s/is/yes | 18:24 |
| notmyname | tdasilva: he's checking :-) | 18:24 |
| tdasilva | oh | 18:24 |
| *** silor has quit IRC | 18:27 | |
| *** silor has joined #openstack-swift | 18:27 | |
| EmilienM | notmyname: I'm actually not sure, tbh | 18:36 |
| EmilienM | on my knowledge no but i'm checking history now | 18:36 |
| *** oshritf has joined #openstack-swift | 18:37 | |
| openstackgerrit | Alistair Coles proposed openstack/swift master: WIP Add CompositeRingBuilder class https://review.openstack.org/453827 | 18:38 |
| *** stradling has joined #openstack-swift | 18:38 | |
| EmilienM | ok so https://review.openstack.org/#/c/427643/ merged and ironic released 7.0.0 after | 18:39 |
| EmilienM | I don't think we need to skip the tag | 18:39 |
| EmilienM | jroll: can you confirm maybe? | 18:39 |
| *** amoralej is now known as amoralej|off | 18:41 | |
| jroll | ? | 18:41 |
| jroll | yeah, afaik you don't need to skip it | 18:41 |
| jroll | I couldn't find the docs yesterday, but found the spec | 18:42 |
| jroll | and the spec said pbr will version it at e.g. 6.1.9999.devXXX | 18:42 |
| jroll | but honestly I'm only 50% sure on this | 18:42 |
| jroll | EmilienM: ^^ | 18:42 |
| acoles | clayg: is this the kind of thing you were thinking re etherpad comments: patch 453827 | 18:43 |
| patchbot | https://review.openstack.org/#/c/453827/ - swift - WIP Add CompositeRingBuilder class | 18:43 |
| acoles | kota_: fyi ^^ that's just to test our understanding | 18:43 |
| notmyname | jroll: how can we get more certain? I'd like to understand the impact | 18:43 |
| * notmyname goes to pick up lunch | 18:43 | |
| jroll | notmyname: the most certain way would be to ask lifeless, read the docs, or read the code. or merge it locally and see what happens. | 18:44 |
| clayg | notmyname: FWIW merging a commit message will *never* break my packages | 18:45 |
| clayg | I mean it might - but all that would really happen is I would cuss then fix it | 18:46 |
| *** hoonetorg has quit IRC | 18:51 | |
| *** SkyRocknRoll has quit IRC | 18:53 | |
| *** silor has quit IRC | 18:53 | |
| *** oshritf has quit IRC | 18:59 | |
| *** oshritf has joined #openstack-swift | 19:03 | |
| *** stradling has quit IRC | 19:10 | |
| *** oshritf has quit IRC | 19:10 | |
| *** oshritf has joined #openstack-swift | 19:12 | |
| *** oshritf has quit IRC | 19:18 | |
| *** oshritf has joined #openstack-swift | 19:21 | |
| tdasilva | just caught up with the conversation on #openstack-dev, seems like the most obvious fix is for rdo package maintainers to use PBR_VERSION variable ??? | 19:39 |
| notmyname | yeah, I think | 19:39 |
| *** stradling has joined #openstack-swift | 19:41 | |
| *** oshritf has quit IRC | 19:48 | |
| caiobrentano_ | Hi all! I'm getting some exceptions from object-expirer, resulting in "Unexpected response: 412 Precondition Failed:"... what could I check? I did some tests with x-delete-at, and everything worked as expected | 19:51 |
| caiobrentano_ | Apparently the object still exists after this 412 error... but with no expirer header | 19:51 |
| notmyname | caiobrentano_: any chance you can pastebin the request headers and response headers? (scrub the token at least) | 19:53 |
| *** oshritf has joined #openstack-swift | 19:53 | |
| caiobrentano_ | notmyname I can paste bin the log... because I'm not doing any request | 19:53 |
| *** silor has joined #openstack-swift | 19:54 | |
| caiobrentano_ | https://pastebin.com/pyhg2EhD | 19:55 |
| notmyname | oh right. expirer. you wouldn't have the request | 19:56 |
| *** silor1 has joined #openstack-swift | 19:57 | |
| openstackgerrit | Thiago da Silva proposed openstack/pyeclib master: DO NOT MERGE - testing new gate job https://review.openstack.org/453847 | 19:58 |
| *** silor has quit IRC | 19:59 | |
| *** silor1 is now known as silor | 19:59 | |
| clayg | caiobrentano_: I think that can happen if you upload at t0 with x-delete-at t2, then upload again at t1 with no x-delete-at | 19:59 |
| notmyname | caiobrentano_: is that all you have for that transaction id? | 20:00 |
| notmyname | as normal, i think clayg is way ahead of me. yeah, looks like a mismatched timestamp | 20:00 |
| clayg | caiobrentano_: the x-delete-at t2 is suposed to be reaped, but if there a race it might still do the delete and find the object not having the matching delete-at metadata so the object-server refused to replace the no-supposed-to-be-expired .data with a .ts | 20:00 |
| *** pcaruana has quit IRC | 20:01 | |
| clayg | which... the expirer *should* be fine with - so the bug to file may just be "not helpful noise in logs in expected failure mode" | 20:01 |
| clayg | notmyname: I'm trying to clease my pallet after freating about automatic package versioning | 20:02 |
| caiobrentano_ | clayg right now I don't have further infos... these are log backups with 7-day life span! so, it was created 7 days ago and never modified | 20:02 |
| notmyname | you should be able to prove out clayg's theory by looking in the logs for that particular object | 20:02 |
| notmyname | clayg: scotch may help ;-) | 20:02 |
| notmyname | caiobrentano_: eg see if there are multiple PUTs or some DELETE or something | 20:02 |
| *** oshritf has quit IRC | 20:06 | |
| caiobrentano_ | notmyname I'll have to talk with the responsible for this objects. This "7-day backup flow" happens for thousands of files... but just this error apparently happens randomly for a small portion | 20:07 |
| clayg | caiobrentano_: and at the end of the day is this object *supposed* to be expired - it should have x-delete-at metadata if so? | 20:09 |
| caiobrentano_ | clagy no! it still exists, but with no x-delete-at metadata | 20:10 |
| *** jamielennox is now known as jamielennox|away | 20:12 | |
| clayg | so the end state is good afayk and the only "problem" is the unexplicable traceback? | 20:13 |
| caiobrentano_ | clayg I just saw this header: X-Object-Meta-Mtime in the object that failed at expirer | 20:13 |
| clayg | afaik that's just a swiftclient thing - no real semantic meaning here | 20:14 |
| caiobrentano_ | clagy No, the end state is the object not deleted and without expirer headers | 20:22 |
| clayg | ... and you want it to be *expired* | 20:23 |
| caiobrentano_ | yes... because I can still GET the object | 20:24 |
| clayg | I can't really imagine a way where an object that doesn't have x-delete-at metadata could possibly be expired; or a process by which an object *with* x-delete-at metadata could lose it - unless you re-uplod the object w/o the x-delete-at set - in which case... that's expected behavior? | 20:24 |
| clayg | maybe something is wrong with some version of post that causes an object to loose it's x-delete-at metadata | 20:25 |
| clayg | well... acctually i'm not sure if POST *should* clear x-delete-at - maybe? | 20:25 |
| clayg | caiobrentano_: do you know what version of swift this is, and if it's configured with post_as_copy in the proxy? | 20:26 |
| clayg | caiobrentano_: if you dig out the hashdir on disk from the ring - can you check if there's a .meta next to the .data file? | 20:26 |
| tdasilva | clayg: didn't someone point out the other day that copying an object that is set to expire has issues with it....something like that | 20:27 |
| tdasilva | nevermind, it was object expiration + versioning | 20:28 |
| caiobrentano_ | clayg version: "2.10.1"... no post_as_copy conf on proxy | 20:29 |
| caiobrentano_ | clayg there's just the .data file | 20:29 |
| *** patchbot has quit IRC | 20:30 | |
| *** patchbot has joined #openstack-swift | 20:30 | |
| notmyname | https://review.openstack.org/453847 https://review.openstack.org/453848 patch 12345 | 20:31 |
| patchbot | patch 453847 - pyeclib - DO NOT MERGE - testing new gate job | 20:31 |
| patchbot | patch 453848 - loci - Update userns info (MERGED) | 20:31 |
| notmyname | skip 234 | 20:32 |
| *** silor has quit IRC | 20:34 | |
| clayg | so 2.10.1 did have 46d61a4dcd9a5d9157625c06d6fe7d916e80c3d2 (SSC mw) | 20:35 |
| clayg | but post_as_copy is probably still default true | 20:35 |
| clayg | caiobrentano_: you should be able to confirm from the timestamp of the .data when the object was created - you may be able to trace that back to the request | 20:36 |
| clayg | it was possibly either a PUT w/o an x-delete-at metadata - or a POST which got translated to a COPY (which on master with post_as_copy=true seems to drop the x-delete-at metadata) | 20:38 |
| *** xinli has joined #openstack-swift | 20:38 | |
| xinli | +acoles: one question in 'All-in-one", in Common Post-Device SetUp, should I define "your-user-name" or I just use my login user name "ubuntu" ? | 20:40 |
| *** patchbot has quit IRC | 20:40 | |
| *** patchbot has joined #openstack-swift | 20:41 | |
| patchbot | hello world! | 20:44 |
| * patchbot is alive | 20:45 | |
| *** rcernin has quit IRC | 20:45 | |
| *** torgomatic has joined #openstack-swift | 20:46 | |
| *** ChanServ sets mode: +v torgomatic | 20:46 | |
| *** stradling has quit IRC | 20:47 | |
| clayg | interesting fast-post also resets x-delete-at it seems - caiobrentano_ I would say you're looking for a PUT or a POST on this object after it was originally uploaded which would have very likely reset the x-delete-at | 20:48 |
| clayg | xinli: just use ubuntu cc zaitcev you normally have opinions about this? ;) | 20:49 |
| notmyname | I think zaitcev always logs into the "ubuntu" user ;-) | 20:50 |
| caiobrentano_ | clayg I've found 2 PUT requests to an object. But we don't log the headers | 20:51 |
| caiobrentano_ | clayg couldn't find any POST | 20:52 |
| *** m_kazuhiro has joined #openstack-swift | 20:52 | |
| *** hoonetorg has joined #openstack-swift | 20:53 | |
| caiobrentano_ | clayg I'm pretty sure they upload this files with a cron job doing "swift upload container logs/" | 20:53 |
| notmyname | meeting time in #openstack-meeting in 3 minutes | 20:57 |
| caiobrentano_ | thanks clayg and notmyname! I'm done for now ;) | 20:58 |
| kota_ | morning | 20:59 |
| notmyname | hello kota_ | 20:59 |
| kota_ | hi notmyname | 20:59 |
| *** dmorita has joined #openstack-swift | 20:59 | |
| notmyname | mattoliverau: good morning. around today? | 21:00 |
| mattoliverau | morning, yup, just a change in time makes it hard (earlier) :) | 21:00 |
| *** mariusv has quit IRC | 21:01 | |
| *** mariusv has joined #openstack-swift | 21:03 | |
| acoles | xinli: you can choose (see https://docs.openstack.org/developer/swift/development_saio.html#what-s-in-a-your-user-name) but I have always created a swift user account and used that. I'm not sure if anything assumes the username is swift, the doc says that is the default but if you are careful to set <your-user-name> wherever needed it should be ok to use another username. | 21:04 |
| rledisez | xinli, acoles: i run with an other user than swift, it works fine | 21:05 |
| tdasilva | acoles, xinli: yep, what rledisez said | 21:06 |
| tdasilva | i run with 'vagrant' | 21:06 |
| rledisez | but there are some hardcoded path (eg: /etc/swift/swift.conf) so you still need a root access to create these files | 21:07 |
| acoles | tdasilva: rledisez thanks for confirming that | 21:07 |
| xinli | thank you very much, | 21:08 |
| tdasilva | xinli: for reference: https://github.com/thiagodasilva/ansible-saio | 21:09 |
| mattoliverau | I just run it with a matt user, cause I'm lazy and remembering :P | 21:09 |
| tdasilva | xinli: also, just fyi, there's a swift community meeting happening right now in #openstack-meeting | 21:22 |
| xinli | +tdasilva | 21:22 |
| xinli | thanks | 21:22 |
| *** chosafine has joined #openstack-swift | 21:28 | |
| *** chosafine has quit IRC | 21:39 | |
| *** dja has quit IRC | 21:47 | |
| *** JimCheung has quit IRC | 21:52 | |
| *** JimCheung has joined #openstack-swift | 21:53 | |
| pdardeau_ | ntata: do you know about the your-user-name question and if there are any ill effects of not using swift? | 21:54 |
| *** catintheroof has quit IRC | 21:54 | |
| acoles | clayg: in atlanta someone from storyboard explained to me how we could (if we wanted) trial storyboard - it would create new topics on indexed from a number that would never collide with anything in launchpad, then if/when launchpad was migrated it would all be happy. | 21:54 |
| pdardeau_ | ntata: i spoke too soon. i see it's already been answered | 21:55 |
| zaitcev | "OpenStack leadership training thing in Ann Arbor" sounds insanely corporate, both the name and location. | 21:57 |
| clarkb | its at a deli | 21:58 |
| tdasilva | zaitcev: it will start with compliance and ethics training | 21:58 |
| tdasilva | clayg: we now have a pyeclib job that tests against libec master: https://review.openstack.org/#/c/453847 | 21:59 |
| patchbot | patch 453847 - pyeclib - DO NOT MERGE - testing new gate job | 21:59 |
| notmyname | zaitcev: tdasilva: lol | 22:00 |
| * mattoliverau is going to find breakfast o/ | 22:00 | |
| kota_ | notmyname: btw, is timburke on vacation? | 22:00 |
| kota_ | yet | 22:01 |
| *** ianychoi has joined #openstack-swift | 22:01 | |
| notmyname | kota_: yes | 22:01 |
| notmyname | kota_: for the rest of this week. will be beack monday | 22:01 |
| kota_ | notmyname: ok, thanks | 22:01 |
| acoles | good night | 22:02 |
| *** chlong has quit IRC | 22:02 | |
| clayg | tdasilva: you're getting good at this! | 22:02 |
| zaitcev | clarkb: he isn't a squad leader for nothing | 22:03 |
| clayg | tdasilva: is it *really* better to commit fixes into project-config instead of a do-the-thing.sh *in* the repo? https://review.openstack.org/#/c/452886/ | 22:04 |
| patchbot | patch 452886 - openstack-infra/project-config - add /usr/local/lib to library cache (MERGED) | 22:04 |
| clayg | tdasilva: do you have any idea how you might demonstrate a test that passes on master but fails/skips on system packages? | 22:08 |
| clayg | tdasilva: ... or vice versa? | 22:09 |
| *** dmorita has quit IRC | 22:09 | |
| *** m_kazuhiro has quit IRC | 22:10 | |
| *** xinli has quit IRC | 22:11 | |
| *** jamielennox|away is now known as jamielennox | 22:13 | |
| *** caiobrentano_ has quit IRC | 22:17 | |
| notmyname | FYI next PTG is in Denver Colorado September 11-15 | 22:20 |
| *** foutatoro has joined #openstack-swift | 22:23 | |
| notmyname | good advice (reviews in general, openstack in particular) https://twitter.com/ihrachyshka/status/844757630748811266 | 22:25 |
| zaitcev | Aww. I may already be in Austin by that time. | 22:33 |
| *** klamath has quit IRC | 22:34 | |
| *** adriant has joined #openstack-swift | 23:06 | |
| *** vint_bra has quit IRC | 23:11 | |
| zaitcev | This TC document is irritating | 23:16 |
| zaitcev | maybe my cultural background is a problem | 23:16 |
| zaitcev | "The old confusion about git namespaces is a thing of the past, given how convenient and user friendly these new views are." | 23:16 |
| *** kei_yama has joined #openstack-swift | 23:35 | |
| *** chsc has quit IRC | 23:38 | |
| *** JimCheung has quit IRC | 23:41 | |
| *** JimCheung has joined #openstack-swift | 23:42 | |
| *** jamielennox is now known as jamielennox|away | 23:42 | |
| *** foutatoro has quit IRC | 23:43 | |
| *** jamielennox|away is now known as jamielennox | 23:45 | |
| *** vint_bra has joined #openstack-swift | 23:45 | |
| *** vint_bra has quit IRC | 23:50 | |
| *** _JZ_ has quit IRC | 23:53 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!