-
Notifications
You must be signed in to change notification settings - Fork 830
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
Bug fix 3.11/hotbackup restore analyzers #20940
Conversation
…t do not exist in HB. Properly react to agency response, if the input is of illegal format.
// Now create the regression. In version 3.6 the entry /arango/Plan/Analyzers did not exist. | ||
|
||
|
||
assertTrue(readAgencyValueAt("Plan/Analyzers") !== undefined, "We should have analyzers in plan by default."); |
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.
nit: instead of assertTrue(x !== undefined)
we could use assertNotUndefined(x)
, which is more legible.
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.
Ah nice, did not know this existed!
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.
LGTM
return {TRI_ERROR_HOT_BACKUP_INTERNAL, | ||
"Failed to restore agency plan from Hotbackup. Please contact " | ||
"ArangoDB support immediately."}; |
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.
Should this be an ADB_PROD_ASSERT?
* Applied fix to set empty objects instead of none for Plan entries that do not exist in HB. Properly react to agency response, if the input is of illegal format. * Actually report an error if replanning of the AgencyPlan fails * Added a failure point for test. * Added hotbackup regression tests * Applied clang-format --------- Co-authored-by: Vadim Kondratev <vadim@arangodb.com>
* Bug fix 3.11/hotbackup restore analyzers (#20940) * Applied fix to set empty objects instead of none for Plan entries that do not exist in HB. Properly react to agency response, if the input is of illegal format. * Actually report an error if replanning of the AgencyPlan fails * Added a failure point for test. * Added hotbackup regression tests * Applied clang-format --------- Co-authored-by: Vadim Kondratev <vadim@arangodb.com> * Bug fix 3.11/hotbackup community test (#20947) * Replaced asserts on undefined with convenience method: * Fixed a race condtion after Hotbackup Restore, and after adding a DBServer, the List of Databases could be empty. * Added CHANGELOG entry --------- Co-authored-by: Vadim Kondratev <vadim@arangodb.com> * Added lost code from cherry-pick --------- Co-authored-by: Vadim Kondratev <vadim@arangodb.com> Co-authored-by: jsteemann <jsteemann@users.noreply.github.com>
Scope & Purpose
This PR fixes a bug which happens on Hotbackup restore, that only shows up on clusters that started with 3.6 or older. And was upgraded ever since. If such a deployment goes to 3.11.8, and wants to restore a backup this will fail.
Checklist
Related Information
(Please reference tickets / specification / other PRs etc)