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

Update NATS to v2.9.18 #192

Merged
merged 4 commits into from
Jun 20, 2023
Merged

Update NATS to v2.9.18 #192

merged 4 commits into from
Jun 20, 2023

Conversation

bruth
Copy link
Contributor

@bruth bruth commented Jun 19, 2023

No description provided.

Signed-off-by: Byron Ruth <byron@nats.io>
@brandond
Copy link
Member

I see that CI has failed, is there anything interesting related to this change in the log?

Signed-off-by: Byron Ruth <byron@nats.io>
@bruth
Copy link
Contributor Author

bruth commented Jun 19, 2023

I forgot to update the actual test script for external NATS, so just did that.

I looked at the failure and tried to track down where/why it would occur, but impacts the older NATS server versions (2.7.x, and 2.8.x, no longer recommended to use). Is there a standard policy arounds backwards compat for kine drivers?

That said personally I want to understand what changed in, where I suspect, the Go NATS library causing the older server versions to fail..

@bruth
Copy link
Contributor Author

bruth commented Jun 19, 2023

Got info on the bug reported here: nats-io/nats.go#1321 and fixed here: nats-io/nats.go#1325. Patch release coming this week, so we can wait to merge this until I bump the Go client with the fix.

Signed-off-by: Byron Ruth <byron@nats.io>
Signed-off-by: Byron Ruth <byron@nats.io>
@brandond brandond merged commit d5dd460 into k3s-io:master Jun 20, 2023
@bruth bruth deleted the nats-2.9.18 branch June 23, 2023 11:10
gshilei pushed a commit to gshilei/kine that referenced this pull request Oct 31, 2023
* Update NATS to v2.9.18
* Update test-run-nats to 2.9.18

Signed-off-by: Byron Ruth <byron@nats.io>
Signed-off-by: zhiyuan <guoshilei.gsl@antgroup.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants