-
Notifications
You must be signed in to change notification settings - Fork 252
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
fix: check recovery backups/snapshots readiness before cluster bootstrap #3663
Conversation
❗ By default, the pull request is configured to backport to all release branches.
|
10eb247
to
824219a
Compare
824219a
to
ec07c85
Compare
a921f27
to
e1f8df8
Compare
cluster.Spec.Bootstrap.Recovery != nil { | ||
var err error | ||
backup, err = r.getOriginBackup(ctx, cluster) | ||
if err != nil { | ||
return ctrl.Result{}, err | ||
} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
this is already there before your PR, but the flow is very confusing.
We look for a backup object in the bootstrap.recovery, and it looks like we depend on it.
But further down we may still proceed if we had boostrap.recovery.volumesnapshots.
I'm going to try to make this more explicit.
347697d
to
1095b94
Compare
/ok-to-merge |
6048082
to
a5470cb
Compare
9013ae0
to
4133c13
Compare
68c4d7b
to
dfdbeaa
Compare
Signed-off-by: YanniHu1996 <yantian.hu@enterprisedb.com>
…ots. Signed-off-by: YanniHu1996 <yantian.hu@enterprisedb.com>
Signed-off-by: YanniHu1996 <yantian.hu@enterprisedb.com>
Signed-off-by: Tao Li <tao.li@enterprisedb.com>
Signed-off-by: Jaime Silvela <jaime.silvela@enterprisedb.com>
Signed-off-by: Jaime Silvela <jaime.silvela@enterprisedb.com>
Signed-off-by: Jaime Silvela <jaime.silvela@enterprisedb.com>
Signed-off-by: Marco Nenciarini <marco.nenciarini@enterprisedb.com>
…rap (#3663) Fix bootstrapping error loop by ensuring recovery backups/snapshots readiness before Cluster construction starts. This patch addresses an issue where Clusters bootstrapping from recovery could enter an error loop if the backups or snapshots were not prepared at the time of cluster creation. It modifies the bootstrapping process to delay the construction of the first instance until the necessary recovery backups or snapshots are fully ready, thus preventing the error loop. Closes #3654 Signed-off-by: YanniHu1996 <yantian.hu@enterprisedb.com> Signed-off-by: Tao Li <tao.li@enterprisedb.com> Signed-off-by: Jaime Silvela <jaime.silvela@enterprisedb.com> Signed-off-by: Marco Nenciarini <marco.nenciarini@enterprisedb.com> Co-authored-by: Tao Li <tao.li@enterprisedb.com> Co-authored-by: Jaime Silvela <jaime.silvela@enterprisedb.com> Co-authored-by: Marco Nenciarini <marco.nenciarini@enterprisedb.com> (cherry picked from commit d7c7212)
…rap (#3663) Fix bootstrapping error loop by ensuring recovery backups/snapshots readiness before Cluster construction starts. This patch addresses an issue where Clusters bootstrapping from recovery could enter an error loop if the backups or snapshots were not prepared at the time of cluster creation. It modifies the bootstrapping process to delay the construction of the first instance until the necessary recovery backups or snapshots are fully ready, thus preventing the error loop. Closes #3654 Signed-off-by: YanniHu1996 <yantian.hu@enterprisedb.com> Signed-off-by: Tao Li <tao.li@enterprisedb.com> Signed-off-by: Jaime Silvela <jaime.silvela@enterprisedb.com> Signed-off-by: Marco Nenciarini <marco.nenciarini@enterprisedb.com> Co-authored-by: Tao Li <tao.li@enterprisedb.com> Co-authored-by: Jaime Silvela <jaime.silvela@enterprisedb.com> Co-authored-by: Marco Nenciarini <marco.nenciarini@enterprisedb.com> (cherry picked from commit d7c7212)
Fix bootstrapping error loop by ensuring recovery backups/snapshots
readiness before Cluster construction starts.
This patch addresses an issue where Clusters bootstrapping from
recovery could enter an error loop if the backups or snapshots were
not prepared at the time of cluster creation. It modifies the
bootstrapping process to delay the construction of the first instance
until the necessary recovery backups or snapshots are fully ready,
thus preventing the error loop.
Closes #3654