Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Allow templating of host and advertisedHost fields #10605

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

scholzj
Copy link
Member

@scholzj scholzj commented Sep 18, 2024

Type of change

  • Enhancement / new feature

Description

This PR implements the Strimzi Proposal 84 and adds support for templating host and advertisedHost fields in the listener configuration. The host fields apply to route and ingress type listeners, the advertised hosts can be used with any listener. The implementation is as describe int he proposal.

Checklist

  • Write tests
  • Make sure all tests pass
  • Update documentation
  • Try your changes from Pod inside your Kubernetes and OpenShift cluster, not just locally
  • Update CHANGELOG.md

@scholzj scholzj added this to the 0.44.0 milestone Sep 18, 2024
@scholzj scholzj changed the title Allow templating of host and advertisedHost fields Allow templating of host and advertisedHost fields Sep 18, 2024
@scholzj
Copy link
Member Author

scholzj commented Sep 18, 2024

/azp run regression

Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@scholzj
Copy link
Member Author

scholzj commented Sep 18, 2024

/azp run zookeeper-regression

Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@scholzj
Copy link
Member Author

scholzj commented Sep 19, 2024

/azp run migration

Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@scholzj
Copy link
Member Author

scholzj commented Sep 19, 2024

/azp run helm-acceptance

Copy link

Azure Pipelines successfully started running 1 pipeline(s).

Copy link
Contributor

@PaulRMellor PaulRMellor left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Great. I've left a few minor suggestions, mainly for consistency with the docs.

@scholzj scholzj force-pushed the allow-templating-of-host-and-advertisedHost-fields branch from b4fc080 to 0411ee7 Compare September 20, 2024 19:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: 0.44.0 (Work in progress)
Development

Successfully merging this pull request may close these issues.

2 participants