News for hackers. Stuff that matters.
Re:Engineer says Google fired her for notifying co-workers o... by jefftk on Tuesday December 17, 2019 @09:49AM (#21813930)
> She did it exactly according to internal policy.

I don't think this person or the other organizers should have been fired, but code review is not the same as organizational approval.

(Disclosure: I work at Google, speaking only for myself)

Re:Engineer says Google fired her for notifying co-workers o... by gnopgnip on Tuesday December 17, 2019 @11:23AM (#21815092)
It doesn't really matter if google would approve. There are a lot of activities that would not be organizationally approved, but are protected concerted activities and it would be unlawful to fire someone for.
Re:Engineer says Google fired her for notifying co-workers o... by jefftk on Tuesday December 17, 2019 @03:38PM (#21818526)
As I said, I don't think any of them should have been fired.

But I don't think glossing "the change went through code review" as "She did it exactly according to internal policy" is helpful.

Re:Engineer says Google fired her for notifying co-workers o... by bduerst on Tuesday December 17, 2019 @01:57PM (#21817241)
GP was referring to approval for code review and organizational approval of security extension notifications, not the right to organize or unionizing activities.
Re:Engineer says Google fired her for notifying co-workers o... by javajosh on Tuesday December 17, 2019 @10:10AM (#21814186)
What is the purview of a code review? Is it only about making sure the code meets technical standards, or are reviewers expected to comment holistically?
Re:Engineer says Google fired her for notifying co-workers o... by jefftk on Tuesday December 17, 2019 @10:40AM (#21814562)
Reviewers are supposed to review holistically, and in cases where something raises larger questions (do we actually have approval for this change?) they shouldn't approve until those questions are resolved.

In this case I suspect the reviewers were supportive of the protest, and so approved it even though their authority didn't cover this kind of change.

I don't think any of these people should have been fired, but "by the book" is not a good way of characterizing this process. That would have been proposing the feature to management, getting their sign-off, and then making the code changes.

Re:Engineer says Google fired her for notifying co-workers o... by javajosh on Tuesday December 17, 2019 @11:13AM (#21814967)
>Reviewers are supposed to review holistically

That sounds about right. So why doesn't Google fire the reviewers? If they don't, then they are inconsistently applying the standard. If they do, then they are punishing a coordinated employee action.

Re:Engineer says Google fired her for notifying co-workers o... by singron on Tuesday December 17, 2019 @12:41PM (#21816168)
If Google just doesn't want unionized engineers, it could just fire all engineers. Obviously they need to balance their various objectives and firing productive people is costly to the business. I think they are hoping more to send a strong message to the whole company with as small a disturbance to the business as possible.
Re:Engineer says Google fired her for notifying co-workers o... by pathseeker on Tuesday December 17, 2019 @12:40PM (#21816137)
Because reviewers aren't held to the same level of responsibility for the content of the change as the author. Making them responsible for back-door vulnerabilities slipped in by employees compromised by a nation-state would grind the company to a halt.
Re:Engineer says Google fired her for notifying co-workers o... by ummonk on Tuesday December 17, 2019 @01:32PM (#21816891)
My understanding is that Google is a rather bottoms-up culture regarding internal tooling, and that you don't normally need sign-off from management to take initiative and make changes to tooling like this. They're only mad that the particular change here involved labor organizing, which is obvious retaliation.
Re:Engineer says Google fired her for notifying co-workers o... by pysxul on Tuesday December 17, 2019 @10:24AM (#21814358)
It kind of depends, not every task are the same. you won't review a trivial change the same way as a brand new feature.

Let's take for instance the fictional case of adding a list of website to a whitelist.txt. Maybe the source file had thousands of websites to be added. Anyone could add or remove some in the middle and no one would ever know

It looks good when you review it. but there is no way to check the data integrity (at review time), you can only rely on the good faith of your coworkers.

I am not saying that's what happened but this is a good example of how anyone could sneak-in arbitrary code that pass the code review.

Re:Engineer says Google fired her for notifying co-workers o... by debt on Tuesday December 17, 2019 @11:19AM (#21815047)
“organizational approval”

Is the literal problem, which necessitated the pop-up.

The Organization Does Not Approve™

It’s like asking someone if you can kick them in the knee before doing so. What do you think they’ll say?

Y’all seem confused: the organization, Google in this case, is antognizing it’s organizing employees. Google absolutely do not, will not, can not approve of unionization efforts. Ever.

Re:Engineer says Google fired her for notifying co-workers o... by ddalex on Tuesday December 17, 2019 @11:33AM (#21815226)
how about not kicking them in the knee ?