Hi.
I've just noticed that OsmocomBB is not on the list of projects at https://gerrit.osmocom.org/#/admin/projects/ Is this because nobody bothered adding it there yet or it's because maintainers do not find it suitable? If it's the latter than I'd love to see it added to streamline contributions and patch review process.
Hi Max,
On Wed, Apr 19, 2017 at 03:31:52PM +0200, Max wrote:
I've just noticed that OsmocomBB is not on the list of projects at https://gerrit.osmocom.org/#/admin/projects/ Is this because nobody bothered adding it there yet or it's because maintainers do not find it suitable? If it's the latter than I'd love to see it added to streamline contributions and patch review process.
I guess it's simply that nobody bothered to do so, as activity on OsmocomBB has been extremely low.
I don't mind gerrit for OsmocomBB, not sure how the others think.
One issue that I fear is that I see very little activity outside of the sysmocom team in gerrit in terms of review. I had the feeling this was better while posting patches still on the mailing lists. I'm not sure there is a causality.
Hi,
One issue that I fear is that I see very little activity outside of the sysmocom team in gerrit in terms of review. I had the feeling this was better while posting patches still on the mailing lists. I'm not sure there is a causality.
I'm not sure if it's just me or if I'm using it wrong but I'm always annoyed when I have to login to gerrit ...
1) I shouldn't be logged out at all ... it's not a high security stuff that session should be kept open for a long time, like > 1 week without issues ... 2) I have to retype the openid login url. I mean there is a login with Yahoo and login with Launchpad Id dedicated link, isn't there a way to add, "login with your osmocom redmine account" link ? 3) Then I'm redirected to redmine, where I have to login as well, because again for some reason I've been logged out. Same comment as above, unless I explicitely log out, that session should last for ever pretty much ... 4) When I then login to redmine, I get redirected to the OpenID end point but at this point the "open id state" or whatever has been lost and so I need to go back to gerrit and re-do the whole login process so it can do it in one go without being interrupted by the redmine login process and finally log me into gerrit ...
I can assure you I gave up on the whole process more than one time ....
Cheers,
Sylvain
On Wed, Apr 19, 2017 at 10:01 PM, Sylvain Munaut 246tnt@gmail.com wrote:
Hi,
One issue that I fear is that I see very little activity outside of the sysmocom team in gerrit in terms of review. I had the feeling this was better while posting patches still on the mailing lists. I'm not sure there is a causality.
I'm not sure if it's just me or if I'm using it wrong but I'm always annoyed when I have to login to gerrit ...
- I shouldn't be logged out at all ... it's not a high security stuff
that session should be kept open for a long time, like > 1 week without issues ... 2) I have to retype the openid login url. I mean there is a login with Yahoo and login with Launchpad Id dedicated link, isn't there a way to add, "login with your osmocom redmine account" link ? 3) Then I'm redirected to redmine, where I have to login as well, because again for some reason I've been logged out. Same comment as above, unless I explicitely log out, that session should last for ever pretty much ... 4) When I then login to redmine, I get redirected to the OpenID end point but at this point the "open id state" or whatever has been lost and so I need to go back to gerrit and re-do the whole login process so it can do it in one go without being interrupted by the redmine login process and finally log me into gerrit ...
I can assure you I gave up on the whole process more than one time ....
I agree with this sentiment. I haven't gave up on the process those few times I did a review, but I still find it very annoying that I need to re-login every time, especially given the process is not so straightforward. It would be nice if my login lasted forever.
Another reason is that by default you're subscribed only to patches you're explicitly mentioned in. So unless you make a conscious decision to enable notifications about all patches, you won't even notice there is something to review. And then you have to figure out how to enable this. So it may help if you was offered to select project you're interested in during user registration and you're subscribed to all patches from those projects.
Hi Sylvain,
On Wed, Apr 19, 2017 at 09:01:30PM +0200, Sylvain Munaut wrote:
I'm not sure if it's just me or if I'm using it wrong but I'm always annoyed when I have to login to gerrit ...
thanks for raising this issue. It is probably as simple as to extend the cookie expiration or something like that. That would at least already resolve '1)' + '4)' from your list.
I've created https://osmocom.org/issues/2015 and https://osmocom.org/issues/2016 to track this. We'll have to see who can look into it, guess Holger and Neels are most familiar with the setup.
On Wed, Apr 19, 2017 at 09:01:30PM +0200, Sylvain Munaut wrote:
Hi,
One issue that I fear is that I see very little activity outside of the sysmocom team in gerrit in terms of review. I had the feeling this was better while posting patches still on the mailing lists. I'm not sure there is a causality.
I'm not sure if it's just me or if I'm using it wrong but I'm always annoyed when I have to login to gerrit ...
I've gotten somewhat used to it but it's a bit of an eyebrow raiser every time. The OpenID seems like it should be really useful, why else would it have been invented, but practically for me it blows up the usual click-on-login with the password remembered from my keystore to a three-step process. I wish gerrit simply had a password. (It does, but that seems to be unusuable for website logins, only for build bots?)
- I shouldn't be logged out at all ... it's not a high security stuff
that session should be kept open for a long time, like > 1 week without issues ... 2) I have to retype the openid login url. I mean there is a login with Yahoo and login with Launchpad Id dedicated link, isn't there a way to add, "login with your osmocom redmine account" link ?
My browser remembers the URL but I still need to type 'h' to trigger the https... URL.
- Then I'm redirected to redmine, where I have to login as well,
because again for some reason I've been logged out. Same comment as above, unless I explicitely log out, that session should last for ever pretty much ... 4) When I then login to redmine, I get redirected to the OpenID end point but at this point the "open id state" or whatever has been lost and so I need to go back to gerrit and re-do the whole login process so it can do it in one go without being interrupted by the redmine login process and finally log me into gerrit ...
A way to shorten the process is to login on osmocom.org first, and only then move on to gerrit -- annoying, I agree. I assume the redmine login dialog loses gerrit's state.
I can assure you I gave up on the whole process more than one time ....
Gave up? :) I pestered about it when we moved to gerrit, but it does work. Admitted, sometimes the '500 Bad Gateway' needs a browser restart...
In summary, I agree the login is unusually cumbersome.
+1 for a "login via osmocom.org" link, not sure where to plug that though.
+1 for longer sessions, also not quite sure how configurable that is...
I think both redmine and gerrit remain logged in as long as the browser keeps the login cookie? I'm usually logged out only when I restart the browser.
In the redmine admin, I see an "Autologin: disabled" item that can be set to 1, 7, 30 or 365 days. I'd switch that to 30 days if everyone agrees.
Gerrit configuration is generally a bit unusual, so far I couldn't find a way to configure login sessions.
~N
On Thu, Apr 20, 2017 at 10:18:57AM +0200, Neels Hofmeyr wrote:
In the redmine admin, I see an "Autologin: disabled" item that can be set to 1, 7, 30 or 365 days. I'd switch that to 30 days if everyone agrees.
Actually, I set it to 30 days now. Please tell me if you disagree and I'll set it back to "disabled".
~N
baseband-devel@lists.osmocom.org
-
Alexander Chemeris -
Harald Welte -
Max -
Neels Hofmeyr -
Sylvain Munaut