Move util to specific package location

This commit is contained in:
Ricardo Pchevuzinske Katz 2022-07-20 18:15:03 -03:00
parent 0cc43d5e52
commit 7304086202
11 changed files with 13 additions and 17 deletions

View file

@ -28,7 +28,7 @@ import (
"k8s.io/ingress-nginx/internal/ingress/annotations/parser" "k8s.io/ingress-nginx/internal/ingress/annotations/parser"
ing_errors "k8s.io/ingress-nginx/internal/ingress/errors" ing_errors "k8s.io/ingress-nginx/internal/ingress/errors"
"k8s.io/ingress-nginx/internal/ingress/resolver" "k8s.io/ingress-nginx/internal/ingress/resolver"
"k8s.io/ingress-nginx/internal/sets" "k8s.io/ingress-nginx/pkg/util/sets"
) )
// Config returns external authentication configuration for an Ingress rule // Config returns external authentication configuration for an Ingress rule

View file

@ -27,7 +27,7 @@ import (
ing_errors "k8s.io/ingress-nginx/internal/ingress/errors" ing_errors "k8s.io/ingress-nginx/internal/ingress/errors"
"k8s.io/ingress-nginx/internal/ingress/resolver" "k8s.io/ingress-nginx/internal/ingress/resolver"
"k8s.io/ingress-nginx/internal/net" "k8s.io/ingress-nginx/internal/net"
"k8s.io/ingress-nginx/internal/sets" "k8s.io/ingress-nginx/pkg/util/sets"
) )
const defaultKey = "$remote_addr" const defaultKey = "$remote_addr"

View file

@ -27,7 +27,7 @@ import (
"k8s.io/ingress-nginx/internal/ingress/annotations/parser" "k8s.io/ingress-nginx/internal/ingress/annotations/parser"
ing_errors "k8s.io/ingress-nginx/internal/ingress/errors" ing_errors "k8s.io/ingress-nginx/internal/ingress/errors"
"k8s.io/ingress-nginx/internal/ingress/resolver" "k8s.io/ingress-nginx/internal/ingress/resolver"
"k8s.io/ingress-nginx/internal/sets" "k8s.io/ingress-nginx/pkg/util/sets"
) )
// SourceRange returns the CIDR // SourceRange returns the CIDR

View file

@ -26,7 +26,7 @@ import (
"k8s.io/ingress-nginx/internal/ingress/annotations/parser" "k8s.io/ingress-nginx/internal/ingress/annotations/parser"
"k8s.io/ingress-nginx/internal/ingress/resolver" "k8s.io/ingress-nginx/internal/ingress/resolver"
"k8s.io/ingress-nginx/internal/net" "k8s.io/ingress-nginx/internal/net"
"k8s.io/ingress-nginx/internal/sets" "k8s.io/ingress-nginx/pkg/util/sets"
) )
const ( const (

View file

@ -58,7 +58,6 @@ import (
"k8s.io/ingress-nginx/internal/net/ssl" "k8s.io/ingress-nginx/internal/net/ssl"
"k8s.io/ingress-nginx/internal/nginx" "k8s.io/ingress-nginx/internal/nginx"
"k8s.io/ingress-nginx/internal/task" "k8s.io/ingress-nginx/internal/task"
"k8s.io/ingress-nginx/internal/watch"
"k8s.io/ingress-nginx/pkg/util/file" "k8s.io/ingress-nginx/pkg/util/file"
klog "k8s.io/klog/v2" klog "k8s.io/klog/v2"
@ -172,7 +171,7 @@ func NewNGINXController(config *Configuration, mc metric.Collector) *NGINXContro
n.t = ngxTpl n.t = ngxTpl
_, err = watch.NewFileWatcher(nginx.TemplatePath, onTemplateChange) _, err = file.NewFileWatcher(nginx.TemplatePath, onTemplateChange)
if err != nil { if err != nil {
klog.Fatalf("Error creating file watcher for %v: %v", nginx.TemplatePath, err) klog.Fatalf("Error creating file watcher for %v: %v", nginx.TemplatePath, err)
} }
@ -196,7 +195,7 @@ func NewNGINXController(config *Configuration, mc metric.Collector) *NGINXContro
} }
for _, f := range filesToWatch { for _, f := range filesToWatch {
_, err = watch.NewFileWatcher(f, func() { _, err = file.NewFileWatcher(f, func() {
klog.InfoS("File changed detected. Reloading NGINX", "path", f) klog.InfoS("File changed detected. Reloading NGINX", "path", f)
n.syncQueue.EnqueueTask(task.GetDummyObject("file-change")) n.syncQueue.EnqueueTask(task.GetDummyObject("file-change"))
}) })

View file

@ -17,7 +17,7 @@ limitations under the License.
package ingress package ingress
import ( import (
"k8s.io/ingress-nginx/internal/sets" "k8s.io/ingress-nginx/pkg/util/sets"
) )
// Equal tests for equality between two Configuration types // Equal tests for equality between two Configuration types

View file

@ -43,9 +43,8 @@ import (
"k8s.io/ingress-nginx/internal/ingress" "k8s.io/ingress-nginx/internal/ingress"
ngx_config "k8s.io/ingress-nginx/internal/ingress/controller/config" ngx_config "k8s.io/ingress-nginx/internal/ingress/controller/config"
"k8s.io/ingress-nginx/internal/watch"
"k8s.io/ingress-nginx/pkg/util/file" "k8s.io/ingress-nginx/pkg/util/file"
klog "k8s.io/klog/v2" klog "k8s.io/klog/v2"
) )
@ -509,8 +508,8 @@ func NewTLSListener(certificate, key string) *TLSListener {
l.load() l.load()
_, _ = watch.NewFileWatcher(certificate, l.load) _, _ = file.NewFileWatcher(certificate, l.load)
_, _ = watch.NewFileWatcher(key, l.load) _, _ = file.NewFileWatcher(key, l.load)
return &l return &l
} }

View file

@ -14,7 +14,7 @@ See the License for the specific language governing permissions and
limitations under the License. limitations under the License.
*/ */
package watch package file
import ( import (
"log" "log"

View file

@ -14,7 +14,7 @@ See the License for the specific language governing permissions and
limitations under the License. limitations under the License.
*/ */
package watch package file
import ( import (
"os" "os"
@ -22,8 +22,6 @@ import (
"path/filepath" "path/filepath"
"testing" "testing"
"time" "time"
"k8s.io/ingress-nginx/pkg/util/file"
) )
func prepareTimeout() chan bool { func prepareTimeout() chan bool {
@ -61,7 +59,7 @@ func TestFileWatcher(t *testing.T) {
t.Fatalf("expected no events before writing a file") t.Fatalf("expected no events before writing a file")
case <-timeoutChan: case <-timeoutChan:
} }
os.WriteFile(f.Name(), []byte{}, file.ReadWriteByUser) os.WriteFile(f.Name(), []byte{}, ReadWriteByUser)
select { select {
case <-events: case <-events:
case <-timeoutChan: case <-timeoutChan: