Skip to content

Commit 60e5186

Browse files
authored
Merge pull request #76 from zongzw/zong-v1.0.0
compat with gatewayapi v1.0.0.
2 parents 5e531f0 + b72fd98 commit 60e5186

Some content is hidden

Large Commits have some content hidden by default. Use the searchbox below for content that may be hidden.

47 files changed

+8136
-1023
lines changed

cmd/controller/main.go

Lines changed: 10 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -30,7 +30,6 @@ import (
3030
// to ensure that exec-entrypoint and run can make use of them.
3131

3232
"github.com/prometheus/client_golang/prometheus"
33-
"github.com/prometheus/client_golang/prometheus/promhttp"
3433
"gopkg.in/yaml.v3"
3534
v1 "k8s.io/api/core/v1"
3635
_ "k8s.io/client-go/plugin/pkg/client/auth"
@@ -42,6 +41,7 @@ import (
4241
"sigs.k8s.io/controller-runtime/pkg/healthz"
4342
"sigs.k8s.io/controller-runtime/pkg/log/zap"
4443
"sigs.k8s.io/controller-runtime/pkg/manager"
44+
"sigs.k8s.io/controller-runtime/pkg/metrics/server"
4545

4646
"github.com/f5devcentral/bigip-kubernetes-gateway/internal/controllers"
4747
"github.com/f5devcentral/bigip-kubernetes-gateway/internal/pkg"
@@ -51,6 +51,7 @@ import (
5151

5252
//+kubebuilder:scaffold:imports
5353

54+
gatewayapi "sigs.k8s.io/gateway-api/apis/v1"
5455
gatewayv1beta1 "sigs.k8s.io/gateway-api/apis/v1beta1"
5556
)
5657

@@ -71,6 +72,7 @@ var (
7172

7273
func init() {
7374
utilruntime.Must(clientgoscheme.AddToScheme(scheme))
75+
utilruntime.Must(gatewayapi.AddToScheme(scheme))
7476
utilruntime.Must(gatewayv1beta1.AddToScheme(scheme))
7577
}
7678

@@ -126,7 +128,7 @@ func main() {
126128
ctrl.SetLogger(zap.New(zap.UseFlagOptions(&opts)))
127129
mgr, err := ctrl.NewManager(ctrl.GetConfigOrDie(), ctrl.Options{
128130
Scheme: scheme,
129-
MetricsBindAddress: metricsAddr,
131+
Metrics: server.Options{BindAddress: metricsAddr},
130132
HealthProbeBindAddress: probeAddr,
131133
LeaderElection: enableLeaderElection,
132134
LeaderElectionID: "303cfed9.f5.com",
@@ -151,8 +153,9 @@ func main() {
151153
prometheus.MustRegister(utils.FunctionDurationTimeCostTotal)
152154
prometheus.MustRegister(f5_bigip.BIGIPiControlTimeCostCount)
153155
prometheus.MustRegister(f5_bigip.BIGIPiControlTimeCostTotal)
154-
mgr.AddMetricsExtraHandler("/stats/", promhttp.Handler())
155-
mgr.AddMetricsExtraHandler("/runtime/", dumpRuntimeHandler())
156+
// TODO: use echo
157+
// mgr.AddMetricsExtraHandler("/stats/", promhttp.Handler())
158+
// mgr.AddMetricsExtraHandler("/runtime/", dumpRuntimeHandler())
156159

157160
setupReconcilers(mgr)
158161

@@ -212,17 +215,17 @@ func setupReconcilers(mgr manager.Manager) {
212215

213216
resources.Register(
214217
&controllers.GatewayClassReconciler{
215-
ObjectType: &gatewayv1beta1.GatewayClass{},
218+
ObjectType: &gatewayapi.GatewayClass{},
216219
Client: mgr.GetClient(),
217220
// LogLevel: cmdflags.LogLevel,
218221
},
219222
&controllers.GatewayReconciler{
220-
ObjectType: &gatewayv1beta1.Gateway{},
223+
ObjectType: &gatewayapi.Gateway{},
221224
Client: mgr.GetClient(),
222225
// LogLevel: cmdflags.LogLevel,
223226
},
224227
&controllers.HttpRouteReconciler{
225-
ObjectType: &gatewayv1beta1.HTTPRoute{},
228+
ObjectType: &gatewayapi.HTTPRoute{},
226229
Client: mgr.GetClient(),
227230
// LogLevel: cmdflags.LogLevel,
228231
},

cmd/webhook/main.go

Lines changed: 16 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -26,8 +26,6 @@ import (
2626
// to ensure that exec-entrypoint and run can make use of them.
2727

2828
"github.com/google/uuid"
29-
"github.com/prometheus/client_golang/prometheus"
30-
"github.com/prometheus/client_golang/prometheus/promhttp"
3129
_ "k8s.io/client-go/plugin/pkg/client/auth"
3230

3331
"k8s.io/apimachinery/pkg/runtime"
@@ -37,14 +35,16 @@ import (
3735
"sigs.k8s.io/controller-runtime/pkg/healthz"
3836
"sigs.k8s.io/controller-runtime/pkg/log/zap"
3937
"sigs.k8s.io/controller-runtime/pkg/manager"
38+
"sigs.k8s.io/controller-runtime/pkg/metrics/server"
39+
"sigs.k8s.io/controller-runtime/pkg/webhook"
4040

4141
// "github.com/f5devcentral/bigip-kubernetes-gateway/internal/pkg"
4242
"github.com/f5devcentral/bigip-kubernetes-gateway/internal/webhooks"
43-
f5_bigip "github.com/f5devcentral/f5-bigip-rest-go/bigip"
4443
"github.com/f5devcentral/f5-bigip-rest-go/utils"
4544

4645
//+kubebuilder:scaffold:imports
4746

47+
gatewayapi "sigs.k8s.io/gateway-api/apis/v1"
4848
gatewayv1beta1 "sigs.k8s.io/gateway-api/apis/v1beta1"
4949
)
5050

@@ -57,6 +57,7 @@ var (
5757

5858
func init() {
5959
utilruntime.Must(clientgoscheme.AddToScheme(scheme))
60+
utilruntime.Must(gatewayapi.AddToScheme(scheme))
6061
utilruntime.Must(gatewayv1beta1.AddToScheme(scheme))
6162
}
6263

@@ -98,11 +99,15 @@ func main() {
9899

99100
ctrl.SetLogger(zap.New(zap.UseFlagOptions(&opts)))
100101
var err error
102+
103+
whServer := webhook.NewServer(webhook.Options{
104+
Port: 9443,
105+
CertDir: cmdflags.CertDir,
106+
})
101107
webhooks.WebhookManager, err = ctrl.NewManager(ctrl.GetConfigOrDie(), ctrl.Options{
102108
Scheme: scheme,
103-
MetricsBindAddress: metricsAddr,
104-
Port: 9443,
105-
CertDir: cmdflags.CertDir,
109+
WebhookServer: whServer,
110+
Metrics: server.Options{BindAddress: metricsAddr},
106111
HealthProbeBindAddress: probeAddr,
107112
LeaderElection: enableLeaderElection,
108113
LeaderElectionID: "303cfed9.f5.com",
@@ -123,11 +128,11 @@ func main() {
123128
os.Exit(1)
124129
}
125130

126-
prometheus.MustRegister(utils.FunctionDurationTimeCostCount)
127-
prometheus.MustRegister(utils.FunctionDurationTimeCostTotal)
128-
prometheus.MustRegister(f5_bigip.BIGIPiControlTimeCostCount)
129-
prometheus.MustRegister(f5_bigip.BIGIPiControlTimeCostTotal)
130-
webhooks.WebhookManager.AddMetricsExtraHandler("/stats/", promhttp.Handler())
131+
// prometheus.MustRegister(utils.FunctionDurationTimeCostCount)
132+
// prometheus.MustRegister(utils.FunctionDurationTimeCostTotal)
133+
// prometheus.MustRegister(f5_bigip.BIGIPiControlTimeCostCount)
134+
// prometheus.MustRegister(f5_bigip.BIGIPiControlTimeCostTotal)
135+
// webhooks.WebhookManager.AddMetricsExtraHandler("/stats/", promhttp.Handler())
131136
setupWebhooks(webhooks.WebhookManager)
132137

133138
if err := webhooks.WebhookManager.AddHealthzCheck("healthz", healthz.Ping); err != nil {

0 commit comments

Comments
 (0)