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 server to 2.10.4 #126

Merged
merged 1 commit into from
Oct 27, 2023
Merged

Update NATS server to 2.10.4 #126

merged 1 commit into from
Oct 27, 2023

Conversation

bruth
Copy link
Member

@bruth bruth commented Oct 27, 2023

No description provided.

@bruth bruth requested a review from wallyqs October 27, 2023 16:01
Copy link
Member

@wallyqs wallyqs left a comment

Choose a reason for hiding this comment

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

LGTM

@@ -1,7 +1,7 @@
#!/usr/bin/env bash
set -ex

ver=(NATS_SERVER 2.9.23)
Copy link
Member

Choose a reason for hiding this comment

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

I think this one has to be 2.9.23 actually

Copy link
Member Author

Choose a reason for hiding this comment

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

Hrm. Let me check, I think the tests run on which ever version is currently being released. At least that is how it worked before.

Copy link
Member Author

Choose a reason for hiding this comment

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

There are only one set of tests for both versions so which is being released is the one that would replace this value. The 2.9.x one has not changed, so no need for testing that.

I believe you did a dual release before correct, so both versions needed to be tested?

Copy link
Member Author

Choose a reason for hiding this comment

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

For posterity, noting @wallyqs and I spoke and no need to change since only one version bump is happening.

@@ -3,7 +3,7 @@ Set-PSDebug -Trace 2
# Exit on error.
$ErrorActionPreference = "Stop"

$ver = "NATS_SERVER 2.9.23".Split(" ")[1]
Copy link
Member

Choose a reason for hiding this comment

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

leave as 2.9.23?

@@ -3,7 +3,7 @@ Set-PSDebug -Trace 2
# Exit on error.
$ErrorActionPreference = "Stop"

$ver = 'NATS_SERVER 2.9.23'.Split(' ')[1]
Copy link
Member

Choose a reason for hiding this comment

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

also has to remain 2.9.23 I think

@bruth bruth merged commit 527dad8 into main Oct 27, 2023
4 checks passed
@bruth bruth deleted the release-2.10.4 branch October 27, 2023 16:28
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