[8.17] (backport #8063) ci: disable link-patrol check (#8067)

* ci: disable link-patrol check (#8063)

* ci: disable link-patrol check

* fix: add issue link for skipping and fix linter issues

* fix: remove extra args

* fix: revert SerialDeps to Deps in Update

(cherry picked from commit 82344c2)

# Conflicts:
#	magefile.go

* Update magefile.go

---------

Co-authored-by: Panos Koutsovasilis <[email protected]>
Co-authored-by: Pierre HILBERT <[email protected]>

Author: 3 people

dev-tools/mage/check.go

@@ -77,18 +77,21 @@ func CheckLicenseHeaders() error {
 		licenser.Check(),
 		licenser.License("Elasticv2"),
 	)
-
 }
 
 // CheckLinksInFileAreLive checks if all links in a file are live.
 func CheckLinksInFileAreLive(filename string) func() error {
 	return func() error {
-		fmt.Printf(">> check: Checking for invalid links in %q\n", filename)
-		mg.Deps(InstallGoLinkCheck)
+		// Skip for now until we can make this work without exceeding the GitHub rate limit
+		// Related: https://github.com/elastic/elastic-agent/issues/8064
+		return nil
 
-		linkcheck := gotool.LinkCheck
-		return linkcheck(
-			linkcheck.Path(filename),
-		)
+		//fmt.Printf(">> check: Checking for invalid links in %q\n", filename)
+		//mg.Deps(InstallGoLinkCheck)
+		//
+		//linkcheck := gotool.LinkCheck
+		//return linkcheck(
+		//	linkcheck.Path(filename),
+		//)
 	}
 }

magefile.go

@@ -121,7 +121,7 @@ var (
 )
 
 func init() {
-	common.RegisterCheckDeps(Update, Check.All)
+	common.RegisterCheckDeps(Update, Check.All, Otel.Readme)
 	test.RegisterDeps(UnitTest)
 	devtools.BeatLicense = "Elastic License 2.0"
 	devtools.BeatDescription = "Elastic Agent - single, unified way to add monitoring for logs, metrics, and other types of data to a host."
@@ -649,7 +649,7 @@ func commitID() string {
 
 // Update is an alias for executing control protocol, configs, and specs.
 func Update() {
-	mg.SerialDeps(Config, BuildPGP, BuildFleetCfg, Otel.Readme)
+	mg.Deps(Config, BuildPGP, BuildFleetCfg)
 }
 
 func EnsureCrossBuildOutputDir() error {