Apiv1beta1 #2
Merged
Apiv1beta1 #2
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Creates new binds (instead of editing them) of the healthz and stats frontends to avoid port conflicts at the first start of the HAProxy when using hostNetwork: true. Since these bind directives are not required, the first start of HAProxy is not affected.
golang.org/x/exp v0.0.0-20231006140011-7918f672742d golang.org/x/net v0.17.0 golang.org/x/oauth2 v0.13.0 golang.org/x/sys v0.13.0 golang.org/x/term v0.13.0
github.com/prometheus/client_golang v1.17.0 github.com/google/go-cmp v0.6.0 github.com/prometheus/client_model v0.5.0 github.com/prometheus/procfs v0.12.0 k8s.io/kube-openapi v0.0.0-20231010175941-2dd684a91f00
For debugging, this will help to match any failed transaction (saved in a file with the transactionID as suffix) with all logs related to it.
Logic to detect if a namespace was updated on K8s update event was wrong and done the opposite way of what it should be
…esses If we need to propagate the Ingress Controller LB's external IP address to all ingresses, then k8s.UpdateAllIngresses is set to true. This value is taken into account in `updateHAProxy`, which is triggered only if there were some changes on the next COMMAND.
…e matches path in Ingress
golang.org/x/exp v0.0.0-20231110203233-9a3e6036ecaa golang.org/x/net v0.18.0 golang.org/x/oauth2 v0.14.0 golang.org/x/sys v0.14.0 golang.org/x/term v0.14.0 golang.org/x/text v0.14.0 golang.org/x/time v0.4.0
k8s.io/api v0.28.3 k8s.io/apiextensions-apiserver v0.28.3 k8s.io/apimachinery v0.28.3 k8s.io/client-go v0.28.3 github.com/fsnotify/fsnotify v1.7.0 github.com/go-logr/logr v1.3.0 github.com/google/uuid v1.4.0 github.com/matttproud/golang_protobuf_extensions/v2 v2.0.0 github.com/prometheus/common v0.45.0 k8s.io/klog/v2 v2.110.1 k8s.io/kube-openapi v0.0.0-20231113174909-778a5567bc1e sigs.k8s.io/structured-merge-diff/v4 v4.4.1 sigs.k8s.io/yaml v1.4.0
CRDs are not properly handled with external tools, Helm and similar options cannot handle upgrading of custom resource definitions
with this change, v1 of backends, defaults and global are introduced, and at the same time alpha2 versions are deprecated Co-authored-by: Helene Durand <hdurand@haproxy.com>
…d e2e tests This MR fixes several issues around the controller port: - set 6060 as default port, so techdump endpoint is working without adding --controller-port parameter - fix pprof backend, entries were missing in maps - have server for prometheus, pprof and techdump starts on ControllerPort and not hard-coded to 6060 This MR also adds e2e tests for techdump and pprof endpoints. Not testing the bahavior, but at least that: - pprof can be triggered This MR also adds Documentation
Adds also some documentation with the unsupported haproxy 2.9 fields.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
No description provided.