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

Add consul v1.11.3 support in consul k8s and helm chart #1035

Closed
FelipeEmerim opened this issue Feb 16, 2022 · 6 comments
Closed

Add consul v1.11.3 support in consul k8s and helm chart #1035

FelipeEmerim opened this issue Feb 16, 2022 · 6 comments
Labels
type/enhancement New feature or request waiting-reply Waiting on the issue creator for a response before taking further action

Comments

@FelipeEmerim
Copy link

Community Note

  • Please vote on this issue by adding a 👍 reaction to the original issue to help the community and maintainers prioritize this request. Searching for pre-existing feature requests helps us consolidate datapoints for identical requirements into a single place, thank you!
  • Please do not leave "+1" or other comments that do not add relevant new information or questions, they generate extra noise for issue followers and do not help prioritize the request.
  • If you are interested in working on this issue or have submitted a pull request, please leave a comment.

Is your feature request related to a problem? Please describe.

Consul 1.11.3 introduces several fixes. This one is very critical and is causing downtime on our ingress controller.

Feature Description

I would like consul k8s and its helm chart to support consul v1.11.3.

Use Case(s)

To get consul's latest bugfixes, including the one mentioned above that solves a problem with ingress controllers which use the dialedDirectly feature of transparent proxy.

Contributions

We could open a PR following #976, but I do not think that is possible since community PRs cannot add Changelog entries.

@FelipeEmerim FelipeEmerim added the type/enhancement New feature or request label Feb 16, 2022
@david-yu
Copy link
Contributor

Hi @FelipeEmerim you could easily upgrade to 1.11.3 using your existing Helm chart as described here: https://www.consul.io/docs/k8s/upgrade#consul-version-upgrade. It is best practice to set the global.image variable in your values.yaml file, so bumping that 1.11.3 would be a solution. We will be making some changes shortly to bump the default version of Consul to 1.11.3 as well.

@david-yu david-yu added the waiting-reply Waiting on the issue creator for a response before taking further action label Feb 16, 2022
@FelipeEmerim
Copy link
Author

Thank you for the quick response, I will try bumping the image as you said.

@FelipeEmerim
Copy link
Author

It worked, than you!

@david-yu
Copy link
Contributor

Great, I'll close this issue but we'll make sure to roll out 1.11.3 as the default shortly. Thanks!

@david-yu
Copy link
Contributor

Hi just an update, we'll actually be rolling out to 1.11.4 instead as the default sometime next week instead of 1.11.3 since there is a UI bug on Consul that folks are hitting.

@FelipeEmerim
Copy link
Author

Thank you for the notice. We will keep an eye out for the new update.

geobeau pushed a commit to geobeau/consul-k8s that referenced this issue May 20, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type/enhancement New feature or request waiting-reply Waiting on the issue creator for a response before taking further action
Projects
None yet
Development

No branches or pull requests

2 participants