Skip to content

Commit

Permalink
add ann parsing of default tcp host
Browse files Browse the repository at this point in the history
TCP services support SNI inspection and has its default host, pretty
much like the HTTP one - it configures the backend that should be used
if there is no matching host. This update adds annotation parsing from
the default host, and does not have impact in the codebase because,
currently, there is no hostname scoped annotation for TCP Services. A
fix for a future problem.
  • Loading branch information
jcmoraisjr committed Apr 13, 2021
1 parent 42fb776 commit c5ab2ce
Showing 1 changed file with 4 additions and 0 deletions.
4 changes: 4 additions & 0 deletions pkg/converters/ingress/ingress.go
Original file line number Diff line number Diff line change
Expand Up @@ -653,6 +653,10 @@ func (c *converter) fullSyncTCP() {
for _, tcpPort := range c.haproxy.TCPServices().Items() {
if ann, found := c.tcpsvcAnnotations[tcpPort]; found {
c.updater.UpdateTCPPortConfig(tcpPort, ann)
tcpHost := tcpPort.DefaultHost()
if tcpHost != nil {
c.updater.UpdateTCPHostConfig(tcpHost, ann)
}
for _, tcpHost := range tcpPort.Hosts() {
c.updater.UpdateTCPHostConfig(tcpHost, ann)
}
Expand Down

0 comments on commit c5ab2ce

Please sign in to comment.