torgomatic | mattoliverau: perfect! looks like "-label:Code-Review>=-2,self" is exactly what I'm after :) | 00:02 |
---|---|---|
mattoliverau | torgomatic: cool :) | 00:13 |
*** kevinc_ has quit IRC | 00:20 | |
*** dmorita has joined #openstack-swift | 00:29 | |
*** mwstorer has quit IRC | 00:33 | |
*** shri has quit IRC | 00:34 | |
*** kopparam has joined #openstack-swift | 00:40 | |
*** kopparam has quit IRC | 00:45 | |
openstackgerrit | OpenStack Proposal Bot proposed a change to openstack/python-swiftclient: Updated from global requirements https://review.openstack.org/89250 | 00:47 |
openstackgerrit | OpenStack Proposal Bot proposed a change to openstack/swift: Updated from global requirements https://review.openstack.org/88736 | 00:48 |
clayg | lol guess what binds more closely than python's ternary conditional operator -> https://github.com/openstack/swift/blob/master/swift/common/db.py#L472 | 00:50 |
torgomatic | clayg: conn.execute("outgoing") you say? | 00:52 |
clayg | torgomatic: sqlite seems to think it's a syntax error - but that codes always been like that - so it's obviously wrong | 00:54 |
torgomatic | clayg: maybe that nobody calls get_syncs() with incoming=False? at least not on my current branch | 00:55 |
clayg | torgomatic: yeah I think everyone uses get_sync - it's vistigial code - not surprising it doesn't work | 00:57 |
torgomatic | clayg: I see a whopping one caller, and that's in db_replicator.py's _usync_db method, and it doesn't pass incoming=False | 00:58 |
torgomatic | but that's pretty funny that this code is obviously broken, but nobody ever calls it that way so we get by without crashing | 00:59 |
mattoliverau | lol, wow | 00:59 |
*** scohen has joined #openstack-swift | 01:14 | |
*** scohen has quit IRC | 01:14 | |
*** matsuhashi has joined #openstack-swift | 01:17 | |
clayg | landmines are awesome | 01:21 |
clayg | zaitcev: portante: so did you guys have some ideas on merge_timestamps? I've been beefing up test coverage assuming some "what about if you did like this" comment was in the works? | 01:26 |
zaitcev | clayg: working on it right this minute. my plan is 1. split merge_timestamps into account (old) container (new, with is_deleted), 2. pass arguments such as conn all through, 2a. watch out for deadlocks | 01:27 |
clayg | zaitcev: interesting... and in building up the diff you feel it'd look better now than later? | 01:29 |
zaitcev | clayg: Not sure about looks as such, after all I introduce some trampolining (versus core duplication). But at least it won't add as many DB open-closes and closes races Peter mentioned. | 01:30 |
zaitcev | clayg: And yes, I know what Knuth said. | 01:30 |
zaitcev | sorry it took so long | 01:31 |
*** nosnos has joined #openstack-swift | 01:31 | |
clayg | the race is the one where two different calls to merge_timestamps are bothing changing the status of the database one from deleted->recreated and the other recreated->deleted and neither end up changing status_changed_at - cause I acctually think we'd be ok to live with that for a little while. | 01:35 |
zaitcev | okay | 01:36 |
clayg | where "live with that for a little while" basically means I don't think it really matters in that case because you have two other db's that already disagree and now one that agrees with at least one of them - regardless the storage_policy_index will eventually resolve to one of them | 01:37 |
clayg | the multiple cursor checkout thing is a little tricker, the old code was a bit more methodical about stitching together the queries - but like in the is_deleted, get_info case it worked out to less actual connections; OTOH I don't really think those were all that expensive to begin with... | 01:40 |
zaitcev | I see... | 01:40 |
clayg | zaitcev: what *did* Knuth say about merge_timestamps anyway? | 01:41 |
zaitcev | He said that the premature optimization is the root of all evil. | 01:41 |
zaitcev | Which is often taken to mean that it's acceptable to make code as crappy as it happens and "tune" it later, which does not necessarily follow. | 01:42 |
clayg | zaitcev: heh | 01:43 |
clayg | I think i made a comment I before that I was attempting to optomize that routine for readability and correctness - but I may have failed to deliver either :D | 01:44 |
*** openfly has joined #openstack-swift | 01:52 | |
*** scohen has joined #openstack-swift | 02:14 | |
*** scohen has quit IRC | 02:16 | |
*** nsquare has quit IRC | 02:19 | |
*** asdfsf has joined #openstack-swift | 02:21 | |
*** annegentle has joined #openstack-swift | 02:21 | |
*** matsuhashi has quit IRC | 02:29 | |
*** matsuhashi has joined #openstack-swift | 02:32 | |
*** haomai___ has quit IRC | 02:36 | |
*** zhiyan_ is now known as zhiyan | 02:44 | |
*** haomaiwa_ has joined #openstack-swift | 02:45 | |
portante | clayg, zaitcev: I don't think that the issue is about readability or optimization, it is about making decisions based on stale knowledge and executing against it | 02:59 |
*** gyee has quit IRC | 02:59 | |
portante | In this case, we have the facilities to make the decision within one transaction that avoids such cases | 03:00 |
*** matsuhashi has quit IRC | 03:00 | |
portante | I think we should consider merging this as one transaction | 03:00 |
*** kajinamit has joined #openstack-swift | 03:02 | |
zaitcev | They are hard to pack into this because get_info() flushes the pending. | 03:04 |
zaitcev | The old is_deleted did it too. | 03:05 |
zaitcev | I am looking at open-coding is_deleted() now. | 03:05 |
zaitcev | The after-SP is_deleted() is even thicker than before. It's huge. | 03:05 |
portante | open-coding? | 03:06 |
zaitcev | row = conn.execute('''SELECT put_timestamp, delete_timestamp, object_count FROM container_stat''').fetchone() | 03:08 |
zaitcev | it's ugly too | 03:09 |
*** scohen has joined #openstack-swift | 03:15 | |
*** scohen has quit IRC | 03:15 | |
*** nosnos has quit IRC | 03:27 | |
openstackgerrit | Zhang Hua proposed a change to openstack/swift: Add distributed tracing capablities in logging. https://review.openstack.org/93677 | 03:29 |
clayg | portante: I think we are considering it, and it ends up being a tradeoff on readability/maintainability | 03:46 |
clayg | zaitcev: right now I'm hung up reasoning about the difference in how accounts/containers reason about the is_deleted (status vs <db_contains>_count) | 03:47 |
zaitcev | clayg: I'm about to e-mail you a diff | 03:47 |
clayg | zaitcev: oh good, everything i'm trying ends up either being a bunch of dupliation or spaghetti | 03:49 |
zaitcev | I'm using the gmail address, is that okay | 03:50 |
clayg | zaitcev: yeah whatever, just get up the diff man! ;) | 03:51 |
zaitcev | I feel like this was the "mountain bourne a mouse" kind of thing | 03:52 |
zaitcev | and an ugly mouse at that | 03:52 |
zaitcev | but hey, it's a transaction like portante observed | 03:52 |
*** matsuhashi has joined #openstack-swift | 03:53 | |
clayg | zaitcev: k i see what you did, i think the idea of passing around the conn is mostly reasonable | 03:53 |
zaitcev | My own motivation was that remember how we discussed that wee shuld not have any "callbacks" anymore (unless passed as arg, like a real callback). It was like the #1 outcome of hackathon to me | 03:54 |
zaitcev | Next week we're at work and your adding these is_deleted() calls into derived class from base class' merge_timestamps | 03:54 |
zaitcev | Like duuuuuude, what did we have the freaking hackathon for | 03:54 |
zaitcev | (well it was fun) | 03:55 |
clayg | zaitcev: I guess common.db shouldn't really have a merge_timestamps method, and the account should just say "with self.get() as conn: self._merge_timestamps(conn, *some_args)" ? | 03:55 |
zaitcev | So I felt challenged to rearrange the code so it follows our agreement | 03:55 |
zaitcev | unless it's impossible, of course - no need to be too dogmatic | 03:56 |
clayg | zaitcev: well as I said, the sp work sorta prompted me to realize how much the existing layout needed some dogmatism | 03:57 |
zaitcev | clayg: oh crap you're right | 03:57 |
zaitcev | clayg: no, wait, scared myself too early. I took yor comment to mean that I forgot the with conn: | 03:58 |
clayg | zaitcev: no i think what you have is correct, I was just looking the duplication in the middle there between container.merge_timestamps and common._merge_timestamps | 04:00 |
zaitcev | clayg: I figured you needed to think in broader categories because SP was so big. So, you declare: status is changed when the deleted status is changed. Then you reason about it, without regard to how much crap is necessary to run is_deleted(). | 04:00 |
zaitcev | clayg: yeah, sadly it's duplicated | 04:00 |
*** charz has joined #openstack-swift | 04:01 | |
zaitcev | clayg: give me a sec to re-upload | 04:03 |
zaitcev | actually, re-running unit tests and pep8 just in case | 04:04 |
*** nosnos has joined #openstack-swift | 04:07 | |
zaitcev | http://www.zaitcev.us/things/swift-sp-merge_del-2.diff | 04:08 |
*** matsuhashi has quit IRC | 04:12 | |
*** matsuhashi has joined #openstack-swift | 04:12 | |
clayg | zaitcev: yeah that's basically where I ended up | 04:13 |
zaitcev | oh | 04:13 |
clayg | zaitcev: but looking at who else uses update_status_changed_at... It's all the same mess in _update_or_create all over again | 04:13 |
clayg | also, it's maybe annoying that accounts don't get their status_changed_at "fixed" | 04:15 |
zaitcev | I meant to ask you about it | 04:15 |
*** scohen has joined #openstack-swift | 04:15 | |
zaitcev | Does it do anything at all? | 04:15 |
zaitcev | I see why it's needed in containers | 04:16 |
*** scohen has quit IRC | 04:16 | |
zaitcev | But I may be missing something crucial. | 04:16 |
clayg | i'm pretty luke warm at this effort, esspecially this late in the cycle (having been burnt on the per policy tempdir stuff) - i agree "mountain out of molehill" | 04:16 |
clayg | zaitcev: no it's not critical for accounts (currently) | 04:16 |
zaitcev | You're just tired. But we can fix it up later, yes. | 04:17 |
*** matsuhashi has quit IRC | 04:17 | |
clayg | zaitcev: but when we were doing the original prototyping of storage polices it ended up being a huge hassel that there was this field called "status_changed_at" that didn't acctually include the time when the damn db changed status - and we're just leaving that for the next guy on accounts | 04:17 |
*** haomaiwa_ has quit IRC | 04:17 | |
*** haomaiwang has joined #openstack-swift | 04:18 | |
*** annegentle has quit IRC | 04:18 | |
zaitcev | I am going to drink some tea and sleep | 04:18 |
clayg | well, g'night | 04:18 |
*** matsuhashi has joined #openstack-swift | 04:21 | |
*** zaitcev has quit IRC | 04:27 | |
*** haomai___ has joined #openstack-swift | 04:27 | |
*** ppai has joined #openstack-swift | 04:29 | |
clayg | well shit, if we're worried that we get the wrong behavior taking action on stale info - I wonder why we shoudln't have to call commit_puts? | 04:29 |
*** haomaiwang has quit IRC | 04:31 | |
*** matsuhashi has quit IRC | 04:36 | |
*** matsuhashi has joined #openstack-swift | 04:36 | |
*** matsuhashi has quit IRC | 04:39 | |
*** matsuhashi has joined #openstack-swift | 04:39 | |
*** ppai has quit IRC | 04:40 | |
*** psharma has joined #openstack-swift | 04:45 | |
*** scohen has joined #openstack-swift | 04:49 | |
*** scohen has quit IRC | 04:53 | |
*** matsuhashi has quit IRC | 05:11 | |
*** matsuhashi has joined #openstack-swift | 05:11 | |
*** annegentle has joined #openstack-swift | 05:14 | |
*** annegentle is now known as Guest25926 | 05:14 | |
*** matsuhashi has quit IRC | 05:16 | |
*** nsquare has joined #openstack-swift | 05:16 | |
*** matsuhashi has joined #openstack-swift | 05:17 | |
*** Guest25926 has quit IRC | 05:19 | |
*** Midnightmyth has joined #openstack-swift | 05:22 | |
*** matsuhashi has quit IRC | 05:24 | |
*** matsuhashi has joined #openstack-swift | 05:24 | |
*** kopparam has joined #openstack-swift | 05:24 | |
*** scohen has joined #openstack-swift | 05:29 | |
*** ppai has joined #openstack-swift | 05:33 | |
*** annegentle has joined #openstack-swift | 05:44 | |
*** annegentle is now known as Guest66581 | 05:45 | |
*** Guest66581 has quit IRC | 05:51 | |
*** kopparam has quit IRC | 06:02 | |
*** kopparam has joined #openstack-swift | 06:05 | |
*** psharma has quit IRC | 06:19 | |
*** openstackgerrit has quit IRC | 06:19 | |
*** omame has quit IRC | 06:19 | |
*** foexle has joined #openstack-swift | 06:23 | |
*** psharma has joined #openstack-swift | 06:23 | |
*** openstackgerrit has joined #openstack-swift | 06:23 | |
*** omame has joined #openstack-swift | 06:23 | |
*** kopparam is now known as vm | 06:24 | |
*** vm is now known as kopparam | 06:25 | |
*** scohen has quit IRC | 06:28 | |
*** scohen has joined #openstack-swift | 06:28 | |
*** miqui has quit IRC | 06:36 | |
*** annegentle has joined #openstack-swift | 06:46 | |
*** annegentle is now known as Guest44191 | 06:46 | |
mattoliverau | I'm calling it a day, night all | 06:46 |
*** Guest44191 has quit IRC | 06:50 | |
*** kopparam has quit IRC | 06:51 | |
*** kopparam has joined #openstack-swift | 06:52 | |
*** kopparam has quit IRC | 07:02 | |
*** kopparam has joined #openstack-swift | 07:03 | |
*** kopparam has quit IRC | 07:07 | |
*** matsuhashi has quit IRC | 07:08 | |
*** matsuhas_ has joined #openstack-swift | 07:08 | |
*** asdfsf has quit IRC | 07:14 | |
*** chandan_kumar has quit IRC | 07:17 | |
*** nshaikh has joined #openstack-swift | 07:22 | |
*** charz has quit IRC | 07:25 | |
*** toolslive has joined #openstack-swift | 07:31 | |
*** kopparam has joined #openstack-swift | 07:44 | |
*** annegentle has joined #openstack-swift | 07:46 | |
*** annegentle is now known as Guest9369 | 07:47 | |
*** nacim has joined #openstack-swift | 07:50 | |
*** jamiehannaford has joined #openstack-swift | 07:50 | |
*** Guest9369 has quit IRC | 07:51 | |
*** nacim has quit IRC | 07:55 | |
openstackgerrit | Clay Gerrard proposed a change to openstack/swift: Fixes probe tests with non-zero default storage policy https://review.openstack.org/96353 | 07:56 |
openstackgerrit | Clay Gerrard proposed a change to openstack/swift: Add two vector timestamps https://review.openstack.org/99315 | 07:56 |
openstackgerrit | Clay Gerrard proposed a change to openstack/swift: Add Storage Policy Support https://review.openstack.org/96027 | 07:56 |
openstackgerrit | Clay Gerrard proposed a change to openstack/swift: Add Storage Policy Documentation https://review.openstack.org/96026 | 07:56 |
openstackgerrit | Clay Gerrard proposed a change to openstack/swift: Add Storage Policy support to Containers https://review.openstack.org/96029 | 07:56 |
openstackgerrit | Clay Gerrard proposed a change to openstack/swift: Update FakeRing and FakeLogger https://review.openstack.org/96028 | 07:56 |
openstackgerrit | Clay Gerrard proposed a change to openstack/swift: Add Storage Policy support to Object Server https://review.openstack.org/96030 | 07:56 |
openstackgerrit | Clay Gerrard proposed a change to openstack/swift: Fix object-expirer for missing objects https://review.openstack.org/98511 | 07:56 |
openstackgerrit | Clay Gerrard proposed a change to openstack/swift: Add functional tests for Storage Policy https://review.openstack.org/96042 | 07:56 |
openstackgerrit | Clay Gerrard proposed a change to openstack/swift: Update bin scripts to be storage policy aware https://review.openstack.org/96043 | 07:56 |
openstackgerrit | Clay Gerrard proposed a change to openstack/swift: Merge container storage_policy_index https://review.openstack.org/96040 | 07:56 |
openstackgerrit | Clay Gerrard proposed a change to openstack/swift: Add Storage Policy Support to Accounts https://review.openstack.org/96041 | 07:56 |
openstackgerrit | Clay Gerrard proposed a change to openstack/swift: Add Storage Policy Support to list_endpoints https://review.openstack.org/96046 | 07:56 |
openstackgerrit | Clay Gerrard proposed a change to openstack/swift: Add Storage Policy Support to Container Sync https://review.openstack.org/96047 | 07:56 |
openstackgerrit | Clay Gerrard proposed a change to openstack/swift: Enqueue misplaced objects during container replication https://review.openstack.org/96044 | 07:56 |
openstackgerrit | Clay Gerrard proposed a change to openstack/swift: Add LRUCache to common.utils https://review.openstack.org/96045 | 07:56 |
openstackgerrit | Clay Gerrard proposed a change to openstack/swift: Add Storage Policy Support to ssync https://review.openstack.org/96034 | 07:56 |
openstackgerrit | Clay Gerrard proposed a change to openstack/swift: Put X-Backend-Timestamp in object 404 responses https://review.openstack.org/96035 | 07:56 |
openstackgerrit | Clay Gerrard proposed a change to openstack/swift: Add Storage Policy Support to the Auditor https://review.openstack.org/96032 | 07:56 |
openstackgerrit | Clay Gerrard proposed a change to openstack/swift: Add storage policy support for the Replicator https://review.openstack.org/96033 | 07:56 |
openstackgerrit | Clay Gerrard proposed a change to openstack/swift: Add container-reconciler daemon https://review.openstack.org/96038 | 07:56 |
openstackgerrit | Clay Gerrard proposed a change to openstack/swift: Add reconciler probetest outline https://review.openstack.org/96039 | 07:56 |
openstackgerrit | Clay Gerrard proposed a change to openstack/swift: Extend interface on InternalClient https://review.openstack.org/96036 | 07:56 |
openstackgerrit | Clay Gerrard proposed a change to openstack/swift: Extend direct_client https://review.openstack.org/96037 | 07:56 |
openstackgerrit | Clay Gerrard proposed a change to openstack/swift: Add Storage Policy support to Object Updates https://review.openstack.org/99979 | 07:56 |
openstackgerrit | Clay Gerrard proposed a change to openstack/swift: Add Storage Policy support to the Account Reaper https://review.openstack.org/96049 | 07:56 |
openstackgerrit | Clay Gerrard proposed a change to openstack/swift: Add Storage Policy Support to Recon Middleware https://review.openstack.org/96048 | 07:56 |
openstackgerrit | Clay Gerrard proposed a change to openstack/swift: Refactoring storage policies merge_timestamps https://review.openstack.org/100807 | 07:56 |
*** nsquare has quit IRC | 07:58 | |
*** nacim has joined #openstack-swift | 08:00 | |
clayg | i think there's still something to do on the timestamp offset width, the list endpoints v2 response, and replacing the header constants with literals | 08:00 |
clayg | but the probetest should be fixed, and I took a stab at something for merge_timestamps | 08:01 |
*** kopparam has quit IRC | 08:05 | |
*** kopparam has joined #openstack-swift | 08:05 | |
*** kopparam has quit IRC | 08:07 | |
*** kopparam has joined #openstack-swift | 08:08 | |
*** Midnightmyth has quit IRC | 08:10 | |
ahale | I can't wait to deploy all this to a prod cluster, see what happens - so much change! | 08:11 |
*** kopparam has quit IRC | 08:12 | |
omame | I'm already preparing the popcorn | 08:12 |
*** ppai has quit IRC | 08:13 | |
*** mmcardle has joined #openstack-swift | 08:22 | |
*** toolslive has quit IRC | 08:22 | |
*** haomai___ has quit IRC | 08:28 | |
*** kopparam has joined #openstack-swift | 08:30 | |
*** ppai has joined #openstack-swift | 08:30 | |
clayg | nom nom nom | 08:34 |
*** toolslive has joined #openstack-swift | 08:35 | |
*** Longgeek_ has joined #openstack-swift | 08:53 | |
*** mkollaro has joined #openstack-swift | 08:53 | |
*** matsuhas_ has quit IRC | 09:01 | |
*** mmcardle has quit IRC | 09:02 | |
*** mmcardle has joined #openstack-swift | 09:03 | |
*** matsuhashi has joined #openstack-swift | 09:09 | |
*** mkerrin has quit IRC | 09:22 | |
*** mkollaro has quit IRC | 09:33 | |
*** matsuhashi has quit IRC | 09:38 | |
*** dANOKELOFF has joined #openstack-swift | 09:39 | |
dANOKELOFF | Hi guys, I try to implement tempurl on swift but when i execute my curl command i have this error : 401 Unauthorized: Temp URL invalid , anyone can help me ? | 09:40 |
*** mkollaro has joined #openstack-swift | 09:41 | |
*** toolslive has quit IRC | 09:48 | |
*** zhiyan is now known as zhiyan_ | 09:51 | |
*** dANOKELOFF has quit IRC | 09:53 | |
*** matsuhashi has joined #openstack-swift | 10:01 | |
*** dANOKELO_ has joined #openstack-swift | 10:03 | |
*** matsuhashi has quit IRC | 10:06 | |
*** Kbee has joined #openstack-swift | 10:08 | |
*** matsuhashi has joined #openstack-swift | 10:08 | |
*** scohen has quit IRC | 10:09 | |
*** scohen has joined #openstack-swift | 10:14 | |
*** kajinamit has quit IRC | 10:14 | |
*** nacim has quit IRC | 10:16 | |
*** matsuhashi has quit IRC | 10:17 | |
*** matsuhashi has joined #openstack-swift | 10:18 | |
*** kopparam has quit IRC | 10:20 | |
*** kopparam has joined #openstack-swift | 10:21 | |
*** Trixboxer has joined #openstack-swift | 10:22 | |
*** dmorita has quit IRC | 10:25 | |
*** kopparam has quit IRC | 10:25 | |
*** mkerrin has joined #openstack-swift | 10:27 | |
*** kopparam has joined #openstack-swift | 10:37 | |
*** Kbee has quit IRC | 10:38 | |
*** mmcardle has quit IRC | 10:41 | |
*** mmcardle has joined #openstack-swift | 10:44 | |
*** mmcardle has quit IRC | 10:44 | |
*** mmcardle has joined #openstack-swift | 10:47 | |
*** Longgeek_ has quit IRC | 10:58 | |
*** tdasilva_ has joined #openstack-swift | 10:59 | |
*** tdasilva has joined #openstack-swift | 11:00 | |
*** tdasilva has quit IRC | 11:00 | |
*** tdasilva_ is now known as tdasilva | 11:00 | |
rieglflo | hi guys, while paginating through objects within an container, is there any way to check whether i'm on the first or the last page? to check if object count is smaller than the limit, is just not valid. and when using the end_marker to page back the same issue comes up for the first page ... i never know when i have reached the end. any recommendations? | 11:30 |
*** d0ugal has quit IRC | 11:33 | |
*** d0ugal has joined #openstack-swift | 11:34 | |
*** mmcardle has quit IRC | 11:40 | |
*** kopparam has quit IRC | 11:43 | |
*** kopparam has joined #openstack-swift | 11:43 | |
*** ppai has quit IRC | 11:46 | |
*** kopparam has quit IRC | 11:47 | |
*** matsuhashi has quit IRC | 11:48 | |
*** kopparam has joined #openstack-swift | 11:49 | |
*** matsuhashi has joined #openstack-swift | 11:51 | |
*** matsuhashi has quit IRC | 11:56 | |
*** matsuhashi has joined #openstack-swift | 11:56 | |
*** scohen has quit IRC | 11:56 | |
dANOKELO_ | can we remove manually the expire url in swift ? | 11:58 |
*** scohen has joined #openstack-swift | 11:59 | |
*** ppai has joined #openstack-swift | 11:59 | |
*** matsuhashi has quit IRC | 12:01 | |
*** nacim has joined #openstack-swift | 12:03 | |
*** ppai has quit IRC | 12:13 | |
*** tdasilva_ has joined #openstack-swift | 12:21 | |
*** tdasilva has quit IRC | 12:22 | |
*** annegent_ has joined #openstack-swift | 12:23 | |
*** annegent_ is now known as annegentle_ | 12:24 | |
*** kopparam has quit IRC | 12:25 | |
*** kopparam has joined #openstack-swift | 12:25 | |
*** mmcardle has joined #openstack-swift | 12:28 | |
*** kopparam has quit IRC | 12:30 | |
ahale | chmouel: do you have to wear a fedora now not your cool homebrew baseball cap ? | 12:32 |
omame | ha | 12:33 |
ahale | dANOKELO_: no you can't expire it earlier without changing the key, you can create a new different one though if its because you had problems with the 401'ing one earlier | 12:33 |
ahale | seems a common request recently though | 12:34 |
dANOKELO_ | ahale: we cant delete the tempurl ? | 12:34 |
ahale | nope, theres nothing to delete really | 12:34 |
chmouel | ahale: i guess i need to work on my redhat pose yeah ;) | 12:34 |
ahale | you can invalidate the hmac thing but thats about it | 12:35 |
ahale | chmouel: heheh | 12:35 |
*** tdasilva_ is now known as tdasilva | 12:37 | |
dANOKELO_ | ahale: ok thank's | 12:43 |
rieglflo | a config setting for the tempurl plugin with max expire seconds would be enough ... e.g. all links with an expiry greater than time() + max_expire_seconds are invalid | 12:45 |
rieglflo | so the max age of tempurls can be configured on the server side ... | 12:46 |
*** annegentle_ has quit IRC | 12:53 | |
*** sombrafam has joined #openstack-swift | 12:56 | |
openstackgerrit | ChangBo Guo(gcb) proposed a change to openstack/swift: Add common units constants https://review.openstack.org/76490 | 13:01 |
*** annegent_ has joined #openstack-swift | 13:03 | |
*** lpabon has joined #openstack-swift | 13:05 | |
peluse_ | clayg: good stuff man - do you some some sort of shift differential for working graveyard? :) | 13:06 |
*** annegent_ has quit IRC | 13:13 | |
*** psharma_ has joined #openstack-swift | 13:14 | |
*** navid__ has joined #openstack-swift | 13:14 | |
*** nshaikh has quit IRC | 13:17 | |
*** psharma has quit IRC | 13:17 | |
*** psharma__ has joined #openstack-swift | 13:19 | |
*** nshaikh has joined #openstack-swift | 13:19 | |
*** navid__ has quit IRC | 13:22 | |
*** psharma_ has quit IRC | 13:22 | |
*** psharma__ has quit IRC | 13:22 | |
*** Kbee has joined #openstack-swift | 13:28 | |
peluse_ | portante: you around? | 13:31 |
*** annegent_ has joined #openstack-swift | 13:35 | |
*** annegent_ is now known as annegentle_ | 13:36 | |
creiht | wow.. congrats to the enovance folkd | 13:37 |
creiht | folks | 13:37 |
*** annegentle_ has quit IRC | 13:40 | |
*** zz_wasmum is now known as wasmum | 13:41 | |
*** annegent_ has joined #openstack-swift | 13:41 | |
wasmum | Great Morning, all. | 13:42 |
wasmum | say, anyone have experience with swift3 from stackforge, using it with keystone? I would like to have some conversation later, if possible. | 13:43 |
*** miqui has joined #openstack-swift | 13:46 | |
*** Longgeek_ has joined #openstack-swift | 13:57 | |
*** dmsimard_away is now known as dmsimard | 14:02 | |
*** nosnos has quit IRC | 14:03 | |
*** asdfsf has joined #openstack-swift | 14:03 | |
*** pberis has joined #openstack-swift | 14:04 | |
*** mmcardle has quit IRC | 14:05 | |
*** mmcardle has joined #openstack-swift | 14:07 | |
*** annegent_ has quit IRC | 14:09 | |
*** annegent_ has joined #openstack-swift | 14:10 | |
peluse_ | portante: saw in process note on gerrit... OK, cool. I was fixing things up as well but will stop. I'm sure you noticed that reload_storage_policies() is using local swift.conf and not the one defined in __init__.py, that's what I was just looking at... | 14:15 |
portante | that is what I was fixing and have that working now | 14:18 |
peluse_ | portante: awesome | 14:19 |
*** esmute has quit IRC | 14:22 | |
*** haomaiwang has joined #openstack-swift | 14:23 | |
*** esmute has joined #openstack-swift | 14:25 | |
*** mwstorer has joined #openstack-swift | 14:29 | |
*** asdfsf has quit IRC | 14:30 | |
*** kopparam has joined #openstack-swift | 14:31 | |
*** Longgeek_ has quit IRC | 14:43 | |
*** scohen has quit IRC | 14:43 | |
*** annegent_ has quit IRC | 14:57 | |
thurloat | wasmum I'm going to be looking at deploying swift3 on keystone in the coming months, care to document your woes for my future self? :) | 14:59 |
*** foexle has quit IRC | 15:00 | |
*** mmcardle has quit IRC | 15:00 | |
*** tdasilva has quit IRC | 15:03 | |
*** kevinc_ has joined #openstack-swift | 15:03 | |
*** jergerber has joined #openstack-swift | 15:04 | |
*** annegent_ has joined #openstack-swift | 15:04 | |
*** tdasilva has joined #openstack-swift | 15:05 | |
*** kopparam has quit IRC | 15:05 | |
*** kopparam has joined #openstack-swift | 15:06 | |
*** zaitcev has joined #openstack-swift | 15:06 | |
*** ChanServ sets mode: +v zaitcev | 15:06 | |
*** mmcardle has joined #openstack-swift | 15:08 | |
notmyname | good morning, world | 15:09 |
*** byeager has joined #openstack-swift | 15:10 | |
*** kopparam has quit IRC | 15:11 | |
dmsimard | notmyname: good morning :) | 15:11 |
notmyname | another day, another Red Hat acquisition :-) | 15:11 |
zaitcev | now what | 15:13 |
zaitcev | Not SwiftStack I'm sure | 15:13 |
notmyname | zaitcev: you bought eNovance | 15:15 |
notmyname | chmouel: cschwede: does this mean drinks are on you in Paris? ;-) | 15:15 |
*** nshaikh has quit IRC | 15:17 | |
peluse_ | that sounds logical to me | 15:18 |
*** Kbee has quit IRC | 15:19 | |
creiht | haha | 15:21 |
notmyname | the SP patches have been under review for quite some time now. soft freeze of master as well. I know that there are improvements being made and bugs found, but we need some resolution soon | 15:21 |
zaitcev | It was my sense too. But I'm behind on reviewing and testing it and get focused on small things. | 15:22 |
portante | notmyname: true, but what is that saying? no cheese before wine? ;) | 15:24 |
notmyname | portante: oh I can whine quite a bit | 15:24 |
peluse_ | notmyname: is it reasonable to vote on merge to master and address any other lingering items prior to cutting RC (so as to end the freeze and keep the ball moving forward) | 15:24 |
portante | whether it lands today or another day, still going to be reviewing that patch chain as we go | 15:25 |
chmouel | notmyname: yeah there would be plenty of champagne tonight :) | 15:25 |
notmyname | peluse_: I think that is a reasonable suggestion (it does make progress), but also working towards an actual release is important | 15:26 |
peluse_ | yup, agree just seems like we're a little "stuck" and can make faster progress on the smaller things by getting this monster chain mgmt out of the way | 15:27 |
notmyname | the current plan is to merge (and see what else needs to land, but that's mostly handled already), cut an RC and have that for a couple of weeks, then release | 15:27 |
notmyname | peluse_: so you're saying just get the mechanics of the merge out of the way (ie propose the merge commit and then just vote on that)? rather than focus on the patch chain itself more? | 15:28 |
notmyname | once SP is merged, "normal" life should return where we address new patches and fix discovered bugs | 15:29 |
peluse_ | notmyname: yes, I'm saying if there are now showstoppers lets merge to master and relieve clayg of managing the chain. This also allows other to directly fix SP related items (right now nobody pushes on one of claygs patches because it messes up the process) | 15:29 |
notmyname | peluse_: s/now/no/ | 15:30 |
notmyname | peluse_: yes, that makes a lot of sense to me | 15:30 |
peluse_ | yes :) | 15:30 |
notmyname | peluse_: why is it always the single most important word in the sentence that gets the typo? ;-) | 15:30 |
peluse_ | notmyname: lots of recent examples of tweaks that could have been by several other folks that we saddled clayg with because of the chain... even one current (in process func tests) | 15:31 |
portante | can I be a small thorn, and just request a little time to update my patch chain locally in my SAIOs and verify that latest? | 15:31 |
peluse_ | notmyname: yeah, my typos always totally change the meaning of what I'm trying to say :) | 15:31 |
notmyname | portante: of course. I don't think anyone is doing anything in the next couple of hours :-) | 15:32 |
notmyname | portante: suspect clayg might still be asleep ;-) | 15:32 |
peluse_ | notmyname: I think clayg just went to bed a few hrs ago :) | 15:32 |
peluse_ | ha! | 15:32 |
*** kevinc_ has quit IRC | 15:33 | |
portante | notmyname: great, thanks | 15:33 |
notmyname | peluse_: I'm told the avotons are back online now (thanks mlanner!!). I'm going to check on them today | 15:34 |
goodes | good morning all | 15:34 |
notmyname | peluse_: portante: I myself am going to go get ready now. I'll be back online later | 15:35 |
*** jyoti_ranjan has joined #openstack-swift | 15:35 | |
portante | k | 15:35 |
peluse_ | notmyname: cool, curious to see the root cause of those dead mobos... not super surprising though, that's a brand new board and you got one of the first batches | 15:36 |
jyoti_ranjan | I have one question related to Swift deployment. | 15:36 |
jyoti_ranjan | Can we have more than one Swift deployed in a given cloud? If yes how endpoint will be registered to cloud? | 15:36 |
* peluse_ also needs to get organized and head into the office... | 15:36 | |
*** scohen has joined #openstack-swift | 15:37 | |
*** kevinc_ has joined #openstack-swift | 15:39 | |
*** pberis has quit IRC | 15:46 | |
jyoti_ranjan | I know that people do not use this use case typically (i.e. two Swift). I am asking because there is need to have two Swift setup: one for internal cloud usage and another for end consumer.. | 15:47 |
*** mmcardle has quit IRC | 15:47 | |
jyoti_ranjan | Certianly alterante option is to have on Swift and use different account for differnt purpose. But wanted to know whether it is technially feasable to deploy two Swift in a given cloud or not. | 15:48 |
*** mmcardle has joined #openstack-swift | 15:49 | |
*** pberis has joined #openstack-swift | 15:49 | |
*** bsdkurt has joined #openstack-swift | 15:50 | |
*** Kbee has joined #openstack-swift | 15:50 | |
*** haomaiwang has quit IRC | 15:51 | |
*** haomaiwang has joined #openstack-swift | 15:52 | |
creiht | jyoti_ranjan: yeah you can do that, you just have two different end points | 15:52 |
zaitcev | A typical client won't be able to select them unless region is specified | 15:55 |
zaitcev | I don't know how clients default when facing two. | 15:56 |
jyoti_ranjan | zaitcev: can I use same keystone for both deployment? | 15:56 |
zaitcev | jyoti_ranjan: Yes, by specifying a region. | 15:56 |
jyoti_ranjan | Also, I do see that there is a field call region as an attribute in keystone which we use at the time of registering end point. | 15:57 |
*** Kbee has quit IRC | 15:57 | |
jyoti_ranjan | let us say, I have two endpoitns: region 1 for internal, region 2 for external. | 15:57 |
creiht | correct, each one has to be a separate region | 15:58 |
jyoti_ranjan | Can cinder running in region 1 can use end point of Swift running in region 2/ | 15:58 |
creiht | that I do not know | 15:58 |
jyoti_ranjan | I want o use for volume backup | 15:58 |
jyoti_ranjan | Cinder just needs Swift backpoint but not sure across region it will work or not. | 15:59 |
*** rieglflo has quit IRC | 15:59 | |
*** tdasilva has quit IRC | 15:59 | |
creiht | if it is just an endpoint in cinder, then you should be fine | 15:59 |
*** gyee has joined #openstack-swift | 16:00 | |
jyoti_ranjan | Great, it helps. We need something like backup_swift_url=http://localhost:8080/v1/AUTH_ and its associated user name and password etc | 16:04 |
*** dANOKELO_ has quit IRC | 16:07 | |
*** haomaiwang has quit IRC | 16:08 | |
*** pberis has quit IRC | 16:11 | |
*** pberis has joined #openstack-swift | 16:12 | |
openstackgerrit | Donagh McCabe proposed a change to openstack/swift: Add swift-checker utility for use by monitoring tools https://review.openstack.org/99961 | 16:12 |
*** chandan_kumar has joined #openstack-swift | 16:18 | |
mlanner | peluse_: i think we may actually have another (5th) bad mobo. i'll have to do some surgery later today. <- notmyname | 16:18 |
*** elambert has joined #openstack-swift | 16:21 | |
wasmum | thurloat: re: documentation, I'll try and share the discovery with you | 16:21 |
thurloat | wasmum thanks a bunch. | 16:22 |
*** cHilDPROdigY1337 is now known as nottrobin | 16:26 | |
gholt | notmyname: Do we have a copyright policy yet? https://review.openstack.org/#/c/76490/ | 16:31 |
*** ezml has quit IRC | 16:32 | |
notmyname | gholt: actually, I think the copyright line(s) in that patch are doing the right thing. openstack doesn't require copyright assignment (and in fact the foundation doesn't want assignment) | 16:36 |
*** nacim has quit IRC | 16:36 | |
notmyname | gholt: so aside from the weirdness of "powers of 2 are now (c) IBM" the copyright in that patch is good | 16:37 |
notmyname | gholt: note that I think the patch is pretty frivolous, though, and my comments on the copyright are independent of if that patch should land (I'd tend to "no") | 16:37 |
notmyname | I'll leave a comment | 16:38 |
gholt | Okay cool, so my listing of everybody that touched a file for copyright purposes is still fine? :) | 16:39 |
notmyname | gholt: maybe. in that case you're getting into the concepts of "work for hire" etc. which may in fact be different for different companies and different countries :-) | 16:40 |
notmyname | which is why I mostly stay away from all of it :-) | 16:40 |
gholt | Hehe, yeah. It's still the root swift/__init__.py suddenly being all "HP!" that riles me. I don't mind folks adding their stamps, but kind of pretending you did it all is a bit off. | 16:41 |
*** haomaiwa_ has joined #openstack-swift | 16:43 | |
gholt | If I submitted a pull request to somebody's project and plopped my copyright on their root file, giving no other credit, I would expect to be soundly told "no". | 16:45 |
notmyname | gholt: even if you replaced the entire contents of the file, except for the license header? | 16:45 |
notmyname | (not disagreeing, just continuing the conversation) | 16:46 |
gholt | Yes, unless that was the entire project in that one file. | 16:46 |
gholt | For an analogy, it'd be like a book cover artist only having his name on the cover. | 16:47 |
*** haomaiwa_ has quit IRC | 16:48 | |
notmyname | once the SP patches land, I think there will be some churn to that file. If you remember, it turns out that our versioning stuff isn't "right", so we'll be adjusting anyway. And if we end up touching __init__, then we can fix it :-) | 16:49 |
torgomatic | does this mean I need to figure out if I put my name or my company's name in copyright statements now? :| | 16:49 |
notmyname | torgomatic: only after you figure out how much you care | 16:50 |
gholt | Heheh, you could just use my "who worked on this?" script, modified to use a .companymap file. ;) | 16:51 |
gholt | Or .copyrightmap file I guess, heh | 16:51 |
torgomatic | notmyname: I'm not particularly upset about it... maybe 7 or 8 angststroms? | 16:51 |
*** byeager has quit IRC | 16:52 | |
gholt | On a scale of 1-10! omg! | 16:53 |
gholt | Oh, maybe angststroms are logarithmic... | 16:54 |
torgomatic | :) | 16:55 |
*** Midnightmyth has joined #openstack-swift | 16:57 | |
gholt | btw, I think I'm about done as much as I can be with the SP reviews. My brain has gotten to the point where it's not really seeing new things anymore. Though I did realize after a bit that I wasn't even working off the tail anymore. :/ | 16:57 |
torgomatic | gholt: got any feedback to post? | 16:58 |
gholt | Just what I have already | 16:58 |
torgomatic | cool | 16:59 |
peluse_ | gholt: anything specific for discussion here? | 16:59 |
gholt | Just be sure, 96026 is the tail now, right? :) | 16:59 |
peluse_ | yes | 17:00 |
torgomatic | I'm sure there'll be a trail of bugs to fix post-merge, but that's true of just about any patch on just about any project | 17:00 |
peluse_ | especially of this magnitude.. totally agree | 17:00 |
peluse_ | there'd better only be 1 or 2 more tmp-N dir bugs though :) | 17:01 |
gholt | I do like that when I diff against master, about 2/3 are tests. | 17:02 |
notmyname | :-) | 17:02 |
peluse_ | gholt: and that was no accident! | 17:02 |
notmyname | ya, the fact that such a huge feature is actually increasing test coverage is nice :-) | 17:02 |
notmyname | good job peluse_ and clayg and torgomatic and portante! | 17:03 |
*** shri has joined #openstack-swift | 17:03 | |
torgomatic | so it looks like there's a little more pending work on the 2-vector timestamp thing, but the rest seems okay...? | 17:04 |
* torgomatic is just trying to get a sense of known problems | 17:04 | |
*** jyoti_ranjan has quit IRC | 17:06 | |
peluse_ | torgomatic: you mean extending the offset width? | 17:06 |
torgomatic | peluse_: yeah, that | 17:06 |
torgomatic | I think clayg's on top of it | 17:07 |
peluse_ | torgomatic: OK, I didn't have a strong opinion either way but if extending then agree now would be the time | 17:07 |
torgomatic | peluse_: I'm with you there; I don't really care how wide it is, but adding more later will be hard | 17:07 |
* torgomatic goes quiet for the walk from train to office | 17:08 | |
gholt | Mostly I think we just need to be sure no one ever uses the second stamp to change actual content. I don't think POST or delete could use them, for instance. It could cause sync problems external to the cluster. | 17:10 |
gholt | Oh, I guess I should leave that as a comment on the commit message. | 17:13 |
clayg | morning all, just read back through my scroll buffer | 17:13 |
notmyname | good morning clayg | 17:13 |
clayg | not sure how to say redhat totally baffles me without sounding like I don't think the enovance team is compleatly awesome - which of course they are - but i really don't get redhat... | 17:14 |
peluse_ | morning | 17:14 |
peluse_ | watch out or they may "get" you :) | 17:15 |
clayg | i think i said before i went to bed - there's like three things outstanding... | 17:15 |
*** cutforth has joined #openstack-swift | 17:15 | |
clayg | timestamp offset width, the list endpoints v2 response, and replacing the header constants with literals | 17:15 |
peluse_ | clayg: I think the last 2 can wait til SP lands on master no? | 17:16 |
clayg | gholt: i think we need the offset on delete, don't know if I could/should exclude it on post exactly... | 17:16 |
clayg | oh idk, maybe - yeah acctually the header replace would be a lot easier in one patch than 27 | 17:17 |
clayg | the list endpoints thing should be easy enough - it's just more to review - so it's really mostly about how effectively I can write obviously correct code :\ | 17:17 |
*** annegent_ has quit IRC | 17:17 | |
clayg | did anyone look at the merge_timestamps change? Is that better - after it was all said in done I think I liked it better - but maybe it was just different | 17:18 |
clayg | anyway, i need to get out of the house, i think the timestamp offset width fix is just changing a constant - so i'll do that and update the effected tests. | 17:18 |
*** nsquare has joined #openstack-swift | 17:20 | |
*** byeager has joined #openstack-swift | 17:22 | |
*** tdasilva has joined #openstack-swift | 17:24 | |
clayg | acoles: I wonder if we shouldn't do <post_timestamp>_<data_file_timestamp> or something so that the last_modified in the container listing looks more like when the post happened than when the datafile got uploaded? dunno. | 17:25 |
clayg | bbiab | 17:28 |
*** byeager has quit IRC | 17:31 | |
acoles | clayg: hmm, but data file timestamp must always take precedence when sorting. could maybe add the offset (which would be post_time - put_time) to 'normal' part when reporting. | 17:39 |
acoles | clayg: commute time for me. | 17:42 |
*** haomaiwa_ has joined #openstack-swift | 17:44 | |
*** haomaiwa_ has quit IRC | 17:49 | |
*** tong_ has joined #openstack-swift | 17:52 | |
*** mmcardle has quit IRC | 17:52 | |
*** Trixboxer has quit IRC | 17:55 | |
*** sombrafam has quit IRC | 17:59 | |
*** scohen has quit IRC | 18:00 | |
*** scohen has joined #openstack-swift | 18:01 | |
*** kevinc_ has quit IRC | 18:04 | |
*** scohen has quit IRC | 18:05 | |
torgomatic | for those trying to figure out which SP patches they've left reviews on and which they haven't, particularly after a rebase, here's a Gerrit search I find useful: | 18:16 |
torgomatic | status:open branch:master topic:bp/storage-policies -label:Code-Review>=-2,self <--- patches without a vote from me on their latest change set | 18:16 |
*** byeager has joined #openstack-swift | 18:17 | |
torgomatic | now that we're on Gerrit 2.newer-than-before, we have the "self" user, which refers to the currently-logged-in user, so there's no need to go s/torgomatic/$me/ on the query | 18:17 |
*** byeager has quit IRC | 18:22 | |
portante | torgomatic: thanks | 18:22 |
torgomatic | portante: np | 18:22 |
torgomatic | if you want to see things you *have* left reviews on, just s/-label/label/ | 18:22 |
peluse_ | torgomatic: hmm, it didn't like 'self' for me, I had to use 'peluse' | 18:26 |
torgomatic | peluse_: are you logged in? | 18:26 |
peluse_ | oh jeeze... | 18:26 |
peluse_ | thanks :) | 18:26 |
torgomatic | peluse_: np :) | 18:28 |
*** gvernik has joined #openstack-swift | 18:30 | |
*** wasmum is now known as zz_wasmum | 18:36 | |
*** zz_wasmum is now known as wasmum | 18:38 | |
clayg | acoles: yeah i was thinking something like that - it maybe makes sense to continue to update the user facing last_modified on post like post-as-copy does today | 18:40 |
notmyname | swift team meeting in 20 minutes in #openstack-meeting | 18:41 |
*** byeager has joined #openstack-swift | 18:42 | |
clayg | so my train was full of kids on there way to some summer camp field trip - didn't make any progress on the timestamp change - also sorta waiting to see if gholt could elaborate on comment on the two vector timestamps | 18:42 |
peluse_ | clayg: sounds like you missed a good opportunity to collaborate with some fresh thinkers... | 18:46 |
portante | notmyname: gonna be late to the swift team meeting, bunch of stuff goin' on here today | 18:48 |
creiht | notmyname: I'll be there for a bit, but will hae to jet a bit early for the meeting | 18:48 |
creiht | for a meeting | 18:48 |
notmyname | ok. mostly I want to raise the question I asked this morning. how do we continue to make progress and actually deliver software? clayg has mentioned just a few outstanding things, and others have said that at least some can be done post-merge | 18:50 |
*** annegent_ has joined #openstack-swift | 18:50 | |
creiht | I'm kind of in the let's get it merged and iterate group | 18:50 |
zaitcev | yeah | 18:51 |
notmyname | creiht: I think most of us are trending that way :-) | 18:51 |
*** annegent_ has quit IRC | 18:53 | |
*** annegent_ has joined #openstack-swift | 18:54 | |
torgomatic | yep, same here | 18:58 |
*** ozialien has joined #openstack-swift | 18:59 | |
zaitcev | It's like Obamacare: you've got to pass it to find what's in it. And if you like your current containers, you can keep them! | 19:00 |
notmyname | and it's noon | 19:01 |
peluse_ | zaitcev: yikes! | 19:01 |
notmyname | zaitcev: well except that you've been able to see what's in it every step of the way ;-) | 19:01 |
zaitcev | notmyname: I feel like I'm too stupid to deduce the implications, so it's no use. I have to rely on Sam and Clay in the end | 19:03 |
notmyname | clayg: we're talking storage policies in #openstack-meeting | 19:05 |
*** lpabon has quit IRC | 19:06 | |
*** openstackgerrit_ has joined #openstack-swift | 19:06 | |
*** openstackgerrit has quit IRC | 19:06 | |
*** openstackgerrit_ is now known as openstackgerrit | 19:08 | |
*** annegent_ has quit IRC | 19:13 | |
*** ozialien has quit IRC | 19:18 | |
*** jamiehannaford has quit IRC | 19:26 | |
*** wasmum is now known as zz_wasmum | 19:27 | |
*** kevinc_ has joined #openstack-swift | 19:35 | |
*** jamiehannaford has joined #openstack-swift | 19:39 | |
notmyname | gvernik: I don't think we'll be able to cover apache spark stuff in today's meeting. I'm interested though, and maybe later this afternoon (if you are traveling) or tomorrow am (so TZs work) I'd love to hear about it | 19:45 |
gvernik | sure, let's cover it next time. | 19:46 |
*** zz_wasmum is now known as wasmum | 19:48 | |
*** gvernik has quit IRC | 19:51 | |
peluse_ | notmyname: missed the tail end of the mtg, are we merging today? | 19:52 |
mattoliverau | Well I'm heading back to bed, see everyone in a few hours :) | 19:54 |
notmyname | peluse_: clayg will widen the timestamps and push the last copy of the chain this afternoon or tonight. from there, I'll work with mordred and get the force push of the chain upstream, and propose the merge commit to master. at that point I'll ask for a "Review of record" for people on the merge commit and then we'll merge that. | 19:55 |
peluse_ | awesome! | 19:55 |
mordred | notmyname: I did a test of the process on sandbox and it works just fine, btw | 19:56 |
notmyname | mordred: cool! | 19:56 |
notmyname | and anything else will be done on master after SP lands. and at the SP merge time, we'll tag the RC and ask the community to test it. then after 2 weeks, we'll cut the final (having backported any major fixes as necessary to the RC) | 19:56 |
mordred | notmyname: otoh - the gate is _WAY_ more healthy today than it was last week | 19:56 |
mordred | notmyname: so there is the chance that merging normally could Just Work™ | 19:57 |
mordred | :) | 19:57 |
notmyname | mordred: yes, but still the idea of having just one commit for SP included is much better | 19:57 |
mordred | sure. just saying | 19:57 |
*** kevinc_ has quit IRC | 19:58 | |
*** tdasilva has quit IRC | 19:58 | |
*** scohen has joined #openstack-swift | 20:04 | |
shri | Hey guys, question about containers… can a container be deleted when it is not empty? | 20:04 |
*** scohen has quit IRC | 20:05 | |
*** mwstorer has quit IRC | 20:11 | |
glange | no | 20:17 |
peluse_ | shri: a container can't be deleted if it has stuff in it (but I think the swift CLI for example has a flag that will let you do this because it deletes contents behind the scenes for you) | 20:17 |
shri | I see that the swift command line client deletes all blobs and then deletes the container | 20:18 |
shri | I am looking at a swift instance where the /srv/node/r0/objects directory has several files in it but there are no containers. Is that because of a bug? | 20:18 |
peluse_ | shri: I think there is a bug there that one of my colleagues ran into as well... one sec | 20:19 |
peluse_ | wait, thought you were asking something different | 20:19 |
peluse_ | shri: there could still be stuff in there, depends on what you are seeing as to whether or not it would be expected or not | 20:20 |
*** annegent_ has joined #openstack-swift | 20:21 | |
peluse_ | .ts files for example could still be there for a while after you've deleted everything | 20:21 |
shri | oh.. thats right.. Let me check | 20:22 |
*** foexle has joined #openstack-swift | 20:23 | |
*** annegent_ has quit IRC | 20:26 | |
*** byeager has quit IRC | 20:28 | |
*** byeager has joined #openstack-swift | 20:29 | |
*** annegent_ has joined #openstack-swift | 20:33 | |
clayg | zaitcev: portante: can you guys put a high pass filter on https://review.openstack.org/#/c/100807/ - if i'm going to be doing more work there tonight i'd like to know it asap | 20:34 |
portante | clayg: looking at it now | 20:41 |
clayg | portante: cool, thanks! | 20:46 |
*** nsquare has quit IRC | 20:51 | |
*** byeager has quit IRC | 20:53 | |
*** byeager has joined #openstack-swift | 20:53 | |
*** miqui has quit IRC | 20:54 | |
*** mmcardle has joined #openstack-swift | 21:00 | |
*** kevinc_ has joined #openstack-swift | 21:03 | |
*** tongli has joined #openstack-swift | 21:12 | |
*** tongli has quit IRC | 21:13 | |
*** byeager has quit IRC | 21:16 | |
*** byeager has joined #openstack-swift | 21:17 | |
*** byeager has quit IRC | 21:19 | |
*** wasmum is now known as zz_wasmum | 21:20 | |
*** tong_ has quit IRC | 21:20 | |
clayg | ok, i think i got the 16 digit timestamp fix in | 21:21 |
*** byeager has joined #openstack-swift | 21:22 | |
*** zz_wasmum is now known as wasmum | 21:23 | |
*** jergerber has quit IRC | 21:26 | |
*** fifieldt has quit IRC | 21:27 | |
*** wasmum is now known as zz_wasmum | 21:36 | |
*** jamiehannaford has quit IRC | 21:37 | |
*** fifieldt has joined #openstack-swift | 21:40 | |
*** byeager has quit IRC | 21:40 | |
*** Midnightmyth has quit IRC | 21:43 | |
*** annegent_ has quit IRC | 21:44 | |
notmyname | FYI there is a low-impact security bug that has been reported for Swift. I've just marked the bug public, and I'm about to push the patch for master and the backport to Icehouse | 21:46 |
notmyname | https://bugs.launchpad.net/swift/+bug/1327414 | 21:46 |
*** byeager has joined #openstack-swift | 21:47 | |
openstackgerrit | John Dickinson proposed a change to openstack/swift: properly quote www-authenticate header value https://review.openstack.org/101031 | 21:47 |
*** mmcardle has quit IRC | 21:50 | |
*** mmcardle has joined #openstack-swift | 21:50 | |
*** mmcardle has quit IRC | 21:51 | |
*** byeager has quit IRC | 22:03 | |
*** byeager has joined #openstack-swift | 22:07 | |
portante | clayg, notmyname: fwiw, I have run the sp patch chain through it paces in my two SAIOs and saw no problems. Not that that will really indicate much, ... | 22:07 |
notmyname | thanks | 22:07 |
*** scohen has joined #openstack-swift | 22:07 | |
portante | in the coming days after it lands, I'll post the patches that get the in-process functional tests working with sp, and addition of the in-memory diskfile backend as a storage policy | 22:07 |
*** scohen has quit IRC | 22:07 | |
peluse_ | portante: cool | 22:08 |
portante | I'd like to see the core devs contribute to ramping up the coverage of some of the base infrastructure code once sp lands on master (like for common/db.py, etc.) as a way to help vet this work further | 22:10 |
*** dmsimard is now known as dmsimard_away | 22:10 | |
notmyname | and it looks like a pep8 error on that security patch. *sigh* | 22:14 |
*** byeager_ has joined #openstack-swift | 22:16 | |
clayg | heheheheh -> http://openstackreactions.enovance.com/2014/03/paeeeep8/ | 22:16 |
*** byeager has quit IRC | 22:21 | |
torgomatic | yep, still good :) | 22:25 |
notmyname | mtreinish: sorry about that. the patch had a flak8 bug. fixed now. https://review.openstack.org/#/c/101032/ | 22:29 |
mtreinish | notmyname: heh, ok. +2'd again | 22:36 |
notmyname | thnaks | 22:36 |
*** byeager_ has quit IRC | 22:49 | |
mattoliverau | morning all | 22:55 |
notmyname | mattoliverau: hello, again. | 23:02 |
notmyname | mattoliverau: you got up for the meeting! wasn't it exciting? | 23:02 |
*** ozialien has joined #openstack-swift | 23:03 | |
mattoliverau | notmyname: lol, yeah I made it, tired but made it :) The meeting was fun, exciting as descisions were made and looks like things are moving forward (SP wise). And it isn't as early as I thought it would be, it was only 5am (on the east coast of australia).. I still went back to bed after, but still not as early :P | 23:12 |
notmyname | A group of swifts are collectively known as a "screaming frenzy" | 23:16 |
mattoliverau | notmyname: ROFL!!! that is a t-shirt for the next sumit :) | 23:21 |
* mattoliverau going to find some breakfast, brb. | 23:22 | |
*** mordred has quit IRC | 23:24 | |
*** mordred has joined #openstack-swift | 23:27 | |
*** zz_wasmum is now known as wasmum | 23:29 | |
*** mordred has quit IRC | 23:30 | |
*** mordred has joined #openstack-swift | 23:30 | |
openstackgerrit | A change was merged to openstack/swift: properly quote www-authenticate header value https://review.openstack.org/101031 | 23:38 |
*** asdfsf has joined #openstack-swift | 23:48 | |
clayg | so in extending the tests for the merge_timestmaps change to cover the account and container brokers I rememberd that the create_account_stat_table code wasn't updated to set status_changed_at to put_timestamp during initialize - i'm inclined to fix that now | 23:58 |
*** sungju has joined #openstack-swift | 23:59 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!