Differential D41751

release: remove inet blackhole sysctls from GCE config
ClosedPublic
Actions

Authored by emaste on Sep 6 2023, 12:06 AM.

Tags

None

Referenced Files

	Unknown Object (File)
	Tue, Feb 10, 12:37 AM

	Unknown Object (File)
	Tue, Feb 3, 5:06 AM

	Unknown Object (File)
	Tue, Feb 3, 5:06 AM

	Unknown Object (File)
	Tue, Feb 3, 5:06 AM

	Unknown Object (File)
	Tue, Feb 3, 4:14 AM

	Unknown Object (File)
	Fri, Jan 23, 5:54 PM

	Unknown Object (File)
	Fri, Jan 23, 5:54 PM

	Unknown Object (File)
	Thu, Jan 22, 10:47 PM

Subscribers

freebsd-transport

Details

Reviewers

swills
jlduran
delphij

Group Reviewers

Commits

rGbebd79e63847: release: remove inet blackhole sysctls from GCE config
rG3284a5929d50: release: remove inet blackhole sysctls from GCE config
rG413687812ba7: release: remove inet blackhole sysctls from GCE config
rG5623352ec005: release: remove inet blackhole sysctls from GCE config
rGb7528b16c603: release: remove inet blackhole sysctls from GCE config

Summary

Other cloud images do not do this, and it can produce confusing results.

Diff Detail

Lint

Lint Skipped

Unit

Tests Skipped

Event Timeline

emaste created this revision.Sep 6 2023, 12:06 AM

Herald added a subscriber: jrtc27. · View Herald TranscriptSep 6 2023, 12:06 AM

See e.g. https://github.com/cirruslabs/cirrus-ci-docs/issues/483 and https://github.com/golang/go/issues/28883

And discussion in https://github.com/python/cpython/pull/91961

Also https://github.com/smol-rs/async-io/issues/110

lwhsu added a subscriber: lwhsu.Sep 6 2023, 8:07 AM

Nice! Thank you!
Also PMTUD problems, UNIX traceroute "not working", the list of issues can be long.
I'll go ahead and disturb @freebsd-transport, to ask if they have something to say against this change.

emaste added a reviewer: jlduran.Sep 6 2023, 2:05 PM

emaste added a subscriber: cperciva.

jlduran accepted this revision.Sep 6 2023, 2:32 PM

This revision is now accepted and ready to land.Sep 6 2023, 2:32 PM

delphij accepted this revision.Sep 6 2023, 4:29 PM

Closed by commit rGb7528b16c603: release: remove inet blackhole sysctls from GCE config (authored by emaste). · Explain WhySep 6 2023, 6:57 PM

This revision was automatically updated to reflect the committed changes.

emaste added a commit: rGb7528b16c603: release: remove inet blackhole sysctls from GCE config.

Herald added a subscriber: imp. · View Herald TranscriptSep 6 2023, 6:57 PM

emaste added a commit: rG5623352ec005: release: remove inet blackhole sysctls from GCE config.Sep 8 2023, 8:37 PM

emaste added a commit: rG413687812ba7: release: remove inet blackhole sysctls from GCE config.Sep 9 2023, 3:15 PM

emaste added a commit: rG3284a5929d50: release: remove inet blackhole sysctls from GCE config.

emaste added a commit: rGbebd79e63847: release: remove inet blackhole sysctls from GCE config.Sep 9 2023, 4:56 PM

Revision Contents
Changeset List

Path

Size

release/

tools/

2 lines

Diff 126924

release/tools/gce.conf

Loading...