Skip to content

Commit

Permalink
[release-17.0] Update known issues in v16.x and v17.0.0 (#13618) (#…
Browse files Browse the repository at this point in the history
…13646)

Co-authored-by: vitess-bot[bot] <108069721+vitess-bot[bot]@users.noreply.github.com>
  • Loading branch information
vitess-bot[bot] authored Jul 27, 2023
1 parent 8c4c1e2 commit 67111e4
Show file tree
Hide file tree
Showing 8 changed files with 154 additions and 8 deletions.
22 changes: 21 additions & 1 deletion changelog/16.0/16.0.0/release_notes.md
Original file line number Diff line number Diff line change
Expand Up @@ -6,6 +6,8 @@
- **[Known Issues](#known-issues)**
- [MySQL & Xtrabackup known issue](#mysql-xtrabackup-ddl)
- [VTTablet Restore Metrics](#vttablet-restore-metrics)
- [Schema-initialization stuck on semi-sync ACKs while upgrading to v16.0.0](#schema-init-upgrade)
- [Broken downgrade from v17.x.x when super_read_only turned on by default](#init-db-sql-turned-on)
- **[Major Changes](#major-changes)**
- **[Breaking Changes](#breaking-changes)**
- [VTGate Advertised MySQL Version](#advertised-mysql-version)
Expand Down Expand Up @@ -101,11 +103,29 @@ or
> ALTER TABLE your_table ENGINE=InnoDB;
```

#### <a id="vttablet-restore-metrics">VTTablet Restore Metrics
### <a id="vttablet-restore-metrics">VTTablet Restore Metrics

As part of the VTTablet Sidecar Schema Maintenance Refactor in v16.0.0, we dropped the `local_metadata` table from the sidecar database schema. This table was storing a couple of metrics related to restores from backup, which have now been lost.
They have been re-introduced in v17.0.0 as metrics that can be accessed from `/debug/vars`.

The original issue can be found [here](https://github.com/vitessio/vitess/issues/13336).

### <a id="schema-init-upgrade"/>Schema-initialization stuck on semi-sync ACKs while upgrading to `v16.0.0`

During upgrades from `<= v15.x.x` to `v16.0.0`, as part of `PromoteReplica` call, the schema-init realizes that there are schema diffs to apply and ends up writing to the database.
The issue is that if [semi-sync](https://vitess.io/docs/16.0/reference/features/mysql-replication/#semi-sync) is enabled, all of these writes get blocked indefinitely.
Eventually, `PromoteReplica` fails, and this fails the entire PRS call.

A fix for this issue was merged on `release-16.0` in [PR#13441](https://github.com/vitessio/vitess/pull/13441), read the [corresponding bug report to learn more](https://github.com/vitessio/vitess/issues/13426).

This issue is fixed in `v16.0.3` and later patch releases.

### <a id="init-db-sql-turned-on"/>Broken downgrade from v17.x.x when super_read_only turned on by default

In `v17.x.x` `super_read_only` is turned on by default meaning that downgrading from `v17` to `v16.0.0` breaks due to `init_db.sql` needing write access.

This issue is fixed in `>= v16.0.3` thanks to [PR #13525](https://github.com/vitessio/vitess/pull/13525)

## <a id="major-changes"/>Major Changes

### <a id="breaking-changes"/>Breaking Changes
Expand Down
27 changes: 26 additions & 1 deletion changelog/16.0/16.0.0/summary.md
Original file line number Diff line number Diff line change
Expand Up @@ -3,7 +3,10 @@
### Table of Contents

- **[Known Issues](#known-issues)**
- [MySQL & Xtrabackup known issue](#mysql-xtrabackup-ddl)
- [MySQL & Xtrabackup known issue](#mysql-xtrabackup-ddl)
- [VTTablet Restore Metrics](#vttablet-restore-metrics)
- [Schema-initialization stuck on semi-sync ACKs while upgrading to v16.0.0](#schema-init-upgrade)
- [Broken downgrade from v17.x.x when super_read_only turned on by default](#init-db-sql-turned-on)
- **[Major Changes](#major-changes)**
- **[Breaking Changes](#breaking-changes)**
- [VTGate Advertised MySQL Version](#advertised-mysql-version)
Expand Down Expand Up @@ -99,6 +102,28 @@ or
> ALTER TABLE your_table ENGINE=InnoDB;
```

### <a id="vttablet-restore-metrics">VTTablet Restore Metrics

As part of the VTTablet Sidecar Schema Maintenance Refactor in v16.0.0, we dropped the `local_metadata` table from the sidecar database schema. This table was storing a couple of metrics related to restores from backup, which have now been lost.
They have been re-introduced in v17.0.0 as metrics that can be accessed from `/debug/vars`.

The original issue can be found [here](https://github.com/vitessio/vitess/issues/13336).

### <a id="schema-init-upgrade"/>Schema-initialization stuck on semi-sync ACKs while upgrading to `v16.0.0`

During upgrades from `<= v15.x.x` to `v16.0.0`, as part of `PromoteReplica` call, the schema-init realizes that there are schema diffs to apply and ends up writing to the database if [semi-sync](https://vitess.io/docs/16.0/reference/features/mysql-replication/#semi-sync) is enabled, all of these writes get blocked indefinitely.
Eventually, `PromoteReplica` fails, and this fails the entire PRS call.

A fix for this issue was merged on `release-16.0` in [PR#13441](https://github.com/vitessio/vitess/pull/13441), read the [corresponding bug report to learn more](https://github.com/vitessio/vitess/issues/13426).

This issue is fixed in `v16.0.3` and later patch releases.

### <a id="init-db-sql-turned-on"/>Broken downgrade from v17.x.x when super_read_only turned on by default

In `v17.x.x` `super_read_only` is turned on by default meaning that downgrading from `v17` to `v16.0.0` breaks due to `init_db.sql` needing write access.

This issue is fixed in `>= v16.0.3` thanks to [PR #13525](https://github.com/vitessio/vitess/pull/13525).

## <a id="major-changes"/>Major Changes

### <a id="breaking-changes"/>Breaking Changes
Expand Down
25 changes: 24 additions & 1 deletion changelog/16.0/16.0.1/release_notes.md
Original file line number Diff line number Diff line change
@@ -1,5 +1,23 @@
# Release of Vitess v16.0.1
## Summary

## Known Issues

### Schema-initialization stuck on semi-sync ACKs while upgrading to `v16.0.1`

During upgrades from `<= v15.x.x` to `v16.0.1`, as part of `PromoteReplica` call, the schema-init realizes that there are schema diffs to apply and ends up writing to the database if [semi-sync](https://vitess.io/docs/16.0/reference/features/mysql-replication/#semi-sync) is enabled, all of these writes get blocked indefinitely.
Eventually, `PromoteReplica` fails, and this fails the entire PRS call.

A fix for this issue was merged on `release-16.0` in [PR#13441](https://github.com/vitessio/vitess/pull/13441), read the [corresponding bug report to learn more](https://github.com/vitessio/vitess/issues/13426).

This issue is fixed in `v16.0.3` and later patch releases.

### <a id="init-db-sql-turned-on"/>Broken downgrade from v17.x.x when super_read_only turned on by default

In `v17.x.x` `super_read_only` is turned on by default meaning that downgrading from `v17` to `v16.0.1` breaks due to `init_db.sql` needing write access.

This issue is fixed in `>= v16.0.3` thanks to [PR #13525](https://github.com/vitessio/vitess/pull/13525)

## Major Changes

### Upgrade to `go1.20.2`

Expand All @@ -8,6 +26,11 @@ Below is a summary of this Go patch release. You can learn more [here](https://g

> go1.20.2 (released 2023-03-07) includes a security fix to the crypto/elliptic package, as well as bug fixes to the compiler, the covdata command, the linker, the runtime, and the crypto/ecdh, crypto/rsa, crypto/x509, os, and syscall packages.
### Keyspace name validation in TopoServer

Prior to v16.0.1, it was possible to create a keyspace with invalid characters, which would then be inaccessible to various cluster management operations.

Keyspace names may no longer contain the forward slash ("/") character, and TopoServer's `GetKeyspace` and `CreateKeyspace` methods return an error if given such a name.

------------
The entire changelog for this release can be found [here](https://github.com/vitessio/vitess/blob/main/changelog/16.0/16.0.1/changelog.md).
Expand Down
21 changes: 19 additions & 2 deletions changelog/16.0/16.0.1/summary.md
Original file line number Diff line number Diff line change
@@ -1,4 +1,21 @@
## Summary
## Known Issues

### Schema-initialization stuck on semi-sync ACKs while upgrading to `v16.0.1`

During upgrades from `<= v15.x.x` to `v16.0.1`, as part of `PromoteReplica` call, the schema-init realizes that there are schema diffs to apply and ends up writing to the database if [semi-sync](https://vitess.io/docs/16.0/reference/features/mysql-replication/#semi-sync) is enabled, all of these writes get blocked indefinitely.
Eventually, `PromoteReplica` fails, and this fails the entire PRS call.

A fix for this issue was merged on `release-16.0` in [PR#13441](https://github.com/vitessio/vitess/pull/13441), read the [corresponding bug report to learn more](https://github.com/vitessio/vitess/issues/13426).

This issue is fixed in `v16.0.3` and later patch releases.

### <a id="init-db-sql-turned-on"/>Broken downgrade from v17.x.x when super_read_only turned on by default

In `v17.x.x` `super_read_only` is turned on by default meaning that downgrading from `v17` to `v16.0.1` breaks due to `init_db.sql` needing write access.

This issue is fixed in `>= v16.0.3` thanks to [PR #13525](https://github.com/vitessio/vitess/pull/13525)

## Major Changes

### Upgrade to `go1.20.2`

Expand All @@ -11,5 +28,5 @@ Below is a summary of this Go patch release. You can learn more [here](https://g

Prior to v16.0.1, it was possible to create a keyspace with invalid characters, which would then be inaccessible to various cluster management operations.

Keyspace names may no longer contain the forward slash ("/") character, and TopoServer's `GetKeyspace` and `CreateKeyspace` methods return an error if given such a name.
Keyspace names may no longer contain the forward slash ("/") character, and TopoServer's `GetKeyspace` and `CreateKeyspace` methods return an error if given such a name.

22 changes: 20 additions & 2 deletions changelog/16.0/16.0.2/release_notes.md
Original file line number Diff line number Diff line change
@@ -1,5 +1,23 @@
# Release of Vitess v16.0.2
## Summary

## Known Issues

### Schema-initialization stuck on semi-sync ACKs while upgrading to `v16.0.2`

During upgrades from `<= v15.x.x` to `v16.0.2`, as part of `PromoteReplica` call, the schema-init realizes that there are schema diffs to apply and ends up writing to the database if [semi-sync](https://vitess.io/docs/16.0/reference/features/mysql-replication/#semi-sync) is enabled, all of these writes get blocked indefinitely.
Eventually, `PromoteReplica` fails, and this fails the entire PRS call.

A fix for this issue was merged on `release-16.0` in [PR#13441](https://github.com/vitessio/vitess/pull/13441), read the [corresponding bug report to learn more](https://github.com/vitessio/vitess/issues/13426).

This issue is fixed in `v16.0.3` and later patch releases.

### <a id="init-db-sql-turned-on"/>Broken downgrade from v17.x.x when super_read_only turned on by default

In `v17.x.x` `super_read_only` is turned on by default meaning that downgrading from `v17` to `v16.0.2` breaks due to `init_db.sql` needing write access.

This issue is fixed in `>= v16.0.3` thanks to [PR #13525](https://github.com/vitessio/vitess/pull/13525)

## Major Changes

### Upgrade to `go1.20.3`

Expand All @@ -11,7 +29,7 @@ Below is a summary of this Go patch release. You can learn more [here](https://g
### EffectiveCallerId in Vtgate gRPC calls

A new flag `grpc-use-static-authentication-callerid` is added to gate the behavior introduced in https://github.com/vitessio/vitess/pull/12050.
Earlier, we used to automatically set immediateCallerID to user from static authentication context that overrode the EffectiveCallerId.
Earlier, we used to automatically set immediateCallerID to user from static authentication context that overrode the EffectiveCallerId.


### Shard name validation in TopoServer
Expand Down
20 changes: 19 additions & 1 deletion changelog/16.0/16.0.2/summary.md
Original file line number Diff line number Diff line change
@@ -1,4 +1,22 @@
## Summary
## Known Issues

### Schema-initialization stuck on semi-sync ACKs while upgrading to `v16.0.2`

During upgrades from `<= v15.x.x` to `v16.0.2`, as part of `PromoteReplica` call, the schema-init realizes that there are schema diffs to apply and ends up writing to the database if [semi-sync](https://vitess.io/docs/16.0/reference/features/mysql-replication/#semi-sync) is enabled, all of these writes get blocked indefinitely.
Eventually, `PromoteReplica` fails, and this fails the entire PRS call.

A fix for this issue was merged on `release-16.0` in [PR#13441](https://github.com/vitessio/vitess/pull/13441), read the [corresponding bug report to learn more](https://github.com/vitessio/vitess/issues/13426).

This issue is fixed in `v16.0.3` and later patch releases.

### <a id="init-db-sql-turned-on"/>Broken downgrade from v17.x.x when super_read_only turned on by default

In `v17.x.x` `super_read_only` is turned on by default meaning that downgrading from `v17` to `v16.0.2` breaks due to `init_db.sql` needing write access.

This issue is fixed in `>= v16.0.3` thanks to [PR #13525](https://github.com/vitessio/vitess/pull/13525)


## Major Changes

### Upgrade to `go1.20.3`

Expand Down
12 changes: 12 additions & 0 deletions changelog/17.0/17.0.0/release_notes.md
Original file line number Diff line number Diff line change
Expand Up @@ -3,6 +3,8 @@

### Table of Contents

- **[Known Issues](#known-issues)**
- [Schema-initialization stuck on semi-sync ACKs while upgrading to v17.0.0](#schema-init-upgrade)
- **[Major Changes](#major-changes)**
- **[Breaking Changes](#breaking-changes)**
- [Default Local Cell Preference for TabletPicker](#tablet-picker-cell-preference)
Expand Down Expand Up @@ -39,6 +41,16 @@
- [Deprecated `vtgr`](#deprecated-vtgr)
- [Deprecated `k8stopo`](#deprecated-k8stopo)

## <a id="known-issues"/>Known Issues
### <a id="schema-init-upgrade"/>Schema-initialization stuck on semi-sync ACKs while upgrading to `v17.0.0`

During upgrades from `<= v16.x.x` to `v17.0.0`, as part of `PromoteReplica` call, the schema-init realizes that there are schema diffs to apply and ends up writing to the database if [semi-sync](https://vitess.io/docs/17.0/reference/features/mysql-replication/#semi-sync) is enabled, all of these writes get blocked indefinitely.
Eventually, `PromoteReplica` fails, and this fails the entire PRS call.

A fix for this issue was merged on `release-17.0` in [PR#13441](https://github.com/vitessio/vitess/pull/13411), read the [corresponding bug report to learn more](https://github.com/vitessio/vitess/issues/13426).

This issue is addressed in the `>= v17.0.1` patch releases.


## <a id="major-changes"/>Major Changes

Expand Down
13 changes: 13 additions & 0 deletions changelog/17.0/17.0.0/summary.md
Original file line number Diff line number Diff line change
Expand Up @@ -2,6 +2,8 @@

### Table of Contents

- **[Known Issues](#known-issues)**
- [Schema-initialization stuck on semi-sync ACKs while upgrading to v17.0.0](#schema-init-upgrade)
- **[Major Changes](#major-changes)**
- **[Breaking Changes](#breaking-changes)**
- [Default Local Cell Preference for TabletPicker](#tablet-picker-cell-preference)
Expand Down Expand Up @@ -38,6 +40,17 @@
- [Deprecated `k8stopo`](#deprecated-k8stopo)


## <a id="known-issues"/>Known Issues
### <a id="schema-init-upgrade"/>Schema-initialization stuck on semi-sync ACKs while upgrading to `v17.0.0`

During upgrades from `<= v16.x.x` to `v17.0.0`, as part of `PromoteReplica` call, the schema-init realizes that there are schema diffs to apply and ends up writing to the database if [semi-sync](https://vitess.io/docs/17.0/reference/features/mysql-replication/#semi-sync) is enabled, all of these writes get blocked indefinitely.
Eventually, `PromoteReplica` fails, and this fails the entire PRS call.

A fix for this issue was merged on `release-17.0` in [PR#13441](https://github.com/vitessio/vitess/pull/13411), read the [corresponding bug report to learn more](https://github.com/vitessio/vitess/issues/13426).

This issue is addressed in the `>= v17.0.1` patch releases.


## <a id="major-changes"/>Major Changes

### <a id="breaking-changes"/>Breaking Changes
Expand Down

0 comments on commit 67111e4

Please sign in to comment.