tailscale/tailscale

util/cloudenv: improve AWS detection to cover Windows and other non-Linux OSes

Open

#4.984 geöffnet am 30. Juni 2022

Auf GitHub ansehen
 (0 Kommentare) (0 Reaktionen) (0 zugewiesene Personen)Go (2.554 Forks)batch import
dnshelp wantedneeds-fix

Repository-Metriken

Stars
 (31.565 Stars)
PR-Merge-Metriken
 (Durchschn. Merge 5T 22h) (173 gemergte PRs in 30 T)

Beschreibung

#4980 adds DNS-specific tweaks when running on AWS (e.g. bootstrap DNS, not shadowing *.internal) but only works if we can detect that we're running on AWS.

Our current implementation of that detection is super minimal:

func getCloud() Cloud {
	// TODO(bradfitz): also detect AWS on Windows, etc. Just try to hit the metadata server
	// and see if it's there? But it might be turned off. Do some small-timeout DNS request
	// to 169.254.169.253 and see if it replies? But which DNS request?
	if runtime.GOOS == "linux" {
		biosVendorB, _ := os.ReadFile("/sys/class/dmi/id/bios_vendor")
		biosVendor := strings.TrimSpace(string(biosVendorB))
		if biosVendor == "Amazon EC2" || strings.HasSuffix(biosVendor, ".amazon") {
			return AWS
		}
	}
....

We should do something that works for BSDs & Windows on AWS too.

See that TODO above.

Contributor Guide

Research-Richtung
Erkunden Sie plattformübergreifende Methoden zur Erkennung der AWS Umgebung, z. B. durch Verbindungsversuch zum AWS Metadatenserver (169.254.169.254) mit kurzem Timeout oder durch Überprüfung von DNS Antworten von 169.254.169.253. Erwägen Sie die Verwendung von Netzwerksonden anstelle des Lesens der BIOS Herstellerdatei. Suchen Sie nach vorhandenen Go Bibliotheken oder betriebssystemspezifischen Ansätzen.
Tech Stack
go
Domain
backendcloudnetworking
Issue Type
Funktion
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
3
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
Halber Tag
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
Frisch
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
Klar
Voraussetzungen
Go
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
70