JobSink: Delete secrets associated with jobs when jobs are deleted #6903
knative-style.yaml
on: pull_request
style
/
changes
2s
style
/
...
/
Auto-format and Check
20s
style
/
...
/
Lint
2m 25s
style
/
...
/
Do Not Submit
7s
style
/
...
/
shell
0s
style
/
...
/
yaml
0s
style
/
...
/
github_actions
0s
Matrix: style / Golang / Boilerplate Check
Annotations
8 errors and 4 warnings
style / Golang / Boilerplate Check (go):
cmd/jobsink/main_test.go#L1
[Go headers] reported by reviewdog 🐶
missing boilerplate:
Raw Output:
cmd/jobsink/main_test.go:1: missing boilerplate:
/*
Copyright 2024 The Knative Authors
Licensed under the Apache License, Version 2.0 (the "License");
you may not use this file except in compliance with the License.
You may obtain a copy of the License at
http://www.apache.org/licenses/LICENSE-2.0
Unless required by applicable law or agreed to in writing, software
distributed under the License is distributed on an "AS IS" BASIS,
WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
See the License for the specific language governing permissions and
limitations under the License.
*/
|
style / Golang / Boilerplate Check (go)
Process completed with exit code 1.
|
style / Golang / Auto-format and Check:
cmd/jobsink/main.go#L1
Please run goimports.
diff --git a/cmd/jobsink/main.go b/cmd/jobsink/main.go
index 99f6b1f..d430744 100644
--- a/cmd/jobsink/main.go
+++ b/cmd/jobsink/main.go
@@ -19,6 +19,7 @@ package main
import (
"context"
"crypto/md5"
+
//nolint:gosec
"crypto/tls"
"encoding/hex"
@@ -450,6 +451,6 @@ func jobLabelSelector(ref types.NamespacedName, id string) string {
}
func toJobName(js string, source, id string) string {
- h := md5.Sum([]byte(source+id))
- return kmeta.ChildName(js, "-" + utils.ToDNS1123Subdomain(hex.EncodeToString(h[:])))
+ h := md5.Sum([]byte(source + id))
+ return kmeta.ChildName(js, "-"+utils.ToDNS1123Subdomain(hex.EncodeToString(h[:])))
}
|
style / Golang / Auto-format and Check:
cmd/jobsink/main_test.go#L1
Please run goimports.
diff --git a/cmd/jobsink/main_test.go b/cmd/jobsink/main_test.go
index 0862d40..0bf31b5 100644
--- a/cmd/jobsink/main_test.go
+++ b/cmd/jobsink/main_test.go
@@ -29,7 +29,7 @@ func TestToJobName(t *testing.T) {
}
for _, tc := range testcases {
- t.Run(tc.JobSinkName + "_" + tc.Source + "_" + tc.Id, func(t *testing.T) {
+ t.Run(tc.JobSinkName+"_"+tc.Source+"_"+tc.Id, func(t *testing.T) {
if errs := validation.NameIsDNS1035Label(tc.JobSinkName, false); len(errs) != 0 {
t.Errorf("Invalid JobSinkName: %v", errs)
}
|
style / Golang / Auto-format and Check
Process completed with exit code 1.
|
style / Golang / Lint:
cmd/jobsink/main.go#L21
G501: Blocklisted import crypto/md5: weak cryptographic primitive (gosec)
|
style / Golang / Lint:
cmd/jobsink/main.go#L453
G401: Use of weak cryptographic primitive (gosec)
|
style / Golang / Lint
issues found
|
style / Golang / Boilerplate Check (go)
Restore cache failed: Dependencies file is not found in /home/runner/work/eventing/eventing. Supported file pattern: go.sum
|
style / Golang / Auto-format and Check
Restore cache failed: Dependencies file is not found in /home/runner/work/eventing/eventing. Supported file pattern: go.sum
|
style / Golang / Boilerplate Check (sh)
Restore cache failed: Dependencies file is not found in /home/runner/work/eventing/eventing. Supported file pattern: go.sum
|
style / Golang / Lint
Restore cache failed: Dependencies file is not found in /home/runner/work/eventing/eventing. Supported file pattern: go.sum
|