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 CHANGELOG for #16029 #16165 #16193 #16194

Merged
merged 1 commit into from
Jul 7, 2023
Merged

Conversation

wenjiaswe
Copy link
Contributor

@@ -13,6 +13,9 @@ Previous change logs can be found at [CHANGELOG-3.3](https://github.com/etcd-io/
### Dependencies
- Compile binaries using [go 1.19.10](https://github.com/etcd-io/etcd/pull/16038).

### etcdutl v3
Copy link
Member

Choose a reason for hiding this comment

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

3.4 doesn't have etcdutl.

Suggested change
### etcdutl v3
### etcdctl v3

Copy link
Member

Choose a reason for hiding this comment

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

minor comment: also usually we put the section ### Dependencies at the end. So suggest to put this section to above it.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Ah, too long since I updated last time.. Thank you very much! I will update changelog for 3.5 and 3.6 once this is merged.

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 we can update all changelog in this PR. But it isn't a big deal.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Done.

@wenjiaswe wenjiaswe force-pushed the patch-1 branch 2 times, most recently from bee7c38 to 4190405 Compare July 7, 2023 07:18
@wenjiaswe wenjiaswe changed the title [WIP] Update CHANGELOG-3.4.md for #16193 Update CHANGELOG for #16029 #16165 #16193 Jul 7, 2023
@wenjiaswe wenjiaswe force-pushed the patch-1 branch 2 times, most recently from afc0a94 to ef4802f Compare July 7, 2023 07:24
@@ -10,6 +10,9 @@ Previous change logs can be found at [CHANGELOG-3.4](https://github.com/etcd-io/
- Fix [corruption check may get a `ErrCompacted` error when server has just been compacted](https://github.com/etcd-io/etcd/pull/16048)
- Improve [Lease put performance for the case that auth is disabled or the user is admin](https://github.com/etcd-io/etcd/pull/16019)

### etcdutl v3
- Add [optional --bump-revision and --mark-compacted flag to etcdutl snapshot restore operation](https://github.com/etcd-io/etcd/pull/16165).
Copy link
Member

Choose a reason for hiding this comment

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

We also updated the etcdctl snapshot restore command in 3.5.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Oops, added. Thanks!

Signed-off-by: Wenjia Zhang <wenjiazhang@google.com>
Copy link
Member

@ahrtr ahrtr left a comment

Choose a reason for hiding this comment

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

LGTM

thx @wenjiaswe

@serathius serathius merged commit 82f6cb4 into etcd-io:main Jul 7, 2023
27 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Development

Successfully merging this pull request may close these issues.

None yet

3 participants