Most active commenters
  • lexicality(5)

←back to thread

650 points clcaev | 18 comments | | HN request time: 0s | source | bottom
Show context
metaphor ◴[] No.45063162[source]
> Immediately after the wreck at 9:14 p.m. on April 25, 2019, the crucial data detailing how it unfolded was automatically uploaded to the company’s servers and stored in a vast central database, according to court documents. Tesla’s headquarters soon sent an automated message back to the car confirming that it had received the collision snapshot.

> Moments later, court records show, the data was just as automatically “unlinked” from the 2019 Tesla Model S at the scene, meaning the local copy was marked for deletion, a standard practice for Teslas in such incidents, according to court testimony.

Wow...just wow.

replies(5): >>45063302 #>>45063632 #>>45063687 #>>45063980 #>>45064115 #
1. raincole ◴[] No.45063632[source]
The 'wow' part is that they deleted data from server. The part you quoted sounds like nothing unusual to me.
replies(1): >>45063732 #
2. lexicality ◴[] No.45063732[source]
You don't think it's unusual that the software is designed to delete crash data from the crashed car?
replies(3): >>45063809 #>>45063911 #>>45064066 #
3. foobarian ◴[] No.45063809[source]
Think of it as the scripts that run on CI/CD actions running unit tests. If a unit test fails, the test artifacts are uploaded to an artifact repository, and then, get this - the test runner instance is destroyed! But we don't think of that as unusual or nefarious.
replies(4): >>45063848 #>>45063858 #>>45063869 #>>45070944 #
4. lexicality ◴[] No.45063848{3}[source]
That's because typically the test runner hasn't just crashed into another test runner at full highway speed
5. smallpipe ◴[] No.45063858{3}[source]
No one dies when your unit test fails. Different stakes, different practices, what are all the Tesla apologists smoking here?
6. Ambroisie ◴[] No.45063869{3}[source]
I don't think you can equate CI/CD unit tests and killing humans with 2 tons of metal.
replies(1): >>45064462 #
7. Thorrez ◴[] No.45063911[source]
The question is whether this is code that's special for crashes, or code that runs the exact same way for all data uploads, regardless of whether there's a crash.

You're implying it's special for crashes, but we don't know that.

replies(3): >>45064033 #>>45064087 #>>45064288 #
8. lexicality ◴[] No.45064033{3}[source]
The crash system uses this code, therefore they chose to do something that would delete the crash data after a crash.

Saying "hey, the upload_and_delete function is used in loads of places!" doesn't free you of the responsibility that you used that function in the crash handler.

replies(1): >>45064052 #
9. Thorrez ◴[] No.45064052{4}[source]
Is this a crash handler, or is it their normal telemetry upload loop?
replies(1): >>45064083 #
10. phkahler ◴[] No.45064066[source]
>> You don't think it's unusual that the software is designed to delete crash data from the crashed car?

After it confirmed upload to the server? What if it was a minor collision? The car may be back on the road the same day, or get repaired and on the road next week. How long should it retain data (that is not legally required to be logged) that has already been archived, and how big does the buffer need to be?

replies(3): >>45064162 #>>45064716 #>>45070934 #
11. lexicality ◴[] No.45064083{5}[source]
Yes, it's a crash handler that uploads a blackbox "collision snapshot" of the entire car's state leading up to a crash. It's very well documented that Tesla does this, including in the article.
12. dylan604 ◴[] No.45064087{3}[source]
You have it backwards. The fact that after the special condition of a crash it still allows the data to be deleted is an issue. Sure, deleting of normal data is fine, but it clearly detected a crash and did not mark the file in the special crash mode as do not delete is mind boggling. Everyone knows that in a crash detection mode that the data is very important. Not having code to ensure data retention is the laziest at best way of doing things or malevolently designed at worst. Tesla and its leadership do not deserve at best as our default choice.
13. lexicality ◴[] No.45064162{3}[source]
A very simple answer is "until the next time the car crashes", you just replace the previous crash data with the new data.

If the car requires that a certain amount of storage is always available to write crash data to, then it doesn't matter what's in that particular area of storage. That reserved storage is always going to be unavailable for other general use.

14. sim7c00 ◴[] No.45064288{3}[source]
if its not special for crashes thats criminally bad design in a safety critical system.

u know if for instance u weld a gas pipeline and an xray machine reveal a crack in your work, you can go to jail.... but if you treat car software as an appstore item, totally fine??

stop defending ridiculously bad design and corporate practices.

15. foobarian ◴[] No.45064462{4}[source]
And yet, that's what you get when your software org comes from that kind of devops culture. And here we are
16. kergonath ◴[] No.45064716{3}[source]
> What if it was a minor collision?

Then, I don’t know… Check if it was the case? Seriously, it’s unbelievable. It’s a company with a protocol to delete possibly incriminating evidence in a situation where it can be responsible for multiple deaths.

17. ndsipa_pomu ◴[] No.45070934{3}[source]
Keep it until any pending court cases or insurance investigation is complete.
18. ndsipa_pomu ◴[] No.45070944{3}[source]
That's a particularly unhelpful analogy. Car telemetry is linked to real world physical objects and real human suffering, whereas CI/CD actions may be linked to the happiness of a project manager etc.