| *** Jeffrey4l has joined #openstack-swift | 00:08 | |
| openstackgerrit | weizj proposed openstack/python-swiftclient master: Update hacking version https://review.openstack.org/627724 | 02:40 |
|---|---|---|
| openstackgerrit | weizj proposed openstack/python-swiftclient master: Update hacking version https://review.openstack.org/627724 | 02:43 |
| *** psachin has joined #openstack-swift | 03:37 | |
| *** psachin has quit IRC | 05:00 | |
| openstackgerrit | weizj proposed openstack/python-swiftclient master: Update hacking version https://review.openstack.org/627724 | 05:09 |
| *** psachin has joined #openstack-swift | 05:34 | |
| *** ccamacho has quit IRC | 06:41 | |
| *** rcernin has quit IRC | 07:09 | |
| *** ccamacho has joined #openstack-swift | 07:24 | |
| *** ccamacho has quit IRC | 07:24 | |
| *** ccamacho has joined #openstack-swift | 07:25 | |
| *** ccamacho has quit IRC | 08:09 | |
| *** ccamacho has joined #openstack-swift | 08:54 | |
| *** mikecmpbll has joined #openstack-swift | 09:12 | |
| *** e0ne has joined #openstack-swift | 09:42 | |
| *** dosaboy has quit IRC | 12:02 | |
| *** Kimmo_ has joined #openstack-swift | 12:06 | |
| *** dosaboy has joined #openstack-swift | 12:07 | |
| *** ianychoi has joined #openstack-swift | 13:17 | |
| *** jistr is now known as jistr|mtg | 14:04 | |
| *** jistr|mtg is now known as jistr | 14:55 | |
| *** e0ne has quit IRC | 15:28 | |
| *** itlinux has joined #openstack-swift | 16:40 | |
| *** mrjk has joined #openstack-swift | 16:44 | |
| *** ccamacho has quit IRC | 16:55 | |
| notmyname | interesting. perhaps something to consider http://lists.openstack.org/pipermail/openstack-discuss/2018-December/001304.html | 17:11 |
| tdasilva | +1 | 17:16 |
| notmyname | tdasilva: it does look interesting, but I wonder how it compares to our wiki. I tried looking in designate, but I wasn't really able to see how the new voting thing is used | 17:17 |
| tdasilva | yeah, i'm looking for either a dashboard or a patch to see if I notice anything different | 17:17 |
| tdasilva | https://review.openstack.org/#/c/625140/ | 17:18 |
| patchbot | patch 625140 - cinder (stable/queens) - Add secret=true to fixed_key configuration parameter - 2 patch sets | 17:18 |
| notmyname | I can imagine how extra info could be used for a dashboard | 17:18 |
| notmyname | https://review.openstack.org/#/c/625263/ | 17:18 |
| patchbot | patch 625263 - designate - Fix test failures for new dnspython module - 1 patch set | 17:18 |
| notmyname | hmm... yeah, not too clear how it's used in either of those | 17:19 |
| tdasilva | https://review.openstack.org/#/c/433341/ | 17:19 |
| patchbot | patch 433341 - cinder - Hedvig Cinder driver implementation - 70 patch sets | 17:19 |
| tdasilva | there's also some good feedback from Sean on that mail thread | 17:21 |
| notmyname | he implies that the possible votes are configurable | 17:22 |
| notmyname | I wonder if it's possible to have {unset 1, 2, 3, 4, 5} | 17:23 |
| notmyname | or something similar | 17:23 |
| notmyname | mapping to something similar as in launchpad (or storyboard, I'd guess) | 17:23 |
| notmyname | critical, high, med, low. something like that | 17:23 |
| notmyname | summing up the priority votes gives you a priority | 17:24 |
| tdasilva | yeah, i was trying to understand their use of -1, +1, +2 | 17:24 |
| notmyname | which is close to what I'm trying to approximate with the "community starred patches" on http://d.not.mn/swift_community_dashboard.html | 17:25 |
| *** gyee has joined #openstack-swift | 17:25 | |
| tdasilva | OTOH we do have priorities on bugs | 17:27 |
| notmyname | I'm going to put some of these questions/thoughts into an email to the ML | 17:32 |
| *** mikecmpbll has quit IRC | 17:36 | |
| notmyname | all open, mergable backports have been approved | 19:10 |
| timburke | notmyname: MOAR BACKPORTS! | 19:42 |
| notmyname | tdasilva: fyi: https://tiny.cc/CinderPriorities | 20:18 |
| tdasilva | notmyname: heh, was about to bring that up | 20:18 |
| tdasilva | from Sean's message it seems they are doing more of a priority page like our wiki rather than http://d.not.mn/swift_community_dashboard.html | 20:19 |
| tdasilva | where our wiki is really curated mostly by you (PTL) | 20:19 |
| notmyname | yup | 20:19 |
| tdasilva | and the community dashboard allows people to vote | 20:19 |
| *** ccamacho has joined #openstack-swift | 20:20 | |
| tdasilva | and based on fungi's message i dont think we would be able to do aggregation | 20:20 |
| tdasilva | well...unless everyone can +1 and we just aggregate on that one value | 20:21 |
| tdasilva | i guess that counts as a vote | 20:21 |
| *** e0ne has joined #openstack-swift | 20:21 | |
| *** ccamacho has quit IRC | 20:22 | |
| fungi | yeah, limiting the range to -1..+1 i think would cause "all" the votes to carry over, at least in situations where gerrit is willing to carry over any votes (limited to whitespace-only code changes and commit message rewording) | 20:26 |
| fungi | ((oh, and trivial rebases of course)) | 20:27 |
| openstackgerrit | Tim Burke proposed openstack/swift master: s3api: Block ETag header from CompleteMultipartUpload requests https://review.openstack.org/628292 | 20:32 |
| tdasilva | i'm not sure i understand the point of -1... | 20:34 |
| *** e0ne has quit IRC | 20:37 | |
| *** psachin has quit IRC | 20:48 | |
| *** e0ne has joined #openstack-swift | 20:54 | |
| *** e0ne has quit IRC | 21:50 | |
| *** e0ne has joined #openstack-swift | 21:52 | |
| fungi | perhaps could indicate lower priority reviews? i really don't know having not tried to make use of something like this myself | 21:53 |
| fungi | more that it gives you two prioritization vote values (-1 and +1) which will carry over to some subsequent patches, and no values (as 0 is implicit) which don't carry over | 21:53 |
| fungi | if you used a 0..+2 range, the +1 wouldn't carry over thanks to how gerrit implemented their vote retention mechanism | 21:54 |
| *** e0ne has quit IRC | 21:57 | |
| notmyname | I guess that'd be the same problem with a [1..5] vote set | 22:13 |
| *** itlinux has quit IRC | 22:24 | |
| openstackgerrit | Tim Burke proposed openstack/swift-bench master: Add long_description to fix gate https://review.openstack.org/628316 | 22:25 |
| timburke | well, figuring out what was going on with https://review.openstack.org/#/c/622832/ was a total rabbit hole... | 22:25 |
| patchbot | patch 622832 - swift-bench - Change openstack-dev to openstack-discuss - 1 patch set | 22:25 |
| openstackgerrit | Tim Burke proposed openstack/swift master: Let developers/operators add watchers to object audit https://review.openstack.org/212824 | 22:39 |
| *** rcernin has joined #openstack-swift | 22:42 | |
| openstackgerrit | Merged openstack/swift-bench master: Add long_description to fix gate https://review.openstack.org/628316 | 23:30 |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!