Make PolyGerrit the default ui
Open, Needs TriagePublic

Tokens
"Love" token, awarded by Liuxinyu970226."Orange Medal" token, awarded by hashar."Goat" token, awarded by Bstorm."Love" token, awarded by dbarratt."Dislike" token, awarded by MarcoAurelio."Love" token, awarded by Mainframe98.
Assigned To
Authored By
Paladox, Jun 9 2018

Description

We should make PolyGerrit the default ui for wikimedia gerrit.

It would let new users get use to the new ui (as the old ui is going away very soon upstream possibly in the next couple of weeks or when ever the next release gets branched).

PolyGerrit in 2.15 is mature enough for most users, users can still go back to the old ui (which on the footer has a nice switcher to ease this)

Paladox created this task.Jun 9 2018, 2:27 PM
Restricted Application added a subscriber: Aklapper. · View Herald TranscriptJun 9 2018, 2:27 PM

Change 439444 had a related patch set uploaded (by Paladox; owner: Paladox):
[operations/puppet@production] Gerrit: Make PolyGerrit the default ui

https://gerrit.wikimedia.org/r/439444

Framawiki added a subscriber: Framawiki.

Proposition of summary for User-notice / tech news:
New users to Wikimedia' Gerrit instance will have the new interface enabled by default as opt-out mode. Every user can already switch the UI using a link in the bottom of the page.

Paladox updated the task description. (Show Details)Jun 9 2018, 6:33 PM
Paladox moved this task from Bugs & stuff to Local hacks on the Gerrit board.Jun 10 2018, 9:12 PM
Tgr added a subscriber: Tgr.Jun 11 2018, 1:02 PM

The lack of editing support is a pretty serious limitation IMO, especially for beginners who might find that a much easier way of creating a patch then setting up git, virtualbox, vagrant & git-review locally.

Paladox added a comment.EditedJun 11 2018, 1:03 PM

Yeh, upstream have added that support in 2.16 / 3.0.

Though they can still switch to the old ui.

While I'm not a huge fan of forcing a new ui, if this ui is going away in future releases I agree it's better to default to the new one and let the ui-switcher provide the old version while it can.

I am thinking about doing this next monday (which will give users plenty of time to give there opinion).

I will update the email with the date.

hashar assigned this task to Paladox.Jun 12 2018, 1:11 PM
hashar added a subscriber: hashar.

We definitely should use polygerrit as the default. It still beta in 2.15 but that will prepare people to switch to it when later on there is no other choice.

Surely it lacks the inline editing, but overall the experience is way more pleasant :]

Yeh, upstream have implemented a really nice inline editing ui for 2.16 / 3.0 :).

Upstream are also offically supporting polygerrit from 2.15+.

@Framawiki wondering can we get the user notice out so we can do this next monday please? :)

We will have to do it on friday as no ops are around to do it from next monday due to offsite.

Or delay it to after the SRE offsite? It is not that urgent anyway :] That also let time for people to react on the announcement, specially volunteers who are usually active solely during the week-end.

Hmm, though i revised the email by sending out the updated date :).

Also they can easily ?polygerrit=0 to get back to the old ui :).

This also fixes a font issue with safari apparently a bug in the way polygerrit and gwtui are switched but upstream are unlikely to fix that as gwtui is being removed soon.

greg added a subscriber: greg.Jun 12 2018, 5:09 PM

We will have to do it on friday as no ops are around to do it from next monday due to offsite.

Please delay until after the SRE offsite :)

And next time, please do coordinate with me/RelEng on any planned changes like this so we can get the timeline correct.

Done and ok :)

@Framawiki wondering can we get the user notice out so we can do this next monday please? :)

Done :)

Paladox added a comment.EditedJun 13 2018, 8:00 PM

@Framawiki oh sorry forgot to tell you that we have postponed it until after the sre offsite (and then a date agreed with releng).

Should probably remove it from there.

Paladox added a comment.EditedJun 15 2018, 11:57 AM

PolyGerrit is now the default ui upstream https://gerrit-review.googlesource.com/c/gerrit/+/184550 (patch by me)

Though that’s for the master branch, gwtui is still the default for 2.15

Johan added a subscriber: Johan.Jun 15 2018, 2:05 PM

Proposition of summary for User-notice / tech news:
New users to Wikimedia' Gerrit instance will have the new interface enabled by default as opt-out mode. Every user can already switch the UI using a link in the bottom of the page.

We've typically excluded these kind of updates from Tech News, as they don't affect editors on the wikis and as there have always been better ways to reach out to developers – especially wikitech-l but also mediawiki-l etc. I don't really see that this has changed. Am I mistaken?

Proposition of summary for User-notice / tech news:
New users to Wikimedia' Gerrit instance will have the new interface enabled by default as opt-out mode. Every user can already switch the UI using a link in the bottom of the page.

We've typically excluded these kind of updates from Tech News, as they don't affect editors on the wikis and as there have always been better ways to reach out to developers – especially wikitech-l but also mediawiki-l etc. I don't really see that this has changed. Am I mistaken?

Hello @Johan, sorry for the delay.
Gerrit is the mandatory gateway for all volunteers wishing to propose code.
So far its interface considered not very welcoming has discouraged more than one. Many of them did not dare or could not come up with their ideas.
The fact that the interface of the site will change soon will I think attract these people. But they still need to be informed! I think it's a big step forward that should be announced. Technews is the best place for this.
The vast majority of the readers have probably already opened a Gerrit page, by following links from phabricator tasks, but did not have to understand much.
With the new interface this constraint should disappear.

Vvjjkkii renamed this task from Make PolyGerrit the default ui to bcbaaaaaaa.Sun, Jul 1, 1:04 AM
Vvjjkkii triaged this task as High priority.
Vvjjkkii removed Paladox as the assignee of this task.
Vvjjkkii updated the task description. (Show Details)
Vvjjkkii removed subscribers: gerritbot, Aklapper.
CommunityTechBot assigned this task to Paladox.
CommunityTechBot raised the priority of this task from High to Needs Triage.
CommunityTechBot renamed this task from bcbaaaaaaa to Make PolyGerrit the default ui.
CommunityTechBot added subscribers: gerritbot, Aklapper.
Johan added a comment.Thu, Jul 5, 9:29 PM

OK, I simplified a bit and geared it a bit more towards the main audience of Tech News (non-developers), but it's included.

Do pre-existing users need to do anything to turn this on?

They only need to add ?polygerrit=1 which sets a cookie (which then sets PolyGerrit to the ui).

But making it the default ui, nope they won't need to do anything.

OK, I simplified a bit and geared it a bit more towards the main audience of Tech News (non-developers), but it's included.

Thanks !